Skip to content

fix: test if admin-openrc.sh exists before deploying os-capacity #1345

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 1 commit into from
Nov 6, 2024

Conversation

jackhodgkiss
Copy link
Contributor

During service deploys using Kayobe Automation this playbook will fail as the admin-openrc.sh file is not generated during the deployment process.

This in turn causes the workflow to be reported as a failure even though the service deployment succeeded.

@jackhodgkiss jackhodgkiss added the bug Something isn't working label Oct 25, 2024
@jackhodgkiss jackhodgkiss self-assigned this Oct 25, 2024
@jackhodgkiss jackhodgkiss requested a review from a team as a code owner October 25, 2024 13:58
Copy link
Contributor

@assumptionsandg assumptionsandg left a comment

Choose a reason for hiding this comment

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

LGTM, Nice catch!

@jackhodgkiss jackhodgkiss marked this pull request as draft October 29, 2024 09:51
@jackhodgkiss jackhodgkiss force-pushed the 2023.1-os-capacity-no-admin branch from 90e2631 to 5f54d97 Compare October 29, 2024 12:45
@jackhodgkiss jackhodgkiss marked this pull request as ready for review October 29, 2024 12:45
@jackhodgkiss jackhodgkiss force-pushed the 2023.1-os-capacity-no-admin branch from 5f54d97 to 2a7e720 Compare October 29, 2024 13:14
During service deploys using Kayobe Automation this playbook will fail
as the `admin-openrc.sh` file is not generated during the deployment
process.

This in turn causes the workflow to be reported as a failure even though
the service deployment succeeded.
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.

lgtm

@Alex-Welsh Alex-Welsh merged commit e8c03a2 into stackhpc/2023.1 Nov 6, 2024
12 checks passed
@Alex-Welsh Alex-Welsh deleted the 2023.1-os-capacity-no-admin branch November 6, 2024 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants