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

[Task 1 - Customer Scenario Checks] Replacing Migrated BZs docstrings token to Jira based #15812

Merged
merged 1 commit into from
Aug 9, 2024

Conversation

jyejare
Copy link
Member

@jyejare jyejare commented Jul 31, 2024

Problem Statement 1

The customer scenario check is based on the Bugzilla today, but since bug management is moved to Jira and the bugs have been closed as CLOSED MIGRATED, the customer scenario check is broken.

Also, the new bugs are being created on the Jira, hence it's vital to update the test docstrings to have the details from Jira instead.

Solution 1

  • Migrate all the bugs closed as CLOSED MIGRATED to get their latest updates from Jira as a new destination.

Also,
The BZs which are not migrated are kept for reference
Some of the tests have already covered the verified token and hence only the BZ reference is removed.

Next

  • Step 2 / Next PR would switch the logic of customer scenario checks from BZ to Jira.

Notes

Why are we linking to the parent Jira to the automated test even though there are clones an subtasks?

  • The today's focus is Just to remediate the broken customer scenario check based on BZ
  • The fix_version was missing on some of the clones at the time of writing this PR
  • Parents has the customer cases attached, whereas some clones don’t hence finding the link to customer scenarios is hard

The issue that could happen to link the test to parent Jira ?

  • The broken status-based test collection or any other logic

Should we be needing the revisit, if some logic/implementation needs stricter status of a jira ids linked to the tests?

  • Yes, we may. But for now, fixing the customer scenario checks is more important.

@jyejare jyejare requested review from a team as code owners July 31, 2024 12:13
@jyejare jyejare added the CherryPick PR needs CherryPick to previous branches label Jul 31, 2024
@jyejare jyejare added 6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 labels Jul 31, 2024
Copy link
Contributor

@vsedmik vsedmik left a comment

Choose a reason for hiding this comment

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

Just a nitpick - till now we are using uppercase :Verifies: everywhere, shall we stick to it?
(yes, I am aware of this)

Copy link
Contributor

@shweta83 shweta83 left a comment

Choose a reason for hiding this comment

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

Ack

@jyejare jyejare requested a review from a team August 7, 2024 09:42
Copy link
Contributor

@jameerpathan111 jameerpathan111 left a comment

Choose a reason for hiding this comment

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

@jyejare Ack, but let's be consistent and use uppercase Verifies.

@shweta83
Copy link
Contributor

shweta83 commented Aug 7, 2024

@jyejare Ack, but let's be consistent and use uppercase Verifies.

I am in favor to not stick with case, and we have added ignorecase for the same reason so we don't have to worry about it.

@jyejare
Copy link
Member Author

jyejare commented Aug 7, 2024

@jameerpathan111 @vsedmik @shweta83 Yes, Tokens are case-insensitive. I mean I would do it in this PR but cant promise this would be followed by everyone later. Since there is no impact of case here, let it be any case.

@jyejare jyejare enabled auto-merge (squash) August 8, 2024 12:04
@jyejare jyejare disabled auto-merge August 8, 2024 12:05
@jyejare jyejare merged commit d0308ee into SatelliteQE:master Aug 9, 2024
8 checks passed
jyejare added a commit to jyejare/robottelo that referenced this pull request Aug 9, 2024
… token to Jira based (SatelliteQE#15812)

Replacing Migrated BZs docstrings token to verifies
jyejare added a commit to jyejare/robottelo that referenced this pull request Aug 9, 2024
… token to Jira based (SatelliteQE#15812)

Replacing Migrated BZs docstrings token to verifies
jyejare added a commit to jyejare/robottelo that referenced this pull request Aug 9, 2024
… token to Jira based (SatelliteQE#15812)

Replacing Migrated BZs docstrings token to verifies
jyejare added a commit to jyejare/robottelo that referenced this pull request Aug 9, 2024
… token to Jira based (SatelliteQE#15812)

Replacing Migrated BZs docstrings token to verifies
jyejare added a commit to jyejare/robottelo that referenced this pull request Aug 16, 2024
… token to Jira based (SatelliteQE#15812)

Replacing Migrated BZs docstrings token to verifies
jyejare added a commit to jyejare/robottelo that referenced this pull request Aug 16, 2024
… token to Jira based (SatelliteQE#15812)

Replacing Migrated BZs docstrings token to verifies
jyejare added a commit to jyejare/robottelo that referenced this pull request Aug 21, 2024
… token to Jira based (SatelliteQE#15812)

Replacing Migrated BZs docstrings token to verifies
jyejare added a commit to jyejare/robottelo that referenced this pull request Aug 21, 2024
… token to Jira based (SatelliteQE#15812)

Replacing Migrated BZs docstrings token to verifies
jyejare added a commit to jyejare/robottelo that referenced this pull request Aug 21, 2024
… token to Jira based (SatelliteQE#15812)

Replacing Migrated BZs docstrings token to verifies
jyejare added a commit that referenced this pull request Aug 22, 2024
…cstrings token to Jira based (#15908)

[Task 1 - Customer Scenario Checks] Replacing Migrated BZs docstrings token to Jira based  (#15812)

Replacing Migrated BZs docstrings token to verifies
jyejare added a commit to jyejare/robottelo that referenced this pull request Aug 22, 2024
… token to Jira based (SatelliteQE#15812)

Replacing Migrated BZs docstrings token to verifies
jyejare added a commit that referenced this pull request Aug 22, 2024
…cstrings token to Jira based (#15909)

[Task 1 - Customer Scenario Checks] Replacing Migrated BZs docstrings token to Jira based  (#15812)

Replacing Migrated BZs docstrings token to verifies
jyejare added a commit that referenced this pull request Aug 22, 2024
…cstrings token to Jira based (#15910)

[Task 1 - Customer Scenario Checks] Replacing Migrated BZs docstrings token to Jira based  (#15812)

Replacing Migrated BZs docstrings token to verifies
jyejare added a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
… token to Jira based (SatelliteQE#15812)

Replacing Migrated BZs docstrings token to verifies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 CherryPick PR needs CherryPick to previous branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants