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.

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

Fix foreman service test #14583

Merged
merged 1 commit into from
Apr 3, 2024

Conversation

Griffin-Sullivan
Copy link
Contributor

Problem Statement

After turning off the foreman service, the check-tftp-storage requires the user to input the admin password since the foreman service isn't running. This causes the command to time out while waiting for input.

Solution

Let's whitelist the tftp storage check since it's not necessary for the test.

Related Issues

@Griffin-Sullivan
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/maintain/test_service.py::test_positive_foreman_service

@Griffin-Sullivan Griffin-Sullivan added CherryPick PR needs CherryPick to previous branches 6.13.z Introduced in or relating directly to Satellite 6.13 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 labels Apr 1, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6261
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/maintain/test_service.py::test_positive_foreman_service --external-logging
Test Result : ================== 1 passed, 4 warnings in 1534.49s (0:25:34) ==================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Apr 1, 2024
@shubhamsg199 shubhamsg199 merged commit 932b588 into SatelliteQE:master Apr 3, 2024
12 of 13 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 3, 2024
(cherry picked from commit 932b588)
github-actions bot pushed a commit that referenced this pull request Apr 3, 2024
(cherry picked from commit 932b588)
github-actions bot pushed a commit that referenced this pull request Apr 3, 2024
(cherry picked from commit 932b588)
shweta83 pushed a commit to shweta83/robottelo that referenced this pull request Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.13.z Introduced in or relating directly to Satellite 6.13 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants