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

Fix test_content_guarded_distributions_option #14817

Merged
merged 1 commit into from
Apr 18, 2024
Merged

Fix test_content_guarded_distributions_option #14817

merged 1 commit into from
Apr 18, 2024

Conversation

jameerpathan111
Copy link
Contributor

Problem Statement

  • Fix test_content_guarded_distributions_option failing with assert "403: [('PEM routines', 'get_name', 'no start line')]" in "403: [('PEM routines', '', 'no start line')]" on RHEL9.

Solution

  • Update assertion to only check 403 error code

Related Issues

  • SAT-24616

@jameerpathan111 jameerpathan111 added No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master labels Apr 17, 2024
@jameerpathan111 jameerpathan111 self-assigned this Apr 17, 2024
@jameerpathan111 jameerpathan111 requested a review from a team as a code owner April 17, 2024 14:09
@jameerpathan111
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/installer/test_installer.py -k test_content_guarded_distributions_option

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6575
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/installer/test_installer.py -k test_content_guarded_distributions_option --external-logging
Test Result : ========== 1 passed, 24 deselected, 41 warnings in 3210.03s (0:53:30) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Apr 17, 2024
pondrejk
pondrejk approved these changes Apr 18, 2024
View reviewed changes
Copy link
Contributor

@pondrejk pondrejk left a comment

Choose a reason for hiding this comment

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

ack, I assume prt ran on rhel8 backend. Do you have it checked on rhel9?

@jameerpathan111
Copy link
Contributor Author

@pondrejk triggered prt for rhel9, let's see how it goes

@jameerpathan111
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/installer/test_installer.py -k test_content_guarded_distributions_option
deploy_rhel_version: 9

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6594
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/installer/test_installer.py -k test_content_guarded_distributions_option --external-logging
Test Result : ========== 1 passed, 24 deselected, 41 warnings in 2726.44s (0:45:26) ==========

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 91
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/installer/test_installer.py -k test_content_guarded_distributions_option --external-logging
Test Result : ========== 1 passed, 24 deselected, 41 warnings in 2637.74s (0:43:57) ==========

@pondrejk pondrejk merged commit bc7f9b0 into SatelliteQE:master Apr 18, 2024
11 checks passed
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
@jameerpathan111 @jyejare
Fix test_content_guarded_distributions_option (SatelliteQE#14817)
4795828
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pondrejk pondrejk pondrejk approved these changes

Assignees

@jameerpathan111 jameerpathan111

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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jameerpathan111 @Satellite-QE @pondrejk