Project

General

Profile

Bug #22498

Improve help and error messages when adding CVs to a CCV

Added by Peter Ondrejka over 4 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Low
Assignee:
-
Category:
Content Views
Target version:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

This came from discussion in https://bugzilla.redhat.com/show_bug.cgi?id=1309944

trying to add a different version of a CV that is already in the CCV, fails, which is I suppose correct but the message could be more informative:

~]# hammer content-view add-version --id 3 --content-view test2 --content-view-version-id 7
Could not add version:
Validation failed: Content view components is invalid, Content view components is invalid

Also the help descriptions could indicate which options relate to ccv and which to cv being added, currently we have:

~]# hammer content-view add-version -h
Usage:
hammer content-view add-version [OPTIONS]

Options:
--content-view CONTENT_VIEW_NAME Content view name to search by
--content-view-id CONTENT_VIEW_ID content view numeric identifier
--content-view-version CONTENT_VIEW_VERSION_VERSION Content view version number
--content-view-version-id CONTENT_VIEW_VERSION_ID Content view version identifier
--id ID content view numeric identifier
--name NAME Content view name to search by

Associated revisions

Revision 0ff1b88f (diff)
Added by Michael Johnson almost 4 years ago

Fixes #22498 - Improve error messages for content views

Revision e1d924f0 (diff)
Added by Michael Johnson almost 4 years ago

Refs #22498 - Improve error messages for content views

History

#1 Updated by Andrew Kofink over 4 years ago

  • Bugzilla link set to 1541393

#2 Updated by The Foreman Bot almost 4 years ago

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

#3 Updated by Michael Johnson almost 4 years ago

  • Status changed from Ready For Testing to Closed

#4 Updated by The Foreman Bot almost 4 years ago

  • Pull request https://github.com/Katello/katello/pull/7708 added

Also available in: Atom PDF