Monday, 2018-11-19

*** slaweq has joined #openstack-neutron00:11
*** slaweq has quit IRC00:24
*** longkb has joined #openstack-neutron00:53
openstackgerritMiguel Lavalle proposed openstack/neutron master: Define qos-rules-alias extension  https://review.openstack.org/61382000:56
*** longkb has quit IRC00:57
*** slaweq has joined #openstack-neutron01:16
*** jaewook_oh has joined #openstack-neutron01:21
*** slaweq has quit IRC01:24
*** zhanglong has joined #openstack-neutron01:29
*** cshen has joined #openstack-neutron01:35
*** cshen has quit IRC01:39
*** yamamoto has joined #openstack-neutron01:51
*** yamamoto has quit IRC01:55
*** dconde has joined #openstack-neutron02:10
*** slaweq has joined #openstack-neutron02:16
*** slaweq has quit IRC02:24
*** ihar has quit IRC02:24
*** ihar has joined #openstack-neutron02:25
*** tbachman has quit IRC02:27
*** sridharg has joined #openstack-neutron02:29
*** sambetts_ has quit IRC02:35
*** sambetts_ has joined #openstack-neutron02:39
*** annp has joined #openstack-neutron02:54
*** Chenjie has joined #openstack-neutron03:00
*** Chenjie has quit IRC03:05
*** slaweq has joined #openstack-neutron03:11
*** tbachman has joined #openstack-neutron03:19
*** maddtux has joined #openstack-neutron03:19
openstackgerritDeepak proposed openstack/python-neutronclient master: Add support for TaaS in NeutronClient  https://review.openstack.org/44918803:22
*** slaweq has quit IRC03:25
*** blake has joined #openstack-neutron03:44
*** ramishra has joined #openstack-neutron03:56
*** ramishra_ has joined #openstack-neutron03:58
*** ramishra has quit IRC04:01
*** ramishra_ is now known as ramishra04:10
*** slaweq has joined #openstack-neutron04:13
*** ramishra has quit IRC04:21
*** janki has joined #openstack-neutron04:24
*** slaweq has quit IRC04:25
*** ramishra has joined #openstack-neutron04:29
*** blake has quit IRC04:30
*** ivve has joined #openstack-neutron04:37
*** njohnston has quit IRC04:47
*** njohnston has joined #openstack-neutron04:48
*** ircuser-1 has joined #openstack-neutron04:51
*** slaweq has joined #openstack-neutron05:16
*** slaweq has quit IRC05:24
*** kailun_ has joined #openstack-neutron05:51
*** ratailor has joined #openstack-neutron05:56
openstackgerritzhufl proposed openstack/neutron master: Add missing ws seperator between words  https://review.openstack.org/61867905:59
*** ramishra_ has joined #openstack-neutron06:04
*** ramishra has quit IRC06:07
*** moshele has joined #openstack-neutron06:10
*** slaweq has joined #openstack-neutron06:11
*** moshele has quit IRC06:13
*** kailun_ has quit IRC06:14
*** slaweq has quit IRC06:24
*** Dinesh_Bhor has joined #openstack-neutron06:25
*** Luzi has joined #openstack-neutron07:03
*** ratailor has quit IRC07:03
*** ratailor has joined #openstack-neutron07:05
*** ccamposr has joined #openstack-neutron07:08
*** ramishra_ is now known as ramishra07:12
*** lajoskatona has joined #openstack-neutron07:13
*** mosulica has joined #openstack-neutron07:18
*** pcaruana has joined #openstack-neutron07:24
*** mosulica has quit IRC07:25
*** trident has quit IRC07:29
*** trident has joined #openstack-neutron07:31
*** abaindur has quit IRC07:33
*** lajoskatona has left #openstack-neutron07:40
*** maciejjozefczyk has joined #openstack-neutron07:41
*** velizarx has joined #openstack-neutron07:44
*** mosulica has joined #openstack-neutron07:46
*** lajoskatona has joined #openstack-neutron07:50
*** yamamoto has joined #openstack-neutron07:53
*** hoonetorg has quit IRC07:55
*** yamamoto has quit IRC07:58
*** gkadam has joined #openstack-neutron07:58
*** Dinesh_Bhor has quit IRC08:00
*** yamamoto has joined #openstack-neutron08:01
*** aojea has joined #openstack-neutron08:05
*** jangutter has joined #openstack-neutron08:06
*** slaweq has joined #openstack-neutron08:11
*** hoonetorg has joined #openstack-neutron08:13
*** slaweq has quit IRC08:16
*** yamamoto has quit IRC08:23
*** tmorin has joined #openstack-neutron08:24
*** jlibosva has joined #openstack-neutron08:37
*** Dinesh_Bhor has joined #openstack-neutron08:40
*** yamamoto has joined #openstack-neutron08:47
*** jpena|off is now known as jpena08:48
openstackgerritFederico Ressi proposed openstack/neutron-tempest-plugin master: Add ip command tool for manually manage devices on VM.  https://review.openstack.org/61331008:48
*** pguimaraes has quit IRC08:56
*** cshen has joined #openstack-neutron08:57
openstackgerritFederico Ressi proposed openstack/neutron-tempest-plugin master: Add ip command tool for manually manage devices on VM.  https://review.openstack.org/61331008:58
openstackgerritFederico Ressi proposed openstack/neutron-tempest-plugin master: Add ip command tool for manually manage devices on VM.  https://review.openstack.org/61331009:00
openstackgerritFederico Ressi proposed openstack/neutron-tempest-plugin master: Update test_trunk after adding new ip command tool.  https://review.openstack.org/61332609:02
*** s10 has joined #openstack-neutron09:10
*** kukacz has quit IRC09:10
*** rossella_s has joined #openstack-neutron09:11
*** slaweq has joined #openstack-neutron09:11
*** kukacz has joined #openstack-neutron09:13
*** pguimaraes has joined #openstack-neutron09:25
*** tssurya has joined #openstack-neutron09:42
*** maciejjozefczyk has quit IRC09:50
openstackgerritSlawek Kaplonski proposed openstack/neutron master: Duplicate words were deleted in openvswitch_agent.rst  https://review.openstack.org/61755609:54
openstackgerritLIU Yulong proposed openstack/neutron master: Check if agent can reach neutron server  https://review.openstack.org/61872010:02
*** sridharg has quit IRC10:05
*** k_mouza has joined #openstack-neutron10:06
*** eglute has quit IRC10:13
*** eglute has joined #openstack-neutron10:13
*** sean-k-mooney has quit IRC10:15
*** Dinesh_Bhor has quit IRC10:17
*** mosulica has quit IRC10:18
*** sean-k-mooney has joined #openstack-neutron10:23
*** zhanglong has quit IRC10:26
sean-k-mooneyslaweq: o/10:26
slaweqsean-k-mooney: hi10:26
*** betherly has joined #openstack-neutron10:27
sean-k-mooneyslaweq: regarding https://review.openstack.org/#/c/602384 i was wondering what you opipion would be regarding enableing this by default in devstack? i cant enable it by default in os-vif but do you think it would be worth enableing by default in devstack when using ml2/ovs10:28
sean-k-mooneysorry this https://review.openstack.org/#/c/61253410:28
sean-k-mooneye.g. the isolate config option10:28
*** rubasov has joined #openstack-neutron10:29
slaweqsean-k-mooney: I will take a look when I will finish checking something with kforde10:29
sean-k-mooneyi plan to add more gate jobs to os-vif before the end of the year so i will test both values but wanted some neutron input on the devstack default10:29
sean-k-mooneyslaweq: sure no rush10:30
sean-k-mooneyim going to ask the nova folks too10:30
*** lucasagomes has joined #openstack-neutron10:30
*** pguimaraes has quit IRC10:32
*** nplanel_ has joined #openstack-neutron10:33
*** maciejjozefczyk has joined #openstack-neutron10:35
*** moshele has joined #openstack-neutron10:36
*** yamamoto has quit IRC10:36
openstackgerritFederico Ressi proposed openstack/neutron-tempest-plugin master: Update test_trunk after adding new ip command tool.  https://review.openstack.org/61332610:36
*** panda|rover|off is now known as panda|rover10:49
openstackgerritFederico Ressi proposed openstack/neutron-tempest-plugin master: Update test_trunk after adding new ip command tool.  https://review.openstack.org/61332610:58
*** cshen has quit IRC10:58
openstackgerritFederico Ressi proposed openstack/neutron-tempest-plugin master: [WIP] Refactor security groups scenario test  https://review.openstack.org/61581310:59
slaweqsean-k-mooney: if You will set this to True in devstack, You will have to update all jobs in projects like networking-odl and others which can't use it, right?11:05
slaweqsean-k-mooney: if so, maybe it's better to change it to True only in neutron jobs and left all others as they are now11:06
*** yamamoto has joined #openstack-neutron11:07
sean-k-mooneyslaweq: i was considering setting it to true by default only if the ml2 ovs agent was deployed11:14
sean-k-mooneyso for odl or other backend it would be still false11:14
sean-k-mooneyi could also just set it to true for select jobs but as we will be recommending that people set it to true when using ml2 ovs going forward i wanted to make sure we tested with that setting when correct to do so11:15
sean-k-mooneybut i also what to test without it when not correct to ensure we do the right thing in both cases11:16
*** janki has quit IRC11:18
*** rodolof has joined #openstack-neutron11:29
*** cshen has joined #openstack-neutron11:31
*** jaewook_oh has quit IRC11:33
*** moshele has quit IRC11:34
*** cshen has quit IRC11:36
*** k_mouza has quit IRC11:38
*** k_mouza has joined #openstack-neutron11:39
*** moshele has joined #openstack-neutron11:40
slaweqsean-k-mooney: making it default for ml2/ovs only would be fine IMO11:40
*** moshele has quit IRC11:44
*** sridharg has joined #openstack-neutron11:44
*** janki has joined #openstack-neutron11:49
*** njohnston has quit IRC11:50
*** njohnston has joined #openstack-neutron11:51
*** ramishra has quit IRC12:09
*** yamamoto has quit IRC12:10
*** ramishra has joined #openstack-neutron12:13
*** cshen has joined #openstack-neutron12:17
*** moshele has joined #openstack-neutron12:17
*** cshen has quit IRC12:22
*** goutham1 has joined #openstack-neutron12:22
*** moshele has quit IRC12:23
*** velizarx has quit IRC12:24
*** ratailor has quit IRC12:24
*** rodolof has quit IRC12:29
*** rodolof has joined #openstack-neutron12:30
*** jpena is now known as jpena|lunch12:31
openstackgerritFederico Ressi proposed openstack/neutron-tempest-plugin master: Update test_trunk after adding new ip command tool.  https://review.openstack.org/61332612:39
*** yamamoto has joined #openstack-neutron12:40
*** nplanel__ has joined #openstack-neutron12:42
*** nplanel_ has quit IRC12:45
*** velizarx has joined #openstack-neutron12:53
*** mriedem has joined #openstack-neutron12:55
*** amuller has joined #openstack-neutron12:55
*** maddtux has quit IRC12:55
*** moshele has joined #openstack-neutron12:56
*** dave-mccowan has joined #openstack-neutron12:57
*** cshen has joined #openstack-neutron12:58
*** ondrejme has joined #openstack-neutron12:59
*** moshele has quit IRC13:09
*** sapd1 has joined #openstack-neutron13:13
*** k_mouza has quit IRC13:14
*** tmorin has quit IRC13:14
*** boden has joined #openstack-neutron13:16
*** nplanel__ has quit IRC13:17
openstackgerritLajos Katona proposed openstack/neutron-tempest-plugin master: tests for port-resource-request  https://review.openstack.org/59842313:19
*** tbachman has quit IRC13:20
*** aojea_ has joined #openstack-neutron13:29
*** jpena|lunch is now known as jpena13:29
*** aojea_ has quit IRC13:30
*** aojea_ has joined #openstack-neutron13:31
*** k_mouza has joined #openstack-neutron13:31
*** k_mouza_ has joined #openstack-neutron13:33
openstackgerritSlawek Kaplonski proposed openstack/neutron master: Get centralized FIP only on router's snat host  https://review.openstack.org/61875013:33
*** aojea_ has quit IRC13:35
openstackgerritMerged openstack/ovsdbapp master: Add WaitEvent to the API  https://review.openstack.org/61312113:35
*** k_mouza has quit IRC13:36
*** dave-mccowan has quit IRC13:36
*** nplanel_ has joined #openstack-neutron13:37
openstackgerritLucas Alvares Gomes proposed openstack/ovsdbapp master: Fix functional tests timeout race condition  https://review.openstack.org/61875213:43
*** tbachman has joined #openstack-neutron13:45
*** mvkr has quit IRC13:47
*** tbachman_ has joined #openstack-neutron13:48
*** moshele has joined #openstack-neutron13:50
*** tbachman has quit IRC13:51
*** tbachman_ is now known as tbachman13:51
*** moshele has quit IRC13:54
*** tmorin has joined #openstack-neutron13:57
*** aojea_ has joined #openstack-neutron13:58
*** moshele has joined #openstack-neutron14:03
openstackgerritFederico Ressi proposed openstack/neutron-tempest-plugin master: Update test_trunk after adding new ip command tool.  https://review.openstack.org/61332614:04
*** moshele has quit IRC14:06
openstackgerritMiguel Angel Ajo proposed openstack/networking-ovn master: Migration document update.  https://review.openstack.org/60972114:08
openstackgerritSlawek Kaplonski proposed openstack/neutron-tempest-plugin master: Remove unstable_test from FIP test_east_west  https://review.openstack.org/61855714:11
*** aojea_ has quit IRC14:31
*** aojea_ has joined #openstack-neutron14:32
*** k_mouza has joined #openstack-neutron14:34
*** mvkr has joined #openstack-neutron14:35
*** aojea_ has quit IRC14:36
*** k_mouza_ has quit IRC14:37
*** mchlumsky has joined #openstack-neutron14:43
openstackgerritLakshmi Prasanna Goutham Pratapa proposed openstack/neutron master: Add get_availability_zone rule to policy.json  https://review.openstack.org/61854914:46
*** nplanel_ has quit IRC14:52
*** nplanel_ has joined #openstack-neutron14:52
*** mlavalle has joined #openstack-neutron15:00
*** k_mouza_ has joined #openstack-neutron15:01
*** sapd1 has quit IRC15:02
*** k_mouza__ has joined #openstack-neutron15:02
*** Luzi has quit IRC15:03
*** k_mouza has quit IRC15:04
*** k_mouza_ has quit IRC15:06
*** sapd1 has joined #openstack-neutron15:09
*** slaweq_ has joined #openstack-neutron15:10
*** slaweq has quit IRC15:12
*** velizarx has quit IRC15:12
*** k_mouza__ has quit IRC15:17
openstackgerritDaniel Alvarez proposed openstack/networking-ovn master: Metadata agent: fetch ovn-bridge from OVSDB and not from config  https://review.openstack.org/61240615:18
*** ijw has quit IRC15:18
openstackgerritMerged openstack/networking-ovn master: Migration document update.  https://review.openstack.org/60972115:19
*** sapd1 has quit IRC15:20
openstackgerritboden proposed openstack/neutron master: remove unused methods from CommonDbMixin  https://review.openstack.org/61700415:24
openstackgerritDaniel Alvarez proposed openstack/networking-ovn master: metadata-agent: Remove 'ovs_integration_bridge' option  https://review.openstack.org/61876715:26
*** gkadam has quit IRC15:31
openstackgerritMerged openstack/neutron-lib master: Add ipv6 canonical address converter to API  https://review.openstack.org/49328015:35
openstackgerritFilippo Inzaghi proposed openstack/neutron-lib master: Change python3.5 job to python3.7 job on Stein+  https://review.openstack.org/61095515:39
*** k_mouza has joined #openstack-neutron15:40
*** moshele has joined #openstack-neutron15:43
*** moshele has quit IRC15:52
*** slaweq_ has quit IRC15:54
*** slaweq_ has joined #openstack-neutron15:54
*** yamamoto has quit IRC15:55
*** dklyle has quit IRC15:55
*** yamamoto has joined #openstack-neutron15:56
*** lajoskatona has left #openstack-neutron15:58
*** mosulica has joined #openstack-neutron16:01
*** dklyle has joined #openstack-neutron16:03
*** tbachman has quit IRC16:04
*** pcaruana has quit IRC16:07
*** moshele has joined #openstack-neutron16:08
*** s10 has quit IRC16:11
*** moshele has quit IRC16:14
*** goutham1 has quit IRC16:17
openstackgerritNate Johnston proposed openstack/neutron master: [DNR] test devstack conversion from bridge-utils to iproute2  https://review.openstack.org/61877616:17
*** jamesdenton has joined #openstack-neutron16:17
*** janki has quit IRC16:26
*** mosulica has quit IRC16:29
openstackgerritNate Johnston proposed openstack/neutron master: Add bulk IP address assignment to ipam driver  https://review.openstack.org/59216416:30
*** mchlumsky has quit IRC16:31
*** mosulica has joined #openstack-neutron16:33
*** mchlumsky has joined #openstack-neutron16:34
*** cshen has quit IRC16:36
*** tbachman has joined #openstack-neutron16:36
*** mosulica has quit IRC16:36
*** velizarx has joined #openstack-neutron16:43
*** sapd1 has joined #openstack-neutron16:45
*** rodolof has quit IRC16:51
*** moshele has joined #openstack-neutron16:52
*** moshele has quit IRC16:53
*** k_mouza_ has joined #openstack-neutron16:57
*** tmorin has quit IRC16:58
*** k_mouza__ has joined #openstack-neutron16:58
*** k_mouza has quit IRC17:00
*** k_mouza_ has quit IRC17:02
*** aojea has quit IRC17:02
*** cshen has joined #openstack-neutron17:04
*** dsneddon has joined #openstack-neutron17:04
*** k_mouza__ has quit IRC17:04
*** tssurya has quit IRC17:07
*** srini_ has quit IRC17:07
*** sapd1 has quit IRC17:07
*** velizarx has quit IRC17:07
*** ccamposr has quit IRC17:08
*** itlinux has joined #openstack-neutron17:12
*** ijw has joined #openstack-neutron17:15
*** imacdonn has quit IRC17:17
*** imacdonn has joined #openstack-neutron17:17
*** ijw has quit IRC17:18
*** tmorin has joined #openstack-neutron17:19
*** k_mouza has joined #openstack-neutron17:24
*** salmankhan has joined #openstack-neutron17:29
*** jlibosva has quit IRC17:34
*** rodolof has joined #openstack-neutron17:36
*** cshen has quit IRC17:36
*** salmankhan has quit IRC17:36
*** cshen has joined #openstack-neutron17:37
*** rodolof has quit IRC17:40
*** rodolof has joined #openstack-neutron17:40
*** rodolof has quit IRC17:41
*** rodolof has joined #openstack-neutron17:41
*** tmorin has quit IRC17:43
*** pcaruana has joined #openstack-neutron17:44
*** k_mouza has quit IRC17:46
*** yamahata has quit IRC17:49
*** sridharg has quit IRC17:49
*** rodolof has quit IRC17:51
*** rodolof has joined #openstack-neutron17:51
*** rodolof has quit IRC17:56
*** rodolof has joined #openstack-neutron17:56
*** dsneddon has quit IRC17:58
*** cshen has quit IRC18:02
*** jpena is now known as jpena|off18:03
*** tbachman has quit IRC18:07
*** dsneddon has joined #openstack-neutron18:14
*** amuller has quit IRC18:19
*** yamahata has joined #openstack-neutron18:25
*** dsneddon has quit IRC18:27
*** rodolof has quit IRC18:28
*** tbachman has joined #openstack-neutron18:29
openstackgerritMerged openstack/neutron master: Add test cases for port number 0 for port_forwarding  https://review.openstack.org/61439418:30
openstackgerritMerged openstack/neutron master: Duplicate words were deleted in openvswitch_agent.rst  https://review.openstack.org/61755618:31
*** k_mouza has joined #openstack-neutron18:49
*** k_mouza has quit IRC18:54
njohnstontidwellr_: Regarding your comment here, do you think I need to define a new type of exception?  https://review.openstack.org/#/c/585028/14/neutron/db/db_base_plugin_v2.py,unified19:00
openstackgerritMerged openstack/networking-odl master: devstack: enable flow based tunnels for sfc  https://review.openstack.org/61415519:01
*** hjensas has quit IRC19:03
openstackgerritSlawek Kaplonski proposed openstack/neutron master: Get centralized FIP only on router's snat host  https://review.openstack.org/61875019:05
*** cshen has joined #openstack-neutron19:11
*** Swami has joined #openstack-neutron19:13
Swami@haleyb: Quick question, do you know where in neutron agent/linux we can pass in or add an argument for SNAT using iptables extension. (such as 'random' and 'random-fully')19:14
*** cshen has quit IRC19:19
*** mvkr has quit IRC19:20
Swami@haleyb: I assume the rules are hard-coded with the required arguments  as in here. https://github.com/openstack/neutron/blob/master/neutron/agent/l3/router_info.py#L85119:23
haleybSwami: yes, that's all done in the agent19:24
Swamihaleyb: ok thanks.19:24
tidwellr_njohnston: Let me look at the exceptions we have :)19:28
*** moshele has joined #openstack-neutron19:29
*** abaindur has joined #openstack-neutron19:31
tidwellr_njohnston: After taking a quick look over the exceptions we already have, a new type of exception may make sense. The alternative is to raise a BadRequest with a friendly message, but I'm partial to the idea of having specific type for this sort of thing19:35
*** moshele has quit IRC19:43
*** cshen has joined #openstack-neutron19:48
njohnstontidwellr_: Sounds good to me, I just wanted to sanity check that it was a good idea.  Thanks!19:51
*** cshen has quit IRC19:52
*** gianpietro has joined #openstack-neutron19:54
*** aojea has joined #openstack-neutron19:56
*** hjensas has joined #openstack-neutron19:58
*** jobewan has joined #openstack-neutron19:59
*** rodolof has joined #openstack-neutron20:00
*** aojea has quit IRC20:01
abaindurDHCP agent is releasing all leases and causing a storm of DHCPRELEASE, whenever a single port or two is deleted20:06
abaindurit seems to think every port is stale, in  _release_unused_leases()20:06
abaindurAre there any known issues around this?20:07
*** mvkr has joined #openstack-neutron20:09
*** mchlumsky has quit IRC20:10
*** mchlumsky has joined #openstack-neutron20:11
*** rodolof has quit IRC20:11
*** rodolof has joined #openstack-neutron20:12
*** rodolof has quit IRC20:16
*** rodolof has joined #openstack-neutron20:17
*** gianpietro has quit IRC20:19
*** gianpietro has joined #openstack-neutron20:19
*** gianpietro has quit IRC20:22
*** gianpietro has joined #openstack-neutron20:22
*** rodolof has quit IRC20:26
*** rodolof has joined #openstack-neutron20:27
*** rodolof has quit IRC20:31
*** aojea has joined #openstack-neutron20:32
*** rodolof has joined #openstack-neutron20:32
haleybabaindur: i don't know of any issues there, if you can reproduce it please file a bug with the file contents and related information20:36
abaindurI suspect the network cache is empty or incomplete20:36
abaindurI am adding more logging around that area to print the cache's network.ports20:36
abaindurentries_to_release = (v4_leases | old_leases) - new_leases20:37
*** cshen has joined #openstack-neutron20:37
abaindurIt seems to me it would only try to release a bunch of entries for ACTIVE ports, if new_leases was empty or incomplete20:37
abaindurhaleyb: i've been trying to figure out exactly how this is populated, can you confirm:20:37
abaindur1. when agent is started, it fully p[opulates the network's current ports20:38
abaindur2. when it received a port_update_end notification, it adds the port the network cache20:38
abainduris that it...?20:38
*** rodolof has quit IRC20:38
*** rodolof has joined #openstack-neutron20:39
abaindur3. Also when it gets a network_update or subnet_update, during which it re-queries via RPC to fully populates network ports20:39
abaindurI cant seem to find anything else that triggers updating the network cache's ports20:40
*** dsneddon has joined #openstack-neutron20:40
*** rodolof has quit IRC20:41
*** rodolof has joined #openstack-neutron20:42
haleybabaindur: when a network is removed it will update the cache as well, network_delete_end20:45
abaindurright, no networks being deleted here. Just some VM churn20:47
abaindurThere are quite a lot of ports on the network - 1600 or so20:47
*** cshen has quit IRC20:51
*** rodolof has quit IRC20:56
*** cshen has joined #openstack-neutron20:57
*** dconde has quit IRC20:58
*** dconde has joined #openstack-neutron21:00
*** abaindur has quit IRC21:00
*** abaindur has joined #openstack-neutron21:01
haleybabaindur: i don't know then, it shouldn't be sending releases for known good IPs21:05
*** rmart04 has joined #openstack-neutron21:18
*** rmart04 has quit IRC21:19
*** dsneddon has quit IRC21:19
*** dsneddon has joined #openstack-neutron21:19
*** larsks has joined #openstack-neutron21:30
*** dsneddon has quit IRC21:37
*** dsneddon has joined #openstack-neutron21:38
openstackgerritNate Johnston proposed openstack/neutron master: Add bulk port creation of DB objects  https://review.openstack.org/58502821:40
*** cshen has quit IRC21:43
*** dsneddon has quit IRC21:46
*** jobewan has quit IRC21:47
*** dsneddon has joined #openstack-neutron21:49
*** pcaruana has quit IRC21:51
*** gianpietro has quit IRC21:52
*** gianpietro has joined #openstack-neutron21:53
abaindurhaleyb: I think we found the issue21:55
abaindurthe client_id field in the leases file is getting updated from a * to something like 01:<mac address>21:55
abaindurentries_to_release = (v4_leases | old_leases) - new_leases21:56
*** ivve has quit IRC21:56
abaindurthe client_id in new_leases (obtained from self.network.ports) is None, and so is client_id in the old_leases tuple (obtained from host file)21:56
abaindurv4_leases is not None, therefore "- new leases" does not filter the entry out21:57
*** dsneddon has quit IRC21:57
haleybabaindur: is this a newer version of dnsmasq?  or did we just not take this into account?21:58
abaindur2.7621:58
abaindurHeres strange thing though. at one point in time, the client_id in leases file, is a * (the last entry in each line in leases file)21:59
abaindurover time, few ports slowly get their client id updated somehow, and leases file changes from * to 01:MAC22:00
abaindurWhat might cause client_id in leases file ot get changed? I dont see/think anything in neutron or dhcp-agent that would do so. Something the VM is doing perhaps?22:01
*** dsneddon has joined #openstack-neutron22:02
haleybthat is written by dnsmasq, right?  agent might just try and populate for initial startup22:02
abaindurThere are no port updates affecting the ports that change, so somehow dnsmasq is changing it on its own22:02
abainduryea, i think we populate it on startup for bootstrapping22:02
abaindurother than that we invoke dhcp_release when VM is cleaned up22:03
haleybi guess we should verify dnsmasq updates it, then try to account for it.  at least we have the info to match it22:04
openstackgerritMerged openstack/neutron-specs master: Add placeholder for Train specs  https://review.openstack.org/61787322:05
*** cshen has joined #openstack-neutron22:07
abaindurIt is most definitely updating it... dont see anything dhcp-agent is doing to write the file.22:08
abaindurim not too familiar with client-id field... why is it included in the lease tuple first place?22:09
abaindurwhen would it ever not be None as per the host file or from the network cache?22:09
*** dsneddon has quit IRC22:10
*** HenryG has quit IRC22:12
haleybabaindur: there is a comment in the code that maybe helps...22:12
haleyb# all that matters is the mac address and IP. the hostname and22:12
haleyb            # client ID will be overwritten on the next renewal.22:12
haleybi think for ipv6 the client-id is not the MAC, but something unique, my RFC memory escapes me22:12
*** HenryG has joined #openstack-neutron22:14
*** dsneddon has joined #openstack-neutron22:14
haleyband i guess client-id can also be specified in the port as an extra option22:16
*** dsneddon has quit IRC22:22
*** dsneddon has joined #openstack-neutron22:22
abaindurhaleyb: right, thats why im not sure why client id is inluded in tuple at all?22:24
abaindur('10.81.96.186', 'fa:16:3e:eb:a1:13', None)22:24
abaindurvs22:24
abaindur('10.81.96.186', 'fa:16:3e:eb:a1:13', '01:fa:16:3e:eb:a1:13')22:25
haleybthere must be a minimum number of fields required in the leases file?22:25
abaindurthat tuple is formed after parsing it, wondering why its included in it at all?22:25
abaindurthere are more than 3 fields per line in leases file22:25
abaindurleases file entry looks actually like:22:26
haleybabaindur: the problem is that the release code will correctly insert the client-id from the leases file, but self._get_client_id() will only ever return the extra opt value, so that is the disconnect22:26
abaindur1542749043 fa:16:3e:f7:11:af 10.81.96.225 test-vm-l-10-81-96-225 *22:26
haleyband that last * can turn into the 01:MAC right?22:26
abaindur*something* over time is updating the leases file entry there from a * to 01:fa:16:3e:f7:11:af22:26
abainduryes22:26
haleybi think it's the first update/request from the client22:27
abainduryou mean a DHCP packet?22:27
abainduror on the neutron control plane for the port (we are not seeing any port updates)22:28
haleybdnsmasq getting a DHCP from the instance, it must then update the leases file with it?22:28
abainduryea we arent sure why dnsmasq is updating the client id however22:29
haleybthe note in our file says it will be overwritten on lease renewal22:29
haleybabaindur: the (possibly) simple fix is in the loop where we do new_leases.add() - if there is no client_id specified, and this is IPv4, then we should set the client_id to 01:MAC - that should match the leases file in this case.  of course it doesn't account for the case where it is still * in the file22:31
*** abaindur_ has joined #openstack-neutron22:32
haleybso we almost have to add two entries, then filter the "extra" out if that makes sense22:32
abaindur_sorry lost internet here, missed your last msg(s)22:33
haleyblet me replay22:33
haleybthe note in our file says it will be overwritten on lease renewal22:33
haleybabaindur: the (possibly) simple fix is in the loop where we do new_leases.add() - if there is no client_id specified, and this is IPv4, then we should set the client_id to 01:MAC - that should match the leases file in this case.  of course it doesn't account for the case where it is still * in the file22:33
haleybso we almost have to add two entries, then filter the "extra" out if that makes sense22:33
*** abaindur has quit IRC22:33
abaindur_yea new_leases set would need a tuples with both None and 01:MAC22:34
*** nplanel__ has joined #openstack-neutron22:34
*** boden has quit IRC22:34
abaindur_For what its worth, these machines do have ipv6 enabled22:35
abaindur_but we are not using ipv6 actually22:35
haleybbut if it has both and one matches, the other will still trigger a release22:35
haleybs/we add both22:35
haleybit's tricky22:35
abaindur_we are not seeing 2 entries in leases file for None/Mac client id22:36
openstackgerritboden proposed openstack/neutron master: remove profiler opt setup in db api  https://review.openstack.org/61669422:37
haleybright, but if we add two entries to new_leases, one gets filtered, the other triggers a release22:37
*** dklyle has quit IRC22:37
*** slaweq__ has joined #openstack-neutron22:37
*** dayou_ has joined #openstack-neutron22:37
haleybabaindur_: perhaps the best thing is to always write the initial leases file with the correct client-id and not *22:37
abaindur_That would cause an even larger storm of DHCPRELEASES22:38
abaindur_right...?22:38
*** dklyle has joined #openstack-neutron22:38
*** wznoinsk has joined #openstack-neutron22:38
haleybit shouldn't, assuming that we always construct it as dnsmasq would re-write it22:39
*** dsneddon has quit IRC22:39
haleybi don't know if using the * was just a quick hack, or if the 01:MAC is standard for dnsmasq22:39
*** tbachman has quit IRC22:39
*** slaweq_ has quit IRC22:39
*** dayou has quit IRC22:39
*** bcafarel has quit IRC22:39
*** imacdonn has quit IRC22:39
*** nplanel_ has quit IRC22:39
*** mriedem has quit IRC22:39
*** sambetts_ has quit IRC22:39
*** annp has quit IRC22:39
*** jmlowe has quit IRC22:39
*** manjeets has quit IRC22:39
haleybor if your clients are just advanced and sending a client-id back?22:40
haleybit would require looking at the dnsmasq code, or asking on their mailing list22:40
*** tbachman has joined #openstack-neutron22:41
*** bcafarel has joined #openstack-neutron22:41
*** imacdonn has joined #openstack-neutron22:41
*** jmlowe has joined #openstack-neutron22:42
*** dsneddon has joined #openstack-neutron22:42
abaindur_we think it may be VMs in particular or something with dhcpv6?22:43
*** sambetts_ has joined #openstack-neutron22:43
haleybi don't know, will require further investigation22:44
*** abaindur has joined #openstack-neutron22:44
haleybbut please file a bug with all the info we discussed, i can look in the dnsmasq code later, or ask their devs22:44
* haleyb must run, thanks abaindur_ for all your work22:46
abaindurIt seems to just be these VMs, at least as far as i can tell.... they are windows with ipv6....22:46
abaindurNp, thanks for the help22:46
abaindurbut basically end result is when any other port or VM is deleted, all these other ports get a storm of DHCPRELEASE.22:47
*** abaindur_ has quit IRC22:48
*** mlavalle has quit IRC23:16
*** njohnston has quit IRC23:29
*** njohnston has joined #openstack-neutron23:30
*** gibi has quit IRC23:34
*** gibi has joined #openstack-neutron23:35
*** manjeets has joined #openstack-neutron23:36
*** aojea has quit IRC23:39
*** gianpietro has quit IRC23:39
*** gianpietro has joined #openstack-neutron23:40
openstackgerritMerged openstack/neutron-lib master: fix DBResourceExtendFixture fixture  https://review.openstack.org/61663123:40
*** gianpietro has quit IRC23:45
*** hjensas has quit IRC23:47
*** klindgren has joined #openstack-neutron23:49
*** nplanel__ has quit IRC23:50
*** hjensas has joined #openstack-neutron23:52

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