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

Update busybox source and fix runc exec bug #2080

Merged
merged 1 commit into from
Jul 27, 2019

Conversation

zhlhahaha
Copy link
Contributor

Currently, the id verification in integration test failed on arm
platform due to the inconsistent /etc/group in the busybox images
for arm and x86. To be specific, the nogroup id in x86 is 99 while
that in arm is 65534.

99 is old id for nogroup, and no longer be used in recent system,
so sync the busybox image for arm and x86 to the image in busybox
github. Also change the id verification rule in integration test.

Signed-off-by: Howard Zhang howard.zhang@arm.com

Currently, the id verification in integration test failed on arm
platform due to the inconsistent /etc/group in the busybox images
for arm and x86. To be specific, the nogroup id in x86 is 99 while
that in arm is 65534.

99 is old id for nogroup, and no longer be used in recent system,
so sync the busybox image for arm and x86 to the image in busybox
github. Also change the id verification rule in integration test.

Signed-off-by: Howard Zhang <howard.zhang@arm.com>
@crosbymichael
Copy link
Member

crosbymichael commented Jul 8, 2019

LGTM

Approved with PullApprove

@zhlhahaha
Copy link
Contributor Author

ping @avagin @caniszczyk @cyphar @dqminh

@caniszczyk
Copy link
Contributor

ping @opencontainers/runc-maintainers

@hqhq
Copy link
Contributor

hqhq commented Jul 27, 2019

LGTM

Approved with PullApprove

@hqhq hqhq merged commit 9ae7901 into opencontainers:master Jul 27, 2019
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.

4 participants