Skip to content

Conversation

@priteau
Copy link
Member

@priteau priteau commented Jan 13, 2026

No description provided.

@priteau priteau self-assigned this Jan 13, 2026
@priteau priteau requested a review from a team as a code owner January 13, 2026 12:11
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 primarily consists of updates to comments and documentation across various configuration files, aligning them with upstream changes from the unmaintained/2024.1 branch. Key changes include clarifying default values, updating image URLs, and improving examples. The main functional change is the fix for NTP configuration on infrastructure VMs by adding the infra-vms group to the ntp configuration, as indicated by the pull request title. A release note for this fix has also been added. My review focuses on improving the clarity and correctness of some of the updated comments.

openstackadmin and others added 3 commits January 22, 2026 09:24
Change-Id: Ied6385e21755cd289c16f098eb7fe87719521eba
Signed-off-by: OpenStack Release Bot <infra-root@openstack.org>
Generated-By: openstack/project-config:roles/copy-release-tools-scripts/files/release-tools/functions
Change-Id: I32a9cf99576fef2e4379606f90d2e9563e36b797
Signed-off-by: Pierre Riteau <pierre@stackhpc.com>
@Alex-Welsh Alex-Welsh force-pushed the merge-upstream-2024.1 branch from 79e2e51 to a7479ab Compare January 22, 2026 09:25
@Alex-Welsh Alex-Welsh enabled auto-merge January 22, 2026 09:26
@Alex-Welsh
Copy link
Member

Merge conflict, rebased

@priteau priteau disabled auto-merge January 23, 2026 15:05
@priteau priteau merged commit 4594fe9 into stackhpc/2024.1 Jan 23, 2026
43 of 45 checks passed
@priteau priteau deleted the merge-upstream-2024.1 branch January 23, 2026 15:05
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.

4 participants