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

[6.15.z] improve installer tests compatibility with pit (#15308) #16424

Merged
merged 1 commit into from
Oct 9, 2024
Merged

[6.15.z] improve installer tests compatibility with pit (#15308) #16424

merged 1 commit into from
Oct 9, 2024

Conversation

rmynar
Copy link
Contributor

@rmynar rmynar commented Sep 19, 2024

cherrypick of #15308
closes #16200

@rmynar rmynar added the No-CherryPick PR doesnt need CherryPick to previous branches label Sep 19, 2024
@rmynar rmynar self-assigned this Sep 19, 2024
@rmynar rmynar force-pushed the cherrypick-16200 branch 3 times, most recently from f472455 to ac5c8aa Compare September 20, 2024 14:03
@rmynar rmynar changed the title improve installer tests compatibility with pit (#15308) [6.15.z] improve installer tests compatibility with pit (#15308) Sep 20, 2024
@rmynar rmynar added the Failed_AutoCherryPick Issue for failed/conflicted auto cherry pick of PRs label Sep 20, 2024
@rmynar rmynar marked this pull request as ready for review September 23, 2024 11:37
@rmynar rmynar requested review from a team as code owners September 23, 2024 11:37
@rmynar
Copy link
Contributor Author

rmynar commented Sep 23, 2024

trigger: test-robottelo
pytest: tests/foreman/installer/test_installer.py::test_satellite_installation tests/foreman/installer/test_installer.py::test_capsule_installation

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8768
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/installer/test_installer.py::test_satellite_installation tests/foreman/installer/test_installer.py::test_capsule_installation --external-logging
Test Result : ============ 1 failed, 1 passed, 260 warnings in 3108.06s (0:51:48) ============

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Sep 23, 2024
@Gauravtalreja1
Copy link
Collaborator

trigger: test-robottelo
pytest: tests/foreman/installer/test_installer.py::test_satellite_installation tests/foreman/installer/test_installer.py::test_capsule_installation

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8785
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/installer/test_installer.py::test_satellite_installation tests/foreman/installer/test_installer.py::test_capsule_installation --external-logging
Test Result : ============ 1 failed, 1 passed, 313 warnings in 2940.59s (0:49:00) ============

@lhellebr
Copy link
Contributor

lhellebr commented Oct 4, 2024

PRT both times:

requests.exceptions.ConnectionError: HTTPSConnectionPool(host='<FQDN>', port=443): Max retries exceeded with url: /katello/api/organizations (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f2fce62aff0>: Failed to establish a new connection: [Errno 111] Connection refused'))

@lhellebr
Copy link
Contributor

lhellebr commented Oct 4, 2024

trigger: test-robottelo
pytest: tests/foreman/installer/test_installer.py::test_satellite_installation tests/foreman/installer/test_installer.py::test_capsule_installation

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8870
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/installer/test_installer.py::test_satellite_installation tests/foreman/installer/test_installer.py::test_capsule_installation --external-logging
Test Result : ============ 1 failed, 1 passed, 329 warnings in 3193.02s (0:53:13) ============

@rmynar
Copy link
Contributor Author

rmynar commented Oct 8, 2024

Previous PRTs have failed on a known bug (SAT-28665). It is not yet fixed, but there's already a known workaround, so I updated the test to apply the workaround if bug is still open.

@rmynar
Copy link
Contributor Author

rmynar commented Oct 8, 2024

trigger: test-robottelo
pytest: tests/foreman/installer/test_installer.py::test_satellite_installation tests/foreman/installer/test_installer.py::test_capsule_installation

@rmynar
Copy link
Contributor Author

rmynar commented Oct 8, 2024

trigger: test-robottelo
pytest: tests/foreman/installer/test_installer.py::test_satellite_installation tests/foreman/installer/test_installer.py::test_capsule_installation
deploy_rhel_version: 8

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8903
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/installer/test_installer.py::test_satellite_installation tests/foreman/installer/test_installer.py::test_capsule_installation --external-logging
Test Result : ================= 2 passed, 463 warnings in 4644.96s (1:17:24) =================

@Satellite-QE Satellite-QE added PRT-Passed Indicates that latest PRT run is passed for the PR and removed PRT-Failed Indicates that latest PRT run is failed for the PR labels Oct 8, 2024
@rmynar rmynar requested review from lhellebr and ogajduse October 8, 2024 14:03
@ogajduse ogajduse merged commit fb8cb7c into SatelliteQE:6.15.z Oct 9, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ogajduse ogajduse ogajduse approved these changes

@lhellebr lhellebr Awaiting requested review from lhellebr

Assignees

@rmynar rmynar

Labels
Failed_AutoCherryPick Issue for failed/conflicted auto cherry pick of PRs 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.
5 participants
@rmynar @Satellite-QE @Gauravtalreja1 @lhellebr @ogajduse