Monday, 2018-11-26

« Sunday, 2018-11-25 Index Tuesday, 2018-11-27 »
openstackgerritMerged openstack/puppet-openstack-integration master: Move out debug to keystone::logging  https://review.openstack.org/61947500:21
*** tosky has quit IRC00:39
openstackgerritzhongshengping proposed openstack/puppet-senlin master: Remove pki related options  https://review.openstack.org/61940601:14
*** rcernin has quit IRC01:36
*** ykarel has joined #puppet-openstack04:23
*** chkumar has joined #puppet-openstack04:34
*** ykarel has quit IRC04:42
*** ykarel has joined #puppet-openstack05:02
*** chkumar is now known as chkumar|ruck05:27
*** chkumar has joined #puppet-openstack05:44
*** chkumar|ruck has quit IRC05:47
*** chkumar has quit IRC05:48
*** chkumar246 has joined #puppet-openstack06:03
*** chkumar246 is now known as chkumar|ruck06:44
openstackgerrit98k proposed openstack/puppet-mistral master: Add bugs link in README  https://review.openstack.org/61693207:22
*** skramaja has joined #puppet-openstack07:25
*** ykarel is now known as ykarel|lunch07:26
*** iurygregory has joined #puppet-openstack07:39
*** agurenko has joined #puppet-openstack07:41
iurygregorygood morning07:41
*** mschuppert has joined #puppet-openstack07:44
*** gfidente has joined #puppet-openstack07:47
*** jtomasek has joined #puppet-openstack07:47
*** ykarel|lunch is now known as ykarel08:14
*** shardy has joined #puppet-openstack08:16
*** iurygregory has quit IRC08:21
*** iurygregory has joined #puppet-openstack08:23
*** shardy has quit IRC08:32
*** chem has joined #puppet-openstack08:43
*** tosky has joined #puppet-openstack08:46
*** jpena|off is now known as jpena08:49
*** shardy has joined #puppet-openstack08:57
*** shardy has quit IRC09:06
*** shardy has joined #puppet-openstack09:08
*** shardy has quit IRC09:17
*** mgagne_ has joined #puppet-openstack09:23
*** zigo has quit IRC09:25
*** EmilienM has quit IRC09:25
*** mgagne has quit IRC09:25
*** chkumar|ruck has quit IRC09:29
*** chkumar246 has joined #puppet-openstack09:31
*** chkumar246 is now known as chkumar|ruck09:31
*** jaosorior has joined #puppet-openstack09:38
*** derekh has joined #puppet-openstack09:38
*** agurenko has quit IRC09:45
lyarwoodtobias-urdin: https://review.openstack.org/#/c/615568/ - morning, if I understand the zuul job correctly we can't move forward with this until the initial puppet-placement change (https://review.openstack.org/#/c/604182/) lands correct?10:12
lyarwoodAFAICT the Depends-On isn't used and we are just pulling puppet-placement master that's currently empty10:12
openstackgerritLee Yarwood proposed openstack/puppet-openstack-integration master: WIP Extract the Placement service from Nova  https://review.openstack.org/61556810:19
lyarwood^ trying to force it to use the WIP change10:20
*** skramaja_ has joined #puppet-openstack10:29
*** skramaja has quit IRC10:30
*** salmankhan has joined #puppet-openstack10:36
*** skramaja_ is now known as skramaja10:38
*** gfidente has quit IRC10:46
*** zigo has joined #puppet-openstack10:52
*** gfidente has joined #puppet-openstack10:53
*** salmankhan has quit IRC11:08
*** salmankhan has joined #puppet-openstack11:08
*** dpawlik has joined #puppet-openstack11:33
tobias-urdinlyarwood: hm, yeah it doesn't seems like it's loading those commits11:49
tobias-urdinzuul hasn't or openstack infra hasn't dropped the Depends-On: <change id> syntax yet?11:50
tobias-urdinor is the change ids correct?11:50
openstackgerritVieri proposed openstack/puppet-neutron master: Add bugs link in README  https://review.openstack.org/61693811:52
openstackgerritVieri proposed openstack/puppet-zaqar master: Always build universal wheels  https://review.openstack.org/60779611:53
openstackgerritLee Yarwood proposed openstack/puppet-openstack-integration master: WIP Extract the Placement service from Nova  https://review.openstack.org/61556811:54
lyarwoodtobias-urdin: I didn't think so, ^ is using the new URL syntax11:54
tobias-urdinlyarwood: seems like puppet-placement is broken12:25
tobias-urdinnot a dependency issue12:25
tobias-urdinatleast for debian based12:27
tobias-urdinlets see what centos says12:27
tobias-urdins/debian/ubuntu/g12:27
lyarwoodtobias-urdin: yeah I need to work on the params for debian12:31
lyarwoodtobias-urdin: testing this has been pretty awkward tbh12:31
tobias-urdini pinged corey from canonical on that, will relay back to you if i recieve a reponse12:31
tobias-urdinresponse12:31
lyarwoodthanks12:32
*** jpena is now known as jpena|lunch12:38
tobias-urdinlyarwood: Keystone::Cron::Fernet_rotate/Cron[keystone-manage fernet_rotate] (notice): Dependency File[/etc/placement/ssl] has failures: true12:44
tobias-urdinlooks wrong12:44
tobias-urdin2018-11-26 12:22:18 +0000 Puppet (err): Cannot create /etc/placement/ssl; parent directory /etc/placement does not exist12:46
lyarwoodtobias-urdin: yeah looking at that now12:47
tobias-urdinthe openstack_integration::ssl_key placement resource needs to be chained after placement packages is installed12:48
*** EmilienM has joined #puppet-openstack12:57
openstackgerritLee Yarwood proposed openstack/puppet-openstack-integration master: WIP Extract the Placement service from Nova  https://review.openstack.org/61556812:57
*** dtantsur|afk is now known as dtantsur|mtg12:59
openstackgerritLee Yarwood proposed openstack/puppet-placement master: Initial cookiecutter and import from nova::placement  https://review.openstack.org/60418213:20
*** trown|outtypewww is now known as trown13:27
*** jpena|lunch is now known as jpena13:29
*** gouthamr has quit IRC13:34
*** rpittau has joined #puppet-openstack13:38
*** gouthamr has joined #puppet-openstack13:40
tobias-urdinlyarwood: corey will hopefully work on placement packages this week, maybe next week we can test with ubuntu packages as well13:44
tobias-urdinthere is also stein packages for ubuntu but they are not really tested, and is py3 only so will break a lot of stuff13:44
lyarwoodtobias-urdin: ack thanks13:45
openstackgerritTobias Urdin proposed openstack/puppet-openstack_extras master: Change repos to stein  https://review.openstack.org/62005413:48
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration master: Change ubuntu repos to stein  https://review.openstack.org/62005513:49
*** gfidente has quit IRC13:53
*** gfidente has joined #puppet-openstack13:55
*** mmethot has joined #puppet-openstack14:06
*** chkumar|ruck has quit IRC14:14
tobias-urdinEmilienM: when you have time, could you review my latest ~15 patches that is passing, ty14:24
EmilienMtobias-urdin: sure14:25
tobias-urdinEmilienM: know tripleo is busy with a lot of other stuff, but do you think these kind of efforts could be completed this cycle from tripleo perspective?14:28
tobias-urdinhttps://review.openstack.org/#/c/619380/ https://review.openstack.org/#/c/619257/ https://review.openstack.org/#/c/619201/14:28
tobias-urdinsome heavy deprecations/removals and changing keystone to port 5000 only (one port only atleast)14:28
tobias-urdinif it's even used that is... i have no insight there :)14:29
EmilienMjaosorior: ^14:30
EmilienMthis work needs to be more visible14:30
jaosoriornice! been wanting to get rid of 35357 for a long time :D14:31
jaosoriorI don't think we use it much anymore.14:31
jaosoriorin tripleo14:31
jaosoriorBUT.... I think some third party integrations use it14:31
jaosoriorIIRC14:31
jaosoriorso... that might be problematic14:31
*** bnemec has joined #puppet-openstack14:33
tobias-urdinjust guessing here, but might be possible to pass an array to $port that is passed to openstacklib -> puppetlabs-apache14:33
tobias-urdinand that way still listen to port 35357 but from same vhost14:33
tobias-urdinassuming you have dropped keystone v2.0 support, v3 is the same app anyways14:34
*** skramaja has quit IRC14:38
jaosoriortrue that14:43
jaosoriortobias-urdin: that would be ideal14:43
jaosoriorif we could still keep 35357, ven if it points towards the same host14:43
tobias-urdinjaosorior: if that works keystone::wsgi::apache::port would default to 5000 and users could pass port=[35357, 5000] to be backward compatible14:50
jaosoriortobias-urdin: sounds good to me.14:51
*** hjensas has quit IRC14:55
*** rpittau has quit IRC14:55
openstackgerritTobias Urdin proposed openstack/puppet-keystone master: Remove port 35357 deployment  https://review.openstack.org/61925715:03
*** dpawlik has quit IRC15:08
*** ykarel is now known as ykarel|away15:10
*** dpawlik has joined #puppet-openstack15:10
*** chkumar|away has joined #puppet-openstack15:14
*** ykarel|away has quit IRC15:16
*** rpittau has joined #puppet-openstack15:19
*** dpawlik has quit IRC15:39
*** ykarel|away has joined #puppet-openstack15:41
*** gfidente has quit IRC15:42
*** gfidente has joined #puppet-openstack15:43
*** dpawlik has joined #puppet-openstack15:55
*** ykarel|away has quit IRC15:55
openstackgerritAlan Bishop proposed openstack/puppet-cinder stable/queens: Add cinder::nova class to configure nova section  https://review.openstack.org/62009315:56
openstackgerritAlan Bishop proposed openstack/puppet-cinder stable/queens: Improve cinder::nova support for auth_type password  https://review.openstack.org/62009415:56
*** dpawlik has quit IRC15:59
*** salmankhan has left #puppet-openstack16:03
openstackgerritAlex Schultz proposed openstack/puppet-qdr master: Support broker connector configuration.  https://review.openstack.org/61736616:06
*** chkumar|away has quit IRC16:11
*** hjensas has joined #puppet-openstack16:28
openstackgerritMerged openstack/puppet-qdr master: Revert "Add a broker connector configuration"  https://review.openstack.org/61856716:49
*** rpittau has quit IRC16:52
*** ykarel|away has joined #puppet-openstack16:53
openstackgerritNagasai Vinaykumar Kapalavai proposed openstack/puppet-qdr master: Changes to include broker connector configuration.  https://review.openstack.org/61736616:57
cgoncalvestobias-urdin, hi. have you seen my last comment on https://review.openstack.org/#/c/619307/ ?17:06
tobias-urdinwhy?17:07
tobias-urdinsorry been afk for some hour17:07
cgoncalvestobias-urdin, touche! :)17:07
openstackgerritLee Yarwood proposed openstack/puppet-placement master: Initial cookiecutter and import from nova::placement  https://review.openstack.org/60418217:08
cgoncalveswe had a short discussion last week here on the channel where mwhahaha also participated. my understanding from it was that my original commit was good17:08
cgoncalvesif that is correct, we should revert https://review.openstack.org/#/c/619013/17:09
tobias-urdini abandoned all that, changed my use case to have octavia::controller defined before octavia::worker17:09
tobias-urdinusing it in production right now17:09
tobias-urdiniiuc if we revert that it will be broken again17:11
tobias-urdinthose includes does not honor that octavia::controller is defined in manifest before them and will cause redeclaration if you use all of them in the same manifest17:11
mwhahahai guess that's a hiera driven composition layer vs static class definitions17:12
tobias-urdinneed to leave now, will be online in ~40 mins when i get home17:13
* tobias-urdin afk -- back sson17:13
cgoncalvesmwhahaha, right. so which one should win? :)17:14
mwhahahayea it gets tricky. I think we've tried to not force too many of those if possible17:15
*** ykarel|away has quit IRC17:15
mwhahahawe've had to do it for teh db stuff because that's required, but i'm not sure about the controller bits17:15
mwhahahawe could have probably gotten away w/o including it everywhere maybe?17:15
cgoncalvestobias-urdin, in here https://review.openstack.org/#/c/619013/2/manifests/api.pp you half-reverted it: you removed the include of controller but did not re-include the db (https://review.openstack.org/#/c/612395/11/manifests/api.pp)17:16
cgoncalvesmwhahaha, I think we could live without including everywhere but then we'd need to include it puppet-tripleo17:16
cgoncalveshttps://github.com/openstack/puppet-tripleo/tree/master/manifests/profile/base/octavia17:17
openstackgerritNagasai Vinaykumar Kapalavai proposed openstack/puppet-qdr master: Changes to include broker connector configuration.  https://review.openstack.org/61736617:24
cgoncalvesmwhahaha, could you have a look at these patches and confirm this is the proper way forward? I have NOT tested them yet. https://review.openstack.org/#/c/620106/ & https://review.openstack.org/#/c/620118/17:46
mwhahahacgoncalves: yes i believe so17:47
cgoncalveslol, okay thanks! I've got a record for breaking stuff even when I test them so... ;)17:48
*** derekh has quit IRC17:52
*** jpena is now known as jpena|off18:13
* tobias-urdin is online again18:39
*** hjensas has quit IRC18:50
*** josephrsandoval has joined #puppet-openstack18:55
openstackgerritRajini Karthik proposed openstack/puppet-cinder stable/rocky: Dell EMC SC: Add use_multipath_for_image_xfer  https://review.openstack.org/61461819:35
openstackgerritRajini Karthik proposed openstack/puppet-cinder stable/rocky: Dell EMC SC: Add use_multipath_for_image_xfer  https://review.openstack.org/61461819:37
openstackgerritRajini Karthik proposed openstack/puppet-cinder stable/rocky: Dell EMC SC: Add use_multipath_for_image_xfer  https://review.openstack.org/61461819:38
openstackgerritRajini Karthik proposed openstack/puppet-cinder stable/queens: Dell EMC SC: Add use_multipath_for_image_xfer  https://review.openstack.org/61461919:40
*** josephrsandoval has quit IRC19:41
openstackgerritRajini Karthik proposed openstack/puppet-cinder stable/queens: Dell EMC SC: Add use_multipath_for_image_xfer  https://review.openstack.org/61461919:42
openstackgerritRajini Karthik proposed openstack/puppet-cinder stable/queens: Dell EMC SC: Add use_multipath_for_image_xfer  https://review.openstack.org/61461919:42
openstackgerritRajini Karthik proposed openstack/puppet-cinder stable/queens: Dell EMC SC: Add use_multipath_for_image_xfer  https://review.openstack.org/61461919:43
openstackgerritRajini Karthik proposed openstack/puppet-cinder stable/rocky: Dell EMC SC: Add use_multipath_for_image_xfer  https://review.openstack.org/61461819:45
cgoncalvestobias-urdin, in here https://review.openstack.org/#/c/619013/2/manifests/api.pp you half-reverted it: you removed the include of controller but did not re-include the db (https://review.openstack.org/#/c/612395/11/manifests/api.pp)19:49
tobias-urdinthanks for noticing that, we should probably add that back right?19:52
cgoncalvestobias-urdin, yes. the api service talks to the db19:54
openstackgerritCarlos Goncalves proposed openstack/puppet-octavia master: Re-include octavia::db in octavia::api  https://review.openstack.org/62014720:05
cgoncalvestobias-urdin, ^20:05
tobias-urdinty20:09
openstackgerritAlex Schultz proposed openstack/puppet-nova master: Update python packaging version for RHEL8  https://review.openstack.org/62014920:23
openstackgerritAlex Schultz proposed openstack/puppet-glance master: Update python packaging version for RHEL8  https://review.openstack.org/62015020:24
cgoncalveshmm, the housekeeping and health manager services also need to include db btw20:29
cgoncalvesif one installs them together with the api and/or worker the db params will be set but if standalone they will not20:30
*** salmankhan has joined #puppet-openstack20:30
cgoncalvesnothing super urgent at least for me and for the time being, though20:30
openstackgerritTobias Urdin proposed openstack/puppet-monasca master: Convert to rspec-puppet-facts and fix monasca::alarmdefs  https://review.openstack.org/61988820:37
*** morazi has joined #puppet-openstack20:38
*** morazi has quit IRC21:00
openstackgerritMerged openstack/puppet-nova master: Deprecate neutron options and add alternatives  https://review.openstack.org/61352921:04
*** gfidente has quit IRC21:35
*** salmankhan has quit IRC21:37
*** salmankhan has joined #puppet-openstack21:37
*** jtomasek has quit IRC21:39
*** salmankhan has quit IRC21:49
*** salmankhan has joined #puppet-openstack21:49
*** salmankhan has quit IRC21:54
*** rcernin has joined #puppet-openstack23:01
*** tosky has quit IRC23:32
*** tosky has joined #puppet-openstack23:32
« Sunday, 2018-11-25 Index Tuesday, 2018-11-27 »

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