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

[BUG] Docker distribution builds are failing. Switching to http://vault.centos.org #2024

Merged
merged 1 commit into from
Feb 1, 2022

Conversation

reta
Copy link
Collaborator

@reta reta commented Jan 31, 2022

Signed-off-by: Andriy Redko [email protected]

Description

Docker distribution builds are failing across the board

CentOS Linux 8 - AppStream                      144  B/s |  38  B     00:00    
| �[91mError: Failed to download metadata for repo 'appstream': Cannot prepare internal mirrorlist: No URLs in mirrorlist
| �[0myum error: retry 1 in 10s
| CentOS Linux 8 - AppStream                      176  B/s |  38  B     00:00    
| �[91mError: Failed to download metadata for repo 'appstream': Cannot prepare internal mirrorlist: No URLs in mirrorlist
| �[0myum error: retry 2 in 10s
| CentOS Linux 8 - AppStream                      228  B/s |  38  B     00:00    
| �[91mError: Failed to download metadata for repo 'appstream': Cannot prepare internal mirrorlist: No URLs in mirrorlist
| �[0myum error: retry 3 in 10s
| CentOS Linux 8 - AppStream                      209  B/s |  38  B     00:00    
| �[91mError: Failed to download metadata for repo 'appstream': Cannot prepare internal mirrorlist: No URLs in mirrorlist
| �[0myum error: retry 4 in 10s
| CentOS Linux 8 - AppStream                      405  B/s |  38  B     00:00    
| �[91mError: Failed to download metadata for repo 'appstream': Cannot prepare internal mirrorlist: No URLs in mirrorlist
| �[0myum error: retry 5 in 10s
| CentOS Linux 8 - AppStream                       86  B/s |  38  B     00:00    
| �[91mError: Failed to download metadata for repo 'appstream': Cannot prepare internal mirrorlist: No URLs in mirrorlist
| �[0myum error: retry 6 in 10s

Issues Resolved

Closes #2023

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@reta reta requested a review from a team as a code owner January 31, 2022 22:15
@opensearch-ci-bot
Copy link
Collaborator

Can one of the admins verify this patch?

@owaiskazi19
Copy link
Member

owaiskazi19 commented Jan 31, 2022

Thanks for the change @reta. I was looking into the same issue and found the below

RUN sed -i 's/mirrorlist/#mirrorlist/g' /etc/yum.repos.d/CentOS-Linux-*

RUN sed -i 's|#baseurl=http://mirror.centos.org|baseurl=http://vault.centos.org|g' /etc/yum.repos.d/CentOS-Linux-*

Let me know what you think?

@reta
Copy link
Collaborator Author

reta commented Jan 31, 2022

Thanks for the change @reta. I was looking into the same issue and found the below

RUN sed -i 's/mirrorlist/#mirrorlist/g' /etc/yum.repos.d/CentOS-Linux-*

RUN sed -i 's|#baseurl=http://mirror.centos.org|baseurl=http://vault.centos.org|g' /etc/yum.repos.d/CentOS-Linux-*

Let me know what you think?

Thanks @owaiskazi19, I haven't tried this way yet (seems like it may not be sufficient) but let me check it

@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure 8d553d0dd8283f73a8c625bf371bec02a01054b9
Log 2137

Reports 2137

@reta
Copy link
Collaborator Author

reta commented Jan 31, 2022

@owaiskazi19 I think your approach with changing the repo is better and does keep CentOS 8 distribution (instead of updating to Stream), thank you, I updated the pull request with this change.

@reta reta changed the title [BUG] Docker distribution builds are failing. Moving from CentOS Linux 8 to CentOS Stream 8 [BUG] Docker distribution builds are failing. Switching to http://vault.centos.org Jan 31, 2022
@owaiskazi19
Copy link
Member

owaiskazi19 commented Jan 31, 2022

@owaiskazi19 I think your approach with changing the repo is better and does keep CentOS 8 distribution (instead of updating to Stream), thank you, I updated the pull request with this change.

I'm glad it worked. Thanks for considering the suggestion. Keeping the reference here: https://www.centos.org/centos-linux-eol/

Copy link
Member

@owaiskazi19 owaiskazi19 left a comment

Choose a reason for hiding this comment

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

Can you please edit the issue with the proposed solution? Thanks.

@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure 3d39225f5194f85ba049e0e40f9b6474b65c56fe
Log 2138

Reports 2138

@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success 8364c27
Log 2139

Reports 2139

@reta
Copy link
Collaborator Author

reta commented Feb 1, 2022

@andrross @dblock sorry guys, mind taking a look? we have all builds stuck on this issue, thanks

@andrross andrross merged commit 6c2f01a into opensearch-project:main Feb 1, 2022
@owaiskazi19 owaiskazi19 added the pending backport Identifies an issue or PR that still needs to be backported label Feb 1, 2022
@owaiskazi19
Copy link
Member

@reta @andrross do we need to backport this change to other branches like 1.1, 1.2.. as well?

@reta
Copy link
Collaborator Author

reta commented Feb 1, 2022

@reta @andrross do we need to backport this change to other branches like 1.1, 1.2.. as well?

Sorry, missed that, for 1.2 - definitely, for 1.1 - not sure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending backport Identifies an issue or PR that still needs to be backported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Docker distribution builds are failing
5 participants