Bug 1310645

Summary:	Both Matcher Value and puppet-default value can be set at a time in sc-param
Product:	Red Hat Satellite	Reporter:	Jitendra Yejare <jyejare>
Component:	Hammer	Assignee:	Adam Ruzicka <aruzicka>
Status:	CLOSED ERRATA	QA Contact:	Peter Ondrejka <pondrejk>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	Nightly	CC:	aruzicka, bbuckingham, bkearney, ehelms
Target Milestone:	Unspecified	Keywords:	Triaged
Target Release:	Unused
Hardware:	Unspecified
OS:	Unspecified
URL:	http://projects.theforeman.org/issues/13832
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2018-02-21 16:49:54 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Bug Depends On:
Bug Blocks:	1278718

Description Jitendra Yejare 2016-02-22 12:23:42 UTC

Description of problem:
Matcher value and puppet default can be set as matcher value for smart class parameter and at-a time. Actually If 'Use-puppet-default' option is checked for matcher, matcher value cannot be set from UI. And same is expected in CLI.

Version-Release number of selected component (if applicable):
foreman-1.11.0-0.develop.201602171602gitd5b5059.el7.noarch

How reproducible:
Always

Steps to Reproduce:
1. Attempt to create matcher with override value and puppet default value at a time from CLI.
# hammer -u admin -p changeme sc-param add-override-value --match <match> --value <value> --use-puppet-default 1

Actual results:
Success! Both value and puppet default is set.

Expected results:
Either one is value is set -or- exception raised for wrong attempt.

Additional info:

Comment 3 Bryan Kearney 2016-02-22 13:09:08 UTC

Upstream bug component is Provisioning

Comment 4 Bryan Kearney 2016-06-07 12:13:12 UTC

Upstream bug component is Hammer

Comment 5 Bryan Kearney 2016-06-10 08:18:33 UTC

Moving to POST since upstream bug http://projects.theforeman.org/issues/13832 has been closed

Comment 6 Peter Ondrejka 2016-12-13 10:21:35 UTC

Verified in Sat 6.3 snap 8

~]# hammer sc-param add-override-value --match os=* --value false --smart-class-parameter-id 202 --use-puppet-default 1
Could not create the override value:
  Error: Cannot use --value when --use-puppet-default is true
  
  See: 'hammer sc-param add-override-value --help'

Comment 7 Satellite Program 2018-02-21 16:49:54 UTC

Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA.

For information on the advisory, and where to find the updated files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2018:0336