Bug #2758: On class reimport, Foreman runs class parameter validations even when parameter override is disabled - Foreman

Custom queries

3.11.0 release DONE
3.11.0 release TODO
3.11.1 release DONE
3.11.1 release TODO
3.8.1 release DONE
3.8.1 release TODO
3.9.2 release DONE
3.9.2 release TODO
All bugs by votes
All issues closed in the last 3 weeks
Easy and Trivial Issues
Easy Issues (only)
Foreman - new open issues in last 10 days
Good first issues - Foreman core
high prio open bugs
Host registration [open]
Mine
My issues closed in the last 3 weeks
old bugs
Open with target version set (aka blockers)
registration
taken-none
Team Atlas untriaged bugs
templates
Trivial Issues
Untriaged and opened in the past two weeks

Actions

Copy link

Bug #2758

closed

On class reimport, Foreman runs class parameter validations even when parameter override is disabled

Added by Jon McKenzie about 11 years ago. Updated almost 11 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Category:

Puppet integration

Target version:

1.3.0

Difficulty:

Triaged:

Bugzilla link:

Pull request:

Fixed in Releases:

Found in Releases:

Red Hat JIRA:

Description

This is running Foreman 1.2 RC3 (although I believe 1.2 would have the issue as well).

I believe the issue arose when I enabled override on an inherited parameter, changed its value, changed its type from string to something else, disabled the override, and then later tried to reimport after I made a class change.

Related issues 1 (0 open — 1 closed)

Has duplicate Foreman - Bug #3773: puppet class key update type change results in default_value invalid (1.2.2)

Duplicate

11/27/2013

Actions

Issue # Delay: days Cancel

History
Notes
Property changes
Associated revisions

Actions

Copy link

Updated by Jon McKenzie about 11 years ago

Sorry for brevity, will include more details when I get time to sit down and reproduce the issue.

Actions

Copy link

Updated by Dominic Cleal about 11 years ago

Category set to Puppet integration
Target version set to 1.3.0

This results in a message saying "Validation failed: Default value is invalid" when you try a puppet class import.

See also https://groups.google.com/forum/#!topic/foreman-users/TKi3ublJnSA for more discussion.

I'm setting this to 1.3 as we get lots of reports of this on IRC, but I didn't know what caused it until now. Thanks Jon!

Actions

Copy link

Updated by Andy Bohne almost 11 years ago

I've got a fix for this. Should I branch off 1.2-stable or should I branch off develop? Assuming if I branch off 1.2-stable, the fix will make it into a release in the 1.2 series and we won't have to wait until 1.3.

Actions

Copy link

Updated by Dominic Cleal almost 11 years ago

Description updated (diff)

Andy Bohne wrote:

I've got a fix for this. Should I branch off 1.2-stable or should I branch off develop? Assuming if I branch off 1.2-stable, the fix will make it into a release in the 1.2 series and we won't have to wait until 1.3.

Great, please branch from develop and send a PR there. I'll try and get it in 1.2 if we do another release (1.2.1 is being finalised, it won't make that).

Actions

Copy link

Updated by Andy Bohne almost 11 years ago

Dominic Cleal wrote:

Andy Bohne wrote:

I've got a fix for this. Should I branch off 1.2-stable or should I branch off develop? Assuming if I branch off 1.2-stable, the fix will make it into a release in the 1.2 series and we won't have to wait until 1.3.

Great, please branch from develop and send a PR there. I'll try and get it in 1.2 if we do another release (1.2.1 is being finalised, it won't make that).

OK. PR 831 has been submitted.

Actions

Copy link