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.

Sign up for GitHub

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

Jump to bottom

CC Automation/Comp Eval coverage - promote CCV with specific registry_name #14451

Merged
merged 7 commits into from
Apr 30, 2024
Merged

CC Automation/Comp Eval coverage - promote CCV with specific registry_name #14451

merged 7 commits into from
Apr 30, 2024

Conversation

sambible
Copy link
Contributor

CC Automation/Comp Eval for CV work for BZ: https://bugzilla.redhat.com/show_bug.cgi?id=2153523

@sambible sambible added CherryPick PR needs CherryPick to previous branches 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 Mar 19, 2024
@sambible sambible self-assigned this Mar 19, 2024
@sambible sambible requested a review from a team as a code owner March 19, 2024 17:50
@sambible sambible changed the title CC Automation/Comp Eval coverage CC Automation/Comp Eval coverage - promote CCV with specific registry_name Mar 19, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6154
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_contentview.py -k test_ccv_promote_registry_name_change --external-logging
Test Result : ================= 87 deselected, 1 error in 786.88s (0:13:06) ==================

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Mar 21, 2024
@lhellebr
Copy link
Contributor

PRT isn't happy

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6287
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_contentview.py -k test_ccv_promote_registry_name_change --external-logging
Test Result : =========== 87 deselected, 3 warnings, 1 error in 705.16s (0:11:45) ============

@Satellite-QE Satellite-QE removed the PRT-Failed Indicates that latest PRT run is failed for the PR label Apr 8, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6360
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_contentview.py -k test_ccv_promote_registry_name_change --external-logging
Test Result : =========== 87 deselected, 3 warnings, 1 error in 784.75s (0:13:04) ============

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Apr 8, 2024
@sambible
Copy link
Contributor Author

sambible commented Apr 8, 2024

trigger: test-robottelo
pytest: tests/foreman/api/test_contentview.py -k 'test_ccv_promote_registry_name_change'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6363
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_contentview.py -k test_ccv_promote_registry_name_change --external-logging
Test Result : =========== 1 passed, 87 deselected, 4 warnings in 854.96s (0:14:14) ===========

@Satellite-QE Satellite-QE added PRT-Passed Indicates that latest PRT run is passed for the PR and removed PRT-Failed Indicates that latest PRT run is failed for the PR labels Apr 8, 2024
damoore044
damoore044 approved these changes Apr 11, 2024
View reviewed changes
Copy link
Contributor

@damoore044 damoore044 left a comment

Choose a reason for hiding this comment

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

ack nice work !

@omkarkhatavkar
Copy link

trigger: test-robottelo
pytest: tests/foreman/api/test_contentview.py -k 'test_ccv_promote_registry_name_change'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 76
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_contentview.py -k test_ccv_promote_registry_name_change --external-logging
Test Result : ========== 1 passed, 88 deselected, 97 warnings in 771.90s (0:12:51) ===========

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6514
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_contentview.py -k test_ccv_promote_registry_name_change --external-logging
Test Result : ========== 1 passed, 88 deselected, 97 warnings in 776.53s (0:12:56) ===========

vsedmik
vsedmik approved these changes Apr 30, 2024
View reviewed changes
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.

Looks nice. @sambible could you rebase, resolve the conflict and push once again?

tests/foreman/api/test_contentview.py Outdated Show resolved Hide resolved
vijaysawant
vijaysawant requested changes Apr 30, 2024
View reviewed changes
Copy link
Contributor

@vijaysawant vijaysawant left a comment

Choose a reason for hiding this comment

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

please look over steps.

tests/foreman/api/test_contentview.py Outdated Show resolved Hide resolved
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Apr 30, 2024
@sambible
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_contentview.py -k 'test_ccv_promote_registry_name_change'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6753
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_contentview.py -k test_ccv_promote_registry_name_change --external-logging
Test Result : ========== 1 passed, 88 deselected, 97 warnings in 804.24s (0:13:24) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Apr 30, 2024
vsedmik
vsedmik approved these changes Apr 30, 2024
View reviewed changes
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.

ACK

@vsedmik vsedmik merged commit 3f1b9e4 into SatelliteQE:master Apr 30, 2024
10 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 30, 2024
@sambible @web-flow
CC Automation/Comp Eval coverage - promote CCV with specific registry…
48bbfd1 
…_name (#14451)

* CC Automation/Comp Eval coverage

* change to non-sca org

* CC Automation/Comp Eval coverage

* change to non-sca org

* Fix formatting issues

* Adjust assert statement, and fix docstring issue

* Adjust assert statement, and fix docstring issue

(cherry picked from commit 3f1b9e4)
@Satellite-QE Satellite-QE mentioned this pull request Apr 30, 2024
Merged
github-actions bot pushed a commit that referenced this pull request Apr 30, 2024
@sambible @web-flow
CC Automation/Comp Eval coverage - promote CCV with specific registry…
00f77f1 
…_name (#14451)

* CC Automation/Comp Eval coverage

* change to non-sca org

* CC Automation/Comp Eval coverage

* change to non-sca org

* Fix formatting issues

* Adjust assert statement, and fix docstring issue

* Adjust assert statement, and fix docstring issue

(cherry picked from commit 3f1b9e4)
@Satellite-QE Satellite-QE mentioned this pull request Apr 30, 2024
Merged
Gauravtalreja1 pushed a commit that referenced this pull request May 2, 2024
@Satellite-QE @sambible
[6.15.z] CC Automation/Comp Eval coverage - promote CCV with specific…
6245bb5 
… registry_name (#14927)

CC Automation/Comp Eval coverage - promote CCV with specific registry_name (#14451)

* CC Automation/Comp Eval coverage

* change to non-sca org

* CC Automation/Comp Eval coverage

* change to non-sca org

* Fix formatting issues

* Adjust assert statement, and fix docstring issue

* Adjust assert statement, and fix docstring issue

(cherry picked from commit 3f1b9e4)

Co-authored-by: Samuel Bible <[email protected]>
Gauravtalreja1 pushed a commit that referenced this pull request May 2, 2024
@Satellite-QE @sambible
[6.14.z] CC Automation/Comp Eval coverage - promote CCV with specific…
88b050c 
… registry_name (#14928)

CC Automation/Comp Eval coverage - promote CCV with specific registry_name (#14451)

* CC Automation/Comp Eval coverage

* change to non-sca org

* CC Automation/Comp Eval coverage

* change to non-sca org

* Fix formatting issues

* Adjust assert statement, and fix docstring issue

* Adjust assert statement, and fix docstring issue

(cherry picked from commit 3f1b9e4)

Co-authored-by: Samuel Bible <[email protected]>
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
@sambible @jyejare
CC Automation/Comp Eval coverage - promote CCV with specific registry…
54102f7 
…_name (SatelliteQE#14451)

* CC Automation/Comp Eval coverage

* change to non-sca org

* CC Automation/Comp Eval coverage

* change to non-sca org

* Fix formatting issues

* Adjust assert statement, and fix docstring issue

* Adjust assert statement, and fix docstring issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vijaysawant vijaysawant vijaysawant requested changes

@damoore044 damoore044 damoore044 approved these changes

@vsedmik vsedmik vsedmik approved these changes

Assignees

@sambible sambible

Labels
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 PRT-Passed Indicates that latest PRT run is passed for the PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@sambible @Satellite-QE @lhellebr @omkarkhatavkar @vijaysawant @vsedmik @damoore044