| *** trad511 has quit IRC | 00:00 | |
| *** alexpilotti has quit IRC | 00:01 | |
| *** claudiub has joined #openstack-neutron | 00:02 | |
| *** dave-mcc_ has joined #openstack-neutron | 00:02 | |
| *** alexpilotti has joined #openstack-neutron | 00:02 | |
| *** alexpilotti has quit IRC | 00:03 | |
| *** yamamoto has joined #openstack-neutron | 00:04 | |
| *** dave-mccowan has quit IRC | 00:04 | |
| *** jp_at_hp has joined #openstack-neutron | 00:04 | |
| *** alexpilotti has joined #openstack-neutron | 00:04 | |
| *** padkrish has joined #openstack-neutron | 00:05 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Reuse nova batch notifier https://review.openstack.org/128013 | 00:05 |
|---|---|---|
| *** alexpilotti has quit IRC | 00:06 | |
| *** shaohe_feng1 has quit IRC | 00:06 | |
| *** shz has joined #openstack-neutron | 00:07 | |
| *** jamielennox is now known as jamielennox|away | 00:09 | |
| *** liusheng has quit IRC | 00:10 | |
| *** hanzhang has quit IRC | 00:10 | |
| *** lykinsbd has joined #openstack-neutron | 00:10 | |
| *** liusheng has joined #openstack-neutron | 00:10 | |
| *** SumitNaiksatam has quit IRC | 00:11 | |
| *** vmtrooper has joined #openstack-neutron | 00:11 | |
| *** dims__ is now known as dims | 00:13 | |
| *** tidwellr has left #openstack-neutron | 00:13 | |
| *** lykinsbd has quit IRC | 00:14 | |
| *** achanda has quit IRC | 00:15 | |
| *** vmtrooper has quit IRC | 00:16 | |
| *** ozialien has joined #openstack-neutron | 00:16 | |
| *** aranjan_ has joined #openstack-neutron | 00:17 | |
| *** absubram has joined #openstack-neutron | 00:18 | |
| *** rushil has joined #openstack-neutron | 00:19 | |
| *** aranjan has quit IRC | 00:19 | |
| *** aranjan_ has quit IRC | 00:21 | |
| *** rushil has quit IRC | 00:21 | |
| openstackgerrit | Sandhya Dasu proposed openstack/neutron: Cisco Nexus1000V ML2 Mechanism Driver https://review.openstack.org/157243 | 00:22 |
| *** jamielennox|away is now known as jamielennox | 00:23 | |
| *** lykinsbd has joined #openstack-neutron | 00:23 | |
| *** rotbeard has quit IRC | 00:26 | |
| openstackgerrit | Dane LeBlanc proposed openstack/neutron: Support Dual-Stack Gateway Ports on Neutron Routers https://review.openstack.org/149068 | 00:27 |
| openstackgerrit | Dane LeBlanc proposed openstack/neutron: Follow up on review for Dual-Stack Gateway Ports https://review.openstack.org/165664 | 00:27 |
| *** aranjan has joined #openstack-neutron | 00:28 | |
| *** lykinsbd has quit IRC | 00:29 | |
| *** pavel_bondar has quit IRC | 00:30 | |
| *** pavel_bondar has joined #openstack-neutron | 00:31 | |
| *** __TheDodd__ has quit IRC | 00:32 | |
| *** ZZelle_ has quit IRC | 00:34 | |
| *** yalie has joined #openstack-neutron | 00:34 | |
| *** Sukhdev has quit IRC | 00:34 | |
| *** absubram has quit IRC | 00:36 | |
| *** zilberstein has quit IRC | 00:36 | |
| *** shivharis has quit IRC | 00:37 | |
| *** markvoelker has quit IRC | 00:37 | |
| *** shivharis has joined #openstack-neutron | 00:37 | |
| *** zilberstein has joined #openstack-neutron | 00:37 | |
| *** trad511 has joined #openstack-neutron | 00:38 | |
| *** oda-g has quit IRC | 00:40 | |
| *** _cjones_ has quit IRC | 00:42 | |
| *** baoli has quit IRC | 00:43 | |
| *** marun has quit IRC | 00:45 | |
| *** oda-g has joined #openstack-neutron | 00:45 | |
| openstackgerrit | Juergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration https://review.openstack.org/141130 | 00:46 |
| *** yalie has left #openstack-neutron | 00:47 | |
| *** shivharis has quit IRC | 00:48 | |
| *** claudiub has quit IRC | 00:50 | |
| openstackgerrit | badveli_vishnuus proposed openstack/neutron-fwaas: Adding mandatory parameters for Router Info in vArmour fwaas agent https://review.openstack.org/161424 | 00:51 |
| *** yamamoto has quit IRC | 00:53 | |
| *** aranjan has quit IRC | 00:53 | |
| *** topshare has joined #openstack-neutron | 00:55 | |
| *** arosen has quit IRC | 00:56 | |
| *** shaohe_feng1 has joined #openstack-neutron | 00:56 | |
| openstackgerrit | Merged openstack/neutron-fwaas: Refactoring related to 'FWaaS insertion Model on Routers' https://review.openstack.org/161881 | 00:57 |
| *** isq has quit IRC | 00:57 | |
| openstackgerrit | Merged openstack/neutron: Adding VLAN Transparency support for ML2 along with REST API changes https://review.openstack.org/158420 | 00:58 |
| *** jp_at_hp has quit IRC | 00:58 | |
| openstackgerrit | Merged openstack/neutron: DHCP Service LoadBalancing Scheduler https://review.openstack.org/137017 | 00:59 |
| *** sbalukoff has quit IRC | 01:00 | |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 01:00 |
| *** Sukhdev has joined #openstack-neutron | 01:00 | |
| *** absubram has joined #openstack-neutron | 01:01 | |
| *** nlahouti has quit IRC | 01:04 | |
| *** dane_leblanc has quit IRC | 01:07 | |
| *** topshare_ has joined #openstack-neutron | 01:07 | |
| *** Longgeek has joined #openstack-neutron | 01:09 | |
| *** topshare has quit IRC | 01:09 | |
| *** isq has joined #openstack-neutron | 01:11 | |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 01:11 |
| armax | HenryG: ping | 01:12 |
| *** vivek-eb_ has joined #openstack-neutron | 01:13 | |
| *** vivek-ebay has quit IRC | 01:13 | |
| *** YorikSar has quit IRC | 01:15 | |
| *** achanda has joined #openstack-neutron | 01:15 | |
| *** markvoelker has joined #openstack-neutron | 01:17 | |
| *** pritesh has joined #openstack-neutron | 01:20 | |
| *** achanda has quit IRC | 01:21 | |
| *** moshele has quit IRC | 01:21 | |
| *** markvoelker has quit IRC | 01:22 | |
| *** tflynn_ has joined #openstack-neutron | 01:22 | |
| *** mwagner_lap has joined #openstack-neutron | 01:22 | |
| *** tflynn has quit IRC | 01:26 | |
| *** tflynn_ has quit IRC | 01:26 | |
| *** s3wong has quit IRC | 01:26 | |
| *** mwagner_lap has quit IRC | 01:27 | |
| *** sridhar_ram has quit IRC | 01:28 | |
| *** stanzgy has joined #openstack-neutron | 01:33 | |
| *** stanzgy_ has joined #openstack-neutron | 01:33 | |
| *** dave-mcc_ has quit IRC | 01:38 | |
| *** mwagner_lap has joined #openstack-neutron | 01:40 | |
| *** popw has joined #openstack-neutron | 01:42 | |
| *** ivar-laz_ has joined #openstack-neutron | 01:43 | |
| *** Marga_ has quit IRC | 01:43 | |
| *** sthillma has quit IRC | 01:46 | |
| *** ivar-lazzaro has quit IRC | 01:46 | |
| *** dave-mccowan has joined #openstack-neutron | 01:46 | |
| *** ivar-laz_ has quit IRC | 01:47 | |
| *** isq has quit IRC | 01:48 | |
| *** ChuckC has joined #openstack-neutron | 01:49 | |
| *** trad511 has quit IRC | 01:52 | |
| *** ozialien has quit IRC | 01:53 | |
| *** apuimedo has quit IRC | 01:55 | |
| *** isq has joined #openstack-neutron | 01:55 | |
| *** marun has joined #openstack-neutron | 01:56 | |
| *** mwagner_lap has quit IRC | 01:58 | |
| *** SumitNaiksatam has joined #openstack-neutron | 02:00 | |
| *** vmtrooper has joined #openstack-neutron | 02:00 | |
| *** marun has quit IRC | 02:01 | |
| *** ajmiller has joined #openstack-neutron | 02:01 | |
| *** vmtrooper has quit IRC | 02:05 | |
| *** lykinsbd has joined #openstack-neutron | 02:07 | |
| *** busterswt has joined #openstack-neutron | 02:07 | |
| *** mwagner_lap has joined #openstack-neutron | 02:11 | |
| *** Sukhdev has quit IRC | 02:11 | |
| *** lykinsbd has quit IRC | 02:11 | |
| *** harlowja_ is now known as harlowja_away | 02:14 | |
| *** ozialien has joined #openstack-neutron | 02:14 | |
| *** pritesh has quit IRC | 02:14 | |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 02:16 |
| *** markvoelker has joined #openstack-neutron | 02:18 | |
| *** vishwanathj has joined #openstack-neutron | 02:18 | |
| *** reed has quit IRC | 02:20 | |
| *** erkules_ has joined #openstack-neutron | 02:20 | |
| *** markvoelker has quit IRC | 02:23 | |
| *** tflynn has joined #openstack-neutron | 02:23 | |
| *** erkules has quit IRC | 02:23 | |
| *** padkrish has quit IRC | 02:24 | |
| *** yamahata has quit IRC | 02:24 | |
| *** padkrish has joined #openstack-neutron | 02:24 | |
| *** achanda has joined #openstack-neutron | 02:27 | |
| *** tflynn has quit IRC | 02:27 | |
| *** baoli has joined #openstack-neutron | 02:28 | |
| *** dims has quit IRC | 02:28 | |
| *** padkrish has quit IRC | 02:29 | |
| *** absubram has quit IRC | 02:33 | |
| *** vivek-eb_ has quit IRC | 02:34 | |
| *** shwetaap has joined #openstack-neutron | 02:37 | |
| *** lykinsbd has joined #openstack-neutron | 02:38 | |
| *** tflynn has joined #openstack-neutron | 02:40 | |
| *** lykinsbd has quit IRC | 02:43 | |
| *** aswadr has joined #openstack-neutron | 02:43 | |
| *** vivek-ebay has joined #openstack-neutron | 02:45 | |
| *** beagles has quit IRC | 02:46 | |
| *** b3nt_pin has joined #openstack-neutron | 02:47 | |
| *** b3nt_pin is now known as Guest87543 | 02:47 | |
| *** tflynn has quit IRC | 02:49 | |
| *** Murali has joined #openstack-neutron | 02:51 | |
| *** busterswt has quit IRC | 02:52 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Send notification to controller about HA router state change https://review.openstack.org/126188 | 02:53 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Expose ha_state per router to agent binding via API https://review.openstack.org/126189 | 02:53 |
| *** leenheer has quit IRC | 02:54 | |
| *** strictlyb has quit IRC | 02:55 | |
| *** vivek-ebay has quit IRC | 02:56 | |
| *** leenheer has joined #openstack-neutron | 02:57 | |
| *** Murali has quit IRC | 02:58 | |
| *** coolsvap|afk is now known as coolsvap | 03:04 | |
| *** aranjan_ has joined #openstack-neutron | 03:06 | |
| dougwig | the gate seems downright tame. | 03:07 |
| *** aranjan__ has joined #openstack-neutron | 03:08 | |
| *** aranjan_ has quit IRC | 03:11 | |
| blogan | unlike you | 03:11 |
| *** absubram has joined #openstack-neutron | 03:12 | |
| *** amuller has quit IRC | 03:12 | |
| *** lykinsbd has joined #openstack-neutron | 03:13 | |
| *** lykinsbd has quit IRC | 03:18 | |
| *** Marga_ has joined #openstack-neutron | 03:21 | |
| *** Marga_ has quit IRC | 03:21 | |
| *** Marga_ has joined #openstack-neutron | 03:22 | |
| *** tflynn has joined #openstack-neutron | 03:22 | |
| *** vmtrooper has joined #openstack-neutron | 03:23 | |
| *** vmtrooper has quit IRC | 03:23 | |
| openstackgerrit | Merged openstack/neutron-vpnaas: Functional tests of ipsec strongswan vpnaas driver https://review.openstack.org/158560 | 03:25 |
| *** akanksha has quit IRC | 03:27 | |
| *** nati_ueno has quit IRC | 03:27 | |
| *** tflynn has quit IRC | 03:28 | |
| *** tflynn has joined #openstack-neutron | 03:30 | |
| *** coolsvap is now known as coolsvap|afk | 03:32 | |
| *** changbl has joined #openstack-neutron | 03:34 | |
| *** stanzgy_ has quit IRC | 03:37 | |
| *** stanzgy has quit IRC | 03:39 | |
| *** stanzgy has joined #openstack-neutron | 03:39 | |
| *** coolsvap|afk is now known as coolsvap | 03:39 | |
| *** tflynn has quit IRC | 03:40 | |
| *** yamamoto has joined #openstack-neutron | 03:40 | |
| *** achanda has quit IRC | 03:42 | |
| *** achanda has joined #openstack-neutron | 03:44 | |
| openstackgerrit | Merged openstack/neutron: Replace keepalived notifier bash script with Python ip monitor https://review.openstack.org/125384 | 03:50 |
| openstackgerrit | Merged openstack/neutron: Fix minor decomp progress chart issues https://review.openstack.org/165267 | 03:50 |
| *** jckasper has joined #openstack-neutron | 03:57 | |
| openstackgerrit | Itsuro Oda proposed openstack/neutron: Keep existing services available on agents with admin_state_up False https://review.openstack.org/147032 | 04:00 |
| *** ozialien has quit IRC | 04:01 | |
| *** ozialien has joined #openstack-neutron | 04:02 | |
| *** leenheer has quit IRC | 04:02 | |
| *** sudipto_ has joined #openstack-neutron | 04:02 | |
| *** sbalukoff has joined #openstack-neutron | 04:03 | |
| *** tflynn has joined #openstack-neutron | 04:04 | |
| *** achanda has quit IRC | 04:05 | |
| *** jamielennox is now known as jamielennox|away | 04:06 | |
| *** jckasper has quit IRC | 04:09 | |
| *** gongysh_ has joined #openstack-neutron | 04:09 | |
| *** baoli has quit IRC | 04:10 | |
| *** sudipto_ has quit IRC | 04:10 | |
| *** tflynn has quit IRC | 04:12 | |
| *** tflynn has joined #openstack-neutron | 04:13 | |
| *** lazy_prince is now known as killer_prince | 04:15 | |
| *** Marga_ has quit IRC | 04:18 | |
| *** dims has joined #openstack-neutron | 04:19 | |
| *** markvoelker has joined #openstack-neutron | 04:19 | |
| *** markvoelker has quit IRC | 04:24 | |
| *** carl_baldwin has joined #openstack-neutron | 04:26 | |
| *** erkules_ is now known as erkules | 04:26 | |
| *** erkules has joined #openstack-neutron | 04:26 | |
| *** leenheer has joined #openstack-neutron | 04:26 | |
| *** dims has quit IRC | 04:27 | |
| *** Murali has joined #openstack-neutron | 04:28 | |
| *** Sukhdev has joined #openstack-neutron | 04:31 | |
| *** padkrish has joined #openstack-neutron | 04:34 | |
| *** padkrish has quit IRC | 04:35 | |
| *** padkrish has joined #openstack-neutron | 04:36 | |
| *** achanda has joined #openstack-neutron | 04:37 | |
| *** dboik_ has quit IRC | 04:39 | |
| *** amotoki has joined #openstack-neutron | 04:40 | |
| *** leenheer has quit IRC | 04:40 | |
| *** achanda has quit IRC | 04:41 | |
| *** sbfox has joined #openstack-neutron | 04:42 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: Fix a usage error of joinedload + filter in dhcp scheduler https://review.openstack.org/150284 | 04:44 |
| *** dboik has joined #openstack-neutron | 04:46 | |
| *** Marga_ has joined #openstack-neutron | 04:47 | |
| *** pradipta has joined #openstack-neutron | 04:52 | |
| *** ijw has joined #openstack-neutron | 04:56 | |
| *** trinaths has joined #openstack-neutron | 04:56 | |
| *** yuanying has quit IRC | 05:01 | |
| *** lykinsbd has joined #openstack-neutron | 05:02 | |
| *** adelinatuvenie has joined #openstack-neutron | 05:02 | |
| *** __afazekas is now known as afazekas | 05:03 | |
| *** tflynn has quit IRC | 05:04 | |
| *** yuanying has joined #openstack-neutron | 05:06 | |
| *** lykinsbd has quit IRC | 05:07 | |
| *** yamamoto has quit IRC | 05:07 | |
| *** achanda has joined #openstack-neutron | 05:08 | |
| *** SridharG has joined #openstack-neutron | 05:12 | |
| *** dave-mccowan has quit IRC | 05:12 | |
| *** tflynn has joined #openstack-neutron | 05:13 | |
| *** yalie has joined #openstack-neutron | 05:13 | |
| *** nati_ueno has joined #openstack-neutron | 05:14 | |
| *** iymht has joined #openstack-neutron | 05:16 | |
| yalie | hi trinaths? | 05:18 |
| yalie | I have updated the patch base on your comments https://review.openstack.org/#/c/152093/ | 05:18 |
| yalie | could you help to review? | 05:18 |
| *** markvoelker has joined #openstack-neutron | 05:20 | |
| *** anilvenkata has joined #openstack-neutron | 05:23 | |
| openstackgerrit | Merged openstack/neutron: Include IPv6 SLAAC addresses implicitly for port create https://review.openstack.org/113339 | 05:23 |
| openstackgerrit | Merged openstack/neutron: Include IPv6 SLAAC addresses implicitly for port create https://review.openstack.org/113339 | 05:23 |
| *** jyuso1 has joined #openstack-neutron | 05:25 | |
| *** nlahouti has joined #openstack-neutron | 05:25 | |
| *** markvoelker has quit IRC | 05:25 | |
| *** garyk has quit IRC | 05:26 | |
| *** liusheng has quit IRC | 05:26 | |
| *** liusheng has joined #openstack-neutron | 05:26 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: OVS_LIB API addition - change bridge controller connection-mode https://review.openstack.org/165326 | 05:27 |
| *** armax has quit IRC | 05:27 | |
| *** ferest has joined #openstack-neutron | 05:28 | |
| *** yuanying has quit IRC | 05:30 | |
| *** yuanying has joined #openstack-neutron | 05:31 | |
| *** yfried|afk is now known as yfried | 05:31 | |
| *** yamamoto has joined #openstack-neutron | 05:32 | |
| *** trinaths has left #openstack-neutron | 05:32 | |
| *** trinaths has joined #openstack-neutron | 05:32 | |
| *** ozialien has quit IRC | 05:32 | |
| *** yatin has joined #openstack-neutron | 05:32 | |
| *** trinaths has left #openstack-neutron | 05:33 | |
| *** lykinsbd has joined #openstack-neutron | 05:33 | |
| *** ferest has quit IRC | 05:33 | |
| *** trinaths has joined #openstack-neutron | 05:34 | |
| *** dannywilson has quit IRC | 05:34 | |
| *** tflynn has quit IRC | 05:35 | |
| *** tflynn has joined #openstack-neutron | 05:35 | |
| *** leenheer has joined #openstack-neutron | 05:36 | |
| *** padkrish has quit IRC | 05:36 | |
| trinaths | yalie: Hi | 05:37 |
| trinaths | yalie: ^^ into the patchset | 05:37 |
| *** sudipto has joined #openstack-neutron | 05:37 | |
| *** lykinsbd has quit IRC | 05:38 | |
| *** nlahouti has quit IRC | 05:38 | |
| *** numan has joined #openstack-neutron | 05:38 | |
| *** nlahouti has joined #openstack-neutron | 05:38 | |
| *** carl_baldwin has quit IRC | 05:39 | |
| *** coolsvap is now known as coolsvap|afk | 05:41 | |
| *** nati_ueno has quit IRC | 05:41 | |
| *** nmagnezi has joined #openstack-neutron | 05:42 | |
| *** ChuckC has quit IRC | 05:45 | |
| *** yuanying has quit IRC | 05:46 | |
| *** yuanying has joined #openstack-neutron | 05:47 | |
| *** coolsvap|afk is now known as coolsvap | 05:48 | |
| trinaths | yalie: ping | 05:50 |
| trinaths | yalie: can you make another patch with respect to comments from SumitNaiksatam on your patch. they are simple and easy to do them now. | 05:51 |
| SumitNaiksatam | trinaths: that is not required | 05:53 |
| SumitNaiksatam | trinaths: i have already mentioned that in my comment | 05:53 |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Basic subnetpool CRUD https://review.openstack.org/148698 | 05:54 |
| *** shwetaap has quit IRC | 05:56 | |
| *** yfried is now known as yfried|afk | 05:59 | |
| trinaths | SumitNaiksatam: true said, okay with that. But re-organizing the imports is a good thing right? | 06:02 |
| *** yuanying has quit IRC | 06:05 | |
| *** gongysh_ has quit IRC | 06:05 | |
| *** jschwarz has joined #openstack-neutron | 06:06 | |
| *** yuanying has joined #openstack-neutron | 06:08 | |
| *** leenheer has quit IRC | 06:08 | |
| trinaths | SumitNaiksatam: +1 to your comments, the patch is ready to go. :) | 06:09 |
| yalie | trinaths, SumitNaiksatam: thank you! | 06:09 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex https://review.openstack.org/165705 | 06:10 |
| *** irenab has quit IRC | 06:12 | |
| *** yfried|afk has quit IRC | 06:12 | |
| *** dims has joined #openstack-neutron | 06:13 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: Validate OVS Agent configurations https://review.openstack.org/165311 | 06:19 |
| *** dims has quit IRC | 06:19 | |
| *** killer_prince is now known as lazy_prince | 06:20 | |
| *** markvoelker has joined #openstack-neutron | 06:21 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Subnet allocation from a subnet pool https://review.openstack.org/157597 | 06:23 |
| *** markvoelker has quit IRC | 06:26 | |
| *** achanda has quit IRC | 06:28 | |
| *** yog_ has joined #openstack-neutron | 06:28 | |
| *** leenheer has joined #openstack-neutron | 06:29 | |
| *** leenheer has quit IRC | 06:30 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Simple subnetpool allocation quotas https://review.openstack.org/165264 | 06:30 |
| *** Sukhdev has quit IRC | 06:31 | |
| *** pcaruana has quit IRC | 06:33 | |
| *** aepifanov has joined #openstack-neutron | 06:35 | |
| *** moshele has joined #openstack-neutron | 06:39 | |
| *** ishant has joined #openstack-neutron | 06:43 | |
| *** numan has quit IRC | 06:44 | |
| *** yatin has quit IRC | 06:44 | |
| openstackgerrit | Yushiro FURUKAWA proposed openstack/neutron-fwaas: Insert validation in creating/updating firewall https://review.openstack.org/147396 | 06:45 |
| *** irenab has joined #openstack-neutron | 06:46 | |
| *** yatin has joined #openstack-neutron | 06:47 | |
| *** itzikb has joined #openstack-neutron | 06:52 | |
| *** garyk has joined #openstack-neutron | 06:52 | |
| *** achanda has joined #openstack-neutron | 06:52 | |
| *** tflynn has quit IRC | 06:53 | |
| *** lizk has quit IRC | 06:57 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: Validate when DVR enabled, l2_pop is also enabled https://review.openstack.org/165311 | 06:58 |
| *** achanda has quit IRC | 07:00 | |
| *** achanda has joined #openstack-neutron | 07:04 | |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add full-stack tests framework https://review.openstack.org/128259 | 07:07 |
| *** jyuso1 has quit IRC | 07:08 | |
| *** Marga_ has quit IRC | 07:09 | |
| *** nlahouti has quit IRC | 07:13 | |
| *** ijw has quit IRC | 07:13 | |
| *** neeti has joined #openstack-neutron | 07:14 | |
| *** kickinz1|afk is now known as kickinz1 | 07:14 | |
| *** lykinsbd has joined #openstack-neutron | 07:20 | |
| *** markvoelker has joined #openstack-neutron | 07:22 | |
| *** yfried|afk has joined #openstack-neutron | 07:25 | |
| *** kawa2014 has joined #openstack-neutron | 07:25 | |
| *** markvoelker has quit IRC | 07:26 | |
| *** lykinsbd has quit IRC | 07:30 | |
| *** yfried has joined #openstack-neutron | 07:30 | |
| *** yfried|afk has quit IRC | 07:30 | |
| *** scheuran has joined #openstack-neutron | 07:33 | |
| sudipto | jschwarz: Hi, good morning! | 07:36 |
| *** achanda has quit IRC | 07:37 | |
| jschwarz | sudipto, good morning friend. how are you? | 07:37 |
| sudipto | jschwarz: I am good, how are you? I have added you for the review we talked about yesterday: https://review.openstack.org/#/c/165539/ | 07:38 |
| jschwarz | sudipto, I know - I was just reviewing it when you pinged me | 07:39 |
| jschwarz | sudipto, check your mail :) | 07:39 |
| sudipto | jschwarz: Thanks for the review! I will work on the UT :) | 07:40 |
| *** lionelz has quit IRC | 07:43 | |
| *** Bsony has joined #openstack-neutron | 07:45 | |
| *** chlong has quit IRC | 07:47 | |
| *** SumitNaiksatam has quit IRC | 07:51 | |
| *** belmoreira has joined #openstack-neutron | 07:53 | |
| *** yfried is now known as yfried|afk | 07:54 | |
| *** bobmel has joined #openstack-neutron | 07:57 | |
| *** numan has joined #openstack-neutron | 07:57 | |
| *** abehl has joined #openstack-neutron | 07:58 | |
| *** scheuran has quit IRC | 07:58 | |
| *** scheuran has joined #openstack-neutron | 07:58 | |
| openstackgerrit | Merged openstack/neutron-fwaas: tests: stop overwriting neutron BaseTestCase configuration files https://review.openstack.org/165419 | 07:59 |
| openstackgerrit | Merged openstack/neutron-vpnaas: tests: stop overwriting neutron BaseTestCase configuration files https://review.openstack.org/165421 | 07:59 |
| *** safchain has joined #openstack-neutron | 08:00 | |
| *** nyechiel has joined #openstack-neutron | 08:00 | |
| *** yfried|afk is now known as yfried | 08:01 | |
| jschwarz | sudipto, great :) | 08:03 |
| *** dims has joined #openstack-neutron | 08:04 | |
| openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: Add a missing mock in DHCPAgentWeightSchedulerTestCase https://review.openstack.org/165722 | 08:04 |
| *** yuanying_ has joined #openstack-neutron | 08:05 | |
| *** ronis has joined #openstack-neutron | 08:06 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V https://review.openstack.org/160596 | 08:07 |
| openstackgerrit | Adelina Tuvenie proposed openstack/neutron: Add native OVSDB implmentation of OVSDB API https://review.openstack.org/152761 | 08:07 |
| *** yuanying has quit IRC | 08:08 | |
| *** dims has quit IRC | 08:10 | |
| *** moha_hunt has joined #openstack-neutron | 08:10 | |
| *** yatin has quit IRC | 08:16 | |
| *** jp_at_hp has joined #openstack-neutron | 08:17 | |
| *** sbfox has quit IRC | 08:17 | |
| *** matrohon has joined #openstack-neutron | 08:17 | |
| *** jp_at_hp has quit IRC | 08:18 | |
| *** yfried is now known as yfried|afk | 08:19 | |
| *** yatin has joined #openstack-neutron | 08:21 | |
| *** kickinz1 is now known as kickinz1|afk | 08:26 | |
| jschwarz | ZZelle, may I ask you for a review on https://review.openstack.org/#/c/128259/ ? :) | 08:28 |
| jschwarz | (when you're out of the K-3 road, of course) | 08:28 |
| *** rotbeard has joined #openstack-neutron | 08:28 | |
| openstackgerrit | Yushiro FURUKAWA proposed openstack/neutron: Enable to specify context on POST requests during unittests https://review.openstack.org/157355 | 08:29 |
| *** lpetrut has joined #openstack-neutron | 08:29 | |
| *** dboik has quit IRC | 08:31 | |
| openstackgerrit | Yushiro FURUKAWA proposed openstack/neutron: Enable to apply policies to resources with special plural https://review.openstack.org/145998 | 08:35 |
| *** jistr has joined #openstack-neutron | 08:37 | |
| *** chandankumar has joined #openstack-neutron | 08:38 | |
| *** carlp has quit IRC | 08:38 | |
| *** kickinz1|afk is now known as kickinz1 | 08:38 | |
| *** pask81 has joined #openstack-neutron | 08:39 | |
| *** yfried|afk is now known as yfried | 08:39 | |
| *** criptoverso has joined #openstack-neutron | 08:40 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V https://review.openstack.org/160596 | 08:42 |
| *** apuimedo has joined #openstack-neutron | 08:46 | |
| *** iymht has quit IRC | 08:49 | |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: WIP: Remove downgrade from existed migrations https://review.openstack.org/165740 | 08:53 |
| *** marun has joined #openstack-neutron | 08:53 | |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: WIP: Remove downgrade from existing migrations https://review.openstack.org/165740 | 08:54 |
| *** ygbo has joined #openstack-neutron | 08:55 | |
| *** jlanoux has joined #openstack-neutron | 08:56 | |
| *** marun has quit IRC | 08:58 | |
| ZZelle | jschwarz, hi, you can | 08:58 |
| *** iymht has joined #openstack-neutron | 08:59 | |
| jschwarz | ZZelle, thanks. I'm going through your patches as we speak | 09:00 |
| *** nyechiel_ has joined #openstack-neutron | 09:03 | |
| *** YorikSar has joined #openstack-neutron | 09:05 | |
| *** nyechiel has quit IRC | 09:05 | |
| *** ronis has quit IRC | 09:07 | |
| *** ChrisNBlum has joined #openstack-neutron | 09:10 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V https://review.openstack.org/160596 | 09:10 |
| *** ronis has joined #openstack-neutron | 09:11 | |
| zigo_ | I'm wondering if it is completely reasonable to have a 90 MB Git repository for stuff like neutron-vpnaas. | 09:12 |
| zigo_ | It's annoying for me to do the first upload to git.debian.org over ADSL... :( | 09:12 |
| zigo_ | Did we really need all the history? | 09:12 |
| *** lykinsbd has joined #openstack-neutron | 09:14 | |
| *** kawa2014 has quit IRC | 09:14 | |
| *** kawa2014 has joined #openstack-neutron | 09:16 | |
| *** rdo has quit IRC | 09:16 | |
| *** ronis has quit IRC | 09:17 | |
| jschwarz | zigo_, history ftw | 09:17 |
| *** apuimedo has quit IRC | 09:17 | |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: Add devices to update in RPC call security_groups_provider_updated https://review.openstack.org/148235 | 09:18 |
| *** apuimedo has joined #openstack-neutron | 09:18 | |
| *** lykinsbd has quit IRC | 09:19 | |
| *** dims has joined #openstack-neutron | 09:22 | |
| *** logan2 has quit IRC | 09:22 | |
| *** logan2 has joined #openstack-neutron | 09:22 | |
| *** markvoelker has joined #openstack-neutron | 09:23 | |
| *** dims has quit IRC | 09:26 | |
| *** Longgeek has quit IRC | 09:27 | |
| *** markvoelker has quit IRC | 09:28 | |
| *** Longgeek has joined #openstack-neutron | 09:29 | |
| *** kawa2014 has quit IRC | 09:29 | |
| *** Guest87543 is now known as b3nt_pin | 09:29 | |
| openstackgerrit | Han Zhou proposed openstack/neutron: Do not send notifications to inactive dhcp-agents. https://review.openstack.org/165749 | 09:31 |
| *** Longgeek has quit IRC | 09:31 | |
| *** yog_ has quit IRC | 09:31 | |
| openstackgerrit | watanabe isao proposed openstack/neutron: Validate DHCP max port when creating/updating subnet https://review.openstack.org/160214 | 09:32 |
| *** adelinatuvenie has quit IRC | 09:35 | |
| *** ihrachyshka has joined #openstack-neutron | 09:35 | |
| *** Longgeek has joined #openstack-neutron | 09:37 | |
| *** liusheng has quit IRC | 09:41 | |
| *** liusheng has joined #openstack-neutron | 09:42 | |
| *** ZZelle has quit IRC | 09:43 | |
| *** ZZelle has joined #openstack-neutron | 09:43 | |
| *** ihrachyshka has quit IRC | 09:44 | |
| *** watanabe_isao has quit IRC | 09:44 | |
| *** ChrisNBlum has quit IRC | 09:45 | |
| *** ihrachyshka has joined #openstack-neutron | 09:45 | |
| *** dims has joined #openstack-neutron | 09:45 | |
| *** yog_ has joined #openstack-neutron | 09:47 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: Validate when DVR enabled, l2_pop is also enabled https://review.openstack.org/165311 | 09:48 |
| *** amotoki has quit IRC | 09:48 | |
| *** Trozz has quit IRC | 09:50 | |
| *** Trozz has joined #openstack-neutron | 09:53 | |
| *** yamamoto has quit IRC | 09:54 | |
| *** Trozz is now known as Trozz_ | 09:55 | |
| *** Trozz_ is now known as Trozz | 09:55 | |
| *** absubram has quit IRC | 10:00 | |
| openstackgerrit | Zhang Hua proposed openstack/neutron-vpnaas: Add some unit tests for strongswan driver https://review.openstack.org/165159 | 10:01 |
| *** yog_ has quit IRC | 10:02 | |
| *** gildub has quit IRC | 10:02 | |
| *** gildub has joined #openstack-neutron | 10:02 | |
| *** chandankumar has quit IRC | 10:04 | |
| *** oreillyd has joined #openstack-neutron | 10:05 | |
| *** wznoinsk_ has joined #openstack-neutron | 10:10 | |
| *** shaohe_feng2 has joined #openstack-neutron | 10:10 | |
| *** hichihara has quit IRC | 10:11 | |
| *** iymht has quit IRC | 10:11 | |
| *** shaohe_robot has joined #openstack-neutron | 10:11 | |
| *** alex_xu has quit IRC | 10:12 | |
| *** adelinatuvenie has joined #openstack-neutron | 10:12 | |
| *** shaohe_feng has quit IRC | 10:12 | |
| *** Longgeek has quit IRC | 10:13 | |
| *** shaohe_feng1 has quit IRC | 10:13 | |
| *** Longgeek has joined #openstack-neutron | 10:13 | |
| *** wznoinsk has quit IRC | 10:13 | |
| *** alex_xu has joined #openstack-neutron | 10:14 | |
| *** yog_ has joined #openstack-neutron | 10:15 | |
| *** lazy_prince has quit IRC | 10:17 | |
| openstackgerrit | Abishek Subramanian proposed openstack/neutron: Support IPv6 Router https://review.openstack.org/156283 | 10:17 |
| *** ihrachyshka has quit IRC | 10:17 | |
| *** Murali has quit IRC | 10:17 | |
| *** yamamoto has joined #openstack-neutron | 10:18 | |
| openstackgerrit | Claudiu Belu proposed openstack/neutron: Hyper-V Agent decomposition https://review.openstack.org/153259 | 10:18 |
| *** claudiub has joined #openstack-neutron | 10:18 | |
| *** chandankumar has joined #openstack-neutron | 10:18 | |
| *** pc_m has joined #openstack-neutron | 10:19 | |
| *** yog_ has quit IRC | 10:19 | |
| *** kawa2014 has joined #openstack-neutron | 10:21 | |
| ZZelle | kevinbenton, hi | 10:24 |
| *** markvoelker has joined #openstack-neutron | 10:24 | |
| *** killer_prince has joined #openstack-neutron | 10:24 | |
| *** lazy_prince has joined #openstack-neutron | 10:24 | |
| *** Murali has joined #openstack-neutron | 10:27 | |
| *** john-davidge has joined #openstack-neutron | 10:27 | |
| *** markvoelker has quit IRC | 10:29 | |
| *** ihrachyshka has joined #openstack-neutron | 10:29 | |
| *** yfried is now known as yfried|afk | 10:34 | |
| openstackgerrit | Adelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V https://review.openstack.org/160596 | 10:35 |
| *** vthapar has joined #openstack-neutron | 10:38 | |
| *** nyechiel_ has quit IRC | 10:40 | |
| *** coolsvap is now known as coolsvap|afk | 10:41 | |
| *** chandankumar_ has joined #openstack-neutron | 10:44 | |
| *** stanzgy_ has joined #openstack-neutron | 10:46 | |
| *** chandankumar has quit IRC | 10:47 | |
| *** stanzgy has quit IRC | 10:48 | |
| *** SridharG has left #openstack-neutron | 10:49 | |
| *** stanzgy_ has quit IRC | 10:51 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform Base(OVS)LinuxTestCase methods in helpers https://review.openstack.org/159852 | 10:51 |
| *** roeyc has joined #openstack-neutron | 10:53 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Use a finite loop in create_resource test method https://review.openstack.org/161808 | 10:53 |
| *** yfried|afk is now known as yfried | 10:54 | |
| *** gampel has joined #openstack-neutron | 10:55 | |
| *** yuanying_ has quit IRC | 10:57 | |
| *** yrn has quit IRC | 10:58 | |
| openstackgerrit | Merged openstack/neutron: Remove unused L3 HA RPC method https://review.openstack.org/155370 | 10:59 |
| *** amotoki has joined #openstack-neutron | 11:00 | |
| *** jschwarz has quit IRC | 11:01 | |
| *** nyechiel_ has joined #openstack-neutron | 11:03 | |
| *** lykinsbd has joined #openstack-neutron | 11:03 | |
| *** jaypipes has joined #openstack-neutron | 11:04 | |
| *** pcaruana has joined #openstack-neutron | 11:05 | |
| *** lykinsbd has quit IRC | 11:08 | |
| *** sambetts has quit IRC | 11:09 | |
| *** yalie has quit IRC | 11:09 | |
| *** yalie has joined #openstack-neutron | 11:10 | |
| *** sambetts has joined #openstack-neutron | 11:11 | |
| *** britthouser has joined #openstack-neutron | 11:13 | |
| openstackgerrit | garyk proposed openstack/neutron: Revert "Adding VLAN Transparency support for ML2 along with REST API changes" https://review.openstack.org/165776 | 11:14 |
| *** sudipto has quit IRC | 11:14 | |
| garyk | Any cores around to talk about ^ | 11:15 |
| *** topshare_ has quit IRC | 11:15 | |
| openstackgerrit | Merged openstack/neutron: Reduce code duplication and fix argument order in test_wsgi https://review.openstack.org/162059 | 11:16 |
| *** devweasel has joined #openstack-neutron | 11:17 | |
| garyk | Anyone know why an extnsion was update directly on the attributes.py and not added as a seperate extension. Maybe I missed somethings | 11:17 |
| *** trinaths has left #openstack-neutron | 11:17 | |
| *** mwagner_lap has quit IRC | 11:21 | |
| *** wznoinsk_ is now known as wznoinsk | 11:24 | |
| *** yamamoto has quit IRC | 11:25 | |
| *** markvoelker has joined #openstack-neutron | 11:25 | |
| *** markvoelker has quit IRC | 11:30 | |
| *** kawa2014 has quit IRC | 11:33 | |
| *** kawa2014 has joined #openstack-neutron | 11:33 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests https://review.openstack.org/143412 | 11:34 |
| claudiub | garyk: hello | 11:34 |
| garyk | claudiub: hi | 11:34 |
| openstackgerrit | Adelina Tuvenie proposed openstack/neutron: OVS agent support on Hyper-V https://review.openstack.org/160596 | 11:34 |
| claudiub | garyk: can you take a look at the hyper-v agent decomposition? :) it got 3x+2 and +A last night, but there was a merge conflict a few hours ago. :( https://review.openstack.org/#/c/153259/ | 11:35 |
| claudiub | garyk: trivial rebase, only conflict was on setup.cfg | 11:35 |
| *** setuid has quit IRC | 11:35 | |
| *** aranjan__ has quit IRC | 11:35 | |
| *** coolsvap|afk is now known as coolsvap | 11:36 | |
| claudiub | garyk: thanks. :D | 11:36 |
| garyk | claudiub: sure, done. just one heads up - the extenal plugins break every onw and then due to neutron upstream changes :) | 11:36 |
| garyk | its a trade off | 11:36 |
| openstackgerrit | Merged openstack/neutron: Add a netns-cleanup functional test https://review.openstack.org/163860 | 11:37 |
| claudiub | garyk: yeah, I agree. But it's ok, Hyper-V CI should run on neutron, just to detect such cases. :) But thanks for the heads up. :) | 11:37 |
| *** pradipta has quit IRC | 11:40 | |
| *** kerpan has joined #openstack-neutron | 11:41 | |
| *** setuid has joined #openstack-neutron | 11:43 | |
| *** aranjan has joined #openstack-neutron | 11:43 | |
| *** lori|away is now known as lori | 11:44 | |
| *** akanksha has joined #openstack-neutron | 11:44 | |
| *** yamamoto has joined #openstack-neutron | 11:50 | |
| *** setuid has quit IRC | 11:50 | |
| *** setuid has joined #openstack-neutron | 11:53 | |
| *** markvoelker has joined #openstack-neutron | 11:54 | |
| *** ronis has joined #openstack-neutron | 11:58 | |
| lpetrut | garyk: Hi | 12:00 |
| garyk | lpetrut: hi | 12:01 |
| *** gildub has quit IRC | 12:02 | |
| lpetrut | garyk: We're planning on adding network QoS support on Hyper-V. I've thought about using the flavor extra specs for setting IOPS caps. The issue is that we need this on the neutron agent side. Would it be ok to retrieve such flavor information in mechanism drivers and put it in the device details that are used by the agent? | 12:02 |
| *** sudipto has joined #openstack-neutron | 12:04 | |
| garyk | lpetrut: i guess that the challenge is getting the data from the service to the agent | 12:04 |
| *** aranjan has quit IRC | 12:04 | |
| lpetrut | garyk: exactly | 12:05 |
| *** claudiub_ has joined #openstack-neutron | 12:05 | |
| garyk | lpetrut: there is a QOS extension that passes it from the flavor to the service - https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L636 | 12:05 |
| *** mwagner_lap has joined #openstack-neutron | 12:07 | |
| *** claudiub has quit IRC | 12:07 | |
| lpetrut | garyk: well that looks like a good starting point. Although that flavor rxtx_factor relies on a rxtx_base, which was previously set by nova-network per network. Neutron networks don't have this property. | 12:08 |
| *** baribal has joined #openstack-neutron | 12:10 | |
| garyk | lpetrut: yes, agree. | 12:10 |
| garyk | maybe try and sync with aaron rosen. he wrote the support. maybe you guys collaborate and have a general extension that the whole community can use | 12:11 |
| *** diegows_ has joined #openstack-neutron | 12:11 | |
| lpetrut | that would be great. thanks a lot! | 12:11 |
| *** chlong has joined #openstack-neutron | 12:14 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: Validate when DVR enabled, l2_pop is also enabled https://review.openstack.org/165311 | 12:14 |
| *** Longgeek has quit IRC | 12:20 | |
| *** Longgeek has joined #openstack-neutron | 12:20 | |
| baribal | Hi All. Could you please tell if it is possible to re-use deallocated/released floating ips from the project? When allocating the new fip to the project it seems it uses "ipavailabilityranges" table in "ovs_neutron" DB and the "first_ip" of the "allocation_pool_id" record to choose the next fip and increase the "first_ip" by 1. When we release/deallocate the fip from the project it does not change this table so every time you | 12:21 |
| baribal | deallocate/release fips we can't re-use them in different or the same project. The only possibility is not manually update the record in this table to set first_ip to the fip which has just been deallocated/released from the project which is total mess. Neutron manages private network ips just fine and doesn't have such an issue. Could you please advise? We are using the Icehouse release from RDO repository. | 12:21 |
| *** ajmiller has quit IRC | 12:23 | |
| *** nyechiel__ has joined #openstack-neutron | 12:24 | |
| *** nyechiel_ has quit IRC | 12:25 | |
| *** Marga_ has joined #openstack-neutron | 12:26 | |
| *** chlong has quit IRC | 12:26 | |
| *** yatin has quit IRC | 12:26 | |
| sudipto | baribal: There's a bug that has been reported on the same line. | 12:28 |
| sudipto | baribal: https://bugs.launchpad.net/neutron/+bug/1427022 Is this something similar to your concern? | 12:28 |
| openstack | Launchpad bug 1427022 in neutron "not releasing IP address when deleting VM, port, LB VIP" [Undecided,Incomplete] - Assigned to Sudipta Biswas (sbiswas7) | 12:28 |
| *** rdo has joined #openstack-neutron | 12:30 | |
| *** yalie has quit IRC | 12:31 | |
| garyk | mestery: you around? | 12:33 |
| garyk | there are some very dodgy things with the extensions | 12:33 |
| openstackgerrit | garyk proposed openstack/neutron: Revert "Expose Rest Api access to mtu attributes" https://review.openstack.org/165801 | 12:33 |
| garyk | mestery: ^ | 12:33 |
| *** yalie has joined #openstack-neutron | 12:34 | |
| baribal | sudipto: this bug looks like not a bug but a normal behaviour in Icehouse. I've seen somewhere that the private network ranges will be recalculated only when the all the ips are consumed. | 12:34 |
| baribal | I tried to do the same with fip range - put the first_ip=last_ip to trigger recalculation but it didn't happen. | 12:35 |
| sudipto | baribal: Interesting! | 12:35 |
| sudipto | I'd try it with the private IPs in a while and see if the re-calculation is triggered. Do you know any precise reason for waiting till the end? | 12:36 |
| baribal | written here from IRC http://lists.openstack.org/pipermail/openstack/2014-June/008130.html | 12:36 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform Base(OVS)LinuxTestCase methods in helpers https://review.openstack.org/159852 | 12:36 |
| openstackgerrit | Gal Sagie proposed openstack/neutron: Remove redundant unit tests from OVS DVR Agent https://review.openstack.org/165802 | 12:37 |
| *** dave-mccowan has joined #openstack-neutron | 12:37 | |
| baribal | sudipto: I can test with private IPs. But my issue is for fips :) | 12:37 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Use a finite loop in create_resource test method https://review.openstack.org/161808 | 12:37 |
| sudipto | baribal: Yeah, i will try that as well :) | 12:37 |
| *** Longgeek_ has joined #openstack-neutron | 12:37 | |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add full-stack tests framework https://review.openstack.org/128259 | 12:38 |
| *** diegows_ has quit IRC | 12:38 | |
| sudipto | unless ofcourse someone has thoughts on it. | 12:38 |
| *** diegows has joined #openstack-neutron | 12:38 | |
| *** adelinatuvenie has quit IRC | 12:39 | |
| *** shaohe_feng2 has quit IRC | 12:40 | |
| *** atuvenie has joined #openstack-neutron | 12:40 | |
| *** Longgeek has quit IRC | 12:41 | |
| *** achanda has joined #openstack-neutron | 12:42 | |
| *** ronis has quit IRC | 12:42 | |
| *** chlong has joined #openstack-neutron | 12:42 | |
| *** chem has joined #openstack-neutron | 12:43 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: Validate when DVR enabled, l2_pop is also enabled https://review.openstack.org/165311 | 12:45 |
| openstackgerrit | Sudipta Biswas proposed openstack/neutron: Improve error detection on agent registration https://review.openstack.org/165539 | 12:45 |
| *** achanda has quit IRC | 12:47 | |
| sudipto | garyk: added you for a code review. Whenever you have some time :) | 12:48 |
| otherwiseguy | mestery: I've seen a weird py27 test failure in the gate on the OVSDB patch regarding KeyError: 'L3_ROUTER_NAT' (http://logs.openstack.org/61/152761/20/check/gate-neutron-python27/e516e2f/console.html) which shouldn't at all be related to my patch. Very odd. Any ideas before I do another recheck? | 12:50 |
| openstackgerrit | Dane LeBlanc proposed openstack/neutron: Follow up on review for Dual-Stack Gateway Ports https://review.openstack.org/165664 | 12:51 |
| *** lykinsbd has joined #openstack-neutron | 12:52 | |
| garyk | sudipto: i added one minor comment. i need to run... | 12:54 |
| *** dane_leblanc has joined #openstack-neutron | 12:54 | |
| *** vthapar has quit IRC | 12:54 | |
| *** lazy_prince has quit IRC | 12:55 | |
| sudipto | garyk: thanks for the review. I will correct it. | 12:55 |
| *** killer_prince has joined #openstack-neutron | 12:55 | |
| *** killer_prince is now known as lazy_prince | 12:56 | |
| *** lykinsbd has quit IRC | 12:57 | |
| *** garyk has quit IRC | 12:58 | |
| *** aveiga has joined #openstack-neutron | 12:58 | |
| openstackgerrit | Roey Chen proposed openstack/neutron: Allow plugin to specify security-group rules ids upon creation https://review.openstack.org/165553 | 12:59 |
| *** dims has quit IRC | 13:01 | |
| *** dims has joined #openstack-neutron | 13:02 | |
| *** dboik has joined #openstack-neutron | 13:03 | |
| *** chandankumar_ has quit IRC | 13:04 | |
| *** xuhanp_ has joined #openstack-neutron | 13:05 | |
| *** xuhanp has joined #openstack-neutron | 13:05 | |
| *** neeti has quit IRC | 13:05 | |
| *** ronis has joined #openstack-neutron | 13:06 | |
| otherwiseguy | atuvenie: It looks like when you based your patch off of the ovsdb native patch, you updated both patches, which removed the workflow +1 on it. | 13:06 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform Base(OVS)LinuxTestCase methods in helpers https://review.openstack.org/159852 | 13:06 |
| otherwiseguy | Any cores around to re-workflow +1 https://review.openstack.org/#/c/152761/ | 13:06 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Use a finite loop in create_resource test method https://review.openstack.org/161808 | 13:07 |
| atuvenie | otherwiseguy: both were updated because the ovsdb native patch was a little behind | 13:07 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Replace BaseIPVethTestCase by FakeMachine https://review.openstack.org/160290 | 13:08 |
| *** jistr has quit IRC | 13:08 | |
| *** cference has joined #openstack-neutron | 13:08 | |
| *** jistr has joined #openstack-neutron | 13:09 | |
| *** Networkn3rd has quit IRC | 13:10 | |
| atuvenie | otherwiseguy: also, there is one test that kept failing before and after the update, and it's failing in my patch as well since it is based on this. I was looking arround on a fix, maybe you have an idea what the problem might be? | 13:11 |
| *** kbringard has joined #openstack-neutron | 13:14 | |
| openstackgerrit | Roey Chen proposed openstack/neutron: Allow plugin to specify security-group rules ids upon creation https://review.openstack.org/165553 | 13:15 |
| *** amuller has joined #openstack-neutron | 13:18 | |
| *** igordcard has joined #openstack-neutron | 13:18 | |
| *** dboik has quit IRC | 13:18 | |
| baribal | sudipto: tested with private networks recalculation works. | 13:18 |
| baribal | was | 22e33986-af2c-4430-9f80-8d685ed3e2f2 | 192.168.0.73 | 192.168.0.73 | | 13:19 |
| baribal | after I assigned this last IP the range disappeared from the table | 13:19 |
| baribal | then on the next machine creation it recalculated all the subnet | 13:20 |
| baribal | | 22e33986-af2c-4430-9f80-8d685ed3e2f2 | 192.168.0.104 | 192.168.0.134 | | 13:20 |
| baribal | | 22e33986-af2c-4430-9f80-8d685ed3e2f2 | 192.168.0.136 | 192.168.0.140 | | 13:20 |
| baribal | | 22e33986-af2c-4430-9f80-8d685ed3e2f2 | 192.168.0.143 | 192.168.0.162 | | 13:20 |
| baribal | | 22e33986-af2c-4430-9f80-8d685ed3e2f2 | 192.168.0.164 | 192.168.0.197 | | 13:20 |
| baribal | | 22e33986-af2c-4430-9f80-8d685ed3e2f2 | 192.168.0.18 | 192.168.0.45 | | 13:20 |
| baribal | | 22e33986-af2c-4430-9f80-8d685ed3e2f2 | 192.168.0.199 | 192.168.0.219 | | 13:20 |
| sudipto | baribal: Great. So now the problem is for FIP only. | 13:20 |
| baribal | | 22e33986-af2c-4430-9f80-8d685ed3e2f2 | 192.168.0.2 | 192.168.0.4 | | 13:20 |
| baribal | | 22e33986-af2c-4430-9f80-8d685ed3e2f2 | 192.168.0.221 | 192.168.0.234 | | 13:20 |
| baribal | | 22e33986-af2c-4430-9f80-8d685ed3e2f2 | 192.168.0.236 | 192.168.0.254 | | 13:20 |
| baribal | | 22e33986-af2c-4430-9f80-8d685ed3e2f2 | 192.168.0.47 | 192.168.0.61 | | 13:20 |
| baribal | | 22e33986-af2c-4430-9f80-8d685ed3e2f2 | 192.168.0.6 | 192.168.0.16 | | 13:20 |
| baribal | | 22e33986-af2c-4430-9f80-8d685ed3e2f2 | 192.168.0.63 | 192.168.0.73 | | 13:20 |
| baribal | | 22e33986-af2c-4430-9f80-8d685ed3e2f2 | 192.168.0.75 | 192.168.0.101 | | 13:20 |
| baribal | yes | 13:20 |
| baribal | does anyone know if fip also can be recalculated? | 13:20 |
| *** chlong has quit IRC | 13:21 | |
| openstackgerrit | Erik Colnick proposed openstack/neutron: Improve DVR scale performance https://review.openstack.org/150110 | 13:22 |
| *** zhipeng has joined #openstack-neutron | 13:22 | |
| mestery | otherwiseguy: I don't have any obvious ideas, just getting in now and catching up on all the things | 13:23 |
| mestery | otherwiseguy: amotoki seems to think the py27 failure is unrelated to this patch | 13:23 |
| mestery | otherwiseguy: See his last comment in the review | 13:23 |
| amotoki | mestery: otherwiseguy: i am running unit test in my local env. | 13:24 |
| mestery | amotoki: Me too | 13:24 |
| mestery | amotoki: Thanks! | 13:24 |
| *** moha_hunt has quit IRC | 13:25 | |
| *** ilyashakhat has quit IRC | 13:25 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Reuse nova batch notifier https://review.openstack.org/128013 | 13:26 |
| amuller | amotoki: ^ | 13:26 |
| otherwiseguy | amotoki: mestery: it worked locally for me, though I don't think I pulled in the most recent rebased version. | 13:26 |
| baribal | sudipto: bingo! | 13:26 |
| baribal | deleted the whole fip pool id from the table | 13:26 |
| amotoki | otherwiseguy: I think so too. | 13:26 |
| baribal | allocated the new fip to the project and it recalculated fip range | 13:27 |
| *** jckasper has joined #openstack-neutron | 13:27 | |
| amotoki | otherwiseguy: it succeeded in my side too. | 13:27 |
| sudipto | baribal: ok. So what went wrong on the first place? | 13:28 |
| *** zzzeek has joined #openstack-neutron | 13:28 | |
| sudipto | baribal: I am trying some stuff now at my end too. | 13:28 |
| mestery | amotoki: Me too | 13:29 |
| otherwiseguy | Hmm, I guess if someone workflow+1s it we can try again... | 13:29 |
| otherwiseguy | Very odd. | 13:29 |
| mestery | otherwiseguy: I'll go for it. | 13:29 |
| mestery | otherwiseguy: In the merge queue, lets see what happens | 13:30 |
| * otherwiseguy sacrifices a chicken | 13:30 | |
| mestery | lol | 13:31 |
| mestery | otherwiseguy: Sometimes it's a goat, but we won't know for 1.5 hours or so. | 13:31 |
| otherwiseguy | mestery: :D | 13:31 |
| *** baoli has joined #openstack-neutron | 13:32 | |
| otherwiseguy | looks like there are a couple of things stuck in the check queue 14 hours waiting on large-ops. | 13:33 |
| otherwiseguy | nm, apparently my page just hadn't updated correctly or something. | 13:33 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Prepare Base(OVS)LinuxTestCase transformation in helpers https://review.openstack.org/165818 | 13:33 |
| otherwiseguy | Oh, no, there they are. | 13:33 |
| otherwiseguy | huh. | 13:33 |
| baribal | sudipto: I dunno what went wrong. I just had first_ip in DB for the fip pool which was assigned to some VM already. So I can't create the new fips as it complained on duplicate record (ipallocations table had this ip record as it was assigned to some VM). | 13:35 |
| *** Longgeek_ has quit IRC | 13:35 | |
| *** Marga_ has quit IRC | 13:35 | |
| sudipto | ok | 13:36 |
| baribal | at least I now know how to trigger range recalculation | 13:36 |
| sudipto | yeah :) | 13:36 |
| baribal | just delete a pool record from ipavailabilityranges | 13:36 |
| mestery | otherwiseguy: That's an ominous sign :) | 13:36 |
| *** lykinsbd has joined #openstack-neutron | 13:38 | |
| mestery | HenryG: I think both V6 BPs are not goign to make fully make Kilo. | 13:39 |
| mestery | HenryG: Thoughts? The V6 router one had a single patch but it's stuck behind some other patches I believe. | 13:39 |
| *** dboik has joined #openstack-neutron | 13:40 | |
| *** popw has quit IRC | 13:42 | |
| openstackgerrit | enikanorov proposed openstack/neutron: Use convenience method from db api to create nested transaction https://review.openstack.org/165822 | 13:43 |
| HenryG | mestery: The multiple prefix patch is https://review.openstack.org/149068 which mainly needs to have a functional test added. I agree that makes it unlikely to be ready for today. | 13:44 |
| *** topshare has joined #openstack-neutron | 13:44 | |
| mestery | HenryG: Yup, it's possible we could do an FFE for one of these if they are close. | 13:44 |
| HenryG | mestery: Thanks, I feel if adding the functional test confirms that the patch is working fine, then I think asking for a FFE is in order. | 13:45 |
| mestery | HenryG: Please proceed with that path, it may be the best. | 13:45 |
| *** julien_ZTE has joined #openstack-neutron | 13:46 | |
| *** shwetaap has joined #openstack-neutron | 13:46 | |
| *** ishant has quit IRC | 13:47 | |
| atuvenie | amotoki: I am looking over the tests right now, patch https://review.openstack.org/#/c/160596/18 | 13:48 |
| *** numan has quit IRC | 13:51 | |
| amotoki | atuvenie: my -1 is not strong. I just want to clarify we need more check. As far as I check, the failure occurs consistently in the past patch sets. | 13:51 |
| *** ronis has quit IRC | 13:53 | |
| pavel_bondar | mestery,carl_baldwin: hi | 13:54 |
| *** zhhuabj_ has quit IRC | 13:55 | |
| *** rushil has joined #openstack-neutron | 13:56 | |
| *** akanksha has quit IRC | 13:57 | |
| *** irenab has quit IRC | 13:57 | |
| *** padkrish has joined #openstack-neutron | 13:57 | |
| *** wuhg has joined #openstack-neutron | 13:59 | |
| *** yeungp has joined #openstack-neutron | 13:59 | |
| *** yfried is now known as yfried|afk | 14:01 | |
| *** Murali has quit IRC | 14:07 | |
| *** thomasem has joined #openstack-neutron | 14:07 | |
| openstackgerrit | Henry Gessau proposed openstack/neutron: WIP: Drop support for SQL Schema Downgrades https://review.openstack.org/163787 | 14:08 |
| amuller | amotoki: about https://review.openstack.org/#/c/126188/17/neutron/api/rpc/handlers/l3_rpc.py | 14:08 |
| amotoki | amuller: hi | 14:08 |
| amuller | hi :) | 14:08 |
| amuller | Juno agents don't call the old or the new method | 14:08 |
| *** yfried|afk is now known as yfried | 14:09 | |
| *** padkrish has quit IRC | 14:09 | |
| amuller | Kilo agents use the new method | 14:09 |
| *** thomasem has quit IRC | 14:09 | |
| openstackgerrit | Henry Gessau proposed openstack/neutron: WIP: Drop support for SQL Schema Downgrades https://review.openstack.org/163787 | 14:09 |
| amotoki | amuller: does juno agent use version 1.4 RPC? | 14:09 |
| *** csoukup has joined #openstack-neutron | 14:09 | |
| amuller | amotoki: no | 14:09 |
| *** padkrish has joined #openstack-neutron | 14:09 | |
| *** lukasa has joined #openstack-neutron | 14:09 | |
| *** thomasem has joined #openstack-neutron | 14:09 | |
| *** lukasa_ has joined #openstack-neutron | 14:10 | |
| *** padkrish_ has joined #openstack-neutron | 14:11 | |
| amotoki | amuller: it is good from that point of view. Generally speaking RPC version is used to check the method is available or not even if it is in the release. This is the reason behind my comment. | 14:11 |
| amotoki | amuller: do you have any concern about bumping the version to 1.5? | 14:11 |
| *** lukasa_work has quit IRC | 14:11 | |
| amuller | amotoki: Not at all, I'm just trying to fully understand the reason for it, which I think I do | 14:11 |
| amotoki | this is the point I haven't fully understood. | 14:11 |
| *** zhipeng has quit IRC | 14:11 | |
| amuller | amotoki: so I'll bump it to 1.5 and rename the method also | 14:11 |
| *** claudiub has joined #openstack-neutron | 14:12 | |
| amotoki | amuller: thanks for understanding. At now neutorn does not support live upgrade completely, but I would like to keep the usage of RPC versioning consistently. | 14:13 |
| amuller | amotoki: About that... | 14:13 |
| amuller | I talked to russellb a bit | 14:13 |
| *** julien_ZTE has quit IRC | 14:13 | |
| amuller | amotoki: He told me that Nova has a job for rolling upgrades with Nova network | 14:13 |
| *** claudiub_ has quit IRC | 14:14 | |
| *** padkrish has quit IRC | 14:14 | |
| *** ajmiller has joined #openstack-neutron | 14:14 | |
| amuller | amotoki: I might ask John Schwarz to look in to activating a similar job with Neutron | 14:14 |
| amotoki | amuller: yes. nova supports rolling upgrade. | 14:14 |
| amuller | amotoki: And I understood from a couple of operators that they successfully performed a rolling upgrade from I to J | 14:14 |
| *** lukasa has quit IRC | 14:14 | |
| amuller | amotoki: Do you know of a specific reason why Neutron does not support rolling upgrades properly, or what is the gap there? | 14:14 |
| *** TrevorV_ has joined #openstack-neutron | 14:14 | |
| amotoki | amuller: Precisely speaking we need version capping as nova does to support rolliing upgrade. | 14:15 |
| *** setuid has quit IRC | 14:15 | |
| *** julien_ZTE has joined #openstack-neutron | 14:15 | |
| amuller | amotoki: What is 'version capping'? Can you point me to a document or any resource about that? | 14:15 |
| *** ijw has joined #openstack-neutron | 14:16 | |
| *** ijw has quit IRC | 14:16 | |
| amotoki | amuller: on the other hand, neutron RPC interfaces are split into several pieces and each interface is not changed so much between releases. I think this is the reason that neutron rolling upgrade works. | 14:16 |
| *** ijw has joined #openstack-neutron | 14:17 | |
| amotoki | amuller: nova has "compat" mode in its settings. it means that nova use the old version of RPC. | 14:17 |
| amuller | amotoki: btw did you see https://review.openstack.org/#/c/163676/ ? | 14:17 |
| amotoki | amuller: this is "version capping" I mean. | 14:17 |
| amuller | amotoki: gotcha | 14:17 |
| *** setuid has joined #openstack-neutron | 14:17 | |
| mestery | HenryG: I'm going to move both of the V6 BPs to RC1 for now, if your functional work doesn't yield results we'll punt them from Kilo. FYI. | 14:17 |
| amotoki | amuller: rpc namespace is a different thing. | 14:18 |
| HenryG | mestery: ack, thanks! | 14:18 |
| amotoki | amuller: I am very happy with your oslo.messaging work. | 14:18 |
| amuller | amotoki: :) just waiting for an oslo messaging release | 14:18 |
| openstackgerrit | Ed Bak proposed openstack/neutron: Break calls to get_devices_details_list into subsets https://review.openstack.org/163594 | 14:19 |
| mestery | amuller: I don't think your HA Router master patches will land today, I'm moving that to the RC, I'll work with marun to see if we can finish it in the RC. Thanks! | 14:19 |
| amuller | mestery: We might surprise you :) | 14:19 |
| *** dhellmann has quit IRC | 14:19 | |
| HenryG | mestery: Also for consideration in Kilo maybe? https://review.openstack.org/152337 | 14:19 |
| *** dhellmann has joined #openstack-neutron | 14:19 | |
| HenryG | mestery: patch series here: https://review.openstack.org/163787 | 14:20 |
| *** localloop127 has joined #openstack-neutron | 14:20 | |
| mestery | HenryG: Tagged as RC1, thanks! | 14:21 |
| HenryG | mestery: another -1000 LOC :) | 14:21 |
| *** ronis has joined #openstack-neutron | 14:21 | |
| mestery | HenryG: Yay! | 14:21 |
| *** rushil_ has joined #openstack-neutron | 14:22 | |
| *** rushil has quit IRC | 14:23 | |
| *** xuhanp_ has quit IRC | 14:23 | |
| *** xuhanp has quit IRC | 14:23 | |
| *** jprovazn has joined #openstack-neutron | 14:27 | |
| *** yamamoto has quit IRC | 14:27 | |
| salv-orlando | HenryG: We already realized in the past release cycle that the downgrade was useful only in relatively small circumstances. | 14:28 |
| salv-orlando | Like aborting an upgrade immediately | 14:28 |
| salv-orlando | but generally useless | 14:28 |
| HenryG | salv-orlando: Seems like the TC has finally seen the light | 14:29 |
| salv-orlando | as the cross-project spec has been approved, I guess we can get rid of them | 14:29 |
| salv-orlando | it's a low-hanging-fruit blueprint I think | 14:29 |
| mestery | otherwiseguy: https://review.openstack.org/#/c/152761/ failed again | 14:29 |
| salv-orlando | it does not take a lot of experience in neutron to remove the downgrade migrations ;) | 14:29 |
| HenryG | salv-orlando: OK. I made a bug which I thought could be marked as wishlist. | 14:29 |
| mestery | otherwiseguy: We need to sort that out, and if we can't do it in the next few hours, I'll move it to RC and we can have a bit more time, sound ok? | 14:29 |
| akamyshnikova | salv-orlando, in fact I already create a change for this :) | 14:30 |
| otherwiseguy | I'll take a look. It's weird, because it didn't used to happen and no changes were made between when it worked and when it didn't. | 14:30 |
| otherwiseguy | And it is a unit test and my stuff doesn't even touch anything around it. | 14:31 |
| mestery | otherwiseguy: No worries, we'll get it, and even if it doesn't make Kilo-3, we'll get it in for RC | 14:31 |
| *** rpothier has joined #openstack-neutron | 14:32 | |
| *** ByPasS has joined #openstack-neutron | 14:32 | |
| *** absubram has joined #openstack-neutron | 14:32 | |
| otherwiseguy | mestery: At 12:04 it passed the check py27, but failed the gate py27 at 1:05. And has failed the check py27 since. | 14:33 |
| *** yamamoto has joined #openstack-neutron | 14:33 | |
| otherwiseguy | That is really really odd. | 14:33 |
| salv-orlando | akamyshnikova: I've got too many changes in my review queue... can you give me a quick link to yours? | 14:33 |
| mestery | otherwiseguy: Weird | 14:33 |
| salv-orlando | otherwiseguy: some partch which at 12:04 was in the gate queue caused your tests to fail | 14:34 |
| mestery | Fols: FYI, once https://review.openstack.org/152093 lands, we're likely to tag Kilo-3. | 14:34 |
| mestery | Folks | 14:34 |
| mestery | not fols | 14:34 |
| salv-orlando | because patches are stacked one upon the other when they enter the gate queue | 14:34 |
| * mestery doesn't even know what a fols is | 14:34 | |
| akamyshnikova | HenryG, salv-orlando probably it will be good to create a testcase that will check that migration do not contain downgrade | 14:34 |
| pc_m | mestery: fools? :) | 14:35 |
| akamyshnikova | salv-orlando, I based on HenryG change https://review.openstack.org/165740 | 14:35 |
| mestery | pc_m: lol, more likely ;) | 14:35 |
| *** achanda has joined #openstack-neutron | 14:35 | |
| -amotoki- is surprised the gate queue is so short. | 14:35 | |
| *** shaohe_feng has joined #openstack-neutron | 14:36 | |
| otherwiseguy | salv-orlando: yeah, but also checking out the patch locally, tests pass. | 14:37 |
| otherwiseguy | salv-orlando: And it seems very odd that my ovsdb patch would be able to have any effect on get_service_plugins() returning a KeyError for L3_ROUTER_NAT. | 14:38 |
| otherwiseguy | The patch is almost completely self-contained even. | 14:41 |
| otherwiseguy | SO ODD. | 14:42 |
| *** leenheer has joined #openstack-neutron | 14:42 | |
| salv-orlando | otherwiseguy: k I'll run your patch locally too and I'll let you know | 14:42 |
| HenryG | akamyshnikova: Yes, a test case is a good idea. | 14:42 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Send notification to controller about HA router state change https://review.openstack.org/126188 | 14:42 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Expose ha_state per router to agent binding via API https://review.openstack.org/126189 | 14:42 |
| *** baoli has quit IRC | 14:42 | |
| amuller | amotoki: ^ | 14:42 |
| openstackgerrit | Gal Sagie proposed openstack/neutron: Validate when DVR enabled, l2_pop is also enabled https://review.openstack.org/165311 | 14:42 |
| *** baoli has joined #openstack-neutron | 14:43 | |
| *** achanda has quit IRC | 14:43 | |
| salv-orlando | otherwiseguy: running them now... if you'll have to post another patchset to gerrit pls remember you have a typo in the commit message | 14:44 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Prepare Base(OVS)LinuxTestCase transformation in helpers https://review.openstack.org/165818 | 14:45 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform BaseLinuxTestCase methods in helpers https://review.openstack.org/159852 | 14:45 |
| *** baoli has quit IRC | 14:45 | |
| otherwiseguy | salv-orlando: I do? | 14:45 |
| otherwiseguy | implementation? | 14:46 |
| otherwiseguy | I can never type that. | 14:46 |
| otherwiseguy | :p | 14:46 |
| *** anilvenkata has quit IRC | 14:46 | |
| *** baoli has joined #openstack-neutron | 14:47 | |
| openstackgerrit | Sudipta Biswas proposed openstack/neutron: Improve error detection on agent registration https://review.openstack.org/165539 | 14:47 |
| openstackgerrit | Nikolay Fedotov proposed openstack/neutron: Update test_create_show_delete_firewall test to make it test 'fwaas insertion mode' https://review.openstack.org/165859 | 14:47 |
| *** yfried is now known as yfried|afk | 14:48 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform BaseLinuxTestCase methods in helpers https://review.openstack.org/159852 | 14:48 |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Fix validation of physical network name for flat nets https://review.openstack.org/160842 | 14:49 |
| *** timbyr_ has joined #openstack-neutron | 14:49 | |
| *** prad has joined #openstack-neutron | 14:49 | |
| *** timbyr_ has quit IRC | 14:49 | |
| *** abehl has quit IRC | 14:50 | |
| *** jlanoux has quit IRC | 14:51 | |
| haleyb | otherwiseguy: fwiw i do get two errors running tox -e py27 with https://review.openstack.org/#/c/152761/ - one is that L3_ROUTER_NAT one, another keyerror in port security test | 14:51 |
| otherwiseguy | haleyb: do you get the failures if you run just the failing test by itself? | 14:52 |
| *** yfried|afk is now known as yfried | 14:52 | |
| *** carl_baldwin has joined #openstack-neutron | 14:54 | |
| salv-orlando | otherwiseguy: I had success on my dev vm which is a 2-core | 14:54 |
| salv-orlando | so we have two possibilities | 14:54 |
| otherwiseguy | haleyb: or does it just happen when you run all of the tests (some weird import issue?) | 14:55 |
| salv-orlando | 1) the ut failures are introduced by some patch which is still in the gate queue | 14:55 |
| haleyb | otherwiseguy: i also see that failure on master, so i'm confused | 14:55 |
| otherwiseguy | haleyb: that makes me feel both better and worse. :) | 14:55 |
| salv-orlando | 2) your patch is introducing a change in unit tests which expose some issue arising from concurrency among threads | 14:55 |
| salv-orlando | like, for instance, concurrent modifications to global variables | 14:56 |
| haleyb | but just running 'tox -e py27 neutron.tests.unit.test_extension_ext_gw_mode' is fine | 14:56 |
| otherwiseguy | salv-orlando: the weird thing is, none of my code should be involved in *any* unit tests. | 14:56 |
| salv-orlando | otherwiseguy: but you do add unit tests don't you? | 14:56 |
| otherwiseguy | salv-orlando: no, just functional. | 14:56 |
| *** mlavalle has joined #openstack-neutron | 14:57 | |
| otherwiseguy | The only change that should effect unit tests is adding the get_port_ofport call in ovs_lib. | 14:57 |
| salv-orlando | that 16 lines you remove might be enough to trigger a disaster ;) | 14:57 |
| salv-orlando | otherwiseguy: not really | 14:57 |
| otherwiseguy | And those should only effect the two replace_port tests (since unit tests with the Retry stuff are dicey and that stuff is covered by existing functional tests). | 14:57 |
| *** irenab has joined #openstack-neutron | 14:58 | |
| otherwiseguy | Everything else I do is in its own little world. | 14:58 |
| *** lukasa_ has quit IRC | 14:58 | |
| *** chandankumar_ has joined #openstack-neutron | 14:58 | |
| otherwiseguy | and disabled by default. | 14:58 |
| *** baribal has quit IRC | 14:58 | |
| haleyb | salv-orlando: i do see that "KeyError: 'L3_ROUTER_NAT'" failure even on master | 14:58 |
| salv-orlando | otherwiseguy: I'm sure it's not something you did. But it seems things indicate your patch is triggering this UT failure. So we need to understand exactly why... otherwise until jenkins says -1 this patch won't go in | 14:59 |
| salv-orlando | haleyb: perhaps this patch is exacerbating the issue | 14:59 |
| *** armax has joined #openstack-neutron | 14:59 | |
| haleyb | i only see it running the full tox suite, not the test individually. i'm on a 16-core dev system | 15:00 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform BaseOVSLinuxTestCase methods in helpers https://review.openstack.org/165866 | 15:01 |
| *** dboik_ has joined #openstack-neutron | 15:01 | |
| * otherwiseguy is only on two core | 15:01 | |
| *** padkrish_ has quit IRC | 15:01 | |
| *** padkrish has joined #openstack-neutron | 15:01 | |
| * otherwiseguy preemptively blames eventlet | 15:01 | |
| otherwiseguy | Just because. | 15:02 |
| * otherwiseguy is going to push a rebased version with a fix to the commit message and help message change | 15:02 | |
| *** yfried is now known as yfried|afk | 15:02 | |
| mestery | otherwiseguy: lol | 15:03 |
| *** localloop127 has quit IRC | 15:03 | |
| openstackgerrit | Terry Wilson proposed openstack/neutron: Add native OVSDB implementation of OVSDB API https://review.openstack.org/152761 | 15:03 |
| * otherwiseguy kind of wants to scream | 15:03 | |
| mestery | otherwiseguy: That bad? | 15:04 |
| *** dboik has quit IRC | 15:04 | |
| otherwiseguy | mestery: it's just so close and the failure seems to be something that should be nearly impossible to even be related to any of the code in the patch. maddening. | 15:04 |
| mestery | otherwiseguy: :) | 15:04 |
| otherwiseguy | It's a rollercoaster. YAY! NOOO! YAY! NOOO! | 15:05 |
| *** dave-mccowan has quit IRC | 15:05 | |
| haleyb | otherwiseguy: that time tox was fine for me, i'll keep running and try a bisect but it is a race condition | 15:05 |
| mestery | otherwiseguy: To avoid the rush, moving your BP into RC1 | 15:05 |
| mestery | otherwiseguy: Less pressure that way | 15:05 |
| *** dave-mccowan has joined #openstack-neutron | 15:06 | |
| *** dave-mccowan has quit IRC | 15:06 | |
| *** Networkn3rd has joined #openstack-neutron | 15:06 | |
| *** roeyc has quit IRC | 15:07 | |
| *** lukasa has joined #openstack-neutron | 15:07 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron: Validate when DVR enabled, l2_pop is also enabled https://review.openstack.org/165311 | 15:09 |
| *** evgenyf has joined #openstack-neutron | 15:11 | |
| *** Marga_ has joined #openstack-neutron | 15:11 | |
| *** TrevorV_ has quit IRC | 15:11 | |
| openstackgerrit | Nikolay Fedotov proposed openstack/neutron-vpnaas: Scenario test for vpnaas: ipsec-site-connection https://review.openstack.org/159746 | 15:12 |
| openstackgerrit | Sandhya Dasu proposed openstack/neutron: Cisco UCS Manager ML2 Mechanism Driver https://review.openstack.org/155436 | 15:12 |
| *** ahmed__ has joined #openstack-neutron | 15:13 | |
| openstackgerrit | Merged openstack/neutron-fwaas: Implementation of Mcafee NGFW Driver https://review.openstack.org/152093 | 15:13 |
| *** devweasel has quit IRC | 15:13 | |
| *** jorgem has joined #openstack-neutron | 15:13 | |
| ahmed__ | Hello guys I have a problem when setup openstack Juno and integrate it with Opendaylight | 15:13 |
| *** lionelz has joined #openstack-neutron | 15:13 | |
| atuvenie | mestery: since my patch https://review.openstack.org/#/c/160596/ depends on otherwiseguy's patch, how should I proceed given that that one was moved to RC1? Should I still keep the dependency? | 15:13 |
| ahmed__ | I have this error : Failure creating Net_ID | 15:14 |
| *** xgerman has joined #openstack-neutron | 15:14 | |
| mestery | atuvenie: Yes, and I moved your BP to RC1 as well | 15:14 |
| * otherwiseguy stares at zuul | 15:16 | |
| otherwiseguy | mestery: The OpenStack foundation could probably make quite a bit of money if zuul had the option to allow you to pay to move your patch to the front of the line. Just sayin'. | 15:17 |
| mestery | otherwiseguy: rofl | 15:17 |
| mestery | otherwiseguy: Have you considered a career in marketing? | 15:17 |
| atuvenie | mesery: ok, btw, when exactly in april is RC1? | 15:17 |
| atuvenie | mesery: also, do I have to ask for a official FFE? | 15:18 |
| mestery | atuvenie: No, I've granted it for now, RCs are cut in the coming weeks, sooner is better. | 15:19 |
| mestery | atuvenie: Lets get otherwiseguy's patch in and then focus on yours, maybe you can have him review yorus even to provide extra ammo :) | 15:20 |
| otherwiseguy | mestery: A career in marketing would probably kill me. | 15:20 |
| otherwiseguy | I'd at least be dead inside. :p | 15:21 |
| mestery | lol | 15:21 |
| *** dannywilson has joined #openstack-neutron | 15:21 | |
| *** padkrish has quit IRC | 15:22 | |
| *** yfried|afk is now known as yfried | 15:23 | |
| openstackgerrit | Nikolay Fedotov proposed openstack/neutron-vpnaas: Scenario test for vpnaas: ipsec-site-connection https://review.openstack.org/159746 | 15:24 |
| *** ChuckC has joined #openstack-neutron | 15:24 | |
| *** dannywil_ has joined #openstack-neutron | 15:24 | |
| *** dannywilson has quit IRC | 15:26 | |
| haleyb | otherwiseguy: i can't get any failures now with your patch or master, so maybe jenkins will +1 you this time :) | 15:26 |
| * otherwiseguy nominates haleyb's dev machine for inclusion in the jenkins cluster | 15:27 | |
| otherwiseguy | haleyb: thanks for help trying to track this crazy thing down. | 15:27 |
| otherwiseguy | I'm kind of at a loss. | 15:28 |
| otherwiseguy | Every single time I've looked at failing code and said that it was impossible that it could be failing I've been wrong, but I'm going to keep trying. | 15:28 |
| HenryG | Where's kevinbenton? He seems to be very good at tracking this kind of thing down. | 15:28 |
| *** jlanoux has joined #openstack-neutron | 15:29 | |
| *** chandankumar_ has quit IRC | 15:29 | |
| *** aepifanov has quit IRC | 15:30 | |
| openstackgerrit | Assaf Muller proposed openstack/python-neutronclient: Add HA router state to l3-agent-list-hosting-router https://review.openstack.org/126182 | 15:33 |
| haleyb | otherwiseguy: np | 15:33 |
| mestery | HenryG: It's between 4AM-12PM PDT, so he's asleep | 15:34 |
| *** localloop127 has joined #openstack-neutron | 15:35 | |
| HenryG | mestery: ah yes, the KB timezone | 15:36 |
| mestery | HenryG: rofl | 15:36 |
| *** reed has joined #openstack-neutron | 15:37 | |
| *** topshare has quit IRC | 15:37 | |
| *** lazy_prince is now known as killer_prince | 15:37 | |
| *** Bsony has quit IRC | 15:39 | |
| openstackgerrit | Rajeev Grover proposed openstack/neutron: HA for DVR - Neutron Server side code changes https://review.openstack.org/143169 | 15:40 |
| *** sudipto has quit IRC | 15:40 | |
| *** _cjones_ has joined #openstack-neutron | 15:41 | |
| *** rotbeard has quit IRC | 15:44 | |
| amuller | armax: Good morning | 15:45 |
| *** chandankumar_ has joined #openstack-neutron | 15:45 | |
| *** rotbeard has joined #openstack-neutron | 15:46 | |
| *** Trozz has quit IRC | 15:46 | |
| *** Trozz has joined #openstack-neutron | 15:47 | |
| *** marun has joined #openstack-neutron | 15:49 | |
| armax | amuller: morning | 15:51 |
| amuller | armax: marun: Remaining patches for your consideration: https://review.openstack.org/#/q/status:open+branch:master+topic:bp/report-ha-router-master,n,z | 15:52 |
| *** chandankumar_ has quit IRC | 15:52 | |
| *** padkrish has joined #openstack-neutron | 15:52 | |
| armax | amuller: ok | 15:53 |
| marun | amuller: re: https://review.openstack.org/#/c/126189/ | 15:54 |
| marun | amuller: this is affecting the rest api? | 15:54 |
| *** nlahouti has joined #openstack-neutron | 15:54 | |
| amuller | marun: yes, a verb will now return an extra attribute | 15:54 |
| amuller | DocImpact? :) | 15:54 |
| marun | amuller: api test change | 15:55 |
| openstackgerrit | Merged openstack/neutron: Add a missing mock in DHCPAgentWeightSchedulerTestCase https://review.openstack.org/165722 | 15:56 |
| *** gampel has quit IRC | 15:56 | |
| amuller | marun: Keep in mind we can't create HA routers unless you have 2+ L3 agents registered | 15:57 |
| marun | amuller: hmmm | 15:57 |
| *** ferest has joined #openstack-neutron | 15:59 | |
| *** jorgem has quit IRC | 15:59 | |
| *** localloop127 has quit IRC | 16:00 | |
| *** garyk has joined #openstack-neutron | 16:00 | |
| amuller | marun: looks like there's 1 API test that uses the affected verb: test_add_list_remove_router_on_l3_agent | 16:00 |
| *** tidwellr has joined #openstack-neutron | 16:01 | |
| *** baoli has quit IRC | 16:02 | |
| *** apuimedo has quit IRC | 16:02 | |
| *** baoli has joined #openstack-neutron | 16:03 | |
| *** ferest has quit IRC | 16:03 | |
| dougwig | marun: are we clear now to move the tempest lbaas api tests from neutron to neutron-lbaas? | 16:03 |
| marun | dougwig: I think so, yes. | 16:04 |
| openstackgerrit | Dane LeBlanc proposed openstack/neutron: No allocation needed for specific IPv6 SLAAC addr assignment https://review.openstack.org/164943 | 16:04 |
| dougwig | marun: ok, expect that next week. | 16:04 |
| *** yatin has joined #openstack-neutron | 16:04 | |
| marun | dougwig: The outstanding issue is going to be running the tests against stable branches and allowing reuse by refstack so that we can remove from tempest entirely. | 16:05 |
| marun | dougwig: But no point in trying to do everything at once. | 16:05 |
| *** rook-tmp has quit IRC | 16:06 | |
| dougwig | marun: i wonder if tempest needs to be having "external plugin" references, like devstack (and now our CLI), so they can be a single gatekeeper without needing to control the content. | 16:06 |
| *** rook_ has joined #openstack-neutron | 16:06 | |
| marun | dougwig: I don't think so, frankly. | 16:06 |
| dougwig | might also save burning yet another node for every repo's tests. | 16:07 |
| marun | dougwig: short-term, we'll waste some vm's | 16:07 |
| marun | dougwig: long-term, we can run api tests against test-managed daemons. | 16:07 |
| marun | dougwig: It's likely they won't have any more requirements than unit tests | 16:08 |
| dougwig | marun: ok. you're closer to it, so i'll leave that worry to you. :) | 16:08 |
| *** tflynn has joined #openstack-neutron | 16:08 | |
| marun | dougwig: so tempest should be able to run api tests for multiple projects via sequential 'tox -eapi' calls on a single node | 16:08 |
| *** Rajeev has joined #openstack-neutron | 16:08 | |
| *** julien_ZTE has quit IRC | 16:08 | |
| dougwig | marun: i was planning on doing lbaasv1/v2 on the same node, in a similar fashion. | 16:09 |
| *** julien_ZTE has joined #openstack-neutron | 16:09 | |
| marun | dougwig: have you done any work on test-managed daemons? or are you waiting on john's fullstack stuff? | 16:09 |
| *** tflynn has quit IRC | 16:09 | |
| dougwig | marun: waiting. right now we're just using a pinned tempest via tox, inside a standard devstack-gate. | 16:10 |
| openstackgerrit | Michael Smith proposed openstack/neutron: manual add/remove router for dvr_snat agent https://review.openstack.org/122024 | 16:10 |
| marun | dougwig: and do you do separate deployment for v1 and v2? or both at the same time? | 16:10 |
| dougwig | marun: right now, the only in-tree job is v2. when I move the tempest stuff out of neutron (which is v1), i was going to try to merge it into the same job and just call tox twice, or i can burn yet another node. i hate that, since 85% of the job time is devstack setup. | 16:11 |
| *** belmoreira has quit IRC | 16:12 | |
| marun | dougwig: yeah, devstack is too expensive for this kind of testing | 16:12 |
| marun | dougwig: I think the example of the neutron functional job setup is instructive. the job doesn't take much more than the unit test job | 16:13 |
| *** julien_ZTE has quit IRC | 16:13 | |
| dougwig | marun: it still requires an underlying devstack, right? | 16:13 |
| marun | dougwig: it does, but it only uses devstack as a way to consistently install and configure system dependencies | 16:14 |
| *** roeyc has joined #openstack-neutron | 16:14 | |
| marun | dougwig: no installation or init of openstack services | 16:14 |
| dougwig | ahh. | 16:14 |
| *** aswadr has quit IRC | 16:14 | |
| marun | dougwig: and it limits the repos that get deployed to the node via a control variable in the job config | 16:14 |
| marun | dougwig: repo deployment to a new node, by itself, takes 3-4m | 16:15 |
| *** scheuran has quit IRC | 16:15 | |
| dougwig | so linux only, needs network, but a VM and one-time setup can mostly cover that for everyone's local environments. | 16:16 |
| marun | I think so, yes | 16:16 |
| dougwig | and the neutron api job, it launches neutron, or is it doing similar magic? | 16:16 |
| marun | the api job is still the traditional 'deploy a cloud with devstack' | 16:16 |
| marun | we'll probably need to keep that as an option, but the goal is to run against test-managed daemon by default | 16:17 |
| *** jorgem has joined #openstack-neutron | 16:17 | |
| marun | probably api testing against devstack-deployed daemon will become a periodic job to make sure nothing falls through the cracks | 16:17 |
| marun | e.g. keytone interaction | 16:18 |
| marun | but we shouldn't have to validate that on every patch | 16:18 |
| dougwig | makes sense. would be nice to get those check runs under 10 minutes or so. | 16:18 |
| openstackgerrit | Zengfa Gao proposed openstack/python-neutronclient: Add Neutron subnetpool API https://review.openstack.org/159618 | 16:18 |
| marun | it's doable | 16:18 |
| marun | at least, until we start piling on the expensive tests ;) | 16:19 |
| marun | it's been a struggle at times to get people to use functional testing responsibly | 16:19 |
| marun | unit tests should still be the default, since they are cheap | 16:19 |
| openstackgerrit | Carl Baldwin proposed openstack/neutron: Move external port processing to router classes https://review.openstack.org/158495 | 16:19 |
| *** leenheer has quit IRC | 16:20 | |
| *** dboik_ has quit IRC | 16:21 | |
| *** amotoki has quit IRC | 16:22 | |
| blogan | ihrachyshka: ping | 16:22 |
| *** dboik has joined #openstack-neutron | 16:22 | |
| *** ihrachyshka has quit IRC | 16:22 | |
| *** Marga_ has quit IRC | 16:23 | |
| *** emagana has joined #openstack-neutron | 16:26 | |
| *** padkrish has quit IRC | 16:28 | |
| *** sbfox has joined #openstack-neutron | 16:29 | |
| openstackgerrit | Carl Baldwin proposed openstack/neutron: Move create_dvr_fip_interfaces in to DVR https://review.openstack.org/159663 | 16:30 |
| *** localloop127 has joined #openstack-neutron | 16:30 | |
| *** mlavalle has quit IRC | 16:31 | |
| otherwiseguy | haleyb: looks like it failed with the same stupid error in the check queue again. | 16:32 |
| * otherwiseguy sighs | 16:32 | |
| *** Bsony has joined #openstack-neutron | 16:32 | |
| *** tsg_ has joined #openstack-neutron | 16:33 | |
| *** ronis has quit IRC | 16:36 | |
| *** chandankumar_ has joined #openstack-neutron | 16:37 | |
| *** yfried has quit IRC | 16:37 | |
| *** padkrish has joined #openstack-neutron | 16:40 | |
| openstackgerrit | Zengfa Gao proposed openstack/python-neutronclient: Aadd Neutron subnet-create with subnetpool https://review.openstack.org/165910 | 16:41 |
| *** sbelous has quit IRC | 16:41 | |
| openstackgerrit | Zengfa Gao proposed openstack/python-neutronclient: Add Neutron subnet-create with subnetpool https://review.openstack.org/165910 | 16:41 |
| *** sbelous has joined #openstack-neutron | 16:41 | |
| *** thomasem has quit IRC | 16:41 | |
| *** ahmed__ has left #openstack-neutron | 16:42 | |
| *** nyechiel__ has quit IRC | 16:43 | |
| *** aranjan has joined #openstack-neutron | 16:43 | |
| *** pritesh has joined #openstack-neutron | 16:43 | |
| *** jorgem has quit IRC | 16:45 | |
| sbelous | carl_baldwin: ping | 16:49 |
| carl_baldwin | sbelous: pong | 16:49 |
| sbelous | Hello, Carl! Long ago you been asked to wait with this patch: https://review.openstack.org/#/c/147554/ Do we need wait more time or we already ok with this patch? | 16:49 |
| *** kerpan has quit IRC | 16:51 | |
| *** yamahata has joined #openstack-neutron | 16:52 | |
| carl_baldwin | sbelous: Could you rebase it on top of https://review.openstack.org/#/c/163222/, which I intend to get in to shape in the next day or two? | 16:52 |
| *** evgenyf has quit IRC | 16:53 | |
| *** baoli has quit IRC | 16:54 | |
| sbelous | carl_baldwin: ok, will be | 16:54 |
| carl_baldwin | sbelous: Thank you. | 16:54 |
| *** baoli has joined #openstack-neutron | 16:55 | |
| ZZelle | carl_baldwin, hi | 16:57 |
| carl_baldwin | ZZelle: hi | 16:57 |
| ZZelle | carl_baldwin, could you look at https://review.openstack.org/133506 when you have time? | 16:57 |
| carl_baldwin | ZZelle: Yes, I will. | 16:58 |
| ZZelle | carl_baldwin, thx | 16:58 |
| carl_baldwin | ZZelle: Glad to help. | 16:59 |
| *** itzikb has quit IRC | 16:59 | |
| *** banix has joined #openstack-neutron | 16:59 | |
| *** ihrachyshka has joined #openstack-neutron | 17:00 | |
| ZZelle | carl_baldwin, do you think i should/could restore https://review.openstack.org/143869 and sisters ? | 17:01 |
| *** Swami has joined #openstack-neutron | 17:01 | |
| *** sthillma has joined #openstack-neutron | 17:02 | |
| *** ygbo has quit IRC | 17:04 | |
| *** dave-mccowan has joined #openstack-neutron | 17:04 | |
| *** Networkn3rd has quit IRC | 17:05 | |
| *** pcaruana has quit IRC | 17:05 | |
| *** xgerman has quit IRC | 17:05 | |
| carl_baldwin | ZZelle: i can support it. I like encapsulating things like manipulating ip cidrs with strings. | 17:06 |
| *** jistr has quit IRC | 17:07 | |
| *** padkrish has quit IRC | 17:07 | |
| ZZelle | carl_baldwin, should they depend on a specific (l3-refactoring) change? | 17:08 |
| *** harlowja_away is now known as harlowja_ | 17:09 | |
| *** julien_ZTE has joined #openstack-neutron | 17:09 | |
| carl_baldwin | ZZelle: At this point, I don’t think so. I think the conflicts in this case would be pretty easy to deal with. I think I’d rather do conflict resolution with them than lengthen the dependency chain. | 17:09 |
| ZZelle | carl_baldwin, ok, will do! | 17:10 |
| otherwiseguy | salv-orlando: Looking at the test case that keeps throwing the L3_ROUTER_NAT KeyError, I see the test does a backup of the l3.RESOURCE_ATTRIBUTE_MAP, then at cleanup restores it. Is that not totally unsafe if you are running lots of tests at once, all possibly stepping all over the l3.RESOURCE_ATTRIBUTE_MAP? | 17:10 |
| garyk | mestery: armax: you guys around? | 17:11 |
| armax | garyk: around | 17:11 |
| * otherwiseguy isn't really that familiar with that part of the code | 17:11 | |
| garyk | armax: not sure if you saw my mails about the reverts to the list. i think that we should really revert the changes to the network attributes with the mtu and trasparent vlans. | 17:11 |
| garyk | i just wanted to know what you think and why we should actully allow these changes on the base definition. it seems to break our whole model (unless I missed that memo) | 17:12 |
| armax | garyk: I looked at the patches…I am still digging through and I spoke briefly with mestery and salv-orlando | 17:12 |
| *** moshele has quit IRC | 17:12 | |
| garyk | ok. let me know what the concensus is. i just feel it is a lilttle dodgy | 17:13 |
| *** Marga_ has joined #openstack-neutron | 17:13 | |
| armax | garyk: makes sense, I recall a discussion where it was agreed that those attribute really make no sense as extension attributes | 17:13 |
| armax | garyk: but I may be wrong | 17:13 |
| garyk | mtu should be a provider cofnig and transparent vlan too (whatever that is | 17:13 |
| armax | garyk: that said, we need to ensure that we do have a reference implementation that support those attributes wherever they are | 17:13 |
| armax | garyk: otherwise their moot | 17:13 |
| garyk | i am really fine with them as additional extenstions | 17:13 |
| sc68cal | Can we just admit that the process for creating API Extensions is fundamentally broken? At this point it's basically saying to code authors "we don't want your work" | 17:13 |
| *** atuvenie has quit IRC | 17:14 | |
| garyk | in the patches i did not see a reference implementation (maybe they were on additional patches) | 17:14 |
| garyk | sc68cal: i do not understand | 17:14 |
| garyk | sc68cal: you can always add an extension | 17:14 |
| sc68cal | oh? so why is my qos api extension dead? | 17:14 |
| armax | garyk: yes, I think they work is not yet complete, and it may need a FF extension | 17:14 |
| sc68cal | I keep having operators ask about rate limits and other stuff at ops meetups | 17:14 |
| sc68cal | and I have to shrug and say well we don't have that in Neutron | 17:15 |
| garyk | sc68cal: i have no idea. sc68cal if reviews are a concern then i will be happy to look at that | 17:15 |
| *** padkrish has joined #openstack-neutron | 17:15 | |
| armax | sc68cal: not sure how this is relevant to this discussion | 17:15 |
| armax | sc68cal: what are you trying to say? | 17:15 |
| sc68cal | so asking the NFV people to do API extensions, based on the history of other API extensions is basically the same as rejecting the work IMO | 17:15 |
| *** Networkn3rd has joined #openstack-neutron | 17:15 | |
| garyk | sc68cal: lpetrut approached me about qos today. so collaborate | 17:15 |
| ZZelle | sc68cal, we need a way to publish available APIs features ... currently we use extensions | 17:16 |
| garyk | sc68cal: why? i add an extension a few weeks ago. | 17:16 |
| sc68cal | well guess I must be doing something wrong, because my experience trying to propose and submit an api extension upstream has pretty much turned me off to the whole proces | 17:16 |
| *** Marga_ has quit IRC | 17:17 | |
| armax | sc68cal: the reasons why work can be turned down may be very different from one proposal to another | 17:17 |
| *** claudiub has quit IRC | 17:17 | |
| garyk | sc68cal: lets agree on one thing - the process sucks, but having said that, when code is added it should at least follow the guidelines or principles of the project (we all make mistakes - and I am certainly to blame for a ton of them) | 17:17 |
| *** Bsony has quit IRC | 17:17 | |
| armax | garyk: so let’s circle back and revise the approach taken | 17:18 |
| garyk | armax: ok. | 17:18 |
| armax | garyk: again, I was under the impression that the attributed were deemed to be core, but that was an oversight and we do need them to be extension attributes, then so be it | 17:18 |
| garyk | i am really not in favour of this being part of the base stuff - it just does not seem right. i'll wait till you guys come to a conclusion | 17:18 |
| armax | garyk: we can revise, rather than revert | 17:18 |
| armax | garyk: having said that, if we have no functional, and well tested reference implementation, these might as well be removed from Kilo, in the RC window | 17:19 |
| garyk | armax: sure, that works for me. | 17:19 |
| garyk | armax: k. | 17:19 |
| *** Bsony has joined #openstack-neutron | 17:19 | |
| sc68cal | That's a more valid argument, that there is no implementation behind it | 17:19 |
| garyk | the transparent vlan broke our external plugin unit tests | 17:19 |
| sc68cal | I asked the author yesterday in this channel and he said patches are forthcoming | 17:19 |
| armax | garyk: agreed, we should definitely have had this discussion earlier but better late than never ;) | 17:19 |
| openstackgerrit | Abhishek Raut proposed openstack/neutron: Cisco Nexus1000V ML2 Mechanism Driver https://review.openstack.org/157243 | 17:20 |
| armax | garyk: yeah, I noticed that the NSX CI has been silent for quite a bit | 17:20 |
| armax | garyk: any ETA when it’s going to be back online? | 17:20 |
| *** shwetaap has quit IRC | 17:20 | |
| garyk | armax: as soon as we fix this :) | 17:21 |
| armax | garyk: um…ok :) | 17:21 |
| garyk | armax: i need to check and will get back to you | 17:21 |
| armax | garyk: ok | 17:21 |
| pritesh | sc68cal: yep i am still working on it, but if all the ci's are passing i have no way to know that someone has their external stuff broken. | 17:21 |
| pritesh | garyk: is there any log which i can check to fix it instead of reverting that patch ? | 17:22 |
| armax | pritesh: there were initial patches where the CI voted negatively | 17:22 |
| armax | pritesh: I believe... | 17:22 |
| garyk | pritesh: according to the new model we are responsible for the breakages in the CI (so please don't be bothered about that) | 17:23 |
| armax | anyhoo, we’ll fix this | 17:23 |
| pritesh | armax: ok thanks. | 17:23 |
| sc68cal | Does not appear to have voted negatively, on https://review.openstack.org/#/c/158420/ | 17:23 |
| armax | it’s been a combination of factors, lack of CI and lack of thorough review | 17:23 |
| sc68cal | i'm seeing all greens | 17:23 |
| garyk | pritesh: i just do not want you to take me in the wrong way. i am in favor of the addition of the extensions. i just think that they should be done as all other extensions | 17:23 |
| armax | sc68cal: the NSX CI did vote negatively on PS11 | 17:24 |
| armax | but then disappeared | 17:24 |
| armax | I failed to notice until this morning | 17:24 |
| *** pritesh_ has joined #openstack-neutron | 17:24 | |
| garyk | it also feel like this stiff should be in the proivider networks extension | 17:25 |
| sc68cal | That's a problem. | 17:25 |
| armax | garyk: how so? | 17:25 |
| garyk | armax: we have some issues with our ci at the moment. not sure if they are internal cloud of code issues. | 17:25 |
| garyk | mtu? | 17:25 |
| garyk | vlan? | 17:25 |
| garyk | when someone creates a virtyual network they do not know what the backing is? for example GRE | 17:26 |
| *** pask81 has quit IRC | 17:26 | |
| garyk | but then again I was not involved in the review process on these specific patches | 17:26 |
| sc68cal | Well, we already have problems with virtual network MTUs exceeding the MTU of the underlying interface | 17:26 |
| sc68cal | when the encap protocol is added | 17:26 |
| garyk | sc68cal: but that is not something that a tenant should be aware of - it is an admin user | 17:26 |
| sc68cal | Unless the tenant wishes to do advanced NFV things | 17:27 |
| *** john-davidge has quit IRC | 17:27 | |
| *** mwagner_lap has quit IRC | 17:27 | |
| garyk | the lets have a NFV extension | 17:27 |
| *** dane_leblanc has quit IRC | 17:27 | |
| *** vivek-ebay has joined #openstack-neutron | 17:27 | |
| *** sridhar_ram has joined #openstack-neutron | 17:27 | |
| garyk | sc68cal: can you please elaborate on the things that the tenant would want to do? | 17:27 |
| *** achanda has joined #openstack-neutron | 17:28 | |
| openstackgerrit | Merged openstack/neutron: Include IPv6 SLAAC addresses implicitly for port update https://review.openstack.org/161033 | 17:28 |
| *** achanda has quit IRC | 17:28 | |
| garyk | i am not familiar with this stuff so please forgive me | 17:28 |
| *** pritesh has quit IRC | 17:28 | |
| *** sripriya has joined #openstack-neutron | 17:28 | |
| armax | garyk: it feels like this debate should’ve happened ages ago…not sure how fruitful this is going to be now | 17:28 |
| *** fredlhsu has joined #openstack-neutron | 17:28 | |
| *** achanda has joined #openstack-neutron | 17:28 | |
| *** nmagnezi has quit IRC | 17:29 | |
| *** padkrish has quit IRC | 17:29 | |
| pritesh_ | armax: i agree. | 17:29 |
| armax | garyk: all I can say is that this MTU and VLAN stuff is way far from being complete | 17:29 |
| *** wuhg has quit IRC | 17:29 | |
| *** s3wong has joined #openstack-neutron | 17:29 | |
| armax | it needs more time to bake and if we are unable to get to a point where it does what it says on the tin | 17:30 |
| armax | I will be the first one to +2 gary’s revert | 17:30 |
| *** shwetaap has joined #openstack-neutron | 17:30 | |
| armax | for now, let’s make sure we focus on the remaining patches | 17:30 |
| armax | that’s where the help is needed the most | 17:30 |
| sc68cal | garyk: https://github.com/stackforge/telcowg-usecases | 17:31 |
| *** mestery_ has joined #openstack-neutron | 17:31 | |
| sc68cal | I think that's where they plan to put some | 17:31 |
| sc68cal | think that's a skeleton currently, | 17:31 |
| pritesh_ | armax: i am not sure what it means by more time bake? does it mean more things are pending? or something missing in there ? | 17:31 |
| sc68cal | https://wiki.opnfv.org/ipv6_opnfv_project/topdown_usecase | 17:32 |
| garyk | armax: agree - as i said i was not involved in the review of these patches - so i am really coming into the game at the 91st minute | 17:32 |
| garyk | i just think that we have an issue here. i hope that this does not break things. | 17:32 |
| garyk | i have a feeling that it may | 17:32 |
| dougwig | garyk: I agree with you now | 17:33 |
| *** mestery has quit IRC | 17:33 | |
| armax | ipritesh_: it’s my understanding is that you have remaining patches that need to go in right? | 17:33 |
| *** coolsvap is now known as coolsvap|afk | 17:33 | |
| *** patrickeast has joined #openstack-neutron | 17:33 | |
| pritesh_ | armax: the only remainig patch is in python neutron client, which adds support for it | 17:33 |
| *** rotbeard has quit IRC | 17:33 | |
| armax | pritesh_: so as soon as more people get a chance to get their hands on this | 17:34 |
| pritesh_ | armax: the feature works as of now as in specs. | 17:34 |
| armax | pritesh_: more issues will come up and we need to strengthen the codebase | 17:34 |
| * mestery_ catches up on the discussion | 17:34 | |
| armax | pritesh_: no feature has ever been merged bug-free :) | 17:34 |
| ijw | armax: what's your problem with the MTU and VLAN stuff? | 17:35 |
| amuller | otherwiseguy: another failure here: https://review.openstack.org/#/c/128013/ | 17:35 |
| amuller | the py27 thing | 17:35 |
| *** packet has joined #openstack-neutron | 17:35 | |
| amuller | otherwiseguy: would recheck help? | 17:35 |
| armax | ijw: I have no problem with those per se | 17:35 |
| ijw | woah, loads of backlog. Anyone care to summarise? | 17:35 |
| garyk | ijw: do you have your shoes on? if so I will explain | 17:35 |
| garyk | ijw: in short it just seems to add the two new params to the base network attributes | 17:36 |
| pritesh_ | ijw: garyk wants to revert the mtu and vlan patch becuase they ahve ci failure (aparently which doesnt show up when merging the patch in) | 17:36 |
| *** ijw has quit IRC | 17:36 | |
| *** kawa2014 has quit IRC | 17:36 | |
| *** leenheer has joined #openstack-neutron | 17:36 | |
| *** kawa2014 has joined #openstack-neutron | 17:36 | |
| garyk | pritesh_: no, not due to the CI failure. that is our responsibility to fix | 17:36 |
| *** ijw has joined #openstack-neutron | 17:36 | |
| *** thomasem has joined #openstack-neutron | 17:37 | |
| *** claudiub has joined #openstack-neutron | 17:37 | |
| garyk | my problem is that the values for the mtu and tranparent_vlan were added to the base network attributes | 17:37 |
| pritesh_ | garyk: as the time of merge there isnt a single failure care to elaborate? | 17:37 |
| garyk | ijw: pritesh_: https://github.com/openstack/neutron/blob/master/neutron/api/v2/attributes.py#L695 | 17:37 |
| *** kawa2014 has quit IRC | 17:37 | |
| garyk | that is my problem. this should be in a sperate extensuon which is imported into the plugin | 17:37 |
| garyk | that is what is broken | 17:37 |
| dougwig | Is there time to fix it without killing the feature? | 17:38 |
| garyk | dougwig: i think so. it is just a matter of declaring an extension and adding it to the plugin that wishies to consume and expose it | 17:38 |
| *** yalie has quit IRC | 17:38 | |
| ijw | Reading back, I'm interested to see garyk's proposal of an NFV extension, if only he would care to define 'NFV' - the stuff thats' there is pretty basic networking stuff and while it has NFV use cases it's absolutely not NFV (as QoS wouldn't be, either) | 17:39 |
| *** yalie has joined #openstack-neutron | 17:39 | |
| armax | I need to go back to the review patches that introduced these specs | 17:39 |
| mestery_ | I think an NFV extension is going to cause issues, because what exactly is NFV and what isn't? | 17:39 |
| *** mestery_ is now known as mestery | 17:39 | |
| armax | because I have a faint recollection that we discussed/agreeed for these to be core attributes, but I might be wrong | 17:39 |
| dougwig | Can the extension and revert get submitted in the same patch today? | 17:39 |
| armax | also, I recall these these attributes should’ve been admin only | 17:40 |
| armax | and clearly at the moment they aren't | 17:40 |
| mestery | dougwig: We already cut Kilo-3, this will be cleaned up in the rc | 17:40 |
| ijw | And in terms of MTU - an app may know what MTU it needs before it can work, and a network controller may be able to find a bigger MTU segment at some higher cost, which is why it's not an admin extension. Every admin hack to MTU to date has been broken for various reasons | 17:40 |
| pritesh_ | dougwig: i agree, that would be a good way to handle it instead of reverting the feature. | 17:40 |
| armax | dougwig: I think we can just iterate on what we got | 17:40 |
| mestery | armax: ++ | 17:40 |
| armax | there’s a reason why we have an RC window | 17:40 |
| armax | to flush out the remaining issues that we encounter while more eyes are on the damn feature | 17:41 |
| armax | not just VLAN or MTU but any other feature that merged in Kilo | 17:41 |
| armax | or am I just rambling? | 17:42 |
| ijw | garyk: every single user of Openstack benefits from the MTU patches, so I don't really understand why you would want that as an extension. Right now it's pretty much good luck that stops MTUs from screwing over users big time, and the performance frequently sucks that said | 17:42 |
| sc68cal | ++ for mtu stuff. I'm working on seeing how bad we're getting bit by it in a lab environment .... | 17:42 |
| ijw | armax: they're not supposed to be admin only | 17:42 |
| garyk | ijw: as far as I understand MTU is something fo the underlying architecture | 17:43 |
| pritesh_ | ijw, sc68cal: ++ same here | 17:43 |
| *** ivar-lazzaro has joined #openstack-neutron | 17:43 | |
| aveiga | garyk: what if the enants are allowed to use the provider net, sans overlay? I have use cases for this | 17:43 |
| ijw | garyk: that would be why I wrote the spec ;) | 17:43 |
| aveiga | tenants, even | 17:43 |
| amuller | We've had these discussions on the MTU and NFV specs months ago, discussing this a day after the patches get merged is unfortunate | 17:43 |
| garyk | amuller: it is never too late. | 17:44 |
| *** yamamoto has quit IRC | 17:44 | |
| garyk | i just find that we break the model for these and i do not think that is healthy | 17:44 |
| *** Marga_ has joined #openstack-neutron | 17:44 | |
| garyk | if we have a new model then why do we have extensions and an api | 17:44 |
| *** julien_ZTE has quit IRC | 17:44 | |
| garyk | lets just sent one massive blob of data - that will save all of the extensions :) | 17:44 |
| ijw | garyk: it was intended to address two use cases. One is working out the max MTU that would work from the underlying architecture, which is what you're talking about. The other, which is less popular, is discovering if an MTU will work for a network and/or setting a specific MTU regardless of what the system's preference is | 17:45 |
| *** ivar-lazzaro has quit IRC | 17:45 | |
| ijw | The latter's more of an NFV use case but it's also useful if you have VMs that can't pick up their MTU automatically, for instance - if I have a VM that has a 1500MTU and no DHCP server then if the cloud supports 9000MTUs I need to reduce that to get anywhere with running my VM | 17:46 |
| ijw | DHCP client, even | 17:46 |
| *** aranjan has quit IRC | 17:46 | |
| ijw | (IPv6 is worse, because RA advertisement of MTUs has certain extra constraints) | 17:46 |
| garyk | ijw: bu that is something that an admin should control - kind of like a provider network. | 17:46 |
| garyk | the MTU in my opinion is part of a provider network extension | 17:46 |
| ijw | garyk: for the parameters that determine the max MTU the admin does control it - it's all in config | 17:46 |
| garyk | transparent vlan too | 17:46 |
| *** ivar-lazzaro has joined #openstack-neutron | 17:47 | |
| garyk | so we can use the policy.json to enable that | 17:47 |
| ijw | garyk: no, you're wrong | 17:47 |
| *** mlavalle has joined #openstack-neutron | 17:47 | |
| garyk | does amazon let you configure mtu? | 17:47 |
| armax | ijw: I think what gary is proposing is only a technical change | 17:47 |
| armax | ijw: that won’t change the semantic or how this would be used | 17:47 |
| ijw | Firstly VLAN trunks and provider networks have nothing to do with each other | 17:47 |
| *** Marga__ has joined #openstack-neutron | 17:47 | |
| *** crc32 has joined #openstack-neutron | 17:47 | |
| *** chandankumar_ has quit IRC | 17:48 | |
| garyk | ijw: my thinking is that the mtu depends on the underlying networking infra | 17:48 |
| ijw | Secondly provider networks and tenant networks and the external network, to name the three types that exist, can all have different MTUs - and - via admin config - the current extension works out exactly what they are for you. It's not provider network specific but it does have to fold that in | 17:48 |
| ijw | There are two definitions of MTU here | 17:48 |
| *** Marga_ has quit IRC | 17:48 | |
| ijw | One is the max packet size that network will transmit, which is absolutely an infra requirement | 17:48 |
| armax | ijw: moving these attributes from core to extension attributes will have no visible difference for those plugins that support them | 17:48 |
| garyk | armax: agree | 17:48 |
| armax | so really this discussion is moot | 17:48 |
| ijw | The other is the MTU that is set in your VMs - which has to be under that first number and consistent across the VMs | 17:48 |
| *** jorgem has joined #openstack-neutron | 17:49 | |
| armax | so, let’s agree where we want them and get on with it | 17:49 |
| ijw | What I'm missing is why you would not want them to be core | 17:49 |
| ijw | Last I checked every network has an MTU | 17:49 |
| ijw | You might not want them to be user selectable, I will happily concede that | 17:49 |
| armax | ijw: agreed, I am not swayed one way or the other | 17:49 |
| armax | ijw: however, if we force them to be in the core | 17:50 |
| garyk | ijw: i agree that every network has an MTU. i just do not think that the V2 api of a neutron network has a MTU | 17:50 |
| garyk | v3 yes. but not v2. a v2 extsnsion yes. | 17:50 |
| armax | ijw: but the model has always been that if they’re in teh core, | 17:50 |
| armax | ijw: every plugin must support it | 17:50 |
| ijw | Both APIs are designed with backward compatibility - an undiscoverable value leaves you with an unset property on the network | 17:50 |
| armax | ijw: whereas clearly some don’t today | 17:50 |
| garyk | ijw: no, only plugins that wish to support it should | 17:50 |
| garyk | ijw: core impleies that everyone has to support it | 17:51 |
| *** matrohon has quit IRC | 17:51 | |
| armax | ijw: the other question is: can all plugins support this out of the box? | 17:51 |
| ijw | armax: the spec's quite explicit about the way they work - if the plugin can determine the MTU, then it should set the value. If it can't it shouldn't. That means that a plugin that is not setting the property is in fact supporting the feature | 17:51 |
| mestery | VLAN transparent yes, because by default they can just indicate it's "False" I think. | 17:51 |
| *** aranjan has joined #openstack-neutron | 17:51 | |
| pritesh_ | garyk: its is optional attribute so how does it matter if you support it or not, even if it is in core? | 17:51 |
| ijw | mestery: yup | 17:51 |
| armax | ijw: if so, then sure, they can be core, if they can’t then they should be extension attributes | 17:51 |
| openstackgerrit | Dane LeBlanc proposed openstack/neutron: No allocation needed for specific IPv6 SLAAC addr assignment https://review.openstack.org/164943 | 17:52 |
| ijw | armax: we can double check this but I think the compatibility with plugins that don't really understand MTU is already in the patches submitted | 17:52 |
| garyk | pritesh_: it is not optional | 17:52 |
| armax | ijw: indeed..let’s look at that and be sure that the chosen approach indeed makes sense | 17:52 |
| ijw | We happen to have implemented the ML2 version for drivers tat understand it, and the code in the plugin that checks all the drivers in use for a network. | 17:52 |
| garyk | pritesh_: i will try and explain. | 17:52 |
| ijw | armax: you might want to go back to the specs, especially the MTU spec, and have a check to see if you like what it does | 17:53 |
| ijw | It's quite detailed on compatibility because I did want a core attribute for preference and I didn't want to go hacking every plugin in the world | 17:53 |
| garyk | pritesh_: we as a community say that people can work off master. so yesterday the core features did not have mtu and todya they do. | 17:53 |
| garyk | this is consuing for a user | 17:53 |
| garyk | confusing for a user | 17:53 |
| garyk | not only that if a user is using X plugin today they do not have this and with a upgrade they do. | 17:54 |
| armax | garyk: I don’t think that’s a fair statement | 17:54 |
| ijw | garyk: I understand your concern but every added feature adds something to the API, and we consider adding properties to be backward compatible | 17:54 |
| armax | garyk: just because the system grows in features it doesn’t mean that the user gets confused | 17:54 |
| armax | as a matter of fact the user might be pleased :) | 17:54 |
| garyk | if i do network show it will show different values? | 17:54 |
| ijw | I would happily have a debate about whether this feature is sensible in the form it's in but I'm not sure there's an argument about whether adding an attribute is an immediate blocker to merging in a stable API | 17:54 |
| pritesh_ | garyk:so no new feature should go in you mean in kilo ? | 17:54 |
| garyk | will they be pleased when they try and configure and the driver does nothing? | 17:55 |
| ijw | garyk: if you add an extension then net-show will show different values. | 17:55 |
| garyk | due to the fact that it is not core | 17:55 |
| *** SumitNaiksatam has joined #openstack-neutron | 17:55 | |
| ijw | garyk: read the specs, they both have backward compatibility that ensures the user can't config values that unknowing drivers can't support | 17:55 |
| garyk | ijw: an admin exposing extension to his./her users will informat them of this | 17:55 |
| ijw | garyk: you're a hopeless optimist, you know ;) | 17:56 |
| garyk | ijw: i tend to disagree on this. | 17:56 |
| mestery | I thought these extensions were admin only? | 17:56 |
| armax | garyk: that’s not what I meant | 17:56 |
| mestery | garyk ijw: Are they not? | 17:56 |
| ijw | mestery: the attributes are readable by anyone | 17:56 |
| ijw | and should be writeable too, though I think there was some debate about that | 17:56 |
| mestery | ijw: OK, then I'm confused, I thought they were admin only | 17:56 |
| ijw | I mean, if my app needs a big MTU I need to be able to check the MTU is big | 17:56 |
| garyk | mestery: as far as i understand they are intended to be consumed by everyone (so I learned a few minutes ago) | 17:57 |
| ijw | mestery: oh, you said extensions | 17:57 |
| ijw | Ignore me | 17:57 |
| ijw | These patches are intended to be consumed by everyone. I'll stay out of the extensions question. | 17:57 |
| garyk | i am not sure that i have much more to add. i think that we need to decide if these are core values or not. i am not in favor of them being core, if they are then cool let us know and we can expose this in all of the plugins | 17:58 |
| ijw | garyk: coming back to your properties comment, we added properties to networks for ipv6 advertisement in an API. They weren't optional, all clouds that upgraded got them, and they turned up in a stable version. So there's precedent for adding properties on object | 17:58 |
| garyk | ijw: i too think that was a mistake. | 17:58 |
| sc68cal | Hang on, thhe ipv6 attributions are optional | 17:59 |
| sc68cal | it just so happens that the reference implementation is totally broken without using them however. | 17:59 |
| armax | so the question here really is: | 17:59 |
| openstackgerrit | Michael Smith proposed openstack/neutron: L3 Agent support of manual dvr snat add/remove https://review.openstack.org/123815 | 17:59 |
| *** reed has quit IRC | 17:59 | |
| armax | now the attributes are exposed to all plugins | 18:00 |
| armax | can all plugins handle them consistently? | 18:00 |
| *** padkrish has joined #openstack-neutron | 18:00 | |
| *** jlanoux has quit IRC | 18:00 | |
| garyk | armax: what if the tenant defines an mtu greater than the mtu of the underlying network? | 18:01 |
| armax | in other words, if a user set them, update them, etc, is the expected behavior honored by the plugin? | 18:01 |
| armax | if not, then, these must be extension attributes, if yes, then I suppose they can stay core | 18:01 |
| garyk | armax: i can say for the vware nsx this is not surrently supported (I only discovered them today :)) | 18:01 |
| ijw | From memory: the default value for them is NULL - which means the plugin isn't going to tell you anything. The default for a plugin that doesn't actively handle setting them on net-create is that the value is rejected. This is documented behaviour - a plugin is at liberty and in spec if it doesn't let you choose the values and doesn't tell you what values it's using. | 18:01 |
| garyk | in addition to that i have no idea what a transparent vlan is | 18:02 |
| ijw | garyk: if the MTU is too big the net-create will fail | 18:02 |
| *** aranjan has quit IRC | 18:02 | |
| ijw | garyk: read the spec | 18:02 |
| *** boris-42 has quit IRC | 18:02 | |
| ijw | armax: and they aren't updateable after create | 18:02 |
| armax | ijw: right, that doesn’t change my question | 18:03 |
| garyk | ijw: that means that the plugin will need to check. i am not sure that i saw that validation. can you point that outr to me please | 18:03 |
| ijw | I hope that the specs actually answer these questions, and it's really the specs you're critcising rather than the code, here. I think it's easier if you go read them and see whether there's something you object to there. | 18:03 |
| *** ZZelle_ has joined #openstack-neutron | 18:03 | |
| armax | ijw: if I set vlan_transparent=True, can any plugin honor the request out of the box? | 18:03 |
| ijw | armax: no | 18:03 |
| armax | so that’s your answer | 18:03 |
| *** enikanorov has joined #openstack-neutron | 18:03 | |
| mestery | armax: Linuxbridge agent I think | 18:03 |
| mestery | ML2 + Linuxbridge | 18:03 |
| armax | these must move to extension attribute | 18:03 |
| ijw | armax: for an obvious one, if you're using OVS in ML2 or VLANs on the net, VLAN trunks cannot be created under any circumstances, but at least the user can now find that out | 18:03 |
| ijw | Before, they got a network that might pass vlans and no clues | 18:04 |
| armax | ijw: that’s not the point | 18:04 |
| *** yatin has quit IRC | 18:04 | |
| ijw | armax: alternatively, you're asking 'what happens if I'm using some plugin that's never heard of VLANs?' | 18:04 |
| armax | we can’t show an attribute that the plugin cannot understand | 18:04 |
| armax | now the other question is: | 18:04 |
| armax | if a user set the MTU, can any plugin honor it right now, i.e. can it enforce the right MTU on the network they provision? | 18:05 |
| ijw | And that's addressed by the backward compatibility bits of the spec | 18:05 |
| ijw | And again, no - not all plugins know what an MTU is | 18:05 |
| ijw | And that's *also* addressed by the backward compatibility bits of the spec | 18:05 |
| armax | ijw: I’ll re-read the sections | 18:06 |
| *** enikanorov__ has quit IRC | 18:06 | |
| *** vthapar has joined #openstack-neutron | 18:06 | |
| armax | ijw: but one thing is for sure: I don’t think it’s acceptable if backend actually do nothing about certain attributes | 18:07 |
| armax | ijw: it doesn’t lead to a coherent user experience | 18:07 |
| ijw | I would totally agree with that | 18:07 |
| garyk | ijw: is this not why we have extensions | 18:07 |
| garyk | a plugin should only expose what it supports? | 18:07 |
| ijw | You need to *know* that you asked for somethig the plugin can't provide, which is why there's stuff in the spec that tells you how you know | 18:07 |
| armax | ijw: we’d need to think not just to the plugins that were considered whilst designing this feature | 18:07 |
| armax | ijw: but also those that weren’t | 18:08 |
| armax | I think that’s garyk’s crux of the matter | 18:08 |
| ijw | armax: read the spec. Seriously, I did think of this when I wrote it | 18:08 |
| armax | due | 18:08 |
| armax | dude I get it | 18:08 |
| *** oreillyd has quit IRC | 18:08 | |
| armax | I did read it and I will re-read it | 18:08 |
| *** dane_leblanc has joined #openstack-neutron | 18:08 | |
| armax | but time has gone by I might have forgotten or even agreed at the time ;) | 18:09 |
| ijw | To be fair, this was also written in the season of salv-orlando's 'no more extensions' argument ;) | 18:09 |
| armax | understood | 18:09 |
| armax | I am not taking any side right now | 18:09 |
| armax | I am only trying to making sure we’re clear as to what the issue is | 18:09 |
| ijw | In this instance I think it does make sense for it not to be an extension, it's something we should strongly recommend every plugin implement in a ore grown up manner than 'I don't know what you're talking about' | 18:09 |
| ijw | Even if the answer is 'your MTU is 1500 and I've never heard of these VLAN things' | 18:10 |
| armax | from an external endpoint whether the MTU is an extension attribute or not it really makes no different | 18:10 |
| armax | and this whole discussion becomes a religiious argument | 18:10 |
| ijw | But VLAN one aside, the MTU code in the ML2 driver addresses real user problems so I would bear that in mind - if we don't want it in like this, I think we still want it in | 18:11 |
| armax | all I care is making sure everyone is on the same page and that the user doesn’t get mad | 18:11 |
| ijw | yup | 18:11 |
| *** coolsvap|afk has quit IRC | 18:11 | |
| armax | frankly I couldn’t care less if the attribute is core or not | 18:11 |
| *** crc32 has quit IRC | 18:11 | |
| *** pritesh_ is now known as pritesh | 18:11 | |
| armax | so long as it does what’s intended to do | 18:11 |
| *** Marga__ has quit IRC | 18:11 | |
| armax | the extension attribute debate is a technicality | 18:11 |
| armax | IMO | 18:12 |
| dougwig | given that this argument started when i left for lunch, and i'm back now, and it could've been coded both ways in that timeframe, can i suggest we drive to an action and move on? | 18:12 |
| ijw | My main worry is that if these discussions are coming up they're so much easier to deal with at spec time than pretty much on the day of the FF when the code is written ;) | 18:12 |
| *** Marga_ has joined #openstack-neutron | 18:12 | |
| armax | ijw: agreed, but the world is far less than ideal | 18:12 |
| *** Marga_ has quit IRC | 18:12 | |
| armax | is it not? | 18:12 |
| * ijw wonders who to bribe to -1 all armax's patches ;) | 18:12 | |
| *** Marga_ has joined #openstack-neutron | 18:13 | |
| *** sudipto has joined #openstack-neutron | 18:13 | |
| * dougwig accepts scotch. | 18:13 | |
| armax | not sure that is gonna give you anything but sure, go ahead :) | 18:13 |
| ijw | But yes - I just want to make sure if this gets blocked it gets blocked for a solid reason | 18:13 |
| armax | you gotta get my -2 taken away first :) | 18:13 |
| ijw | armax: entertainment, and I may be in the pricerange for a -2 | 18:13 |
| garyk | iijwno one is blocking. | 18:13 |
| armax | ijw: I don’t want it blocked | 18:13 |
| armax | ijw: I want it right! | 18:13 |
| garyk | ijw: the issue is just to move the definition from a core property to an extension | 18:14 |
| ijw | OK - that being the case then what would make it right for you, do you think? Simply making it an extension? | 18:14 |
| ijw | I have to say I'm not a massive fan | 18:14 |
| garyk | ijw: that works for me | 18:14 |
| armax | ijw: I need more time to look at this | 18:14 |
| garyk | you guys can convince me that the mtu is a core. | 18:14 |
| garyk | vlan_transparency - no way | 18:14 |
| dougwig | does moving it to an extension change the public facing API at all? (i don't think it does.) which means that this is mostly an internal debate, right? which means that there is not a huge hurry. | 18:14 |
| armax | and decide whether these make sense as is, or we should move them | 18:14 |
| *** Marga_ has quit IRC | 18:14 | |
| armax | dougwig: amen | 18:14 |
| armax | dougwig: that’s what I am saying for an hour | 18:15 |
| *** jschwarz has joined #openstack-neutron | 18:15 | |
| *** Marga_ has joined #openstack-neutron | 18:15 | |
| dougwig | armax: i agree. unless it's urgent, today of all days seems like not the day to debate this (or revert it, if we expect it to be in in some fashion.) | 18:16 |
| armax | dougwig: I never advocated for the revert | 18:16 |
| pritesh | dougwig: if people start posting reverting patches it no more a internal debate, it shows you want that feature out. | 18:16 |
| armax | pritesh: yeah, that was a bit hasty | 18:16 |
| garyk | pritesh: i do not want the feature out. i di dnot say that. i just want the definition in a more logical place | 18:16 |
| blogan | whats the main reason to put somethign as an etension versus core? toggling on or off? | 18:17 |
| * salv-orlando waits for the blood to flow | 18:17 | |
| armax | garyk: the revert was not the right step forward | 18:17 |
| dougwig | the features, as they are in today, move the ball down the field. with what garyk identified, is some technical debt. now we're debating when/how to fix that technical debt, really. which we can do without pulling the feature. and we can also do without urgency. | 18:17 |
| garyk | armax: then can you please explain what the correct process is? | 18:17 |
| armax | iterate over the existing code | 18:17 |
| armax | once agreement has been reached | 18:17 |
| blogan | dougwig: what if moving it from core to to extension does break backwards compatibility? | 18:17 |
| armax | stuff can just be shuffled to the right place | 18:17 |
| dougwig | garyk: have the original authors do the revert/move in the same gerrit patch would be my preference, so it's seamless. | 18:18 |
| armax | once the right place has been determined | 18:18 |
| mestery | We're not reverting the patches, we'll fix it. Reverting isn't the right move. | 18:18 |
| mestery | armax: Right | 18:18 |
| mestery | Please stop talking about reverting | 18:18 |
| garyk | yay, now i am to blame :) | 18:18 |
| armax | kilo hasn’t shipped yet | 18:18 |
| dougwig | blogan: i don't think that's true in this case, since it's attribute adds, not endpoint adds. | 18:18 |
| * salv-orlando throws some meat to the lions... | 18:18 | |
| mestery | right | 18:18 |
| salv-orlando | well when i do something wrong you guys revert my patches ;) | 18:18 |
| garyk | next time will be very sensitive about how i do things, but when something is not done correctly it should be fixed. | 18:18 |
| blogan | dougwig: removing an attribute break backwards compatibility, and if moving from core to extension then allows an attribute to be disabled, doesn't that break it? | 18:19 |
| *** criptoverso has quit IRC | 18:19 | |
| armax | I guess there are reasons that justify a revert | 18:19 |
| mestery | Examples like this are why I think the specs process is completely broken BTW | 18:19 |
| garyk | the fact that this changes the way that we deal with chagnges to attributes is setting a precedent. | 18:19 |
| dougwig | blogan: point. doesn't worry me, but point. | 18:19 |
| armax | but I don’t see any of the reasons we discussed today, valid ones to warrant a revert of this feature, just yet | 18:19 |
| salv-orlando | yes - but can we discuss this tomorrow? or monday? | 18:19 |
| salv-orlando | if that code stays in trunk one or 2 days no puppy will day | 18:20 |
| mestery | maybe tuesday? | 18:20 |
| salv-orlando | die | 18:20 |
| dougwig | garyk: i think your objection to this being precedent is duly noted. :-) | 18:20 |
| armax | as I said, let’s have more eyes on this, get everyone on board and happy that it does what it’s supposed to do and we’ll make the call, then and only then | 18:20 |
| garyk | i am going to abandon the reverts. guys let me know what you decide. its borken and should be fixed. | 18:20 |
| *** lori is now known as lori|away | 18:20 | |
| armax | broken where? | 18:20 |
| garyk | armax: it changes core values that are not supported by all plaugins | 18:21 |
| garyk | it exposes data to a tenant that should be admin only | 18:21 |
| garyk | that is just on the api level. | 18:21 |
| sc68cal | We have attributes like that today | 18:21 |
| armax | garyk: ok, this is becoming a circular discussion | 18:21 |
| *** ronis has joined #openstack-neutron | 18:21 | |
| *** _cjones_ has quit IRC | 18:21 | |
| * sc68cal points to ipv6_ra_mode and ipv6_address_mode | 18:21 | |
| garyk | exactly | 18:21 |
| sc68cal | So should we revert those? | 18:21 |
| garyk | but you wanted to know where i thiink that it is broken. no? | 18:22 |
| *** catohornet has joined #openstack-neutron | 18:22 | |
| dougwig | who is willing to take the action item to own this debate/submit some patches? | 18:22 |
| sc68cal | Or should I make an API extension for ipv6 support? | 18:22 |
| garyk | sc68cal: since when do two wrongs make a right? | 18:22 |
| mestery | lol | 18:22 |
| armax | I am gonna go back and reading the specs once again and make sure the code is sound | 18:22 |
| garyk | but whatever the community decides | 18:22 |
| salv-orlando | anyway might it be relevant to the discussion that I proposed abolition of attribute extensions and nobody objected? | 18:23 |
| * armax moves his eyes on his other screen | 18:23 | |
| *** atuvenie has joined #openstack-neutron | 18:23 | |
| garyk | at least armax is not rolling his eyes | 18:23 |
| garyk | it paves the way for parting with extensions … | 18:23 |
| * armax rolling his eyes to the left ;) | 18:23 | |
| garyk | what do i care south africa is into the semi finals! | 18:23 |
| dougwig | so i asked who owns solving this, and heard nothing. if no one cares about this feature and consensus that much, then let's revert it. | 18:24 |
| salv-orlando | because while I agree garyk has some very valid concern especially around one of the two blueprints, I also think that if we keep going with everything is an extension we end up in a sitation where we do not know what neutron is or does | 18:24 |
| dougwig | otherwise, speak up. | 18:24 |
| salv-orlando | is just whatever you want it to be | 18:24 |
| pritesh | dougwig: i can submit new patch which will move it to extension | 18:24 |
| mestery | armax owns this :) | 18:24 |
| sc68cal | I can help too | 18:24 |
| mestery | dougwig: ^^^ | 18:24 |
| mestery | sc68cal: Tahnks! | 18:24 |
| sc68cal | if needed | 18:24 |
| dougwig | pritesh: can you also take on driving the consensus, via the ML or the neutron meeting? | 18:24 |
| dougwig | if so, let's table and let priggish work. | 18:24 |
| pritesh | dougwig: sure will do. | 18:24 |
| *** jschwarz has quit IRC | 18:24 | |
| garyk | pritesh: thanks! | 18:24 |
| dougwig | pritesh (sorry, autocorrect) | 18:24 |
| garyk | if you mail or ping me I will be very happy to review | 18:25 |
| *** _cjones_ has joined #openstack-neutron | 18:25 | |
| pritesh | garyk: ok thanks, will do. | 18:25 |
| garyk | pritesh: thanks. i have abandoned the patches and the balls in your court. | 18:25 |
| pritesh | dougwig: no, worries, thanks. | 18:25 |
| pritesh | garyk: ok sounds good, will send it new patch soon to move it. | 18:26 |
| pritesh | garyk: just to be sure, you mean the vlan transprent one right, and you are ok with the mtu one being in core? | 18:26 |
| *** safchain has quit IRC | 18:28 | |
| garyk | pritesh: i am fine with the mtu to be core. we will need to work on our plugin to make sure that this is correctly implemented… | 18:28 |
| *** banix has quit IRC | 18:28 | |
| garyk | i still am not 100% comfortable with it. but i seem to be in the minority | 18:28 |
| pritesh | garyk: sounds good, so will get the vlan transparent one moved to extension. | 18:28 |
| *** reed has joined #openstack-neutron | 18:29 | |
| garyk | ok, thanks | 18:29 |
| garyk | mestery: armax: marun: i was just sent a mail asking for me to let people that they can vote for our session :). oh my. armax roll those eyes | 18:29 |
| mestery | garyk: lol | 18:30 |
| mestery | :) | 18:30 |
| *** emagana has quit IRC | 18:31 | |
| *** leenheer has quit IRC | 18:31 | |
| *** leenheer has joined #openstack-neutron | 18:32 | |
| *** emagana has joined #openstack-neutron | 18:32 | |
| sc68cal | just got it too | 18:32 |
| *** aepifanov has joined #openstack-neutron | 18:34 | |
| *** aranjan has joined #openstack-neutron | 18:35 | |
| *** banix has joined #openstack-neutron | 18:36 | |
| garyk | salv-orlando: thanks for fixing the unit tests | 18:37 |
| garyk | one the patch is in i will rebase the rest... | 18:37 |
| kevinbenton | mestery: ping | 18:37 |
| mestery | kevinbenton: You're awake before noon pacific? Who are you? | 18:38 |
| kevinbenton | mestery: i'm getting my life in order :) | 18:38 |
| ijw | I've posted to the mailing list on both the patch thread and the compatibilty thread Gary started. Let's work there. | 18:38 |
| ijw | Quick decisions appreciated, but at least we can involve everyone who cares that way | 18:39 |
| *** jbrendel has joined #openstack-neutron | 18:39 | |
| mestery | kevinbenton: Good for you! Did you get your first sobriety chip? :) | 18:39 |
| kevinbenton | mestery: "up before noon for 1 day" | 18:39 |
| mestery | kevinbenton: lol | 18:39 |
| *** aranjan has quit IRC | 18:40 | |
| kevinbenton | mestery: when do we propose FFE's? I really want the ebtables patch to make it in this cycle. It's important to enable an equivalent of flatdhcp | 18:40 |
| mestery | kevinbenton: You'll have to work with marun on that, but I don't think that one can make it. If you can convince him, come and talk to me. | 18:41 |
| kevinbenton | mestery: ok | 18:41 |
| jbrendel | Mestery: All the ARP-related patches are lined up and have been reviewed many times. | 18:41 |
| kevinbenton | marun: what were your main concerns around the ebtables stuff? | 18:41 |
| *** tflynn has joined #openstack-neutron | 18:42 | |
| jbrendel | None of the comments were concerns about the way it worked, only minor ones about style and such. | 18:42 |
| *** Marga_ has quit IRC | 18:42 | |
| *** jprovazn has quit IRC | 18:43 | |
| *** yamamoto has joined #openstack-neutron | 18:44 | |
| *** Marga_ has joined #openstack-neutron | 18:45 | |
| *** claudiub has quit IRC | 18:46 | |
| *** abehl has joined #openstack-neutron | 18:48 | |
| *** yfried has joined #openstack-neutron | 18:48 | |
| *** yamamoto has quit IRC | 18:49 | |
| *** marrusl has quit IRC | 18:51 | |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: L2 agent RPC add new RPC calls https://review.openstack.org/147435 | 18:51 |
| *** safchain has joined #openstack-neutron | 18:52 | |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: OVS agent react to events instead of polling https://review.openstack.org/164880 | 18:54 |
| openstackgerrit | Rossella Sblendido proposed openstack/neutron: Add get_events to OVSDB monitor https://review.openstack.org/161806 | 18:54 |
| *** marrusl has joined #openstack-neutron | 18:54 | |
| openstackgerrit | Merged openstack/neutron: Hyper-V Agent decomposition https://review.openstack.org/153259 | 18:54 |
| openstackgerrit | Roey Chen proposed openstack/neutron: Fix create_security_group_rule_bulk_native to return all created rules https://review.openstack.org/165970 | 18:57 |
| *** vivek-ebay has quit IRC | 18:58 | |
| *** vivek-ebay has joined #openstack-neutron | 19:01 | |
| *** vivek-ebay has quit IRC | 19:01 | |
| marun | kevinbenton: re: ebtables, I grew pretty frustrated at the lack of testability in the design. | 19:01 |
| *** vivek-ebay has joined #openstack-neutron | 19:02 | |
| marun | kevinbenton: I directed salv-orlando and armax to review in my place because I didn't have the energy to educate the author. | 19:02 |
| marun | kevinbenton: It solves a real problem so I didn't want to block it just because the implementation offended my tender sensibilities. The same can be said of a great deal of the neutron codebase, after all. | 19:03 |
| kevinbenton | marun: maybe we can chat a bit tomorrow morning about some changes to make it a little easier to test? | 19:04 |
| marun | kevinbenton: sure thing | 19:04 |
| kevinbenton | marun: sounds good | 19:04 |
| jbrendel | Hi there! | 19:04 |
| jbrendel | Thank you for looking into that ARP patch. | 19:04 |
| jbrendel | We've had a few +2s before, all the patches are lined up and and ready to go. | 19:05 |
| kevinbenton | jbrendel: talking to me or maru ? :) | 19:05 |
| jbrendel | All of you! :-) | 19:05 |
| jbrendel | kevinbenton: maru: Robert Clark gave it a +1 last night, requesting that it please be considered for inclusion due to security. | 19:05 |
| kevinbenton | jbrendel: i have to step out for a meeting. i'll be back in an hour or so | 19:06 |
| mestery | Keep in mind we have 9 BPs for RC1 already folks, which is likely already 4 too many. I'm not saying we can't make it an even 10 though, but it's a lot. | 19:06 |
| *** fredlhsu has quit IRC | 19:06 | |
| *** Marga_ has quit IRC | 19:06 | |
| kevinbenton | mestery: right, i only suggested it because it seems pretty close. i'll work with Maru on the testing aspect to see if there is a change we can make | 19:07 |
| jbrendel | mestery: Does it help in any way that the code has seen a lot of reviews and seems to be acceptable? | 19:07 |
| *** jobewan has joined #openstack-neutron | 19:07 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Transform BaseOVSLinuxTestCase methods in helpers https://review.openstack.org/165866 | 19:07 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define FakeMachine helper for functional/fullstack tests https://review.openstack.org/143412 | 19:07 |
| mestery | jbrendel kevinbenton: It does indeed, like I said, lets see what the investigation turns up. Worst case we can fast-track this into Liberty once we branch the RC. | 19:07 |
| *** _cjones_ has quit IRC | 19:08 | |
| *** claudiub has joined #openstack-neutron | 19:08 | |
| kevinbenton | mestery: okay. i'll update on the review what Maru an i can work out | 19:09 |
| mestery | kevinbenton: Gracias | 19:09 |
| *** marun has quit IRC | 19:09 | |
| *** dsneddon is now known as dsneddon_lunch | 19:09 | |
| *** bobmel has quit IRC | 19:09 | |
| HenryG | kevinbenton: I too will help where I can | 19:09 |
| kevinbenton | HenryG: ack | 19:10 |
| *** bobmel has joined #openstack-neutron | 19:10 | |
| sc68cal | I need to stop joking and propose a summit session "OpenStack Neutron: Hug it out" where everyone just has to give eachother a hug and say it's all going to be OK | 19:12 |
| pritesh | sounds like a good idea! | 19:13 |
| *** jbrendel has quit IRC | 19:14 | |
| *** bobmel has quit IRC | 19:15 | |
| aveiga | sc68cal: that's what a whiskey BOF is for | 19:15 |
| sc68cal | :) | 19:15 |
| *** evgenyf has joined #openstack-neutron | 19:16 | |
| *** marun has joined #openstack-neutron | 19:19 | |
| *** ijw has quit IRC | 19:20 | |
| *** shaohe_feng has quit IRC | 19:20 | |
| *** banix has quit IRC | 19:22 | |
| mestery | sc68cal: lol | 19:23 |
| *** jbrendel has joined #openstack-neutron | 19:24 | |
| *** Marga_ has joined #openstack-neutron | 19:24 | |
| *** Marga_ has quit IRC | 19:25 | |
| *** vthapar has quit IRC | 19:27 | |
| *** julien_ZTE has joined #openstack-neutron | 19:27 | |
| *** aranjan has joined #openstack-neutron | 19:29 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define extract_ip helper function https://review.openstack.org/143811 | 19:31 |
| *** rotbeard has joined #openstack-neutron | 19:32 | |
| *** pc_m has quit IRC | 19:32 | |
| *** aranjan has quit IRC | 19:34 | |
| *** _cjones_ has joined #openstack-neutron | 19:34 | |
| *** banix has joined #openstack-neutron | 19:36 | |
| *** SumitNaiksatam has quit IRC | 19:39 | |
| *** jorgem has quit IRC | 19:39 | |
| openstackgerrit | Merged openstack/neutron: Basic subnetpool CRUD https://review.openstack.org/148698 | 19:42 |
| *** dsneddon_lunch is now known as dsneddon | 19:43 | |
| *** britthouser has quit IRC | 19:43 | |
| *** evgenyf has quit IRC | 19:49 | |
| otherwiseguy | mestery: should I be able to find logs from my py27 failure on logstash? Because I seem to be completely incapable of that. | 19:51 |
| mestery | otherwiseguy: Indeed | 19:52 |
| otherwiseguy | don't seem to see anything with build_name:"gate-neutron-python27". | 19:52 |
| *** sbelous1 has joined #openstack-neutron | 19:53 | |
| *** sudipto has quit IRC | 19:53 | |
| otherwiseguy | didn't know if it was more than me just having no idea what I'm doing or not. :p | 19:53 |
| *** julien_ZTE has quit IRC | 19:53 | |
| *** sbelous1 has left #openstack-neutron | 19:54 | |
| *** aveiga has quit IRC | 19:54 | |
| *** flwang1 has joined #openstack-neutron | 19:55 | |
| otherwiseguy | mestery: https://review.openstack.org/#/c/155470/15/neutron/db/l3_db.py has been stuck in the gate for 20 hours. | 19:55 |
| flwang1 | garyk: ping | 19:55 |
| otherwiseguy | It does L3-y things. | 19:55 |
| openstackgerrit | Salvatore Orlando proposed openstack/neutron: IPAM reference driver https://review.openstack.org/150485 | 19:55 |
| *** Marga_ has joined #openstack-neutron | 19:55 | |
| mestery | otherwiseguy: Lets go #openstack-infra | 19:56 |
| *** sbfox has quit IRC | 19:56 | |
| * otherwiseguy tries checking it out and seeing if it causes his patch to fail | 19:56 | |
| *** shwetaap has quit IRC | 19:56 | |
| *** britthouser has joined #openstack-neutron | 20:00 | |
| *** fredlhsu has joined #openstack-neutron | 20:01 | |
| *** sbfox has joined #openstack-neutron | 20:01 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define extract_prefixlen helper function https://review.openstack.org/143812 | 20:01 |
| *** dave-mccowan has quit IRC | 20:04 | |
| *** gildub has joined #openstack-neutron | 20:04 | |
| *** fredlhsu has quit IRC | 20:04 | |
| *** sambetts has quit IRC | 20:06 | |
| *** sambetts has joined #openstack-neutron | 20:06 | |
| mestery | otherwiseguy: Once armax comes back he can push a new verison of that stuck patch. If he's at lunch, feel free to push it yourself :) | 20:07 |
| *** Marga_ has quit IRC | 20:08 | |
| otherwiseguy | mestery: k. still beating my head against the seemingly impossible. :) | 20:08 |
| *** roeyc has quit IRC | 20:09 | |
| otherwiseguy | I guess in the check queue it shouldn't affect me at all. Just grasping at straws at this point. | 20:09 |
| *** atuvenie has quit IRC | 20:10 | |
| *** shwetaap has joined #openstack-neutron | 20:11 | |
| mestery | otherwiseguy: Hang in there, it's almost Friday :) | 20:12 |
| *** aranjan has joined #openstack-neutron | 20:15 | |
| *** vivek-ebay has quit IRC | 20:15 | |
| *** vivek-eb_ has joined #openstack-neutron | 20:15 | |
| *** dave-mccowan has joined #openstack-neutron | 20:18 | |
| kevinbenton | otherwiseguy: what's stuck? | 20:19 |
| kevinbenton | otherwiseguy: or failing? | 20:19 |
| *** aranjan has quit IRC | 20:20 | |
| *** thomasem has quit IRC | 20:20 | |
| *** emagana has quit IRC | 20:20 | |
| otherwiseguy | kevinbenton: 152761 fails py27 with KeyError: 'L3_ROUTER_NAT' even though I can't see how the code could ever even come close to touching that. http://logs.openstack.org/61/152761/21/check/gate-neutron-python27/d29809b/console.html | 20:21 |
| kevinbenton | otherwiseguy: you've probably been lucky enough to hit the UT failure lottery! | 20:22 |
| otherwiseguy | kevinbenton: It passed py27 fine around midnight, then failed in the gate later in the morning without any code change, and has failed the check since. | 20:22 |
| kevinbenton | otherwiseguy: which patch is it? | 20:22 |
| kevinbenton | otherwiseguy: i will find the issue | 20:23 |
| kevinbenton | otherwiseguy: i love this game :) | 20:23 |
| otherwiseguy | kevinbenton: here's the patch: https://review.openstack.org/#/c/152761/ | 20:23 |
| kevinbenton | otherwiseguy: usually some other test is mutating something | 20:23 |
| *** c_soukup has joined #openstack-neutron | 20:23 | |
| kevinbenton | otherwiseguy: and your patch just happened to change the test order enough to cause it | 20:23 |
| otherwiseguy | Something somewhere is probably mutating service_plugins or something. | 20:23 |
| kevinbenton | otherwiseguy: right | 20:23 |
| mestery | kevinbenton is like sherlock holmes! | 20:24 |
| otherwiseguy | I've just been flailing around at it. :p | 20:24 |
| kevinbenton | mestery: minus the good looks, physical fitness, and general aura of coolness | 20:24 |
| otherwiseguy | kevinbenton: So just the cocaine? | 20:24 |
| mestery | kevinbenton: Heh. | 20:24 |
| kevinbenton | otherwiseguy: oh, none of that either because i can't afford it | 20:24 |
| otherwiseguy | :) | 20:24 |
| kevinbenton | otherwiseguy: diet coke instead :) | 20:24 |
| *** banix has quit IRC | 20:25 | |
| *** emagana has joined #openstack-neutron | 20:25 | |
| *** aranjan has joined #openstack-neutron | 20:25 | |
| amuller | otherwiseguy: one of my patches is hitting that as well: https://review.openstack.org/#/c/128013/ | 20:26 |
| *** Swami has quit IRC | 20:26 | |
| *** csoukup has quit IRC | 20:26 | |
| otherwiseguy | amuller: I'M NOT ALONE! | 20:26 |
| *** salv-orlando has quit IRC | 20:27 | |
| * amuller leaves | 20:27 | |
| otherwiseguy | So far it looks like an anti-Red Hat conspiracy! :p | 20:27 |
| *** flwang1 has quit IRC | 20:29 | |
| mestery | lol | 20:30 |
| mestery | kevinbenton: I think the RH guys are on to us :) | 20:30 |
| kevinbenton | mestery: I HAVE NO IDEA WHAT YOU ARE TALKING ABOUT | 20:30 |
| kevinbenton | mestery: i have an alibi | 20:30 |
| kevinbenton | mestery: i was at starbucks during that time | 20:30 |
| mestery | kevinbenton: lol | 20:31 |
| amuller | carl_baldwin: The pluggable ipam + reference ipam implementation is still targeted for Kilo I see? | 20:32 |
| carl_baldwin | amuller: Yes, it is. | 20:32 |
| amuller | I guess I should start reviewing that =D *feels guilty* | 20:32 |
| *** flwang1 has joined #openstack-neutron | 20:32 | |
| *** dims has quit IRC | 20:34 | |
| *** Marga_ has joined #openstack-neutron | 20:38 | |
| *** yfried is now known as yfried|afk | 20:42 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Subnet allocation from a subnet pool https://review.openstack.org/157597 | 20:46 |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Subnet allocation from a subnet pool https://review.openstack.org/157597 | 20:47 |
| kevinbenton | otherwiseguy: unreproducable, case closed | 20:47 |
| kevinbenton | otherwiseguy: :) | 20:47 |
| *** jorgem has joined #openstack-neutron | 20:48 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define get_cidr helper function https://review.openstack.org/143869 | 20:48 |
| *** claired has joined #openstack-neutron | 20:48 | |
| *** cference has quit IRC | 20:48 | |
| *** jaypipes has quit IRC | 20:50 | |
| *** safchain has quit IRC | 20:51 | |
| *** Marga_ has quit IRC | 20:52 | |
| *** gildub has quit IRC | 20:52 | |
| *** Marga_ has joined #openstack-neutron | 20:52 | |
| kevinbenton | otherwiseguy: i lied, i found the issue | 20:52 |
| otherwiseguy | kevinbenton: !!! | 20:53 |
| *** rotbeard has quit IRC | 20:53 | |
| *** emagana has quit IRC | 20:53 | |
| *** Marga_ has quit IRC | 20:54 | |
| *** Marga_ has joined #openstack-neutron | 20:54 | |
| *** emagana has joined #openstack-neutron | 20:54 | |
| otherwiseguy | kevinbenton: what was it? | 20:55 |
| kevinbenton | otherwiseguy: working on figuring out why | 20:55 |
| kevinbenton | otherwiseguy: neutron.tests.unit.test_dhcp_scheduler.DHCPAgentWeightSchedulerTestCase.test_scheduler_equal_distribution | 20:55 |
| kevinbenton | otherwiseguy: that test, followed by the one you see failing | 20:55 |
| ZZelle_ | carl_baldwin, re | 20:55 |
| ZZelle_ | carl_baldwin, about https://review.openstack.org/133506 | 20:56 |
| carl_baldwin | ZZelle_: hi | 20:56 |
| *** Sukhdev has joined #openstack-neutron | 20:57 | |
| *** matrohon has joined #openstack-neutron | 20:58 | |
| ZZelle_ | carl_baldwin, my concern is that on one side routers are reachable from external networks (internet) and on the other side metadata proxies run on infra nodes | 20:58 |
| *** absubram has quit IRC | 20:59 | |
| ZZelle_ | carl_baldwin, we should as much as possible reduce the attack surface of our services | 20:59 |
| otherwiseguy | kevinbenton: nice, I can reproduce that way as well. | 20:59 |
| HenryG | kevinbenton: because it's overriding DB_PLUGIN_KLASS ? | 21:00 |
| *** raildo is now known as raildo|away | 21:00 | |
| ZZelle_ | carl_baldwin, typically metadata proxies providing a "bridge" to infra side | 21:00 |
| kevinbenton | HenryG: possibly | 21:00 |
| *** padkrish has quit IRC | 21:01 | |
| kevinbenton | otherwiseguy, HenryG: working on a fix now | 21:01 |
| otherwiseguy | kevinbenton: cool, thanks. Relocating from coffee shop to house. | 21:01 |
| carl_baldwin | enikanorov: Could you comment on https://bugs.launchpad.net/neutron/+bug/1187102 ? I’m wondering about your comment #6 questioning whether we should fix this. | 21:01 |
| openstack | Launchpad bug 1187102 in neutron "quantum-ns-metadata-proxy listens on external interfaces too" [Low,In progress] - Assigned to Cedric Brandily (cbrandily) | 21:01 |
| kevinbenton | otherwiseguy: ack | 21:01 |
| *** padkrish has joined #openstack-neutron | 21:01 | |
| carl_baldwin | ZZelle_: I tend to agree with you. Let me give enikanorov a chance to chime in. | 21:01 |
| *** sthillma has quit IRC | 21:01 | |
| *** jckasper has quit IRC | 21:01 | |
| *** yfried|afk is now known as yfried | 21:02 | |
| *** padkrish_ has joined #openstack-neutron | 21:03 | |
| *** tsg_ has quit IRC | 21:03 | |
| *** ronis has quit IRC | 21:03 | |
| *** aepifanov has quit IRC | 21:03 | |
| *** dannywil_ has quit IRC | 21:03 | |
| *** patrickeast has quit IRC | 21:03 | |
| kevinbenton | HenryG: seems to be because it calls the neutron manager to get_plugin() | 21:03 |
| kevinbenton | might actually be the l3 one that is brittle | 21:04 |
| *** dannywilson has joined #openstack-neutron | 21:04 | |
| *** ivar-laz_ has joined #openstack-neutron | 21:04 | |
| *** ivar-laz_ has quit IRC | 21:05 | |
| *** padkrish has quit IRC | 21:06 | |
| *** ihrachyshka has quit IRC | 21:06 | |
| *** ivar-laz_ has joined #openstack-neutron | 21:06 | |
| kevinbenton | HenryG: oh, dhcp tests don't clean up after themselves | 21:06 |
| *** ivar-laz_ has quit IRC | 21:07 | |
| HenryG | kevinbenton: what don't they clean up? | 21:07 |
| *** ivar-laz_ has joined #openstack-neutron | 21:07 | |
| *** aranjan has quit IRC | 21:08 | |
| *** ivar-lazzaro has quit IRC | 21:08 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define extract_ip helper function https://review.openstack.org/143811 | 21:09 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define extract_prefixlen helper function https://review.openstack.org/143812 | 21:09 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define get_cidr helper function https://review.openstack.org/143869 | 21:09 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Make DHCP tests cleanup neutron manager reference https://review.openstack.org/166014 | 21:11 |
| kevinbenton | HenryG: ^^ | 21:11 |
| *** yfried is now known as yfried|afk | 21:12 | |
| *** aranjan_ has joined #openstack-neutron | 21:13 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Make DHCP tests cleanup neutron manager reference https://review.openstack.org/166014 | 21:13 |
| *** Networkn3rd has quit IRC | 21:13 | |
| HenryG | kevinbenton: you need to open source your sleuthing methods | 21:14 |
| *** Marga_ has quit IRC | 21:14 | |
| *** Marga_ has joined #openstack-neutron | 21:15 | |
| *** lpetrut has quit IRC | 21:15 | |
| *** ivar-laz_ has quit IRC | 21:15 | |
| *** yamamoto has joined #openstack-neutron | 21:16 | |
| otherwiseguy | kevinbenton: works for me. thank you! I owe you a beverage-of-your-choice. (I owe lots of beverage-of-choices this release). | 21:18 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Break calls to get_devices_details_list into subsets https://review.openstack.org/163594 | 21:18 |
| *** claired has quit IRC | 21:19 | |
| HenryG | kevinbenton: Do you use OS_CHECK_PLUGIN_DEALLOCATION? | 21:19 |
| kevinbenton | HenryG: no, it's really slow | 21:20 |
| *** dboik_ has joined #openstack-neutron | 21:20 | |
| openstackgerrit | Terry Wilson proposed openstack/neutron: Add native OVSDB implementation of OVSDB API https://review.openstack.org/152761 | 21:20 |
| *** dboik_ has quit IRC | 21:20 | |
| *** yamamoto has quit IRC | 21:21 | |
| *** dboik_ has joined #openstack-neutron | 21:21 | |
| kevinbenton | HenryG: and unfortunately that's only called on tests that do the cleanup | 21:21 |
| kevinbenton | HenryG: so that wouldn't have helped here | 21:21 |
| otherwiseguy | kevinbenton: I have depended my patch on yours. Go, zuul, go! | 21:21 |
| kevinbenton | mestery: which timezone does it have to be thursday in for feature freeze? | 21:22 |
| mestery | kevinbenton: FF was earlier today, we've already cut Kilo-3 :) | 21:22 |
| mestery | kevinbenton: https://launchpad.net/neutron/+milestone/kilo-3 | 21:22 |
| mestery | kevinbenton: We're on to a packed RC1 now: https://launchpad.net/neutron/+milestone/kilo-rc1 | 21:22 |
| kevinbenton | mestery: scary! | 21:22 |
| mestery | kevinbenton: Indeed | 21:23 |
| *** dboik has quit IRC | 21:23 | |
| *** ivar-lazzaro has joined #openstack-neutron | 21:24 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Replace BaseIPVethTestCase by FakeMachine https://review.openstack.org/160290 | 21:25 |
| *** salv-orlando has joined #openstack-neutron | 21:27 | |
| *** yeungp1 has joined #openstack-neutron | 21:28 | |
| *** yeungp has quit IRC | 21:28 | |
| *** achanda has quit IRC | 21:28 | |
| *** gildub has joined #openstack-neutron | 21:35 | |
| openstackgerrit | Michael Smith proposed openstack/neutron: L3 Agent support for routers with HA and DVR https://review.openstack.org/139686 | 21:35 |
| *** sripriya has quit IRC | 21:35 | |
| *** carl_baldwin has quit IRC | 21:35 | |
| *** sripriya has joined #openstack-neutron | 21:36 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Clean neutron.tests.functional.agent.linux.base https://review.openstack.org/161913 | 21:36 |
| *** lpetrut has joined #openstack-neutron | 21:36 | |
| *** ChuckC has quit IRC | 21:36 | |
| HenryG | kevinbenton: your HEAD^ in that patch is really out of date | 21:36 |
| *** achanda has joined #openstack-neutron | 21:37 | |
| *** julien_ZTE has joined #openstack-neutron | 21:37 | |
| kevinbenton | HenryG: want me to rebase? | 21:38 |
| *** aranjan_ has quit IRC | 21:39 | |
| *** alexpilotti has joined #openstack-neutron | 21:39 | |
| *** dims has joined #openstack-neutron | 21:39 | |
| HenryG | kevinbenton: You touched an unchanged area, zuul may be clever enough to merge it | 21:39 |
| *** aranjan has joined #openstack-neutron | 21:39 | |
| kevinbenton | HenryG: yeah, there shouldn't be any merge conflicts otherwise we would have heard already | 21:40 |
| *** salv-orlando has quit IRC | 21:40 | |
| *** localloop127 has quit IRC | 21:40 | |
| *** safchain has joined #openstack-neutron | 21:41 | |
| HenryG | kevinbenton: but my guess is this introduced the problem: https://review.openstack.org/137017 | 21:42 |
| *** safchain has quit IRC | 21:42 | |
| *** salv-orlando has joined #openstack-neutron | 21:42 | |
| kevinbenton | HenryG: looks like it | 21:43 |
| *** moha_hunt has joined #openstack-neutron | 21:43 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Define get_cidr helper function https://review.openstack.org/143869 | 21:43 |
| *** aranjan has quit IRC | 21:43 | |
| kevinbenton | HenryG: i don't see any others that call report_state | 21:44 |
| kevinbenton | HenryG: so that must be it | 21:44 |
| HenryG | kevinbenton: line 298,299 should have used setup_coreplugin() | 21:44 |
| *** dboik_ has quit IRC | 21:44 | |
| *** dboik has joined #openstack-neutron | 21:45 | |
| *** kbringard has quit IRC | 21:45 | |
| kevinbenton | HenryG: yeah, i wonder if i should adjust my fix to just touch that class | 21:45 |
| kevinbenton | HenryG: what do you think? | 21:45 |
| *** aranjan has joined #openstack-neutron | 21:46 | |
| *** dboik_ has joined #openstack-neutron | 21:46 | |
| HenryG | kevinbenton: might be safer. Calling cleanup_core_plugin() without setup_coreplugin() is untested :) | 21:46 |
| *** xgerman has joined #openstack-neutron | 21:49 | |
| *** sripriya has quit IRC | 21:49 | |
| *** tsg has joined #openstack-neutron | 21:49 | |
| *** sripriya has joined #openstack-neutron | 21:50 | |
| *** dboik has quit IRC | 21:50 | |
| *** dboik_ has quit IRC | 21:51 | |
| *** rpothier has quit IRC | 21:51 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Make DHCP tests cleanup neutron manager reference https://review.openstack.org/166014 | 21:52 |
| kevinbenton | HenryG: ^^ | 21:52 |
| *** lpetrut has quit IRC | 21:53 | |
| *** achanda has quit IRC | 21:53 | |
| *** ijw has joined #openstack-neutron | 21:56 | |
| *** abehl has quit IRC | 21:57 | |
| *** absubram has joined #openstack-neutron | 21:57 | |
| otherwiseguy | kevinbenton: my patch has successfully passed gate-neutron-python27. :) | 21:57 |
| *** yfried|afk has quit IRC | 21:58 | |
| *** yfried|afk has joined #openstack-neutron | 21:58 | |
| *** oomichi has joined #openstack-neutron | 21:59 | |
| kevinbenton | otherwiseguy: i just pushed another one so i think you need to rebase now :) | 22:00 |
| *** lykinsbd has quit IRC | 22:00 | |
| openstackgerrit | Terry Wilson proposed openstack/neutron: Add native OVSDB implementation of OVSDB API https://review.openstack.org/152761 | 22:00 |
| ijw | dougwig: ping | 22:00 |
| *** achanda has joined #openstack-neutron | 22:01 | |
| *** cference has joined #openstack-neutron | 22:02 | |
| *** dannywilson has quit IRC | 22:03 | |
| *** dannywilson has joined #openstack-neutron | 22:07 | |
| *** dsneddon is now known as dsneddon_afk | 22:07 | |
| dougwig | ijw: ack | 22:08 |
| *** jamielennox|away is now known as jamielennox | 22:09 | |
| *** dboik has joined #openstack-neutron | 22:10 | |
| *** nati_ueno has joined #openstack-neutron | 22:11 | |
| *** dboik_ has joined #openstack-neutron | 22:11 | |
| *** julien_ZTE has quit IRC | 22:12 | |
| *** Rajeev has quit IRC | 22:13 | |
| *** dboik has quit IRC | 22:15 | |
| *** packet has quit IRC | 22:15 | |
| ijw | dougwig: just trying to establish what actually got 'agreed' in IRC this morning on the VLAN trunking stuff, because garyk has reported to the mailing list that an unnamed quorum of people have for unexlained reasons decided things | 22:16 |
| dougwig | ijw: as far as i know, we ended with pritesh taking the action to figure out the next step. i didn't think anything was decided. | 22:17 |
| *** apuimedo has joined #openstack-neutron | 22:17 | |
| dougwig | if that happened, it was without me. | 22:17 |
| ijw | Given the specific choice (stick VLAN trunking in an extension) is exactly how the spec started out and something I changed because reviewers insisted I would like to make sure that you're making a valid choice for good reasons if you're going to override our decision making process | 22:17 |
| *** yamamoto has joined #openstack-neutron | 22:17 | |
| ijw | ok, then he might be overstating things somewhat... | 22:17 |
| dougwig | reading the thread, yes. | 22:18 |
| *** dims has quit IRC | 22:19 | |
| *** catohornet has quit IRC | 22:20 | |
| *** Marga_ has quit IRC | 22:21 | |
| *** Marga_ has joined #openstack-neutron | 22:21 | |
| *** yamamoto has quit IRC | 22:21 | |
| *** ChuckC has joined #openstack-neutron | 22:24 | |
| *** rook has quit IRC | 22:25 | |
| * kevinbenton makes popcorn for the mailing list fight | 22:30 | |
| *** shwetaap has quit IRC | 22:31 | |
| *** boris-42 has joined #openstack-neutron | 22:35 | |
| *** rushil_ has quit IRC | 22:35 | |
| amuller | kevinbenton: yay you rebase the py27 fix | 22:36 |
| amuller | rebased* | 22:36 |
| amuller | now I can rebase my crap on it | 22:36 |
| *** aveiga has joined #openstack-neutron | 22:36 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Send notification to controller about HA router state change https://review.openstack.org/126188 | 22:38 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Expose ha_state per router to agent binding via API https://review.openstack.org/126189 | 22:38 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Reuse nova batch notifier https://review.openstack.org/128013 | 22:38 |
| kevinbenton | amuller: yep! | 22:38 |
| amuller | bam | 22:38 |
| amuller | marun: Heya, can you approve https://review.openstack.org/#/c/128013/ again? | 22:39 |
| amuller | had to rebase on Kevin's fix for the py27 issue | 22:39 |
| amuller | it still has two +2's | 22:39 |
| otherwiseguy | kevinbenton: new version of fix fails: https://jenkins04.openstack.org/job/gate-neutron-python27/5447/console | 22:50 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Reload DHCP interface when its port is updated https://review.openstack.org/163758 | 22:50 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Reload DHCP interface when its port is updated https://review.openstack.org/163758 | 22:51 |
| *** jobewan has quit IRC | 22:51 | |
| otherwiseguy | (at least with my patch on top of it) | 22:52 |
| *** matrohon has quit IRC | 22:52 | |
| *** tiswanso has joined #openstack-neutron | 22:57 | |
| *** Sukhdev has quit IRC | 22:57 | |
| marun | amuller: sure | 22:58 |
| amuller | marun: looks like there's still issues with kevin's latest PS though | 22:58 |
| *** sbfox has quit IRC | 22:58 | |
| *** dannywilson has quit IRC | 22:58 | |
| otherwiseguy | kevinbenton: which is weird because the other patches that use eit seem to have passed py27. | 22:58 |
| kevinbenton | otherwiseguy: crap. looks like another leaked mock | 22:59 |
| *** rushil has joined #openstack-neutron | 22:59 | |
| *** marios has quit IRC | 23:01 | |
| *** ijw has quit IRC | 23:01 | |
| *** oomichi has quit IRC | 23:02 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Reload DHCP interface when its port is updated https://review.openstack.org/163758 | 23:02 |
| *** c_soukup has quit IRC | 23:03 | |
| kevinbenton | otherwiseguy: what patch is that failure on? | 23:03 |
| otherwiseguy | 152761 | 23:03 |
| otherwiseguy | https://review.openstack.org/#/c/152761/ | 23:03 |
| *** rook has joined #openstack-neutron | 23:03 | |
| *** pritesh has quit IRC | 23:05 | |
| *** yalie has quit IRC | 23:05 | |
| *** yalie1 has joined #openstack-neutron | 23:05 | |
| openstackgerrit | Juergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration https://review.openstack.org/141130 | 23:08 |
| openstackgerrit | Juergen Brendel proposed openstack/neutron: ARP spoofing patch: Data structures for rules. https://review.openstack.org/157097 | 23:08 |
| *** dane_leblanc has quit IRC | 23:08 | |
| *** julien_ZTE has joined #openstack-neutron | 23:10 | |
| *** s3wong has quit IRC | 23:14 | |
| *** tiswanso_ has joined #openstack-neutron | 23:15 | |
| *** tiswanso has quit IRC | 23:15 | |
| *** s3wong has joined #openstack-neutron | 23:16 | |
| *** banix has joined #openstack-neutron | 23:16 | |
| *** julien_ZTE has quit IRC | 23:17 | |
| *** julien_ZTE has joined #openstack-neutron | 23:17 | |
| *** ByPasS has quit IRC | 23:18 | |
| *** xgerman has quit IRC | 23:18 | |
| *** kutija has quit IRC | 23:24 | |
| *** moha_hunt has quit IRC | 23:24 | |
| *** leenheer has quit IRC | 23:25 | |
| *** chlong has joined #openstack-neutron | 23:27 | |
| *** leenheer has joined #openstack-neutron | 23:27 | |
| *** ijw has joined #openstack-neutron | 23:28 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Ml2 Mechanism Driver for OVSvApp Solution https://review.openstack.org/160247 | 23:29 |
| *** ijw has quit IRC | 23:29 | |
| *** ijw has joined #openstack-neutron | 23:30 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Subnet allocation from a subnet pool https://review.openstack.org/157597 | 23:30 |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Subnet allocation from a subnet pool https://review.openstack.org/157597 | 23:31 |
| *** shaohe_feng has joined #openstack-neutron | 23:35 | |
| *** dannywilson has joined #openstack-neutron | 23:35 | |
| *** zzzeek has quit IRC | 23:35 | |
| *** Marga_ has quit IRC | 23:36 | |
| *** Marga_ has joined #openstack-neutron | 23:36 | |
| *** yuanying has joined #openstack-neutron | 23:39 | |
| *** yuanying has quit IRC | 23:41 | |
| *** yuanying has joined #openstack-neutron | 23:41 | |
| *** ijw has quit IRC | 23:44 | |
| *** dsneddon_afk is now known as dsneddon | 23:45 | |
| *** dane_leblanc has joined #openstack-neutron | 23:46 | |
| *** ajmiller has quit IRC | 23:46 | |
| *** julien_ZTE has quit IRC | 23:53 | |
| *** watanabe_isao has joined #openstack-neutron | 23:54 | |
| *** dane_leblanc has quit IRC | 23:54 | |
| *** jckasper has joined #openstack-neutron | 23:55 | |
| *** padkrish_ has quit IRC | 23:56 | |
| *** padkrish has joined #openstack-neutron | 23:57 | |
| *** jorgem has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!