Skip to content

Test bump to runc rc95#2125

Closed
hinshun wants to merge 2 commits intomoby:masterfrom
hinshun:runc-rc95
Closed

Test bump to runc rc95#2125
hinshun wants to merge 2 commits intomoby:masterfrom
hinshun:runc-rc95

Conversation

@hinshun
Copy link
Copy Markdown
Collaborator

@hinshun hinshun commented May 20, 2021

Checking if there is a CI failure when bumping to rc95.
Related to: #2123

@hinshun hinshun requested a review from tonistiigi May 20, 2021 22:24
@hinshun hinshun force-pushed the runc-rc95 branch 2 times, most recently from b927ab9 to 79bf7ad Compare May 20, 2021 22:54
@tonistiigi
Copy link
Copy Markdown
Member

Looks like doesn't cause failure atm. Btw, we don't usually update runc vendoring on its own but do it with containerd vendor update to the same version they are using. A binary update should be enough for this case.

@hinshun
Copy link
Copy Markdown
Collaborator Author

hinshun commented May 20, 2021

Ok I'll try to write a test that fails in this PR before I port it over to #2123

Signed-off-by: Edgar Lee <edgarl@netflix.com>
Signed-off-by: Edgar Lee <edgarl@netflix.com>
@hinshun
Copy link
Copy Markdown
Collaborator Author

hinshun commented May 26, 2021

@tonistiigi Added a failing test that only bumps the binary, which passes when ran with #2123

@hinshun
Copy link
Copy Markdown
Collaborator Author

hinshun commented May 27, 2021

Closing this since #2123 is merged.

@hinshun hinshun closed this May 27, 2021
@tonistiigi
Copy link
Copy Markdown
Member

we can still update to rc95

@hinshun
Copy link
Copy Markdown
Collaborator Author

hinshun commented May 27, 2021

@tonistiigi Do we want to wait until containerd bumps to rc95 or is buildkit going to move independently?

@tonistiigi
Copy link
Copy Markdown
Member

You can bump containerd binary to 1.5.2 as well. Vendor changes can be done separately when needed.

@AkihiroSuda
Copy link
Copy Markdown
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants