Tuesday, 2017-11-21

« Monday, 2017-11-20 Index Wednesday, 2017-11-22 »
*** rwsu has joined #puppet-openstack02:06
*** ilbot3 has quit IRC02:44
*** ilbot3 has joined #puppet-openstack02:56
*** ykarel|away has joined #puppet-openstack04:09
*** ykarel|away is now known as ykarel04:13
*** skramaja has joined #puppet-openstack04:55
*** ratailor has joined #puppet-openstack04:59
*** markvoelker has quit IRC05:00
*** jaganathan has joined #puppet-openstack05:14
*** udesale has joined #puppet-openstack05:32
*** markvoelker has joined #puppet-openstack06:00
*** vinsh has quit IRC06:22
*** vinsh has joined #puppet-openstack06:25
*** jtomasek has joined #puppet-openstack06:50
*** rcernin has quit IRC06:59
*** dtantsur|afk is now known as dtantsur07:01
*** spectr has joined #puppet-openstack07:11
*** spectr has quit IRC07:12
*** oidgar has joined #puppet-openstack07:12
*** udesale has quit IRC07:32
*** agurenko has joined #puppet-openstack07:41
*** ykarel is now known as ykarel|lunch07:44
*** shardy has joined #puppet-openstack07:49
*** karthiks has quit IRC08:06
*** yprokule has joined #puppet-openstack08:07
*** ccamacho has joined #puppet-openstack08:14
*** rmart04 has joined #puppet-openstack08:16
*** udesale has joined #puppet-openstack08:19
*** tesseract has joined #puppet-openstack08:23
*** karthiks has joined #puppet-openstack08:23
*** ykarel|lunch is now known as ykarel08:31
*** karthiks has quit IRC08:33
*** karthiks has joined #puppet-openstack08:33
*** jpena|off is now known as jpena08:43
*** amoralej|off is now known as amoralej09:01
*** tdasilva has quit IRC09:09
*** melodyzhou has joined #puppet-openstack09:12
*** melodyzhou has quit IRC09:13
*** oidgar has quit IRC09:13
*** tdasilva has joined #puppet-openstack09:15
*** etingof has quit IRC09:16
*** dtantsur_ has joined #puppet-openstack09:21
*** dtantsur has quit IRC09:22
*** dtantsur_ is now known as dtantsur09:22
*** etingof has joined #puppet-openstack09:30
*** etingof has quit IRC09:35
*** derekh has joined #puppet-openstack09:41
*** etingof has joined #puppet-openstack09:48
*** gfidente has joined #puppet-openstack09:56
*** gfidente has quit IRC09:56
*** gfidente has joined #puppet-openstack09:56
*** chem has joined #puppet-openstack09:56
*** rcernin has joined #puppet-openstack10:01
*** salmankhan has joined #puppet-openstack10:18
*** ratailor has quit IRC10:30
*** ratailor has joined #puppet-openstack10:47
*** ratailor_ has joined #puppet-openstack10:53
*** ratailor has quit IRC10:56
*** udesale has quit IRC11:30
*** ansmith has quit IRC11:39
*** oidgar has joined #puppet-openstack11:48
*** salmankhan has quit IRC12:21
*** asilenkov has quit IRC12:24
*** asilenkov has joined #puppet-openstack12:26
*** salmankhan has joined #puppet-openstack12:27
*** morazi has joined #puppet-openstack12:43
*** jpena is now known as jpena|lunch12:58
*** amoralej is now known as amoralej|lunch13:00
*** ansmith has joined #puppet-openstack13:12
*** dprince has joined #puppet-openstack13:15
*** shardy is now known as shardy_lunch13:22
openstackgerritMateusz Kowalski proposed openstack/puppet-ironic master: Allow setting inspector timeout value  https://review.openstack.org/52165713:25
openstackgerritMateusz Kowalski proposed openstack/puppet-ironic master: Allow setting inspector timeout value  https://review.openstack.org/52165713:29
*** rcernin has quit IRC13:30
*** ratailor_ is now known as ratailor13:31
*** alee has quit IRC13:37
*** amoralej|lunch is now known as amoralej13:44
*** trown|outtypewww is now known as trown13:44
*** udesale has joined #puppet-openstack13:45
*** shardy_lunch is now known as shardy13:45
*** rmart04 has quit IRC13:48
*** jpena|lunch is now known as jpena13:50
*** foxhunt_away is now known as foxhunt7213:51
*** skramaja has quit IRC14:03
*** jaganathan has quit IRC14:12
*** ratailor has quit IRC14:21
*** clayton has quit IRC14:27
*** clayton has joined #puppet-openstack14:30
*** markvoelker has quit IRC14:31
*** markvoelker has joined #puppet-openstack14:31
openstackgerritPradeep Kilambi proposed openstack/puppet-ceilometer stable/ocata: Add support to set host param  https://review.openstack.org/52186214:39
*** pradk has joined #puppet-openstack14:41
openstackgerritPradeep Kilambi proposed openstack/puppet-ceilometer stable/newton: Add support to set host param  https://review.openstack.org/52186814:45
*** foxhunt72 is now known as foxhunt_away14:48
*** oidgar has quit IRC15:03
*** udesale has quit IRC15:08
*** agurenko has quit IRC15:08
*** agurenko has joined #puppet-openstack15:09
*** vinsh has quit IRC15:11
*** vinsh has joined #puppet-openstack15:12
*** vinsh_ has joined #puppet-openstack15:16
*** agurenko has quit IRC15:17
*** agurenko has joined #puppet-openstack15:18
*** ykarel is now known as ykarel|away15:19
*** vinsh has quit IRC15:19
*** ykarel|away has quit IRC15:24
*** jistr is now known as jistr|mtg15:59
openstackgerritMerged openstack/puppet-cinder stable/pike: Dell EMC VMAX: Fixes driver dependency using ensure  https://review.openstack.org/52015016:00
*** vinsh_ has quit IRC16:01
*** oidgar has joined #puppet-openstack16:12
*** jistr|mtg is now known as jistr16:16
*** chem has quit IRC16:25
*** vinsh has joined #puppet-openstack16:35
EmilienMmnaser, iurygregory : can you please review https://review.openstack.org/#/c/521593/ ? thanks16:38
EmilienMI'll need to backport it as well16:38
iurygregoryEmilienM, doing now16:40
EmilienMthx!16:41
iurygregory+A ;)16:41
*** ccamacho has quit IRC16:42
*** salmankhan has quit IRC16:42
*** salmankhan has joined #puppet-openstack16:44
*** chem has joined #puppet-openstack16:45
EmilienMiurygregory: thx!!16:48
*** salmankhan has quit IRC16:51
*** salmankhan has joined #puppet-openstack16:53
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration stable/pike: install_modules_unit: list modules  https://review.openstack.org/52193816:53
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration stable/ocata: install_modules_unit: list modules  https://review.openstack.org/52193916:54
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration stable/newton: install_modules_unit: list modules  https://review.openstack.org/52194016:54
*** agurenko has quit IRC16:54
iurygregoryyw =D16:54
*** dtantsur is now known as dtantsur|afk16:55
*** openstackstatus has quit IRC16:58
*** openstackstatus has joined #puppet-openstack17:00
*** ChanServ sets mode: +v openstackstatus17:00
openstackgerritAlex Schultz proposed openstack/puppet-nova stable/ocata: Pass arguments as array for nova-manage provider  https://review.openstack.org/52194517:01
colonwqmwhahaha, (or anyone else) My last unit test failure is something basic that I have not figured out how to pass. Any suggestions would be greatly appreciated: http://logs.openstack.org/31/519531/6/check/legacy-puppet-tripleo-puppet-unit-4.8-centos-7/56aa6dd/job-output.txt.gz#_2017-11-16_16_36_57_93970617:02
*** ratailor has joined #puppet-openstack17:04
mnasercolonwq: looking17:05
mnaseroh thats a puppet-tripleo thing but ill look :p17:05
mnasercolonwq: Ceph::Key["client.${cinder_rbd_client_name}"]  could not necessarily be defined what you want to do is use a resource collector so id replace it with17:06
colonwqOne of THOSE patches. :)17:06
mnaserCeph::Key< title == "client.${cinder_rbd_client_name}" >17:06
mnaserCeph::Key< title == "client.${cinder_rbd_client_name}" > -> Exec["exec-setfacl-${cinder_rbd_client_name}-cinder"]17:06
colonwqmnaser, I did not know that. I'll update and add the title and test.17:07
mnasercolonwq: or alternatively you can add the ceph::key in the unit tests17:08
*** ntpttr_laptop has joined #puppet-openstack17:08
*** tesseract has quit IRC17:09
*** ratailor has quit IRC17:10
*** ansmith has quit IRC17:24
mwhahahawould be beter to add the proper keys to the test rather than relying on the spaceships17:27
EmilienMI think we have a big problem17:27
EmilienMbut puppet-tripleo/newton unit tests deploys modules from master17:27
EmilienMhttp://logs.openstack.org/81/521281/2/check/puppet-openstack-unit-4.8-centos-7/c6eadd6/job-output.txt.gz#_2017-11-21_17_18_09_13257317:27
iurygregory:O17:28
* EmilienM eating french croissant17:28
EmilienMoh wait17:31
EmilienMno17:31
EmilienMnevermind, it's all good17:31
* EmilienM should really go back to bed17:31
iurygregoryor prob you need more coffee EmilienM17:34
mwhahahaEmilienM: we need to pin haproxy17:36
mwhahahaEmilienM: http://logs.openstack.org/81/521281/2/check/puppet-openstack-unit-4.8-centos-7/c6eadd6/job-output.txt.gz#_2017-11-21_17_19_37_977693 absolutepath wasn't added until after 4.12.0 in puppet stdlib i think17:36
mwhahahaEmilienM: http://logs.openstack.org/81/521281/2/check/puppet-openstack-unit-4.8-centos-7/c6eadd6/job-output.txt.gz#_2017-11-21_17_18_11_13994317:37
EmilienMmwhahaha: ahhh17:37
EmilienMmwhahaha: and we didn't catch it because we don't use haproxy in p-o-i17:37
mwhahahatest is pulling in haproxy 1.5.0 which needs 4.1317:37
mwhahahaso that's the fix17:37
mwhahahaeither up stdlib or match the packaging of haproxy17:37
EmilienMok, let me check that now17:38
mwhahahayea we package 4.13.117:38
mwhahahaup stdlib17:38
EmilienMin newton: puppet-stdlib-4.13.1-0.20161117202107.dcef77a.el7.centos.noarch17:38
mwhahahahttps://github.com/redhat-openstack/rdoinfo/blob/master/rdo.yml#L84217:39
EmilienMok17:39
EmilienMI'll do that17:39
mwhahahahttps://github.com/redhat-openstack/rdoinfo/blob/master/rdo.yml#L64917:39
mwhahahawe use puppet-haproxy 1.5.017:39
EmilienMare you a bot? you go too fast.17:39
mwhahahai'm in a good mood today17:39
mwhahahaci isn't royally screwed up17:39
mwhahaha:D17:39
EmilienMgood because I'm not17:39
EmilienMok so '4.21.0' in master, but :ref => '4.15.0' in ocata17:40
mwhahahaso we're testing with newer17:41
mwhahahacause 4.14.0 is in ocata and 4.18.0 is pike17:41
EmilienMmaster :-O17:41
EmilienMI think we should pin puppetlabs-haproxy, no?17:42
mwhahahaprobably if we're not going to unpin it17:42
mwhahaha1.5.0 is the latest17:42
EmilienMit's newton, not going to change I think17:42
mwhahahain newton yea17:42
mwhahahabut since it's 1.5.0-2 there might be some hash we have to pin to17:42
EmilienMalso, master is a bad pin for newton17:42
EmilienMimho, we need to pin newton/ocata/pike17:43
EmilienMlike we do in p-o-i17:43
EmilienMotherwise it can break at anytime17:43
mwhahahayes17:43
EmilienMok, I'll do that17:43
EmilienMeasy task, I can do it17:43
openstackgerritMerged openstack/puppet-openstack-integration master: install_modules_unit: list modules  https://review.openstack.org/52159317:43
*** derekh has quit IRC17:58
*** agurenko has joined #puppet-openstack17:58
*** trown is now known as trown|lunch18:03
*** jistr is now known as jistr|off|trng18:04
*** yprokule has quit IRC18:13
*** amoralej is now known as amoralej|off18:16
openstackgerritMerged openstack/puppet-glance stable/pike: Switch to Zuul v3 testing  https://review.openstack.org/51993418:18
*** jpena is now known as jpena|off18:20
*** ccamacho has joined #puppet-openstack18:24
colonwqmnaser, I got them all fixed. thanks for the help.18:25
*** shardy has quit IRC18:26
iurygregoryEmilienM, if you need the backports merged fast just let me know =)18:26
EmilienMthat's fine, mwhahaha will get them18:27
*** ccamacho has quit IRC18:33
*** ccamacho has joined #puppet-openstack18:36
*** ntpttr_laptop has quit IRC18:45
*** ansmith has joined #puppet-openstack18:49
*** ntpttr_laptop has joined #puppet-openstack18:51
*** oidgar has quit IRC19:02
*** trown|lunch is now known as trown19:03
*** gfidente is now known as gfidente|afk19:20
openstackgerritMerged openstack/puppet-nova stable/ocata: Pass arguments as array for nova-manage provider  https://review.openstack.org/52194519:26
*** ccamacho has quit IRC19:28
KvisleCan someone look at why https://review.openstack.org/#/c/511730/ is stuck with a downvote from CI?19:36
openstackgerritMerged openstack/puppet-openstack-integration stable/newton: install_modules_unit: list modules  https://review.openstack.org/52194019:36
Kvisleoh, it has a merge conflict now.19:37
openstackgerritMerged openstack/puppet-openstack-integration stable/pike: install_modules_unit: list modules  https://review.openstack.org/52193819:46
*** jtomasek has quit IRC19:51
openstackgerritMerged openstack/puppet-openstack-integration stable/pike: Add unit jobs in zuul v3  https://review.openstack.org/51651820:04
*** morazi has quit IRC20:10
*** salmankhan has quit IRC20:24
openstackgerritMateusz Kowalski proposed openstack/puppet-ironic master: Allow setting inspector timeout value  https://review.openstack.org/52165720:57
*** gfidente|afk has quit IRC21:07
mgagneis there a way to not rely on admin_token to provision keystone resources? I see that keystone types still rely on DEFAULT/admin_token21:08
*** etingof has quit IRC21:20
*** etingof has joined #puppet-openstack21:22
mwhahahathought we made that optional21:23
mwhahahamgagne: what version? i don't see any references to auth_token21:25
mgagnemwhahaha: https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/provider/keystone.rb#L187-L18921:26
mgagnebut maybe I'm misreading the code21:27
mgagneand in fact, I didn't see the call to super21:27
mwhahahaoh i was looking for auth_token21:27
mgagneand I suspect the parent class is the one supporting password auth21:27
mwhahahait falls back to admin token21:27
mwhahahaif normal auth fails21:28
mwhahahahttps://github.com/openstack/puppet-keystone/blob/master/lib/puppet/provider/keystone.rb#L23321:28
mwhahahaso if you have normal auth it should use that21:28
mgagneawesome, will try, thanks!21:28
mwhahahahttps://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack.rb#L7721:29
mwhahahaand our credentials stuff https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack/credentials.rb21:29
mgagneI'm just missing the link between those 2, what injects credentials?21:30
mwhahahadidn't you write this stuff? :D21:30
mgagneno lol21:30
mgagneI'll keep digging and will figure out something for sure21:30
mwhahahait's all black magic21:31
mgagnenot a bad thing to get familiar with this part of the code21:31
mgagnehehe21:31
mwhahahahttps://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack/auth.rb21:31
mgagneprovider extends auth which provides request already: https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack/auth.rb#L3521:31
mwhahahabecause https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/provider/keystone.rb#L921:31
mgagneyea21:31
mgagnegood stuff21:31
*** etingof has quit IRC21:31
mgagnenow to remove that middleware from our internal boxes :D21:31
mgagnethanks for the help!21:32
*** etingof has joined #puppet-openstack21:32
mwhahahayea that was 2 cycles ago i think when they killed of that21:32
mgagneone last thing21:32
mgagne /root/openrc21:32
mgagneI don't really like that it's hardcoded here21:32
mwhahahait also accepts ENV vars from the run if that file isn't there21:33
mwhahahaso you can just pass them in to the puppet execution21:33
mgagneyea, but not gonna happen :P21:33
* mwhahaha shrugs 21:33
mwhahahaunfortunately we have to know where to go looking21:33
mgagnewould there be a way to make it configurable and/or consider more Puppetish path?21:33
mwhahahaso you could extend it but there has to be somewhere21:33
mgagnelike is there a "standard" path where Puppet stores secrets or credentials?21:33
mwhahahathere isn't21:34
mwhahahaunless you do like eyaml or some trickery with hiera21:34
mgagne=(21:34
mwhahahawhich is way too complex for this21:34
mgagneyea21:34
mwhahahait's not really hard coded as you could in theory run it as a different user21:34
mgagneok, will continue to check then. thanks!21:34
mgagnewell... wouldn't be an easy task ;)21:35
mwhahaharight so that's why it's /root/openrc :D21:35
mgagne /etc/puppetlabs/puppet/openrc? =)21:35
mwhahahano because perms21:35
mwhahahait's your admin creds for your cloud21:36
mgagneisn't puppet already running as root?21:36
mwhahaharight so why not keep the important cred file locked up in /root21:36
mwhahahawhich other users probably don't have access to21:36
mgagnebecause I feel root is for user stuff, not system stuff21:36
*** dprince has quit IRC21:37
mwhahahaif you want to suggest an optional path then sure21:37
mwhahahabut it would still need to look in /root/openrc for backwards compatibility21:37
mwhahahapatches welcome :D21:37
mwhahahabut it's not a foreign concept to keep the cloud rc file in /root21:38
mgagneoh yea, I'm not the kind of guy to break backward compat =)21:38
mwhahahait's been this way since kilo, https://github.com/openstack/puppet-openstacklib/blob/kilo-eol/lib/puppet/provider/openstack/auth.rb#L521:38
mwhahahaso21:38
mgagneI agree but we do have human using root and messing with /root/openrc without knowing that a service/system reads it21:38
mwhahahait's got a long history21:38
mwhahahawell perhaps your humans should keep their dirty hands off that file :D21:39
mgagnethey are humans, what can you do :D21:39
mwhahahaelectro shock therapy21:39
mwhahahaor puppetize it https://github.com/openstack/puppet-openstack_extras/blob/master/manifests/auth_file.pp21:40
mgagnesuggestion duly noted :D21:40
*** salmankhan has joined #puppet-openstack21:49
*** rcernin has joined #puppet-openstack21:50
*** salmankhan has quit IRC21:54
*** trown is now known as trown|outtypewww21:58
*** hjensas has quit IRC22:10
*** etingof has quit IRC22:13
*** markvoelker has quit IRC22:16
*** markvoelker has joined #puppet-openstack22:27
*** etingof has joined #puppet-openstack22:35
*** salmankhan has joined #puppet-openstack22:57
*** salmankhan has quit IRC23:06
*** bnemec has quit IRC23:09
*** hjensas has joined #puppet-openstack23:11
*** bnemec has joined #puppet-openstack23:12
*** hjensas has quit IRC23:36
*** ntpttr_laptop has quit IRC23:56
« Monday, 2017-11-20 Index Wednesday, 2017-11-22 »

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