Thursday, 2018-05-24

*** rpittau has quit IRC01:01
*** markvoelker has joined #puppet-openstack01:08
*** colonwq has joined #puppet-openstack01:24
openstackgerritMerged openstack/puppet-cinder master: Remove deprecated parameters  https://review.openstack.org/56964001:40
*** ilbot3 has quit IRC01:47
openstackgerritzhongshengping proposed openstack/puppet-cloudkitty master: [DNM] test  https://review.openstack.org/57028801:54
*** ilbot3 has joined #puppet-openstack01:57
*** skramaja has joined #puppet-openstack02:09
openstackgerritzhongshengping proposed openstack/puppet-heat master: Remove deprecated rabbit params  https://review.openstack.org/57029502:11
openstackgerritzhongshengping proposed openstack/puppet-ironic master: Remove deprecated rabbit params  https://review.openstack.org/57029602:14
openstackgerritzhongshengping proposed openstack/puppet-magnum master: Remove deprecated rabbit params  https://review.openstack.org/57029802:19
openstackgerritMerged openstack/puppet-barbican master: Remove deprecated parameters  https://review.openstack.org/56960702:31
openstackgerritzhongshengping proposed openstack/puppet-magnum master: Remove deprecated rabbit params  https://review.openstack.org/57029802:36
openstackgerritzhongshengping proposed openstack/puppet-manila master: Remove deprecated rabbit params  https://review.openstack.org/57030102:37
openstackgerritzhongshengping proposed openstack/puppet-mistral master: Remove deprecated rabbit params  https://review.openstack.org/57030302:43
openstackgerritzhongshengping proposed openstack/puppet-heat master: Remove deprecated rabbit params  https://review.openstack.org/57029502:48
openstackgerritzhongshengping proposed openstack/puppet-ironic master: Remove deprecated rabbit params  https://review.openstack.org/57029602:49
openstackgerritzhongshengping proposed openstack/puppet-magnum master: Remove deprecated rabbit params  https://review.openstack.org/57029802:50
openstackgerritzhongshengping proposed openstack/puppet-manila master: Remove deprecated rabbit params  https://review.openstack.org/57030102:51
openstackgerritzhongshengping proposed openstack/puppet-mistral master: Remove deprecated rabbit params  https://review.openstack.org/57030302:52
openstackgerritzhongshengping proposed openstack/puppet-neutron master: Remove deprecated rabbit params  https://review.openstack.org/57030703:01
openstackgerritzhongshengping proposed openstack/puppet-octavia master: Remove deprecated rpc_backend param  https://review.openstack.org/57030803:04
openstackgerritzhongshengping proposed openstack/puppet-sahara master: Remove deprecated rabbit params  https://review.openstack.org/57030903:09
openstackgerritzhongshengping proposed openstack/puppet-tacker master: Remove deprecated rpc_backend param  https://review.openstack.org/57031003:11
openstackgerritzhongshengping proposed openstack/puppet-trove master: Remove deprecated rabbit params  https://review.openstack.org/57031103:15
openstackgerritzhongshengping proposed openstack/puppet-trove master: Remove deprecated heat_service_type param  https://review.openstack.org/57031203:19
openstackgerritzhongshengping proposed openstack/puppet-vitrage master: Remove deprecated rpc_backend param  https://review.openstack.org/57031303:20
openstackgerritzhongshengping proposed openstack/puppet-watcher master: Remove deprecated rpc_backend param  https://review.openstack.org/57031403:25
*** skramaja has quit IRC03:43
*** skramaja has joined #puppet-openstack03:43
*** rcernin has quit IRC03:58
*** rcernin has joined #puppet-openstack04:17
*** rcernin has quit IRC04:23
*** rcernin has joined #puppet-openstack04:23
*** openstackgerrit has quit IRC04:34
*** ykarel|afk has joined #puppet-openstack04:35
*** rcernin has quit IRC04:56
*** openstackgerrit has joined #puppet-openstack05:19
openstackgerritMerged openstack/puppet-openstack-integration master: Revert "Debian: Setup Cinder using cinder-api uwsgi"  https://review.openstack.org/56942005:19
openstackgerritMerged openstack/puppet-ec2api stable/queens: Configure metadata_proxy_shared_secret as secret  https://review.openstack.org/57012605:20
openstackgerritMerged openstack/puppet-ec2api stable/pike: Configure metadata_proxy_shared_secret as secret  https://review.openstack.org/57012705:20
openstackgerritMerged openstack/puppet-ironic master: Add pool_timeout option  https://review.openstack.org/57013505:25
openstackgerritMerged openstack/puppet-ironic master: Add support for db_max_retries param  https://review.openstack.org/57013805:25
openstackgerritMerged openstack/puppet-swift stable/queens: Configure www_authenticate_uri instead of auth_uri in authtoken  https://review.openstack.org/56992505:29
*** markvoelker_ has joined #puppet-openstack05:36
*** markvoelker has quit IRC05:37
*** jaganathan has quit IRC05:50
*** jaganathan has joined #puppet-openstack05:50
openstackgerritzhongshengping proposed openstack/puppet-trove master: Remove deprecated rabbit params  https://review.openstack.org/57031105:58
openstackgerritzhongshengping proposed openstack/puppet-mistral master: Remove deprecated rabbit params  https://review.openstack.org/57030306:01
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/56901906:07
openstackgerritzhongshengping proposed openstack/puppet-heat master: Remove deprecated rabbit params  https://review.openstack.org/57029506:07
openstackgerritzhongshengping proposed openstack/puppet-ironic master: Remove deprecated rabbit params  https://review.openstack.org/57029606:14
openstackgerritzhongshengping proposed openstack/puppet-manila master: Remove deprecated rabbit params  https://review.openstack.org/57030106:15
*** olap has joined #puppet-openstack06:18
*** olap has quit IRC06:18
*** olap has joined #puppet-openstack06:18
*** rpittau has joined #puppet-openstack06:37
openstackgerritMerged openstack/puppet-neutron stable/pike: Fixes missing 'geneve' network type  https://review.openstack.org/57012906:38
*** rcernin has joined #puppet-openstack06:43
*** alee has joined #puppet-openstack06:43
*** rpittau has quit IRC06:51
*** tesseract has joined #puppet-openstack07:23
*** ykarel|afk is now known as ykarel|lunch07:30
openstackgerritThomas Goirand proposed openstack/puppet-neutron master: WIP: Support neutron-{api,rpc-server} services  https://review.openstack.org/57025007:36
*** jpena|off is now known as jpena07:37
openstackgerritzhongshengping proposed openstack/puppet-trove master: Remove deprecated rabbit params  https://review.openstack.org/57031107:51
*** jistr is now known as jistr|iview07:57
*** ykarel|lunch is now known as ykarel08:12
openstackgerritThomas Goirand proposed openstack/puppet-neutron master: WIP: Support neutron-{api,rpc-server} services  https://review.openstack.org/57025008:13
*** jaganathan has quit IRC08:22
*** jaganathan has joined #puppet-openstack08:29
*** rcernin has quit IRC08:36
*** derekh has joined #puppet-openstack08:42
*** amoralej|off is now known as amoralej08:55
openstackgerritThomas Goirand proposed openstack/puppet-neutron master: WIP: Support neutron-{api,rpc-server} services  https://review.openstack.org/57025009:06
*** jistr|iview is now known as jistr09:07
*** Tengu has quit IRC09:07
openstackgerritThomas Goirand proposed openstack/puppet-neutron master: WIP: Support neutron-{api,rpc-server} services  https://review.openstack.org/57025009:08
*** morazi has joined #puppet-openstack09:14
*** salmankhan has joined #puppet-openstack09:16
openstackgerritThomas Goirand proposed openstack/puppet-neutron master: Support neutron-{api,rpc-server} services  https://review.openstack.org/57025009:50
*** gfidente has joined #puppet-openstack10:15
*** gfidente has quit IRC10:15
*** gfidente has joined #puppet-openstack10:15
*** dtantsur|afk is now known as dtantsur10:30
*** olap has quit IRC10:38
*** jpena is now known as jpena|lunch10:58
*** amoralej is now known as amoralej|off11:07
*** jpena|lunch is now known as jpena|off11:10
*** vinsh has quit IRC11:33
*** vinsh has joined #puppet-openstack11:33
*** olap has joined #puppet-openstack11:42
*** rasca has quit IRC11:44
*** gfidente has quit IRC11:51
*** gfidente has joined #puppet-openstack11:55
*** jpena|off is now known as jpena11:58
*** skramaja has quit IRC12:07
*** vinsh has quit IRC12:21
*** vinsh has joined #puppet-openstack12:22
zigomnaser: mwhahaha: With these 2, scenario001 passes here: https://review.openstack.org/#/c/570250/ https://review.openstack.org/#/c/564328/12:30
zigoThat last one needs unit tests fixups, of course.12:30
zigoCan you look at the first one, ie: https://review.openstack.org/#/c/570250/ please ?12:30
zigoIt probably needs some variable updates, like neutron-server-eventlet, maybe, please do tell me...12:31
*** trown|outtypewww is now known as trown12:44
*** rasca has joined #puppet-openstack12:49
*** gfidente has quit IRC12:57
*** jistr is now known as jistr|mtg13:14
*** ykarel is now known as ykarel|away13:17
*** gfidente has joined #puppet-openstack13:20
*** amoralej|off is now known as amoralej13:35
*** ykarel|away has quit IRC13:46
*** tesseract-RH has joined #puppet-openstack13:54
*** tesseract-RH has quit IRC13:55
*** tesseract-RH has joined #puppet-openstack13:55
*** tesseract has quit IRC13:56
*** derekh has quit IRC13:59
*** jistr|mtg is now known as jistr14:02
*** olap has quit IRC14:13
openstackgerritThomas Goirand proposed openstack/puppet-neutron master: Support neutron-{api,rpc-server} services  https://review.openstack.org/57025014:13
*** rook has quit IRC14:15
*** rook has joined #puppet-openstack14:17
*** rook is now known as Guest1107914:18
*** derekh has joined #puppet-openstack14:21
*** Guest11079 has quit IRC14:26
*** rook_ has joined #puppet-openstack14:29
*** tesseract-RH has quit IRC14:47
*** jtomasek has joined #puppet-openstack14:54
*** tesseract-RH has joined #puppet-openstack15:02
*** tesseract-RH has quit IRC15:03
*** tesseract has joined #puppet-openstack15:03
*** markvoelker_ has quit IRC15:07
*** markvoelker has joined #puppet-openstack15:07
*** markvoelker has quit IRC15:11
*** rook_ is now known as rook15:20
*** jtomasek has quit IRC15:35
*** ykarel|away has joined #puppet-openstack15:51
*** ykarel|away has quit IRC16:00
*** markvoelker has joined #puppet-openstack16:00
*** markvoelker has quit IRC16:05
*** markvoelker has joined #puppet-openstack16:12
*** jaganathan has quit IRC16:19
*** markvoelker has quit IRC16:21
*** alee has quit IRC16:25
*** trown is now known as trown|lunch16:29
*** markvoelker has joined #puppet-openstack16:31
*** markvoelker has quit IRC16:44
*** alee has joined #puppet-openstack16:56
*** derekh has quit IRC17:00
*** amoralej is now known as amoralej|off17:03
openstackgerritThomas Goirand proposed openstack/puppet-neutron master: Support neutron-{api,rpc-server} services  https://review.openstack.org/57025017:09
*** dtantsur is now known as dtantsur|afk17:11
*** tesseract has quit IRC17:25
*** jpena is now known as jpena|off17:28
*** gfidente has quit IRC17:39
*** trown|lunch is now known as trown17:44
*** alee has quit IRC17:46
*** beagles is now known as beagles_brb17:46
*** rpittau has joined #puppet-openstack17:53
*** alee has joined #puppet-openstack17:53
*** salmankhan has quit IRC18:14
*** morazi has quit IRC18:31
*** rajinir has joined #puppet-openstack18:37
*** beagles_brb is now known as beagles18:40
*** alee has quit IRC18:40
*** rpittau has quit IRC18:41
*** rpittau has joined #puppet-openstack18:46
*** alee has joined #puppet-openstack18:52
openstackgerritThomas Goirand proposed openstack/puppet-nova master: Fix VNC console in Debian  https://review.openstack.org/56432819:06
*** markvoelker has joined #puppet-openstack19:31
*** markvoelker has quit IRC19:33
*** beagles is now known as Guest8785119:34
*** Hunner is now known as Guest327119:34
*** tlbr is now known as Guest3411619:34
*** dmsimard is now known as Guest9444819:34
*** salmankhan has joined #puppet-openstack19:34
*** alee has quit IRC19:35
*** rpittau has quit IRC19:36
*** Guest3271 is now known as Hunner19:48
*** Guest94448 has quit IRC19:51
*** salmankhan has quit IRC19:51
*** dmsimard has joined #puppet-openstack19:52
*** markvoelker has joined #puppet-openstack19:56
*** salmankhan has joined #puppet-openstack19:56
*** markvoelker has quit IRC20:04
*** salmankhan has quit IRC20:11
*** markvoelker has joined #puppet-openstack20:14
*** trown is now known as trown|outtypewww20:27
*** salmankhan has joined #puppet-openstack20:38
*** Guest87851 is now known as beagles20:57
*** rpittau has joined #puppet-openstack21:02
*** shardy has quit IRC21:08
*** shardy has joined #puppet-openstack21:09
*** beagles is now known as beagles_biab21:14
*** salmankhan has quit IRC21:15
*** salmankhan has joined #puppet-openstack21:20
*** rpittau has quit IRC21:36
*** alee has joined #puppet-openstack21:38
*** salmankhan has quit IRC21:40
*** rpittau has joined #puppet-openstack21:42
*** beagles_biab is now known as beagles21:45
openstackgerritThomas Goirand proposed openstack/puppet-nova master: Fix VNC console in Debian  https://review.openstack.org/56432821:46
*** salmankhan has joined #puppet-openstack21:50
*** salmankhan has quit IRC21:56
*** dpawlik has joined #puppet-openstack22:06
*** rajinir has quit IRC22:07
*** dpawlik_ has joined #puppet-openstack22:08
*** dpawlik_ has quit IRC22:08
zigomwhahaha: On my patch here: https://review.openstack.org/#/c/564328/ I believe that the $enable parameter for both manifests/vncproxy.pp and manifests/spicehtml5proxy.pp should be set to true by default, which is why I'm getting unit tests failures. Your thoughts?22:09
mwhahahazigo: it wasn't in testing because ::nova::compute is not included in the test22:10
mwhahahawhich was my comment about passing the param in22:10
mwhahahathe unit tests are for testing just the specific class, so you'd have to pass in the appropriate logic22:10
mwhahahathe deployment tests would likely have the actual enable for both22:11
zigomwhahaha: But it doesn't make sense to include compute in a machine which is a proxy node.22:11
*** dpawlik has quit IRC22:11
*** rcernin has joined #puppet-openstack22:11
zigomwhahaha: It's also going to fail deployments that aren't all-in-one, because proxy nodes wont include ::nova::compute ...22:11
mwhahaharight22:12
mwhahahazigo: so it's likely you'd leave true for nova::spicehtml5proxy22:12
mwhahahabecause you'd only include that if you need it22:12
zigomwhahaha: Yeah, and in vncproxy.pp as well.22:12
zigoOk, I'm happy that it feels logical... :P22:12
zigoI'm expecting that this last patch finally passes.22:13
zigomwhahaha: What about https://review.openstack.org/#/c/570250/ ? Can I push neutron-rpc-server in a separate .pp in a follow-up patch ?22:14
mwhahahazigo: no it's in the wrong spot22:14
mwhahahaif it always needs to be included it shouldn't be where it is22:15
mwhahahathe if $rpce_service_name should be outside of the if {} else {}  block22:15
mwhahahathe if else block it is currently in is just for neutron service22:16
zigomwhahaha: No, it must be in.22:16
mwhahahawhy22:16
zigomwhahaha: The choice is: either you run neutron-server, in such case, you have API + RPC, either you run neutron-api and neutron-rpc-server.22:16
zigoIn other words, neutron-api is a WSGI app that doesn't including listening in the bus like neutron-server do.22:17
mwhahahabut the if else block is about running the api portion22:17
mwhahahaso if the api gets split out but still runs under wsgi, your code wouldn't run rpc22:17
mwhahahaor are you saying if you run under wsgi you would be running both22:18
mwhahahain the single wsgi config22:18
*** salmankhan has joined #puppet-openstack22:18
zigoI think I'm very bad at explaining the situation, so let me try again.22:18
mwhahahayes it is not clear how this stuff pieces together22:18
zigoCurrently situation: neutron-server is running Eventlet, and does 2 jobs: API server, and RPC (aka rabbitmq) agent (ie: it sends network port connection signal to Nova, for example).22:19
zigoNow, Debian can't run neutron-server, because of lack of Python 3 + SSL, so I switched to neutron-api22:20
zigoBut neutron-api does *not* include the RPC agent functionality.22:20
zigoIt does only API.22:20
zigoFor that, you have the neutron-rpc-server that does this part of the job.22:20
openstackgerritThomas Goirand proposed openstack/puppet-nova master: Fix VNC console in Debian  https://review.openstack.org/56432822:21
mwhahahai think the confusion i have is the lack of the ability to run neutron-api-service via wsgi here22:22
mwhahahaso what i was thinking is that the existing configuration for neutron-server would be for neutron-api-server22:22
mwhahahaand then you'd always run neutron-rpc-server if available22:22
mwhahahahonestly this probably needs more review from neutron folks on the sanity of this setup22:23
zigoYou should *not* run /usr/bin/neutron-rpc-server if you're running /usr/bin/neutron-server.22:23
zigoI'm quite sure of what I'm saying, I experienced the problem.22:23
zigoIt's also documented.22:24
mwhahahaeh i guess we can roll with this one22:24
mwhahahaseems like a terrible service setup22:24
*** salmankhan has quit IRC22:24
zigoSee this patch https://review.openstack.org/#/c/555608/ fixing this bug https://bugs.launchpad.net/neutron/+bug/170838922:25
openstackLaunchpad bug 1708389 in neutron "wsgi and rpc-server has issues with dhcp when creating subnet" [Undecided,In progress] - Assigned to Nguyen Phuong An (annp)22:25
zigoThat's how I figured it out.22:26
zigoAh no...22:26
zigothat's the solution to another problem ! :)22:26
zigoEmilienM: mnaser: https://review.openstack.org/#/c/570250/ <--- Help? :)22:28
zigoThey are probably both at the summit at this time, I guess...22:28
mnaserzigo: sitting right next to each other too22:28
zigomnaser: Hi there !22:28
*** alee has quit IRC22:28
zigoSay hi to Emilien.22:28
zigomnaser: What are you listening to?22:29
EmilienMhi zigo :D22:29
zigo:P22:29
mnaserare you sure that neutron-rpc-server works properly?22:29
mnaserbecause i have had problems with it falling apart when i tried using it22:29
zigomnaser: With the patch I applied (ie: https://review.openstack.org/#/c/555608 ) then it works.22:30
zigomnaser: This afternoon, with it, I just had one single functional test failure on scenario001 and Debian.22:31
mnaserright but we can't really merge things based on something that hasn't been merged? :x22:31
zigoI'm still investigating it.22:31
mnaseryou'll find that the second you create a port22:31
mnaserit falls apart22:31
mnaserneutron-rpc-server that is22:31
zigomnaser: My patch doesn't force anyone to use it, it just happens to make it the default for Debian only.22:31
zigomnaser: It fails appart with binding:host_id not being recognized as a parameter indeed, and that's because it doesn't know about ml2 params in the config file, and that's what the patch I referenced fixes.22:33
mnaseri mean i have had issues but if it doesn't change default behaviour and doesn't hurt anyone22:33
mnaserdont think that's too much of an issuse22:33
zigoRight, default behavior is unchanged, and even in Debian, one could force old parameters (though my package don't support that anymore, but that's another story).22:34
zigoWhat I want to do later (but not on the same patch), is a bit of clean-up with variables names. For example the tag "neutron-server-eventlet" should be renamed as "neutron-api-service-up" or something ...22:36
zigoThen rename $service_name with $server_service_name and deprecate $service_name.22:37
zigoMaybe $server_package -> $server_package_name too.22:38
zigoThough I very much prefer doing this on a *later* patch, so I don't break what I wrote.22:39
zigo(and that clean-up phase is isolated from this patch)22:39
*** rpittau has quit IRC23:17
*** markvoelker has quit IRC23:19
*** markvoelker has joined #puppet-openstack23:20
zigoCan I do BGP ECMP with puppet-openstack ?23:20
zigoWe want to implement that for both the Ceph nodes and compute.23:21
*** rpittau has joined #puppet-openstack23:23
*** markvoelker has quit IRC23:24
*** jtomasek has joined #puppet-openstack23:31
*** alee has joined #puppet-openstack23:40

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