*** vishwanathj has quit IRC | 00:00 | |
*** banix has joined #openstack-neutron | 00:01 | |
*** tflynn has quit IRC | 00:01 | |
*** tflynn_ has quit IRC | 00:02 | |
*** Longgeek has joined #openstack-neutron | 00:03 | |
*** annegentle has joined #openstack-neutron | 00:06 | |
*** Longgeek has quit IRC | 00:07 | |
openstackgerrit | Sukhdev Kapur proposed openstack/neutron: Arista L3 Service Plugin decomposition https://review.openstack.org/161984 | 00:07 |
---|---|---|
*** VijayB has quit IRC | 00:07 | |
*** leenheer has quit IRC | 00:08 | |
*** VijayB has joined #openstack-neutron | 00:12 | |
*** tflynn has joined #openstack-neutron | 00:12 | |
*** VijayB has quit IRC | 00:13 | |
*** VijayB_ has joined #openstack-neutron | 00:14 | |
*** bandarji has quit IRC | 00:15 | |
*** boris-42 has joined #openstack-neutron | 00:15 | |
shivharis | armax: hi | 00:15 |
armax | shivharis: hello | 00:15 |
*** xgerman has quit IRC | 00:16 | |
shivharis | armax: you seem to have a concern, please describe | 00:16 |
armax | shivharis: well, how clarify! :) | 00:16 |
armax | shivharis: I don’t understand what that plugin is for | 00:16 |
armax | shivharis: it’s a dummy class, no? | 00:16 |
shivharis | armax: yes this is dummy - the real import is done by the ml2 plugin which is now moved to networking_brocade | 00:17 |
openstackgerrit | Merged openstack/neutron: Add README and requirements.txt for VMware plugins https://review.openstack.org/163244 | 00:18 |
armax | shivharis: ok, so why do you keep this at all? | 00:18 |
shivharis | armax: the dummy is only to get folks off this in a clean manner | 00:18 |
armax | shivharis: besides ml2 != l3 | 00:18 |
armax | shivharis: nah, that’s not the right way at all | 00:18 |
armax | shivharis: why do you think this is clean? | 00:18 |
shivharis | armax: it could be nothing at all here as well | 00:18 |
armax | shivharis: it’s only masking an error in configuration | 00:19 |
armax | shivharis: it’s a time bomb, that’s what it is :) | 00:19 |
*** Longgeek has joined #openstack-neutron | 00:19 | |
shivharis | armax: what is ur suggestion | 00:20 |
armax | shivharis: have you considered dropping it altogether? | 00:20 |
shivharis | armax: i could remove everything | 00:20 |
armax | shivharis: is that going to be replaced by this one? | 00:20 |
armax | https://review.openstack.org/#/c/159056/? | 00:20 |
shivharis | armax: no that is different | 00:21 |
*** melwitt has quit IRC | 00:21 | |
armax | ok | 00:21 |
*** mwagner_lap has joined #openstack-neutron | 00:21 | |
shivharis | armax: the same stackforge has stuff for the vdx hardware as well, which is what will be used | 00:22 |
armax | so the BrocadeSVIPlugin is in stackforge, correct? | 00:22 |
shivharis | exactly | 00:23 |
shivharis | yes | 00:23 |
shivharis | I can remove the class in the shim completely if that is what u recommend | 00:23 |
armax | shivharis: well, I think that your actions need to be consistent | 00:23 |
armax | shivharis: If you do what you do here: https://review.openstack.org/#/c/152383/17/neutron/plugins/ml2/drivers/brocade/mechanism_brocade.py | 00:24 |
armax | shivharis: it makes sense to do the same for the l3 service plugin | 00:24 |
shivharis | i was being nice to the end customer - but consistency is a good argument you gave | 00:24 |
shivharis | armax: will that satisfy the concern, anything else u see? | 00:25 |
*** Longgeek has quit IRC | 00:26 | |
armax | shivharis: I’d argue the opposite, you’re nasty to the end customer who see things starting and nothing working :) | 00:26 |
shivharis | :-) hmmm.. :-) | 00:26 |
armax | shivharis: nothing else caught my eye, but I have been pretty congested | 00:26 |
*** banix has quit IRC | 00:26 | |
armax | shivharis: so my eye is a bit watery :) | 00:26 |
shivharis | armax: anything else u see that is of concern - or is that the only issue? (i dont want it the start flooding..) | 00:27 |
armax | what flooding? | 00:28 |
shivharis | ok, i will take care of this in the next 10 mins | 00:28 |
shivharis | watery.. never mind | 00:28 |
armax | :) | 00:28 |
armax | shivharis: ic | 00:28 |
shivharis | armax: thanks, it will be there shortly, no more class definition | 00:29 |
armax | shivharis: ok | 00:30 |
shivharis | armax: ok, ttyl | 00:30 |
armax | shivharis: thanks, ttyl | 00:30 |
*** shwetaap has joined #openstack-neutron | 00:30 | |
*** annegentle has quit IRC | 00:31 | |
*** shwetaap1 has joined #openstack-neutron | 00:33 | |
*** absubram has joined #openstack-neutron | 00:33 | |
*** natarajk1 has joined #openstack-neutron | 00:34 | |
*** shwetaap has quit IRC | 00:37 | |
*** annegentle has joined #openstack-neutron | 00:38 | |
*** vishy has quit IRC | 00:41 | |
*** vmtrooper has joined #openstack-neutron | 00:41 | |
*** Sayaji has quit IRC | 00:42 | |
*** vishy has joined #openstack-neutron | 00:43 | |
*** Rajeev_ has quit IRC | 00:44 | |
*** zzzeek has quit IRC | 00:44 | |
*** tflynn has quit IRC | 00:45 | |
*** vmtrooper has quit IRC | 00:46 | |
*** tsg_ has quit IRC | 00:47 | |
*** csoukup has quit IRC | 00:47 | |
*** annegentle has quit IRC | 00:48 | |
*** annegentle has joined #openstack-neutron | 00:49 | |
*** tflynn has joined #openstack-neutron | 00:50 | |
*** tiswanso has joined #openstack-neutron | 00:51 | |
*** tiswanso has quit IRC | 00:54 | |
*** tflynn has quit IRC | 00:54 | |
openstackgerrit | Henry Gessau proposed openstack/neutron: Add Mellanox decomposition progress to chart https://review.openstack.org/161651 | 00:55 |
*** tiswanso has joined #openstack-neutron | 00:55 | |
*** chlong has quit IRC | 00:55 | |
openstackgerrit | Shiv Haris proposed openstack/neutron: Brocade vendor code decomposition from neutron repo. https://review.openstack.org/152383 | 00:55 |
*** chlong has joined #openstack-neutron | 00:55 | |
*** dannywilson has joined #openstack-neutron | 00:57 | |
openstackgerrit | Salvatore Orlando proposed openstack/neutron: Support for reservations https://review.openstack.org/163659 | 00:58 |
*** patrickeast has joined #openstack-neutron | 00:58 | |
*** nati_uen_ has quit IRC | 00:58 | |
*** marun has joined #openstack-neutron | 01:00 | |
dannywilson | I am seeing some weird behavior in neutron. I am trying to associate a floatingip address to an instance. The command succeeds without warning but I can't use the ip address, but the logs show an error saying "TooManyExternalNetworks: More than one external network exists". Is this a known issue? | 01:00 |
*** _cjones_ has quit IRC | 01:02 | |
*** topshare has joined #openstack-neutron | 01:02 | |
*** igordcard_ has quit IRC | 01:03 | |
*** topshare has quit IRC | 01:03 | |
*** claired has quit IRC | 01:04 | |
*** rwsu is now known as rwsu-afk | 01:04 | |
*** sripriya has quit IRC | 01:05 | |
*** marun has quit IRC | 01:05 | |
*** shivharis has quit IRC | 01:06 | |
*** shivharis has joined #openstack-neutron | 01:06 | |
openstackgerrit | Joe Jiang proposed openstack/python-neutronclient: PEP8(version: 1.6.2) test fails in root directory of neutronclient. https://review.openstack.org/163663 | 01:06 |
*** topshare has joined #openstack-neutron | 01:07 | |
*** apuimedo has quit IRC | 01:08 | |
*** annegentle has quit IRC | 01:08 | |
*** Bhavya has quit IRC | 01:10 | |
*** VijayB_ has quit IRC | 01:12 | |
*** SumitNaiksatam has quit IRC | 01:13 | |
*** shivharis has quit IRC | 01:17 | |
*** shivharis has joined #openstack-neutron | 01:17 | |
*** VijayB_ has joined #openstack-neutron | 01:20 | |
*** VijayB_ has quit IRC | 01:22 | |
*** pal has joined #openstack-neutron | 01:26 | |
*** markvoelker has quit IRC | 01:27 | |
*** ivar-laz_ has joined #openstack-neutron | 01:27 | |
*** markvoelker has joined #openstack-neutron | 01:27 | |
*** ivar-lazzaro has quit IRC | 01:30 | |
*** ivar-laz_ has quit IRC | 01:32 | |
*** markvoelker has quit IRC | 01:32 | |
*** topshare_ has joined #openstack-neutron | 01:33 | |
*** nlahouti has quit IRC | 01:34 | |
*** pal_ has joined #openstack-neutron | 01:35 | |
*** topshare has quit IRC | 01:35 | |
*** stanzgy_ has joined #openstack-neutron | 01:36 | |
*** stanzgy has joined #openstack-neutron | 01:36 | |
*** pal has quit IRC | 01:36 | |
*** sbfox has joined #openstack-neutron | 01:38 | |
openstackgerrit | Timothy Swanson proposed openstack/neutron: Add MTU selection & advertisement settings to Neutron config https://review.openstack.org/153733 | 01:39 |
*** sbfox has quit IRC | 01:39 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Immediately schedule net to a DHCP agt by default https://review.openstack.org/163672 | 01:42 |
*** clds_ has quit IRC | 01:42 | |
*** popw has joined #openstack-neutron | 01:44 | |
*** wwriverrat has left #openstack-neutron | 01:44 | |
*** clds has joined #openstack-neutron | 01:44 | |
*** natarajk1 has quit IRC | 01:47 | |
*** sballe__ has quit IRC | 01:48 | |
*** nati_ueno has joined #openstack-neutron | 01:50 | |
*** Longgeek has joined #openstack-neutron | 01:52 | |
*** pal_ has quit IRC | 01:52 | |
*** VijayB has joined #openstack-neutron | 01:52 | |
*** dims_ has quit IRC | 01:53 | |
*** ajmiller has joined #openstack-neutron | 01:54 | |
*** VijayB_ has joined #openstack-neutron | 01:54 | |
*** VijayB has quit IRC | 01:54 | |
*** ajmiller_ has joined #openstack-neutron | 01:55 | |
*** ajmiller has quit IRC | 01:55 | |
*** Longgeek has quit IRC | 01:56 | |
*** salv-orlando has quit IRC | 01:58 | |
*** VijayB_ has quit IRC | 01:58 | |
*** yamahata has quit IRC | 01:59 | |
*** armax has quit IRC | 02:01 | |
*** shivharis has quit IRC | 02:01 | |
*** shivharis has joined #openstack-neutron | 02:02 | |
*** shivharis has quit IRC | 02:04 | |
openstackgerrit | yalei wang proposed openstack/neutron-fwaas: Implementation of Mcafee NGFW Driver https://review.openstack.org/152093 | 02:05 |
HenryG | mestery: still around? | 02:07 |
*** VijayB_ has joined #openstack-neutron | 02:07 | |
*** s3wong has quit IRC | 02:07 | |
*** armax has joined #openstack-neutron | 02:08 | |
*** banix has joined #openstack-neutron | 02:11 | |
*** nati_ueno has quit IRC | 02:11 | |
openstackgerrit | Merged openstack/neutron: linuxbridge UT: Do not create the same instance in each cases https://review.openstack.org/162825 | 02:13 |
*** jckasper has quit IRC | 02:13 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Test the shiny new WeightScheduler https://review.openstack.org/162231 | 02:14 |
*** ijw has quit IRC | 02:14 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Stop using RPC namespace to unbreak rolling upgrades https://review.openstack.org/163676 | 02:15 |
openstackgerrit | Assaf Muller proposed openstack/neutron: Stop using RPC namespace to unbreak rolling upgrades https://review.openstack.org/163676 | 02:16 |
*** patrickeast has quit IRC | 02:18 | |
*** marrusl has quit IRC | 02:18 | |
*** yfried|afk is now known as yfried|pto | 02:20 | |
*** ivar-lazzaro has joined #openstack-neutron | 02:23 | |
*** _cjones_ has joined #openstack-neutron | 02:24 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Replace keepalived notifier bash script with Python ip monitor https://review.openstack.org/125384 | 02:27 |
*** erkules_ has joined #openstack-neutron | 02:28 | |
*** VijayB_ has quit IRC | 02:30 | |
*** amuller has quit IRC | 02:30 | |
*** vmtrooper has joined #openstack-neutron | 02:30 | |
*** yfried|pto is now known as yfried|afk | 02:30 | |
*** erkules has quit IRC | 02:31 | |
*** marrusl has joined #openstack-neutron | 02:32 | |
*** vishwanathj has joined #openstack-neutron | 02:32 | |
openstackgerrit | Merged openstack/neutron: Add Mellanox decomposition progress to chart https://review.openstack.org/161651 | 02:33 |
*** vmtrooper has quit IRC | 02:35 | |
openstackgerrit | Phillip Toohill proposed openstack/python-neutronclient: Updates pool session persistence options https://review.openstack.org/163682 | 02:38 |
*** chandankumar has joined #openstack-neutron | 02:40 | |
*** leenheer has joined #openstack-neutron | 02:40 | |
*** achanda has quit IRC | 02:42 | |
*** ivar-lazzaro has quit IRC | 02:43 | |
*** david-lyle is now known as david-lyle_afk | 02:45 | |
*** yfried|afk is now known as yfried|pto | 02:50 | |
*** Marga_ has quit IRC | 02:51 | |
*** oomichi has joined #openstack-neutron | 02:52 | |
*** marun has joined #openstack-neutron | 02:53 | |
*** dims_ has joined #openstack-neutron | 02:54 | |
*** shwetaap1 has quit IRC | 02:55 | |
*** tflynn has joined #openstack-neutron | 02:57 | |
marun | kevinbenton: Sorry, had to take off. | 02:58 |
*** shwetaap has joined #openstack-neutron | 02:58 | |
*** salv-orlando has joined #openstack-neutron | 02:59 | |
marun | kevinbenton: Having the l2 agent worry about dhcp seems strange. | 02:59 |
marun | kevinbenton: But it saves a coordination step. | 02:59 |
*** dims_ has quit IRC | 02:59 | |
*** yfried|pto is now known as yfried|afk | 03:00 | |
marun | kevinbenton: We're conflating 'port up' with 'port ready for vm' at the server currently. If we treat those differently, I think it might make sense for the l2 agent to report on dhcp status since it can verify as close to the vm as possible that dhcp will be available. | 03:01 |
marun | kevinbenton: The other option might be adding yet another agent that can report on that same state, but I'm not sure the added complexity is justified. | 03:02 |
*** tflynn has quit IRC | 03:02 | |
*** zz_jgrimm is now known as jgrimm | 03:03 | |
*** tflynn has joined #openstack-neutron | 03:06 | |
*** popw1 has joined #openstack-neutron | 03:07 | |
openstackgerrit | Isaku Yamahata proposed openstack/neutron: iptables firewall: add framework for iptables firewall functional test https://review.openstack.org/160051 | 03:08 |
*** popw has quit IRC | 03:08 | |
openstackgerrit | Merged openstack/neutron: Arista L3 Service Plugin decomposition https://review.openstack.org/161984 | 03:09 |
*** nati_ueno has joined #openstack-neutron | 03:09 | |
*** dboik_ has quit IRC | 03:10 | |
*** dkehn has quit IRC | 03:10 | |
openstackgerrit | Merged openstack/neutron: Imported Translations from Transifex https://review.openstack.org/163302 | 03:10 |
*** salv-orlando has quit IRC | 03:10 | |
*** nati_ueno has quit IRC | 03:11 | |
*** yfried|afk is now known as yfried|pto | 03:12 | |
*** drjones has joined #openstack-neutron | 03:13 | |
*** dkehn has joined #openstack-neutron | 03:15 | |
*** _cjones_ has quit IRC | 03:16 | |
*** salv-orlando has joined #openstack-neutron | 03:19 | |
*** ivar-lazzaro has joined #openstack-neutron | 03:19 | |
*** SumitNaiksatam has joined #openstack-neutron | 03:22 | |
*** radez_g0n3 is now known as radez | 03:22 | |
*** salv-orlando has quit IRC | 03:23 | |
*** Networkn3rd has joined #openstack-neutron | 03:23 | |
*** salv-orlando has joined #openstack-neutron | 03:24 | |
*** tflynn has quit IRC | 03:26 | |
*** aranjan has joined #openstack-neutron | 03:26 | |
*** aranjan has quit IRC | 03:27 | |
*** aranjan has joined #openstack-neutron | 03:27 | |
*** aranjan has quit IRC | 03:28 | |
*** aranjan has joined #openstack-neutron | 03:29 | |
*** salv-orlando has quit IRC | 03:29 | |
openstackgerrit | Merged openstack/neutron: linuxbridge UT: Mock get_interface_by_ip https://review.openstack.org/162826 | 03:30 |
*** chandankumar has quit IRC | 03:30 | |
*** tflynn has joined #openstack-neutron | 03:31 | |
*** dannywilson has quit IRC | 03:31 | |
*** topshare_ has quit IRC | 03:33 | |
*** topshare has joined #openstack-neutron | 03:35 | |
*** nlahouti has joined #openstack-neutron | 03:36 | |
*** ijw has joined #openstack-neutron | 03:36 | |
*** Longgeek has joined #openstack-neutron | 03:37 | |
*** yfried|pto is now known as yfried|afk | 03:38 | |
*** zhhuabj has quit IRC | 03:39 | |
*** yfried|afk is now known as yfried|pto | 03:41 | |
*** topshare has quit IRC | 03:41 | |
*** ijw has quit IRC | 03:41 | |
*** banix has quit IRC | 03:41 | |
*** nlahouti has quit IRC | 03:42 | |
*** nlahouti has joined #openstack-neutron | 03:43 | |
*** topshare has joined #openstack-neutron | 03:43 | |
*** claudiub has quit IRC | 03:46 | |
openstackgerrit | Zengfa Gao proposed openstack/neutron: Use common agent.linux.utils.ensure_dir method https://review.openstack.org/154103 | 03:46 |
*** chandankumar has joined #openstack-neutron | 03:48 | |
*** topshare has quit IRC | 03:53 | |
*** Marga_ has joined #openstack-neutron | 03:53 | |
openstackgerrit | Sridar Kandaswamy proposed openstack/neutron-fwaas: FWaaS Insertion Model on Routers https://review.openstack.org/152697 | 03:54 |
*** tflynn has quit IRC | 03:54 | |
*** tflynn has joined #openstack-neutron | 03:54 | |
*** topshare has joined #openstack-neutron | 03:55 | |
*** shwetaap1 has joined #openstack-neutron | 03:55 | |
*** yfried|pto is now known as yfried|afk | 03:55 | |
*** yfried|afk is now known as yfried|pto | 03:56 | |
*** shwetaap has quit IRC | 03:57 | |
*** drjones has quit IRC | 03:59 | |
*** chandankumar has quit IRC | 04:00 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Schedule net to a DHCP agt on subnet create https://review.openstack.org/163672 | 04:00 |
*** SridharRamaswamy has joined #openstack-neutron | 04:00 | |
*** Longgeek has quit IRC | 04:03 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow driver https://review.openstack.org/153946 | 04:04 |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Separate ovs-ofctl using code as a driver https://review.openstack.org/160245 | 04:04 |
openstackgerrit | Kevin Benton proposed openstack/neutron: Schedule net to a DHCP agt on subnet create https://review.openstack.org/163672 | 04:06 |
kevinbenton | marun: still around? | 04:08 |
marun | kevinbenton: yes | 04:08 |
marun | kevinbenton: thoughts? | 04:09 |
kevinbenton | marun: so how does the L2 agent on the compute node know that DHCP is ready? | 04:09 |
kevinbenton | marun: would we have logic like, 'if not.port_owner.startswith(neutron): poll for dhcp port' | 04:10 |
kevinbenton | 'once dhcp port is ready, set port status to up' | 04:10 |
marun | kevinbenton: something like that, yeah | 04:10 |
kevinbenton | marun: that's just so crazy it might work! | 04:10 |
marun | kevinbenton: as far as how to tell the dhcp port is ready, the only guaranteed way is to actually invoke dhcping | 04:11 |
kevinbenton | marun: right | 04:11 |
kevinbenton | marun: i chatted with salvatore about that a bit after you left | 04:11 |
kevinbenton | marun: seeing that the DHCP port is up is only half of it | 04:11 |
marun | kevinbenton: right. | 04:11 |
kevinbenton | marun: it's sort of a sliding scale of complexity vs complete safety | 04:12 |
marun | kevinbenton: and if we actually verified that dhcp was working before booting a vm it should reduce the potential failure domain | 04:12 |
marun | kevinbenton: agreed | 04:12 |
kevinbenton | marun: can dhcping bind to an arbitrary tap port to send requests? | 04:12 |
marun | kevinbenton: not sure, tbh. If not, we could always implement a minimal client entirely in python. | 04:13 |
*** oomichi has quit IRC | 04:13 | |
*** pal has joined #openstack-neutron | 04:14 | |
*** chandankumar has joined #openstack-neutron | 04:14 | |
*** shwetaap has joined #openstack-neutron | 04:14 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: ofagent: Update after networking-ofagent release https://review.openstack.org/163273 | 04:14 |
kevinbenton | marun: what do you think about starting with that patch i just proposed? | 04:14 |
kevinbenton | marun: https://review.openstack.org/163672 | 04:14 |
*** Marga_ has quit IRC | 04:15 | |
marun | kevinbenton: Seems reasonable | 04:16 |
marun | kevinbenton: I think the proposed patch is an optimization if we verify dhcp is live before reporting to nova. | 04:16 |
*** Marga_ has joined #openstack-neutron | 04:16 | |
marun | kevinbenton: but I don't think it would hurt. | 04:16 |
kevinbenton | marun: yeah, i think this just buys us a little more room | 04:16 |
marun | kevinbenton: what did salvatore have to say about why the port wasn't previously created on subnet creation? | 04:17 |
*** shwetaap1 has quit IRC | 04:17 | |
*** jckasper has joined #openstack-neutron | 04:17 | |
kevinbenton | marun: he said he thought it was done to save resources | 04:18 |
marun | kevinbenton: which resources in particular? | 04:18 |
kevinbenton | marun: but he wasn't strongly sold on the argument so he was fine with a change | 04:18 |
kevinbenton | marun: on the dhcp agent | 04:18 |
kevinbenton | marun: spinning up an unnecessary dnsmasq process | 04:18 |
marun | kevinbenton: hmmm | 04:18 |
marun | kevinbenton: well, if it proves a problem the change can be reversed pretty easiliy | 04:19 |
marun | kevinbenton: and a dhcp liveness check will ensure that vm's don't fail due to the previous race | 04:19 |
*** vmtrooper has joined #openstack-neutron | 04:19 | |
kevinbenton | marun: so do you think i should go all out and try the dhcping full verification | 04:20 |
marun | kevinbenton: I have mixed feelings. | 04:21 |
marun | kevinbenton: It is clearly a potential failure mode | 04:21 |
marun | kevinbenton: I don't have a good sense of how likely it actually is | 04:21 |
*** radez is now known as radez_g0n3 | 04:21 | |
*** amotoki has joined #openstack-neutron | 04:22 | |
marun | kevinbenton: I would tend to work on the thing that is causing us the most pain. | 04:22 |
kevinbenton | marun: let's start with this and i'll see what kind of dhcp agent failures i still get when working on the scaling stuff | 04:22 |
marun | kevinbenton: Is your sense that this is a serious enough problem to address before other issues? | 04:22 |
marun | kevinbenton: fair enough | 04:22 |
marun | kevinbenton: if you're seeing failure due to this problem, then it would seem a good problem to solve | 04:22 |
kevinbenton | marun: it was pretty rare to encounter, and i think this will make it even harder to hit | 04:23 |
kevinbenton | marun: the patch i just proposed, that is | 04:23 |
*** vmtrooper has quit IRC | 04:24 | |
*** chandankumar has quit IRC | 04:24 | |
marun | kevinbenton: fair enough. | 04:24 |
*** topshare has quit IRC | 04:24 | |
marun | kevinbenton: well, I look forward to hearing how it goes | 04:24 |
*** topshare has joined #openstack-neutron | 04:24 | |
*** devlaps has quit IRC | 04:25 | |
*** gridinv_ has joined #openstack-neutron | 04:28 | |
*** topshare has quit IRC | 04:28 | |
*** ijw has joined #openstack-neutron | 04:31 | |
*** achanda has joined #openstack-neutron | 04:31 | |
*** zhhuabj has joined #openstack-neutron | 04:32 | |
*** yatin has joined #openstack-neutron | 04:32 | |
*** ivar-lazzaro has quit IRC | 04:33 | |
*** ivar-lazzaro has joined #openstack-neutron | 04:34 | |
*** SridharRamaswam1 has joined #openstack-neutron | 04:36 | |
*** sthillma has joined #openstack-neutron | 04:37 | |
*** ijw has quit IRC | 04:37 | |
*** SridharRamaswamy has quit IRC | 04:38 | |
*** ivar-lazzaro has quit IRC | 04:39 | |
*** killer_prince is now known as lazy_prince | 04:39 | |
*** Bhavya has joined #openstack-neutron | 04:40 | |
*** jckasper has quit IRC | 04:41 | |
openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver https://review.openstack.org/158560 | 04:41 |
openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: VPNaaS breakage by refactoring commit https://review.openstack.org/162840 | 04:41 |
openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention https://review.openstack.org/144391 | 04:41 |
*** marun has quit IRC | 04:47 | |
*** topshare has joined #openstack-neutron | 04:47 | |
*** salv-orlando has joined #openstack-neutron | 04:49 | |
*** lazy_prince has quit IRC | 04:49 | |
*** junbo has joined #openstack-neutron | 04:50 | |
*** marun has joined #openstack-neutron | 04:50 | |
*** yfried|pto is now known as yfried|afk | 04:50 | |
*** lazy_prince has joined #openstack-neutron | 04:51 | |
*** neeti has joined #openstack-neutron | 04:53 | |
*** lazy_prince is now known as killer_prince | 04:53 | |
*** trinaths has joined #openstack-neutron | 04:55 | |
*** killer_prince is now known as lazy_prince | 04:58 | |
*** junbo has left #openstack-neutron | 04:59 | |
*** salv-orlando has quit IRC | 05:00 | |
*** yamahata has joined #openstack-neutron | 05:04 | |
*** wwriverrat_ has joined #openstack-neutron | 05:04 | |
*** achanda has quit IRC | 05:04 | |
*** david-ly_ has joined #openstack-neutron | 05:04 | |
*** aleksandr_null_ has joined #openstack-neutron | 05:04 | |
*** tflynn has quit IRC | 05:04 | |
*** lazy_pri- has joined #openstack-neutron | 05:05 | |
*** kevinbenton_ has joined #openstack-neutron | 05:05 | |
*** blogan_ has joined #openstack-neutron | 05:06 | |
*** ijw has joined #openstack-neutron | 05:06 | |
*** mestery_ has joined #openstack-neutron | 05:07 | |
*** anilvenkata has joined #openstack-neutron | 05:07 | |
*** SumitNaiksatam has quit IRC | 05:07 | |
*** shadyabh1 has joined #openstack-neutron | 05:08 | |
*** rook-tmp_ has joined #openstack-neutron | 05:08 | |
*** mestery has quit IRC | 05:08 | |
*** terryw has joined #openstack-neutron | 05:09 | |
*** shaohe_robot1 has joined #openstack-neutron | 05:09 | |
*** jyuso has quit IRC | 05:09 | |
*** lazy_prince has quit IRC | 05:09 | |
*** lazy_pri- is now known as lazy_prince | 05:09 | |
*** david-lyle_afk has quit IRC | 05:09 | |
*** jyuso has joined #openstack-neutron | 05:09 | |
*** otherwiseguy has quit IRC | 05:09 | |
*** beagles has quit IRC | 05:09 | |
*** shaohe_robot has quit IRC | 05:09 | |
*** mikal_ has joined #openstack-neutron | 05:09 | |
*** aleksandr_null has quit IRC | 05:09 | |
*** kevinbenton has quit IRC | 05:09 | |
*** blogan has quit IRC | 05:09 | |
*** shadyabhi has quit IRC | 05:09 | |
*** frickler has quit IRC | 05:09 | |
*** mikal has quit IRC | 05:09 | |
*** frickler has joined #openstack-neutron | 05:09 | |
*** raginbajin has quit IRC | 05:09 | |
*** rook-tmp has quit IRC | 05:09 | |
*** b3nt_pin has joined #openstack-neutron | 05:10 | |
*** b3nt_pin is now known as Guest27074 | 05:10 | |
*** yfried|afk is now known as yfried|pto | 05:11 | |
*** topshare has quit IRC | 05:11 | |
*** ijw has quit IRC | 05:12 | |
*** raginbajin has joined #openstack-neutron | 05:13 | |
*** absubram has quit IRC | 05:15 | |
*** britthouser has joined #openstack-neutron | 05:16 | |
openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver https://review.openstack.org/158560 | 05:16 |
openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention https://review.openstack.org/144391 | 05:16 |
*** SridharRamaswam1 has quit IRC | 05:17 | |
*** britthouser has quit IRC | 05:19 | |
*** marun has quit IRC | 05:19 | |
*** chandankumar has joined #openstack-neutron | 05:20 | |
*** yfried|pto has quit IRC | 05:20 | |
*** topshare has joined #openstack-neutron | 05:21 | |
*** marun has joined #openstack-neutron | 05:22 | |
*** topshare_ has joined #openstack-neutron | 05:23 | |
*** shwetaap has quit IRC | 05:23 | |
*** britthouser has joined #openstack-neutron | 05:24 | |
trinaths | Hi all, Very Good Morning :) | 05:25 |
*** markvoelker has joined #openstack-neutron | 05:25 | |
*** gridinv_ has quit IRC | 05:25 | |
*** topshare has quit IRC | 05:26 | |
*** pradipta has joined #openstack-neutron | 05:26 | |
*** Murali has joined #openstack-neutron | 05:27 | |
*** achanda has joined #openstack-neutron | 05:28 | |
*** yuanying has joined #openstack-neutron | 05:30 | |
*** markvoelker has quit IRC | 05:30 | |
*** reed has quit IRC | 05:30 | |
*** leenheer has quit IRC | 05:33 | |
*** britthouser has quit IRC | 05:34 | |
*** tflynn has joined #openstack-neutron | 05:36 | |
*** chandankumar has quit IRC | 05:36 | |
*** enikanorov has joined #openstack-neutron | 05:40 | |
*** leenheer has joined #openstack-neutron | 05:40 | |
*** britthouser has joined #openstack-neutron | 05:40 | |
*** harlowja_ is now known as harlowja_away | 05:41 | |
*** chandankumar has joined #openstack-neutron | 05:41 | |
*** enikanorov__ has quit IRC | 05:41 | |
*** obondarev_ has joined #openstack-neutron | 05:42 | |
*** leenheer has quit IRC | 05:42 | |
*** obondarev has quit IRC | 05:42 | |
*** marun has quit IRC | 05:42 | |
*** yog_ has joined #openstack-neutron | 05:45 | |
*** Longgeek has joined #openstack-neutron | 05:45 | |
*** britthouser has quit IRC | 05:46 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Introduce Ryu based OpenFlow driver https://review.openstack.org/153946 | 05:46 |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Separate ovs-ofctl using code as a driver https://review.openstack.org/160245 | 05:46 |
*** yamamoto has joined #openstack-neutron | 05:47 | |
*** britthouser has joined #openstack-neutron | 05:50 | |
*** chandankumar has quit IRC | 05:52 | |
*** yog_ has quit IRC | 05:55 | |
*** wuhg has joined #openstack-neutron | 05:56 | |
*** adelinatuvenie has joined #openstack-neutron | 06:00 | |
*** anilvenkata has quit IRC | 06:04 | |
*** ijw has joined #openstack-neutron | 06:06 | |
*** yshafrir has joined #openstack-neutron | 06:07 | |
*** Murali_ has joined #openstack-neutron | 06:08 | |
*** vmtrooper has joined #openstack-neutron | 06:08 | |
*** Murali has quit IRC | 06:08 | |
*** Murali_ is now known as Murali | 06:08 | |
openstackgerrit | Hong Hui Xiao proposed openstack/neutron: Always run dnsmasq as root https://review.openstack.org/161567 | 06:11 |
*** yog_ has joined #openstack-neutron | 06:11 | |
*** ijw has quit IRC | 06:11 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex https://review.openstack.org/163708 | 06:12 |
*** rook-tmp_ has quit IRC | 06:12 | |
*** numan has joined #openstack-neutron | 06:12 | |
*** vmtrooper has quit IRC | 06:13 | |
*** aranjan_ has joined #openstack-neutron | 06:14 | |
openstackgerrit | Sridar Kandaswamy proposed openstack/neutron-fwaas: FWaaS Insertion Model on Routers https://review.openstack.org/152697 | 06:15 |
*** Longgeek has quit IRC | 06:18 | |
*** anilvenkata has joined #openstack-neutron | 06:21 | |
*** nlahouti has quit IRC | 06:23 | |
*** yog_ has quit IRC | 06:24 | |
*** afazekas is now known as __afazekas | 06:25 | |
*** markvoelker has joined #openstack-neutron | 06:26 | |
*** yamahata has quit IRC | 06:26 | |
*** leenheer has joined #openstack-neutron | 06:27 | |
*** markvoelker has quit IRC | 06:32 | |
*** dims_ has joined #openstack-neutron | 06:32 | |
*** leenheer has quit IRC | 06:32 | |
*** chandankumar has joined #openstack-neutron | 06:34 | |
*** anilvenkata has quit IRC | 06:35 | |
*** lazy_prince is now known as killer_prince | 06:35 | |
*** yog_ has joined #openstack-neutron | 06:36 | |
*** killer_prince is now known as lazy_prince | 06:37 | |
*** oda-g has quit IRC | 06:37 | |
*** dims_ has quit IRC | 06:38 | |
*** itzikb has joined #openstack-neutron | 06:42 | |
*** irenab has joined #openstack-neutron | 06:43 | |
*** setuid has quit IRC | 06:43 | |
*** setuid has joined #openstack-neutron | 06:46 | |
openstackgerrit | Ryan Tidwell proposed openstack/neutron: WIP Subnet allocation from a subnet pool https://review.openstack.org/157597 | 06:47 |
*** gildub has quit IRC | 06:49 | |
*** rook-tmp has joined #openstack-neutron | 06:50 | |
*** yfried|pto has joined #openstack-neutron | 06:51 | |
*** shaohe_robot1 is now known as shaohe_feng | 06:55 | |
*** yfried|pto has quit IRC | 06:55 | |
*** yfried|pto has joined #openstack-neutron | 06:56 | |
*** anilvenkata has joined #openstack-neutron | 06:57 | |
*** yfried|pto is now known as yfried|afk | 06:57 | |
*** yfried|afk is now known as yfried|pto | 07:00 | |
*** yfried|pto is now known as yfried | 07:00 | |
*** nmagnezi_ has joined #openstack-neutron | 07:01 | |
*** kawa2014 has joined #openstack-neutron | 07:02 | |
*** gongysh has joined #openstack-neutron | 07:05 | |
*** bobmel has joined #openstack-neutron | 07:06 | |
*** armax has quit IRC | 07:06 | |
*** ijw has joined #openstack-neutron | 07:06 | |
*** kawa2014 has quit IRC | 07:07 | |
*** Marga_ has quit IRC | 07:07 | |
*** kawa2014 has joined #openstack-neutron | 07:07 | |
openstackgerrit | Mamta Prabhu proposed openstack/neutron: IBM SDN-VE Plugin decomposition https://review.openstack.org/130471 | 07:09 |
*** bobmel_ has quit IRC | 07:09 | |
*** achanda has quit IRC | 07:11 | |
*** vthapar has joined #openstack-neutron | 07:11 | |
*** ijw has quit IRC | 07:11 | |
*** kawa2014 has quit IRC | 07:12 | |
*** kawa2014 has joined #openstack-neutron | 07:12 | |
*** lazy_prince has quit IRC | 07:15 | |
*** lazy_prince has joined #openstack-neutron | 07:16 | |
*** watanabe_isao has joined #openstack-neutron | 07:17 | |
*** SridharG1 has joined #openstack-neutron | 07:17 | |
*** _cjones_ has joined #openstack-neutron | 07:19 | |
*** anshul has joined #openstack-neutron | 07:20 | |
*** _cjones_ has quit IRC | 07:21 | |
*** achanda has joined #openstack-neutron | 07:24 | |
*** anshul has quit IRC | 07:26 | |
*** anshul has joined #openstack-neutron | 07:26 | |
*** salv-orlando has joined #openstack-neutron | 07:27 | |
*** markvoelker has joined #openstack-neutron | 07:28 | |
*** evgenyf has joined #openstack-neutron | 07:32 | |
*** lazy_prince has quit IRC | 07:33 | |
*** markvoelker has quit IRC | 07:33 | |
*** lazy_prince has joined #openstack-neutron | 07:34 | |
*** yfried is now known as yfried|afk | 07:34 | |
*** lazy_prince is now known as killer_prince | 07:35 | |
*** yfried|afk is now known as yfried | 07:35 | |
*** salv-orlando has quit IRC | 07:35 | |
*** nyechiel_ has joined #openstack-neutron | 07:36 | |
openstackgerrit | Itzik Brown proposed openstack/neutron: Adding a cleanup for 'fip-' and 'snat-' namespaces in netns_cleanup https://review.openstack.org/163403 | 07:36 |
*** bjornar_ has joined #openstack-neutron | 07:38 | |
*** salv-orlando has joined #openstack-neutron | 07:39 | |
*** killer_prince is now known as lazy_prince | 07:39 | |
*** salv-orl_ has joined #openstack-neutron | 07:42 | |
*** tflynn has quit IRC | 07:42 | |
*** sluo_wfh has joined #openstack-neutron | 07:42 | |
*** salv-orlando has quit IRC | 07:43 | |
*** achanda has quit IRC | 07:44 | |
*** anilvenkata has quit IRC | 07:46 | |
*** salv-orl_ has quit IRC | 07:46 | |
*** terryw has quit IRC | 07:48 | |
*** ChrisNBlum has joined #openstack-neutron | 07:49 | |
*** safchain has joined #openstack-neutron | 07:51 | |
*** lazy_prince has quit IRC | 07:52 | |
*** otherwiseguy has joined #openstack-neutron | 07:53 | |
*** lazy_prince has joined #openstack-neutron | 07:53 | |
*** sluo_wfh has quit IRC | 07:53 | |
*** lazy_prince is now known as killer_prince | 07:54 | |
*** jlanoux has joined #openstack-neutron | 07:54 | |
*** vmtrooper has joined #openstack-neutron | 07:57 | |
*** killer_prince is now known as lazy_prince | 07:57 | |
*** scheuran has joined #openstack-neutron | 07:59 | |
openstackgerrit | Romil Gupta proposed openstack/neutron: Ml2 Mechanism Driver for OVSvApp Solution https://review.openstack.org/160247 | 08:01 |
*** salv-orlando has joined #openstack-neutron | 08:01 | |
*** vmtrooper has quit IRC | 08:02 | |
*** Longgeek has joined #openstack-neutron | 08:03 | |
*** kickinz1|afk is now known as kickinz1 | 08:03 | |
*** jistr has joined #openstack-neutron | 08:03 | |
*** roeyc has joined #openstack-neutron | 08:04 | |
*** salv-orlando has quit IRC | 08:06 | |
*** ijw has joined #openstack-neutron | 08:06 | |
openstackgerrit | Hong Hui Xiao proposed openstack/neutron: Always run dnsmasq as root https://review.openstack.org/161567 | 08:06 |
*** afazekas_ has joined #openstack-neutron | 08:06 | |
*** kawa2014 has quit IRC | 08:08 | |
*** pask81 has joined #openstack-neutron | 08:08 | |
*** sudipto has joined #openstack-neutron | 08:09 | |
*** apuimedo has joined #openstack-neutron | 08:10 | |
*** gildub has joined #openstack-neutron | 08:11 | |
*** nyechiel__ has joined #openstack-neutron | 08:11 | |
*** matrohon has joined #openstack-neutron | 08:11 | |
*** nyechiel_ has quit IRC | 08:12 | |
*** ijw has quit IRC | 08:12 | |
*** Longgeek has quit IRC | 08:12 | |
*** yuanying has quit IRC | 08:13 | |
*** pask81 has quit IRC | 08:15 | |
*** pradipta has quit IRC | 08:16 | |
*** adelinatuvenie has quit IRC | 08:16 | |
*** sluo_wfh has joined #openstack-neutron | 08:18 | |
*** ronis has joined #openstack-neutron | 08:20 | |
*** yfried is now known as yfried|afk | 08:20 | |
*** pask81 has joined #openstack-neutron | 08:21 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Schedule net to a DHCP agt on subnet create https://review.openstack.org/163672 | 08:21 |
*** markvoelker has joined #openstack-neutron | 08:29 | |
*** watanabe_isao has quit IRC | 08:32 | |
*** aranjan_ has quit IRC | 08:34 | |
*** markvoelker has quit IRC | 08:34 | |
*** salv-orlando has joined #openstack-neutron | 08:34 | |
*** aranjan has joined #openstack-neutron | 08:35 | |
openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Return 404 when executing net-list-on-dhcp-agent with invalid agent_id https://review.openstack.org/159046 | 08:36 |
*** erkules_ is now known as erkules | 08:40 | |
*** erkules has quit IRC | 08:40 | |
*** erkules has joined #openstack-neutron | 08:40 | |
*** jschwarz has joined #openstack-neutron | 08:40 | |
*** yfried|afk is now known as yfried | 08:40 | |
*** kawa2014 has joined #openstack-neutron | 08:41 | |
openstackgerrit | enikanorov proposed openstack/neutron: Refactor retry mechanism used in some DB operations https://review.openstack.org/149261 | 08:46 |
openstackgerrit | enikanorov proposed openstack/neutron: Set default of api_workers to number of CPUs https://review.openstack.org/140493 | 08:46 |
*** gongysh has quit IRC | 08:49 | |
*** yfried is now known as yfried|afk | 08:50 | |
*** ygbo has joined #openstack-neutron | 08:51 | |
*** yfried|afk is now known as yfried | 08:54 | |
*** moshele has joined #openstack-neutron | 08:55 | |
openstackgerrit | pritesh proposed openstack/neutron: Adding VLAN Transparency support for ML2 along with REST API changes https://review.openstack.org/158420 | 09:00 |
*** neeti has quit IRC | 09:05 | |
openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Return 404 when executing net-list-on-dhcp-agent with invalid agent_id https://review.openstack.org/159046 | 09:05 |
*** neeti has joined #openstack-neutron | 09:05 | |
*** ijw has joined #openstack-neutron | 09:06 | |
*** ChrisNBlum has quit IRC | 09:06 | |
*** ijw has quit IRC | 09:11 | |
*** gongysh has joined #openstack-neutron | 09:13 | |
matrohon | rossella_s : hi | 09:13 |
*** nyechiel has joined #openstack-neutron | 09:13 | |
*** lazy_prince is now known as killer_prince | 09:15 | |
*** nyechiel__ has quit IRC | 09:15 | |
*** kerpan has joined #openstack-neutron | 09:15 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Schedule net to a DHCP agt on subnet create https://review.openstack.org/163672 | 09:17 |
*** yamamoto has quit IRC | 09:21 | |
*** Bhavya has quit IRC | 09:22 | |
*** topshare_ has quit IRC | 09:26 | |
*** stanzgy_ has quit IRC | 09:26 | |
*** stanzgy_ has joined #openstack-neutron | 09:26 | |
*** anilvenkata has joined #openstack-neutron | 09:28 | |
rossella_s | matrohon: hi | 09:29 |
*** yog_ has quit IRC | 09:30 | |
*** markvoelker has joined #openstack-neutron | 09:30 | |
*** ihrachyshka has joined #openstack-neutron | 09:32 | |
jschwarz | ihrachyshka, congratz, friend! :) | 09:33 |
openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Fix l3_agentschedulers_db for consistency of code https://review.openstack.org/163740 | 09:34 |
ihrachyshka | jschwarz, tnx | 09:34 |
* ihrachyshka is a bit scared of possessing +2 hammer. I need to learn to use it with due care. | 09:34 | |
*** Bhavya has joined #openstack-neutron | 09:35 | |
*** markvoelker has quit IRC | 09:35 | |
*** jpena has joined #openstack-neutron | 09:36 | |
*** aepifanov has joined #openstack-neutron | 09:36 | |
*** aranjan has quit IRC | 09:38 | |
*** yog_ has joined #openstack-neutron | 09:39 | |
openstackgerrit | Merged openstack/neutron: Stop using RPC namespace to unbreak rolling upgrades https://review.openstack.org/163676 | 09:39 |
*** aranjan has joined #openstack-neutron | 09:40 | |
*** Guest27074 is now known as b3nt_pin | 09:41 | |
*** kawa2014 has quit IRC | 09:43 | |
*** kawa2014 has joined #openstack-neutron | 09:44 | |
*** vmtrooper has joined #openstack-neutron | 09:45 | |
amotoki | ihrachyshka: congrats! | 09:47 |
ihrachyshka | amotoki, thanks, will do my best | 09:48 |
*** yfried is now known as yfried|afk | 09:49 | |
*** yfried|afk is now known as yfried | 09:50 | |
*** vmtrooper has quit IRC | 09:51 | |
*** yamamoto has joined #openstack-neutron | 09:51 | |
*** dims__ has joined #openstack-neutron | 09:54 | |
*** Longgeek has joined #openstack-neutron | 09:54 | |
*** moha_hunt has joined #openstack-neutron | 09:56 | |
trinaths | ihrachyshka: Congratulations.. | 10:02 |
*** ijw has joined #openstack-neutron | 10:06 | |
openstackgerrit | Akihiro Motoki proposed openstack/python-neutronclient: Copy functional tests from tempest cli https://review.openstack.org/162848 | 10:08 |
*** killer_prince is now known as lazy_prince | 10:09 | |
*** carlp has quit IRC | 10:10 | |
*** ijw has quit IRC | 10:11 | |
*** ajo has quit IRC | 10:12 | |
*** jp_at_hp has joined #openstack-neutron | 10:13 | |
*** Longgeek has quit IRC | 10:14 | |
*** jlanoux has quit IRC | 10:18 | |
*** salv-orl_ has joined #openstack-neutron | 10:18 | |
*** lazy_pri- has joined #openstack-neutron | 10:18 | |
*** imcsk8 is now known as imcsk8|zZz | 10:18 | |
*** ajo has joined #openstack-neutron | 10:18 | |
*** lazy_prince has quit IRC | 10:18 | |
*** lazy_pri- is now known as lazy_prince | 10:18 | |
*** stanzgy has quit IRC | 10:19 | |
*** stanzgy has joined #openstack-neutron | 10:20 | |
*** salv-orlando has quit IRC | 10:20 | |
*** gongysh has quit IRC | 10:21 | |
*** mlavalle has quit IRC | 10:21 | |
*** _cjones_ has joined #openstack-neutron | 10:22 | |
*** salv-orl_ has quit IRC | 10:22 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Migrate to oslo.log https://review.openstack.org/159638 | 10:23 |
*** lazy_pri- has joined #openstack-neutron | 10:24 | |
ihrachyshka | yamamoto, btw thanks for your comments, that are usually very detailed | 10:25 |
*** lazy_prince has quit IRC | 10:26 | |
*** lazy_pri- is now known as lazy_prince | 10:26 | |
*** _cjones_ has quit IRC | 10:26 | |
*** lazy_prince is now known as killer_prince | 10:27 | |
openstackgerrit | yalei wang proposed openstack/neutron: iptables firewall: add framework for iptables firewall functional test https://review.openstack.org/160051 | 10:28 |
openstackgerrit | yalei wang proposed openstack/neutron: Add portsecurity extension support https://review.openstack.org/126552 | 10:28 |
yamamoto | ihrachyshka: np | 10:29 |
openstackgerrit | Praveen Kumar SM proposed openstack/neutron: DHCP Service LoadBalancing Scheduler https://review.openstack.org/137017 | 10:31 |
*** markvoelker has joined #openstack-neutron | 10:31 | |
*** killer_prince is now known as lazy_prince | 10:33 | |
*** yamamoto has quit IRC | 10:35 | |
*** markvoelker has quit IRC | 10:36 | |
*** jistr has quit IRC | 10:36 | |
*** safchain has quit IRC | 10:43 | |
*** yog_ has quit IRC | 10:44 | |
kerpan | What's the best practice to implement an old abandon blueprint started by someone else? Contact the author? Start a new one? (The design is 'drafting' and there is some code waiting for review with no activity for more then 1 year) | 10:47 |
*** ParsectiX_ has joined #openstack-neutron | 10:47 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Add the rebinding chance in _bind_port_if_needed https://review.openstack.org/162260 | 10:48 |
openstackgerrit | Kevin Benton proposed openstack/neutron: Reload DHCP interface when port is updated https://review.openstack.org/163758 | 10:48 |
ParsectiX_ | Hello guys. I'm trying to create a router and attach it to a network | 10:48 |
ParsectiX_ | neutron router-create router_2 | 10:48 |
ParsectiX_ | neutron router-interface-add router_2 <UUID of the subnet> | 10:48 |
ParsectiX_ | but I'm getting Bad request | 10:49 |
ParsectiX_ | The subnet must have DHCP enabled ? | 10:49 |
*** stanzgy has quit IRC | 10:50 | |
*** pc_m has joined #openstack-neutron | 10:51 | |
*** stanzgy_ has quit IRC | 10:51 | |
*** skraynev has quit IRC | 10:52 | |
*** jlanoux has joined #openstack-neutron | 10:52 | |
*** skraynev has joined #openstack-neutron | 10:52 | |
*** skraynev has quit IRC | 10:53 | |
*** jistr has joined #openstack-neutron | 10:53 | |
*** skraynev has joined #openstack-neutron | 10:53 | |
*** yog_ has joined #openstack-neutron | 11:00 | |
*** skraynev_ has joined #openstack-neutron | 11:03 | |
*** ijw has joined #openstack-neutron | 11:06 | |
amotoki | ParsectiX_: there is no need to enable DHCP for a subnet. | 11:07 |
amotoki | ParsectiX_: what message did you get in BadRequest response? | 11:07 |
*** chandankumar has quit IRC | 11:09 | |
*** ijw has quit IRC | 11:11 | |
*** chandankumar has joined #openstack-neutron | 11:16 | |
*** aix has joined #openstack-neutron | 11:19 | |
*** safchain has joined #openstack-neutron | 11:19 | |
*** yog_ has quit IRC | 11:21 | |
*** bjornar_ has quit IRC | 11:23 | |
*** yuanying has joined #openstack-neutron | 11:23 | |
*** yuanying has quit IRC | 11:28 | |
*** yfried is now known as yfried|afk | 11:31 | |
*** markvoelker has joined #openstack-neutron | 11:33 | |
ParsectiX_ | amotoki: Bad Request (HTTP 400) (Request-ID: | 11:33 |
*** hichihara has quit IRC | 11:34 | |
*** vmtrooper has joined #openstack-neutron | 11:34 | |
amotoki | ParsectiX_: no detail message? Some versions of neutronclient behaves so. IIRC, in this case neutron -v router-interface-add would show the detail message. | 11:35 |
*** markvoelker has quit IRC | 11:37 | |
*** sudipto has quit IRC | 11:38 | |
*** nihilifer has quit IRC | 11:39 | |
ParsectiX_ | <amotoki> Thanks for the info | 11:39 |
*** nihilifer has joined #openstack-neutron | 11:39 | |
*** vmtrooper has quit IRC | 11:39 | |
*** yfried|afk is now known as yfried | 11:42 | |
*** scheuran_ has joined #openstack-neutron | 11:46 | |
*** kickinz1 is now known as kickinz1|afk | 11:48 | |
*** scheuran has quit IRC | 11:49 | |
*** kickinz1|afk is now known as kickinz1 | 11:54 | |
openstackgerrit | Henry Gessau proposed openstack/neutron: WIP: Drop support for SQL Schema Downgrades https://review.openstack.org/163787 | 11:55 |
*** gildub has quit IRC | 11:57 | |
*** aix has quit IRC | 11:59 | |
*** Longgeek has joined #openstack-neutron | 11:59 | |
*** tsg_ has joined #openstack-neutron | 11:59 | |
raorn | hi | 12:00 |
*** salv-orlando has joined #openstack-neutron | 12:00 | |
raorn | I am trying to backport two commits into stable/icehouse | 12:00 |
raorn | git review decided to squash them into one, but this is cherry-picks... | 12:01 |
HenryG | raorn: which commits? | 12:01 |
raorn | https://review.openstack.org/163783 https://review.openstack.org/163784 | 12:02 |
raorn | dhcp in v6-only networks | 12:02 |
HenryG | raorn: Have you tried the "Cherry Pick To" button in the gerrit UI? | 12:03 |
*** yfried is now known as yfried|afk | 12:03 | |
raorn | HenryG: there are two different comits and I tried git review -X | 12:03 |
raorn | on both master commits | 12:04 |
*** yfried|afk is now known as yfried | 12:04 | |
*** moha_hunt has quit IRC | 12:04 | |
*** mwagner_lap has quit IRC | 12:04 | |
HenryG | raorn: I have only done backports on already merged patches. Maybe check with ihrachyshka | 12:05 |
*** nyechiel_ has joined #openstack-neutron | 12:05 | |
raorn | both these patches are merged and even backported to juno | 12:06 |
*** ijw has joined #openstack-neutron | 12:06 | |
raorn | i don't think it's right to squash these two patches, since this is cherry-picks or two different commits. but decided to ask here. | 12:07 |
*** nyechiel has quit IRC | 12:07 | |
ihrachyshka | raorn, hm, do we even have ipv6 supported in icehouse? | 12:07 |
*** salv-orlando has quit IRC | 12:08 | |
*** baoli has joined #openstack-neutron | 12:08 | |
*** salv-orlando has joined #openstack-neutron | 12:08 | |
raorn | ihrachyshka: even in havana | 12:11 |
ihrachyshka | raorn, really? are you sure? afaik ipv6 attrs were disabled for icehouse | 12:11 |
raorn | only dhcpv6-statefull supported | 12:11 |
*** ijw has quit IRC | 12:12 | |
ihrachyshka | raorn, are you sure we speak about upstream releases and not some local changes in some distro? | 12:12 |
openstackgerrit | John Davidge proposed openstack/neutron: Implement Neutron IPv6 Prefix Delegation https://review.openstack.org/158697 | 12:13 |
raorn | openstack knows nothing about SLAAC | 12:13 |
ihrachyshka | raorn, "With the IceHouse release of OpenStack, only IPv6 Upstream Provider Network support with VLANs is feasible." | 12:14 |
ihrachyshka | that's from http://techs.enovance.com/7199/journey-of-ipv6-in-openstack | 12:14 |
*** Bhavya has quit IRC | 12:14 | |
raorn | ihrachyshka: I build openstack myself from gihtub repos, but our local changes are minimal. | 12:14 |
*** markvoelker has joined #openstack-neutron | 12:15 | |
ihrachyshka | so how do you create a ipv6 subnet while https://review.openstack.org/#/q/I69a1a571b5beb566641200e60b84f0716c1ec138,n,z is merged in icehouse? | 12:15 |
raorn | ihrachyshka: we only have provider networks and IPv6 (-only and dualstack) works fine since havana | 12:15 |
ihrachyshka | ah right, so provider networks. ok, looking. | 12:16 |
*** marun has joined #openstack-neutron | 12:17 | |
*** kawa2014 has quit IRC | 12:19 | |
*** Longgeek has quit IRC | 12:19 | |
*** salv-orlando has quit IRC | 12:20 | |
openstackgerrit | vikram.choudhary proposed openstack/neutron: Time Sync As Service https://review.openstack.org/163793 | 12:21 |
openstackgerrit | Moshe Levi proposed openstack/neutron: Move mlnx agent to be under ml2/drivers/mlnx https://review.openstack.org/162693 | 12:21 |
*** ferest has joined #openstack-neutron | 12:21 | |
*** skraynev has quit IRC | 12:23 | |
*** skraynev has joined #openstack-neutron | 12:23 | |
*** lazy_prince is now known as killer_prince | 12:24 | |
*** ferest has quit IRC | 12:25 | |
*** ajmiller_ has quit IRC | 12:26 | |
*** bjornar_ has joined #openstack-neutron | 12:28 | |
raorn | ihrachyshka: I guess "Hide ipv6 subnet API attributes" creates some problems with IPv6, but I'm not shure what is wrong. mayme my view is limited by provider networks... | 12:29 |
*** pal has quit IRC | 12:29 | |
*** tiswanso has quit IRC | 12:29 | |
ihrachyshka | raorn, sorry if my questions are dumb, but do you really run dhcp agent for provider networks? | 12:30 |
*** dims__ has quit IRC | 12:35 | |
*** dims_ has joined #openstack-neutron | 12:36 | |
ihrachyshka | raorn, ok, I think I get how your setup looks like. makes sense. | 12:37 |
raorn | ihrachyshka: yes. we are give VLAN and set of subnets (real addresses). "provider" does routing and RA, we only can manage flags in announces | 12:38 |
*** sudipto has joined #openstack-neutron | 12:39 | |
*** Longgeek has joined #openstack-neutron | 12:39 | |
ihrachyshka | raorn, so back to your original problem with patch squashing... looking at the patches, I don't see any problems. These two patches are properly linked in a list of dependencies. What's your problem with them? | 12:40 |
*** Marga_ has joined #openstack-neutron | 12:40 | |
*** moha_hunt has joined #openstack-neutron | 12:40 | |
*** txomon|fon has quit IRC | 12:41 | |
*** marun has quit IRC | 12:41 | |
raorn | ihrachyshka: no problem. this is the fist time I backport several commits at once, better ask ;-) | 12:42 |
*** txomon|fon has joined #openstack-neutron | 12:43 | |
*** amuller has joined #openstack-neutron | 12:44 | |
*** jpena is now known as jpena|lunch | 12:44 | |
*** elmiko has left #openstack-neutron | 12:45 | |
*** radez_g0n3 is now known as radez | 12:48 | |
*** txomon|fon has quit IRC | 12:49 | |
*** nyechiel_ has quit IRC | 12:50 | |
*** pavel_bondar has joined #openstack-neutron | 12:51 | |
*** mwagner_lap has joined #openstack-neutron | 12:52 | |
*** sudipto has quit IRC | 12:52 | |
*** amotoki has quit IRC | 12:57 | |
*** aix has joined #openstack-neutron | 12:58 | |
*** yfried_ has joined #openstack-neutron | 13:00 | |
*** yfried__ has joined #openstack-neutron | 13:03 | |
*** dboik has joined #openstack-neutron | 13:03 | |
*** yfried has quit IRC | 13:03 | |
*** jschwarz has quit IRC | 13:03 | |
*** yshafrir has quit IRC | 13:04 | |
*** setuid has quit IRC | 13:05 | |
*** setuid has joined #openstack-neutron | 13:05 | |
*** dboik_ has joined #openstack-neutron | 13:05 | |
*** yfried_ has quit IRC | 13:05 | |
*** ijw has joined #openstack-neutron | 13:06 | |
*** raildo has joined #openstack-neutron | 13:06 | |
*** yatin has quit IRC | 13:07 | |
*** mestery_ has quit IRC | 13:08 | |
*** dboik has quit IRC | 13:08 | |
*** nyechiel_ has joined #openstack-neutron | 13:09 | |
*** nyechiel__ has joined #openstack-neutron | 13:11 | |
*** dboik_ has quit IRC | 13:11 | |
*** ijw has quit IRC | 13:12 | |
*** yamamoto has joined #openstack-neutron | 13:13 | |
*** nyechiel_ has quit IRC | 13:14 | |
*** aveiga has joined #openstack-neutron | 13:17 | |
*** mestery has joined #openstack-neutron | 13:20 | |
*** shwetaap has joined #openstack-neutron | 13:23 | |
*** vmtrooper has joined #openstack-neutron | 13:23 | |
*** Longgeek has quit IRC | 13:24 | |
*** jckasper has joined #openstack-neutron | 13:25 | |
*** cference has joined #openstack-neutron | 13:27 | |
*** vmtrooper has quit IRC | 13:29 | |
*** dboik has joined #openstack-neutron | 13:32 | |
*** dboik has quit IRC | 13:32 | |
*** cference has quit IRC | 13:32 | |
*** dboik has joined #openstack-neutron | 13:32 | |
*** julien_ZTE has joined #openstack-neutron | 13:32 | |
*** tiswanso has joined #openstack-neutron | 13:34 | |
*** cference has joined #openstack-neutron | 13:35 | |
openstackgerrit | Merged openstack/neutron: Return 404 when executing net-list-on-dhcp-agent with invalid agent_id https://review.openstack.org/159046 | 13:36 |
openstackgerrit | John Davidge proposed openstack/neutron: Implement Neutron IPv6 Prefix Delegation https://review.openstack.org/158697 | 13:36 |
*** Murali has quit IRC | 13:36 | |
*** annegentle has joined #openstack-neutron | 13:37 | |
*** Longgeek has joined #openstack-neutron | 13:37 | |
*** kbringard has joined #openstack-neutron | 13:39 | |
*** shwetaap1 has joined #openstack-neutron | 13:42 | |
*** kawa2014 has joined #openstack-neutron | 13:44 | |
*** adelinatuvenie has joined #openstack-neutron | 13:45 | |
*** shwetaap has quit IRC | 13:45 | |
*** bjornar_ has quit IRC | 13:46 | |
*** Marga_ has quit IRC | 13:49 | |
*** anilvenkata has left #openstack-neutron | 13:50 | |
*** Marga_ has joined #openstack-neutron | 13:50 | |
*** zhhuabj has quit IRC | 13:51 | |
*** anshul has quit IRC | 13:54 | |
*** pal has joined #openstack-neutron | 13:57 | |
*** trinaths1 has joined #openstack-neutron | 13:57 | |
*** trinaths has quit IRC | 13:57 | |
*** neeti has quit IRC | 13:59 | |
*** baoli has quit IRC | 14:00 | |
*** sudipto has joined #openstack-neutron | 14:01 | |
*** jpena|lunch is now known as jpena | 14:01 | |
*** baoli_ has joined #openstack-neutron | 14:02 | |
*** apuimedo has quit IRC | 14:02 | |
*** baoli_ has quit IRC | 14:02 | |
openstackgerrit | Elena Ezhova proposed openstack/neutron: [WIP]: Add reset methods to WsgiWorker and RPCWorker https://review.openstack.org/161732 | 14:03 |
*** r-daneel has joined #openstack-neutron | 14:03 | |
*** apuimedo has joined #openstack-neutron | 14:03 | |
*** baoli_ has joined #openstack-neutron | 14:04 | |
*** ajmiller has joined #openstack-neutron | 14:06 | |
*** ijw has joined #openstack-neutron | 14:06 | |
openstackgerrit | ZongKai LI proposed openstack/neutron: Remove router binding with router-interface-delete https://review.openstack.org/163454 | 14:06 |
*** johnbelamaric has joined #openstack-neutron | 14:07 | |
*** ParsectiX_ has quit IRC | 14:07 | |
*** yamamoto has quit IRC | 14:07 | |
*** pal has quit IRC | 14:08 | |
*** irenab has quit IRC | 14:08 | |
*** Marga_ has quit IRC | 14:08 | |
*** chandankumar has quit IRC | 14:09 | |
*** ijw has quit IRC | 14:12 | |
*** amotoki has joined #openstack-neutron | 14:12 | |
*** carlp has joined #openstack-neutron | 14:14 | |
*** thomasem_ has quit IRC | 14:14 | |
openstackgerrit | enikanorov proposed openstack/neutron: Set default of api_workers to number of CPUs https://review.openstack.org/140493 | 14:14 |
*** yfried__ has quit IRC | 14:14 | |
*** thomasem has joined #openstack-neutron | 14:18 | |
*** aepifanov has quit IRC | 14:19 | |
*** yamamoto has joined #openstack-neutron | 14:19 | |
*** criptoverso has joined #openstack-neutron | 14:19 | |
*** adelinatuvenie has quit IRC | 14:19 | |
*** yfried__ has joined #openstack-neutron | 14:20 | |
*** yfried__ has quit IRC | 14:21 | |
*** topshare has joined #openstack-neutron | 14:23 | |
*** julien_ZTE has quit IRC | 14:24 | |
*** blues-man has joined #openstack-neutron | 14:25 | |
*** julien_ZTE has joined #openstack-neutron | 14:26 | |
*** dsundarraj has joined #openstack-neutron | 14:26 | |
*** yfried has joined #openstack-neutron | 14:27 | |
*** csoukup has joined #openstack-neutron | 14:31 | |
*** popw1 has quit IRC | 14:31 | |
*** jprovazn has joined #openstack-neutron | 14:31 | |
*** prad has joined #openstack-neutron | 14:34 | |
*** yfried is now known as yfried|afk | 14:35 | |
*** yfried|afk is now known as yfried | 14:35 | |
*** yfried has quit IRC | 14:35 | |
*** marun has joined #openstack-neutron | 14:35 | |
*** achanda has joined #openstack-neutron | 14:36 | |
*** adelinatuvenie has joined #openstack-neutron | 14:39 | |
*** seizadi has joined #openstack-neutron | 14:40 | |
openstackgerrit | Rob Pothier proposed openstack/neutron: Add MTU selection to ML2 https://review.openstack.org/156318 | 14:41 |
*** tflynn has joined #openstack-neutron | 14:41 | |
*** tflynn has quit IRC | 14:42 | |
*** apuimedo is now known as apuimedo|away | 14:42 | |
*** r-daneel has quit IRC | 14:42 | |
*** jschwarz has joined #openstack-neutron | 14:43 | |
*** tflynn has joined #openstack-neutron | 14:43 | |
*** achanda has quit IRC | 14:44 | |
*** r-daneel has joined #openstack-neutron | 14:45 | |
*** topshare has quit IRC | 14:47 | |
openstackgerrit | stephen-ma proposed openstack/neutron: Router is not unscheduled when the last VM is deleted https://review.openstack.org/159296 | 14:48 |
*** aveiga has quit IRC | 14:49 | |
*** carl_baldwin has joined #openstack-neutron | 14:49 | |
*** apuimedo|away has quit IRC | 14:49 | |
*** chandankumar has joined #openstack-neutron | 14:50 | |
*** tflynn has quit IRC | 14:50 | |
*** aveiga has joined #openstack-neutron | 14:50 | |
*** johnbelamaric has quit IRC | 14:51 | |
matrohon | rkukura : hi? | 14:52 |
*** numan has quit IRC | 14:53 | |
*** thedodd has joined #openstack-neutron | 14:53 | |
*** aveiga has quit IRC | 14:53 | |
openstackgerrit | Brian Bowen proposed openstack/neutron: Fix missing spaces in strings split across lines. https://review.openstack.org/163859 | 14:54 |
*** Longgeek has quit IRC | 14:55 | |
*** pal has joined #openstack-neutron | 14:55 | |
*** xgerman has joined #openstack-neutron | 14:57 | |
*** wwriverrat_ has left #openstack-neutron | 14:58 | |
*** dboik_ has joined #openstack-neutron | 14:58 | |
openstackgerrit | Paul Michali proposed openstack/neutron: Fix missing spaces in strings split across lines. https://review.openstack.org/163859 | 14:58 |
*** mlavalle has joined #openstack-neutron | 14:58 | |
*** yfried has joined #openstack-neutron | 14:59 | |
*** ajmiller has quit IRC | 14:59 | |
*** tiswanso has quit IRC | 14:59 | |
openstackgerrit | Paul Michali proposed openstack/neutron: Remove dead code https://review.openstack.org/163586 | 15:00 |
*** baoli_ has quit IRC | 15:00 | |
*** dboik has quit IRC | 15:00 | |
*** claudiub has joined #openstack-neutron | 15:01 | |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Add a netns-cleanup functional test https://review.openstack.org/163860 | 15:01 |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Fix netns-cleanup broken by ProcessMonitor refactor https://review.openstack.org/163401 | 15:02 |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Add a netns-cleanup functional test https://review.openstack.org/163860 | 15:02 |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Add a netns-cleanup functional test https://review.openstack.org/163860 | 15:04 |
ajo | ... sorry for the noise... | 15:04 |
trinaths1 | ajo: :) okay | 15:05 |
ajo | trinaths1 :) | 15:06 |
*** ijw has joined #openstack-neutron | 15:06 | |
ajo | ihrachyshka, amuller, marun ^ | 15:06 |
ajo | let me know how it looks | 15:06 |
*** tiswanso has joined #openstack-neutron | 15:07 | |
*** baoli has joined #openstack-neutron | 15:08 | |
*** moha_hunt has quit IRC | 15:09 | |
*** ajmiller has joined #openstack-neutron | 15:11 | |
*** ijw has quit IRC | 15:11 | |
openstackgerrit | Romil Gupta proposed openstack/neutron: Added Flag for default External Networks type in ML2 https://review.openstack.org/149546 | 15:12 |
*** chandankumar has quit IRC | 15:12 | |
mestery | marun HenryG: The experimental job passed for the functional job for the client, this is ready to merge now: https://review.openstack.org/#/c/162848/ | 15:12 |
*** vmtrooper has joined #openstack-neutron | 15:12 | |
mestery | marun HenryG: It's a straight move of hte existing tests into the client repo | 15:12 |
openstackgerrit | Adelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V https://review.openstack.org/160596 | 15:14 |
*** zhhuabj has joined #openstack-neutron | 15:14 | |
*** trinaths1 has quit IRC | 15:15 | |
*** yamahata has joined #openstack-neutron | 15:17 | |
*** prad has quit IRC | 15:17 | |
*** vmtrooper has quit IRC | 15:17 | |
rkukura | matrohon: hi | 15:18 |
matrohon | rkukura : hi | 15:18 |
*** pritesh has joined #openstack-neutron | 15:18 | |
matrohon | rkukura : I left a comment on DVR tech debt bug | 15:19 |
*** adelinatuvenie has quit IRC | 15:19 | |
*** zhhuabj has quit IRC | 15:19 | |
rkukura | matrohon: OK, I’ll look | 15:19 |
matrohon | rkukura : I just wanted to know if there is still a chance that a patch make it for k3 | 15:19 |
*** SridharG1 has left #openstack-neutron | 15:19 | |
openstackgerrit | John Davidge proposed openstack/neutron: Implement Neutron IPv6 Prefix Delegation https://review.openstack.org/158697 | 15:20 |
rkukura | matrohon: I’m hoping so. I made some progress last week, but haven’t been able to get back to it this week. | 15:20 |
rkukura | matrohon: But plan to work on it today | 15:20 |
matrohon | rkukura : ok, cool! | 15:20 |
*** dannywilson has joined #openstack-neutron | 15:20 | |
*** dsundarraj has quit IRC | 15:21 | |
matrohon | rkukura : this patch add tech debt to : https://review.openstack.org/#/c/141114/ | 15:21 |
matrohon | rkukura : but we move forward to have a chance to have a patch for k | 15:22 |
ihrachyshka | mestery, hey | 15:22 |
*** dannywil_ has joined #openstack-neutron | 15:22 | |
*** dannywilson has quit IRC | 15:23 | |
matrohon | rkukura : BTW rossella_s proposed to rebind the port when get_device_details is received : https://review.openstack.org/#/c/163178/ | 15:23 |
*** david-ly_ is now known as david-lyle | 15:23 | |
rkukura | matrohon: I thought it already did that if not already bound. | 15:23 |
matrohon | rkukura : it address the case when the port is already bound, typically in Live-migration | 15:24 |
*** Murali has joined #openstack-neutron | 15:24 | |
*** chandankumar has joined #openstack-neutron | 15:25 | |
matrohon | rkukura : I think it's a good idea, this would avoid a nova patch | 15:26 |
rkukura | matrohon: The get_device_details calling plugin.get_bound_port_context should result in rebinding if the host has changed. | 15:26 |
*** aranjan has quit IRC | 15:26 | |
rkukura | matrohon: Nova updates the port’s binding:host_id on migration, right? | 15:26 |
*** ChuckC has joined #openstack-neutron | 15:26 | |
matrohon | rkukura : hummm... this is not the case in our experiments... | 15:26 |
rossella_s | rkukura: it doesn't bind it if the host change because the vif type is not VIF_TYPE_UNBOUND | 15:27 |
matrohon | rkukura : yes, at least during Juno | 15:27 |
rossella_s | rkukura but if the port update from nova is processed after the get_device_detail we have a problem | 15:27 |
*** rpothier has joined #openstack-neutron | 15:28 | |
rkukura | matrohon, rossella_s: Anything that changes the port’s host_id value should change invalidate the binding. | 15:28 |
rossella_s | rkukura: I totally agree, that's why I proposed that the port binding should be updated if the host change, regardless of the port vif type | 15:29 |
matrohon | rkukura, rossella_s : so you have to think about how get_device_details is legitimate to modify the host | 15:29 |
rkukura | matrohon, rossella_s: BTW, my patch will be changing the way binding states (unbound vs. binding failed vs. bound) are represented in the DB, so if there is a case not covered, please let me know | 15:29 |
rkukura | rossella_s: Am I wrong that the only way the host_id can change is through a port_update? | 15:29 |
rossella_s | rkukura I didn't have the chance to review your patch, will look for it | 15:30 |
rkukura | Just receiving an RPC from a different host should not change the host_id, IMHO. | 15:30 |
rkukura | rossella_s: Its not posted yet | 15:30 |
matrohon | rkukura : so you consider get_device_detail RPC msg should not be allowed to change the binding? | 15:31 |
rkukura | matrohon: It can change the binding - i.e. cause a new binding attempt if the previous attempt failed. | 15:31 |
rossella_s | rkukura: right now if a port comes up on an l2 agent, it sends a get_device_details and the port is bound to that agent | 15:31 |
rkukura | matrohon: But nova should be doing a port_update to change the host_id before the port gets created on the new host during a migration, right? | 15:32 |
rkukura | rossella_s: I really don’t think get_device_details should be changing the host_id. The REST call should be the only way to do that. | 15:32 |
matrohon | rkukura : we can't be sure of that | 15:32 |
*** rwsu-afk is now known as rwsu | 15:33 | |
*** chandankumar has quit IRC | 15:33 | |
rossella_s | rkukura: there's no order guarantee, right? if the port update is processed after the messages sent by the l2 agent, we can't set the port state correctly | 15:33 |
matrohon | rossella_s : +1 | 15:33 |
*** shwetaap1 has quit IRC | 15:34 | |
rkukura | matrohon: Are you saying nova might not update the host_id first, or just that the neutron process handling the RPC might have stale state? | 15:34 |
matrohon | rkukura : last time I check yes... | 15:34 |
*** reed has joined #openstack-neutron | 15:34 | |
rossella_s | rkukura: I'd like to understand your point. If the get_device_details is allowed to set the host binding in case of a "new" port, why shouldn't it update the binding in case of a migration? | 15:34 |
rkukura | matrohon: The port update is synchronous, so nova shouldn’t re-plug the VIF until after the update REST call has returned. | 15:35 |
matrohon | rkukura : that's why I reported this bug, but it was while ago : https://bugs.launchpad.net/neutron/+bug/1274160 | 15:35 |
openstack | Launchpad bug 1274160 in neutron "port_update from nova could potentially occurs after get_device_details/update_device_up" [Medium,In progress] - Assigned to Mathieu Rohon (mathieu-rohon) | 15:35 |
*** setuid has quit IRC | 15:35 | |
*** prad has joined #openstack-neutron | 15:35 | |
*** setuid has joined #openstack-neutron | 15:36 | |
rkukura | matrohon: I hadn’t seen that. Is this really a nova bug? | 15:36 |
*** shwetaap has joined #openstack-neutron | 15:36 | |
matrohon | rkukura : I first though about a nova bug but sdague removed it because this bug hasn't been seen, just a potential race to confirm | 15:37 |
*** itzikb has quit IRC | 15:37 | |
rkukura | matrohon, rossella_s: I’ve considered the REST calls to be the only way to change normal “input” attributes like binding:host_id. | 15:38 |
*** Murali has quit IRC | 15:38 | |
rkukura | matrohon, rossella_s: I’m concerned if we allow the get_device_details RPC to update the host_id, we could potentially have race conditions between L2 agents on the two hosts during a migtration. The old host could possible to a get_device_details RPC for the port at about the same time as the new host, and either could win. | 15:40 |
*** _cjones_ has joined #openstack-neutron | 15:40 | |
openstackgerrit | ZongKai LI proposed openstack/neutron: Remove router binding with router-interface-delete https://review.openstack.org/163454 | 15:41 |
rkukura | matrohon, rossella_s: I wish we could discuss these kinds of issues at the ML2 IRC meetings. | 15:41 |
openstackgerrit | Sandhya Dasu proposed openstack/neutron: Cisco UCS Manager ML2 Mechanism Driver https://review.openstack.org/155436 | 15:42 |
*** seizadi has quit IRC | 15:42 | |
matrohon | rkukura, rossella_s : yep, but unfortunately I'm available for every ML2 meetings | 15:42 |
mlavalle | pc_m: do you want to talk now or would you rather wait for carl_baldwin to be back and we all talk together? | 15:42 |
pc_m | doesn't matter to me. I'm adding some items in the bug right now. | 15:42 |
*** Swami_ has joined #openstack-neutron | 15:42 | |
matrohon | rossella_s : do you think the race rkukura mentioned could happen | 15:42 |
*** aranjan has joined #openstack-neutron | 15:43 | |
rossella_s | matrohon, rkukura I don't see how that race could happen. get_device_details is sent by the l2 agent when a new device is discovered. So the old l2 agent won't send it | 15:43 |
*** zilberstein has joined #openstack-neutron | 15:43 | |
zilberstein | Good evening #openstack-neutron | 15:44 |
HenryG | marun: ping | 15:44 |
rossella_s | matrohon, rkukura the only possibility is a port going up and then down on the old host before being migrated...but not sure how likely it is | 15:44 |
zilberstein | what interface_driver I should use in dhcp_agent.ini if I use ofagent,l2population with openvswitch ? | 15:44 |
rkukura | rossella_s, matrohon: Couldn’t the old host’s agent just be coming up at the same time as the migration, or maybe the VM was migrated immediately after it was created? | 15:44 |
rkukura | rossella_s, matrohon: Seems a migration might be triggered by a compute host being down, but it could recover during the migration. | 15:45 |
*** tflynn has joined #openstack-neutron | 15:46 | |
*** armax has joined #openstack-neutron | 15:46 | |
rkukura | rossella_s, matrohon: I think its safer for it to be nova’s responsibility to make sure the port_update is called when it decides to migrate the VM. | 15:46 |
*** ajo has quit IRC | 15:47 | |
*** aranjan has quit IRC | 15:47 | |
*** sudipto has quit IRC | 15:48 | |
matrohon | rkukura : the scenario where the migration is triggered by a host going down makes sense | 15:48 |
*** ajo has joined #openstack-neutron | 15:48 | |
rkukura | rossella_s, matrohon: I do think get_device_details should check the host_id, and return an error if the agent making the call isn’t the current host. I think I saw a recent patch for this. | 15:49 |
rossella_s | rkukura: you read my mind. That's what I was going to suggest | 15:49 |
rossella_s | rkukura: I am working on that, maybe you've seen one of my patches | 15:49 |
matrohon | rossella_s : do you have the link to this patch? | 15:50 |
rkukura | rossella_s: I do try to look at them ;) | 15:50 |
rossella_s | rossella_s, matrohon do we agree that the best would be to return an error if the host_id doesn't match so that we give the time to the neutron server to process the port update? | 15:50 |
*** tflynn has quit IRC | 15:50 | |
HenryG | ajo: ping | 15:51 |
ajo | hi HenryG | 15:51 |
ajo | how're you doing? | 15:51 |
HenryG | ajo: good thanks, how about you? | 15:51 |
*** dsundarraj has joined #openstack-neutron | 15:51 | |
ajo | HenryG, very well, playing a bit with IPv6 ;) all my home exposed via v6... I feel naked ;D | 15:52 |
ajo | I will need to firewall a bit, I guess ;D | 15:52 |
*** wwriverrat has joined #openstack-neutron | 15:52 | |
mlavalle | pc_m: Plese read the comment I made here: https://review.openstack.org/#/c/147744/18/neutron/agent/l3/dvr_snat_ns.py. It is a little long. My conclusion, is that snat namepsaces were sometimes created implicitely prior to the refactoring and now we have made that evident after the refactoring | 15:52 |
*** ihrachyshka has quit IRC | 15:52 | |
matrohon | rkukura, rossella_s : I agree, this won't fix the potential race but I think it will fix this error : https://bugs.launchpad.net/neutron/+bug/1274160/comments/7 | 15:53 |
openstack | Launchpad bug 1274160 in neutron "port_update from nova could potentially occurs after get_device_details/update_device_up" [Medium,In progress] - Assigned to Mathieu Rohon (mathieu-rohon) | 15:53 |
mlavalle | pc_m: I need to relocate to the office, but I will ping you later in a bit after you have read that comment. is that ok? | 15:53 |
HenryG | ajo: I am looking at https://review.openstack.org/152512 | 15:53 |
pc_m | ok | 15:53 |
ajo | HenryG, opening | 15:53 |
*** wwriverrat has left #openstack-neutron | 15:53 | |
pc_m | mlavalle: will loook at it | 15:53 |
matrohon | rkukura, rossella_s : If we don't modify the port status of course | 15:53 |
ajo | HenryG, how does it look did you find anything strange? :) | 15:53 |
HenryG | ajo: The fix is good. No problems there. | 15:53 |
rossella_s | matrohon: there are several patches for the l2 agent, but I don't exactly check the host id, not sure if Bob meant the patch by somebody else... | 15:53 |
mlavalle | pc_m: it is not excatly the problem you are dealing with, but I think it is all related | 15:53 |
*** vivek-ebay has joined #openstack-neutron | 15:54 | |
ajo | HenryG: thanks for looking at it :) | 15:54 |
rkukura | rossella_s: The patch I’m thinking of (or imagining) change get_device_details in neutron-server to check the host_id. | 15:54 |
rossella_s | matrohon: if you are interested just check the patch owned by me...it's still WIP but I am trying to modify the way the ovs agent handles arrors | 15:54 |
HenryG | ajo: I ended up with many nits on the tests. Nothing I want to block on, but I wonder if you want to have a chance to read them before I approve? | 15:54 |
ajo | sure HenryG , makes sense | 15:55 |
rossella_s | rkukura: I haven't seen it then...if you find it please share :) | 15:55 |
ajo | may be I can sort them out tomorrow morning | 15:55 |
rkukura | rossella_s, matrohon: Do we know for sure that nova may re-plug the port before the update_port has returned? | 15:55 |
matrohon | rkukura : may be you're talking about my current proposal : https://review.openstack.org/#/c/163178/4/neutron/plugins/ml2/rpc.py | 15:55 |
matrohon | rkukura : I'll recheck | 15:55 |
*** mlavalle has quit IRC | 15:55 | |
rkukura | matrohon: that’s the one | 15:56 |
ajo | HenryG, so feel free to publish the comments without approving, and I will fix :) | 15:56 |
*** Longgeek has joined #openstack-neutron | 15:56 | |
rossella_s | rkukura, matrohon mistery solved :) ...shall we continue the discussion on the review directly? | 15:56 |
matrohon | rkukura : but by the time I reported the bug, the race could be seen with a debugger that slows down nova | 15:56 |
rkukura | rossella_s: makes sense, but feel free to ping me on IRC | 15:57 |
HenryG | ajo: thanks. Also marun might have something to say, maybe. | 15:57 |
*** pal_ has joined #openstack-neutron | 15:57 | |
ajo | ack, marun ^ https://review.openstack.org/152512 | 15:58 |
matrohon | rkukura : fine, I'll try to summarize this discussion on gerrit, and propose a patch that returns an error when the port is not bound to host submitted by get_device_details | 15:58 |
marun | ajo: ack | 15:58 |
rossella_s | matrohon, rkukura thanks for the nice discussion! | 15:58 |
matrohon | rossella_s : thanks for your review! | 15:59 |
rossella_s | matrohon you can't return an error right now, the l2 agent will resync everything | 15:59 |
rossella_s | matrohon: my pleasure! | 15:59 |
HenryG | ajo: comments posted | 15:59 |
*** pal has quit IRC | 15:59 | |
*** ronis has quit IRC | 15:59 | |
*** vthapar has quit IRC | 15:59 | |
rossella_s | matrohon: I mean right now the code in the l2 agent is dummy, in case of error it's going to resync. I am changing that...after the agent behaves better in case of error, you can modify get_device_details to return an error if the host id doesn't match | 16:00 |
*** nlahouti has joined #openstack-neutron | 16:00 | |
*** thedodd has quit IRC | 16:00 | |
*** roeyc has quit IRC | 16:01 | |
*** carl_baldwin has quit IRC | 16:01 | |
matrohon | rossella_s : ok, so right now, returning an error will trigger a loop of RPC get_device_details call? | 16:01 |
*** aveiga has joined #openstack-neutron | 16:02 | |
rossella_s | matrohon right now, the error will trigger a resync in the OVS agent, which will send get_device_details again and if it gets another error...well loop | 16:03 |
rossella_s | matrohon: it will probably loop till the port update is processed...that's very bad | 16:03 |
*** andreykurilin has quit IRC | 16:03 | |
*** prad has quit IRC | 16:04 | |
rossella_s | matrohon: if there are many devices, the resync is expensive | 16:04 |
matrohon | rossella_s : ok I'd better to propose my patch on top of yours | 16:04 |
*** killer_prince is now known as lazy_prince | 16:04 | |
rossella_s | matrohon: if you are not in an hurry :) | 16:04 |
ajo | thanks HenryG , I commented back, I will fix, but I disagree about two points, one in the iptables_firewall @staticmethod thing, and the other about duplicating set and list (line 1485 @ test), the 2nd one not very strong, but check if what I said makes sense. :) | 16:05 |
matrohon | rossella_s : which one of your patch introduces error handling in l2 agent? | 16:05 |
ajo | HenryG if you have a second, please check my comments, and thanks for the review, not all were nits ;) | 16:05 |
*** ijw has joined #openstack-neutron | 16:06 | |
HenryG | ajo: I certainly did not expect agreement to all my opinions! Thanks for looking, I will check your comments. | 16:06 |
rossella_s | matrohon: https://review.openstack.org/#/c/157839 | 16:07 |
ajo | :D HenryG :) | 16:07 |
rossella_s | matrohon: I have to update it a little bit, anyway you get the general idea there | 16:07 |
matrohon | rossella_s : ok thanks | 16:08 |
*** nmagnezi_ has quit IRC | 16:08 | |
*** claudiub has quit IRC | 16:09 | |
*** scheuran_ has quit IRC | 16:09 | |
*** tiswanso has quit IRC | 16:09 | |
matrohon | rossella_s : my patch is mainly motivated by errors seen will running this patch : https://review.openstack.org/#/c/141114/ | 16:09 |
matrohon | rossella_s : we would love to have l2pop compatible with HA router in k, so we are more or less in a hurry :) | 16:10 |
*** dboik_ has quit IRC | 16:10 | |
*** tiswanso has joined #openstack-neutron | 16:10 | |
*** dboik has joined #openstack-neutron | 16:10 | |
rossella_s | matrohon: I understand, well you know that having dependencies doesn't speed up | 16:11 |
*** chandankumar has joined #openstack-neutron | 16:11 | |
*** ijw has quit IRC | 16:12 | |
*** ParsectiX has joined #openstack-neutron | 16:12 | |
*** vivek-eb_ has joined #openstack-neutron | 16:12 | |
*** dsundarraj has quit IRC | 16:12 | |
*** setuid has quit IRC | 16:13 | |
matrohon | rossella_s : of course, but there is another alternative proposed for 141114, so I'll move forward with dependencies | 16:13 |
openstackgerrit | vishwanath jayaraman proposed openstack/neutron-fwaas: Refactoring related to 'FWaaS insertion Model on Routers' https://review.openstack.org/161881 | 16:13 |
rossella_s | matrohon just read your comment on that patch. I think 2 is a good option | 16:14 |
*** yfried has quit IRC | 16:14 | |
*** setuid has joined #openstack-neutron | 16:14 | |
*** vivek-ebay has quit IRC | 16:16 | |
*** tsg_ has quit IRC | 16:17 | |
*** setuid has quit IRC | 16:18 | |
*** claudiub has joined #openstack-neutron | 16:20 | |
*** setuid has joined #openstack-neutron | 16:21 | |
*** dsundarraj has joined #openstack-neutron | 16:22 | |
*** bandarji has joined #openstack-neutron | 16:23 | |
*** prad has joined #openstack-neutron | 16:23 | |
*** seizadi has joined #openstack-neutron | 16:24 | |
*** seizadi has quit IRC | 16:25 | |
*** seizadi has joined #openstack-neutron | 16:25 | |
*** rkukura has quit IRC | 16:25 | |
*** dsundarraj has quit IRC | 16:25 | |
*** vivek-eb_ has quit IRC | 16:26 | |
*** Longgeek has quit IRC | 16:26 | |
openstackgerrit | Zengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API https://review.openstack.org/159618 | 16:27 |
*** afazekas_ has quit IRC | 16:27 | |
*** vivek-ebay has joined #openstack-neutron | 16:27 | |
*** pritesh has quit IRC | 16:28 | |
*** baoli has quit IRC | 16:28 | |
*** jlanoux has quit IRC | 16:28 | |
*** irenab has joined #openstack-neutron | 16:28 | |
*** vivek-ebay has quit IRC | 16:28 | |
*** matrohon has quit IRC | 16:30 | |
*** salv-orlando has joined #openstack-neutron | 16:31 | |
openstackgerrit | Merged openstack/python-neutronclient: Copy functional tests from tempest cli https://review.openstack.org/162848 | 16:32 |
enikanorov | salv-orlando: hi! so... how do you think, who's +A do we need? :) https://review.openstack.org/#/c/149261/ | 16:32 |
*** bjornar_ has joined #openstack-neutron | 16:36 | |
*** absubram has joined #openstack-neutron | 16:37 | |
*** rkukura has joined #openstack-neutron | 16:38 | |
*** baoli has joined #openstack-neutron | 16:38 | |
*** chandankumar has quit IRC | 16:38 | |
*** zhhuabj has joined #openstack-neutron | 16:38 | |
claudiub | armax: hello | 16:38 |
openstackgerrit | Merged openstack/python-neutronclient: Remove unused AlreadyAttachedClient https://review.openstack.org/163419 | 16:38 |
*** zzzeek has joined #openstack-neutron | 16:40 | |
openstackgerrit | ZongKai LI proposed openstack/neutron: fix check_ports_exist_on_l3agent in no subnet case https://review.openstack.org/159338 | 16:40 |
*** SridharRamaswamy has joined #openstack-neutron | 16:41 | |
* zilberstein loses connectivity when neutron starts | 16:42 | |
zilberstein | What to check ? | 16:42 |
larsks | zilberstein: start with why your connectivity is gone...routing problem? Address got deleted? iptables blocking traffic? Also, you probably want to move this to #openstack; this channel is primarily for development questions. | 16:44 |
*** jistr has quit IRC | 16:45 | |
zilberstein | larsks, no, routering table hasnt changed in linux. What is strange is that when I stop neutron-ofagent, it works again | 16:45 |
*** vivek-ebay has joined #openstack-neutron | 16:45 | |
zilberstein | and ovs-vsctl shows br-int and its only port eth0 that I added | 16:45 |
zilberstein | as it should be | 16:45 |
openstackgerrit | Sridar Kandaswamy proposed openstack/neutron-fwaas: Adds FWaaS service plugin for Cisco CSR1kv https://review.openstack.org/152282 | 16:46 |
*** pal_ has quit IRC | 16:47 | |
*** yamahata has quit IRC | 16:47 | |
*** Miouge has joined #openstack-neutron | 16:48 | |
*** yamamoto has quit IRC | 16:48 | |
*** carl_baldwin has joined #openstack-neutron | 16:48 | |
*** Marga_ has joined #openstack-neutron | 16:52 | |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Introduce External IPAM Interface https://review.openstack.org/134339 | 16:53 |
carl_baldwin | mestery: ^ working around new pylint requirement. | 16:53 |
*** aranjan has joined #openstack-neutron | 16:53 | |
*** leenheer has joined #openstack-neutron | 16:54 | |
*** ParsectiX has quit IRC | 16:54 | |
*** ParsectiX has joined #openstack-neutron | 16:54 | |
*** Marga__ has joined #openstack-neutron | 16:55 | |
*** Swami_ has quit IRC | 16:55 | |
*** Marga_ has quit IRC | 16:57 | |
*** sbfox has joined #openstack-neutron | 16:57 | |
*** VijayB_ has joined #openstack-neutron | 16:58 | |
*** Marga__ has quit IRC | 16:58 | |
*** aranjan has quit IRC | 16:58 | |
*** ihrachyshka has joined #openstack-neutron | 16:58 | |
*** aranjan has joined #openstack-neutron | 16:58 | |
*** Marga_ has joined #openstack-neutron | 16:58 | |
*** mwagner_lap has quit IRC | 16:59 | |
*** seizadi has quit IRC | 16:59 | |
*** SridharRamaswamy has quit IRC | 16:59 | |
*** seizadi has joined #openstack-neutron | 16:59 | |
*** afazekas has joined #openstack-neutron | 16:59 | |
*** yfauser has joined #openstack-neutron | 17:00 | |
*** vmtrooper has joined #openstack-neutron | 17:01 | |
*** yfauser has left #openstack-neutron | 17:02 | |
*** kickinz1 is now known as kickinz1|afk | 17:03 | |
*** jlanoux has joined #openstack-neutron | 17:04 | |
*** harlowja_away is now known as harlowja_ | 17:06 | |
*** ijw has joined #openstack-neutron | 17:06 | |
*** vmtrooper has quit IRC | 17:06 | |
*** Swami has joined #openstack-neutron | 17:07 | |
*** VijayB_ has quit IRC | 17:07 | |
*** chandankumar has joined #openstack-neutron | 17:08 | |
*** yamahata has joined #openstack-neutron | 17:09 | |
*** ijw has quit IRC | 17:11 | |
*** VijayB_ has joined #openstack-neutron | 17:13 | |
*** aranjan has quit IRC | 17:14 | |
*** ronis has joined #openstack-neutron | 17:16 | |
*** aranjan has joined #openstack-neutron | 17:17 | |
*** ygbo has quit IRC | 17:17 | |
*** aranjan has quit IRC | 17:18 | |
ihrachyshka | HenryG, hey. up to a challenge? ;) | 17:19 |
HenryG | ihrachyshka: not the oslo.log patch? ;) | 17:19 |
ihrachyshka | ok, moving on... | 17:20 |
ihrachyshka | :) | 17:20 |
HenryG | ihrachyshka: hit me | 17:20 |
ihrachyshka | well... yeah, the oslo.log: https://review.openstack.org/#/c/159638/ | 17:20 |
HenryG | ihrachyshka: Ouch! ;) | 17:20 |
ihrachyshka | I will need two cores to get it in the queue before something breaks it (and that can be anything) | 17:21 |
HenryG | ihrachyshka: sure, I will do it. I have been keeping track of it so it should not be too bad. | 17:21 |
ihrachyshka | HenryG, even if you track patches like that, the number of files is just... unmanageable | 17:22 |
ihrachyshka | HenryG, thanks a lot, I owe you | 17:22 |
*** sbalukoff has quit IRC | 17:22 | |
*** melwitt has joined #openstack-neutron | 17:22 | |
*** aranjan has joined #openstack-neutron | 17:23 | |
ajo | HenryG, your list vs set comment makes sense, thanks for responding so fast ;) | 17:23 |
*** aix has quit IRC | 17:25 | |
*** aranjan has quit IRC | 17:25 | |
*** kawa2014 has quit IRC | 17:25 | |
*** britthouser has quit IRC | 17:26 | |
*** aranjan has joined #openstack-neutron | 17:26 | |
*** britthouser has joined #openstack-neutron | 17:26 | |
*** devlaps has joined #openstack-neutron | 17:26 | |
*** evgenyf has quit IRC | 17:29 | |
*** pask81 has quit IRC | 17:29 | |
*** wuhg has quit IRC | 17:29 | |
*** jp_at_hp has quit IRC | 17:31 | |
*** aranjan_ has joined #openstack-neutron | 17:32 | |
*** aranjan has quit IRC | 17:32 | |
*** achanda has joined #openstack-neutron | 17:32 | |
*** blues-man has quit IRC | 17:32 | |
*** ivar-lazzaro has joined #openstack-neutron | 17:33 | |
*** tflynn has joined #openstack-neutron | 17:35 | |
*** sripriya has joined #openstack-neutron | 17:37 | |
*** lazy_prince is now known as killer_prince | 17:41 | |
*** yfried has joined #openstack-neutron | 17:42 | |
*** sudipto has joined #openstack-neutron | 17:43 | |
*** aranjan_ has quit IRC | 17:44 | |
*** shwetaap1 has joined #openstack-neutron | 17:45 | |
*** aranjan has joined #openstack-neutron | 17:47 | |
*** mrsmith has joined #openstack-neutron | 17:47 | |
*** pritesh has joined #openstack-neutron | 17:47 | |
*** shwetaap has quit IRC | 17:48 | |
*** natarajk has joined #openstack-neutron | 17:51 | |
*** safchain has quit IRC | 17:52 | |
*** Marga_ has quit IRC | 17:53 | |
*** claudiub has quit IRC | 17:54 | |
*** aranjan has quit IRC | 17:56 | |
*** Marga_ has joined #openstack-neutron | 17:56 | |
*** aranjan has joined #openstack-neutron | 17:56 | |
pc_m | carl_baldwin: hi | 17:57 |
carl_baldwin | pc_m: hi | 17:57 |
pc_m | carl_baldwin: Wondering if you had time to chat about the bug reported with DVR and namespace for VPN | 17:58 |
*** SumitNaiksatam has joined #openstack-neutron | 17:59 | |
carl_baldwin | pc_m: Yes, what is the latest? | 17:59 |
*** annegentle has quit IRC | 18:00 | |
*** yfried is now known as yfried|afk | 18:00 | |
*** shivharis has joined #openstack-neutron | 18:00 | |
pc_m | carl_baldwin: Well, in VPN, when creating a process manager for OpenSwan, it expects there to be a namespace for the router. | 18:00 |
*** SridharRamaswamy has joined #openstack-neutron | 18:01 | |
carl_baldwin | Right. | 18:01 |
pc_m | This call happens when VPN service is created, or when router added (I guess for case of router add taking longer than service create). | 18:01 |
pc_m | In the past, this was always true, but is not, now for DVR. | 18:02 |
*** igordcard_ has joined #openstack-neutron | 18:02 | |
pc_m | Now, for the latter case, which could happen with just a router add and no service, it would not create the process. | 18:03 |
*** dims__ has joined #openstack-neutron | 18:03 | |
*** dims__ has quit IRC | 18:03 | |
*** bjornar_ has quit IRC | 18:04 | |
pc_m | However, it calls code as part of the "sync" that will try to delete the process for any processes where there is no VPN service. | 18:04 |
carl_baldwin | pc_m: Just need 3-5 minutes to clean out a workspace to start looking at it. Bear with me. | 18:04 |
*** dims__ has joined #openstack-neutron | 18:04 | |
pc_m | OK | 18:04 |
*** flaviof has quit IRC | 18:04 | |
*** flaviof has joined #openstack-neutron | 18:05 | |
*** mwagner_lap has joined #openstack-neutron | 18:06 | |
*** seizadi has quit IRC | 18:06 | |
*** ijw has joined #openstack-neutron | 18:06 | |
*** seizadi1 has joined #openstack-neutron | 18:06 | |
*** jlanoux has quit IRC | 18:06 | |
pc_m | carl_baldwin: Let em know when you're ready, and I'll get all the links to the code for the issue. | 18:06 |
*** fredlhsu has joined #openstack-neutron | 18:07 | |
*** dims_ has quit IRC | 18:07 | |
carl_baldwin | pc_m: ok, ready | 18:07 |
* carl_baldwin reading back... | 18:07 | |
pc_m | Here's the sequence that causes the issue.... | 18:08 |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Move internal port processing to router classes https://review.openstack.org/154513 | 18:08 |
pc_m | carl_baldwin: Router is created, and notify() calls this: https://github.com/openstack/neutron-vpnaas/blob/master/neutron_vpnaas/services/vpn/vpn_service.py#L57 | 18:08 |
pc_m | It will add the router to the driver, and do a sync(). https://github.com/openstack/neutron-vpnaas/blob/master/neutron_vpnaas/services/vpn/device_drivers/ipsec.py#L772 | 18:09 |
*** jpena has quit IRC | 18:09 | |
*** dims__ has quit IRC | 18:09 | |
pc_m | For the sync, nothing will happen for _sync_vpn_processes(), but it will try to get rid of processes without VPN. | 18:09 |
pc_m | Note process is an object that manages the OpenSwan process. | 18:10 |
pc_m | For some reason, that module first creates the process, if it doesn't exist: https://github.com/openstack/neutron-vpnaas/blob/master/neutron_vpnaas/services/vpn/device_drivers/ipsec.py#L795 | 18:10 |
pc_m | And that's the issue, as there is no namespace. | 18:10 |
pc_m | Now, there is a fix to that. Here: https://review.openstack.org/#/c/162840/8/neutron_vpnaas/services/vpn/device_drivers/ipsec.py | 18:11 |
*** shivharis has quit IRC | 18:11 | |
pc_m | It is part of another fix, where I suggested this line be removed (couldn't see why it was there). It would avoid the problem | 18:11 |
*** shivharis has joined #openstack-neutron | 18:11 | |
*** Longgeek has joined #openstack-neutron | 18:11 | |
*** ijw has quit IRC | 18:12 | |
pc_m | carl_baldwin: However, the question is, would we have an issue with the namespace not being set, if a router is created and a VPN connection added immediately? | 18:12 |
*** annegentle has joined #openstack-neutron | 18:12 | |
*** fredlhsu has quit IRC | 18:12 | |
* SridharRamaswamy pulls a chair to learn his router -> vpn flow | 18:12 | |
carl_baldwin | pc_m: I’m following along, just trying to soak it in still | 18:13 |
pc_m | carl_baldwin: So it creates the router entry, does a sync operation, and tries to create the process, so it can delete it...duh | 18:13 |
pc_m | The fix out for review on another item, would prevent this issue too, IMHO. | 18:14 |
pc_m | However, I'm concerned we could create a router and VN connection, and maybe not have the namespace in time, for a normal VPN create flow. | 18:15 |
*** mlavalle has joined #openstack-neutron | 18:15 | |
*** nyechiel__ has quit IRC | 18:15 | |
*** annegentle has quit IRC | 18:15 | |
carl_baldwin | pc_m: It doesn’t look right to “ensure_process” just to destroy the router. | 18:15 |
*** iwi has joined #openstack-neutron | 18:15 | |
*** annegentle has joined #openstack-neutron | 18:15 | |
pc_m | exactly, so when this fix was out for review, I asked to have that pulled. No idea why it was there. | 18:15 |
carl_baldwin | The normal VPN flow happens on “after_router_added”, right? | 18:15 |
iwi | Hi there | 18:16 |
*** baoli has quit IRC | 18:16 | |
pc_m | carl_baldwin: Yes, you'd add the router, and then once added, the VPN service could be created and connection created. | 18:16 |
pc_m | carl_baldwin: Wondering if it is always guaranteed that the namespace will be there by then. | 18:16 |
iwi | is it possible to use an external network as a normal tenant network (so for both - DHCP for VMs and Floating IPs) | 18:16 |
iwi | ? | 18:16 |
carl_baldwin | pc_m: Then the namespace will be there. | 18:16 |
HenryG | ihrachyshka: done, now you need one more. | 18:17 |
*** kawa2014 has joined #openstack-neutron | 18:17 | |
pc_m | carl_baldwin: If so, then I think the fix out will kill two birds with one stone. | 18:17 |
*** achanda has quit IRC | 18:17 | |
ihrachyshka | HenryG, thanks, let's see who hasn't hidden under a bench yet... | 18:17 |
carl_baldwin | pc_m: Still speaking of this fix: https://review.openstack.org/#/c/162840 ? | 18:18 |
pc_m | carl_baldwin: yes | 18:18 |
ihrachyshka | mestery, up to adventure?.. ;) I need someone brave to dive into oslo.log migration patch to push it in gate. | 18:19 |
pc_m | carl_baldwin: It clears up some other issues with a recent refactoring that I had done, but changes this area too (as we need to delete the process, but not the router we created, when we delete a service). | 18:19 |
*** jschwarz has quit IRC | 18:20 | |
*** yfried|afk is now known as yfried | 18:20 | |
*** Marga_ has quit IRC | 18:20 | |
pc_m | SridharRamaswamy: I've got some questions for you too, once done here, if you're still around. | 18:20 |
SridharRamaswamy | pc_m: sure | 18:21 |
carl_baldwin | pc_m: Reviewing it now. | 18:21 |
pc_m | carl_baldwin: Thanks. So will the namespace be there shortly after the after_router_added() notify? | 18:21 |
mlavalle | carl_baldwin, pc_m: I see you are weel underway in your discussion about the namespace issue. Don't want to get in the way. Please ping me if you think I might add something | 18:22 |
*** achanda has joined #openstack-neutron | 18:23 | |
*** tsg_ has joined #openstack-neutron | 18:23 | |
pc_m | mlavalle: yeah, sorry forgot to ping you too. If you can scroll up, you'll see we have a fix. Though I do have a question on when the namespace is ready. | 18:23 |
carl_baldwin | pc_m: The namespace will be created for a new router before notifying. | 18:23 |
*** achanda has quit IRC | 18:23 | |
pc_m | carl_baldwin: Hmmm. Then wondering why there was no namespace. | 18:24 |
openstackgerrit | Merged openstack/neutron: Fix non-existent self.local_subnets in DvrRouter class https://review.openstack.org/157621 | 18:24 |
*** achanda has joined #openstack-neutron | 18:24 | |
*** kawa2014 has quit IRC | 18:24 | |
*** rushil_ has quit IRC | 18:24 | |
*** kawa2014 has joined #openstack-neutron | 18:24 | |
pc_m | SridharRamaswamy: For the Brocade VPN plugin I had some questions about out of tree code... | 18:26 |
SridharRamaswamy | pc_m: sure, i replied after your discussion w/ armax | 18:26 |
SridharRamaswamy | pc_m: I think you logged off by then | 18:26 |
pc_m | SridharRamaswamy: (may have missed) . The code has an import of networking_brocade. | 18:26 |
*** afazekas has quit IRC | 18:26 | |
pc_m | SridharRamaswamy: How does the package get installed, say for devstack or UT use? | 18:27 |
SridharRamaswamy | pcm: We got that working by mocking the networking_brocade imports. See #link https://github.com/openstack/neutron-vpnaas/blob/master/neutron_vpnaas/tests/unit/services/vpn/device_drivers/test_vyatta_ipsec.py | 18:27 |
*** sudipto has quit IRC | 18:28 | |
pc_m | SridharRamaswamy: Ah. Makes sense. Then for devstack, do you have a devstack plugin to do the install? | 18:28 |
SridharRamaswamy | pc_m: this is was standard suggestion for all the decomposed plugins - to get their UT working *without* doing the import | 18:28 |
SridharRamaswamy | pc_m: No. Our CI scripts does add a separate entry point to invoke out vyatta-vpn-agent. But that happens only in our CI env | 18:30 |
*** yfried is now known as yfried|afk | 18:30 | |
*** yfried|afk is now known as yfried | 18:31 | |
*** Longgeek has quit IRC | 18:31 | |
pc_m | SridharRamaswamy: So for functional tests, you could create a devstack plugin to load the package, I guess. | 18:31 |
carl_baldwin | pc_m: If I understood correctly, there was no namespace because it wasn’t a new router, it was a router being cleaned up. | 18:31 |
pc_m | carl_baldwin: Oh, do you mean in that test case they were cleaning up a router? | 18:32 |
Swami | pcm: that might be true | 18:32 |
pc_m | carl_baldwin: Why would it call after_router_added (and not after/before_router_removed)? | 18:33 |
carl_baldwin | pc_m: I thought that a different router was added which notified. Then a sync routers was called that was trying to destroy another router, not the one that was just added. | 18:34 |
pc_m | carl_baldwin: Ah, ok that would make sense. | 18:34 |
*** dims has joined #openstack-neutron | 18:35 | |
SridharRamaswamy | pc_m: Yes, we have devstack patches in the CI env. We can augment that if needed to get the functional tests going. | 18:35 |
openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver https://review.openstack.org/158560 | 18:36 |
openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention https://review.openstack.org/144391 | 18:36 |
*** claudiub has joined #openstack-neutron | 18:36 | |
pc_m | SridharRamaswamy: Thanks for the info. We may need to do the same in Cisco code. | 18:37 |
*** numan has joined #openstack-neutron | 18:37 | |
*** numan has quit IRC | 18:37 | |
SridharRamaswamy | pc_m: sounds good | 18:38 |
*** Sukhdev has joined #openstack-neutron | 18:38 | |
Sukhdev | mestery: Ping | 18:39 |
*** skraynev has quit IRC | 18:39 | |
Sukhdev | mestery: Can you please tag this for K3? - https://bugs.launchpad.net/neutron/+bug/1429968 | 18:40 |
openstack | Launchpad bug 1429968 in neutron "In Arista ML2 delete tenant if no resources associated with it" [Undecided,Fix committed] - Assigned to Sukhdev Kapur (sukhdev-8) | 18:40 |
pc_m | carl_baldwin: Thanks for approving. What do we do about the bug filed? Have the submitter check and close, if fixed? | 18:40 |
carl_baldwin | pc_m: Sounds like a good plan. | 18:40 |
Swami | pc_m: what bug are you talking about here. | 18:41 |
pc_m | carl_baldwin: I updated bug. | 18:41 |
pc_m | Swami: 1431077 | 18:41 |
*** sbalukoff has joined #openstack-neutron | 18:42 | |
pc_m | Swami: Is that the one you filed? | 18:42 |
Swami | pc_m: Yes this is the one that I filed. | 18:42 |
Swami | I will check this with your patch and then update the LP. | 18:42 |
pc_m | Swami: Thanks! | 18:42 |
pc_m | Swami: I think it may resolve the issue. | 18:43 |
pc_m | carl_baldwin: If you get a chance, look at https://review.openstack.org/#/c/163586/. Easy peasy review and removes dead code related to this namespace thing. | 18:44 |
*** claudiub has quit IRC | 18:47 | |
carl_baldwin | pc_m: Done | 18:47 |
*** SridharRamaswamy is now known as sridhar_ram | 18:48 | |
pc_m | SridharRamaswamy: Can you test the latest StrongSwan patch (61)? The previous one looked good, other than changes interfered with running connections. I think Zhang has a fix for that. I'm going to re-test as well. | 18:48 |
pc_m | carl_baldwin: Thank you. Don't you wish they all were that easy? :) | 18:48 |
carl_baldwin | pc_m: :) | 18:48 |
sridhar_ram | pc_m: sure, will give it a spin | 18:49 |
*** sbfox has quit IRC | 18:49 | |
pc_m | sridhar_ram: THanks. Looks like we're close to getting this in... | 18:49 |
*** vmtrooper has joined #openstack-neutron | 18:50 | |
*** Marga_ has joined #openstack-neutron | 18:50 | |
enikanorov_ | mestery: hi, who could be a core to give +A https://review.openstack.org/#/c/149261/ ? This has raised a lot of discussions, but i think we may want to have a fix for the issue and move on with enabling multiple api and rpc workers by default | 18:52 |
*** vmtrooper has quit IRC | 18:55 | |
*** thedodd has joined #openstack-neutron | 18:55 | |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Move internal port processing to router classes https://review.openstack.org/154513 | 18:56 |
sridhar_ram | pc_m: i agree, but state transition from pending-create --> down --> active still bothers me. I'm going to check if that got fixed in the latest PS. | 19:00 |
pc_m | sridhar_ram: Yeah, I mentioned that I prefer PENDING_CREATE - > ACTIVE, but it will take more complex logic to check that, because of the way StrongSwan reports status. | 19:03 |
pc_m | sridhar_ram: I mentioned that I'd be OK with doing that as a bug fix later. | 19:03 |
*** yfried is now known as yfried|afk | 19:04 | |
*** shivharis has quit IRC | 19:04 | |
sridhar_ram | pc_m: okay | 19:04 |
*** shivharis has joined #openstack-neutron | 19:05 | |
pc_m | sridhar_ram: it reports CONNECTING, when connecting, but for a period of time, it reports INSTALLED and CONNECTING, so need to check multiple status lines. | 19:05 |
sridhar_ram | pc_m: I see | 19:05 |
*** kawa2014 has quit IRC | 19:05 | |
*** bobmel_ has joined #openstack-neutron | 19:06 | |
*** yfried|afk is now known as yfried | 19:06 | |
pc_m | sridhar_ram: I'm wondering if openswan drops traffic on a config change (adding a second connection). I think it will. | 19:06 |
*** ijw has joined #openstack-neutron | 19:06 | |
*** yfauser has joined #openstack-neutron | 19:07 | |
*** yfauser has left #openstack-neutron | 19:07 | |
*** bobmel has quit IRC | 19:09 | |
*** tgohad has joined #openstack-neutron | 19:09 | |
*** carl_baldwin has quit IRC | 19:10 | |
*** Marga_ has quit IRC | 19:11 | |
*** ijw has quit IRC | 19:11 | |
*** seizadi1 has quit IRC | 19:12 | |
*** tsg_ has quit IRC | 19:13 | |
*** seizadi has joined #openstack-neutron | 19:18 | |
*** chandankumar has quit IRC | 19:19 | |
*** seizadi has quit IRC | 19:19 | |
*** ParsectiX has quit IRC | 19:22 | |
*** crc32 has joined #openstack-neutron | 19:22 | |
*** ParsectiX has joined #openstack-neutron | 19:22 | |
*** shivharis has quit IRC | 19:23 | |
openstackgerrit | Brian Bowen proposed openstack/neutron: Fix missing spaces in strings split across lines. https://review.openstack.org/163859 | 19:25 |
*** claudiub has joined #openstack-neutron | 19:26 | |
*** leenheer has quit IRC | 19:30 | |
*** leenheer has joined #openstack-neutron | 19:33 | |
*** chandankumar has joined #openstack-neutron | 19:34 | |
*** YorikSar has joined #openstack-neutron | 19:34 | |
*** jorgem has joined #openstack-neutron | 19:35 | |
YorikSar | otherwiseguy: Do you know if there's a consensus about whether we should enable daemon mode by default? | 19:35 |
*** ParsectiX has quit IRC | 19:36 | |
otherwiseguy | YorikSar: A lot of people have asked that. I tend to be cautious about such things. I mean, technically even using rootwrap at all is non-default. | 19:36 |
otherwiseguy | Since root_helper defaults to 'sudo'. | 19:37 |
YorikSar | otherwiseguy: Is it? Oh... | 19:37 |
otherwiseguy | But rootwrap daemon should be faster than sudo too... | 19:37 |
YorikSar | otherwiseguy: Ok, then the question should be whether we want devstack to configure daemon mode by default. | 19:38 |
YorikSar | otherwiseguy: Or we'll have to run 2 jobs: one for daemon mode, another for rootwrap case. | 19:38 |
*** pc_m has quit IRC | 19:38 | |
*** sbfox has joined #openstack-neutron | 19:38 | |
*** _cjones_ has quit IRC | 19:39 | |
otherwiseguy | YorikSar: If it was up to me, I'd say yes just from the perspective that 'rootwrap' code doesn't really change a lot and is unlikely to break, also even neutron will use rootwrap and rootwrap daemon both (because of long-running processes), so both will get exercised. | 19:39 |
*** openstackgerrit has quit IRC | 19:40 | |
otherwiseguy | Others may like the 'do both' approach, but I hate doubling effort for testing. | 19:40 |
YorikSar | otherwiseguy: I guess this discussion will be held in devstack CR | 19:40 |
*** openstackgerrit has joined #openstack-neutron | 19:41 | |
YorikSar | otherwiseguy: Yeah. Besides rootwrap is well-tested (including daemon mode). | 19:41 |
otherwiseguy | YorikSar: so does 158805 look good now that the devstack patch tests are passing? | 19:42 |
*** aranjan has quit IRC | 19:42 | |
YorikSar | otherwiseguy: Let's wait for another round... As I said it's very strange that lbaas counldn't start daemon while other services could. | 19:43 |
YorikSar | (I mean dhcp and l3 agents) | 19:43 |
*** leenheer has quit IRC | 19:43 | |
*** leenheer has joined #openstack-neutron | 19:44 | |
*** chandankumar has quit IRC | 19:45 | |
*** Marga_ has joined #openstack-neutron | 19:45 | |
*** Marga_ has quit IRC | 19:50 | |
*** yfried is now known as yfried|afk | 19:51 | |
*** marun has quit IRC | 19:54 | |
*** Padmini has joined #openstack-neutron | 19:58 | |
*** _cjones_ has joined #openstack-neutron | 19:59 | |
*** imcsk8|zZz is now known as imcsk8 | 20:01 | |
*** melwitt_ has joined #openstack-neutron | 20:02 | |
*** mikal_ is now known as mikal | 20:02 | |
*** seizadi has joined #openstack-neutron | 20:02 | |
*** melwitt has quit IRC | 20:02 | |
*** melwitt_ is now known as melwitt | 20:02 | |
*** nlahouti has quit IRC | 20:03 | |
*** seizadi has quit IRC | 20:03 | |
*** seizadi has joined #openstack-neutron | 20:03 | |
*** setuid has quit IRC | 20:03 | |
*** baoli has joined #openstack-neutron | 20:03 | |
*** sbfox has quit IRC | 20:04 | |
*** baoli has quit IRC | 20:04 | |
*** setuid has joined #openstack-neutron | 20:05 | |
*** baoli has joined #openstack-neutron | 20:05 | |
*** aranjan has joined #openstack-neutron | 20:05 | |
*** jgrimm is now known as zz_jgrimm | 20:07 | |
*** leenheer has quit IRC | 20:08 | |
*** matrohon has joined #openstack-neutron | 20:10 | |
*** yfried|afk is now known as yfried | 20:11 | |
*** aepifanov has joined #openstack-neutron | 20:11 | |
*** sbfox has joined #openstack-neutron | 20:11 | |
*** yfauser has joined #openstack-neutron | 20:12 | |
*** rushil has joined #openstack-neutron | 20:13 | |
*** yfauser has left #openstack-neutron | 20:13 | |
*** seizadi has quit IRC | 20:13 | |
*** seizadi has joined #openstack-neutron | 20:14 | |
*** zz_jgrimm is now known as jgrimm | 20:14 | |
*** Longgeek has joined #openstack-neutron | 20:16 | |
*** amuller has quit IRC | 20:16 | |
*** chandankumar has joined #openstack-neutron | 20:16 | |
ihrachyshka | armax, hey. may I ask you to check oslo.log migration patch? It needs the 2nd +2: https://review.openstack.org/159638 (CI failures are expected and were communicated to 3party maintainers) | 20:17 |
openstackgerrit | Michael Smith proposed openstack/neutron: delay fip namespace cleanup/destroy https://review.openstack.org/151758 | 20:18 |
*** sambetts has quit IRC | 20:18 | |
openstackgerrit | Jerome Bertolino proposed openstack/neutron: Update Cisco decomposition URL https://review.openstack.org/163950 | 20:19 |
*** jorgem has quit IRC | 20:20 | |
*** sambetts has joined #openstack-neutron | 20:20 | |
*** yfried is now known as yfried|afk | 20:21 | |
*** Longgeek has quit IRC | 20:23 | |
*** jorgem has joined #openstack-neutron | 20:23 | |
*** rpothier has quit IRC | 20:25 | |
*** Marga_ has joined #openstack-neutron | 20:35 | |
*** jorgem has quit IRC | 20:37 | |
*** Marga_ has quit IRC | 20:37 | |
*** jorgem has joined #openstack-neutron | 20:38 | |
*** Marga_ has joined #openstack-neutron | 20:38 | |
*** Sukhdev has quit IRC | 20:38 | |
*** vmtrooper has joined #openstack-neutron | 20:39 | |
*** vmtrooper has quit IRC | 20:44 | |
armax | ihrachyshka: question | 20:46 |
ihrachyshka | armax, shoot | 20:47 |
armax | ihrachyshka: do we know of the uptake of the change for the vendor repos? | 20:47 |
armax | ihrachyshka: i.e. have people started moving away from common/log to oslo log, as we preemptively done for the aas repos? | 20:48 |
*** carl_baldwin has joined #openstack-neutron | 20:48 | |
ihrachyshka | armax, some of them already did, like ryu or brocade. I expect others to follow once they actually notice breakage. | 20:48 |
ihrachyshka | armax, the complecity of the fix is low - import changes. | 20:49 |
armax | ihrachyshka: ok | 20:49 |
*** VijayB_ has quit IRC | 20:49 | |
*** chandankumar has quit IRC | 20:49 | |
ihrachyshka | in very rear case - an import path change, or similar. but I would expect no vendor libs will really need even that | 20:49 |
armax | ihrachyshka: let’s make sure we bring that up again during hte next irc meeting | 20:49 |
armax | ihrachyshka: we need to evangelize that wiki link of yours relentelessy :) | 20:50 |
ihrachyshka | ok. | 20:50 |
*** thedodd has quit IRC | 20:50 | |
ihrachyshka | armax, I should have also reach out their 3party team meeting on Mon to communicate the issue. though I realized that late in Tue. | 20:50 |
armax | ihrachyshka: it’s okay | 20:51 |
armax | ihrachyshka: we’ll get more proactive next time | 20:51 |
ihrachyshka | yeah, but next time I will try to be more vocal :) | 20:51 |
HenryG | armax: ihrachyshka: working on a patch for ours | 20:52 |
*** nlahouti has joined #openstack-neutron | 20:52 | |
*** cference has quit IRC | 20:52 | |
armax | HenryG: great, | 20:52 |
armax | ihrachyshka: let me go over the oslo patch and I’ll nudge it in | 20:52 |
ihrachyshka | armax, thanks sir :) | 20:53 |
*** thedodd has joined #openstack-neutron | 20:53 | |
armax | ihrachyshka: is there a defined schedule as to when these changes are best brought in? | 20:53 |
armax | ihrachyshka: or is it to whim of the oslo releases? | 20:53 |
openstackgerrit | vishwanath jayaraman proposed openstack/neutron-fwaas: Refactoring related to 'FWaaS insertion Model on Routers' https://review.openstack.org/161881 | 20:54 |
HenryG | armax: you will need to upgrade your glasses prescription when you are done looking at the oslo.log patch :) | 20:54 |
*** thedodd has quit IRC | 20:54 | |
*** marun has joined #openstack-neutron | 20:54 | |
ihrachyshka | armax, well, neutron team decides what's in each cycle, we do it on our own schedule. | 20:54 |
armax | HenryG: I change a pair a day | 20:54 |
dhellmann | armax: it's up to the projects, but we like to have adoption of new libs finished by the end of the cycle after we release the lib | 20:54 |
dhellmann | armax: what ihrachyshka said :-) | 20:54 |
*** yfried|afk is now known as yfried | 20:55 | |
armax | ok, so technically we can line them all up and do them, for instance during the RC window? | 20:55 |
ihrachyshka | armax, though I would vote for the more adoptions the better (except those we are scared e.g. due to position in the cycle) | 20:55 |
*** leenheer has joined #openstack-neutron | 20:55 | |
ihrachyshka | armax, I wouldn't do that that late in the cycle. there are glitches and bugs that may be introduced during adoption. | 20:56 |
armax | but then things will be fairly quite and fixes would be done more easily? | 20:56 |
ihrachyshka | armax, I would say, kilo-3 mile should be the last stone to consider any new merges. we also have dep freeze that is kinda related. | 20:57 |
*** jprovazn has quit IRC | 20:57 | |
ihrachyshka | because each adoption means a new dep | 20:57 |
armax | ihrachyshka: ok, fair point | 20:57 |
openstackgerrit | Pradeep Kilambi proposed openstack/neutron: Expose Rest Api access to mtu attributes https://review.openstack.org/154921 | 20:57 |
openstackgerrit | Pradeep Kilambi proposed openstack/neutron: Add MTU selection to ML2 https://review.openstack.org/156318 | 20:57 |
openstackgerrit | Pradeep Kilambi proposed openstack/neutron: Advertise mtu over dhcp https://review.openstack.org/155874 | 20:57 |
openstackgerrit | Pradeep Kilambi proposed openstack/neutron: Add MTU selection & advertisement settings to Neutron config https://review.openstack.org/153733 | 20:57 |
armax | ihrachyshka: so, still close to the end | 20:57 |
ihrachyshka | + doing it at rc time is a bit uncomfortable for those who package stuff I guess (too many work at the end of the cycle) | 20:58 |
dhellmann | armax, ihrachyshka: you wouldn't want to do them during the RC window because you shouldn't be adding dependencies at that point | 20:58 |
dhellmann | the feature freeze date is also a requirements freeze date | 20:58 |
*** raildo has quit IRC | 20:58 | |
ihrachyshka | dhellmann, right, dep freeze | 20:58 |
*** ajo has quit IRC | 20:58 | |
*** aranjan has quit IRC | 20:59 | |
*** dboik_ has joined #openstack-neutron | 20:59 | |
claudiub | armax: hello | 20:59 |
ihrachyshka | claudiub, hey man, I had a question to you! | 20:59 |
*** sbfox has quit IRC | 20:59 | |
claudiub | ihrachyshka: hello. :) | 20:59 |
*** ajmiller_ has joined #openstack-neutron | 21:00 | |
armax | dhellmann, ihrachyshka: makes sense, close to the third milestone it is | 21:00 |
armax | claudiub: hello | 21:00 |
ihrachyshka | claudiub, in RDO, we traditionally packaged hyperv agent in a separate package. but as I understand it, there is no real reason to ship it at all, since RDO is not meant to run on Windows :) so are we safe to kill the agent during build process? | 21:00 |
dhellmann | armax: well, *by* the 3rd milestone. You could also do it earlier. :-) | 21:00 |
*** mrsmith has quit IRC | 21:01 | |
*** VijayB_ has joined #openstack-neutron | 21:01 | |
ihrachyshka | armax, I don't actually think it's easier to introduce patches like that near 3rd milestone. it's exactly the time you get lots of reviews and updates and fixes, because everyone rushes to push his stuff in the tree that cycle. | 21:01 |
claudiub | armax: so, about the decomposition... well, you were saying that it should be moved to neutron/plugins/hyperv... well, I'd argue that it is more of an agent, since there is nothing else that can bind the neutron ports in hyper-v.. the reason it was in neutron/plugins (other than the fact that everyone was there) was because it had a HyperVNeutronPlugin, which we removed in favor of Ml2Plugin | 21:02 |
*** dboik has quit IRC | 21:02 | |
kevinbenton_ | rossella_s: hey | 21:02 |
*** ronis has quit IRC | 21:02 | |
*** leenheer has quit IRC | 21:02 | |
claudiub | armax: plus, having some naming issues, which I cannot solve. :D | 21:02 |
*** natarajk1 has joined #openstack-neutron | 21:03 | |
armax | dhellmann, ihrachyshka: I was only thinking whether it would be beneficial if we were more specific about the times these changes are introduced | 21:03 |
*** harlowja_ has quit IRC | 21:03 | |
armax | claudiub: I am not sure I follow | 21:03 |
*** ajmiller has quit IRC | 21:03 | |
claudiub | ihrachyshka: yeah, the hyper-v agent only runs on windows and has windows specific packages that have nothing to do with linux (wmi, pywin32, win32com) | 21:03 |
armax | claudiub: let’s take the discussion on the review page itself | 21:03 |
*** natarajk has quit IRC | 21:04 | |
claudiub | ihrachyshka: just one thing though | 21:04 |
*** harlowja has joined #openstack-neutron | 21:04 | |
claudiub | ihrachyshka: if there are RDO users that will use Hyper-V nodes, they will probably use Ml2Plugin to bind them, with hyperv mechanism.. for that to work, networking_hyperv package will be needed | 21:04 |
*** moshele has quit IRC | 21:05 | |
claudiub | ihrachyshka: since the mech driver will be imported from there | 21:05 |
*** seizadi has quit IRC | 21:05 | |
claudiub | ihrachyshka: other than that, nothing else is needed. :) | 21:05 |
claudiub | armax: Ok. | 21:05 |
armax | claudiub: review # again? | 21:05 |
armax | :) | 21:05 |
claudiub | https://review.openstack.org/#/c/153259/ | 21:05 |
ihrachyshka | armax, I agree I should be more specific about when patches are to be published for review and expected to be merged | 21:06 |
ihrachyshka | claudiub, right. we don't kill ml2 driver. so probably we'll kill agent, but keep the openstack-neutron-hyperv package to pull in the needed dependency (if we get to packaging it) | 21:06 |
armax | ihrachyshka: yeah, maybe we can announce when they are about to be introduced so that we can set expectations | 21:06 |
armax | ihrachyshka: not it’s a small thing really | 21:07 |
ihrachyshka | armax, I announced expected adoptions this cycle, but milestone wise, not specific dates. I'll consider being more specific, sounds like a good thing for everyone | 21:07 |
*** padkrish has joined #openstack-neutron | 21:08 | |
armax | ihrachyshka: ok, I am pretty sure you did and I did not pay attention ;) | 21:08 |
*** yfried is now known as yfried|afk | 21:08 | |
*** vivek-ebay has quit IRC | 21:09 | |
*** leenheer has joined #openstack-neutron | 21:12 | |
*** blogan_ is now known as blogan | 21:12 | |
openstackgerrit | Carl Baldwin proposed openstack/neutron: Move external port processing to router classes https://review.openstack.org/158495 | 21:13 |
*** thedodd has joined #openstack-neutron | 21:14 | |
armax | claudiub: ping | 21:15 |
claudiub | armax: sorry, was posting on the review. :) https://review.openstack.org/#/c/153259/ | 21:16 |
claudiub | i hope I explained myself a little bit better. :) | 21:17 |
*** mlavalle has quit IRC | 21:17 | |
armax | claudiub: ok | 21:17 |
* armax reading | 21:17 | |
*** yfried|afk is now known as yfried | 21:17 | |
claudiub | armax: PS: forgot to mention, the hyper-v agent is an L2 agent. :) | 21:19 |
openstackgerrit | Timothy Swanson proposed openstack/neutron: Add MTU selection & advertisement settings to Neutron config https://review.openstack.org/153733 | 21:19 |
armax | claudiub: ok I still don’t understand your argument, I’ll try to be more specific on the review as to what I mean | 21:19 |
claudiub | armax: I understand what you mean. :) | 21:20 |
*** Marga_ has quit IRC | 21:20 | |
claudiub | armax: but honestly, neutron.plugins.hyperv.agent.agent.HyperVNeutronAgent is such an ugly name... :D | 21:20 |
*** tiswanso has quit IRC | 21:21 | |
claudiub | armax: while neutron.agent.hyperv.HyperVNeutronAgent would be much better... | 21:21 |
*** aranjan has joined #openstack-neutron | 21:21 | |
*** ajo has joined #openstack-neutron | 21:21 | |
armax | claudiub: I am afraid your argument doens’t fly | 21:22 |
armax | claudiub: we’re not writing poems or novels | 21:22 |
armax | claudiub: it’s a place for a class is not appropriate, I don’t think we should be conditionated by the fully qualified name | 21:23 |
*** leenheer has quit IRC | 21:23 | |
*** amuller has joined #openstack-neutron | 21:24 | |
*** Marga_ has joined #openstack-neutron | 21:24 | |
*** yfauser has joined #openstack-neutron | 21:24 | |
*** shwetaap1 has quit IRC | 21:24 | |
*** yfauser has left #openstack-neutron | 21:25 | |
*** tim_o has joined #openstack-neutron | 21:26 | |
*** shwetaap has joined #openstack-neutron | 21:27 | |
openstackgerrit | Phillip Toohill proposed openstack/python-neutronclient: Updating lbaas cli for TLS https://review.openstack.org/161404 | 21:27 |
tim_o | is mestery or emagma here? | 21:27 |
salv-orlando | armax: so won't you take my patch for refactoring code in alternate rhyme? | 21:28 |
armax | salv-orlando: um probably not | 21:28 |
*** aranjan_ has joined #openstack-neutron | 21:28 | |
*** aranjan has quit IRC | 21:28 | |
matrohon | armax, dougwig : hi | 21:29 |
*** tgohad has quit IRC | 21:29 | |
*** jorgem has quit IRC | 21:29 | |
tim_o | just trying to get ahold of someone to look at the CI log file | 21:30 |
salv-orlando | armax: what about refactoring methods into sonnets? | 21:30 |
armax | salv-orlando: all I can accept is a limerick | 21:30 |
*** kevinbenton_ is now known as kevinbenton | 21:30 | |
*** jorgem has joined #openstack-neutron | 21:31 | |
matrohon | armax, dougwig : as you validated https://review.openstack.org/#/c/148318/, could you please have an eye on : https://review.openstack.org/#/c/163478/ ? | 21:31 |
*** SridharRamaswamy has joined #openstack-neutron | 21:31 | |
openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 21:32 |
*** sridhar_ram has quit IRC | 21:32 | |
*** sripriya has quit IRC | 21:33 | |
*** sripriya has joined #openstack-neutron | 21:33 | |
dougwig | matrohon: commented | 21:34 |
matrohon | dougwig : thanks | 21:34 |
matrohon | douwig : I was betting that somebody would ask for UT :) | 21:35 |
matrohon | dougwig ^ | 21:36 |
dougwig | good bet. :) | 21:36 |
openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 21:36 |
*** leenheer has joined #openstack-neutron | 21:37 | |
openstackgerrit | Ed Bak proposed openstack/neutron: Break calls to get_devices_details_list into subsets https://review.openstack.org/163594 | 21:38 |
matrohon | dougwig : however do you consider that extending the client the way I described in the bug report is the good way? personally, I find it very useful! | 21:38 |
*** Marga_ has quit IRC | 21:38 | |
*** iwi has left #openstack-neutron | 21:39 | |
openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 21:39 |
*** Marga_ has joined #openstack-neutron | 21:39 | |
dougwig | matrohon: seemed reasonable to me | 21:39 |
matrohon | dougwig : fine I'll submit UT asap, thanks | 21:40 |
openstackgerrit | Juergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration https://review.openstack.org/141130 | 21:40 |
openstackgerrit | Juergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration https://review.openstack.org/141130 | 21:43 |
*** dboik_ has quit IRC | 21:43 | |
*** dboik has joined #openstack-neutron | 21:44 | |
*** Sukhdev has joined #openstack-neutron | 21:44 | |
*** ihrachyshka has quit IRC | 21:44 | |
Sukhdev | mestery: are you there? | 21:45 |
*** sbfox has joined #openstack-neutron | 21:46 | |
*** amotoki has quit IRC | 21:49 | |
*** oda-g has joined #openstack-neutron | 21:49 | |
armax | HenryG: ping | 21:51 |
*** prad has quit IRC | 21:52 | |
dougwig | salv-orlando: i want to see a gerrit review of sonnets. | 21:52 |
*** kbringard has quit IRC | 21:53 | |
*** annegentle has quit IRC | 21:56 | |
*** moha_hunt has joined #openstack-neutron | 21:56 | |
*** pritesh has quit IRC | 22:01 | |
*** SridharRamaswamy has quit IRC | 22:01 | |
*** melwitt_ has joined #openstack-neutron | 22:01 | |
*** melwitt_ has quit IRC | 22:02 | |
*** harlowja has quit IRC | 22:02 | |
*** melwitt has quit IRC | 22:03 | |
*** SridharRamaswamy has joined #openstack-neutron | 22:04 | |
*** padkrish has quit IRC | 22:04 | |
*** apuimedo has joined #openstack-neutron | 22:04 | |
*** padkrish has joined #openstack-neutron | 22:04 | |
*** harlowja has joined #openstack-neutron | 22:04 | |
*** marrusl has quit IRC | 22:08 | |
*** Longgeek has joined #openstack-neutron | 22:08 | |
*** marrusl has joined #openstack-neutron | 22:08 | |
*** vivek-ebay has joined #openstack-neutron | 22:10 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Update contribute.rst with Big Switch decomp https://review.openstack.org/163996 | 22:10 |
openstackgerrit | Claudiu Belu proposed openstack/neutron: Hyper-V Agent decomposition https://review.openstack.org/153259 | 22:11 |
*** tsg has joined #openstack-neutron | 22:12 | |
salv-orlando | dougwig: I made an attempt a while ago.... not really a sonnet though: https://review.openstack.org/#/c/69579/1//COMMIT_MSG | 22:12 |
openstackgerrit | Abhishek Raut proposed openstack/neutron: Cisco Nexus1000V ML2 Mechanism Driver https://review.openstack.org/157243 | 22:12 |
*** aranjan_ has quit IRC | 22:12 | |
*** vivek-ebay has quit IRC | 22:14 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Update contribute.rst with Big Switch decomp https://review.openstack.org/163996 | 22:15 |
claudiub | armax: uploaded a new patchset | 22:15 |
dougwig | salv-orlando: ha, love it. | 22:15 |
*** aranjan has joined #openstack-neutron | 22:17 | |
amuller | armax: And you say I have too much time... | 22:17 |
*** shivharis has joined #openstack-neutron | 22:17 | |
*** carl_baldwin has quit IRC | 22:18 | |
openstackgerrit | Nachi Ueno proposed openstack/neutron: OpenContrail plugin code split https://review.openstack.org/163218 | 22:18 |
apuimedo | amuller: no weekend for you? | 22:18 |
amuller | apuimedo: It's 6:20pm here right now :) | 22:19 |
apuimedo | ah yes... I keep forgetting that you are in the other side | 22:19 |
apuimedo | so you do monday-friday now? | 22:19 |
amuller | yeah | 22:20 |
*** aranjan has quit IRC | 22:20 | |
amuller | still kind of confused when I see calendars starting on a Monday | 22:20 |
*** aranjan has joined #openstack-neutron | 22:20 | |
apuimedo | heh, for me it's the other way around, a lot of software starting the week no Sunday | 22:21 |
*** cference has joined #openstack-neutron | 22:21 | |
amuller | apuimedo: How's it going in Barcelona? | 22:22 |
*** yfried is now known as yfried|afk | 22:22 | |
apuimedo | amuller: going there too much for my taste | 22:22 |
*** luqas has joined #openstack-neutron | 22:22 | |
apuimedo | I enjoy more working from my town | 22:22 |
*** carl_baldwin has joined #openstack-neutron | 22:23 | |
amuller | apuimedo: Where's that? | 22:24 |
apuimedo | 1h commuting by bus | 22:25 |
apuimedo | Olesa de Montserrat | 22:25 |
*** logan2 has quit IRC | 22:26 | |
openstackgerrit | Merged openstack/neutron: Remove dead code https://review.openstack.org/163586 | 22:28 |
*** vmtrooper has joined #openstack-neutron | 22:28 | |
armax | amuller: what did I do that doesn’t please you? | 22:28 |
amuller | armax: You claim I have too much free time, your team mate is writing sonnets in commit messages :) | 22:29 |
armax | amuller: true, I shouldn’t single you out like I did | 22:29 |
amuller | heh, fair enough | 22:29 |
armax | amuller: but I blame salv-orlando all the time too | 22:31 |
*** Longgeek has quit IRC | 22:32 | |
*** vmtrooper has quit IRC | 22:33 | |
*** SridharRamaswamy has quit IRC | 22:35 | |
*** tiswanso has joined #openstack-neutron | 22:35 | |
*** csoukup has quit IRC | 22:35 | |
*** melwitt has joined #openstack-neutron | 22:36 | |
*** Marga_ has quit IRC | 22:36 | |
*** SridharRamaswamy has joined #openstack-neutron | 22:38 | |
*** thedodd has quit IRC | 22:39 | |
openstackgerrit | pritesh proposed openstack/neutron: Adding VLAN Transparency support for ML2 along with REST API changes https://review.openstack.org/158420 | 22:40 |
*** tiswanso has quit IRC | 22:40 | |
salv-orlando | amuller: what's your job title? | 22:41 |
amuller | code monkey probably | 22:41 |
*** aranjan has quit IRC | 22:41 | |
amuller | what's yours? | 22:41 |
salv-orlando | that does not entitle to you to waste time. | 22:41 |
salv-orlando | check it out for yourself: https://uk.linkedin.com/in/taturiello | 22:41 |
*** yfried|afk is now known as yfried | 22:42 | |
* b3nt_pin apologizes for eavesdropping while rofl | 22:42 | |
amuller | salv-orlando: Yeah definitely going to do something about my title next time I go to print cards | 22:42 |
amuller | Well played | 22:43 |
*** aranjan has joined #openstack-neutron | 22:44 | |
*** dims_ has joined #openstack-neutron | 22:46 | |
*** dims has quit IRC | 22:46 | |
*** SridharRamaswamy has quit IRC | 22:47 | |
*** SridharRamaswamy has joined #openstack-neutron | 22:47 | |
*** aranjan has quit IRC | 22:47 | |
*** aranjan_ has joined #openstack-neutron | 22:47 | |
*** dboik has quit IRC | 22:48 | |
*** dboik has joined #openstack-neutron | 22:48 | |
*** r-daneel has quit IRC | 22:48 | |
*** amotoki has joined #openstack-neutron | 22:49 | |
*** ajo has quit IRC | 22:51 | |
*** carl_baldwin has quit IRC | 22:51 | |
*** aranjan_ has quit IRC | 22:52 | |
*** yfried is now known as yfried|afk | 22:52 | |
*** achanda has quit IRC | 22:52 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Introduce RPC tests https://review.openstack.org/162811 | 22:53 |
amuller | armax: ^ | 22:53 |
*** achanda has joined #openstack-neutron | 22:53 | |
amuller | Changed to addCleanup and made some other simplifications | 22:53 |
kevinbenton | salv-orlando: here is the easy change from our discussion yesterday https://review.openstack.org/#/c/163672/ | 22:53 |
*** carl_baldwin has joined #openstack-neutron | 22:54 | |
*** amotoki has quit IRC | 22:55 | |
*** sridhar_ram has joined #openstack-neutron | 22:55 | |
*** baoli has quit IRC | 22:55 | |
*** thomasem has quit IRC | 22:55 | |
*** achanda_ has joined #openstack-neutron | 22:56 | |
*** annegentle has joined #openstack-neutron | 22:56 | |
*** achanda_ has quit IRC | 22:56 | |
*** achanda_ has joined #openstack-neutron | 22:56 | |
*** natarajk1 has left #openstack-neutron | 22:56 | |
*** sridhar_ram has left #openstack-neutron | 22:57 | |
*** achanda has quit IRC | 22:57 | |
*** jorgem has quit IRC | 22:58 | |
*** melwitt has quit IRC | 22:59 | |
*** dims_ has quit IRC | 23:00 | |
*** dims has joined #openstack-neutron | 23:00 | |
*** apuimedo has quit IRC | 23:01 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Start metadata agent without trying to connect db https://review.openstack.org/159365 | 23:01 |
*** annegentle has quit IRC | 23:02 | |
openstackgerrit | Shiv Haris proposed openstack/neutron: Brocade vendor code decomposition from neutron repo. https://review.openstack.org/152383 | 23:03 |
*** apuimedo has joined #openstack-neutron | 23:03 | |
*** ajmiller_ has quit IRC | 23:04 | |
*** melwitt has joined #openstack-neutron | 23:05 | |
*** dims has quit IRC | 23:05 | |
*** dboik has quit IRC | 23:05 | |
*** shwetaap has quit IRC | 23:06 | |
*** SridharRamaswamy has quit IRC | 23:08 | |
*** achanda has joined #openstack-neutron | 23:08 | |
*** SridharRamaswamy has joined #openstack-neutron | 23:08 | |
*** hichihara has joined #openstack-neutron | 23:09 | |
*** Marga_ has joined #openstack-neutron | 23:09 | |
*** SridharRamaswamy has quit IRC | 23:09 | |
*** matrohon has quit IRC | 23:10 | |
*** Marga_ has quit IRC | 23:10 | |
*** aranjan has joined #openstack-neutron | 23:10 | |
*** Marga_ has joined #openstack-neutron | 23:10 | |
*** achanda_ has quit IRC | 23:11 | |
*** igordcard_ has quit IRC | 23:12 | |
HenryG | armax: pong | 23:13 |
armax | HenryG: yo | 23:13 |
armax | HenryG: line https://github.com/openstack/neutron/blob/master/setup.cfg#L91 | 23:14 |
armax | HenryG: not sure if we should have entry points to external code like this | 23:15 |
armax | HenryG: what do you reckon? | 23:15 |
HenryG | armax: I see, yes, there should be a shim in tree | 23:15 |
armax | or no entry point at all | 23:16 |
armax | that could be added to the stackforge repo | 23:16 |
HenryG | armax: True. I can poke the owners to do one or the other. | 23:16 |
shivharis | armax: hi, need to clarify something. waiting for you to get free from the other conversation ... | 23:16 |
armax | shivharis: shoot | 23:16 |
shivharis | armax: refering to your comment in the patch-set, the 2 products are different may have different needs for invokation | 23:17 |
*** yamamoto has joined #openstack-neutron | 23:18 | |
shivharis | one depends on ml2 other does not | 23:18 |
armax | shivharis: that table is not the place to document vendor specific technologies | 23:18 |
armax | shivharis: I appreciate the differrence | 23:18 |
armax | shivharis: but we don’t need to capture those there | 23:18 |
armax | shivharis: imagine if all plugins start doing the same | 23:18 |
*** Marga_ has quit IRC | 23:18 | |
armax | HenryG’s employer especially | 23:18 |
armax | they have tons of different flavors | 23:18 |
*** Marga_ has joined #openstack-neutron | 23:19 | |
armax | shivharis: can you imagine if we start documenting those there? The table is going to bloat out of proportion | 23:19 |
*** crc32 has quit IRC | 23:20 | |
armax | shivharis: we should simply capture the flavor of plugin/driver being available in the repo | 23:20 |
armax | so that we know what to look for…as for the specific, user documentation and other resources are more appropriate tools to drive a user to know what options are available from a Brocade OpenStack solution | 23:21 |
shivharis | i am at a loss, need clarification: should the other patch-set update the vyatta column? | 23:21 |
armax | shivharis: where? | 23:22 |
shivharis | or should this distinction not be there at all? | 23:22 |
armax | you mean change 15906? | 23:22 |
armax | sorry 159056? | 23:22 |
shivharis | there are 2 product and 2 service plugins | 23:22 |
armax | I get it | 23:22 |
shivharis | yes 159056 | 23:22 |
armax | all I am saying, wrt that table | 23:22 |
armax | we simply do not need to care | 23:23 |
shivharis | ok, i'll remove the distinction | 23:23 |
armax | if you want to specify the various flavors of l3 plugins Brocade has | 23:23 |
armax | it’s best to put taht in the README | 23:23 |
armax | all we need to know from that table is that the L3 code that was once in the tree, is now avaialable out of tree | 23:24 |
armax | it’s that simple | 23:24 |
shivharis | sounds like a plan. will send update shortly. We will mainting this in the stackforge README.dm | 23:24 |
shivharis | s/dm/md/ | 23:24 |
armax | I think that’s best, thank you | 23:24 |
armax | shivharis: bear in mind the other comment on the entry point | 23:24 |
armax | if you plan on not having it, for consistency we’d want to remove the one from review 159056 | 23:25 |
armax | so perhaps sync up with your colleague to get that sorted | 23:25 |
shivharis | that worries me | 23:25 |
shivharis | they may have a different requirement for invokation.. we depend on ml2 they do not | 23:25 |
*** natarajk1 has joined #openstack-neutron | 23:26 | |
*** padkrish has quit IRC | 23:26 | |
armax | shivharis: do you understand what entry points are? | 23:27 |
*** padkrish has joined #openstack-neutron | 23:27 | |
*** julien_ZTE has quit IRC | 23:27 | |
armax | shivharis: I am not sure I fully grasp the implications of your statement | 23:28 |
shivharis | armax: yes | 23:28 |
armax | ok, so maybe I don't | 23:28 |
*** julien_ZTE has joined #openstack-neutron | 23:28 | |
armax | what has the invocation have to do with anything? | 23:28 |
shivharis | i am using that term instead of entry point | 23:29 |
*** yamamoto has quit IRC | 23:29 | |
armax | shivharis: ok, leave it as you wish | 23:29 |
armax | we’ll sort it out on the other patch | 23:30 |
shivharis | armax: ok | 23:31 |
sripriya | armax:hi | 23:31 |
armax | as much as I am trying to explain myself | 23:31 |
armax | shivharis: it seems like I am unable to get my message across | 23:32 |
armax | sripriya: hi | 23:32 |
*** achanda has quit IRC | 23:32 | |
*** padkrish has quit IRC | 23:32 | |
sripriya | we are just following the published guidelines that was referred in 159056 | 23:32 |
shivharis | ok, we have both plugins represented, shivharis and sripriya | 23:32 |
*** achanda has joined #openstack-neutron | 23:32 | |
*** sbfox has quit IRC | 23:32 | |
*** julien_ZTE has quit IRC | 23:33 | |
armax | sripriya, shivharis: yes, however I see two different approaches for the same vendor; albeit they are both acceptable IMO | 23:33 |
armax | i’d rather see one or the other in the context of the same vendor | 23:33 |
armax | now, shivharis’s l3 plugin is removed altogether, sripriya’s l3 plugin has a shim | 23:33 |
armax | to me this is confusing | 23:34 |
armax | as in the former case, obviosuly there’s no need of the entry point, in the latter there is | 23:34 |
armax | you with me? | 23:35 |
shivharis | armax: i appreciate your point but the implementation/deployments are dissimilar | 23:35 |
sripriya | armax: vyatta only has a new L3 plugin so we are doing a shim based on the guidelines | 23:35 |
*** absubram has quit IRC | 23:35 | |
*** melwitt has quit IRC | 23:35 | |
armax | shivharis: how so? | 23:36 |
shivharis | armax: whereas the vdx plugin has an ml2 component and an l3 plugin as well | 23:36 |
armax | shivharis: an l3 service plugin is an l3 service plugin | 23:36 |
armax | shivharis: ok, fine, keep it inconsistent | 23:36 |
*** mlavalle has joined #openstack-neutron | 23:36 | |
shivharis | correct, however devstack and other apps creating neutron.conf use a mechanism for this specific purpose | 23:37 |
armax | shivharis: if it was me, I wouldn’t do it in two separate ways, but I can’t force you | 23:37 |
armax | shivharis: be warned that if you came later and want to add the entry point, you’re sure gonna get shot down :) | 23:37 |
* armax kids | 23:38 | |
shivharis | armax: :-), that exactly why I like your approach | 23:38 |
armax | shivharis: I think you’ll change your mind when I slap a -2 on your patches | 23:38 |
sripriya | armax: just an FYI, we have few more new Brocade L3 plugins that are following vyatta approach: https://review.openstack.org/#/c/161489/7/setup.cfg | 23:39 |
shivharis | armax: you really want this changed? | 23:39 |
*** aepifanov has quit IRC | 23:39 | |
*** sridhar_ram has joined #openstack-neutron | 23:39 | |
armax | shivharis, sripriya the more the merrier, that makes me question why shivharis’ approach is an outlier | 23:40 |
armax | anyhow my energy in discussing this is ran out | 23:40 |
armax | thanks for listening for now :) | 23:40 |
*** david-lyle is now known as david-lyle_afk | 23:41 | |
shivharis | armax: what do you want me to do? | 23:41 |
armax | my suggestion would be to sort yourselves out so that the brocade’s contributions are cleaner | 23:41 |
armax | at this point, I am happy to let this go | 23:41 |
armax | I need to drink a gallon of water after all this talking :) | 23:42 |
shivharis | ok | 23:42 |
shivharis | i will let this stand, in the background think about it | 23:43 |
shivharis | however, i would also want to chat with you why i am using this approach | 23:43 |
shivharis | (at a later time) | 23:43 |
armax | shivharis: ok | 23:44 |
armax | sounds fair | 23:44 |
*** Sukhdev has quit IRC | 23:44 | |
shivharis | fair, thanks | 23:44 |
openstackgerrit | Shiv Haris proposed openstack/neutron: Brocade vendor code decomposition from neutron repo. https://review.openstack.org/152383 | 23:47 |
*** sripriya has quit IRC | 23:50 | |
*** dboik has joined #openstack-neutron | 23:52 | |
*** sripriya has joined #openstack-neutron | 23:52 | |
*** yamamoto has joined #openstack-neutron | 23:52 | |
*** dboik_ has joined #openstack-neutron | 23:52 | |
*** sripriya has quit IRC | 23:52 | |
*** padkrish has joined #openstack-neutron | 23:53 | |
*** shwetaap has joined #openstack-neutron | 23:54 | |
*** reed has quit IRC | 23:55 | |
*** ChuckC has quit IRC | 23:56 | |
*** dboik has quit IRC | 23:56 | |
*** annegentle has joined #openstack-neutron | 23:58 | |
*** dannywil_ has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!