Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Update container path separator #17374

Merged
merged 1 commit into from
Jan 21, 2025
Merged

Update container path separator #17374

merged 1 commit into from
Jan 21, 2025

Conversation

vsedmik
Copy link
Contributor

@vsedmik vsedmik commented Jan 17, 2025 •
edited
Loading

Problem Statement

The test_positive_sync_container_repo_end_to_end is failing since the container path separator has been changed in 6.17 from - to / recently.

Solution

This PR should fix it.

Related Issues

Katello/katello#11208

PRT test Cases example

trigger: test-robottelo
pytest: tests/foreman/api/test_capsulecontent.py -k test_positive_sync_container_repo_end_to_end

@vsedmik vsedmik added the No-CherryPick PR doesnt need CherryPick to previous branches label Jan 17, 2025
@vsedmik vsedmik self-assigned this Jan 17, 2025
@vsedmik
Copy link
Contributor Author

vsedmik commented Jan 17, 2025 

trigger: test-robottelo
pytest: tests/foreman/api/test_capsulecontent.py -k test_positive_sync_container_repo_end_to_end

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9927
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_capsulecontent.py -k test_positive_sync_container_repo_end_to_end --external-logging
Test Result : ========== 1 passed, 26 deselected, 45 warnings in 2424.37s (0:40:24) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jan 17, 2025
@vsedmik vsedmik force-pushed the fix-container-separator branch from 3f36cb4 to 169faa6 Compare January 17, 2025 18:06
@vsedmik
Copy link
Contributor Author

vsedmik commented Jan 17, 2025 

trigger: test-robottelo
pytest: tests/foreman/api/test_capsulecontent.py -k test_positive_sync_container_repo_end_to_end

@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Jan 17, 2025
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9929
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_capsulecontent.py -k test_positive_sync_container_repo_end_to_end --external-logging
Test Result : ========== 1 passed, 26 deselected, 45 warnings in 2135.32s (0:35:35) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jan 17, 2025
@vsedmik vsedmik marked this pull request as ready for review January 17, 2025 18:57
@vsedmik vsedmik requested a review from a team as a code owner January 17, 2025 18:57
@pondrejk pondrejk merged commit 707e023 into SatelliteQE:master Jan 21, 2025
12 checks passed
@vsedmik vsedmik deleted the fix-container-separator branch January 21, 2025 14:30
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@pondrejk pondrejk pondrejk approved these changes

@synkd synkd synkd approved these changes

Assignees

@vsedmik vsedmik

Labels
No-CherryPick PR doesnt need 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.
4 participants
@vsedmik @Satellite-QE @pondrejk @synkd