| *** yamahata_ has quit IRC | 00:04 | |
| *** kfox1111 has joined #openstack-neutron | 00:04 | |
| kfox1111 | I'm seeing a case where if I disassociate an ip from an instance, it is getting deallocated as well. This is bad. Any ideas what might cause it? | 00:04 |
|---|---|---|
| *** gdubreui has quit IRC | 00:09 | |
| *** aymenfrikha has quit IRC | 00:15 | |
| *** rudrarugge has joined #openstack-neutron | 00:15 | |
| openstackgerrit | Shiv Haris proposed a change to openstack/neutron: Testing tempest with neutron https://review.openstack.org/61273 | 00:18 |
| openstackgerrit | A change was merged to openstack/neutron: Sync global requirements to pin sphinx to sphinx>=1.1.2,<1.2 https://review.openstack.org/61245 | 00:19 |
| *** alagalah has joined #openstack-neutron | 00:19 | |
| *** alagalah has joined #openstack-neutron | 00:20 | |
| *** alagalah has left #openstack-neutron | 00:21 | |
| openstackgerrit | Shiv Haris proposed a change to openstack/neutron: Testing tempest with neutron https://review.openstack.org/61273 | 00:22 |
| *** rudrarug_ has joined #openstack-neutron | 00:24 | |
| *** rudrarugge has quit IRC | 00:24 | |
| *** otherwiseguy has quit IRC | 00:25 | |
| openstackgerrit | Shiv Haris proposed a change to openstack/neutron: Testing tempest with neutron https://review.openstack.org/61273 | 00:25 |
| anteaya | nati_ueno: you around? | 00:26 |
| *** banix has quit IRC | 00:26 | |
| nati_ueno | anteaya: sorry in mtg now | 00:26 |
| anteaya | can you -2 this patch? https://review.openstack.org/#/c/61273/ | 00:26 |
| nati_ueno | anteaya: done | 00:27 |
| anteaya | thanks | 00:27 |
| nati_ueno | anteaya: your welcome | 00:27 |
| *** carl_baldwin has quit IRC | 00:28 | |
| *** salv-orlando has joined #openstack-neutron | 00:30 | |
| anteaya | salv-orlando: shouldn't you be asleep | 00:32 |
| salv-orlando | anteaya: it's half past midnight; still early. And I'm on the train back from the london meetup | 00:33 |
| *** rudrarug_ has quit IRC | 00:33 | |
| anteaya | salv-orlando: ah, how was the meetup? | 00:33 |
| anteaya | still such a shame you didn't make it to Paris | 00:33 |
| salv-orlando | anteaya: meet up was good, room packed. It's a shame about Paris, but I had a dentist appointment I could not postpone | 00:34 |
| anteaya | awesome about the packed room, shame about the dentist | 00:35 |
| anteaya | hope all is good now? | 00:35 |
| anteaya | or more appointments on the horizon? | 00:35 |
| * anteaya has to see the dentist tomorrow | 00:35 | |
| openstackgerrit | stephen-ma proposed a change to openstack/neutron: L3 Agent restart causes network outage https://review.openstack.org/30988 | 00:35 |
| *** gdubreui has joined #openstack-neutron | 00:36 | |
| salv-orlando | nope it's a long running thing but I'm painless at the moment, which is good | 00:38 |
| anteaya | yay for painless | 00:39 |
| anteaya | :( for chronic issue | 00:39 |
| anteaya | so back to the meetup, do you follow an agenda and have presentations? | 00:40 |
| anteaya | or just gather and chat | 00:40 |
| openstackgerrit | Shiv Haris proposed a change to openstack/neutron: Testing ... https://review.openstack.org/61273 | 00:43 |
| *** dims has joined #openstack-neutron | 00:46 | |
| *** salv-orlando has quit IRC | 00:46 | |
| *** yamahata_ has joined #openstack-neutron | 00:55 | |
| *** yamahata_ has quit IRC | 00:55 | |
| *** yamahata_ has joined #openstack-neutron | 00:56 | |
| *** aymenfrikha has joined #openstack-neutron | 01:00 | |
| *** x86brandon has quit IRC | 01:04 | |
| *** julim has joined #openstack-neutron | 01:09 | |
| *** x86brandon has joined #openstack-neutron | 01:09 | |
| *** dims has quit IRC | 01:21 | |
| *** kfox1111 has left #openstack-neutron | 01:21 | |
| *** SumitNaiksatam has quit IRC | 01:21 | |
| *** salv-orlando has joined #openstack-neutron | 01:25 | |
| *** salv-orlando has quit IRC | 01:28 | |
| *** salv-orlando has joined #openstack-neutron | 01:29 | |
| *** otherwiseguy has joined #openstack-neutron | 01:32 | |
| *** djbkd has quit IRC | 01:34 | |
| *** dims has joined #openstack-neutron | 01:36 | |
| *** suresh12 has quit IRC | 01:37 | |
| *** julim has quit IRC | 01:40 | |
| *** SumitNaiksatam has joined #openstack-neutron | 01:45 | |
| *** SumitNaiksatam has quit IRC | 01:55 | |
| *** banix has joined #openstack-neutron | 01:58 | |
| *** Jianyong has joined #openstack-neutron | 01:59 | |
| *** Alienyyg_ has quit IRC | 01:59 | |
| *** suresh12 has joined #openstack-neutron | 02:08 | |
| *** aymenfrikha has quit IRC | 02:14 | |
| *** x86brandon has quit IRC | 02:15 | |
| *** carl_baldwin has joined #openstack-neutron | 02:16 | |
| *** wenjianhn has joined #openstack-neutron | 02:16 | |
| *** suresh12 has quit IRC | 02:17 | |
| *** oda-g has joined #openstack-neutron | 02:19 | |
| *** banix has quit IRC | 02:19 | |
| *** terence has joined #openstack-neutron | 02:21 | |
| *** oda-g has left #openstack-neutron | 02:21 | |
| *** oda-g has joined #openstack-neutron | 02:22 | |
| *** banix has joined #openstack-neutron | 02:22 | |
| *** suresh12 has joined #openstack-neutron | 02:23 | |
| *** banix has quit IRC | 02:24 | |
| *** x86brandon has joined #openstack-neutron | 02:24 | |
| oda-g | Hi, I submitted a BP https://blueprints.launchpad.net/neutron/+spec/enable-to-set-dhcp-port-attributes . Please check. | 02:25 |
| *** banix has joined #openstack-neutron | 02:25 | |
| *** oda-g has left #openstack-neutron | 02:25 | |
| *** banix has quit IRC | 02:30 | |
| *** banix has joined #openstack-neutron | 02:31 | |
| *** hua has joined #openstack-neutron | 02:32 | |
| *** hua has quit IRC | 02:42 | |
| *** zhhuabj has joined #openstack-neutron | 02:43 | |
| *** carl_baldwin has quit IRC | 02:46 | |
| *** SumitNaiksatam has joined #openstack-neutron | 02:49 | |
| *** nati_ueno has quit IRC | 02:54 | |
| *** marun has joined #openstack-neutron | 03:00 | |
| mestery | oda-g: Thanks for pinging the channel on this. | 03:11 |
| marun | salv-orlando: piong | 03:23 |
| marun | mestery: ping | 03:23 |
| marun | 769081 | 03:24 |
| *** clev has joined #openstack-neutron | 03:37 | |
| *** pcm_ has quit IRC | 03:43 | |
| openstackgerrit | Jianing Yang proposed a change to openstack/neutron: Implement basic functionalities for port forwarding https://review.openstack.org/60512 | 03:53 |
| openstackgerrit | Jianing Yang proposed a change to openstack/neutron: Implement basic functionalities for port forwarding https://review.openstack.org/60512 | 03:55 |
| *** suresh12 has quit IRC | 04:00 | |
| *** alex_klimov has joined #openstack-neutron | 04:02 | |
| *** alex_klimov has quit IRC | 04:05 | |
| *** yfried has quit IRC | 04:22 | |
| *** zhhuabj has quit IRC | 04:24 | |
| *** yamahata_ has quit IRC | 04:24 | |
| *** lari has quit IRC | 04:25 | |
| *** lari has joined #openstack-neutron | 04:25 | |
| *** hdd_ has joined #openstack-neutron | 04:25 | |
| *** hdd_ has left #openstack-neutron | 04:26 | |
| *** yamahata_ has joined #openstack-neutron | 04:28 | |
| openstackgerrit | Sushil Kumar proposed a change to openstack/neutron: Updates tox.ini to use new features https://review.openstack.org/60825 | 04:28 |
| *** yfried has joined #openstack-neutron | 04:33 | |
| *** changbl has quit IRC | 04:33 | |
| *** changbl has joined #openstack-neutron | 04:36 | |
| *** alex_klimov has joined #openstack-neutron | 04:36 | |
| *** clev has quit IRC | 04:36 | |
| *** yfried has quit IRC | 04:37 | |
| *** zhhuabj has joined #openstack-neutron | 04:37 | |
| *** yfried has joined #openstack-neutron | 04:39 | |
| *** alagalah has joined #openstack-neutron | 04:43 | |
| *** yfried has quit IRC | 04:59 | |
| *** wenjianhn is now known as _wenjianhn_ | 04:59 | |
| openstackgerrit | Aaron Rosen proposed a change to openstack/neutron: Bump api_workers from 0 to 4 https://review.openstack.org/59787 | 04:59 |
| *** alagalah has left #openstack-neutron | 05:08 | |
| *** suresh12 has joined #openstack-neutron | 05:11 | |
| *** suresh12 has quit IRC | 05:15 | |
| *** terence has quit IRC | 05:18 | |
| *** sc68cal has quit IRC | 05:18 | |
| *** sc68cal has joined #openstack-neutron | 05:18 | |
| *** suresh12 has joined #openstack-neutron | 05:36 | |
| openstackgerrit | Maru Newby proposed a change to openstack/neutron: Send DHCP notifications regardless of agent status https://review.openstack.org/61168 | 05:39 |
| *** x86brandon has quit IRC | 05:41 | |
| marun | a pox on your mixins! | 05:44 |
| *** banix has quit IRC | 05:54 | |
| *** vkozhukalov has quit IRC | 06:09 | |
| *** otherwiseguy has quit IRC | 06:15 | |
| *** x86brandon has joined #openstack-neutron | 06:23 | |
| openstackgerrit | Praneet Bachheti proposed a change to openstack/neutron: Juniper Contrail plug-in implementation for core resources https://review.openstack.org/43793 | 06:27 |
| *** gongysh has joined #openstack-neutron | 06:28 | |
| openstackgerrit | Jenkins proposed a change to openstack/neutron: Imported Translations from Transifex https://review.openstack.org/61325 | 06:33 |
| *** amritanshu_RnD has joined #openstack-neutron | 06:43 | |
| *** SushilKM has joined #openstack-neutron | 06:46 | |
| openstackgerrit | Maru Newby proposed a change to openstack/neutron: Send DHCP notifications regardless of agent status https://review.openstack.org/61168 | 06:49 |
| openstackgerrit | gongysh proposed a change to openstack/neutron: move rpc_setup to the last step of __init__ https://review.openstack.org/60805 | 06:53 |
| *** gdubreui has quit IRC | 06:54 | |
| *** yfried has joined #openstack-neutron | 06:56 | |
| *** yfujioka has joined #openstack-neutron | 06:59 | |
| *** zhhuabj has quit IRC | 07:01 | |
| *** yfried has quit IRC | 07:02 | |
| openstackgerrit | Oleg Bondarev proposed a change to openstack/neutron: LBaaS: check for associations before deleting health monitor https://review.openstack.org/53821 | 07:05 |
| *** lifeless has quit IRC | 07:05 | |
| *** zhhuabj has joined #openstack-neutron | 07:08 | |
| *** simon-AS559 has joined #openstack-neutron | 07:16 | |
| *** nati_ueno has joined #openstack-neutron | 07:36 | |
| *** simon-AS559 has quit IRC | 07:38 | |
| *** simon-AS559 has joined #openstack-neutron | 07:38 | |
| *** simon-AS559 has quit IRC | 07:42 | |
| *** Abhishek_ has joined #openstack-neutron | 07:57 | |
| *** vkozhukalov has joined #openstack-neutron | 07:57 | |
| *** suresh12 has quit IRC | 08:04 | |
| openstackgerrit | Oleg Bondarev proposed a change to openstack/neutron: LBaaS: move agent based driver files into a separate dir https://review.openstack.org/61339 | 08:08 |
| openstackgerrit | gongysh proposed a change to openstack/neutron: merge add-flow() calls https://review.openstack.org/61341 | 08:15 |
| *** simon-AS559 has joined #openstack-neutron | 08:22 | |
| *** _wenjianhn_ has quit IRC | 08:29 | |
| *** nati_ueno has quit IRC | 08:38 | |
| *** fouxm has joined #openstack-neutron | 08:39 | |
| *** yongli is now known as yongli_away | 08:41 | |
| openstackgerrit | Siming Yin proposed a change to openstack/neutron: fix --excluded of meter-label-rule-create is not working https://review.openstack.org/61344 | 08:41 |
| *** roeyc has joined #openstack-neutron | 08:56 | |
| *** jistr has joined #openstack-neutron | 08:57 | |
| *** jlibosva has joined #openstack-neutron | 09:01 | |
| *** afazekas has joined #openstack-neutron | 09:02 | |
| *** Abhishek_ has quit IRC | 09:08 | |
| openstackgerrit | Zhang Hua proposed a change to openstack/neutron: Clean up ML2 Manager https://review.openstack.org/61351 | 09:10 |
| *** jpich has joined #openstack-neutron | 09:14 | |
| *** suresh12 has joined #openstack-neutron | 09:14 | |
| *** Abhishek_ has joined #openstack-neutron | 09:17 | |
| *** suresh12 has quit IRC | 09:19 | |
| marun | gongysh: ping | 09:39 |
| gongysh | marun: pong | 09:39 |
| marun | gongysh: Good catch regarding proper detection of 'active' status. | 09:39 |
| marun | gongysh: why does admin_state_up even exist? | 09:40 |
| marun | gongysh: is it ever set to False for agents, and if so, why? | 09:40 |
| gongysh | marun: admin can force an agent to be excluded. | 09:40 |
| gongysh | marun: it is set by neutron API, not by agent itself. | 09:40 |
| marun | gongysh: When I was doing load testing and seeing dhcp notifications not going through, I guess the agents hadn't been created in the db at all | 09:41 |
| marun | gongysh: I see now that there is actually no check for 'liveness' in the notification class. | 09:41 |
| marun | gongysh: only a check for 'admin_state_up' | 09:42 |
| gongysh | marun: the logics is that when the agent reports its first state, the agent record will be created in db. | 09:42 |
| marun | gongysh: yeah, that makes sense. I just wish I had figured that out earlier. | 09:43 |
| *** lifeless has joined #openstack-neutron | 09:43 | |
| marun | gongysh: the only real benefit of the current patch is that missing agents will log an error | 09:43 |
| marun | gongysh: active/inactive agents were previously notified so long as they had a record in the db | 09:43 |
| gongysh | marun: I have a new comment in the patch. https://review.openstack.org/#/c/61168/3/neutron/api/rpc/agentnotifiers/dhcp_rpc_agent_api.py | 09:46 |
| marun | gongysh: ok | 09:47 |
| marun | gongysh: ah, i'm seeing why i'm confused. get_dhcp_agents_hosting_networks is calling is_eligible_networks | 09:47 |
| gongysh | marun: :) | 09:48 |
| marun | gongysh: It took me a few passes to figure it out I'm afraid. | 09:48 |
| *** gdubreui has joined #openstack-neutron | 09:49 | |
| marun | gongysh: Is there a reason we can't add methods to models? It sure would be useful. :/ | 09:50 |
| gongysh | marun: I don't think we have a rule not to add method to models. | 09:52 |
| marun | gongysh: it would seem to make sense to have agent models able to determine if they are 'active' | 09:52 |
| marun | instances, rather | 09:52 |
| gongysh | marun: yes | 09:53 |
| *** gongysh has quit IRC | 10:00 | |
| *** safchain has joined #openstack-neutron | 10:03 | |
| *** heyongli has joined #openstack-neutron | 10:10 | |
| *** x86brandon has quit IRC | 10:13 | |
| *** rossella_s has joined #openstack-neutron | 10:14 | |
| *** Jianyong has left #openstack-neutron | 10:14 | |
| *** x86brandon has joined #openstack-neutron | 10:15 | |
| *** _jj_ has quit IRC | 10:56 | |
| *** yhe_ has joined #openstack-neutron | 11:04 | |
| *** heyongli has quit IRC | 11:06 | |
| *** jp_at_hp has joined #openstack-neutron | 11:08 | |
| *** pcm_ has joined #openstack-neutron | 11:18 | |
| *** nati_ueno has joined #openstack-neutron | 11:21 | |
| *** pcm_ has quit IRC | 11:22 | |
| *** pcm_ has joined #openstack-neutron | 11:22 | |
| *** Abhishek_ has quit IRC | 11:23 | |
| *** roeyc has quit IRC | 11:24 | |
| *** Sreedhar has joined #openstack-neutron | 11:25 | |
| *** yhe_ is now known as heyongli | 11:33 | |
| *** yamahata_ has quit IRC | 11:33 | |
| *** jianingy_afk is now known as jianingy | 11:55 | |
| *** gdubreui has quit IRC | 11:59 | |
| *** yfujioka has quit IRC | 12:12 | |
| *** bvandenh has joined #openstack-neutron | 12:14 | |
| *** beagles has quit IRC | 12:44 | |
| *** jianingy is now known as jianingy_afk | 12:45 | |
| openstackgerrit | Oleg Bondarev proposed a change to openstack/neutron: LBaaS: agent monitoring and instance rescheduling https://review.openstack.org/59743 | 12:48 |
| *** vkozhukalov has quit IRC | 13:00 | |
| *** x86brandon has quit IRC | 13:03 | |
| pcm_ | Anyone: I'm using the requests package, which on my DevStack setup appears to be V1.2.3 (and 1.2.0 on another system). I see that the latest is 2.1.0. Is there any reason why we cannot use the newer version? | 13:08 |
| *** b3nt_pin has joined #openstack-neutron | 13:10 | |
| pcm_ | Asking because I'm seeing some odd errors with v1.2.0 | 13:11 |
| *** b3nt_pin has quit IRC | 13:12 | |
| *** b3nt_pin has joined #openstack-neutron | 13:12 | |
| *** ygbo has joined #openstack-neutron | 13:12 | |
| *** b3nt_pin is now known as beagles | 13:19 | |
| *** b3nt_pin has joined #openstack-neutron | 13:22 | |
| openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: Do not trigger agent notification if bindings do not change https://review.openstack.org/58860 | 13:25 |
| *** b3nt_pin has quit IRC | 13:26 | |
| *** aveiga has quit IRC | 13:31 | |
| marios | salv-orlando: ping - hi - do you think https://bugs.launchpad.net/neutron/+bug/1111572 is worth working on? I ask because you commented there. I can't see dan around now. It seems was abandoned as 'not supported by spec' but has been revived recently | 13:38 |
| *** safchain has quit IRC | 13:38 | |
| *** jorisroovers has joined #openstack-neutron | 13:39 | |
| *** gongysh has joined #openstack-neutron | 13:43 | |
| gongysh | salv-orlando: ping | 13:43 |
| *** banix has joined #openstack-neutron | 13:44 | |
| *** banix has quit IRC | 13:47 | |
| *** banix has joined #openstack-neutron | 13:48 | |
| pcm_ | mestery: ping | 13:49 |
| mestery | pcm_: pong | 13:49 |
| pcm_ | mestery: Hitting an issue using the requests package. See that we are using v1.2.3, but 2.1.0 is latest. | 13:50 |
| pcm_ | mestery: Wondering if can require newer version? | 13:50 |
| mestery | pcm_: Is this with the latest on Ubuntu? I haven't seen this one yet. | 13:51 |
| pcm_ | mestery: In DevStack, under Ubuntu, it is using requests 1.2.3. I'm wondering if 2.1.0 would solve my issue. | 13:52 |
| pcm_ | mestery: I get SSLError instead of Timeout, when a request times out. | 13:52 |
| mestery | pcm_: Have you tried it locally to see if it helps? I haven't caught up on emails yet this morning, but I see one from you on this in the queue. | 13:52 |
| pcm_ | mestery: Not yet, planning on giving it a try. What's the best way to facilitate that change in Neutron? | 13:53 |
| *** safchain has joined #openstack-neutron | 13:54 | |
| pcm_ | mestery: Not sure how one specifies a new version (I know I can pip install update it, but wondering the procedure within Neutron). | 13:54 |
| mestery | pcm_: I'm not 100% sure either, but perhaps the requirements.txt file is the way? | 13:55 |
| pcm_ | Yeah I see I can change that. Didn't know what I need to do, once I do that to cause the change to be picked up. | 13:56 |
| * pcm_ confused on the packaging stuff... | 13:56 | |
| *** rkukura has quit IRC | 13:57 | |
| *** rkukura has joined #openstack-neutron | 13:57 | |
| mestery | So, I think you can either manually run "sudo python ./setup.py install" or re-run devstack with OFFLINE=True commented out. | 13:58 |
| mestery | Either way should install things with your requirements.txt changes. | 13:59 |
| HenryG | pcm_: in my neutron workspace, requests version is 2.0.0 | 13:59 |
| pcm_ | mestery: I'll give that a try | 13:59 |
| HenryG | pcm_: in another neutron workspace it is 2.1.0 | 13:59 |
| pcm_ | HenryG: Did you manually do anything to cause a newer version? | 14:00 |
| HenryG | pcm_: that is without any change to requirements.txt | 14:00 |
| HenryG | pcm_: no, but I did run UT at some point. I think that installs requests (in the venv) | 14:00 |
| * mestery nods in agreement with HenryG. | 14:01 | |
| pcm_ | HenryG: I'm using commit 3014e1e021b3fe59c75daae1734472c3a11582ee 12/7 | 14:01 |
| *** aymenfrikha has joined #openstack-neutron | 14:02 | |
| HenryG | pcm_: do "tox --notest -e py27", then "source .tox/py27/bin/activate | 14:02 |
| HenryG | ", then "pip show requests" | 14:03 |
| * pcm_ trying... | 14:04 | |
| salv-orlando | gongysh: poong | 14:06 |
| pcm_ | HenryG: So, on a system with requirements.txt saying >=1.1, it is 1.2.0. Run tox and then under that env it shows 2.1.0 | 14:08 |
| gongysh | salv-orlando: https://bugs.launchpad.net/neutron/+bug/1259937 | 14:08 |
| salv-orlando | marios: The issue with unknown attributes was sorted. But making allocation_pools updatable is still something we'd like to have | 14:08 |
| * pcm_ confused as to why tox uses a different version than production code | 14:08 | |
| salv-orlando | gongysh: I know, it | 14:09 |
| marios | salv-orlando: awesome. i'll give dan a day to respond (do you know if he is still active here?). I'll take it tomorrow otherwise | 14:09 |
| salv-orlando | is vestigial code. | 14:09 |
| salv-orlando | Dan is now mostly active on nova | 14:09 |
| *** banix has quit IRC | 14:09 | |
| marios | salv-orlando: i see, thanks | 14:10 |
| salv-orlando | he's surely not going to do this patch for neutron, so go ahead | 14:10 |
| marios | salv-orlando: yeah just cos in the comments: "I have the code to make allocation pools writable similar to what we already do for host_routes or dns_nameservers ... " @ https://bugs.launchpad.net/neutron/+bug/1111572/comments/4 | 14:10 |
| HenryG | pcm_: the system version won't get updated unless you actually install openstack and leave it installed | 14:11 |
| marios | salv-orlando: can work on something else in the meantime. thanks! | 14:11 |
| salv-orlando | well, it's over 10 months ago and that code was never pushed. I guess that rebasing it wouldn't be trivial either | 14:11 |
| pcm_ | HenryG: The setup.py install... just wondering why the difference between what runs via DevStack and what runs from tox. | 14:12 |
| marios | salv-orlando: ok. i'm new here, so don't want to tread on toes ;) | 14:12 |
| marios | salv-orlando: adding to my list. thanks again | 14:12 |
| HenryG | pcm_: I don't know :( | 14:12 |
| *** alagalah has joined #openstack-neutron | 14:12 | |
| HenryG | pcm_: definitely you should try setting requests>=2.0 in requirements.txt and if that fixes your issue then propose a change upstream | 14:13 |
| pcm_ | HenryG: Yeah, I'm going to try it. | 14:14 |
| pcm_ | HenryG: Just trying to include another package I need too (that aren't in requirements.txt) | 14:15 |
| *** gongysh has quit IRC | 14:16 | |
| *** aveiga has joined #openstack-neutron | 14:17 | |
| *** clev has joined #openstack-neutron | 14:17 | |
| HenryG | pcm_: regarding tox venv vs devstack ... Looks like when creating a venv all the latest pip package versions get installed. When devstack runs the project installer it first checks if the requirement is satisfied and if it is then it does not update the pip package to latest version. | 14:19 |
| pcm_ | HenryG: Just seems risky to me... not testing with the same versions...odd | 14:21 |
| HenryG | pcm_: Well, "requests>=1.1" means it is expected to work with any version >=1.1 | 14:23 |
| HenryG | pcm_: And you are finding otherwise, so you may need to get that fixed | 14:23 |
| pcm_ | HenryG: mestery: Well, I updated to requests 2.1.0 and ran my UTs and all the test cases work with Timeout exceptions instead of SSLError exceptions, as expected. | 14:26 |
| *** jgrimm has joined #openstack-neutron | 14:27 | |
| HenryG | pcm_: good. It's about time neutron moved to a new version of requests IMO. | 14:31 |
| pcm_ | mestery: Should I create a bug to update requirements.txt, or wait until I'm doing my blueprint commit? | 14:32 |
| *** x86brandon has joined #openstack-neutron | 14:33 | |
| HenryG | pcm_: I think it's fine to wait. Let's not distract the cores with non-critical reviews right now. | 14:33 |
| *** nati_ueno has quit IRC | 14:33 | |
| HenryG | pcm_: The only requests user a.t.m. is neutron/plugins/ml2/drivers/mechanism_ncs.py | 14:35 |
| *** alagalah has left #openstack-neutron | 14:35 | |
| * HenryG needs to upgrade IRC client. BRB. | 14:36 | |
| *** HenryG has quit IRC | 14:36 | |
| mestery | HenryG: The as-yet-not-upstreamed ODL MechanismDriver also uses requests. | 14:42 |
| openstackgerrit | Edward Hope-Morley proposed a change to openstack/neutron: Adds optional timeout for neutronclient https://review.openstack.org/61183 | 14:43 |
| openstackgerrit | Maru Newby proposed a change to openstack/neutron: Send DHCP notifications regardless of agent status https://review.openstack.org/61168 | 14:44 |
| openstackgerrit | Jeremy Stanley proposed a change to openstack/neutron: Add X-Tenant-ID to metadata request https://review.openstack.org/61439 | 14:46 |
| *** jorisroovers has quit IRC | 14:49 | |
| *** rpodolyaka has joined #openstack-neutron | 14:50 | |
| *** SushilKM has quit IRC | 14:50 | |
| *** marun has quit IRC | 14:53 | |
| *** dhellmann has joined #openstack-neutron | 14:53 | |
| *** carl_baldwin has joined #openstack-neutron | 14:53 | |
| *** jp_at_hp has quit IRC | 14:54 | |
| *** fouxm has quit IRC | 14:56 | |
| *** fouxm has joined #openstack-neutron | 14:57 | |
| *** fouxm has quit IRC | 14:59 | |
| *** fouxm has joined #openstack-neutron | 15:00 | |
| pcm_ | mestery: Well hopefully, whoever upstreams can also update requests. | 15:01 |
| mestery | pcm_: :) | 15:02 |
| *** jecarey has joined #openstack-neutron | 15:04 | |
| *** vkozhukalov has joined #openstack-neutron | 15:07 | |
| *** HenryG has joined #openstack-neutron | 15:13 | |
| *** safchain has quit IRC | 15:14 | |
| *** otherwiseguy has joined #openstack-neutron | 15:20 | |
| *** litong has joined #openstack-neutron | 15:20 | |
| *** julim has joined #openstack-neutron | 15:23 | |
| *** alex_klimov has quit IRC | 15:39 | |
| *** jorisroovers has joined #openstack-neutron | 15:40 | |
| *** heyongli has quit IRC | 15:41 | |
| *** carl_baldwin has quit IRC | 15:44 | |
| *** jorisroovers has quit IRC | 15:46 | |
| *** jroovers has joined #openstack-neutron | 15:46 | |
| *** changbl has quit IRC | 15:46 | |
| *** dims has quit IRC | 15:46 | |
| *** jp_at_hp has joined #openstack-neutron | 15:52 | |
| *** fouxm has quit IRC | 15:52 | |
| *** julim has quit IRC | 15:54 | |
| *** banix has joined #openstack-neutron | 15:57 | |
| *** changbl has joined #openstack-neutron | 15:59 | |
| *** jorisroovers has joined #openstack-neutron | 15:59 | |
| *** mestery has quit IRC | 16:00 | |
| *** dims has joined #openstack-neutron | 16:01 | |
| jlibosva | EmilienM: hi | 16:01 |
| *** jroovers has quit IRC | 16:03 | |
| jlibosva | EmilienM: I got back to work on neutron grenade. I haven't dedicate time for exercise devstack scripts w/ neutron. Now I see it's failing before upgrade process goes in. Did you experience something similar? | 16:03 |
| *** otherwiseguy has quit IRC | 16:05 | |
| *** carl_baldwin has joined #openstack-neutron | 16:07 | |
| EmilienM | jlibosva: no, but did you see that my patch in infra got merged ? | 16:07 |
| *** jorisroovers has quit IRC | 16:11 | |
| *** safchain has joined #openstack-neutron | 16:14 | |
| *** networkstatic has joined #openstack-neutron | 16:14 | |
| *** networkstatic has quit IRC | 16:14 | |
| *** otherwiseguy has joined #openstack-neutron | 16:20 | |
| *** clev has quit IRC | 16:21 | |
| *** julim has joined #openstack-neutron | 16:25 | |
| jlibosva | EmilienM: yes, good job :) That's why I started to work on it again. But I was told exercise tests from devstack are deprecated | 16:27 |
| *** aveiga has quit IRC | 16:27 | |
| *** aveiga has joined #openstack-neutron | 16:30 | |
| *** mlavalle has joined #openstack-neutron | 16:32 | |
| *** amritanshu_RnD has quit IRC | 16:36 | |
| *** SushilKM has joined #openstack-neutron | 16:39 | |
| *** fouxm has joined #openstack-neutron | 16:44 | |
| *** bvandenh has quit IRC | 16:45 | |
| *** afazekas has quit IRC | 16:46 | |
| openstackgerrit | dekehn proposed a change to openstack/neutron: extra_dhcp_opt add checks for empty strings https://review.openstack.org/59858 | 16:51 |
| *** SushilKM has quit IRC | 16:51 | |
| *** SushilKM has joined #openstack-neutron | 16:56 | |
| *** mestery has joined #openstack-neutron | 16:56 | |
| openstackgerrit | James E. Blair proposed a change to openstack/neutron: Have tox install via setup.py develop https://review.openstack.org/61490 | 16:57 |
| *** mlavalle has quit IRC | 16:58 | |
| *** SushilKM has quit IRC | 16:58 | |
| *** SushilKM has joined #openstack-neutron | 16:59 | |
| *** HenryG has quit IRC | 17:04 | |
| *** HenryG has joined #openstack-neutron | 17:05 | |
| *** jlibosva has quit IRC | 17:07 | |
| *** safchain has quit IRC | 17:11 | |
| *** simon-AS559 has quit IRC | 17:14 | |
| *** simon-AS559 has joined #openstack-neutron | 17:14 | |
| *** wally247 has joined #openstack-neutron | 17:15 | |
| *** wally247 has quit IRC | 17:18 | |
| *** simon-AS559 has quit IRC | 17:18 | |
| openstackgerrit | James E. Blair proposed a change to openstack/python-neutronclient: Add -U to pip install command in tox.ini https://review.openstack.org/61508 | 17:20 |
| *** SumitNaiksatam has quit IRC | 17:25 | |
| *** JoeHazzers has quit IRC | 17:25 | |
| *** JoeHazzers has joined #openstack-neutron | 17:26 | |
| *** yfried has joined #openstack-neutron | 17:26 | |
| *** SushilKM has quit IRC | 17:28 | |
| *** dims has quit IRC | 17:33 | |
| *** fouxm has quit IRC | 17:36 | |
| *** dims has joined #openstack-neutron | 17:39 | |
| *** x86brandon has quit IRC | 17:40 | |
| openstackgerrit | Carl Baldwin proposed a change to openstack/neutron: Simplify ip allocation/recycling to relieve db pressure https://review.openstack.org/58017 | 17:40 |
| *** ygbo has quit IRC | 17:41 | |
| *** chandankumar has quit IRC | 17:41 | |
| *** SumitNaiksatam has joined #openstack-neutron | 17:47 | |
| *** SushilKM has joined #openstack-neutron | 17:56 | |
| SumitNaiksatam | does anyone know if Mark Mcclain is around today? | 18:06 |
| *** devlaps has joined #openstack-neutron | 18:06 | |
| SumitNaiksatam | we were waiting for him in the FWaaS meeting (currently on) | 18:06 |
| *** terryw has joined #openstack-neutron | 18:12 | |
| *** jistr has quit IRC | 18:13 | |
| *** otherwiseguy has quit IRC | 18:15 | |
| *** csd has joined #openstack-neutron | 18:22 | |
| mestery | SumitNaiksatam: I believe he said he was going to be on a plane during the FWaaS meeting. | 18:28 |
| SumitNaiksatam | mestery: ah ok, based on his comment during the neutron IRC meeting, my impression was that he was planning in attending | 18:29 |
| SumitNaiksatam | mestery: no worries | 18:30 |
| mestery | SumitNaiksatam: I could be wrong, but that's how I remember it. :) Carry on! | 18:30 |
| *** garyk has joined #openstack-neutron | 18:34 | |
| *** nati_ueno has joined #openstack-neutron | 18:40 | |
| *** SushilKM has quit IRC | 18:42 | |
| *** rossella_s has quit IRC | 18:45 | |
| *** simon-AS559 has joined #openstack-neutron | 18:46 | |
| *** simon-AS5591 has joined #openstack-neutron | 18:46 | |
| *** simon-AS559 has quit IRC | 18:46 | |
| *** jpich has quit IRC | 18:47 | |
| *** simon-AS5591 has quit IRC | 18:51 | |
| *** nati_uen_ has joined #openstack-neutron | 19:05 | |
| *** terryw has quit IRC | 19:08 | |
| *** nati_ueno has quit IRC | 19:08 | |
| *** x86brandon has joined #openstack-neutron | 19:11 | |
| *** suresh12 has joined #openstack-neutron | 19:12 | |
| *** jorisroovers has joined #openstack-neutron | 19:17 | |
| *** bvandenh has joined #openstack-neutron | 19:22 | |
| *** nati_uen_ has quit IRC | 19:30 | |
| *** openstackgerrit has quit IRC | 19:34 | |
| *** openstackgerrit has joined #openstack-neutron | 19:34 | |
| *** nati_ueno has joined #openstack-neutron | 19:40 | |
| *** clev has joined #openstack-neutron | 19:41 | |
| *** carl_baldwin has quit IRC | 19:50 | |
| *** jecarey has quit IRC | 20:00 | |
| *** jecarey has joined #openstack-neutron | 20:02 | |
| openstackgerrit | Anita Kuno proposed a change to openstack/python-neutronclient: Pin Sphinx to <1.2 in test-requirements. https://review.openstack.org/61553 | 20:02 |
| *** balar has quit IRC | 20:05 | |
| *** Sreedhar has quit IRC | 20:07 | |
| *** jistr has joined #openstack-neutron | 20:08 | |
| *** jistr has quit IRC | 20:08 | |
| *** jistr_ has joined #openstack-neutron | 20:09 | |
| *** jistr_ has quit IRC | 20:09 | |
| openstackgerrit | A change was merged to openstack/neutron: Fix a race condition in agents status update code https://review.openstack.org/58814 | 20:09 |
| *** jistr has joined #openstack-neutron | 20:10 | |
| *** jistr has quit IRC | 20:11 | |
| *** jistr has joined #openstack-neutron | 20:11 | |
| *** simon-AS559 has joined #openstack-neutron | 20:11 | |
| *** jroovers has joined #openstack-neutron | 20:12 | |
| *** jorisroovers has quit IRC | 20:12 | |
| *** thedodd has joined #openstack-neutron | 20:13 | |
| *** simon-AS5591 has joined #openstack-neutron | 20:14 | |
| *** simon-AS559 has quit IRC | 20:16 | |
| *** alex_klimov has joined #openstack-neutron | 20:16 | |
| *** terryw has joined #openstack-neutron | 20:18 | |
| *** terryw is now known as otherwiseguy | 20:18 | |
| *** bvandenh has quit IRC | 20:22 | |
| anteaya | jlibosva why would exercise tests from devstack being deprecated affect your work (if that is in fact the case)? | 20:26 |
| *** simon-AS5591 has left #openstack-neutron | 20:28 | |
| *** mlavalle has joined #openstack-neutron | 20:28 | |
| mlavalle | tr3buchet, BeardyMcBeards: the instance is 3d16b47e-10e9-44d5-87b9-01d92e86daf8 | 20:28 |
| *** vkozhukalov has quit IRC | 20:29 | |
| mlavalle | tr3buchet, BeardyMcBeards: root password is N45nY3iLHMEi | 20:29 |
| pcm_ | Any able some general REST questions that I have related to OpenStack? | 20:30 |
| *** mriedem has joined #openstack-neutron | 20:34 | |
| mriedem | does anyone here know Sukhdev Kapur and know why he's running 3rd party CI against nova patches for Arista ML2 neutron code? | 20:34 |
| *** devlaps has quit IRC | 20:35 | |
| openstackgerrit | enikanorov proposed a change to openstack/python-neutronclient: Add cli commands to manage loadbalancer instance https://review.openstack.org/61561 | 20:40 |
| anteaya | mlavalle: did you mean to post root passwords in channel? | 20:41 |
| anteaya | mlavalle: nice job on the ml post too btw | 20:41 |
| anteaya | pcm_: ask away, we can do our best | 20:41 |
| mlavalle | anteaya: wrong channel | 20:41 |
| anteaya | mriedem: do you have a link? | 20:41 |
| anteaya | mlavalle: figured as much, might want to change that password | 20:42 |
| mlavalle | anteaya: we'll throw away that instance in a little bit…. no big deal | 20:42 |
| mlavalle | thanks anyway | 20:42 |
| *** x86brandon has quit IRC | 20:43 | |
| *** jecarey has quit IRC | 20:43 | |
| anteaya | mlavalle: k | 20:44 |
| anteaya | mlavalle: I am rerunning nati_ueno's patch through check, it tripped up on the sphinx pin issue | 20:44 |
| *** jecarey has joined #openstack-neutron | 20:45 | |
| mriedem | anteaya: https://review.openstack.org/#/c/60716/ | 20:45 |
| mlavalle | anteaya: ok, i'll keep an eye on it | 20:45 |
| *** carl_baldwin has joined #openstack-neutron | 20:45 | |
| anteaya | mriedem: looking | 20:47 |
| *** armax has joined #openstack-neutron | 20:50 | |
| *** alagalah_ has joined #openstack-neutron | 20:51 | |
| anteaya | mriedem: he doesn't appear to be on irc, I can't find a nick for him | 20:51 |
| anteaya | he is a neutron dev | 20:51 |
| mriedem | anteaya: that's what i figured | 20:52 |
| mriedem | he must be setting up vendor plugin 3rd party CI for neutron | 20:52 |
| pcm_ | anteaya: I was wondering what to do, if a client does a POST, and times out waiting for server. | 20:52 |
| mriedem | but apparently he's getting triggered on other project's patches too | 20:52 |
| anteaya | there is a push for plugin folks to do third party testing on their plugins, he might be practicing | 20:52 |
| anteaya | mriedem: yeah, I think so | 20:52 |
| mriedem | anteaya: yeah, that's my guess | 20:52 |
| anteaya | mriedem: okay thanks for the heads up, if you see it happening again, please hit me with links to patches and I'll compose an email | 20:53 |
| pcm_ | anteaya: I could just raise an exception, but wondering if there is any way to retry (can't think of a way). | 20:53 |
| *** armax has left #openstack-neutron | 20:53 | |
| mriedem | anteaya: sounds good, thanks | 20:53 |
| anteaya | mriedem: or if he isn't responding and cleaning up after himself | 20:53 |
| anteaya | np, thanks for asking | 20:53 |
| anteaya | pcm_: good question | 20:54 |
| anteaya | pcm_: do you have a specific POST example in mind? | 20:54 |
| anteaya | or better yet, a patch? | 20:54 |
| pcm_ | anteaya: Its for a vendor VPNaaS driver I'm creating. | 20:55 |
| anteaya | k | 20:55 |
| anteaya | anything remotely code like to look at? | 20:55 |
| anteaya | I give better answers the more detail I have to work with | 20:55 |
| pcm_ | anteaya: Yeah, though the question is more workflow related. | 20:55 |
| anteaya | ah | 20:55 |
| anteaya | can't you fire off a retry if a timeout exception is raised? | 20:56 |
| pcm_ | anteaya: For example, if I try to create a user via POST, and it times out, I can try the POST again, but I'll get a 400 response, because the server did do the request (just slow) | 20:56 |
| anteaya | and add a counter so 3 retries and it fails out noting it tried 3 times | 20:56 |
| anteaya | ah | 20:57 |
| anteaya | how much time needs to expire between requests to avoid the 400? | 20:57 |
| pcm_ | anteaya: I can catch the retry, and handle the counters and all (I'll paste that in a sec). But the issue is the server rejects the second attempt. | 20:57 |
| anteaya | ah I see | 20:57 |
| anteaya | I have not tried to do this myself so mostly I am just listening and asking questions | 20:58 |
| anteaya | no personal experience to draw on in this situation | 20:58 |
| anteaya | but what you ask makes sense to me | 20:58 |
| *** suresh12 has quit IRC | 20:58 | |
| *** suresh12 has joined #openstack-neutron | 20:59 | |
| *** jistr has quit IRC | 20:59 | |
| *** aymenfrikha has left #openstack-neutron | 20:59 | |
| pcm_ | anteaya: http://paste.openstack.org/show/54848/ | 21:02 |
| pcm_ | anteaya: So I have the mechanics, but I don't know how to retry. Wondering if the general thought is to set a ridiculously long timeout and then raise and exception. | 21:03 |
| *** aveiga has quit IRC | 21:04 | |
| *** suresh12 has quit IRC | 21:04 | |
| roaet | rkukura: howdy, I have a minor ml2 bug fix I've been trying to get approved, do you mind reviewing it and getting whoever else (a core) to get it approved? It's been waiting since september 3rd | 21:12 |
| rkukura | roaet: Happy to look. What's the URL? Also, does it need rebasing? | 21:13 |
| roaet | rkukura: https://review.openstack.org/#/c/44942/ it was recently rebased | 21:15 |
| roaet | the docs bug was blocking it | 21:15 |
| rkukura | roaet: OK, I've looked at this before, and will review it in detail as soon as possible. | 21:19 |
| anteaya | pcm_: hmmm, yes I don't know what the general thought is | 21:20 |
| anteaya | perhaps salv-orlando might have a thought or two, if he is free | 21:20 |
| openstackgerrit | Carl Baldwin proposed a change to openstack/neutron: Simplify ip allocation/recycling to relieve db pressure https://review.openstack.org/58017 | 21:21 |
| salv-orlando | anteaya: I am always free. I'm reading. | 21:21 |
| salv-orlando | the timeout we're talking about, is it a timeout in the neutron-server, or a timeout in some backend? | 21:22 |
| anteaya | salv-orlando :D thanks | 21:23 |
| openstackgerrit | Anita Kuno proposed a change to openstack/python-neutronclient: Pin Sphinx to <1.2 in test-requirements https://review.openstack.org/61553 | 21:23 |
| *** jecarey has quit IRC | 21:29 | |
| *** alagalah_ has left #openstack-neutron | 21:30 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: This is a test, please ignore. https://review.openstack.org/61574 | 21:30 |
| *** mriedem has left #openstack-neutron | 21:41 | |
| pcm_ | anteaya: salv-orlando: Missed your responses... | 22:02 |
| salv-orlando | I did not have a response rather a question | 22:02 |
| *** suresh12 has joined #openstack-neutron | 22:03 | |
| salv-orlando | re-post: the timeout we're talking about, is it a timeout in the neutron-server, or a timeout in some backend? | 22:03 |
| pcm_ | salv-orlando: I'm doing a client for VNaaS that will talk to a vendor box using REST. If I try to do a REST request and the server delays, I could timeout. | 22:03 |
| pcm_ | salv-orlando: If I wait forever, it would hang, obviously. So... if I timeout, I'm wondering what my recourse is. | 22:04 |
| pcm_ | salv-orlando: I could raise an exception and say I lost connection to server. | 22:04 |
| salv-orlando | Ok, so it's the backend timing out, is that right? I'm a bit confused as you say "client for VPNaaS" and then "talk to a vendor box" | 22:04 |
| pcm_ | salv-orlando: yes. I'm doing a device driver for VPNaaS. | 22:05 |
| salv-orlando | In the NVP plugin if we lose the controller we start raising 503 server unavailable - the timeout seems a similar scenario | 22:05 |
| pcm_ | salv-orlando: Yeah, I was thinking that. Wondered if there was a way to retry, but if I do, the server gives a 400, because it already did the POST. | 22:06 |
| *** jroovers has quit IRC | 22:06 | |
| salv-orlando | Is that because the server processed the request asynchronously? | 22:06 |
| pcm_ | salv-orlando: No, but it did process the request. The client timed out, considered it a failure. | 22:07 |
| salv-orlando | pcm_: so basically the request actually succeeded but the client does not know that | 22:07 |
| pcm_ | salv-orlando: I'm using Requests package to hande the requests, and use a timeout to prevent it for hanging. | 22:08 |
| pcm_ | salv-orlando: yes. exactly | 22:08 |
| salv-orlando | so perhaps you can do a retry but before doing the POST you can do a GET to check whether the request succeeded after all | 22:08 |
| salv-orlando | or you can interpret the 400 from the server on the 2nd post as something that tells you that the retry is not actually necessary | 22:08 |
| pcm_ | salv-orlando: Issue is that the POST create would normally return the ID in the location header item. Without that, I don;t know what to GET | 22:09 |
| salv-orlando | very good point. | 22:09 |
| pcm_ | salv-orlando: Yeah, 400=OK is a possibility, it masks any real 'bad request' | 22:09 |
| salv-orlando | But my gut feeling it seems you might be better off with an asynchronous processing of the request - executing a callback when the request returns; is that possible with vpn drivers at the moment? | 22:10 |
| salv-orlando | pcm_: perhaps you can parse the response content when you get a 400? | 22:10 |
| pcm_ | salv-orlando: I don't think it supports async, but am not sure. Some things require sync (have to have the policies, before creating connection) | 22:11 |
| *** x86brandon has joined #openstack-neutron | 22:11 | |
| pcm_ | salv-orlando: I'm not sure that 400 provides the location though. | 22:11 |
| salv-orlando | pcm_: then fail in neutron upon a backend timeout; but you'd need to find a way to ensure consistency - meaning that somehow you need to undo the operation that neutron thinks failed whereas it was just slow | 22:13 |
| pcm_ | salv-orlando: Seems like 3 choices... 1) 503 response on timeout, 2) consider 400 on retry as OK, 3) implement async processing. | 22:13 |
| pcm_ | salv-orlando: Hmmm. Not sure how I can ensure consistency (other than get of all and exam to see if there is a match). | 22:14 |
| salv-orlando | 1) would need a cleanup; 2) is dangerous if the 400 cannot be disambiguated and 3) is the best but perhaps not feasible now | 22:14 |
| salv-orlando | pcm_ | 22:14 |
| salv-orlando | : | 22:14 |
| pcm_ | salv-orlando: 4) fail and cleanup by get all | 22:15 |
| pcm_ | salv-orlando: I guess 4) would have to be part of 1) | 22:15 |
| salv-orlando | yes that would be a cleanup option, but what ab out if the operation which failed is still in progress? | 22:15 |
| openstackgerrit | rcurran proposed a change to openstack/neutron: Calls in delete_port() should be in reverse order as create. https://review.openstack.org/61583 | 22:16 |
| pcm_ | salv-orlando: I'm not sure. Good question. | 22:16 |
| pcm_ | salv-orlando: 5) Have a really long timeout such that it would be considered failure - requiring restart of the server. Not sure about that. | 22:17 |
| salv-orlando | I am thinking that you can wrap the backend request in a thread perhaps. In the caller you might able to execute a wait on that thread with a timeout | 22:18 |
| salv-orlando | if the wait completes before the timeout expires - fine, that's a success | 22:19 |
| salv-orlando | otherwise, if the timeout expires you fail, and in the thread you started, you do the cleanup | 22:19 |
| salv-orlando | the hard bit is to avoid the cleanup when you succeed, but that should be doable, somehow | 22:20 |
| *** gdubreui has joined #openstack-neutron | 22:20 | |
| * pcm_ trying to absorb... | 22:21 | |
| pcm_ | salv-orlando: How would that be different from single thread and doing a long timeout and cleanup? | 22:22 |
| salv-orlando | pcm_: fail fast | 22:22 |
| salv-orlando | or at least faster than a long timeout | 22:23 |
| salv-orlando | the thing is that if the timeout is something like 20 seconds, you will be blocking the client of the neutron api for 20 secs. | 22:23 |
| pcm_ | salv-orlando: so, let's say we fail... the async thread cleans up, and then we retry? | 22:24 |
| pcm_ | salv-orlando: or just abort the request? | 22:24 |
| *** rkukura has quit IRC | 22:24 | |
| pcm_ | salv-orlando: good point on the client effect. | 22:25 |
| salv-orlando | as you're doing the cleanup maybe you can abort. I don't have enough knowledge of the use case to say what's better; but as long as it's consistent and reasonably efficient, it's good. | 22:25 |
| *** dims has quit IRC | 22:26 | |
| pcm_ | Right now, it looks like 1.x secs, but have some commands taking 2-6 seconds (an issue to be tracked). | 22:26 |
| pcm_ | salv-orlando: I think for connection create, VPNaaS goes to 'PENDING CREATE', and then polling is done to update status (for the reference impl). | 22:27 |
| salv-orlando | pcm_: sound async to me | 22:28 |
| pcm_ | salv-orlando: Not sure about the other commands though. I think they all expect to complete immed. | 22:28 |
| pcm_ | salv-orlando: Maybe I can defer all operations until the connect and at that time, send all the requests to server and immed. return. Use a second thread to wait for the response and update the status once replies occur? | 22:30 |
| pcm_ | salv-orlando: essentially batch and then async handle? could have a coarse timeout to detect server failure. | 22:31 |
| salv-orlando | pcm_: that would be the best course of action | 22:31 |
| salv-orlando | in my opinion | 22:31 |
| pcm_ | salv-orlando: Thanks for the advice! | 22:32 |
| salv-orlando | pcm_: np | 22:32 |
| pcm_ | salv-orlando: din din time... ttyl | 22:32 |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: This is a test, please ignore https://review.openstack.org/61574 | 22:36 |
| anteaya | salv-orlando: feel like reviewing a patch blocking neutronclient? https://review.openstack.org/#/c/61553/ | 22:37 |
| anteaya | what other cores are around? | 22:37 |
| anteaya | nati_ueno: ^^ | 22:37 |
| nati_ueno | anteaya: what's up? | 22:38 |
| salv-orlando | anteaya: is this for doc problem? | 22:38 |
| anteaya | can you review 61553? | 22:38 |
| *** alex_klimov has quit IRC | 22:38 | |
| anteaya | it is the sphinx pin for neutronclient | 22:38 |
| nati_ueno | ah OK | 22:38 |
| salv-orlando | approved | 22:38 |
| anteaya | salv-orlando: yes | 22:38 |
| anteaya | thank you | 22:38 |
| nati_ueno | so auto update isn't working yet for neutron client? | 22:38 |
| anteaya | yes it was | 22:39 |
| *** dims has joined #openstack-neutron | 22:39 | |
| nati_ueno | anteaya: gotcha. Thank you for your working on this too :) | 22:39 |
| anteaya | saw a failure on jeblair patch and cooked up a fix for neutronclient before I looked | 22:39 |
| anteaya | my fault | 22:39 |
| anteaya | nati_ueno: thanks :D | 22:39 |
| anteaya | will fix up the other Jenkins requirements patchs and get them in line | 22:39 |
| anteaya | meant to do it today already, how can it be dark out now? | 22:40 |
| anteaya | oh and snow, any skiers in the audience, going to be a great season | 22:40 |
| *** gdubreui has quit IRC | 22:40 | |
| *** SumitNaiksatam has quit IRC | 22:45 | |
| *** enikanorov__ has quit IRC | 22:49 | |
| *** csd has quit IRC | 22:49 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: This is a test, please ignore https://review.openstack.org/61574 | 22:49 |
| *** csd has joined #openstack-neutron | 22:50 | |
| *** litong has quit IRC | 22:51 | |
| *** jp_at_hp has quit IRC | 22:55 | |
| *** rkukura has joined #openstack-neutron | 22:59 | |
| *** pasquier-s has quit IRC | 23:05 | |
| anteaya | mlavalle: this passed Jenkins check https://review.openstack.org/#/c/61234/ if you would like to review | 23:06 |
| mlavalle | anteaya: reviewing…… | 23:07 |
| anteaya | thanks | 23:07 |
| *** pasquier-s has joined #openstack-neutron | 23:08 | |
| *** clev has quit IRC | 23:12 | |
| *** carl_baldwin has quit IRC | 23:16 | |
| openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: NVP plugin: Do backend router delete out from db transaction https://review.openstack.org/61604 | 23:20 |
| *** SumitNaiksatam has joined #openstack-neutron | 23:21 | |
| *** thedodd has quit IRC | 23:24 | |
| *** thedodd has joined #openstack-neutron | 23:24 | |
| anteaya | mlavalle: 61234 is in the gate queue | 23:30 |
| anteaya | hey thedodd | 23:30 |
| *** thedodd has quit IRC | 23:30 | |
| anteaya | and there he goes | 23:31 |
| mlavalle | anteaya: well, I might be missing something but I don't think it's going to work | 23:31 |
| mlavalle | anteaya: since it was approved by sdague and mtreinish, I'll just leave my comments with a 0 in the patch set | 23:33 |
| anteaya | mlavalle: okay | 23:33 |
| *** jgrimm has quit IRC | 23:33 | |
| *** dims has quit IRC | 23:34 | |
| anteaya | you think the test will fail, or you think the patch will fail to fix the bug it purports to fix? | 23:34 |
| mtreinish | mlavalle: there should be no harm from the patch | 23:35 |
| *** openstackgerrit has quit IRC | 23:35 | |
| *** openstackgerrit has joined #openstack-neutron | 23:36 | |
| mtreinish | whether it'll fix the bug I can't say | 23:36 |
| *** x86brandon has quit IRC | 23:36 | |
| mlavalle | mtreinish, anteaya: I think the patch is incorrect in the way the external gateway is specified in the when the router is created….. I just leave my comments in the patch set. As I say in my comments, I might be messing something | 23:37 |
| anteaya | mlavalle: mtreinish okay, well sounds like we are willing to let it merge and then evaluate its effectiveness once it is in master | 23:39 |
| anteaya | mlavalle: let's make sure we take a look at this tomorrow and evaluate | 23:39 |
| anteaya | unless of course something goes boom before that | 23:40 |
| mlavalle | anteaya; sure…. i'll be here | 23:40 |
| anteaya | thanks | 23:40 |
| mlavalle | anteaya: I left my comments in the patch set. As I said, I might be missing something | 23:45 |
| *** carl_baldwin has joined #openstack-neutron | 23:45 | |
| *** gdubreui has joined #openstack-neutron | 23:48 | |
| *** carl_baldwin has quit IRC | 23:49 | |
| *** openstackgerrit has quit IRC | 23:56 | |
| *** openstackgerrit has joined #openstack-neutron | 23:56 | |
| anteaya | mlavalle: okay thanks, fair enough | 23:58 |
| anteaya | let's observe what happens | 23:58 |
| mlavalle | anteaya: cool, I'm going offline….time to git the hit the lap pool and work out for a little bit | 23:58 |
| mlavalle | good night | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!