Skip to content

test: update pinned alpine and busybox images#4033

Merged
crazy-max merged 2 commits into
moby:masterfrom
crazy-max:test-images-update
Jul 20, 2023
Merged

test: update pinned alpine and busybox images#4033
crazy-max merged 2 commits into
moby:masterfrom
crazy-max:test-images-update

Conversation

@crazy-max
Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
@crazy-max crazy-max requested a review from jedevc July 20, 2023 13:25
@crazy-max crazy-max marked this pull request as ready for review July 20, 2023 13:25
Copy link
Copy Markdown
Member

@jedevc jedevc left a comment

Choose a reason for hiding this comment

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

Can we also bump to busybox 1.36.1? It looks like that release has the fix for https://bugs.busybox.net/show_bug.cgi?id=15236 which is why we originally introduced the pins.

@crazy-max
Copy link
Copy Markdown
Member Author

Ah good call out, will do

https://bugs.busybox.net/show_bug.cgi?id=15236 fixes SIGILL
on accessing sha256 on some CPUs

Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Copy link
Copy Markdown
Member

@jedevc jedevc left a comment

Choose a reason for hiding this comment

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

Assuming tests pass... LGTM!

@crazy-max crazy-max merged commit b4c1183 into moby:master Jul 20, 2023
@crazy-max crazy-max deleted the test-images-update branch July 20, 2023 15:07
@crazy-max crazy-max changed the title test: update pinned alpine image to 3.18 test: update pinned alpine and busybox images Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants