-
Notifications
You must be signed in to change notification settings - Fork 23
Sync wallaby into xena #235
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
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This prevents workflows from stacking up in the queue. Prior to this change, a new workflow was being created for each commit to a PR. If multiple commits are pushed in quick sucession, a large backlog can be built up in the queue. There is not much point in testing an old version of a PR. See: https://stackoverflow.com/questions/66335225/how-to-cancel-previous-runs-in-the-pr-when-you-push-new-commitsupdate-the-curre
Without the --all flag, only dangling images are pruned. This doesn't help much. Also, only prune images over 1 week old on the builder runner.
[CI] Cancel stale workflows when pushing a new change
CI: fix docker image prune
* Adds pulp_proxy_url Pulp requires you to the proxy for each remote. * Default to https_proxy Depends on https://review.opendev.org/c/openstack/kayobe/+/814046 * Update pulp_repo_default The default did not work well when using a proxy deployed on the seed. Leave it up the user to explicitly enable.
* Adds Rocky 8 pulp repositories to dnf.yml (#152) * Adds Rocky 8 pulp repositories to dnf.yml This configures the hosts to use the mirrors in the local pulp. Prior to this change kayobe would attempt to configure CentOS stream mirrors on a Rocky host. * Use facts to pick dnf repositories * Adds Rocky 8 repository config for local pulp (#146) * Adds Rocky 8 repository config for local pulp We support Rocky 8 as a host OS with os_distribution == 'rocky'. This adds the configuration necessary to use the RPM snapshots from ark. * Abstract over version number v1 Unsure if it would be better to: 1) Update the remote on the repository 2) Change the publication to point to a new repository and keep repositories for every minor version Advantages of 1: - Single repository in pulp Advantages of 2: - Less chance of conflicts (although I think an old repo is just an old snapshot) * Fixups * Address code review comment Upadting variable name after feedback * Update docker baseurl to work with Rocky 8 (#153) This is the equivalent change of: https://review.opendev.org/c/openstack/ansible-collection-kolla/+/831642 * Adds a Rocky 8 AIO test * Use ansible fork with Rocky 8 support * Install fork of ansible unconditionally The kayobe docker image fron kayobe-automation was being built without Rocky Linux support. This is because os_distribution was not being set add build time. * Workaround lack of ovs-vsctl on the host. * Switch back to stackhpc/wallaby The PR has now been merged. * centos user has changed to cloud-user The images on smslab are now based on centos-minimal. * Add os-distribution to tempest artifact name This allows you to differentiate the artifacts across the distros. * Remove bootstrap override This is no longer necessary now the centos image also uses cloud-user. * Revert volume_size to 100GB 200G was only necessary for testing the DIB images from kayobe-config with the default LVM logical volume sizes. * Remove configure-local-networking
This was referenced Nov 17, 2022
Merged
markgoddard
approved these changes
Nov 17, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.