Thursday, 2015-07-16

*** dannywilson has quit IRC00:01
*** padkrish has quit IRC00:02
*** ijw has quit IRC00:02
*** jbell8 has joined #openstack-neutron00:02
*** ijw has joined #openstack-neutron00:03
*** Sukhdev has quit IRC00:03
*** mutoulbj has quit IRC00:03
*** minwang2 has quit IRC00:04
*** lnicolas has quit IRC00:05
openstackgerritMerged openstack/neutron: populate port security default into network  https://review.openstack.org/18820900:05
*** TrevorV has quit IRC00:05
*** carl_baldwin has joined #openstack-neutron00:05
*** sbalukoff has quit IRC00:06
*** rm_work is now known as rm_work|away00:07
*** lnicolas has joined #openstack-neutron00:07
*** padkrish has joined #openstack-neutron00:09
*** ivar-laz_ has quit IRC00:11
*** ivar-lazzaro has joined #openstack-neutron00:12
*** carl_baldwin has quit IRC00:12
*** nlahouti has quit IRC00:17
*** ivar-laz_ has joined #openstack-neutron00:18
*** madhu_ak has quit IRC00:18
*** nlahouti has joined #openstack-neutron00:19
*** nlahouti has quit IRC00:19
*** pavel_bondar has quit IRC00:20
*** pavel_bondar has joined #openstack-neutron00:21
*** ivar-lazzaro has quit IRC00:22
*** ivar-laz_ has quit IRC00:23
*** padkrish has quit IRC00:23
*** padkrish has joined #openstack-neutron00:23
*** salv-orlando has quit IRC00:24
openstackgerritMerged openstack/neutron: Create dvr base class and stop passing around snat_ports  https://review.openstack.org/20029400:24
*** pritesh has joined #openstack-neutron00:25
*** salv-orlando has joined #openstack-neutron00:26
*** sridhar_ram1 has quit IRC00:26
*** achanda_ has quit IRC00:26
*** padkrish has quit IRC00:28
*** britthouser has joined #openstack-neutron00:28
*** _cjones_ has quit IRC00:29
*** ajmiller has quit IRC00:31
*** sripriya has quit IRC00:31
*** ijw_ has joined #openstack-neutron00:31
*** ajmiller has joined #openstack-neutron00:31
*** mitchjameson has quit IRC00:32
*** pritesh has quit IRC00:32
*** salv-orlando has quit IRC00:34
*** ijw has quit IRC00:34
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port  https://review.openstack.org/19444100:36
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent  https://review.openstack.org/19444600:36
openstackgerritenikanorov proposed openstack/neutron: Graceful ovs-agent restart  https://review.openstack.org/18292000:36
*** ujjain has quit IRC00:36
*** tflynn has quit IRC00:36
*** yuanying_ has joined #openstack-neutron00:36
*** ujjain has joined #openstack-neutron00:37
*** ujjain has quit IRC00:37
*** ujjain has joined #openstack-neutron00:37
*** topshare has joined #openstack-neutron00:37
*** vivek-ebay has joined #openstack-neutron00:38
*** yuanying has quit IRC00:40
*** regXboi has quit IRC00:41
*** sballe has quit IRC00:42
*** yuanying_ has quit IRC00:42
*** yonglihe has joined #openstack-neutron00:43
*** dsneddon is now known as dsneddon_biab00:43
*** yuanying has joined #openstack-neutron00:45
*** padkrish has joined #openstack-neutron00:45
*** padkrish has quit IRC00:45
*** padkrish has joined #openstack-neutron00:46
*** eckelcu has joined #openstack-neutron00:49
*** cing has joined #openstack-neutron00:50
openstackgerritArmando Migliaccio proposed openstack/neutron: Try to decentralize the managemement of service providers  https://review.openstack.org/20220700:50
*** ijw_ has quit IRC00:50
*** banix has joined #openstack-neutron00:50
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port  https://review.openstack.org/19444100:50
*** ijw has joined #openstack-neutron00:51
*** signed8bit_ZZZzz is now known as signed8bit00:51
*** yuanying has quit IRC00:52
*** padkrish has quit IRC00:52
*** baoli has joined #openstack-neutron00:52
*** eckelcu has quit IRC00:53
*** shashank_hegde_ has quit IRC00:55
*** spandhe has quit IRC00:57
openstackgerritshihanzhang proposed openstack/neutron: Unplug the VIF if dhcp port is deleted  https://review.openstack.org/19692700:58
*** eckelcu has joined #openstack-neutron00:58
openstackgerrityalei wang proposed openstack/neutron: pass the extention driver exception to plugin  https://review.openstack.org/20206100:59
*** Longgeek has joined #openstack-neutron01:00
*** dsneddon_biab is now known as dsneddon01:00
*** signed8bit is now known as signed8bit_ZZZzz01:02
*** mutoulbj has joined #openstack-neutron01:03
*** nati_ueno has joined #openstack-neutron01:04
*** nati_ueno has quit IRC01:04
*** mikeymeitbual has joined #openstack-neutron01:08
*** mutoulbj has quit IRC01:09
openstackgerritshihanzhang proposed openstack/neutron: Install arp spoofing protection flow after setting port tag  https://review.openstack.org/19942501:11
*** yuanying has joined #openstack-neutron01:12
*** vhosakot has joined #openstack-neutron01:14
*** erkules_ has joined #openstack-neutron01:14
*** lazy_prince has joined #openstack-neutron01:14
*** openstack has joined #openstack-neutron01:25
*** annegentle has quit IRC01:26
*** yamahata has quit IRC01:26
*** minwang2 has quit IRC01:28
*** minwang2 has joined #openstack-neutron01:28
*** breitz has quit IRC01:29
*** breitz has joined #openstack-neutron01:30
*** ijw has quit IRC01:32
*** mestery_ is now known as mestery01:34
*** stanzgy has joined #openstack-neutron01:35
*** rushil has joined #openstack-neutron01:35
*** patrickeast has quit IRC01:35
*** fitoduarte has joined #openstack-neutron01:36
*** ijw has joined #openstack-neutron01:36
*** baoli_ has joined #openstack-neutron01:36
*** annegentle has joined #openstack-neutron01:37
*** baoli has quit IRC01:38
*** ThiagoCMC has quit IRC01:40
*** signed8bit_ZZZzz is now known as signed8bit01:41
*** annegentle has quit IRC01:42
*** lazy_prince has quit IRC01:43
*** shashank_hegde_ has joined #openstack-neutron01:46
openstackgerritgongysh proposed openstack/neutron: Replace internal calls of create_{network, subnet, port}  https://review.openstack.org/12977301:47
*** jbell8 has quit IRC01:47
*** abhiraut has quit IRC01:48
*** ijw has quit IRC01:49
*** park_ has joined #openstack-neutron01:50
*** ivar-lazzaro has joined #openstack-neutron01:50
openstackgerrityalei wang proposed openstack/neutron: Pass the extension driver exception to plugin  https://review.openstack.org/20206101:51
*** rdo has quit IRC01:51
*** signed8bit is now known as signed8bit_ZZZzz01:52
*** tflynn has joined #openstack-neutron01:53
*** tflynn_ has joined #openstack-neutron01:54
*** signed8bit_ZZZzz is now known as signed8bit01:56
*** MentalRay__ has joined #openstack-neutron01:57
*** tflynn has quit IRC01:57
*** dims_ has quit IRC01:58
*** tflynn_ has quit IRC01:59
*** minwang2 has quit IRC01:59
*** dlundquist has joined #openstack-neutron01:59
*** tflynn has joined #openstack-neutron01:59
*** vivek-ebay has quit IRC02:01
*** aranjan has quit IRC02:03
*** minwang2 has joined #openstack-neutron02:03
*** doude has quit IRC02:04
*** tflynn has quit IRC02:04
*** tflynn has joined #openstack-neutron02:05
*** dave-mccowan has joined #openstack-neutron02:05
*** signed8bit has quit IRC02:06
*** aranjan has joined #openstack-neutron02:07
*** doude has joined #openstack-neutron02:07
*** doude has quit IRC02:07
*** doude has joined #openstack-neutron02:07
*** clawft has quit IRC02:07
*** annegentle has joined #openstack-neutron02:10
*** annegentle has quit IRC02:13
*** minwang2 has quit IRC02:14
*** minwang2 has joined #openstack-neutron02:16
*** dims__ has joined #openstack-neutron02:18
openstackgerritZhi Chang proposed openstack/neutron: Keep dns nameserver order consistency  https://review.openstack.org/19958802:20
*** yamamoto has joined #openstack-neutron02:22
*** markvoelker has quit IRC02:22
*** aranjan has quit IRC02:22
*** baoli_ has quit IRC02:23
*** ijw has joined #openstack-neutron02:24
*** yamamoto_ has joined #openstack-neutron02:24
*** yamamoto has quit IRC02:24
*** dsneddon is now known as dsneddon_away02:24
*** wwriverrat has quit IRC02:25
*** shashank_hegde_ has quit IRC02:25
*** shashank_hegde_ has joined #openstack-neutron02:25
*** tflynn has quit IRC02:26
*** dlundquist has left #openstack-neutron02:28
*** vivek-ebay has joined #openstack-neutron02:29
*** tflynn has joined #openstack-neutron02:30
*** Longgeek has joined #openstack-neutron02:30
*** dims__ has quit IRC02:30
*** jwy has quit IRC02:31
*** mickeys has joined #openstack-neutron02:32
*** jyuso1 has quit IRC02:32
*** tflynn has quit IRC02:33
*** tflynn has joined #openstack-neutron02:33
*** yamahata has joined #openstack-neutron02:34
*** baoli has joined #openstack-neutron02:35
*** Longgeek has quit IRC02:35
*** yamahata has quit IRC02:36
*** rushil has quit IRC02:37
openstackgerritIsaku Yamahata proposed openstack/neutron: l3_db: it updates port attribute without l2 plugin  https://review.openstack.org/20235702:37
*** yamahata has joined #openstack-neutron02:37
openstackgerritIsaku Yamahata proposed openstack/neutron: l3_db: not use l2 plugin _get_port unnecessarily  https://review.openstack.org/20235802:37
openstackgerritIsaku Yamahata proposed openstack/neutron: l3_db: not sue l3 plugin _get_subnet unnecessarily  https://review.openstack.org/20235902:37
*** minwang2 has quit IRC02:37
openstackgerritZhi Chang proposed openstack/neutron: Keep dns nameserver order consistency  https://review.openstack.org/19958802:38
openstackgerritLynn Li proposed openstack/neutron: Update device info for FIP paired port w/unbound  https://review.openstack.org/20236002:40
*** vivek-ebay has quit IRC02:43
*** shivrao has joined #openstack-neutron02:43
*** tflynn has quit IRC02:43
*** minwang2 has joined #openstack-neutron02:45
*** mickeys has quit IRC02:45
*** baoli has quit IRC02:47
*** tflynn has joined #openstack-neutron02:49
*** hitalia has quit IRC02:49
*** baoli has joined #openstack-neutron02:49
openstackgerritMerged openstack/neutron: Fix a regression in a recent IPAM change  https://review.openstack.org/20190702:50
*** ivar-lazzaro has quit IRC02:50
*** yamahata has quit IRC02:50
*** changzhi has joined #openstack-neutron02:51
*** julim has joined #openstack-neutron02:52
*** hakimo_ has joined #openstack-neutron02:53
*** sdake_ has joined #openstack-neutron02:53
*** minwang2_ has joined #openstack-neutron02:53
*** minwang2 has quit IRC02:54
*** minwang2_ is now known as minwang202:54
*** hakimo has quit IRC02:54
*** ijw has quit IRC02:56
openstackgerritLynn Li proposed openstack/neutron: Update device info for FIP paired port w/unbound  https://review.openstack.org/20236002:56
*** shivrao has quit IRC02:56
*** jbell8 has joined #openstack-neutron02:56
*** dave-mccowan has quit IRC02:56
*** sdake has quit IRC02:57
openstackgerritZhaoBo proposed openstack/neutron: Fixed pass wrong cidr check during create subnet  https://review.openstack.org/20194203:04
*** JonB has quit IRC03:05
*** tflynn has quit IRC03:10
*** crc32 has quit IRC03:10
*** tflynn has joined #openstack-neutron03:10
openstackgerritZhaoBo proposed openstack/neutron: Fixed pass wrong cidr check during create subnet  https://review.openstack.org/20194203:11
openstackgerritshihanzhang proposed openstack/neutron: Install arp spoofing protection flow after setting port tag  https://review.openstack.org/19942503:13
*** bradjones has quit IRC03:14
*** minwang2 has quit IRC03:15
*** yuanying_ has joined #openstack-neutron03:16
*** yuanying_ has quit IRC03:18
*** yuanying has quit IRC03:18
*** yuanying_ has joined #openstack-neutron03:18
*** bradjones has joined #openstack-neutron03:19
*** bradjones has quit IRC03:19
*** bradjones has joined #openstack-neutron03:19
*** minwang2 has joined #openstack-neutron03:22
*** tflynn has quit IRC03:25
*** pal has joined #openstack-neutron03:26
*** tflynn has joined #openstack-neutron03:27
*** spandhe has joined #openstack-neutron03:27
*** spandhe_ has joined #openstack-neutron03:30
*** britthou_ has joined #openstack-neutron03:30
*** yuanying_ has quit IRC03:30
*** minwang2 has quit IRC03:31
*** spandhe has quit IRC03:31
*** spandhe_ is now known as spandhe03:31
*** yuanying has joined #openstack-neutron03:32
*** minwang2 has joined #openstack-neutron03:32
*** jyuso1 has joined #openstack-neutron03:32
*** baoli has quit IRC03:33
*** liudong has joined #openstack-neutron03:33
*** britthouser has quit IRC03:33
*** markvoelker has joined #openstack-neutron03:33
*** dannywilson has joined #openstack-neutron03:34
*** markvoelker has quit IRC03:34
*** markvoelker has joined #openstack-neutron03:34
*** wwriverrat has joined #openstack-neutron03:36
*** abregman has quit IRC03:36
*** markvoelker_ has joined #openstack-neutron03:37
*** yuanying has quit IRC03:37
*** minwang2 has quit IRC03:38
*** yuanying has joined #openstack-neutron03:38
*** lazy_prince has joined #openstack-neutron03:38
*** markvoelker has quit IRC03:39
*** tflynn has quit IRC03:40
*** y_sawai has joined #openstack-neutron03:40
changzhiyamamoto: ping03:43
yamamoto_changzhi: hi03:43
changzhiyamammoto: please review https://review.openstack.org/#/c/199588/ :-)03:43
*** rward has quit IRC03:44
*** sweston has quit IRC03:44
yamamoto_changzhi: will look later today03:44
*** liudong has quit IRC03:44
*** briancurtin has quit IRC03:45
changzhiyamamoto: Thx :-)03:45
*** zhiyan has quit IRC03:45
*** ctracey has quit IRC03:45
*** vivek-ebay has joined #openstack-neutron03:45
*** sthillma has quit IRC03:45
*** rex_lee has quit IRC03:46
*** y_sawai has quit IRC03:46
*** X-Rob has quit IRC03:46
*** y_sawai has joined #openstack-neutron03:46
*** edwarnicke has quit IRC03:46
*** ramishra has quit IRC03:46
*** vivek-ebay has quit IRC03:46
*** boris-42 has quit IRC03:46
*** y_sawai has quit IRC03:47
*** coolsvap|away is now known as coolsvap03:47
*** banix has quit IRC03:48
*** liudong has joined #openstack-neutron03:48
*** tflynn has joined #openstack-neutron03:49
*** shashank_hegde has quit IRC03:50
*** dougwig has quit IRC03:50
*** shashank_hegde_ has quit IRC03:51
*** StevenK has quit IRC03:51
*** dannywilson has quit IRC03:52
*** rm_work|away has quit IRC03:52
*** dannywilson has joined #openstack-neutron03:52
*** vivek-ebay has joined #openstack-neutron03:53
*** zhiyan has joined #openstack-neutron03:54
*** sdake_ has quit IRC03:55
*** StevenK has joined #openstack-neutron03:55
*** tflynn has quit IRC03:55
*** vivek-eb_ has joined #openstack-neutron03:56
*** briancurtin has joined #openstack-neutron03:56
openstackgerritZhiyuan Cai proposed openstack/neutron-specs: Add resource timestamp  https://review.openstack.org/19695903:57
*** vivek-ebay has quit IRC03:58
*** yuanying has quit IRC04:00
*** edwarnicke has joined #openstack-neutron04:00
*** yuanying has joined #openstack-neutron04:01
*** rex_lee has joined #openstack-neutron04:02
*** ctracey has joined #openstack-neutron04:02
*** yeungp has joined #openstack-neutron04:02
*** achanda has joined #openstack-neutron04:02
*** yuanying has quit IRC04:02
*** yeungp has quit IRC04:02
*** mkolesni has joined #openstack-neutron04:02
*** X-Rob has joined #openstack-neutron04:04
*** oomichi has quit IRC04:05
*** yuanying has joined #openstack-neutron04:07
*** boris-42 has joined #openstack-neutron04:07
*** iwamoto has joined #openstack-neutron04:09
*** ijw has joined #openstack-neutron04:10
*** sthillma has joined #openstack-neutron04:11
*** rward has joined #openstack-neutron04:11
*** sbalukoff has joined #openstack-neutron04:11
*** changzhi has quit IRC04:12
*** kiran-r has joined #openstack-neutron04:15
*** britthouser has joined #openstack-neutron04:15
*** britthou_ has quit IRC04:16
*** sweston has joined #openstack-neutron04:16
*** Longgeek has joined #openstack-neutron04:17
*** spandhe_ has joined #openstack-neutron04:18
*** spandhe has quit IRC04:19
*** spandhe_ is now known as spandhe04:19
*** gberginc has joined #openstack-neutron04:20
*** shivrao has joined #openstack-neutron04:21
*** shashank_hegde has joined #openstack-neutron04:22
*** tflynn has joined #openstack-neutron04:23
*** yamamoto_ has quit IRC04:23
*** shivrao_ has joined #openstack-neutron04:24
*** Longgeek has quit IRC04:25
*** Longgeek has joined #openstack-neutron04:25
*** yamamoto has joined #openstack-neutron04:26
*** shivrao has quit IRC04:26
*** shivrao_ is now known as shivrao04:26
*** tflynn has quit IRC04:26
*** tflynn has joined #openstack-neutron04:27
*** cing has quit IRC04:27
*** dougwig has joined #openstack-neutron04:29
*** ramishra has joined #openstack-neutron04:29
*** tflynn has quit IRC04:31
*** tflynn has joined #openstack-neutron04:31
*** yatin has joined #openstack-neutron04:34
*** TrevorV has joined #openstack-neutron04:34
*** cing has joined #openstack-neutron04:35
*** shashank_hegde has quit IRC04:36
*** zzzeek has joined #openstack-neutron04:37
*** vikram has joined #openstack-neutron04:37
*** rm_work has joined #openstack-neutron04:40
*** irenab has quit IRC04:40
*** MentalRay__ has quit IRC04:41
*** shashank_hegde has joined #openstack-neutron04:42
*** tflynn has quit IRC04:43
*** tflynn has joined #openstack-neutron04:43
*** daneyon has quit IRC04:45
*** yog__ has joined #openstack-neutron04:47
*** kiran-r has quit IRC04:47
*** y_sawai has joined #openstack-neutron04:47
*** shashank_hegde_ has joined #openstack-neutron04:48
*** markvoelker has joined #openstack-neutron04:49
*** zzzeek has quit IRC04:52
*** doude has quit IRC04:52
*** y_sawai has quit IRC04:52
*** markvoelker_ has quit IRC04:53
*** ekuris has joined #openstack-neutron04:56
*** abregman has joined #openstack-neutron05:00
*** yfried|afk has quit IRC05:01
*** SourabhP has joined #openstack-neutron05:02
*** sthillma_ has joined #openstack-neutron05:07
*** shashank_hegde_ has quit IRC05:07
*** kiran-r has joined #openstack-neutron05:09
*** sthillma has quit IRC05:10
*** sthillma_ is now known as sthillma05:10
*** veena has joined #openstack-neutron05:12
*** TrevorV has quit IRC05:13
*** killer_prince has joined #openstack-neutron05:16
*** lazy_prince has quit IRC05:19
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: OVS-agent: Fix a docstring typo  https://review.openstack.org/20238605:21
*** SridharG has joined #openstack-neutron05:23
*** rm_work is now known as rm_work|away05:23
*** daneyon has joined #openstack-neutron05:24
*** hitalia has joined #openstack-neutron05:26
*** shashank_hegde_ has joined #openstack-neutron05:29
openstackgerritMadhusudhan Kandadai proposed openstack/neutron: Flavor Framework implementation  https://review.openstack.org/13975805:30
*** madhu_ak has joined #openstack-neutron05:30
*** anilvenkata has joined #openstack-neutron05:30
*** vivek-eb_ has quit IRC05:31
*** _kiran_ has joined #openstack-neutron05:33
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: test_l2_ovs_agent: _bind_port fixes  https://review.openstack.org/20238805:34
*** kiranr has joined #openstack-neutron05:35
*** kiranr has quit IRC05:35
*** kiran-r has quit IRC05:37
*** lajos-katona has joined #openstack-neutron05:37
*** _kiran_ has quit IRC05:39
*** sudipto has joined #openstack-neutron05:39
*** SourabhP has quit IRC05:42
*** vishwana_ has joined #openstack-neutron05:43
*** jbell8 has quit IRC05:43
*** moshele has joined #openstack-neutron05:43
*** irenab has joined #openstack-neutron05:44
*** Swami has quit IRC05:46
*** vishwanathj has quit IRC05:47
*** y_sawai has joined #openstack-neutron05:48
*** y_sawai_ has joined #openstack-neutron05:48
*** NightKhaos has joined #openstack-neutron05:49
*** Murali has joined #openstack-neutron05:50
*** fitoduarte has quit IRC05:51
*** itzikb_ has joined #openstack-neutron05:52
*** y_sawai has quit IRC05:52
*** madhu_ak has quit IRC05:54
openstackgerritMerged openstack/neutron: Avoid using logging in signal handler  https://review.openstack.org/20147405:55
*** juliancash has quit IRC05:55
*** rm_work|away is now known as rm_work05:56
*** dboik has joined #openstack-neutron05:57
*** rm_work is now known as rm_work|away05:58
*** lazy_prince has joined #openstack-neutron05:59
*** spandhe has quit IRC06:02
*** killer_prince has quit IRC06:02
*** dboik has quit IRC06:02
*** yfried has joined #openstack-neutron06:03
openstackgerritHirofumi Ichihara proposed openstack/neutron: Enable to update external network subnet's gateway-ip  https://review.openstack.org/19493906:05
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/20192806:09
*** Sukhdev has joined #openstack-neutron06:11
*** scheuran has joined #openstack-neutron06:11
*** y_sawai_ has quit IRC06:13
*** moshele has quit IRC06:13
*** ildikov has quit IRC06:18
*** obondarev__ has joined #openstack-neutron06:19
*** lukasa has joined #openstack-neutron06:21
*** spligak has quit IRC06:25
*** trinaths has joined #openstack-neutron06:25
*** dannywilson has quit IRC06:26
*** vivek-ebay has joined #openstack-neutron06:28
*** vivek-ebay has quit IRC06:32
*** obondarev__ has quit IRC06:38
openstackgerritfumihiko kakuma proposed openstack/neutron: Stop use of oslo_utils.timeutils.strtime()  https://review.openstack.org/20240106:40
openstackgerritNuman Siddique proposed openstack/neutron: Add address scope API tests  https://review.openstack.org/19885006:42
mkolesniajo: yo06:42
yaliehi, anyone knows that in neutron cmd line, type=list equals to list=True?06:44
*** Sukhdev has quit IRC06:45
*** jschwarz has joined #openstack-neutron06:46
NightKhaosSo, my qrouters aren't passing traffic through. This means I can't ping anything on the other side of the router (i.e. instance going out, or client coming into floating). Any traffic that should be SNATed or DNATed across the interface is dropped, but not by IPTables. Anyone come across this issue before?06:47
jschwarzirenab, ping06:47
NightKhaosSo, my qrouters aren't passing traffic through. This means I can't ping anything on the other side of the router (i.e. instance going out, or client coming into floating). Any traffic that should be SNATed or DNATed across the interface is dropped, but not by IPTables. Anyone come across this issue before?06:47
NightKhaosWhoops. didn't mean to double send sorry06:47
irenabjschwarz: hey06:48
jschwarzirenab, morning :)06:48
jschwarzirenab, how are you?06:48
irenabjschwarz: good morning. I am fine. How about you?06:48
jschwarzirenab, a bit tired but that's to be expected so early in the morning ;-)06:49
jschwarzirenab, what can I help you with?06:49
irenabjschwarz: Are we in the same time zone :-)?06:49
jschwarzirenab, we are. I'm just lazier than you06:50
irenabjschwarz: :-)06:50
*** pal has quit IRC06:51
*** Longgeek has quit IRC06:51
mkolesniajo: ping06:51
*** Longgeek has joined #openstack-neutron06:52
*** kawa2014 has joined #openstack-neutron06:54
trinathsHi, Good Morning All06:56
NightKhaosI'm in the midddle of writing a huge ask openstack post with excessive detail, so if any of you have experienced my issue and can save me the effort I'd really apperciate it. :)06:57
openstackgerritMerged openstack/neutron: Different approach to indicate failure on SystemExit  https://review.openstack.org/18341106:57
openstackgerritArmando Migliaccio proposed openstack/neutron: Decentralize the managemement of service providers  https://review.openstack.org/20220706:58
*** alex_klimov has joined #openstack-neutron06:59
jschwarzmkolesni, what's the status of https://review.openstack.org/#/c/197564/ ?06:59
jschwarzmkolesni, will it merge today?06:59
*** moshele has joined #openstack-neutron07:03
*** ildikov has joined #openstack-neutron07:04
mkolesnijschwarz: dont ask me, ask the gate :)07:05
*** pal has joined #openstack-neutron07:05
jschwarzmkolesni, roge07:05
jschwarzr07:05
*** Tiancheng has joined #openstack-neutron07:06
*** x58 has quit IRC07:09
*** adjohn has quit IRC07:09
*** x58 has joined #openstack-neutron07:10
*** gildub has quit IRC07:12
*** jlibosva has joined #openstack-neutron07:13
*** wim_dc has joined #openstack-neutron07:15
openstackgerritNuman Siddique proposed openstack/neutron: Support subnetpool association to an address scope  https://review.openstack.org/19755207:15
*** adjohn- has joined #openstack-neutron07:15
*** devvesa has joined #openstack-neutron07:16
wim_dcHey, is this also the channel to ask about the neutron pythonclient?07:17
*** abehl has joined #openstack-neutron07:19
openstackgerritgongysh proposed openstack/neutron: Create fip on subnet id  https://review.openstack.org/19870307:21
openstackgerritRen Qiaowei proposed openstack/neutron: Remove unnecessary executable permission  https://review.openstack.org/20241607:21
openstackgerritgongysh proposed openstack/neutron: Create fip on subnet id  https://review.openstack.org/19870307:24
*** changzhi has joined #openstack-neutron07:25
*** erkules_ is now known as erkules07:30
*** erkules has quit IRC07:30
*** erkules has joined #openstack-neutron07:30
*** ihrachyshka has joined #openstack-neutron07:32
*** jbell8 has joined #openstack-neutron07:32
*** evgenyf has joined #openstack-neutron07:36
*** jbell8 has quit IRC07:37
openstackgerritNuman Siddique proposed openstack/neutron: Add the subnetpool created in the api test to cleanup  https://review.openstack.org/20241907:37
*** jlanoux has joined #openstack-neutron07:41
*** pal_ has joined #openstack-neutron07:42
*** nijaba has quit IRC07:43
openstackgerritArmando Migliaccio proposed openstack/neutron: Decentralize the managemement of service providers  https://review.openstack.org/20220707:44
*** nijaba has joined #openstack-neutron07:44
*** nijaba has quit IRC07:44
*** nijaba has joined #openstack-neutron07:44
*** pal has quit IRC07:44
*** achanda has quit IRC07:46
*** yamamoto has quit IRC07:47
jschwarzmkolesni, ping07:48
mkolesnijschwarz: pong07:48
jschwarzmkolesni, so the code in https://review.openstack.org/#/c/197564/17/neutron/services/qos/qos_plugin.py (line 175) is broken07:48
jschwarzmkolesni, policy.create() returns None and 'to_dict()' fails07:49
jschwarz:<07:49
mkolesnijschwarz: yeah we know, the next patch there fixes it07:49
jschwarzahh07:49
mkolesniu testing locally?07:49
jschwarzmkolesni, yes07:49
mkolesnican u test some new code if i push it?07:49
jschwarzmkolesni, can you link me the patch that fixes there?07:49
jschwarzmkolesni, what kind of new code? :)07:50
mkolesnijschwarz: a new version of the code that's supposed to fix that07:50
jschwarzmkolesni, that's the idea.. trying to write some basic infra for QoS API testing so irenab can continue work on this next week07:50
mkolesnijschwarz: ok let me send a new version and give you the linlk07:51
jschwarzmkolesni, mostly 'create_qos_policy' in API client side and cleanup, etc07:51
jschwarzack07:51
*** anilvenkata has quit IRC07:51
*** tflynn has quit IRC07:52
*** fzdarsky has joined #openstack-neutron07:53
changzhimestery: please review https://review.openstack.org/#/c/199588/ :-)07:54
mkolesnijschwarz: if i rebase a change, will it reset the +2 +A flags?07:54
mkolesniihrachyshka: maybe u know07:54
ihrachyshkamkoderer, it will reset +A for sure07:55
jschwarzmkolesni, you want to rebase the QoS plugin one which was approved? not ure07:55
jschwarzsure*07:55
ihrachyshkamkolesni, ^^07:55
ihrachyshkamkoderer, sorry07:55
ihrachyshkamkolesni, as for +2, it depends on whether there are changes in the patch07:55
mkolesniihrachyshka: ok so not rebase it?07:55
mkolesniihrachyshka: i need to push some patches on top and git is complaining07:55
*** SridharG has left #openstack-neutron07:56
jschwarzmkolesni, if you're rebasing a patch on top of the QoS plugin one - feel free07:56
openstackgerritshihanzhang proposed openstack/neutron: Install arp spoofing protection flow after setting port tag  https://review.openstack.org/19942507:56
jschwarzmkolesni, I just did it with https://review.openstack.org/#/c/197126/ and it didn't reset your +A07:56
mkolesnijschwarz: no it also has to be rebased07:56
ihrachyshkamkolesni, I don't know what's your problem with git. stdout may help.07:56
*** neeti has joined #openstack-neutron07:56
jschwarzmkolesni, ah07:56
ihrachyshkajschwarz, for api tests, do we need new neutronclient?07:57
*** tmorin has joined #openstack-neutron07:57
jschwarzihrachyshka, negative. it uses a variation of tempest client07:57
ihrachyshkagood.07:57
mkolesniihrachyshka:  ! [remote rejected] HEAD -> refs/publish/feature/qos/neutron-qos (no new changes)07:57
ihrachyshkathat's one thing I realized I don't know for sure yesterday :)07:57
mkolesniihrachyshka: which is a liw07:57
*** sthillma has quit IRC07:57
mkolesni*lie07:57
mkolesniihrachyshka: then i saw it aint rebased07:57
jschwarz:)07:57
ihrachyshkamkolesni, apparently, your patch in gerrit is the same (identical git hash) as in your HEAD07:58
mkolesniihrachyshka: yeah that the way it should be07:58
mkolesni91c8932 (HEAD, qos_update_profile_id) Extend the resources using the QoS extension class07:58
mkolesni27eded2 Handle qos_profile_id on network/port create/update07:58
mkolesniebd6453 Implement QoS plugin07:58
mkolesnithe bottom one should be identical07:59
mkolesnithe middle one changed07:59
mkolesniand the top one is new07:59
mkolesnibut..07:59
ihrachyshkaajo, pff, service plugin is now failing on git timeout in gate07:59
jlibosvamkolesni: I had the same issue ~2 weeks ago07:59
* ajo tries not to look worried :/07:59
mkolesniebd6453 Implement QoS plugin <- should be rebased either way07:59
ihrachyshkashould we pray for its merge? maybe some live stock offer to infra will help?..07:59
jlibosvamkolesni: I had to make a change in the very bottom change (like put something to commit msg) but it reset then all votes on the bottom patch07:59
mkolesnijlibosva: thats wrong it should be like that08:00
ihrachyshkamkolesni, why rebase needed for service plugin?08:00
mkolesnijlibosva: the bottom change should have the same hash otherwise ur pushing a new patchset which definitely resets flags08:00
mkolesniihrachyshka: because the HEAD on the feature/qos branch moved08:01
ihrachyshkamkolesni, so what? you still can merge from whatever parent commit08:01
jlibosvamkolesni: it had the same changeid, it just refused to push patch on top of that due to no changes in bottom patch08:01
mkolesniihrachyshka: from git fetch: cdcffc7..a375e73  feature/qos -> origin/feature/qos08:01
*** shashank_hegde_ has quit IRC08:02
mkolesnijlibosva: yeah but i dont wanna push bottom patch, i want to depend on it08:02
ihrachyshkajlibosva, mkolesni I always send with -R, and I haven't experienced anything like that08:02
jlibosvamkolesni: ok, I don't know why I'm telling you this :) I wanted to do the same as what you are trying to do but I failed08:02
jlibosvamkolesni: anyway, if you'll find a way how to do it, please share your experience :)08:03
mkolesnijlibosva: ok no prob :)08:03
*** markvoelker has quit IRC08:04
*** tmorin has quit IRC08:04
mkolesniihrachyshka: ok -R did the trick08:04
mkolesnijlibosva: fyi ^08:04
jlibosvaihrachyshka++08:04
ihrachyshkajlibosva, ^^ huh, it seems it's not the default or what?08:04
mkolesniihrachyshka: still, that patch needs rebase it cant be merged like that anymore08:04
jlibosvaihrachyshka: based on our observing I thought it is08:05
ihrachyshkamkolesni, jlibosva once looked in git-review code and I think the result was it's default in fresh versions08:05
ihrachyshkamkolesni, maybe you use an old git-review08:05
ajoihrachyshka, I saw yesterday that the same DBDeadlocks on ipam during ml2 plugin extensions was also happening in master08:05
ihrachyshkamkolesni, which patch? please be more explicit08:05
ajobut, still I'm not sure if our local rate for the service plugin is higher08:05
*** pck has quit IRC08:06
ajoI also saw some patch from kevinbenton which could be related, removing the db-access lock, and restoring transactions in ml2 plugin08:06
ihrachyshkaajo, for that latest failure, I think it was smth about LOG from extending a port for port_security extension, in ml208:06
*** pck has joined #openstack-neutron08:06
*** armax has quit IRC08:06
mkolesniihrachyshka: ebd6453 Implement QoS plugin08:06
jschwarzmkolesni, so what should I depend my patch on? :)08:06
*** pal_ has quit IRC08:07
mkolesnijschwarz: https://review.openstack.org/#/c/202425/08:07
mkolesnijschwarz: try this08:07
ajoI was thinking of trying an statistical test, start 1 review patching the callback reg, 1 review patching return into the callback function, 1 review with kevinbenton  patch for ml2 transactions restored cherrypicked on top08:07
jschwarzmkolesni, trying08:07
kevinbentonajo, ihrachyshka: keep in mind that deadlocks are annoyingly logged as exceptions right now08:07
ihrachyshkamkolesni, why rebase needed for that? explain08:07
kevinbentonajo: even though they are now not failing events08:07
mkolesnijschwarz: 10x08:07
ihrachyshkakevinbenton, good to know.08:08
*** ygbo has joined #openstack-neutron08:08
ajoand issue a few rechecks on them08:08
ajothen gather statistics08:08
mkolesniihrachyshka: the branch moved, or does gate job hanlde that silently?08:08
kevinbentonajo, ihrachyshka: as long as the retry decorator is behaving as expected, they are essentially non-events08:08
ihrachyshkamkolesni, of course it does! gate merges branches08:08
ajoahh08:08
*** rossella_s has joined #openstack-neutron08:09
ajokevinbenton, we were seeing deadlocks in our branch again, but with the db-access lock still in08:09
ajobut, just because, we call for transactions inside our "AFTER_READ" callback for extensions, I  guess08:10
ajokevinbenton: I think the retry decorator was set long ago in June?08:10
jschwarzmkolesni, still no go08:10
jschwarzmkolesni, http://pastebin.com/exMmdiWg08:10
kevinbentonajo: yeah, but we weren't relying on it as heavily until the db-access lock was removed08:10
ihrachyshkamkolesni, just to make sure, please don't touch service plugin patch, it should merge as-is08:10
mkolesniihrachyshka: im not08:10
*** markvoelker has joined #openstack-neutron08:11
*** yamamoto has joined #openstack-neutron08:11
ajoihrachyshka: https://review.openstack.org/#/c/192084/08:11
mkolesnijschwarz: ok but what passed?08:11
jschwarzmkolesni, i didn't understand your question08:12
ajokevinbenton: ack08:12
ajoso as soon as retries are happening we should be safe,08:12
mkolesnijschwarz: what did u do, run the tests?08:12
*** safchain has joined #openstack-neutron08:12
mkolesniajo: i pushed code to remove those callbacks08:12
kevinbentonajo: yeah, for API operations as least08:12
mkolesniajo: https://review.openstack.org/#/c/202425/08:12
jschwarzmkolesni, I just invoked the 'create_policy' function08:12
kevinbentonajo: internal calls from AMQP or periodic tasks may encounter issues08:13
jschwarzmkolesni, not sure what's happening there... sec08:13
openstackgerritshihanzhang proposed openstack/neutron: Check the validity of security group rule port range  https://review.openstack.org/20152408:13
ajokevinbenton: I believe , btw that we're seeing instances of DBDeadlock, without retries happening, thus tempest failing08:13
ajoI will ping you if we find those back again08:13
kevinbentonajo: ack08:13
mkolesnijschwarz: seems like the policy wasnt created or something08:13
ajomkolesni, but that was not the plan :-??08:14
ajothat was the 2nd option plan?08:14
jschwarzmkolesni, also creating a bandwidth rule is broken... I assume you guys changed the spec again?08:14
ajoyou're pushing other plugins out08:14
ajomkolesni ^08:14
mkolesnijschwarz: no i didnt change anything about that08:14
*** SridharG has joined #openstack-neutron08:14
mkolesnijschwarz: i was just fiddleing with the qos_policy_id field on the port/network08:14
jschwarzmkolesni, anyway the policy wasn't created as you say08:14
mkolesniajo: what, no08:15
*** enikanorov_ has joined #openstack-neutron08:15
jschwarzso the qos plugin's create_policy is broken for some reason08:15
*** y_sawai has joined #openstack-neutron08:15
mkolesniajo: it's all in QosExtensionHandler class, easily reusable by any plugin08:15
ajomkolesni, if you remove the AFTER_READ callback, other plugins can't extend the port info, can they?08:15
ajomkolesni, ok, let me read more carefully ;)08:15
*** markvoelker has quit IRC08:16
mkolesniajo: sure they can, look at the patch and see the simplicity..08:16
*** y_sawai has quit IRC08:16
*** jistr has joined #openstack-neutron08:16
*** gildub has joined #openstack-neutron08:16
jschwarzihrachyshka, ping08:16
mkolesnijschwarz: well was something created in the db?08:16
ihrachyshkamkolesni, do we have that in review? I mean, qos_policy_id08:16
ihrachyshkajschwarz, shoot08:16
jschwarzihrachyshka, so i'm invoking create_policy to.. well... create a policy. the function succeeds (to_dict() returns an actual object) but the object doesn't exist in the DB08:16
jschwarzmkolesni, ^08:16
ihrachyshkajschwarz, is .create() called for the policy object?08:17
mkolesnijschwarz: ah so its created but not actually persisted?08:17
jschwarzihrachyshka, yes08:17
*** markvoelker has joined #openstack-neutron08:17
jschwarzmkolesni, yes08:17
mkolesnijschwarz: dunno i didnt touch that at all08:17
*** topshare has quit IRC08:17
*** enikanorov has quit IRC08:17
ihrachyshkajschwarz, that is weird, I have unit tests for that. is it more or less latest neutron?08:18
jschwarzihrachyshka, I'm running the latest patch on Mike's chain08:18
jschwarzihrachyshka, https://review.openstack.org/#/c/202425/08:18
*** pal has joined #openstack-neutron08:18
mkolesnijschwarz: u can run even the qos plugin patch for that, it doesnt matter08:18
jschwarzihrachyshka, let me run those unit tests... sec08:18
ihrachyshkajschwarz, QosPolicyDbObjectTestCase08:19
*** topshare has joined #openstack-neutron08:19
*** shivrao has quit IRC08:19
jschwarzihrachyshka, qlalchemy.exc.OperationalError: (OperationalError) no such table: networks u'INSERT INTO networks (tenant_id, id, name, status, admin_state_up, shared, mtu, vlan_transparent) VALUES (?, ?, ?, ?, ?, ?, ?, ?)' (None, '806e00ba-7ce0-4f6b-adaa-1907adead278', 'test-network1', None, None, None, None, None)08:20
jschwarzihrachyshka, did you add a new migration script when I was away?08:20
*** Kiall_ has joined #openstack-neutron08:20
*** Kiall has quit IRC08:20
ajomkolesni: https://review.openstack.org/#/c/202425/1/neutron/plugins/ml2/extensions/qos.py I'm not sure if you can get "context" from inside "session"08:21
ajohmmm08:21
ajoprobably not08:21
ajosession is the sqlalchemy database session?, then not...08:21
ihrachyshkajschwarz, we modified models and all08:21
jschwarzihrachyshka, :<08:21
jschwarzihrachyshka, I'm running the db manage stuff but it doesn't update anything08:22
ihrachyshkabecause we modified in place08:22
ihrachyshkanot with a new migration08:22
jschwarzuh08:22
jschwarzso how do I fix that? :)08:22
ajomkolesni, I think it can be good as a start,08:22
*** rossella_s has quit IRC08:22
ihrachyshkaI don't think it's relevant to your db state actually. tests should run independently.08:22
*** garyk has joined #openstack-neutron08:23
ajomkolesni, move out from the automatic callback based extension08:23
ihrachyshkajschwarz, it looks like your schema is not bootstrapped by the test. not sure why.08:23
ajoand then we can look back for ways to register those kind of extensions automatically08:23
ihrachyshkait's done in SqlTestCase, you can trace08:23
ajoso plugins don't need to call on the specific methods08:23
garykajo: ihrachyshka: gsagie: irenab: quick question - do we support qos-marking? or is that planned in the next version?08:23
*** topshare has quit IRC08:23
ajogaryk, planned for next version08:24
mkolesniajo: sure08:24
garykajo: thanks!08:24
jschwarzihrachyshka, unit tests aside, the actual server doesn't save the policies I create08:24
ajogaryk, I think first thing is DSCP, vhoward from comcast was looking at this already08:24
jschwarzihrachyshka, actually hold on, let me check something08:24
ajogaryk, I think he prepared an RFE/spec or something (or was on it)08:24
*** markvoelker has quit IRC08:25
irenabgaryk: next iteration08:25
ajogaryk: https://review.openstack.org/#/c/190285/08:25
jschwarzihrachyshka, mkolesni, nevermind, I put in some code to delete the resource post-test and didn't expect it to actually work <.<08:25
jschwarzso false alarm08:25
garyktx08:25
ajoirenab, have you seen mikes patch?08:25
jschwarzajo, irenab, did you guys change the API specs re: bandwidth_rule creation?08:26
mkolesnijschwarz: http://cdndata.bigfooty.com/2014/05/57646_14d1c405d9b9fa1de2189aa5660a101a.jpg08:26
irenabgaryk: the vendor can add new rule types easily even if it not supported  upstresm08:26
jschwarzirenab, serverside is throwing me a "TypeError: create_policy_bandwidth_limit_rule() got an unexpected keyword argument 'bandwidth_limit_rule'" error upon creating a rule08:27
ihrachyshkajschwarz, gooooood08:27
jschwarzlol mkolesni08:27
mkolesnijschwarz: so can u please try create a policy and attach it to port/network and see if that works?08:27
ajomkolesni: QosExtensionHandler, I wonder if it makes sense to move all the "update_%s_policy"  to the extension itself08:27
*** topshare has joined #openstack-neutron08:27
mkolesniajo: what u mean, out of the plugin?08:27
jschwarzmkolesni, erm... I can try and get to that later down the line, once the code I do have in place works08:27
ihrachyshkaajo, see jschwarz's error for kwarg.08:27
ajoirenab: they need the datamodels, not sure08:27
mkolesniajo: i e qos_plugin?08:27
* jschwarz wasn't supposed to be working today :>08:27
irenabjschwarz: I think maybe method parameters renaming got in08:27
jschwarzhaha!08:28
jschwarzi'll look at the spec08:28
ajoirenab: the idea is to force upstream discussion for new rules, and get consensus, otherwise it'd be a mess08:28
irenabajo: agree for long term, but for experimenting this should be easy to add08:28
mkolesnijschwarz: ah i thought u have a test for that08:28
jschwarzirenab, looking at https://review.openstack.org/#/c/199112/6/specs/liberty/qos-api-extension.rst - doesn't seem to be any change there08:28
ihrachyshkairenab, yeah, it seems it could actually break something. we'll handle it one by one.08:28
jschwarzmkolesni, I don't08:28
ajomkolesni, from qos_plugin to... the extension itself, so totally decoupling the extension logic from the plugin08:28
*** redbeard has joined #openstack-neutron08:29
ajoirenab: true08:29
garykirenab: thanks.08:29
*** markvoelker has joined #openstack-neutron08:29
irenabjschwarz: can you try to change parameters of the methods to previous full name as in API?08:29
mkolesniajo: hmm but then the extension handler needs to be aware of the versioned objects08:29
garykmy concern for the first version is the base api that some vendores support ingrees and egrees rules. we decide to address this in the second version.08:29
jschwarzirenab, I can. what are the parameters supposed to be named?08:30
irenabjschwarz: just the same before renaming, let me check08:30
*** evgenyf has quit IRC08:30
irenabjschwarz: https://review.openstack.org/#/c/197564/17/neutron/extensions/qos.py08:31
irenabplease check line 228 change08:31
irenabbandwidth_limit_rule08:32
*** boris-42 has quit IRC08:32
ajomkolesni, yes, that's a drawback, but it should be ok, we lighten the service plugin,08:33
*** iwamoto has quit IRC08:33
ajoyou remove one dependency (the plugin) and add another,08:33
ajodependency count ==08:33
*** markvoelker has quit IRC08:33
mkolesniajo: ok i'll see if it's possible to do quickly08:35
mkolesniajo: do u have capacity to quickly test it on a running env?08:35
mkolesniajo: i dont have a ready env right now..08:35
jschwarzirenab, looking08:35
*** gildub is now known as gildub_bbl08:35
jschwarzirenab, that fixed it08:37
jschwarzihrachyshka, now we have: OperationalError: (OperationalError) (1054, "Unknown column 'id' in 'field list'") 'INSERT INTO qos_bandwidth_limit_rules (max_kbps, max_burst_kbps, id) VALUES (%s, %s, %s)' (100, 1000, 'a4a0dab1-e8df-4c37-a3f9-417e3c9cfff0')08:37
jschwarzihrachyshka, seems like something I definitely need to fix before going further08:37
ajojschwarz, did you run the db migrations?08:38
*** alex_klimov has quit IRC08:38
jschwarzajo, yeah, it was a no-op because changes were made in-place08:38
ihrachyshkayeah, qos_rule_id was renamed into id for bandwidth rule model08:38
ajoyes08:38
ihrachyshkajschwarz, so drop db, and recreate it08:38
ajobut you need to reset the DB08:38
ajocorrect08:38
openstackgerritAnn Kamyshnikova proposed openstack/neutron: Graceful ovs-agent restart  https://review.openstack.org/18292008:39
ajojschwarz, that's because we modified the migration in place instead of creating a new one08:39
ajomkolesni, I can try, but I'd like to focus on getting the db deadlocks out08:39
ajoand get the basic service plugin merged08:39
ajothen we can refine the extension approach08:40
jschwarzajo, ihrachyshka, aye08:40
irenabjschwarz: I tried to keep parameters unchanged :-)08:40
mkolesniajo: if we dont use read callbacks it might lower the count?08:40
jschwarzihrachyshka, how do I easily re-create the table once dropped?08:40
jschwarzirenab, :)08:40
ihrachyshkaajo, yes, plugin is top08:40
jschwarzmkolesni, I have a change you need to make in one of your patches08:40
irenabihrachyshka:  so lets not merge this patch without reverting it08:40
ihrachyshkajschwarz, create database neutron; and then neutron-db-manage upgrade head08:40
mkolesniajo: anyway if it's not in the qos_plugin then the extension will work even if the service plugin is not enabled08:40
mkolesniajo: is that desirable?08:40
jschwarzihrachyshka, ack, thanks08:40
mkolesnijschwarz: what do u need, sir?08:41
ihrachyshkairenab, not sure which patch you mean08:41
irenabjschwarz: I away for 1 hour for a meeting08:41
ajomkolesni, it's going to be the same thing I think, since we call for port extension manually within ml208:41
*** gongysh has quit IRC08:41
jschwarzirenab, ack08:41
ajomkolesni, hmm08:41
ajothat's a good question08:41
mkolesniajo: only if the qos ext driver is loaded08:41
irenabihrachyshka: https://review.openstack.org/#/c/197564/17/neutron/extensions/qos.py line 22808:41
mkolesniajo: and its not right now so should affect the tests08:41
ajomkolesni, I would check that the plugin is installed and otherwise don't let any call pass thru08:42
ajobut I think having the logic separate is beneficial...08:42
ajogood point08:42
mkolesniajo: so u think still keep the logic there, but on each call check if the service plugin is active?08:42
ihrachyshkairenab, yeah. we probably should do it. ajo: we'll revert arg names before merge, ok?08:42
ihrachyshkaajo, but first, we need to get into that deadlock08:42
ajomkolesni, I'd check and cache it08:43
ajomkolesni, the plugin can't go inactive once active08:43
ajomkolesni, when having an extension manager, we can delegate that to it...08:43
mkolesniajo: yes of course08:43
mkolesniajo: i was asking to make sure if thats the intended logic08:43
ajo"ResourceExtensionManager" ...08:43
jschwarzmkolesni, made you a diff: http://pastebin.com/HH7jYQ5Y08:43
jschwarzmkolesni, :)08:43
*** markvoelker has joined #openstack-neutron08:44
ajomkolesni, I can deploy yo a devstack on my tiny cloud08:44
ajoif that helps you08:44
mkolesnijschwarz: you are a friend and a gentleman08:44
mkolesnijschwarz: i think the consensus was not to touch that patch anymore, but i might be wrong08:44
mkolesnijschwarz: if it;s indeed the case this needs to be a follow-up patch08:44
mkolesniajo: a devstack supporting qos?08:45
mkolesni:)08:45
*** rdo has joined #openstack-neutron08:45
ajomkolesni, if you have a devstack, you just have to service_plugins += qos08:45
jschwarzmkolesni, ack. I'll fix it locally on the QoS API patch but it will need to be removed afterwards08:45
ajoand run the db migrations08:46
*** salv-orlando has joined #openstack-neutron08:46
mkolesniajo: ill try to focus on the code08:47
openstackgerritJakub Libosvar proposed openstack/neutron: Change prefix for namespace fixture  https://review.openstack.org/20243908:47
*** markvoelker has quit IRC08:48
*** aix has joined #openstack-neutron08:48
ihrachyshkakevinbenton, ping re deadlocks08:50
kevinbentonihrachyshka: shoot08:50
ajoshoot08:50
ihrachyshkakevinbenton, http://logs.openstack.org/64/197564/17/check/gate-tempest-dsvm-neutron-full/1493b21/logs/screen-q-svc.txt.gz#_2015-07-16_07_37_50_91308:51
*** lukasa has left #openstack-neutron08:51
ihrachyshkathere, requests grabs the db-access lock08:51
* jschwarz hates re-stacking08:51
ihrachyshkaand then there is timeout, and release of the lock after it08:51
jlibosvaajo: do you have links to neutronclient qos patches?08:51
gsagieif anyone can please review https://review.openstack.org/#/c/198983/  , will be great. trying to clear reviews :)08:51
ihrachyshkathe last I see from tre request is "Ignoring state change"08:51
ihrachyshkakevinbenton, question: I see patch that removes the semaphore in master. will it help?08:52
ajojlibosva, look in the neutronclient repo, there are just a few open patches08:52
ajojlibosva 1st one is ok for policy create/remove association/disociation08:52
ihrachyshkait's also not clear for me why we still use MySQLdb. wasn't that a plan to switch to pymysql?08:52
jlibosvaajo: ok, I thought it's neutronclient and it's python-neutronclient, that's why I couldn't find anything :-/08:52
ajoahh08:53
kevinbentonihrachyshka: yes, the switch was dependent on getting us to handle deadlocks correctly08:53
ajo ok :)08:53
kevinbentonihrachyshka: i need to check with armax, but i think we were ready to switch to pymysql now08:53
ihrachyshkakevinbenton, aha. but we are not there yet. ok. so speaking of that patch, will it make it for us?08:53
*** park_ has quit IRC08:56
kevinbentonihrachyshka: looking now08:57
*** markvoelker has joined #openstack-neutron08:58
*** evgenyf has joined #openstack-neutron08:58
ihrachyshkakevinbenton, note that in the patch that fails that way, we subscribe to AFTER_READ events for ports. I suspect it's the culprit, though I don't see any potential yielding in extension function we pass for the event09:00
kevinbentonihrachyshka: i missed a spot with the decorator09:02
kevinbentonihrachyshka: working on a patch now09:02
*** doude has joined #openstack-neutron09:03
*** doude has quit IRC09:03
*** doude has joined #openstack-neutron09:03
ihrachyshkakevinbenton, great. you're awesome. :) keep us in the loop. anything we can patch locally for now to unblock our branch?09:03
*** killer_prince has joined #openstack-neutron09:03
*** markvoelker has quit IRC09:03
kevinbentonihrachyshka: yeah, just let me confirm this really quick and i'll push the patch up and you can cherry-pick it in09:04
ihrachyshkawoot. ajo^^09:04
*** yatin has quit IRC09:06
*** yonglihe has quit IRC09:06
ihrachyshkajschwarz, can you paste in https://review.openstack.org/#/c/197564/ what we need to fix there before merge? we'll need to respin09:06
*** lazy_prince has quit IRC09:06
*** alex_klimov has joined #openstack-neutron09:06
jschwarzsure09:07
*** hichihara has quit IRC09:07
*** lazy_prince has joined #openstack-neutron09:07
jschwarzihrachyshka, done09:07
*** killer_prince has quit IRC09:08
changzhiajo: hi, please review https://review.openstack.org/#/c/199588/ if necessary :-)09:08
openstackgerritKevin Benton proposed openstack/neutron: Add oslo db retry decorator to non-CRUD actions  https://review.openstack.org/20244909:11
kevinbentonihrachyshka: ^^09:12
ajokevinbenton++09:12
ajoI will try pulling that in instead09:12
ihrachyshkaajo, I will handle that09:13
ihrachyshkaajo, I already started updating service plugin patch09:13
ajoihrachyshka, what are you changing?09:13
ihrachyshkaajo, just arg names, as per jschwarz's request09:13
ajoahh09:13
ihrachyshkacontrollers pass fields as kwargs09:13
openstackgerritKevin Benton proposed openstack/neutron: Add oslo db retry decorator to non-CRUD actions  https://review.openstack.org/20244909:14
ajook, ack, thanks09:14
ihrachyshkaand I will base it on a cherry-pick of ^^09:14
kevinbentonihrachyshka, ajo: make sure you get that second version09:14
ihrachyshkakevinbenton, ack09:14
ajoihrachyshka, let me do the cherry pick, and provide you the link09:14
ajounless you already finished the changes09:14
ihrachyshkaajo, I will handle it. find something other to tackle ;)09:14
ajooook :)09:15
ajoihrachyshka, I will update devref doc then09:15
*** yfried is now known as yfried|afk09:15
ihrachyshkaright!09:15
ihrachyshkathere was a proof reader for that ;)09:15
ihrachyshkaI think we should push that after respin (I mean devref)09:15
ihrachyshkathen we'll grow it from there09:15
ajoihrachyshka, we broke neutron-dsvm-full with devref, can you believe it? ;)09:15
ajogate it's quite unstable :(09:15
ajoihrachyshka I agree09:16
ihrachyshkakevinbenton, do we also need db-access semaphor killer in feature/qos? or that one with decorator is enough for our needs?09:16
ajoihrachyshka: I think the decorator should be ok09:16
ajoihrachyshka removing the db-acesss lock in fact may make it more likely to happen , I guess09:16
ajokevinbenton, am I right?09:16
ajohappen, and then be retried..09:17
kevinbentonihrachyshka, ajo: yeah, i don't think you need to remove it09:17
kevinbentonit's just a performance cost09:17
*** numan has joined #openstack-neutron09:17
kevinbentonand does reduce the possibility of deadlocks09:17
ajoyeah, caffeine wasn't working still when I thought of that09:17
ajostill working09:17
ihrachyshkaack, thanks a lot09:18
*** alex_xu_ is now known as alex_xu09:27
mkolesniajo: quick question09:28
ihrachyshkaajo, I've rebased plugin09:29
mkolesniajo: manager.NeutronManager.get_service_plugins()[plugin_constants.QOS] will throw a value error if the plugin isnt loaded?09:29
mkolesniajo: i mean key error09:29
ajoihrachyshka: ++ /me crosses fingers09:29
ajomkolesni, true09:30
mkolesniajo: in that case shouldn't it be .get(plugin_constants.QOS) instead?09:30
ajoyou may just want to look for the key09:30
ajoto see if the plugin is in09:30
ajoor get, yes09:30
ajothat's ok too09:30
ihrachyshka.get09:30
mkolesniok..09:30
ajoif you need the plugin .get09:30
ajothanks mkolesni++09:31
*** rubasov has joined #openstack-neutron09:31
mkolesniajo: oki09:32
*** yog__ has quit IRC09:32
ajohttps://c2.staticflickr.com/4/3454/5711805975_1f24e24419_b.jpg09:34
ajothe gate09:34
*** yatin has joined #openstack-neutron09:34
*** yfried|afk is now known as yfried09:35
*** lazy_prince has quit IRC09:39
*** x-ray has joined #openstack-neutron09:40
*** salv-orlando has quit IRC09:40
*** ArchiFleKs has joined #openstack-neutron09:41
openstackgerritvikram.choudhary proposed openstack/neutron: [WIP] BGP Dynamic Routing: introduce entities and model  https://review.openstack.org/20162109:42
openstackgerritvikram.choudhary proposed openstack/neutron: [WIP] BGP Dynamic Routing: adding agent and scheduling  https://review.openstack.org/20246809:42
*** sudipto has quit IRC09:42
NightKhaosHi everyone, so I've managed to write up my problem in a more detailed ask openstack question: https://ask.openstack.org/en/question/78586/neutron-qrouters-not-passing-traffic/?09:43
*** ArchiFleKs has quit IRC09:44
*** ArchiFleKs has joined #openstack-neutron09:44
*** Bhavya has joined #openstack-neutron09:46
*** sudipto has joined #openstack-neutron09:48
jschwarzgah09:48
*** obondarev_ has joined #openstack-neutron09:50
*** nmagnezi has joined #openstack-neutron09:52
*** yog__ has joined #openstack-neutron09:56
*** obondarev_ has quit IRC09:56
jschwarzajo, ihrachyshka, mkolesni, running into more issues with the API tests (could be server side)09:56
openstackgerritZhaoBo proposed openstack/neutron: Fixed pass wrong cidr check during create subnet  https://review.openstack.org/20194209:56
jschwarzI'm thinking about letting it rest and coming back to it on Wednesday when I'll be back to work fulltime09:56
jschwarzwhen do we need the API tests to be ready for?09:57
ajoihrachyshka, I will check the current one you just updated manually09:58
vikramamotoki: ping09:58
*** aix has quit IRC09:58
ihrachyshkajschwarz, oh, you are back Wed? cool09:58
*** pal has quit IRC09:59
ajojschwarz, what kind of issues are you running into?09:59
ihrachyshkajschwarz, ideally, we need something right after we push service plugin, even if some operations are not there09:59
ihrachyshkathen you could grow on it09:59
jschwarzajo, have re-stacked and now post requests to v2.0/qos/policies are failing with '404 not found'09:59
*** evgenyf has quit IRC10:00
jschwarzit worked before the re-stack so i'm thinking something is broken if the DB is empty10:00
ajojschwarz, did you set the service_plugins in /etc/neutron/neutron.conf ?10:00
ajore-stacking may clear it10:00
jschwarzajo, that would make sense! sec10:00
jschwarzwhich service plugin do i need to set?10:01
ajoihrachyshka, is there any way to tell devstack to pull neutron from feature/qos in localrc conf?10:01
ajojschwarz: "qos"10:01
ajojust like that10:01
ajoadd it to the end of the list, with a comma in front10:01
jschwarzright so false alarm :P10:01
jschwarzstill getting more issues further down the line though, but I'll see about getting them fixed10:01
ihrachyshkaajo, NEUTRON_BRANCH10:02
ajoNEUTRON_BRANCH=feature/qos10:02
ajothat's it?10:02
ihrachyshkaright10:02
ajothanks10:02
ihrachyshka"should be"10:03
*** claudiub has joined #openstack-neutron10:03
*** zhenguo has quit IRC10:03
ihrachyshkagaryk, https://review.openstack.org/#/c/202461/ we need that cherry-pick for qos service to pass tempest-full10:05
ajomkolesni, how is it going there? do you want me to try anything manually?10:09
*** afazekas has joined #openstack-neutron10:10
*** evgenyf has joined #openstack-neutron10:11
jschwarzirenab, ajo, what do we want a test to do? just create a policy with a rule and associate it to a port?10:11
ajojschwarz, yes, also dissociation10:12
ajothen rule modification10:12
ajo(while associated to a port)10:12
irenabjschwarz: I started to add list of thest we need to support. For the basic one, just create, show and delete policy10:12
ajothat's working :)10:12
irenabajo: :-)10:12
*** lazy_prince has joined #openstack-neutron10:12
jschwarzirenab, I got something that creates and lists all policies10:12
ajoat least something ':D10:12
jschwarz(which works)10:12
ajojschwarz, also "show" :)10:13
ajoand modify (basically description)10:13
ajobut that's very low prio, I guess10:13
jschwarzajo, aye. also I don't think we ever went through the 'update' route so it might take some more time10:13
jschwarzI'll see about the association stuff10:13
ajojschwarz, ack, thanks man10:13
irenabjschwarz: lets try to pucsh small patches, I guess even simple test will be very useful. Will keep ajo more sain with all changes going around10:13
ajoirenab: +110:14
jschwarz^_^10:14
ajomaking a fisrt small one, will allow others to come after and extend testing other stuff10:14
jschwarzirenab, what's the APi address for port-policy association?10:14
openstackgerritRamanjaneya Reddy Palleti proposed openstack/python-neutronclient: Support CLI changes for QoS (2/2).  https://review.openstack.org/19827710:14
ajojschwarz: just port10:14
openstackgerritNuman Siddique proposed openstack/neutron: Support subnetpool association to an address scope  https://review.openstack.org/19755210:15
irenaband network10:15
ajojschwarz, you update the port with qos_policy_id in dictionary10:15
ajocorrect10:15
jschwarzoh I see that10:15
ajoalso network10:15
*** Apsu has quit IRC10:15
ajojschwarz, that depends on the patches mkolesni is currently working on10:15
jschwarzso I don't think that'll be much of a problem10:15
*** pc_m has joined #openstack-neutron10:15
ajo(net/port association/disociation)10:15
jschwarzsince the network/port create API calls are very robust so it shouldn't be hard to just add a key there10:15
*** pal has joined #openstack-neutron10:17
*** Apsu has joined #openstack-neutron10:17
*** garyk has quit IRC10:18
*** john-davidge has joined #openstack-neutron10:19
*** numan has quit IRC10:20
*** gampel has joined #openstack-neutron10:22
*** dims_ has joined #openstack-neutron10:24
jschwarzirenab, do you remember how did we fix an error such as "PolicyCheckError: Failed to check policy tenant_id:%(tenant_id)s because Unable to verify match:%(tenant_id)s as the parent resource: tenant was not found"?10:28
*** aix has joined #openstack-neutron10:28
*** nplanel has joined #openstack-neutron10:28
*** yfried is now known as yfried|afk10:29
*** yfried|afk is now known as yfried10:31
*** amotoki has joined #openstack-neutron10:32
*** afazekas has quit IRC10:35
*** yfried_ has joined #openstack-neutron10:36
* ajo looks at http://status.openstack.org//zuul/ filter: 19756410:37
ajoihrachyshka  ^  ;)10:37
ajocross your fingers10:37
*** yfried has quit IRC10:38
*** dims_ has quit IRC10:39
*** salv-orlando has joined #openstack-neutron10:40
*** changzhi has quit IRC10:40
mosheleajo, ihrachyshka: this patch  https://review.openstack.org/#/c/201975/  now pass Jenkins please review10:44
ajomoshele, thanks10:46
*** sudipto has quit IRC10:46
gsagieis the devref patch commited yet so i can edit it?10:47
*** hitalia has quit IRC10:47
*** afazekas has joined #openstack-neutron10:48
*** ArchiFleKs has quit IRC10:49
*** arosen has quit IRC10:50
ajogsagie, add a linked patch,10:50
ajoI believe we can merge in the current form.10:50
ajoand then iterate.10:50
*** jprovazn has joined #openstack-neutron10:50
*** obondarev__ has joined #openstack-neutron10:51
gsagieby linked you mean dependent on it?10:51
*** arosen has joined #openstack-neutron10:51
*** dsneddon_away has quit IRC10:51
*** stanzgy has quit IRC10:53
ajoihrachyshka: the plugin with cherry pick of the non-CRUD retry decorator is working! ;)10:53
*** Tiancheng has quit IRC10:53
ajogsagie: correct :) yest10:53
ajoyes10:53
*** garyk has joined #openstack-neutron10:54
irenabjschwarz: does it happen with noauth?10:54
*** obondarev__ has quit IRC10:55
jschwarzirenab, let me try10:56
ihrachyshkamoshele, I'm a bad boy, so left -1 and some comments10:56
ihrachyshkaajo, gooood10:56
irenabjschwarz: this may have similar problem resolved, I guessit matter of including tenant_id on rule resource10:57
jschwarzirenab, weird because I don't include it in any other request.10:57
irenabjschwarz: they maybe included on other resources10:57
irenabas policy10:57
openstackgerritfumihiko kakuma proposed openstack/neutron: Python 3: Wrap map() in a list call  https://review.openstack.org/20249710:58
*** coolsvap has quit IRC10:58
jschwarzirenab, can you clarify10:58
jschwarz?10:58
ajogsagie: I fixed it for merge, so you can safely iterate over devref without expected changes10:58
*** coolsvap has joined #openstack-neutron10:58
ajoI mean, in a follow up patch10:58
irenabjschwarz: it is defined in policy.json10:59
ihrachyshkaajo, +2/+A https://review.openstack.org/#/c/197564/ ?10:59
jschwarzwill have a looksie10:59
jschwarzlooksee?10:59
ajoihrachyshka, yeah, even if it's mine partly, it had positive votes before, and it's just a fix I didn't11:00
*** sudipto has joined #openstack-neutron11:02
*** yfried__ has joined #openstack-neutron11:02
ajolet's wish good luck to the 202461 recheck11:03
*** topshare has quit IRC11:03
*** mwagner_lap has quit IRC11:05
*** yfried_ has quit IRC11:06
*** yog__ has quit IRC11:07
openstackgerritshihanzhang proposed openstack/neutron: Install arp spoofing protection flow after setting port tag  https://review.openstack.org/19942511:08
mosheleihrachyshka: please see my comments11:09
* ajo goes for food11:10
mosheleihrachyshka: the resource_rpc is generic so it can be use for other things as well11:10
*** yfried has joined #openstack-neutron11:11
*** yfried__ has quit IRC11:12
openstackgerritshihanzhang proposed openstack/neutron: Check the validity of security group rule port range  https://review.openstack.org/20152411:14
ihrachyshkamoshele, sure, but it's not as generic that we should bind it to extension manager11:15
ihrachyshkamoshele, I replied11:15
yaliehi all,11:16
yaliewhen I am updaing a port like below, it failed11:16
yalieneutron port-update d9f6f33a-8527-47bf-92d3-c1f0db7ce245 --fixed-ips type=dict  {subnet_id=4d2c4e42-29ae-4b12-b8c0-cb1dce23fa84}11:16
yalieInvalid input for fixed_ips. Reason: Invalid data format for fixed IP: '{u'{subnet_id': u'4d2c4e42-29ae-4b12-b8c0-cb1dce23fa84}'}'.11:16
yaliebut what is the right format11:16
*** dsneddon_away has joined #openstack-neutron11:17
*** pal has quit IRC11:19
*** yog__ has joined #openstack-neutron11:20
jschwarzajo, ihrachyshka, pushing a new revision of the API test now - it has a policy/rule create and policy show (rule show a bit broken, irenab will look into it)11:20
ihrachyshkajschwarz, will we be able to get something mergeable, even if incomplete, this week?11:21
*** sambetts has quit IRC11:21
ihrachyshkaI think it would be nice to have at least some pieces to guard from regressions in tree, even if not for everything11:21
jschwarzihrachyshka, what's currently there works locally, but it depends on a bunch of mkolesni's patches11:22
*** gildub_bbl is now known as gildub11:22
jschwarzmaybe we can remove the dependency and then it's a matter of passing tests IMO11:22
ihrachyshkaok, got it11:22
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: ovs_lib: Fix native implementation of db_list  https://review.openstack.org/20250511:23
jschwarzihrachyshka, let me push it and we can discuss what can be done to speed it up11:23
ihrachyshkawe can try. ajo ^^ what do you think?11:23
ihrachyshkajschwarz, tnx11:23
jschwarzihrachyshka, ajo, irenab, https://review.openstack.org/#/c/197126/11:23
jschwarzmkolesni, ^11:24
*** veena has quit IRC11:26
openstackgerritRoman Bogorodskiy proposed openstack/neutron: [RFC] mech_sriov: vf cleanup on delete  https://review.openstack.org/20204411:26
openstackgerritVivekanandan Narasimhan proposed openstack/neutron: vm migration needs persistent state for l2pop  https://review.openstack.org/17538311:27
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: ovs_lib: Fix native implementation of db_list  https://review.openstack.org/20250511:29
*** abregman is now known as abregman|brb11:30
*** Kiall_ has quit IRC11:34
*** Kiall has joined #openstack-neutron11:34
*** nplanel has quit IRC11:36
*** john-davidge has quit IRC11:37
*** sambetts has joined #openstack-neutron11:37
*** john-davidge has joined #openstack-neutron11:38
*** x-ray has quit IRC11:38
*** gberginc has quit IRC11:42
wim_dcAnyone knows much about the extension mechanism in the neutron pythonclient? I know about neutronclient.common.extension  - But I'm mainly interested in extending already existing resources (like router) and have parameters added.11:43
openstackgerritZhaoBo proposed openstack/neutron: Fixed pass wrong cidr check during create subnet  https://review.openstack.org/20194211:43
*** devvesa has quit IRC11:45
*** thorst has joined #openstack-neutron11:47
*** jagter has quit IRC11:48
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: VPNaaS: Store local side's tunnel IP for later retrieval  https://review.openstack.org/19967011:50
*** trinaths has left #openstack-neutron11:51
*** topshare has joined #openstack-neutron11:52
*** topshare has quit IRC11:52
*** NightKhaos has left #openstack-neutron11:53
*** NightKhaos has joined #openstack-neutron11:53
*** Longgeek has quit IRC11:54
*** jagter has joined #openstack-neutron11:56
*** devvesa has joined #openstack-neutron11:56
*** amuller has joined #openstack-neutron11:58
*** lazy_prince has quit IRC11:59
*** lazy_prince has joined #openstack-neutron12:01
ihrachyshkajlibosva, ajo: I've moved to metaclass: https://review.openstack.org/#/c/200608/612:01
mosheleihrachyshka: this commit has a bug https://review.openstack.org/#/c/201544/12:01
mkolesniajo: ping12:01
moshelejlibosva: ^12:01
mosheleihrachyshka: it doen't create the instance of the class12:02
*** cing has quit IRC12:02
*** topshare has joined #openstack-neutron12:02
ihrachyshkamoshele, ah right. please fix.12:02
openstackgerritPaul Michali proposed openstack/neutron: DO NOT MERGE - test change  https://review.openstack.org/19614912:03
jschwarzajo, ihrachyshka, status report on the API test - currently we have a test that creates a policy and a rule, shows the policy (not the rule - bug there) and then deletes them. No updates, no associates.12:03
jschwarzready for your reviews :)12:03
jschwarzif the patch has to be re-spinned we (me and irenab) will probably add some other small things12:03
mosheleihrachyshka: can I do it in my commit or different one12:03
ihrachyshkamoshele, it hides it via mock :) we probably need a test to check it's actually a driver and not a class there12:04
*** amotoki has quit IRC12:04
ihrachyshkamoshele, if possible, separate it12:04
mosheleihrachyshka: ok will do12:04
ihrachyshkamoshele, thanks12:04
ajomkolesni, pong12:04
ajohow's it going?12:04
ihrachyshkajschwarz, on my way.12:04
jschwarzihrachyshka, excellent. will check back later today and see what's what12:05
*** Longgeek has joined #openstack-neutron12:05
mkolesniajo: good12:05
mkolesniajo: i pushed new version where moved the code inside the qos_extension12:05
jschwarzciao for now12:05
mkolesniajo: also u/t were added12:05
ajomkolesni++12:05
ajoniice12:05
mkolesniajo: i still want to add integ tests but won't have time for that until next week12:06
ajook, we can follow up with that12:06
mkolesniajo: hopefully the code works now, it was a bit broken (thank you unit tests :))12:06
ajoI will test it manually12:06
mkolesniok12:06
mkolesnii can't guarantee it really works, but at least it should flow normally12:06
*** prad has joined #openstack-neutron12:06
mkolesnidont forget to add qos in the ml2 config12:06
mkolesni'extension_drivers=qos' or something like that12:07
mkolesniajo: ^12:07
*** jschwarz has quit IRC12:07
ajoahh, ok, good to know12:07
ihrachyshkayeah, I will need to patch devstack-gate after it's in12:08
ajoihrachyshka ^ probably we will need a patch to the features/qos gate script when that's merged12:08
ajolol :)12:08
ajo1 sec diff12:08
ihrachyshkaI beat you!12:08
ihrachyshka:P12:08
ajoyou did12:08
ajoXD12:08
ajomkolesni, do you want to hangout to see it tested? :)12:10
openstackgerritshihanzhang proposed openstack/neutron: Use fanout RPC message to nofity the security group's change  https://review.openstack.org/18917112:10
openstackgerritshihanzhang proposed openstack/neutron: Destroy ipset when the corresponding rule is removed  https://review.openstack.org/19099112:10
mkolesniajo: sure y not12:11
ajomkolesni, gtalk id? :)12:11
openstackgerritMike Kolesnik proposed openstack/neutron: Update port bindings for master router  https://review.openstack.org/14111412:12
*** yfried has quit IRC12:13
*** dims_ has joined #openstack-neutron12:14
*** yatin has quit IRC12:20
jlibosvamoshele: oops12:22
jlibosvamoshele: I'll fix that12:22
*** redbeard has quit IRC12:22
jlibosvamoshele: ajo ihrachyshka https://review.openstack.org/#/c/202530/12:24
ihrachyshkajlibosva, we need a test for that12:24
jlibosvait should be covered by fullstack tests12:24
ihrachyshkathat does not mock the result of the function12:24
*** Shikha has joined #openstack-neutron12:24
ihrachyshkajlibosva, it's cool, but I'm not sure we'll get there before merge-back12:25
jlibosvahmm, that's right. The current UT doesn't correspond with what happens there12:25
ihrachyshkait should be easy to create the extension and check its qos_driver12:25
*** pradk has joined #openstack-neutron12:27
*** doude has quit IRC12:28
*** yatin has joined #openstack-neutron12:29
*** markvoelker has joined #openstack-neutron12:29
*** doude has joined #openstack-neutron12:29
*** dims_ has quit IRC12:30
*** salv-orlando has quit IRC12:31
*** mkolesni has quit IRC12:31
*** ArchiFleKs has joined #openstack-neutron12:31
*** dims__ has joined #openstack-neutron12:32
*** markvoelker has quit IRC12:34
openstackgerritElena Ezhova proposed openstack/neutron-fwaas: Fix wrong argument name in _rpc_update_firewall  https://review.openstack.org/20253312:35
mosheleihrachyshka: how can I rebase my patch on to https://review.openstack.org/#/c/202530/  (like ajo did yesterday) ?12:37
*** topshare has quit IRC12:38
ajomoshele, let me show you12:38
*** redbeard has joined #openstack-neutron12:38
*** sudipto has quit IRC12:39
ajomoshele, what's your patch?12:39
Muralipc_m: Hi12:39
mosheleajo: https://review.openstack.org/#/c/201975/12:39
pc_mMurali: hi12:39
*** dims__ has quit IRC12:39
ajook, moshele , you go to the "check out cmdline part"12:39
openstackgerritNir Magnezi proposed openstack/neutron: Adds garp_master_repeat and garp_master_refresh to keepalived.conf  https://review.openstack.org/20112812:39
openstackgerritRamanjaneya Reddy Palleti proposed openstack/python-neutronclient: Support QoS neutron-client (1/2).  https://review.openstack.org/18965512:39
Muralipc_m: How are you?12:39
scheuranmestery: Kyle, regarding the discussion if the macvtap support should land in it's own repo or somewhere else: can I put this on next weeks neutron meeting agenda? Is that the right place to get things forward?12:39
openstackgerritIhar Hrachyshka proposed openstack/neutron: [devref] db_layer: expand on how new migration scripts look like  https://review.openstack.org/20253412:39
scheuranmestery: refering to this one: https://review.openstack.org/19590712:39
pc_mMurali: good!12:40
Muralipc_m:  I seeing some performance issue12:40
*** park_ has joined #openstack-neutron12:40
Muralicould you give me your sight in there12:40
*** park__ has joined #openstack-neutron12:40
ihrachyshkamoshele, check out the patch (git review -d 202530), then cherry-pick your patch on top, then git review -R12:41
ajomoshele, you will see: git fetch https://review.openstack.org/openstack/neutron refs/changes/75/201975/4 && git checkout FETCH_HEAD12:41
*** park_ has quit IRC12:41
ajoyou just do it, to be on your patch12:41
MuraliActaully I accessing metedata inside the instance using "http://169.254.169.254/openstack/2012-08-10/meta_data.json"12:41
ajonow12:41
ihrachyshkaajo, it's git review -d XXX12:41
*** park__ has quit IRC12:41
pc_mMurali: I can try. What is the performance issue?12:41
ajoihrachyshka, crap, git review does it automatically?12:41
*** park_ has joined #openstack-neutron12:41
ajoahh12:41
ajoto download12:41
*** park_ has quit IRC12:42
*** park_heijlong_ has joined #openstack-neutron12:42
ajook moshele , you first download your patch, the way you prefer12:42
ajothen12:42
Muraliand I am launching the instances one by one after the 80 instances creating I saw error while accessing the metadata12:42
ajomoshele: you go to the patch you want to rebase on12:42
ajoand get the same checkout string, but remove the checkout part, leave only the fetch12:43
openstackgerritKyle Mestery proposed openstack/neutron: dhcp: Default to using local DNS resolution  https://review.openstack.org/19271212:43
ajomoshele, in this case:12:43
*** markvoelker has joined #openstack-neutron12:43
ajomoshele: git fetch https://review.openstack.org/openstack/neutron refs/changes/30/202530/212:43
ajomoshele, then:  git rebase HEAD~1 --onto refs/changes/30/202530/212:43
*** merooney has joined #openstack-neutron12:43
Muralipc_m: this is the error I am seeing http://pastebin.com/FUHg1Dv012:44
Muraliinside the instance12:44
ajomoshele: if any conflicts... fix conflicts, add files, rebase --continue (luckily you can skip that part)12:44
*** krtaylor has quit IRC12:44
mosheleajo: thanks12:44
ajomoshele: and then  git review12:44
ajocheck that the commit ID of the patch you rebased on, didn't change12:44
*** topshare has joined #openstack-neutron12:44
ajootherwise you may be screwing up the other patch..12:44
*** topshare has quit IRC12:45
ajomoshele, let me know if anything goes wrong ;)12:45
mosheleajo: sure12:45
*** topshare has joined #openstack-neutron12:45
*** topshare has quit IRC12:45
pc_mMurali: Afraid I don't know much at all about issues with metadata access for instances (was thinking you may have a VPN question).12:46
*** topshare has joined #openstack-neutron12:46
*** topshare has quit IRC12:46
Muralipc_m : no probs whom I can contact on metadata agent related issues12:47
*** topshare has joined #openstack-neutron12:47
*** markvoelker has quit IRC12:48
pc_mMurali: not sure... see if someone on this channel responds?12:48
openstackgerrityalei wang proposed openstack/python-neutronclient: Extend the way to update the fixed-ips by cmdline  https://review.openstack.org/20253812:48
*** chem has quit IRC12:48
*** chem has joined #openstack-neutron12:49
pc_mMurali: One wait to get some idea is to do 'git blame' on one of the source files in that area, and look for people doing recent changes.12:50
*** yatin has quit IRC12:51
Muralipc_m: ok let me check thanks for response12:51
*** krtaylor has joined #openstack-neutron12:51
pc_mMurali: Looking at the agent. I see obondarev as one person from last year.12:52
pc_mobondarev: Do you know why Murali can contact about metadata agent issues?12:52
*** ArchiFleKs has quit IRC12:53
pc_mMurali: You may have to try later today, for some people to see if they are on.12:53
*** salv-orlando has joined #openstack-neutron12:53
Muralipc_m: sure thank you12:53
ajoirenab, ihrachyshka12:54
pc_mamuller: Hi. Do you know who may be knowledgable about metadata agent. Murali is looking for some help.12:54
irenabajo: yep12:54
ajowe have an issue with using ml2 extensions to handle qos "port/net" extension.12:54
ajothis: ValueError: Multiple plugins for service QOS were configured12:54
ajonow ml2 registers to provide "qos" and so does our pluggin12:54
*** yamamoto has quit IRC12:54
ajoplugin12:54
*** yfried has joined #openstack-neutron12:54
ajowhich, makes sense12:55
irenabajo: why ml2 reqiesters?12:55
ajosince an ml2 extension also needs to provide show-port, etc...12:55
pc_mMurali: you may want to try later in the day. There are several people that have modified the agent.py file, so some of them may know who to contact.12:55
ajoyou declare the api extension in one end12:55
ajoand the ml2 extension in the other,12:55
ajothat's all12:55
amullerpc_m: I'm OK with the metadata agent, jlibosva as well12:56
ajosince we don't want to be just an ml2 extension, we cannot use ml2 extensions as-is12:56
irenabajo: can ml2 extension leave extension empty?12:56
ajoirenab, let me try what happens if we do that12:56
amullerMurali: You had a Q about the metadata agent?12:57
irenabajo: another creazy idea, we can add MD to do the same as ML2 extension. ML2 QoS Proxy MD…12:57
pc_mMurali: amuller may be able to help.12:57
*** markvoelker has joined #openstack-neutron12:57
Muraliamuller: yes12:57
irenabajo: this won’t require extension declaration12:57
ajoirenab, MD?12:58
irenabMech. Driver12:58
*** thorst has quit IRC12:58
*** jbell8 has joined #openstack-neutron12:58
irenabLike L2 population12:58
ajoirenab, but that won't work for extending the resource12:58
ajoI don't follow12:58
ajobtw: ExtensionsNotFound: Extensions not found: [None]12:58
*** yalie has left #openstack-neutron12:59
ajoit will try to look for the extension, to declare himself12:59
MuraliAmuller: I am aceessing the metadata inside instance like http://169.254.169.254/openstack/2012-08-10/meta_data.json12:59
irenabajo: assumed so…12:59
ajolet me check if I can patch it to not look for the extension when "None"12:59
irenabso just try to change Extension into Mech Driver what implements the similar methods12:59
ajoirenab, let's try to do it with what we have, or even patch it to the port_update13:00
Muraliamuller: and I am trying to create bulk instances one by one and I saw that metadata is received by some 50 instances. other instances after 50 are failed to get the metadata and got error http://pastebin.com/FUHg1Dv013:00
ajoI believe we don't have more time to try different approaches, we can refine that later13:00
ajobefore merging back to master13:01
ajobut we need to make the fullstack working properly13:01
*** zhenguo has joined #openstack-neutron13:01
MuraliAmuller: are you there?13:01
amullerMurali: yes13:01
amullerMurali: any errors in metadata agent or proxy?13:01
irenabajo: this should be not difficult , following similar as here https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/opendaylight/driver.py13:01
ajoI will look at mech drivers if this fails, but I don't get how does it fit us13:01
irenabbut only precommit methods for port/network13:01
ajoahh13:01
*** markvoelker has quit IRC13:02
MuraliAmuller: I didnt see any error in metadataagent.log13:02
ajoirenab, so we'd keep the AFTER_READ callback,13:02
ajoto extend the port13:02
ajoand, then the mech driver to handle updates13:02
*** mwagner_lap has joined #openstack-neutron13:02
ajothough the class mike declared?13:02
MuraliAmuller: let me check in proxy13:02
irenabajo: sounds like a plan13:02
*** jbell8 has quit IRC13:03
ajoirenab, thanks for the more detailed pointer ;)13:03
MuraliAmuller: I am many proxy logs do need to check with my dhcp related ID proxy log?13:03
irenabajo: this double extension error, is exactly what I tried to say yesterday during the sync13:03
MuraliAmuller: I have many proxy logs13:03
*** gildub has quit IRC13:04
ajoirenab, hmm, I didn't get that,13:04
ajoirenab, sorry13:04
irenabwe may have to split qos extension in two parts to make it work. ML2 extension for qos_assosiation, and qos for QoS Plugin manage policies/rules….13:04
*** kbringard has joined #openstack-neutron13:04
ajoirenab, I will try to patch the ml2 extensions driver manager to not register for an extension when it's None13:04
*** thorst has joined #openstack-neutron13:04
ajoirenab, let's try to avoid that13:04
irenabajo: agree, lets try the hack with not registering13:05
*** pal has joined #openstack-neutron13:05
ajoirenab, I don't think it's a hack, just another use case :)13:05
ajowe're extending core resources, but, we don't want to extend the api13:05
*** markvoelker has joined #openstack-neutron13:05
irenabajo: thank you for trying to convince me its not a hack, its feature :-)13:06
ajoihrachyshka sometimes says I could be a politician ;)13:06
*** john-davidge has quit IRC13:06
irenabajo: lets try your suggestion, should work13:06
amullerMurali: also you can play with metadata_workers in metadata_agent.ini, see if that has an effect13:06
*** markvoelker_ has joined #openstack-neutron13:06
MuraliAmuller: I am seeing any error proxy log for my dhcp namespace13:07
ajoirenab: http://paste.fedoraproject.org/244993/7052078113:08
ajoit seems logical :)13:08
*** pal_ has joined #openstack-neutron13:08
irenabajo: and elegant :-)13:08
*** itzikb_ has quit IRC13:09
MuraliAmuller: I am running network node on controller itself.13:09
ajouhh, it's even showing the qos_profile_id for a port_show13:09
MuraliAmuller: I made metadata_workers=413:09
*** pal has quit IRC13:10
*** markvoelker has quit IRC13:10
ajoalmost working, irenab , attach failed, but... debugging ;)13:10
amullerMurali: you can bump it to 8, see if it makes a difference13:10
MuraliAmuller: Let me try13:10
*** alegacy has joined #openstack-neutron13:11
*** coolsvap is now known as coolsvap|away13:13
openstackgerritSam Betts proposed openstack/neutron: Fix order of calls in update_port  https://review.openstack.org/20254913:14
sambettsCan I get some eyes on this ^ its causing UTs to fail in networking-cisco and blocking our merges13:15
sambetts:)13:16
*** abregman|brb is now known as abregman13:16
sambettsHenryG: ^^13:16
*** dims_ has joined #openstack-neutron13:16
openstackgerritPavel Bondar proposed openstack/neutron: Add Pluggable IPAM Backend Part 2  https://review.openstack.org/18309913:17
*** Tiancheng has joined #openstack-neutron13:18
*** vivek-ebay has joined #openstack-neutron13:23
*** zzzeek has joined #openstack-neutron13:24
*** eckelcu has joined #openstack-neutron13:24
openstackgerritvikram.choudhary proposed openstack/neutron: [WIP] BGP Dynamic Routing: introduce entities and model  https://review.openstack.org/20162113:24
*** crose has joined #openstack-neutron13:25
*** sdake has joined #openstack-neutron13:25
*** dane_leblanc has joined #openstack-neutron13:26
*** rossella_s has joined #openstack-neutron13:27
*** topshare has quit IRC13:27
*** edmondsw has joined #openstack-neutron13:28
*** rpothier has joined #openstack-neutron13:28
*** eckelcu has quit IRC13:28
*** eckelcu has joined #openstack-neutron13:29
*** Bhavya has quit IRC13:30
MuraliAmuller: metadata_workers=8 not much helping me13:33
*** topshare has joined #openstack-neutron13:33
amullerMurali: you see 8 workers in ps output?13:33
MuraliAmuller: Yes see 8 workers13:34
amullerMurali: cut off is still at 50?13:34
MuraliAmuller: Yes I see after 55 instances I see the problem13:35
*** jaypipes has joined #openstack-neutron13:35
MuraliAmuller: meta-proxy namespace not getting the request from the instances13:35
MuraliAmuller: I saw many connections not closed in controller http://pastebin.com/QmUvY0LQ13:39
openstackgerritvikram.choudhary proposed openstack/neutron: [WIP] BGP Dynamic Routing: adding agent and scheduling  https://review.openstack.org/20246813:39
*** eckelcu has quit IRC13:40
*** ronis has joined #openstack-neutron13:41
*** prad has quit IRC13:41
jlibosvapc_m: amuller what, where? I was afk and I miss context13:41
jlibosvaok, gotcha13:42
*** arajagopal has joined #openstack-neutron13:42
*** prad has joined #openstack-neutron13:42
*** ferest has joined #openstack-neutron13:44
*** yamamoto has joined #openstack-neutron13:46
openstackgerritBerezovsky Irena proposed openstack/neutron: Add documentation for SRIOV NIC agent (previously missing)  https://review.openstack.org/20256913:46
*** ferest has quit IRC13:47
*** vivek-eb_ has joined #openstack-neutron13:48
Sam-I-Amhmm, y'all should be adding docs to the networking guide13:48
Sam-I-Amimo :)13:48
*** jckasper has quit IRC13:48
*** irenab has quit IRC13:49
*** vivek-ebay has quit IRC13:50
*** yamamoto has quit IRC13:51
amullerMurali: I might have misunderstood you, if you spawn 50 instances it works. If you wait for them to finish booting, then 10 more, those fail?13:52
*** pradk has quit IRC13:52
amullerMurali: even if you wait a while?13:52
*** gsagie has quit IRC13:53
ihrachyshkaajo, https://review.openstack.org/#/c/20253013:53
MuraliAmuller: After 50 came-up I tried launch some more which are failing13:53
MuraliAmuller: I waited for some time in b/n13:54
amullerMurali: ok that's definitely unusual / not normal13:54
amullerMurali: never heard of this13:54
*** dboik has joined #openstack-neutron13:55
*** dboik has quit IRC13:55
*** dboik has joined #openstack-neutron13:55
MuraliAmuller: after 50  randomly 10 of 1 will get the metadata13:56
Muralinot all13:56
amullerjlibosva: ever heard of something like this? ^13:56
amullerMurali: you're thinking the problem is that the proxy is not getting the requests?13:56
*** signed8bit has joined #openstack-neutron13:57
MuraliAmuller: yes and I see "class 'socket.error'>: [Errno 113] No route to host)"13:57
jlibosvanope13:58
Muralihttp://pastebin.com/FUHg1Dv013:58
*** vikram has quit IRC13:58
jlibosvaMurali: do you use isolated networks or natting is done in router namespace?13:59
*** banix has joined #openstack-neutron13:59
*** redbeard has quit IRC13:59
*** topshare has quit IRC13:59
*** topshare has joined #openstack-neutron14:00
MuraliJlibosva: used isolated networks14:00
*** dboik_ has joined #openstack-neutron14:00
MuraliAll are launched in same network14:00
*** yamahata has joined #openstack-neutron14:01
jlibosvamancdaz: and can you see all requests in proxy log? The debug log messages start with "Request: "14:01
Muralihttp://pastebin.com/ScmnZGHt14:02
MuraliJlibosva: all my requests are in net114:02
jlibosvamancdaz: sorry, that was for Murali :)14:02
pc_mmestery: hi14:02
*** Kennan has quit IRC14:02
mesterypc_m: hello my friend14:02
*** alex_klimov has quit IRC14:03
*** Kennan has joined #openstack-neutron14:03
pc_mmestery: On https://review.openstack.org/#/c/200592/ it was approved, but I needed to rebase to pick up changes to tox.ini (for pep8). can you +A it again?14:03
*** dboik has quit IRC14:04
mesterypc_m: Done!14:04
pc_mmestery: Thanks!14:04
*** topshare has quit IRC14:04
pc_mmestery: Also, Nikolay's scenario test for VPN looks good to me. Could use another core review so this upstreams (https://review.openstack.org/#/c/159746/). Can you help?14:05
mesterypc_m: Sure, looking14:05
MuraliJlibosva:2015-07-16 19:28:15.293 26168 DEBUG neutron.agent.metadata.namespace_proxy [-] Request: GET /openstack/2012-08-10/meta_data.json HTTP/1.0^M14:05
openstackgerritMerged openstack/neutron: Python 3: do not use types.ClassType  https://review.openstack.org/20055314:05
openstackgerritMerged openstack/neutron: Lower log level for extending network/subnet/port  https://review.openstack.org/20101314:05
MuraliJlibosva: I see the requests upto 5014:05
*** Tiancheng has quit IRC14:05
pc_mmestery: Appreciate the help!14:06
jlibosvaMurali: ok, can you look into the namespace where proxy is running for packets that were natted? 'ip netns exec <your_namespace> iptables -t nat -L -nv' - you may want to reset iptables before by 'ip netns exec <ns> iptables -t nat -Z'14:07
MuraliJlibosva: will check14:08
jlibosvaMurali: search for rule that translates 169.254.169.254:80 to :9796 port14:08
*** yamamoto has joined #openstack-neutron14:08
pc_mmestery: Thanks for approving!14:09
*** yamamoto_ has joined #openstack-neutron14:09
*** neophy has joined #openstack-neutron14:10
MuraliJlibosva: here  is my  http://pastebin.com/94hzPieN14:10
jlibosvaMurali: ok, so 76 were natted14:11
jlibosvaMurali: can you reset stats and test again? To make sure all the packets where actually redirected correctly14:11
jlibosvabut actually I'm not sure how many should we see :)14:11
MuraliJlibosva: why it was only 7614:12
MuraliI am sending many14:12
Muralilet me reset14:13
*** yamamoto has quit IRC14:13
*** mutoulbj has joined #openstack-neutron14:13
*** redbeard has joined #openstack-neutron14:14
*** alex_klimov has joined #openstack-neutron14:14
*** sudipto has joined #openstack-neutron14:15
*** claudiub has quit IRC14:15
*** alex_klimov has quit IRC14:15
*** fifieldt_ has quit IRC14:19
*** markvoelker_ has quit IRC14:24
*** Shikha has quit IRC14:25
*** lajos-katona has quit IRC14:26
*** alex_klimov has joined #openstack-neutron14:27
*** pal has joined #openstack-neutron14:29
*** mlavalle has joined #openstack-neutron14:30
*** annegentle has joined #openstack-neutron14:30
*** pal_ has quit IRC14:30
*** Shikha has joined #openstack-neutron14:31
*** jckasper has joined #openstack-neutron14:31
openstackgerritIsaku Yamahata proposed openstack/neutron: l3_db: not use l2 plugin _get_subnet unnecessarily  https://review.openstack.org/20235914:31
*** sudipto has quit IRC14:32
ajoihrachyshka: https://review.openstack.org/#/c/202425/ and https://review.openstack.org/#/c/20197914:34
ajoboth manually tested and working14:34
ajogaryk  ^14:34
*** arajagopal has quit IRC14:35
ajoI'm ok with them as they are, unless there's something big, I'd merge (ha.. as if it was so easy...) and iterate over the design later14:35
ajobut all (net/port association and dissociation from policies, port/next dict extension, etc..) works14:35
*** neeti has quit IRC14:36
*** ronis has quit IRC14:36
garykajo: there is no rest for the wicked14:37
ajogaryk there is not.... there is not... ;)14:37
openstackgerritAmir Sadoughi proposed openstack/python-neutronclient: Support for child resource extensions  https://review.openstack.org/20259714:38
scheuranmestery: regarding the discussion if the new macvtap support should land in it's own repo or somewhere else: can I put this on next weeks neutron meeting agenda? Is that the right place to get things forward?14:38
scheuranmestery: refering to this one: https://review.openstack.org/19590714:38
ajogaryk, of course, if there's something unclear or that needs discussion, -1 and let's discuss14:38
mesteryscheuran: Yes please do, that way we can get armax to chime in as well14:39
mesterythanks!14:39
*** rkukura has quit IRC14:40
scheuranmestery: thx14:40
garykajo: some silly stuff - otherwise its good14:41
garykajo: once you guys fix those i'll +214:43
ajogaryk I can fix silly stuff too :)14:43
ajogaryk, going14:43
garykif only i could be a fly on the wall to hear the curses :)14:43
Sam-I-Ammestery: is there any way to get people writing neutron docs to put them in the networking guide instead of devdocs?14:43
mesterySam-I-Am: Lets talk about itn ext week in the meeting, please add an agenda item14:44
mestery:)14:44
*** ekuris has quit IRC14:44
ajogaryk: what do you mean by this comment?14:44
ajohttps://review.openstack.org/#/c/201979/8/neutron/plugins/ml2/plugin.py14:44
ajoI don't see the difference :D14:44
ajoahh14:44
ajook, the parenthesis14:44
ajo:)14:44
ajovery subtle14:45
*** SridharG has quit IRC14:45
openstackgerritYusuke Ide proposed openstack/neutron: Add breakages in public API of devref  https://review.openstack.org/19848314:46
sambettsCan we get some eyes on this please: https://review.openstack.org/#/c/202549/ the bug its fixing is causing UT failures in networking-cisco14:47
sambetts:)14:47
*** SridharG has joined #openstack-neutron14:47
*** SridharG has left #openstack-neutron14:47
Sam-I-Ammestery: is next week an odd week?14:47
*** sbalukoff has quit IRC14:47
mesterySam-I-Am: It's the Monday afternoon meeting next week, 2100 UTC14:47
*** johnbelamaric has joined #openstack-neutron14:48
Sam-I-Ammestery: hmm ok... my brain is having a fencepost error14:48
*** cing has joined #openstack-neutron14:49
*** arajagopal has joined #openstack-neutron14:49
*** obondarev_ has joined #openstack-neutron14:49
scheuranmestery: The neutron-meeting wiki still has the last meeting listed as next meeting. Will you just move the old agenda forward? So can I just add my stuff to the old agenda or shall I wait until you updated that wiki?14:50
*** carl_baldwin has joined #openstack-neutron14:50
*** Vikram has joined #openstack-neutron14:50
mesteryscheuran: Put it on "On Demand" and I'll adjust as needed14:50
scheuranmestery: ok, thanks14:51
*** regXboi has joined #openstack-neutron14:51
*** markvoelker has joined #openstack-neutron14:53
*** arajagopal1 has joined #openstack-neutron14:54
*** arajagopal has quit IRC14:57
*** gampel has quit IRC14:57
Sam-I-Ammestery: do you want this item under docs, or on demand?14:58
*** Longgeek has quit IRC14:58
mesterySam-I-Am: On Demand please14:58
ajogaryk, updated: https://review.openstack.org/#/c/20197914:58
*** shashank_hegde_ has joined #openstack-neutron14:59
*** pal_ has joined #openstack-neutron14:59
*** jschwarz has joined #openstack-neutron14:59
*** obondarev_ has quit IRC14:59
*** pal has quit IRC14:59
ajokevinbenton, your non-CRUD db retry patch works like a charm ;)15:00
*** cing has quit IRC15:00
*** redbeard has quit IRC15:00
*** Longgeek has joined #openstack-neutron15:00
*** daneyon_ has joined #openstack-neutron15:02
*** Murali has quit IRC15:03
*** Shikha has quit IRC15:04
*** john-davidge has joined #openstack-neutron15:04
*** Shikha has joined #openstack-neutron15:04
*** sudipto has joined #openstack-neutron15:04
*** daneyon has quit IRC15:04
*** pal_ has quit IRC15:05
*** obondarev__ has joined #openstack-neutron15:05
*** tflynn has joined #openstack-neutron15:05
openstackgerritIhar Hrachyshka proposed openstack/neutron-vpnaas: migrations: rearrange the tree to support split migration phases  https://review.openstack.org/20262715:06
*** alex_klimov has quit IRC15:06
*** tflynn_ has joined #openstack-neutron15:06
*** coolsvap|away is now known as coolsvap15:06
openstackgerritPaul Michali proposed openstack/neutron: DO NOT MERGE - test change  https://review.openstack.org/19614915:07
*** Longgeek has quit IRC15:07
ihrachyshkapc_m, https://review.openstack.org/20262715:07
*** pal has joined #openstack-neutron15:07
pc_mihrachyshka: hi. Will check in a min. BTW, I had to push a temp commit to disable migration check in PEP8 for VPN (Neutron changes for new migration cause this to fail).15:08
*** sudipto_ has joined #openstack-neutron15:09
*** tflynn has quit IRC15:09
*** amotoki has joined #openstack-neutron15:10
*** Shikha has quit IRC15:11
*** absubram has joined #openstack-neutron15:11
*** sdake has quit IRC15:11
*** sudipto has quit IRC15:12
*** abehl has quit IRC15:12
*** shashank_hegde_ has quit IRC15:13
*** ParsectiX has joined #openstack-neutron15:13
*** pal has quit IRC15:14
*** sballe has joined #openstack-neutron15:15
*** redbeard has joined #openstack-neutron15:15
*** tflynn_ has quit IRC15:15
*** mutoulbj has quit IRC15:16
*** dannywilson has joined #openstack-neutron15:16
alegacyIs this subproject still the officially adopted way of implementing router port forwarding?   http://docs.openstack.org/developer/neutron/devref/sub_projects.html#portforwarding15:17
*** pal has joined #openstack-neutron15:17
*** sdake has joined #openstack-neutron15:17
alegacyi found other references to port forwarding in reviews and BPs but they seemed to be abondoned in favor of that project15:18
*** armax has joined #openstack-neutron15:18
*** jbell8 has joined #openstack-neutron15:19
*** afazekas has quit IRC15:20
*** batkins has joined #openstack-neutron15:21
pc_mihrachyshka: Had some questions. Put them in review for 202627.15:22
*** obondarev__ has quit IRC15:22
*** obondarev_ has joined #openstack-neutron15:23
*** batkins is now known as batkins6115:24
*** pal_ has joined #openstack-neutron15:24
*** pal has quit IRC15:25
*** obondarev_ has quit IRC15:25
*** jbell8 has quit IRC15:25
*** jbell8 has joined #openstack-neutron15:26
*** obondarev__ has joined #openstack-neutron15:27
*** jbell8 has quit IRC15:27
*** jbell8 has joined #openstack-neutron15:28
batkins61I'm using rhel osp and ended up with no br-tun ports on my controller node, although the ports on my compute node are present.  Where would I look for the cause?  (Kilo, rhel 7, OSP 6).15:28
*** dsneddon_away is now known as dsneddon15:29
openstackgerritCarl Baldwin proposed openstack/neutron: Enable pluggable IPAM implementation by default  https://review.openstack.org/18102315:29
*** yog__ has quit IRC15:34
*** jbell8 has quit IRC15:35
*** pal has joined #openstack-neutron15:36
*** vivek-eb_ is now known as vivek_ebay15:36
*** scheuran has quit IRC15:36
*** pal_ has quit IRC15:36
*** jbell8 has joined #openstack-neutron15:37
*** jobewan has joined #openstack-neutron15:37
ihrachyshkapc_m, re check_migration - ack15:37
ihrachyshkaI'll try to fix it15:37
*** yfried has quit IRC15:37
openstackgerritIlya Shakhat proposed openstack/neutron: Verify that DHCP namespace process is killed by netns_cleanup  https://review.openstack.org/19955615:39
*** MentalRay__ has joined #openstack-neutron15:39
*** obondarev__ has quit IRC15:40
*** jbell8 has quit IRC15:40
*** ig0r_ has quit IRC15:40
*** sudipto_ has quit IRC15:41
*** jbell8 has joined #openstack-neutron15:41
*** jbell8 has quit IRC15:42
*** jbell8 has joined #openstack-neutron15:42
*** moha_hunt has quit IRC15:43
*** tflynn has joined #openstack-neutron15:43
*** afazekas has joined #openstack-neutron15:44
*** obondarev__ has joined #openstack-neutron15:44
*** madhu_ak has joined #openstack-neutron15:45
*** vishwanathj has joined #openstack-neutron15:45
*** nlahouti has joined #openstack-neutron15:46
*** vishwana_ has quit IRC15:47
*** chem has quit IRC15:50
*** pal_ has joined #openstack-neutron15:50
*** pal has quit IRC15:51
*** shashank_hegde_ has joined #openstack-neutron15:52
*** annegentle has quit IRC15:52
*** annegentle has joined #openstack-neutron15:52
*** obondarev__ has quit IRC15:53
*** jbell8 has quit IRC15:54
*** mickeys has joined #openstack-neutron15:54
*** vivek_ebay is now known as vivek_15:54
*** jbell8 has joined #openstack-neutron15:55
*** obondarev_ has joined #openstack-neutron15:56
*** TrevorV has joined #openstack-neutron15:57
*** mkolesni has joined #openstack-neutron15:58
*** sripriya has joined #openstack-neutron15:58
*** shashank_hegde_ has quit IRC15:58
*** evgenyf has quit IRC15:59
*** banix has quit IRC16:00
*** jlibosva has quit IRC16:00
*** shivrao has joined #openstack-neutron16:01
*** banix has joined #openstack-neutron16:01
*** moha_hunt has joined #openstack-neutron16:01
*** jlibosva has joined #openstack-neutron16:02
openstackgerritSalvatore Orlando proposed openstack/neutron: Introduce usage data tracking for Neutron  https://review.openstack.org/18888316:02
openstackgerritSalvatore Orlando proposed openstack/neutron: Add DB support for resource usage tracking  https://review.openstack.org/18860816:02
openstackgerritSalvatore Orlando proposed openstack/neutron: Devref for quotas  https://review.openstack.org/19079816:02
openstackgerritSalvatore Orlando proposed openstack/neutron: Reservations support  https://review.openstack.org/16365916:02
openstackgerritSalvatore Orlando proposed openstack/neutron: Enable resource usage tracking for reference plugins.  https://review.openstack.org/18888416:02
openstackgerritSalvatore Orlando proposed openstack/neutron: Create packages for quota modules  https://review.openstack.org/18861816:02
openstackgerritSalvatore Orlando proposed openstack/neutron: Add plural names for quota resources  https://review.openstack.org/19486716:02
*** jlanoux has quit IRC16:02
johnbelamariccarl_baldwin, neiljerram: ok, I think this would be good to track down and see what can be done here. i agree pluggable IPAM is appropriate place to address it if the workflow makes it possible16:02
*** jbell8 has quit IRC16:02
*** ronis has joined #openstack-neutron16:02
*** dboik_ has quit IRC16:03
*** sthillma_ has joined #openstack-neutron16:03
carl_baldwinThere are some other things my model did that would have to be rethought.16:03
neiljerramjohnbelamaric: I can take action to check the sequencing here, if that would be helpful16:03
*** jbell8 has joined #openstack-neutron16:03
johnbelamaricneiljerram: yes, that would be great.16:03
johnbelamaricOk, well, let's just keep moving it forward and see if we can get it all to come together16:04
carl_baldwin1) By modeling each segment with a Neutron network, the L2 details were kept completely separate between the networks.16:04
*** shivrao_ has joined #openstack-neutron16:04
*** dane_leblanc has quit IRC16:04
*** sridhar_ram has joined #openstack-neutron16:05
*** coolsvap is now known as coolsvap|away16:05
*** hitalia has joined #openstack-neutron16:05
carl_baldwin2) It was easy to keep the static routed subnets, the ones confined to a segment, separate from the floating subnets, the ones that could be routed with BGP or other mechanism.16:05
neiljerramSo, e.g., they could in principle use different provider types?16:05
*** ihrachyshka has quit IRC16:05
carl_baldwinIf the whole thing is modeled as one Neutron network, there is nothing in the model to help us distinguish.16:05
*** pal_ has quit IRC16:05
carl_baldwinneiljerram: Yes, or just different segementation parameters.16:05
*** shivrao has quit IRC16:05
*** shivrao_ is now known as shivrao16:05
carl_baldwin#1 might be solvable with hierarchical port binding but I don’t understand it well enough to know yet.16:06
johnbelamariccarl_baldwin: perhaps I missed something, I know the discussion continued on during the mid-cycle. is there a write up on the current proposal? I thought were going to do the tree structure for networks to solve this16:06
*** park_heijlong_ has quit IRC16:06
*** sdake_ has joined #openstack-neutron16:07
carl_baldwinjohnbelamaric: front/back networks.  Right?16:07
johnbelamariccarl_baldwin: yes16:07
carl_baldwinjohnbelamaric: If so, that was still my plan.16:07
openstackgerritMadhusudhan Kandadai proposed openstack/neutron: Flavor Framework implementation  https://review.openstack.org/13975816:07
*** mutoulbj has joined #openstack-neutron16:07
carl_baldwinjohnbelamaric: But there is concern about creating tenant ports on a network that would not be visible to tenants.16:07
johnbelamariccarl_baldwin: ok, so i am confused perhaps16:07
*** ijw has quit IRC16:07
johnbelamariccarl_baldwin: ah, interesting. i see16:08
*** ivar-lazzaro has joined #openstack-neutron16:09
johnbelamaric(sorry, in another meeting may not respond quickly...)16:09
*** pal has joined #openstack-neutron16:09
*** s3wong has joined #openstack-neutron16:10
*** sdake has quit IRC16:10
*** obondarev_ has quit IRC16:10
*** crc32 has joined #openstack-neutron16:11
*** obondarev__ has joined #openstack-neutron16:12
*** merooney has quit IRC16:12
*** sudipto has joined #openstack-neutron16:12
*** tflynn has quit IRC16:13
*** jbell8 has quit IRC16:13
carl_baldwinjohnbelamaric: neiljerram:  I’ll try to get my thoughts gathered and send a message by the end of the week.16:14
*** jbell8 has joined #openstack-neutron16:15
*** absubram has quit IRC16:15
*** tflynn has joined #openstack-neutron16:16
openstackgerritMerged openstack/neutron-vpnaas: VPNaaS DevStack Plugin support  https://review.openstack.org/20059216:17
openstackgerritMerged openstack/neutron-vpnaas: Scenario test for vpnaas: ipsec-site-connection  https://review.openstack.org/15974616:17
*** ihrachyshka has joined #openstack-neutron16:17
*** MentalRay__ has quit IRC16:17
*** mitchjameson has joined #openstack-neutron16:18
*** jbell8 has quit IRC16:18
*** thorst has quit IRC16:18
*** kawa2014 has quit IRC16:18
*** jbell8 has joined #openstack-neutron16:19
*** jbell8 has quit IRC16:19
*** ZZelle has quit IRC16:20
*** ZZelle has joined #openstack-neutron16:20
*** dboik has joined #openstack-neutron16:20
*** tflynn has quit IRC16:21
*** sudipto has quit IRC16:21
*** tflynn has joined #openstack-neutron16:22
*** pal has quit IRC16:25
*** edmondsw has quit IRC16:26
*** redbeard has quit IRC16:27
*** tflynn has quit IRC16:27
*** Swami has joined #openstack-neutron16:27
*** shashank_hegde_ has joined #openstack-neutron16:27
*** achanda has joined #openstack-neutron16:28
ihrachyshkagaryk, hi. wanna check some versioned objects changes? https://review.openstack.org/#/c/200608/ it's one of high prio pieces for qos integration16:28
*** evgenyf has joined #openstack-neutron16:28
*** achanda has quit IRC16:29
*** dims_ has quit IRC16:30
*** _cjones_ has joined #openstack-neutron16:30
*** armax has quit IRC16:31
*** dims_ has joined #openstack-neutron16:32
*** dims_ has quit IRC16:32
*** tflynn has joined #openstack-neutron16:32
*** sbalukoff has joined #openstack-neutron16:33
*** fzdarsky has quit IRC16:34
*** mwagner_lap has quit IRC16:34
*** spandhe has joined #openstack-neutron16:34
*** achanda has joined #openstack-neutron16:35
*** fitoduarte has joined #openstack-neutron16:36
*** mutoulbj has quit IRC16:37
*** redbeard has joined #openstack-neutron16:37
*** obondarev__ has quit IRC16:37
*** crc32 has quit IRC16:38
*** ivar-lazzaro has quit IRC16:38
*** dane_leblanc has joined #openstack-neutron16:39
*** devvesa has quit IRC16:39
*** crc32 has joined #openstack-neutron16:39
ihrachyshkaajo, it's time to review https://review.openstack.org/#/c/200608/16:41
ajooh, true16:42
ajocan you also review...16:42
ajohttps://review.openstack.org/201979 ?16:42
ajo:)16:42
ajoand https://review.openstack.org/#/c/202425/16:42
ihrachyshkaajo, will do, not sure whether today though16:43
*** rwsu has quit IRC16:44
ajoack16:44
*** padkrish has joined #openstack-neutron16:44
*** ygbo has quit IRC16:47
*** shivrao has quit IRC16:47
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port  https://review.openstack.org/19444116:48
*** ajmiller has quit IRC16:48
ajoihrachyshka, diff reviewed from my last review, +2 it looks good.16:49
*** safchain has quit IRC16:49
ihrachyshkagaryk, amotoki, mestery: it begs for another reviewer: https://review.openstack.org/200608 (patch adding rules-inside-policies fields to policy objects)16:49
ihrachyshkaajo, thanks :)16:50
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: ovs_lib: Fix native implementation of db_list  https://review.openstack.org/20250516:50
ajoihrachyshka, thank you, awesome work :)16:50
amotokiihrachyshka: let me check after lbaas meeting16:50
ihrachyshkaamotoki, cool! :)16:50
*** hichihara has joined #openstack-neutron16:51
*** y_sawai has joined #openstack-neutron16:52
*** thorst has joined #openstack-neutron16:52
yamamoto_rossella_s: ping16:52
*** mutoulbj has joined #openstack-neutron16:52
ihrachyshkapc_m, I will look into check_migration tomorrow. hope it works for you16:52
*** ParsectiX has quit IRC16:53
amotokiihrachyshka: seems garyk beat me16:53
rossella_syamamoto_ pong16:53
ihrachyshkaamotoki, you're still welcome if you like to get deeper into objects :)16:53
*** absubram has joined #openstack-neutron16:53
*** obondarev__ has joined #openstack-neutron16:53
pc_mihrachyshka: Sure. It's disabled right now. Looks like we need to get VPN repo based grenade tests working, and transition to the new migration method.16:53
yamamoto_rossella_s: do you want to fold https://review.openstack.org/#/c/202505/ into yours?16:54
amotokiit is interesting one, but I haven't caught up fully... try to cathc up16:54
ajoihrachyshka, amotoki  VOs are nice, and we aren't still using them at full power, just very lightly16:54
pc_mihrachyshka: we = community16:54
*** ajmiller has joined #openstack-neutron16:54
rossella_syamamoto_ that was my suggestion, what do you think16:54
ihrachyshkapc_m, you mean, grenade first?16:54
*** ajmiller__ has joined #openstack-neutron16:54
*** ajmiller__ has quit IRC16:55
*** y_sawai has quit IRC16:55
*** padkrish has quit IRC16:55
*** obondarev__ has quit IRC16:56
*** crc32 has quit IRC16:56
*** padkrish has joined #openstack-neutron16:56
pc_mihrachyshka: Right now the check-migration is disabled. Also, there is a commit to disable services from grenade job. So... any migration code in VPN is not going to be tested (unless done manually, like I'm doing for a current commit).16:56
*** aranjan has joined #openstack-neutron16:56
*** minwang2 has joined #openstack-neutron16:56
ihrachyshkapc_m, yeah, that sucks. do you have specific plan/ETA/idea for grenade?16:57
yamamoto_rossella_s: i don't think your if_exists-related tweaks are neceesary if it's done within an ovsdb transaction.  how do you think?16:57
pc_mihrachyshka: So I guess we need the new migration scheme, modification of migration scripts, VPN grenade job (with plugin) working, and check-migration re-enabled for VPN PEP8.16:57
pc_mihrachyshka: I'm starting to look at grenade plugin to understand what is needed. Not very familiar with it, so slow going.16:58
yamamoto_rossella_s: it's entirely possible that i'm misunderstanding ovsdb code, though.16:58
ihrachyshkapc_m, check-migration is independent, I'll handle it16:59
ihrachyshkaideally, we should fix in neutron, not in *aas16:59
*** dane_leblanc has quit IRC16:59
rossella_syamamoto_ with your patch you don't need that tweak because you removed "if_exists" from the query at line 38616:59
ihrachyshkabecause old scheme should work with no changes16:59
ihrachyshkaif not, that's my bug16:59
*** obondarev__ has joined #openstack-neutron16:59
pc_mihrachyshka: thanks! lot on my plate.16:59
pc_mihrachyshka: currently old scheme fails with latest check-migration.17:00
*** aix has quit IRC17:00
*** jistr has quit IRC17:00
*** banix has quit IRC17:00
ihrachyshkapc_m, yeah, tomorrow I'll have smth17:00
pc_mihrachyshka: awesome17:01
rossella_syamamoto_ your patch fixes 3 issues 1) the if_exists with no record, 2) the generator that is wrongly built 3) the operation done outside the transaction17:01
sambettsSwami: ping17:01
Swamisambetts: hi17:01
*** tflynn has quit IRC17:02
rossella_syamamoto_, since issue 1 and 2 were fixed by my patches that were committed before yours...usually my patches should "win" since they are older...you should just fix issue 317:02
sambettsSwami: can you check this review out https://review.openstack.org/#/c/202549/ its related to your DVR update_port work17:02
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/20180317:02
*** abehl has joined #openstack-neutron17:02
sambettsSwami: its causing one of my UTs to fail in openstack/networking-cisco17:03
Swamisambetts: yes will take a look at it.17:03
*** merooney has joined #openstack-neutron17:03
rossella_syamamoto_ if you look at your patch, you didn't simply move all the operation inside the transaction, you also changed the code to address issue 1 and 217:03
*** merooney has quit IRC17:03
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements  https://review.openstack.org/20068217:03
pc_mihrachyshka: on 202627 review, I seemed to be confused a bit.17:03
sambettsSwami: Thanks :D17:03
*** obondarev__ has quit IRC17:03
*** neiljerram has quit IRC17:03
pc_mihrachyshka: 56893333aa52 is in contract directory, but has label for liberty_expand. The new migration file has a similar thing.17:04
*** merooney has joined #openstack-neutron17:04
ihrachyshkapc_m, ouch :)17:04
pc_mihrachyshka: Should the branch label be liberty_contract?17:04
yamamoto_rossella_s: ok, i guess it's best to fold all of three patches.  how do you think?17:05
pc_mihrachyshka: Same for the new migration file. Seems like labels are reversed.17:05
pc_mihrachyshka: hence the -1.17:05
ihrachyshkapc_m, yeah, if comments were inline, I would get the point :) thanks17:05
rossella_syamamoto_, what do you mean by folding?17:05
*** ildikov has quit IRC17:05
yamamoto_rossella_s: make them a single patch17:05
pc_mihrachyshka: Sorry.17:06
rossella_syamamoto_, they fix different issues17:06
pc_mihrachyshka: I marked one. You have the other marked.17:06
*** evgenyf has quit IRC17:07
rossella_syamamoto_ I guess we can fold if we state clearly in the commit message what issues are fixed and if we are coauthors17:07
yamamoto_rossella_s: i disagree.  all three bugs are caused by a single problem.17:07
Swamisambetts: can you add a test to check the order of the functions being called.17:07
sambettsSwami: to prevent regressions?17:07
rossella_syamamoto_ it's not a single problem, you changed multiple things17:08
*** tflynn has joined #openstack-neutron17:08
rossella_syamamoto_ I will comment on your review to point it out, anyway it's just a detail17:08
Swamisambetts: yes at least if the order is predefined then there will not be regressions.17:08
yamamoto_rossella_s: my change moved code into a transaction.  i consider it a signale change.17:08
rossella_syamamoto_, I am adding some comments to point out17:09
*** dane_leblanc has joined #openstack-neutron17:09
sambettsSwami: Moving those functions doesn't affect how your patch works?17:09
yamamoto_rossella_s: sorry, it's time to go bed for me.  if yours get merged, i'll rebase mine. :-)17:09
yamamoto_rossella_s: thank you for discussion17:09
*** abhiraut has joined #openstack-neutron17:09
rossella_syamamoto_ good night, let's talk tomorrow then17:09
jschwarzihrachyshka, I've pushed another revision for the qos tests. I'll try to follow up during the weekend - mail me if there's a need for a re-spin17:10
Swamisambetts: yes I understand it does not affect, but if the order is captured somewhere it would be safe for ever.17:10
yamamoto_rossella_s: sure.  good night!17:10
*** yamamoto_ has quit IRC17:10
jschwarzotherwise see you wednesday :D17:10
*** jbell8 has joined #openstack-neutron17:10
ihrachyshkajschwarz, cu. I'll also baby sit that one in case there are nits to fix17:10
jschwarzthanks17:11
jschwarzhave a nice weekend!17:11
*** jschwarz has quit IRC17:11
*** absubram has quit IRC17:12
sambettsSwami: awesome, I'll work on getting that test added, and then I hope we can rush that fix throgh because its blocking the cisco plugin repo17:12
Swamisambetts: sure17:12
*** tflynn has quit IRC17:13
*** mkolesni has quit IRC17:13
*** tflynn has joined #openstack-neutron17:13
*** krtaylor has quit IRC17:13
*** rm_work|away is now known as rm_work17:13
*** armax has joined #openstack-neutron17:14
*** ihrachyshka has quit IRC17:17
haleybSwami: i was just responding to the change with the same comment - need to check order17:17
*** signed8bit has quit IRC17:19
*** abregman has quit IRC17:19
*** SourabhP has joined #openstack-neutron17:19
*** signed8bit has joined #openstack-neutron17:20
*** tflynn_ has joined #openstack-neutron17:21
Swamihaleyb: thanks17:22
*** tflynn has quit IRC17:24
*** rossella_s has quit IRC17:25
*** achanda has quit IRC17:25
*** mlavalle has quit IRC17:25
*** jlibosva has quit IRC17:26
*** mitchjameson has quit IRC17:26
*** y_sawai has joined #openstack-neutron17:27
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port  https://review.openstack.org/19444117:29
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent  https://review.openstack.org/19444617:29
*** neophy has quit IRC17:30
*** sthillma_ has quit IRC17:31
*** yamahata has quit IRC17:31
*** dims_ has joined #openstack-neutron17:32
*** sthillma has joined #openstack-neutron17:32
*** annegentle has quit IRC17:33
*** armax has quit IRC17:34
*** banix has joined #openstack-neutron17:35
*** achanda has joined #openstack-neutron17:35
*** shivrao has joined #openstack-neutron17:35
*** vivek__ has joined #openstack-neutron17:36
openstackgerritJohn Davidge proposed openstack/neutron: DB & API changes for IPv6 Prefix Delegation  https://review.openstack.org/15869717:37
*** evgenyf has joined #openstack-neutron17:37
*** hitalia has quit IRC17:38
*** tflynn_ has quit IRC17:38
*** john-davidge has left #openstack-neutron17:39
*** dims_ has quit IRC17:39
*** john-davidge has joined #openstack-neutron17:39
john-davidgecarl_baldwin: Hi17:39
*** achanda has quit IRC17:40
carl_baldwinjohn-davidge: hi17:40
john-davidgecarl_baldwin: Freshly rebased PD patch is up https://review.openstack.org/#/c/158697/17:40
*** vivek_ has quit IRC17:40
*** madhu_ak_ has joined #openstack-neutron17:40
carl_baldwinExcellent, thanks.17:40
*** SumitNaiksatam has joined #openstack-neutron17:40
*** ajmiller__ has joined #openstack-neutron17:41
john-davidgecarl_baldwin: I did some digging into your comments about the new subnet model attribute https://review.openstack.org/#/c/158697/47/neutron/api/v2/attributes.py17:41
*** lazy_prince has quit IRC17:41
john-davidgecarl_baldwin: I’ve described a couple options in my reply17:41
*** rushil has joined #openstack-neutron17:41
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent  https://review.openstack.org/19444617:41
carl_baldwinjohn-davidge: Thanks, I’ll take a look soon.17:41
john-davidgecarl_baldwin: great, thanks17:42
*** tidwellr has left #openstack-neutron17:42
*** ajmiller has quit IRC17:43
*** zhenguo has quit IRC17:43
*** madhu_ak has quit IRC17:44
*** vivek__ has quit IRC17:44
*** achanda has joined #openstack-neutron17:46
*** rushil has quit IRC17:46
*** krtaylor has joined #openstack-neutron17:46
*** rushil has joined #openstack-neutron17:47
*** ivar-lazzaro has joined #openstack-neutron17:47
*** ivar-lazzaro has quit IRC17:48
regXboicarl_baldwin: ping - is https://bugs.launchpad.net/neutron/+bug/1317363 considered part of L3 "space" ?17:48
openstackLaunchpad bug 1317363 in neutron "Unable to update external network subnet's gateway-ip" [Medium,In progress] - Assigned to Hirofumi Ichihara (ichihara-hirofumi)17:48
*** rushil has quit IRC17:49
*** jckasper has quit IRC17:49
*** edmondsw has joined #openstack-neutron17:49
*** merooney has quit IRC17:49
*** ivar-lazzaro has joined #openstack-neutron17:49
*** igordcard has joined #openstack-neutron17:50
*** ivar-lazzaro has quit IRC17:50
*** iymht has joined #openstack-neutron17:51
*** edmondsw has quit IRC17:51
*** sthillma has quit IRC17:51
*** tflynn has joined #openstack-neutron17:51
*** ivar-lazzaro has joined #openstack-neutron17:51
*** amuller is now known as amuller_afk17:51
*** edmondsw has joined #openstack-neutron17:51
*** jwy has joined #openstack-neutron17:52
*** moshele has quit IRC17:52
*** dims_ has joined #openstack-neutron17:52
*** dims_ has quit IRC17:53
*** tflynn has quit IRC17:56
*** carl_baldwin has quit IRC17:56
*** hichihara has quit IRC17:57
*** dims_ has joined #openstack-neutron17:57
*** merooney has joined #openstack-neutron17:59
*** Vikram has quit IRC17:59
*** y_sawai has quit IRC17:59
*** rkukura has joined #openstack-neutron18:00
*** sthillma has joined #openstack-neutron18:00
*** amotoki has quit IRC18:00
*** padkrish has quit IRC18:01
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port  https://review.openstack.org/19444118:01
*** padkrish has joined #openstack-neutron18:02
*** minwang2 has quit IRC18:02
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent  https://review.openstack.org/19444618:02
*** Kennan has quit IRC18:02
*** dboik has quit IRC18:02
*** ivar-laz_ has joined #openstack-neutron18:02
*** redbeard has quit IRC18:03
*** Kennan has joined #openstack-neutron18:03
*** Sukhdev has joined #openstack-neutron18:04
*** Marga_ has quit IRC18:06
*** madhu_ak_ is now known as madhu_ak18:06
*** ivar-lazzaro has quit IRC18:06
*** Marga_ has joined #openstack-neutron18:07
*** patrickeast has joined #openstack-neutron18:07
*** jprovazn has quit IRC18:07
*** dane_leblanc has quit IRC18:08
*** merooney has quit IRC18:08
*** shashank_hegde_ has quit IRC18:09
*** abehl has quit IRC18:09
*** xek has quit IRC18:10
*** isq has quit IRC18:10
*** yamamoto has joined #openstack-neutron18:10
*** doude has quit IRC18:11
*** amuller_afk is now known as amuller18:11
*** minwang2 has joined #openstack-neutron18:11
*** vivek_ has joined #openstack-neutron18:11
*** xek has joined #openstack-neutron18:12
*** isq has joined #openstack-neutron18:13
*** sdake has joined #openstack-neutron18:13
*** yamamoto has quit IRC18:15
*** MentalRay__ has joined #openstack-neutron18:15
*** signed8bit is now known as signed8bit_ZZZzz18:16
*** achanda has quit IRC18:16
*** nlahouti has quit IRC18:16
*** sdake_ has quit IRC18:16
*** nmagnezi has quit IRC18:17
*** nlahouti has joined #openstack-neutron18:18
*** baoli has joined #openstack-neutron18:19
openstackgerritAssaf Muller proposed openstack/neutron: Update port bindings for master router  https://review.openstack.org/14111418:19
*** merooney has joined #openstack-neutron18:20
*** rushil has joined #openstack-neutron18:21
*** y_sawai has joined #openstack-neutron18:21
*** ildikov has joined #openstack-neutron18:22
*** rpothier has quit IRC18:22
*** y_sawai has quit IRC18:22
*** padkrish has quit IRC18:22
*** padkrish has joined #openstack-neutron18:23
*** dboik has joined #openstack-neutron18:23
*** crc32 has joined #openstack-neutron18:24
*** mickeys has quit IRC18:25
*** tidwellr has joined #openstack-neutron18:26
*** achanda has joined #openstack-neutron18:26
*** dane_leblanc has joined #openstack-neutron18:29
*** agireud has joined #openstack-neutron18:29
*** MentalRay__ has quit IRC18:30
*** jckasper has joined #openstack-neutron18:32
*** shashank_hegde_ has joined #openstack-neutron18:34
*** absubram has joined #openstack-neutron18:34
*** mutoulbj has quit IRC18:35
*** merooney has quit IRC18:38
*** hitalia has joined #openstack-neutron18:38
*** achanda has quit IRC18:38
*** minwang2 has quit IRC18:39
johnbelamariccarl_baldwin: ping18:39
*** sbalukoff has quit IRC18:39
johnbelamariccarl_baldwin: looks like #link https://review.openstack.org/#/c/183099 needs one more round unfortunately18:39
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port  https://review.openstack.org/19444118:44
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent  https://review.openstack.org/19444618:44
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Cleanup the fip agent gateway port delete routines  https://review.openstack.org/19445518:44
*** rm_work is now known as rm_work|away18:45
*** sridhar_ram has quit IRC18:45
*** vichow has joined #openstack-neutron18:46
*** signed8bit_ZZZzz is now known as signed8bit18:47
*** mutoulbj has joined #openstack-neutron18:48
*** ivar-laz_ has quit IRC18:49
*** merooney has joined #openstack-neutron18:49
*** gsagie has joined #openstack-neutron18:51
*** dims_ has quit IRC18:51
*** dims_ has joined #openstack-neutron18:52
*** mutoulbj has quit IRC18:53
*** setuid has quit IRC18:55
*** dims_ has quit IRC18:56
*** minwang2 has joined #openstack-neutron18:56
*** minwang2 has quit IRC19:01
*** shashank_hegde_ has quit IRC19:01
*** ParsectiX has joined #openstack-neutron19:01
*** setuid has joined #openstack-neutron19:03
*** madhu_ak has quit IRC19:03
abhirautping Sukhdev19:04
Sukhdevabhiraut: Hi19:05
abhirautquestion around L2gw19:05
Sukhdevabhiraut: sure19:05
abhirautthe upstream implementation is a service plugin.. is it a generic one?19:05
abhirautfor example i would like to write up something which interacts with my backend using the L2gw API19:06
abhirauthow do u go about it?19:06
*** SourabhP has quit IRC19:06
*** regXboi has quit IRC19:06
abhirautwrite a new service plugin referencing L2GW extension in the repo?19:06
*** sdake_ has joined #openstack-neutron19:08
abhirautSukhdev: did i make sense :S?19:08
Sukhdevabhiraut: the upstream is service plugin - correct - what do you mean by generic?19:09
*** minwang2 has joined #openstack-neutron19:09
Sukhdevabhiraut: the front-end is service plugin - backend is L2 GW agent19:10
*** sthillma has quit IRC19:10
Sukhdevso, you will use the service plugin as-is19:10
*** padkrish has quit IRC19:10
Sukhdevand will make any modification that you need to the L2 GQ agent side19:10
Sukhdevs/GQ/GW19:11
*** padkrish has joined #openstack-neutron19:11
*** sdake has quit IRC19:11
abhirautah ok..19:11
Sukhdevabhiraut: what do you want to do on the back-end?19:12
abhirautthanks Sukhdev.. i will look into it19:12
Sukhdevpresently it supports OVSDB - if you need to use OVSDB on the back-end, you can use the whole solution as-is19:13
abhirautbasically map the API and send it to our controller19:13
Sukhdevotherwise, you will modify the back-end to work with your platform19:13
Sukhdevabhiraut: correct19:13
*** pal has joined #openstack-neutron19:14
*** sbalukoff has joined #openstack-neutron19:14
abhirautfor example .. l2-gateway-connection will result in a API call which I would send to my backend19:14
abhirautand my controller will do the necessary configurations19:14
*** padkrish has quit IRC19:15
abhirautso from what i understand.. i will be writing an agent of some sort which would relay this information back to my controller .. correct?19:15
abhirautand use the service plugin as is19:16
*** sridhar_ram has joined #openstack-neutron19:16
*** ParsectiX has quit IRC19:17
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Fix DVR interface delete by port when gateway is set  https://review.openstack.org/17300119:18
*** ParsectiX has joined #openstack-neutron19:18
*** ivar-lazzaro has joined #openstack-neutron19:20
*** agireud has quit IRC19:20
*** evgenyf has quit IRC19:21
Sukhdevabhiraut: sorry - was distracted - back now -19:21
Sukhdevabhiraut: yes, what you said is correct19:21
*** annegentle has joined #openstack-neutron19:21
*** regXboi has joined #openstack-neutron19:22
*** madhu_ak has joined #openstack-neutron19:22
*** agireud has joined #openstack-neutron19:22
*** baoli has quit IRC19:23
*** ivar-lazzaro has quit IRC19:24
*** dguerri` is now known as dguerri19:26
*** agireud has quit IRC19:26
*** baoli has joined #openstack-neutron19:28
*** carl_baldwin has joined #openstack-neutron19:29
*** agireud has joined #openstack-neutron19:29
*** spandhe has quit IRC19:29
*** pal has quit IRC19:31
*** spandhe has joined #openstack-neutron19:31
*** banix has quit IRC19:32
*** mitchjameson has joined #openstack-neutron19:36
*** padkrish has joined #openstack-neutron19:38
*** mlavalle has joined #openstack-neutron19:39
*** john-davidge has quit IRC19:40
*** padkrish has quit IRC19:43
*** ronis has quit IRC19:43
*** harlowja has joined #openstack-neutron19:49
*** harlowja_ has quit IRC19:49
*** tmorin has joined #openstack-neutron19:52
*** harlowja has quit IRC19:52
*** harlowja has joined #openstack-neutron19:53
*** Swami has quit IRC19:55
*** tflynn has joined #openstack-neutron19:56
*** tflynn has joined #openstack-neutron19:56
*** ParsectiX has quit IRC20:01
*** achanda has joined #openstack-neutron20:03
*** Sukhdev has quit IRC20:04
*** gampel has joined #openstack-neutron20:04
*** ParsectiX has joined #openstack-neutron20:04
*** sridhar_ram1 has joined #openstack-neutron20:05
*** abhiraut has quit IRC20:05
*** padkrish has joined #openstack-neutron20:07
*** sridhar_ram has quit IRC20:07
*** rm_work|away is now known as rm_work20:07
*** achanda has quit IRC20:09
*** regXboi has quit IRC20:09
*** regXboi has joined #openstack-neutron20:09
*** padkrish has quit IRC20:11
*** salv-orlando has quit IRC20:11
*** sridhar_ram has joined #openstack-neutron20:11
*** agireud has quit IRC20:13
*** Sukhdev has joined #openstack-neutron20:15
*** merooney has quit IRC20:15
*** sridhar_ram1 has quit IRC20:15
*** agireud has joined #openstack-neutron20:17
*** ajmiller__ has quit IRC20:17
*** achanda has joined #openstack-neutron20:19
*** abhiraut has joined #openstack-neutron20:22
*** ParsectiX has quit IRC20:23
*** ZZelle_ has joined #openstack-neutron20:23
abhirautSukhdev: thanks .. i will look into it .. and will follow up if I have more questions. This was helpful.20:23
*** sthillma has joined #openstack-neutron20:23
Sukhdevabhiraut: no problem20:23
*** spandhe has quit IRC20:24
*** igordcard has quit IRC20:25
*** vichow has quit IRC20:26
*** sridhar_ram1 has joined #openstack-neutron20:27
*** annegentle has quit IRC20:28
*** sridhar_ram has quit IRC20:29
*** mickeys has joined #openstack-neutron20:29
*** gsagie has quit IRC20:30
*** jasondotstar has joined #openstack-neutron20:30
*** SourabhP has joined #openstack-neutron20:30
*** ParsectiX has joined #openstack-neutron20:31
*** agireud has quit IRC20:31
*** ivar-laz_ has joined #openstack-neutron20:31
*** rossella_s has joined #openstack-neutron20:31
*** agireud has joined #openstack-neutron20:32
openstackgerritCarl Baldwin proposed openstack/neutron: Remove perform_snat_action indirection  https://review.openstack.org/20022020:32
*** sridhar_ram has joined #openstack-neutron20:32
openstackgerritRyan Tidwell proposed openstack/neutron: [WIP] Add devref for BGP Speaker API extension  https://review.openstack.org/19645220:33
openstackgerritRyan Tidwell proposed openstack/neutron: [WIP] Add devref for BGP Speaker API extension  https://review.openstack.org/19645220:33
*** doude has joined #openstack-neutron20:33
*** doude has quit IRC20:33
*** doude has joined #openstack-neutron20:33
*** sridhar_ram1 has quit IRC20:34
*** doude has quit IRC20:34
*** doude has joined #openstack-neutron20:34
*** doude has quit IRC20:34
*** doude has joined #openstack-neutron20:34
*** krtaylor has quit IRC20:36
*** dims_ has joined #openstack-neutron20:37
*** alex_klimov has joined #openstack-neutron20:37
*** dane_leblanc has quit IRC20:39
*** dave-mccowan has joined #openstack-neutron20:41
*** salv-orlando has joined #openstack-neutron20:45
*** ivar-laz_ has quit IRC20:46
*** ivar-lazzaro has joined #openstack-neutron20:46
*** dane_leblanc has joined #openstack-neutron20:47
*** ivar-lazzaro has quit IRC20:47
*** ivar-lazzaro has joined #openstack-neutron20:47
*** ivar-lazzaro has quit IRC20:52
*** rushil has quit IRC20:54
openstackgerritCarl Baldwin proposed openstack/neutron: Use only the lower 16 bits of iptables mark for marking  https://review.openstack.org/20172720:54
*** pc_m has quit IRC20:55
*** aranjan has quit IRC20:57
*** ajmiller has joined #openstack-neutron20:58
*** aranjan has joined #openstack-neutron20:58
*** aranjan has quit IRC20:58
*** rossella_s has quit IRC20:58
amullersc68cal: I read the email thread you just posted about FWaaS, the thing that concerns me is the centralized nature of it. With DVR and security groups you get the entire flow distributed, both routing and iptables.21:00
amullerFWaaS (At least the reference impl. of it) is really a huge step back in that sense21:00
amullerAre you aware of any plans to implement the FWaaS API with iptables on the compute nodes instead of a centralized node?21:01
sc68calamuller: agreed - however that's a problem of the implementation, not really the API21:01
sc68calIf I am not mistaken, it just is that the current fwaas is implicitly doing perimeter filtering21:01
amullerand yet the implementation often decides if the feature gets adoption21:01
*** aranjan has joined #openstack-neutron21:02
*** absubram has quit IRC21:02
sc68calamuller: very true21:02
sc68calamuller: there were some cards that we marked yellow, for stuff the ref impl does but needs to do better21:03
amullerI'm concerned with the community spending any time on developing an API that has a backwards implementation, unless there are concrete plans to scrape the centralized nature of it and use an agent (Be it the existing L2 agents, or a separate one?) on the compute node to do the work21:03
*** armax has joined #openstack-neutron21:03
sc68calamuller: I was interested in doing the fwaas API agent side 2 years ago21:03
openstackgerritCarl Baldwin proposed openstack/neutron: Make external_gateway_nat_rules easier to understand  https://review.openstack.org/20129421:03
sc68calfor provider networking case21:03
sc68calwhere the l3 agent is not running21:04
openstackgerritCarl Baldwin proposed openstack/neutron: Use only the lower 16 bits of iptables mark for marking  https://review.openstack.org/20172721:04
sc68calamuller: so, I think that's something we need to capture somewhere, that the ref impl needs to stop depending on the l3 agent, maybe look into distributed21:04
amullersc68cal: Another question I had - I remember you analyzed the current SGs API VS Amazon's API and found a bunch of gaps, so since we can't claim compatability anyhow, is the plan to change the API to ensure compatability (While none of the other Neutron APIs are compatible...), or just freeze / maintain the current API? If so what's the reasoning behind that, if the API is not compatible, you either make it compatible or just deviate21:05
amullercompletely, what's the point of freezing it?21:05
*** ijw has joined #openstack-neutron21:06
sc68calamuller: I think the only gaps are basically the attribute names, but conceptually they remain similar21:06
amullerI wonder why we care about Amazon compatability specifically with the SGs API21:06
sc68calamuller: mostly because it's a well known concept in the "cloud" world, where it's easy for people who have worked with IaaS systems21:07
sc68calthe attrs may not match, but people know what to expect and how to use21:07
*** padkrish has joined #openstack-neutron21:08
sc68calso it's not really code compat- more user experience and concept21:08
sc68calI freely admit, pretty squishy and subjective21:08
*** hitalia has quit IRC21:09
amullerI'm looking at this page: http://docs.aws.amazon.com/AWSEC2/latest/APIReference/OperationList-query.html21:09
sc68calI'd rather see us make a better, clean API for doing more advanced filtering things, then a hodgepodge21:09
amullerwe have so much stuff in the Neutron API that doesn't really correlate to Amazon's API, and the inverse is true as well21:09
*** gangil has joined #openstack-neutron21:10
sc68calI will point out that Amazon even saw the SG API doesn't cover everything - http://docs.aws.amazon.com/AWSEC2/latest/APIReference/API_CreateNetworkAcl.html21:10
amullersc68cal: Note that NetworkAcl is a stateless firewall21:10
amullerwhich is pretty different from both SGs and FWaaS21:11
*** signed8bit is now known as signed8bit_ZZZzz21:11
*** dane_leblanc has quit IRC21:11
*** flaviof is now known as flaviof_afk21:11
sc68calamuller: are they? I dn't see that documented on http://docs.aws.amazon.com/AWSEC2/latest/APIReference/API_CreateNetworkAcl.html21:12
amullerhttp://docs.aws.amazon.com/AmazonVPC/latest/UserGuide/VPC_ACLs.html, 6th bullet point21:12
sc68calah yes thanks21:12
*** padkrish has quit IRC21:12
amuller sc68cal: I don't know, I'm just having a very hard time justifying work on FWaaS where the rationale is that we don't want to change the SGs API, when we conceptually differ a lot from Amazon APIs anyway21:12
*** signed8bit_ZZZzz is now known as signed8bit21:12
amullerseems like a very weak reason to me21:12
*** gangil has left #openstack-neutron21:13
amullerI'd rather see one API with a distributed reference impl.21:13
*** gangil has joined #openstack-neutron21:13
amullerthere, I took it off my chest21:13
amullernow you do what you will :)21:13
sc68calamuller: understood - the feature that is most valuable is the distributed part, yes?21:14
amullerPerhaps it is my Red Hat perspective but worrying about Amazon API compat at this stage in OpenStack's lifecycle seems like betting on a dead horse21:14
amullersc68cal: + a single API21:15
amullersc68cal: not two APIs with two implementations21:15
amulleralthoug that API would have to be pluggable, which is one place where FWaaS wins out21:16
sc68calamuller: We did discuss with dougwig and markmcclain that the APIs could be different but the implementations could begin to converge on a single21:16
sc68calobviously the features of the fwaas API, and the distributed nature of the SG implementation21:17
*** ivar-lazzaro has joined #openstack-neutron21:17
amullerand the21:17
amullerif you could converge the DB models as well...21:18
amullersorry hit an early enter there :)21:18
*** jbell8 has quit IRC21:18
sc68calamuller: agreed - there was talk about behind the API merging the models, since most likely the firewall API objects are a superset of the security group rules21:18
sc68calamuller: but again we have to differentiate between the issue with the reference implementation (being not distributed), with the actual API and the expressiveness of it21:19
*** doude has quit IRC21:20
dougwigamuller, sc68cal - to me, ideally, fwaas isn't centralized, but is rather an SDN widget like a router, that you can plop anywhere in your topology (and also becomes a good interface/home for things like qos).  whereas security groups are more port based.  i don't see them as *needing* to converge.21:20
*** gampel has quit IRC21:20
dougwigbut that's all IMO.21:20
*** ivar-lazzaro has quit IRC21:21
sc68caldougwig: there may be some objects behind the API that can be shared, like the actual Rule object, but yeah we also identified the fact that the FWaaS API doesnt have an actual Firewall object, like the L3 api ext has a router object21:22
sc68calwhich makes service insertion difficult. If we made a Firewall API object that might make service insertion easier since there's an actual "thing" they can reference21:22
*** ajmiller has quit IRC21:24
*** gangil has left #openstack-neutron21:25
*** ajmiller has joined #openstack-neutron21:25
*** ajmiller__ has joined #openstack-neutron21:27
*** ajmiller has quit IRC21:27
*** ajmiller__ has quit IRC21:28
*** ajmiller has joined #openstack-neutron21:29
*** ajmiller has quit IRC21:32
openstackgerritDoug Wiegley proposed openstack/neutron: RFE bug state decoder ring  https://review.openstack.org/20279721:33
*** Swami has joined #openstack-neutron21:33
*** abhiraut has quit IRC21:34
openstackgerritDoug Wiegley proposed openstack/neutron: RFE bug state decoder ring  https://review.openstack.org/20279721:35
*** sbalukoff has quit IRC21:36
*** krtaylor has joined #openstack-neutron21:36
dougwigsc68cal: right, agreed.21:37
*** sridhar_ram has quit IRC21:38
openstackgerritLynn Li proposed openstack/neutron: Update device info for FIP paired port w/unbound  https://review.openstack.org/20236021:38
openstackgerritLynn Li proposed openstack/neutron: Update device info for FIP paired port w/unbound  https://review.openstack.org/20236021:40
*** hitalia has joined #openstack-neutron21:41
*** boris-42 has joined #openstack-neutron21:42
*** tmorin has quit IRC21:42
*** johnbelamaric has quit IRC21:42
*** ajmiller has joined #openstack-neutron21:43
*** arajagopal1 has quit IRC21:43
Swamiamuller: quick question21:44
amullerSwami: yessir21:44
Swamiamuller: I could not run functional test anymore in my repo21:44
Swamiamuller: it fails21:45
amullerok, what's the issue?21:45
Swamiamuller: ERROR: InvocationError: '/usr/local/bin/ostestr --regex21:45
*** edmondsw has quit IRC21:45
amullerSwami: did you try rm -rf .tox21:46
amullerrun tox again?21:46
Swamiamuller: I run "tox -e dsvm-functional" but it fails immediately with this error.21:46
amullerprobably missing ostestr, which is test-requirements21:46
amulleris in*21:46
sc68calSwami: try tox --recreate -e dsvm-functional21:46
Swamiamuller: Yes I removed the whole ".tox" folder.21:46
Swamisc68cal: ok will try21:46
amullerremoving the .tox dir should have the same effect as running -r or --recreate21:46
Swamiamuller: yes that's what I felt.21:47
*** alex_klimov has quit IRC21:47
*** mickeys has quit IRC21:48
*** mlavalle has quit IRC21:49
*** abhiraut has joined #openstack-neutron21:49
*** agireud has quit IRC21:49
*** shashank_hegde_ has joined #openstack-neutron21:49
*** sbalukoff has joined #openstack-neutron21:50
openstackgerritsean mooney proposed openstack/neutron: Add config option to specify ovs datapath.  https://review.openstack.org/19720921:50
*** vivek_ has quit IRC21:51
amullerSwami: "which ostestr" I assume returns nothing?21:51
*** ivar-lazzaro has joined #openstack-neutron21:51
*** mutoulbj has joined #openstack-neutron21:51
Swamiamuller: what do you mean by "which ostestr". - /usr/local/bin/ostestr21:52
amullerSwami: so, so it is installed21:52
amullerok, can you paste the error you're getting?21:52
Swamiamuller: sure21:52
* amuller is checking if there was an ostestr release today21:53
Swamiamuller: #link http://paste.openstack.org/show/381862/21:53
*** shashank_hegde_ has quit IRC21:53
amullerSwami: hmm looks like an exception in os-testr itself21:54
amullerthat's interesting21:54
amullerSwami: ok if you just want to get on with your life you can use testr for example21:54
amullersource .tox/dsvm-functional/bin/activate21:54
Swamiamuller: ok, will try it out21:54
amullertestr neutron.tests.functional21:54
Swamiamuller: thanks will work for now.21:55
amulleroh and you also need export OS_TEST_PATH=./neutron/tests/fullstack21:55
amullerfrom /opt/openstack/neutron21:55
amullerSwami: that works for me21:55
Swamiamuller: will use it.21:55
*** mickeys has joined #openstack-neutron21:56
*** mutoulbj has quit IRC21:56
*** lynnlee has joined #openstack-neutron21:56
amullerSwami: latest release is 0.2.021:57
amullerSwami: so no new release that screwed us up21:57
Swamiok.21:57
*** vivek_ has joined #openstack-neutron21:58
Swamiamuller: #link https://bugs.launchpad.net/os-testr/+bug/147239521:58
openstackLaunchpad bug 1472395 in os-testr "ostestr has no default input and causes tox to fail to work" [Undecided,Confirmed] - Assigned to Assaf Muller (amuller)21:58
Swamiamuller: is this related to the issue that I am seeing or is it different21:58
amullerI think it's different21:58
Swamiamuller: yes never mind that is different.21:58
*** lilynn has quit IRC21:59
*** padkrish has joined #openstack-neutron21:59
*** padkrish has quit IRC22:00
*** padkrish has joined #openstack-neutron22:01
amullerSwami: do you have "subunit-trace" installed?22:01
Swamiamuller: no22:02
amullerah ha22:02
*** ChuckC has joined #openstack-neutron22:02
*** dboik has quit IRC22:02
*** lynnlee has quit IRC22:03
amullerSwami: can you sudo pip install tempest-lib?22:04
amullermy copy of subunit-trace comes from there22:04
Swamiamuller: will do22:04
amullerand that is used by ostestr22:04
*** ajmiller__ has joined #openstack-neutron22:05
*** madhu_ak_ has joined #openstack-neutron22:05
amullerSwami: although tempest-lib is in test-requirements for neutron so I don't know why that wouldn't be installed for you22:05
amullervery odd22:05
*** cing has joined #openstack-neutron22:05
Swamiyes, that is strange.22:05
Swamiamuller: I just did a pull today and things got messed up in my environment.22:06
*** ijw_ has joined #openstack-neutron22:06
*** ChuckC has quit IRC22:06
amullerSwami: so I had to walk through the ostestr code to understand what binary it wasn't finding22:07
*** ajmiller has quit IRC22:07
Swamiamuller: Thanks, let me see if I can spawn a new fresh VM and if things are fine with that.22:07
*** sbalukoff has quit IRC22:08
*** madhu_ak has quit IRC22:08
amullerSwami: maybe I'll try to send a patch to subprocess.Popen to make it print the the binary name it doesn't find22:08
Swamiamuller: ok22:08
*** madhu_ak_ is now known as madhu_ak22:08
*** ijw has quit IRC22:09
*** kbringard has quit IRC22:09
abhirautping Sukhdev22:10
*** gangil has joined #openstack-neutron22:10
*** achanda has quit IRC22:10
Sukhdevabhiraut: hey22:11
abhirautper the wiki.. it says the L2GW service plugin operates in conjunction with ML2 plugin/drivers22:12
abhirautwhat about monolithic plugins?22:12
abhirautcan they be used in conjunction with L2gw service plugins?22:13
madhu_akmestery, dougwig: thanks for iterating over for your votes on flavor framework22:13
*** Sukhdev has quit IRC22:14
*** jasondotstar has quit IRC22:15
*** vivek__ has joined #openstack-neutron22:15
*** vivek_ has quit IRC22:15
*** amuller is now known as amuller_afk22:15
*** Sukhdev has joined #openstack-neutron22:16
*** signed8bit is now known as signed8bit_ZZZzz22:19
*** thorst has quit IRC22:21
*** padkrish has quit IRC22:23
openstackgerritsean mooney proposed openstack/neutron: Adds configurable agent type  https://review.openstack.org/19721022:23
*** padkrish has joined #openstack-neutron22:26
*** achanda has joined #openstack-neutron22:27
Sukhdevabhiraut: no - monolithic plugins are on their own22:27
abhirautgot it.. thanks again Sukhdev!22:27
Sukhdevabhiraut: they provide all services through the monolithic plugin22:27
*** padkrish has quit IRC22:32
*** padkrish has joined #openstack-neutron22:32
*** mickeys has quit IRC22:35
*** padkrish has quit IRC22:37
*** sridhar_ram has joined #openstack-neutron22:38
*** achanda has quit IRC22:41
madhu_akamuller: I am seeing error for gate-neutron-dsvm-functional job. Isn't not stable now?22:44
*** ijw_ has quit IRC22:44
madhu_akoops, looks like amuller is away, anyone else seeing error with the above job?22:45
*** signed8bit_ZZZzz is now known as signed8bit22:45
madhu_akthe error snapshot is at https://jenkins04.openstack.org/job/gate-neutron-dsvm-functional/292/console22:46
*** jobewan has quit IRC22:48
*** gildub has joined #openstack-neutron22:49
*** thorst has joined #openstack-neutron22:54
*** signed8bit is now known as signed8bit_ZZZzz22:55
*** thorst has quit IRC22:56
*** padkrish has joined #openstack-neutron22:58
*** shwetaap has joined #openstack-neutron22:59
*** zzzeek has quit IRC23:08
*** achanda has joined #openstack-neutron23:09
*** sdake_ has quit IRC23:12
*** tidwellr has left #openstack-neutron23:12
*** daneyon_ has left #openstack-neutron23:15
*** daneyon_ has joined #openstack-neutron23:15
*** salv-orlando has quit IRC23:16
*** daneyon_ has quit IRC23:16
*** sridhar_ram has quit IRC23:17
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/20180323:17
*** crose has quit IRC23:17
*** abhiraut has quit IRC23:21
*** fitoduarte has quit IRC23:21
*** zigo has quit IRC23:21
openstackgerritSean M. Collins proposed openstack/neutron: Security Group API: Ensure attributes are validated as UUIDs  https://review.openstack.org/19701723:22
*** zigo has joined #openstack-neutron23:22
*** abhiraut has joined #openstack-neutron23:23
openstackgerritCarl Baldwin proposed openstack/neutron: Allow passing table argument to construct IpRouteCommand  https://review.openstack.org/19814323:23
*** cing has quit IRC23:23
*** jasondotstar has joined #openstack-neutron23:25
*** vivek__ has quit IRC23:26
*** jwy has quit IRC23:27
*** sbalukoff has joined #openstack-neutron23:28
*** aranjan_ has joined #openstack-neutron23:28
*** aranjan has quit IRC23:28
*** abhiraut has quit IRC23:29
*** aranjan_ has quit IRC23:30
*** bharath has joined #openstack-neutron23:33
*** openstackstatus has joined #openstack-neutron23:34
*** ChanServ sets mode: +v openstackstatus23:34
*** padkrish has quit IRC23:36
*** padkrish has joined #openstack-neutron23:36
*** ijw has joined #openstack-neutron23:38
openstackgerritfumihiko kakuma proposed openstack/neutron: Python 3: Wrap map() in a list call  https://review.openstack.org/20249723:39
*** sinh has joined #openstack-neutron23:39
openstackgerritZhaoBo proposed openstack/neutron: Fixed pass wrong cidr check during create subnet  https://review.openstack.org/20194223:40
*** padkrish has quit IRC23:41
*** padkrish has joined #openstack-neutron23:43
*** jwy has joined #openstack-neutron23:45
*** aranjan has joined #openstack-neutron23:46
regXboitidwellr: ping?23:46
*** amuller_afk is now known as amuller23:46
*** britthouser has quit IRC23:47
*** shwetaap1 has joined #openstack-neutron23:49
*** britthouser has joined #openstack-neutron23:50
*** shwetaap has quit IRC23:50
*** jaypipes has quit IRC23:50
*** TrevorV has quit IRC23:51
*** vivek_ has joined #openstack-neutron23:52
*** ParsectiX has quit IRC23:53
*** dguerri is now known as dguerri`23:54
*** barra204 has joined #openstack-neutron23:54
*** aranjan has quit IRC23:55
*** sridhar_ram has joined #openstack-neutron23:56
*** sridhar_ram has quit IRC23:57
*** sridhar_ram has joined #openstack-neutron23:57

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