Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2013-04-10

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

All times shown according to UTC.

Time Nick Message
00:10 openstackgerrit Dan Bode proposed a change to stackforge/puppet-keystone: Run 'keystone db_sync' as keystone user.  https://review.openstack.org/26154
00:45 openstackgerrit A change was merged to stackforge/puppet-keystone: Run 'keystone db_sync' as keystone user.  https://review.openstack.org/26154
01:55 openstackgerrit Yu Xingchao proposed a change to stackforge/puppet-nova: Fix nova::quota param quota_ram misspelling.  https://review.openstack.org/26564
02:37 openstackgerrit A change was merged to stackforge/puppet-swift: Remove travis test status from README  https://review.openstack.org/26549
05:02 aimon ah we are missing the .gitreview file in puppet-openstack
05:03 openstackgerrit Dan Bode proposed a change to stackforge/puppet-nova: Fix nova::quota param quota_ram misspelling.  https://review.openstack.org/26564
05:04 aimon bodepd_ do I need to set up a gitreview file manually?
05:22 openstackgerrit Dan Bode proposed a change to stackforge/puppet-nova: Add label support to nova::manage::network.  https://review.openstack.org/26486
05:24 aimon nm I got it working
05:27 openstackgerrit Aimon Bustardo proposed a change to stackforge/puppet-openstack: add rabbit_virtual_host passthrough to cinder::base  https://review.openstack.org/26573
05:53 openstackgerrit Dan Bode proposed a change to stackforge/puppet-swift: release 1.0.0  https://review.openstack.org/26575
05:54 dachary joined #puppet-openstack
06:05 bodepd_ aimon: we probably should
06:05 bodepd_ aimon: all of the are missing that
06:05 aimon +1
06:05 aimon makes it easier for everyone
06:05 bodepd_ aimon: can I get you to review: https://review.openstack.org/26575
06:05 aimon sure
06:05 bodepd_ I really need to cut a release of swift before I start merging grizzly specific stuff
06:11 aimon done
06:12 bodepd_ aimon: I just added you to the group.
06:12 aimon ok
06:12 bodepd_ its hte +2 that matters (and you did not have the ability to do that)
06:12 aimon ah ok
06:12 bodepd_ that brings up the review group to 7!
06:12 bodepd_ I hope we have at least 12 active folks by the end of the summit
06:13 aimon nice! Yes 12 is a good solid number
06:13 aimon gave the +2
06:13 bodepd_ thanks!
06:14 bodepd_ I also manually lanuched your submission through CI
06:14 bodepd_ it's a shame that just as soon as I got test automation where I wanted it, it was time to tear down and start over :)
06:16 bodepd_ aimon: the unit tests for that patch do not look complete.
06:16 bodepd_ aimon: have you guys been successful with the modules? Do you have alot of patches you need to get in?
06:16 aimon I have only a few.. most of it looks good so far
06:16 aimon all are minor
06:17 bodepd_ I just noticed that swift is broken on grizzly
06:17 aimon using openstack, nova, glance, cinder, keystone, ceilometer
06:17 bodepd_ that is why I want to merge that last one
06:17 bodepd_ what ceilometer module?
06:17 aimon no swift in grizz yet on our end
06:17 bodepd_ I'm looking at that now
06:18 bodepd_ part of this move to stackforge is to get dprince using the modules with smokestack
06:18 bodepd_ (which is pretty cool)
06:18 aimon enovance/puppet-ceilometer
06:18 bodepd_ so, they will follow master
06:18 bodepd_ cool, I know someone else who gave that a spin
06:19 aimon seems like there are only 2 options.. one is fork of the other
06:19 aimon I grabbed the most active
06:19 aimon So far its working.. havent fully tested all connections
06:19 aimon but I am definately getting glance data.. rest is just plumming
06:20 bodepd_ whose fork?
06:20 bodepd_ and quantum?
06:21 bodepd_ I REALLY wanted to get quantum sorted before the conference, but this stackforge migration may have killed the rest of my time
06:21 aimon I also loked at ekarlso fork of ceilometer
06:22 aimon I am turning on quantum tomorrow.. I ahve rest done at this point
06:22 aimon waiting for QA to do their thing with nova-netowrk enabled , then will conver tit and they can re QA to ensure feature parity
06:22 bodepd_ I have some code I need to push out
06:22 bodepd_ related to quantum.
06:23 bodepd_ I have code that comes REALLY close to deploying it for folsom
06:23 bodepd_ the only thing that did not work was metadata service
06:23 aimon :)
06:23 aimon heh, we dont use the metadata service anyways :p
06:23 bodepd_ I looked at enovance and cisco, and neither seemed to really work
06:23 bodepd_ I'm hoping to get around to that tommorrow
06:24 aimon interesting… I'll definately keep you posted on my findings over next 48 hours
06:24 aimon .. I kleep getting slowed down by nova and cinder bugs
06:24 bodepd_ let me see if I can push some code up
06:25 aimon openstack RCs are more like Betas
06:25 aimon and realease is still pretty rough generally
06:25 aimon better then Folsom so I am pleased we are going in correct direction
06:25 aimon Ubuntu really got their act together this time also on releasing debs on time
06:25 bodepd_ cool
06:26 bodepd_ I've been using their grizzly stuff for a few weeks now
06:26 aimon goig on 2 weeks this week for me
06:27 aimon big issue we have atm is inifile on compute
06:28 aimon it tears up the nova.conf every other run.. leaves half out or enters lines with 'nil'
06:28 aimon are you seeing that issue at all?
06:38 bodepd_ wtc
06:38 bodepd_ wtf
06:38 bodepd_ what version?
06:39 bodepd_ the version of quantum I was using is here: https://github.com/bodepd/pu​ppet-quantum/commits/folsom
06:42 bodepd_ I just force pushed that branch b/c I had to fix something
06:44 bodepd_ qauntum branch of opensack: https://github.com/bodepd/pupp​etlabs-openstack/tree/quantum
06:54 bodepd_ also, this branch: https://github.com/bodepd/puppetla​bs-openstack_dev_env/tree/quantum
06:56 aimon head bodepd_ on cprice/inifile
06:57 aimon what inifile you using?
07:07 bodepd_ the last released version 0.10.0
07:07 bodepd_ I have not seen this issue you are referring to
07:08 bodepd_ when did you start seeing this?
07:09 bodepd_ looks like 0.10.0 is the same as master
07:11 bodepd_ aimon: are you purging?
07:12 aimon bodepd_ yes, I have tried both ways.. it didnt seem to change its behavior.. It always purges
07:13 bodepd_ I recreated
07:13 bodepd_ I can see what the problem is
07:17 aimon well thats good
07:22 bodepd_ aimon: shit. I can see the problem...
07:22 bodepd_ uggh. I wonder if there is an easy fix
07:24 bodepd_ this issue is with purging
07:28 openstackgerrit Mehdi Abaakouk proposed a change to stackforge/puppet-openstack: Change rabbit_host/rabbit_port to rabbit_hosts array  https://review.openstack.org/26577
07:33 bodepd_ aimon: this is the fix: https://github.com/puppetlabs/​puppetlabs-nova/pull/207/files
07:33 bodepd_ well, a variation of that.
07:36 aimon ahh missing header
07:37 bodepd_ yeah, with purging, it can't tell the difference between teh entries
07:37 bodepd_ in teh file, and the things without DEFAULT
07:37 aimon regarding my addirion.. I see what I missed.. nothing in compute spec.. will add
07:37 bodepd_ thus the swapping
07:37 bodepd_ it's a nasty bug, and will be a pain to fix.
07:38 openstackgerrit Mehdi Abaakouk proposed a change to stackforge/puppet-horizon: Fix debian compatibility  https://review.openstack.org/26579
07:41 dachary joined #puppet-openstack
07:53 openstackgerrit Mehdi Abaakouk proposed a change to stackforge/puppet-swift: Allow customise log_facility in storage server configuration  https://review.openstack.org/26581
07:55 openstackgerrit Mehdi Abaakouk proposed a change to stackforge/puppet-swift: Fix swift storage service start on debian  https://review.openstack.org/26582
07:56 openstackgerrit Mehdi Abaakouk proposed a change to stackforge/puppet-swift: Ensure python-swiftclient is installed to use swift-dispersion  https://review.openstack.org/26583
08:00 openstackgerrit Aimon Bustardo proposed a change to stackforge/puppet-cinder: Add nexenta support  https://review.openstack.org/26584
08:01 aimon ^ I submitted a patch to Cinder project to fix the Nexenta driver for that ^^^.  If anyone is using nexenta I can help.
08:02 bodepd_ aimon: https://gist.github.com/bodepd/5352705
08:02 bodepd_ uggh. too much code, can't possible review all...
08:03 bodepd_ aimon: that is the shortest fix I can do.
08:03 openstackgerrit Mehdi Abaakouk proposed a change to stackforge/puppet-swift: Allow to have a different internal address and public address  https://review.openstack.org/26585
08:03 bodepd_ it's hack, its just a few lines of code
08:03 bodepd_ the REAL solution is to globally replace DEFAULT
08:04 bodepd_ see you folks tomorrow, I'm out
09:08 openstackgerrit Aimon Bustardo proposed a change to stackforge/puppet-openstack: add rabbit_virtual_host passthrough to cinder::base  https://review.openstack.org/26589
09:09 aimon 2013-04-10 09:09:03.101 | Failed to clone git://github.com/bodepd/openstack-quantum-puppet into spec/fixtures/modules/quantum
09:21 aimon submitting fix with repointed repos
09:23 openstackgerrit Aimon Bustardo proposed a change to stackforge/puppet-openstack: Repoint quantum and keystone repos to boped/puppet-<foo>  https://review.openstack.org/26590
09:23 aimon ^ That fixes rake
09:27 openstackgerrit Aimon Bustardo proposed a change to stackforge/puppet-openstack: Repoint quantum and keystone repos to bodepd/puppet-<foo>  https://review.openstack.org/26592
10:14 taziden joined #puppet-openstack
10:17 aimon or that ^ ;)
12:33 dprince joined #puppet-openstack
13:02 openstackgerrit Dan Prince proposed a change to stackforge/puppet-cinder: Require cinder package before running sync.  https://review.openstack.org/26398
13:30 d3u joined #puppet-openstack
13:45 dachary joined #puppet-openstack
14:08 openstackgerrit Dan Prince proposed a change to stackforge/puppet-swift: Drop ringbuilder.sh from swift module.  https://review.openstack.org/26212
14:15 d3u joined #puppet-openstack
15:04 openstackgerrit A change was merged to stackforge/puppet-swift: Drop ringbuilder.sh from swift module.  https://review.openstack.org/26212
15:07 openstackgerrit A change was merged to stackforge/puppet-cinder: Require cinder package before running sync.  https://review.openstack.org/26398
15:38 dachary joined #puppet-openstack
15:44 openstackgerrit Dan Prince proposed a change to stackforge/puppet-nova: Add support for configuring qpid.  https://review.openstack.org/26230
15:54 mgagne1 joined #puppet-openstack
15:59 d3u joined #puppet-openstack
16:38 mgagne joined #puppet-openstack
16:47 mgagne joined #puppet-openstack
16:52 dachary joined #puppet-openstack
17:13 openstackgerrit A change was merged to stackforge/puppet-cinder: Allow to use of rabbit_hosts  https://review.openstack.org/26507
17:22 d3u joined #puppet-openstack
17:33 mgagne w00t
18:11 bodepd_ wow, things are getting merged that I didnt even have to look at!
18:11 bodepd_ as an FYI, purging for nova_config is totally hosed
18:13 dprince bodepd_: how would you feel about nuking storeconfigs from the nova modules. They all throw warnings which isn't going to be handled well in my setup (where warnings are considered failures)
18:13 bodepd_ dprince: I am all for it
18:14 bodepd_ there is a bug in puppet, that syntax is checked at parse time
18:14 dprince bodepd_: looking online see several puppet tickets on this topic
18:14 bodepd_ when it should be validated as run time
18:14 dprince bodepd_: exactly.
18:14 bodepd_ kill it
18:14 bodepd_ dprince: I am officially under water with all of this code review
18:14 bodepd_ dprince: gotta start preparing for my talk at some point :)
18:15 bodepd_ to fix puring in nova_config, I cannot support both : nova_config { foo: } and nova_config { DEFAULT/foo }
18:15 bodepd_ I can fix it in two ways:
18:16 dprince I quite using purge. Why do we need it?
18:16 bodepd_ quire section names (like everything else). This will break backwards compat
18:16 dprince And by quite I mean quit.
18:16 bodepd_ it's an option available for anyone who wants to lock down the settings allowed to be in the files
18:16 bodepd_ (I gotcha)
18:16 bodepd_ so that you can say only these things
18:16 bodepd_ for folks with stricter compliance issues, it may be required
18:17 dprince Gotcha. Now I remember.
18:18 bodepd_ basically, I have to fix it by requiring section names, this is a pretty substantial baskwards compat break, but I dont see a way around it
18:18 bodepd_ or dissallow DEFAULT, and say if you want to set DEFAULT, omit section name
18:18 dprince bodepd_: using DEFAULT for all our _config resources is fine w/ me.
18:18 bodepd_ this will maintain backwards compat
18:19 bodepd_ ok, I will start putting this patch together.
18:19 dprince bodepd_: either way is fine actually. Perhaps worth taking this to the list first?
18:19 bodepd_ doing now
18:19 dprince bodepd_: I personally think DEFAULT is clearer. But compat is good too.
18:22 bodepd_ yeah, I thought I could be clever and support both.
18:22 bodepd_ not clever enough :(
18:52 openstackgerrit Dan Bode proposed a change to stackforge/puppet-keystone: update module for stackforge  https://review.openstack.org/26649
18:54 openstackgerrit Dan Prince proposed a change to stackforge/puppet-nova: Update fixtures location to use stackforge.  https://review.openstack.org/26651
18:56 openstackgerrit A change was merged to stackforge/puppet-keystone: update module for stackforge  https://review.openstack.org/26649
18:56 openstackgerrit Dan Prince proposed a change to stackforge/puppet-glance: Update fixtures location to use stackforge.  https://review.openstack.org/26652
18:57 openstackgerrit Dan Prince proposed a change to stackforge/puppet-swift: Update fixtures location to use stackforge.  https://review.openstack.org/26653
18:58 openstackgerrit A change was merged to stackforge/puppet-nova: Update fixtures location to use stackforge.  https://review.openstack.org/26651
18:58 openstackgerrit A change was merged to stackforge/puppet-glance: Update fixtures location to use stackforge.  https://review.openstack.org/26652
18:58 openstackgerrit Dan Prince proposed a change to stackforge/puppet-cinder: Update fixtures location to use stackforge.  https://review.openstack.org/26655
19:00 openstackgerrit A change was merged to stackforge/puppet-swift: Update fixtures location to use stackforge.  https://review.openstack.org/26653
19:00 openstackgerrit A change was merged to stackforge/puppet-cinder: Update fixtures location to use stackforge.  https://review.openstack.org/26655
19:13 bodepd_ dprince: you beat me to those updates
19:15 dprince bodepd_: Ha! Just noticed it this morning when I added some qpid tests.
19:20 openstackgerrit Dan Prince proposed a change to stackforge/puppet-nova: Remove storeconfigs from nova::init.  https://review.openstack.org/26660
19:20 openstackgerrit Dan Prince proposed a change to stackforge/puppet-nova: Remove storeconfigs from nova::db::mysql.  https://review.openstack.org/26661
19:20 openstackgerrit Dan Prince proposed a change to stackforge/puppet-nova: Remove storeconfigs from nova::compute.  https://review.openstack.org/26662
19:36 openstackgerrit Dan Bode proposed a change to stackforge/puppet-nova: fix purging for nova_config  https://review.openstack.org/26664
20:08 bodepd_ aimon: I submitted a patch that fixes the nova purging issue
20:09 bodepd_ aimon: want to try it out?
20:14 openstackgerrit Dan Prince proposed a change to stackforge/puppet-cinder: Renames cinder::base to cinder.  https://review.openstack.org/26499
20:26 openstackgerrit Dan Bode proposed a change to stackforge/puppet-swift: release 1.0.0  https://review.openstack.org/26575
20:28 bodepd_ mgagne: can I get another review. I had to rebase for some reason
20:28 bodepd_ https://review.openstack.org/#/c/26575/
20:30 openstackgerrit A change was merged to stackforge/puppet-swift: release 1.0.0  https://review.openstack.org/26575
20:32 openstackgerrit A change was merged to stackforge/puppet-nova: Fix nova::quota param quota_ram misspelling.  https://review.openstack.org/26564
20:32 openstackgerrit A change was merged to stackforge/puppet-nova: Add label support to nova::manage::network.  https://review.openstack.org/26486
20:37 openstackgerrit Dan Bode proposed a change to stackforge/puppet-swift: Add gitreview  https://review.openstack.org/26669
20:39 openstackgerrit A change was merged to stackforge/puppet-swift: Add gitreview  https://review.openstack.org/26669
20:39 openstackgerrit Dan Bode proposed a change to stackforge/puppet-glance: Add gitreview  https://review.openstack.org/26670
20:40 openstackgerrit A change was merged to stackforge/puppet-glance: Add gitreview  https://review.openstack.org/26670
20:41 openstackgerrit Dan Bode proposed a change to stackforge/puppet-cinder: add gitreview  https://review.openstack.org/26671
20:42 openstackgerrit A change was merged to stackforge/puppet-cinder: add gitreview  https://review.openstack.org/26671
20:44 openstackgerrit Dan Bode proposed a change to stackforge/puppet-nova: add gitreview  https://review.openstack.org/26672
20:44 * mgagne thinks someone should maintain librarian-puppet
20:45 openstackgerrit Dan Bode proposed a change to stackforge/puppet-horizon: add gitreview  https://review.openstack.org/26673
20:46 openstackgerrit Dan Bode proposed a change to stackforge/puppet-openstack: add gitreview  https://review.openstack.org/26674
20:47 openstackgerrit A change was merged to stackforge/puppet-horizon: add gitreview  https://review.openstack.org/26673
20:49 openstackgerrit Dan Bode proposed a change to stackforge/puppet-openstack_dev_env: add gitreview  https://review.openstack.org/26675
21:09 bodepd_ mgagne: maesto-dev forked it and they maintain it
21:09 bodepd_ maestro
21:09 bodepd_ should be able to find it with a quick google search
21:31 aimon mgagne - can you explain a bit further what you mean on https://review.openstack.org/#/c/26584/1​/spec/classes/cinder_volume_nexenta_spec.rb
21:32 aimon There arent an OS specific params in that class...
21:32 aimon *any
21:33 mgagne aimon: try to put garbage in your loop to make your test fails.
21:33 mgagne params isn't about OS specific, it's about params passed to the class.
21:34 aimon I pretty much copied the existing netapp one…
21:35 mgagne it's broken too then =)
21:35 mgagne I'm working on a new class too and found out about this problem
21:36 aimon hmm I want to fix this but haivng problem understanding what you are explaining…
21:36 mgagne I can't officially propose code because of the CLA, waiting on it
21:36 aimon could be the fact that I am still half asleep ;)
21:36 mgagne we are all asleep here ;)
21:37 aimon ah how about pointing me to an example that displays the proper way
21:37 aimon ?
21:37 mgagne http://rspec-puppet.com/tutorial/ ;)
21:39 mgagne To test a class, you need to define :params (not params_hash or whatever), it's a magic variable
21:39 mgagne I'll push my branch so you can see
21:40 aimon yeah I think I am being obtuse here.. my head feels like a brick atm.. ugh need more sleep!
21:40 aimon or more cafeine
21:41 mgagne https://github.com/mgagne/puppet-cinder/comm​it/f1f5537127a2ddffbd39a963e78492edf8dbf8e0
21:41 aimon ahh!
21:42 mgagne it's mainly about using params and not params_hash
21:42 aimon ok, ok, 'params' is magic.. my code uses an abritrary hash
21:42 mgagne yes
21:42 aimon which bypasses the real tests
21:42 aimon got it. Tahnk you!
21:43 mgagne sorry, my patch modifies a lot of things so it might not be easy to figure out what's going on
21:44 aimon totally makes sense.. initially I thought you were talking about params.pp class
21:44 mgagne oh, no sorry =)
21:44 aimon lol naming overload ;)
21:44 mgagne =)
21:44 mgagne you could remove include cinder::params too
21:45 mgagne There isn't any OS specific values required from cinder::params
21:47 mgagne in your test, it's only a matter of injection the nexenta_host parameter too
21:54 openstackgerrit A change was merged to stackforge/puppet-nova: add gitreview  https://review.openstack.org/26672
21:57 openstackgerrit A change was merged to stackforge/puppet-swift: Allow to have a different internal address and public address  https://review.openstack.org/26585
22:18 openstackgerrit Dan Prince proposed a change to stackforge/puppet-cinder: Renames cinder::base to cinder.  https://review.openstack.org/26499
22:21 openstackgerrit A change was merged to stackforge/puppet-cinder: Renames cinder::base to cinder.  https://review.openstack.org/26499
22:27 openstackgerrit Aimon Bustardo proposed a change to stackforge/puppet-cinder: Add nexenta support  https://review.openstack.org/26584
22:28 openstackgerrit Dan Bode proposed a change to stackforge/puppet-openstack: update fixtures  https://review.openstack.org/26683
22:29 bodepd_ mgagne: that last one is to resolve your issue: ^^^^
22:29 mgagne cool
22:30 mgagne anyone with folsom?
22:31 bodepd_ what about it?
22:31 mgagne https://review.openstack.org/#/c/26508/
22:31 bodepd_ I have some integration tests that I can launch from jenkins
22:31 bodepd_ but its manyal atm
22:31 mgagne just to make sure it does not break anything although I did not ask for puppet-cinder
22:32 bodepd_ it will take me some time. I have two or three things I need first
22:32 mgagne alright
22:33 openstackgerrit A change was merged to stackforge/puppet-openstack: update fixtures  https://review.openstack.org/26683
22:46 openstackgerrit Aimon Bustardo proposed a change to stackforge/puppet-openstack: add rabbit_virtual_host passthrough to cinder::base  https://review.openstack.org/26589
22:54 e1mer joined #puppet-openstack
22:55 openstackgerrit A change was merged to stackforge/puppet-openstack: add gitreview  https://review.openstack.org/26674
23:05 JoeJulian joined #puppet-openstack
23:05 bodepd_ I fixed the permission on openstack_dev_env. Well, by I , I mean I asked clarkb if he would fix it :)
23:05 mgagne ok
23:07 JoeJulian I've seen other modules that have a param.pp also have a config.pp that extends param. You node has to instance {module}::config before {module} but it allows you to override params. What about doing something like that?
23:08 JoeJulian "You node"... <sigh> I can't type today. :/
23:12 openstackgerrit Dan Bode proposed a change to stackforge/puppet-openstack_dev_env: update repos to point at stackforge  https://review.openstack.org/26687
23:13 bodepd_ JoeJulian: I would hope that any overriding would be done through hiera
23:14 mgagne hehe, still trying to wrap my head around hiera
23:14 bodepd_ it just an external way to lookup data that supports a hierarchical lookup
23:14 openstackgerrit Aimon Bustardo proposed a change to stackforge/puppet-openstack: add rabbit_virtual_host passthrough to cinder::base  https://review.openstack.org/26589
23:14 bodepd_ my use case now, is that I have a jenkins layer of the lookup so that jenkins can inject overriding data
23:15 mgagne hmm
23:15 bodepd_ (like what version of openstack to test, or what networking mode to use)
23:15 JoeJulian I need to get openstack working reliably so I can make room on some box to implement hiera...
23:15 bodepd_ I have implemented it in the dev_environment
23:16 mgagne There is 3-4 backends available and I'm still trying to find the right one for my purpose and one easy to maintain/use
23:16 bodepd_ puppet-openstack_dev_env that is
23:16 bodepd_ what backends?
23:16 bodepd_ ah, hiera backends?
23:16 bodepd_ start with yaml :)
23:16 mgagne yes, hiera backends
23:16 bodepd_ work your way to something more complicated
23:16 bodepd_ the openstack-infra team uses hiera-gpg for example
23:17 mgagne I'll have to find time for this :P
23:17 JoeJulian +1
23:17 mgagne Will puppetlabs/tempest stay under the control of puppetlabs or will it be moved to stackforge someday?
23:19 JoeJulian So I just make yaml files somewhere? Is there some mechanism in place that override those params or is that functionality that still needs added?
23:19 mgagne with puppet 3.0, it's builtin
23:20 JoeJulian Damn. I guess I know what I'll be reading tonight.
23:20 mgagne with previous version, you have to install hiera but you won't have automatic parameter lookup
23:24 JoeJulian I've been on 3 longer than I should have anyway. :D
23:31 bodepd_ mgagne: I woudl like to move everything to stackforge, but I just started with the critical stuff
23:31 bodepd_ mgagne: you're actually using puppetlabs-tempest :)
23:32 mgagne I'm not using tempest for now. I had to ask after the update of Puppetfile in stackforge/puppet-openstack_dev_env
23:32 bodepd_ JoeJulian: this sets up hiera: https://github.com/puppetlabs/puppetlabs-opensta​ck_dev_env/blob/master/manifests/setup/hosts.pp
23:33 bodepd_ gotcha.
23:33 bodepd_ I use it for CI
23:34 JoeJulian bodepd_: Do you want a whole pull request to fix "enalbles" in keystone's init.pp?
23:35 JoeJulian actually, I suppose I should check git before I ask..
23:35 mgagne can't do a partial pull request :P
23:35 JoeJulian :D
23:35 JoeJulian But since he's got to do readme changes since moving the modules anyway, he could just sneak that one in.
23:41 openstackgerrit A change was merged to stackforge/puppet-openstack_dev_env: add gitreview  https://review.openstack.org/26675
23:43 openstackgerrit A change was merged to stackforge/puppet-openstack_dev_env: update repos to point at stackforge  https://review.openstack.org/26687
23:53 bodepd_ JoeJulian: yes. we're not doing pull requests any more
23:53 bodepd_ JoeJulian: its all under review.openstack.org

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