*** rpittau has quit IRC | 01:01 | |
*** markvoelker has joined #puppet-openstack | 01:08 | |
*** colonwq has joined #puppet-openstack | 01:24 | |
openstackgerrit | Merged openstack/puppet-cinder master: Remove deprecated parameters https://review.openstack.org/569640 | 01:40 |
---|---|---|
*** ilbot3 has quit IRC | 01:47 | |
openstackgerrit | zhongshengping proposed openstack/puppet-cloudkitty master: [DNM] test https://review.openstack.org/570288 | 01:54 |
*** ilbot3 has joined #puppet-openstack | 01:57 | |
*** skramaja has joined #puppet-openstack | 02:09 | |
openstackgerrit | zhongshengping proposed openstack/puppet-heat master: Remove deprecated rabbit params https://review.openstack.org/570295 | 02:11 |
openstackgerrit | zhongshengping proposed openstack/puppet-ironic master: Remove deprecated rabbit params https://review.openstack.org/570296 | 02:14 |
openstackgerrit | zhongshengping proposed openstack/puppet-magnum master: Remove deprecated rabbit params https://review.openstack.org/570298 | 02:19 |
openstackgerrit | Merged openstack/puppet-barbican master: Remove deprecated parameters https://review.openstack.org/569607 | 02:31 |
openstackgerrit | zhongshengping proposed openstack/puppet-magnum master: Remove deprecated rabbit params https://review.openstack.org/570298 | 02:36 |
openstackgerrit | zhongshengping proposed openstack/puppet-manila master: Remove deprecated rabbit params https://review.openstack.org/570301 | 02:37 |
openstackgerrit | zhongshengping proposed openstack/puppet-mistral master: Remove deprecated rabbit params https://review.openstack.org/570303 | 02:43 |
openstackgerrit | zhongshengping proposed openstack/puppet-heat master: Remove deprecated rabbit params https://review.openstack.org/570295 | 02:48 |
openstackgerrit | zhongshengping proposed openstack/puppet-ironic master: Remove deprecated rabbit params https://review.openstack.org/570296 | 02:49 |
openstackgerrit | zhongshengping proposed openstack/puppet-magnum master: Remove deprecated rabbit params https://review.openstack.org/570298 | 02:50 |
openstackgerrit | zhongshengping proposed openstack/puppet-manila master: Remove deprecated rabbit params https://review.openstack.org/570301 | 02:51 |
openstackgerrit | zhongshengping proposed openstack/puppet-mistral master: Remove deprecated rabbit params https://review.openstack.org/570303 | 02:52 |
openstackgerrit | zhongshengping proposed openstack/puppet-neutron master: Remove deprecated rabbit params https://review.openstack.org/570307 | 03:01 |
openstackgerrit | zhongshengping proposed openstack/puppet-octavia master: Remove deprecated rpc_backend param https://review.openstack.org/570308 | 03:04 |
openstackgerrit | zhongshengping proposed openstack/puppet-sahara master: Remove deprecated rabbit params https://review.openstack.org/570309 | 03:09 |
openstackgerrit | zhongshengping proposed openstack/puppet-tacker master: Remove deprecated rpc_backend param https://review.openstack.org/570310 | 03:11 |
openstackgerrit | zhongshengping proposed openstack/puppet-trove master: Remove deprecated rabbit params https://review.openstack.org/570311 | 03:15 |
openstackgerrit | zhongshengping proposed openstack/puppet-trove master: Remove deprecated heat_service_type param https://review.openstack.org/570312 | 03:19 |
openstackgerrit | zhongshengping proposed openstack/puppet-vitrage master: Remove deprecated rpc_backend param https://review.openstack.org/570313 | 03:20 |
openstackgerrit | zhongshengping proposed openstack/puppet-watcher master: Remove deprecated rpc_backend param https://review.openstack.org/570314 | 03:25 |
*** skramaja has quit IRC | 03:43 | |
*** skramaja has joined #puppet-openstack | 03:43 | |
*** rcernin has quit IRC | 03:58 | |
*** rcernin has joined #puppet-openstack | 04:17 | |
*** rcernin has quit IRC | 04:23 | |
*** rcernin has joined #puppet-openstack | 04:23 | |
*** openstackgerrit has quit IRC | 04:34 | |
*** ykarel|afk has joined #puppet-openstack | 04:35 | |
*** rcernin has quit IRC | 04:56 | |
*** openstackgerrit has joined #puppet-openstack | 05:19 | |
openstackgerrit | Merged openstack/puppet-openstack-integration master: Revert "Debian: Setup Cinder using cinder-api uwsgi" https://review.openstack.org/569420 | 05:19 |
openstackgerrit | Merged openstack/puppet-ec2api stable/queens: Configure metadata_proxy_shared_secret as secret https://review.openstack.org/570126 | 05:20 |
openstackgerrit | Merged openstack/puppet-ec2api stable/pike: Configure metadata_proxy_shared_secret as secret https://review.openstack.org/570127 | 05:20 |
openstackgerrit | Merged openstack/puppet-ironic master: Add pool_timeout option https://review.openstack.org/570135 | 05:25 |
openstackgerrit | Merged openstack/puppet-ironic master: Add support for db_max_retries param https://review.openstack.org/570138 | 05:25 |
openstackgerrit | Merged openstack/puppet-swift stable/queens: Configure www_authenticate_uri instead of auth_uri in authtoken https://review.openstack.org/569925 | 05:29 |
*** markvoelker_ has joined #puppet-openstack | 05:36 | |
*** markvoelker has quit IRC | 05:37 | |
*** jaganathan has quit IRC | 05:50 | |
*** jaganathan has joined #puppet-openstack | 05:50 | |
openstackgerrit | zhongshengping proposed openstack/puppet-trove master: Remove deprecated rabbit params https://review.openstack.org/570311 | 05:58 |
openstackgerrit | zhongshengping proposed openstack/puppet-mistral master: Remove deprecated rabbit params https://review.openstack.org/570303 | 06:01 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints https://review.openstack.org/569019 | 06:07 |
openstackgerrit | zhongshengping proposed openstack/puppet-heat master: Remove deprecated rabbit params https://review.openstack.org/570295 | 06:07 |
openstackgerrit | zhongshengping proposed openstack/puppet-ironic master: Remove deprecated rabbit params https://review.openstack.org/570296 | 06:14 |
openstackgerrit | zhongshengping proposed openstack/puppet-manila master: Remove deprecated rabbit params https://review.openstack.org/570301 | 06:15 |
*** olap has joined #puppet-openstack | 06:18 | |
*** olap has quit IRC | 06:18 | |
*** olap has joined #puppet-openstack | 06:18 | |
*** rpittau has joined #puppet-openstack | 06:37 | |
openstackgerrit | Merged openstack/puppet-neutron stable/pike: Fixes missing 'geneve' network type https://review.openstack.org/570129 | 06:38 |
*** rcernin has joined #puppet-openstack | 06:43 | |
*** alee has joined #puppet-openstack | 06:43 | |
*** rpittau has quit IRC | 06:51 | |
*** tesseract has joined #puppet-openstack | 07:23 | |
*** ykarel|afk is now known as ykarel|lunch | 07:30 | |
openstackgerrit | Thomas Goirand proposed openstack/puppet-neutron master: WIP: Support neutron-{api,rpc-server} services https://review.openstack.org/570250 | 07:36 |
*** jpena|off is now known as jpena | 07:37 | |
openstackgerrit | zhongshengping proposed openstack/puppet-trove master: Remove deprecated rabbit params https://review.openstack.org/570311 | 07:51 |
*** jistr is now known as jistr|iview | 07:57 | |
*** ykarel|lunch is now known as ykarel | 08:12 | |
openstackgerrit | Thomas Goirand proposed openstack/puppet-neutron master: WIP: Support neutron-{api,rpc-server} services https://review.openstack.org/570250 | 08:13 |
*** jaganathan has quit IRC | 08:22 | |
*** jaganathan has joined #puppet-openstack | 08:29 | |
*** rcernin has quit IRC | 08:36 | |
*** derekh has joined #puppet-openstack | 08:42 | |
*** amoralej|off is now known as amoralej | 08:55 | |
openstackgerrit | Thomas Goirand proposed openstack/puppet-neutron master: WIP: Support neutron-{api,rpc-server} services https://review.openstack.org/570250 | 09:06 |
*** jistr|iview is now known as jistr | 09:07 | |
*** Tengu has quit IRC | 09:07 | |
openstackgerrit | Thomas Goirand proposed openstack/puppet-neutron master: WIP: Support neutron-{api,rpc-server} services https://review.openstack.org/570250 | 09:08 |
*** morazi has joined #puppet-openstack | 09:14 | |
*** salmankhan has joined #puppet-openstack | 09:16 | |
openstackgerrit | Thomas Goirand proposed openstack/puppet-neutron master: Support neutron-{api,rpc-server} services https://review.openstack.org/570250 | 09:50 |
*** gfidente has joined #puppet-openstack | 10:15 | |
*** gfidente has quit IRC | 10:15 | |
*** gfidente has joined #puppet-openstack | 10:15 | |
*** dtantsur|afk is now known as dtantsur | 10:30 | |
*** olap has quit IRC | 10:38 | |
*** jpena is now known as jpena|lunch | 10:58 | |
*** amoralej is now known as amoralej|off | 11:07 | |
*** jpena|lunch is now known as jpena|off | 11:10 | |
*** vinsh has quit IRC | 11:33 | |
*** vinsh has joined #puppet-openstack | 11:33 | |
*** olap has joined #puppet-openstack | 11:42 | |
*** rasca has quit IRC | 11:44 | |
*** gfidente has quit IRC | 11:51 | |
*** gfidente has joined #puppet-openstack | 11:55 | |
*** jpena|off is now known as jpena | 11:58 | |
*** skramaja has quit IRC | 12:07 | |
*** vinsh has quit IRC | 12:21 | |
*** vinsh has joined #puppet-openstack | 12:22 | |
zigo | mnaser: mwhahaha: With these 2, scenario001 passes here: https://review.openstack.org/#/c/570250/ https://review.openstack.org/#/c/564328/ | 12:30 |
zigo | That last one needs unit tests fixups, of course. | 12:30 |
zigo | Can you look at the first one, ie: https://review.openstack.org/#/c/570250/ please ? | 12:30 |
zigo | It probably needs some variable updates, like neutron-server-eventlet, maybe, please do tell me... | 12:31 |
*** trown|outtypewww is now known as trown | 12:44 | |
*** rasca has joined #puppet-openstack | 12:49 | |
*** gfidente has quit IRC | 12:57 | |
*** jistr is now known as jistr|mtg | 13:14 | |
*** ykarel is now known as ykarel|away | 13:17 | |
*** gfidente has joined #puppet-openstack | 13:20 | |
*** amoralej|off is now known as amoralej | 13:35 | |
*** ykarel|away has quit IRC | 13:46 | |
*** tesseract-RH has joined #puppet-openstack | 13:54 | |
*** tesseract-RH has quit IRC | 13:55 | |
*** tesseract-RH has joined #puppet-openstack | 13:55 | |
*** tesseract has quit IRC | 13:56 | |
*** derekh has quit IRC | 13:59 | |
*** jistr|mtg is now known as jistr | 14:02 | |
*** olap has quit IRC | 14:13 | |
openstackgerrit | Thomas Goirand proposed openstack/puppet-neutron master: Support neutron-{api,rpc-server} services https://review.openstack.org/570250 | 14:13 |
*** rook has quit IRC | 14:15 | |
*** rook has joined #puppet-openstack | 14:17 | |
*** rook is now known as Guest11079 | 14:18 | |
*** derekh has joined #puppet-openstack | 14:21 | |
*** Guest11079 has quit IRC | 14:26 | |
*** rook_ has joined #puppet-openstack | 14:29 | |
*** tesseract-RH has quit IRC | 14:47 | |
*** jtomasek has joined #puppet-openstack | 14:54 | |
*** tesseract-RH has joined #puppet-openstack | 15:02 | |
*** tesseract-RH has quit IRC | 15:03 | |
*** tesseract has joined #puppet-openstack | 15:03 | |
*** markvoelker_ has quit IRC | 15:07 | |
*** markvoelker has joined #puppet-openstack | 15:07 | |
*** markvoelker has quit IRC | 15:11 | |
*** rook_ is now known as rook | 15:20 | |
*** jtomasek has quit IRC | 15:35 | |
*** ykarel|away has joined #puppet-openstack | 15:51 | |
*** ykarel|away has quit IRC | 16:00 | |
*** markvoelker has joined #puppet-openstack | 16:00 | |
*** markvoelker has quit IRC | 16:05 | |
*** markvoelker has joined #puppet-openstack | 16:12 | |
*** jaganathan has quit IRC | 16:19 | |
*** markvoelker has quit IRC | 16:21 | |
*** alee has quit IRC | 16:25 | |
*** trown is now known as trown|lunch | 16:29 | |
*** markvoelker has joined #puppet-openstack | 16:31 | |
*** markvoelker has quit IRC | 16:44 | |
*** alee has joined #puppet-openstack | 16:56 | |
*** derekh has quit IRC | 17:00 | |
*** amoralej is now known as amoralej|off | 17:03 | |
openstackgerrit | Thomas Goirand proposed openstack/puppet-neutron master: Support neutron-{api,rpc-server} services https://review.openstack.org/570250 | 17:09 |
*** dtantsur is now known as dtantsur|afk | 17:11 | |
*** tesseract has quit IRC | 17:25 | |
*** jpena is now known as jpena|off | 17:28 | |
*** gfidente has quit IRC | 17:39 | |
*** trown|lunch is now known as trown | 17:44 | |
*** alee has quit IRC | 17:46 | |
*** beagles is now known as beagles_brb | 17:46 | |
*** rpittau has joined #puppet-openstack | 17:53 | |
*** alee has joined #puppet-openstack | 17:53 | |
*** salmankhan has quit IRC | 18:14 | |
*** morazi has quit IRC | 18:31 | |
*** rajinir has joined #puppet-openstack | 18:37 | |
*** beagles_brb is now known as beagles | 18:40 | |
*** alee has quit IRC | 18:40 | |
*** rpittau has quit IRC | 18:41 | |
*** rpittau has joined #puppet-openstack | 18:46 | |
*** alee has joined #puppet-openstack | 18:52 | |
openstackgerrit | Thomas Goirand proposed openstack/puppet-nova master: Fix VNC console in Debian https://review.openstack.org/564328 | 19:06 |
*** markvoelker has joined #puppet-openstack | 19:31 | |
*** markvoelker has quit IRC | 19:33 | |
*** beagles is now known as Guest87851 | 19:34 | |
*** Hunner is now known as Guest3271 | 19:34 | |
*** tlbr is now known as Guest34116 | 19:34 | |
*** dmsimard is now known as Guest94448 | 19:34 | |
*** salmankhan has joined #puppet-openstack | 19:34 | |
*** alee has quit IRC | 19:35 | |
*** rpittau has quit IRC | 19:36 | |
*** Guest3271 is now known as Hunner | 19:48 | |
*** Guest94448 has quit IRC | 19:51 | |
*** salmankhan has quit IRC | 19:51 | |
*** dmsimard has joined #puppet-openstack | 19:52 | |
*** markvoelker has joined #puppet-openstack | 19:56 | |
*** salmankhan has joined #puppet-openstack | 19:56 | |
*** markvoelker has quit IRC | 20:04 | |
*** salmankhan has quit IRC | 20:11 | |
*** markvoelker has joined #puppet-openstack | 20:14 | |
*** trown is now known as trown|outtypewww | 20:27 | |
*** salmankhan has joined #puppet-openstack | 20:38 | |
*** Guest87851 is now known as beagles | 20:57 | |
*** rpittau has joined #puppet-openstack | 21:02 | |
*** shardy has quit IRC | 21:08 | |
*** shardy has joined #puppet-openstack | 21:09 | |
*** beagles is now known as beagles_biab | 21:14 | |
*** salmankhan has quit IRC | 21:15 | |
*** salmankhan has joined #puppet-openstack | 21:20 | |
*** rpittau has quit IRC | 21:36 | |
*** alee has joined #puppet-openstack | 21:38 | |
*** salmankhan has quit IRC | 21:40 | |
*** rpittau has joined #puppet-openstack | 21:42 | |
*** beagles_biab is now known as beagles | 21:45 | |
openstackgerrit | Thomas Goirand proposed openstack/puppet-nova master: Fix VNC console in Debian https://review.openstack.org/564328 | 21:46 |
*** salmankhan has joined #puppet-openstack | 21:50 | |
*** salmankhan has quit IRC | 21:56 | |
*** dpawlik has joined #puppet-openstack | 22:06 | |
*** rajinir has quit IRC | 22:07 | |
*** dpawlik_ has joined #puppet-openstack | 22:08 | |
*** dpawlik_ has quit IRC | 22:08 | |
zigo | mwhahaha: 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 |
mwhahaha | zigo: it wasn't in testing because ::nova::compute is not included in the test | 22:10 |
mwhahaha | which was my comment about passing the param in | 22:10 |
mwhahaha | the unit tests are for testing just the specific class, so you'd have to pass in the appropriate logic | 22:10 |
mwhahaha | the deployment tests would likely have the actual enable for both | 22:11 |
zigo | mwhahaha: But it doesn't make sense to include compute in a machine which is a proxy node. | 22:11 |
*** dpawlik has quit IRC | 22:11 | |
*** rcernin has joined #puppet-openstack | 22:11 | |
zigo | mwhahaha: It's also going to fail deployments that aren't all-in-one, because proxy nodes wont include ::nova::compute ... | 22:11 |
mwhahaha | right | 22:12 |
mwhahaha | zigo: so it's likely you'd leave true for nova::spicehtml5proxy | 22:12 |
mwhahaha | because you'd only include that if you need it | 22:12 |
zigo | mwhahaha: Yeah, and in vncproxy.pp as well. | 22:12 |
zigo | Ok, I'm happy that it feels logical... :P | 22:12 |
zigo | I'm expecting that this last patch finally passes. | 22:13 |
zigo | mwhahaha: 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 |
mwhahaha | zigo: no it's in the wrong spot | 22:14 |
mwhahaha | if it always needs to be included it shouldn't be where it is | 22:15 |
mwhahaha | the if $rpce_service_name should be outside of the if {} else {} block | 22:15 |
mwhahaha | the if else block it is currently in is just for neutron service | 22:16 |
zigo | mwhahaha: No, it must be in. | 22:16 |
mwhahaha | why | 22:16 |
zigo | mwhahaha: 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 |
zigo | In other words, neutron-api is a WSGI app that doesn't including listening in the bus like neutron-server do. | 22:17 |
mwhahaha | but the if else block is about running the api portion | 22:17 |
mwhahaha | so if the api gets split out but still runs under wsgi, your code wouldn't run rpc | 22:17 |
mwhahaha | or are you saying if you run under wsgi you would be running both | 22:18 |
mwhahaha | in the single wsgi config | 22:18 |
*** salmankhan has joined #puppet-openstack | 22:18 | |
zigo | I think I'm very bad at explaining the situation, so let me try again. | 22:18 |
mwhahaha | yes it is not clear how this stuff pieces together | 22:18 |
zigo | Currently 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 |
zigo | Now, Debian can't run neutron-server, because of lack of Python 3 + SSL, so I switched to neutron-api | 22:20 |
zigo | But neutron-api does *not* include the RPC agent functionality. | 22:20 |
zigo | It does only API. | 22:20 |
zigo | For that, you have the neutron-rpc-server that does this part of the job. | 22:20 |
openstackgerrit | Thomas Goirand proposed openstack/puppet-nova master: Fix VNC console in Debian https://review.openstack.org/564328 | 22:21 |
mwhahaha | i think the confusion i have is the lack of the ability to run neutron-api-service via wsgi here | 22:22 |
mwhahaha | so what i was thinking is that the existing configuration for neutron-server would be for neutron-api-server | 22:22 |
mwhahaha | and then you'd always run neutron-rpc-server if available | 22:22 |
mwhahaha | honestly this probably needs more review from neutron folks on the sanity of this setup | 22:23 |
zigo | You should *not* run /usr/bin/neutron-rpc-server if you're running /usr/bin/neutron-server. | 22:23 |
zigo | I'm quite sure of what I'm saying, I experienced the problem. | 22:23 |
zigo | It's also documented. | 22:24 |
mwhahaha | eh i guess we can roll with this one | 22:24 |
mwhahaha | seems like a terrible service setup | 22:24 |
*** salmankhan has quit IRC | 22:24 | |
zigo | See this patch https://review.openstack.org/#/c/555608/ fixing this bug https://bugs.launchpad.net/neutron/+bug/1708389 | 22:25 |
openstack | Launchpad 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 |
zigo | That's how I figured it out. | 22:26 |
zigo | Ah no... | 22:26 |
zigo | that's the solution to another problem ! :) | 22:26 |
zigo | EmilienM: mnaser: https://review.openstack.org/#/c/570250/ <--- Help? :) | 22:28 |
zigo | They are probably both at the summit at this time, I guess... | 22:28 |
mnaser | zigo: sitting right next to each other too | 22:28 |
zigo | mnaser: Hi there ! | 22:28 |
*** alee has quit IRC | 22:28 | |
zigo | Say hi to Emilien. | 22:28 |
zigo | mnaser: What are you listening to? | 22:29 |
EmilienM | hi zigo :D | 22:29 |
zigo | :P | 22:29 |
mnaser | are you sure that neutron-rpc-server works properly? | 22:29 |
mnaser | because i have had problems with it falling apart when i tried using it | 22:29 |
zigo | mnaser: With the patch I applied (ie: https://review.openstack.org/#/c/555608 ) then it works. | 22:30 |
zigo | mnaser: This afternoon, with it, I just had one single functional test failure on scenario001 and Debian. | 22:31 |
mnaser | right but we can't really merge things based on something that hasn't been merged? :x | 22:31 |
zigo | I'm still investigating it. | 22:31 |
mnaser | you'll find that the second you create a port | 22:31 |
mnaser | it falls apart | 22:31 |
mnaser | neutron-rpc-server that is | 22:31 |
zigo | mnaser: My patch doesn't force anyone to use it, it just happens to make it the default for Debian only. | 22:31 |
zigo | mnaser: 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 |
mnaser | i mean i have had issues but if it doesn't change default behaviour and doesn't hurt anyone | 22:33 |
mnaser | dont think that's too much of an issuse | 22:33 |
zigo | Right, 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 |
zigo | What 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 |
zigo | Then rename $service_name with $server_service_name and deprecate $service_name. | 22:37 |
zigo | Maybe $server_package -> $server_package_name too. | 22:38 |
zigo | Though 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 IRC | 23:17 | |
*** markvoelker has quit IRC | 23:19 | |
*** markvoelker has joined #puppet-openstack | 23:20 | |
zigo | Can I do BGP ECMP with puppet-openstack ? | 23:20 |
zigo | We want to implement that for both the Ceph nodes and compute. | 23:21 |
*** rpittau has joined #puppet-openstack | 23:23 | |
*** markvoelker has quit IRC | 23:24 | |
*** jtomasek has joined #puppet-openstack | 23:31 | |
*** alee has joined #puppet-openstack | 23:40 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!