Tuesday, 2016-11-29

kevinbentonraj_: thx00:01
*** ijw has joined #openstack-neutron00:01
*** wolverineav has joined #openstack-neutron00:02
raj_I assume there is some way to tag it for Mitaka but I'm something of a cluser in that regard presently.00:03
*** yuanying has joined #openstack-neutron00:03
kevinbentonraj_: is this pretty easy for you to reproduce?00:03
*** yuanying_ has joined #openstack-neutron00:03
*** ijw has quit IRC00:03
*** ijw has joined #openstack-neutron00:04
raj_Yes.  With 448 instances spread across a little more than half a dozen computes, I have seen it happen two times out of three.  With 1000 instances I've seen it happen every time.00:04
kevinbentonraj_: I will push up a patch for you to test to make sure it is the problem I think it is00:05
raj_Hopefully it isn't too terribly involved a patch :)00:05
kevinbentonraj_: will be late tonight or tomorrow00:05
kevinbentonraj_: not too bad I don't think00:05
raj_No worries.  This isn't a hot issue.  Just something I happened across pretty much by chance.00:06
*** dffco has quit IRC00:06
*** wolverineav has quit IRC00:07
*** yuanying has quit IRC00:08
*** catintheroof has joined #openstack-neutron00:08
*** dsneddon has quit IRC00:10
*** fnaval has quit IRC00:10
openstackgerritReedip proposed openstack/neutron-lib: Add converter to convert IPv6 addresses to canonical format  https://review.openstack.org/35767000:13
*** tbachman has joined #openstack-neutron00:13
*** bobmel has joined #openstack-neutron00:16
*** tidwellr has left #openstack-neutron00:19
*** bobmel has quit IRC00:21
raj_kevinbenton, side question - the revision that is checked against in Newton in the change you pointed me at - that is a per-port attribute00:21
*** Swami has joined #openstack-neutron00:21
raj_s/attribute/attribute?/00:22
openstackgerritBrandon Logan proposed openstack/neutron: Pecan: Change Quotas to use original controllers  https://review.openstack.org/39979100:23
kevinbentonraj_: it's for the whole port00:23
kevinbentonraj_: so when an attribute on the port is updated, the revision is bumped00:23
raj_Thanks.  I was wondering about the probability of wrap-around :)00:24
openstackgerritBrandon Logan proposed openstack/neutron: Pecan: Change Quotas to use original controllers  https://review.openstack.org/39979100:24
kevinbentonraj_: it's not 32-bit00:24
kevinbentonraj_: it's huuuge :)00:24
raj_Heh.  Even if it were 32 bit, so long as each port has its own revision number I suspect it would be fine.  What keeps the cache of deleted ports "in check" as it were?00:25
kevinbentonraj_: so we still have the issue of messages getting completely lost00:25
kevinbentonraj_: if that's what you are thinking00:25
*** thorst_ has joined #openstack-neutron00:26
raj_I was wondering if every delete port (for that network?) goes into the cache and stays there, or if something kept the cache from growing without bound.00:26
*** reedip_ has quit IRC00:27
raj_Hmm, looking closer I think I see places where it gets reset.  nm I suppose.00:27
*** thorst_ has quit IRC00:27
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: DVR: Add forwarding routes based on address_scopes  https://review.openstack.org/35506200:28
*** limao has joined #openstack-neutron00:29
*** jtollet has joined #openstack-neutron00:29
*** amuller has quit IRC00:30
openstackgerritMerged openstack/python-neutronclient: Show team and repo badges on README  https://review.openstack.org/40268800:30
*** hfu has joined #openstack-neutron00:30
openstackgerritMerged openstack/neutron-dynamic-routing: Use ExtensionDescriptor from neutron-lib  https://review.openstack.org/40329300:31
*** wolverineav has joined #openstack-neutron00:31
*** john-davidge has joined #openstack-neutron00:32
armaxabhiraut: ping00:33
*** jtollet has quit IRC00:34
*** longlq_ has quit IRC00:36
*** wolverineav has quit IRC00:36
*** john-davidge has quit IRC00:36
*** mfuruta has joined #openstack-neutron00:39
abhirautarmax pong00:43
armaxabhiraut: hi00:43
abhirauthello00:43
abhirautssup00:43
*** vhosakot_ has quit IRC00:43
armaxabhiraut: I was wondering if you were still interested/had time in helping out with the python-neutronclient transition to OSC00:43
abhirautyeah.. i was some holidays.. so didnt get to look at it much.. back now00:44
armaxreview-wise you have a good eye for spotting good OSC guidelines00:44
armaxabhiraut: cool00:44
abhirautill start eye balling client reviews00:44
armaxabhiraut: great00:45
abhirauti did push one patch regarding osc.. i wanted to get rtheis opinion00:45
abhirauton it00:45
abhirautto see whether im on the right path00:45
abhirautjust added you on that review00:45
armaxlink?00:45
armaxack00:46
abhirauthttps://review.openstack.org/#/c/401033/00:46
armaxI’ll have a look00:46
abhirautbut this is regarding the refactor00:46
*** shashank_hegde has quit IRC00:46
armaxbtw we have a section decidecated on client updates during the team meeting00:47
abhirautwhat CLI remain to be transitioned?00:47
abhirautok cool00:47
armaxabhiraut: it’s all on here: https://bugs.launchpad.net/neutron/+bug/152129100:47
openstackLaunchpad bug 1521291 in neutron "[RFE] Transition neutron CLI from python-neutronclient to python-openstackclient" [Wishlist,Triaged] - Assigned to Akihiro Motoki (amotoki)00:47
*** hoangcx has joined #openstack-neutron00:47
abhirautill be on the meetings now to monitor the status00:47
*** rubasov has quit IRC00:49
armaxthe etherpad with all reviews I believe is https://etherpad.openstack.org/p/osc-neutron-support00:49
armaxthere are a few that I’ll be tracking for consolidating OSC support from neutron subprojects00:49
*** rubasov has joined #openstack-neutron00:50
armaxthose should show up under topic:stadium-implosion00:50
abhirautack00:50
armaxbut I’ll tag you, just in case00:50
abhirautyeah thanks..00:50
armaxwell, thank you!00:50
abhirauti have gkotton blessings so far to work on this :)00:51
*** wolverineav has joined #openstack-neutron00:51
armaxI hope it won’t take much of your time00:53
armaxbut if you want more on your plate, all you have to do is ask ;)00:53
abhirautcool.. i know whom to ;)00:53
*** tflynn_ has joined #openstack-neutron00:54
*** dsneddon has joined #openstack-neutron00:54
*** stevemar__ has quit IRC00:54
*** wolverineav has quit IRC00:56
*** dave-mccowan has quit IRC00:56
*** tflynn has quit IRC00:57
*** tflynn_ has quit IRC00:58
armaxabhiraut: I added the review # on https://etherpad.openstack.org/p/osc-network-command-sdk-support00:59
armaxline 6401:00
abhirautah .. got it.. will get on to the remaining TODOs01:00
abhirautand update accordingly01:00
abhirautthanks for your review!01:01
*** wolverineav has joined #openstack-neutron01:01
*** tbachman has quit IRC01:01
*** ankur-gupta-f1 has left #openstack-neutron01:03
*** TuHV has joined #openstack-neutron01:04
*** gvrangan has quit IRC01:04
*** longlq_ has joined #openstack-neutron01:05
*** s3wong has quit IRC01:05
*** wolverineav has quit IRC01:06
*** tbachman has joined #openstack-neutron01:07
*** ducnc has joined #openstack-neutron01:07
*** beagles_biab is now known as beagles01:09
*** gvrangan has joined #openstack-neutron01:10
*** tovin07 has joined #openstack-neutron01:10
*** annp has joined #openstack-neutron01:10
*** wolverineav has joined #openstack-neutron01:11
*** wolverineav has quit IRC01:16
*** donghao has joined #openstack-neutron01:18
*** shashank_hegde has joined #openstack-neutron01:21
*** wolverineav has joined #openstack-neutron01:21
openstackgerritArmando Migliaccio proposed openstack/neutron: Reuse plugin's update method for changing trunk status  https://review.openstack.org/39162001:23
*** wolverineav has quit IRC01:25
*** donghao has quit IRC01:27
*** jtollet has joined #openstack-neutron01:31
*** wolverineav has joined #openstack-neutron01:31
*** Swami has quit IRC01:31
*** nmagnezi has quit IRC01:32
openstackgerritVictor Morales proposed openstack/neutron: Integration of RouterRoute OVO  https://review.openstack.org/36707101:33
*** wu_ng has joined #openstack-neutron01:33
openstackgerritMerged openstack/neutron-fwaas: Reenable FWaaS v2 tempest tests  https://review.openstack.org/40094801:33
*** numans has joined #openstack-neutron01:34
*** manjong has joined #openstack-neutron01:34
*** thorst_ has joined #openstack-neutron01:35
*** wolverineav has quit IRC01:35
*** jtollet has quit IRC01:35
*** mriedem has joined #openstack-neutron01:38
*** yamamot__ has joined #openstack-neutron01:41
*** thorst_ has quit IRC01:41
*** kevo has quit IRC01:43
*** yamamot__ has quit IRC01:44
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/40114601:45
*** haleyb_ has joined #openstack-neutron01:46
openstackgerritArmando Migliaccio proposed openstack/neutron-fwaas: Add os-testr to test requirements  https://review.openstack.org/39734801:48
*** mlavalle has quit IRC01:48
openstackgerritOpenStack Proposal Bot proposed openstack/python-neutronclient: Updated from global requirements  https://review.openstack.org/39282501:50
openstackgerritArmando Migliaccio proposed openstack/neutron-specs: Ocata: Assessment summary  https://review.openstack.org/38939701:51
*** zkassab has joined #openstack-neutron01:55
*** shashank_hegde has quit IRC01:56
*** shashank_hegde has joined #openstack-neutron01:57
*** dffco has joined #openstack-neutron01:59
*** wolverineav has joined #openstack-neutron02:00
*** numans has quit IRC02:01
*** yamamot__ has joined #openstack-neutron02:04
*** iheartbutter has joined #openstack-neutron02:04
*** wolverineav has quit IRC02:05
openstackgerritIWAMOTO Toshihiro proposed openstack/neutron: DONOTMERGE: use ovsfw  https://review.openstack.org/39940002:06
reediparmax : ping02:07
*** paw has quit IRC02:08
*** iwamoto has joined #openstack-neutron02:09
*** baoli has joined #openstack-neutron02:11
*** fnaval has joined #openstack-neutron02:14
*** catintheroof has quit IRC02:16
*** catintheroof has joined #openstack-neutron02:16
*** bobmel has joined #openstack-neutron02:17
*** wolverineav has joined #openstack-neutron02:20
*** catintheroof has quit IRC02:20
*** shashank_hegde has quit IRC02:20
*** bobmel has quit IRC02:21
*** wolverineav has quit IRC02:25
*** rajinir has quit IRC02:26
*** dffco has quit IRC02:28
*** reedip has quit IRC02:29
*** jtollet has joined #openstack-neutron02:32
*** reedip has joined #openstack-neutron02:32
*** stanzgy has joined #openstack-neutron02:33
*** numans has joined #openstack-neutron02:35
*** jtollet has quit IRC02:36
*** thorst_ has joined #openstack-neutron02:39
*** thorst_ has quit IRC02:40
*** thorst_ has joined #openstack-neutron02:40
*** thorst_ has quit IRC02:41
*** achadha_ has joined #openstack-neutron02:43
*** iheartbutter has quit IRC02:43
*** paw has joined #openstack-neutron02:45
*** jschwarz has quit IRC02:45
*** achadha has quit IRC02:46
openstackgerritMerged openstack/neutron: Add missing revises_on_change attribute  https://review.openstack.org/40016102:47
*** achadha_ has quit IRC02:47
*** achadha has joined #openstack-neutron02:47
*** jschwarz has joined #openstack-neutron02:49
*** wolverineav has joined #openstack-neutron02:49
*** achadha has quit IRC02:51
*** dsneddon has quit IRC02:52
*** ijw has quit IRC02:53
*** ijw has joined #openstack-neutron02:53
*** ijw has quit IRC02:53
*** wolverineav has quit IRC02:54
*** stevemar__ has joined #openstack-neutron02:55
*** john-davidge has joined #openstack-neutron02:59
*** stevemar__ has quit IRC03:00
*** yuanying_ has quit IRC03:00
*** sudipto has joined #openstack-neutron03:01
*** sudipto_ has joined #openstack-neutron03:01
*** zkassab has quit IRC03:02
*** bkopilov has quit IRC03:02
*** john-davidge has quit IRC03:03
openstackgerritSpencer Yu proposed openstack/os-vif: Os-vif already uses PBR:- setuptools.setup(  setup_requires=['pbr>=1.8'],  pbr=True)  https://review.openstack.org/40399503:04
*** haleyb__ has joined #openstack-neutron03:06
*** tlian has quit IRC03:06
*** dsneddon has joined #openstack-neutron03:06
*** oanson has joined #openstack-neutron03:07
*** thorst_ has joined #openstack-neutron03:08
*** haleyb_ has quit IRC03:09
*** wolverineav has joined #openstack-neutron03:09
*** links has joined #openstack-neutron03:10
*** links has quit IRC03:10
*** jhershbe has joined #openstack-neutron03:11
*** dffco has joined #openstack-neutron03:12
openstackgerritOmer Anson proposed openstack/neutron: Use pyroute2 to read routing table  https://review.openstack.org/39474103:13
*** Leom has joined #openstack-neutron03:13
*** wolverineav has quit IRC03:14
*** Leom has quit IRC03:15
*** tlian has joined #openstack-neutron03:15
*** gcb has joined #openstack-neutron03:15
*** tbachman has quit IRC03:16
*** thorst_ has quit IRC03:16
*** bobmel has joined #openstack-neutron03:18
*** wolverineav has joined #openstack-neutron03:19
*** yamamo___ has joined #openstack-neutron03:22
*** bobmel has quit IRC03:23
*** lnicolas has joined #openstack-neutron03:23
armaxreedip: pong03:23
*** wolverineav has quit IRC03:24
*** dffco has quit IRC03:24
*** catintheroof has joined #openstack-neutron03:25
*** yamamot__ has quit IRC03:26
*** abhiraut has quit IRC03:27
openstackgerritMerged openstack/neutron-fwaas: Add os-testr to test requirements  https://review.openstack.org/39734803:31
*** tlian has quit IRC03:31
*** jtollet has joined #openstack-neutron03:32
*** udesale has joined #openstack-neutron03:32
*** trinaths has joined #openstack-neutron03:34
*** manjong has quit IRC03:35
openstackgerritSpencer Yu proposed openstack/os-vif: Drop MANIFEST.in - it's not needed by pbr  https://review.openstack.org/40399503:35
*** jtollet has quit IRC03:37
openstackgerritCao Xuan Hoang proposed openstack/neutron-fwaas: [PoC] Netlink solution to improve FWaaS performance  https://review.openstack.org/38965403:47
reediparmax : nothing, I was looking up the documents for OSC and found the information which you shared above... thanks ...03:48
*** baoli has quit IRC03:48
*** wolverineav has joined #openstack-neutron03:48
*** liuyulong__ is now known as liuyulong03:49
*** wolverineav has quit IRC03:53
*** Jeffrey4l has quit IRC03:54
*** Jeffrey4l has joined #openstack-neutron03:55
*** stevemar__ has joined #openstack-neutron03:55
*** yuanying has joined #openstack-neutron03:59
*** nicolasbock has quit IRC04:02
*** mdnadeem has joined #openstack-neutron04:03
*** nplanel has joined #openstack-neutron04:06
*** trinaths has quit IRC04:06
*** tuanluong has joined #openstack-neutron04:06
*** Dong_Zhi has joined #openstack-neutron04:06
*** wolverineav has joined #openstack-neutron04:08
*** stevemar__ has quit IRC04:11
*** wolverineav has quit IRC04:13
*** thorst_ has joined #openstack-neutron04:14
*** armax has quit IRC04:16
*** adriant has quit IRC04:19
*** thorst_ has quit IRC04:22
*** catinthe_ has joined #openstack-neutron04:23
*** catintheroof has quit IRC04:26
*** catinthe_ has quit IRC04:26
*** catintheroof has joined #openstack-neutron04:27
*** catintheroof has quit IRC04:27
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: gate_hook: Remove a stale comment  https://review.openstack.org/40402004:29
*** oanson has quit IRC04:30
*** manjong has joined #openstack-neutron04:32
openstackgerritNguyen Phuong An proposed openstack/neutron-specs: (Operator-only) Logging API for security groups  https://review.openstack.org/20350904:34
*** bkopilov has joined #openstack-neutron04:35
*** wolverineav has joined #openstack-neutron04:38
*** yamahata has quit IRC04:38
*** dsneddon has quit IRC04:39
*** dsneddon has joined #openstack-neutron04:41
*** reedip has quit IRC04:42
*** hichihara has joined #openstack-neutron04:42
*** wolverineav has quit IRC04:42
*** iyamahat has quit IRC04:43
*** numans has quit IRC04:48
*** baoli has joined #openstack-neutron04:49
*** tidwellr has joined #openstack-neutron04:57
*** wolverineav has joined #openstack-neutron04:58
*** sudipto has quit IRC04:59
*** sudipto_ has quit IRC04:59
*** sam_nowitzki has joined #openstack-neutron05:00
*** longlq_ has quit IRC05:00
*** baoli has quit IRC05:01
*** wolverineav has quit IRC05:02
*** numans has joined #openstack-neutron05:04
*** shashank_hegde has joined #openstack-neutron05:05
*** prateek has joined #openstack-neutron05:05
*** gkadam has joined #openstack-neutron05:06
*** dkehn has quit IRC05:07
*** wolverineav has joined #openstack-neutron05:07
*** dkehn has joined #openstack-neutron05:09
*** moshele has joined #openstack-neutron05:09
*** mkolesni has joined #openstack-neutron05:11
*** shashank_hegde has quit IRC05:12
*** wolverineav has quit IRC05:12
*** ratailor has joined #openstack-neutron05:12
*** gouthamr has joined #openstack-neutron05:12
*** haleyb__ has quit IRC05:12
*** garyk1 has joined #openstack-neutron05:15
*** ramishra has quit IRC05:16
*** wolverineav has joined #openstack-neutron05:17
*** ramishra has joined #openstack-neutron05:18
*** bobmel has joined #openstack-neutron05:19
*** thorst_ has joined #openstack-neutron05:20
*** tidwellr has left #openstack-neutron05:22
*** wolverineav has quit IRC05:22
openstackgerritYushiro FURUKAWA proposed openstack/python-neutronclient: Add FWaaS V2 commands for OSC plugin  https://review.openstack.org/35158205:22
*** bobmel has quit IRC05:23
*** thorst_ has quit IRC05:26
*** wolverineav has joined #openstack-neutron05:27
openstackgerritZainub Wahid proposed openstack/neutron: Make UUIDField actually validate UUIDs  https://review.openstack.org/39315005:28
*** donghao has joined #openstack-neutron05:29
*** sudipto has joined #openstack-neutron05:29
*** sudipto_ has joined #openstack-neutron05:29
*** wolverineav has quit IRC05:31
*** jtollet has joined #openstack-neutron05:33
*** donghao has quit IRC05:33
*** shashank_hegde has joined #openstack-neutron05:35
*** shausy has joined #openstack-neutron05:35
*** gouthamr has quit IRC05:37
openstackgerritHirofumi Ichihara proposed openstack/neutron: Add notify for tag operations  https://review.openstack.org/29813305:37
*** jtollet has quit IRC05:38
*** bks has joined #openstack-neutron05:46
*** gouthamr has joined #openstack-neutron05:46
*** Mr_W has joined #openstack-neutron05:51
*** Mr_W has quit IRC05:53
*** ivasilevskaya has joined #openstack-neutron05:55
*** gouthamr has quit IRC05:56
*** wolverineav has joined #openstack-neutron05:56
*** gtrxcb has joined #openstack-neutron05:58
*** wolverineav has quit IRC06:01
*** gcb has quit IRC06:08
*** gcb has joined #openstack-neutron06:09
*** haleyb__ has joined #openstack-neutron06:09
*** hichihara has quit IRC06:10
*** achadha has joined #openstack-neutron06:11
*** achadha has quit IRC06:11
*** gvrangan has quit IRC06:11
*** achadha has joined #openstack-neutron06:11
*** Jeffrey4l has quit IRC06:12
*** Jeffrey4l has joined #openstack-neutron06:12
*** haleyb__ has quit IRC06:14
*** trinaths has joined #openstack-neutron06:14
*** shashank_hegde has quit IRC06:16
*** ushkalim has quit IRC06:16
*** wolverineav has joined #openstack-neutron06:16
*** jtollet has joined #openstack-neutron06:18
*** itzikb_ has joined #openstack-neutron06:20
*** wolverineav has quit IRC06:21
*** gcb has quit IRC06:21
*** jaosorior has joined #openstack-neutron06:22
*** jtollet has quit IRC06:23
*** tflynn has joined #openstack-neutron06:24
*** tflynn has quit IRC06:24
*** tflynn has joined #openstack-neutron06:24
*** irenab has joined #openstack-neutron06:24
*** thorst_ has joined #openstack-neutron06:25
*** wolverineav has joined #openstack-neutron06:26
*** shashank_hegde has joined #openstack-neutron06:27
*** wolverineav has quit IRC06:31
openstackgerritMerged openstack/python-neutronclient: Disallow specifying name for meter-label-rule  https://review.openstack.org/35216906:31
*** anilvenkata has joined #openstack-neutron06:31
*** thorst_ has quit IRC06:32
*** dsneddon has quit IRC06:33
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: l3_db: Put next-hop finding code into a separate method  https://review.openstack.org/40403406:34
*** tflynn has quit IRC06:35
*** wolverineav has joined #openstack-neutron06:36
*** tflynn has joined #openstack-neutron06:36
*** FunTara has joined #openstack-neutron06:39
*** gvrangan has joined #openstack-neutron06:40
*** wolverineav has quit IRC06:41
*** gcb has joined #openstack-neutron06:41
*** oanson has joined #openstack-neutron06:41
openstackgerritOmer Anson proposed openstack/neutron: Use pyroute2 to read routing table  https://review.openstack.org/39474106:43
*** mfuruta has quit IRC06:45
*** mfuruta has joined #openstack-neutron06:45
*** wolverineav has joined #openstack-neutron06:46
*** dsneddon has joined #openstack-neutron06:46
*** wolverineav has quit IRC06:50
openstackgerritMerged openstack/neutron: Reuse plugin's update method for changing trunk status  https://review.openstack.org/39162006:51
openstackgerritYushiro FURUKAWA proposed openstack/python-neutronclient: Add FWaaS V2 commands for OSC plugin  https://review.openstack.org/35158206:51
openstackgerritMerged openstack/neutron: ovs-agent: Close ryu app on all exceptions  https://review.openstack.org/40256906:52
*** josecastroleon has joined #openstack-neutron06:53
*** wolverineav has joined #openstack-neutron06:56
*** ijw has joined #openstack-neutron06:56
*** baoli has joined #openstack-neutron06:58
*** wolverineav has quit IRC07:00
*** yamamo___ has quit IRC07:02
*** baoli has quit IRC07:02
*** ihrachys has joined #openstack-neutron07:03
*** jaosorior has quit IRC07:07
*** dsneddon has quit IRC07:07
*** eezhova has joined #openstack-neutron07:07
*** ihrachys has quit IRC07:08
*** jaosorior has joined #openstack-neutron07:08
*** haleyb__ has joined #openstack-neutron07:10
*** iranzo has joined #openstack-neutron07:11
*** oshvartz has joined #openstack-neutron07:11
*** kevo has joined #openstack-neutron07:13
*** haleyb__ has quit IRC07:15
*** bobmel has joined #openstack-neutron07:20
*** tflynn_ has joined #openstack-neutron07:20
openstackgerritZainub Wahid proposed openstack/neutron: Make UUIDField actually validate UUIDs  https://review.openstack.org/39315007:20
*** yuanying has quit IRC07:20
*** pcaruana has joined #openstack-neutron07:21
*** sridharg has joined #openstack-neutron07:22
*** anilvenkata is now known as anilvenkata_afk07:22
*** dsneddon has joined #openstack-neutron07:22
*** tflynn has quit IRC07:23
*** bobmel has quit IRC07:25
*** wolverineav has joined #openstack-neutron07:25
*** yuanying has joined #openstack-neutron07:26
*** thorst_ has joined #openstack-neutron07:28
openstackgerritMerged openstack/neutron: Revert "Deprecate SR-IOV 'physical_device_mappings' config option"  https://review.openstack.org/40368607:28
*** iyamahat has joined #openstack-neutron07:29
*** wolverineav has quit IRC07:30
*** tflynn_ has quit IRC07:33
*** eezhova has quit IRC07:35
*** lihi has joined #openstack-neutron07:35
*** thorst_ has quit IRC07:36
*** ctayal has joined #openstack-neutron07:36
*** korzen has joined #openstack-neutron07:36
*** TuHV_ has joined #openstack-neutron07:37
*** dimak has joined #openstack-neutron07:37
*** eezhova has joined #openstack-neutron07:37
*** TuHV has quit IRC07:38
*** yamamot__ has joined #openstack-neutron07:39
*** yamahata has joined #openstack-neutron07:46
*** ihrachys has joined #openstack-neutron07:47
*** jtollet has joined #openstack-neutron07:50
*** andreas_s has joined #openstack-neutron07:51
*** nherciu has joined #openstack-neutron07:54
*** baoli has joined #openstack-neutron07:59
*** abregman has joined #openstack-neutron08:02
*** iyamahat has quit IRC08:03
*** gouthamr has joined #openstack-neutron08:03
*** achadha_ has joined #openstack-neutron08:04
*** baoli has quit IRC08:04
*** achadha has quit IRC08:04
*** achadha has joined #openstack-neutron08:07
*** iyamahat has joined #openstack-neutron08:07
*** matrohon has joined #openstack-neutron08:08
*** lezbar has joined #openstack-neutron08:08
*** ihrachys has quit IRC08:10
*** kobis has joined #openstack-neutron08:10
*** achadha_ has quit IRC08:10
*** haleyb__ has joined #openstack-neutron08:11
*** Dong_Zhi has quit IRC08:12
*** ataraday has joined #openstack-neutron08:12
*** akamyshnikova has left #openstack-neutron08:12
*** tmorin has joined #openstack-neutron08:15
*** ayogi has joined #openstack-neutron08:15
*** haleyb__ has quit IRC08:15
*** rmart04 has joined #openstack-neutron08:16
openstackgerritHa Van Tu proposed openstack/neutron-fwaas: [PoC] Netlink solution to improve FWaaS performance  https://review.openstack.org/38965408:17
*** ralonsoh has joined #openstack-neutron08:17
*** lezbar has quit IRC08:17
*** eezhova has quit IRC08:17
*** yuanying has quit IRC08:17
*** nmagnezi has joined #openstack-neutron08:19
*** iyamahat has quit IRC08:19
*** mickeys has quit IRC08:19
*** donghao has joined #openstack-neutron08:19
*** lezbar has joined #openstack-neutron08:19
*** mickeys has joined #openstack-neutron08:20
*** nmagnezi has quit IRC08:21
*** mickeys has quit IRC08:24
*** donghao has quit IRC08:25
*** amoralej|off is now known as amoralej08:33
*** jlibosva has joined #openstack-neutron08:33
*** thorst_ has joined #openstack-neutron08:34
*** qwebirc899 has joined #openstack-neutron08:36
*** irenab has quit IRC08:37
*** ijw has quit IRC08:38
*** jpena|off is now known as jpena08:38
*** bjornar_ has joined #openstack-neutron08:39
*** irenab has joined #openstack-neutron08:39
*** thorst_ has quit IRC08:41
*** qwebirc899 has quit IRC08:42
*** tbachman has joined #openstack-neutron08:42
*** pgadiya has joined #openstack-neutron08:43
*** eezhova has joined #openstack-neutron08:43
*** zhhuabj has quit IRC08:44
*** anilvenkata_afk is now known as anilvenkata08:44
*** ihrachys has joined #openstack-neutron08:46
*** ihrachys has quit IRC08:49
*** gtrxcb has quit IRC08:49
*** mfuruta has quit IRC08:49
*** ekuris has joined #openstack-neutron08:49
*** shashank_hegde has quit IRC08:50
*** kashyap has joined #openstack-neutron08:54
*** ctayal has quit IRC08:54
kashyapI'm trying to setup as fewer Neutron services as possible to "minify" a DevStack setup.  Among these, what can I elide?  'quantum,q-svc,q-agt,q-dhcp,q-l3,q-meta'08:54
*** rcernin has joined #openstack-neutron08:56
*** zhhuabj has joined #openstack-neutron08:56
*** rossella_s has joined #openstack-neutron08:58
*** shashank_hegde has joined #openstack-neutron08:59
jschwarzkashyap, hey08:59
*** shashank_hegde has quit IRC08:59
*** zzzeek has quit IRC09:00
kashyapOkay, on a Fresh Fedora 25, Neutron alone takes up a whopping 770 MB09:00
kashyap406.1 MB (neutron-server) + 77 MB (neutron-openvswitch)+ 65.4 (neutron-rootwrap) + 58.6 (neutron-l3-agent) + 58.6 (neutron-dhcp-agent) + 54.9 (neutron-metadata) + 50.1 (neutron-ns-meta) == 770.7 MB09:00
kashyapjschwarz: Heya09:00
*** zzzeek has joined #openstack-neutron09:00
jschwarzkashyap, how many nodes are you running? what are you planning to put on those nodes?09:00
*** baoli has joined #openstack-neutron09:00
kashyapjschwarz: Normally I wouldn't care about the memory usage.  But I'm doing an exercise where I'm condensing a minimal DevStack into a 300MB file (xz / qcow2 compressed)09:00
kashyapjschwarz: It's a single DevStack node09:00
*** pmannidi has quit IRC09:00
kashyapWith these services:09:01
kashyapENABLED_SERVICES=g-api,key,n-api,n-cpu,n-sch,n-cond,mysql,rabbit,dstat,quantum,q-svc,q-agt,q-dhcp,q-l3,q-meta09:01
kashyapNOVA_ENABLED_APIS=osapi_compute,metadata09:01
openstackgerritKevin Benton proposed openstack/neutron: Skip larger than /64 subnets in DHCP agent  https://review.openstack.org/39801609:01
kashyapjschwarz: I already also disabled config_drive in Nova.09:01
kashyap[[post-config|$NOVA_CONF]]09:01
kashyap[DEFAULT]09:01
kashyapforce_config_drive = False09:01
jschwarzkashyap, that's 770MB on the RAM, right?09:02
kashyapjschwarz: Yep09:02
iwamotojlibosva, ping09:02
jschwarzkashyap, well, neutron-server deals with the API, DB and RPC to agents09:03
*** tbachman has quit IRC09:03
* kashyap bbiab; coffee is finished, goes to bring some quickly09:03
jschwarzkashyap, neutron-openvswitch is for l2 connectivity and neutron-l3-agent is for l3 (and neutron-rootwrap is for, well, rootwrap)09:03
kashyapjschwarz: Sure :-) I'm not denying it.  Is 'neutron-metadata' mandatory?  (I guess "no" if you have config_drive enabled)09:03
jschwarzkashyap, I'm not sure what config_drive means09:04
* kashyap back in 10 min09:04
jschwarzkashyap, ack09:04
*** baoli has quit IRC09:05
*** jtollet has quit IRC09:06
openstackgerritAnn Taraday proposed openstack/neutron: [WIP]New enginefacade for networks, subnets.  https://review.openstack.org/40275009:06
*** ushkalim has joined #openstack-neutron09:08
*** jtollet has joined #openstack-neutron09:08
*** jtollet has quit IRC09:09
*** stevemar__ has joined #openstack-neutron09:12
*** haleyb__ has joined #openstack-neutron09:12
*** mvk has quit IRC09:13
*** gvrangan has quit IRC09:14
*** stevemar__ has quit IRC09:16
openstackgerritYushiro FURUKAWA proposed openstack/neutron-fwaas: Fix removing rule_association on updating a policy  https://review.openstack.org/40410309:17
*** haleyb__ has quit IRC09:17
*** yamahata has quit IRC09:18
*** jtollet has joined #openstack-neutron09:18
*** mickeys has joined #openstack-neutron09:20
*** oreillyd has joined #openstack-neutron09:22
*** mickeys has quit IRC09:25
*** lucas-afk is now known as lucasagomes09:25
*** jtollet has quit IRC09:29
*** ababich has quit IRC09:29
*** jtollet has joined #openstack-neutron09:29
*** jtollet has quit IRC09:30
*** jtollet has joined #openstack-neutron09:31
*** rossella_s has quit IRC09:31
garyk1anyone familiar with neutron-ovs-cleanup ?09:31
*** rossella_s has joined #openstack-neutron09:31
kashyapjschwarz: "config_drive" == http://docs.openstack.org/user-guide/cli-config-drive.html09:33
jschwarzkashyap, ah09:34
jschwarzkashyap, then yes, you can remove q-meta for a whooping 50.1MB (:O) per subnet09:34
*** abregman has quit IRC09:35
jschwarzkashyap, neutron-dhcp-agent are used for providing an instance with an IP address (dnsmasq) so you probably need that as well as neutron-openvswitch-agent09:35
*** abregman has joined #openstack-neutron09:35
jschwarzkashyap, might be able to leave out neutron-l3-agent but then you don't have routers, external connectivity, etc09:35
kashyapjschwarz: Forgot to note - I don't have 'config_drive' enabled.09:35
jschwarzkashyap, ah09:36
kashyapjschwarz: Hmm, about neutron-l3-agent...09:36
jschwarzkashyap, lol then yeah, you need q-meta as well :P09:36
jschwarzkashyap, yes?09:37
kashyapThis is just a one-off "OpenStack in a disk image" that just lets people try something cool once they fire up the disk image.  :-)09:37
kashyapjschwarz: Yes, you're right09:37
jschwarzkashyap, well, then the question is whether or not "ping to google from VM" is cool enough ;-)09:37
kashyapjschwarz: Don't demotivate please...09:38
jschwarzkashyap, so if you have 300MB HD space you still need the entirety of openstack/neutron09:39
jschwarzkashyap, the question is how much RAM you have available09:39
*** thorst_ has joined #openstack-neutron09:39
*** kevo has quit IRC09:39
jschwarzkashyap, if it's reasonable to require 2GBs of RAM for the image, then the additional 58MB for the neutron-l3-agent is negligible.09:40
jschwarzkashyap, the rest is kinda mandatory for l2 and IP connectivity09:41
kashyapjschwarz: Yeah, I'm profiling the entire memory usage09:41
jschwarzkashyap, though 406MB for neutron-server seems high - what plugins do you have in your neutron.conf?09:41
kashyapYeah, that's a monster09:41
*** davidsha has joined #openstack-neutron09:42
*** hfu has quit IRC09:42
*** ihrachys has joined #openstack-neutron09:44
*** lezbar has quit IRC09:44
*** mvk has joined #openstack-neutron09:46
*** thorst_ has quit IRC09:47
*** yfried has joined #openstack-neutron09:47
*** openstackgerrit has quit IRC09:48
*** openstackgerrit has joined #openstack-neutron09:49
*** jtollet has quit IRC09:49
*** john-davidge has joined #openstack-neutron09:51
openstackgerritFrancesco Santoro proposed openstack/os-vif: os-vif: add vif_name to VIFVHostUser class  https://review.openstack.org/39022509:52
*** fzdarsky|afk has joined #openstack-neutron09:58
*** achadha has quit IRC09:58
*** jschwarz has quit IRC09:59
jlibosvaiwamoto: hi09:59
iwamotojlibosva, hi new test lieutenant :)09:59
iwamotojlibosva, have you tried ovsfw+tempest?10:00
*** kashyap has left #openstack-neutron10:00
jlibosvaiwamoto: hi :) I did during ovsfw development - they don't cover much of security groups though, it helped on some basic ipv6 stuff10:00
*** baoli has joined #openstack-neutron10:01
jlibosvaiwamoto: do you see any issues? I'd also like to introduce ovsfw to some of tempest job10:01
iwamotojlibosva, so it worked before?10:01
iwamotojlibosva, https://review.openstack.org/#/c/399400/10:01
jlibosvaiwamoto: it did, it doesn't know?10:01
openstackgerritGergely Csatari proposed openstack/neutron-lib: api-ref: binding:vnic_type added to Create Port  https://review.openstack.org/40378910:02
iwamotojlibosva, local testing suggested ovsfw uses stale ofport number10:02
*** jtollet has joined #openstack-neutron10:03
*** wu_ng has quit IRC10:03
iwamotojlibosva, after server soft reboot, ssh connection doesn't work. but i'm not sure if a soft reboot changes ofport number10:04
iwamotojlibosva, i'll look into tomorrow, but if you know anything about this, it'll help10:05
jlibosvaiwamoto: hmm, I'll try to reproduce it locally10:05
jlibosvaiwamoto: I don't think reboot changes ofport, it doesn't do anything with the tap10:05
jlibosvaI *thik*10:05
jlibosvathink*10:05
*** baoli has quit IRC10:06
iwamotoi spent some time looking at differences in >100 ovs flows, but probably there are no meaningful differences10:07
* iwamoto confused10:07
*** jschwarz has joined #openstack-neutron10:10
iwamotojlibosva, on gate testing, at least an experimental ovsfw job needs to be set up, imo ;)10:10
*** reedip has joined #openstack-neutron10:11
*** odyssey4me has left #openstack-neutron10:11
*** limao has quit IRC10:11
*** haleyb__ has joined #openstack-neutron10:13
*** fzdarsky|afk is now known as fzdarsky10:13
iwamotobye10:13
*** iwamoto has quit IRC10:13
jlibosvaiwamoto: I agree10:13
*** haleyb__ has quit IRC10:18
*** manjong has quit IRC10:19
*** sambetts|afk is now known as sambetts10:19
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron: Add IPLink class to Windows ip_lib implementation  https://review.openstack.org/40379910:21
*** mickeys has joined #openstack-neutron10:21
*** jtollet has quit IRC10:21
*** TuHV_ has quit IRC10:21
*** bobmel has joined #openstack-neutron10:22
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron: Fix broken Windows compatibility in ovs_lib  https://review.openstack.org/40380410:22
*** manjong has joined #openstack-neutron10:24
*** hoangcx has quit IRC10:26
*** bobmel has quit IRC10:26
*** mickeys has quit IRC10:27
*** manjong has quit IRC10:27
*** tovin07 has quit IRC10:29
*** stanzgy has quit IRC10:29
*** tpsilva has joined #openstack-neutron10:34
*** anilvenkata is now known as anilvenkata_afk10:34
*** donghao has joined #openstack-neutron10:38
*** jtollet has joined #openstack-neutron10:43
*** xiaofandh12 has joined #openstack-neutron10:43
*** xiaofandh12 has quit IRC10:44
*** thorst_ has joined #openstack-neutron10:45
*** donghao has quit IRC10:47
*** jtollet has quit IRC10:49
*** manjong has joined #openstack-neutron10:51
*** thorst_ has quit IRC10:51
*** bfernando has joined #openstack-neutron10:51
*** fnaval_ has joined #openstack-neutron10:52
*** jtollet has joined #openstack-neutron10:53
*** bobmel has joined #openstack-neutron10:54
*** fnaval has quit IRC10:54
*** manjong has quit IRC10:55
*** gouthamr has quit IRC10:58
*** udesale has quit IRC10:58
*** tbachman has joined #openstack-neutron10:59
*** achadha has joined #openstack-neutron10:59
*** baoli has joined #openstack-neutron11:02
*** sudipto_ has quit IRC11:03
*** sudipto has quit IRC11:03
*** abregman has quit IRC11:03
openstackgerritDaniel Alvarez proposed openstack/neutron: Kill processes when cleaning up namespaces  https://review.openstack.org/40214011:03
*** mhickey has joined #openstack-neutron11:03
*** achadha has quit IRC11:04
*** jtollet has quit IRC11:07
*** baoli has quit IRC11:07
*** mvk has quit IRC11:08
*** tbachman_ has joined #openstack-neutron11:14
*** haleyb__ has joined #openstack-neutron11:14
*** tbachman has quit IRC11:14
*** tbachman_ is now known as tbachman11:14
*** haleyb__ has quit IRC11:19
*** mvk has joined #openstack-neutron11:20
*** bks has quit IRC11:21
*** mickeys has joined #openstack-neutron11:23
*** anilvenkata_afk is now known as anilvenkata11:25
*** mickeys has quit IRC11:27
*** hfu has joined #openstack-neutron11:28
*** hfu has quit IRC11:30
*** ociuhandu has joined #openstack-neutron11:31
*** mdnadeem has quit IRC11:34
openstackgerritgaryk proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/40114611:35
*** Alex_Stef has joined #openstack-neutron11:37
*** boden has joined #openstack-neutron11:37
*** tbachman has quit IRC11:38
*** nicolasbock has joined #openstack-neutron11:38
Alex_Stefhaleyb, Hi , https://bugs.launchpad.net/neutron/+bug/1645317 this is me :) , ping me if u need additional ingo11:39
openstackLaunchpad bug 1645317 in neutron "FIP and router iptables are not apdated when associated port ip address is changed " [Undecided,New]11:39
*** josecastroleon has quit IRC11:42
*** donghao has joined #openstack-neutron11:45
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: l3_db: Put next-hop finding code into a separate method  https://review.openstack.org/40403411:45
*** yamamot__ has quit IRC11:46
*** thorst_ has joined #openstack-neutron11:49
*** mdnadeem has joined #openstack-neutron11:50
*** donghao has quit IRC11:50
*** bjornar_ has quit IRC11:54
sambettsarmax: Hey, I see you abandoned your tempest patch, I was struggling to give feedback on it, but I was keeping up with the discussions, did you see the mailer discussion around drivers?11:54
*** andymaier has joined #openstack-neutron11:55
*** thorst_ has quit IRC11:57
*** andymaier has quit IRC11:57
openstackgerritIhar Hrachyshka proposed openstack/neutron: Integration of RouterRoute OVO  https://review.openstack.org/36707111:57
*** baoli has joined #openstack-neutron12:03
ihrachysajo: would be cool to see you bless https://review.openstack.org/#/c/400660/12:04
*** abregman has joined #openstack-neutron12:06
*** baoli has quit IRC12:07
*** sridharg has quit IRC12:09
*** rtheis has joined #openstack-neutron12:10
*** davidsha has quit IRC12:11
*** gkadam has quit IRC12:13
*** dsneddon has quit IRC12:13
*** wolverineav has joined #openstack-neutron12:14
*** tuanluong has quit IRC12:14
*** haleyb__ has joined #openstack-neutron12:15
*** bkopilov has quit IRC12:17
*** annp has quit IRC12:18
openstackgerritAnn Taraday proposed openstack/neutron: [WIP] Use new enginefacade for dvr mac db  https://review.openstack.org/40418212:19
*** haleyb__ has quit IRC12:20
*** bjornar_ has joined #openstack-neutron12:24
*** josecastroleon has joined #openstack-neutron12:28
openstackgerritReedip proposed openstack/neutron-lib: Add converter to convert IPv6 addresses to canonical format  https://review.openstack.org/35767012:30
*** sridharg has joined #openstack-neutron12:33
*** jtollet has joined #openstack-neutron12:36
*** catinthe_ has joined #openstack-neutron12:37
openstackgerritReedip proposed openstack/neutron-lib: Add validator for device owner  https://review.openstack.org/38177912:40
*** ratailor has quit IRC12:40
*** fnaval has joined #openstack-neutron12:42
*** gouthamr has joined #openstack-neutron12:42
*** stevemar__ has joined #openstack-neutron12:44
*** limao has joined #openstack-neutron12:45
*** fnaval_ has quit IRC12:45
*** wolverineav has quit IRC12:46
*** mdnadeem has quit IRC12:46
*** thorst_ has joined #openstack-neutron12:49
*** mnasiadka has joined #openstack-neutron12:50
*** lucasagomes is now known as lucas-hungry12:52
*** prateek has quit IRC12:54
*** nyechiel has joined #openstack-neutron12:54
*** phil_h has quit IRC12:54
*** tlian has joined #openstack-neutron12:55
*** hfu has joined #openstack-neutron12:55
*** gcb has quit IRC12:56
openstackgerritLIU Yulong proposed openstack/neutron-dynamic-routing: Let the bgp_plugin to query floating IP bgp next_hop  https://review.openstack.org/37231012:57
*** amuller has joined #openstack-neutron12:57
*** edand has joined #openstack-neutron13:00
*** achadha has joined #openstack-neutron13:01
*** trown|outtypewww is now known as trown13:03
*** baoli has joined #openstack-neutron13:03
*** limao_ has joined #openstack-neutron13:04
*** xinliang has quit IRC13:06
*** limao has quit IRC13:07
*** achadha has quit IRC13:07
*** amoralej is now known as amoralej|lunch13:08
*** baoli has quit IRC13:08
openstackgerritArtur Korzeniewski proposed openstack/neutron: devref: docs about how to use NeutronDbObject.  https://review.openstack.org/33651813:10
*** liuyulong_ has joined #openstack-neutron13:10
*** manikanta_tadi has quit IRC13:11
*** phil_h has joined #openstack-neutron13:11
*** edmondsw has joined #openstack-neutron13:11
*** ayogi has quit IRC13:13
*** pgadiya has quit IRC13:14
*** trinaths has left #openstack-neutron13:14
*** fnaval_ has joined #openstack-neutron13:14
*** fnaval has quit IRC13:16
*** stevemar__ has quit IRC13:19
*** xinliang has joined #openstack-neutron13:19
*** stevemar__ has joined #openstack-neutron13:20
*** FunTara has quit IRC13:20
*** yamamoto has joined #openstack-neutron13:20
*** yamamoto_ has joined #openstack-neutron13:22
*** andymaier has joined #openstack-neutron13:23
*** stevemar__ has quit IRC13:24
*** yamamoto has quit IRC13:25
*** haleyb_ has joined #openstack-neutron13:25
*** mickeys has joined #openstack-neutron13:25
*** davidsha has joined #openstack-neutron13:26
*** hoangcx has joined #openstack-neutron13:28
*** andymaier has quit IRC13:29
*** mickeys has quit IRC13:29
*** srobert has joined #openstack-neutron13:31
*** rossella_s has quit IRC13:31
*** rossella_s has joined #openstack-neutron13:31
*** tbachman has joined #openstack-neutron13:32
openstackgerritJohn Schwarz proposed openstack/neutron: Migrate device_owner for router's interface  https://review.openstack.org/40421513:33
jschwarzataraday, hey13:34
*** leitan has joined #openstack-neutron13:34
ataradayjschwarz, hi13:34
jschwarzataraday, congratulations on the new nickname! XD13:34
ataradayjschwarz, xD13:34
jschwarzataraday, could you have a look at https://review.openstack.org/#/c/404215/ ?13:35
jschwarzit's a purely DB patch so it's right up your alley :)13:35
*** jpena is now known as jpena|lunch13:35
leitanHi guys, is there a chance to disable snat by default as a router property or i have to hack horizon for no to set it ?13:35
leitanim having a high public ip consumption due to this, and its a feature that we dont provide, but since the snat l3 agent is the responsable for scheduling i need to have a dvr_snat enabled agent ... so im kinda strugling with that13:37
jschwarzhaleyb, ^ :)13:37
*** tbachman has quit IRC13:38
haleyb_jschwarz: will look, thanks13:38
*** tbachman has joined #openstack-neutron13:38
leitani went around the router_dict from neutronclient to try to pass it as false always ... with no luck13:39
haleyb_leitan: in newton you can use subnet types to have the snat interface use an IP from a private subnet13:40
*** sahid has joined #openstack-neutron13:41
leitanhaleyb_: yeah, i know its a great feature, but this specific cloud is running the old migthy kilo with dvr13:41
*** wolverineav has joined #openstack-neutron13:43
haleyb_kilo, you might be stuck13:43
*** mdnadeem has joined #openstack-neutron13:43
*** dave-mccowan has joined #openstack-neutron13:43
openstackgerritMerged openstack/neutron-lib: Make the get_random_mac more versatile  https://review.openstack.org/40040813:43
*** wolverin_ has joined #openstack-neutron13:43
wdeclercqleitan, you probably will have to hack around here: https://github.com/openstack/horizon/blob/master/openstack_dashboard/dashboards/project/routers/forms.py#L96  There are no variables from local_settings or anything being used, so I guess you have to pass it manually.13:44
openstackgerritAnn Taraday proposed openstack/neutron: [WIP] Use new enginefacade for dvr mac db  https://review.openstack.org/40418213:45
leitanhaleyb_: poor grandpa kilo, ill keep looking for a way to hack around it, but just to clarify i manually updated a router to snat_false, and when i set the gateway, a public IP gets assigned anyways to the router, thats a normal behaviour ?13:45
*** hfu has quit IRC13:45
*** zhhuabj has quit IRC13:45
leitanwdeclercq: yes, and also change the policy.json from neutron, since its and admin_only action13:45
*** sudipto has joined #openstack-neutron13:45
*** sudipto_ has joined #openstack-neutron13:45
*** jtollet has quit IRC13:45
*** tmorin has quit IRC13:46
*** tmorin has joined #openstack-neutron13:46
*** udesale has joined #openstack-neutron13:47
haleyb_leitan: i believe so, it might just not create the iptables rules for snat, been a while since i looked at that code13:47
*** wolverineav has quit IRC13:47
*** itzikb_ has quit IRC13:48
*** jtollet has joined #openstack-neutron13:48
leitanhaleyb_: hmmm, so snat_enable = False will not fix my public ip allocation issue since its going to take a public ip anyways from the pool13:49
*** fnaval has joined #openstack-neutron13:53
*** jtollet has quit IRC13:53
*** chandanc has joined #openstack-neutron13:53
*** lucas-hungry is now known as lucasagomes13:54
leitanthanks anyway haleyb_ and wdeclercq13:54
*** thorst_ has quit IRC13:54
*** sridharg has quit IRC13:54
*** thorst_ has joined #openstack-neutron13:54
*** bobmel has quit IRC13:55
haleyb_leitan: just looked at code, snat_enabled will just trigger the iptables snat rules, doesn't change IP allocation.  So all these subnets are isolated, floating IP only?  I don't know what else is using snat unnoticed, like dns traffic?13:55
*** fnaval_ has quit IRC13:55
*** bobmel has joined #openstack-neutron13:56
*** ralonsoh_ has joined #openstack-neutron13:56
*** dsneddon has joined #openstack-neutron13:56
leitanhaleyb_: all the tenant networks are vxlan isolated, access is only allowed trough floating ips, so i was looking on how to avoid snat from stealing one public ip from the floating ip range when i "setgateway" on the tenant router13:56
leitanto use on the snat namespace13:57
*** ralonsoh has quit IRC13:57
*** jheroux has joined #openstack-neutron13:58
openstackgerritReedip proposed openstack/neutron-lib: Add validator for device owner  https://review.openstack.org/38177913:58
haleyb_leitan: right, but have you tried booting an instance without that IP configured, or with it being a blackhole somehow?  I'm curious if it's being used at all13:59
*** xionchen_ has joined #openstack-neutron13:59
*** mhickey has quit IRC13:59
leitanhaleyb_: yes, besides if the instance has a floating ip or not assigned, the instant i set the gateway to the external network on the tenant router it assings an ip address for it on the external public network14:00
*** dsneddon has quit IRC14:01
*** bobmel has quit IRC14:01
leitanbesides snat_enable is set to true or false, thing that now makes more sense to me with what youre saying14:01
*** mkolesni has quit IRC14:01
*** lezbar has joined #openstack-neutron14:01
*** oshvartz has quit IRC14:01
*** bobmel has joined #openstack-neutron14:02
*** ajo has left #openstack-neutron14:02
*** baoli has joined #openstack-neutron14:03
*** zhhuabj has joined #openstack-neutron14:03
*** rmart04_ has joined #openstack-neutron14:04
*** rmart04 has quit IRC14:04
*** rmart04_ is now known as rmart0414:04
*** lindycoder has joined #openstack-neutron14:05
*** jtollet has joined #openstack-neutron14:07
*** Worry has joined #openstack-neutron14:09
*** janzian has joined #openstack-neutron14:11
*** zkassab has joined #openstack-neutron14:12
*** donghao has joined #openstack-neutron14:12
*** abregman is now known as abregman|afk14:13
*** jperry has joined #openstack-neutron14:14
*** sam_nowitzki has quit IRC14:15
*** bfernando has quit IRC14:15
*** stevemar__ has joined #openstack-neutron14:17
*** abregman|afk is now known as abregman14:17
*** jamesdenton has joined #openstack-neutron14:17
*** ajo has joined #openstack-neutron14:17
ajoihrachys thanks for pinging about https://review.openstack.org/#/c/400660/4   it's almost correct :)14:18
ajobut there's a part being deleted (policy-create) which I believe we should not delete14:18
*** sballe_ has quit IRC14:18
*** trevormc has joined #openstack-neutron14:18
*** sballe_ has joined #openstack-neutron14:19
ihrachysajo: please check my latest comment14:19
ihrachysajo: the file is not attached anywhere14:19
ihrachysajo: hence it's removed14:19
ajohmm14:19
jschwarzkevinbenton, ping14:19
jschwarzkevinbenton, re: https://review.openstack.org/#/c/404215/114:19
ihrachysajo: I grepped, no occurrence in the tree14:19
ajoihrachys may be correct patch is attaching it then to create?14:19
* ajo reads the api doc14:19
ihrachysajo: maybe, but that's a separate thing. and since the snippet is plain wrong...14:19
ajoihrachys , it's also unrelated to patch message ;)14:20
ihrachysajo: you want to split out that bit? leaving the file untouched?14:21
*** wolverin_ has quit IRC14:22
davidshatmorin: ping14:23
ajoihrachys yes14:23
tmorinhi davidsha !  yes ?14:23
*** vijaykc4 has joined #openstack-neutron14:23
*** jamesdenton has quit IRC14:23
*** amoralej|lunch is now known as amoralej14:23
ihrachysajo: are you going to follow up with attaching the snippet?14:23
*** mdnadeem has quit IRC14:23
ajoihrachys yep, almost have it14:24
ihrachysajo: ack, cool14:24
davidshatmorin: Hey, I was just wondering are you still working on: https://bugs.launchpad.net/neutron/+bug/162845514:24
openstackLaunchpad bug 1628455 in neutron "openvswitch native native bridge implementation redefines delete_flows with a different signature" [Medium,In progress] - Assigned to Thomas Morin (tmmorin-orange)14:24
*** mickeys has joined #openstack-neutron14:26
*** zjf has joined #openstack-neutron14:26
davidshatmorin: This was the patch you submitted: https://review.openstack.org/#/c/38032914:26
tmorindavidsha: I still plan to, but haven't had time to revisit the issue in the past 10 days14:26
*** bfernando has joined #openstack-neutron14:26
tmorindavidsha: yes, I have the issue well in mind14:26
davidshatmorin: kk, no problem!14:26
tmorinI'll try to comment on the change to the discuss bw. you and yamamoto14:27
kevinbentonjschwarz: pong14:29
jschwarzkevinbenton, hey14:29
*** mriedem has quit IRC14:29
davidshatmorin: no problem, it isn't a blocker for me but it is something I'm interested in addressing, inconsistent flow commands are a nightmare with writing the flow manager X(14:29
jschwarzkevinbenton, so haleyb suggested we might want to rebase https://review.openstack.org/#/c/399479/ on top of my new patch https://review.openstack.org/#/c/404215/114:29
*** srobert has quit IRC14:29
jschwarzkevinbenton, since we'll want to backport 404215 anyway14:30
jschwarzkevinbenton, what do you think?14:30
tmorindavidsha: indeed, the issue is requiring an ugly workaround in networking-bgpvpn ...14:30
*** mickeys has quit IRC14:30
*** jamesdenton has joined #openstack-neutron14:30
openstackgerritMiguel Angel Ajo proposed openstack/neutron-lib: Fix QoS Policy create json response, and link it  https://review.openstack.org/40424214:32
davidshatmorin: kk, thanks for the response!14:32
jianghuawihrachys, could you help to look into this patch at some time? https://review.openstack.org/#/c/390931/14:32
tmorindavidsha: don't hesitate to take over the change, if you're frustrated by my lack of responsiveness :)14:33
jschwarzkevinbenton, :) thanks14:33
jschwarzhaleyb, can you relook at 404215?14:33
*** mlavalle has joined #openstack-neutron14:34
jianghuawihrachys, the filed bug is: https://bugs.launchpad.net/neutron/+bug/158551014:34
openstackLaunchpad bug 1585510 in oslo.rootwrap "[RFE] openvswitch-agent support rootwrap daemon when hypervisor is XenServer" [Undecided,New]14:34
haleyb_jschwarz: will do14:34
jschwarzmuchos appreciatos!14:34
*** itzikb_ has joined #openstack-neutron14:35
jianghuawihrachys, I need some advice on how I can proceed for this bug. Thanks very much.14:35
*** kun_huang has joined #openstack-neutron14:36
ihrachysjianghuaw: yeah it's in my queue, but best effort. I will eventually get there some time this or next week.14:36
jianghuawihrachys, great. Thanks.14:37
*** manjong has joined #openstack-neutron14:37
davidshatmorin:  Ack, I have some time tomorrow so I might see if I could give it a go. It isn't out of frustration, just interest, it bothers me that the commands are not consistent across ofctl and native :P14:37
*** wolverineav has joined #openstack-neutron14:37
*** zjf has quit IRC14:38
*** dasm has left #openstack-neutron14:38
*** mhickey has joined #openstack-neutron14:39
*** itzikb_ has quit IRC14:40
*** SinaMaleki has joined #openstack-neutron14:41
*** manjong has quit IRC14:41
*** wolverineav has quit IRC14:41
*** liuyulong_ has quit IRC14:42
*** bkopilov has joined #openstack-neutron14:42
*** NeverWorry has joined #openstack-neutron14:42
*** Worry has quit IRC14:42
*** jpena|lunch is now known as jpena14:43
openstackgerritMiguel Angel Ajo proposed openstack/neutron-lib: Fix QoS Policy create json response, and link it  https://review.openstack.org/40424214:43
*** manjong has joined #openstack-neutron14:44
*** moshele has quit IRC14:44
*** irenab has quit IRC14:45
*** moshele has joined #openstack-neutron14:46
*** fnaval has quit IRC14:46
*** dasm_away has joined #openstack-neutron14:46
*** cleong has joined #openstack-neutron14:47
*** kbringard has joined #openstack-neutron14:47
openstackgerritMerged openstack/neutron-vpnaas: Use ExtensionDescriptor from neutron-lib  https://review.openstack.org/40327914:48
*** manjong has quit IRC14:48
ihrachysgaryk: sc68cal: jschwarz noticed that we landed backport in devstack without master fix https://review.openstack.org/#/q/I6f441c6febb5070ad885569d9c798634d0272b6c,n,z14:48
ihrachysplease chime in to unravel the issue14:49
garyk1shit, my bas. i did not see that this was a stable patch. sorry14:50
jschwarzwell, if we merge the master patch fast enough it wont matter, right? :)14:50
*** rmart04 has quit IRC14:51
*** zhhuabj has quit IRC14:51
garyk1i guess i am going to be tarred and featherd.14:51
openstackgerritDaniel Alvarez proposed openstack/neutron: Kill processes when cleaning up namespaces  https://review.openstack.org/40214014:51
garyk1please only cherrypick once a master patch has landed14:51
jschwarzgaryk1, aye - we needed to check it specifically against a DNM, should have marked that one as DNM as well14:52
jschwarzsorryt14:52
jschwarzsorry*14:52
ihrachysjschwarz: you could WIP though ;)14:52
*** rmart04 has joined #openstack-neutron14:52
jschwarzihrachys, yep14:53
*** wolverineav has joined #openstack-neutron14:53
*** NeverWorry has quit IRC14:54
*** abregman is now known as abregman|afk14:54
*** edmondsw has quit IRC14:55
*** zhhuabj has joined #openstack-neutron14:55
*** prateek has joined #openstack-neutron14:56
*** jtollet has quit IRC14:56
*** prateek has quit IRC14:57
*** mriedem has joined #openstack-neutron14:57
*** saggi has joined #openstack-neutron14:57
*** prateek has joined #openstack-neutron14:57
*** dasm_away is now known as dasm14:57
*** trown is now known as trown|brb14:57
*** jtollet has joined #openstack-neutron14:58
openstackgerritJohn Davidge proposed openstack/neutron: Fix DHCP Port Creation on Service Subnets  https://review.openstack.org/39218315:03
*** achadha has joined #openstack-neutron15:04
*** fnaval has joined #openstack-neutron15:05
*** wolverineav has quit IRC15:06
openstackgerritIhar Hrachyshka proposed openstack/neutron: objects: add delete_objects public method  https://review.openstack.org/40383215:06
openstackgerritIhar Hrachyshka proposed openstack/neutron: tests: introduce update_obj_fields method in base objects test class  https://review.openstack.org/40425915:06
*** jaosorior has quit IRC15:07
*** jaosorior has joined #openstack-neutron15:08
*** tbachman has quit IRC15:09
*** achadha has quit IRC15:09
*** Jeffrey4l has quit IRC15:09
openstackgerritKevin Benton proposed openstack/neutron: [WIP]: server-side push notifications for ML2  https://review.openstack.org/38893915:10
openstackgerritKevin Benton proposed openstack/neutron: Skip larger than /64 subnets in DHCP agent  https://review.openstack.org/39801615:10
openstackgerritKevin Benton proposed openstack/neutron: Pass context and resource_type in RPC callback  https://review.openstack.org/40426415:10
openstackgerritKevin Benton proposed openstack/neutron: [wip]: agent-side receiver cache for ML2 OVO  https://review.openstack.org/40426515:10
*** crose has joined #openstack-neutron15:11
*** jhershbe has quit IRC15:11
*** hoangcx has quit IRC15:11
*** slaweq_ has quit IRC15:11
*** slaweq_ has joined #openstack-neutron15:12
*** eezhova has quit IRC15:13
openstackgerritMerged openstack/neutron: Using a new security group in scenario tests  https://review.openstack.org/39183115:14
*** ankur-gupta-f1 has joined #openstack-neutron15:14
*** shausy has quit IRC15:14
*** dasm_away has joined #openstack-neutron15:14
*** garyk1 has quit IRC15:15
*** trown|brb is now known as trown15:15
*** wolverineav has joined #openstack-neutron15:15
*** eezhova has joined #openstack-neutron15:18
*** eezhova has quit IRC15:18
*** dasm has quit IRC15:18
*** dasm_away is now known as dasm15:18
*** tonytan4ever has joined #openstack-neutron15:19
*** wolverineav has quit IRC15:20
*** moshele has quit IRC15:20
*** ekuris has quit IRC15:21
*** mnasiadka has quit IRC15:24
*** TTris has joined #openstack-neutron15:24
*** prateek has quit IRC15:24
*** abregman|afk is now known as abregman15:25
*** rajinir has joined #openstack-neutron15:26
*** mickeys has joined #openstack-neutron15:26
*** chlong has joined #openstack-neutron15:27
*** jhershbe has joined #openstack-neutron15:28
*** mriedem is now known as mriedem_meeting15:29
*** korzen has quit IRC15:29
*** mickeys has quit IRC15:31
*** jprovazn has joined #openstack-neutron15:32
*** mdnadeem has joined #openstack-neutron15:33
*** dmsimard has joined #openstack-neutron15:34
*** vijaykc4 has quit IRC15:35
*** dffco has joined #openstack-neutron15:35
*** wolverineav has joined #openstack-neutron15:35
*** corey_ has joined #openstack-neutron15:40
*** wolverineav has quit IRC15:40
openstackgerritIhar Hrachyshka proposed openstack/neutron: tests: introduce update_obj_fields method in base objects test class  https://review.openstack.org/40425915:40
*** cleong has quit IRC15:40
*** chandanc has quit IRC15:42
anilvenkatashihanzhang, ping15:42
*** Jeffrey4l has joined #openstack-neutron15:44
openstackgerritVictor Morales proposed openstack/neutron: DistributedVirtualRouter mac address to OVO  https://review.openstack.org/30487315:44
*** donghao has quit IRC15:45
*** tlian has quit IRC15:46
openstackgerritIhar Hrachyshka proposed openstack/neutron: objects: add delete_objects public method  https://review.openstack.org/40383215:46
*** oanson has quit IRC15:46
*** ataraday_ has joined #openstack-neutron15:49
haleyb_jschwarz: see my comment on that change and tell me if i'm crazy15:49
jschwarzhaleyb_, you mean, the one on the test file? not crazy at all15:50
*** fnaval has quit IRC15:50
haleyb_jschwarz: yes, that and anil's comment15:51
jschwarzhaleyb_, I can do it (it's already covered in another test but oh well)15:51
jschwarzhaleyb_, I'll submit a new one in a minute15:51
haleyb_jschwarz: which one?  the web ui gives little context15:51
*** Jeffrey4l has quit IRC15:51
*** Jeffrey4l has joined #openstack-neutron15:51
jschwarzhaleyb_, https://github.com/openstack/neutron/blob/master/neutron/tests/unit/db/test_l3_hamode_db.py#L31615:52
dmsimardHi cores, I'm still looking for reviews on https://review.openstack.org/#/c/403253/ to reduce the amount of tests tagged as smoke in lbaas, there's >100 tests and it defeats the purpose intent of smoke testing15:52
jschwarzhaleyb_, nevermind though, it's a small change15:53
*** tlian has joined #openstack-neutron15:53
openstackgerritAnindita Das proposed openstack/neutron: [WIP] Extending ml2_port_binding table PK  https://review.openstack.org/40429315:54
openstackgerritJohn Schwarz proposed openstack/neutron: Migrate device_owner for router's interface  https://review.openstack.org/40421515:54
*** manjong has joined #openstack-neutron15:54
jschwarzhaleyb_, kevinbenton, ^15:54
*** baoli has quit IRC15:55
*** john-davidge has quit IRC15:55
*** udesale has quit IRC15:55
*** fnaval has joined #openstack-neutron15:55
*** baoli has joined #openstack-neutron15:55
haleyb_jschwarz: thanks15:56
jschwarzhaleyb_, :)15:56
*** bkopilov_ has joined #openstack-neutron15:56
openstackgerritIhar Hrachyshka proposed openstack/neutron-vpnaas: Use DB field sizes instead of _MAX_LEN constants  https://review.openstack.org/40331915:57
openstackgerritIhar Hrachyshka proposed openstack/neutron-vpnaas: Remove PLURALS  https://review.openstack.org/40331815:57
*** bkopilov has quit IRC15:58
*** tbachman has joined #openstack-neutron15:58
*** manjong has quit IRC15:59
garykjschwarz: ping me when that patch gets a +116:00
*** mriedem_meeting is now known as mriedem16:00
*** iyamahat has joined #openstack-neutron16:01
*** dsneddon has joined #openstack-neutron16:03
*** rcernin has quit IRC16:03
jschwarzgaryk, sure thing16:03
jschwarzgaryk, thanks16:03
*** jhershbe has quit IRC16:03
*** lihi has quit IRC16:04
*** jaosorior has quit IRC16:05
jschwarzgaryk, btw https://review.openstack.org/#/c/400258/ passed jenkins (only got non-voting stuff to complete) so it only needs another +216:05
jschwarzI added Sean so maybe he'll be the second one16:05
jschwarzihrachys, ^16:05
*** vijaykc4 has joined #openstack-neutron16:06
*** chlong has quit IRC16:07
*** jhershbe has joined #openstack-neutron16:08
*** haleyb__ has joined #openstack-neutron16:09
*** manjong has joined #openstack-neutron16:09
*** kobis has quit IRC16:09
garyksc68cal: can you please see https://review.openstack.org/#/c/400258/16:09
*** SinaMaleki has quit IRC16:10
*** haleyb_ has quit IRC16:11
ihrachysjschwarz: btw we should recheck https://review.openstack.org/#/c/345428/ once both sides of devstack change land. maybe it's the MTU fix we need.16:12
*** ygbo has joined #openstack-neutron16:12
jschwarzihrachys, agreed16:12
*** manjong has quit IRC16:14
ihrachysrossella_s: jlibosva: code reuse in objects tests: https://review.openstack.org/#/c/404259/ would be cool to see it landing so that we can start switching existing patches to the new approach16:14
ihrachysthat chain/zip copy-paste drives me nuts16:14
*** haleyb__ has quit IRC16:16
jlibosvaihrachys: I'm wondering whether self.db_objs, self.obj_fields, self.objs16:16
jlibosvaihrachys: can have some kind of object that would always point to db_obj, fields and obj16:16
ihrachysjlibosva: not sure what you mean16:17
jlibosvaihrachys: then you would have just one attribute containing collection of such objects and you'd always pick one and work with it instead of zipping16:17
*** yamahata has joined #openstack-neutron16:17
ihrachyswell yeah but even then I would need to update all elements of such an object16:17
*** chandanc has joined #openstack-neutron16:17
ihrachyswith the same value16:17
ihrachys(with slight key difference)16:17
jlibosvayou could encapsulate the consistency of state inside of the class16:18
ihrachysjlibosva: I already do encapsulate it in the test case itself. doesn't help much with code reuse.16:19
ihrachysjlibosva: but maybe I miss something16:19
ihrachysjlibosva: can you write an example on how it would be used?16:19
ihrachysit may give me an idea of what you envision16:19
jlibosvaihrachys: I will try, not sure when though. :)16:20
*** chlong has joined #openstack-neutron16:20
ihrachysjlibosva: ok. note that this patch is a dep for another one that introduces delete_objects() API for objects that we want to use in plugin code16:21
*** mdnadeem has quit IRC16:21
*** tbachman has quit IRC16:23
jlibosvaihrachys: noted16:24
*** mickeys has joined #openstack-neutron16:27
*** Leo_ has joined #openstack-neutron16:27
*** achadha has joined #openstack-neutron16:28
*** victorhe has joined #openstack-neutron16:28
*** victorhe has quit IRC16:28
*** dimak has quit IRC16:28
*** victorhe has joined #openstack-neutron16:28
*** chandanc has quit IRC16:29
*** victorhe has quit IRC16:30
*** victorhe has joined #openstack-neutron16:30
*** tbachman has joined #openstack-neutron16:30
*** bjornar_ has quit IRC16:31
*** victorhe has quit IRC16:31
*** mickeys has quit IRC16:31
*** jschwarz has quit IRC16:32
*** armax has joined #openstack-neutron16:32
*** mmalik4 has joined #openstack-neutron16:32
*** andreas_s has quit IRC16:32
*** victorhe has joined #openstack-neutron16:32
*** victorhe has quit IRC16:32
rossella_sihrachys, ack, I will put it in my list16:32
*** dffco has quit IRC16:34
*** achadha has quit IRC16:34
openstackgerritMerged openstack/neutron-vpnaas: Remove PLURALS  https://review.openstack.org/40331816:34
*** jschwarz has joined #openstack-neutron16:35
*** dffco has joined #openstack-neutron16:36
*** limao_ has quit IRC16:36
*** pcaruana has quit IRC16:36
*** mvk has quit IRC16:36
*** sayalilunkad has quit IRC16:38
*** dsneddon has quit IRC16:39
*** janzian has quit IRC16:40
*** dsneddon has joined #openstack-neutron16:40
openstackgerritAnkur proposed openstack/python-neutronclient: Implement Network Tag Plugin into OSC  https://review.openstack.org/39672716:41
openstackgerritvenkata anil proposed openstack/neutron: Avoid apply firewall if remote sg members update  https://review.openstack.org/35177316:44
openstackgerritvenkata anil proposed openstack/neutron: Delete conntrack when remote ipset member removed  https://review.openstack.org/35244016:44
*** josecastroleon has quit IRC16:44
*** kun_huang has quit IRC16:45
*** jamesdenton has quit IRC16:45
*** rmart04 has quit IRC16:47
*** ataraday_ has quit IRC16:47
*** trown is now known as trown|lunch16:47
*** anilvenkata has quit IRC16:48
*** vhosakot has joined #openstack-neutron16:49
*** tbachman has quit IRC16:51
*** tmorin has quit IRC16:52
*** haleyb__ has joined #openstack-neutron16:54
*** jamesdenton has joined #openstack-neutron16:54
*** nplanel has quit IRC16:55
*** janzian has joined #openstack-neutron16:55
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron: [WIP] QoS NIC bandwidth manager for Linux Bridge backend  https://review.openstack.org/40212116:57
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron: [WIP] QoS NIC bandwidth manager for SR-IOV backend  https://review.openstack.org/40125416:57
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron: [WIP] QoS NIC bandwidth manager for OVS backend  https://review.openstack.org/40434316:58
openstackgerritIhar Hrachyshka proposed openstack/neutron: tests: introduce update_obj_fields method in base objects test class  https://review.openstack.org/40425917:00
openstackgerritIhar Hrachyshka proposed openstack/neutron: objects: add delete_objects public method  https://review.openstack.org/40383217:00
*** yfried has quit IRC17:01
*** jtollet has quit IRC17:02
*** ataraday_ has joined #openstack-neutron17:02
*** shashank_hegde has joined #openstack-neutron17:02
openstackgerritVictor Morales proposed openstack/neutron: Router, RouterPort and FloatingIP to OVO  https://review.openstack.org/30796417:03
*** manjong has joined #openstack-neutron17:03
*** xionchen_ has quit IRC17:04
*** baoli has quit IRC17:05
*** nyechiel has quit IRC17:05
*** achadha has joined #openstack-neutron17:07
*** haleyb__ has quit IRC17:07
*** asettle has quit IRC17:08
*** abregman has quit IRC17:09
*** achadha_ has joined #openstack-neutron17:09
*** chlong has quit IRC17:10
*** ataraday_ has quit IRC17:10
*** achadha has quit IRC17:10
*** wolverineav has joined #openstack-neutron17:11
*** bandarji has joined #openstack-neutron17:11
*** baoli has joined #openstack-neutron17:12
*** shashank_hegde has quit IRC17:12
*** shashank_hegde has joined #openstack-neutron17:16
*** neil_ has joined #openstack-neutron17:16
*** neiljerram has quit IRC17:17
*** achadha_ has quit IRC17:17
*** achadha has joined #openstack-neutron17:18
*** sahid has quit IRC17:18
*** zzzeek has quit IRC17:20
*** ataraday_ has joined #openstack-neutron17:22
*** kevo has joined #openstack-neutron17:23
*** chlong has joined #openstack-neutron17:24
*** zzzeek has joined #openstack-neutron17:24
*** numans has quit IRC17:25
*** matrohon has quit IRC17:26
*** vhosakot has quit IRC17:27
*** vijaykc4 has quit IRC17:27
*** mickeys has joined #openstack-neutron17:28
*** jhershbe has quit IRC17:28
*** saggi has quit IRC17:29
*** vhosakot has joined #openstack-neutron17:30
*** manjong has quit IRC17:32
*** davidsha has quit IRC17:32
*** mickeys has quit IRC17:32
*** Leo_ has quit IRC17:36
*** Leo_ has joined #openstack-neutron17:36
*** phil_h has quit IRC17:36
openstackgerritShashank Kumar Shankar proposed openstack/neutron-lib: api-ref: Fix api-ref for fwaas  https://review.openstack.org/36396717:38
*** dffco has quit IRC17:39
*** ctayal has joined #openstack-neutron17:40
*** matrohon has joined #openstack-neutron17:42
sc68calgaryk: whoops :)17:42
sc68calgaryk: just +A'd https://review.openstack.org/#/c/400258/17:42
openstackgerritIhar Hrachyshka proposed openstack/neutron: Expose [agent] extensions option into l3_agent.ini  https://review.openstack.org/40085217:43
openstackgerritIhar Hrachyshka proposed openstack/neutron: Renamed all [AGENT] config sections into [agent]  https://review.openstack.org/40436217:43
*** ygbo has quit IRC17:44
*** vijaykc4 has joined #openstack-neutron17:44
*** neil_ has quit IRC17:44
*** gvrangan has joined #openstack-neutron17:45
*** zzzeek has quit IRC17:45
*** markvoelker has quit IRC17:45
*** dffco has joined #openstack-neutron17:47
*** irenab has joined #openstack-neutron17:48
*** lucasagomes is now known as lucas-afk17:48
*** zzzeek has joined #openstack-neutron17:48
*** numans has joined #openstack-neutron17:50
*** mvk has joined #openstack-neutron17:50
*** markvoelker has joined #openstack-neutron17:50
*** shashank_hegde has quit IRC17:51
ihrachysafazekas: anything that holds us back on https://review.openstack.org/#/c/403289/ ? I understand your concern about keeping the test in tempest but I believe it's a different matter.17:51
*** numans has quit IRC17:52
*** numans has joined #openstack-neutron17:52
*** trevormc has quit IRC17:53
*** beagles is now known as beagles_brb17:53
*** iyamahat has quit IRC17:54
*** mickeys has joined #openstack-neutron17:56
*** yamahata has quit IRC17:57
*** raj_ has quit IRC17:58
*** jperry has quit IRC17:59
*** jperry has joined #openstack-neutron17:59
*** jlibosva has quit IRC17:59
*** neil_ has joined #openstack-neutron18:00
*** itzikb_ has joined #openstack-neutron18:00
*** manjong has joined #openstack-neutron18:00
*** asettle has joined #openstack-neutron18:01
*** rodrigods is now known as rodrigods_18:02
*** ralonsoh_ has quit IRC18:02
*** edand has quit IRC18:04
*** ijw has joined #openstack-neutron18:05
*** manjong has quit IRC18:06
*** haleyb__ has joined #openstack-neutron18:06
*** tonytan4ever has quit IRC18:06
*** tidwellr has joined #openstack-neutron18:06
*** rodrigods_ is now known as rodrigods18:06
*** tonytan4ever has joined #openstack-neutron18:07
*** rodrigods has quit IRC18:07
*** rodrigods has joined #openstack-neutron18:07
*** nyechiel has joined #openstack-neutron18:09
*** jpena is now known as jpena|off18:10
*** tbachman has joined #openstack-neutron18:10
*** irenab has quit IRC18:11
*** vijaykc4 has quit IRC18:12
*** Alex_Stef has quit IRC18:13
openstackgerritArmando Migliaccio proposed openstack/python-neutronclient: Disable VPNaaS functional tests in the neutronclient gate  https://review.openstack.org/40436618:13
*** sudipto_ has quit IRC18:13
*** sudipto has quit IRC18:13
tidwellrliuyulong: ping18:13
*** baoli has quit IRC18:13
*** baoli has joined #openstack-neutron18:14
*** iyamahat has joined #openstack-neutron18:15
openstackgerritArmando Migliaccio proposed openstack/neutron-vpnaas: DNM: switch to strongswan to validate xenial setup  https://review.openstack.org/40436718:15
openstackgerritArmando Migliaccio proposed openstack/python-neutronclient: DNM: see how strongswan works on xenial  https://review.openstack.org/40436918:16
armaxihrachys: ^18:16
armaxihrachys: the xenial switch for the neutronclient’s functional jobs has not been flawless as I anticipated18:17
ihrachysarmax: you still feel obligated to make vpnaas work? :)18:17
armaxihrachys: not quite18:17
ihrachysah client...18:17
armaxihrachys: the switch to xenial unveiled the fact that xenial doesn’t ship with openswan any longer18:17
ihrachyshaha18:18
armaxso we have two options: we either stop testing vpnaas in the client gate (the one I prefer)18:18
armaxor switch to strongswan18:18
armaxihrachys: the gist here: https://review.openstack.org/#/q/topic:bug/164581918:18
ihrachysis it hard to disable vpnaas in the repo?18:18
armaxnot at all18:19
armaxhttps://review.openstack.org/#/c/404366/18:19
armaxI am speculating if keeping vpn’s functional coverage is as easy to switch to strongswan18:19
armaxwith https://review.openstack.org/#/c/404369/18:19
armaxihrachys: but I think it’s safer to proceed with https://review.openstack.org/#/c/404366/ because we can’t gate on something we don'18:20
armaxmaintain18:20
ihrachysyeap18:20
armaxthis will have to be backported to18:20
armaxfor newton though18:20
armaxas newton’s default is xenial18:21
armaxunless we pin trusty on newton18:21
armaxwhat do you think?18:21
*** shashank_hegde has joined #openstack-neutron18:21
ihrachysarmax: I feel ok not to pursue the fix for their gate. I guess the longer term plan for bits out of stadium is to move it to other repos or kill it if no one cares?18:21
armaxeither that, or switch to a non-voting job18:22
*** trown|lunch is now known as trown18:22
ihrachysarmax: I think it's fair to backport the gate fix, it's not like we drop the code, but it's not gated on anymore18:22
armaxihrachys: aye18:23
ihrachysarmax: I feel it's unfair to remove coverage for other services just because vpnaas sucks :)18:23
armaxihrachys: it doesn’t suck, it’s being neglected18:23
*** Leo_ has quit IRC18:23
armaxit may be neglected because it might suck, but that’s a different argument I am not willing to make18:24
ihrachysI don't see any practical difference. anyhoo...18:24
*** shashank_hegde has quit IRC18:25
* ihrachys -> life18:27
*** ihrachys has quit IRC18:27
*** tflynn has joined #openstack-neutron18:28
openstackgerritRyan Tidwell proposed openstack/neutron-dynamic-routing: Let the bgp_plugin to query floating IP bgp next_hop  https://review.openstack.org/37231018:28
*** edand has joined #openstack-neutron18:30
*** rossella_s has quit IRC18:32
*** garyk has quit IRC18:32
*** yamahata has joined #openstack-neutron18:33
*** abhiraut has joined #openstack-neutron18:34
*** dffco has quit IRC18:37
*** itzikb_ has quit IRC18:38
*** dsneddon has quit IRC18:41
*** dsneddon has joined #openstack-neutron18:42
*** raj_ has joined #openstack-neutron18:44
*** TTris has quit IRC18:45
*** s3wong has joined #openstack-neutron18:46
*** shashank_hegde has joined #openstack-neutron18:50
*** eezhova has joined #openstack-neutron18:52
*** kevo has quit IRC18:53
*** manjong has joined #openstack-neutron18:55
*** rkukura has quit IRC18:57
*** kevo has joined #openstack-neutron18:57
*** baoli has quit IRC18:57
*** kevo has quit IRC18:58
*** manjong has quit IRC18:59
*** kevo has joined #openstack-neutron19:02
*** brianstajkowski has joined #openstack-neutron19:02
*** haleyb__ has quit IRC19:04
*** ctayal has quit IRC19:06
*** awidders_ has joined #openstack-neutron19:06
*** ctayal has joined #openstack-neutron19:06
*** abregman has joined #openstack-neutron19:08
*** abregman has quit IRC19:09
*** abregman has joined #openstack-neutron19:10
*** gvrangan has quit IRC19:10
*** kevo has quit IRC19:11
*** achadha_ has joined #openstack-neutron19:11
*** baoli has joined #openstack-neutron19:11
*** iyamahat has quit IRC19:12
*** oreillyd has quit IRC19:13
*** crose has quit IRC19:13
*** matrohon has quit IRC19:14
*** dsneddon has quit IRC19:14
*** dsneddon_ is now known as dsneddon19:15
*** achadha has quit IRC19:15
*** achadha_ has quit IRC19:15
*** edand has quit IRC19:15
*** brault_ has joined #openstack-neutron19:16
*** brault has quit IRC19:19
*** kevo has joined #openstack-neutron19:19
*** amoralej is now known as amoralej|off19:21
openstackgerritArmando Migliaccio proposed openstack/python-neutronclient: Make extension functional tests conditional  https://review.openstack.org/40440019:25
openstackgerritArmando Migliaccio proposed openstack/python-neutronclient: Disable VPNaaS functional tests in the neutronclient gate  https://review.openstack.org/40436619:26
*** yfried has joined #openstack-neutron19:26
*** trevormc has joined #openstack-neutron19:26
*** oanson has joined #openstack-neutron19:27
*** ijw has quit IRC19:28
openstackgerritArmando Migliaccio proposed openstack/python-neutronclient: Make extension functional tests conditional  https://review.openstack.org/40440019:30
openstackgerritArmando Migliaccio proposed openstack/python-neutronclient: Disable VPNaaS functional tests in the neutronclient gate  https://review.openstack.org/40436619:30
*** achadha has joined #openstack-neutron19:30
*** haleyb__ has joined #openstack-neutron19:31
*** haleyb__ has quit IRC19:36
*** tbachman has quit IRC19:37
*** ankur-gupta-f1 has left #openstack-neutron19:37
*** ankur-gupta-f1 has joined #openstack-neutron19:37
*** eezhova has quit IRC19:44
*** josecastroleon has joined #openstack-neutron19:45
*** nyechiel has quit IRC19:46
*** lezbar has quit IRC19:47
*** manjong has joined #openstack-neutron19:49
*** josecastroleon has quit IRC19:50
*** wolverineav has quit IRC19:51
*** mmalik4_ has joined #openstack-neutron19:52
*** wolverineav has joined #openstack-neutron19:52
*** edand has joined #openstack-neutron19:52
openstackgerritNate Johnston proposed openstack/neutron-lib: Migrate neutron agent extensions to neutron-lib  https://review.openstack.org/38504519:52
*** Leo_ has joined #openstack-neutron19:53
*** ociuhandu has quit IRC19:53
*** rkukura has joined #openstack-neutron19:53
*** manjong has quit IRC19:54
*** mmalik4 has quit IRC19:55
*** ctayal_ has joined #openstack-neutron19:55
*** ataraday_ has quit IRC19:56
*** slaweq_ has quit IRC19:57
*** slaweq_ has joined #openstack-neutron19:58
*** ctayal has quit IRC19:59
*** ankur-gupta-f1 has left #openstack-neutron20:01
*** tmorin has joined #openstack-neutron20:01
*** gvrangan has joined #openstack-neutron20:02
openstackgerritNakul Dahiwade proposed openstack/neutron: OVO for Ipam(Allocation,Subnet,Pool)  https://review.openstack.org/36079920:03
*** ctayal_ has quit IRC20:05
*** kevo has quit IRC20:05
*** ctayal has joined #openstack-neutron20:05
*** ociuhandu has joined #openstack-neutron20:05
*** bobmel has quit IRC20:05
openstackgerritArmando Migliaccio proposed openstack/python-neutronclient: Disable VPNaaS functional tests in the neutronclient gate  https://review.openstack.org/40436620:07
*** wolverineav has quit IRC20:07
*** bobmel has joined #openstack-neutron20:09
*** wolverineav has joined #openstack-neutron20:10
*** ctayal_ has joined #openstack-neutron20:10
*** ctayal_ has quit IRC20:11
*** ctayal_ has joined #openstack-neutron20:13
*** iranzo has quit IRC20:14
*** ctayal has quit IRC20:14
*** beagles_brb is now known as beagles20:14
*** wolverineav has quit IRC20:15
*** nyechiel has joined #openstack-neutron20:15
*** slaweq_ has quit IRC20:16
*** slaweq_ has joined #openstack-neutron20:17
*** jtollet has joined #openstack-neutron20:19
*** jtollet has quit IRC20:19
*** slaweq_ has quit IRC20:22
*** bobmel has quit IRC20:23
*** baoli has quit IRC20:23
*** slaweq_ has joined #openstack-neutron20:23
*** bobmel has joined #openstack-neutron20:23
*** baoli has joined #openstack-neutron20:23
*** tmorin has quit IRC20:24
*** wolverineav has joined #openstack-neutron20:24
*** adriant has joined #openstack-neutron20:25
*** itzikb_ has joined #openstack-neutron20:29
openstackgerritShashank Kumar Shankar proposed openstack/neutron: [WIP] Get standard attribute ID from DeclarativeObject.  https://review.openstack.org/40041220:29
*** jtollet has joined #openstack-neutron20:29
*** wolverineav has quit IRC20:29
*** Swami has joined #openstack-neutron20:32
*** fidgi has quit IRC20:33
*** wolverineav has joined #openstack-neutron20:35
*** catinthe_ has quit IRC20:35
*** tbachman has joined #openstack-neutron20:37
*** magg has joined #openstack-neutron20:39
*** sambetts is now known as sambetts|afk20:39
*** jtollet has quit IRC20:40
*** wolverineav has quit IRC20:40
*** jtollet has joined #openstack-neutron20:40
*** slaweq_ has quit IRC20:42
*** oanson has quit IRC20:43
*** manjong has joined #openstack-neutron20:43
*** slaweq_ has joined #openstack-neutron20:44
*** wolverineav has joined #openstack-neutron20:45
magghi guys: i have a devstack multi node installation where only one node has access to internet. the rest nodes are inside a private newtork20:46
maggbut the rest of the servers have to pass the traffic through this one server. the controller was installed in one of the nodes of the private network20:46
maggeverything works, but my vms cannot ping my DNS server and so they cannot reach the internet20:46
maggany idea where to debug?20:47
*** manjong has quit IRC20:48
*** wolverineav has quit IRC20:50
sc68calmagg: verify that the vxlan traffic from all your nodes is flowing correctly to the controller20:52
sc68calmagg: then verify that traffic is exiting correctly20:52
*** abregman has quit IRC20:53
maggsc68cal: how can i do that?20:54
sc68calis the node that has access to the internet also your controller node?20:54
*** tonytan4ever has quit IRC20:54
maggsc68cal: no20:55
*** matrohon has joined #openstack-neutron20:57
sc68calmagg: the controller node needs to be the one that has internet access..20:57
*** nplanel has joined #openstack-neutron20:58
sc68calso make sure your control node has internet access20:58
maggsc68cal: it has because the traffic is routed through the node with access20:59
sc68calmagg: so then I would suggest starting with the node with access, and packet dump20:59
maggthe controller can ping google.com and my internal dns at my company20:59
maggok21:00
sc68calwhat about traffic from the compute node? Either trace it starting from a compute node, or start from the internet access node21:00
*** dalvarez has quit IRC21:01
*** mickeys has quit IRC21:02
maggthanks21:02
*** mhickey has quit IRC21:04
*** wolverineav has joined #openstack-neutron21:05
*** ijw has joined #openstack-neutron21:06
*** jtollet has quit IRC21:06
*** jtollet has joined #openstack-neutron21:06
*** dave-mccowan has quit IRC21:06
*** ijw has quit IRC21:07
*** ijw has joined #openstack-neutron21:08
*** magg has quit IRC21:08
*** dalvarez has joined #openstack-neutron21:09
*** wolverineav has quit IRC21:09
*** yfried has quit IRC21:10
*** tidwellr has left #openstack-neutron21:11
*** gouthamr has quit IRC21:11
openstackgerritShashank Kumar Shankar proposed openstack/neutron: [WIP] Integration of Floating IP OVO  https://review.openstack.org/39635121:11
*** bandarji has left #openstack-neutron21:13
*** jschwarz has quit IRC21:16
*** ajo has quit IRC21:16
*** jschwarz has joined #openstack-neutron21:17
*** ajo has joined #openstack-neutron21:17
*** wolverineav has joined #openstack-neutron21:25
*** Leo_ has quit IRC21:25
*** jschwarz has quit IRC21:26
*** jschwarz has joined #openstack-neutron21:27
*** wolverineav has quit IRC21:29
*** Leo_ has joined #openstack-neutron21:31
*** janzian has quit IRC21:33
*** haleyb__ has joined #openstack-neutron21:33
*** mickeys has joined #openstack-neutron21:34
*** catintheroof has joined #openstack-neutron21:34
*** Jeffrey4l has quit IRC21:35
*** tbachman has quit IRC21:35
*** haleyb__ has quit IRC21:37
*** manjong has joined #openstack-neutron21:37
raj_kevinbenton, I've run a couple more iterations of the create a bunch of instances/ports, delete them, and your proposed patch seems to be holding-up well.21:38
*** catintheroof has quit IRC21:38
*** ctayal_ has quit IRC21:41
*** manjong has quit IRC21:42
*** ajo has quit IRC21:43
*** wolverineav has joined #openstack-neutron21:44
*** jprovazn has quit IRC21:45
*** leitan has quit IRC21:46
*** ajo has joined #openstack-neutron21:46
*** jtollet has quit IRC21:46
*** leitan has joined #openstack-neutron21:46
*** jtollet has joined #openstack-neutron21:46
*** Jeffrey4l has joined #openstack-neutron21:47
*** catintheroof has joined #openstack-neutron21:47
*** janzian has joined #openstack-neutron21:47
*** wolverineav has quit IRC21:49
*** ijw has quit IRC21:49
*** ijw has joined #openstack-neutron21:50
*** leitan has quit IRC21:50
*** jtollet has quit IRC21:51
*** rtheis has quit IRC21:52
*** nmagnezi has joined #openstack-neutron21:52
*** matrohon has quit IRC21:54
*** matrohon has joined #openstack-neutron21:54
*** tpsilva has quit IRC21:55
*** ijw has quit IRC21:55
*** rodrigods is now known as rodrigods_21:56
*** ctayal has joined #openstack-neutron21:56
*** ihrachys has joined #openstack-neutron21:57
*** trown is now known as trown|outtypewww22:00
*** bobmel has quit IRC22:02
*** slaweq_ has quit IRC22:03
*** tbachman has joined #openstack-neutron22:04
bodenarmax HenryG: maybe I missed this, but for new API extensions are we proposing they go right into neutron-lib or follow the neutron -> lib rehome cycle?22:04
armaxboden: at this point I’d suggest to go straight to neutron-lib22:04
bodensorry.. API extension definitions22:04
*** wolverineav has joined #openstack-neutron22:04
HenryG+122:05
bodenarmax: that makes neutron dev difficult tho, doesn’t it (without a lib release)?22:05
armaxboden: I wouldn’t label it as difficult22:05
armaxboden: the API must be agreed on22:05
boden(perhaps) elongated22:05
armaxupfront22:05
armaxno22:05
*** teclator has quit IRC22:06
bodenarmax: fine by me.. thanks22:06
armaxit’s slightly more elaborated in that if you want to test your API definition you gotta do the depends-on trick22:06
armaxthat’s it22:06
*** teclator has joined #openstack-neutron22:06
*** nmagnezi has quit IRC22:06
armaxbut other than that, development proceeds as the usual pace22:06
bodenarmax: ack… lets see it in action now!22:07
*** corey_ has quit IRC22:07
armaxboden: what API do you have in midn?22:07
bodenarmax: I’m just reviewing a spec (https://review.openstack.org/#/c/203509)22:08
armaxboden: oh right that one22:09
*** wolverineav has quit IRC22:09
*** tlian2 has joined #openstack-neutron22:11
*** chlong has quit IRC22:12
*** tlian has quit IRC22:14
armaxboden: is this dead code right now? https://review.openstack.org/#/c/379804/22:14
armaxbecause of https://review.openstack.org/#/c/379804/7/neutron_lib/hacking/checks.py@261 I mean22:14
*** jamesdenton has quit IRC22:15
*** tlian2 has quit IRC22:15
*** lindycoder has quit IRC22:16
bodenarmax: technically consumers can use the assert_equal_none() check in various ways directly, but the fact that its commented out in factory() means it won’t get used “by default” for neutron-lib consumers who call our factory() via tox.ini hacking config… so its not dead, its just sleeping until someone wakes it22:17
armaxboden: waking up how?22:17
armaxby uncommenting?22:18
armaxis there a plan to uncommenting at some point in the near future?22:19
bodenarmax: (a) consumer who uses their own factory() (a few do I think) adds the check to their factory()… (b) manually used via local-check in tox.ini.. (c) uncommented in lib as you said22:19
bodena-b are probably unlikely22:19
armaxotherwise this has very much a chance to rot22:19
bodenagreed22:20
armaxbefore it becomes useful22:20
*** ctayal has quit IRC22:20
bodenarmax: regarding hacking checks in lib.. I’m still actively working that (https://review.openstack.org/#/c/350723/).. I’m trying to get a new release of hacking that includes my updates to it before proceeding22:20
*** ctayal has joined #openstack-neutron22:21
openstackgerritNate Johnston proposed openstack/neutron-lib: Migrate neutron-fwaas API definitions to neutron-lib  https://review.openstack.org/38938822:21
armaxboden: so you’re saying that https://review.openstack.org/#/c/379804/ should be lined up behind it?22:22
kevinbentonraj_: excellent22:22
*** matrohon has quit IRC22:22
*** teclator_ has joined #openstack-neutron22:22
*** teclator has quit IRC22:22
bodenarmax: yes if we want it to be “useful”22:22
armaxok22:23
*** Leo_ has quit IRC22:23
armaxboden: thanks22:23
*** nyechiel has quit IRC22:23
armaxboden: one more thing22:24
bodenarmax: shoot22:24
*** nyechiel has joined #openstack-neutron22:24
armaxboden: the patches whose commit headers start with Verify valid_*22:24
armaxdo we have use cases ready to adopt any of them?22:25
bodenarmax: nothing I’m aware of today22:26
bodentechnical debt if you will22:26
raj_kevinbenton, I don't want to look too closely at the teeth of the gift-horse, but while I was applying the patch (by hand because my destination system didn't have "patch" installed) I noticed that what I thought was emptying the set of deleted ports was just the init.  So, what (eventually) takes a port out of the set?22:26
armaxboden: techincal debt being created by these patches you mean? :)22:27
raj_s/a port/a deleted port/22:27
kevinbentonraj_: OOM killier22:27
kevinbentonraj_: that is an issue with it currently22:27
kevinbentonraj_: at some point we will need to expire them22:27
raj_heh :)  so, rapid, long-term port "churn" could be hazardous to the agent's health.22:27
armaxboden: if we don’t plan to use these in practice how can we validate the design is sound?22:28
kevinbentonraj_: yeah, it was one of those things where the fix for the functional issue was more important than the millions of ports you would have to churn through to make that a problem22:28
bodenarmax: In a nutshell the existing validators define a kwarg in their (public API) validator function but don’t even use it; bad practice… those patches address that22:28
openstackgerritVictor Morales proposed openstack/neutron: Flavor and Service Profile to OVO  https://review.openstack.org/30668522:28
kevinbentonraj_: it's just storing the UUID so it's not the full port body22:28
raj_kevinbenton, fair enough.  I'll not harp on that further.22:29
kevinbentonraj_: it's worth following up on though if you want to file a bug22:29
armaxboden: you’re doing so by adding complexity though22:29
kevinbentonraj_: just haven't gotten to it yet22:29
armaxboden: complexity is not free22:29
bodenarmax: validating the valid values (the unused kwarg) is fairly well defined IMO.. some validator functions already do it, some just ignore the kwarg22:29
armaxboden: what’s wrong with parameters left for ‘future use’?22:30
armaxboden: I mean, don’t get me wrong, I also don’t like the idea of having a signature that doesn’t exactly match what the method is supposed to do22:31
bodenarmax: example of existing use: https://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/validators.py#L24922:31
armaxboden: that’s different though22:31
*** rcernin has joined #openstack-neutron22:31
armaxboden: that’s using an existing method22:31
*** manjong has joined #openstack-neutron22:32
bodenarmax: I assume you mean by calling validate_values()… that’s what’s done in the patches as well22:32
bodenarmax: perhaps you are referring to some assumptions made in the patches about the format of the valid_values?22:33
armaxboden: right, it’s more explicit and easier to understand and reason about22:33
armaxI am looking at https://review.openstack.org/#/c/390398/8/neutron_lib/api/validators.py22:33
armaxand you’re making validate_boolean validate values being passed in22:33
*** wolverineav has joined #openstack-neutron22:34
armaxnow I have two problems with this, but I might be off track22:34
*** haleyb__ has joined #openstack-neutron22:34
armaxa) by using a decorator is overly complex for the aim of the change itself. I hate acronyms but in this case KISS seems appropriate22:34
armaxb) may be prone to become non-backward compatible if we’re not careful22:35
armaxthe iterate_on_data name is a bit obscure to me22:36
*** manjong has quit IRC22:36
bodena) ack, complexity vs redundency can be a fine line…22:37
openstackgerritKevin Benton proposed openstack/neutron: Skip larger than /64 subnets in DHCP agent  https://review.openstack.org/39801622:37
armaxboden: bottom line what is that you’re trying to address?22:38
kevinbentonhaleyb: ^^22:38
armaxif this is just about the fact that some validators don’t use an arg22:38
kevinbentonhaleyb: respun that patch to cover the larger prefixes in the tests22:38
armaxdropping the arg and make the caller handle the difference in signature seems perfectly legit to22:38
haleybkevinbenton: ack22:38
armaxme22:38
*** wolverineav has quit IRC22:38
armaxwhen the arg is required, the arg can either be added or a new method can be introduced22:39
*** haleyb__ has quit IRC22:39
armaxso my take is: pushing complexity to the caller seems like a better trade off22:39
armaxoverall22:39
*** jschwarz has quit IRC22:39
bodenarmax: it is about just makeing the API consistent and (more) proper… dropping the kwarg doesn’t seem great because then we have to technically “deprecate” it don’t we?22:39
*** ajo has quit IRC22:40
*** jschwarz has joined #openstack-neutron22:40
armaxconsistency for consistency’s sake seems futile22:40
armaxif it comes at a cost22:40
armaxthat doesn’t seem justified22:40
armaxas for the deprecation, I think the caller can already be made defensive22:41
armaxwithout having to deprecate the argument drop22:41
*** ajo has joined #openstack-neutron22:41
armaxassuming that the only caller being affected here is neutron, it should also be relatively easy thing to do22:41
bodenarmax: thats fine.. it does seem a bit conflicting with us wanting to pay-up technical debt in neutron-lib, but perhaps this isn’t something we want to pay-up on22:42
*** kevo has joined #openstack-neutron22:42
armaxboden: I personally think that this whole decorator business is overly convoluted22:43
armaxbut maybe it’s just me22:43
bodenarmax: understand… go ahead and -1 with that… I’ll revisit this when I get time.. it’s not a pressing matter for sure…22:44
armaxboden: I wonder if kevinbenton has an opinion too22:44
armaxhe loves decorators22:44
*** jheroux has quit IRC22:45
bodenarmax: the decorator is just an attempt to remove reduant code.. the redundant code is surely easier, but redunant.. I’m just ODC and hate functions that define unused args22:45
*** amuller has quit IRC22:45
armaxboden: don’t you have many levels of hate?22:46
armax:)22:46
armaxor is it just that you hate or love something?22:46
armaxthis is nor marmite22:47
bodenarmax: boy this discussion is going down a rabbit hole now.. at the end of the day I won’t loose sleep either way22:47
armaxfor the reference: https://en.wikipedia.org/wiki/Marmite22:47
armaxboden: ok sorry, next time I’ll slap you with a -2 without telling you anything22:48
*** mriedem has quit IRC22:48
openstackgerritAnindita Das proposed openstack/neutron: Integration of RouterL3AgentBinding  https://review.openstack.org/37707422:48
bodenarmax: I do appreciate you reaching out here. but I don’t take offense to -2’s either…22:49
armaxboden: I just wanted to get a better grasp of your intentions22:49
armaxthat’s all22:49
*** tlian has joined #openstack-neutron22:50
*** ijw has joined #openstack-neutron22:51
*** baoli has quit IRC22:52
*** thorst_ has quit IRC22:53
*** wolverineav has joined #openstack-neutron22:53
*** ijw has quit IRC22:54
*** ijw has joined #openstack-neutron22:55
*** sahilsinha has quit IRC22:55
openstackgerritAnindita Das proposed openstack/neutron: Refactoring config options for mech_sriov opts  https://review.openstack.org/35209822:57
bodenarmax: if you’re open to makeing the validators more consistent; I’m willing to rework.. otherwise we can even defer this activity since no one is really “asking” for it22:58
*** wolverineav has quit IRC22:58
*** kevo has quit IRC22:59
kevinbentonarmax: what do i have an opinion on?23:00
armaxboden: I’d rather wrap up with https://review.openstack.org/#/c/346554/, which has a greater impact23:00
armaxand a much more direct use23:00
bodenarmax: agreed.. I’m still actively working that one, just been busy the last day or 223:00
*** itzikb_ has quit IRC23:01
armaxkevinbenton: boden  and I were pondering on https://review.openstack.org/#/c/390398/ and whether the use of decorators is a bit too much23:01
armaxboden: tag me anytime you need me23:01
armaxboden: don’t be shy ;)23:01
*** lindycoder has joined #openstack-neutron23:01
bodenarmax: also given we have fewer cores these days; it does make sense to prioritize our efforts23:01
*** rcernin has quit IRC23:01
armaxboden: focus is my middle name23:01
armaxcapital f23:02
bodenarmax kevingbenton: lets just table this one… we have more important fish to fry… I’ll abandon these changes and close up the bugs for now.. if we ever get bored we can revisit23:02
bodenkevinbenton ^23:02
*** kbringard has quit IRC23:03
*** rcernin has joined #openstack-neutron23:03
*** catinthe_ has joined #openstack-neutron23:03
*** wolverineav has joined #openstack-neutron23:03
kevinbentonboden: ack23:03
*** rcernin has quit IRC23:04
*** lindycoder has quit IRC23:04
*** rcernin has joined #openstack-neutron23:04
*** lindycoder has joined #openstack-neutron23:04
*** sahilsinha has joined #openstack-neutron23:05
*** quade has quit IRC23:05
*** quade has joined #openstack-neutron23:06
quadebuffer 623:06
openstackgerritNakul Dahiwade proposed openstack/neutron: Integration of DistributedPortBinding OVO  https://review.openstack.org/38256723:06
*** catintheroof has quit IRC23:06
*** Leo_ has joined #openstack-neutron23:07
*** asettle has quit IRC23:08
*** wolverineav has quit IRC23:08
*** boden has quit IRC23:09
*** Leom has joined #openstack-neutron23:09
*** abhiraut has quit IRC23:10
*** tidwellr has joined #openstack-neutron23:11
*** trevormc has quit IRC23:11
*** Leo_ has quit IRC23:11
*** ijw has quit IRC23:13
*** ihrachys has quit IRC23:13
*** mriedem has joined #openstack-neutron23:14
*** yuanying has joined #openstack-neutron23:16
*** dsneddon_ has joined #openstack-neutron23:16
*** rodrigods_ is now known as rodrigods23:17
*** jperry has quit IRC23:17
*** ijw has joined #openstack-neutron23:17
openstackgerritVictor Morales proposed openstack/neutron: OVO for Allocation and Endpoint  https://review.openstack.org/39632723:18
haleybkevinbenton: regarding https://review.openstack.org/#/c/398016/ - mind if i add a unit test?  Not much work23:18
kevinbentonhaleyb: sure23:20
*** ijw has quit IRC23:22
*** stevemar__ has quit IRC23:22
*** fnaval has quit IRC23:22
openstackgerritBrian Haley proposed openstack/neutron: Skip larger than /64 subnets in DHCP agent  https://review.openstack.org/39801623:22
*** stevemar__ has joined #openstack-neutron23:23
*** wolverineav has joined #openstack-neutron23:25
*** manjong has joined #openstack-neutron23:26
openstackgerritArmando Migliaccio proposed openstack/neutron-specs: Ocata: Assessment summary  https://review.openstack.org/38939723:26
*** stevemar__ has quit IRC23:27
*** liuyulong_ has joined #openstack-neutron23:28
openstackgerritMerged openstack/neutron-fwaas: Fix removing rule_association on updating a policy  https://review.openstack.org/40410323:28
*** janzian has quit IRC23:28
*** nyechiel has quit IRC23:28
liuyulong_tidwellr, hi, finally you're back. : )23:29
*** wolverineav has quit IRC23:30
tidwellrliuyulong_: I am :)23:30
*** manjong has quit IRC23:30
tidwellrliuyulong_: I wanted to ask you about https://review.openstack.org/#/c/357722/23:31
armaxsc68cal: ping?23:31
liuyulong_tidwellr, yeah, I think this neutron change depends on neutron-dynamic-routing change https://review.openstack.org/#/c/372310/823:32
*** vhosakot has quit IRC23:32
liuyulong_tidwellr, neutron-dynamic-routing once has ability to query fip next hop, then neutron can get rid of it.23:33
tidwellrliuyulong_: I went ahead and tagged the change in neutron-dynamic-routing with a Depends-On23:33
tidwellrliuyulong_: my only concern was the comment yamamoto made about the coupling with ML223:34
openstackgerritMerged openstack/neutron-lib: Add alembic branch constants  https://review.openstack.org/40147923:34
*** haleyb__ has joined #openstack-neutron23:35
openstackgerritMerged openstack/neutron-lib: Replace six.iteritems() with .items()  https://review.openstack.org/40059623:35
openstackgerritMerged openstack/neutron-lib: Correcting Networking API v2.0 flavor extension  https://review.openstack.org/35838723:35
openstackgerritMerged openstack/neutron-lib: Typo in ExtensionDescriptor.get_request_extensions()  https://review.openstack.org/40380023:35
liuyulong_tidwellr, https://review.openstack.org/#/c/357722/ also has a depends-on.23:35
*** pmannidi has joined #openstack-neutron23:35
tidwellrliuyulong_: as long we won't actually break midonet I want to go ahead and merge what you have, then we can work on decoupling from ML223:36
liuyulong_tidwellr, currently ml2 port binding in neutron-dynamic-routing is widely used. right?23:36
tidwellrliuyulong_: it is, not just in your patch23:36
*** bobmel has joined #openstack-neutron23:37
tidwellrliuyulong_: we should break the dependency with a focused set of patches if we can23:37
*** lindycoder has quit IRC23:37
liuyulong_tidwellr, https://review.openstack.org/#/c/404034/, here I left some comments to yamahata_.23:37
tidwellrliuyulong_:  ok thanks, is https://review.openstack.org/#/c/357722/ in the shape you want it?23:38
liuyulong_tidwellr, yes, neutron should get rid of it.23:39
liuyulong_tidwellr, let's move on the following patches, https://review.openstack.org/#/c/357716/23:39
tidwellrliuyulong_: ok, if you think it's ready to go I'll try to round up some reviewers23:40
*** haleyb__ has quit IRC23:40
liuyulong_https://review.openstack.org/#/c/372310/23:40
liuyulong_https://review.openstack.org/#/c/357722/23:40
liuyulong_^^^23:40
*** rcernin has quit IRC23:41
liuyulong_tidwellr, yeah, it's sitting there a very long time, : )23:41
*** bobmel has quit IRC23:42
*** thorst_ has joined #openstack-neutron23:42
tidwellrliuyulong_: my apologies for the delay, we got caught with the end of the newton cycle and I've had my priorities shifted all over the place, but I've come full circle and I'm back now :)23:43
*** abhiraut has joined #openstack-neutron23:45
*** mmalik4_ has quit IRC23:46
liuyulong_tidwellr, it's ok, I'm glad you came back.  : )23:46
*** thorst_ has quit IRC23:46
tidwellrliuyulong_: I think you're right, I set up the dependency backwards23:47
liuyulong_tidwellr, https://bugs.launchpad.net/neutron/+bug/1614866 this bug is marked as High level, I think the fix https://review.openstack.org/#/c/357716/ should get merged asap.23:48
openstackLaunchpad bug 1614866 in neutron "BGP: Failed to withdraw route while dissocating floating IP" [High,In progress] - Assigned to LIU Yulong (dragon889)23:48
tidwellrliuyulong_: the Depends-On belongs on the neutron side, I can publish an edit23:48
liuyulong_tidwellr, then it's the dvr fip next hop issue.23:48
liuyulong_tidwellr, OK23:48
tidwellrliuyulong_: I just want to hear back from yamamoto before merging, I don't think you would break things for him but I want to be sure before we merge it23:49
tidwellrliuyulong_: I'll publish the edits to the commit messages and round up some reviewers, thanks for working on this23:50
liuyulong_tidwellr, you're welcome. : ) and thank you.23:53
*** edand has quit IRC23:54
*** Swami_ has joined #openstack-neutron23:54
*** thorst_ has joined #openstack-neutron23:56
*** Swami has quit IRC23:57
*** thorst_ has quit IRC23:58
*** ijw has joined #openstack-neutron23:59
*** liuyulong_ has quit IRC23:59

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