Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2014-04-22

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

All times shown according to UTC.

Time Nick Message
00:13 openstackgerrit Lars Kellogg-Stedman proposed a change to stackforge/puppet-neutron: Neutron Nova Interactions Support  https://review.openstack.org/88410
00:17 openstackgerrit Lars Kellogg-Stedman proposed a change to stackforge/puppet-neutron: Neutron Nova Interactions Support  https://review.openstack.org/88410
00:38 morazi joined #puppet-openstack
00:40 rmoe joined #puppet-openstack
01:16 openstackgerrit Lars Kellogg-Stedman proposed a change to stackforge/puppet-neutron: Neutron Nova Interactions Support  https://review.openstack.org/88410
01:21 larsks EmilienM markvoelker I've finally wrapped my head around provider testing.  There are now new spec tests -- that are passing -- that test the new provider functionality.
01:31 xarses joined #puppet-openstack
01:43 iheartwhiskey joined #puppet-openstack
02:02 xingchao joined #puppet-openstack
02:27 thumpba_ joined #puppet-openstack
02:57 starmer joined #puppet-openstack
03:05 thumpba_ joined #puppet-openstack
03:10 Aimon joined #puppet-openstack
03:13 Aimon joined #puppet-openstack
03:15 Aimon joined #puppet-openstack
03:18 Aimon joined #puppet-openstack
03:56 openstackgerrit Xingchao Yu proposed a change to stackforge/puppet-ceilometer: Introduce ceilometer:config to manage custom options  https://review.openstack.org/84999
04:00 iheartwhiskey joined #puppet-openstack
04:28 thumpba_ joined #puppet-openstack
04:37 openstackgerrit A change was merged to stackforge/puppet-openstack: Fixed typos in readme  https://review.openstack.org/83530
05:06 sputnik1_ joined #puppet-openstack
05:10 EmilienM good morning
05:10 EmilienM larsks: ok, reviewing it today, (yesterday was off here)
05:18 sanjayu joined #puppet-openstack
05:23 openstackgerrit Loic Dachary proposed a change to stackforge/puppet-ceph: ceph::mon : remove useless comments from tests  https://review.openstack.org/89476
05:23 openstackgerrit Loic Dachary proposed a change to stackforge/puppet-ceph: ceph::mon : tag cephx spec test  https://review.openstack.org/89477
05:23 openstackgerrit Loic Dachary proposed a change to stackforge/puppet-ceph: ceph::mon : control client.admin key generation  https://review.openstack.org/89351
05:32 m0ikz joined #puppet-openstack
05:51 m0ikz joined #puppet-openstack
05:52 uberj joined #puppet-openstack
05:54 uberj_ joined #puppet-openstack
05:55 nigelk joined #puppet-openstack
06:02 mkoderer joined #puppet-openstack
06:10 sputnik1_ joined #puppet-openstack
06:15 tfz joined #puppet-openstack
06:21 comptona joined #puppet-openstack
06:53 sanjayu joined #puppet-openstack
06:54 starmer joined #puppet-openstack
07:05 openstackgerrit Sergey Vasilenko proposed a change to stackforge/puppet-keystone: Make service provider as changeble incoming parameter.  https://review.openstack.org/86551
07:16 tfz joined #puppet-openstack
07:21 openstackgerrit Loic Dachary proposed a change to stackforge/puppet-ceph: ceph::key : add keyring and id key injection arguments  https://review.openstack.org/89258
07:27 qba73 joined #puppet-openstack
07:28 openstackgerrit Loic Dachary proposed a change to stackforge/puppet-ceph: update the first use case to reflect that it works  https://review.openstack.org/89356
07:31 dachary EmilienM: morning
07:32 EmilienM dachary: hey
07:35 artem_panchenko joined #puppet-openstack
07:46 tfz joined #puppet-openstack
07:54 tfz joined #puppet-openstack
08:03 derekh joined #puppet-openstack
08:03 tfz joined #puppet-openstack
08:22 tfz joined #puppet-openstack
08:30 tfz joined #puppet-openstack
08:34 bauzas joined #puppet-openstack
08:42 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-neutron: Neutron Nova Interactions Support  https://review.openstack.org/88410
08:43 EmilienM larsks: with more rspec tests ^
09:02 openstackgerrit Loic Dachary proposed a change to stackforge/puppet-ceph: add missing puppetlabs-inifile to Puppetfile  https://review.openstack.org/89513
09:33 sanjayu joined #puppet-openstack
10:25 artem_panchenko joined #puppet-openstack
10:26 starmer joined #puppet-openstack
11:59 rcrit joined #puppet-openstack
12:00 rcrit left #puppet-openstack
12:09 morazi joined #puppet-openstack
12:19 Mr_D joined #puppet-openstack
12:22 ashp joined #puppet-openstack
12:29 [1]Mr_D joined #puppet-openstack
12:37 EmilienM thanks markvoelker
12:37 EmilienM I'm continuing the work now
12:41 derekh joined #puppet-openstack
12:42 prad joined #puppet-openstack
12:50 openstackgerrit Lars Kellogg-Stedman proposed a change to stackforge/puppet-neutron: Neutron Nova Interactions Support  https://review.openstack.org/88410
12:51 larsks EmilienM markvoelker : That resolves most of markvoelker comments, I think.
12:51 EmilienM larsks: I was on it :-/
12:51 EmilienM like I said 15 min earlier
12:51 EmilienM anyway
12:52 larsks Sorry, just getting on irc now.
12:53 Pax joined #puppet-openstack
12:53 EmilienM larsks: still not good: https://review.openstack.org/#/c/88410/22/lib/puppet/provider/nova_admin_tenant_id_setter/ini_setting.rb
12:53 EmilienM you need a header
12:53 bauzas joined #puppet-openstack
12:53 EmilienM just move the Puppet::Type.type(:nova_admin_tenant_id_setter).provide(:ruby) do
12:53 EmilienM I think
12:54 larsks EmilienM: I didn't follow that.  You mean, those functions should be defined within the puppet type rather than outside of it?
12:55 EmilienM larsks: i think, yes
12:55 larsks EmilienM: I located them outside of the provide class itself because they operate exclusively with paramters rather than relying on any of the resource attributes.
12:56 larsks I'm happy to move them inside the definition, but it seems like the wrong place for them.
12:56 EmilienM mhh maybe markvoelker is around to confirm that
12:56 EmilienM larsks: and you also dropped the Co-Authored-By, would mind to keep it?
12:57 m0ikz joined #puppet-openstack
12:57 larsks You'll note that I didn't even replace it with an authored-by :).  My inclination is to keep my name out of source files (what if someone else changes them?), and rely on the git history to show who made what.  But if it's important to you I will add comments back to...which files?
12:59 dprince joined #puppet-openstack
12:59 EmilienM larsks: I agree to not use names within file names
12:59 larsks EmilienM: So, commit message?
13:00 EmilienM larsks: yes, just keep the commit message with Co-Authored, it makes more sense
13:00 larsks Okay, I will update that.
13:01 EmilienM larsks: thanks
13:01 larsks I'm going to wait and see if markvoelker has an opinion on those utility functions.  Two out of three wins :)
13:14 openstackgerrit Lars Kellogg-Stedman proposed a change to stackforge/puppet-neutron: Neutron Nova Interactions Support  https://review.openstack.org/88410
13:18 openstackgerrit Lars Kellogg-Stedman proposed a change to stackforge/puppet-neutron: Neutron Nova Interactions Support  https://review.openstack.org/88410
13:18 larsks (I rewrote the commit message and fix the co-authored-by: tag.  I apologize for dropping that earlier; I hadn't noticed it)
13:20 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet_openstack_builder: Fix typo in horizon bind address  https://review.openstack.org/89567
13:20 EmilienM larsks: no problem :)
13:20 EmilienM larsks: let's see reviews now
13:22 dinosaurpt joined #puppet-openstack
13:28 kitp joined #puppet-openstack
13:33 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Fix typo in horizon bind address  https://review.openstack.org/89567
13:35 openstackgerrit Lars Kellogg-Stedman proposed a change to stackforge/puppet-neutron: Neutron Nova Interactions Support  https://review.openstack.org/88410
13:51 markvoelker EmilienM: larsks: I don't have a real strong opinion, but I'm ok with it staying outside of the provider class.  Might actually make it slightly more reusable since it doesn't rely on resource attributes.
13:51 larsks markvoelker: That was kind of my thinking.
13:51 larsks markvoelker: I'm addressing your most recent comments right now...
13:52 markvoelker larsks: cool
13:54 larsks markvoelker: re: default for auth_password, I tried nil but got: /home/lkellogg/.gem/ruby/gems/puppet-3.5.1/lib/puppet/parameter.rb:102:in `defaultto': Either a default value or block must be provided (Puppet::DevError)
13:55 larsks markvoelker: I guess "defaultto nil" is really saying "no default".
13:55 larsks So maybe just remove the "defaultto" there.
13:56 markvoelker larsks: Yeah, probably
13:57 openstackgerrit Lars Kellogg-Stedman proposed a change to stackforge/puppet-neutron: Neutron Nova Interactions Support  https://review.openstack.org/88410
14:03 prad joined #puppet-openstack
14:03 markvoelker1 joined #puppet-openstack
14:09 rwsu joined #puppet-openstack
14:11 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet_openstack_builder: Fixes for HA proxy start issue  https://review.openstack.org/89582
14:11 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet_openstack_builder: Fix typo in horizon bind address  https://review.openstack.org/89583
14:17 bauzas joined #puppet-openstack
14:41 marun joined #puppet-openstack
14:48 daneyon joined #puppet-openstack
14:50 xingchao joined #puppet-openstack
14:54 openstackgerrit Christopher Dearborn proposed a change to stackforge/puppet-cinder: Add support for volumes backed by Dell EqualLogic  https://review.openstack.org/89602
14:58 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Fix the mongo connection string to use ip  https://review.openstack.org/88414
14:59 starmer joined #puppet-openstack
15:00 Mr_D joined #puppet-openstack
15:06 openstackgerrit Lukas Bezdicka proposed a change to stackforge/puppet-neutron: Fix neutron subnets with empty values.  https://review.openstack.org/88305
15:16 kkirkpatrick joined #puppet-openstack
15:42 openstackgerrit Lars Kellogg-Stedman proposed a change to stackforge/puppet-nova: support for managing ssh keys  https://review.openstack.org/89620
15:44 EmilienM larsks: nice one :) ^
15:44 larsks EmilienM: This is the first step to https://bugs.launchpad.net/packstack/+bug/1311168
15:45 fvollero larsks: thanks for your contributions :)
15:45 EmilienM larsks: this is how we manage it: https://github.com/enovance/puppet-openstack-cloud/blob/master/manifests/compute/hypervisor.pp#L47-L77
15:45 EmilienM very ugly for now
15:45 EmilienM but your patch will be helpful :)
15:45 larsks Yay :).  And thank you (and markvoelker!) for all your help working through that notifications patch.  That was a lot of new material for me to cover.
15:47 EmilienM larsks: since I'm part of the patch, I can't +A
15:47 EmilienM but I'm sure fvollero will find the time to review it :P
15:47 fvollero EmilienM: Without any dubt :) You'll have time to review 88305 ? :)
15:48 EmilienM maybe
15:48 fvollero EmilienM: :)
15:50 rmoe joined #puppet-openstack
15:53 derekh joined #puppet-openstack
16:02 morazi joined #puppet-openstack
16:10 starmer joined #puppet-openstack
16:11 sputnik1_ joined #puppet-openstack
16:21 saneax_-_ joined #puppet-openstack
16:25 uberj_ is there not an icehouse release yet?
16:26 EmilienM uberj: no, current master supports icehouse
16:26 uberj ah, thanks
16:27 Aimon joined #puppet-openstack
16:29 iheartwhiskey joined #puppet-openstack
16:30 angdraug joined #puppet-openstack
16:31 bauzas1 joined #puppet-openstack
16:31 EmilienM uberj: an official release will happen soon tough
16:32 markvoelker2 joined #puppet-openstack
16:41 tfz joined #puppet-openstack
16:50 morazi joined #puppet-openstack
16:51 xarses joined #puppet-openstack
17:14 marun joined #puppet-openstack
17:36 hogepodge uberj_ I’m starting on the Icehouse release today.
17:37 hogepodge It needs to be integration tested and hava release notes written.
17:37 hogepodge It’s going to take a least a week, more likely two.
17:37 hogepodge Regardless, we’re ahead of the curve on this one. Previous releases have been after the summit.
17:37 hogepodge I’m committed to getting this one out before.
17:41 EmilienM hogepodge: cool :)
17:44 uberj hogepodge: awesome
17:53 openstackgerrit Christopher Dearborn proposed a change to stackforge/puppet-cinder: Add support for volumes backed by Dell EqualLogic  https://review.openstack.org/89602
17:56 [1]Mr_D joined #puppet-openstack
18:01 markvoelker hogepodge: You're definitely going to want that nova interactions patch in Icehouse. =)  It's about ready to land though, so should fit easily in that timeframe.  I'll get rickerc to review it if fvollero doesn't first.
18:02 fvollero markvoelker: which one ?
18:03 markvoelker fvollero: https://review.openstack.org/#/c/88410/
18:03 fvollero markvoelker: because i was looking at the one of lars
18:03 markvoelker That's the one
18:04 fvollero markvoelker: for me it's also +2
18:04 fvollero markvoelker: i'll go to approve it since you already +2 it ?
18:06 markvoelker fvollero: Yes, I think it's good to go.  EmilienM has also been working with it (but abstained from voting since he coauthored bits of it), and I think rickerc did a bit of testing on it too.
18:07 markvoelker fvollero: merge away as far as I'm concerned
18:07 EmilienM markvoelker: it makes sense since I started the code
18:08 markvoelker EmilienM: yep.  Thanks for getting the ball rolling on this one.
18:10 fvollero :)
18:10 fvollero EmilienM: markvoelker : i worked a bit helping this guy here https://review.openstack.org/#/c/88305/ so I would like to not vote on this one
18:11 markvoelker fvollero: Perhaps you and EmilienM could vote +1 if you think it's ready?  I'll round up another core to do the final core review if you'd like to abstain.
18:12 markvoelker oh wait, different patch than I was thinking of...
18:12 markvoelker fvollero: ok, so you need another review on that subnets-with-empty-values patch?
18:13 fvollero markvoelker: yeah, I just helped Lukas on this one because he's a new entry in our team :)
18:13 openstackgerrit A change was merged to stackforge/puppet-neutron: Neutron Nova Interactions Support  https://review.openstack.org/88410
18:13 fvollero openstackgerrit: ORLY? :)
18:13 larsks \o/
18:14 fvollero larsks: you should thanks openstackgerrit as well
18:14 markvoelker heh
18:14 markvoelker fvollero: I'll sign myself up for a review on that one then
18:15 * fvollero is always scared to give +2 && +1 since he fears mgagne
18:15 fvollero s/mgagne/mgagne rage/
18:15 fvollero markvoelker: thanks a bunch Mark :)
18:15 fvollero larsks: if you want you could sign up as well, https://review.openstack.org/#/c/88305/
18:15 larsks Sure.
18:15 larsks I'll take a look.
18:25 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Fixes for HA proxy start issue  https://review.openstack.org/89582
18:25 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Fix typo in horizon bind address  https://review.openstack.org/89583
18:31 openstackgerrit Lars Kellogg-Stedman proposed a change to stackforge/puppet-nova: support for managing ssh keys  https://review.openstack.org/89620
18:47 thumpba joined #puppet-openstack
18:49 openstackgerrit joined #puppet-openstack
19:03 derekh joined #puppet-openstack
19:17 openstackgerrit John Eckersberg proposed a change to stackforge/puppet-nova: Do not set special_service_provider on RHEL7.  https://review.openstack.org/89682
19:19 dprince joined #puppet-openstack
19:21 kkirkpatrick joined #puppet-openstack
19:24 tfz1 joined #puppet-openstack
19:43 Mr_D joined #puppet-openstack
19:46 starmer joined #puppet-openstack
19:53 Pax joined #puppet-openstack
20:15 xarses joined #puppet-openstack
20:19 qba73 joined #puppet-openstack
20:54 dmassey joined #puppet-openstack
20:54 angdraug joined #puppet-openstack
20:56 hogepodge thanks markvoelker
20:56 hogepodge I’m also concerned about ml2
20:57 hogepodge I just wrapped up my neverending Havana work.
20:57 dmassey Does anyone know of any method to convert puppet code/manifest into human readable format?  I have to make a compliance report and I use puppet to configure everything.  It would be awesome if I could generate a report about how each node is configure that my boss can read
21:04 hogepodge joined #puppet-openstack
21:11 xarses joined #puppet-openstack
21:26 rmoe joined #puppet-openstack
21:46 daneyon joined #puppet-openstack
21:50 dmassey Does anyone know of any method to convert puppet code/manifest into human readable format?  I have to make a compliance report and I use puppet to configure everything.  It would be awesome if I could generate a report about how each node is configure that my boss can read?
22:13 derekh joined #puppet-openstack
22:28 dtalton joined #puppet-openstack
22:29 bitblt joined #puppet-openstack
22:29 bitblt joined #puppet-openstack
22:29 bitblt left #puppet-openstack
22:31 dmassey i am new to this IRC, can someone just reply so i know that I acually joined a channel please
22:32 xavpaice dmassey: got your message... but don't have an answer :(
22:33 xavpaice if your boss can't understand puppet code, chances are your bos also won't understand any human readable format either.  Generate a pie chart and tell the boss that it's all OK ;)
22:33 dmassey lol
22:34 rharrison xavpaice, oh I've seen puppet code nobody can understand...
22:34 rharrison xavpaice, especially from people who talk about writing "puppet scripts"...
22:34 [1]Mr_D joined #puppet-openstack
22:35 dmassey its really more for the business guys.  The will not read it that just have to beable to point to something during a compliance review that looks more like words than code
22:35 dmassey *they
22:36 dmassey thanks for replying though, just got a IRC client im no to sure about
22:37 rharrison dmassey, I'd use rspec-puppet or rspec-system for that.  You can write your tests to their compliance docs and show them the results.  i.e. test 1234 (for requirement #1234) = passed
22:38 hogepodge ping markvoelker
22:38 rharrison dmassey, the real upside of this will be you'll know if you (or a jr admin who knows nothing about requirement #1234) ever do something that breaks / violates the requirement.
22:38 hogepodge or michchap or cricker
22:39 dmassey hogepodge: awesome thanks, looking into this right now
22:46 dmassey any ideas on how to list all manifests that apply to a particular node?
22:47 xavpaice dmassey: take a look in /var/lib/puppet/state - there's a ton of great stuff in there
22:52 dmassey xavpaice: thanks, i did not see anyhitng that show per node though, should there be?
22:58 starmer joined #puppet-openstack
23:07 xavpaice oh sorry - that's on the node itself...
23:10 dmassey xavpaice: oh cool thanks, i was looking at the master
23:14 dmassey very nice, /var/lib/puppet/state/resources.txt and /var/lib/puppet/state/last_run_report.yaml could be really helpful for what i need, this might save me alot of time
23:15 xarses joined #puppet-openstack
23:26 xarses joined #puppet-openstack
23:44 bodepd what are the most important days in Atlanta for the puppet-openstack community stuff?

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