Project

General

Profile

Bug #33182

Warnings should be improved for hammer host errata apply, when not passing errata_ids

Added by Lucy Fu over 1 year ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Lucy Fu
Category:
Errata Management
Target version:
Katello 4.2.0
Difficulty:
Triaged:
Yes
Bugzilla link:
1979239
Pull request:
https://github.com/Katello/hammer-cli-katello/pull/812, https://github.com/Katello/katello/pull/9516
Fixed in Releases:
Katello 4.3.0
Found in Releases:
Red Hat JIRA:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1979239

Description of problem:

Version-Release number of selected component (if applicable):
Satellite 6.10.0 Snap 7.0

How reproducible:
always

Steps to Reproduce:
1. # hammer host errata apply --host-id <host-id>
This command uses katello agent and will be removed in favor of remote execution in a future release.
The remote execution equivalent is `hammer job-invocation create --feature katello_errata_install`.
Could not apply errata:
no implicit conversion of nil into String
2. # echo $?
70

Actual results:
no implicit conversion of nil into String

Expected results:
A warning should exist to pass errata_ids, like when not passing --errata to command `hammer host-collection erratum install --id 1 --organization-id 1`

Additional info:
On Satellite 6.9.3 Snap 5.0
  1. hammer host errata apply --host-id 3
    Could not apply errata:
    undefined method `[]' for nil:NilClass

Associated revisions

Revision ece0b63a (diff)
Added by Lucy Fu over 1 year ago

Refs #33182 - hide broken options --ids

Revision c35f3100 (diff)
Added by Lucy Fu over 1 year ago

Fixes #33182 - make --errata-ids required for host errata apply

History

#1 Updated by The Foreman Bot over 1 year ago

  • Assignee set to Lucy Fu
  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/9516 added

#2 Updated by The Foreman Bot over 1 year ago

  • Pull request https://github.com/Katello/hammer-cli-katello/pull/812 added

#3 Updated by The Foreman Bot over 1 year ago

  • Fixed in Releases Katello 4.3.0 added

#4 Updated by Lucy Fu over 1 year ago

  • Status changed from Ready For Testing to Closed

#5 Updated by Justin Sherrill over 1 year ago

  • Triaged changed from No to Yes
  • Target version set to Katello 4.2.0
  • Subject changed from Warnings should be improved for hammer host errata apply, when not passing errata_ids to Warnings should be improved for hammer host errata apply, when not passing errata_ids

Also available in: Atom PDF