Project

General

Profile

Actions

Bug #12525

closed

exit code when enabling repository-set changed, message is not that helpful as well

Added by Brad Buckingham over 8 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Category:
API
Target version:
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1249655
Description of problem:
Exit code when enabling repository-set changed, message is not that helpful as well

Version-Release number of selected component (if applicable):
Satellite-6.1.0-RHEL-6-20150721.7-Satellite-x86_64
vs.
Satellite-6.1.0-RHEL-6-20150731.1-Satellite-x86_64

How reproducible:
always

Steps to Reproduce:
1. Attempt to enable repository-set without providing all the substitutions: # hammer --username <user> --password <pass> repository-set enable \
--product-id 6 --id 1952

Actual results:
  1. hammer --username admin --password changeme repository-set enable --product-id 6 --id 1952
    Could not enable repository: {} are not valid substitutions for /content/dist/rhel/server/6/$releasever/$basearch/kickstart
  2. echo $?
    65

Expected results:
Error message from hammer from the older compose seems to be more helpful to me:

  1. hammer --username admin --password changeme repository-set enable --product-id 6 --id 1952
    Could not enable repository:
    Missing arguments for 'basearch', 'releasever'
  2. echo $?
    64

Additional info:
Please feel free to close as notabug. I just want to be 100% sure this is intentional.

Actions #1

Updated by The Foreman Bot over 8 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Brad Buckingham
  • Pull request https://github.com/Katello/katello/pull/5606 added
Actions #2

Updated by Brad Buckingham over 8 years ago

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

Updated by Eric Helms over 8 years ago

  • Category set to API
  • translation missing: en.field_release set to 86
  • Triaged changed from No to Yes
Actions

Also available in: Atom PDF