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

[Comp Eval] Fixing tests and adding support for errata install by search #14975

Merged
merged 10 commits into from
May 22, 2024
Merged

[Comp Eval] Fixing tests and adding support for errata install by search #14975

merged 10 commits into from
May 22, 2024

Conversation

ColeHiggins2
Copy link
Member

Reporting component eval. Adding cloning to end to end tests, fixing failing tests, and adding new tests for installing errata by search

RHSAT6-44110
RHSAT6-44106
RHSAT6-40840
RHSAT6-45564

@ColeHiggins2 ColeHiggins2 added CherryPick PR needs CherryPick to previous branches 6.15.z Introduced in or relating directly to Satellite 6.15 labels May 6, 2024
@ColeHiggins2 ColeHiggins2 self-assigned this May 6, 2024
@ColeHiggins2 ColeHiggins2 requested a review from a team as a code owner May 6, 2024 14:38
@ColeHiggins2 ColeHiggins2 requested a review from vijaysawant May 9, 2024 15:23
damoore044
damoore044 approved these changes May 9, 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! quick comment regarding rhel version param, nonblocking

tests/foreman/api/test_reporttemplates.py Outdated Show resolved Hide resolved
@ColeHiggins2 ColeHiggins2 force-pushed the reporting-comp-eval-errata branch from 95a350c to f7f9d9b Compare May 9, 2024 17:28
sambible
sambible approved these changes May 9, 2024
View reviewed changes
Copy link
Contributor

@sambible sambible 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 PRT

@ColeHiggins2 ColeHiggins2 force-pushed the reporting-comp-eval-errata branch from f9ab330 to 3e436d7 Compare May 10, 2024 14:04
@ColeHiggins2 ColeHiggins2 requested a review from vsedmik May 10, 2024 14:06
@ColeHiggins2
Copy link
Member Author

trigger: test-robottelo
pytest: tests/foreman/api/test_reporttemplates.py tests/foreman/cli/test_reporttemplates.py -k 'test_positive_applied_errata_by_search or test_positive_installable_errata or
test_positive_installable_products or
test_positive_end_to_end_crud_and_list'

@ColeHiggins2
Copy link
Member Author

trigger: test-robottelo
pytest: tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search or test_positive_installable_errata

@ColeHiggins2
Copy link
Member Author

trigger: test-robottelo
pytest: tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search

vsedmik
vsedmik approved these changes May 10, 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.

PRT pending

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6885
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search --external-logging
Test Result : ========= 3 failed, 38 deselected, 278 warnings in 1894.62s (0:31:34) ==========

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label May 10, 2024
damoore044
damoore044 requested changes May 10, 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.

This might be a quick fix for PRT failure-

tests/foreman/api/test_reporttemplates.py Outdated Show resolved Hide resolved
@ColeHiggins2 ColeHiggins2 force-pushed the reporting-comp-eval-errata branch from 3e436d7 to b9dd7a1 Compare May 10, 2024 18:25
@ColeHiggins2
Copy link
Member Author

trigger: test-robottelo
pytest: tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6896
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search --external-logging
Test Result : ========= 3 failed, 38 deselected, 299 warnings in 2111.49s (0:35:11) ==========

@ColeHiggins2
Copy link
Member Author

trigger: test-robottelo
pytest: tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6899
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search --external-logging
Test Result : ========= 3 failed, 38 deselected, 273 warnings in 1948.49s (0:32:28) ==========

@ColeHiggins2 ColeHiggins2 force-pushed the reporting-comp-eval-errata branch from 167c891 to 7a4b565 Compare May 13, 2024 16:27
@ColeHiggins2
Copy link
Member Author

trigger: test-robottelo
pytest: tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6907
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search --external-logging
Test Result : ==== 1 failed, 2 passed, 38 deselected, 270 warnings in 1899.27s (0:31:39) =====

@damoore044
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6929
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search --external-logging
Test Result : ==== 2 failed, 1 passed, 38 deselected, 270 warnings in 1842.82s (0:30:42) =====

@ColeHiggins2
Copy link
Member Author

trigger: test-robottelo
pytest: tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7047
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search --external-logging
Test Result : ========= 3 passed, 37 deselected, 267 warnings in 1834.61s (0:30:34) ==========

@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 May 21, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7048
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search --external-logging
Test Result : ==== 1 failed, 2 passed, 37 deselected, 260 warnings in 1793.47s (0:29:53) =====

@Satellite-QE Satellite-QE added PRT-Failed Indicates that latest PRT run is failed for the PR and removed PRT-Passed Indicates that latest PRT run is passed for the PR labels May 21, 2024
@ColeHiggins2
Copy link
Member Author

trigger: test-robottelo
pytest: tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7073
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search --external-logging
Test Result : ==== 1 failed, 2 passed, 37 deselected, 258 warnings in 1874.96s (0:31:14) =====

@ColeHiggins2 ColeHiggins2 force-pushed the reporting-comp-eval-errata branch from bebfc65 to 754c296 Compare May 22, 2024 19:10
@ColeHiggins2
Copy link
Member Author

trigger: test-robottelo
pytest: tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7074
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_reporttemplates.py -k test_positive_applied_errata_by_search --external-logging
Test Result : ========= 3 passed, 37 deselected, 265 warnings in 1764.30s (0:29:24) ==========

@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 May 22, 2024
@ColeHiggins2 ColeHiggins2 requested a review from damoore044 May 22, 2024 21:50
@damoore044 damoore044 merged commit 2a52b85 into SatelliteQE:master May 22, 2024
11 checks passed
github-actions bot pushed a commit that referenced this pull request May 22, 2024
@ColeHiggins2 @web-flow
[Comp Eval] Fixing tests and adding support for errata install by sea…
ccfa599 
…rch (#14975)

* Fixing tests and adding errata by search

* removing max_tries

* updating pytest mark

* fixing other tests

* fixing docstring

* Moving errataname search

* removing location selection

* Adding sub man repo command

* updating erratum

* updating search rate and max tries

(cherry picked from commit 2a52b85)
@Satellite-QE Satellite-QE mentioned this pull request May 22, 2024
Draft
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
@ColeHiggins2 @jyejare
[Comp Eval] Fixing tests and adding support for errata install by sea…
0018f93 
…rch (SatelliteQE#14975)

* Fixing tests and adding errata by search

* removing max_tries

* updating pytest mark

* fixing other tests

* fixing docstring

* Moving errataname search

* removing location selection

* Adding sub man repo command

* updating erratum

* updating search rate and max tries
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@damoore044 damoore044 damoore044 approved these changes

@vsedmik vsedmik vsedmik approved these changes

@vijaysawant vijaysawant vijaysawant approved these changes

@sambible sambible sambible approved these changes

Assignees

@ColeHiggins2 ColeHiggins2

Labels
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.
6 participants
@ColeHiggins2 @Satellite-QE @damoore044 @sambible @vijaysawant @vsedmik