Wednesday, 2016-04-20

« Tuesday, 2016-04-19 Index Thursday, 2016-04-21 »
openstackgerritMerged openstack/puppet-gnocchi: Gemfile: rely on puppet-openstack_spec_helper for dependencies  https://review.openstack.org/30720100:05
*** spredzy has quit IRC00:09
*** spredzy has joined #puppet-openstack00:15
*** fawadkhaliq has joined #puppet-openstack00:25
openstackgerritzhongshengping proposed openstack/puppet-trove: Remove the extra spaces in the conductor.pp  https://review.openstack.org/30809600:41
*** mesamoo has quit IRC00:46
*** haplo37 has joined #puppet-openstack00:49
*** Matum has quit IRC00:54
*** fawadkhaliq has quit IRC00:56
*** fawadkhaliq has joined #puppet-openstack00:57
*** johnmilton has quit IRC01:05
*** ayoung has joined #puppet-openstack01:09
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Correct typo resulting in a hardcoded paramter used incorrectly  https://review.openstack.org/29961501:21
*** saneax is now known as saneax_AFK01:27
openstackgerritMatt Fischer proposed openstack/puppet-trove: deprecate templated guestagent config  https://review.openstack.org/30790301:51
*** ducttape_ has joined #puppet-openstack01:59
*** ducttape_ has quit IRC02:09
*** fedexo has joined #puppet-openstack02:37
*** delattec has quit IRC02:40
*** ducttape_ has joined #puppet-openstack02:49
*** richm has quit IRC02:50
*** fawadkhaliq has quit IRC02:53
mwhahahaah damn it my puppet-nova failed on the verify step03:00
mwhahahastupid puppet labs repo 40303:01
mwhahahabut if anyone cares, the centos7 puppet jobs appear to be broken because puppetlab's release rpm is returning a 40303:09
mwhahahahttp://logs.openstack.org/77/306677/9/gate/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/061bc66/console.html#_2016-04-20_01_23_14_21203:09
colonwqmwhahaha, thanks. I am waiting for mine to (slowly) finish. I'll be on the look out for it.03:11
mwhahahait's going to fail03:12
crinklemwhahaha: i think someone already reported the issue https://tickets.puppetlabs.com/browse/CPR-33203:12
mwhahahaover in #puppet the el6 is missing, looks like they renamed it03:12
mwhahahahttp://yum.puppetlabs.com/puppetlabs-release-pc1-el-7.noarch.rpm03:13
crinklebut i think the release team is mostly US-based and therefor not at work right now03:13
* mwhahaha flips tables03:13
colonwqhopefully the debian test will pass. I had a keyboard/brain issue in my patch03:15
*** ducttape_ has quit IRC03:24
*** haplo37 has quit IRC03:39
*** ayoung has quit IRC03:40
*** chandankumar has joined #puppet-openstack04:26
*** fawadkhaliq has joined #puppet-openstack04:28
*** fedexo has quit IRC04:29
*** markvoelker has quit IRC04:34
*** markvoelker has joined #puppet-openstack04:35
*** markvoelker has quit IRC04:40
*** imcsk8 has quit IRC04:42
*** imcsk8 has joined #puppet-openstack04:44
*** chandankumar has quit IRC04:54
*** chandankumar has joined #puppet-openstack05:10
*** saneax_AFK is now known as saneax05:40
*** jmikolajczak has joined #puppet-openstack05:44
*** dc_mattj has joined #puppet-openstack05:47
*** dc_mattj has quit IRC05:53
*** ericksonsantos has quit IRC06:13
*** rcernin has joined #puppet-openstack06:21
*** hjensas has quit IRC06:22
*** amoralej|off is now known as amoralej06:45
*** tesseract has joined #puppet-openstack06:47
*** tesseract is now known as Guest6708206:48
*** matt__ has joined #puppet-openstack06:50
*** jaosorior has joined #puppet-openstack06:57
*** matt__ has left #puppet-openstack06:58
*** dgurtner has joined #puppet-openstack06:58
*** dgurtner has quit IRC06:58
*** dgurtner has joined #puppet-openstack06:58
*** matt__ has joined #puppet-openstack06:59
*** permalac has joined #puppet-openstack07:00
*** matt__ has left #puppet-openstack07:00
*** dc_mattj has joined #puppet-openstack07:00
*** magicboiz has joined #puppet-openstack07:01
*** magicboiz has left #puppet-openstack07:02
*** dgurtner has quit IRC07:16
*** dgurtner has joined #puppet-openstack07:18
*** samuelson has quit IRC07:22
*** foxhunt_away is now known as foxhunt7207:22
*** jpena|off is now known as jpena07:22
*** jprs has joined #puppet-openstack07:24
*** leanderthal|afk is now known as leanderthal07:31
*** hjensas has joined #puppet-openstack07:34
*** hjensas has joined #puppet-openstack07:34
*** spredzy has quit IRC07:35
*** paramite has joined #puppet-openstack07:36
*** shardy has joined #puppet-openstack07:36
*** fawadkhaliq has quit IRC07:45
*** fawadkhaliq has joined #puppet-openstack07:45
*** paramite is now known as paramite|afk07:47
*** gfidente has joined #puppet-openstack07:59
*** fawadkhaliq has quit IRC08:02
openstackgerritMykyta Karpin proposed openstack/puppet-ironic: Switch ironic to use puppet-oslo module  https://review.openstack.org/30529908:08
*** karimb has joined #puppet-openstack08:08
*** jistr has joined #puppet-openstack08:18
openstackgerritAlexey Deryugin proposed openstack/puppet-trove: Use oslo module for messaging (rabbit) configuration  https://review.openstack.org/30790708:36
*** derekh has joined #puppet-openstack08:37
*** shardy has quit IRC08:38
*** shardy has joined #puppet-openstack08:38
*** _ody has quit IRC08:41
*** karimb has quit IRC08:42
*** mgould has joined #puppet-openstack08:47
*** karimb has joined #puppet-openstack08:53
openstackgerritAlexey Deryugin proposed openstack/puppet-trove: Use oslo module for loggind and database configuration  https://review.openstack.org/30820708:58
*** dgurtner has quit IRC08:59
openstackgerritAlexey Deryugin proposed openstack/puppet-trove: Use oslo module for logging and database configuration  https://review.openstack.org/30820709:00
*** dgurtner has joined #puppet-openstack09:02
*** dgurtner has quit IRC09:02
*** dgurtner has joined #puppet-openstack09:02
*** dtantsur|afk is now known as dtantsur09:18
*** _degorenko|afk is now known as degorenko09:20
*** _ody has joined #puppet-openstack09:50
*** _ody has quit IRC09:56
*** _ody has joined #puppet-openstack09:56
openstackgerritAlexey Deryugin proposed openstack/puppet-trove: Use oslo module for messaging (rabbit) configuration  https://review.openstack.org/30790709:56
openstackgerritAlexey Deryugin proposed openstack/puppet-trove: Use oslo module for logging and database configuration  https://review.openstack.org/30820709:57
*** dc_mattj has quit IRC10:03
*** dc_mattj has joined #puppet-openstack10:05
openstackgerritDenis Egorenko proposed openstack/puppet-tempest: Update spec tests for Tempest  https://review.openstack.org/30824110:05
*** chandankumar has quit IRC10:19
*** paramite|afk is now known as paramite10:26
*** karimb has quit IRC10:33
*** karimb has joined #puppet-openstack10:37
*** paramite is now known as paramite|afk10:37
*** chandankumar has joined #puppet-openstack10:38
*** paramite|afk is now known as paramite10:39
*** gfidente has quit IRC10:46
*** dc_mattj has quit IRC10:48
*** jaosorior has quit IRC10:52
*** jaosorior has joined #puppet-openstack10:53
openstackgerritMiles Gould proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/30827111:03
openstackgerritMiles Gould proposed openstack/puppet-ironic: Copy iPXE ROMs into /tftpboot  https://review.openstack.org/30827211:03
openstackgerritXingchao Yu proposed openstack/puppet-oslo: Add oslo.middleware defines  https://review.openstack.org/27442211:08
*** johnmilton has joined #puppet-openstack11:13
*** trown|outtypewww is now known as trown11:25
*** karimb has quit IRC11:33
*** chandankumar has quit IRC11:33
*** ducttape_ has joined #puppet-openstack11:45
openstackgerritXingchao Yu proposed openstack/puppet-keystone: Refactor keystone::resource::authtoken  https://review.openstack.org/26672311:45
*** gfidente has joined #puppet-openstack11:46
*** morazi has joined #puppet-openstack11:56
*** dprince has joined #puppet-openstack11:57
*** ansmith has joined #puppet-openstack12:02
openstackgerritSergii Golovatiuk proposed openstack/puppet-horizon: Move notification from File to Concat  https://review.openstack.org/30829412:05
*** amoralej is now known as amoralej|lunch12:14
*** cdelatte has joined #puppet-openstack12:17
*** michchap_ has quit IRC12:17
*** markvoelker has joined #puppet-openstack12:18
*** michchap has joined #puppet-openstack12:22
*** ducttape_ has quit IRC12:24
*** michchap_ has joined #puppet-openstack12:31
*** michchap has quit IRC12:32
*** iurygregory has joined #puppet-openstack12:37
*** karimb has joined #puppet-openstack12:44
iurygregorymorning12:44
iurygregorymfisch, if you have time, can you review https://review.openstack.org/#/c/307940/ ?12:44
*** jpena is now known as jpena|lunch12:45
*** mgould is now known as mgould|lunch12:55
*** ducttape_ has joined #puppet-openstack12:56
*** karimb has quit IRC13:00
*** trown is now known as trown|brb13:00
*** ducttape_ has quit IRC13:01
*** ducttape_ has joined #puppet-openstack13:02
*** ducttape_ has quit IRC13:02
openstackgerritJames Slagle proposed openstack/puppet-tripleo: Add destination parameter to firewall rule  https://review.openstack.org/30831713:04
*** karimb has joined #puppet-openstack13:05
*** ducttape_ has joined #puppet-openstack13:06
*** karimb_ has joined #puppet-openstack13:14
*** richm has joined #puppet-openstack13:15
*** karimb has quit IRC13:16
*** cwolferh has quit IRC13:17
*** ntpttr has quit IRC13:18
*** trown|brb is now known as trown13:19
*** cwolferh has joined #puppet-openstack13:20
*** saneax is now known as saneax_AFK13:36
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: functions: add defaults for ZUUL_BRANCH & ZUUL_REF  https://review.openstack.org/30833313:41
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: functions: add defaults for ZUUL_BRANCH & ZUUL_REF  https://review.openstack.org/30833313:42
EmilienMhello folks13:44
iurygregorymorning13:44
mjblackmorning13:45
*** amoralej|lunch is now known as amoralej13:47
*** jpena|lunch is now known as jpena13:48
*** cdelatte has quit IRC13:48
mnaserjust wanted to give feedback that the puppet-ceph module has been great, centos + storage sig + puppet-ceph = excellent combination13:49
mjblackdegorenko: 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 stopped13:49
mjblackat least not without making it more complicated13:50
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/30588613:50
EmilienMmnaser: excellent news!13:50
*** ayoung has joined #puppet-openstack13:50
colonwqGood morning13:50
mjblackEmilienM: any suggestion?13:51
EmilienMmjblack: no, I like the patch13:53
mjblackEmilienM: 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 better13:53
*** dfisher has joined #puppet-openstack13:54
EmilienMif 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 ahead13:54
degorenkomjblack, can we check port or process instead of sleeps?13:56
EmilienMbahhh13:56
mjblackdegorenko: you mean like have an "onlyif" if the port is detected to be in use?13:56
EmilienMthat's terrible13:56
EmilienMthere is no way to know the port13:57
EmilienMspecially when running in wsgi13:57
degorenkoit's not a strong suggestion13:57
EmilienMwhere port is defined in a separated class13:57
degorenkobut sleep 35 looks ugly13:57
degorenkowe can have some hardware when we need 3613:57
degorenko3713:57
degorenkoand more and more13:57
mjblackdegorenko: 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 uglier13:57
mjblackdegorenko,EmilienM: the alternative, besides messing with policy-rc.d, would be to delay the apache start13:59
openstackgerritLukas Bezdicka proposed openstack/puppet-neutron: LBaaSv2 support  https://review.openstack.org/30834613:59
mjblackhave 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 time13:59
mjblackEmilienM,degorenko: https://jpetazzo.github.io/2013/10/06/policy-rc-d-do-not-start-services-automatically/14:00
*** xarses has quit IRC14:00
mjblackwe can have that as an option maybe in openstack_extras that is turned off but in our CI its turned on14:00
degorenkomjblack, which port uses keystone under apache?14:02
mjblackwhichever the user configures14:02
mjblackby default its 5000 and 3535714:02
*** cdelatte has joined #puppet-openstack14:02
degorenkoin which config/class we specifying this?14:03
mjblackmain class14:03
*** mgould|lunch is now known as mgould14:04
mjblackhttps://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L560-L56114:04
degorenkoso, we are on main class, why we can't check port then?14:06
degorenkoopen it or not?14:06
mwhahahadegorenko: https://review.openstack.org/#/c/305798/ was because at the rate my puppet-nova patch is going to get merged it'll be after summit14:06
mwhahahagiven the CI shenanigans currently occuring14:06
mwhahahabecause UCA is currently broken due to that bug, so i still need to cherry pick one or the other to stable/mitaka14:07
degorenkomwhahaha, so, the plan is to revert 305978 after backport to mitaka?14:09
mwhahahayea14:09
degorenkothis fix we need asap?14:09
mwhahahayea14:09
degorenkook14:09
*** jaosorior has quit IRC14:09
*** jaosorior has joined #puppet-openstack14:09
mwhahahawe want to announce the UCA support in 9.0 :) and it's kinda hard to do when it's broken14:09
degorenkomwhahaha, i've dropped my -114:10
mwhahahacool i've retriggered the neutron test so we'll see14:10
mwhahahait a race between openstack-infra and fuel-ci14:11
mwhahahawhich patch will land first!14:11
*** v1k0d3n has joined #puppet-openstack14:12
degorenkomwhahaha, also, for https://review.openstack.org/#/c/30801214:12
colonwqmwhahaha, will the slow one have to refactor?14:12
degorenkowhy we can't check port here?14:12
mwhahahais there a standard provider for that?14:12
mjblackdegorenko: 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
mwhahahacolonwq: nope they are compatible, just duplicates14:13
mwhahahapolicy-rc.d gets dangerous and that's a lot of code to add for one service14:13
mjblackmwhahaha: there is a package and a config file14:13
colonwqmwhahaha, a ok.14:13
mjblackI'm taking a look at it right now to see if its just a simply install and line in a conf file14:14
mwhahahamjblack: 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
degorenkoi'm agree with mwhahaha for policy-rc.d14:14
mjblackok14:14
mwhahahathe best solution is to make sure the port is available14:14
degorenkoyeah, we can have exec with port checking14:15
mwhahahayea i guess it could be an exec with retry14:15
degorenkoi think it will be best solution, also i don't like useless sleeps :D14:15
degorenkoyeah, retries and sleeps FOR exec, not IN exec14:16
*** jaosorior has quit IRC14:16
mjblackok I'll update with a check/sleep in loop14:16
degorenkothanks14:16
*** chandankumar has joined #puppet-openstack14:17
*** Matum has joined #puppet-openstack14:17
mjblackdegorenko: you ok with an exec that does the loop/check?14:19
openstackgerritJames Slagle proposed openstack/puppet-tripleo: Add destination parameter to firewall rule  https://review.openstack.org/30831714:20
openstackgerritJames Slagle proposed openstack/puppet-tripleo: Add dport/sport parameter to firewall rule  https://review.openstack.org/30835814:20
degorenkomjblack, loop/check for port checks - yep14:20
*** twm2016 has joined #puppet-openstack14:21
*** xarses has joined #puppet-openstack14:22
*** Matum has quit IRC14:25
*** dhollinger has joined #puppet-openstack14:27
*** paramite has quit IRC14:28
openstackgerritSergii Rizvan proposed openstack/puppet-keystone: Remove user/role prefetch to support multi-domain.  https://review.openstack.org/30836514:28
dfishera poll() loop is almost always more desirable than arbitrary sleep calls14:28
dfishera root canal is usually more desirable than arbitrary sleep calls too.14:29
mwhahahai'd rather have arbitrary sleeps than a root canal, but maybe that's my years of dental work speaking :D14:32
*** astupnikov has joined #puppet-openstack14:32
*** chandankumar has quit IRC14:33
*** dprince has quit IRC14:33
EmilienMdfisher: first tarball: http://tarballs.openstack.org/puppet-aodh/14:35
EmilienMdfisher: i'm now ajusting to have all releases, with a better name, etc14:35
EmilienMdfisher: also other modules don't have tarballs yet, I'm on it, like I told you yesterday14:35
dfisherhot damn.14:36
openstackgerritMerged openstack/puppet-openstack-integration: Enable Sahara integration job on Ubuntu  https://review.openstack.org/30791114:37
*** chandankumar has joined #puppet-openstack14:37
EmilienMmnaser: just for the record, ceph -s results in our CI: http://logs.openstack.org/00/307900/1/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7/b6cbcc3/console.html#_2016-04-19_17_05_35_05814:39
*** jaosorior has joined #puppet-openstack14:40
mnaseri guess i broke something somewhere \o/14:40
EmilienMmwhahaha: can we get https://review.openstack.org/#/c/307330/ please?14:40
mnaseroh i did want to submit a review for something14:40
mjblackdegorenko: going to warn you...it's gonna be ugly14:41
*** magicboiz has joined #puppet-openstack14:41
mnaserEmilienM: if ceph::profile::mon is included after ceph::profile::client, puppet poops out.  https://github.com/openstack/puppet-ceph/blob/master/manifests/profile/client.pp#L26-L34 and https://github.com/openstack/puppet-ceph/blob/master/manifests/profile/mon.pp#L39-L4414:43
mnaseris it okay if we add the same if clause in mon.pp?14:43
*** darkxploit has joined #puppet-openstack14:43
mnaseror maybe a warning (looks like it has diff paths)14:44
degorenkomjblack, what's wrong? :)14:49
mjblackbasically writing a shell script in an exec call :D14:49
*** magicboiz has quit IRC14:51
mnaseruh if i can chime in14:51
mnaseri know its a bit more work but maybe you can setup something in openstacklib that can check if a port is free14:51
mjblackmnaser: sure14:51
mnasercause this is likely to be reused across a few more modules14:51
openstackgerritJerzy Mikolajczak proposed openstack/puppet-neutron: Remove deprecated options from agents  https://review.openstack.org/30110614:51
mjblackmnaser: honestly, its a good idea but doing policy-rc.d would ultimately be the best way to handle it :D14:52
mnasertrue, better to have never started it, but what if a user decides to implement this midway14:52
degorenkomjblack, why shell script will be difficult?14:57
degorenkomjblack, netstat -lnt | awk '$6 == "LISTEN" && $4 ~ ".5000"' for example14:57
degorenkowith only substitution for var $port14:57
mjblackwe dont need to sub port14:57
mjblacksince the default port coming out of the deb is 500014:58
degorenkobut we can override it14:58
mjblackbut we also need to make sure admin port is down too14:58
EmilienMit sounds like over-complicated14:58
mjblackdegorenko: we dont need to override it14:58
mjblackI think if we go simple thats fine14:58
degorenkomjblack, netstat -lnt | awk '$6 == "LISTEN" && ($4 ~ ".5000" || $4 ~ ".35357")'14:59
degorenko:D14:59
EmilienMdo not hardcode the ports14:59
EmilienMthey can be changes in deployments14:59
mjblackEmilienM: but thats not the case here14:59
degorenkoas i said above - with sub for ports14:59
*** dprince has joined #puppet-openstack14:59
EmilienMit's a terrible hack, imho14:59
mjblackEmilienM: the default ports get installed as part of the deb, which is what we care about14:59
mjblackdegorenko: btw that doesnt work, it always returns 0 :D15:00
mjblackdegorenko: much rather use netstat -lnt|egrep "(5000|35357)"15:02
degorenkomjblack, it was just example15:02
degorenkoso, yeah, having grep|egprep can help15:03
mjblackI only care about return code15:03
mjblacknow if puppet exec provider would work the way the docs say...15:03
degorenkoit will be different return code on situatuon when we have match and when we don't15:03
mjblackor I can stop being stupid and get my expected return code right :P15:04
mjblackthere we go...15:04
mwhahahause ss15:04
mwhahahanetstat is deprecated15:04
nibalizerjust like ifconfig15:05
mwhahahayup15:05
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/30801215:06
mjblackbah, netstat will never die! :P15:06
degorenkomjblack, ss -lt | egrep "(5000|35357)" looks like we need that15:07
mwhahahahttp://logs.openstack.org/77/306677/9/check/gate-puppet-nova-puppet-beaker-rspec-centos-7/3ce2502/console.html#_2016-04-20_14_24_39_34415:07
mjblackdegorenko: yeah I'll change the command over15:07
mwhahaha(╯ರ ~ ರ）╯︵ ┻━┻15:07
mjblackI just updated it with netstat15:07
degorenko:D15:07
degorenkoand yeah, netstat will never die :D15:08
mjblackdegorenko: funny thing though15:09
mjblackif ss is not installed...egrep will return 1...and exec will exit out thinking everything is fine15:09
mwhahahaEmilienM: Thoughts on the key conflict issue -^15:09
degorenkomjblack, you can update path: /bin, /usr/bin, /sbin15:10
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/30801215:10
mjblackdegorenko: why? both commands are in /bin15:10
degorenkomjblack, weeeeeeell :D http://unix.stackexchange.com/questions/65245/ss-command-for-checking-sockets-not-found15:11
mjblack...ok I'll go the full measure15:11
degorenkomjblack, let's wait for ci :)15:11
mjblackyou want me to go /usr/local/bin to be safe? :P15:12
degorenkomwhahaha, will netstat be removed in xenial?15:12
mnaserhttps://github.com/kwilczynski/puppet-functions/blob/master/lib/puppet/parser/functions/is_port_open.rb15:12
mwhahahai have no idea15:12
openstackgerritXingchao Yu proposed openstack/puppet-keystone: Refactor keystone::resource::authtoken  https://review.openstack.org/26672315:12
degorenkomjblack, nope :D15:12
mnasersomething based of that15:12
mnaserwould be interesting15:12
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/30801215:12
mjblackdegorenko: well too late :P15:12
mnaserjust write a ruby function/provider/etc to do that15:12
degorenkomwhahaha, mwhahaha why we can't use then netstat? :D15:12
mwhahahai don't actually care enough one way or the other. This is primarily a <=mitaka problem15:13
mwhahahai'm sure i'll get address in newton15:13
mwhahahajust get it working :D15:13
degorenko:D well, then we can use netstat, i guess15:14
* mjblack grumbles 15:14
degorenkoso this one looks good to me: https://review.openstack.org/#/c/308012/6/manifests/init.pp15:14
mwhahahaegrep -q?15:15
EmilienMdfisher: https://review.openstack.org/#/q/topic:puppet/tarball15:15
*** michchap_ has quit IRC15:15
EmilienMdfisher: once both patches are merged, I think we're good15:15
mjblackdegorenko: except I messed up15:15
*** cdelatte has quit IRC15:15
degorenkomwhahaha, yes, right, -q option missed15:15
degorenkomjblack, no problem :D15:15
degorenkomwhahaha, so, you're ok with 6 patch set + -q option?15:16
mjblackdegorenko,mwhahaha: so before I submit again, we using netstat or ss?15:16
mwhahahayea15:16
mwhahahanetstat is fine15:16
degorenkoEmilienM, https://review.openstack.org/#/c/308012/6/manifests/init.pp - your thoughts?15:16
EmilienMwell, if it works15:16
*** cdelatte has joined #puppet-openstack15:17
degorenkomjblack, :D you know what you should to do, right?15:17
*** michchap has joined #puppet-openstack15:17
mjblackviva la'netstat15:17
degorenkoo/15:17
*** astupnikov has quit IRC15:18
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/30801215:20
mjblackdegorenko,EmilienM,mwhahaha: ^15:21
mjblackso now the question becomes how long till the tests get execute :D15:22
degorenko:D yeah15:23
mjblackmwhahaha: you able to fix fuel ci issues? :D15:27
mwhahahafor which one15:28
mjblackhttps://ci.fuel-infra.org/job/master.puppet-openstack.fuel.noop/485/console15:28
mjblackits complaining of an error cloning15:28
*** jaosorior has quit IRC15:29
mwhahahayea i can retrigger15:29
mwhahaharestarted15:30
*** foxhunt72 is now known as foxhunt_away15:32
*** alee has joined #puppet-openstack15:35
aleeEmilienM, any progress in https://review.openstack.org/#/c/305886/ ?15:35
EmilienMalee: it's failing, I need to investigate. I was on something else until now15:36
aleeEmilienM, ok15:36
*** mdorman has joined #puppet-openstack15:37
openstackgerritMerged openstack/puppet-tripleo: IPv6 dual-stack support  https://review.openstack.org/28634415:37
mjblackEmilienM: just out of curiosity, why is it that the CI jobs that ubuntu is delayed?15:38
mwhahahait shouldn't be today15:39
mwhahahabut the ubuntu vms are the most used15:39
EmilienMbecause OpenStack Infra is consumming clouds, and have limited resoruces15:39
EmilienMand yeah, ubuntu are very used by devstack jobs, etc15:39
EmilienMso we better deploy on centos715:39
mjblackah ok, I was figuring as much...15:40
openstackgerritMerged openstack/puppet-openstack-integration: repos/ubuntu: install repos before any package  https://review.openstack.org/30733015:40
mwhahahafyi beaker on centos7 is hosed15:40
EmilienMmwhahaha: what?15:41
mwhahahayou can scroll back in #openstack-infra but the puppetlabs release rpm is conflicting15:42
mwhahaha http://logs.openstack.org/77/306677/9/check/gate-puppet-nova-puppet-beaker-rspec-centos-7/3ce2502/console.html#_2016-04-20_14_24_39_34415:42
mjblackyeah it just happened with the puppet-keystone change of mine15:42
mwhahahaso my virtlock/virtlog patch that i've been trying to land for 3  days now is once again hosed by CI :D15:42
mwhahahai'm just glad we don't run beaker in fuel so i can land it over there15:43
*** cdelatte has quit IRC15:43
mjblackrpm should be replacing the gpg file15:44
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Change source for puppet-corosync  https://review.openstack.org/30842415:44
*** iurygregory has quit IRC15:45
mjblackEmilienM,mwhahaha: it looks like we should be passing in --replacefiles too15:45
EmilienMI'm wondering if it's related to latest beaker version15:45
EmilienMcrinkle: should we pin beaker to previous release?15:45
mwhahahano that code hasn't changed in beaker in a year15:45
EmilienMoh ok15:46
mwhahahathey are working on fixing the images i think see #openstack-infra15:46
EmilienMso all our beaker jobs are failing atm15:46
mjblackthe issue could be puppetlabs changed their gpg key15:46
crinkleEmilienM: no its related to the images having puppet preinstalled on them15:46
EmilienMoh oki15:46
mjblackEmilienM: if we pass in --replacefiles we could get around it15:46
crinkleyolanda is working on updating the images15:46
EmilienMmwhahaha: can you get https://review.openstack.org/#/c/307972/ and https://review.openstack.org/#/c/308001/ please?15:47
mwhahahayea15:48
*** cdelatte has joined #puppet-openstack15:50
yolandaEmilienM, i pushed  https://review.openstack.org/30842615:52
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: test ironic on Ubuntu again  https://review.openstack.org/30797715:52
EmilienMyolanda: thx!15:53
*** twm2016 has quit IRC15:54
*** iurygregory has joined #puppet-openstack15:55
*** michchap_ has joined #puppet-openstack15:57
*** chandankumar has quit IRC16:00
*** michchap has quit IRC16:01
*** dtantsur is now known as dtantsur|afk16:08
*** magicboiz has joined #puppet-openstack16:08
*** cdelatte has quit IRC16:10
*** cdelatte has joined #puppet-openstack16:12
*** magicboiz has quit IRC16:14
*** cdelatte has quit IRC16:14
*** magicboiz has joined #puppet-openstack16:15
*** cwolferh has quit IRC16:15
*** magicboiz has quit IRC16:15
*** cdelatte has joined #puppet-openstack16:16
*** haplo37 has joined #puppet-openstack16:18
*** jistr has quit IRC16:22
*** karimb_ has quit IRC16:23
*** karimb has joined #puppet-openstack16:23
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Change scheduler configurations to use entrypoints  https://review.openstack.org/30845216:24
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/30588616:26
*** bnemec has quit IRC16:26
openstackgerritMerged openstack/puppet-openstack-integration: scenario001: test telemetry on Ubuntu again  https://review.openstack.org/30797216:27
*** twm2016 has joined #puppet-openstack16:29
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: test telemetry on Ubuntu again  https://review.openstack.org/30845416:29
*** karimb has quit IRC16:29
*** cdelatte has quit IRC16:31
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: Update spec tests for Tempest  https://review.openstack.org/30824116:32
*** cwolferh has joined #puppet-openstack16:32
openstackgerritMerged openstack/puppet-openstack-integration: centos: deploy cinder in SSL  https://review.openstack.org/30800116:32
*** gchamoul has quit IRC16:35
*** bnemec has joined #puppet-openstack16:35
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: venv: create venv & run pip in exec  https://review.openstack.org/30785316:35
*** leanderthal is now known as leanderthal|afk16:37
openstackgerritMerged openstack/puppet-openstack-integration: functions: add defaults for ZUUL_BRANCH & ZUUL_REF  https://review.openstack.org/30833316:41
openstackgerritIvan Berezovskiy proposed openstack/puppet-glance: Remove docs duplication for notification_driver  https://review.openstack.org/30846716:42
*** gchamoul has joined #puppet-openstack16:42
*** derekh has quit IRC16:44
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: Add support for Gnocchi tests  https://review.openstack.org/30846916:45
*** Guest67082 has quit IRC16:46
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos: pin RDO to Newton & UCA to stable/mitaka  https://review.openstack.org/30588616:48
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: run Gnocchi tempest tests  https://review.openstack.org/30847316:48
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario001: run Gnocchi tempest tests  https://review.openstack.org/30847316:49
*** cdelatte has joined #puppet-openstack16:50
*** dalees` has quit IRC16:54
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: Add missing bit to the user prefetch removal.  https://review.openstack.org/30631316:55
*** chem has joined #puppet-openstack17:00
EmilienMcrinkle, 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:00
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: CI test - never merge  https://review.openstack.org/30847817:01
crinkleEmilienM: we could replace the run_puppet_install_helper in puppet-openstack_spec_helper with something else but i think it would be better to wait17:05
EmilienMcrinkle: how do you estimate the waiting time?17:05
crinkleEmilienM: not sure, yolanda ^17:05
EmilienMif it's breaking our CI > 24h I'll find a workaround17:06
crinklei'm sure it will be less than 24h17:06
mjblackEmilienM: sigh... http://logs.openstack.org/12/308012/9/check/gate-puppet-keystone-puppet-beaker-rspec-ubuntu-trusty/b3303de/console.html#_2016-04-20_17_03_08_93117:08
*** v1k0d3n has quit IRC17:12
mjblackEmilienM,mwhahaha,degorenko: going to do this... netstat -ltnp|egrep "(5000|35357)"|awk '{split($7,pids,"/"); print pids[1]}'|xargs -r kill -917:14
mwhahahanooooes17:14
degorenkomjblack, why we need this perfect structure? :)17:14
mjblackmwhahaha,degorenko: in the beaker tests it still didnt die after 60 seconds17:15
mwhahahanow that's a bug that we could probably get pushed to be fixed17:16
mwhahahait should stop :D17:16
mwhahahamight need to turn on debug logs17:17
* mwhahaha ducks17:17
mjblackmwhahaha: I'm at this point right now... http://imgur.com/Qt1ZCMI17:17
openstackgerritIvan Berezovskiy proposed openstack/puppet-heat: Cleanup rabbit ssl checks  https://review.openstack.org/30848717:17
mjblackdegorenko: I'll even add a comment quote "why wont you die"....17:19
degorenkomjblack, i hope not for me :D17:19
mjblackno, just around the exec call :P17:19
degorenkomjblack, heh :)17:19
mjblackif a kill -9 doesnt fix this...17:20
degorenkomjblack, how then sleep 35 works?17:21
mjblackhey hey17:21
mjblackwe were using logs to see it take 31 seconds17:21
*** jpena is now known as jpena|off17:22
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/30801217:22
mjblackdegorenko: ^ please review :D17:22
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/30801217:23
degorenkomjblack, looking L)17:23
degorenko:)17:23
mjblackdegorenko: look at the one I just pushed17:23
iberezovskiycool exec name17:23
degorenkoyeah17:23
degorenkoah, so beautiful command in exec17:24
degorenko:D17:24
mjblackdegorenko: 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
degorenkomjblack, anyway, it works for sleep 35, why it can't work for netstatt?17:25
*** ron____ has joined #puppet-openstack17:25
*** trown is now known as trown|lunch17:27
mjblackdegorenko: what works for sleep?17:28
mjblackdegorenko: if this this doesnt work, I'm going to just for loop it 60 times...17:29
degorenkomjblack, your commit with sleep 35 option - passed CI, why CI wont pass with netstat and 60 seconds?17:30
mwhahahathats why i said we might need debug logs cause i'm not sure it's looping (or working correctly)17:31
mjblackdegorenko: I wonder if the load on the hypervisor running the instance is high enough to slow everything down17:31
*** mgould is now known as mgould|afk17:32
mjblackmwhahaha: yeah but I dont think debug will be as helpful, I wonder if hte ports are still in use in a fin_wait17:33
mwhahahaif you turn on debug and ditch the -q, we could see the output17:33
mjblackwe're only looking for LISTEN17:33
mjblackso fin_wait wont show up17:33
mjblackwe would have to take out the -l in netstat17:33
mjblackbut lets see if this new change does it17:34
mwhahahano -l includes listen17:34
mwhahahabut it shouldn't exclude fin_wait17:34
mjblackno it does17:35
mjblackyou would need -a17:35
mjblack-l is only listening17:35
degorenkothat's strange, but this one https://review.openstack.org/#/c/308012/9/manifests/init.pp - should work17:35
mjblackI dont disagree17:36
mjblackbut at this point17:36
mjblacklets go with the nuclear option and kill the process17:36
degorenkowe want just wait for free port17:37
degorenkowe don't want nuclear weapon :)17:37
mjblackI think at this point we just want the port freed so CI doesnt fail randomly :D17:38
mjblackbut I can go back to the old one and set it for 5 minute retry....17:39
mjblackif after 5 minutes its still failing we can go back to the kill process route17:39
degorenko:D17:39
degorenkolooks like - you're wanna die? no? i will wait. What's about now?17:40
mjblacktoo bad puppet doesnt have an on_fail option with exec provider17:40
iberezovskiymjblack, can we use unless there?17:41
*** hjensas has quit IRC17:42
mjblackiberezovskiy: in the current change?17:42
mjblackwe can look to add an unless in there, just want to make sure it works17:43
degorenkolol17:43
degorenkoyou know17:43
degorenkowe can use sleep 2 + onlyif netstat -ltn|egrep -q "(5000|35357)17:44
degorenko:D17:44
degorenkowith tries17:44
mjblackdegorenko: hey hey17:44
mjblackif we're going to go this route...I'm going to write a type/provider17:45
mjblackensure_dead17:45
degorenkono, i'm joking17:45
mjblackI'm not :P17:45
degorenkoD:17:45
mjblackensure_dead { 'keystone': with_extreme_prejudice => true }17:46
degorenkoi really don't understand, why this is not working: https://review.openstack.org/#/c/308012/9/manifests/init.pp17:46
degorenkomjblack, can you do this: ensure_dead { 'keystone': head_shot => true } ?17:46
mjblackthat would be like deleting the init/upstart/systemd script17:47
degorenkothat doesn't matter, i want head_shot feature :D17:47
degorenkomjblack, do you have any hardware to check your patch?17:48
mjblackI do17:48
mjblackit works fine here17:48
degorenkomagic17:48
degorenkoi have same result17:48
mjblackI say we pre-bake the beaker/scenario images with keystone installed, update-rc.d with it turned off17:49
degorenkomjblack, you can back your 9 patch set + depends-on test commit to openstack integration with enabling debug and verbose.17:50
degorenkomay be we will see reason for -117:50
openstackgerritMerged openstack/puppet-octavia: Totally drop Qpid support  https://review.openstack.org/30718217:50
mjblackdegorenko: which commit to depend on?17:51
degorenkomjblack, keystone commit + depends-on to openstack integration17:51
degorenkoyour commit :)17:51
mjblackwhich commit specifically on openstack-integration?17:55
degorenkomjblack, no, upload a new commit to openstack-intehration with enabling debug logging for puppet in scenario17:56
colonwqEmilienM 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/gate-puppet-ceph-puppet-beaker-rspec-centos-7/b3e1305/console.html#_2016-04-20_13_10_30_71917:57
degorenkooops, that's not scenario failes17:57
degorenkofails*17:57
*** gfidente has quit IRC17:57
mjblackcolonwq: they know about it, there is a change to fix the issue17:57
degorenkomjblack, scenario jobs are under keystone+apache17:58
degorenkoso want work only beakers. that's double magic17:58
degorenkowont*17:58
mjblackcolonwq: https://review.openstack.org/#/c/308426/17:58
colonwqmjblack, ok. thanks. I have not been around long enough today to follow a conversation.17:58
mjblackcolonwq: np, thats the review pertaining to the issue17:58
mjblackdegorenko: can we wait to see if the nuclear option works? :D17:59
mjblackat the very minimum we have a fall back option :)17:59
colonwqmjblack, Ok. I'll keep an eye on it working through the process.18:00
degorenkomjblack, we can :) sure18:01
mjblackdegorenko: co-worker says the provider should be "nuke_it_from_space_its_the_only_way_to_be_sure { 'keystone': }"18:01
degorenkomjblack, then you need inspect logs with radiation shield :D18:02
mjblackdegorenko: I can always change the exec call to a reboot18:02
mjblackthats one way to be sure :D18:02
mjblackupdate-rc.d keystone disable;reboot18:02
degorenkowe want kill only keystone, not a full server18:03
mjblackhmmm18:03
mjblackdegorenko: I have a better idea18:04
mjblackif $::osfamily == 'Debian' { Package<| title == 'keystone' |> { provider => pip } }18:04
EmilienMlol18:04
mjblackproblem solved18:04
degorenkono please18:05
mjblackdegorenko: how aobut if $::osfamily == 'Debian' { fail('Use redhat foo') }18:06
degorenkomjblack, if $::osfamily == 'Debian' { fail('Hey, relax and deploy Centos') }18:06
mjblackthat'll be patch #2 for the review18:07
degorenko:D18:07
mjblackafter getting -1's18:07
mjblackfor #118:07
mjblackdegorenko: you know this could be soooooo much easier if dpkg allowed us to noscripts packages18:08
degorenkomjblack, you mean no init scripts?18:09
mjblackI mean like rpm has noscripts option18:09
degorenkomjblack, ah, do not execute pre and post scripts18:10
mjblackalthough...18:11
mjblackI'm sensing a potential avenue here... --post-invoke=command18:11
degorenkoi don't see such option in my dpkg18:13
mjblacko_O18:14
mjblackyou on 14.04?18:15
openstackgerritIvan Berezovskiy proposed openstack/puppet-heat: Drop all current deprecated parameters for heat  https://review.openstack.org/30851118:15
degorenkomjblack, yeah18:15
mjblackweird cause its on mine18:15
degorenkomjblack, no, its here, it is not showed on --help - man shows18:16
*** amoralej is now known as amoralej|off18:17
openstackgerritDan Prince proposed openstack/puppet-tripleo: Add neutron profiles  https://review.openstack.org/29343618:17
openstackgerritMerged openstack/puppet-oslo: Totally drop Qpid support  https://review.openstack.org/30709418:17
*** hjensas has joined #puppet-openstack18:18
mjblackoh oh18:18
mjblackI have an idea18:18
mjblackdegorenko: what if...we install keystone via exec18:21
mjblackthen we can pass RUNLEVEL=1 and it wont start up keystone...18:21
*** dgurtner has quit IRC18:24
*** ayoung has quit IRC18:25
*** yolanda has quit IRC18:26
degorenko:D oh my gosh18:26
degorenkoplease no18:26
degorenko:D18:26
mjblackdegorenko: no no its ok I got an idea18:27
mjblackif $::osfamily == 'Debian' { exec { 'install_keystone': command => 'apt-get -y install keystone'; environmnet => 'RUNLEVEL=1' } -> Package['keystone'] }18:28
mjblackit'll be so much easier that way18:28
mjblackEmilienM: what do you think?18:28
* degorenko cries18:28
mjblackI can throw an awk command in there18:28
degorenkoplease stop :)18:28
mjblackyou think I'm joking but that'll actually solve the problem according to docs I read :D18:29
*** yolanda has joined #puppet-openstack18:29
degorenkolet's just find out why 9 patch set fails18:29
degorenkomay be don't like you?18:30
degorenkoCI i mean18:30
degorenko:D18:30
mjblackeverybody loves me18:31
mjblackif they dont they're lying18:31
degorenkohe he18:31
degorenkook, i'm off for today :) it's 21-30 in my time18:32
degorenkowe will continue tomorrow :D18:32
*** degorenko is now known as _degorenko|afk18:32
mjblacksee ya18:34
EmilienM_ody: do you know if puppetlans plans to provide Xenial packaging on http://apt.puppetlabs.com/ ?18:34
mjblackEmilienM: what are your thoughts on the most ugly hack to fix the CI problem?18:35
EmilienMI don't know18:36
EmilienMI haven't followed everything here, I was pretty busy by other things18:36
EmilienMmjblack: why just we don't add a sleep?18:36
mjblackEmilienM: essentially install keystone via an exec where we can pass RUNLEVEL=1 as an env var18:36
mjblackand it wont start up the service18:36
mjblackexec { .. } -> Package['keystone']18:37
mjblackwith an unless => 'dpkg -l|grep keystone'18:37
EmilienMthe problem is we install repos before Package resource18:38
mjblackI know18:38
EmilienMso you'll need to make sure orchestration is correct18:38
mjblackits not perfect, just has to be there for mitaka/liberty18:38
EmilienMgo for it then18:39
mjblackdegorenko isnt going to like it18:39
openstackgerritBradley Klein proposed openstack/puppet-monasca: Support pruning metrics by dimension(s)  https://review.openstack.org/30852218:40
EmilienMwhat does he suggest?18:40
*** trown|lunch is now known as trown18:46
mjblackEmilienM: trying to see why its not dying quickly18:47
mjblackEmilienM: this is what I'm proposing though... http://paste.openstack.org/show/494873/18:47
* mwhahaha runs away18:48
mjblackmwhahaha: I'm still all for doing if $::osfamily == 'Debian' { fail('use centos') }18:49
mwhahahaheh18:49
mjblackmwhahaha: but I'm sure that change will get -2 from like everyone18:51
mwhahahayes, me included :D\18:51
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Support of Ubuntu 16.04 LTS Xenial deployments  https://review.openstack.org/30853018:54
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: Support of Ubuntu 16.04 LTS Xenial deployments  https://review.openstack.org/30853018:55
*** v1k0d3n has joined #puppet-openstack18:56
*** yolanda has quit IRC18:58
*** rcernin has quit IRC19:05
mwhahahaEmilienM: should we switch beaker centos7 to non-voting? :/19:08
mjblackmwhahaha: only if we make ubuntu scenario testing non-voting too :D19:12
mwhahahaso this is the one case where ubuntu is working :D19:12
mjblackyeah...it does rspec...but thats about it :D19:13
mwhahahawell at least it gets to rspec19:13
* mwhahaha throws things19:13
colonwqif it passes lint, it can be good enough. :)19:15
EmilienMmwhahaha: if there is no fix today, we'll workaround in spec_helper19:17
mwhahahak19:17
EmilienMbut I'm not in favor of disabling voting on centos719:18
*** colonwq is now known as colonwq_afk19:18
EmilienMI'll work on the workaround a bit later, so it's ready in case of we need it19:19
mwhahahaat this rate i think we need it :(19:19
*** ducttape_ has quit IRC19:20
*** yolanda has joined #puppet-openstack19:23
openstackgerritAlex Schultz proposed openstack/puppet-openstack_spec_helper: Remote puppetlabs-release prior to running beaker  https://review.openstack.org/30854019:24
mjblackEmilienM: hey, we can merge my kill option...it passed tripleo :P19:25
openstackgerritAlex Schultz proposed openstack/puppet-openstack_spec_helper: Remove puppetlabs-release prior to running beaker  https://review.openstack.org/30854019:25
mjblackand 2 out of 3 fuel ci tests19:25
mwhahahaಠ_ಠ19:25
*** ducttape_ has joined #puppet-openstack19:27
mjblackmwhahaha: ლ(ಠ益ಠლ)19:28
openstackgerritAlex Schultz proposed openstack/puppet-nova: CI test  https://review.openstack.org/30854319:29
*** rcernin has joined #puppet-openstack19:31
mjblackmwhahaha: so good news...we get to see if the kill option works :D19:35
mjblackit finally started doing the ubuntu tets19:35
mjblack*tests19:35
mjblackwell that failed...19:39
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/30801219:40
dfisherthanks for localizing that to Debian.19:41
dfisheron Solaris we only run Keystone as WSGI so …. yeah.19:41
mjblackdfisher: at this point I want to see about just putting out policy-rc.d before the package and then remove it after package install19:43
dfisherthat's … magic Debian/Ubuntu things … right?19:44
mjblackmaybe we fix the issue by converting the ubuntu system to centos19:44
dfisherhah19:44
mjblackI'm sure thats less pain19:44
dfisheryou mispelled 'solaris' again19:44
mjblackwhat this you're talking about? you're acting like this is 1998?19:45
* dfisher sobs19:45
mwhahahaslowaris?19:46
dfisherpile it on.19:46
*** grimlock86 has joined #puppet-openstack19:47
mwhahahatouch the sparc, go on... touch it19:47
mjblackwhy would I want to run linux at a 100000% mark up price?19:47
dfisheryep.  keep going....19:47
dfisherspread the FUD thick.19:47
* dfisher waits19:47
*** ducttape_ has quit IRC19:47
dfisheri'm sadly used to this.19:47
mjblackdfisher: its not FUD when you account for Oracle pricing19:47
dfisher:(19:47
EmilienMmjblack: we'll review it when all CI will have tested ths change19:48
EmilienMok /me working on workaround for beaker jobs now19:48
dfisherit is when I point out things from 3rd party sites that can clearly demonstrate TCO for SPARC is less than Linux but … whatever19:48
mwhahahaEmilienM: i proposed one -^19:48
mjblackEmilienM: previous patch failed, same issue...19:48
dfisheragain, this is the wrong room to discuss this :)19:48
dfishergoozeyx poked the bear (me) last week19:49
EmilienMmwhahaha: you're faster, sorry I was afk for lunch19:49
mjblackdfisher: Oracle wanted to charge $25k per socket for an ldap server19:49
EmilienMmwhahaha: this is not enough19:49
mwhahahayea19:49
mwhahahai just saw the failure19:49
EmilienMmwhahaha: we need to patch the lib/19:49
EmilienMmwhahaha: and not the bash script19:49
dfishermjblack: even I'll ask why you wanted to use SPARC for LDAP.19:50
* dfisher shrugs19:50
mwhahahayea wasn't sure the best place, i was trying to go simple first19:50
EmilienMwe need to replace run_puppet_install_helper19:50
mjblackdfisher: that wasnt sparc19:51
mjblackI think that was their ldap server for oracle linux19:51
EmilienMmwhahaha: we need to rewrite https://github.com/puppetlabs/beaker-puppet_install_helper/blob/master/lib/beaker/puppet_install_helper.rb19:51
dfisheri have nothing to say about Oracle Linux19:51
mwhahahai don't know if we need to go that far19:51
mjblackdfisher: I'm sure they had one for slowaris :D19:52
* dfisher sighs19:52
EmilienMdfisher, mjblack: you guys need a room19:52
dfisherthis one works fine :)19:52
EmilienM /join #openstack-solaris19:52
mjblacknoooo, thats like hell19:52
dfisherIM SO LONELY THERE19:52
EmilienMmwhahaha: we need to modify install_puppet_on19:53
mwhahahawhy can't we just remove puppetlabs-release19:54
mwhahahaand skip all that trouble19:54
EmilienMoh19:54
EmilienMyeah19:54
mwhahahalike in our beaker_spec_helper19:54
mwhahahalet me fix my patch real quick19:54
EmilienMmwhahaha: sure19:54
openstackgerritAlex Schultz proposed openstack/puppet-openstack_spec_helper: Remove puppetlabs-release prior to running beaker  https://review.openstack.org/30854019:55
*** ducttape_ has joined #puppet-openstack19:56
EmilienMmwhahaha: it will fail on ubuntu :P19:57
mwhahahaat the moment i'm just trying to see if it'll work ;)19:57
EmilienMok sure19:57
*** shardy has quit IRC20:03
mwhahahawe don't have an ubuntu beaker job for openstack_spec_helper20:03
* mwhahaha waits patiently20:03
EmilienMmwhahaha: oh right20:03
mwhahahai've got a puppet-nova change to check it on next patchset if it passes20:03
openstackgerritMatthew J Black proposed openstack/puppet-keystone: Add delay before apache start  https://review.openstack.org/30801220:05
*** shardy has joined #puppet-openstack20:12
mwhahahanope failed :/20:13
* mwhahaha waits for logs20:13
*** ntpttr_ is now known as ntpttr20:17
*** mjblack is now known as mjblack|afk20:25
*** grimlock861 has joined #puppet-openstack20:26
*** grimlock86 has quit IRC20:26
openstackgerritEmilien Macchi proposed openstack/puppet-nova: Deprecations and updates for Newton  https://review.openstack.org/30857320:27
mwhahahaI think you're right. I think we have to override that class :/20:28
EmilienMmwhahaha: can't we drop the package before?20:29
mwhahahait's a matter of being able to do that20:29
EmilienMwhy could not we be able to do that?20:30
mwhahahahttps://review.openstack.org/#/c/308540/ failed20:30
mwhahahaand i didn't see my yum remove20:30
mwhahahaeven tho i put it in the beaker_spec_helper20:31
EmilienMis it good syntax to put it like this?20:31
mwhahahayea that's just an exec20:31
mwhahahabut i think the inclusion of require 'beaker/puppet_install_helper' gets ran before my remove20:32
EmilienMmwhahaha: this inclusion does not run it20:32
mwhahahayea that's what i am trying to figure out20:33
EmilienMI'm testing something else20:33
mwhahahai think we have to run the removal on the host beaker is running on20:35
mwhahahacause doesn't beaker fire up a vm of some sort? and editing the spec helper runs it on the parent vm20:35
*** mjblack|afk is now known as mjblack20:37
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: Remove puppetlabs-release before installing puppet  https://review.openstack.org/30858020:38
EmilienMmwhahaha: can it work? ^20:38
mwhahahamaybe20:39
EmilienMmwhahaha: you remember the picture with the dog?20:39
mwhahahayea i'm already there :D20:39
EmilienMit's me, right now20:39
mjblackwhat picture of a dog?20:40
EmilienMmjblack: http://goo.gl/DLkLXR20:40
dfisherhaha20:40
dfisherhttp://images.performgroup.com/di/library/sporting_news/12/66/this-is-fine-dog-embed_c1wcflsukty01jr4jtcy11rrx.jpg   <— better version20:41
mjblackdfisher: oh come on, you're missing the other two panels20:43
mjblackhttp://i0.kym-cdn.com/photos/images/newsfeed/000/962/640/658.png20:43
dfisheri know :)20:43
mjblackEmilienM: I thought this was you... http://s2.quickmeme.com/img/e0/e03c7470565050e7c9dbcaf21368e62b23653a73b101fcd71e7f895620ae1619.jpg20:45
*** agrebennikov has joined #puppet-openstack20:45
dfisherthat's all of us.20:46
EmilienMmjblack: oh man, this guy looks like me20:46
EmilienMmjblack: have we meet each other?20:46
mjblackEmilienM: lol thats why I posted it20:46
mjblackno we havent20:46
EmilienMyou found my twin20:47
EmilienMhe's java dev20:47
EmilienMI wanted to keep it secret, you found him.20:47
mjblackwhy java? that could be that puppet ide20:47
mjblackit looks like eclipse20:47
mwhahahavi4life20:50
mjblackmwhahaha: guy I work with uses nano, drives me crazy20:51
mjblackI keep telling him to use a real editor20:52
mwhahahai knew a guy who did that20:52
mjblackEmilienM: you can use this one though... http://digitalsynopsis.com/wp-content/uploads/2015/03/web-designer-developer-jokes-humour-funny-41.jpg20:52
*** trown is now known as trown|outtypewww20:53
EmilienMI'm more a guy at the right20:54
mjblackEmilienM: you know why they skipped windows 9?20:55
mwhahahaseven ate nine?20:55
EmilienMI didn't even know we were so far20:55
*** shardy has quit IRC20:55
ntpttrEmilienM: 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/1020:55
mjblacksomeone probably had an if condition trying to look for windows 95 or 98 that was "Windows 9"20:56
EmilienMntpttr: yes go ahead please20:56
*** dprince has quit IRC20:57
ntpttrEmilienM: sweet will do20:57
*** ansmith has quit IRC21:00
dfisherpuppet resource requires a full type definition, right?  so since puppetlabs-ntp doesn't provide a type, there's nothing for puppet to assemble?21:00
EmilienMmwhahaha: I don't see the results yet but beaker job is running for 11 min :P21:03
mwhahahaoo progress then21:03
EmilienMit should have failed before, isn't?21:03
EmilienM(if not working)21:03
mwhahahabeaker failed before the unit test last time21:05
mwhahahaso  it might be working21:05
EmilienMSUCCESS21:06
mjblackdfisher: thats right, a define is not a type21:06
EmilienMok, I think we have a workaround21:06
mwhahahak21:06
mjblackdfisher: and puppet resource will only work with types21:07
dfisherthanks.21:07
EmilienMmwhahaha: 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_21721:07
EmilienMcrinkle: I'll let you +A (or not) https://review.openstack.org/#/c/308580 please21:08
openstackgerritBradley Klein proposed openstack/puppet-monasca: Add ability to include libvirt scale group metadata  https://review.openstack.org/30858921:08
crinkleEmilienM: +A21:09
EmilienMmwhahaha: the bad news is it will fix stable/mitaka and master21:10
EmilienMmwhahaha: I did not change stable/liberty spec/spec_helper_acceptance.rb21:10
EmilienMbut I think we can live without liberty during 1 or 2 days21:10
EmilienMcrinkle: thanks21:10
EmilienMwe'll revert that patch once it's fixed by infra21:11
openstackgerritJames Slagle proposed openstack/puppet-tripleo: Add dport/sport parameter to firewall rule  https://review.openstack.org/30835821:11
mwhahahayea i think it's ok21:11
*** colonwq_afk is now known as colonwq21:12
openstackgerritJames Slagle proposed openstack/puppet-tripleo: Add dport/sport parameter to firewall rule  https://review.openstack.org/30835821:12
EmilienMI'm out, but back later in late evening.21:15
*** ron____ has quit IRC21:20
openstackgerritMerged openstack/puppet-openstack_spec_helper: Remove puppetlabs-release before installing puppet  https://review.openstack.org/30858021:26
*** alee has quit IRC21:37
*** jmikolajczak has quit IRC21:41
*** mdorman has quit IRC21:42
*** cdelatte has quit IRC21:44
*** v1k0d3n has quit IRC21:45
openstackgerritMerged openstack/puppet-monasca: Support pruning metrics by dimension(s)  https://review.openstack.org/30852221:54
mjblackwell21:58
mjblackso far so good21:58
openstackgerritMerged openstack/puppet-tripleo: Add Glance profiles  https://review.openstack.org/29607622:16
*** grimlock861 has quit IRC22:23
*** grimlock86 has joined #puppet-openstack22:23
*** dhollinger has quit IRC22:26
openstackgerritNate Potter proposed openstack/puppet-magnum: Add magnum-conductor  https://review.openstack.org/28584722:32
*** ayoung has joined #puppet-openstack22:33
*** johnmilton has quit IRC22:33
*** haplo37 has quit IRC22:38
*** ducttape_ has quit IRC22:40
*** ducttape_ has joined #puppet-openstack22:41
*** twm2016 has quit IRC22:43
*** dfisher has quit IRC22:58
*** johnmilton has joined #puppet-openstack23:08
*** johnmilton has quit IRC23:13
*** grimlock86 has quit IRC23:16
*** ducttape_ has quit IRC23:22
*** johnmilton has joined #puppet-openstack23:25
dmsimardmjblack: I'm late but re: nano https://xkcd.com/378/23:31
mjblacklol23:32
mjblackvi is the best tho23:32
dmsimardvim*23:32
dmsimardvi sux.23:32
mjblackwell23:32
mjblackthats why my zshrc has alias vi=23:32
mjblack':D23:32
mjblackbah23:32
mjblackalias vi='vim'23:32
mjblackI save a whole character now23:33
dmsimardI think most of the time when you install vim, vi redirects to vim automagically23:39
dmsimardI've never really wondered why23:39
*** jprs is now known as samuelson23:39
mjblackdmsimard: not always23:45
mjblackonly on some linux distros23:45
*** xarses has quit IRC23:54
« Tuesday, 2016-04-19 Index Thursday, 2016-04-21 »

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!