Project

General

Profile

Bug #8892

content-override does not produce an error when --label is missing

Added by Christine Fouant about 8 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Category:
Hammer
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1173723
Description of problem:
when running activation-key content-override, you must include the label of the product being overridden. However, a user can currently run content-override without passing --label. Of course, no change is made.

Version-Release number of selected component (if applicable):
Nightly

How reproducible:
always

Steps to Reproduce:
1. create an activation key and add a subscription
2. run hammer -u admin -p ** activation-key content-override --id=<activation key id> --organization-id=<organization id>
3. view results

Actual results:
Command runs and returns "Updated content override"

Expected results:
An error should be returned informing the user that a product label is needed.

Additional info:

Associated revisions

Revision bc668761 (diff)
Added by Christine Fouant about 8 years ago

fixes #8892, #9340 - Validates content label and override value, BZ1173723, BZ1187750

Revision 272ccaec
Added by Christine Fouant about 8 years ago

Merge pull request #5002 from cfouant/refactor-content-override

fixes #8892, #9340 - Validates content label and override value, BZ1173723, BZ1187750

History

#1 Updated by Eric Helms about 8 years ago

  • Legacy Backlogs Release (now unused) set to 23
  • Triaged changed from No to Yes

#2 Updated by The Foreman Bot about 8 years ago

  • Status changed from New to Ready For Testing
  • Target version set to 66
  • Pull request https://github.com/Katello/katello/pull/5002 added
  • Pull request deleted ()

#3 Updated by Eric Helms about 8 years ago

  • Legacy Backlogs Release (now unused) changed from 23 to 31

#4 Updated by Eric Helms about 8 years ago

  • Target version changed from 66 to 67

#5 Updated by Eric Helms about 8 years ago

  • Legacy Backlogs Release (now unused) changed from 31 to 23

#6 Updated by Christine Fouant about 8 years ago

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

Also available in: Atom PDF