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.14.z] Fix katello-agent #12518

Merged
merged 1 commit into from
Sep 8, 2023
Merged

[6.14.z] Fix katello-agent #12518

merged 1 commit into from
Sep 8, 2023

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #12395

In this PR I propose two changes:

  1. Since 6.14 the custom repos are disabled by default, we need to have them overridden to enabled.
  2. Katello-agent fails in old (RHEL6 and 7) containers (IIRC goferd failed to start since it was missing some system service), but it pass on a regular VMs. In order to get this component tested for 6.14 RC properly I propose to set it run on the standard VMs. The price is 1.5 - 2 h of extra run time, which I find acceptable since the katello-agent gets removed from 6.15 anyway and these tests would be deleted in 18 months.

PRT results:
no_containers:

  • 19 passed, 1 broker provider err (PRT#3689, 6:28:38)
  • all 20 passed (PRT#3790, 6:08:16)

with containers:

  • 10 (RHEL6/7) failed, 10 (RHEL8/9) passed (PRT#3700, ~4:33:00)

@vsedmik @web-flow
Fix katello-agent (#12395)
b482968 
Let katello-agent run on a standard VM

(cherry picked from commit f85639f)
@Satellite-QE Satellite-QE added 6.14.z Introduced in or relating directly to Satellite 6.14 Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels Sep 7, 2023
@Satellite-QE
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/destructive/test_katello_agent.py

@Satellite-QE Satellite-QE added the AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing label Sep 7, 2023
@vsedmik
Copy link
Contributor

vsedmik commented Sep 7, 2023

trigger: test-robottelo
pytest: tests/foreman/destructive/test_katello_agent.py

@vsedmik
Copy link
Contributor

vsedmik commented Sep 8, 2023

PRT:
04:29:12 tests/foreman/destructive/test_katello_agent.py .....E.............. [100%]

The single E is broker.exceptions.ProviderError

@Gauravtalreja1 Gauravtalreja1 merged commit 6f5b342 into 6.14.z Sep 8, 2023
@Gauravtalreja1 Gauravtalreja1 deleted the cherry-pick-6.14.z-f85639f1ca6fff7bf6dbc9f1850cbd27cf5885a0 branch September 8, 2023 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

Assignees

@vsedmik vsedmik

@Satellite-QE Satellite-QE

Labels
6.14.z Introduced in or relating directly to Satellite 6.14 Auto_Cherry_Picked Automatically cherrypicked PR using GHA AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing No-CherryPick PR doesnt need CherryPick to previous branches
Projects
None yet
Milestone
No milestone
Development

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