Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2014-03-20

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:24 hogepodge joined #puppet-openstack
00:26 thuc joined #puppet-openstack
00:56 hogepodge joined #puppet-openstack
01:03 mgagne joined #puppet-openstack
01:19 mgagne1 joined #puppet-openstack
01:33 Aimon joined #puppet-openstack
01:37 thuc joined #puppet-openstack
01:37 Aimon joined #puppet-openstack
01:40 Aimon joined #puppet-openstack
01:44 xingchao joined #puppet-openstack
01:52 kitp joined #puppet-openstack
02:12 thuc joined #puppet-openstack
02:19 xarses joined #puppet-openstack
02:25 thuc_ joined #puppet-openstack
02:32 thuc joined #puppet-openstack
02:44 daneyon joined #puppet-openstack
02:53 kitp joined #puppet-openstack
03:00 thuc joined #puppet-openstack
03:24 thuc joined #puppet-openstack
03:54 kitp joined #puppet-openstack
04:12 saju_m joined #puppet-openstack
04:13 Akshik joined #puppet-openstack
04:21 thuc joined #puppet-openstack
04:29 kitp joined #puppet-openstack
04:41 thuc joined #puppet-openstack
04:56 Akshik_ joined #puppet-openstack
05:17 PEPPERONIE joined #puppet-openstack
05:33 kitp joined #puppet-openstack
05:57 thuc joined #puppet-openstack
05:59 xingchao_ joined #puppet-openstack
06:04 saju_m joined #puppet-openstack
06:08 saju_m joined #puppet-openstack
07:19 Akshik_ joined #puppet-openstack
07:22 Akshik joined #puppet-openstack
07:56 saju_m joined #puppet-openstack
08:04 morazi joined #puppet-openstack
08:11 EmilienM good morning
08:25 Akshik_ joined #puppet-openstack
08:29 bauzas joined #puppet-openstack
09:06 Akshik joined #puppet-openstack
09:07 Akshik_ joined #puppet-openstack
09:10 openstackgerrit Benedikt Trefzer proposed a change to stackforge/puppet-heat: replace rabbit_virtualhost parameter  https://review.openstack.org/81736
09:19 andi- joined #puppet-openstack
09:19 andi- joined #puppet-openstack
09:20 derekh joined #puppet-openstack
09:44 mattymo EmilienM, have you guys fooled around with docker yet?
09:50 mmagr joined #puppet-openstack
09:54 nigelk joined #puppet-openstack
09:57 EmilienM mattymo: not for real, i know leseb did some tests, you can read his blog
10:33 rwsu joined #puppet-openstack
11:08 uberj joined #puppet-openstack
11:44 openstackgerrit Michael Chapman proposed a change to stackforge/puppet_openstack_builder: Rewrite data model to be pure hiera  https://review.openstack.org/81241
12:26 saju_m joined #puppet-openstack
12:33 openstackgerrit Gonéri Le Bouder proposed a change to stackforge/puppet-horizon: ability to pass extra parameters to the vhost  https://review.openstack.org/81797
12:50 dgollub joined #puppet-openstack
13:03 thuc joined #puppet-openstack
13:04 thuc_ joined #puppet-openstack
13:06 saju_m joined #puppet-openstack
13:33 dmsimard joined #puppet-openstack
13:39 saju_m joined #puppet-openstack
13:39 francois1 joined #puppet-openstack
13:52 kitp joined #puppet-openstack
14:16 dgollub joined #puppet-openstack
14:28 xarses joined #puppet-openstack
14:30 dprince joined #puppet-openstack
14:31 thuc joined #puppet-openstack
14:34 kitp joined #puppet-openstack
14:37 nigelk joined #puppet-openstack
14:54 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet_openstack_builder: Support separate apt repo for puppet modules  https://review.openstack.org/81602
14:56 thuc joined #puppet-openstack
14:57 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet_openstack_builder: Support separate apt repo for puppet modules  https://review.openstack.org/81602
14:58 dmsimard1 joined #puppet-openstack
14:59 ryanycoleman joined #puppet-openstack
15:01 dgollub joined #puppet-openstack
15:02 thuc joined #puppet-openstack
15:02 morazi joined #puppet-openstack
15:07 dtalton joined #puppet-openstack
15:11 kkirkpatrick joined #puppet-openstack
15:20 xarses joined #puppet-openstack
15:20 kkirkpatrick joined #puppet-openstack
15:23 mgagne joined #puppet-openstack
15:25 mattymo joined #puppet-openstack
15:33 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet_openstack_builder: Support separate apt repo for puppet modules  https://review.openstack.org/81602
15:34 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet_openstack_builder: Support separate apt repo for puppet modules  https://review.openstack.org/81602
15:35 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet_openstack_builder: Support separate apt repo for puppet modules  https://review.openstack.org/81602
15:35 kitp joined #puppet-openstack
15:37 daneyon joined #puppet-openstack
15:40 Aimon joined #puppet-openstack
15:56 mmagr joined #puppet-openstack
16:01 rmoe joined #puppet-openstack
16:02 kevin joined #puppet-openstack
16:04 ryanycoleman joined #puppet-openstack
16:06 dtalton joined #puppet-openstack
16:07 otherwiseguy joined #puppet-openstack
16:12 ryanycoleman joined #puppet-openstack
16:22 hogepodge EmilienM have you had a chance to check out https://review.openstack.org/#/c/81408/
16:22 EmilienM hogepodge: tonight, still at school atm
16:22 hogepodge If anyone else with type/provider experience want to take a look too (say bodepd) https://review.openstack.org/#/c/81408/
16:23 thuc joined #puppet-openstack
16:28 thuc joined #puppet-openstack
16:33 kkirkpatrick joined #puppet-openstack
16:35 hogepodge joined #puppet-openstack
16:36 kitp joined #puppet-openstack
16:37 mgagne joined #puppet-openstack
16:43 thuc_ joined #puppet-openstack
16:52 kitp joined #puppet-openstack
16:55 pabelanger left #puppet-openstack
17:01 mgagne joined #puppet-openstack
17:12 hogepodge joined #puppet-openstack
17:15 openstackgerrit A change was merged to stackforge/puppet-glance: set ssl options for glance-api  https://review.openstack.org/78907
17:23 otherwiseguy joined #puppet-openstack
17:25 openstackgerrit A change was merged to stackforge/puppet-glance: set ssl options for glance-registry  https://review.openstack.org/78915
17:28 kevin joined #puppet-openstack
17:47 openstackgerrit Benedikt Trefzer proposed a change to stackforge/puppet-glance: set ssl options for glance-api  https://review.openstack.org/81861
17:50 dprince_ joined #puppet-openstack
17:53 kitp joined #puppet-openstack
18:00 dprince joined #puppet-openstack
18:24 dgollub joined #puppet-openstack
18:27 openstackgerrit A change was merged to stackforge/puppet-cinder: Add default_volume_type as cinder api parameter  https://review.openstack.org/81473
18:28 openstackgerrit Benedikt Trefzer proposed a change to stackforge/puppet-glance: set ssl options for glance-registry  https://review.openstack.org/81880
18:29 mgagne EmilienM: ping
18:29 PEPPERONIE joined #puppet-openstack
18:30 mgagne EmilienM: I don't know where my comment went but this change clashes with cinder::backends
18:30 mgagne EmilienM: default_volume_type is already managed elsewhere
18:30 EmilienM mgagne: yeah, i know I just did the backport of his patch
18:30 EmilienM mgagne: i abandoned mine though
18:31 mgagne EmilienM: you know? why backporting a broken change?
18:31 mgagne EmilienM: cinder::backends is now broken for everyone using it
18:32 EmilienM mgagne: sorry I don't get it. I took a patch  from markvoelker and backported it into havana
18:32 EmilienM that's all I did, I don't see what I'm breaking
18:33 thuc joined #puppet-openstack
18:33 mgagne EmilienM: you said you knew that this change broke cinder::backends. Or I misunderstood what you meant.
18:34 EmilienM mgagne: you -2 a patch I did. I'm OK with that, I did a mistake and I immediatly abandoned it
18:34 EmilienM mgagne: then I looked at who did the patch in master and just did the backport. What's wrong?
18:34 mgagne EmilienM: but a similar one got approved and then backported
18:34 mgagne EmilienM: damn, I approved it
18:34 mgagne EmilienM: I'm an idiot
18:35 mgagne EmilienM: it should be reverted
18:35 EmilienM mgagne: talking about https://review.openstack.org/#/c/81473/ ?
18:35 mgagne EmilienM: yes
18:35 mgagne EmilienM: https://review.openstack.org/#/c/81473/
18:36 angdraug joined #puppet-openstack
18:36 EmilienM mgagne: I don't see that it breaks
18:36 EmilienM it's a backport to havana
18:36 mgagne EmilienM: there is no test
18:36 mgagne EmilienM: that's the problem and you probably don't use cinder::backends
18:37 mgagne EmilienM: because unit tests pass doesn't mean it won't break anything, some stuff might not be tested
18:37 EmilienM mgagne: i use multi backend in our setup here
18:37 mgagne EmilienM: do you use cinder::backends ?
18:37 PEPPERONIE joined #puppet-openstack
18:37 EmilienM mgagne: yes, netapp and rbd drivers
18:37 EmilienM mgagne: so the patch in master from markvoelker broke something?
18:38 mgagne EmilienM: how is this https://github.com/stackforge/puppet-cind​er/blob/master/manifests/backends.pp#L32 not conflicting with this? https://github.com/stackforge/puppet-ci​nder/blob/master/manifests/api.pp#L175
18:39 EmilienM mgagne: gotcha
18:39 EmilienM mgagne: so we need to revert the backport and also the patch from markvoelker in master
18:39 mgagne EmilienM: yep, the mistake I made is approving the one in master
18:39 EmilienM mgagne: nah, I did the error to backport, my bad
18:39 mgagne EmilienM: will add tests for cinder::backends
18:40 EmilienM mgagne: I let you revert and I can +2 +A if you want
18:40 mgagne EmilienM: ok
18:40 mgagne EmilienM: One thing thought
18:40 mgagne EmilienM: though*
18:41 mgagne EmilienM: is cinder::backends supposed to be configured on the cinder-api or cinder-volume?
18:41 EmilienM mgagne: cinder volume
18:41 mgagne EmilienM: which services use default_volume_type then? both?
18:42 EmilienM mgagne: volume only
18:42 mgagne EmilienM: we should also find a way to check if all classes can be included/used in the same manifest/node
18:42 EmilienM if you look at cinder code
18:42 mgagne EmilienM: isn't it in cinder-scheduler?
18:42 EmilienM mgagne: nop
18:43 mgagne EmilienM: api volume encryption uses it
18:44 mgagne EmilienM: api/volumes too
18:44 EmilienM mgagne: should we move it in init.pp?
18:45 mgagne EmilienM: will have to check/test
18:45 EmilienM mgagne: are you using multi backend?
18:46 mgagne EmilienM: no but I have eagle eyes (at my time)
18:51 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-cinder: Revert "Add default_volume_type as cinder api parameter"  https://review.openstack.org/81886
18:52 EmilienM mgagne: can I +2 +A ?
18:53 kitp joined #puppet-openstack
18:55 markvoelker EmilienM: sorry, just catching backlog.  What patch are we talking about? Can't remember the last time I committed something to cinder...
18:55 EmilienM markvoelker: https://review.openstack.org/#q,cd7391c5e2567a081d5a214a80cb773caa1ae2c6,n,z
18:56 markvoelker Ah.  That's Marcd Koderer, not Mark Voelker. =)
18:56 EmilienM markvoelker: oops
18:56 EmilienM markvoelker: i own you a beer for this one
18:56 markvoelker EmilienM: Nah, no worries.
18:58 mgagne markvoelker, EmilienM: we have to check which service(s) use it and then move it (if needed)
18:58 EmilienM mgagne: yes. Definitely
19:03 thuc joined #puppet-openstack
19:11 thumpba_ joined #puppet-openstack
19:12 thuc_ joined #puppet-openstack
19:23 openstackgerrit Chris Ricker proposed a change to stackforge/puppet_openstack_builder: Drop rubygems install for trusty / icehouse  https://review.openstack.org/81892
19:27 openstackgerrit A change was merged to stackforge/puppet-cinder: Revert "Add default_volume_type as cinder api parameter"  https://review.openstack.org/81886
19:35 openstackgerrit Chris Ricker proposed a change to stackforge/puppet_openstack_builder: Remove automatic configuration of nagios  https://review.openstack.org/81897
19:54 kitp joined #puppet-openstack
19:58 otherwiseguy joined #puppet-openstack
19:58 thuc joined #puppet-openstack
19:59 thuc_ joined #puppet-openstack
20:08 daneyon joined #puppet-openstack
20:14 thuc joined #puppet-openstack
20:18 PEPPERONIE joined #puppet-openstack
20:33 bcrochet joined #puppet-openstack
20:50 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet_openstack_builder: Support separate apt repo for puppet modules  https://review.openstack.org/81602
20:54 dmsimard hogepodge: You have an idea who I can poke about https://github.com/puppetlabs/puppetlabs-corosync ? Seems largely unmaintained from looking at the amount of pull requests :(
20:55 kitp joined #puppet-openstack
20:57 EmilienM dmsimard: and not supported on RHEL :(
20:59 dmsimard EmilienM: Yeah, I think I saw hogepodge looking to implement HA in puppetlabs-havana/puppet_openstack_builder using corosync, hence why I'm asking him :p
21:00 EmilienM dmsimard: i think it's for some services, we do the same here
21:00 EmilienM dmsimard: heat-engine and ceilometer central agent
21:03 thuc joined #puppet-openstack
21:04 rwsu joined #puppet-openstack
21:12 hogepodge joined #puppet-openstack
21:12 hogepodge dmsimard checking
21:13 dmsimard EmilienM: which reminds me, I configured the ceilometer middleware for swift but it doesn't seem to set up all the meters that seem to be available as per http://docs.openstack.org/develo​per/ceilometer/measurements.html. Do I need ceilometer agent central to poll swift or something ?
21:14 EmilienM dmsimard: for sure you need it
21:14 EmilienM dmsimard: there is some meters get by central agent
21:14 dmsimard EmilienM: That would make sense then.. need to read up on that
21:16 thuc joined #puppet-openstack
21:16 hogepodge dmsimard um, we agree?
21:17 hogepodge nobody has been assigned that module. It may fall to me at some point. I hate to pass the buck, but Florian probably has the knowledge of the best pacemaker/corosync module to use.
21:18 hogepodge I'm very much in FIFO triage right now, as I imagine lots of people are.
21:20 dmsimard Florian.. fghaas ?
21:21 EmilienM yep I guess
21:24 xavpaice pacemaker/corosync on Ubuntu Precise works well with puppetlabs-corosync
21:24 xavpaice I followed Florian's guide on setup first
21:25 EmilienM on debian too
21:25 EmilienM but forget about RH
21:25 EmilienM in RH they don't have 'crm' cli command
21:26 xavpaice corosync changed quite a lot fairly recently, I think RHEL has the 'new' version with a more separate pacemaker
21:27 xavpaice hmm, which isn't in the puppetlabs-corosync module :(
21:28 openstackgerrit Gonéri Le Bouder proposed a change to stackforge/puppet-horizon: ability to pass extra parameters to the vhost  https://review.openstack.org/81797
21:29 hogepodge Tell you what. If dmsimard and xavpaice do a review of the PRs for corosync I'll approve anything that they thumbs up on.
21:30 hogepodge and anyone else with pacemaker experience.
21:30 hogepodge I'd be happy to just do the reviews myself, but I need to familiarize myself with it more.
21:31 dmsimard I'm not comfortable enough with corosync (especially under Redhat-like) to do reviews :(
21:33 daneyon joined #puppet-openstack
21:35 openstackgerrit A change was merged to stackforge/puppet-nova: Added the ability to manage the nova uid/gid.  https://review.openstack.org/76298
21:35 * xavpaice hasn't touched RedHat in a while either
21:56 kitp joined #puppet-openstack
22:07 marun joined #puppet-openstack
22:25 openstackgerrit Jacques Uber proposed a change to stackforge/puppet-glance: Explicity set default notifier strategy  https://review.openstack.org/81930
22:26 uberj uhg, I think I named my branch wrong
22:26 openstackgerrit A change was merged to stackforge/puppet-glance: set ssl options for glance-registry  https://review.openstack.org/81880
22:27 openstackgerrit A change was merged to stackforge/puppet-glance: set ssl options for glance-api  https://review.openstack.org/81861
22:31 openstackgerrit Jacques Uber proposed a change to stackforge/puppet-glance: Explicity set default notifier strategy  https://review.openstack.org/81930
22:31 thuc_ joined #puppet-openstack
22:31 uberj there!
22:32 uberj http://25.media.tumblr.com/tumb​lr_lwohhaNr2p1r8cvzdo1_250.gif
22:36 thuc joined #puppet-openstack
22:36 uberj dtalton: can you tell me if I did the spec test thing right in https://review.openstack.org/81930 ?
22:36 openstackgerrit Gonéri Le Bouder proposed a change to stackforge/puppet-horizon: ability to pass extra parameters to the vhost  https://review.openstack.org/81797
22:37 dtalton uberj, looks ok, did you test it?
22:37 uberj yes
22:37 dtalton cool
22:37 uberj hopefully jenkins will agree with my :)
22:37 uberj *me
22:38 uberj I think I see an improvement, though
22:41 uberj should I test for the absence of notifier_strategy=noop here https://github.com/stackforge/puppet​-glance/blob/stable/havana/spec/clas​ses/glance_notify_qpid_spec.rb#L16 ?
22:42 uberj huh, the review failed jenkins.
22:45 openstackgerrit Jacques Uber proposed a change to stackforge/puppet-glance: Explicity set default notifier strategy  https://review.openstack.org/81930
22:46 thuc joined #puppet-openstack
22:46 uberj missed adding a comma during a rebase against havana
22:52 uberj there, *now* jenkins is happy
22:57 kitp joined #puppet-openstack
23:05 daneyon joined #puppet-openstack
23:11 kkirkpatrick joined #puppet-openstack
23:23 derekh joined #puppet-openstack
23:33 thumpba joined #puppet-openstack
23:36 xarses joined #puppet-openstack
23:43 openstackgerrit Michael Chapman proposed a change to stackforge/puppet-neutron: Make haproxy package management optional  https://review.openstack.org/81430
23:57 kitp joined #puppet-openstack

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary