Project

General

Profile

Bug #14871

hammer-cli-katello: 'capsule content synchronize' generates error with 'name' input

Added by Brad Buckingham almost 3 years ago. Updated 7 months ago.

Status:
Closed
Priority:
Normal
Category:
Hammer
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

If the user attempts to sync a capsule using the CLI, they will receive an error, if using the capsule 'name' vs 'id'.

E.g.

hammer> capsule content synchronize --name katello.example.com
Could not synchronize capsule content:
Error: undefined method `[]' for nil:NilClass

This should look more like one of the following:

hammer> capsule content synchronize --name katello.example.com
Could not synchronize capsule content:
This request may only be performed on a Capsule that has the Pulp Node feature.

hammer> capsule content synchronize --organization-id 1 --environment dev --name capsule.example.com
[...................................................................................................] [100%]

Associated revisions

Revision a167e20e (diff)
Added by Brad Buckingham almost 3 years ago

fixes #14871 - capsule content synchronize - fix for --name

Before:

hammer> capsule content synchronize --name katello.example.com
Could not synchronize capsule content:
Error: undefined method `[]' for nil:NilClass

After:

hammer> capsule content synchronize --name katello.example.com
Could not synchronize capsule content:
This request may only be performed on a Capsule that has the Pulp Node feature.

Revision c4521099
Added by Brad Buckingham almost 3 years ago

Merge pull request #389 from bbuckingham/issue-14871

fixes #14871 - capsule content synchronize - fix error on using name

History

#1 Updated by The Foreman Bot almost 3 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/hammer-cli-katello/pull/389 added

#2 Updated by Eric Helms almost 3 years ago

  • Legacy Backlogs Release (now unused) set to 143

#3 Updated by Brad Buckingham almost 3 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF