Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2017-02-09

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

All times shown according to UTC.

Time Nick Message
00:01 ducttape_ joined #puppet-openstack
00:20 jaosorior joined #puppet-openstack
00:29 xarses joined #puppet-openstack
00:29 xarses joined #puppet-openstack
00:34 gbarros joined #puppet-openstack
00:38 OGtrilliams joined #puppet-openstack
01:05 jpena joined #puppet-openstack
01:31 jperry joined #puppet-openstack
02:16 dtantsur joined #puppet-openstack
02:30 owalsh joined #puppet-openstack
02:38 ducttape_ joined #puppet-openstack
02:48 ilbot3 joined #puppet-openstack
02:48 Topic for #puppet-openstack is now CI status: green - Puppet OpenStack modules http://docs.openstack.org/developer/puppet-openstack-guide/
02:59 ccamacho joined #puppet-openstack
03:02 openstackstatus joined #puppet-openstack
03:22 gbarros joined #puppet-openstack
03:44 ducttape_ joined #puppet-openstack
04:18 udesale joined #puppet-openstack
04:29 nmathew joined #puppet-openstack
04:55 ducttape_ joined #puppet-openstack
05:02 openstackgerrit Merged openstack/puppet-openstack-guide master: Update release page for Ocata  https://review.openstack.org/431069
05:06 acormier joined #puppet-openstack
05:16 acormier joined #puppet-openstack
05:18 acormier joined #puppet-openstack
05:35 acormier joined #puppet-openstack
05:38 owalsh_ joined #puppet-openstack
06:00 openstackgerrit Merged openstack/puppet-watcher stable/ocata: Update .gitreview for stable/ocata  https://review.openstack.org/430828
06:00 openstackgerrit Merged openstack/puppet-watcher master: Fix typo in api.pp  https://review.openstack.org/423102
06:12 imcsk8_ joined #puppet-openstack
06:15 acormier joined #puppet-openstack
06:29 NachoDuck_ joined #puppet-openstack
06:29 patrickeast_ joined #puppet-openstack
06:33 foxhunt_away joined #puppet-openstack
06:40 jaganathan joined #puppet-openstack
06:42 fxpester joined #puppet-openstack
06:49 acormier joined #puppet-openstack
06:55 dgurtner joined #puppet-openstack
06:55 dgurtner joined #puppet-openstack
06:55 jpich joined #puppet-openstack
07:00 rcernin joined #puppet-openstack
07:03 ducttape_ joined #puppet-openstack
07:04 imcsk8 joined #puppet-openstack
07:11 tesseract joined #puppet-openstack
07:17 agurenko joined #puppet-openstack
07:17 udesale__ joined #puppet-openstack
07:32 udesale joined #puppet-openstack
07:35 udesale joined #puppet-openstack
07:43 openstackgerrit Eyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/429705
07:53 udesale joined #puppet-openstack
07:56 hjensas joined #puppet-openstack
07:56 hjensas joined #puppet-openstack
08:08 bogdando joined #puppet-openstack
08:12 rasca joined #puppet-openstack
08:14 yprokule joined #puppet-openstack
08:14 udesale__ joined #puppet-openstack
08:21 agurenko joined #puppet-openstack
08:21 paramite joined #puppet-openstack
08:24 paramite joined #puppet-openstack
08:26 Mosibi joined #puppet-openstack
08:44 chem joined #puppet-openstack
08:49 openstackgerrit Merged openstack/puppet-openstack-integration master: Enable using tempest rpm in a configurable mode  https://review.openstack.org/427578
08:50 openstackgerrit Merged openstack/puppet-ironic master: Add separate manifest for configuring access to neutron  https://review.openstack.org/428795
08:57 yprokule_ joined #puppet-openstack
09:06 chem joined #puppet-openstack
09:06 rcernin joined #puppet-openstack
09:07 yprokule_ joined #puppet-openstack
09:16 openstackgerrit joined #puppet-openstack
09:16 openstackgerrit Merged openstack/puppet-tacker master: Add control_exchange and rpc_response_timeout options  https://review.openstack.org/430132
09:25 gfidente joined #puppet-openstack
09:25 gfidente joined #puppet-openstack
09:28 openstackgerrit Eyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/429705
09:29 openstackgerrit Eyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/429705
09:32 bogdando joined #puppet-openstack
09:36 openstackgerrit Eyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/429705
09:40 openstackgerrit Merged openstack/puppet-trove master: Add rpc_response_timeout option  https://review.openstack.org/430516
09:41 openstackgerrit Eyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/429705
10:04 salmankhan joined #puppet-openstack
10:11 ducttape_ joined #puppet-openstack
10:12 snecklifter joined #puppet-openstack
10:18 snecklifter Morning puppeteers
10:18 snecklifter Anyone able to tell me why I end up with a blank string when using the following in control_params.yaml
10:18 snecklifter http://paste.openstack.org/show/598197/
10:19 snecklifter I am using tripleo to deploy
10:19 snecklifter Pretty sure I just have syntax problem ...
10:24 openstackgerrit Merged openstack/puppet-nova master: Add control_exchange and option  https://review.openstack.org/429957
10:29 enanrag joined #puppet-openstack
10:30 enanrag hi
10:31 enanrag One question..
10:33 enanrag We have implemented a composable service and when we deploy via Heat the package is not getting installed in overcloud..
10:33 openstackgerrit Eyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/429705
10:33 dtantsur snecklifter, my only guess is that you may not need '' around the string, as it may be interpreted literally when using : | syntax
10:35 dtantsur enanrag, IIRC by default installation of packages is forbidden; you have to have the package on the overcloud-full image already.
10:35 amorin joined #puppet-openstack
10:36 snecklifter dtantsur: ah, ok, possibly one of the few combinations I have not tried, thanks for response!
10:42 salmankhan joined #puppet-openstack
10:45 salmankhan joined #puppet-openstack
10:47 enanrag <dtantsur> Thanks for the response.. Is there a way we can overcome this and install package?
10:48 dtantsur yeah, lemme find
10:51 dtantsur enanrag, do a similar thing in your environment files: https://github.com/openstack/tripleo-heat-templates/blob/e4d28996273ff6c8a1e53351b875686527898358/environments/deployed-server-bootstrap-environment-centos.yaml#L7
10:51 dtantsur enanrag, still, a better way is to install the package on your images. then it will work even without internet access
10:54 enanrag dtantsur, thanks a lot.. I will try this now..
11:12 richm joined #puppet-openstack
11:13 amorin left #puppet-openstack
11:30 openstackgerrit Eyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/429705
11:36 danpawlik joined #puppet-openstack
11:41 danpawlik 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#L11
11:41 danpawlik Hello
11:42 danpawlik Im 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#11
11:43 danpawlik it use CedentialsV2.0
11:43 danpawlik How can I switch the version without changing puppet module?
11:45 openstackgerrit Eyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/429705
11:51 openstackgerrit Arnaud Morin proposed openstack/puppet-nova stable/newton: Enable Keystone v3 support for nova_flavor and nova_aggregate  https://review.openstack.org/431463
11:54 EmilienM hey oh
11:56 danpawlik EmilienM: hello
11:58 derekh joined #puppet-openstack
11:59 danpawlik I don't know if it will be a problem
11:59 danpawlik to apply this patch https://review.openstack.org/#/c/431463/ for Newton
12:00 degorenko danpawlik, if you are using v3 in configs/openrc then you need to use OS_CREDENTIALS_API variable
12:00 degorenko as possible way without changing puppets/ configs
12:00 danpawlik and should we change other providers to version 3 or not?
12:00 danpawlik degorenko: checking
12:00 degorenko possibly changing v3 for newton is not a good idea, can it be backported? from my point of view - this is a feature
12:01 skramaja joined #puppet-openstack
12:02 snecklifter dtantsur: thanks for your help but I got
12:02 snecklifter Syntax error: {"vendor_id"=>"15b3", "product_id"=>"1007", "name"=>"ib0"} is not a String
12:03 snecklifter I need to understand yaml syntax better
12:04 dtantsur then I guess I can't help much, sorry :(
12:04 snecklifter dtantsur: no worries, thanks for taking a look anyway, if I solve it I will post back
12:08 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration master: DNM: test centos7/consistent  https://review.openstack.org/424915
12:13 ducttape_ joined #puppet-openstack
12:13 dprince joined #puppet-openstack
12:16 danpawlik degorenko: I guess OS_CREDENTIALS_API  is deprecated or?
12:17 degorenko danpawlik, OS_IDENTITY_API_VERSION the correct one
12:19 danpawlik degorenko: I have it in openrc file. TBH it doesn't help puppet to pass :)
12:22 degorenko danpawlik, so you have this in openrc right? And you have v3 rigth?
12:23 danpawlik degorenko: yes
12:24 degorenko danpawlik, 1) change it to v2; 2) run puppet manifests with env variable above set to v2.0
12:26 danpawlik degorenko: so I aslo should remove OS_USER_DOMAIN_NAME and OS_PROJECT_DOMAIN_NAME
12:26 degorenko danpawlik, i'm not sure, because v2 credentials class just will not use those parameters, you can try to leave them as is
12:34 pester joined #puppet-openstack
12:46 danpawlik degorenko: nope, it doesn't work
12:49 degorenko danpawlik, what's exactly?
12:51 openstackgerrit Eyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/429705
12:55 danpawlik about using v2 and not removing domain/project name:
12:55 danpawlik Ignoring domain related config project_domain_name because identity API version is 2.0
12:55 danpawlik Ignoring domain related config user_domain_name because identity API version is 2.0
12:55 danpawlik degorenko: checking one more time with removing domain/project
12:56 degorenko it's ignoring, so not used
13:02 ducttape_ joined #puppet-openstack
13:02 danpawlik degorenko: 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 danpawlik a puppet error*
13:02 mangerduchien joined #puppet-openstack
13:03 ansmith joined #puppet-openstack
13:03 danpawlik degorenko: and when you normally call by openstack client you will get empty list.
13:03 danpawlik degorenko: what I would like to add is that I want to use v3 not v2.0
13:03 degorenko well, actually backport to newton with v3 https://review.openstack.org/431463 - passed ci
13:04 danpawlik yeah
13:04 degorenko mwhahaha, EmilienM can we accept 431463 ? :)
13:05 mangerduchien hey all
13:05 danpawlik but now the main question is: should we enable v3 in newton or it should be enabled since ocata release
13:05 danpawlik ?
13:08 morazi joined #puppet-openstack
13:20 mwhahaha degorenko: yea because the v3 creds provider is backwards compatible with v2
13:21 degorenko mwhahaha, should we wait for fuel?
13:21 mwhahaha does fuel use those?
13:21 mwhahaha you can wait
13:21 degorenko only nova flavor i guess
13:21 degorenko let me check
13:22 degorenko https://github.com/openstack/fuel-library/blob/stable/newton/deployment/puppet/openstack_tasks/manifests/roles/controller.pp#L19-L24
13:22 gbarros joined #puppet-openstack
13:23 mwhahaha yea feel free to wait until fuel comes back, it should pass because the v3 creds are backwards compatible
13:23 degorenko yeah, i hope :D
13:23 danpawlik mwhahaha: so I
13:23 danpawlik :d
13:24 openstackgerrit Luigi Toscano proposed openstack/puppet-openstack-integration master: tempest-lib should not be required anymore  https://review.openstack.org/431518
13:27 fultonj_ joined #puppet-openstack
13:27 pradk joined #puppet-openstack
13:45 OGtrilliams joined #puppet-openstack
13:47 openstackgerrit Dan Radez proposed openstack/puppet-congress master: Adding acceptance test to Red Hat family  https://review.openstack.org/430492
13:47 dcritch joined #puppet-openstack
13:48 amorin joined #puppet-openstack
13:57 gbarros joined #puppet-openstack
14:04 ducttape_ joined #puppet-openstack
14: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_175117
14:08 xb_ joined #puppet-openstack
14:09 openstackgerrit Dan Radez proposed openstack/puppet-congress master: Adding acceptance test to Red Hat family  https://review.openstack.org/430492
14:10 degorenko danpawlik, +A :) v3 creds for newton
14:13 mwhahaha ansiwen: I'll look more in a minute but it looks like a parse error
14:22 chandankumar degorenko: please have a look on this https://review.openstack.org/#/c/430732/ when you are free
14:26 jperry joined #puppet-openstack
14:26 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration master: DNM: test centos7/consistent  https://review.openstack.org/424915
14:27 salmankhan joined #puppet-openstack
14:29 EmilienM mwhahaha: our puppet ci didn't promote to nova/trunk yet
14:29 EmilienM we still deploy ocata-3
14:29 EmilienM though tripleo did promote
14:30 acormier joined #puppet-openstack
14:30 acormier joined #puppet-openstack
14:31 owalsh-brb joined #puppet-openstack
14:33 dgurtner joined #puppet-openstack
14:33 dgurtner joined #puppet-openstack
14:34 acormier joined #puppet-openstack
14:39 mwhahaha ansiwen: looked transient since the recheck is passing
14:50 ansiwen mwhahaha: right, just saw it
14:53 snecklifter dtantsur: finally got the syntax working with [] - ie. http://paste.openstack.org/show/598252/
14:53 snecklifter FYI and for anyone else who stumbles across this archived on eavesdrop :)
14:53 dtantsur cool!
14:56 mwhahaha snecklifter: does that have to be a string? or could you do a yaml array of hashes?
14:57 mwhahaha snecklifter: http://paste.openstack.org/show/598254/
14:59 snecklifter mwhahaha: good question, I'm pretty sure it has to be a string https://github.com/openstack/puppet-nova/blob/master/manifests/api.pp#L77
14:59 mwhahaha snecklifter: https://github.com/openstack/puppet-nova/blob/master/manifests/api.pp#L481
14:59 udesale joined #puppet-openstack
15:00 mwhahaha though i don't know what check_array_of_hash is
15:00 snecklifter mwhahaha: ack but the example appears to show a string being passed
15:00 snecklifter mwhahaha: also, https://github.com/openstack/puppet-nova/blob/master/spec/classes/nova_api_spec.rb#L162
15:01 openstackgerrit Eyal proposed openstack/puppet-openstack-integration master: add vitrage  https://review.openstack.org/429705
15:01 snecklifter which is what led me to success in the end
15:01 mwhahaha yea it looks like  the string works just wondering if there's a cleaner way to do it :D
15:01 paramite joined #puppet-openstack
15:01 mwhahaha stringifying stuff always ends badly :D
15:02 mwhahaha snecklifter: looks like either, https://github.com/openstack/puppet-nova/blob/8938e4cff9fb7aa02f6b01037f3afef22792b8c8/lib/puppet/parser/functions/check_array_of_hash.rb
15:02 snecklifter mwhahaha: I'm sure there is, the SR-IOV code is only just starting to get some attention with OSP 10
15:02 mwhahaha snecklifter: so it's up to you on what you want to use, my preference would be proper yaml :)
15:02 snecklifter mwhahaha: your way looks much cleaner i agree
15:03 snecklifter I will try one more run with your config and report back!
15:04 gbarros joined #puppet-openstack
15:05 dfisher joined #puppet-openstack
15:05 paramite joined #puppet-openstack
15:17 agrebennikov_ joined #puppet-openstack
15:20 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration master: nova: start compute after keystone endpoints/services  https://review.openstack.org/431596
15:21 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration master: DNM: test centos7/consistent  https://review.openstack.org/424915
15:21 EmilienM mwhahaha: ^ might be the fix they suggest
15:21 * EmilienM cries
15:21 EmilienM mwhahaha: doing it in CI now and I'll think at a longterm solution in puppet-nova
15:21 mwhahaha HEH
15:29 snecklifter mwhahaha: hash array bombed out, it really wants a string
15:29 snecklifter https://paste.fedoraproject.org/551798/65412014/
15:29 mwhahaha interesting
15:30 snecklifter i think that the puppet_alias paramter wants a string to pass on
15:31 snecklifter *pci_alias
15:33 udesale joined #puppet-openstack
15:33 udesale joined #puppet-openstack
15:34 udesale joined #puppet-openstack
15:36 karthiks joined #puppet-openstack
15:38 karts joined #puppet-openstack
15:40 finchd joined #puppet-openstack
15:42 karts joined #puppet-openstack
15:46 udesale joined #puppet-openstack
15:48 acabot joined #puppet-openstack
15:50 salmankhan1 joined #puppet-openstack
16:00 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration master: DNM: test centos7/consistent  https://review.openstack.org/424915
16:00 xarses joined #puppet-openstack
16:00 mwhahaha snecklifter: 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 though
16:01 xarses joined #puppet-openstack
16:01 snecklifter mwhahaha: 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, thanks
16:02 andreww joined #puppet-openstack
16:02 snecklifter but would be good to know its definitely a bug before doing so
16:02 mwhahaha snecklifter: 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 it
16:03 mwhahaha snecklifter: stringified json in configs makes me sad
16:03 snecklifter mwhahaha: it makes me lose hours of my life trying to understand yaml and json...
16:04 mwhahaha stringify json in yaml to puppet to write out to an ini to be read by python
16:04 mwhahaha brilliant!
16:04 snecklifter mwhahaha: maybe its missing being piped through Go?
16:05 mwhahaha openstack deployment, http://wonderopolis.org/wp-content/uploads//2014/05/Rubenvent1.jpg
16:06 EmilienM mwhahaha: that's the step 1 of the deployment
16:08 salmankhan joined #puppet-openstack
16:10 gbarros joined #puppet-openstack
16:31 jaosorior joined #puppet-openstack
16:34 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration master: DNM: test centos7/consistent  https://review.openstack.org/424915
17:02 openstackgerrit Merged openstack/puppet-nova stable/newton: Enable Keystone v3 support for nova_flavor and nova_aggregate  https://review.openstack.org/431463
17:04 dgurtner joined #puppet-openstack
17:04 dgurtner joined #puppet-openstack
17:43 EmilienM mwhahaha: it seems it helped https://review.openstack.org/#/c/424915/
17:43 EmilienM only mistral is broken now
17:43 EmilienM dmsimard, iurygregory, mwhahaha: I need your power on https://review.openstack.org/#/c/431596/1
17:43 mwhahaha well that's good i guess
17:44 iurygregory EmilienM, boom
17:44 iurygregory ;)
17:45 dprince joined #puppet-openstack
18:04 ducttape_ joined #puppet-openstack
18:04 openstackgerrit Merged openstack/puppet-openstack-integration master: tempest-lib should not be required anymore  https://review.openstack.org/431518
18:04 openstackgerrit Merged openstack/puppet-nova stable/newton: Add a class to run the db online_data_migrations  https://review.openstack.org/427623
18:13 EmilienM so the mistral issue on trunk
18: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:14 EmilienM in 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.gz
18:15 jaosorior_mtg EmilienM: seems like a mistral error "cannot register CLI option"
18:16 EmilienM yeah but why we have it today and not yesterday :D
18:16 EmilienM https://github.com/openstack/mistral/commits/master
18:16 EmilienM I don't find anything useful
18:35 jaosorior joined #puppet-openstack
18:46 xarses joined #puppet-openstack
19:34 ducttape_ joined #puppet-openstack
19:36 mwhahaha did oslo cfg get updated?
19:37 EmilienM mwhahaha: that was my first thought
19:37 EmilienM if you look on #rdo
19:37 EmilienM let me check
19:38 * mwhahaha just got from lunch and goes a scrollin'
19:39 EmilienM python2-oslo-config-3.22.0-0.20170120192858.3025b6f.el7.centos.noarch when it worked
19:39 EmilienM python2-oslo-config-3.22.0-0.20170120192858.3025b6f.el7.centos.noarch when it fails
19:39 EmilienM so no
19:39 mwhahaha loles
19:39 mwhahaha got the diff handy?
19:40 EmilienM let me check
19:40 EmilienM https://www.diffchecker.com/DrhMDFoO
19:40 EmilienM so much things
19:41 mwhahaha i need another monitor
19:41 mwhahaha too many windows
19:41 EmilienM lol
19:42 EmilienM mwhahaha: http://infovis.cs.vt.edu/gigapixel/pictures/curved24mon/quake_front.jpg
19:42 mwhahaha which 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-accessories
19:42 bkero Oh man that's like 4 polygons per monitor
19:43 mwhahaha when i worked at rackspace there were guys with like 12
19:43 mwhahaha that's a bit much
19:43 mwhahaha i've always had 2, i was hoping 1 34" would suffice but it doesn't :(
19:43 bkero I need to just start doing work on my oculus rift
19:43 EmilienM are they more productive?
19:43 EmilienM they fix more nova bugs in the same time? #troll
19:43 mwhahaha :o
19:45 mwhahaha it's this change
19:45 mwhahaha EmilienM: https://review.openstack.org/#/c/425581/3/mistral/actions/openstack/action_generator/base.py@38
19:45 mwhahaha that's where it's blowing up
19:47 EmilienM that !
19:47 EmilienM let's revert it :D
19:47 mwhahaha my work here is done. http://i.imgur.com/xoi91ZR.png
19:47 EmilienM i'm worrying about why tripleo CI passed
19:47 EmilienM we're running Mistral on the undercloud
19:47 mwhahaha cause we don't deploy it in wsgi?
19:49 mwhahaha https://github.com/openstack/instack-undercloud/blob/master/elements/puppet-stack-config/puppet-stack-config.pp#L524
19:49 mwhahaha because we don't deploy it in wsgi.
19:49 EmilienM HE
19:49 EmilienM I hate when you're right
19:50 mwhahaha me too
19:50 mwhahaha so i guess the next question is, is 'wsgi' supported by mistral? or are we getting nove'd
19:50 EmilienM I'm going to ping mistral folks
19:50 mwhahaha s/nove/nova
19:51 EmilienM /join #openstack-mistral
20:09 EmilienM mwhahaha: nobody replies on mistral side
20:09 EmilienM I'm proposing a revert and disabling it in p-o-i
20:09 mwhahaha sounds reasonable
20:11 openstackgerrit Merged openstack/puppet-openstack-integration master: nova: start compute after keystone endpoints/services  https://review.openstack.org/431596
20:11 jaosorior joined #puppet-openstack
20:13 owalsh joined #puppet-openstack
20:13 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration master: scenario003: disable mistral  https://review.openstack.org/431733
20:14 mwhahaha EmilienM: we don't have to completely disable it, we can just switch to non wsgi
20:14 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration master: DNM: test centos7/consistent  https://review.openstack.org/424915
20:15 EmilienM mwhahaha: ssl / ipv6 is a pain to make it work outside apache
20:15 EmilienM I don' thave time today to do it
20:15 mwhahaha oh nm
20:15 mwhahaha turning it off it is
20:15 mwhahaha :D
20:15 EmilienM yes
20:21 openstackstatus NOTICE: Restarting gerrit due to performance problems
20:23 EmilienM please review https://review.openstack.org/#/c/431733/
20:32 salmankhan joined #puppet-openstack
20:33 openstackgerrit Dan Radez proposed openstack/puppet-congress master: Updating service name for Red Hat platform  https://review.openstack.org/431739
20:34 openstackgerrit Dan Radez proposed openstack/puppet-congress master: Adding acceptance test to Red Hat family  https://review.openstack.org/430492
20:42 rajinir_ joined #puppet-openstack
20:47 fultonj I 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#L239
20:47 fultonj [2] https://github.com/openstack/tripleo-heat-templates/blob/6b8c34b27a324120f6945f7873f479d37ba2e516/puppet/services/ceph-external.yaml#L84-L101
20:47 fultonj This same code worked in previous versions. What would be the correct fix for this?
20:48 fultonj a. 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:49 mwhahaha fultonj: question would be where is that used in puppet-ceph
20:50 fultonj mwhahaha: /etc/puppet/modules/ceph/manifests/keys.pp:31
20:50 mwhahaha fultonj: that validate_hash is 2 years old, so i assume this is failing because we recently added that class be included
20:51 fultonj mwhahaha: i removing that validate_hash to experiment, but it was rejected further down the line by keys.pp for the same reason
20:51 mwhahaha fultonj: where is ceph::profile::params::client_keys used
20:52 mwhahaha https://github.com/openstack/puppet-ceph/blob/d4cc06688c084a3c8570fdd42e75445d68129c12/manifests/profile/mon.pp#L41
20:52 mwhahaha fultonj: it has to be a hash
20:52 mwhahaha fultonj: create_resources needs a hash not a string, so fix THT
20:52 fultonj mwhahaha: agreed, it should be a hash
20:52 fultonj ok
20:53 fultonj mwhahaha: i'm used to thinking of tht as generating strings, i need to figure out how to make it generate a hash
20:53 fultonj mwhahaha: thanks for recommending a direction
20:53 mwhahaha fultonj: it's just a special string into yaml :D
20:53 fultonj so it should be a simple wrapper around that join
20:53 fultonj i will look for prior art and copy the style
20:53 xarses ++ on the hash
20:54 fultonj thanks mwhahaha and xarses
20:55 xarses aren't you reading the yaml in?
20:55 xarses to puppet?
20:55 ducttape_ joined #puppet-openstack
20:56 mwhahaha xarses: he needs to figure out how to generate the yaml in the heat template since it's dynamic
20:56 fultonj xarses: the yaml ends up in hiera inbetween
20:57 fultonj http://paste.openstack.org/show/598302/
20:58 fultonj that looks like a hash to me, i.e. it's not quoted
20:58 fultonj not on the outside
20:59 fultonj am 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?
21:00 fultonj i will go to tht to see about generating a hash, but i was just curious about that part
21:04 mwhahaha fultonj: that's not a yaml hash
21:05 mwhahaha fultonj: sec
21:07 mwhahaha fultonj: what do you get for hiera -h ceph::profile::params::client_keys
21:07 fultonj http://paste.openstack.org/show/598305/
21:08 mwhahaha fultonj: yea it's a string in the hiera files
21:08 fultonj i think i see your point now
21:08 fultonj thanks, i'll take note of that -h option
21:08 mwhahaha fultonj: sec it should be really easy
21:08 mwhahaha actually doing what's there is harder than it needs to be
21:10 mwhahaha fultonj: https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/services/ceph-external.yaml#L91-L108
21:11 mwhahaha fultonj: whatever version you were looking at needs to have that format backported
21:12 fultonj mwhahaha: got it map_replace vs string_replace
21:14 openstackgerrit Dan Prince proposed openstack/puppet-nova master: Nova placement doesn't require nova::api  https://review.openstack.org/431744
21:14 fultonj dah...
21:14 fultonj gfidente already fixed it :)
21:14 fultonj https://github.com/openstack/tripleo-heat-templates/commit/8bf04682013472dcc696dde7dc7b5e48d33bb7bc
21:14 fultonj that's what i get for not updating. thanks sorry i didn't see that earlier
21:14 mwhahaha yea that's why i was like it needs to be backported :D
21:19 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration master: scenario003: disable mistral  https://review.openstack.org/431733
21:19 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration master: DNM: test centos7/consistent  https://review.openstack.org/424915
21:19 paramite joined #puppet-openstack
21:23 radez mwhahaha: EmilienM: if you guys have a sec would you mind taking a look at: https://review.openstack.org/#/c/431739/
21:23 EmilienM no way
21:24 EmilienM something 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_245919
21:24 mwhahaha how did that pass
21:24 EmilienM http://logs.openstack.org/92/430492/4/check/gate-puppet-congress-puppet-beaker-rspec-centos-7/8736738/logs/syslog.txt.gz#_Feb_09_20_46_16
21:25 EmilienM yeah
21:25 EmilienM mwhahaha: because he put the tests on top of the patch
21:25 EmilienM radez: ^
21:26 radez EmilienM: hm, k thx, that looks like it's at the package level
21:29 mwhahaha tho it'd probably be ok to at least merge the name fix
21:30 mwhahaha cause that's not going to change :D
21:30 EmilienM ok
21:30 mwhahaha and it'd be nice to get it in the rc1
21:30 EmilienM I'm +2 it
21:30 mwhahaha which hasn't been cut yet
21:35 openstackgerrit Merged openstack/puppet-congress master: Updating service name for Red Hat platform  https://review.openstack.org/431739
21:54 radez thx for merging the name fix, I'll get the package fixed so the acceptance passes
22:10 ducttape_ joined #puppet-openstack
22:15 EmilienM radez: cool
22:15 EmilienM radez: and tripleo too please
22:16 openstackgerrit Merged openstack/puppet-tempest master: Add Octavia Tempest plugin support.  https://review.openstack.org/430732
22:38 jaosorior joined #puppet-openstack
22:43 PagliaccisCloud joined #puppet-openstack
22:49 openstackgerrit Merged openstack/puppet-openstack-integration master: scenario003: disable mistral  https://review.openstack.org/431733
23:11 jaosorior joined #puppet-openstack
23:12 hecliunyx joined #puppet-openstack
23:28 openstackgerrit Merged openstack/puppet-ceph master: Automatically zap a disk if no valid gpt label is present  https://review.openstack.org/420992
23:50 dprince joined #puppet-openstack

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