Project

General

Profile

Bug #18499

hammer throws error "Uuid can't be blank" while adding groups to a content view filter of content type "Package Group"

Added by Nagoor Shaik over 2 years ago. Updated 11 months ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

hammer throws error "Uuid can't be blank" while adding groups to a content view filter of content type "Package Group"

Version-Release number of selected component (if applicable):
Red Hat Satellite 6.2.7

How reproducible:
100 %

Steps to Reproduce:

  1. export ORG="RedHat"
  2. export CV="SOE-RHEL7"
  1. hammer content-view create --name $CV --description "RHEL7 Content View for SOE" --organization $ORG
  1. hammer content-view add-repository --organization $ORG --name $CV --repository "Red Hat Enterprise Linux 7 Server RPMs x86_64 7Server" --product "Red Hat Enterprise Linux Server"
  1. hammer content-view filter create --organization $ORG --content-view $CV --name "SOE Package Groups" --description "Package Groups needed for SOE" --inclusion true --type package_group
  1. hammer content-view filter rule create --organization $ORG --content-view $CV --content-view-filter "SOE Package Groups" --name Core
    Could not create the filter rule:
    Validation failed: Uuid can't be blank

Note: The base group here in RHEL7 is called Core

Here are the logs captured when adding the package group from the webUI

  1. grep Core /var/log/foreman/production.log
    2015-10-06 16:22:29 [I] Parameters: {"name"=>"Core", "api_version"=>"v2", "content_view_filter_id"=>"1", "content_view_filter_rule"=>{"name"=>"Core"}}
    2015-10-06 16:24:27 [I] Parameters: {"uuid"=>"f402611b-0125-4a94-ae14-363edc8c638d", "name"=>"Core", "api_version"=>"v2", "content_view_filter_id"=>"1", "content_view_filter_rule"=>{"uuid"=>"f402611b-0125-4a94-ae14-363edc8c638d", "name"=>"Core"}}
    2015-10-06 16:26:03 [I] Parameters: {"uuid"=>"f402611b-0125-4a94-ae14-363edc8c638d", "name"=>"Core", "api_version"=>"v2", "content_view_filter_id"=>"1", "content_view_filter_rule"=>{"uuid"=>"f402611b-0125-4a94-ae14-363edc8c638d", "name"=>"Core"}}
  1. hammer content-view filter rule create --organization $ORG --content-view $CV --content-view-filter "SOE Package Groups" --help
    Usage:
    hammer content-view filter rule create [OPTIONS]

Options:
--content-view CONTENT_VIEW_NAME Content view name
--content-view-filter CONTENT_VIEW_FILTER_NAME Name to search by
--content-view-filter-id CONTENT_VIEW_FILTER_ID filter identifier
--content-view-id CONTENT_VIEW_ID content view numeric identifier
--end-date END_DATE erratum: end date (YYYY-MM-DD)
--errata-id ERRATA_ID erratum: id
--errata-ids ERRATA_IDS erratum: IDs or a select all object
Comma separated list of values.
--max-version MAX_VERSION package: maximum version
--min-version MIN_VERSION package: minimum version
--name NAME package or package group: name
--organization ORGANIZATION_NAME Organization name to search by
--organization-id ORGANIZATION_ID organization ID
--organization-label ORGANIZATION_LABEL Organization label to search by
--start-date START_DATE erratum: start date (YYYY-MM-DD)
--types TYPES erratum: types (enhancement, bugfix, security)
Comma separated list of values.
--version VERSION package: version
-h, --help print help

As hammer command is expecting a parameter uuid which is not listed in the hammer command help, causing it to fail.

Actual results:
hammer fails as its expecting a UUID to be passed along with the name for package group

Expected results:
Either UUID should enabled in the hammer

Additional info:

Associated revisions

Revision 0abf4af2 (diff)
Added by Nagoor Shaik over 2 years ago

Fixes #18499 - Add UUID in ContentView Filter Rule API

Revision e605f0cb (diff)
Added by Nagoor Shaik about 2 years ago

Fixes #18499 - Hammer CV filter rule accepts PackageGroup UUID

Revision fe244084
Added by Brad Buckingham about 2 years ago

Merge pull request #480 from snagoor/18499

Fixes #18499 - Hammer CV filter rule accepts PackageGroup UUID

History

#1 Updated by Dominic Cleal over 2 years ago

  • Project changed from Hammer CLI to Katello

#2 Updated by Nagoor Shaik over 2 years ago

Hi,

I am working on a PR and will submit it by end of tomorrow.

#3 Updated by The Foreman Bot over 2 years ago

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

#4 Updated by The Foreman Bot over 2 years ago

  • Pull request https://github.com/Katello/hammer-cli-katello/pull/480 added

#5 Updated by Eric Helms over 2 years ago

  • Legacy Backlogs Release (now unused) set to 219

#6 Updated by Eric Helms over 2 years ago

  • Legacy Backlogs Release (now unused) changed from 219 to 211

#7 Updated by Nagoor Shaik over 2 years ago

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

#8 Updated by Nagoor Shaik over 2 years ago

  • Bugzilla link changed from 1391704 to 1270659

Also available in: Atom PDF