Skip to content
This repository has been archived by the owner on Jul 18, 2023. It is now read-only.

Remove useless polarion upgrade testrun suffix #1924

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lpramuk
Copy link
Contributor

@lpramuk lpramuk commented Apr 30, 2021

Currently for Upgrades the script produces testrun named:
Satellite 6.9.2-1.0 rhel7 upgrade - Upgrade

This PR removes useless - Upgrade suffix

@lpramuk lpramuk requested a review from jyejare April 30, 2021 17:20
@lpramuk lpramuk self-assigned this Apr 30, 2021
@lpramuk lpramuk requested a review from mshriver April 30, 2021 17:21
@@ -31,7 +31,6 @@ TEST_RUN_GROUP_ID="$(echo ${TEST_RUN_ID} | cut -d' ' -f2)"
# Prepare the XML files

if [[ "${TEST_RUN_ID}" = *"upgrade"* ]]; then
TEST_RUN_ID="${TEST_RUN_ID} - Upgrade"
Copy link
Contributor Author

@lpramuk lpramuk Apr 30, 2021

Choose a reason for hiding this comment

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

It already contains upgrade so put addition Upgrade yet for double portion of Upgrades :-D

Copy link
Member

Choose a reason for hiding this comment

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

Make sure that the polarion reports used by mgmt aren't dependent on that testrun ID pattern

Copy link
Member

@JacobCallahan JacobCallahan left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@jyejare jyejare left a comment

Choose a reason for hiding this comment

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

ACK pending Mikes Comments !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants