Re: [OpenStack-Infra] [jjb] What's the deal with {{?

2015-08-13 Thread Ian Wienand
On 08/13/2015 08:19 PM, Darragh Bailey wrote: macros do not get substitution performed unless you provide a variable to be substituted in. Thanks; that makes some sense when you grok what's going on, especially as to why job-templates require it but other macros don't. I have proposed [1] to b

Re: [OpenStack-Infra] ProphetStor CI account

2015-08-13 Thread Asselin, Ramy
Hi Rick, In general, to get your account re-enabled, you need permission from the team that requested the account to be disabled. In this case it's the cinder team, not the infra team [1] So you should post to the general developer list [2]. Add the tags [cinder] and [third-party] to get bett

Re: [OpenStack-Infra] ProphetStor CI account

2015-08-13 Thread Mike Perez
On 23:21 Aug 13, Rick Chen wrote: > Dear Infra team, > > > > I already fixed my CI machine about failed to build up the devstack issue. > Can you re-enable my CI gerrit account? > > Account : prophetstor-ci > > Account email: prophetstor...@prophetstor.com >

Re: [OpenStack-Infra] Using ensure=>running by default in Puppet modules

2015-08-13 Thread Yolanda Robla Mota
Ok, but i continue seeing ths problem again between what infra says it's reasonable, and i fully believe your are right, and what downstream users are expecting on that. So why be opinionated on it? My opinion is to add the possibility to the module, defaulting to false so when an user enables

Re: [OpenStack-Infra] Using ensure=>running by default in Puppet modules

2015-08-13 Thread Jeremy Stanley
On 2015-08-13 09:16:46 -0700 (-0700), Colleen Murphy wrote: [...] > Downstream users of a puppet module will always expect the > module to [...] start services. [...] I find it hard to believe that this is a common expectation among system administrators, but it may be a cultural/generational bias

Re: [OpenStack-Infra] Using ensure=>running by default in Puppet modules

2015-08-13 Thread Colleen Murphy
On Wed, Aug 12, 2015 at 2:15 PM, Jeremy Stanley wrote: > Change https://review.openstack.org/168306 for puppet-zuul came to > my attention earlier today when it merged. After a quick discussion > on IRC, Spencer proposed a revert which I approved so that we can > get a little more discussion goin

Re: [OpenStack-Infra] Using ensure=>running by default in Puppet modules

2015-08-13 Thread James E. Blair
Fabien Boucher writes: > Hi, > > I apologize I wasn't here yesterday to discuss about that on IRC. > I was surprised also about not letting puppet ensuring that the service is > running and > now I understand well the reason behind it. But I think too, that for > most of downstream users (like t

Re: [OpenStack-Infra] Using ensure=>running by default in Puppet modules

2015-08-13 Thread Paul Belanger
On Wed, Aug 12, 2015 at 09:15:37PM +, Jeremy Stanley wrote: > Change https://review.openstack.org/168306 for puppet-zuul came to > my attention earlier today when it merged. After a quick discussion > on IRC, Spencer proposed a revert which I approved so that we can > get a little more discussi

[OpenStack-Infra] ProphetStor CI account

2015-08-13 Thread Rick Chen
Dear Infra team, I already fixed my CI machine about failed to build up the devstack issue. Can you re-enable my CI gerrit account? Account : prophetstor-ci Account email: prophetstor...@prophetstor.com Latest CI verify report: http://download.pro

Re: [OpenStack-Infra] [Openstack-Infra][CI][Keystone] Starting keystone failing in Devstack for CI

2015-08-13 Thread Jeremy Stanley
On 2015-08-13 13:48:17 +0530 (+0530), Abhishek Shrivastava wrote: > In my CI while starting Keystone the service is failing and showing the > following ERROR: [...] > 2015-08-13 08:08:58.937 | +++ openstack --os-url=http://127.0.0.1:5000/v3 > --os-identity-api-version=3 project create admin --domai

Re: [OpenStack-Infra] Using ensure=>running by default in Puppet modules

2015-08-13 Thread Fabien Boucher
Hi, I apologize I wasn't here yesterday to discuss about that on IRC. I was surprised also about not letting puppet ensuring that the service is running and now I understand well the reason behind it. But I think too, that for most of downstream users (like those of puppet-openstackci) the most o

Re: [OpenStack-Infra] IBM XIV CI account

2015-08-13 Thread Mikhail Medvedev
Hi Isaac, I have added your new account (ibmxi...@il.ibm.com) to the Third-Party CI gerrit group and have removed the old one. Someone from the Infra team would still need to deactivate your old gerrit account. Mikhail Medvedev (irc:mmedvede) IBM PowerKVM CI On Wed, Aug 12, 2015 at 12:58 AM,

Re: [OpenStack-Infra] [jjb] What's the deal with {{?

2015-08-13 Thread Darragh Bailey
Hi, macros do not get substitution performed unless you provide a variable to be substituted in. Following would trigger the behaviour: - job-template: name: '{foo}-test' builders: - test_builder: FOO_BAR: hello - shell: | echo ${{FOO_3}} Definitely a

Re: [OpenStack-Infra] [Openstack-Infra][CI][Keystone] Starting keystone failing in Devstack for CI

2015-08-13 Thread Abhishek Shrivastava
Hi Folks, Please reply for this issue as its causing a lot of trouble now. On Thu, Aug 13, 2015 at 2:43 PM, Abhishek Shrivastava < abhis...@cloudbyte.com> wrote: > Hi Folks, > > In my CI while starting Keystone the service is failing and showing the > following ERROR: > > 2015-08-13 08:08:58.935

Re: [OpenStack-Infra] Using ensure=>running by default in Puppet modules

2015-08-13 Thread Ricardo Carrillo Cruz
I lean towards the second option that Jeremy pointed out. External users just expect Puppet to bring up services configured by the module, it makes sense to me having 'running' as default and override that at the node or wrapper module. Regards El 13/8/2015 8:30, "Yolanda Robla Mota" escribió:

Re: [OpenStack-Infra] Puppet lint checks for system-config

2015-08-13 Thread Yolanda Robla Mota
I agree with you Jim, but sadly it's a practice in the reality to -1 for nitpicks or even for questions. I'd prefer if that nitpicks can be flagged, but as a comment, so they can be corrected in future iterations if the patch needs more work, but not a blocking -1. Also same as -1 for questions, i

[OpenStack-Infra] [Openstack-Infra][CI][Keystone] Starting keystone failing in Devstack for CI

2015-08-13 Thread Abhishek Shrivastava
Hi Folks, In my CI while starting Keystone the service is failing and showing the following ERROR: 2015-08-13 08:08:58.935 | + export OS_TOKEN=111222333444 2015-08-13 08:08:58.935 | + OS_TOKEN=111222333444 2015-08-13 08:08:58.935 | + export OS_URL=http://127.0.0.1:35357/v2.0 2015-08-13 08:08:58.9