Thursday, 2017-02-09

*** morazi has quit IRC00:01
*** ducttape_ has joined #puppet-openstack00:01
*** dfisher has quit IRC00:14
*** jaosorior has joined #puppet-openstack00:20
*** xarses has joined #puppet-openstack00:29
*** xarses has quit IRC00:29
*** xarses has joined #puppet-openstack00:29
*** jaosorior has quit IRC00:31
*** gbarros has joined #puppet-openstack00:34
*** OGtrilliams has quit IRC00:38
*** OGtrilliams has joined #puppet-openstack00:38
*** gbarros has quit IRC00:39
*** jpena has quit IRC01:03
*** jpena has joined #puppet-openstack01:05
*** ducttape_ has quit IRC01:22
*** jperry has joined #puppet-openstack01:31
*** OGtrilliams has quit IRC01:59
*** dtantsur|afk has quit IRC02:15
*** dtantsur has joined #puppet-openstack02:16
*** owalsh has joined #puppet-openstack02:30
*** owalsh-afk has quit IRC02:31
*** ducttape_ has joined #puppet-openstack02:38
*** ilbot3 has quit IRC02:45
*** ilbot3 has joined #puppet-openstack02:48
*** ccamacho has quit IRC02:58
*** openstackstatus has quit IRC02:58
*** ccamacho has joined #puppet-openstack02:59
*** openstackstatus has joined #puppet-openstack03:02
*** ChanServ sets mode: +v openstackstatus03:02
*** saneax-_-|AFK is now known as saneax03:04
*** gbarros has joined #puppet-openstack03:22
*** gbarros has quit IRC03:22
*** saneax is now known as saneax-_-|AFK03:32
*** jperry has quit IRC03:40
*** ducttape_ has quit IRC03:44
*** ducttape_ has joined #puppet-openstack03:44
*** ducttape_ has quit IRC03:56
*** xarses has quit IRC04:09
*** udesale has joined #puppet-openstack04:18
*** nmathew has joined #puppet-openstack04:29
*** saneax-_-|AFK is now known as saneax04:51
*** ducttape_ has joined #puppet-openstack04:55
openstackgerritMerged openstack/puppet-openstack-guide master: Update release page for Ocata  https://review.openstack.org/43106905:02
*** acormier has joined #puppet-openstack05:06
*** acormier has quit IRC05:10
*** saneax is now known as saneax-_-|AFK05:16
*** acormier has joined #puppet-openstack05:16
*** acormier has quit IRC05:17
*** acormier has joined #puppet-openstack05:18
*** acormier has quit IRC05:25
*** acormier has joined #puppet-openstack05:35
*** owalsh_ has joined #puppet-openstack05:38
*** owalsh has quit IRC05:39
openstackgerritMerged openstack/puppet-watcher stable/ocata: Update .gitreview for stable/ocata  https://review.openstack.org/43082806:00
openstackgerritMerged openstack/puppet-watcher master: Fix typo in api.pp  https://review.openstack.org/42310206:00
*** saneax-_-|AFK is now known as saneax06:02
*** ducttape_ has quit IRC06:02
*** rajinir_ has quit IRC06:10
*** imcsk8_ has joined #puppet-openstack06:12
*** imcsk8 has quit IRC06:12
*** acormier has quit IRC06:15
*** acormier has joined #puppet-openstack06:15
*** NachoDuck_ has joined #puppet-openstack06:29
*** patrickeast_ has joined #puppet-openstack06:29
*** foxhunt7z has quit IRC06:33
*** patrickeast has quit IRC06:33
*** NachoDuck has quit IRC06:33
*** foxhunt_away has joined #puppet-openstack06:33
*** patrickeast_ is now known as patrickeast06:33
*** NachoDuck_ is now known as NachoDuck06:34
*** jaganathan has joined #puppet-openstack06:40
*** richm has quit IRC06:42
*** fxpester has joined #puppet-openstack06:42
*** acormier has quit IRC06:45
*** acormier has joined #puppet-openstack06:49
*** dgurtner has joined #puppet-openstack06:55
*** dgurtner has quit IRC06:55
*** dgurtner has joined #puppet-openstack06:55
*** jpich has joined #puppet-openstack06:55
*** rcernin has joined #puppet-openstack07:00
*** ducttape_ has joined #puppet-openstack07:03
*** imcsk8_ has quit IRC07:04
*** imcsk8 has joined #puppet-openstack07:04
*** acormier has quit IRC07:06
*** tesseract has joined #puppet-openstack07:11
*** dgurtner has quit IRC07:14
*** agurenko has joined #puppet-openstack07:17
*** udesale__ has joined #puppet-openstack07:17
*** udesale has quit IRC07:20
*** udesale__ has quit IRC07:32
*** udesale has joined #puppet-openstack07:33
*** udesale has quit IRC07:35
*** udesale has joined #puppet-openstack07:35
openstackgerritEyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/42970507:43
*** udesale has quit IRC07:52
*** udesale has joined #puppet-openstack07:53
*** hjensas has joined #puppet-openstack07:56
*** hjensas has joined #puppet-openstack07:56
*** agurenko has quit IRC08:05
*** bogdando has joined #puppet-openstack08:08
*** rasca has joined #puppet-openstack08:12
*** yprokule has joined #puppet-openstack08:14
*** udesale__ has joined #puppet-openstack08:14
*** ducttape_ has quit IRC08:14
*** bogdando has quit IRC08:16
*** udesale has quit IRC08:17
*** agurenko has joined #puppet-openstack08:21
*** paramite has joined #puppet-openstack08:21
*** paramite has quit IRC08:22
*** paramite has joined #puppet-openstack08:24
*** Mosibi has joined #puppet-openstack08:26
*** chem has joined #puppet-openstack08:45
openstackgerritMerged openstack/puppet-openstack-integration master: Enable using tempest rpm in a configurable mode  https://review.openstack.org/42757808:49
openstackgerritMerged openstack/puppet-ironic master: Add separate manifest for configuring access to neutron  https://review.openstack.org/42879508:50
*** rcernin has quit IRC08:50
*** owalsh_ is now known as owalsh-afk08:55
*** yprokule_ has joined #puppet-openstack08:57
*** yprokule has quit IRC08:58
*** yprokule_ is now known as yprokule08:58
*** openstackgerrit has quit IRC09:03
*** chem has quit IRC09:05
*** chem has joined #puppet-openstack09:06
*** rcernin has joined #puppet-openstack09:06
*** yprokule_ has joined #puppet-openstack09:07
*** yprokule has quit IRC09:09
*** yprokule_ is now known as yprokule09:09
*** openstackgerrit has joined #puppet-openstack09:16
openstackgerritMerged openstack/puppet-tacker master: Add control_exchange and rpc_response_timeout options  https://review.openstack.org/43013209:16
*** amoralej|off is now known as amoralej09:17
*** gfidente has joined #puppet-openstack09:25
*** gfidente has quit IRC09:25
*** gfidente has joined #puppet-openstack09:25
*** lucas-afk is now known as lucasagomes09:26
openstackgerritEyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/42970509:28
openstackgerritEyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/42970509:29
*** bogdando has joined #puppet-openstack09:32
*** bogdando has quit IRC09:32
openstackgerritEyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/42970509:36
openstackgerritMerged openstack/puppet-trove master: Add rpc_response_timeout option  https://review.openstack.org/43051609:40
openstackgerritEyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/42970509:41
*** salmankhan has joined #puppet-openstack10:04
*** ducttape_ has joined #puppet-openstack10:11
*** snecklifter has joined #puppet-openstack10:12
*** udesale__ has quit IRC10:15
*** ducttape_ has quit IRC10:17
snecklifterMorning puppeteers10:18
snecklifterAnyone able to tell me why I end up with a blank string when using the following in control_params.yaml10:18
snecklifterhttp://paste.openstack.org/show/598197/10:18
snecklifterI am using tripleo to deploy10:19
snecklifterPretty sure I just have syntax problem ...10:19
openstackgerritMerged openstack/puppet-nova master: Add control_exchange and option  https://review.openstack.org/42995710:24
*** enanrag has joined #puppet-openstack10:29
enanraghi10:30
*** cgfbee has quit IRC10:31
enanragOne question..10:31
enanragWe have implemented a composable service and when we deploy via Heat the package is not getting installed in overcloud..10:33
openstackgerritEyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/42970510:33
dtantsursnecklifter, my only guess is that you may not need '' around the string, as it may be interpreted literally when using : | syntax10:33
dtantsurenanrag, IIRC by default installation of packages is forbidden; you have to have the package on the overcloud-full image already.10:35
*** amorin has joined #puppet-openstack10:35
snecklifterdtantsur: ah, ok, possibly one of the few combinations I have not tried, thanks for response!10:36
*** salmankhan has quit IRC10:39
*** salmankhan has joined #puppet-openstack10:42
*** salmankhan has quit IRC10:44
*** salmankhan has joined #puppet-openstack10:45
enanrag<dtantsur> Thanks for the response.. Is there a way we can overcome this and install package?10:47
dtantsuryeah, lemme find10:48
dtantsurenanrag, do a similar thing in your environment files: https://github.com/openstack/tripleo-heat-templates/blob/e4d28996273ff6c8a1e53351b875686527898358/environments/deployed-server-bootstrap-environment-centos.yaml#L710:51
dtantsurenanrag, still, a better way is to install the package on your images. then it will work even without internet access10:51
*** skramaja has quit IRC10:54
enanragdtantsur, thanks a lot.. I will try this now..10:54
*** nmathew has quit IRC10:59
*** agurenko has quit IRC11:03
*** richm has joined #puppet-openstack11:12
*** amorin has left #puppet-openstack11:13
*** amorin has quit IRC11:13
*** enanrag has quit IRC11:15
openstackgerritEyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/42970511:30
*** danpawlik has joined #puppet-openstack11:36
danpawliknot authenticate. So I was checking what was bad and I found that in newton release https://github.com/openstack/puppet-nova/blob/stable/newton/lib/puppet/provider/nova_flavor/openstack.rb#L1111:41
danpawlikHello11:41
danpawlikIm using keystone v3 and I have a problem that when puppet call second time e.g. nova flavor provider it shows me an error that he can not authenticate. So I was checking what was bad and I found that in newton release https://github.com/openstack/puppet-nova/blob/stable/newton/lib/puppet/provider/nova_flavor/openstack.rb#1111:42
danpawlikit use CedentialsV2.011:43
danpawlikHow can I switch the version without changing puppet module?11:43
openstackgerritEyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/42970511:45
openstackgerritArnaud Morin proposed openstack/puppet-nova stable/newton: Enable Keystone v3 support for nova_flavor and nova_aggregate  https://review.openstack.org/43146311:51
EmilienMhey oh11:54
danpawlikEmilienM: hello11:56
*** derekh has joined #puppet-openstack11:58
danpawlikI don't know if it will be a problem11:59
danpawlikto apply this patch https://review.openstack.org/#/c/431463/ for Newton11:59
degorenkodanpawlik, if you are using v3 in configs/openrc then you need to use OS_CREDENTIALS_API variable12:00
*** jpena is now known as jpena|lunch12:00
degorenkoas possible way without changing puppets/ configs12:00
danpawlikand should we change other providers to version 3 or not?12:00
danpawlikdegorenko: checking12:00
degorenkopossibly changing v3 for newton is not a good idea, can it be backported? from my point of view - this is a feature12:00
*** owalsh-afk is now known as owalsh12:01
*** skramaja has joined #puppet-openstack12:01
snecklifterdtantsur: thanks for your help but I got12:02
snecklifterSyntax error: {"vendor_id"=>"15b3", "product_id"=>"1007", "name"=>"ib0"} is not a String12:02
snecklifterI need to understand yaml syntax better12:03
dtantsurthen I guess I can't help much, sorry :(12:04
snecklifterdtantsur: no worries, thanks for taking a look anyway, if I solve it I will post back12:04
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration master: DNM: test centos7/consistent  https://review.openstack.org/42491512:08
*** ducttape_ has joined #puppet-openstack12:13
*** dprince has joined #puppet-openstack12:13
danpawlikdegorenko: I guess OS_CREDENTIALS_API  is deprecated or?12:16
degorenkodanpawlik, OS_IDENTITY_API_VERSION the correct one12:17
*** ducttape_ has quit IRC12:18
*** amoralej is now known as amoralej|lunch12:18
*** jaganathan has quit IRC12:18
danpawlikdegorenko: I have it in openrc file. TBH it doesn't help puppet to pass :)12:19
degorenkodanpawlik, so you have this in openrc right? And you have v3 rigth?12:22
danpawlikdegorenko: yes12:23
degorenkodanpawlik, 1) change it to v2; 2) run puppet manifests with env variable above set to v2.012:24
danpawlikdegorenko: so I aslo should remove OS_USER_DOMAIN_NAME and OS_PROJECT_DOMAIN_NAME12:26
degorenkodanpawlik, i'm not sure, because v2 credentials class just will not use those parameters, you can try to leave them as is12:26
*** pester has joined #puppet-openstack12:34
*** fxpester has quit IRC12:37
danpawlikdegorenko: nope, it doesn't work12:47
degorenkodanpawlik, what's exactly?12:50
openstackgerritEyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/42970512:51
danpawlikabout using v2 and not removing domain/project name:12:55
danpawlikIgnoring domain related config project_domain_name because identity API version is 2.012:55
danpawlikIgnoring domain related config user_domain_name because identity API version is 2.012:55
danpawlikdegorenko: checking one more time with removing domain/project12:55
degorenkoit's ignoring, so not used12:56
*** jpena|lunch is now known as jpena13:01
*** ducttape_ has joined #puppet-openstack13:02
danpawlikdegorenko: an pupper error: Error: Could not prefetch keystone_service provider 'openstack': Execution of '/usr/bin/openstack service list --quiet --format csv --long' returned 1: Could not find requested endpoint in Service Catalog. (tried 35, for a total of 170 seconds)13:02
danpawlika puppet error*13:02
*** mangerduchien has joined #puppet-openstack13:02
*** ansmith has joined #puppet-openstack13:03
danpawlikdegorenko: and when you normally call by openstack client you will get empty list.13:03
danpawlikdegorenko: what I would like to add is that I want to use v3 not v2.013:03
degorenkowell, actually backport to newton with v3 https://review.openstack.org/431463 - passed ci13:03
danpawlikyeah13:04
*** lucasagomes is now known as lucas-hungry13:04
degorenkomwhahaha, EmilienM can we accept 431463 ? :)13:04
mangerduchienhey all13:05
danpawlikbut now the main question is: should we enable v3 in newton or it should be enabled since ocata release13:05
danpawlik?13:05
*** mangerduchien is now known as amorin13:05
*** jayg|g0n3 is now known as jayg13:06
*** morazi has joined #puppet-openstack13:08
*** trown|outtypewww is now known as trown13:09
mwhahahadegorenko: yea because the v3 creds provider is backwards compatible with v213:20
degorenkomwhahaha, should we wait for fuel?13:21
mwhahahadoes fuel use those?13:21
mwhahahayou can wait13:21
degorenkoonly nova flavor i guess13:21
degorenkolet me check13:21
degorenkohttps://github.com/openstack/fuel-library/blob/stable/newton/deployment/puppet/openstack_tasks/manifests/roles/controller.pp#L19-L2413:22
*** gbarros has joined #puppet-openstack13:22
mwhahahayea feel free to wait until fuel comes back, it should pass because the v3 creds are backwards compatible13:23
degorenkoyeah, i hope :D13:23
danpawlikmwhahaha: so I13:23
danpawlik:d13:23
openstackgerritLuigi Toscano proposed openstack/puppet-openstack-integration master: tempest-lib should not be required anymore  https://review.openstack.org/43151813:24
*** fultonj_ has joined #puppet-openstack13:27
*** gbarros has quit IRC13:27
*** pradk has joined #puppet-openstack13:27
*** amoralej|lunch is now known as amoralej13:28
*** ducttape_ has quit IRC13:36
*** amorin has quit IRC13:42
*** OGtrilliams has joined #puppet-openstack13:45
openstackgerritDan Radez proposed openstack/puppet-congress master: Adding acceptance test to Red Hat family  https://review.openstack.org/43049213:47
*** dcritch has joined #puppet-openstack13:48
*** amorin has joined #puppet-openstack13:48
*** lucas-hungry is now known as lucasagomes13:57
*** gbarros has joined #puppet-openstack13:57
*** ducttape_ has joined #puppet-openstack14:04
ansiwen_mwhahaha: could you help me with this error? http://logs.openstack.org/23/427623/1/check/gate-puppet-nova-puppet-syntax-4-centos-7/5f8a419/console.html.gz#_2017-02-01_10_18_08_17511714:04
*** ansiwen_ is now known as ansiwen14:04
*** gbarros has quit IRC14:04
*** xb_ has joined #puppet-openstack14:08
openstackgerritDan Radez proposed openstack/puppet-congress master: Adding acceptance test to Red Hat family  https://review.openstack.org/43049214:09
degorenkodanpawlik, +A :) v3 creds for newton14:10
*** saneax is now known as saneax-_-|AFK14:12
mwhahahaansiwen: I'll look more in a minute but it looks like a parse error14:13
*** salmankhan has quit IRC14:14
*** owalsh is now known as owalsh-brb14:22
chandankumardegorenko: please have a look on this https://review.openstack.org/#/c/430732/ when you are free14:22
*** jperry has joined #puppet-openstack14:26
*** owalsh-brb has quit IRC14:26
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration master: DNM: test centos7/consistent  https://review.openstack.org/42491514:26
*** salmankhan has joined #puppet-openstack14:27
EmilienMmwhahaha: our puppet ci didn't promote to nova/trunk yet14:29
EmilienMwe still deploy ocata-314:29
EmilienMthough tripleo did promote14:29
*** OGtrilliams is now known as PagliaccisCloud14:29
*** acormier has joined #puppet-openstack14:30
*** acormier has quit IRC14:30
*** acormier has joined #puppet-openstack14:30
*** owalsh-brb has joined #puppet-openstack14:31
*** owalsh-brb is now known as owalsh14:31
*** dgurtner has joined #puppet-openstack14:33
*** dgurtner has quit IRC14:33
*** dgurtner has joined #puppet-openstack14:33
*** acormier has quit IRC14:34
*** acormier has joined #puppet-openstack14:34
*** acormier has quit IRC14:35
*** paramite has quit IRC14:36
mwhahahaansiwen: looked transient since the recheck is passing14:39
*** acabot has quit IRC14:42
ansiwenmwhahaha: right, just saw it14:50
*** dgurtner has quit IRC14:52
snecklifterdtantsur: finally got the syntax working with [] - ie. http://paste.openstack.org/show/598252/14:53
snecklifterFYI and for anyone else who stumbles across this archived on eavesdrop :)14:53
dtantsurcool!14:53
mwhahahasnecklifter: does that have to be a string? or could you do a yaml array of hashes?14:56
mwhahahasnecklifter: http://paste.openstack.org/show/598254/14:57
sneckliftermwhahaha: good question, I'm pretty sure it has to be a string https://github.com/openstack/puppet-nova/blob/master/manifests/api.pp#L7714:59
mwhahahasnecklifter: https://github.com/openstack/puppet-nova/blob/master/manifests/api.pp#L48114:59
*** udesale has joined #puppet-openstack14:59
mwhahahathough i don't know what check_array_of_hash is15:00
sneckliftermwhahaha: ack but the example appears to show a string being passed15:00
*** karthiks has quit IRC15:00
sneckliftermwhahaha: also, https://github.com/openstack/puppet-nova/blob/master/spec/classes/nova_api_spec.rb#L16215:00
openstackgerritEyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/42970515:01
snecklifterwhich is what led me to success in the end15:01
mwhahahayea it looks like  the string works just wondering if there's a cleaner way to do it :D15:01
*** paramite has joined #puppet-openstack15:01
mwhahahastringifying stuff always ends badly :D15:01
mwhahahasnecklifter: looks like either, https://github.com/openstack/puppet-nova/blob/8938e4cff9fb7aa02f6b01037f3afef22792b8c8/lib/puppet/parser/functions/check_array_of_hash.rb15:02
sneckliftermwhahaha: I'm sure there is, the SR-IOV code is only just starting to get some attention with OSP 1015:02
mwhahahasnecklifter: so it's up to you on what you want to use, my preference would be proper yaml :)15:02
sneckliftermwhahaha: your way looks much cleaner i agree15:02
snecklifterI will try one more run with your config and report back!15:03
*** gbarros has joined #puppet-openstack15:04
*** paramite has quit IRC15:05
*** dfisher has joined #puppet-openstack15:05
*** paramite has joined #puppet-openstack15:05
*** gbarros has quit IRC15:12
*** agrebennikov_ has joined #puppet-openstack15:17
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration master: nova: start compute after keystone endpoints/services  https://review.openstack.org/43159615:20
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration master: DNM: test centos7/consistent  https://review.openstack.org/42491515:21
EmilienMmwhahaha: ^ might be the fix they suggest15:21
* EmilienM cries15:21
EmilienMmwhahaha: doing it in CI now and I'll think at a longterm solution in puppet-nova15:21
mwhahahaHEH15:21
*** saneax-_-|AFK is now known as saneax15:27
*** pester has quit IRC15:28
*** finchd has quit IRC15:28
sneckliftermwhahaha: hash array bombed out, it really wants a string15:29
snecklifterhttps://paste.fedoraproject.org/551798/65412014/15:29
mwhahahainteresting15:29
snecklifteri think that the puppet_alias paramter wants a string to pass on15:30
*** udesale has quit IRC15:31
snecklifter*pci_alias15:31
*** udesale has joined #puppet-openstack15:33
*** udesale has quit IRC15:33
*** udesale has joined #puppet-openstack15:33
*** udesale has quit IRC15:33
*** udesale has joined #puppet-openstack15:34
*** karthiks has joined #puppet-openstack15:36
*** karts has joined #puppet-openstack15:38
*** karthiks has quit IRC15:38
*** finchd has joined #puppet-openstack15:40
*** karts has quit IRC15:42
*** karts has joined #puppet-openstack15:42
*** udesale has quit IRC15:43
*** udesale has joined #puppet-openstack15:46
*** udesale has quit IRC15:48
*** acabot has joined #puppet-openstack15:48
*** salmankhan1 has joined #puppet-openstack15:50
*** salmankhan has quit IRC15:53
*** salmankhan1 is now known as salmankhan15:53
*** derekh has quit IRC15:53
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration master: DNM: test centos7/consistent  https://review.openstack.org/42491516:00
*** xarses has joined #puppet-openstack16:00
mwhahahasnecklifter: i think that's a bug because we really should support the complex types but i'll take a look later. we might not properly convert to a string for config storage though16:00
*** xarses has quit IRC16:00
*** xarses has joined #puppet-openstack16:01
*** paramite has quit IRC16:01
sneckliftermwhahaha: ok, that would be appreciated, please ping me if you raise a bug as I can apply some additional resource to get it looked at, thanks16:01
*** andreww has joined #puppet-openstack16:02
snecklifterbut would be good to know its definitely a bug before doing so16:02
mwhahahasnecklifter: oh i'm reading it wrong, yea that function checks that it is a string. I wonder if there would be an issue to convert it16:02
mwhahahasnecklifter: stringified json in configs makes me sad16:03
sneckliftermwhahaha: it makes me lose hours of my life trying to understand yaml and json...16:03
mwhahahastringify json in yaml to puppet to write out to an ini to be read by python16:04
mwhahahabrilliant!16:04
sneckliftermwhahaha: maybe its missing being piped through Go?16:04
*** xarses has quit IRC16:05
mwhahahaopenstack deployment, http://wonderopolis.org/wp-content/uploads//2014/05/Rubenvent1.jpg16:05
EmilienMmwhahaha: that's the step 1 of the deployment16:06
*** salmankhan has quit IRC16:08
*** salmankhan has joined #puppet-openstack16:09
*** gbarros has joined #puppet-openstack16:10
*** saneax is now known as saneax-_-|AFK16:11
*** jpena is now known as jpena|away16:15
*** gbarros has quit IRC16:19
*** yprokule has quit IRC16:24
*** jaosorior has joined #puppet-openstack16:31
*** rcernin has quit IRC16:32
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration master: DNM: test centos7/consistent  https://review.openstack.org/42491516:34
*** morazi is now known as morazi-mtg16:37
*** skramaja has quit IRC17:02
openstackgerritMerged openstack/puppet-nova stable/newton: Enable Keystone v3 support for nova_flavor and nova_aggregate  https://review.openstack.org/43146317:02
*** jaosorior is now known as jaosorior_mtg17:03
*** dgurtner has joined #puppet-openstack17:05
*** dgurtner has quit IRC17:05
*** dgurtner has joined #puppet-openstack17:05
*** ducttape_ has quit IRC17:12
*** tesseract has quit IRC17:18
*** dgurtner has quit IRC17:20
*** trown is now known as trown|lunch17:26
*** dtantsur is now known as dtantsur|afk17:27
*** jpich has quit IRC17:30
EmilienMmwhahaha: it seems it helped https://review.openstack.org/#/c/424915/17:43
EmilienMonly mistral is broken now17:43
EmilienMdmsimard, iurygregory, mwhahaha: I need your power on https://review.openstack.org/#/c/431596/117:43
mwhahahawell that's good i guess17:43
*** dprince has quit IRC17:44
iurygregoryEmilienM, boom17:44
iurygregory;)17:44
*** dprince has joined #puppet-openstack17:45
*** andreww has quit IRC17:55
*** lucasagomes is now known as lucas-afk18:02
*** ducttape_ has joined #puppet-openstack18:04
openstackgerritMerged openstack/puppet-openstack-integration master: tempest-lib should not be required anymore  https://review.openstack.org/43151818:04
openstackgerritMerged openstack/puppet-nova stable/newton: Add a class to run the db online_data_migrations  https://review.openstack.org/42762318:04
*** jpena|away is now known as jpena18:11
EmilienMso the mistral issue on trunk18:13
EmilienM[Thu Feb 09 17:09:46.429312 2017] [:error] [pid 22500] [remote ::1:180] mod_wsgi (pid=22500): Exception occurred processing WSGI script '/var/www/cgi-bin/mistral/app'.18:13
EmilienMin http://logs.openstack.org/15/424915/17/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/f828cbc/logs/apache/mistral_wsgi_error_ssl.txt.gz18:14
jaosorior_mtgEmilienM: seems like a mistral error "cannot register CLI option"18:15
EmilienMyeah but why we have it today and not yesterday :D18:16
EmilienMhttps://github.com/openstack/mistral/commits/master18:16
EmilienMI don't find anything useful18:16
*** jaosorior_mtg has quit IRC18:35
*** jaosorior has joined #puppet-openstack18:35
*** xarses has joined #puppet-openstack18:46
*** rasca has quit IRC18:49
*** ducttape_ has quit IRC18:52
*** trown|lunch is now known as trown19:03
*** owalsh has quit IRC19:04
*** salmankhan has quit IRC19:18
*** jaosorior has quit IRC19:20
*** jpena is now known as jpena|away19:27
*** ducttape_ has joined #puppet-openstack19:34
mwhahahadid oslo cfg get updated?19:36
EmilienMmwhahaha: that was my first thought19:37
EmilienMif you look on #rdo19:37
EmilienMlet me check19:37
* mwhahaha just got from lunch and goes a scrollin'19:38
EmilienMpython2-oslo-config-3.22.0-0.20170120192858.3025b6f.el7.centos.noarch when it worked19:39
EmilienMpython2-oslo-config-3.22.0-0.20170120192858.3025b6f.el7.centos.noarch when it fails19:39
EmilienMso no19:39
mwhahahaloles19:39
mwhahahagot the diff handy?19:39
EmilienMlet me check19:40
EmilienMhttps://www.diffchecker.com/DrhMDFoO19:40
EmilienMso much things19:40
mwhahahai need another monitor19:41
mwhahahatoo many windows19:41
EmilienMlol19:41
EmilienMmwhahaha: http://infovis.cs.vt.edu/gigapixel/pictures/curved24mon/quake_front.jpg19:42
mwhahahawhich is bad cause i have one of these, http://www.dell.com/en-us/shop/dell-ultrasharp-34-curved-ultrawide-monitor-u3415w/apd/210-adtr/monitors-monitor-accessories19:42
bkeroOh man that's like 4 polygons per monitor19:42
mwhahahawhen i worked at rackspace there were guys with like 1219:43
mwhahahathat's a bit much19:43
mwhahahai've always had 2, i was hoping 1 34" would suffice but it doesn't :(19:43
bkeroI need to just start doing work on my oculus rift19:43
EmilienMare they more productive?19:43
EmilienMthey fix more nova bugs in the same time? #troll19:43
mwhahaha:o19:43
mwhahahait's this change19:45
mwhahahaEmilienM: https://review.openstack.org/#/c/425581/3/mistral/actions/openstack/action_generator/base.py@3819:45
mwhahahathat's where it's blowing up19:45
EmilienMthat !19:47
EmilienMlet's revert it :D19:47
mwhahahamy work here is done. http://i.imgur.com/xoi91ZR.png19:47
EmilienMi'm worrying about why tripleo CI passed19:47
EmilienMwe're running Mistral on the undercloud19:47
mwhahahacause we don't deploy it in wsgi?19:47
mwhahahahttps://github.com/openstack/instack-undercloud/blob/master/elements/puppet-stack-config/puppet-stack-config.pp#L52419:49
mwhahahabecause we don't deploy it in wsgi.19:49
EmilienMHE19:49
EmilienMI hate when you're right19:49
mwhahahame too19:50
mwhahahaso i guess the next question is, is 'wsgi' supported by mistral? or are we getting nove'd19:50
EmilienMI'm going to ping mistral folks19:50
mwhahahas/nove/nova19:50
EmilienM /join #openstack-mistral19:51
*** amoralej is now known as amoralej|off19:51
EmilienMmwhahaha: nobody replies on mistral side20:09
EmilienMI'm proposing a revert and disabling it in p-o-i20:09
mwhahahasounds reasonable20:09
*** gfidente is now known as gfidente|afk20:10
openstackgerritMerged openstack/puppet-openstack-integration master: nova: start compute after keystone endpoints/services  https://review.openstack.org/43159620:11
*** jaosorior has joined #puppet-openstack20:11
*** owalsh has joined #puppet-openstack20:13
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration master: scenario003: disable mistral  https://review.openstack.org/43173320:13
mwhahahaEmilienM: we don't have to completely disable it, we can just switch to non wsgi20:14
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration master: DNM: test centos7/consistent  https://review.openstack.org/42491520:14
EmilienMmwhahaha: ssl / ipv6 is a pain to make it work outside apache20:15
EmilienMI don' thave time today to do it20:15
mwhahahaoh nm20:15
mwhahahaturning it off it is20:15
mwhahaha:D20:15
EmilienMyes20:15
*** morazi-mtg has quit IRC20:18
-openstackstatus- NOTICE: Restarting gerrit due to performance problems20:21
EmilienMplease review https://review.openstack.org/#/c/431733/20:23
*** salmankhan has joined #puppet-openstack20:32
openstackgerritDan Radez proposed openstack/puppet-congress master: Updating service name for Red Hat platform  https://review.openstack.org/43173920:33
openstackgerritDan Radez proposed openstack/puppet-congress master: Adding acceptance test to Red Hat family  https://review.openstack.org/43049220:34
*** salmankhan has quit IRC20:36
*** rajinir_ has joined #puppet-openstack20:42
fultonjI see an openstack-puppet module [1] rejecting a string generated by THT [2] becuase it's not a hash.20:47
fultonj[1] https://github.com/openstack/puppet-ceph/blob/master/manifests/profile/params.pp#L23920:47
fultonj[2] https://github.com/openstack/tripleo-heat-templates/blob/6b8c34b27a324120f6945f7873f479d37ba2e516/puppet/services/ceph-external.yaml#L84-L10120:47
fultonjThis same code worked in previous versions. What would be the correct fix for this?20:48
fultonja. Modify THT to make it generate a hash? OR b. have puppet try to convert the string to a hash and if it works continue?20:48
mwhahahafultonj: question would be where is that used in puppet-ceph20:49
*** ducttape_ has quit IRC20:50
fultonjmwhahaha: /etc/puppet/modules/ceph/manifests/keys.pp:3120:50
mwhahahafultonj: that validate_hash is 2 years old, so i assume this is failing because we recently added that class be included20:50
fultonjmwhahaha: i removing that validate_hash to experiment, but it was rejected further down the line by keys.pp for the same reason20:51
mwhahahafultonj: where is ceph::profile::params::client_keys used20:51
mwhahahahttps://github.com/openstack/puppet-ceph/blob/d4cc06688c084a3c8570fdd42e75445d68129c12/manifests/profile/mon.pp#L4120:52
mwhahahafultonj: it has to be a hash20:52
mwhahahafultonj: create_resources needs a hash not a string, so fix THT20:52
fultonjmwhahaha: agreed, it should be a hash20:52
fultonjok20:52
fultonjmwhahaha: i'm used to thinking of tht as generating strings, i need to figure out how to make it generate a hash20:53
fultonjmwhahaha: thanks for recommending a direction20:53
mwhahahafultonj: it's just a special string into yaml :D20:53
fultonjso it should be a simple wrapper around that join20:53
fultonji will look for prior art and copy the style20:53
xarses++ on the hash20:53
fultonjthanks mwhahaha and xarses20:54
xarsesaren't you reading the yaml in?20:55
xarsesto puppet?20:55
*** ducttape_ has joined #puppet-openstack20:55
mwhahahaxarses: he needs to figure out how to generate the yaml in the heat template since it's dynamic20:56
fultonjxarses: the yaml ends up in hiera inbetween20:56
fultonjhttp://paste.openstack.org/show/598302/20:57
fultonjthat looks like a hash to me, i.e. it's not quoted20:58
fultonjnot on the outside20:58
fultonjam I not reading that right? i.e. even though hiera returns what looks like a hash, is it really a string when puppet gets it?20:59
fultonji will go to tht to see about generating a hash, but i was just curious about that part21:00
mwhahahafultonj: that's not a yaml hash21:04
mwhahahafultonj: sec21:05
mwhahahafultonj: what do you get for hiera -h ceph::profile::params::client_keys21:07
fultonjhttp://paste.openstack.org/show/598305/21:07
mwhahahafultonj: yea it's a string in the hiera files21:08
fultonji think i see your point now21:08
fultonjthanks, i'll take note of that -h option21:08
mwhahahafultonj: sec it should be really easy21:08
mwhahahaactually doing what's there is harder than it needs to be21:08
mwhahahafultonj: https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/services/ceph-external.yaml#L91-L10821:10
mwhahahafultonj: whatever version you were looking at needs to have that format backported21:11
fultonjmwhahaha: got it map_replace vs string_replace21:12
openstackgerritDan Prince proposed openstack/puppet-nova master: Nova placement doesn't require nova::api  https://review.openstack.org/43174421:14
fultonjdah...21:14
fultonjgfidente already fixed it :)21:14
fultonjhttps://github.com/openstack/tripleo-heat-templates/commit/8bf04682013472dcc696dde7dc7b5e48d33bb7bc21:14
fultonjthat's what i get for not updating. thanks sorry i didn't see that earlier21:14
mwhahahayea that's why i was like it needs to be backported :D21:14
*** gfidente|afk has quit IRC21:15
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration master: scenario003: disable mistral  https://review.openstack.org/43173321:19
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration master: DNM: test centos7/consistent  https://review.openstack.org/42491521:19
*** paramite has joined #puppet-openstack21:20
radezmwhahaha: EmilienM: if you guys have a sec would you mind taking a look at: https://review.openstack.org/#/c/431739/21:23
EmilienMno way21:23
EmilienMsomething is not idempotent: http://logs.openstack.org/92/430492/4/check/gate-puppet-congress-puppet-beaker-rspec-centos-7/8736738/console.html#_2017-02-09_20_46_52_24591921:24
mwhahahahow did that pass21:24
EmilienMhttp://logs.openstack.org/92/430492/4/check/gate-puppet-congress-puppet-beaker-rspec-centos-7/8736738/logs/syslog.txt.gz#_Feb_09_20_46_1621:25
EmilienMyeah21:25
EmilienMmwhahaha: because he put the tests on top of the patch21:25
EmilienMradez: ^21:25
radezEmilienM: hm, k thx, that looks like it's at the package level21:26
mwhahahatho it'd probably be ok to at least merge the name fix21:29
mwhahahacause that's not going to change :D21:30
EmilienMok21:30
mwhahahaand it'd be nice to get it in the rc121:30
EmilienMI'm +2 it21:30
mwhahahawhich hasn't been cut yet21:30
openstackgerritMerged openstack/puppet-congress master: Updating service name for Red Hat platform  https://review.openstack.org/43173921:35
*** ducttape_ has quit IRC21:40
*** trown is now known as trown|outtypewww21:40
radezthx for merging the name fix, I'll get the package fixed so the acceptance passes21:54
*** fultonj has quit IRC22:06
*** ducttape_ has joined #puppet-openstack22:10
*** paramite has quit IRC22:13
EmilienMradez: cool22:15
EmilienMradez: and tripleo too please22:15
openstackgerritMerged openstack/puppet-tempest master: Add Octavia Tempest plugin support.  https://review.openstack.org/43073222:16
*** jaosorior has quit IRC22:37
*** jaosorior has joined #puppet-openstack22:38
*** PagliaccisCloud has quit IRC22:42
*** PagliaccisCloud has joined #puppet-openstack22:43
openstackgerritMerged openstack/puppet-openstack-integration master: scenario003: disable mistral  https://review.openstack.org/43173322:49
*** xb_ has quit IRC22:57
*** jperry has quit IRC23:03
*** jaosorior has quit IRC23:10
*** jaosorior has joined #puppet-openstack23:11
*** hecliunyx has joined #puppet-openstack23:12
*** dprince has quit IRC23:18
openstackgerritMerged openstack/puppet-ceph master: Automatically zap a disk if no valid gpt label is present  https://review.openstack.org/42099223:28
*** jaosorior has quit IRC23:29
*** dprince has joined #puppet-openstack23:50
*** dprince has quit IRC23:59

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