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

[CustomerCase] Test Errata type "Other", with inclusive CV Filter, verify content counts #17602

Merged
merged 2 commits into from
Feb 17, 2025

Conversation

damoore044
Copy link
Contributor

@damoore044 damoore044 commented Feb 14, 2025

Purpose:

Covers CustomerCase: SAT-24725 (BZ: 2160804)

  • We use the EPEL 10 repository to sync real Erratum of types that fall under 'Other' (ie newpackage , unspecified)
  • We need to fetch a dynamically created PGP key from https://dl.fedoraproject.org/pub/epel/RPM-GPG-KEY-EPEL-10
  • Syncing EPEL version 10, because it matches the newest RHEL version from supportability

Closely following DEV Testing Steps from Katello MR: Katello/katello#11188

Steps (condensed):

GET request to receive a new PGP key from FedoraProject.
Create a GPG key on satellite from the PGP key received, create product with the GPG assigned.
Use the GPG key to create and sync, large EPEL 10 (x86_64 - everything) as a custom repository.
Publish the 1st CV version with EPEL repo contents added.
Add inclusive Erratum (by Date) CV filter with rules:

  • 'start_date' left unspecified (None)
  • 'end_date' Today (UTC)
  • 'allow-other-types' True

Add inclusive RPM CV filter (All)
Publish the 2nd CV Version with filtering (~10 minutes).

Expected Results:

  • Erratum and RPM counts are the same between the filtered and unfiltered versions.
  • Inclusive filtering of all Errata (by date), including 'Other' types, does not affect RPM filtering or package counts
    for the next published version.
  • The EPEL repository's Erratum are overwhelmingly types that are 'Other' , we can verify this in the content counts.

PRT Case:

trigger: test-robottelo
pytest: tests/foreman/api/test_errata.py::test_positive_filter_errata_type_other

@damoore044 damoore044 added No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master labels Feb 14, 2025
@damoore044 damoore044 self-assigned this Feb 14, 2025
@damoore044 damoore044 requested a review from a team as a code owner February 14, 2025 22:00
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_errata.py::test_positive_filter_errata_type_other

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10187
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_errata.py::test_positive_filter_errata_type_other --external-logging
Test Result : ================== 1 passed, 9 warnings in 1189.00s (0:19:49) ==================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Feb 14, 2025
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Feb 14, 2025
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_errata.py::test_positive_filter_errata_type_other

@damoore044
Copy link
Contributor Author

pre-commit.ci autofix

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10189
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_errata.py::test_positive_filter_errata_type_other --external-logging
Test Result : ================== 1 passed, 9 warnings in 1317.31s (0:21:57) ==================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Feb 14, 2025
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_errata.py::test_positive_filter_errata_type_other

@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Feb 17, 2025
@damoore044 damoore044 requested a review from a team as a code owner February 17, 2025 15:57
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10208
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_errata.py::test_positive_filter_errata_type_other --external-logging
Test Result : ================== 1 passed, 9 warnings in 1254.30s (0:20:54) ==================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Feb 17, 2025
Copy link
Contributor

@jeremylenz jeremylenz left a comment

Choose a reason for hiding this comment

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

Nicely written test, thanks for the comments explaining everything :)

Copy link
Member

@JacobCallahan JacobCallahan left a comment

Choose a reason for hiding this comment

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

That's a beefy test!

@JacobCallahan JacobCallahan merged commit f1e22a9 into SatelliteQE:master Feb 17, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR Stream Introduced in or relating directly to Satellite Stream/Master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants