Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ubuntu import 1.0-rc93 #5

Merged
merged 7 commits into from
Mar 16, 2021
Merged

Ubuntu import 1.0-rc93 #5

merged 7 commits into from
Mar 16, 2021

Conversation

lucaskanashiro
Copy link
Collaborator

Version 1.0.0~rc93 contains an important fix for this bug:

https://bugs.launchpad.net/ubuntu/+source/glibc/+bug/1916485

I uploaded the package to this PPA:

https://launchpad.net/~lucaskanashiro/+archive/ubuntu/docker-20.10/+packages

As you can see it FTBFS on armhf, there is an integer overflow in the test of the fix we want. I filed an upstream bug here:

opencontainers/runc#2846

I considered to skip this test for now, but since the failing test is seccomp related, which we want to make sure it is working, I turned it down. Let me know if you have a better approach for this issue.

FWIW autopkgtest is still happy on amd64:

autopkgtest [17:10:37]: @@@@@@@@@@@@@@@@@@@@ summary
basic-smoke          PASS
command1             PASS

runc now has special handling for seccomp profiles to avoid making new
syscalls unusable for glibc (LP: #1916485).
@lucaskanashiro lucaskanashiro marked this pull request as draft March 10, 2021 20:13
@lucaskanashiro
Copy link
Collaborator Author

@tianon
Copy link
Owner

tianon commented Mar 12, 2021

(otherwise this looks very benign to me, assuming it works with containerd and Docker 👍)

@lucaskanashiro lucaskanashiro marked this pull request as ready for review March 12, 2021 12:34
@lucaskanashiro
Copy link
Collaborator Author

Thanks @tianon . I backported the same patch and it fixed the issue. I do not know why I did not see this patch before 🤦

I am testing everything using the PPA in the description (there you can find docker.io and containerd packages as well), I am going to wait for your +1 to move forward.

Copy link
Owner

@tianon tianon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@lucaskanashiro
Copy link
Collaborator Author

Thanks! I am just waiting for the FFe approval to get it merged and uploaded.

@lucaskanashiro
Copy link
Collaborator Author

FFe was approved.

@lucaskanashiro lucaskanashiro merged commit e7657a1 into ubuntu Mar 16, 2021
@lucaskanashiro lucaskanashiro deleted the ubuntu-import-1.0-rc93 branch March 16, 2021 18:06
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.

2 participants