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

preparation for new yggdrasil #16789

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

preparation for new yggdrasil #16789

wants to merge 1 commit into from

Conversation

pondrejk
Copy link
Contributor

Problem Statement

SAT-27476

template changes and a different service name on rhel 10 and 9.6+

Solution

this pr

Related Issues

awaits theforeman/foreman#10340

@pondrejk pondrejk added the No-CherryPick PR doesnt need CherryPick to previous branches label Oct 25, 2024
@pondrejk pondrejk self-assigned this Oct 25, 2024
@pondrejk pondrejk requested review from a team as code owners October 25, 2024 13:44
@pondrejk pondrejk marked this pull request as draft October 25, 2024 13:44
@pondrejk pondrejk marked this pull request as ready for review November 20, 2024 09:14
@pondrejk pondrejk added the Stream Introduced in or relating directly to Satellite Stream/Master label Nov 20, 2024
@pondrejk
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_remoteexecution.py -k test_positive_run_pull_job_on_offline_host[rhel9-ipv4]

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9406
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_remoteexecution.py -k test_positive_run_pull_job_on_offline_host[rhel9-ipv4] --external-logging
Test Result : ========== 1 failed, 63 deselected, 80 warnings in 1381.58s (0:23:01) ==========

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Nov 20, 2024
lhellebr
lhellebr reviewed Nov 26, 2024
View reviewed changes
tests/foreman/cli/test_remoteexecution.py
@@ -1020,13 +1020,14 @@ def test_positive_run_job_on_host_converted_to_pull_provider(
)
assert result.status == 0, f'Failed to register host: {result.stderr}'

service_name = (
Copy link
Contributor

Choose a reason for hiding this comment

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

Can't this be in one place? If not, can at least the versions be constants?

@pondrejk
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_remoteexecution.py -k test_positive_run_pull_job_on_offline_host[rhel9-ipv4]

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9577
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_remoteexecution.py -k test_positive_run_pull_job_on_offline_host[rhel9-ipv4] --external-logging
Test Result : ========== 1 failed, 69 deselected, 86 warnings in 1631.63s (0:27:11) ==========

@pondrejk
Copy link
Contributor Author 

trigger: test-robottelo
pytest: tests/foreman/cli/test_remoteexecution.py -k test_positive_run_job_on_host_registered_to_pull_provider
theforeman:
    smart_proxy_remote_execution_ssh:121

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lhellebr lhellebr lhellebr left review comments

Assignees

@pondrejk pondrejk

Labels
No-CherryPick PR doesnt need CherryPick to previous branches PRT-Failed Indicates that latest PRT run is failed 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
@pondrejk @Satellite-QE @lhellebr