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

zed: automate Kolla Ansible tag variable definitions #663

Merged
merged 11 commits into from
Feb 2, 2024

Conversation

markgoddard
Copy link
Contributor

@markgoddard markgoddard commented Sep 27, 2023

This change reworks the way that Kolla image tags are defined, making the following improvements possible:

  • Automatic promotion of images on merge (as is currently done for package repos)
  • New CI job that checks that all specified tags are present in Test Pulp. This shows that they have been built, pushed to Ark, and also synced to Test Pulp on SMS lab
  • Automatic definition of the _tag variables for Kolla Ansible

The main visible change is that container image tags are now defined in the kolla_image_tags variable in etc/kayobe/kolla-image-tags.yml. This avoids the previous churn in the etc/kayobe/kolla/globals.yml file.

A new script, tools/kolla-images.py, understands the Kolla image and container relationships, and is able to display the tags used by each image. This output is parsed in the new container promotion workflows added in stackhpc/stackhpc-release-train#193.

@markgoddard markgoddard self-assigned this Sep 27, 2023
@markgoddard markgoddard changed the title zed auto tags zed: automate Kolla Ansible tag variable definitions Sep 27, 2023
@markgoddard markgoddard force-pushed the zed-auto-tags branch 6 times, most recently from 071ba56 to 66c8335 Compare October 25, 2023 15:51
@markgoddard markgoddard marked this pull request as ready for review October 25, 2023 16:03
@markgoddard markgoddard requested a review from a team as a code owner October 25, 2023 16:03
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.

Nice code 👍 Just a few minor comments inline. Also please fix merge conflicts.

tools/kolla-images.py Outdated Show resolved Hide resolved
tools/kolla-images.py Outdated Show resolved Hide resolved
tools/kolla-images.py Outdated Show resolved Hide resolved
tools/kolla-images.py Show resolved Hide resolved
The previous tag was not in Ark, but this one is present.
The failing case was the haproxy image used by the neutron_tls_proxy
container, if a tag is defined for the 'neutron' prefix but not
'haproxy'.
Copy link
Contributor

@jovial jovial left a comment

Choose a reason for hiding this comment

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

Lets give a whirl. Seems like it will save some time :)

@markgoddard markgoddard merged commit ee5fa79 into stackhpc/zed Feb 2, 2024
10 checks passed
@markgoddard markgoddard deleted the zed-auto-tags branch February 2, 2024 14:03
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