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

[6.14.z] Fix lce id option in host registration #12852

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #12849

Registering host using the LCE id fails with Failed to generate registration command: Error: Unrecognised option '--lifecycle_environment_id'. See: 'hammer host-registration generate-command --help' because the dashes are used for hammer CLI options.

# hammer host-registration generate-command --help
Usage:
    hammer host-registration generate-command [OPTIONS]

Options:
...
 --lifecycle-environment[-id] VALUE/NUMBER Lifecycle environment for the host.

@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 Oct 9, 2023
@Satellite-QE Satellite-QE added the AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing label Oct 9, 2023
@Gauravtalreja1 Gauravtalreja1 enabled auto-merge (squash) October 9, 2023 22:01
@Gauravtalreja1 Gauravtalreja1 merged commit 379b096 into 6.14.z Oct 9, 2023
@Gauravtalreja1 Gauravtalreja1 deleted the cherry-pick-6.14.z-2f3484e18a10873eb8bed975fd78bdc059fa4ca7 branch October 9, 2023 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Development

Successfully merging this pull request may close these issues.

3 participants