Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2014-05-27

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

All times shown according to UTC.

Time Nick Message
00:00 michchap_ markvoelker: ping
00:11 aldavud joined #puppet-openstack
00:19 dvorak what is the etiquette around cherry-picking other people's changes to stable branches?  For example, Chris Ricker has been submitting several changes to normalize mysql charset encoding that I'd like in the icehouse branch.  Should I ask him to do that, or is it kosher to submit that myself?
00:22 michchap_ dvorak: I want to say just do it.
00:22 michchap_ also dvorak: possibly of interest to you: http://terrible-musk-ox-5211.vag​rantshare.com/docs/mappings.html
00:23 dvorak btw, did you see the patch that came across the puppet-users list last week about embedded hiera lookups for arrays and hashes?
00:23 michchap_ yep, it's under review atm
00:24 michchap_ https://github.com/puppetlabs/hiera/pull/188
00:26 dvorak does the Change-Id change for a cherry-picked change?
00:27 michchap_ should just be cherry-pick -x $commit_id
00:28 openstackgerrit Clayton O'Neill proposed a change to stackforge/puppet-nova: Default to utf8 for MySQL  https://review.openstack.org/95612
00:30 dvorak ah, so I guess since the change-id doesn't change, people that that are playing the stackalytics game still get credit for their change.
00:36 thumpba joined #puppet-openstack
00:36 michchap_ I'm not sure how much stackalytics matters here: the puppet repos aren't part of an official project, so the numbers are even less meaningful.
00:41 dvorak yeah, I'm new to all this, but it seems a bit surprising to me that the puppet repos aren't
00:56 michchap_ the party line is 'TripleO is the official deployment project'
01:01 dvorak heh
01:41 alugovoi joined #puppet-openstack
02:26 alugovoi joined #puppet-openstack
03:32 thumpba joined #puppet-openstack
04:13 openstackgerrit Mike Dorman proposed a change to stackforge/puppet-keystone: Allow blank email address for keystone_user type  https://review.openstack.org/93588
04:23 openstackgerrit Mike Dorman proposed a change to stackforge/puppet-keystone: Allow blank email address for keystone_user type  https://review.openstack.org/93588
04:45 xavpaice michchap_: that puppet-lint change works for me
04:47 sanjayu joined #puppet-openstack
04:48 michchap_ xavpaice: cool. We'll see if the maintainer gets a chance to look at it
04:48 michchap_ xavpaice: unfortunately I realised I've invented some extra style requirements
04:48 xavpaice given the doc check is so strict, I expect the maintainer would at least want it to be an optional check :)
04:48 michchap_ xavpaice: the style guide makes no mention of (optional|required) or of mentioning the default on a new line
04:49 michchap_ xavpaice: in fact I now have half a mind to remove those two, since I can get them from the code easily enough, and remove them from all our manifests
04:50 michchap_ xavpaice: that way when I fix the doc generation it will be generally applicable.
04:50 xavpaice who maintains the style guide? :)
04:51 michchap_ puppetlabs
04:51 michchap_ http://docs.puppetlabs.com/guid​es/style_guide.html#puppet-doc
05:06 sanjayu joined #puppet-openstack
05:11 sputnik13 joined #puppet-openstack
05:28 starmer joined #puppet-openstack
05:35 EmilienM good morning
05:51 aldavud joined #puppet-openstack
06:01 openstackgerrit Ivan Chavero proposed a change to stackforge/puppet-ceilometer: Fixes agent_notification_service_name  https://review.openstack.org/95519
06:18 sputnik13 joined #puppet-openstack
06:31 dgollub joined #puppet-openstack
06:38 ricardo joined #puppet-openstack
06:43 berendt joined #puppet-openstack
06:47 aldavud joined #puppet-openstack
06:53 artem_panchenko joined #puppet-openstack
07:33 sputnik13 joined #puppet-openstack
08:16 aldavud joined #puppet-openstack
08:16 aldavud good morning
08:17 derekh_ joined #puppet-openstack
08:22 berendt joined #puppet-openstack
08:27 qba73 joined #puppet-openstack
08:39 dgollub joined #puppet-openstack
08:49 berendt joined #puppet-openstack
08:50 mmagr joined #puppet-openstack
09:30 admin0 joined #puppet-openstack
10:28 admin0 left #puppet-openstack
10:28 admin0 joined #puppet-openstack
10:50 fritzroy15 joined #puppet-openstack
10:52 fritzroy15 hello, openstack newbie here, I'm getting the timed out waiting for nova-conductor is the service running when I boot my nova-compute node on windows, here is the nova config on it: http://pastebin.com/Bw5rKPaL ; the one on the controller node is default...any tips where I can start debugging this?
11:12 comptona joined #puppet-openstack
11:30 fritzroy15 anyone?
11:32 michchap_ fritzroy15: did you mean to ask in the #openstack channel? This channel is just for puppet-openstack.
11:33 fritzroy15 michchap_ whoops...that's correct, thanks :)
11:33 mattymo fritzroy15, check your RPC logs for errors on both controller and compute
11:34 mattymo as a starting point
11:40 comptona joined #puppet-openstack
11:49 berendt joined #puppet-openstack
11:55 dgollub joined #puppet-openstack
11:58 blentz joined #puppet-openstack
12:02 blentz joined #puppet-openstack
12:14 markvoelker joined #puppet-openstack
12:17 comptona joined #puppet-openstack
12:18 dprince joined #puppet-openstack
12:25 dvorak joined #puppet-openstack
12:26 dprince joined #puppet-openstack
12:41 JayJ joined #puppet-openstack
12:48 rmoe joined #puppet-openstack
13:03 britthouser joined #puppet-openstack
13:33 csschwe joined #puppet-openstack
13:34 aldavud dachary: are you around?
13:42 rharrison joined #puppet-openstack
13:47 jehb joined #puppet-openstack
13:47 prad_ joined #puppet-openstack
13:52 comptona joined #puppet-openstack
14:09 dachary aldavud: yes ?
14:10 * dachary lags a little ;-)
14:11 comptona joined #puppet-openstack
14:13 aldavud dachary: :)
14:13 aldavud dachary: I'm almost ready with the patch for https://review.openstack.org/#/c/94287/
14:14 aldavud dachary: I was wondering if I should submit all the needed stuff into a single patch, or if I should create a set of dependant patches for each of mon, key and osd
14:15 dmsimard aldavud: different patches, please. Easier to review and work on if need be
14:15 dmsimard https://review.openstack.org/#/c/94195/ took me a good while :)
14:16 aldavud ;)
14:17 aldavud the issue is, that the integration tests would fail on all but the last
14:17 dmsimard Wouldn't each have it's own set of tests ? Or are they dependant ?
14:17 aldavud it's a bit of chain
14:18 dmsimard You can commit them as separate patches but dependant of each other
14:18 dmsimard You do that by submitting a patch on top of a patch
14:18 dmsimard e.g, git checkout -b huge_change
14:19 dmsimard *patch 1* git review *patch 2* git review
14:19 dmsimard patch2 will have patch1's code and will be marked as dependant on patch1
14:19 dmsimard and won't get merged unless patch1 is
14:19 aldavud yes, that's what i planned to do
14:19 aldavud but patch1 will have failing CI tests
14:20 openstackgerrit A change was merged to stackforge/puppet-ceph: define ceph::pool.  https://review.openstack.org/86491
14:20 aldavud and only patch 2 will fix them
14:20 dmsimard hmm, dachary won't like that I think :p
14:26 dachary dmsimard: :-)
14:26 dmsimard aldavud: Eh, if anything, submit the two very dependant patches in a single commit
14:26 dachary aldavud: I'm happy with a patch series where the head passes integration even if the intermediate do not.
14:27 dachary as dmsimard mentionned, separate patches are easier to review. And as long as we can rely on the head to pass integration tests, we have no reason to frown upon intermediate failures ;-)
14:28 aldavud ok. thank you :)
14:29 sileht joined #puppet-openstack
14:39 sileht joined #puppet-openstack
14:46 alugovoi joined #puppet-openstack
14:48 kkirkpatrick joined #puppet-openstack
14:51 mdorman joined #puppet-openstack
14:51 sileht joined #puppet-openstack
14:54 jehb joined #puppet-openstack
15:27 starmer joined #puppet-openstack
15:28 openstackgerrit Chris Ricker proposed a change to stackforge/puppet-glance: Fix could not find user glance on ensure files  https://review.openstack.org/95800
15:33 jehb joined #puppet-openstack
15:37 starmer joined #puppet-openstack
15:37 alugovoi joined #puppet-openstack
15:52 dgollub joined #puppet-openstack
15:56 shakamunyi joined #puppet-openstack
15:57 rmoe joined #puppet-openstack
16:00 imcsk8 joined #puppet-openstack
16:07 cwolferh_ joined #puppet-openstack
16:08 imcsk8 hello guys i'm having problems with the firewall module in fedora 20 and rhel 7, i've been looking around and it does not appear to support firewalld, is there any module that supports it??
16:12 larsks imcsk8: I am not aware of a module with good support for firewalld (and certainly not one that integrates with the other puppte openstack modules).
16:30 starmer joined #puppet-openstack
16:31 dachary joined #puppet-openstack
16:54 imcsk8 joined #puppet-openstack
16:54 imcsk8 larsks: well i guess i'm gonna have to take a shot with it
16:57 hogepodge joined #puppet-openstack
16:58 aimon joined #puppet-openstack
17:03 badiane_ka joined #puppet-openstack
17:03 starmer joined #puppet-openstack
17:04 mmaglana joined #puppet-openstack
17:19 openstackgerrit Chris Ricker proposed a change to stackforge/puppet-swift: Fix swift quota filter names  https://review.openstack.org/95832
17:20 openstackgerrit Chris Ricker proposed a change to stackforge/puppet-swift: Fix swift quota filter names  https://review.openstack.org/95833
17:29 dtalton joined #puppet-openstack
17:29 openstackgerrit Ivan Chavero proposed a change to stackforge/puppet-ceilometer: Fixes agent_notification_service_name  https://review.openstack.org/95519
17:30 imcsk8 joined #puppet-openstack
17:31 jehb joined #puppet-openstack
17:31 xarses joined #puppet-openstack
17:33 alugovoi joined #puppet-openstack
17:34 abramley left #puppet-openstack
17:52 openstackgerrit A change was merged to stackforge/puppet-glance: Fix could not find user glance on ensure files  https://review.openstack.org/93921
18:41 mgagne Hunner or hogepodge: ping
18:42 hogepodge pong
18:42 mgagne hogepodge: how is the lib/ folder in modules supposed to work? Is it only added to the search path of ruby and the structure happens to fit with what puppet expects?
18:44 JayJ joined #puppet-openstack
18:45 mgagne hogepodge: if I create lib/zoombie/dead.rb in Module A, will it be found by Module B requiring such file?
18:48 Hunner mgagne: lib/ in modules should be added to the rubypath for the puppet process, so if something in module b does a `require 'zoombie/dead'` it should find it, yes
18:49 mgagne Hunner: great news then
18:49 Hunner Additionally, the lib/ dirs in the modules only exist on the master, but they are copied to the ruby lib path on the agent all together
18:49 Hunner You can run `puppet agent --configprint libdir` to see where that is
18:49 Hunner So they won't actually be in separate lib dirs on the agent
18:50 mgagne Hunner: how are name conflicts handled? lib dirs are stacked together and the last one win?
18:54 sputnik13 joined #puppet-openstack
18:55 Hunner mgagne: Its either the last or the first; I forget
18:55 mgagne Hunner: alright thanks
18:58 markvoelker joined #puppet-openstack
19:02 derekh_ joined #puppet-openstack
19:16 openstackgerrit Chris Ricker proposed a change to stackforge/puppet-openstack: Add account and container quotas in swift pipeline  https://review.openstack.org/95867
19:19 hogepodge mgagne assume undefined (when behavior should be ‘fail’ like a good library linker would do)
19:20 openstackgerrit Chris Ricker proposed a change to stackforge/puppet-swift: Add dependency so swift config creation succeeds  https://review.openstack.org/95868
19:25 Hunner That would probably be a good ticket to make; fail if pluginsync syncs two files to the same path
19:26 EmilienM hello there
19:48 openstackgerrit Donald Talton proposed a change to stackforge/puppet_openstack_builder: WIP: Add Cobbler support for baremetal provisioning  https://review.openstack.org/95232
19:49 jehb joined #puppet-openstack
19:53 hogepodge Hi EmilenM
19:53 hogepodge I haven’t forgotten about your patch.
19:54 hogepodge I’m just getting over my conference cold (slept for 17 hours straight Saturday night)
20:12 kkirkpatrick joined #puppet-openstack
20:13 imcsk8 joined #puppet-openstack
20:36 mgagne EmilienM: is there a place/street/city named kraken near Paris or in France? :D
20:36 toabctl joined #puppet-openstack
20:36 toabctl joined #puppet-openstack
20:37 EmilienM mgagne: not afik
20:37 mgagne EmilienM: damn! :D
20:39 sbadia :D kraken, the legendary giant sea monster ?!
20:40 mgagne :D
20:42 EmilienM lol
21:27 aldavud joined #puppet-openstack
21:28 openstackgerrit David Gurtner proposed a change to stackforge/puppet-ceph: Added support for RedHat in ceph::mon  https://review.openstack.org/94287
21:33 openstackgerrit David Gurtner proposed a change to stackforge/puppet-ceph: Added support for RedHat in ceph::mon  https://review.openstack.org/94287
21:50 openstackgerrit David Gurtner proposed a change to stackforge/puppet-ceph: Added support for RedHat in ceph::mon  https://review.openstack.org/94287
21:54 openstackgerrit David Gurtner proposed a change to stackforge/puppet-ceph: Added support for RedHat in ceph::mon  https://review.openstack.org/94287
22:01 openstackgerrit David Gurtner proposed a change to stackforge/puppet-ceph: Added support for RedHat in ceph::keys CI tests  https://review.openstack.org/94287
22:10 shakamunyi joined #puppet-openstack
22:14 mmaglana joined #puppet-openstack
22:16 openstackgerrit David Gurtner proposed a change to stackforge/puppet-ceph: Fixed removal command in ceph::osd  https://review.openstack.org/95955
22:20 openstackgerrit A change was merged to stackforge/puppet-glance: Fix could not find user glance on ensure files  https://review.openstack.org/95800
22:29 imcsk8 joined #puppet-openstack
23:01 xarses joined #puppet-openstack
23:05 openstackgerrit David Gurtner proposed a change to stackforge/puppet-ceph: Added support for RedHat in ceph::mon  https://review.openstack.org/95962
23:10 hogepodge Naming for K is going to be tough.
23:10 hogepodge I’m expecting a new rule exception.
23:23 openstackgerrit David Gurtner proposed a change to stackforge/puppet-ceph: Added support for RedHat in ceph::mon  https://review.openstack.org/95962
23:32 michchap joined #puppet-openstack
23:44 mgagne hogepodge: looks like they have good candidates =)
23:44 hogepodge mgagne I was just looking at the list. I would say “good” rather than good. ;-)
23:45 hogepodge As in “thank goodness they named a street after Kepler”.
23:45 mgagne hogepodge: haha
23:45 mgagne hogepodge: if for you "good" is a name not starting with the letter "K", I don't know what to say :P
23:46 hogepodge I thought about modifying the wiki to say “astrologer”.
23:47 hogepodge To my eye there’s only one name on that entire list that makes sense (aforementioned astronomer)
23:47 mgagne sounds like people like Kepler =)
23:47 openstackgerrit David Gurtner proposed a change to stackforge/puppet-ceph: Changupport for RedHat in ceph::mon  https://review.openstack.org/94287
23:47 dvorak yeah, Kepler is a good name
23:49 openstackgerrit David Gurtner proposed a change to stackforge/puppet-ceph: Support for RedHat in ceph::mon  https://review.openstack.org/94287
23:49 openstackgerrit David Gurtner proposed a change to stackforge/puppet-ceph: Fixed removal command in ceph::osd  https://review.openstack.org/95955

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