Skip to content

Add Rocky 9.2 repo versions #532

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 26, 2023
Merged

Add Rocky 9.2 repo versions #532

merged 1 commit into from
Jun 26, 2023
+19 −8

Conversation

Alex-Welsh
Copy link
Member

This change also fixes the formatting of the rocky CRB and highavailability repo versions so that the minor version is correctly chosen dynamically.

priteau
priteau reviewed Jun 12, 2023
View reviewed changes
@Alex-Welsh Alex-Welsh force-pushed the rocky9.2-repos branch 2 times, most recently from 3ec2f7c to a70a165 Compare June 12, 2023 10:17
priteau
priteau previously approved these changes Jun 12, 2023
View reviewed changes
Alex-Welsh
Alex-Welsh commented Jun 12, 2023
View reviewed changes
@Alex-Welsh Alex-Welsh marked this pull request as ready for review June 13, 2023 08:42
@Alex-Welsh Alex-Welsh requested a review from a team as a code owner June 13, 2023 08:42
@Alex-Welsh
Copy link
Member Author

Promotion is done: https://github.com/stackhpc/stackhpc-release-train/actions/runs/5253811217

@priteau priteau force-pushed the rocky9.2-repos branch 2 times, most recently from 31dcc03 to e5ec3cd Compare June 16, 2023 13:45
@priteau priteau changed the title WIP: Add Rocky 9.2 repo versions Add Rocky 9.2 repo versions Jun 16, 2023
priteau
priteau previously approved these changes Jun 16, 2023
View reviewed changes
Copy link
Member

@priteau priteau left a comment

Choose a reason for hiding this comment

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

It works fine, thanks @Alex-Welsh.

@markgoddard
Copy link
Contributor

RL9 jobs failing:

TASK [dev-tools : Ensure required packages are installed] **********************
Monday 19 June 2023  15:11:49 +0000 (0:00:01.790)       0:01:29.760 *********** 
fatal: [controller0]: FAILED! => changed=false 
  msg: 'Failed to download metadata for repo ''extras'': Cannot download repomd.xml: Cannot download repodata/repomd.xml: All mirrors were tried'
  rc: 1
  results: []

@Alex-Welsh
Add Rocky 9.2 repo versions
b4396af 
This change also fixes the formatting of the rocky CRB and
highavailability repo versions so that the minor version is correctly
chosen dynamically.

The default repos for Rocky linux are now 9.2
@Alex-Welsh
Copy link
Member Author

Alex-Welsh commented Jun 22, 2023 •
edited
Loading

RL9 jobs failing

stackhpc_pulp_repo_rocky_9_minor_version now set at 1 for CI

We should be using workflow-built images in CI soon and it'll be reverted then

@markgoddard
Copy link
Contributor

RL9 jobs failing

stackhpc_pulp_repo_rocky_9_minor_version now set at 1 for CI

We should be using workflow-built images in CI soon and it'll be reverted then

Could you explain the problem?

@Alex-Welsh
Copy link
Member Author

Alex-Welsh commented Jun 22, 2023 •
edited
Loading

RL9 jobs failing

stackhpc_pulp_repo_rocky_9_minor_version now set at 1 for CI
We should be using workflow-built images in CI soon and it'll be reverted then

Could you explain the problem?

My assumption is that because the default minor version is now 2, it's trying to use Rocky 9.2 repos when the CI image is fixed at 9.1 for now, which is causing it to fail

@markgoddard
Copy link
Contributor

RL9 jobs failing

stackhpc_pulp_repo_rocky_9_minor_version now set at 1 for CI
We should be using workflow-built images in CI soon and it'll be reverted then

Could you explain the problem?

My assumption is that because the default minor version is now 2, it's trying to use Rocky 9.2 repos when the CI image is fixed at 9.1 for now, which is causing it to fail

Shouldn't be an issue if we overwrite all of the repos to 9.2. Perhaps we're missing a repo?

@m-bull
Copy link
Contributor

m-bull commented Jun 22, 2023

Some of those repo versions exist on Ark, but not on test-pulp because of failed syncs. The failed syncs are (mostly) fixed now.

@Alex-Welsh
Copy link
Member Author

Some of those repo versions exist on Ark, but not on test-pulp because of failed syncs. The failed syncs are (mostly) fixed now.

Seems to be working now: OVS, OVN
Broken ones for comparison: OVS, OVN

markgoddard
markgoddard approved these changes Jun 26, 2023
View reviewed changes
Copy link
Contributor

@markgoddard markgoddard left a comment

Choose a reason for hiding this comment

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

We need to get this RL CI sorted as we're a bti blind at the moment.

@markgoddard markgoddard merged commit 9cfa4bb into stackhpc/yoga Jun 26, 2023
@markgoddard markgoddard deleted the rocky9.2-repos branch June 26, 2023 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@m-bull m-bull m-bull left review comments

@markgoddard markgoddard markgoddard approved these changes

@priteau priteau priteau left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Alex-Welsh @markgoddard @m-bull @priteau