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 test to sync repo having zst compression #14398

Merged
merged 1 commit into from
Mar 22, 2024
Merged

Add test to sync repo having zst compression #14398

merged 1 commit into from
Mar 22, 2024

Conversation

shubhamsg199
Copy link
Contributor

@shubhamsg199 shubhamsg199 commented Mar 13, 2024 •
edited
Loading

Adding a test to sync repository having zst type of compression
Closed loop BZ#2241934

@shubhamsg199 shubhamsg199 added QETestCoverage Issues and PRs relating to a Satellite bug CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 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 13, 2024
@shubhamsg199 shubhamsg199 self-assigned this Mar 13, 2024
@shubhamsg199 shubhamsg199 requested review from a team as code owners March 13, 2024 17:57
@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_repositories.py -k test_positive_sync_upstream_repo_with_zst_compression

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6062
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_repositories.py -k test_positive_sync_upstream_repo_with_zst_compression --external-logging
Test Result : =========== 1 passed, 10 deselected, 1 warning in 776.43s (0:12:56) ============

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Mar 13, 2024
@vsedmik
Copy link
Contributor

vsedmik commented Mar 13, 2024

I don't think we need to sync the skype repo specifically. All rpm repos from https://fixtures.pulpproject.org (we use a good amount of them in robottelo already) switched to zst compression already - see their repomd.xml.

@shubhamsg199 shubhamsg199 force-pushed the pulp_upstream_zst_repo branch 2 times, most recently from 12ac3b6 to 9a5da12 Compare March 18, 2024 08:27
@shubhamsg199 shubhamsg199 removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Mar 18, 2024
@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_repositories.py -k test_positive_sync_upstream_repo_with_zst_compression

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6098
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_repositories.py -k test_positive_sync_upstream_repo_with_zst_compression --external-logging
Test Result : =========== 1 passed, 10 deselected, 1 warning in 741.81s (0:12:21) ============

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Mar 18, 2024
Gauravtalreja1
Gauravtalreja1 reviewed Mar 19, 2024
View reviewed changes
@shubhamsg199 shubhamsg199 force-pushed the pulp_upstream_zst_repo branch 2 times, most recently from df6792c to 30e7b2a Compare March 19, 2024 11:40
@shubhamsg199 shubhamsg199 force-pushed the pulp_upstream_zst_repo branch from 30e7b2a to 8ae8504 Compare March 22, 2024 07:35
@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_repositories.py -k test_positive_sync_upstream_repo_with_zst_compression

@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Mar 22, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6162
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_repositories.py -k test_positive_sync_upstream_repo_with_zst_compression --external-logging
Test Result : =========== 1 passed, 10 deselected, 1 warning in 653.39s (0:10:53) ============

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Mar 22, 2024
@Gauravtalreja1 Gauravtalreja1 merged commit 1bfd367 into SatelliteQE:master Mar 22, 2024
10 checks passed
github-actions bot pushed a commit that referenced this pull request Mar 22, 2024
@shubhamsg199 @web-flow
Add test to sync repo having zst compression (#14398)
efd99f4 
Signed-off-by: Shubham Ganar <[email protected]>
(cherry picked from commit 1bfd367)
@Satellite-QE Satellite-QE mentioned this pull request Mar 22, 2024
Merged
github-actions bot pushed a commit that referenced this pull request Mar 22, 2024
@shubhamsg199 @web-flow
Add test to sync repo having zst compression (#14398)
f6ad474 
Signed-off-by: Shubham Ganar <[email protected]>
(cherry picked from commit 1bfd367)
@Satellite-QE Satellite-QE mentioned this pull request Mar 22, 2024
Merged
shweta83 pushed a commit to shweta83/robottelo that referenced this pull request Apr 10, 2024
@shubhamsg199 @shweta83
Add test to sync repo having zst compression (SatelliteQE#14398)
0c5a9d0 
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

@shweta83 shweta83 shweta83 left review comments

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

@Griffin-Sullivan Griffin-Sullivan Griffin-Sullivan approved these changes

Assignees

@shubhamsg199 shubhamsg199

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 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs 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
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@shubhamsg199 @Satellite-QE @vsedmik @Gauravtalreja1 @shweta83 @Griffin-Sullivan