Project

General

Profile

Bug #6055

host-collection add-content-host and remove-content-host should take uuids

Added by dustin tsang over 6 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Hammer
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:

Description

bundle exec hammer -v host-collection add-content-host --id 1 --system-id 4923fe7a-f140-4a15-9a98-18a15cf53e12
Could not add content host:
Couldn't find all Katello::Systems with IDs (1, 2, 4923) (found 2 results, but was looking for 3)

Associated revisions

Revision 56191f50 (diff)
Added by dustin tsang over 6 years ago

Fixes #6055 - HC add/remove-content-host take uuid

the host-collection command's --system-id option was accepting database
ids instead of uuids. This uses the
add_host_collections/remove_host_collections action instead of the
AssociatingCommands (which uses the update action). This is a simpler
approach than trying to alter/override the behavior of an
AssociatingCommand to accept uuids.

Revision 471acb33
Added by dustin tsang over 6 years ago

Merge pull request #180 from dustint-rh/hc_rm_associating_commands

Fixes #6055 - HC add/remove-content-host take uuid

History

#1 Updated by dustin tsang over 6 years ago

  • Subject changed from hostcolleciotns add-content-host and remove-content-host should take uuids to host-collection add-content-host and remove-content-host should take uuids

#4 Updated by Eric Helms over 6 years ago

  • Triaged changed from No to Yes

#5 Updated by dustin tsang over 6 years ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

Applied in changeset hammer-cli-katello|commit:56191f50255d0eb4edb8f790c03357a05a670de7.

#6 Updated by dustin tsang over 6 years ago

  • Target version set to 45

#7 Updated by Eric Helms about 6 years ago

  • Legacy Backlogs Release (now unused) set to 13

Also available in: Atom PDF