Wednesday, 2016-06-08

*** thorst has quit IRC00:02
*** jckasper has joined #openstack-neutron00:03
*** ijw has quit IRC00:03
*** ihrachys has quit IRC00:04
openstackgerritCarl Baldwin proposed openstack/neutron: Add index on trunk_id in the subports model  https://review.openstack.org/32668900:04
*** Swami has quit IRC00:05
*** Sukhdev has quit IRC00:07
*** Sukhdev has joined #openstack-neutron00:08
*** ijw has joined #openstack-neutron00:09
*** ssalagame has quit IRC00:10
*** ijw has quit IRC00:11
*** ijw has joined #openstack-neutron00:11
*** mfuruta has joined #openstack-neutron00:11
*** reedip has joined #openstack-neutron00:13
*** reedip has quit IRC00:15
*** ssalagame has joined #openstack-neutron00:18
*** gangil has quit IRC00:19
*** claudiub has quit IRC00:21
*** claudiub has joined #openstack-neutron00:21
*** ihrachys has joined #openstack-neutron00:22
*** lnicolas has joined #openstack-neutron00:26
*** amotoki has joined #openstack-neutron00:31
*** reedip has joined #openstack-neutron00:33
*** ihrachys has quit IRC00:33
*** reedip has quit IRC00:35
*** reedip has joined #openstack-neutron00:35
*** reedip has quit IRC00:36
*** amotoki has quit IRC00:36
*** Sukhdev has quit IRC00:37
*** ihrachys has joined #openstack-neutron00:39
*** ssalagame has quit IRC00:41
*** tidwellr has left #openstack-neutron00:42
*** hoangcx has joined #openstack-neutron00:45
openstackgerritKevin Benton proposed openstack/neutron: Match filter criteria when constructing URI for href based iteration  https://review.openstack.org/32671100:47
*** thorst has joined #openstack-neutron00:48
*** Jabadia has joined #openstack-neutron00:50
*** salv-orlando has quit IRC00:53
*** reedip has joined #openstack-neutron00:53
*** salv-orlando has joined #openstack-neutron00:53
openstackgerritIhar Hrachyshka proposed openstack/neutron: Match filter criteria when constructing URI for href based iteration  https://review.openstack.org/32671100:54
*** lucas has joined #openstack-neutron00:55
*** reedip has quit IRC00:55
openstackgerritIhar Hrachyshka proposed openstack/neutron: Match filter criteria when constructing URI for href based iteration  https://review.openstack.org/32671100:56
*** baoli has joined #openstack-neutron00:57
*** lucas has quit IRC00:59
*** rahuls has quit IRC01:00
*** ihrachys has quit IRC01:01
*** shashank_hegde has quit IRC01:02
*** lucas has joined #openstack-neutron01:03
*** ssalagame has joined #openstack-neutron01:04
*** ihrachys has joined #openstack-neutron01:04
*** kevo has quit IRC01:04
*** ssalagame has quit IRC01:05
kevinbentonihrachys: your patch is nicer i think because it doesn't mutate an argument01:05
*** edand has joined #openstack-neutron01:05
*** fzdarsky_ has joined #openstack-neutron01:05
kevinbentonihrachys: if you want to revert back to it i can re +201:05
openstackgerritMerged openstack/neutron: Adding basic connectivity scenario to Neutron  https://review.openstack.org/32080801:05
*** abhiraut has quit IRC01:05
*** ivar-laz_ has quit IRC01:06
*** ivar-lazzaro has joined #openstack-neutron01:08
*** sjmc7 has quit IRC01:08
*** fzdarsky|afk has quit IRC01:08
*** edand has quit IRC01:09
*** ihrachys has quit IRC01:09
*** jerryz has quit IRC01:10
*** johnbelamaric has joined #openstack-neutron01:11
*** reedip has joined #openstack-neutron01:13
*** markvoelker has joined #openstack-neutron01:13
*** reedip has quit IRC01:15
*** johnbelamaric has quit IRC01:16
*** azbiswas has quit IRC01:17
*** Nepoc has quit IRC01:18
*** Nepoc has joined #openstack-neutron01:18
*** markvoelker has quit IRC01:18
*** jckasper has quit IRC01:19
*** jckasper has joined #openstack-neutron01:20
*** djan has quit IRC01:22
*** jckasper has quit IRC01:25
*** ssalagame has joined #openstack-neutron01:25
*** ramishra has joined #openstack-neutron01:26
*** djan has joined #openstack-neutron01:26
*** tbachman has quit IRC01:27
*** tflynn has joined #openstack-neutron01:29
*** ssalagame has quit IRC01:30
*** frontrunner has quit IRC01:31
*** reedip has joined #openstack-neutron01:31
*** p01nt3r75 has quit IRC01:31
*** amotoki has joined #openstack-neutron01:32
*** numans has joined #openstack-neutron01:32
*** ijw has quit IRC01:32
*** reedip has quit IRC01:35
*** EinstCrazy has joined #openstack-neutron01:35
*** amotoki has quit IRC01:36
*** tflynn has quit IRC01:37
*** gangil has joined #openstack-neutron01:37
*** toddnni_ has joined #openstack-neutron01:39
*** stanzgy has joined #openstack-neutron01:39
*** toddnni has quit IRC01:40
*** toddnni_ is now known as toddnni01:40
*** gouthamr has quit IRC01:40
*** ducnc has joined #openstack-neutron01:41
*** EinstCra_ has joined #openstack-neutron01:41
*** EinstCr__ has joined #openstack-neutron01:44
*** EinstCrazy has quit IRC01:44
*** s3wong has quit IRC01:46
*** EinstCra_ has quit IRC01:46
*** banix has joined #openstack-neutron01:47
*** ivar-lazzaro has quit IRC01:48
openstackgerritMerged openstack/neutron: objects: stop using internal _context attribute  https://review.openstack.org/28361601:50
*** reedip has joined #openstack-neutron01:52
*** gangil has quit IRC01:53
*** yamahata has quit IRC01:53
*** lucas has quit IRC01:53
*** wolverineav has quit IRC01:53
*** iyamahat has quit IRC01:54
*** reedip has quit IRC01:55
*** honghuixiao_ has quit IRC01:57
*** xiaohhui has joined #openstack-neutron01:58
*** thorst has quit IRC01:59
*** thorst has joined #openstack-neutron02:00
*** akshai has quit IRC02:01
*** djan_ has joined #openstack-neutron02:01
*** gvrangan_ has quit IRC02:02
*** djan has quit IRC02:03
*** tbachman has joined #openstack-neutron02:04
*** pai15 has joined #openstack-neutron02:05
*** mickeys has quit IRC02:06
*** mickeys has joined #openstack-neutron02:06
*** mickeys has quit IRC02:06
pai15hello, question regarding https://bugs.launchpad.net/neutron/+bug/1414640 - looks like if a new external subnet was added, the link local router to the new external subnect does not get added to existing routers02:06
openstackLaunchpad bug 1414640 in neutron "HA routers don't implement extra_subnets / multiple subnets per external network" [Medium,Fix released] - Assigned to yong sheng gong (gongysh)02:06
*** mickeys has joined #openstack-neutron02:07
pai15new router created gets the link local to new external subnet, existing router does not get updated until there is some fip gets assigned to a vm on that network02:08
*** thorst has quit IRC02:08
pai15anyone seen that or is that a known issue?02:08
*** tonytan4ever has joined #openstack-neutron02:11
*** reedip has joined #openstack-neutron02:11
*** mickeys has quit IRC02:11
*** akshai has joined #openstack-neutron02:12
*** shashank_hegde has joined #openstack-neutron02:12
*** reedip has quit IRC02:15
*** Jabadia has quit IRC02:16
*** Nepoc has quit IRC02:21
*** Nepoc has joined #openstack-neutron02:21
openstackgerritMerged openstack/neutron: Tox: Remove neutron/openstack/common from excludes list  https://review.openstack.org/32642902:25
*** lucas has joined #openstack-neutron02:26
*** yushiro has quit IRC02:26
*** mickeys has joined #openstack-neutron02:27
*** yamamoto_ has joined #openstack-neutron02:29
*** tbachman has quit IRC02:31
*** mickeys has quit IRC02:31
*** tbachman_ has joined #openstack-neutron02:31
*** amotoki has joined #openstack-neutron02:32
*** ramishra_ has joined #openstack-neutron02:33
*** jckasper has joined #openstack-neutron02:34
*** moshele has joined #openstack-neutron02:34
*** reedip has joined #openstack-neutron02:34
*** reedip has quit IRC02:35
*** ramishra has quit IRC02:36
*** amotoki has quit IRC02:37
*** jckasper has quit IRC02:39
*** jckasper has joined #openstack-neutron02:40
*** singhj has quit IRC02:41
*** jckasper has quit IRC02:41
*** tbachman_ is now known as tbachman02:44
*** jckasper has joined #openstack-neutron02:45
*** jckasper has quit IRC02:45
*** openstackgerrit has quit IRC02:47
*** openstackgerrit has joined #openstack-neutron02:48
*** grassazz has joined #openstack-neutron02:51
*** shashank_hegde has quit IRC02:52
*** ihrachys has joined #openstack-neutron02:53
*** azbiswas has joined #openstack-neutron02:53
*** dave-mccowan has quit IRC02:54
*** jckasper has joined #openstack-neutron02:55
*** pai15 has quit IRC02:55
*** iyamahat has joined #openstack-neutron02:56
*** nplanel has joined #openstack-neutron02:56
*** pai15 has joined #openstack-neutron02:57
*** azbiswas has quit IRC02:58
*** ihrachys has quit IRC02:59
*** moshele has quit IRC02:59
*** shashank_hegde has joined #openstack-neutron03:03
*** gangil has joined #openstack-neutron03:03
*** rossella_s has quit IRC03:04
*** thorst has joined #openstack-neutron03:06
*** pai15 has quit IRC03:07
*** jckasper has quit IRC03:08
*** jckasper has joined #openstack-neutron03:08
*** jckasper has quit IRC03:09
*** annp has joined #openstack-neutron03:09
*** EinstCr__ has quit IRC03:13
*** thorst has quit IRC03:13
*** jckasper has joined #openstack-neutron03:13
*** shashank_hegde has quit IRC03:17
*** pai15 has joined #openstack-neutron03:19
*** fragatina has quit IRC03:19
*** pai15 has quit IRC03:22
*** fnaval_ has joined #openstack-neutron03:23
*** fnaval has quit IRC03:25
openstackgerritbin proposed openstack/neutron: Refactor DNS integration out of DB core plugin  https://review.openstack.org/31329103:25
*** shashank_hegde has joined #openstack-neutron03:25
openstackgerritMerged openstack/neutron: Match filter criteria when constructing URI for href based iteration  https://review.openstack.org/32671103:26
*** pai15 has joined #openstack-neutron03:26
*** grassazz has quit IRC03:27
*** grassaSS has joined #openstack-neutron03:28
*** tbachman has quit IRC03:29
*** dileepr has quit IRC03:30
*** pai15 has quit IRC03:30
*** amotoki has joined #openstack-neutron03:34
*** links has joined #openstack-neutron03:35
*** amotoki has quit IRC03:38
*** azbiswas has joined #openstack-neutron03:40
*** absubram has joined #openstack-neutron03:41
*** lucas has quit IRC03:42
*** shashank_hegde has quit IRC03:43
*** EinstCrazy has joined #openstack-neutron03:44
*** gangil has quit IRC03:44
*** azbiswas has quit IRC03:44
*** julim has quit IRC03:45
*** jhershbe has joined #openstack-neutron03:46
*** EinstCrazy has quit IRC03:47
*** yushiro has joined #openstack-neutron03:52
*** yushiro has left #openstack-neutron03:53
*** yushiro has quit IRC03:53
*** yushiro has joined #openstack-neutron03:53
*** vikasc has joined #openstack-neutron03:55
*** padkrish has joined #openstack-neutron04:00
*** shashank_hegde has joined #openstack-neutron04:05
*** tonytan_brb has joined #openstack-neutron04:07
*** tonytan4ever has quit IRC04:08
*** gangil has joined #openstack-neutron04:08
*** gangil has quit IRC04:08
*** gangil has joined #openstack-neutron04:08
*** claudiub has quit IRC04:10
*** jckasper has quit IRC04:11
*** thorst has joined #openstack-neutron04:11
*** jckasper has joined #openstack-neutron04:11
*** jckasper has quit IRC04:12
openstackgerritIWAMOTO Toshihiro proposed openstack/neutron: OVS-agent: Switch the default to "native" of_interface  https://review.openstack.org/31977004:13
*** jckasper has joined #openstack-neutron04:16
*** wolverineav has joined #openstack-neutron04:17
*** fragatina has joined #openstack-neutron04:17
*** thorst has quit IRC04:18
*** wolverineav has quit IRC04:21
*** fragatina has quit IRC04:22
*** sudipto has joined #openstack-neutron04:22
*** EinstCrazy has joined #openstack-neutron04:22
*** wolverineav has joined #openstack-neutron04:23
*** salv-orl_ has joined #openstack-neutron04:24
*** salv-orlando has quit IRC04:27
openstackgerritKevin Benton proposed openstack/neutron: Support for MultipleExceptions in db_api decorators  https://review.openstack.org/31909004:29
openstackgerritKevin Benton proposed openstack/neutron: Convert multiple exception types in the API  https://review.openstack.org/31907304:29
openstackgerritKevin Benton proposed openstack/neutron: Add revision numbers to standard attribute objects  https://review.openstack.org/30396604:29
*** fnaval_ has quit IRC04:30
*** djan has joined #openstack-neutron04:32
*** grassaSS has quit IRC04:33
*** prateek has joined #openstack-neutron04:33
*** prateek has quit IRC04:33
*** Sukhdev has joined #openstack-neutron04:33
*** baoli has quit IRC04:34
*** djan_ has quit IRC04:34
*** amotoki has joined #openstack-neutron04:34
*** singhj has joined #openstack-neutron04:34
*** tonytan_brb has quit IRC04:36
*** fragatina has joined #openstack-neutron04:38
*** amotoki has quit IRC04:39
*** tflynn has joined #openstack-neutron04:40
*** fragatina has quit IRC04:40
*** tflynn has quit IRC04:40
*** fragatina has joined #openstack-neutron04:40
*** tflynn has joined #openstack-neutron04:41
*** fragatin_ has joined #openstack-neutron04:41
*** gangil has quit IRC04:42
openstackgerritYu Jie proposed openstack/neutron: Improve tunnel_sync  https://review.openstack.org/32681804:43
*** jhershbe has quit IRC04:43
*** fragatina has quit IRC04:44
*** azbiswas has joined #openstack-neutron04:45
*** Alex_Stef has joined #openstack-neutron04:46
*** ratailor has joined #openstack-neutron04:49
*** fzdarsky_ has quit IRC04:49
*** azbiswas has quit IRC04:50
*** banix has quit IRC04:50
*** vishwanathj has joined #openstack-neutron04:51
*** njohnston is now known as njohnston|sleep04:53
*** fnaval has joined #openstack-neutron04:53
*** yamahata has joined #openstack-neutron04:55
*** iyamahat has quit IRC04:56
*** shashank_hegde has quit IRC05:02
*** salv-orl_ has quit IRC05:03
*** salv-orlando has joined #openstack-neutron05:03
*** tonytan4ever has joined #openstack-neutron05:03
*** tbachman has joined #openstack-neutron05:04
*** sudipto has quit IRC05:05
*** gangil has joined #openstack-neutron05:05
*** saggi has joined #openstack-neutron05:08
*** tonytan4ever has quit IRC05:09
*** oanson has joined #openstack-neutron05:09
*** singhj has quit IRC05:09
*** tbachman has quit IRC05:11
*** saggi has quit IRC05:14
*** thorst has joined #openstack-neutron05:16
*** itzikb has joined #openstack-neutron05:16
*** tflynn has quit IRC05:17
*** tflynn has joined #openstack-neutron05:17
*** prateek has joined #openstack-neutron05:18
*** shashank_hegde has joined #openstack-neutron05:18
*** manikanta_ has joined #openstack-neutron05:18
*** moshele has joined #openstack-neutron05:20
*** iyamahat has joined #openstack-neutron05:23
*** thorst has quit IRC05:23
*** hdaniel has joined #openstack-neutron05:25
*** gvrangan has joined #openstack-neutron05:27
*** oferby has joined #openstack-neutron05:28
*** tflynn has quit IRC05:28
*** iyamahat has quit IRC05:29
*** fawadkhaliq has joined #openstack-neutron05:33
*** tonytan4ever has joined #openstack-neutron05:33
*** obondarev has joined #openstack-neutron05:34
openstackgerritHong Hui Xiao proposed openstack/neutron: Enable create and delete segments in ML2  https://review.openstack.org/31735805:34
openstackgerritHong Hui Xiao proposed openstack/neutron: Deprecate the functions map after segment moving out of ml2  https://review.openstack.org/29517305:34
openstackgerritHong Hui Xiao proposed openstack/neutron: Use callback to update SegmentHostMapping  https://review.openstack.org/32682905:34
*** obondarev has quit IRC05:35
*** amotoki has joined #openstack-neutron05:35
*** fragatin_ has quit IRC05:36
*** padkrish has quit IRC05:38
*** tonytan4ever has quit IRC05:38
*** amotoki has quit IRC05:39
*** azbiswas has joined #openstack-neutron05:41
*** mohankumar has joined #openstack-neutron05:42
*** pgadiya has joined #openstack-neutron05:48
*** iwamoto has joined #openstack-neutron05:48
*** sudipto has joined #openstack-neutron05:51
*** EinstCrazy has quit IRC05:59
*** EinstCrazy has joined #openstack-neutron05:59
*** EinstCrazy has quit IRC05:59
*** jhershbe has joined #openstack-neutron06:00
*** obondarev has joined #openstack-neutron06:03
gsagieAnyone involved with working on neutron-lib is here by any chance?06:04
*** yushiro has quit IRC06:04
*** andreas_s has joined #openstack-neutron06:04
*** yushiro has joined #openstack-neutron06:04
*** EinstCrazy has joined #openstack-neutron06:05
*** djan has quit IRC06:05
*** djan has joined #openstack-neutron06:06
*** sridharg has joined #openstack-neutron06:06
openstackgerritNam Nguyen Hoai proposed openstack/neutron: Preventing overlap CIDR on *one* network in case neutron active-active  https://review.openstack.org/31405406:08
openstackgerritGal Sagie proposed openstack/neutron-lib: Add Neutron L3 agent types  https://review.openstack.org/32683906:09
*** lifeless has quit IRC06:13
*** rcernin has joined #openstack-neutron06:13
openstackgerritItzik Brown proposed openstack/neutron: qos basic scenario  https://review.openstack.org/32588906:15
*** mickeys has joined #openstack-neutron06:15
*** mickeys has quit IRC06:16
*** mickeys has joined #openstack-neutron06:17
*** manikanta_ has quit IRC06:18
*** korzen has joined #openstack-neutron06:19
*** abregman|nb has quit IRC06:19
*** mickeys has quit IRC06:21
*** apoorv has joined #openstack-neutron06:21
*** thorst has joined #openstack-neutron06:21
*** obondarev has quit IRC06:23
*** Alex_Stef has quit IRC06:23
*** gangil has quit IRC06:26
*** thorst has quit IRC06:29
*** vikram has joined #openstack-neutron06:29
*** faizy has joined #openstack-neutron06:31
openstackgerritGal Sagie proposed openstack/neutron: Move L3 agent type constants to use neutron-lib  https://review.openstack.org/32685006:31
*** Sukhdev has quit IRC06:33
*** vishwanathj has quit IRC06:36
*** vishwanathj has joined #openstack-neutron06:36
*** azbiswas has quit IRC06:37
*** jhershbe has quit IRC06:38
*** kawa2014 has joined #openstack-neutron06:40
*** kobis has joined #openstack-neutron06:40
gsagiesunshine :)06:41
*** claudiub has joined #openstack-neutron06:44
vikramgaryk: ping06:46
*** gampel has joined #openstack-neutron06:46
*** openstackgerrit has quit IRC06:48
*** ajmiller_ has quit IRC06:48
*** mkolesni has joined #openstack-neutron06:48
*** ajmiller_ has joined #openstack-neutron06:48
*** openstackgerrit has joined #openstack-neutron06:49
*** jpena|off is now known as jpena06:50
vikramgaryk: Can you check the doubt for https://review.openstack.org/#/c/325844/06:52
*** mohankumar has quit IRC06:56
*** itamarl has joined #openstack-neutron06:57
*** jhershbe has joined #openstack-neutron06:57
openstackgerritChangBo Guo(gcb) proposed openstack/neutron: Use is_valid_port from oslo.utils  https://review.openstack.org/32686306:58
*** martinus__ has joined #openstack-neutron06:58
*** EinstCra_ has joined #openstack-neutron06:59
*** gampel has left #openstack-neutron06:59
*** fzdarsky_ has joined #openstack-neutron06:59
*** tesseract has joined #openstack-neutron07:00
*** shihanzhang has joined #openstack-neutron07:01
openstackgerritMerged openstack/neutron: Make service plugins' get_plugin_type classmethods  https://review.openstack.org/32671607:01
*** EinstCrazy has quit IRC07:02
*** salv-orlando has quit IRC07:02
*** ajmiller_ has quit IRC07:03
*** ajmiller has joined #openstack-neutron07:03
*** salv-orlando has joined #openstack-neutron07:03
*** EinstCra_ has quit IRC07:05
*** EinstCrazy has joined #openstack-neutron07:05
*** saggi has joined #openstack-neutron07:07
*** dane_leblanc has quit IRC07:10
*** dane_leblanc has joined #openstack-neutron07:10
openstackgerritLiuYong proposed openstack/neutron: Validate '-F' parameter for 'net-ip-availability-list' command  https://review.openstack.org/32316807:16
*** roeyc has joined #openstack-neutron07:16
*** jschwarz has joined #openstack-neutron07:17
*** tinbar has joined #openstack-neutron07:17
*** garyk1 has joined #openstack-neutron07:17
*** eddima has joined #openstack-neutron07:18
*** hynekm has joined #openstack-neutron07:18
*** watanabe_isao has joined #openstack-neutron07:20
*** yfried has joined #openstack-neutron07:21
*** abregman|mtg has joined #openstack-neutron07:23
*** fawadkhaliq has quit IRC07:23
*** mikelk has joined #openstack-neutron07:25
*** thorst has joined #openstack-neutron07:26
*** matrohon has joined #openstack-neutron07:29
*** jlanoux has joined #openstack-neutron07:29
*** jlibosva has joined #openstack-neutron07:29
*** watanabe_isao has quit IRC07:31
*** sleviim has joined #openstack-neutron07:31
*** salv-orlando has quit IRC07:32
*** salv-orlando has joined #openstack-neutron07:32
*** thorst has quit IRC07:33
*** obondarev has joined #openstack-neutron07:35
*** ajmiller has quit IRC07:35
*** mohankumar_ has joined #openstack-neutron07:36
*** amotoki has joined #openstack-neutron07:36
*** djan has quit IRC07:38
*** amotoki has quit IRC07:41
*** gvrangan has quit IRC07:43
*** rossella_s has joined #openstack-neutron07:44
*** manikanta_ has joined #openstack-neutron07:45
*** fzdarsky_ has quit IRC07:47
*** openstackgerrit has quit IRC07:48
*** openstackgerrit has joined #openstack-neutron07:49
*** tmorin has joined #openstack-neutron07:50
*** jlanoux_ has joined #openstack-neutron07:50
*** jlanoux has quit IRC07:54
*** yushiro has quit IRC07:56
*** pai15 has joined #openstack-neutron07:57
*** nmagnezi has joined #openstack-neutron07:57
*** bcafarel has joined #openstack-neutron07:58
*** zzzeek has quit IRC08:00
*** chandankumar has joined #openstack-neutron08:00
*** Alex_Stef has joined #openstack-neutron08:00
*** pai15 has quit IRC08:01
*** zzzeek has joined #openstack-neutron08:02
*** numans has quit IRC08:04
*** ajmiller has joined #openstack-neutron08:05
*** s1061123 has joined #openstack-neutron08:06
*** ushkalim has joined #openstack-neutron08:06
openstackgerritItzik Brown proposed openstack/neutron: qos basic scenario  https://review.openstack.org/32588908:08
*** fawadkhaliq has joined #openstack-neutron08:09
openstackgerritDavanum Srinivas (dims) proposed openstack/neutron: [WIP] Testing latest u-c  https://review.openstack.org/30334608:10
*** salv-orlando has quit IRC08:10
*** ajmiller_ has joined #openstack-neutron08:10
*** lucas-dinner is now known as lucasagomes08:10
*** salv-orlando has joined #openstack-neutron08:11
*** ygbo has joined #openstack-neutron08:11
*** ajmiller has quit IRC08:14
*** vishwanathj has quit IRC08:14
*** EinstCrazy has quit IRC08:15
*** tflynn has joined #openstack-neutron08:16
*** abregman|mtg has quit IRC08:16
*** yamahata has quit IRC08:16
*** EinstCrazy has joined #openstack-neutron08:18
*** claudiub has quit IRC08:20
openstackgerritSongming Yan proposed openstack/python-neutronclient: Fix the problem of "qos-bandwidth-limit-rule-show"  https://review.openstack.org/32690208:25
*** davidsha has joined #openstack-neutron08:26
*** tflynn_ has joined #openstack-neutron08:27
*** abregman|mtg has joined #openstack-neutron08:28
*** tflynn has quit IRC08:31
*** jhershbe has quit IRC08:32
*** tflynn_ has quit IRC08:35
*** yfried has quit IRC08:35
*** haukebruno has joined #openstack-neutron08:35
*** dmk0202 has joined #openstack-neutron08:37
*** amotoki has joined #openstack-neutron08:37
ajo_nmagnezi, can you have an eye on https://bugs.launchpad.net/neutron/+bug/1590310  ?08:38
openstackLaunchpad bug 1590310 in neutron "Reduce excessive LBaaSv2 logging" [Low,New] - Assigned to Nir Magnezi (nmagnezi)08:38
*** neiljerram has joined #openstack-neutron08:38
ajo_please ? :)08:38
ajo_just to confirm at least, it's low prio08:38
*** abregman|mtg is now known as abregman08:38
ajo_an it's related to lbaasv2 + haproxy08:38
*** neiljerram has quit IRC08:38
openstackgerritzhurong proposed openstack/python-neutronclient: Fix typo  https://review.openstack.org/32691708:38
*** neiljerram has joined #openstack-neutron08:40
*** amotoki has quit IRC08:42
*** EinstCrazy has quit IRC08:43
*** EinstCrazy has joined #openstack-neutron08:43
*** EinstCrazy has quit IRC08:45
*** fawadkhaliq has quit IRC08:45
nmagneziajo_, i will check that, but it will take me time to handle it due to other tasks08:47
ajo_nmagnezi, we just need some triaging when possible, I just know that you had haproxy driver on-hand ;)08:47
*** EinstCrazy has joined #openstack-neutron08:47
ajo_it's just creating a load balancer with no listener, and looking to the logs :D anytime will be good08:47
ajo_nmagnezi, but it's low prio, so ... of course, take priority into consideration :)08:48
*** openstackgerrit has quit IRC08:48
nmagneziajo_, :-)08:48
*** openstackgerrit has joined #openstack-neutron08:49
*** yfried has joined #openstack-neutron08:49
*** jhershbe has joined #openstack-neutron08:50
*** wolverin_ has joined #openstack-neutron08:50
*** nyechiel has joined #openstack-neutron08:51
*** saggi has quit IRC08:51
*** wolverin_ has quit IRC08:51
*** wolverineav has quit IRC08:52
openstackgerritKevin Benton proposed openstack/neutron: Support for MultipleExceptions in db_api decorators  https://review.openstack.org/31909008:54
openstackgerritKevin Benton proposed openstack/neutron: Convert multiple exception types in the API  https://review.openstack.org/31907308:54
openstackgerritKevin Benton proposed openstack/neutron: Add revision numbers to standard attribute objects  https://review.openstack.org/30396608:54
openstackgerritKevin Benton proposed openstack/neutron: Check for StaleData errors in retry decorator  https://review.openstack.org/32692608:54
openstackgerritKevin Benton proposed openstack/neutron: Check for mysql SAVEPOINT error in retry decorator  https://review.openstack.org/32692708:54
openstackgerritKevin Benton proposed openstack/neutron: Re-raise retriable DB errors in callback exc handling  https://review.openstack.org/32692808:54
*** vishwanathj has joined #openstack-neutron08:55
*** garyk has quit IRC08:59
*** garyk has joined #openstack-neutron08:59
*** garyk has quit IRC09:03
*** mvk has quit IRC09:03
*** mickeys has joined #openstack-neutron09:04
*** numans has joined #openstack-neutron09:06
*** fzdarsky_ has joined #openstack-neutron09:07
*** fzdarsky_ is now known as fzdarsky09:07
*** garyk has joined #openstack-neutron09:08
openstackgerritbin proposed openstack/neutron: Refactor DNS integration out of DB core plugin  https://review.openstack.org/31329109:08
openstackgerritYu Jie proposed openstack/neutron: Improve tunnel_sync  https://review.openstack.org/32681809:08
*** lujinluo has quit IRC09:10
*** chandankumar has quit IRC09:11
*** chandankumar has joined #openstack-neutron09:12
*** EinstCrazy has quit IRC09:18
*** yushiro has joined #openstack-neutron09:19
*** shashank_hegde has quit IRC09:20
*** EinstCrazy has joined #openstack-neutron09:21
*** roeyc has quit IRC09:22
*** ajmiller__ has joined #openstack-neutron09:22
*** azbiswas has joined #openstack-neutron09:23
openstackgerritgaryk proposed openstack/neutron-lib: A translation validations to the hacking policy  https://review.openstack.org/32586509:23
*** mickeys has quit IRC09:25
*** mickeys has joined #openstack-neutron09:25
*** mickeys has quit IRC09:26
*** ajmiller_ has quit IRC09:26
*** azbiswas has quit IRC09:27
*** vikasc has quit IRC09:30
*** manikanta_ has quit IRC09:30
*** fawadkhaliq has joined #openstack-neutron09:32
*** fawadkhaliq has quit IRC09:33
*** fawadkhaliq has joined #openstack-neutron09:33
*** fawadkhaliq has quit IRC09:33
*** mvk has joined #openstack-neutron09:34
*** amotoki has joined #openstack-neutron09:38
*** sambetts|afk is now known as sambetts09:41
*** vikasc has joined #openstack-neutron09:42
*** mfuruta has quit IRC09:42
*** fawadkhaliq has joined #openstack-neutron09:42
*** amotoki has quit IRC09:42
*** namnh has joined #openstack-neutron09:43
*** eezhova has joined #openstack-neutron09:43
*** manikanta_ has joined #openstack-neutron09:44
*** yushiro has quit IRC09:44
namnhzzzeek: Hello Mr.Mike. I updated my patch set for IPv6. Could you please review it : https://review.openstack.org/#/c/314054/ . Thank you so much.09:45
*** roeyc has joined #openstack-neutron09:51
*** roeyc has quit IRC09:51
*** wolverineav has joined #openstack-neutron09:52
*** mhickey has joined #openstack-neutron09:52
*** roeyc has joined #openstack-neutron09:52
*** markvoelker has joined #openstack-neutron09:53
*** jhershbe has quit IRC09:55
*** amotoki has joined #openstack-neutron09:56
*** wolverineav has quit IRC09:57
*** yushiro has joined #openstack-neutron09:59
*** rossella_s has quit IRC10:02
*** sridharg has quit IRC10:03
*** john-davidge has joined #openstack-neutron10:07
*** roeyc has quit IRC10:08
*** vishwanathj has quit IRC10:08
*** iwamoto has quit IRC10:08
*** john-dav_ has joined #openstack-neutron10:09
openstackgerritDavanum Srinivas (dims) proposed openstack/neutron: [WIP] Testing latest u-c  https://review.openstack.org/30334610:10
*** john-davidge has quit IRC10:12
*** jamie_h has joined #openstack-neutron10:12
*** yfried has quit IRC10:21
*** jhershbe has joined #openstack-neutron10:21
*** matrohon has quit IRC10:21
*** john-dav_ is now known as john-davidge10:22
*** stanzgy has quit IRC10:22
*** hoangcx has quit IRC10:22
*** salv-orl_ has joined #openstack-neutron10:25
*** yushiro has quit IRC10:26
*** tmorin has quit IRC10:27
*** salv-orlando has quit IRC10:28
*** yushiro has joined #openstack-neutron10:30
*** chandankumar has quit IRC10:31
openstackgerritHong Hui Xiao proposed openstack/neutron: Return fixed_ips from port update  https://review.openstack.org/32115210:33
*** salv-orl_ has quit IRC10:34
*** salv-orlando has joined #openstack-neutron10:34
*** lifeless has joined #openstack-neutron10:35
*** garyk has quit IRC10:39
*** eezhova has quit IRC10:39
*** garyk has joined #openstack-neutron10:39
openstackgerritAndreas Scheuring proposed openstack/neutron: [WIP] Extend DB schema for unify portbinding for dvr and normal ports  https://review.openstack.org/32698510:40
openstackgerritAndreas Scheuring proposed openstack/neutron: [WIP] Remove vif_type & details from normal port db  https://review.openstack.org/32698610:40
openstackgerritAndreas Scheuring proposed openstack/neutron: [WIP] unify portbinding for normal and dvr ports  https://review.openstack.org/32698710:41
*** sdague has joined #openstack-neutron10:42
*** andreas_s has quit IRC10:42
*** javeriak has joined #openstack-neutron10:43
*** vikram has quit IRC10:43
*** yamamoto_ has quit IRC10:43
*** garyk has quit IRC10:44
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Refactor the rpc callback version discovery mechanism.  https://review.openstack.org/32285710:44
*** garyk has joined #openstack-neutron10:45
Alex_Stefkevinbenton, ping10:46
*** namnh has quit IRC10:47
*** lifeless has quit IRC10:47
*** lifeless_ has joined #openstack-neutron10:47
*** jschwarz has left #openstack-neutron10:48
*** jschwarz has joined #openstack-neutron10:48
*** yfried has joined #openstack-neutron10:49
*** slunkad has joined #openstack-neutron10:49
*** tesseract has quit IRC10:50
*** rtheis has joined #openstack-neutron10:50
*** tesseract has joined #openstack-neutron10:51
*** numans has quit IRC10:54
*** rossella_s has joined #openstack-neutron10:58
openstackgerritDaniel Berrange proposed openstack/os-vif: os_vif: register objects before loading plugins  https://review.openstack.org/32699611:01
openstackgerritDaniel Berrange proposed openstack/os-vif: os_vif: add logging for each plugin that is loaded  https://review.openstack.org/32699711:01
openstackgerritDaniel Berrange proposed openstack/os-vif: vif_plug_ovs: clarify that the plugin was not in fact renamed  https://review.openstack.org/32699811:01
*** ihrachys has joined #openstack-neutron11:02
*** ihrachys has quit IRC11:02
*** ihrachys has joined #openstack-neutron11:02
*** obondarev has quit IRC11:02
*** roeyc has joined #openstack-neutron11:03
ihrachysajo_: nice latest version of rpc callbacks patch. I left some nitty comments, overall good.11:03
ajo_ihrachys, thanks, let me check comments now11:03
*** roeyc has quit IRC11:03
*** banix has joined #openstack-neutron11:04
ajo_dammit, I must proofread my documents and "english texts" ':) thanks ihrachys11:04
ihrachysthat's easier to offload it to reviewers :)11:05
ihrachysit's hard to spot typos in your own text11:05
*** afazekas is now known as __afazekas11:05
jschwarzihrachys, aye I'm aware11:07
jschwarzihrachys, apologies, I'm swamped with work :<11:07
ihrachysjschwarz: sounds like my life11:08
*** doug-fish has joined #openstack-neutron11:09
jschwarzihrachys, hopefully I might have time soon, but unfortunately I'm going on PTO for 3 weeks soon for exams :<11:10
ihrachysthat's ok. if it does not happen, I won't cry11:10
ihrachyssomeone really interested in it can always chime in and take over11:10
ajo_ihrachys, I could have detected many of those typos with a simple read, since they came from altering existing senteces more than ignorance (and I'm plenty of that)11:11
*** fawadkhaliq has quit IRC11:12
fricklerso why would booting an instance lower the MTU for the integration bridge to 1450? (see https://bugs.launchpad.net/neutron/+bug/1376446/comments/11 for the context)11:12
openstackLaunchpad bug 1376446 in neutron "Neutron LBAAS agent ignores MTU settings used by subnets" [Medium,In progress] - Assigned to Dr. Jens Rosenboom (j-rosenboom-j)11:12
*** banix has quit IRC11:13
*** sudipto has quit IRC11:14
*** annp has quit IRC11:18
*** fawadkhaliq has joined #openstack-neutron11:19
*** vikram has joined #openstack-neutron11:20
*** roeyc has joined #openstack-neutron11:20
*** chandankumar has joined #openstack-neutron11:23
*** javeriak has quit IRC11:23
*** javeriak has joined #openstack-neutron11:23
*** fawadkhaliq has quit IRC11:25
*** garyk2 has joined #openstack-neutron11:28
*** garyk has quit IRC11:28
*** yamamoto has joined #openstack-neutron11:33
*** garyk2 has quit IRC11:33
*** garyk has joined #openstack-neutron11:34
*** yamamoto_ has joined #openstack-neutron11:34
*** lucasagomes is now known as lucas-afk11:35
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Refactor the rpc callback version discovery mechanism  https://review.openstack.org/32285711:36
ajo_ihrachys, there it goes  ^11:36
* ajo_ goes for quick lunch11:36
openstackgerritItzik Brown proposed openstack/neutron: qos basic scenario  https://review.openstack.org/32588911:37
*** fawadkhaliq has joined #openstack-neutron11:38
*** yamamoto has quit IRC11:38
garyk1ajo_: eat slowly and run fats!!!11:38
garyk1fast11:38
*** baoli has joined #openstack-neutron11:40
fricklerso this looks indeed like a bug to me: https://bugs.launchpad.net/neutron/+bug/1590397 but I haven't yet found out where that MTU change is happening, if someone could have a look that would be great11:40
openstackLaunchpad bug 1590397 in neutron "MTU setting too low when mixing Vlan and Vxlan" [Undecided,New]11:40
*** fawadkhaliq has quit IRC11:40
*** fawadkhaliq has joined #openstack-neutron11:40
*** jpena is now known as jpena|lunch11:40
*** doug-fish has quit IRC11:41
*** baoli_ has joined #openstack-neutron11:41
*** tpsilva has joined #openstack-neutron11:41
*** obondarev has joined #openstack-neutron11:43
*** banix has joined #openstack-neutron11:43
*** fawadk has joined #openstack-neutron11:43
*** javeriak_ has joined #openstack-neutron11:43
*** davidsha has quit IRC11:43
*** fawadkhaliq has quit IRC11:44
*** fawadkhaliq has joined #openstack-neutron11:44
*** javeria__ has joined #openstack-neutron11:44
*** javeriak has quit IRC11:44
*** baoli has quit IRC11:45
*** banix has quit IRC11:46
*** javeriak_ has quit IRC11:48
*** fawadk has quit IRC11:48
*** __afazekas is now known as afazekas11:49
*** jbell8 has quit IRC11:50
*** jbell8 has joined #openstack-neutron11:51
*** eezhova has joined #openstack-neutron11:51
*** c64cosmin has joined #openstack-neutron11:52
*** jhershbe has quit IRC11:53
*** faizy has quit IRC11:53
*** jhershbe has joined #openstack-neutron11:54
*** jbell8 has quit IRC11:55
*** markvoelker has quit IRC11:56
*** jbell8 has joined #openstack-neutron11:56
*** sridharg has joined #openstack-neutron11:56
*** salv-orlando has quit IRC11:59
*** amuller has joined #openstack-neutron11:59
*** salv-orlando has joined #openstack-neutron12:00
*** davidsha has joined #openstack-neutron12:02
*** mohankumar_ has quit IRC12:03
*** mohankumar_ has joined #openstack-neutron12:03
*** fawadkhaliq has quit IRC12:03
*** sridharg has quit IRC12:09
*** ihrachys has quit IRC12:10
*** ihrachys has joined #openstack-neutron12:10
*** markvoelker has joined #openstack-neutron12:12
*** andreas_s_ has joined #openstack-neutron12:12
*** hynekm has quit IRC12:13
*** apoorv has quit IRC12:13
*** EinstCrazy has quit IRC12:14
*** EinstCrazy has joined #openstack-neutron12:14
*** frontrunner has joined #openstack-neutron12:14
*** salv-orl_ has joined #openstack-neutron12:15
*** fnaval_ has joined #openstack-neutron12:15
*** itamarl has quit IRC12:16
*** fnaval has quit IRC12:19
*** EinstCrazy has quit IRC12:19
*** salv-orlando has quit IRC12:19
*** aurelien__ has joined #openstack-neutron12:20
*** aurelien__ has quit IRC12:21
*** fawadkhaliq has joined #openstack-neutron12:22
openstackgerritgaryk proposed openstack/neutron: Hacking: make use of neutron lib  https://review.openstack.org/32706012:23
*** sridharg has joined #openstack-neutron12:23
*** banix has joined #openstack-neutron12:24
*** yamamoto_ has quit IRC12:26
*** fawadkhaliq has quit IRC12:26
*** vikasc has quit IRC12:27
*** slunkad has quit IRC12:27
*** vikram has quit IRC12:28
*** yamamoto has joined #openstack-neutron12:28
*** slunkad has joined #openstack-neutron12:29
*** yamamoto has quit IRC12:30
*** pradk has quit IRC12:30
openstackgerritHong Hui Xiao proposed openstack/neutron: Mark dhcp port as ready after enabling dhcp at agent  https://review.openstack.org/32706212:31
*** eezhova has quit IRC12:32
*** Nepoc has quit IRC12:33
*** itamarl has joined #openstack-neutron12:33
*** fawadkhaliq has joined #openstack-neutron12:33
*** Nepoc has joined #openstack-neutron12:34
*** annemccormick has joined #openstack-neutron12:35
openstackgerritMerged openstack/neutron: Revert "Revert "Remove threading before process forking""  https://review.openstack.org/31327712:35
*** rossella_s has quit IRC12:36
*** annemccormick has quit IRC12:37
*** javeria__ has quit IRC12:39
*** pgadiya has quit IRC12:40
*** vishwanathj has joined #openstack-neutron12:41
*** singhj has joined #openstack-neutron12:42
*** rossella_s has joined #openstack-neutron12:42
*** lucas-afk is now known as lucasagomes12:44
*** thorst has joined #openstack-neutron12:45
*** neil_ has joined #openstack-neutron12:46
*** obondarev has quit IRC12:47
jschwarzihrachys, ping, re backports of bugs12:47
jschwarzihrachys, I've commented on 2 of the 3 patches' launchpads with the effects of the current code and how they solve the race conditions12:48
jschwarzI'm not completely sure we *must* have the 3rd fix though. I need to look into it and remember what it does first12:48
jschwarz(i didn't write that one so...)12:49
*** hynekm has joined #openstack-neutron12:49
*** thorst has quit IRC12:49
*** pai15 has joined #openstack-neutron12:49
*** jpena|lunch is now known as jpena12:49
*** neiljerram has quit IRC12:49
*** eezhova has joined #openstack-neutron12:49
*** eezhova has quit IRC12:51
*** eezhova has joined #openstack-neutron12:51
openstackgerritNate Johnston proposed openstack/neutron: QoS DSCP fullstack tests  https://review.openstack.org/28839212:51
*** sridharg has quit IRC12:53
*** manikanta_ has quit IRC12:53
*** sridharg has joined #openstack-neutron12:53
*** gampel has joined #openstack-neutron12:54
*** oferb has joined #openstack-neutron12:54
*** julim has joined #openstack-neutron12:55
*** sudipto has joined #openstack-neutron12:55
*** singhj has quit IRC12:55
*** rodrigods has quit IRC12:56
*** rodrigods has joined #openstack-neutron12:56
*** oferby has quit IRC12:57
openstackgerritJakub Libosvar proposed openstack/neutron: objects: switch base plugin class to using subnetpool object  https://review.openstack.org/30005612:58
*** jhershbe has quit IRC12:58
*** pai15 has quit IRC12:58
*** jlibosva has quit IRC12:58
*** jlibosva has joined #openstack-neutron13:00
*** thorst has joined #openstack-neutron13:00
*** edmondsw has joined #openstack-neutron13:00
*** claudiub has joined #openstack-neutron13:00
*** azbiswas has joined #openstack-neutron13:00
openstackgerritItzik Brown proposed openstack/neutron: qos basic scenario  https://review.openstack.org/32588913:01
*** oanson has quit IRC13:01
*** gouthamr has joined #openstack-neutron13:02
*** links has quit IRC13:03
*** fawadkhaliq has quit IRC13:04
*** azbiswas has quit IRC13:05
*** efoley has joined #openstack-neutron13:06
*** otherwiseguy has joined #openstack-neutron13:09
openstackgerritIhar Hrachyshka proposed openstack/neutron: tests: added sorting/pagination api tests for subnetpools  https://review.openstack.org/32708113:09
ihrachysjlibosva: ^ subnetpools api tests for sorting/pagination13:10
openstackgerritAkihiro Motoki proposed openstack/neutron: Drop neutron/i18n.py in favor of neutron/_i18n.py  https://review.openstack.org/32077213:10
ihrachysouch, I need to change uuids..13:10
ihrachysajo_: where does the patch that was supposed to catch uuid duplicates for api tests stand?13:11
*** obondarev has joined #openstack-neutron13:13
ihrachysI mean https://review.openstack.org/#/c/300483/13:14
openstackgerritIhar Hrachyshka proposed openstack/neutron: tests: added sorting/pagination api tests for subnetpools  https://review.openstack.org/32708113:14
*** tbachman has joined #openstack-neutron13:17
*** gampel has left #openstack-neutron13:18
jlibosvaihrachys: ack, I'll rebase mine13:18
ihrachysjlibosva: maybe better to Depends-On13:18
ihrachysjlibosva: I don't expect clashes in code13:18
ihrachysand it will make rebases easier13:18
*** links has joined #openstack-neutron13:18
*** watanabe_isao has joined #openstack-neutron13:20
*** yushiro1 has joined #openstack-neutron13:20
*** yushiro1 has left #openstack-neutron13:20
*** yushiro is now known as yushiro_afk13:21
*** yushiro_afk is now known as yushiro13:22
*** chandankumar has quit IRC13:23
haleybSam-I-Am: baaa13:26
*** njohnston|sleep is now known as njohnston13:28
ihrachysajo_: I will respin the uuid patch13:29
ajo_ihrachys, which one?13:30
*** roeyc has quit IRC13:31
*** yushiro is now known as yushiro_afk13:31
*** vhoward has joined #openstack-neutron13:32
*** yushiro_afk is now known as yushiro13:32
Sam-I-Amhaleyb: moo13:32
ihrachysajo_: the one adding pep8 check for unique uuids for tempest tests13:32
haleybSam-I-Am: you must have had an mtu question13:32
*** Leo_ has joined #openstack-neutron13:33
Sam-I-Amhaleyb: well, sort of... re 32012113:33
Sam-I-Ami had a comment in ps 8 suggesting another way to approach the problem13:33
*** Leom has joined #openstack-neutron13:34
*** sridharg has quit IRC13:36
haleybSam-I-Am: yes, i punted on those just to get PS9 out (with a lot less code), but you gave two options: 1) determine IP version from first agent that registers; 2) disable agents that don't match IP version13:36
*** Leo_ has quit IRC13:37
haleybI'm all for #2, on the fence for #1, mostly because I don't know the code, but also because what if someone creates a network before an agent is running?13:38
openstackgerritIhar Hrachyshka proposed openstack/neutron: Add sanity check for idempotent id's in tempest tests  https://review.openstack.org/30048313:38
*** dane_leblanc has quit IRC13:38
*** johnbelamaric has joined #openstack-neutron13:38
Sam-I-Amhaleyb: i suppose thats a corner case. last time i tried creating a network before there was an agent, i ended up with a bunch of errors.13:39
Sam-I-Amnot during creation, but when the agent finally came up13:39
ihrachysajo_: ^ respinned13:40
ihrachysamuller: please review https://review.openstack.org/#/c/300483/5 in terms of tempest dep in test-requirements13:40
*** pradk has joined #openstack-neutron13:41
*** roeyc has joined #openstack-neutron13:42
*** janzian has joined #openstack-neutron13:43
*** bdemers has joined #openstack-neutron13:43
haleybSam-I-Am: i don't want to create new options either, but i think it would help with mis-configurations13:44
fricklerhaleyb: could you take a look at https://bugs.launchpad.net/bugs/1590397 please? It seems to me that mixing different MTUs is quite pointless currently at least for OVS deployments, not sure whether LB behaves differently13:44
openstackLaunchpad bug 1590397 in neutron "MTU setting too low when mixing Vlan and Vxlan" [Undecided,New] - Assigned to Manikantha Srinivas Tadi (manikantha-tadi)13:44
Sam-I-Amhaleyb: yeah, there's explicit config vs. implicit config. my idea avoids creation of another option, but may not hit the corner cases13:44
haleybfrickler: yes, that bug is interesting, Sam-I-Am ^^^13:47
*** jlanoux_ has quit IRC13:47
Sam-I-Amyeah, i posted a comment13:47
Sam-I-Amwhat the host reports for ovs bridge mtu can be misleading13:47
Sam-I-Amfor example, when i was testing large mtus (9000), br-int and br-tun were still 1500 and worked fine13:48
*** yushiro has quit IRC13:48
Sam-I-Amthe host also had those bridges marked as down, which is also interesting13:48
haleybSam-I-Am: does that get back to OVS as treating bridges as having 64K mtus to support jumbo?13:48
*** fnaval_ has quit IRC13:49
Sam-I-Amit gets back to ovs being wonky about mtu13:49
Sam-I-Amit says one thing according to the host, but does another13:49
haleybfrickler: did you see an error sending packets of various sizes?13:49
fricklerhaleyb: yes, the issue is that e.g. router ports get created with the low MTU anyway because it seems thats what "ovs-vsctl add-port br-int routerport" does13:50
fricklerso for my vlan based network, the instance interface has MTU 1500 but the router has 1450 and will drop packets of size 150013:50
Sam-I-Amthe only place mtu is defined is interface inside namespaces and linux bridges13:51
Sam-I-Amnot on the ovs ends of things (because there is no way to define it)13:51
fricklerthe question is what is the mtu for new interfaces, see https://bugs.launchpad.net/neutron/+bug/1376446/comments/11 for how I stumbled about this13:52
openstackLaunchpad bug 1376446 in neutron "Neutron LBAAS agent ignores MTU settings used by subnets" [Medium,In progress] - Assigned to Dr. Jens Rosenboom (j-rosenboom-j)13:52
*** roeyc has quit IRC13:52
Sam-I-Amthis patch was not merged, so the mtu of br-int/tun should not change from the default system value - https://review.openstack.org/#/c/295917/13:52
haleybfrickler: i would have thought the mtu of the network would have been min(1450, 1500) and everyone would have been 1450, at least I see dnsmasq always running with 1450 on my vxlan setup13:52
haleybso clients get that in the dhcp response13:53
*** dane_leblanc has joined #openstack-neutron13:54
Sam-I-Amfrickler: so you plug 2 different kinds of project networks into a router, the interfaces in that namespace should have the respective MTUs. br-int shouldn't matter.13:54
*** watanabe_isao has quit IRC13:54
fricklerhaleyb: yes, if you only use vxlan, that is fine, but if you want to mix in some vlan based tenant networks - e.g. because you need some external devices connected there - you'll have trouble13:54
fricklerSam-I-Am: br-int does matter, it determines the MTU of all newly created router interfaces13:55
Sam-I-Amthe interfaces in the namespace?13:55
fricklerSam-I-Am: yes, they get created via "ovs-vsctl add-port br-int routerport" and are placed into the namespace after that13:56
Sam-I-Amwhich ver of os are you using?13:56
Sam-I-Ambecause we should be setting the mtu value of the router interface based on the value of global_physnet_mtu or path_mtu (depending on the network type)13:56
fricklerI've tested with devstack on master running trusty, but I'm seeing the same with xenial/mitaka13:57
*** korzen has quit IRC13:57
*** thorst has quit IRC13:57
*** roeyc has joined #openstack-neutron13:57
Sam-I-Amdid you set the mtu options?13:57
openstackgerritLiuYong proposed openstack/neutron: Fix the problem that 'net-delete' command seems to cause dead loop  https://review.openstack.org/32461713:57
haleybfrickler: is the mtu of the tenant network showing 1450?  maybe there's some place where we forget to use it13:58
fricklerno, the MTU for vxlan based networks is automatically set to 1450 (which is correct) and the vlan ones have 1500 by default, which is also fine for me13:58
fricklerhaleyb: well, lbaas neglegts to use the network mtu, that is https://bugs.launchpad.net/neutron/+bug/137644613:59
openstackLaunchpad bug 1376446 in neutron "Neutron LBAAS agent ignores MTU settings used by subnets" [Medium,In progress] - Assigned to Dr. Jens Rosenboom (j-rosenboom-j)13:59
openstackgerritNate Johnston proposed openstack/neutron-fwaas: DevStack plugin for fwaas  https://review.openstack.org/21435013:59
haleybfrickler: yes, i know of that patch :)14:00
fricklerhaleyb: the other bug is that once there is any instance running with MTU 1450, all new router ports get that MTU, even if they are on networks with higher MTU14:00
*** Swami has joined #openstack-neutron14:01
fricklerhaleyb: and not only router ports, but also e.g. lbaas ports, which is why I was confused about the first bug for some time14:01
*** fnaval has joined #openstack-neutron14:02
*** ametts has joined #openstack-neutron14:02
Sam-I-Amfrickler: 'once there is any instance running with mtu 1450...'14:03
Sam-I-Amthis part confuses me14:03
fricklerI'll add some debug output to 1590397 in order to show the behaviour14:03
Sam-I-Aminstances dont directly touch routers14:03
haleybfrickler: ML2 needs to either pick the lowest, or support both, but there is definetly a bug here14:04
*** pai15 has joined #openstack-neutron14:05
*** p01nt3r75 has joined #openstack-neutron14:05
*** Swami_ has joined #openstack-neutron14:05
haleybSam-I-Am: i think that's because the network mtu gets set to 1450, so all elements inherit the value14:06
*** singhj has joined #openstack-neutron14:06
*** itzikb has quit IRC14:06
Sam-I-Amin this case there's two different networks... one vxlan, and one vlan, no?14:06
*** watanabe_isao has joined #openstack-neutron14:06
Sam-I-Amso each network should have a respective mtu value for its components14:07
*** rossella_s has quit IRC14:07
fricklerSam-I-Am: yes, but this isn't applied currently14:07
Sam-I-Amcurious whats in your config14:08
Sam-I-Amdevstack usually doesnt use the right options14:08
*** roeyc has quit IRC14:08
*** Swami has quit IRC14:09
*** jlibosva has quit IRC14:09
*** roeyc has joined #openstack-neutron14:09
*** jlibosva has joined #openstack-neutron14:09
openstackgerritAndreas Scheuring proposed openstack/neutron: [WIP] unify portbinding for normal and dvr ports  https://review.openstack.org/32698714:10
haleybSam-I-Am: maybe it's time to kill devstack since it's creating all these bugs :)14:10
*** andreas_s_ has quit IRC14:10
*** singhj has quit IRC14:11
Sam-I-Amhaleyb: lol14:11
Sam-I-Ami'd be game for that14:11
njohnstonrewrite devstack in go14:12
Sam-I-Ami find the networking in it just contains a bunch of hacks that dont necessarily validate 'modern' ways to config neutron14:12
haleybin a "real" deployment the data plane should be jumbo, so that we have 1500 for all the tenant networks, and avoid this mtu crap14:12
Sam-I-Amnjohnston: excellent idea. i'll post to the ML.14:12
njohnstonSam-I-Am: +100!14:12
Sam-I-Amhaleyb: well, in mitaka+, that problem is fixed14:12
Sam-I-Amyou can even use jumbo frames all the way to instances now14:13
Sam-I-Ambut unraveling years of mtu hacking has proven difficult14:13
*** kriskend has joined #openstack-neutron14:13
Sam-I-Amnone of those solutions actually resolving the real problem14:13
Sam-I-Amand many of them baking themselves into deployment tools like pupper14:13
*** rossella_s has joined #openstack-neutron14:13
Sam-I-Ampuppet14:13
*** Swami__ has joined #openstack-neutron14:14
fricklerwell, if I had all vxlan networks with MTU 9000 and need one vlan network for the outside, I'd still have the same issue. running everything at 1500 should be fine, though14:15
haleybSam-I-Am: maybe stating the obvious, but what we want is for the internal network mtu of a neutron router interface to be the same as the external network mtu - having that 1500/1450 is sucky enough14:16
haleybi think we just agreed14:16
*** davidsha has quit IRC14:17
*** Swami has joined #openstack-neutron14:17
*** Swami_ has quit IRC14:18
*** pai15 has quit IRC14:18
Sam-I-Amhaleyb: yeah... except what happens if something actually tries to send you a packet at that 'jumboer' size?14:18
*** Swami__ has quit IRC14:19
*** ramishra_ has quit IRC14:19
Sam-I-Amfrickler: so you have a router namespace with 1 qg and 2 qrs, each qr belonging to a different self-service network type?14:20
*** ramishra has joined #openstack-neutron14:20
*** chandankumar has joined #openstack-neutron14:21
haleybSam-I-Am: it should get fragmented at the sender, or dropped.  1500 is the standard no matter how much we want to deny it.  Jumbo should only be used for intra-network traffic - i.e. web server to db backend (imho)14:21
haleybor intra-datacenter14:21
fricklerSam-I-Am: in fact I was having two routers, not connected to each other, except they are hosted on the same node14:21
*** sjmc7 has joined #openstack-neutron14:22
Sam-I-Amhaleyb: lets say you're using 1550 on the physical net so vms using vxlan can use 1500. as long as no packet larger than 1500 enters that phys net, you should be fine. so that implies some sort of l3 device between the internets (assuming 1500 mtu) and your dc network (1550)14:23
*** pai15 has joined #openstack-neutron14:23
Sam-I-Amfrickler: ok, so one router for the vxlan net, one for the vlan net?14:23
*** lixiaoy1 has quit IRC14:23
Swamiobondarev: ping14:23
*** lixiaoy1 has joined #openstack-neutron14:24
obondarevSwami: hi!14:24
fricklerSam-I-Am: right14:24
Swamiobondarev: I did you see your comments on the patch regarding the fix for the regression.14:24
Sam-I-Amfrickler: so whats ip netns show inside those namespaces?14:24
Swamiobondarev: I had one question.14:24
obondarevSwami: yep14:24
*** singhj has joined #openstack-neutron14:24
*** ssalagame has joined #openstack-neutron14:25
*** mlavalle has joined #openstack-neutron14:25
Swamiobondarev: you mentioned that when the gw-port is deleted from the router when the agent was down. How do to reproduce it. or what are the steps to reproduce it.14:25
*** dave-mccowan has joined #openstack-neutron14:26
obondarevSwami: ah, it’s simple, turn off l3 agent on the node hosting snat for router, make sure router is rescheduled to another node, delete router ext gw, turn on l3 agent from step 114:27
*** asingh has joined #openstack-neutron14:27
haleybSam-I-Am: in my case I'm assuming the DC network default external VLAN (Internet-facing) is 1500.  The intra-node VLAN used for the data plane is 1550 or higher, so it's only used for VM traffic.14:27
haleyband it's not visible to the tenant14:28
Sam-I-Amhaleyb: that works14:28
Sam-I-Amassuming you only need 1500-ish mtu to vms14:28
haleybSam-I-Am: but i also want to support a jumbo network, but perhaps that should only live within the confines of neutron, when it needs to get routed external you run into trouble with mtu again14:29
*** ijw has joined #openstack-neutron14:29
*** ratailor has quit IRC14:29
*** links has quit IRC14:29
Swamiobondarev: are you deleting the router external gateway port manually or doing router_gateway_clear on the router.14:29
Sam-I-Amas long as the mtu change occurs in a router, you should be fine with any values14:30
obondarevSwami: second14:30
Swamiobondarev: ok will check it out.14:30
Sam-I-Amthe problem prior to mitaka was mtu changes occuring at layer 2, which causes packets to drop14:30
*** iyamahat has joined #openstack-neutron14:31
obondarevSwami: do you think we should block revert?14:31
*** yamahata has joined #openstack-neutron14:31
Swamiobondarev: yes, I think we should block the revert and proceed with this patch to fix it.14:32
haleybSam-I-Am: just that typically there aren't mtu changes across the internet, but yes, a router should handle it.  anyways, i'm guessing we're tripping over one of these problems here - not separating network elements properly so the mtu details are "leaking" between them for lack of a better explanatiaon14:32
*** sjmc7 has left #openstack-neutron14:32
Swamiobondarev: I am confident that we can address all the cases with this patch.14:32
obondarevSwami: I’d say we should fix high priority issue asap14:32
Sam-I-Amhaleyb: yeah, thats why i'd like to see output14:32
Sam-I-Amand better understand the topo14:32
obondarevSwami: it’s a regression in stable branches14:32
*** p01nt3r75 has quit IRC14:32
Swamiobondarev: yes14:33
Swamiobondarev: agreed.14:33
fricklerSam-I-Am: I'm was rebuilding the stack, almost done14:33
Swamiobondarev: do you see any other issues with this patch.14:33
obondarevSwami: I don’t see anything wrong with reverting, then preparing well tested patch for stale namespaces with no rush14:34
*** tonytan4ever has joined #openstack-neutron14:34
Sam-I-Amfrickler: i'd like to see the config, and then the mtu of all interfaces (including on the host and inside namespaces)14:35
Swamiobondarev: it's your call.14:35
Swamiobondarev: either way is fine for me.14:35
haleybSam-I-Am: thanks for taking a look, it will be worthy of another summit talk :)  i'll try and keep engaged in the discussion between meetings, but you probably need to loop-in kevin as well14:35
obondarevSwami: thanks, will you please reevaluate https://review.openstack.org/#/c/326574/ then?14:36
*** julim has quit IRC14:37
*** rubasov has quit IRC14:37
*** oanson has joined #openstack-neutron14:37
Swamiobondarev: done14:38
obondarevSwami: thanks!14:39
*** rtheis_ has joined #openstack-neutron14:39
*** rtheis has quit IRC14:40
*** julim has joined #openstack-neutron14:40
*** julim has quit IRC14:40
*** cleong has joined #openstack-neutron14:41
*** chagatai has quit IRC14:41
ihrachysjlibosva: that's a solid piece of code indeed14:42
ihrachysjlibosva: I left some comments14:42
jlibosvaihrachys: thanks14:42
jlibosvaihrachys: you wrote most of it :d14:42
ihrachyshm. not really, there is a lot of base class work done14:43
*** woodburn has quit IRC14:44
fricklerSam-I-Am: http://paste.openstack.org/show/508948 and http://paste.openstack.org/show/50895014:45
*** Alex_Stef has quit IRC14:45
*** nplanel has quit IRC14:46
*** rtheis has joined #openstack-neutron14:49
openstackgerritNate Johnston proposed openstack/neutron-fwaas: FWaaS v2 Database  https://review.openstack.org/31115914:50
*** oshvartz has joined #openstack-neutron14:50
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: DVR: Check for snat_namespace existence before add or delete  https://review.openstack.org/30035814:50
*** rtheis_ has quit IRC14:51
*** watanabe_isao has quit IRC14:51
*** davidsha has joined #openstack-neutron14:53
Sam-I-Amfrickler: pondering14:53
*** Leom has quit IRC14:53
*** thumpba has joined #openstack-neutron14:54
*** salv-orl_ has quit IRC14:56
*** salv-orlando has joined #openstack-neutron14:57
*** links has joined #openstack-neutron14:57
*** diga has joined #openstack-neutron14:57
*** abregman_ has joined #openstack-neutron14:58
*** abregman has quit IRC14:59
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: DVR: Add exists method to namespace class to check  https://review.openstack.org/32672914:59
*** krypto has joined #openstack-neutron14:59
*** fawadkhaliq has joined #openstack-neutron15:00
kryptoi am having a strange situation where router status is shown as alive :) but namespace is missing from network node.i have "allow_automatic_l3agent_failover = False" any idea?15:00
*** vikram has joined #openstack-neutron15:01
openstackgerritIhar Hrachyshka proposed openstack/neutron: tests: added sorting/pagination api tests for subnetpools  https://review.openstack.org/32708115:01
*** kobis has quit IRC15:02
*** oshvartz has quit IRC15:02
*** ijw has quit IRC15:02
*** ijw has joined #openstack-neutron15:03
openstackgerritDaniel Berrange proposed openstack/os-vif: os_vif: fix logging of exceptions during plug/unplug  https://review.openstack.org/32716915:03
*** tonytan4ever has quit IRC15:03
*** lucas___ has joined #openstack-neutron15:05
*** gvrangan has joined #openstack-neutron15:05
*** frickler has quit IRC15:06
*** mkolesni has quit IRC15:06
*** frickler has joined #openstack-neutron15:06
*** itamarl has quit IRC15:06
*** Leom has joined #openstack-neutron15:07
ihrachysjlibosva: replied, something to consider there.15:09
*** tonytan4ever has joined #openstack-neutron15:09
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/32483815:10
*** faizy has joined #openstack-neutron15:10
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements  https://review.openstack.org/32483915:11
Sam-I-Amfrickler: did you try setting path_mtu to 1500? looks like its 0 in your config. shouldn't matter, but its something to check.15:12
*** tidwellr_ has joined #openstack-neutron15:13
*** oshvartz has joined #openstack-neutron15:15
*** oshvartz has quit IRC15:15
Sam-I-Amfrickler: also, looking at your output, what network bits are plugged into router 7c2e3d62 ?15:15
openstackgerritCarl Baldwin proposed openstack/neutron: Add index on trunk_id in the subports model  https://review.openstack.org/32668915:15
*** oshvartz has joined #openstack-neutron15:16
fricklerSam-I-Am: that is the default router created by devstack, connected to public and private15:16
jlibosvaihrachys: I have new PS ready but updating the tenant_id15:17
jlibosvaihrachys: I still wonder whether we need to be explicit there about it15:17
jlibosvaalso wonder whether we have it covered by api test15:17
carl_baldwinHenryG: Would you mind reading Armando's comment about an inline change to the model (without migration) on this review ^^15:18
fricklerSam-I-Am: those are both created with network_type=vxlan and have mtu=145015:18
*** haukebruno has quit IRC15:18
fricklerSam-I-Am: I'll be away for a bit now, will try the path_mtu later15:19
*** twm2016 has joined #openstack-neutron15:20
*** davidlenwell has quit IRC15:20
*** julim has joined #openstack-neutron15:20
*** oshvartz has quit IRC15:20
HenryGcarl_baldwin: Will do. I remember discussing something similar on this model with Armando some time back.15:20
*** julim has quit IRC15:20
*** oshvartz has joined #openstack-neutron15:21
*** julim has joined #openstack-neutron15:21
*** akshai has quit IRC15:21
*** akshai has joined #openstack-neutron15:21
*** garyk1 has quit IRC15:22
*** julim has quit IRC15:22
Sam-I-Amfrickler: ohhh... whats external_network_bridge set to in the l3 agent config?15:22
*** julim has joined #openstack-neutron15:22
Sam-I-Amfrickler: also whats the output of neutron net-show external (or whatever the devstack ext net is called)15:23
*** tinbar has quit IRC15:23
ihrachyskorzen__: tiny concern on https://review.openstack.org/#/c/326477/1 (dns server), otherwise looks good15:24
*** faizy has quit IRC15:24
ihrachyskorzen__: I saw grenade multinode failed, nice job :)15:25
fricklerSam-I-Am: http://paste.openstack.org/show/50895815:25
*** oshvartz has quit IRC15:25
dasmihrachys: i believe, first time i see someone, who is glad that job is failing :D15:26
ihrachysdasm: that's by intent15:26
*** jschwarz has quit IRC15:26
*** oshvartz has joined #openstack-neutron15:26
*** fnaval has quit IRC15:26
Sam-I-Amfrickler: yeah, there's your problem.15:26
dasmihrachys: yeah, i noticed it before. but still... the situation is funny :P15:26
Sam-I-Amfrickler: devstack is doing Bad and Wrong :)15:26
*** lucas___ has quit IRC15:26
Sam-I-Amfrickler: is this a recent pull of devstack, or older?15:27
fricklerSam-I-Am: fresh from yesterday15:27
*** yamamoto has joined #openstack-neutron15:28
ihrachysjlibosva: you are right, I am lunatic15:28
jlibosvaihrachys: I added an API test for that15:28
ihrachysjlibosva: heil to you!15:28
openstackgerritJakub Libosvar proposed openstack/neutron: objects: switch base plugin class to using subnetpool object  https://review.openstack.org/30005615:29
*** dmk0202 has quit IRC15:29
jlibosvaihrachys: ^^ there it goes :)15:29
ihrachysspeedy Jakub15:29
*** ajmiller has joined #openstack-neutron15:29
*** jamesdenton has joined #openstack-neutron15:30
Sam-I-Amfrickler: whats happening here... devstack is creating your public/ext net as type vxlan, even though it isn't, hence getting an mtu of 1450 rather than 1500.15:30
jlibosvaihrachys: not that you're the author so we'll probably need HenryG to look at it as well15:30
jlibosvanote*15:30
ihrachysyeah, just realized that15:30
*** oshvartz has quit IRC15:31
*** emagana has joined #openstack-neutron15:31
*** mickeys has joined #openstack-neutron15:31
*** ajmiller__ has quit IRC15:31
*** fnaval has joined #openstack-neutron15:31
*** oshvartz has joined #openstack-neutron15:31
HenryGihrachys: jlibosva: which one?15:31
jlibosvaHenryG:  We would like to ask all of you to help ensure Red Hat is well represented at the show!15:31
*** lucas____ has joined #openstack-neutron15:32
fricklerSam-I-Am: well, in that case it should not matter, because the vlan router is not connected to either public or private15:32
jlibosvawhat?15:32
jlibosvaHenryG: I didn't write that :D15:32
ihrachysWAT15:32
jlibosvaI copied the link15:32
ihrachyssmells like some proprietary data leaked!!!15:32
ihrachys:)15:32
ihrachysHenryG: https://review.openstack.org/30005615:32
*** julim has quit IRC15:32
*** yamamoto has quit IRC15:33
jlibosvathat was weird. I can't paste it here but when I paste it to browser I have the correct link :-/15:33
ihrachysHenryG: also prerequisite adding test coverage for Pager is: https://review.openstack.org/#/c/327081/15:34
openstackgerritImran Hayder Malik proposed openstack/neutron: Fix designate dns driver for SSL based endpoints  https://review.openstack.org/32695815:34
ihrachysthose two will give us base plugin doing objects for pools15:34
jlibosvaihrachys: I realized I forgot to add DependsOn15:34
ihrachysjlibosva: there are pieces in l3, are you covering those?15:35
ihrachysjlibosva: as a follow up obviously15:35
jlibosvaihrachys: what do you mean?15:35
*** mikelk has quit IRC15:36
*** oshvartz has quit IRC15:36
ihrachysjlibosva: http://paste.openstack.org/show/508960/15:36
Sam-I-Amfrickler: did you read http://docs.openstack.org/developer/devstack/guides/neutron.html ?15:36
*** oshvartz has joined #openstack-neutron15:37
ihrachysah wait15:37
ihrachysmaybe those are false flags15:37
*** a_ta_ has joined #openstack-neutron15:38
*** pai15_ has joined #openstack-neutron15:38
ihrachysjlibosva: in SubnetAllocator, in _lock_subnetpool, we fetch hash from the model directly15:38
ihrachysand also update it15:38
*** Swami_ has joined #openstack-neutron15:38
openstackgerritMerged openstack/neutron-dynamic-routing: Fix typo in the installation documentation  https://review.openstack.org/32624215:39
*** roeyc has quit IRC15:39
openstackgerritMerged openstack/neutron-dynamic-routing: Fix exception translation  https://review.openstack.org/32623515:39
jlibosvaihrachys: aha15:39
openstackgerritMerged openstack/neutron-dynamic-routing: Add neutron-dynamic-routing DrAgent & Scheduler document  https://review.openstack.org/31784415:40
ihrachysjlibosva: seems like that's actually the only thing15:40
*** amuller is now known as amuller_afk15:40
*** Swami__ has joined #openstack-neutron15:40
*** pai15 has quit IRC15:40
*** a_ta has quit IRC15:40
*** abregman_ has quit IRC15:41
jlibosvaihrachys: isn't this CaS part? do we need objects there?15:41
*** hdaniel has quit IRC15:41
*** chandankumar has quit IRC15:42
*** oshvartz has quit IRC15:42
*** Swami has quit IRC15:42
ihrachysjlibosva: I think yes. any place that currently uses models will in ideal world do everything thru objects.15:42
*** azbiswas has joined #openstack-neutron15:42
*** julim has joined #openstack-neutron15:42
*** sleviim has quit IRC15:42
ihrachysI think it's the same pattern: fetch pool, update hash attr, .update()15:43
*** julim has quit IRC15:43
*** Swami_ has quit IRC15:43
ihrachysnow, please take it with a grain of salt :)15:43
ihrachysmaybe I am totally out of reality15:43
*** julim has joined #openstack-neutron15:44
*** julim has quit IRC15:44
*** Swami__ has quit IRC15:45
ihrachysmaybe that code just belongs to the object.15:45
*** julim has joined #openstack-neutron15:46
*** julim has quit IRC15:46
*** vhosakot has joined #openstack-neutron15:46
*** yamamoto has joined #openstack-neutron15:46
jlibosvaihrachys: I'm not sure. This seems like a better approach to me - you need to update an old hash with new hash specifically in database, I don't see reason to use objects here15:47
*** julim has joined #openstack-neutron15:47
*** eezhova has quit IRC15:47
jlibosvaI'm thinking how it will influence live upgrade15:47
*** eezhova has joined #openstack-neutron15:47
ihrachysjlibosva: the thing is, if hash field moves somewhere, and we need to handle it smartly without a simple rename, it introduces complexity on CaS too15:48
ihrachysjlibosva: and I think we should keep it into the object.15:48
*** pai15_ has quit IRC15:48
jlibosvaihrachys: ok, let's try that15:48
ihrachysotherwise we forget to update that other piece of code, and boom15:48
ihrachysjlibosva: ok thanks. but let's do follow up, let's try to land what we have quicker15:49
jlibosvaok15:49
*** pai15 has joined #openstack-neutron15:49
*** gangil has joined #openstack-neutron15:49
openstackgerritHenry Gessau proposed openstack/neutron-lib: Make the constant Sentinal() class public  https://review.openstack.org/32719815:52
*** oshvartz has joined #openstack-neutron15:53
*** garyk has quit IRC15:53
*** ivase has joined #openstack-neutron15:54
*** eddima has quit IRC15:55
*** hynekm has quit IRC15:55
*** yamamoto has quit IRC15:56
*** davidlenwell has joined #openstack-neutron15:56
*** yamamoto has joined #openstack-neutron15:56
*** gvrangan has quit IRC15:57
*** julim has quit IRC15:57
*** emagana has quit IRC15:57
*** oshvartz has quit IRC15:58
*** cbouch has joined #openstack-neutron15:58
*** chlong has quit IRC15:58
*** oshvartz has joined #openstack-neutron15:58
*** obondarev has quit IRC15:59
*** yamamoto has quit IRC16:00
*** yamamoto has joined #openstack-neutron16:00
*** chandankumar has joined #openstack-neutron16:00
*** emagana has joined #openstack-neutron16:01
openstackgerritCarl Baldwin proposed openstack/neutron: Clean up subports model  https://review.openstack.org/32720116:02
*** mhickey has quit IRC16:02
carl_baldwinHenryG: ^ I broke out the possibly controversial inline model change in to this new patch16:02
mesterysc68cal: Regarding https://review.openstack.org/#/c/214350/, sorry I don't have time to debug further to understand the config file failure16:02
*** oshvartz has quit IRC16:03
*** ushkalim has quit IRC16:04
*** oshvartz has joined #openstack-neutron16:04
*** gangil has quit IRC16:05
*** garyk has joined #openstack-neutron16:06
*** obondarev has joined #openstack-neutron16:07
*** fawadkhaliq has quit IRC16:07
*** yfried has quit IRC16:07
*** oshvartz has quit IRC16:08
*** Sukhdev_ has joined #openstack-neutron16:08
*** oshvartz has joined #openstack-neutron16:09
*** julim has joined #openstack-neutron16:11
openstackgerritJakub Libosvar proposed openstack/neutron: WIP: _lock_subnetpool with objects  https://review.openstack.org/32720716:12
*** nmagnezi has quit IRC16:12
jlibosvaihrachys: something like ^^ ?16:12
*** julim has quit IRC16:12
*** julim has joined #openstack-neutron16:13
*** diga has quit IRC16:13
*** Sukhdev_ has quit IRC16:14
*** oshvartz_ has joined #openstack-neutron16:14
ihrachysjlibosva: should we extend update() to allow where= argument?16:14
*** links has quit IRC16:15
*** vhosakot has quit IRC16:15
*** kawa2014 has quit IRC16:16
*** bcafarel has quit IRC16:16
*** eezhova has quit IRC16:17
*** eezhova has joined #openstack-neutron16:17
jlibosvaihrachys: we would need to start using query there instead of saving model16:17
jlibosvain update_object() in  dbapi16:18
*** Leom has quit IRC16:18
*** salv-orlando has quit IRC16:18
*** jlibosva has quit IRC16:18
*** oshvartz has quit IRC16:18
*** eezhova has quit IRC16:18
*** salv-orlando has joined #openstack-neutron16:18
ihrachysjlibosva: we can distinguish between too cases under the hood16:18
*** rossella_s has quit IRC16:19
*** slunkad has quit IRC16:19
*** oshvartz_ has quit IRC16:19
*** oshvartz_ has joined #openstack-neutron16:19
*** ygbo has quit IRC16:20
*** brad_behle has quit IRC16:23
*** brad_behle has joined #openstack-neutron16:23
*** oshvartz_ has quit IRC16:24
*** salv-orl_ has joined #openstack-neutron16:24
*** obondarev has quit IRC16:25
*** azbiswas_ has joined #openstack-neutron16:25
*** oshvartz has joined #openstack-neutron16:25
*** ijw has quit IRC16:26
*** Sukhdev_ has joined #openstack-neutron16:26
*** cbouch has quit IRC16:26
*** salv-orlando has quit IRC16:27
*** azbiswas has quit IRC16:27
*** banix has quit IRC16:28
*** lucas____ has quit IRC16:28
*** Sukhdev_ has quit IRC16:29
*** oshvartz has quit IRC16:29
*** obondarev has joined #openstack-neutron16:29
*** oshvartz has joined #openstack-neutron16:30
*** yamahata has quit IRC16:30
*** yamamoto has quit IRC16:30
*** fzdarsky is now known as fzdarsky|afk16:31
*** iyamahat has quit IRC16:31
*** lucas___ has joined #openstack-neutron16:31
*** tesseract has quit IRC16:31
*** jamesdenton has quit IRC16:32
sc68calmestery: no problem16:32
*** lucas____ has joined #openstack-neutron16:32
*** luca_____ has joined #openstack-neutron16:33
*** oshvartz has quit IRC16:34
*** yamamoto has joined #openstack-neutron16:34
*** luca_____ has quit IRC16:34
*** luc______ has joined #openstack-neutron16:34
*** oshvartz has joined #openstack-neutron16:35
*** diga has joined #openstack-neutron16:35
*** prateek has quit IRC16:35
*** lucas___ has quit IRC16:35
*** frontrunner has quit IRC16:36
*** lucas____ has quit IRC16:36
*** jpena is now known as jpena|off16:37
*** eezhova has joined #openstack-neutron16:38
*** pai15 has quit IRC16:38
*** pai15 has joined #openstack-neutron16:38
*** yamamoto has quit IRC16:39
*** luc______ has quit IRC16:39
*** oshvartz has quit IRC16:39
*** oshvartz has joined #openstack-neutron16:39
*** obondarev has quit IRC16:40
*** eezhova has quit IRC16:40
openstackgerritgaryk proposed openstack/neutron-lib: A translation validations to the hacking policy  https://review.openstack.org/32586516:40
*** Guest47242 is now known as med_16:41
*** med_ has joined #openstack-neutron16:41
*** lucas___ has joined #openstack-neutron16:41
*** rcernin has quit IRC16:42
*** fawadkhaliq has joined #openstack-neutron16:42
*** fawadkhaliq has quit IRC16:42
*** fawadkhaliq has joined #openstack-neutron16:42
*** mohankumar_ has quit IRC16:42
*** oanson has quit IRC16:42
*** oshvartz has quit IRC16:43
*** oshvartz has joined #openstack-neutron16:44
*** sudipto has quit IRC16:44
*** wolverineav has joined #openstack-neutron16:45
openstackgerritSean M. Collins proposed openstack/neutron-fwaas: DevStack plugin for fwaas  https://review.openstack.org/21435016:45
sc68calmestery: I forgot a "-"16:45
*** lucas___ has quit IRC16:46
*** frontrunner has joined #openstack-neutron16:47
*** moshele has quit IRC16:48
*** lucasagomes is now known as lucas-dinner16:48
*** oshvartz has quit IRC16:48
*** eezhova has joined #openstack-neutron16:49
*** oshvartz has joined #openstack-neutron16:49
*** mvk has quit IRC16:50
openstackgerritgaryk proposed openstack/neutron-lib: Hacking: update iteritems hacking message  https://review.openstack.org/32724216:52
*** gvrangan has joined #openstack-neutron16:52
*** javeriak has joined #openstack-neutron16:52
*** oshvartz has quit IRC16:53
*** Swami has joined #openstack-neutron16:54
*** iyamahat has joined #openstack-neutron16:54
*** oshvartz has joined #openstack-neutron16:54
openstackgerritgaryk proposed openstack/neutron-lib: Hacking: update iteritems hacking message  https://review.openstack.org/32724216:55
*** gangil has joined #openstack-neutron16:56
openstackgerritMerged openstack/neutron: Hacking: make use of neutron lib  https://review.openstack.org/32706016:56
*** bobmel has joined #openstack-neutron16:56
*** bobmel_ has quit IRC16:57
*** oshvartz has quit IRC16:58
*** thorst_ has joined #openstack-neutron16:58
*** oshvartz has joined #openstack-neutron16:59
*** vikram has quit IRC16:59
*** ijw has joined #openstack-neutron17:00
openstackgerritgaryk proposed openstack/neutron-fwaas: Remove check_i18n files  https://review.openstack.org/32724517:03
*** oshvartz has quit IRC17:03
*** fragatina has joined #openstack-neutron17:03
*** javeriak_ has joined #openstack-neutron17:04
*** fragatin_ has joined #openstack-neutron17:04
*** javeriak has quit IRC17:04
*** tonytan4ever has quit IRC17:04
*** thumpba has quit IRC17:04
*** thumpba has joined #openstack-neutron17:05
*** thorst_ has quit IRC17:05
*** chandankumar has quit IRC17:05
*** Leom has joined #openstack-neutron17:06
*** fragatina has quit IRC17:08
openstackgerritIhar Hrachyshka proposed openstack/neutron: objects: in get_object(s), filter by fields, not model attributes  https://review.openstack.org/32724917:09
*** shashank_hegde has joined #openstack-neutron17:09
*** jerryz has joined #openstack-neutron17:10
*** yamahata has joined #openstack-neutron17:11
*** vishwanathj has quit IRC17:11
*** chandankumar has joined #openstack-neutron17:11
*** lucas___ has joined #openstack-neutron17:11
mesterysc68cal: Testing17:12
*** ivasilevskaya has quit IRC17:14
*** wolverineav has quit IRC17:14
*** javeriak has joined #openstack-neutron17:15
dougwigHenryG, kevinbenton, boden - I'm going to miss the lib meeting today.17:16
*** amuller_afk is now known as amuller17:16
*** lucas___ has quit IRC17:17
*** thorst_ has joined #openstack-neutron17:17
*** javeriak_ has quit IRC17:18
*** dileepr has joined #openstack-neutron17:18
*** c64cosmin has quit IRC17:19
*** vhosakot has joined #openstack-neutron17:20
openstackgerritSreekumar S proposed openstack/neutron: Fix for Cookie Bridge not applying ext cookie to flow ops  https://review.openstack.org/32663717:20
*** thumpba has quit IRC17:20
*** diga has quit IRC17:20
*** thumpba has joined #openstack-neutron17:21
HenryGdougwig: I'll run it17:21
*** techcet has joined #openstack-neutron17:21
*** chandankumar has quit IRC17:21
*** ivar-lazzaro has joined #openstack-neutron17:22
*** techcet has quit IRC17:22
*** thorst_ has quit IRC17:22
*** shashank_hegde has quit IRC17:22
openstackgerritgaryk proposed openstack/neutron-dynamic-routing: Remove check_i18n files  https://review.openstack.org/32725417:22
*** techcet has joined #openstack-neutron17:22
*** thumpba has quit IRC17:23
*** tonytan4ever has joined #openstack-neutron17:23
*** john-davidge has quit IRC17:23
openstackgerritgaryk proposed openstack/neutron-fwaas: Remove check_i18n files  https://review.openstack.org/32724517:24
*** thumpba has joined #openstack-neutron17:25
openstackgerritgaryk proposed openstack/neutron-dynamic-routing: Remove check_i18n files  https://review.openstack.org/32725417:25
*** davidsha has quit IRC17:26
*** itisha has joined #openstack-neutron17:26
*** thorst_ has joined #openstack-neutron17:29
*** thorst_ has quit IRC17:29
*** boden has joined #openstack-neutron17:30
*** singhj has quit IRC17:31
*** ivar-laz_ has joined #openstack-neutron17:31
*** moshele has joined #openstack-neutron17:32
*** yamahata has quit IRC17:33
*** yamahata has joined #openstack-neutron17:33
*** ivar-lazzaro has quit IRC17:34
*** singhj has joined #openstack-neutron17:34
*** Leom has quit IRC17:37
*** singhj has quit IRC17:38
*** wolverin_ has joined #openstack-neutron17:41
*** tidwellr_ has quit IRC17:44
*** abregman has joined #openstack-neutron17:47
openstackgerritIhar Hrachyshka proposed openstack/neutron: WIP: objects: introduce NetworkPortSecurity object  https://review.openstack.org/32725717:47
*** ihrachys has quit IRC17:47
openstackgerritDarek Smigiel proposed openstack/neutron: Update Neutron documentation with `project`  https://review.openstack.org/32725817:48
*** lucas___ has joined #openstack-neutron17:48
*** ijw has quit IRC17:49
*** abhiraut has joined #openstack-neutron17:50
*** eezhova has quit IRC17:50
*** shashank_hegde has joined #openstack-neutron17:50
*** abhiraut has quit IRC17:50
*** abhiraut has joined #openstack-neutron17:52
*** sambetts is now known as sambetts|afk17:53
garyksc68cal: hi, qucik question - do you have offhand the patch that moves away from neutron-legacy.17:53
*** salv-orl_ has quit IRC17:53
*** banix has joined #openstack-neutron17:53
*** salv-orlando has joined #openstack-neutron17:53
*** ivar-laz_ has quit IRC17:55
*** kbringard has joined #openstack-neutron17:55
*** ivar-lazzaro has joined #openstack-neutron17:57
*** singhj has joined #openstack-neutron18:00
*** s3wong has joined #openstack-neutron18:00
*** ijw has joined #openstack-neutron18:01
*** chandankumar has joined #openstack-neutron18:01
*** ducnc has quit IRC18:01
*** ducnc has joined #openstack-neutron18:01
*** kevo has joined #openstack-neutron18:02
*** jamie_h has quit IRC18:07
*** kriskend has quit IRC18:08
*** frontrunner has quit IRC18:10
-openstackstatus- NOTICE: Zuul has been restarted to correct an error condition. Events since 17:30 may have been missed; please 'recheck' your changes if they were uploaded since then, or have "NOT_REGISTERED" errors.18:10
sc68calgaryk: under what context? project-config, devstack-gate?18:11
*** chandankumar has quit IRC18:11
*** oanson has joined #openstack-neutron18:13
garyksc68cal: devstack18:13
*** fragatin_ has quit IRC18:14
sc68calgaryk: my plan is to update project-config and devstack-gate to use neutron-*18:15
sc68calthen eventually delete lib/neutron-legacy18:15
*** Sukhdev has joined #openstack-neutron18:16
garyksc68cal: i understand that i just want to make sure that decomposed prject do not break when you do this18:17
garykwe need to make sure that people have enough time to do this18:17
*** chandankumar has joined #openstack-neutron18:19
*** fragatina has joined #openstack-neutron18:19
sc68calgaryk: indeed. I don't have a patch up that deletes neutron-legacy. I imagine it'll be some time before that happens18:19
openstackgerritMerged openstack/neutron-dynamic-routing: BGP dynamic routing CLI  https://review.openstack.org/32313218:21
*** javeriak_ has joined #openstack-neutron18:21
*** javeriak has quit IRC18:22
*** lucas___ has quit IRC18:22
openstackgerritgaryk proposed openstack/neutron-lib: Hacking: update iteritems hacking message  https://review.openstack.org/32724218:23
garyksc68cal: ok. understood18:24
*** ijw has quit IRC18:24
*** jerryz has quit IRC18:26
*** javeriak has joined #openstack-neutron18:29
*** pai15 has quit IRC18:29
*** pai15 has joined #openstack-neutron18:30
*** mickeys has quit IRC18:30
*** links has joined #openstack-neutron18:32
*** javeriak_ has quit IRC18:33
*** pai15 has quit IRC18:34
*** ijw has joined #openstack-neutron18:36
*** frontrunner has joined #openstack-neutron18:36
*** javeriak has quit IRC18:37
*** javeriak_ has joined #openstack-neutron18:37
*** ssalagame has quit IRC18:39
*** techcet has quit IRC18:39
*** woodburn has joined #openstack-neutron18:39
*** gvrangan has quit IRC18:40
*** jerryz has joined #openstack-neutron18:41
*** javeriak has joined #openstack-neutron18:42
*** Leom has joined #openstack-neutron18:42
*** javeriak_ has quit IRC18:44
*** gouthamr has quit IRC18:45
*** ssalagame has joined #openstack-neutron18:45
*** azbiswas_ has quit IRC18:45
*** efoley has quit IRC18:45
*** boden has quit IRC18:46
*** techcet has joined #openstack-neutron18:46
*** fragatina has quit IRC18:48
*** vhosakot has quit IRC18:50
*** krtaylor has quit IRC18:52
*** Kennan has quit IRC18:52
*** vhosakot has joined #openstack-neutron18:52
*** Kennan has joined #openstack-neutron18:52
*** Sukhdev has quit IRC18:53
*** fnaval has quit IRC18:57
*** nyechiel has quit IRC18:58
*** fnaval has joined #openstack-neutron18:58
*** fnaval has quit IRC18:58
*** a_ta_ has quit IRC18:59
*** fnaval has joined #openstack-neutron18:59
*** a_ta has joined #openstack-neutron18:59
*** Sukhdev has joined #openstack-neutron19:01
*** a_ta_ has joined #openstack-neutron19:01
*** gouthamr has joined #openstack-neutron19:01
*** alex_xu has quit IRC19:02
*** nyechiel has joined #openstack-neutron19:03
*** ssalagame has quit IRC19:03
*** a_ta has quit IRC19:04
*** krtaylor has joined #openstack-neutron19:05
*** pai15 has joined #openstack-neutron19:06
*** abhiraut has quit IRC19:06
*** alex_xu has joined #openstack-neutron19:06
*** claudiub has quit IRC19:06
*** nyechiel has quit IRC19:11
*** harlowja has quit IRC19:12
*** chandankumar has quit IRC19:13
*** salv-orlando has quit IRC19:14
*** salv-orlando has joined #openstack-neutron19:14
*** tflynn has joined #openstack-neutron19:15
*** harlowja has joined #openstack-neutron19:15
*** fzdarsky|afk has quit IRC19:18
*** ssalagame has joined #openstack-neutron19:18
*** abhiraut has joined #openstack-neutron19:18
*** gvrangan has joined #openstack-neutron19:19
*** frontrunner has quit IRC19:21
openstackgerritHenry Gessau proposed openstack/neutron: Remove some options from neutron-db-manage  https://review.openstack.org/31162419:22
*** javeriak has quit IRC19:22
*** javeriak has joined #openstack-neutron19:22
*** evgenyf has joined #openstack-neutron19:23
*** evgenyf has left #openstack-neutron19:24
*** gangil has quit IRC19:24
openstackgerritKevin Benton proposed openstack/neutron: Convert multiple exception types in the API  https://review.openstack.org/31907319:25
*** rcernin has joined #openstack-neutron19:25
*** gvrangan has quit IRC19:26
*** links has quit IRC19:31
*** pai15 has quit IRC19:32
*** boden has joined #openstack-neutron19:33
*** javeriak has quit IRC19:34
*** ducncvn has joined #openstack-neutron19:36
*** javeriak has joined #openstack-neutron19:37
*** ducnc has quit IRC19:37
*** pai15 has joined #openstack-neutron19:38
*** tidwellr has joined #openstack-neutron19:39
*** salv-orlando has quit IRC19:39
*** salv-orlando has joined #openstack-neutron19:40
*** singhj has quit IRC19:40
openstackgerritHenry Gessau proposed openstack/neutron: Check for alembic Add/DropColumn exceptions in migrations  https://review.openstack.org/32569919:42
*** lixiaoy1 has quit IRC19:44
*** lixiaoy1 has joined #openstack-neutron19:44
*** azbiswas has joined #openstack-neutron19:47
*** abhiraut has quit IRC19:50
*** emagana has quit IRC19:52
openstackgerritMiguel Angel Ajo proposed openstack/neutron-specs: L2 extension ovs flow management  https://review.openstack.org/32043919:54
*** ssalagame has quit IRC19:55
*** emagana has joined #openstack-neutron19:59
*** techcet has quit IRC19:59
*** fawadkhaliq has quit IRC20:01
openstackgerritNate Johnston proposed openstack/neutron-fwaas: FWaaS v2 Database  https://review.openstack.org/31115920:01
*** fawadkhaliq has joined #openstack-neutron20:01
*** singhj has joined #openstack-neutron20:03
*** abregman has quit IRC20:04
*** fawadkhaliq has quit IRC20:05
*** cleong has quit IRC20:08
*** techcet has joined #openstack-neutron20:08
*** Sukhdev has quit IRC20:08
slaweqgaryk: hello20:09
*** minwang2 has joined #openstack-neutron20:10
slaweqcan You help me with https://review.openstack.org/#/c/32564420:10
*** gangil has joined #openstack-neutron20:10
slaweqor maybe someone else who knows well rbac can take a look on that and on my comment in https://review.openstack.org/#/c/325644/4/neutron/objects/rbac_db.py20:11
*** minwang2 has quit IRC20:12
*** Leom has quit IRC20:13
Sam-I-Amfrickler: you here?20:14
*** ducnc has joined #openstack-neutron20:17
*** ducncvn has quit IRC20:17
*** singhj has quit IRC20:17
*** minwang2 has joined #openstack-neutron20:19
*** mvk has joined #openstack-neutron20:19
*** singhj has joined #openstack-neutron20:20
*** elopez has quit IRC20:22
*** regXboi has joined #openstack-neutron20:23
*** oanson has quit IRC20:24
*** gvrangan has joined #openstack-neutron20:26
*** thumpba has quit IRC20:27
*** ihrachys has joined #openstack-neutron20:28
*** thumpba has joined #openstack-neutron20:29
*** mickeys has joined #openstack-neutron20:29
carl_baldwinpavel_bondar: tidwellr: ping20:31
*** jbell8 has quit IRC20:31
*** mickeys_ has joined #openstack-neutron20:32
*** thumpba has quit IRC20:33
*** mickeys has quit IRC20:34
*** javeriak_ has joined #openstack-neutron20:36
*** mickeys_ has quit IRC20:36
openstackgerritImran Hayder Malik proposed openstack/neutron: Fix designate dns driver for SSL based endpoints  https://review.openstack.org/32695820:36
*** mickeys has joined #openstack-neutron20:36
*** javeriak_ has quit IRC20:36
openstackgerritImran Hayder Malik proposed openstack/neutron: Fix designate dns driver for SSL based endpoints  https://review.openstack.org/32695820:39
*** javeriak has quit IRC20:39
*** asingh has quit IRC20:39
*** Leo_ has joined #openstack-neutron20:39
*** ksial has joined #openstack-neutron20:41
*** lnicolas has quit IRC20:42
*** thumpba has joined #openstack-neutron20:43
*** gouthamr has quit IRC20:43
tidwellrcarl_baldwin: pong20:44
*** dave-mccowan has quit IRC20:44
carl_baldwintidwellr: Was wondering if you have had your eye on the test results of https://review.openstack.org/#/c/29220720:44
carl_baldwintidwellr: I'm looking through some of the jenkins failures in pavel_bondar 's patch that he rebased to it.20:45
carl_baldwintidwellr: So far, I haven't made much sense of them.20:45
*** kriskend has joined #openstack-neutron20:46
tidwellrcarl_baldwin: 292207 seems to be passing20:46
tidwellrcarl_baldwin: but I haven't rechecked it much20:46
carl_baldwinI just did.  I'm actually pretty confident with that one but then pavel_bondar 's patch has had more failures than successes.20:47
*** openstackgerrit has quit IRC20:48
tidwellrare you talking about https://review.openstack.org/#/c/181023/?20:49
*** openstackgerrit has joined #openstack-neutron20:49
carl_baldwintidwellr: yes20:51
tidwellrcarl_baldwin: I see the last recheck had some success20:53
tidwellrcarl_baldwin: do you have a link to a stack trace?20:54
carl_baldwintidwellr: The previous 7 rechecks before that all failed.  I haven't found any smoking guns but that alone is a little bit concerning.20:55
*** pai15 has quit IRC20:56
*** thumpba has quit IRC20:56
carl_baldwintidwellr: I'll keep looking in to this a bit to see if I see any correlation.  In the meantime, recheck it if you see it isn't running in the check queue.  This is one we need a lot of data on.  It would be great if it passes a lot more going forward.20:57
tidwellrrecheck both or 181023?20:57
tidwellrI can do both20:57
openstackgerritNate Johnston proposed openstack/neutron-fwaas: FWaaS v2 Database  https://review.openstack.org/31115920:58
*** techcet has quit IRC20:58
*** pai15 has joined #openstack-neutron20:59
*** asingh has joined #openstack-neutron21:00
*** dave-mccowan has joined #openstack-neutron21:00
carl_baldwintidwellr: I found something that seems to be unique to this patch set.21:01
carl_baldwintidwellr: http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22u'SAVEPOINT%20sa_savepoint_2%20does%20not%20exist'%5C%2221:01
*** fragatina has joined #openstack-neutron21:01
*** pai15 has quit IRC21:01
carl_baldwintidwellr: It appears in logs/screen-q-svc.txt in multiple rechecks of this patch set.  It only appears on this patch set.21:02
*** julim has quit IRC21:04
*** nyechiel has joined #openstack-neutron21:04
*** chlong has joined #openstack-neutron21:05
carl_baldwintidwellr: Here is an example of a trackback:  http://paste.openstack.org/show/509028/21:06
*** jbell8 has joined #openstack-neutron21:07
*** abhiraut has joined #openstack-neutron21:07
carl_baldwintidwellr: It looks like it does correlate with an actual test failure:  http://paste.openstack.org/show/509029/21:07
*** banix has quit IRC21:11
carl_baldwintidwellr: The good news is that I see the same failure on PS80 which was before rebasing to your change.  That actually gives me some confidence in your patch.21:13
*** ivase has quit IRC21:13
*** ametts has quit IRC21:14
*** nyechiel has quit IRC21:15
carl_baldwintidwellr: I've changed to +2 on https://review.openstack.org/#/c/29220721:15
carl_baldwintidwellr: Great work on that patch.  I know it was a hard one.21:16
*** ssalagame has joined #openstack-neutron21:16
tidwellrcarl_baldwin: brave man giving that a +2, here we go! :)21:17
carl_baldwintidwellr: No guts no glory.21:17
tidwellrcarl_baldwin: we're leaping off the cliff now :)21:17
carl_baldwintidwellr: Seriously though, I think we've given this patch set *a lot* of extra care and due diligence.  There is a point where you've got to pull the trigger and merge it.21:18
tidwellrcarl_baldwin: agreed, finding bugs is unfortunately part of doing business, you can't let fear of bugs block progress forever21:19
carl_baldwintidwellr: I feel pretty good about it.  Let's see if we can get it approved.21:20
*** mvk_ has joined #openstack-neutron21:25
openstackgerritRyan Tidwell proposed openstack/neutron: Override get_object() and get_objects() for Trunk  https://review.openstack.org/32734521:27
*** mvk has quit IRC21:28
*** nplanel has joined #openstack-neutron21:30
*** lifeless_ is now known as lifeless21:30
*** tonytan4ever has quit IRC21:31
*** yamahata has quit IRC21:35
openstackgerritIhar Hrachyshka proposed openstack/neutron: objects: introduce NetworkPortSecurity object  https://review.openstack.org/32725721:36
*** johnbelamaric has quit IRC21:36
*** ihrachys has quit IRC21:36
*** ihrachys has joined #openstack-neutron21:37
*** baoli_ has quit IRC21:38
*** jbell8 has quit IRC21:41
*** regXboi has quit IRC21:43
*** janzian has quit IRC21:44
*** thorst has joined #openstack-neutron21:47
*** boden has quit IRC21:47
*** tpsilva has quit IRC21:47
kevinbentoncarl_baldwin, tidwellr: yo21:48
kevinbenton"the mac address None is in use"21:48
kevinbentonshould be gone21:48
kevinbentonwith https://review.openstack.org/#/c/322626/21:50
*** moshele has quit IRC21:51
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: DVR: Add exists method to namespace class to check  https://review.openstack.org/32672921:51
carl_baldwinkevinbenton: we saw it for a different reason, I think.21:52
tidwellrkevinbenton: yes, different reason21:53
*** asingh has quit IRC21:53
openstackgerritIhar Hrachyshka proposed openstack/neutron: Add sanity check for idempotent id's in tempest tests  https://review.openstack.org/30048321:53
kevinbentontidwellr, carl_baldwin: what was it?21:53
*** ihrachys_ has joined #openstack-neutron21:53
*** rcernin is now known as rcernin|off21:54
*** asingh has joined #openstack-neutron21:54
kevinbentonbecause it was on an update21:55
*** r-daneel has joined #openstack-neutron21:55
kevinbentonand it looks like it was because it leaked the mutated dictionary into the retry21:55
*** ssalagame has quit IRC21:55
carl_baldwinkevinbenton: That patch adds the random window for IP allocation, right?  So, there is a chance that two allocations try to grab the same IP and a duplicate entry exception would be thrown which should restart one of them.21:55
kevinbentonyes21:56
kevinbentonwhich would lead to the bug that patch i linked if it was an update operation21:56
carl_baldwinkevinbenton: So, the operation right before that mac address duplication check is where the new IP allocation is inserted.  But, it wasn't flushed.  So, the duplicate wasn't detected until that mac address code flushed it.21:56
*** ihrachys has quit IRC21:56
carl_baldwinkevinbenton: So, I added this flush:  https://review.openstack.org/#/c/292207/23/neutron/db/db_base_plugin_common.py21:57
carl_baldwinkevinbenton: The mac address None is in use error disappeared when I added that.21:57
carl_baldwinkevinbenton: Then, the duplicate entry exception properly bubbled up to trigger the retry mechanism.21:58
kevinbentoncarl_baldwin: "The mac address None" means that the port data had bogus info in it21:58
*** vhoward has quit IRC21:59
openstackgerritRyan Tidwell proposed openstack/neutron: [WIP] Enable CRUD for trunk ports  https://review.openstack.org/32009221:59
*** shashank_hegde has quit IRC22:00
kevinbentoncarl_baldwin: calling flush may have changed timing to avoid retries, but i don't see how it could have changed the mac address field being None22:00
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/32483822:00
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements  https://review.openstack.org/32483922:00
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-lib: Updated from global requirements  https://review.openstack.org/32307422:00
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements  https://review.openstack.org/32737822:00
openstackgerritSlawek Kaplonski proposed openstack/neutron: Filter supported QoS rules with possible parameters  https://review.openstack.org/31969422:00
carl_baldwinkevinbenton: Let me find another pointer.22:01
*** xagent has joined #openstack-neutron22:02
carl_baldwinkevinbenton: https://review.openstack.org/#/c/321840/1/neutron/db/ipam_backend_mixin.py22:02
carl_baldwinWithout that flush, the duplicate IP address wouldn't be caught until here: https://review.openstack.org/#/c/321840/1/neutron/db/ipam_backend_mixin.py22:03
carl_baldwinThe code you see here is assuming that DBDuplicateEntry is only thrown for duplicate mac address.  But, it was wrong.22:03
kevinbentoncarl_baldwin: oh, i see. so the exception is bad22:03
*** vhosakot has quit IRC22:03
carl_baldwinkevinbenton: It was being raised because of the IP address duplication that was queued up and not detected until the flush right here.22:04
xagentHi, I have a question regarding multiple external networks w/ 1 L3 Agent. I am no longer setting the "external_network_bridge" to my external OVS bridge. Also, I am now defining a new bridge mapping for my external bridge ID in OVS agent config22:04
carl_baldwinkevinbenton: I'm not sure what you mean by "the exception is bad"22:04
*** kriskend has quit IRC22:04
carl_baldwinThe exception was good but raising it was delayed to a point in the code which misinterpreted it.22:04
xagentBut I now have multiple bridge mappings - physnet:br-data for my regular tenant/provider networks, physnet2, etc..22:05
kevinbentoncarl_baldwin: raising duplicate MAC based on duplicate entry22:05
xagentand also a phyext:br-ext for my external bridge22:05
kevinbentoncarl_baldwin: is wrong22:05
carl_baldwinkevinbenton: Ah, yes, that too presumptuous.22:05
xagentbefore when creating tenant networks I wasn't specifying the mapping. i just want it to go on any available physnet22:05
carl_baldwins/that/that is/22:05
xagentnow, how do i prevent it from getting on the external bridge mapping?22:06
*** gouthamr has joined #openstack-neutron22:06
*** singhj has quit IRC22:07
kevinbentonxagent: using vlan networks?22:08
*** shashank_hegde has joined #openstack-neutron22:08
xagentyes22:08
kevinbentoncarl_baldwin: this dup detection is problematic because a flush doesn't really guaruntee anything since the transaction is still open...22:08
kevinbentonuncommitted22:08
kevinbentonxagent: ml2_type_vlan config section22:09
openstackgerritBrandon Logan proposed openstack/neutron: Pecan: handle single fields query parameter  https://review.openstack.org/32739422:09
xagentNot sure I follow?22:09
xagentmy assumption was before, if i have multiple physnet mappings (physnet1, physnet2, physnet3) with VLAN ranges, and i create a tenant network, it just goes on the next available one. when i don't specify the provider:physical_network22:09
*** ssalagame has joined #openstack-neutron22:10
*** dane_leblanc has quit IRC22:10
openstackgerritBrandon Logan proposed openstack/neutron: Pecan: handle single fields query parameter  https://review.openstack.org/32739422:10
kevinbentonxagent: one sec, checking something22:10
xagentnow that i've adding a mapping for my external bridge, i want tenant networks to go on any, except the one for ext networks22:10
*** ihrachys_ has quit IRC22:11
*** ihrachys has joined #openstack-neutron22:12
*** ihrachys has quit IRC22:12
*** twm2016 has quit IRC22:12
kevinbentonxagent: i think you can define an empty range22:12
kevinbentonxagent: IIRC22:12
openstackgerritBrandon Logan proposed openstack/neutron: Pecan: handle single fields query parameter  https://review.openstack.org/32739422:13
blogankevinbenton: ^^22:13
xagentempty range for which mapping?22:14
carl_baldwinkevinbenton: Are you saying that the duplicate error exception could still be deferred until the final commit?  I guess that might be true.22:15
kevinbentonxagent: for the external physnet22:15
kevinbentonxagent: so just don't define a range for it22:16
kevinbentonxagent: just put the physnet name by itself22:16
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: DVR: Add exists method to namespace class to check  https://review.openstack.org/32672922:16
kevinbentonxagent: that will prevent neutron from automatically allocating from it22:16
carl_baldwinkevinbenton: I should say "deferred to any point up to the final commit"22:16
kevinbentonxagent: then only an admin that explicitly specifies the vlan segmentation ID22:16
kevinbentonxagent: can use the external physnet22:16
xagentSo I create a tenant network, and don't speicfy any provider mapping - it'll go on physnet1, physnet2, etc.. exhausting it's VLAN ranges22:17
kevinbentoncarl_baldwin: so once it flushes, i think we are guarunteed that the mac is unique for the rest of the transaction operations22:17
kevinbentoncarl_baldwin: until commit22:17
xagentbut when i create an external network, I can specify an exact VLAN ID and give the provider mapping, this will work?22:17
kevinbentonxagent: yep22:18
*** kbringard has quit IRC22:18
kevinbentonxagent: just tested locally22:18
xagentWhat about if i create a flat external network (just specify the provider:physical_network) and router:external=true?22:18
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: DVR: Add exists method to namespace class to check  https://review.openstack.org/32672922:18
xagentthis would also work and go on my special physnet-ext ?22:18
xagentthanks, let me try that out...22:19
kevinbentonxagent: i believe so. i don't think there is anything that blocks a flat and vlan sharing a physnet22:19
kevinbentonxagent: but you'll have to try it and let me know22:19
carl_baldwinkevinbenton: I'm a little confused.  That's what I thought originally but then you said it doesn't really guarantee anything.  I'm not sure what you're saying.22:19
kevinbentoncarl_baldwin: i'm saying we have to catch duplicates at commit time22:20
kevinbentoncarl_baldwin: (i.e. when the context manager exits and COMMIT is sent to the DB)22:20
kevinbentoncarl_baldwin: because we can get a duplicate at either point22:20
kevinbentoncarl_baldwin: either immediately when the INSERT statement is sent to the DB22:20
carl_baldwinkevinbenton: Did you mean to say "i think we *aren't* guaranteed the the mac in unique for the rest ..." ?22:20
*** rcernin|off has quit IRC22:21
kevinbentoncarl_baldwin: or at COMMIT22:21
carl_baldwinkevinbenton: or at flush22:21
kevinbentoncarl_baldwin: i meant to say that for the rest of the transaction, we don't have to worry about dbduplicate22:21
kevinbentoncarl_baldwin: from the mac22:21
kevinbentoncarl_baldwin: until commit time22:21
carl_baldwinkevinbenton: or, someone else issues a flush.22:21
kevinbentoncarl_baldwin: "at flush" == "carl_baldwin: either immediately when the INSERT statement is sent to the DB22:21
kevinbentonkevinbenton: Did you mean to say "i think we *aren't* guaranteed the the mac in unique for the rest ..." ?"22:21
kevinbenton(ignore second message)22:22
kevinbentonwow my irc client garbled that22:22
kevinbenton "at flush" == " immediately when the INSERT statement is sent to the DB"22:22
kevinbentonwe don't have to worry about other flushes22:22
carl_baldwinkevinbenton: I think I'm following now.22:22
kevinbentononce we have sent the INSERT statement with the mac to the db22:22
kevinbentonif it doesn't raise duplicate then, we are safe until commit time22:23
*** gvrangan has quit IRC22:23
kevinbentonthen we have to catch duplicate there as well i believe22:23
kevinbentonbecause that's a normal constraint failure, not a weird one that will manifest as a deadlock22:24
carl_baldwinAre you thinking that maybe a galera cluster might defer it until commit time when it certifies the commit?22:24
*** gvrangan has joined #openstack-neutron22:24
*** gvrangan has quit IRC22:24
*** gvrangan has joined #openstack-neutron22:25
*** ivar-lazzaro has quit IRC22:25
kevinbentoncarl_baldwin: not even galera if the port table isn't locked22:25
kevinbentoncarl_baldwin: do we lock the port table?22:25
carl_baldwinkevinbenton: So, now the problem is that we might have a DBDuplicate for different reasons.  Some reasons should trigger a retry and some should throw a new exception and abort the operation.22:25
*** salv-orl_ has joined #openstack-neutron22:25
*** Leo_ has quit IRC22:25
*** Leo_ has joined #openstack-neutron22:26
kevinbentoncarl_baldwin: right, but the dup can show up much later as well at commit22:26
kevinbentoncarl_baldwin: definitely with galera22:26
kevinbentoncarl_baldwin: and maybe without galera if we don't still lock the port table22:26
*** Leo_ has quit IRC22:27
carl_baldwinkevinbenton: We'll need code at about the same level as the retry decorator that has the ability to distinguish between failure modes.22:27
carl_baldwinI'm now wondering what data the exception carries with it.22:27
carl_baldwin... and how easy it will be to match it back to a failure mode.22:27
kevinbentoncarl_baldwin: maybe we should just look for the mac in the DB ahead of time to look for duplicates22:28
kevinbentoncarl_baldwin: then treat and DBDublicate errors as retriable22:28
*** salv-orlando has quit IRC22:28
kevinbentoncarl_baldwin: to handle the case where there is a race with duplicate macs22:29
kevinbentoncarl_baldwin: because we already treat other duplicates as retry (https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/plugin.py#L1104)22:29
*** fnaval has quit IRC22:29
*** fnaval has joined #openstack-neutron22:29
kevinbentoncarl_baldwin: basically it would be the same as IPAM in that regard22:29
kevinbentoncarl_baldwin: check for duplicate, if clear, try to use22:30
*** banix has joined #openstack-neutron22:30
kevinbentoncarl_baldwin: if exception, try again22:30
openstackgerritRyan Tidwell proposed openstack/neutron: DO NOT MERGE: Allocate IP addresses backwards to flush out tests reliant on sequential allocation  https://review.openstack.org/31265522:30
carl_baldwinkevinbenton: that'd work22:31
kevinbentoncarl_baldwin: I'll cook it up22:31
*** chandankumar has joined #openstack-neutron22:34
*** chandankumar_ has joined #openstack-neutron22:34
carl_baldwinkevinbenton: Let me know when you have it.22:34
*** techcet has joined #openstack-neutron22:36
*** edmondsw has quit IRC22:37
*** claudiub has joined #openstack-neutron22:37
*** singhj has joined #openstack-neutron22:38
*** mlavalle has quit IRC22:38
xagentkevinbenton: If I do not specify any VLAN ranges for my external physnet, do I list the mapping in ml2_type_vlan or ml2_type_flat? My external networks themselves may be VLAN based22:38
kevinbentonxagent: both if you want to be able to use both types22:39
Sam-I-Amxagent: provider nets can use arbitrary vlan ids outside of the ml2_type_vlan range22:39
xagentah, I can have same bridge/mapping name in both sections? didnt know that22:39
Sam-I-Amprovider/external physnet22:39
kevinbentonSam-I-Am: yes, covered that further up the backlog :)22:39
Sam-I-Amthe ranges only apply to self-service/project networks22:39
*** chandankumar_ has quit IRC22:40
*** chandankumar has quit IRC22:40
kevinbentonxagent: by default you can have any arbitary flat network22:40
kevinbentonxagent: so you don't even need to define the physnet in ml2_type_flat if 'flat_networks' is left as the default *22:41
Sam-I-Amstill needs a mapping tho, no?22:41
*** tidwellr has left #openstack-neutron22:41
kevinbentonyou mean a bridge_mappings entry?22:41
kevinbentonshould already have one for when it was a vlan physnet22:41
kevinbentonbut yeah, it needs one22:41
kevinbentonon the agent22:42
xagentgotcha, so I shall have "physnet-ext:br-ext" in both ml2_type flat and vlan sections for my bridge_mappings22:42
*** minwang2 has quit IRC22:42
*** singhj has quit IRC22:42
kevinbentonxagent: bridge_mappings go on the agent under the OVS section. you don't need any bridge_mappings in the ML2 conf22:42
xagentfor network_vlan_ranges in ml2_type_flat, I shall just specify "physnet-ext" without a range22:42
kevinbentonxagent: yep22:43
*** ksial has quit IRC22:43
kevinbentonxagent: so something like network_vlan_ranges=tenant_phys:1000:1500,physnet-ext22:43
*** fawadkhaliq has joined #openstack-neutron22:44
Sam-I-Amkevinbenton: have you seen https://bugs.launchpad.net/neutron/+bug/1590397 ?22:44
openstackLaunchpad bug 1590397 in neutron "MTU setting too low when mixing Vlan and Vxlan" [Undecided,New] - Assigned to Manikantha Srinivas Tadi (manikantha-tadi)22:44
*** azbiswas has quit IRC22:44
Sam-I-Amthe title is a bit misleading22:44
*** asingh has quit IRC22:44
kevinbentonSam-I-Am: no, hadn't yet22:44
*** jckasper has quit IRC22:44
*** azbiswas has joined #openstack-neutron22:45
*** asingh has joined #openstack-neutron22:45
*** salv-orl_ has quit IRC22:45
*** jckasper has joined #openstack-neutron22:45
kevinbentonSam-I-Am: so this is the OVS bridge MTU?22:46
*** salv-orlando has joined #openstack-neutron22:46
*** woodburn1 has joined #openstack-neutron22:46
Sam-I-Amwell, also misleading. i think the original report thought it might be ovs bridge mtu.22:46
Sam-I-Ami think that was actually mislead to by a devstack misconfig22:47
Sam-I-Amhowever, the post after i said how to config devstack is more interesting. clear gateway, set gateway... mtu does not restore to 1500.22:47
*** woodburn has quit IRC22:48
Sam-I-Amshould be easy to reproduce if its a thing22:48
kevinbentonSam-I-Am: can you see if you can repro it?22:48
Sam-I-Amdoing that now22:49
*** azbiswas has quit IRC22:49
*** jckasper has quit IRC22:49
*** banix has quit IRC22:50
*** ksial has joined #openstack-neutron22:52
*** krypto has quit IRC22:53
*** emagana has quit IRC22:53
*** pradk has quit IRC22:54
*** emagana has joined #openstack-neutron22:55
*** amotoki has quit IRC22:56
*** emagana has quit IRC22:58
*** sdague has quit IRC22:59
*** banix has joined #openstack-neutron23:00
*** Jabadia has joined #openstack-neutron23:00
*** amotoki has joined #openstack-neutron23:02
*** jckasper has joined #openstack-neutron23:06
Sam-I-Amkevinbenton: cant repro, qg comes back up as 1500 or 900023:06
*** jckasper has quit IRC23:07
*** jckasper has joined #openstack-neutron23:07
*** vhosakot has joined #openstack-neutron23:08
Sam-I-Amand ovs int mtu still doesnt seem to matter23:08
*** shashank_hegde has quit IRC23:11
*** jckasper has quit IRC23:12
*** snizar has joined #openstack-neutron23:13
*** markvoelker has quit IRC23:13
*** vhosakot has quit IRC23:14
*** shashank_hegde has joined #openstack-neutron23:14
*** vhosakot has joined #openstack-neutron23:14
*** gouthamr_ has joined #openstack-neutron23:15
*** lnicolas has joined #openstack-neutron23:16
kevinbentonSam-I-Am: hmmm23:17
*** fawadkhaliq has quit IRC23:17
Sam-I-Amstill running more tests23:17
Sam-I-Amsomehow borked my system23:17
kevinbentonSam-I-Am: ack23:17
*** fawadkhaliq has joined #openstack-neutron23:18
*** gouthamr has quit IRC23:18
*** jckasper has joined #openstack-neutron23:20
*** mickeys has quit IRC23:20
*** mickeys has joined #openstack-neutron23:21
*** fawadkhaliq has quit IRC23:21
*** emagana has joined #openstack-neutron23:22
*** mickeys has quit IRC23:25
*** vhosakot has quit IRC23:25
*** vhosakot has joined #openstack-neutron23:26
*** emagana has quit IRC23:26
*** jckasper has quit IRC23:27
Sam-I-Amkevinbenton: dunno what i did, but it'll wait until tomorrow when my brain is less melty23:29
*** snizar has quit IRC23:30
*** mickeys has joined #openstack-neutron23:30
openstackgerritKevin Benton proposed openstack/neutron: Preemptive MAC dup detection with retry  https://review.openstack.org/32741323:31
openstackgerritBrandon Logan proposed openstack/neutron: Add segment_id to DHCP Agent Network Bindings  https://review.openstack.org/32626123:33
*** rustyeddy has joined #openstack-neutron23:33
kevinbentoncarl_baldwin: ^~123:34
kevinbentonSam-I-Am: sounds good23:34
*** banix has quit IRC23:35
*** claudiub has quit IRC23:38
*** markvoelker has joined #openstack-neutron23:44
*** Sukhdev has joined #openstack-neutron23:47
*** lnicolas has quit IRC23:48
*** markvoelker has quit IRC23:49
*** chandankumar has joined #openstack-neutron23:50
*** gangil has quit IRC23:51
*** yushiro has joined #openstack-neutron23:52
*** amuller has quit IRC23:53
*** chandankumar has quit IRC23:54
*** gangil has joined #openstack-neutron23:55
*** banix has joined #openstack-neutron23:57

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