Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-02-17

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

All times shown according to UTC.

Time Nick Message
00:04 markvoelker joined #puppet-openstack
00:24 aimon joined #puppet-openstack
00:24 aimon joined #puppet-openstack
01:05 markvoelker joined #puppet-openstack
01:11 vinsh joined #puppet-openstack
01:29 20WAANEMG joined #puppet-openstack
01:52 lpmulligan joined #puppet-openstack
02:04 badiane_ka joined #puppet-openstack
02:08 markvoelker joined #puppet-openstack
02:18 xingchao joined #puppet-openstack
02:41 aimon joined #puppet-openstack
02:49 ilbot3 joined #puppet-openstack
02:49 Topic for #puppet-openstack is now Place to collaborate on Puppet/OpenStack tools: logs at http://irclog.perlgeek.de/puppet-openstack/today
03:09 vinsh joined #puppet-openstack
03:09 markvoelker joined #puppet-openstack
03:18 xingchao joined #puppet-openstack
03:39 xingchao joined #puppet-openstack
04:04 xingchao joined #puppet-openstack
04:09 xingchao joined #puppet-openstack
04:10 markvoelker joined #puppet-openstack
04:18 openstackgerrit Emilien Macchi proposed stackforge/puppet-ceph: Add EL7 support  https://review.openstack.org/156458
04:27 openstackgerrit Merged stackforge/puppet-keystone: Add support for LDAP connection pools  https://review.openstack.org/156402
05:12 markvoelker joined #puppet-openstack
05:18 xingchao joined #puppet-openstack
05:57 saju_m joined #puppet-openstack
05:59 badiane_ka joined #puppet-openstack
06:06 xingchao joined #puppet-openstack
06:09 dmsimard_away joined #puppet-openstack
06:13 markvoelker joined #puppet-openstack
06:19 sanjayu joined #puppet-openstack
06:45 vinsh joined #puppet-openstack
06:49 tfz joined #puppet-openstack
07:09 saju_m joined #puppet-openstack
07:15 markvoelker joined #puppet-openstack
07:29 xingchao joined #puppet-openstack
07:36 mmagr joined #puppet-openstack
07:55 xingchao joined #puppet-openstack
08:17 markvoelker joined #puppet-openstack
08:24 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Create resource cache upon creation  https://review.openstack.org/156503
08:37 IvanBerezovskiy joined #puppet-openstack
08:45 jpena joined #puppet-openstack
08:51 jistr joined #puppet-openstack
08:53 aimon joined #puppet-openstack
09:15 xingchao joined #puppet-openstack
09:18 markvoelker joined #puppet-openstack
09:25 xingchao joined #puppet-openstack
09:39 stamak joined #puppet-openstack
10:05 dgurtner joined #puppet-openstack
10:05 dgurtner joined #puppet-openstack
10:12 ddmitriev joined #puppet-openstack
10:19 markvoelker joined #puppet-openstack
10:27 dgurtner EmilienM: ping re: https://review.openstack.org/#/c/156458/
10:32 xingchao joined #puppet-openstack
10:39 saju_m joined #puppet-openstack
10:48 social joined #puppet-openstack
10:55 aarzhanov joined #puppet-openstack
11:20 markvoelker joined #puppet-openstack
11:23 bogdando joined #puppet-openstack
11:23 social joined #puppet-openstack
11:24 social joined #puppet-openstack
11:30 dmsimard_away joined #puppet-openstack
11:32 cdent joined #puppet-openstack
11:36 bogdando joined #puppet-openstack
11:47 xingchao joined #puppet-openstack
12:08 social joined #puppet-openstack
12:21 markvoelker joined #puppet-openstack
12:25 dprince joined #puppet-openstack
12:30 aimon joined #puppet-openstack
12:52 markvoelker joined #puppet-openstack
13:17 ericpeterson joined #puppet-openstack
13:29 xingchao joined #puppet-openstack
14:01 openstackgerrit Clayton O'Neill proposed stackforge/puppet-neutron: Add Puppet 4.x lint checks  https://review.openstack.org/156593
14:09 aarefiev joined #puppet-openstack
14:19 aimon joined #puppet-openstack
14:24 richm joined #puppet-openstack
14:35 derekh joined #puppet-openstack
14:38 mschmitt joined #puppet-openstack
14:39 pradk joined #puppet-openstack
14:42 dalgaaf joined #puppet-openstack
14:53 openstackgerrit Clayton O'Neill proposed stackforge/puppet-designate: Add Puppet 4.x lint checks  https://review.openstack.org/156607
15:03 cdent_ joined #puppet-openstack
15:05 ericpeterson joined #puppet-openstack
15:11 rcallawa anyway I could get a little review love on https://review.openstack.org/#/c/137534/? Thanks!
15:13 finnx joined #puppet-openstack
15:18 openstackgerrit Clayton O'Neill proposed stackforge/puppet-designate: Fix spelling mistake  https://review.openstack.org/156616
15:22 finnx joined #puppet-openstack
15:24 finnx joined #puppet-openstack
15:25 finnx joined #puppet-openstack
15:27 vinsh joined #puppet-openstack
15:28 openstackgerrit Clayton O'Neill proposed stackforge/puppet-designate: Fix enabled parameter ordering  https://review.openstack.org/156621
15:33 dvorak looks like rubygems is having problems again.  Getting lots of timeouts retrieving gems during CI
15:33 openstackgerrit Merged stackforge/puppet-designate: Fix spelling mistake  https://review.openstack.org/156616
15:34 dvorak fwiw, I've got a few designate patches outstanding that look like nit picking, but I'm trying to get some cleanup patches in place before putting my virtualenv review up, to hopefully make the diff a little smaller
15:57 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Update CI boxes  https://review.openstack.org/156632
15:59 xarses joined #puppet-openstack
16:06 openstackgerrit Clayton O'Neill proposed stackforge/puppet-designate: Add puppet-lint param docs check  https://review.openstack.org/156638
16:09 mdorman joined #puppet-openstack
16:17 aimon joined #puppet-openstack
16:18 dvorak anyone know how to get gate checks to rerun? I'm getting spurious errors from rubygems not being reliable
16:19 openstackgerrit Clayton O'Neill proposed stackforge/puppet-designate: Fix misc cut-and-paste mistakes  https://review.openstack.org/156644
16:25 EmilienM dvorak: recheck
16:27 dvorak yeah, that's what #openstack-infra said too, but it appears that just ran the check tests, which failed
16:27 dvorak so now I'm trying to get my +1 from jenkins back :)
16:33 dvorak should anyone be able to edit cards in trello?
16:34 EmilienM spredzy|afk: ^
16:35 dvorak I have a review up for this for designate, I was going to update the card - https://trello.com/c/tC7ZhPtN/10-add-pupp​et-lint-param-docs-plugins-to-puppet-lint
16:35 dontalton joined #puppet-openstack
16:36 dvorak it appears that recheck will always rerun the check tests, then will run the gate tests again if applicable
16:38 derekh joined #puppet-openstack
16:39 openstackgerrit Merged stackforge/puppet-neutron: Add Puppet 4.x lint checks  https://review.openstack.org/156593
16:50 aarzhanov joined #puppet-openstack
16:52 rwsu joined #puppet-openstack
17:02 spredzy dvorak, just sent you a trello invite, once you answer you should be able to edit
17:02 dvorak thanks!
17:03 openstackgerrit Merged stackforge/puppet-nova: Add Puppet 4.x lint checks  https://review.openstack.org/156367
17:04 spredzy dvorak, working ?
17:04 dvorak looks like :)
17:05 spredzy goot :)
17:05 spredzy crinkle, ping
17:06 dvorak is there an offical list of modules that is maintained somewhere?  It seems like I always have a hard time remembering all of them, and the list is getting longer :)
17:07 dontalton ./stackforge/puppet-* :)
17:10 rmoe joined #puppet-openstack
17:15 IvanBerezovskiy left #puppet-openstack
17:43 tfz joined #puppet-openstack
17:56 openstackgerrit Gaudenz Steinlin proposed stackforge/puppet-ceph: Fix Exec unless test for set-${name}-size  https://review.openstack.org/156684
18:01 aimon joined #puppet-openstack
18:02 openstackgerrit Merged stackforge/puppet-designate: Add Puppet 4.x lint checks  https://review.openstack.org/156607
18:03 openstackgerrit Matt Fischer proposed stackforge/puppet-keystone: openstackclient should use package_ensure  https://review.openstack.org/156686
18:10 openstackgerrit Gaudenz Steinlin proposed stackforge/puppet-ceph: Fix Exec unless test for set-${name}-size  https://review.openstack.org/156684
18:21 openstackgerrit Matt Fischer proposed stackforge/puppet-monasca: Fix resource warnings  https://review.openstack.org/156697
18:24 mfisch rubygems.org killing me today
18:24 mfisch killing everyone it seems
18:26 openstackgerrit Dulanjalie Ganegedara proposed stackforge/puppet-neutron: Adding portdb and fastpath_flood to n1kv.conf  https://review.openstack.org/155560
18:30 saju_m joined #puppet-openstack
18:30 openstackgerrit Emilien Macchi proposed stackforge/puppet-ceph: Add EL7 support  https://review.openstack.org/156458
18:50 EmilienM hogepodge: crinkle: re: summit - do we have to do something to got a room?
18:51 mgagne anyone using puppet 3.7 (precise + ruby 1.8) and puppetdb 1.6 ?
18:51 hogepodge EmilienM crinkle talk to claire@openstack.org
18:51 mgagne getting weird performance
18:51 mgagne 12 seconds vs 154s for the same catalog
18:52 mgagne with those weird errors: Could not intern from text/pson: undefined method `delete' for #<Puppet::Resource::Catalog:0x7f505f351450>
18:52 hogepodge EmilienM: crinkle I'm not sure what the status of those rooms are with the changes going on. We know they're important, but don't know how to allocate them yet
18:54 crinkle spredzy: pong
18:54 mfisch crinkle: where could I find more info on how the service provider works?
18:55 mfisch are the internals documented?
18:55 mfisch I thinnk its using the exit code of service foo status
18:56 mgagne mfisch: my guess is that the code is the documentation
18:56 crinkle mfisch: I don't think it's really documented
18:56 mfisch ok
18:56 crinkle yeah https://github.com/puppetlabs/puppet/t​ree/master/lib/puppet/provider/service is probably the best you'll get
18:57 mgagne shouldn't be hard anyway, types and provides are "known" concepts. service resources only implement them
18:57 mgagne mfisch: upstart for example checks the output: https://github.com/puppetlabs/puppet/blob/master/​lib/puppet/provider/service/upstart.rb#L162-L167
18:58 mfisch this is sysv init
18:58 mfisch I think its checking status
18:59 mfisch this service always returns 0 from status
18:59 mfisch other services like ntp are smarter
18:59 mfisch I think puppet thinks its running
18:59 mfisch a bug on them really
18:59 mgagne mfisch: main logic is in base: https://github.com/puppetlabs/puppet/blob/mast​er/lib/puppet/provider/service/base.rb#L54-L71
18:59 mgagne mfisch: init.rb only provides/finds the status command to run
19:00 mfisch if $CHILD_STATUS.exitstatus == 0
19:00 mfisch return :running
19:00 mgagne yep
19:00 mgagne there you go
19:00 mfisch I'll send them a patch and then invoke the support contract, whoop
19:13 mgagne I'm starting to regret updating to puppet 3.7 :-/
19:14 EmilienM hogepodge: I shooted an email, thx for info
19:20 hogepodge EmilienM it was bounced back over to me, so I guess I should figure out what I'm responsible for :-D
19:20 openstackgerrit joined #puppet-openstack
19:22 EmilienM hogepodge: lol
19:22 mfisch mgagne: why?
19:24 mfisch whats the minimum version of openstackclient we need for puppet?
19:24 rharwood left #puppet-openstack
19:27 crinkle mfisch: we're successful with 0.3.0 for now
19:27 crinkle preferred is 1.0.1
19:27 crinkle in the future
19:29 mfisch thx
20:00 nosleep77 joined #puppet-openstack
20:01 stamak joined #puppet-openstack
20:01 openstackgerrit Gaudenz Steinlin proposed stackforge/puppet-ceph: Fix Exec unless test for set-${name}-size  https://review.openstack.org/156684
20:34 openstackgerrit Dulanjalie Ganegedara proposed stackforge/puppet-neutron: Adding portdb and fastpath_flood to n1kv.conf  https://review.openstack.org/155560
20:59 openstackgerrit Merged stackforge/puppet-designate: Fix enabled parameter ordering  https://review.openstack.org/156621
21:01 openstackgerrit Dulanjalie Ganegedara proposed stackforge/puppet-neutron: Adding portdb and fastpath_flood to n1kv.conf  https://review.openstack.org/155560
21:02 openstackgerrit Spencer Krum proposed stackforge/puppet-openstack_extras: Add delorean repo class to openstack extras  https://review.openstack.org/156332
21:02 openstackgerrit Merged stackforge/puppet-designate: Add puppet-lint param docs check  https://review.openstack.org/156638
21:07 mgagne mfisch: I think I narrowed the problem. looks like the catalog compilation timeout and the agent tries to apply a cached catalog and fails.
21:08 EmilienM mgagne: you run puppetmaster :-P
21:08 mgagne EmilienM: yes
21:08 EmilienM I've not seen improvement in masterless
21:08 mgagne EmilienM: and it worked very well with puppet 3.2
21:08 dvorak mgagne: you're running dynamic environments?
21:09 EmilienM in term of time to compile the catalog
21:09 mgagne dvorak: how can I check? I don't have multiple environments.
21:09 mgagne dvorak: can it be disabled?
21:09 dvorak if you're not using dynamic environments, then I think it complains to you every time you run puppet :)
21:10 dvorak there was some discussion on puppet-users when 3.7 came out about catalog caching defaults w/dynamic environments.  We've not moved yet because we only have one environment too
21:11 mgagne dvorak: ok, simple question: how can I have a working setup with non-dynamic environments? any configs I should know about?
21:11 dvorak yeah, the default (old) way to do it is with static environments
21:11 dvorak with dynamic environments you end up with something like /etc/puppet/environments/<​environmentname>/<modules>
21:12 mgagne I don't have this layout
21:12 dvorak except that environment name can't be master, which is awesome
21:12 dvorak you're probably using the old ones then
21:12 mgagne right
21:12 dvorak which I think is going away with Puppet 4.x
21:12 mgagne damn
21:12 dvorak yeah, I find it annoying
21:12 mgagne I really hate this change
21:13 dvorak I intend to symlink /etc/puppet/modules into /etc/puppet/environments/prod
21:13 mgagne because puppet modules environment is coupled with hiera environment too
21:13 mgagne and that's a huge problem IMO
21:16 richm mgagne: I have found an annoying problem with using Puppet::Resource.indirection.find(​"Keystone_user/#{get_user}")[:id]
21:16 richm I cannot figure out how to mock it
21:16 richm for spec testing
21:16 mgagne =(
21:17 EmilienM xarses: ping
21:18 xarses pong
21:18 EmilienM xarses: hey, I remember a patch of you on puppet-neutron about refreshing when nova admin id is changed
21:18 EmilienM xarses: on my setup (running master), neutron-server is refreshed every puppet run
21:18 EmilienM and the id does not change
21:19 dvorak I had a similar problem a while back, but it got fixed a month or two ago for me
21:19 dvorak I think mdorman fixed it
21:19 EmilienM dvorak: mdorman how?
21:19 mgagne this puppet 3.7 issue is really annoying me
21:19 mgagne half of my nodes fails to apply the catalog
21:20 mgagne "Could not retrieve file metadata for puppet:///modules/infra/whatever.sh: Could not intern from text/pson: undefined method `delete' for #<Puppet::Resource::Catalog:0x7f97c9913f80>"
21:20 dvorak EmilienM: this might be a different problem, but I had that problem until this change handed: https://review.openstack.org/#/c/134667/
21:20 dvorak s/handed/landed/
21:20 EmilienM how
21:20 EmilienM is it in stable/juno?
21:21 dvorak oh, probably not, I don't see a cherry-pick
21:21 EmilienM no
21:21 EmilienM https://review.openstack.org/#/c/156777/
21:21 EmilienM crinkle: mgagne ^
21:21 EmilienM please
21:21 dvorak you +2'ed the first change :)
21:22 EmilienM dvorak: sometimes I forget some reviews
21:22 dvorak nod :)
21:22 EmilienM mdorman: thx again for the patch BTW
21:22 ericpeterson joined #puppet-openstack
21:22 dvorak I remember this one because the bug was annoying me
21:23 dvorak I thought the change to make it notify was actually mine
21:23 dvorak but maybe I just put in another dep
21:23 openstackgerrit Richard Megginson proposed stackforge/puppet-keystone: support for keystone v3 api  https://review.openstack.org/146926
21:24 mdorman sorry wasn’t paying attention to this channel, catching up
21:25 openstackgerrit Merged stackforge/puppet-designate: Fix misc cut-and-paste mistakes  https://review.openstack.org/156644
21:25 mdorman looks like you all figured out waht you needed :)
21:29 openstackgerrit Dulanjalie Ganegedara proposed stackforge/puppet-neutron: Adding portdb and fastpath_flood to n1kv.conf  https://review.openstack.org/155560
21:33 xarses EmilienM: I don't think I had any problem with nova_admin_id after we patched the order / collection
21:33 EmilienM xarses: mdorman did in fact
21:37 mdorman yeah.  don’t remember the details but i remember doing it
21:40 dgurtner xarses: hei! can I ask you to have a look at https://review.openstack.org/#/c/156632/ ?
21:40 dgurtner dmsimard: same to you! ^
21:41 xarses EmilienM, mdorman my issue was last with https://github.com/stackforge/puppet-neut​ron/blame/master/manifests/server.pp#L233
21:42 xarses the other issue was https://review.openstack.org/#/c/106141/
21:43 mdorman yeah i remember that first one too
21:44 dontalton joined #puppet-openstack
21:46 aimon joined #puppet-openstack
21:47 stamak joined #puppet-openstack
21:48 social joined #puppet-openstack
21:56 vinsh joined #puppet-openstack
22:10 xarses dgurtner: https://review.openstack.org/#/c/156632/1/README.md
22:11 xarses you added {one,two}-{ubuntu-server-1404-x64,centos70-x64}, are we not including them in the list at 100-103?
22:14 xarses hmm, it looks like they are for beaker
22:18 dgurtner xarses: they are intended for rspec-system
22:18 dgurtner xarses: but it's a multi step thing
22:18 openstackgerrit Merged stackforge/puppet-ceph: Update CI boxes  https://review.openstack.org/156632
22:18 dgurtner xarses: we first need the boxes to be able to test
22:19 dgurtner xarses: then we need to actually support the new releases
22:19 dgurtner xarses: and then we can show how to use them in the readme
22:20 xarses dgurtner: mmkay, thanks
22:20 xarses dgurtner: is 151970  ready?
22:20 saneax_-_ joined #puppet-openstack
22:21 openstackgerrit Merged stackforge/puppet-ceph: Fix Exec unless test for set-${name}-size  https://review.openstack.org/156684
22:22 dgurtner xarses: 151970 is ready, yes
22:24 dgurtner https://review.openstack.org/156684 didn't pass CI tests yet thoug (and I expect them to brake atm.)
22:24 xarses ??
22:25 dgurtner xarses: the changed test has some side effects
22:26 * xarses bangs his head against a wall
22:26 dgurtner xarses: :P
22:27 dgurtner xarses: it's not this tests fault
22:27 xarses it looks like it's passing was passing as expected, what am I missing here?
22:27 dgurtner the changed test enforces idempotency
22:28 dgurtner xarses: puppet returns 2 on change, 1 on failure and 0 on nothing changed
22:28 dgurtner xarses: the issue is that the puppet run also contains monitor creation, which in fact is _not_ idempotent and thus will exit with 2
22:30 xarses dgurtner: so the test should still be should_not == 1?
22:30 dgurtner xarses: well yes and no
22:30 dgurtner xarses: the test should be "should == 0", but the mon creation has to be moved out of the test
22:37 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Fix README endpoint example  https://review.openstack.org/156808
22:39 xarses dgurtner: ok, so for mon creation it should still be should_not == 1
22:39 dgurtner xarses: exactly
22:40 dgurtner xarses: actually I once tried working on that, but it requires some big changes
22:40 dgurtner xarses: currently we're creating the mon bootstrap key via a file resource, which we then remove again via an exec
22:41 dgurtner xarses: as it's only a temporary file
22:41 EmilienM crinkle: can you approve a backport please ? > https://review.openstack.org/#/c/154545/
22:42 dvorak dgurtner: it returns 4 or 6 on failure
22:43 dvorak 4 on failure with no changes, 6 on failures with changes
22:45 openstackgerrit Merged stackforge/puppet-horizon: Sort policy files in local_settings.py  https://review.openstack.org/156335
22:46 mfisch crinkle: there some string interpolation issue in the keystone provider code
22:46 mfisch notice("#{service} service is unavailable. Will retry for up to #{wait} seconds.")
22:46 openstackgerrit Mathieu Gagné proposed stackforge/puppet-horizon: Sort policy files in local_settings.py  https://review.openstack.org/156815
22:46 mfisch I get "service service is unavail"
22:46 mfisch Notice: Puppet::Type::Keystone_service::ProviderOpenstack: service service is unavailable. Will retry for up to 10 seconds.
22:47 dgurtner dvorak: thanks for the clarification!
22:47 mfisch I guess more likely is that service comes in wrong
22:48 openstackgerrit Andrew Woodward proposed stackforge/puppet-ceph: WIP: slightly seperate install from create pool  https://review.openstack.org/156817
22:52 xarses dgurtner: re:151970 nodepool,yml is supposed to have an ip of 127.0.0.1?
22:53 dgurtner xarses: yes, afaik. this is run directly on the machine without a hypervisor, so you wouldn't know which IPs other than that are available
22:54 xarses mmkay
22:55 dgurtner I think the way beaker works on openstack gerrit is not very refined yet tbh :)
22:59 xarses dgurtner: 151970 looks good, merge?
23:02 openstackgerrit Emilien Macchi proposed stackforge/puppet-ceph: Add EL7 support  https://review.openstack.org/156458
23:02 dgurtner xarses: sure :)
23:03 * xarses waits for the detonation
23:03 openstackgerrit Merged stackforge/puppet-keystone: Fix README endpoint example  https://review.openstack.org/156808
23:04 dgurtner xarses: haha
23:04 dgurtner xarses: btw. why did you work-in-progress your change https://review.openstack.org/#/c/156817/
23:04 xarses dgurtner: i wanted it to run the tests, but aparently the way i did it makes it not test at all
23:06 dgurtner xarses: no, I think it just takes a while until the tests get run ;)
23:06 openstackgerrit Merged stackforge/puppet-ceph: Adding base infrastructure for beaker CI tests  https://review.openstack.org/151970
23:07 dgurtner xarses: but no I tried to fix the same issue for on reason :P
23:07 xarses =P
23:07 xarses what did you come up with ?
23:07 dgurtner xarses: I didn't even get to hit the review button
23:08 dgurtner xarses: local tests still running
23:08 openstackgerrit Andrew Woodward proposed stackforge/puppet-ceph: Slightly seperate install from create pool  https://review.openstack.org/156817
23:09 crinkle mfisch: it's because it's referring to the keystone service
23:09 crinkle so for other things it'll say the project service or user service
23:10 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Refactor the pool tests to only test pool  https://review.openstack.org/156827
23:10 dgurtner xarses: ^
23:10 crinkle would probably be better to just say identity service but "identity" doesn't really appear in the command
23:11 crinkle maybe "the service action is unavailable"?
23:11 crinkle mfisch: ^
23:15 xarses dgurtner: your's looks nicer
23:20 dgurtner xarses: hm.. but also more complicated for a quick fix.
23:25 dgurtner xarses: I'll give your's the thumbs up, so we can get it merged asap, maybe I'll rebase mine onto yours as a future cleanup later
23:26 xarses provided it passes. dachary already +2'd yours
23:27 dgurtner xarses: as you wish, feel free to +2 mine ;)
23:32 gildub joined #puppet-openstack
23:38 mfisch crinkle: that would make sense
23:42 openstackgerrit Matt Fischer proposed stackforge/puppet-keystone: Add a package ensure for openstackclient  https://review.openstack.org/156686
23:48 xarses dgurtner: heh, I walk away and dachary merges it =)
23:49 openstackgerrit Merged stackforge/puppet-ceph: Slightly seperate install from create pool  https://review.openstack.org/156817
23:49 dgurtner xarses: :) I'll retire mine then
23:52 dachary :-) from then on we should wait for the CI to give the green light before merging anything, right ?
23:52 dachary xarses: ^ ?
23:53 xarses i swore it did
23:53 dgurtner :P
23:53 dachary I mean it was my mistake to not wait for the CI before merging https://review.openstack.org/156817
23:54 dachary I'll behave next time
23:54 xarses You mean to tell us that you haven't been behaving the whole time?
23:55 dgurtner xarses: dachary ok, I'm off for the day. I'll need to wait for all queued up CI tests to finish, so I can add the CentOS 7 tests and then verify EmilienM changes. Please don't merge _even if_ the CI already comes back successfully ;)
23:55 dgurtner dmsimard_away: ^
23:55 xarses or just last time
23:55 dachary dgurtner: ack
23:55 dachary xarses: just last time ;-)
23:55 xarses dgurtner: roger roger
23:56 dgurtner ok, gn8 everyone
23:56 dachary night

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