Skip to content

[6.16.z] Add possibility to specify download-utility#17902

Closed
Satellite-QE wants to merge 1 commit into6.16.zfrom
cherry-pick-6.16.z-880545127a968784511a42571fd3d423f28af635
Closed

[6.16.z] Add possibility to specify download-utility#17902
Satellite-QE wants to merge 1 commit into6.16.zfrom
cherry-pick-6.16.z-880545127a968784511a42571fd3d423f28af635

Conversation

@Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #16912

Problem Statement

newly added download utility option missing theforeman/foreman#9808

Solution

Add field

Related Issues

dependency SatelliteQE/nailgun#1257

* Add possibility to specify download-utility

in host registration

see also theforeman/foreman#9808

* Add test for registration property download utility

(cherry picked from commit 8805451)
@Satellite-QE Satellite-QE requested a review from a team as a code owner March 17, 2025 12:10
@Satellite-QE Satellite-QE added Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels Mar 17, 2025
@Satellite-QE Satellite-QE requested a review from a team as a code owner March 17, 2025 12:10
@ogajduse
Copy link
Member

trigger: test-robottelo
pytest: tests/foreman/cli/test_registration.py::test_positive_register_download_utility

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 10567
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_registration.py::test_positive_register_download_utility --external-logging
Test Result : ================= 31 warnings, 2 errors in 1183.46s (0:19:43) ==================

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Mar 17, 2025
@ogajduse
Copy link
Member

trigger: test-robottelo
pytest: tests/foreman/cli/test_registration.py::test_positive_register_download_utility

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 10601
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_registration.py::test_positive_register_download_utility --external-logging
Test Result : ================= 2 failed, 162 warnings in 2040.09s (0:34:00) =================

@ogajduse
Copy link
Member

robottelo.exceptions.CLIReturnCodeError: CLIReturnCodeError(status=64, stderr="Failed to generate registration command:\n  Error: Unrecognised option '--download-utility'.\n  \n  See: 'hammer host-registration generate-command --help'.\n", msg='Command "host-registration generate-command" finished with status 64\nstderr contains:\nFailed to generate registration command:\n  Error: Unrecognised option \'--download-utility\'.\n  \n  See: \'hammer host-registration generate-command --help\'.\n'

Packages in 6.16 are:

  • rubygem-hammer_cli_foreman-3.12.0-1.el8sat.noarch.rpm
  • foreman-3.12.0.7-1.el8sat.noarch.rpm

It seems that these do not allow using the --download-utility option.

@ogajduse
Copy link
Member

@dosas Are you fine with closing this PR?

@dosas
Copy link
Collaborator

dosas commented Mar 19, 2025

@ogajduse Sure, thanks for checking.

@dosas dosas closed this Mar 19, 2025
@Gauravtalreja1 Gauravtalreja1 deleted the cherry-pick-6.16.z-880545127a968784511a42571fd3d423f28af635 branch March 19, 2025 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.16.z Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches PRT-Failed Indicates that latest PRT run is failed for the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants