Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2014-02-25

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

All times shown according to UTC.

Time Nick Message
00:19 xarses joined #puppet-openstack
00:27 hogepodge codec sorry for the late reply. You'll need to code around it a bit to make sure classes aren't applied twice. It's a pretty opinionated and picky module, but is easy to tweak.
00:30 lnxnut joined #puppet-openstack
00:35 lnxnut joined #puppet-openstack
00:38 xarses joined #puppet-openstack
00:46 britthou_ joined #puppet-openstack
00:50 lnxnut joined #puppet-openstack
00:52 thumpba joined #puppet-openstack
00:54 britthouser joined #puppet-openstack
00:54 lnxnut joined #puppet-openstack
01:01 lnxnut joined #puppet-openstack
01:02 openstackgerrit Michael Chapman proposed a change to stackforge/puppet_openstack_builder: Fix rpc in glance and mappings  https://review.openstack.org/76059
01:09 mgagne joined #puppet-openstack
01:17 rmoe joined #puppet-openstack
01:35 mgagne joined #puppet-openstack
01:36 fandi joined #puppet-openstack
01:39 lnxnut joined #puppet-openstack
01:40 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Fix bad param to setup_test_volume class  https://review.openstack.org/75996
01:56 xingchao joined #puppet-openstack
01:57 lnxnut joined #puppet-openstack
01:57 rongze joined #puppet-openstack
02:16 mgagne joined #puppet-openstack
02:25 andi- joined #puppet-openstack
02:28 britthouser joined #puppet-openstack
02:40 lnxnut joined #puppet-openstack
02:46 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet_openstack_builder: Re-enable nova services  https://review.openstack.org/75959
02:50 markvoelker1 joined #puppet-openstack
02:59 xarses joined #puppet-openstack
03:09 kkirkpatrick joined #puppet-openstack
03:19 lnxnut joined #puppet-openstack
03:56 lnxnut joined #puppet-openstack
04:04 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet-nova: Parameterize cpu_mode, deprecatelibvirt_type  https://review.openstack.org/75748
04:04 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet-nova: Parameterize cpu_mode, deprecate libvirt_type  https://review.openstack.org/75748
04:11 lnxnut joined #puppet-openstack
04:22 thumpba joined #puppet-openstack
04:26 lnxnut joined #puppet-openstack
04:33 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet-nova: Parameterize cpu_mode, deprecate libvirt_type  https://review.openstack.org/75748
04:48 rongze joined #puppet-openstack
04:54 thumpba_ joined #puppet-openstack
04:58 fandi joined #puppet-openstack
05:18 rongze joined #puppet-openstack
05:21 rongze joined #puppet-openstack
05:36 lnxnut joined #puppet-openstack
05:42 openstackgerrit A change was merged to stackforge/puppet-nova: [vmware] Replaced pip with native package manager  https://review.openstack.org/75458
06:06 rongze joined #puppet-openstack
06:12 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet-nova: Parameterize disk_cachemodes  https://review.openstack.org/76099
06:18 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Add Support for mongodb replica sets to the data model  https://review.openstack.org/75999
06:21 saju_m joined #puppet-openstack
06:26 lnxnut joined #puppet-openstack
06:42 saju_m joined #puppet-openstack
06:44 openstackgerrit A change was merged to stackforge/puppet-nova: Add ability to disable endpoint configuration  https://review.openstack.org/72461
06:48 openstackgerrit A change was merged to stackforge/puppet-nova: Deprecates old libvirt config  https://review.openstack.org/63978
06:54 mkoderer joined #puppet-openstack
06:58 openstackgerrit A change was merged to stackforge/puppet-nova: Fix deprecation warnings  https://review.openstack.org/70026
07:13 saju_m joined #puppet-openstack
07:15 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet-nova: Parameterize cpu_mode  https://review.openstack.org/75748
07:26 lnxnut joined #puppet-openstack
07:29 francois1 joined #puppet-openstack
07:43 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet-nova: Parameterize disk_cachemodes  https://review.openstack.org/76099
07:43 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet-nova: Parameterize cpu_mode  https://review.openstack.org/75748
07:54 bauzas joined #puppet-openstack
08:04 EmilienM good morning
08:17 dgollub joined #puppet-openstack
08:19 EmilienM michchap: just read https://review.openstack.org/#/c/72440/4/manifests/compute/rbd.pp
08:19 EmilienM michchap: gonna change for /etc/nova/... it makes sense
08:22 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-nova: compute: RBD backend support  https://review.openstack.org/72440
08:24 markvoelker1 joined #puppet-openstack
08:27 lnxnut joined #puppet-openstack
08:47 mmagr joined #puppet-openstack
09:07 openstackgerrit A change was merged to stackforge/puppet-neutron: Don't be too strict for vlan_ranges  https://review.openstack.org/76000
09:12 michchap EmilienM: Have you asked dachary? I'm not an expert on ceph/nova integration, he may have a better proposal.
09:16 EmilienM michchap: not yet
09:17 EmilienM dachary: if you are around, could you have a look at https://review.openstack.org/72440 ?
09:17 derekh joined #puppet-openstack
09:24 markvoelker1 joined #puppet-openstack
09:26 lnxnut joined #puppet-openstack
09:36 fandi joined #puppet-openstack
09:42 mmagr joined #puppet-openstack
09:53 bauzas joined #puppet-openstack
09:53 openstackgerrit A change was merged to stackforge/puppet-nova: Fix usage of double quote instead of simple quote  https://review.openstack.org/74033
10:15 openstackgerrit Pierre Rambaud proposed a change to stackforge/puppet-nova: Fix usage of double quote instead of simple quote  https://review.openstack.org/76146
10:24 xingchao joined #puppet-openstack
10:24 openstackgerrit Pierre Rambaud proposed a change to stackforge/puppet-nova: Fix usage of double quote instead of simple quote  https://review.openstack.org/76148
10:26 lnxnut joined #puppet-openstack
10:31 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Re-enable nova services  https://review.openstack.org/75959
11:13 rongze joined #puppet-openstack
11:24 markvoelker1 joined #puppet-openstack
11:26 lnxnut joined #puppet-openstack
11:54 dmsimard joined #puppet-openstack
12:10 markvoelker1 joined #puppet-openstack
12:26 lnxnut joined #puppet-openstack
12:28 openstackgerrit Francois Deppierraz proposed a change to stackforge/puppet_openstack_builder: Expose controller_public_protocol parameter  https://review.openstack.org/75858
12:29 openstackgerrit Francois Deppierraz proposed a change to stackforge/puppet_openstack_builder: Expose controller_public_protocol parameter  https://review.openstack.org/75858
13:00 mgagne1 joined #puppet-openstack
13:04 dmsimard puppet-swift should probably have a swift_config provider like the other modules instead of using templates :(
13:05 markvoelker1 joined #puppet-openstack
13:26 lnxnut joined #puppet-openstack
13:45 rongze joined #puppet-openstack
13:56 dprince joined #puppet-openstack
14:02 lnxnut joined #puppet-openstack
14:10 mgagne1 dmsimard: you can inject custom config using concat::fragment with the right name/order
14:11 kkirkpatrick joined #puppet-openstack
14:12 dmsimard mgagne: Yeah, I know. I'm sure you get what I'm saying, though :)
14:12 mgagne dmsimard: =)
14:27 prad_ joined #puppet-openstack
14:37 openstackgerrit Chris Ricker proposed a change to stackforge/puppet-neutron: neutron-metering-agent package name is wrong  https://review.openstack.org/76218
14:48 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-nova: enable libvirt at boot  https://review.openstack.org/75878
14:48 EmilienM markvoelker: fc__ : I'm rebased (jenkins failed to merge it) https://review.openstack.org/#/c/75878/ - could you +2 / +A again? ^ Thanks
14:48 lnxnut joined #puppet-openstack
14:48 * markvoelker looks
14:49 markvoelker EmilienM: done
14:49 EmilienM markvoelker: so fast, thanks :-)
14:49 openstackgerrit Chris Ricker proposed a change to stackforge/puppet-neutron: neutron-metering-agent package name is wrong  https://review.openstack.org/76218
14:51 EmilienM markvoelker: I would need also some +A on https://review.openstack.org/#/c/75914/  - https://review.openstack.org/#/c/75929/ and https://review.openstack.org/#/c/75930/
14:53 lnxnut joined #puppet-openstack
14:58 markvoelker EmilienM: roger that.  I'll have a look as soon as I get out of this meeting (or possibly during =p)
15:01 EmilienM markvoelker: thanks !
15:08 openstackgerrit A change was merged to stackforge/puppet-neutron: neutron-metering-agent package name is wrong  https://review.openstack.org/76218
15:14 openstackgerrit Chris Ricker proposed a change to stackforge/puppet-neutron: neutron-metering-agent package name is wrong  https://review.openstack.org/76232
15:14 otherwiseguy joined #puppet-openstack
15:24 fvollero EmilienM: markvoelker : mgagne : bodepd_ : I need all your power fellas... I recall I made a patch (already merged in master and havana) but this patch was just half complete, now, for bugzilla related thing, I need to complete this patch and have it done (and backported to havana) for today, could you guys help me on that? I would be really really grateful
15:25 EmilienM fvollero: yes
15:25 EmilienM go ahead with the patch and I'll review it today
15:26 fvollero EmilienM: Thank you very much mon ami!
15:26 fvollero EmilienM: markvoelker : mgagne : bodepd_ : for more context https://review.openstack.org/#/c/37493/ this need to have also a change in https://github.com/stackforge/puppet-swift/blob/master/manifests/storage/all.pp
15:28 openstackgerrit A change was merged to stackforge/puppet-neutron: neutron-metering-agent package name is wrong  https://review.openstack.org/76232
15:52 dtalton joined #puppet-openstack
15:55 kkirkpatrick joined #puppet-openstack
16:00 comptona joined #puppet-openstack
16:05 openstackgerrit A change was merged to stackforge/puppet-nova: Deprecate logdir parameter (use log_dir instead)  https://review.openstack.org/75929
16:05 openstackgerrit A change was merged to stackforge/puppet-nova: Allow log_dir to be set to false in order to disable file logging  https://review.openstack.org/75930
16:06 openstackgerrit A change was merged to stackforge/puppet-glance: Add log_dir param. by default and allow users to disable it  https://review.openstack.org/75914
16:13 mfisch markvoelker: morning, I'd like to introduce you to dvorak, he works on our team here and knows a good amount about puppet, he's hoping to contribute some fixes too
16:13 dvorak thanks mfisch :)
16:13 openstackgerrit A change was merged to stackforge/puppet-ironic: Update contributors link according stackforge migration  https://review.openstack.org/75637
16:26 openstackgerrit Francesco Vollero proposed a change to stackforge/puppet-swift: Adding full support for allow_versions in swift containers The previous patch (37493) was just implementing the feature partially  https://review.openstack.org/76253
16:26 fvollero EmilienM: done ^^
16:26 EmilienM fvollero: commit message is too long
16:27 fvollero EmilienM: but i added it on two lines. I'll amend it
16:27 openstackgerrit Francesco Vollero proposed a change to stackforge/puppet-swift: Adding full support for allow_versions in swift containers  https://review.openstack.org/76253
16:28 fvollero EmilienM: better in this way right? :)
16:28 EmilienM yepp
16:28 markvoelker fvollero: still too long though. =)
16:28 ryanycoleman joined #puppet-openstack
16:29 fvollero markvoelker: :( echo 'Adding full support for allow_versions in swift containers' | wc -c                                                                                       (git)-[967308]
16:29 fvollero 59
16:30 markvoelker fvollero: https://wiki.openstack.org/wiki/GitCommitMessages#Information_in_commit_messages
16:30 markvoelker first line should be limited to 50 characters
16:30 markvoelker Subsequent lines should be wrapped at 72 characters.
16:30 fvollero markvoelker: snap!
16:30 fvollero markvoelker: could you help me rephrasing it ?
16:31 markvoelker fvollero: "Full support for allow_versions in containers" ?
16:31 * markvoelker very often has a hard time cramming a meaningful title into 50 chars too
16:31 mgagne joined #puppet-openstack
16:32 openstackgerrit Francesco Vollero proposed a change to stackforge/puppet-swift: Support for allow_versions in swift containers  https://review.openstack.org/76253
16:32 fvollero :)
16:32 fvollero markvoelker: that should do the trick, right?
16:32 markvoelker fvollero: yep
16:32 fvollero 47 chars
16:34 fvollero markvoelker: how was your weekend ?
16:34 fvollero EmilienM: You still have outstanding reviews that need attention?
16:36 EmilienM no
16:36 markvoelker fvollero: good.  Did something besides puppet for a change. =p
16:36 markvoelker fvollero: looks like this is failing the gate on multiple tests
16:38 fvollero markvoelker: i just saw it, but strangely on my machine is all green... maybe it's a stupid comma problem...
16:38 markvoelker I think you left off the "#" on line 13
16:38 markvoelker in all.pp
16:38 fvollero markvoelker: yeah
16:38 fvollero markvoelker: already committed the new change
16:39 openstackgerrit Francesco Vollero proposed a change to stackforge/puppet-swift: Support for allow_versions in swift containers  https://review.openstack.org/76253
16:39 fvollero oh, finally it popped up
16:44 fandi joined #puppet-openstack
16:57 thumpba joined #puppet-openstack
17:04 mmagr joined #puppet-openstack
17:06 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-nova: enable libvirt at boot  https://review.openstack.org/75878
17:07 EmilienM fvollero: done
17:07 fvollero EmilienM: Thank you
17:08 openstackgerrit A change was merged to stackforge/puppet-swift: Support for allow_versions in swift containers  https://review.openstack.org/76253
17:12 otherwiseguy joined #puppet-openstack
17:14 marun joined #puppet-openstack
17:16 ryanycoleman joined #puppet-openstack
17:25 openstackgerrit Francesco Vollero proposed a change to stackforge/puppet-swift: Support for allow_versions in swift containers  https://review.openstack.org/76266
17:28 hogepodge joined #puppet-openstack
17:28 fvollero EmilienM: markvoelker : that's the backport :) :)
17:38 bauzas joined #puppet-openstack
17:50 kitp joined #puppet-openstack
17:58 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-neutron: Migrate log_{file,dir} params to neutron class  https://review.openstack.org/76290
17:58 ryanycoleman joined #puppet-openstack
18:00 openstackgerrit A change was merged to stackforge/puppet-swift: Support for allow_versions in swift containers  https://review.openstack.org/76266
18:01 mjblack joined #puppet-openstack
18:04 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-horizon: Fixed Horizon SSL issues  https://review.openstack.org/69511
18:05 ryanycoleman joined #puppet-openstack
18:08 openstackgerrit Donald Talton proposed a change to stackforge/puppet-nova: Added the option to create the nova user before installation  https://review.openstack.org/76298
18:09 dprince_ joined #puppet-openstack
18:11 rmoe joined #puppet-openstack
18:11 sputnik13net joined #puppet-openstack
18:14 ryanycoleman joined #puppet-openstack
18:15 andi-_ joined #puppet-openstack
18:18 rmoe_ joined #puppet-openstack
18:20 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet_openstack_builder: Support to enable ssl in keystone  https://review.openstack.org/76302
18:21 xarses joined #puppet-openstack
18:23 dtalton joined #puppet-openstack
18:26 mfisch where is the official bug tracker for puppet_openstack_builder?  Here? https://bugs.launchpad.net/openstack-cisco
18:27 kitp_ joined #puppet-openstack
18:30 dgollub joined #puppet-openstack
18:33 markvoelker dvorak: nice to meet you!  Sorry for the late reply, running between meetings today.
18:33 markvoelker mfisch: We've been using that since pretty much anything that affects puppet_openstack_builder winds up affecting COI.
18:33 mfisch markvoelker: thanks
18:35 angdraug joined #puppet-openstack
18:39 ianw joined #puppet-openstack
18:41 fvollero markvoelker: EmilienM : thanks a lot for the quick reviews, i really appreciate it
18:46 openstackgerrit Daneyon Hansen proposed a change to stackforge/puppet_openstack_builder: Fixes rabbit_hosts entries in heat.conf  https://review.openstack.org/76314
18:55 openstackgerrit Daneyon Hansen proposed a change to stackforge/puppet_openstack_builder: Adds Support for bind_host of Heat Services  https://review.openstack.org/76319
18:56 csschwe joined #puppet-openstack
19:05 hogepodge joined #puppet-openstack
19:16 ryanycoleman joined #puppet-openstack
19:22 openstackgerrit A change was merged to stackforge/puppet-nova: enable libvirt at boot  https://review.openstack.org/75878
19:23 xarses joined #puppet-openstack
19:23 ryanycoleman joined #puppet-openstack
19:23 csschwe_ joined #puppet-openstack
19:25 openstackgerrit A change was merged to stackforge/puppet-cinder: volume/rbd: Re-enable conditional for string parameters  https://review.openstack.org/74378
19:30 openstackgerrit A change was merged to stackforge/puppet-nova: Ensure that pip is installed before using it.  https://review.openstack.org/74630
19:34 openstackgerrit Clayton O'Neill proposed a change to stackforge/puppet_openstack_builder: Remove git remote references  https://review.openstack.org/76337
19:34 openstackgerrit A change was merged to stackforge/puppet-nova: Fix usage of double quote instead of simple quote  https://review.openstack.org/76148
19:34 openstackgerrit A change was merged to stackforge/puppet-nova: Fix usage of double quote instead of simple quote  https://review.openstack.org/76146
19:34 ryanycoleman joined #puppet-openstack
19:35 dgollub joined #puppet-openstack
19:39 openstackgerrit Daneyon Hansen proposed a change to stackforge/puppet-heat: Fixes the EC2 Auth Token Settings  https://review.openstack.org/76339
19:40 ryanycoleman joined #puppet-openstack
19:42 csschwe joined #puppet-openstack
20:05 kkirkpatrick joined #puppet-openstack
20:10 ryanycoleman joined #puppet-openstack
20:10 openstackgerrit Daneyon Hansen proposed a change to stackforge/puppet_openstack_builder: Adds Support for the Heat EC2 Auth URI  https://review.openstack.org/76346
20:16 ianw joined #puppet-openstack
20:29 thumpba joined #puppet-openstack
20:34 EmilienM fvollero: anytime ;)
20:35 hogepodge joined #puppet-openstack
20:48 EmilienM hogepodge: hey, could you have a look at my comment on https://review.openstack.org/76290 please?
20:48 sbadia iop
20:49 openstackgerrit Daneyon Hansen proposed a change to stackforge/puppet-heat: Aligns Keystone auth_uri with other OpenStack Services  https://review.openstack.org/76356
20:51 hogepodge Just removing the parameter would be fine with me. Same as a fail, with less code. I just want us to use the appropriate language, and API breaks are fine with me since we have time to correct for them.
20:52 EmilienM sbadia: sounds like a plan ^
20:52 sbadia ok, i agree with that
20:52 markvoelker hogepodge: on 76099, what sort of doc pointers did you have in mind?
20:54 markvoelker Add something like this to the header? http://docs.openstack.org/havana/config-reference/content/list-of-compute-config-options.html
20:54 hogepodge Maybe a link? I'm not beholden to it, I'm mainly throwing it out there for comment.
20:54 markvoelker I'm a tad loathe to use URL's like that b/c (for example), that one is specific to havana.
20:54 ryanycoleman joined #puppet-openstack
20:55 hogepodge Yeah. Maybe it doesn't matter since those lists are out there anyway.
20:55 openstackgerrit A change was merged to stackforge/puppet-neutron: Ensure Neutron DB is populated  https://review.openstack.org/70708
20:55 hogepodge Boy, there's a bit backlog of reviews.
20:56 markvoelker hogepodge: I'd also considered something like https://github.com/openstack/nova/blob/master/etc/nova/nova.conf.sample#L2869-L2871 but same problem really...
20:56 hogepodge I need to be better about getting there daily even when I'm feeling swamped here.
20:56 EmilienM hogepodge: i'm gonna backport 70708 to H
20:56 openstackgerrit A change was merged to stackforge/puppet-nova: Add auth_version option in nova::api  https://review.openstack.org/75656
20:57 markvoelker hogepodge: I'm definitely not opposed to having some convention for this though....perhaps we should take it the ML?
20:58 hogepodge Yeah, I think that's a good idea. It will also spark some conversation about when we should add new parameters and when we should just have people add them manually through the conf providers. I know that Dan P. has mentioned parameter creep previously.
20:58 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-neutron: Ensure Neutron DB is populated  https://review.openstack.org/76358
20:58 markvoelker hogepodge: our metric has generally been: add it when a customer asks for it. =)
21:04 ryanycoleman joined #puppet-openstack
21:05 markvoelker hogepodge: Ok, I'll drum up a conversation on the ML.
21:05 markvoelker hogepodge: I am probably going to need that patch (and the forthcoming backport) in the next couple of days though, so may take you up on that +2 offer if the conversation is slow. =)
21:08 mgagne EmilienM: sorry for late review but https://review.openstack.org/#/c/76358/
21:09 mgagne hogepodge: About parameter overpopulation, I would be open to add a configs parameter where you can inject whatever configs you want as long as the config isn't managed by puppet already.
21:09 mgagne hogepodge: and you pipe the values/configs to create_resource and call it a day
21:10 mgagne hogepodge: https://github.com/jfryman/puppet-nginx/blob/master/manifests/init.pp#L58-L60
21:10 mgagne hogepodge: https://github.com/jfryman/puppet-nginx/blob/master/manifests/init.pp#L148-L150
21:14 EmilienM mgagne: your concern is only for the backport, right? (https://review.openstack.org/#/c/76358/)
21:14 mgagne EmilienM: I'm late, it wasn't only for backport
21:14 mgagne EmilienM: I had serious issue when upgrading neutron database
21:15 EmilienM damn
21:15 mgagne EmilienM: partially due to the fact the database version wasn't stamped
21:16 EmilienM any suggestion? otherwise I'll have a look tomorrow
21:17 mgagne EmilienM: I don't have a suggestion unless we purposely crash catalog compilation if database isn't stamped so the guy has to do it beforehand
21:18 ryanycoleman joined #puppet-openstack
21:20 EmilienM mhh, neutron db is a pain, a lot of bugs are running
21:30 hogepodge mgagne the nova config kind of does that, but do you want it more automated in the init class?
21:32 mgagne hogepodge: nova_config is a define. I can argue that people can already manage nova_config in their own manifests and stop bothering us with changes to nova::init.
21:32 mgagne hogepodge: but they want to be able to manage them through Hiera.
21:32 hogepodge ah. good point.
21:33 mgagne hogepodge: So my suggestion is to add a nova_configs parameter to nova::init so they can now manage them through Hiera and stop adding a plethora of parameters each time to discover a new config they would like to manage.
21:34 hogepodge I think that makes sense. A way to make everyone happy. Plus, fewer configs means fewer specs to write, etc.
21:34 mgagne hogepodge: and same with with nova_paste_api_ini so all bases are covered
21:35 openstackgerrit Donald Talton proposed a change to stackforge/puppet-nova: Added the option to create the nova user before installation  https://review.openstack.org/76298
21:45 sanman joined #puppet-openstack
21:47 lnxnut_ joined #puppet-openstack
21:48 sbadia hum
21:48 sbadia I've an strange issue
21:48 sbadia http://logs.openstack.org/98/76298/2/check/gate-puppet-nova-puppet-unit-2.7/a6e2ba4/console.html
21:49 sbadia but for me it's for neutron specs ("o'! at line 1 on)
21:49 xarses dachary: poking around your open commits
21:49 xarses on puppet-ceph
21:51 xarses sbadia: which review is this?
21:52 sbadia xarses: any
21:52 sbadia this log is for https://review.openstack.org/76298
21:52 sbadia but i think it's a bug in puppetlabs or stackforge upstream fixtures
21:52 sbadia (in my fresh local fixtures I've the issue)
21:54 xarses based on that its "at line 3" with not much describing what's line 3 id guess its what ever is used as the "site.pp" file that puppet is attempting to apply
21:54 sbadia for example https://review.openstack.org/#/c/76358/ works at 10PM but now no
21:56 sbadia bingo !
21:56 sbadia git://github.com/enovance/puppetlabs-inifile
21:56 sbadia was broken…
21:57 sbadia sorry git://github.com/puppetlabs/puppetlabs-inifile
21:58 mgagne what a waste of time...
21:58 sbadia https://travis-ci.org/puppetlabs/puppetlabs-inifile/jobs/19609590
21:58 sbadia mgagne: yep…
21:58 xarses hmm, fun
21:58 mgagne upstream should be more careful
21:59 mgagne that one too: https://github.com/puppetlabs/puppetlabs-inifile/issues/76
21:59 sbadia pffff…
22:01 mgagne if you haven't guessed, I'm not really in a good mood today ^^'
22:01 sbadia :-)
22:04 lnxnut joined #puppet-openstack
22:14 fvollero mgagne: yay!!! luckily i dont need review for today :) Pfiuuuuu
22:14 mgagne fvollero: don't worry =)
22:14 fvollero mgagne: :)
22:15 fvollero mgagne: anyway we know that they're using vim :)
22:15 mgagne fvollero: ^^'
22:26 lnxnut joined #puppet-openstack
22:26 xarses sbadia fvollero: either of you want to find a version to pin that works?
22:27 ryanycoleman joined #puppet-openstack
22:28 fvollero xarses: of inifile? Sorry man but i didn't followed the whole conversation
22:28 JonnyNomad joined #puppet-openstack
22:28 xarses fvollero: sorry, thought you where poking around it too
22:32 fvollero xarses: anyway, sure thing, I can investigate the issue, but tomorrow, i just reached 12 hours of work
22:35 xarses I posted an issue on inifile https://github.com/puppetlabs/puppetlabs-inifile/issues/79
22:36 xarses it looks like https://github.com/puppetlabs/puppetlabs-inifile/commit/c1bc9d056aebf02794a65c441efffdcffc3c32d8 first previous is stable
22:37 ryanycol_ joined #puppet-openstack
22:45 sbadia re
22:45 sbadia xarses: yep we must
22:45 sbadia but it's a bit annoying
22:46 sbadia because ALL openstack modules use inifile :(
22:46 xarses seriously, we will have to set the commit id in each repo, until its fixed
22:46 ryanycoleman joined #puppet-openstack
22:46 xarses I think it might be easier to lean on the puppet labs guy and unbreak inifile
22:46 sbadia i'll submit a PR on inifile repo
22:47 xarses sbadia: he might have just fixed it
22:48 xarses he just closed my bug, and added some comma's over the file he modified
22:48 sbadia \o/ indeed
22:48 sbadia ^^
22:50 xarses its starting to pass its own travis tests
22:50 xarses we should be able to re-run them now
22:50 xarses recheck-nobug?
22:50 sbadia yep
22:53 xarses "recheck no bug"
22:53 xarses mine are good now =)
22:59 openstackgerrit A change was merged to stackforge/puppet-ceph: Added the content of the ceph Blueprint in the USECASES.md and linked it in the README.md  https://review.openstack.org/71033
23:00 openstackgerrit A change was merged to stackforge/puppet-ceph: add the mon string in class names  https://review.openstack.org/74038
23:13 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-neutron: Migrate log_{file,dir} params to neutron class  https://review.openstack.org/76290
23:23 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Expose controller_public_protocol parameter  https://review.openstack.org/75858
23:28 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-neutron: Migrate log_{file,dir} params to neutron class  https://review.openstack.org/76290
23:46 michchap hogepodge: Parameter creep was a big problem with puppet-openstack. I think we can avoid it in the modules for services simply by having more classes. The conf files are generally sectioned, so it's possible we could even auto-generate the missing conf parameters for each module if we wanted to.
23:50 michchap markvoelker: on linking to docs, another option is to go through the current docstrings and where it is a conf setting replace them with whatever the comment is in the relevant conf file verbatim.

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