Skip to content

Conversation

@seunghun1ee
Copy link
Member

To make upgrading tests to work with new versioned terraform-kayobe-multinode branches.

@seunghun1ee seunghun1ee self-assigned this Dec 8, 2025
@seunghun1ee seunghun1ee requested a review from a team as a code owner December 8, 2025 13:01
@gemini-code-assist
Copy link
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

Copy link
Member

@Alex-Welsh Alex-Welsh left a comment

Choose a reason for hiding this comment

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

https://github.com/stackhpc/stackhpc-openstack-gh-workflows/pull/16/files#diff-ac002649dfe09359bfd52742a41205f21d61442615bad584c5b9e85e3bd158aeR101

Will fail on major upgrade if previous version is undefined. You'll need to update multinode-inputs.py

@seunghun1ee
Copy link
Member Author

Oops

@seunghun1ee seunghun1ee force-pushed the bump-multinode-workflow branch from 0aab73f to 35eff5b Compare December 8, 2025 13:58
@Alex-Welsh Alex-Welsh force-pushed the bump-multinode-workflow branch from 35eff5b to d67e196 Compare December 10, 2025 14:05
@Alex-Welsh Alex-Welsh force-pushed the bump-multinode-workflow branch from d67e196 to bf469d4 Compare December 10, 2025 14:08
To make upgrading tests to work with new versioned
terraform-kayobe-multinode branches.

New workflow variable ``terraform_kayobe_multinode_previous_version``
was introduced. multinode-inputs.py and workflow files were updated
accordingly.
@Alex-Welsh
Copy link
Member

Simplified the logic a bit because it was getting less readable. I've essentially reverted 3ba886b. IDK why I thought that was a good idea in the first place

@Alex-Welsh Alex-Welsh merged commit 4deb6f0 into stackhpc/2025.1 Dec 10, 2025
21 checks passed
@Alex-Welsh Alex-Welsh deleted the bump-multinode-workflow branch December 10, 2025 14:26
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