Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-04-20

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

All times shown according to UTC.

Time Nick Message
00:05 openstackgerrit Merged openstack/puppet-gnocchi: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/307201
00:15 spredzy joined #puppet-openstack
00:25 fawadkhaliq joined #puppet-openstack
00:41 openstackgerrit zhongshengping proposed openstack/puppet-trove: Remove the extra spaces in the conductor.pp  https://review.openstack.org/308096
00:49 haplo37 joined #puppet-openstack
00:57 fawadkhaliq joined #puppet-openstack
01:09 ayoung joined #puppet-openstack
01:21 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Correct typo resulting in a hardcoded paramter used incorrectly  https://review.openstack.org/299615
01:51 openstackgerrit Matt Fischer proposed openstack/puppet-trove: deprecate templated guestagent config  https://review.openstack.org/307903
01:59 ducttape_ joined #puppet-openstack
02:37 fedexo joined #puppet-openstack
02:49 ducttape_ joined #puppet-openstack
03:00 mwhahaha ah damn it my puppet-nova failed on the verify step
03:01 mwhahaha stupid puppet labs repo 403
03:09 mwhahaha but if anyone cares, the centos7 puppet jobs appear to be broken because puppetlab's release rpm is returning a 403
03:09 mwhahaha http://logs.openstack.org/77/306677/9/g​ate/gate-puppet-openstack-integration-3​-scenario001-tempest-centos-7/061bc66/c​onsole.html#_2016-04-20_01_23_14_212
03:11 colonwq mwhahaha, thanks. I am waiting for mine to (slowly) finish. I'll be on the look out for it.
03:12 mwhahaha it's going to fail
03:12 crinkle mwhahaha: i think someone already reported the issue https://tickets.puppetlabs.com/browse/CPR-332
03:12 mwhahaha over in #puppet the el6 is missing, looks like they renamed it
03:13 mwhahaha http://yum.puppetlabs.com/puppet​labs-release-pc1-el-7.noarch.rpm
03:13 crinkle but i think the release team is mostly US-based and therefor not at work right now
03:13 * mwhahaha flips tables
03:15 colonwq hopefully the debian test will pass. I had a keyboard/brain issue in my patch
04:26 chandankumar joined #puppet-openstack
04:28 fawadkhaliq joined #puppet-openstack
04:35 markvoelker joined #puppet-openstack
04:44 imcsk8 joined #puppet-openstack
05:10 chandankumar joined #puppet-openstack
05:44 jmikolajczak joined #puppet-openstack
05:47 dc_mattj joined #puppet-openstack
06:21 rcernin joined #puppet-openstack
06:47 tesseract joined #puppet-openstack
06:50 matt__ joined #puppet-openstack
06:57 jaosorior joined #puppet-openstack
06:58 matt__ left #puppet-openstack
06:58 dgurtner joined #puppet-openstack
06:59 matt__ joined #puppet-openstack
07:00 permalac joined #puppet-openstack
07:00 matt__ left #puppet-openstack
07:00 dc_mattj joined #puppet-openstack
07:01 magicboiz joined #puppet-openstack
07:02 magicboiz left #puppet-openstack
07:18 dgurtner joined #puppet-openstack
07:24 jprs joined #puppet-openstack
07:34 hjensas joined #puppet-openstack
07:34 hjensas joined #puppet-openstack
07:36 paramite joined #puppet-openstack
07:36 shardy joined #puppet-openstack
07:45 fawadkhaliq joined #puppet-openstack
07:59 gfidente joined #puppet-openstack
08:08 openstackgerrit Mykyta Karpin proposed openstack/puppet-ironic: Switch ironic to use puppet-oslo module  https://review.openstack.org/305299
08:08 karimb joined #puppet-openstack
08:18 jistr joined #puppet-openstack
08:36 openstackgerrit Alexey Deryugin proposed openstack/puppet-trove: Use oslo module for messaging (rabbit) configuration  https://review.openstack.org/307907
08:37 derekh joined #puppet-openstack
08:38 shardy joined #puppet-openstack
08:47 mgould joined #puppet-openstack
08:53 karimb joined #puppet-openstack
08:58 openstackgerrit Alexey Deryugin proposed openstack/puppet-trove: Use oslo module for loggind and database configuration  https://review.openstack.org/308207
09:00 openstackgerrit Alexey Deryugin proposed openstack/puppet-trove: Use oslo module for logging and database configuration  https://review.openstack.org/308207
09:02 dgurtner joined #puppet-openstack
09:02 dgurtner joined #puppet-openstack
09:50 _ody joined #puppet-openstack
09:56 _ody joined #puppet-openstack
09:56 openstackgerrit Alexey Deryugin proposed openstack/puppet-trove: Use oslo module for messaging (rabbit) configuration  https://review.openstack.org/307907
09:57 openstackgerrit Alexey Deryugin proposed openstack/puppet-trove: Use oslo module for logging and database configuration  https://review.openstack.org/308207
10:05 dc_mattj joined #puppet-openstack
10:05 openstackgerrit Denis Egorenko proposed openstack/puppet-tempest: Update spec tests for Tempest  https://review.openstack.org/308241
10:37 karimb joined #puppet-openstack
10:38 chandankumar joined #puppet-openstack
10:53 jaosorior joined #puppet-openstack
11:03 openstackgerrit Miles Gould proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/308271
11:03 openstackgerrit Miles Gould proposed openstack/puppet-ironic: Copy iPXE ROMs into /tftpboot  https://review.openstack.org/308272
11:08 openstackgerrit Xingchao Yu proposed openstack/puppet-oslo: Add oslo.middleware defines  https://review.openstack.org/274422
11:13 johnmilton joined #puppet-openstack
11:45 ducttape_ joined #puppet-openstack
11:45 openstackgerrit Xingchao Yu proposed openstack/puppet-keystone: Refactor keystone::resource::authtoken  https://review.openstack.org/266723
11:46 gfidente joined #puppet-openstack
11:56 morazi joined #puppet-openstack
11:57 dprince joined #puppet-openstack
12:02 ansmith joined #puppet-openstack
12:05 openstackgerrit Sergii Golovatiuk proposed openstack/puppet-horizon: Move notification from File to Concat  https://review.openstack.org/308294
12:17 cdelatte joined #puppet-openstack
12:18 markvoelker joined #puppet-openstack
12:22 michchap joined #puppet-openstack
12:31 michchap_ joined #puppet-openstack
12:37 iurygregory joined #puppet-openstack
12:44 karimb joined #puppet-openstack
12:44 iurygregory morning
12:44 iurygregory mfisch, if you have time, can you review https://review.openstack.org/#/c/307940/ ?
12:56 ducttape_ joined #puppet-openstack
13:02 ducttape_ joined #puppet-openstack
13:04 openstackgerrit James Slagle proposed openstack/puppet-tripleo: Add destination parameter to firewall rule  https://review.openstack.org/308317
13:05 karimb joined #puppet-openstack
13:06 ducttape_ joined #puppet-openstack
13:14 karimb_ joined #puppet-openstack
13:15 richm joined #puppet-openstack
13:20 cwolferh joined #puppet-openstack
13:41 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: functions: add defaults for ZUUL_BRANCH & ZUUL_REF  https://review.openstack.org/308333
13:42 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: functions: add defaults for ZUUL_BRANCH & ZUUL_REF  https://review.openstack.org/308333
13:44 EmilienM hello folks
13:44 iurygregory morning
13:45 mjblack morning
13:49 mnaser just wanted to give feedback that the puppet-ceph module has been great, centos + storage sig + puppet-ceph = excellent combination
13:49 mjblack degorenko: hey, with regards to your comment on https://review.openstack.org/#/c/308012, I dont disagree with you but I'm not sure there is a better way to ensure the service has been stopped
13:50 mjblack at least not without making it more complicated
13:50 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/305886
13:50 EmilienM mnaser: excellent news!
13:50 ayoung joined #puppet-openstack
13:50 colonwq Good morning
13:51 mjblack EmilienM: any suggestion?
13:53 EmilienM mjblack: no, I like the patch
13:53 mjblack EmilienM: I think its a bit of a hack but unless we do what the fuel people do, I dont think we're going to get better
13:54 dfisher joined #puppet-openstack
13:54 EmilienM if it works in our CI, if it works in Fuel, if it works in TripleO, if it's documented and if we know it's fixed in Newton, I suggest we go ahead
13:56 degorenko mjblack, can we check port or process instead of sleeps?
13:56 EmilienM bahhh
13:56 mjblack degorenko: you mean like have an "onlyif" if the port is detected to be in use?
13:56 EmilienM that's terrible
13:57 EmilienM there is no way to know the port
13:57 EmilienM specially when running in wsgi
13:57 degorenko it's not a strong suggestion
13:57 EmilienM where port is defined in a separated class
13:57 degorenko but sleep 35 looks ugly
13:57 degorenko we can have some hardware when we need 36
13:57 degorenko 37
13:57 degorenko and more and more
13:57 mjblack degorenko: I dont think anyone will disagree with you that it is ugly but unfortunately there is no good way to ensure it has been stopped without doing something even uglier
13:59 mjblack degorenko,EmilienM: the alternative, besides messing with policy-rc.d, would be to delay the apache start
13:59 openstackgerrit Lukas Bezdicka proposed openstack/puppet-neutron: LBaaSv2 support  https://review.openstack.org/308346
13:59 mjblack have package install immediately stop the service and then do other stuff before starting apache...you still run into a potential issue that it might not be enough time
14:00 mjblack EmilienM,degorenko: https://jpetazzo.github.io/2013/10/06/polic​y-rc-d-do-not-start-services-automatically/
14:00 mjblack we can have that as an option maybe in openstack_extras that is turned off but in our CI its turned on
14:02 degorenko mjblack, which port uses keystone under apache?
14:02 mjblack whichever the user configures
14:02 mjblack by default its 5000 and 35357
14:02 cdelatte joined #puppet-openstack
14:03 degorenko in which config/class we specifying this?
14:03 mjblack main class
14:04 mjblack https://github.com/openstack/puppet-keysto​ne/blob/master/manifests/init.pp#L560-L561
14:06 degorenko so, we are on main class, why we can't check port then?
14:06 degorenko open it or not?
14:06 mwhahaha degorenko: https://review.openstack.org/#/c/305798/ was because at the rate my puppet-nova patch is going to get merged it'll be after summit
14:06 mwhahaha given the CI shenanigans currently occuring
14:07 mwhahaha because UCA is currently broken due to that bug, so i still need to cherry pick one or the other to stable/mitaka
14:09 degorenko mwhahaha, so, the plan is to revert 305978 after backport to mitaka?
14:09 mwhahaha yea
14:09 degorenko this fix we need asap?
14:09 mwhahaha yea
14:09 degorenko ok
14:09 jaosorior joined #puppet-openstack
14:09 mwhahaha we want to announce the UCA support in 9.0 :) and it's kinda hard to do when it's broken
14:10 degorenko mwhahaha, i've dropped my -1
14:10 mwhahaha cool i've retriggered the neutron test so we'll see
14:11 mwhahaha it a race between openstack-infra and fuel-ci
14:11 mwhahaha which patch will land first!
14:12 v1k0d3n joined #puppet-openstack
14:12 degorenko mwhahaha, also, for https://review.openstack.org/#/c/308012
14:12 colonwq mwhahaha, will the slow one have to refactor?
14:12 degorenko why we can't check port here?
14:12 mwhahaha is there a standard provider for that?
14:13 mjblack degorenko: would you be okay with a solution that is part of openstack_extras to use policy-rc.d instead to prevent keystone from starting?
14:13 mwhahaha colonwq: nope they are compatible, just duplicates
14:13 mwhahaha policy-rc.d gets dangerous and that's a lot of code to add for one service
14:13 mjblack mwhahaha: there is a package and a config file
14:13 colonwq mwhahaha, a ok.
14:14 mjblack I'm taking a look at it right now to see if its just a simply install and line in a conf file
14:14 mwhahaha mjblack: no we also have a function to check if the package is installed to be idempotent (which won't work in puppetmaster builds)
14:14 degorenko i'm agree with mwhahaha for policy-rc.d
14:14 mjblack ok
14:14 mwhahaha the best solution is to make sure the port is available
14:15 degorenko yeah, we can have exec with port checking
14:15 mwhahaha yea i guess it could be an exec with retry
14:15 degorenko i think it will be best solution, also i don't like useless sleeps :D
14:16 degorenko yeah, retries and sleeps FOR exec, not IN exec
14:16 mjblack ok I'll update with a check/sleep in loop
14:16 degorenko thanks
14:17 chandankumar joined #puppet-openstack
14:17 Matum joined #puppet-openstack
14:19 mjblack degorenko: you ok with an exec that does the loop/check?
14:20 openstackgerrit James Slagle proposed openstack/puppet-tripleo: Add destination parameter to firewall rule  https://review.openstack.org/308317
14:20 openstackgerrit James Slagle proposed openstack/puppet-tripleo: Add dport/sport parameter to firewall rule  https://review.openstack.org/308358
14:20 degorenko mjblack, loop/check for port checks - yep
14:21 twm2016 joined #puppet-openstack
14:21 xarses joined #puppet-openstack
14:27 dhollinger joined #puppet-openstack
14:28 openstackgerrit Sergii Rizvan proposed openstack/puppet-keystone: Remove user/role prefetch to support multi-domain.  https://review.openstack.org/308365
14:28 dfisher a poll() loop is almost always more desirable than arbitrary sleep calls
14:29 dfisher a root canal is usually more desirable than arbitrary sleep calls too.
14:31 mwhahaha i'd rather have arbitrary sleeps than a root canal, but maybe that's my years of dental work speaking :D
14:32 astupnikov joined #puppet-openstack
14:35 EmilienM dfisher: first tarball: http://tarballs.openstack.org/puppet-aodh/
14:35 EmilienM dfisher: i'm now ajusting to have all releases, with a better name, etc
14:35 EmilienM dfisher: also other modules don't have tarballs yet, I'm on it, like I told you yesterday
14:35 dfisher hot damn.
14:37 openstackgerrit Merged openstack/puppet-openstack-integration: Enable Sahara integration job on Ubuntu  https://review.openstack.org/307911
14:37 chandankumar joined #puppet-openstack
14:39 EmilienM mnaser: just for the record, ceph -s results in our CI: http://logs.openstack.org/00/307900/1/c​heck/gate-puppet-openstack-integration-​3-scenario001-tempest-centos-7/b6cbcc3/​console.html#_2016-04-19_17_05_35_058
14:40 jaosorior joined #puppet-openstack
14:40 mnaser i guess i broke something somewhere \o/
14:40 EmilienM mwhahaha: can we get https://review.openstack.org/#/c/307330/ please?
14:40 mnaser oh i did want to submit a review for something
14:41 mjblack degorenko: going to warn you...it's gonna be ugly
14:41 magicboiz joined #puppet-openstack
14:43 mnaser EmilienM: if ceph::profile::mon is included after ceph::profile::client, puppet poops out.  https://github.com/openstack/puppet-ceph/blo​b/master/manifests/profile/client.pp#L26-L34 and https://github.com/openstack/puppet-ceph/bl​ob/master/manifests/profile/mon.pp#L39-L44
14:43 mnaser is it okay if we add the same if clause in mon.pp?
14:43 darkxploit joined #puppet-openstack
14:44 mnaser or maybe a warning (looks like it has diff paths)
14:48 degorenko mjblack, what's wrong? :)
14:49 mjblack basically writing a shell script in an exec call :D
14:51 mnaser uh if i can chime in
14:51 mnaser i know its a bit more work but maybe you can setup something in openstacklib that can check if a port is free
14:51 mjblack mnaser: sure
14:51 mnaser cause this is likely to be reused across a few more modules
14:51 openstackgerrit Jerzy Mikolajczak proposed openstack/puppet-neutron: Remove deprecated options from agents  https://review.openstack.org/301106
14:52 mjblack mnaser: honestly, its a good idea but doing policy-rc.d would ultimately be the best way to handle it :D
14:52 mnaser true, better to have never started it, but what if a user decides to implement this midway
14:57 degorenko mjblack, why shell script will be difficult?
14:57 degorenko mjblack, netstat -lnt | awk '$6 == "LISTEN" && $4 ~ ".5000"' for example
14:57 degorenko with only substitution for var $port
14:57 mjblack we dont need to sub port
14:57 mjblack since the default port coming out of the deb is 5000
14:58 degorenko but we can override it
14:58 mjblack but we also need to make sure admin port is down too
14:58 EmilienM it sounds like over-complicated
14:58 mjblack degorenko: we dont need to override it
14:58 mjblack I think if we go simple thats fine
14:59 degorenko mjblack, netstat -lnt | awk '$6 == "LISTEN" && ($4 ~ ".5000" || $4 ~ ".35357")'
14:59 degorenko :D
14:59 EmilienM do not hardcode the ports
14:59 EmilienM they can be changes in deployments
14:59 mjblack EmilienM: but thats not the case here
14:59 degorenko as i said above - with sub for ports
14:59 dprince joined #puppet-openstack
14:59 EmilienM it's a terrible hack, imho
14:59 mjblack EmilienM: the default ports get installed as part of the deb, which is what we care about
15:00 mjblack degorenko: btw that doesnt work, it always returns 0 :D
15:02 mjblack degorenko: much rather use netstat -lnt|egrep "(5000|35357)"
15:02 degorenko mjblack, it was just example
15:03 degorenko so, yeah, having grep|egprep can help
15:03 mjblack I only care about return code
15:03 mjblack now if puppet exec provider would work the way the docs say...
15:03 degorenko it will be different return code on situatuon when we have match and when we don't
15:04 mjblack or I can stop being stupid and get my expected return code right :P
15:04 mjblack there we go...
15:04 mwhahaha use ss
15:04 mwhahaha netstat is deprecated
15:04 nibalizer just like ifconfig
15:05 mwhahaha yup
15:06 openstackgerrit Matthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/308012
15:06 mjblack bah, netstat will never die! :P
15:07 degorenko mjblack, ss -lt | egrep "(5000|35357)" looks like we need that
15:07 mwhahaha http://logs.openstack.org/77/306677/9/check/g​ate-puppet-nova-puppet-beaker-rspec-centos-7/​3ce2502/console.html#_2016-04-20_14_24_39_344
15:07 mjblack degorenko: yeah I'll change the command over
15:07 mwhahaha (╯ರ ~ ರ)╯︵ ┻━┻
15:07 mjblack I just updated it with netstat
15:07 degorenko :D
15:08 degorenko and yeah, netstat will never die :D
15:09 mjblack degorenko: funny thing though
15:09 mjblack if ss is not installed...egrep will return 1...and exec will exit out thinking everything is fine
15:09 mwhahaha EmilienM: Thoughts on the key conflict issue -^
15:10 degorenko mjblack, you can update path: /bin, /usr/bin, /sbin
15:10 openstackgerrit Matthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/308012
15:10 mjblack degorenko: why? both commands are in /bin
15:11 degorenko mjblack, weeeeeeell :D http://unix.stackexchange.com/questions/6524​5/ss-command-for-checking-sockets-not-found
15:11 mjblack ...ok I'll go the full measure
15:11 degorenko mjblack, let's wait for ci :)
15:11 mjblack you want me to go /usr/local/bin to be safe? :P
15:12 degorenko mwhahaha, will netstat be removed in xenial?
15:12 mnaser https://github.com/kwilczynski/pupp​et-functions/blob/master/lib/puppet​/parser/functions/is_port_open.rb
15:12 mwhahaha i have no idea
15:12 openstackgerrit Xingchao Yu proposed openstack/puppet-keystone: Refactor keystone::resource::authtoken  https://review.openstack.org/266723
15:12 degorenko mjblack, nope :D
15:12 mnaser something based of that
15:12 mnaser would be interesting
15:12 openstackgerrit Matthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/308012
15:12 mjblack degorenko: well too late :P
15:12 mnaser just write a ruby function/provider/etc to do that
15:12 degorenko mwhahaha, mwhahaha why we can't use then netstat? :D
15:13 mwhahaha i don't actually care enough one way or the other. This is primarily a <=mitaka problem
15:13 mwhahaha i'm sure i'll get address in newton
15:13 mwhahaha just get it working :D
15:14 degorenko :D well, then we can use netstat, i guess
15:14 * mjblack grumbles
15:14 degorenko so this one looks good to me: https://review.openstack.org​/#/c/308012/6/manifests/init.pp
15:15 mwhahaha egrep -q?
15:15 EmilienM dfisher: https://review.openstack.or​g/#/q/topic:puppet/tarball
15:15 EmilienM dfisher: once both patches are merged, I think we're good
15:15 mjblack degorenko: except I messed up
15:15 degorenko mwhahaha, yes, right, -q option missed
15:15 degorenko mjblack, no problem :D
15:16 degorenko mwhahaha, so, you're ok with 6 patch set + -q option?
15:16 mjblack degorenko,mwhahaha: so before I submit again, we using netstat or ss?
15:16 mwhahaha yea
15:16 mwhahaha netstat is fine
15:16 degorenko EmilienM, https://review.openstack.org​/#/c/308012/6/manifests/init.pp - your thoughts?
15:16 EmilienM well, if it works
15:17 cdelatte joined #puppet-openstack
15:17 degorenko mjblack, :D you know what you should to do, right?
15:17 michchap joined #puppet-openstack
15:17 mjblack viva la'netstat
15:17 degorenko o/
15:20 openstackgerrit Matthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/308012
15:21 mjblack degorenko,EmilienM,mwhahaha: ^
15:22 mjblack so now the question becomes how long till the tests get execute :D
15:23 degorenko :D yeah
15:27 mjblack mwhahaha: you able to fix fuel ci issues? :D
15:28 mwhahaha for which one
15:28 mjblack https://ci.fuel-infra.org/job/master.p​uppet-openstack.fuel.noop/485/console
15:28 mjblack its complaining of an error cloning
15:29 mwhahaha yea i can retrigger
15:30 mwhahaha restarted
15:35 alee joined #puppet-openstack
15:35 alee EmilienM, any progress in https://review.openstack.org/#/c/305886/ ?
15:36 EmilienM alee: it's failing, I need to investigate. I was on something else until now
15:36 alee EmilienM, ok
15:37 mdorman joined #puppet-openstack
15:37 openstackgerrit Merged openstack/puppet-tripleo: IPv6 dual-stack support  https://review.openstack.org/286344
15:38 mjblack EmilienM: just out of curiosity, why is it that the CI jobs that ubuntu is delayed?
15:39 mwhahaha it shouldn't be today
15:39 mwhahaha but the ubuntu vms are the most used
15:39 EmilienM because OpenStack Infra is consumming clouds, and have limited resoruces
15:39 EmilienM and yeah, ubuntu are very used by devstack jobs, etc
15:39 EmilienM so we better deploy on centos7
15:40 mjblack ah ok, I was figuring as much...
15:40 openstackgerrit Merged openstack/puppet-openstack-integration: repos/ubuntu: install repos before any package  https://review.openstack.org/307330
15:40 mwhahaha fyi beaker on centos7 is hosed
15:41 EmilienM mwhahaha: what?
15:42 mwhahaha you can scroll back in #openstack-infra but the puppetlabs release rpm is conflicting
15:42 mwhahaha http://logs.openstack.org/77/306677/9/check/g​ate-puppet-nova-puppet-beaker-rspec-centos-7/​3ce2502/console.html#_2016-04-20_14_24_39_344
15:42 mjblack yeah it just happened with the puppet-keystone change of mine
15:42 mwhahaha so my virtlock/virtlog patch that i've been trying to land for 3  days now is once again hosed by CI :D
15:43 mwhahaha i'm just glad we don't run beaker in fuel so i can land it over there
15:44 mjblack rpm should be replacing the gpg file
15:44 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Change source for puppet-corosync  https://review.openstack.org/308424
15:45 mjblack EmilienM,mwhahaha: it looks like we should be passing in --replacefiles too
15:45 EmilienM I'm wondering if it's related to latest beaker version
15:45 EmilienM crinkle: should we pin beaker to previous release?
15:45 mwhahaha no that code hasn't changed in beaker in a year
15:46 EmilienM oh ok
15:46 mwhahaha they are working on fixing the images i think see #openstack-infra
15:46 EmilienM so all our beaker jobs are failing atm
15:46 mjblack the issue could be puppetlabs changed their gpg key
15:46 crinkle EmilienM: no its related to the images having puppet preinstalled on them
15:46 EmilienM oh oki
15:46 mjblack EmilienM: if we pass in --replacefiles we could get around it
15:46 crinkle yolanda is working on updating the images
15:47 EmilienM mwhahaha: can you get https://review.openstack.org/#/c/307972/ and https://review.openstack.org/#/c/308001/ please?
15:48 mwhahaha yea
15:50 cdelatte joined #puppet-openstack
15:52 yolanda EmilienM, i pushed  https://review.openstack.org/308426
15:52 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario002: test ironic on Ubuntu again  https://review.openstack.org/307977
15:53 EmilienM yolanda: thx!
15:55 iurygregory joined #puppet-openstack
15:57 michchap_ joined #puppet-openstack
16:08 magicboiz joined #puppet-openstack
16:12 cdelatte joined #puppet-openstack
16:15 magicboiz joined #puppet-openstack
16:16 cdelatte joined #puppet-openstack
16:18 haplo37 joined #puppet-openstack
16:23 karimb joined #puppet-openstack
16:24 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Change scheduler configurations to use entrypoints  https://review.openstack.org/308452
16:26 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/305886
16:27 openstackgerrit Merged openstack/puppet-openstack-integration: scenario001: test telemetry on Ubuntu again  https://review.openstack.org/307972
16:29 twm2016 joined #puppet-openstack
16:29 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: test telemetry on Ubuntu again  https://review.openstack.org/308454
16:32 openstackgerrit Emilien Macchi proposed openstack/puppet-tempest: Update spec tests for Tempest  https://review.openstack.org/308241
16:32 cwolferh joined #puppet-openstack
16:32 openstackgerrit Merged openstack/puppet-openstack-integration: centos: deploy cinder in SSL  https://review.openstack.org/308001
16:35 bnemec joined #puppet-openstack
16:35 openstackgerrit Emilien Macchi proposed openstack/puppet-tempest: venv: create venv & run pip in exec  https://review.openstack.org/307853
16:41 openstackgerrit Merged openstack/puppet-openstack-integration: functions: add defaults for ZUUL_BRANCH & ZUUL_REF  https://review.openstack.org/308333
16:42 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-glance: Remove docs duplication for notification_driver  https://review.openstack.org/308467
16:42 gchamoul joined #puppet-openstack
16:45 openstackgerrit Emilien Macchi proposed openstack/puppet-tempest: Add support for Gnocchi tests  https://review.openstack.org/308469
16:48 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/305886
16:48 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: run Gnocchi tempest tests  https://review.openstack.org/308473
16:49 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario001: run Gnocchi tempest tests  https://review.openstack.org/308473
16:50 cdelatte joined #puppet-openstack
16:55 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Add missing bit to the user prefetch removal.  https://review.openstack.org/306313
17:00 chem joined #puppet-openstack
17:00 EmilienM crinkle, yolanda: so beaker jobs are broken until next centos7 image build & upload to clouds?
17:00 EmilienM (when https://review.openstack.org/#/c/308426/ is merged)
17:01 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_spec_helper: CI test - never merge  https://review.openstack.org/308478
17:05 crinkle EmilienM: we could replace the run_puppet_install_helper in puppet-openstack_spec_helper with something else but i think it would be better to wait
17:05 EmilienM crinkle: how do you estimate the waiting time?
17:05 crinkle EmilienM: not sure, yolanda ^
17:06 EmilienM if it's breaking our CI > 24h I'll find a workaround
17:06 crinkle i'm sure it will be less than 24h
17:08 mjblack EmilienM: sigh... http://logs.openstack.org/12/308012/9/check/gate​-puppet-keystone-puppet-beaker-rspec-ubuntu-trus​ty/b3303de/console.html#_2016-04-20_17_03_08_931
17:14 mjblack EmilienM,mwhahaha,degorenko: going to do this... netstat -ltnp|egrep "(5000|35357)"|awk '{split($7,pids,"/"); print pids[1]}'|xargs -r kill -9
17:14 mwhahaha nooooes
17:14 degorenko mjblack, why we need this perfect structure? :)
17:15 mjblack mwhahaha,degorenko: in the beaker tests it still didnt die after 60 seconds
17:16 mwhahaha now that's a bug that we could probably get pushed to be fixed
17:16 mwhahaha it should stop :D
17:17 mwhahaha might need to turn on debug logs
17:17 * mwhahaha ducks
17:17 mjblack mwhahaha: I'm at this point right now... http://imgur.com/Qt1ZCMI
17:17 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-heat: Cleanup rabbit ssl checks  https://review.openstack.org/308487
17:19 mjblack degorenko: I'll even add a comment quote "why wont you die"....
17:19 degorenko mjblack, i hope not for me :D
17:19 mjblack no, just around the exec call :P
17:19 degorenko mjblack, heh :)
17:20 mjblack if a kill -9 doesnt fix this...
17:21 degorenko mjblack, how then sleep 35 works?
17:21 mjblack hey hey
17:21 mjblack we were using logs to see it take 31 seconds
17:22 openstackgerrit Matthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/308012
17:22 mjblack degorenko: ^ please review :D
17:23 openstackgerrit Matthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/308012
17:23 degorenko mjblack, looking L)
17:23 degorenko :)
17:23 mjblack degorenko: look at the one I just pushed
17:23 iberezovskiy cool exec name
17:23 degorenko yeah
17:24 degorenko ah, so beautiful command in exec
17:24 degorenko :D
17:25 mjblack degorenko: so you're saying that for me to get a +2 out of you, I just need sed/awk stuff in my manifests?
17:25 degorenko mjblack, anyway, it works for sleep 35, why it can't work for netstatt?
17:25 ron____ joined #puppet-openstack
17:28 mjblack degorenko: what works for sleep?
17:29 mjblack degorenko: if this this doesnt work, I'm going to just for loop it 60 times...
17:30 degorenko mjblack, your commit with sleep 35 option - passed CI, why CI wont pass with netstat and 60 seconds?
17:31 mwhahaha thats why i said we might need debug logs cause i'm not sure it's looping (or working correctly)
17:31 mjblack degorenko: I wonder if the load on the hypervisor running the instance is high enough to slow everything down
17:33 mjblack mwhahaha: yeah but I dont think debug will be as helpful, I wonder if hte ports are still in use in a fin_wait
17:33 mwhahaha if you turn on debug and ditch the -q, we could see the output
17:33 mjblack we're only looking for LISTEN
17:33 mjblack so fin_wait wont show up
17:33 mjblack we would have to take out the -l in netstat
17:34 mjblack but lets see if this new change does it
17:34 mwhahaha no -l includes listen
17:34 mwhahaha but it shouldn't exclude fin_wait
17:35 mjblack no it does
17:35 mjblack you would need -a
17:35 mjblack -l is only listening
17:35 degorenko that's strange, but this one https://review.openstack.org​/#/c/308012/9/manifests/init.pp - should work
17:36 mjblack I dont disagree
17:36 mjblack but at this point
17:36 mjblack lets go with the nuclear option and kill the process
17:37 degorenko we want just wait for free port
17:37 degorenko we don't want nuclear weapon :)
17:38 mjblack I think at this point we just want the port freed so CI doesnt fail randomly :D
17:39 mjblack but I can go back to the old one and set it for 5 minute retry....
17:39 mjblack if after 5 minutes its still failing we can go back to the kill process route
17:39 degorenko :D
17:39 degorenko looks like - you're wanna die? no? i will wait. What's about now?
17:40 mjblack too bad puppet doesnt have an on_fail option with exec provider
17:41 iberezovskiy mjblack, can we use unless there?
17:42 mjblack iberezovskiy: in the current change?
17:43 mjblack we can look to add an unless in there, just want to make sure it works
17:43 degorenko lol
17:43 degorenko you know
17:44 degorenko we can use sleep 2 + onlyif netstat -ltn|egrep -q "(5000|35357)
17:44 degorenko :D
17:44 degorenko with tries
17:44 mjblack degorenko: hey hey
17:45 mjblack if we're going to go this route...I'm going to write a type/provider
17:45 mjblack ensure_dead
17:45 degorenko no, i'm joking
17:45 mjblack I'm not :P
17:45 degorenko D:
17:45 mjblack ensure_dead { 'keystone': with_extreme_prejudice => true }
17:46 degorenko i really don't understand, why this is not working: https://review.openstack.org​/#/c/308012/9/manifests/init.pp
17:46 degorenko mjblack, can you do this: ensure_dead { 'keystone': head_shot => true } ?
17:47 mjblack that would be like deleting the init/upstart/systemd script
17:47 degorenko that doesn't matter, i want head_shot feature :D
17:48 degorenko mjblack, do you have any hardware to check your patch?
17:48 mjblack I do
17:48 mjblack it works fine here
17:48 degorenko magic
17:48 degorenko i have same result
17:49 mjblack I say we pre-bake the beaker/scenario images with keystone installed, update-rc.d with it turned off
17:50 degorenko mjblack, you can back your 9 patch set + depends-on test commit to openstack integration with enabling debug and verbose.
17:50 degorenko may be we will see reason for -1
17:50 openstackgerrit Merged openstack/puppet-octavia: Totally drop Qpid support  https://review.openstack.org/307182
17:51 mjblack degorenko: which commit to depend on?
17:51 degorenko mjblack, keystone commit + depends-on to openstack integration
17:51 degorenko your commit :)
17:55 mjblack which commit specifically on openstack-integration?
17:56 degorenko mjblack, no, upload a new commit to openstack-intehration with enabling debug logging for puppet in scenario
17:57 colonwq EmilienM or others, Has anyone else reported a problem with gate-puppet-ceph-puppet-beaker-rspec-centos-7? http://logs.openstack.org/15/299615/5/check/g​ate-puppet-ceph-puppet-beaker-rspec-centos-7/​b3e1305/console.html#_2016-04-20_13_10_30_719
17:57 degorenko oops, that's not scenario failes
17:57 degorenko fails*
17:57 mjblack colonwq: they know about it, there is a change to fix the issue
17:58 degorenko mjblack, scenario jobs are under keystone+apache
17:58 degorenko so want work only beakers. that's double magic
17:58 degorenko wont*
17:58 mjblack colonwq: https://review.openstack.org/#/c/308426/
17:58 colonwq mjblack, ok. thanks. I have not been around long enough today to follow a conversation.
17:58 mjblack colonwq: np, thats the review pertaining to the issue
17:59 mjblack degorenko: can we wait to see if the nuclear option works? :D
17:59 mjblack at the very minimum we have a fall back option :)
18:00 colonwq mjblack, Ok. I'll keep an eye on it working through the process.
18:01 degorenko mjblack, we can :) sure
18:01 mjblack degorenko: co-worker says the provider should be "nuke_it_from_space_its_the_only_way_to_be_sure { 'keystone': }"
18:02 degorenko mjblack, then you need inspect logs with radiation shield :D
18:02 mjblack degorenko: I can always change the exec call to a reboot
18:02 mjblack thats one way to be sure :D
18:02 mjblack update-rc.d keystone disable;reboot
18:03 degorenko we want kill only keystone, not a full server
18:03 mjblack hmmm
18:04 mjblack degorenko: I have a better idea
18:04 mjblack if $::osfamily == 'Debian' { Package<| title == 'keystone' |> { provider => pip } }
18:04 EmilienM lol
18:04 mjblack problem solved
18:05 degorenko no please
18:06 mjblack degorenko: how aobut if $::osfamily == 'Debian' { fail('Use redhat foo') }
18:06 degorenko mjblack, if $::osfamily == 'Debian' { fail('Hey, relax and deploy Centos') }
18:07 mjblack that'll be patch #2 for the review
18:07 degorenko :D
18:07 mjblack after getting -1's
18:07 mjblack for #1
18:08 mjblack degorenko: you know this could be soooooo much easier if dpkg allowed us to noscripts packages
18:09 degorenko mjblack, you mean no init scripts?
18:09 mjblack I mean like rpm has noscripts option
18:10 degorenko mjblack, ah, do not execute pre and post scripts
18:11 mjblack although...
18:11 mjblack I'm sensing a potential avenue here... --post-invoke=command
18:13 degorenko i don't see such option in my dpkg
18:14 mjblack o_O
18:15 mjblack you on 14.04?
18:15 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-heat: Drop all current deprecated parameters for heat  https://review.openstack.org/308511
18:15 degorenko mjblack, yeah
18:15 mjblack weird cause its on mine
18:16 degorenko mjblack, no, its here, it is not showed on --help - man shows
18:17 openstackgerrit Dan Prince proposed openstack/puppet-tripleo: Add neutron profiles  https://review.openstack.org/293436
18:17 openstackgerrit Merged openstack/puppet-oslo: Totally drop Qpid support  https://review.openstack.org/307094
18:18 hjensas joined #puppet-openstack
18:18 mjblack oh oh
18:18 mjblack I have an idea
18:21 mjblack degorenko: what if...we install keystone via exec
18:21 mjblack then we can pass RUNLEVEL=1 and it wont start up keystone...
18:26 degorenko :D oh my gosh
18:26 degorenko please no
18:26 degorenko :D
18:27 mjblack degorenko: no no its ok I got an idea
18:28 mjblack if $::osfamily == 'Debian' { exec { 'install_keystone': command => 'apt-get -y install keystone'; environmnet => 'RUNLEVEL=1' } -> Package['keystone'] }
18:28 mjblack it'll be so much easier that way
18:28 mjblack EmilienM: what do you think?
18:28 * degorenko cries
18:28 mjblack I can throw an awk command in there
18:28 degorenko please stop :)
18:29 mjblack you think I'm joking but that'll actually solve the problem according to docs I read :D
18:29 yolanda joined #puppet-openstack
18:29 degorenko let's just find out why 9 patch set fails
18:29 degorenko may be don't like you?
18:30 degorenko CI i mean
18:30 degorenko :D
18:31 mjblack everybody loves me
18:31 mjblack if they dont they're lying
18:31 degorenko he he
18:31 degorenko ok, i'm off for today :) it's 21-30 in my time
18:32 degorenko we will continue tomorrow :D
18:34 mjblack see ya
18:34 EmilienM _ody: do you know if puppetlans plans to provide Xenial packaging on http://apt.puppetlabs.com/ ?
18:35 mjblack EmilienM: what are your thoughts on the most ugly hack to fix the CI problem?
18:36 EmilienM I don't know
18:36 EmilienM I haven't followed everything here, I was pretty busy by other things
18:36 EmilienM mjblack: why just we don't add a sleep?
18:36 mjblack EmilienM: essentially install keystone via an exec where we can pass RUNLEVEL=1 as an env var
18:36 mjblack and it wont start up the service
18:37 mjblack exec { .. } -> Package['keystone']
18:37 mjblack with an unless => 'dpkg -l|grep keystone'
18:37 EmilienM the problem is we install repos before Package resource
18:38 mjblack I know
18:38 EmilienM so you'll need to make sure orchestration is correct
18:38 mjblack its not perfect, just has to be there for mitaka/liberty
18:39 EmilienM go for it then
18:39 mjblack degorenko isnt going to like it
18:40 openstackgerrit Bradley Klein proposed openstack/puppet-monasca: Support pruning metrics by dimension(s)  https://review.openstack.org/308522
18:40 EmilienM what does he suggest?
18:47 mjblack EmilienM: trying to see why its not dying quickly
18:47 mjblack EmilienM: this is what I'm proposing though... http://paste.openstack.org/show/494873/
18:48 * mwhahaha runs away
18:49 mjblack mwhahaha: I'm still all for doing if $::osfamily == 'Debian' { fail('use centos') }
18:49 mwhahaha heh
18:51 mjblack mwhahaha: but I'm sure that change will get -2 from like everyone
18:51 mwhahaha yes, me included :D\
18:54 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Support of Ubuntu 16.04 LTS Xenial deployments  https://review.openstack.org/308530
18:55 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: Support of Ubuntu 16.04 LTS Xenial deployments  https://review.openstack.org/308530
18:56 v1k0d3n joined #puppet-openstack
19:08 mwhahaha EmilienM: should we switch beaker centos7 to non-voting? :/
19:12 mjblack mwhahaha: only if we make ubuntu scenario testing non-voting too :D
19:12 mwhahaha so this is the one case where ubuntu is working :D
19:13 mjblack yeah...it does rspec...but thats about it :D
19:13 mwhahaha well at least it gets to rspec
19:13 * mwhahaha throws things
19:15 colonwq if it passes lint, it can be good enough. :)
19:17 EmilienM mwhahaha: if there is no fix today, we'll workaround in spec_helper
19:17 mwhahaha k
19:18 EmilienM but I'm not in favor of disabling voting on centos7
19:19 EmilienM I'll work on the workaround a bit later, so it's ready in case of we need it
19:19 mwhahaha at this rate i think we need it :(
19:23 yolanda joined #puppet-openstack
19:24 openstackgerrit Alex Schultz proposed openstack/puppet-openstack_spec_helper: Remote puppetlabs-release prior to running beaker  https://review.openstack.org/308540
19:25 mjblack EmilienM: hey, we can merge my kill option...it passed tripleo :P
19:25 openstackgerrit Alex Schultz proposed openstack/puppet-openstack_spec_helper: Remove puppetlabs-release prior to running beaker  https://review.openstack.org/308540
19:25 mjblack and 2 out of 3 fuel ci tests
19:25 mwhahaha ಠ_ಠ
19:27 ducttape_ joined #puppet-openstack
19:28 mjblack mwhahaha: ლ(ಠ益ಠლ)
19:29 openstackgerrit Alex Schultz proposed openstack/puppet-nova: CI test  https://review.openstack.org/308543
19:31 rcernin joined #puppet-openstack
19:35 mjblack mwhahaha: so good news...we get to see if the kill option works :D
19:35 mjblack it finally started doing the ubuntu tets
19:35 mjblack *tests
19:39 mjblack well that failed...
19:40 openstackgerrit Matthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/308012
19:41 dfisher thanks for localizing that to Debian.
19:41 dfisher on Solaris we only run Keystone as WSGI so …. yeah.
19:43 mjblack dfisher: at this point I want to see about just putting out policy-rc.d before the package and then remove it after package install
19:44 dfisher that's … magic Debian/Ubuntu things … right?
19:44 mjblack maybe we fix the issue by converting the ubuntu system to centos
19:44 dfisher hah
19:44 mjblack I'm sure thats less pain
19:44 dfisher you mispelled 'solaris' again
19:45 mjblack what this you're talking about? you're acting like this is 1998?
19:45 * dfisher sobs
19:46 mwhahaha slowaris?
19:46 dfisher pile it on.
19:47 grimlock86 joined #puppet-openstack
19:47 mwhahaha touch the sparc, go on... touch it
19:47 mjblack why would I want to run linux at a 100000% mark up price?
19:47 dfisher yep.  keep going....
19:47 dfisher spread the FUD thick.
19:47 * dfisher waits
19:47 dfisher i'm sadly used to this.
19:47 mjblack dfisher: its not FUD when you account for Oracle pricing
19:47 dfisher :(
19:48 EmilienM mjblack: we'll review it when all CI will have tested ths change
19:48 EmilienM ok /me working on workaround for beaker jobs now
19:48 dfisher it is when I point out things from 3rd party sites that can clearly demonstrate TCO for SPARC is less than Linux but … whatever
19:48 mwhahaha EmilienM: i proposed one -^
19:48 mjblack EmilienM: previous patch failed, same issue...
19:48 dfisher again, this is the wrong room to discuss this :)
19:49 dfisher goozeyx poked the bear (me) last week
19:49 EmilienM mwhahaha: you're faster, sorry I was afk for lunch
19:49 mjblack dfisher: Oracle wanted to charge $25k per socket for an ldap server
19:49 EmilienM mwhahaha: this is not enough
19:49 mwhahaha yea
19:49 mwhahaha i just saw the failure
19:49 EmilienM mwhahaha: we need to patch the lib/
19:49 EmilienM mwhahaha: and not the bash script
19:50 dfisher mjblack: even I'll ask why you wanted to use SPARC for LDAP.
19:50 * dfisher shrugs
19:50 mwhahaha yea wasn't sure the best place, i was trying to go simple first
19:50 EmilienM we need to replace run_puppet_install_helper
19:51 mjblack dfisher: that wasnt sparc
19:51 mjblack I think that was their ldap server for oracle linux
19:51 EmilienM mwhahaha: we need to rewrite https://github.com/puppetlabs/beaker​-puppet_install_helper/blob/master/l​ib/beaker/puppet_install_helper.rb
19:51 dfisher i have nothing to say about Oracle Linux
19:51 mwhahaha i don't know if we need to go that far
19:52 mjblack dfisher: I'm sure they had one for slowaris :D
19:52 * dfisher sighs
19:52 EmilienM dfisher, mjblack: you guys need a room
19:52 dfisher this one works fine :)
19:52 EmilienM /join #openstack-solaris
19:52 mjblack noooo, thats like hell
19:52 dfisher IM SO LONELY THERE
19:53 EmilienM mwhahaha: we need to modify install_puppet_on
19:54 mwhahaha why can't we just remove puppetlabs-release
19:54 mwhahaha and skip all that trouble
19:54 EmilienM oh
19:54 EmilienM yeah
19:54 mwhahaha like in our beaker_spec_helper
19:54 mwhahaha let me fix my patch real quick
19:54 EmilienM mwhahaha: sure
19:55 openstackgerrit Alex Schultz proposed openstack/puppet-openstack_spec_helper: Remove puppetlabs-release prior to running beaker  https://review.openstack.org/308540
19:56 ducttape_ joined #puppet-openstack
19:57 EmilienM mwhahaha: it will fail on ubuntu :P
19:57 mwhahaha at the moment i'm just trying to see if it'll work ;)
19:57 EmilienM ok sure
20:03 mwhahaha we don't have an ubuntu beaker job for openstack_spec_helper
20:03 * mwhahaha waits patiently
20:03 EmilienM mwhahaha: oh right
20:03 mwhahaha i've got a puppet-nova change to check it on next patchset if it passes
20:05 openstackgerrit Matthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/308012
20:12 shardy joined #puppet-openstack
20:13 mwhahaha nope failed :/
20:13 * mwhahaha waits for logs
20:26 grimlock861 joined #puppet-openstack
20:27 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Deprecations and updates for Newton  https://review.openstack.org/308573
20:28 mwhahaha I think you're right. I think we have to override that class :/
20:29 EmilienM mwhahaha: can't we drop the package before?
20:29 mwhahaha it's a matter of being able to do that
20:30 EmilienM why could not we be able to do that?
20:30 mwhahaha https://review.openstack.org/#/c/308540/ failed
20:30 mwhahaha and i didn't see my yum remove
20:31 mwhahaha even tho i put it in the beaker_spec_helper
20:31 EmilienM is it good syntax to put it like this?
20:31 mwhahaha yea that's just an exec
20:32 mwhahaha but i think the inclusion of require 'beaker/puppet_install_helper' gets ran before my remove
20:32 EmilienM mwhahaha: this inclusion does not run it
20:33 mwhahaha yea that's what i am trying to figure out
20:33 EmilienM I'm testing something else
20:35 mwhahaha i think we have to run the removal on the host beaker is running on
20:35 mwhahaha cause doesn't beaker fire up a vm of some sort? and editing the spec helper runs it on the parent vm
20:38 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_spec_helper: Remove puppetlabs-release before installing puppet  https://review.openstack.org/308580
20:38 EmilienM mwhahaha: can it work? ^
20:39 mwhahaha maybe
20:39 EmilienM mwhahaha: you remember the picture with the dog?
20:39 mwhahaha yea i'm already there :D
20:39 EmilienM it's me, right now
20:40 mjblack what picture of a dog?
20:40 EmilienM mjblack: http://goo.gl/DLkLXR
20:40 dfisher haha
20:41 dfisher http://images.performgroup.com/di/libr​ary/sporting_news/12/66/this-is-fine-d​og-embed_c1wcflsukty01jr4jtcy11rrx.jpg   <— better version
20:42 mjblack dfisher: oh come on, you're missing the other two panels
20:43 mjblack http://i0.kym-cdn.com/photos/ima​ges/newsfeed/000/962/640/658.png
20:43 dfisher i know :)
20:45 mjblack EmilienM: I thought this was you... http://s2.quickmeme.com/img/e0/e03c7470565050e7c9d​bcaf21368e62b23653a73b101fcd71e7f895620ae1619.jpg
20:45 agrebennikov joined #puppet-openstack
20:46 dfisher that's all of us.
20:46 EmilienM mjblack: oh man, this guy looks like me
20:46 EmilienM mjblack: have we meet each other?
20:46 mjblack EmilienM: lol thats why I posted it
20:46 mjblack no we havent
20:46 EmilienM you found my twin
20:47 EmilienM he's java dev
20:47 EmilienM I wanted to keep it secret, you found him.
20:47 mjblack why java? that could be that puppet ide
20:47 mjblack it looks like eclipse
20:50 mwhahaha vi4life
20:51 mjblack mwhahaha: guy I work with uses nano, drives me crazy
20:52 mjblack I keep telling him to use a real editor
20:52 mwhahaha i knew a guy who did that
20:52 mjblack EmilienM: you can use this one though... http://digitalsynopsis.com/wp-cont​ent/uploads/2015/03/web-designer-d​eveloper-jokes-humour-funny-41.jpg
20:54 EmilienM I'm more a guy at the right
20:55 mjblack EmilienM: you know why they skipped windows 9?
20:55 mwhahaha seven ate nine?
20:55 EmilienM I didn't even know we were so far
20:55 ntpttr EmilienM: I noticed that this patch has been idle for quite a while after the -1, and I had a patch that I abandoned in favor of it - do you think it's been long enough that I should just unabandon my patch with the correct acceptance test? https://review.openstack.org/#/c/275223/10
20:56 mjblack someone probably had an if condition trying to look for windows 95 or 98 that was "Windows 9"
20:56 EmilienM ntpttr: yes go ahead please
20:57 ntpttr EmilienM: sweet will do
21:00 dfisher puppet resource requires a full type definition, right?  so since puppetlabs-ntp doesn't provide a type, there's nothing for puppet to assemble?
21:03 EmilienM mwhahaha: I don't see the results yet but beaker job is running for 11 min :P
21:03 mwhahaha oo progress then
21:03 EmilienM it should have failed before, isn't?
21:03 EmilienM (if not working)
21:05 mwhahaha beaker failed before the unit test last time
21:05 mwhahaha so  it might be working
21:06 EmilienM SUCCESS
21:06 mjblack dfisher: thats right, a define is not a type
21:06 EmilienM ok, I think we have a workaround
21:06 mwhahaha k
21:07 mjblack dfisher: and puppet resource will only work with types
21:07 dfisher thanks.
21:07 EmilienM mwhahaha: http://logs.openstack.org/80/308580/1/check/gate-​puppet-openstack-spec-helper-beaker-rspec-centos-​7/5bf184e/console.html#_2016-04-20_20_56_46_217
21:08 EmilienM crinkle: I'll let you +A (or not) https://review.openstack.org/#/c/308580 please
21:08 openstackgerrit Bradley Klein proposed openstack/puppet-monasca: Add ability to include libvirt scale group metadata  https://review.openstack.org/308589
21:09 crinkle EmilienM: +A
21:10 EmilienM mwhahaha: the bad news is it will fix stable/mitaka and master
21:10 EmilienM mwhahaha: I did not change stable/liberty spec/spec_helper_acceptance.rb
21:10 EmilienM but I think we can live without liberty during 1 or 2 days
21:10 EmilienM crinkle: thanks
21:11 EmilienM we'll revert that patch once it's fixed by infra
21:11 openstackgerrit James Slagle proposed openstack/puppet-tripleo: Add dport/sport parameter to firewall rule  https://review.openstack.org/308358
21:11 mwhahaha yea i think it's ok
21:12 openstackgerrit James Slagle proposed openstack/puppet-tripleo: Add dport/sport parameter to firewall rule  https://review.openstack.org/308358
21:15 EmilienM I'm out, but back later in late evening.
21:26 openstackgerrit Merged openstack/puppet-openstack_spec_helper: Remove puppetlabs-release before installing puppet  https://review.openstack.org/308580
21:54 openstackgerrit Merged openstack/puppet-monasca: Support pruning metrics by dimension(s)  https://review.openstack.org/308522
21:58 mjblack well
21:58 mjblack so far so good
22:16 openstackgerrit Merged openstack/puppet-tripleo: Add Glance profiles  https://review.openstack.org/296076
22:23 grimlock86 joined #puppet-openstack
22:32 openstackgerrit Nate Potter proposed openstack/puppet-magnum: Add magnum-conductor  https://review.openstack.org/285847
22:33 ayoung joined #puppet-openstack
22:41 ducttape_ joined #puppet-openstack
23:08 johnmilton joined #puppet-openstack
23:25 johnmilton joined #puppet-openstack
23:31 dmsimard mjblack: I'm late but re: nano https://xkcd.com/378/
23:32 mjblack lol
23:32 mjblack vi is the best tho
23:32 dmsimard vim*
23:32 dmsimard vi sux.
23:32 mjblack well
23:32 mjblack thats why my zshrc has alias vi=
23:32 mjblack ':D
23:32 mjblack bah
23:32 mjblack alias vi='vim'
23:33 mjblack I save a whole character now
23:39 dmsimard I think most of the time when you install vim, vi redirects to vim automagically
23:39 dmsimard I've never really wondered why
23:45 mjblack dmsimard: not always
23:45 mjblack only on some linux distros

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