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

rootless: update to 25.0.4 #397

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

vvoland
Copy link
Contributor

@vvoland vvoland commented Mar 8, 2024

Signed-off-by: Paweł Gronowski [email protected]

Signed-off-by: Paweł Gronowski <[email protected]>
@vvoland vvoland requested a review from thaJeztah March 8, 2024 14:47
@vvoland vvoland self-assigned this Mar 8, 2024
thaJeztah
thaJeztah previously approved these changes Mar 8, 2024
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@thaJeztah
Copy link
Member

Looks like something's broken. Wondering if it's the switch to dnf that we merged; it looks to be hanging on this; no progress, nothing 🤔

+ sh -c 'dnf --best install -y -q dnf-plugins-core'
Error: The operation was canceled.

@vvoland
Copy link
Contributor Author

vvoland commented Mar 8, 2024

Hmm, reverting that PR didn't fix it 🤔

@vvoland
Copy link
Contributor Author

vvoland commented Mar 8, 2024

Looks like the upstream is having some issue:

❱ docker run --rm -it quay.io/centos/centos:stream9
[root@dd732ca3e3e7 /]# yum install -y -q yum-utils

also hangs

@vvoland vvoland force-pushed the update-rootless-25.0.4 branch from c0c22ef to 68fd259 Compare March 8, 2024 16:27
@thaJeztah
Copy link
Member

Thanks for checking! Hm, let me try kicking it again to see if it's already resolved on their side 🤞

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

ci is happy now 🎉

@thaJeztah thaJeztah merged commit 359d71e into docker:master Mar 8, 2024
15 of 17 checks passed
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