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

[6.15.z] Quick fix- CI failure- for number of found CV publish tasks #14920

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #14890

Problem Statement

test_negative_publish_during_repo_sync: Sometimes, more than one content-view publish task is present,
so we should search that there are > 0, and get most recent task from search, [index 0].

  • The reason seems to be because in this module, when the full session is run, an associated composite content-view still exists from another test, which will generate extra task(s).

Related Issues

CI failure for stream, 6.15.z combined sessions, because occasionally we find 2 or 3 matching tasks, instead of exactly 1.

PRT test Cases example

trigger: test-robottelo
pytest: tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_negative_publish_during_repo_sync

CI fix for CV publish tasks

(cherry picked from commit 15101cf)
@Satellite-QE Satellite-QE added the 6.15.z Introduced in or relating directly to Satellite 6.15 label Apr 30, 2024
@Satellite-QE Satellite-QE requested a review from a team as a code owner April 30, 2024 09:14
@Satellite-QE Satellite-QE added Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels Apr 30, 2024
@Gauravtalreja1
Copy link
Collaborator

trigger: test-robottelo
pytest: tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_negative_publish_during_repo_sync

@Gauravtalreja1 Gauravtalreja1 added the AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing label May 2, 2024
@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 6778
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_negative_publish_during_repo_sync --external-logging
Test Result : ================== 1 failed, 10 warnings in 699.40s (0:11:39) ==================

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label May 2, 2024
@Gauravtalreja1
Copy link
Collaborator

@damoore044 Could you resolve the conflicts and investigate the PRT failures?

@damoore044
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_negative_publish_during_repo_sync

@damoore044
Copy link
Contributor

@Gauravtalreja1 This test case was not brought into 6.15, mistakenly added the CP tag it seems.

@damoore044 damoore044 closed this Jun 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.15.z Introduced in or relating directly to Satellite 6.15 Auto_Cherry_Picked Automatically cherrypicked PR using GHA AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing No-CherryPick PR doesnt need CherryPick to previous branches PRT-Failed Indicates that latest PRT run is failed for the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants