Skip to content

Conversation

@technowhizz
Copy link
Contributor

@technowhizz technowhizz commented Jan 22, 2026

Implements commits: 9c85c3d and f23399c

Update docs to get pulp to also publish and promote noble images

Move repo additions to happen before do-release-upgrade

Moves the addition of new Ubuntu repositories to directly before the call to do-release-upgrade. This ensures that any any failures after that point are only due to the upgrade itself, and not due to missing mounts (such as /tmp not being mounted).

Implements commits: 9c85c3d and
f23399c

Update docs to get pulp to also publish and promote noble images

Move repo additions to happen before do-release-upgrade
Moves the addition of new Ubuntu repositories to directly before the
call to do-release-upgrade. This ensures that any any failures after
that point are only due to the upgrade itself, and not due to missing
mounts (such as /tmp not being mounted).
@technowhizz technowhizz self-assigned this Jan 22, 2026
@technowhizz technowhizz requested a review from a team as a code owner January 22, 2026 15:01
@technowhizz technowhizz enabled auto-merge January 22, 2026 15:03
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces several improvements to the Ubuntu Noble upgrade process. The documentation is updated with a critical warning regarding Ceph versions and clarifies the commands for publishing and promoting Pulp repositories. The Ansible playbook for the upgrade is refactored to move the repository addition step, making the process more robust. My review includes suggestions to improve the readability of the updated documentation by reducing command repetition. Overall, these are valuable changes that enhance the clarity and reliability of the upgrade procedure.

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.

3 participants