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

Add coverage for katello-ca certificate refresh #15870

Merged
merged 1 commit into from
Aug 14, 2024
Merged

Add coverage for katello-ca certificate refresh #15870

merged 1 commit into from
Aug 14, 2024

Conversation

shubhamsg199
Copy link
Contributor

Adding coverage for katello-ca certificate refresh

@shubhamsg199 shubhamsg199 added QETestCoverage Issues and PRs relating to a Satellite bug No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master labels Aug 6, 2024
@shubhamsg199 shubhamsg199 self-assigned this Aug 6, 2024
@shubhamsg199 shubhamsg199 requested a review from a team as a code owner August 6, 2024 12:08
@shubhamsg199 shubhamsg199 marked this pull request as draft August 6, 2024 12:09
@shubhamsg199 shubhamsg199 force-pushed the katello-ca-crt branch 2 times, most recently from 64f42ab to 71f8853 Compare August 6, 2024 12:10
@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_registration.py -k test_positive_katello_ca_crt_refresh

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8063
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_registration.py -k test_positive_katello_ca_crt_refresh --external-logging
Test Result : ========= 2 failed, 16 deselected, 138 warnings in 1507.44s (0:25:07) ==========

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Aug 9, 2024
@shubhamsg199 shubhamsg199 marked this pull request as ready for review August 9, 2024 14:09
@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_registration.py -k test_positive_katello_ca_crt_refresh

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8066
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_registration.py -k test_positive_katello_ca_crt_refresh --external-logging
Test Result : ========= 2 failed, 16 deselected, 132 warnings in 1218.73s (0:20:18) ==========

@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_registration.py -k test_positive_katello_ca_crt_refresh

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8069
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_registration.py -k test_positive_katello_ca_crt_refresh --external-logging
Test Result : ========== 16 deselected, 23 warnings, 2 errors in 203.04s (0:03:23) ===========

@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_registration.py -k test_positive_katello_ca_crt_refresh

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8070
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_registration.py -k test_positive_katello_ca_crt_refresh --external-logging
Test Result : ========= 2 passed, 16 deselected, 155 warnings in 1335.22s (0:22:15) ==========

@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 Aug 9, 2024
shweta83
shweta83 requested changes Aug 13, 2024
View reviewed changes
tests/foreman/api/test_registration.py Outdated Show resolved Hide resolved
@shubhamsg199 shubhamsg199 removed their assignment Aug 13, 2024
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Aug 13, 2024
@shweta83
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/api/test_registration.py -k test_positive_katello_ca_crt_refresh

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8096
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_registration.py -k test_positive_katello_ca_crt_refresh --external-logging
Test Result : ========== 16 deselected, 23 warnings, 2 errors in 202.19s (0:03:22) ===========

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Aug 13, 2024
@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_registration.py -k test_positive_katello_ca_crt_refresh

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8097
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_registration.py -k test_positive_katello_ca_crt_refresh --external-logging
Test Result : ========== 16 deselected, 23 warnings, 2 errors in 196.39s (0:03:16) ===========

@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_registration.py -k test_positive_katello_ca_crt_refresh

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8098
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_registration.py -k test_positive_katello_ca_crt_refresh --external-logging
Test Result : ========= 2 passed, 16 deselected, 651 warnings in 7489.38s (2:04:49) ==========

@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 Aug 13, 2024
Gauravtalreja1
Gauravtalreja1 reviewed Aug 13, 2024
View reviewed changes
Copy link
Collaborator

@Gauravtalreja1 Gauravtalreja1 left a comment

Choose a reason for hiding this comment

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

ACK, with minor suggestions

tests/foreman/api/test_registration.py Outdated Show resolved Hide resolved
tests/foreman/api/test_registration.py Outdated Show resolved Hide resolved
tests/foreman/api/test_registration.py Outdated Show resolved Hide resolved
tests/foreman/api/test_registration.py Outdated Show resolved Hide resolved
tests/foreman/api/test_registration.py Show resolved Hide resolved
tests/foreman/api/test_registration.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 Aug 14, 2024
@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_registration.py -k test_positive_katello_ca_crt_refresh

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8101
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_registration.py -k test_positive_katello_ca_crt_refresh --external-logging
Test Result : ========= 2 passed, 16 deselected, 164 warnings in 1688.89s (0:28:08) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Aug 14, 2024
@Gauravtalreja1 Gauravtalreja1 enabled auto-merge (squash) August 14, 2024 13:33
shweta83
shweta83 approved these changes Aug 14, 2024
View reviewed changes
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

@Gauravtalreja1 Gauravtalreja1 merged commit 2fc5de4 into SatelliteQE:master Aug 14, 2024
10 checks passed
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
@shubhamsg199 @jyejare
Add coverage for katello-ca certificate refresh (SatelliteQE#15870)
7a676f2 
Signed-off-by: Shubham Ganar <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

@shweta83 shweta83 shweta83 approved these changes

Assignees

@shubhamsg199 shubhamsg199

Labels
No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR QETestCoverage Issues and PRs relating to a Satellite bug Stream Introduced in or relating directly to Satellite Stream/Master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@shubhamsg199 @Satellite-QE @shweta83 @Gauravtalreja1