Re: [Crowbar] NTP cookbook broken in CB20

2013-12-12 Thread Rob_Hirschfeld
I confirmed that you can edit this with raw mode URL that I sent and fix the json back to an array. From: Victor Lowther [mailto:victor.lowt...@gmail.com] Sent: Thursday, December 12, 2013 1:37 PM To: Sébastien Braun Cc: crowbar; Hirschfeld, Rob Subject: Re: [Crowbar] NTP cookbook broken in CB20

Re: [Crowbar] NTP cookbook broken in CB20

2013-12-12 Thread Victor Lowther
t;> I believe it should be an array, not a single string. >> >> >> >> Here’s the raw template: >> >> >> >> { >> >> "crowbar": { >> >> "ntp": { >> >> "external_servers": [ >> >> >> >> ] >> >&

Re: [Crowbar] NTP cookbook broken in CB20

2013-12-12 Thread Sébastien Braun
} > > } > > > > *From:* Sébastien Braun [mailto:sebbr...@gmail.com] > *Sent:* Thursday, December 12, 2013 9:32 AM > *To:* Hirschfeld, Rob > *Cc:* crowbar > > *Subject:* Re: [Crowbar] NTP cookbook broken in CB20 > > > > CB2.0 code dat

Re: [Crowbar] NTP cookbook broken in CB20

2013-12-12 Thread Rob_Hirschfeld
some more information about your set and which version of the code (date) you are using? The update NTP to an external value has had minimal testing and likely needs to be checked. We've also been making recent changes to update how attributes are handled but those have not bee

Re: [Crowbar] NTP cookbook broken in CB20

2013-12-12 Thread Sébastien Braun
ave not been reflected in the UI. > > The 1st step in troubleshooting would be to look at the actual json on the > node role in "raw" mode. > > Rob > > From: crowbar-bounces On Behalf Of Sébastien Braun > Sent: Thursday, December 12, 2013 6:58 AM > To: crowbar >

Re: [Crowbar] NTP cookbook broken in CB20

2013-12-12 Thread Rob_Hirschfeld
d but those have not been reflected in the UI. The 1st step in troubleshooting would be to look at the actual json on the node role in "raw" mode. Rob From: crowbar-bounces On Behalf Of Sébastien Braun Sent: Thursday, December 12, 2013 6:58 AM To: crowbar Subject: Re: [Crowbar] NTP coo

Re: [Crowbar] NTP cookbook broken in CB20

2013-12-12 Thread Adam Spiers
Sébastien Braun (sebbr...@gmail.com) wrote: > Hi Adam, > > It's not an Array when bootstrapping the admin node from the Crowbar 2.0 > Web UI, but a string. > > When you run the installer with --wizard, you get the opportunity to setup > things before installing the admin node, when you give a NTP

Re: [Crowbar] NTP cookbook broken in CB20

2013-12-12 Thread Sébastien Braun
Hi Adam, It's not an Array when bootstrapping the admin node from the Crowbar 2.0 Web UI, but a string. When you run the installer with --wizard, you get the opportunity to setup things before installing the admin node, when you give a NTP server information in the UI it's given to the NTP cookbo

Re: [Crowbar] NTP cookbook broken in CB20

2013-12-11 Thread Adam Spiers
Adam Spiers (aspi...@suse.com) wrote: > Having said that, in this case, isn't @ntp_servers supposed to be an > Array of Strings, not a String? Maybe you are editing the proposal > JSON incorrectly? Oh, you're talking about Crowbar 2.0. Well, my point still stands: https://github.com/crowbar/bar

Re: [Crowbar] NTP cookbook broken in CB20

2013-12-11 Thread Adam Spiers
Hi Sébastien, Sébastien Braun (sebbr...@gmail.com) wrote: > Ruby 1.9 removed each method for strings. > > But when we provision an admin node, if we edit the ntp-server role > template to add an NTP server, it will break provisionning. Because when > trying to use ntp.conf.erb template, it will c

[Crowbar] NTP cookbook broken in CB20

2013-12-10 Thread Sébastien Braun
Hi, Ruby 1.9 removed each method for strings. But when we provision an admin node, if we edit the ntp-server role template to add an NTP server, it will break provisionning. Because when trying to use ntp.conf.erb template, it will call each on the string containing the NTP servers :( Both chef