Bug 1308860 - CLI Error not raised while setting default value when 'Override' set to False for parameter.
Summary: CLI Error not raised while setting default value when 'Override' set to False...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Provisioning
Version: Nightly
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: Unspecified
Assignee: orabin
QA Contact: Katello QA List
URL: http://projects.theforeman.org/issues...
Whiteboard:
Depends On:
Blocks: 1278718
TreeView+ depends on / blocked
 
Reported: 2016-02-16 09:58 UTC by Jitendra Yejare
Modified: 2019-09-26 13:57 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-07-27 11:43:47 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)
Screenshot Attached (134.69 KB, image/png)
2016-02-16 09:58 UTC, Jitendra Yejare 		no flags Details
View All


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 13727 0 None None None 2016-04-22 16:40:07 UTC
Red Hat Bugzilla 1343378 0 unspecified CLOSED editing puppet class parameter with arrays and hashes fails when not overriding 2021-02-22 00:41:40 UTC

Internal Links: 1343378

Description Jitendra Yejare 2016-02-16 09:58:59 UTC 
Created attachment 1127547 [details]
Screenshot Attached

Description of problem:
User is able to set the default value for parameter even if the 'override' flag is not set for the parameter, which is not expected.
Note: This issue is only observed in CLI and not in UI.

Version-Release number of selected component (if applicable):
foreman-1.11.0-0.develop.201601251557git59dc77b.el6.noarch

How reproducible:
Always

Steps to Reproduce:
1. Attempt to set default value for parameter which override flag is set to False.

Actual results:
Default Value successfully updated and no error raised.

Expected results:
1. Error/Info should be raised to set the 'override' flag to True first.
2. Default Value should not be updated.

Additional info:
Comment 3 Jitendra Yejare 2016-02-16 11:16:12 UTC 
Same scenario is reproducible from API as well in same build.
Comment 4 Bryan Kearney 2016-02-16 13:08:21 UTC 
Upstream bug component is Provisioning
Comment 5 Bryan Kearney 2016-02-17 09:08:19 UTC 
Upstream bug assigned to orabin
Comment 6 Bryan Kearney 2016-02-17 09:08:20 UTC 
Upstream bug assigned to orabin
Comment 7 Bryan Kearney 2016-03-28 16:08:41 UTC 
Moving to POST since upstream bug http://projects.theforeman.org/issues/13727 has been closed
-------------
Ori Rabin
Applied in changeset commit:2a08e8c1deac40c26932dc80ca53e89bfe9e493b.
Comment 8 Mike McCune 2016-03-28 23:15:27 UTC 
This bug was accidentally moved from POST to MODIFIED via an error in automation, please see mmccune with any questions
Comment 9 Jitendra Yejare 2016-05-19 13:01:19 UTC 
Verified !
@Sat 6.2 Snap 11

Proof:

root@qeblade36 ~]# hammer sc-param update --id 322 --default-value [78]
Could not update the parameter:
  Validation failed: Override must be true to edit the parameter

So moving this to verified.
Comment 10 Bryan Kearney 2016-07-27 11:43:47 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/RHBA-2016:1501
Note You need to log in before you can comment on or make changes to this bug.