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 upgrade scenario test for docker manifest indexing #15263

Merged
merged 3 commits into from
Jul 2, 2024
Merged

Add upgrade scenario test for docker manifest indexing #15263

merged 3 commits into from
Jul 2, 2024

Conversation

vsedmik
Copy link
Contributor

@vsedmik vsedmik commented May 31, 2024 •
edited
Loading

Problem Statement

We need a test case to verify that docker manifest labels, annotations and other flags are indexed

  • after 6.15 -> 6.16 upgrade in the post-upgrade task.
  • during repo sync at 6.16

Solution

This PR adds such cases.
Needs SatelliteQE/nailgun#1188

Test resuts

For upgrade scenario (delivered in f-m 1.6.8):
PRE:

Launching pytest with arguments -m pre_upgrade -k test_pre_container_repo_sync tests/upgrades/test_repository.py --no-header --no-summary -q in /home/vsedmik/PycharmProjects/robottelo

============================= test session starts ==============================
collecting ... collected 10 items / 9 deselected / 1 selected

tests/upgrades/test_repository.py::TestScenarioContainerRepoSync::test_pre_container_repo_sync 

======================= 1 passed, 9 deselected in 47.61s =======================

POST:

Launching pytest with arguments -m post_upgrade -k test_post_container_repo_sync tests/upgrades/test_repository.py --no-header --no-summary -q in /home/vsedmik/PycharmProjects/robottelo

============================= test session starts ==============================
collecting ... collected 10 items / 9 deselected / 1 selected

tests/upgrades/test_repository.py::TestScenarioContainerRepoSync::test_post_container_repo_sync 

======================= 1 passed, 9 deselected in 17.82s =======================

For repo sync test case

trigger: test-robottelo
pytest: tests/foreman/api/test_repository.py -k manifest_labels
nailgun: 1188

@vsedmik vsedmik added CherryPick PR needs CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master 6.16.z labels May 31, 2024
@vsedmik vsedmik self-assigned this May 31, 2024
@vsedmik vsedmik force-pushed the container-labels branch from 8444faf to 09f2f27 Compare June 3, 2024 18:12
@vsedmik
Copy link
Contributor Author

vsedmik commented Jun 3, 2024

trigger: test-robottelo
pytest: tests/foreman/api/test_repository.py -k manifest_labels
nailgun: 1188

@vsedmik
Copy link
Contributor Author

vsedmik commented Jun 4, 2024

PRT failure is unrelated, I will rerun once PRT is fixed.

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7267
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_repository.py -k manifest_labels --external-logging
Test Result : ========= 2 passed, 164 deselected, 197 warnings in 657.69s (0:10:57) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jun 4, 2024
@jyejare jyejare removed the 6.16.z label Jun 5, 2024
@vsedmik vsedmik force-pushed the container-labels branch from 09f2f27 to 2ae351d Compare June 10, 2024 14:02
@vsedmik
Copy link
Contributor Author

vsedmik commented Jun 10, 2024

(rebased)

@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Jun 10, 2024
@vsedmik vsedmik added No-CherryPick PR doesnt need CherryPick to previous branches and removed CherryPick PR needs CherryPick to previous branches labels Jun 10, 2024
@vsedmik vsedmik marked this pull request as ready for review June 10, 2024 14:05
@vsedmik vsedmik requested review from a team as code owners June 10, 2024 14:05
@vsedmik
Copy link
Contributor Author

vsedmik commented Jun 10, 2024

trigger: test-robottelo
pytest: tests/foreman/api/test_repository.py -k manifest_labels
nailgun: 1188

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7335
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_repository.py -k manifest_labels --external-logging
Test Result : ========= 2 passed, 164 deselected, 197 warnings in 666.86s (0:11:06) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jun 10, 2024
lhellebr
lhellebr approved these changes Jun 11, 2024
View reviewed changes
tests/foreman/api/test_repository.py Outdated Show resolved Hide resolved
tests/upgrades/test_repository.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 Jun 11, 2024
@damoore044 damoore044 merged commit 09d44b7 into SatelliteQE:master Jul 2, 2024
9 checks passed
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
@vsedmik @jyejare
Add upgrade scenario test for docker manifest indexing (SatelliteQE#1…
c0e0362 
…5263)

* Add upgrade scenario test for docker manifest indexing

* Add repo sync test for docker manifest indexing

* Update loop vars to be less confusing
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

@ColeHiggins2 ColeHiggins2 ColeHiggins2 approved these changes

@lpramuk lpramuk lpramuk approved these changes

@lhellebr lhellebr lhellebr approved these changes

Assignees

@vsedmik vsedmik

Labels
No-CherryPick PR doesnt need CherryPick to previous branches 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.
7 participants
@vsedmik @Satellite-QE @lhellebr @lpramuk @ColeHiggins2 @damoore044 @jyejare