Project

General

Profile

Bug #13727

Error not raised while setting default value when 'Override' set to False for parameter.

Added by Jitendra Yejare almost 5 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Parameters
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1308860
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:


Related issues

Related to Hammer CLI - Bug #13832: Both Matcher Value and puppet-default value can be set at a time in sc-paramClosed2016-02-22
Related to Foreman - Bug #15308: editing puppet class parameter with arrays and hashes fails when not overridingClosed2016-06-06

Associated revisions

Revision 2a08e8c1 (diff)
Added by Ori Rabin almost 5 years ago

Fixes #13727 - Error raised when setting default_value when override is set to false

History

#1 Updated by Dominic Cleal almost 5 years ago

  • Subject changed from CLI Error not raised while setting default value when 'Override' set to False for parameter. to Error not raised while setting default value when 'Override' set to False for parameter.
  • Category set to Parameters

Applies to the API.

#2 Updated by Ori Rabin almost 5 years ago

  • Status changed from New to Assigned
  • Assignee set to Ori Rabin

#3 Updated by The Foreman Bot almost 5 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/3204 added

#4 Updated by Dominic Cleal almost 5 years ago

  • Related to Bug #13832: Both Matcher Value and puppet-default value can be set at a time in sc-param added

#5 Updated by Ori Rabin almost 5 years ago

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

#6 Updated by Dominic Cleal almost 5 years ago

  • Legacy Backlogs Release (now unused) set to 136

#7 Updated by Dominic Cleal over 4 years ago

  • Related to Bug #15308: editing puppet class parameter with arrays and hashes fails when not overriding added

Also available in: Atom PDF