Skip to content

WIP: Set testrun name for fusion and fdf deployments #12771

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

clacroix12
Copy link
Contributor

No description provided.

@clacroix12 clacroix12 requested a review from a team July 23, 2025 20:15
@clacroix12 clacroix12 requested a review from a team as a code owner July 23, 2025 20:15
@clacroix12 clacroix12 added Needs Deployment Testing Needs Deployment and Some Tier1 Testing team/ecosystem Ecosystem team related issues/PRs labels Jul 23, 2025
@pull-request-size pull-request-size bot added the size/S PR that changes 10-29 lines label Jul 23, 2025
Copy link

openshift-ci bot commented Jul 23, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: clacroix12

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Comment on lines +2461 to +2468
if fdf_version:
product_version = f"FDF{fdf_version}"
elif fusion_version:
product_version = f"Fusion{fusion_version}"
elif ocs_version:
product_version = f"OCS{ocs_version}"
else:
product_version = ""
Copy link
Contributor

Choose a reason for hiding this comment

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

above will skip fusion_version if fdf_version exists. I think we need Fusion and FDF version in subject

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was intentional as I didn't want to have both in the name of the launch. If fdf_version exists, I wanted it to take priority over fusion_version.

@clacroix12 clacroix12 changed the title Set testrun name for fusion and fdf deployments WIP: Set testrun name for fusion and fdf deployments Jul 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge/work-in-progress Needs Deployment Testing Needs Deployment and Some Tier1 Testing size/S PR that changes 10-29 lines team/ecosystem Ecosystem team related issues/PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants