Skip to content

xena: wallaby merge #641

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

Merged
merged 5 commits into from
Sep 14, 2023
Merged

xena: wallaby merge #641

merged 5 commits into from
Sep 14, 2023

Conversation

markgoddard
Copy link
Contributor

  • Restore upstream mirrors in container image package repos
  • CI: Only sync old images after build

Currently the Kolla container images configure Yum repositories to use
pulp-server on SMS Cloud:

(nova-compute) $ grep baseurl /etc/yum.repos.d/CentOS-Stream-BaseOS.repo
baseurl=http://pulp-server.internal.sms-cloud:8080/pulp/content/centos/8-stream/BaseOS/x86_64/os/20211122T102435

This makes it difficult to install/update packages for debugging issues
in customer environments.

This change restores Yum repository files at the end of the container image
build to point to distribution repositories. It also makes an attempt to
do the same for Apt repos, although these are not enabled in Wallaby
currently.
Restore upstream mirrors in container image package repos
The new format (without distro & type in the name) is only used from Zed.
@markgoddard markgoddard requested a review from a team as a code owner September 14, 2023 09:00
@markgoddard markgoddard self-assigned this Sep 14, 2023
Copy link
Member

@Alex-Welsh Alex-Welsh left a comment

Choose a reason for hiding this comment

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

LGTM

@markgoddard markgoddard merged commit 4bcfa2b into stackhpc/xena Sep 14, 2023
@markgoddard markgoddard deleted the xena-wallaby-merge branch September 14, 2023 15:28
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