Project

General

Profile

Bug #4639

Syntax of arrays in parameters changed when edit is changed.

Added by Martin Hovmöller about 5 years ago. Updated 11 months ago.

Status:
Closed
Priority:
Normal
Category:
Smart Variables
Target version:
Difficulty:
Triaged:
Bugzilla link:
Pull request:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

As I understand if, the syntax for arrays should be JSON/YAML stype, so if I wish to have an array of hashes the value for a parameter should be something like "[{ "hostname": "host", "password": "secret" }].
Adding this works fine, but when the change is saved, foreman changes the syntax, so the next time I edit the host the value is instead "[{"hostname"=>"host", "password"=>"secret"}]".

Trying to edit anything on this host then gives me the error "(<unknown>): did not find expected ',' or '}' while parsing a flow mapping at line 1 column 2" since foreman doesn't get the syntax it itself created, which means that this value has to be manually changed every time anything is changed on the host in question.

Associated revisions

Revision 801a90d1 (diff)
Added by Stephen Benjamin about 5 years ago

fixes #4639 - array in lookup_value incorrectly displayed in form

Revision 0accaa2d (diff)
Added by Stephen Benjamin about 5 years ago

fixes #4639 - array in lookup_value incorrectly displayed in form

(cherry picked from commit 801a90d196d363aadae7bf2be9347ff6e46e0471)

History

#1 Updated by Dominic Cleal about 5 years ago

  • Category set to Smart Variables

#2 Updated by Stephen Benjamin about 5 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Stephen Benjamin
  • Target version set to 1.9.0

#3 Updated by Anonymous about 5 years ago

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

#4 Updated by Dominic Cleal about 5 years ago

  • Legacy Backlogs Release (now unused) set to 7

#5 Updated by Dominic Cleal about 5 years ago

  • Legacy Backlogs Release (now unused) changed from 7 to 9

Apologies, I missed this from 1.4.2. Retargeting for 1.4.3 if we do it, else 1.5.0.

#6 Updated by Dominic Cleal about 5 years ago

  • Legacy Backlogs Release (now unused) changed from 9 to 4

#7 Updated by Dominic Cleal about 5 years ago

  • Legacy Backlogs Release (now unused) changed from 4 to 17

Also available in: Atom PDF