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

Change ami and ova workflows input upper_case to lower_case #114

Conversation

Enaraque
Copy link
Member

@Enaraque Enaraque commented Nov 7, 2024

Description

Previously, in the workflow_dispatch of the OVA and AMI build workflows, inputs in upper_case were used. In order to follow a standard in the format of the inputs with the rest of the workflows, the style has been changed to lower_case.

Testing

Tests have been performed on both workflows to verify that everything is still working as expected. In the AMI workflow it has been completely executed and in the OVA workflow it has been stopped from the steps that did not use the inputs.
Both workflows have been executed correctly.

Related issue

@Enaraque Enaraque requested a review from a team November 7, 2024 16:35
@Enaraque Enaraque self-assigned this Nov 7, 2024
@Enaraque Enaraque changed the title Change ami and ova workflows input upper_case to lowe_case Change ami and ova workflows input upper_case to lower_case Nov 7, 2024
CarlosALgit
CarlosALgit previously approved these changes Nov 7, 2024
@teddytpc1 teddytpc1 merged commit 55c69fc into 4.10.0 Nov 8, 2024
@teddytpc1 teddytpc1 deleted the bug/113-request-for-changes-to-standardize-parameters-in-workflows-for-ova-and-ami-creation-2 branch November 8, 2024 11:53
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.

Request for changes to standardize parameters in workflows for OVA and AMI creation
3 participants