| *** boris-42 has joined #openstack-neutron | 00:04 | |
| *** EinstCra_ has joined #openstack-neutron | 00:05 | |
| *** neiljerram has quit IRC | 00:06 | |
| *** EinstCrazy has quit IRC | 00:06 | |
| *** shashank_hegde has joined #openstack-neutron | 00:07 | |
| *** EinstCra_ has quit IRC | 00:07 | |
| *** thorst has joined #openstack-neutron | 00:07 | |
| *** thorst has quit IRC | 00:08 | |
| *** thorst has joined #openstack-neutron | 00:08 | |
| *** banix has joined #openstack-neutron | 00:08 | |
| *** hichihara has joined #openstack-neutron | 00:08 | |
| *** apuimedo has quit IRC | 00:09 | |
| *** outofmemory is now known as reedip | 00:11 | |
| *** fedexo has joined #openstack-neutron | 00:16 | |
| *** fedexo has quit IRC | 00:17 | |
| *** fedexo has joined #openstack-neutron | 00:19 | |
| brad_behle | reedip: Do you have a minute? I have a question about a few reviews I have out there that you updated, 276495 and 279273. | 00:20 |
|---|---|---|
| reedip | brad_behle : sure | 00:20 |
| brad_behle | reedip: In 276495, it looks like you uploaded patch sets 7 and 8, and reverted a few changes I had made in those files? | 00:22 |
| reedip | brad_behle: I think you are right. My patch https://review.openstack.org/#/c/281129 is dependent on both these patches. So when I made a change, it was reflected in the above 2 patches | 00:23 |
| brad_behle | It looks like you undid a few changes I had made in previous patch sets, I couldn't tell why? | 00:23 |
| *** thorst has quit IRC | 00:23 | |
| reedip | brad_behle: I guess after I took one of your PS, you made changes, which were reverted when I uploaded my PS ( for subnet set) | 00:23 |
| *** thorst has joined #openstack-neutron | 00:24 | |
| reedip | brad_behle: If you want I can revert the changes in your PS to incorporate your changes and meanwhile put my patch on hold till your PS doesnt get merged | 00:24 |
| brad_behle | reedip: Okay. I'm fine just re-making my changes (I have other changes I need to make in those as well), if that is easier. Just wanted to check with you to see what you thought would be best. | 00:25 |
| reedip | brad_behle: Please proceed with your changes. I will wait for your patches to merge, and then I will publish my modifications. That ways, everything would be synced up | 00:26 |
| *** thorst has quit IRC | 00:27 | |
| *** thorst has joined #openstack-neutron | 00:27 | |
| *** subscope has quit IRC | 00:27 | |
| brad_behle | reedip: Okay, sounds good. I'll work on them my time tomorrow and get new patch sets up. | 00:27 |
| reedip | brad_behle: +1 | 00:28 |
| *** jckasper has quit IRC | 00:36 | |
| *** achanda has quit IRC | 00:36 | |
| *** jckasper has joined #openstack-neutron | 00:36 | |
| *** jckasper has quit IRC | 00:41 | |
| *** salv-orl_ has joined #openstack-neutron | 00:42 | |
| *** thorst has quit IRC | 00:42 | |
| *** thorst has joined #openstack-neutron | 00:43 | |
| *** salv-orlando has quit IRC | 00:45 | |
| *** hoangcx has joined #openstack-neutron | 00:46 | |
| *** thorst has quit IRC | 00:51 | |
| *** amotoki has joined #openstack-neutron | 00:54 | |
| *** mfuruta has joined #openstack-neutron | 00:55 | |
| *** wolverineav has quit IRC | 00:58 | |
| *** salv-orl_ has quit IRC | 01:00 | |
| *** jckasper has joined #openstack-neutron | 01:01 | |
| *** fedexo has quit IRC | 01:03 | |
| *** djan has quit IRC | 01:04 | |
| *** fedexo has joined #openstack-neutron | 01:04 | |
| *** djan has joined #openstack-neutron | 01:05 | |
| *** markvoelker has joined #openstack-neutron | 01:09 | |
| *** NamNH has joined #openstack-neutron | 01:10 | |
| *** thorst has joined #openstack-neutron | 01:11 | |
| *** stanzgy has joined #openstack-neutron | 01:12 | |
| *** markvoelker has quit IRC | 01:13 | |
| *** jckasper has quit IRC | 01:13 | |
| *** jckasper has joined #openstack-neutron | 01:14 | |
| *** tfukushima has joined #openstack-neutron | 01:17 | |
| *** jckasper has quit IRC | 01:18 | |
| *** rkukura has joined #openstack-neutron | 01:20 | |
| *** Sukhdev has joined #openstack-neutron | 01:22 | |
| *** banix has quit IRC | 01:23 | |
| *** annp has joined #openstack-neutron | 01:24 | |
| *** Sukhdev has quit IRC | 01:26 | |
| openstackgerrit | Merged openstack/neutron: Pecan: replace dashes with underscores on controller lookup https://review.openstack.org/272312 | 01:27 |
| openstackgerrit | Reedip proposed openstack/python-neutronclient: Update help information for lbaasv2 CLIs https://review.openstack.org/255093 | 01:28 |
| openstackgerrit | Reedip proposed openstack/python-neutronclient: Update help information for lbaasv2 CLIs https://review.openstack.org/255093 | 01:28 |
| openstackgerrit | Reedip proposed openstack/python-neutronclient: Add timeout attribute to LBaaS v2 Listeners https://review.openstack.org/273911 | 01:29 |
| *** bochi-michael has joined #openstack-neutron | 01:33 | |
| *** achanda has joined #openstack-neutron | 01:37 | |
| *** rkukura has quit IRC | 01:38 | |
| *** djan_ has joined #openstack-neutron | 01:40 | |
| *** Sukhdev has joined #openstack-neutron | 01:40 | |
| *** rkukura has joined #openstack-neutron | 01:40 | |
| *** elopez_ has quit IRC | 01:41 | |
| *** djan has quit IRC | 01:41 | |
| *** achanda has quit IRC | 01:42 | |
| *** jckasper has joined #openstack-neutron | 01:43 | |
| *** elo has joined #openstack-neutron | 01:45 | |
| *** EinstCrazy has joined #openstack-neutron | 01:48 | |
| *** baojg has joined #openstack-neutron | 01:49 | |
| *** achanda has joined #openstack-neutron | 01:52 | |
| *** yamamoto has joined #openstack-neutron | 01:52 | |
| *** wolverineav has joined #openstack-neutron | 01:58 | |
| *** thorst has quit IRC | 01:59 | |
| *** thorst has joined #openstack-neutron | 01:59 | |
| *** wolverineav has quit IRC | 02:00 | |
| *** wolverineav has joined #openstack-neutron | 02:04 | |
| *** wolverineav has quit IRC | 02:06 | |
| *** wolverineav has joined #openstack-neutron | 02:06 | |
| *** thorst has quit IRC | 02:08 | |
| *** miyagishi_t has joined #openstack-neutron | 02:08 | |
| *** yamamoto has quit IRC | 02:14 | |
| openstackgerrit | Reedip proposed openstack/python-neutronclient: Fix assertNotEqual parameters https://review.openstack.org/282933 | 02:14 |
| *** watanabe_isao has joined #openstack-neutron | 02:15 | |
| *** thanhnt-z has joined #openstack-neutron | 02:16 | |
| *** ircuser-1 has joined #openstack-neutron | 02:16 | |
| *** thanhnt-z has quit IRC | 02:20 | |
| *** minwang2 has joined #openstack-neutron | 02:22 | |
| *** fedexo has quit IRC | 02:27 | |
| *** xiaohhui has joined #openstack-neutron | 02:29 | |
| *** bochi-michael has quit IRC | 02:37 | |
| *** bochi-michael has joined #openstack-neutron | 02:38 | |
| *** salv-orlando has joined #openstack-neutron | 02:38 | |
| *** salv-orlando has quit IRC | 02:41 | |
| *** minwang2 has quit IRC | 02:43 | |
| *** dronshaw has joined #openstack-neutron | 02:46 | |
| *** dronshaw has left #openstack-neutron | 02:47 | |
| *** dkehn has quit IRC | 02:49 | |
| *** Marga_ has quit IRC | 02:51 | |
| *** amotoki has quit IRC | 02:52 | |
| *** hoangcx has quit IRC | 02:52 | |
| *** minwang2 has joined #openstack-neutron | 02:53 | |
| *** salv-orlando has joined #openstack-neutron | 02:55 | |
| *** dkehn has joined #openstack-neutron | 02:56 | |
| *** ircuser-1 has quit IRC | 02:57 | |
| *** yamamoto has joined #openstack-neutron | 02:58 | |
| *** bochi-michael has quit IRC | 02:59 | |
| *** salv-orlando has quit IRC | 03:01 | |
| *** vhosakot has joined #openstack-neutron | 03:02 | |
| *** amotoki has joined #openstack-neutron | 03:04 | |
| *** vhosakot has quit IRC | 03:05 | |
| *** thorst has joined #openstack-neutron | 03:06 | |
| *** markvoelker has joined #openstack-neutron | 03:09 | |
| *** wolverineav has quit IRC | 03:12 | |
| *** wenjianhn has joined #openstack-neutron | 03:12 | |
| *** thorst has quit IRC | 03:12 | |
| openstackgerrit | ZhaoBo proposed openstack/neutron: make/update_subnetpool_dict call _dict_extend https://review.openstack.org/282941 | 03:14 |
| *** markvoelker has quit IRC | 03:14 | |
| *** amotoki has quit IRC | 03:15 | |
| *** wolverineav has joined #openstack-neutron | 03:17 | |
| *** shivrao has joined #openstack-neutron | 03:18 | |
| *** minwang2 has quit IRC | 03:18 | |
| *** yuanying has quit IRC | 03:20 | |
| *** amotoki has joined #openstack-neutron | 03:20 | |
| *** wolverineav has quit IRC | 03:21 | |
| openstackgerrit | ZhaoBo proposed openstack/neutron: Add timestamp for neutron core resources https://review.openstack.org/213586 | 03:21 |
| *** wolverineav has joined #openstack-neutron | 03:23 | |
| *** stanzgy has quit IRC | 03:24 | |
| *** minwang2 has joined #openstack-neutron | 03:25 | |
| *** amotoki_ has joined #openstack-neutron | 03:25 | |
| *** jckasper has quit IRC | 03:26 | |
| *** wolverineav has quit IRC | 03:28 | |
| *** Marga_ has joined #openstack-neutron | 03:28 | |
| *** baoli has quit IRC | 03:28 | |
| *** amotoki has quit IRC | 03:29 | |
| *** Marga_ has quit IRC | 03:29 | |
| *** baojg has quit IRC | 03:29 | |
| *** Marga_ has joined #openstack-neutron | 03:29 | |
| *** baojg has joined #openstack-neutron | 03:31 | |
| *** dingboopt has joined #openstack-neutron | 03:32 | |
| openstackgerrit | Hong Hui Xiao proposed openstack/neutron: Correlate address scope with network https://review.openstack.org/282942 | 03:32 |
| *** watanabe_isao has quit IRC | 03:35 | |
| *** amotoki_ has quit IRC | 03:35 | |
| *** stanzgy has joined #openstack-neutron | 03:36 | |
| *** wenjianhn has quit IRC | 03:37 | |
| *** baojg has quit IRC | 03:38 | |
| *** micucci has quit IRC | 03:39 | |
| *** amotoki has joined #openstack-neutron | 03:41 | |
| *** watanabe_isao has joined #openstack-neutron | 03:42 | |
| *** gvrangan has joined #openstack-neutron | 03:42 | |
| *** amotoki has quit IRC | 03:44 | |
| *** Marga_ has quit IRC | 03:44 | |
| *** wolverineav has joined #openstack-neutron | 03:45 | |
| *** Marga_ has joined #openstack-neutron | 03:45 | |
| *** wenjianhn has joined #openstack-neutron | 03:45 | |
| *** julim has quit IRC | 03:46 | |
| *** Marga_ has quit IRC | 03:49 | |
| *** wolverineav has quit IRC | 03:50 | |
| *** Aish has joined #openstack-neutron | 03:52 | |
| *** Aish has quit IRC | 03:52 | |
| *** minwang2 has quit IRC | 03:53 | |
| *** salv-orlando has joined #openstack-neutron | 03:57 | |
| *** links has joined #openstack-neutron | 03:57 | |
| *** amotoki has joined #openstack-neutron | 04:00 | |
| *** allan_h has joined #openstack-neutron | 04:03 | |
| *** salv-orlando has quit IRC | 04:04 | |
| *** Marga_ has joined #openstack-neutron | 04:10 | |
| *** yuanying has joined #openstack-neutron | 04:10 | |
| *** thorst has joined #openstack-neutron | 04:10 | |
| *** jckasper has joined #openstack-neutron | 04:15 | |
| *** brad_behle has quit IRC | 04:17 | |
| *** lnicolas has quit IRC | 04:17 | |
| *** thorst has quit IRC | 04:17 | |
| *** lnicolas has joined #openstack-neutron | 04:18 | |
| *** singhj has quit IRC | 04:18 | |
| *** gvrangan has quit IRC | 04:18 | |
| *** jckasper has quit IRC | 04:19 | |
| *** singhj has joined #openstack-neutron | 04:19 | |
| *** Marga_ has quit IRC | 04:20 | |
| *** singhj has quit IRC | 04:21 | |
| *** z_kassab has joined #openstack-neutron | 04:21 | |
| *** dave-mcc_ has joined #openstack-neutron | 04:24 | |
| *** dave-mccowan has quit IRC | 04:25 | |
| *** gvrangan has joined #openstack-neutron | 04:26 | |
| *** alhu_ has joined #openstack-neutron | 04:26 | |
| *** allan_h has quit IRC | 04:28 | |
| *** dave-mcc_ has quit IRC | 04:28 | |
| *** tfukushima has quit IRC | 04:34 | |
| *** gvrangan has quit IRC | 04:36 | |
| *** alhu__ has joined #openstack-neutron | 04:39 | |
| *** alhu_ has quit IRC | 04:39 | |
| *** dave-mccowan has joined #openstack-neutron | 04:40 | |
| *** watanabe_isao has quit IRC | 04:40 | |
| *** vishwanathj has joined #openstack-neutron | 04:40 | |
| openstackgerrit | Merged openstack/neutron: Fix for adding gateway with IP outside subnet https://review.openstack.org/233287 | 04:43 |
| *** z_kassab has quit IRC | 04:50 | |
| *** bharathk has joined #openstack-neutron | 04:55 | |
| openstackgerrit | ZhaoBo proposed openstack/neutron: Add timestamp for neutron core resources https://review.openstack.org/213586 | 05:00 |
| *** djan_ has quit IRC | 05:01 | |
| *** djan has joined #openstack-neutron | 05:02 | |
| *** Marga_ has joined #openstack-neutron | 05:05 | |
| *** wolverineav has joined #openstack-neutron | 05:05 | |
| *** lnicolas has quit IRC | 05:05 | |
| *** anilvenkata has joined #openstack-neutron | 05:07 | |
| *** ekuris has joined #openstack-neutron | 05:09 | |
| *** markvoelker has joined #openstack-neutron | 05:10 | |
| openstackgerrit | ZhaoBo proposed openstack/neutron: Add timestamp for neutron core resources https://review.openstack.org/213586 | 05:10 |
| *** tflynn has joined #openstack-neutron | 05:11 | |
| *** gberginc_ is now known as gberginc | 05:12 | |
| *** wolverineav has quit IRC | 05:14 | |
| *** rkukura has quit IRC | 05:14 | |
| *** markvoelker has quit IRC | 05:15 | |
| *** thorst has joined #openstack-neutron | 05:15 | |
| *** sridharg has joined #openstack-neutron | 05:20 | |
| *** sudipto has joined #openstack-neutron | 05:22 | |
| *** thorst has quit IRC | 05:22 | |
| *** EinstCra_ has joined #openstack-neutron | 05:23 | |
| *** watanabe_isao has joined #openstack-neutron | 05:23 | |
| *** minwang2 has joined #openstack-neutron | 05:24 | |
| *** EinstCrazy has quit IRC | 05:25 | |
| *** tflynn has quit IRC | 05:28 | |
| *** tflynn has joined #openstack-neutron | 05:29 | |
| *** Marga_ has quit IRC | 05:30 | |
| *** tfukushima has joined #openstack-neutron | 05:32 | |
| *** dave-mccowan has quit IRC | 05:37 | |
| *** ijw has joined #openstack-neutron | 05:40 | |
| *** baoli has joined #openstack-neutron | 05:40 | |
| *** rdo has quit IRC | 05:45 | |
| *** gongysh has joined #openstack-neutron | 05:45 | |
| *** baoli has quit IRC | 05:45 | |
| *** daneyon_ has quit IRC | 05:45 | |
| *** rdo has joined #openstack-neutron | 05:47 | |
| *** ijw has quit IRC | 05:50 | |
| *** ijw has joined #openstack-neutron | 05:50 | |
| openstackgerrit | Sreekumar S proposed openstack/neutron: Deprecate 'force_gateway_on_subnet' configuration option https://review.openstack.org/277303 | 05:51 |
| *** sridharg has quit IRC | 05:52 | |
| *** mkolesni_ has joined #openstack-neutron | 05:53 | |
| openstackgerrit | Reedip proposed openstack/neutron: Return a 404 on quota delete if entry not found https://review.openstack.org/276096 | 06:02 |
| *** hanchao has joined #openstack-neutron | 06:05 | |
| *** ijw_ has joined #openstack-neutron | 06:08 | |
| *** moshele has joined #openstack-neutron | 06:08 | |
| *** minwang2 has quit IRC | 06:08 | |
| *** minwang2 has joined #openstack-neutron | 06:10 | |
| *** hoangcx has joined #openstack-neutron | 06:10 | |
| *** wolverineav has joined #openstack-neutron | 06:11 | |
| *** salv-orlando has joined #openstack-neutron | 06:11 | |
| *** ijw has quit IRC | 06:11 | |
| openstackgerrit | ZhaoBo proposed openstack/neutron: Add timestamp for neutron core resources https://review.openstack.org/213586 | 06:12 |
| *** sleviim has joined #openstack-neutron | 06:13 | |
| *** salv-orlando has quit IRC | 06:14 | |
| *** wolverineav has quit IRC | 06:15 | |
| *** minwang2 has quit IRC | 06:16 | |
| *** minwang2 has joined #openstack-neutron | 06:16 | |
| *** alhu__ has quit IRC | 06:19 | |
| *** thorst has joined #openstack-neutron | 06:20 | |
| *** Marga_ has joined #openstack-neutron | 06:22 | |
| *** ijw has joined #openstack-neutron | 06:24 | |
| *** ijw_ has quit IRC | 06:25 | |
| *** ijw_ has joined #openstack-neutron | 06:25 | |
| *** Guest61736 is now known as mariusv | 06:27 | |
| *** mariusv has joined #openstack-neutron | 06:27 | |
| *** thorst has quit IRC | 06:28 | |
| *** ijw has quit IRC | 06:29 | |
| *** ircuser-1 has joined #openstack-neutron | 06:30 | |
| *** hdaniel has joined #openstack-neutron | 06:30 | |
| *** itzikb_ has joined #openstack-neutron | 06:34 | |
| *** yfried_ has quit IRC | 06:34 | |
| *** shivrao has quit IRC | 06:35 | |
| *** dedery has joined #openstack-neutron | 06:35 | |
| *** sridharg has joined #openstack-neutron | 06:35 | |
| *** mubirru has joined #openstack-neutron | 06:35 | |
| *** john-davidge has joined #openstack-neutron | 06:36 | |
| *** EinstCrazy has joined #openstack-neutron | 06:38 | |
| *** changzhi has joined #openstack-neutron | 06:38 | |
| *** anshul has joined #openstack-neutron | 06:38 | |
| *** javeriak has joined #openstack-neutron | 06:38 | |
| *** EinstCra_ has quit IRC | 06:39 | |
| *** dedery has quit IRC | 06:40 | |
| *** mubirru_ has joined #openstack-neutron | 06:41 | |
| *** mubirru has quit IRC | 06:41 | |
| *** mubirru_ is now known as mubirru | 06:41 | |
| hichihara | amotoki: ping | 06:43 |
| openstackgerrit | Manjeet Singh Bhatia proposed openstack/neutron: Add db migrations test framework with data https://review.openstack.org/124946 | 06:44 |
| amotoki | hichihara: pong | 06:45 |
| hichihara | amotoki: Could you review tag patch again? | 06:45 |
| amotoki | hichihara: sure | 06:45 |
| hichihara | amotoki: I have updated according with your comments. | 06:45 |
| hichihara | amotoki: Thank you! | 06:46 |
| amotoki | hichihara: it seems you succeeded to pass the integrated tests :) | 06:46 |
| hichihara | amotoki: Yeah, I missed nit ;) | 06:46 |
| *** jhershbe has joined #openstack-neutron | 06:46 | |
| amotoki | hichihara: model/head? | 06:46 |
| hichihara | amotoki: Yes | 06:47 |
| amotoki | hichihara: I think it looks good in general. | 06:48 |
| hichihara | amotoki: awesome | 06:48 |
| amotoki | hichihara: btw, do you add tag replaces or something to CLI? | 06:48 |
| amotoki | hichihara: i just would like to cover operations written in the spec. | 06:49 |
| hichihara | amotoki: Do you mean tag-update? | 06:49 |
| amotoki | hichihara: I am not sure which is better : "net-update --tag red --tag blue" or "tag-update" | 06:50 |
| amotoki | tag-XXX looks consistent. | 06:51 |
| hichihara | amotoki: I understand your point. | 06:51 |
| hichihara | amotoki: I'm thinking that I would add tag-update and tag-delete (with --all option) | 06:53 |
| *** john-davidge has quit IRC | 06:54 | |
| amotoki | hichihara: sounds good. | 06:54 |
| *** salv-orlando has joined #openstack-neutron | 06:54 | |
| amotoki | hichihara: I just noticed we have no way to specify tags when creating a network :-( | 06:54 |
| *** dedery has joined #openstack-neutron | 06:55 | |
| hichihara | amotoki: I think that it's not defined in spec. | 06:56 |
| amotoki | hichihara: yeah... there is no mention in the API-WG guideline too. | 06:57 |
| *** javeriak has quit IRC | 06:57 | |
| *** javeriak has joined #openstack-neutron | 06:57 | |
| *** ijw_ has quit IRC | 06:58 | |
| *** ijw has joined #openstack-neutron | 06:58 | |
| hichihara | amotoki: I tried to add POST but I have a issue so I removed it. | 06:58 |
| *** oshvartz has joined #openstack-neutron | 06:58 | |
| hichihara | amotoki: You can see the remains in Patchset 6. | 06:59 |
| amotoki | hichihara: no problem. let's focus on merging the current proposal and then try it :) | 06:59 |
| *** wolverineav has joined #openstack-neutron | 06:59 | |
| hichihara | amotoki: Thanks. I think so. | 07:00 |
| *** josecastroleon has joined #openstack-neutron | 07:00 | |
| *** watanabe_isao has quit IRC | 07:01 | |
| *** neeti has joined #openstack-neutron | 07:01 | |
| *** watanabe_isao has joined #openstack-neutron | 07:01 | |
| *** salv-orlando has quit IRC | 07:02 | |
| *** ijw has quit IRC | 07:03 | |
| *** baojg has joined #openstack-neutron | 07:04 | |
| openstackgerrit | linwei wu proposed openstack/neutron: Remove unclear built-in in subnetpool exception https://review.openstack.org/270649 | 07:07 |
| *** djan_ has joined #openstack-neutron | 07:08 | |
| *** eddima has joined #openstack-neutron | 07:10 | |
| *** djan has quit IRC | 07:10 | |
| *** markvoelker has joined #openstack-neutron | 07:11 | |
| openstackgerrit | linwei wu proposed openstack/neutron: Remove unclear built-in in subnetpool exception https://review.openstack.org/270649 | 07:15 |
| *** markvoelker has quit IRC | 07:15 | |
| *** minwang2 has quit IRC | 07:15 | |
| *** hdaniel has quit IRC | 07:16 | |
| *** kobis has joined #openstack-neutron | 07:21 | |
| amotoki | hichihara: I am reading the tag spec in neutron-spec again. | 07:22 |
| amotoki | hichihara: when we put an existing tag, it says "If the tag already exists, no error is raised, it just returns the 409 Conflict". | 07:22 |
| amotoki | hichihara: which is correct "no error is raised" and "409 conflict"? | 07:23 |
| amotoki | hichihara: it seems inconsistent. | 07:23 |
| hichihara | amotoki: woops, I forgot to remove "no error is raised" | 07:23 |
| amotoki | hichihara: thanks. let's clean up later :) | 07:24 |
| hichihara | amotoki: Sorry for my mistake ;) | 07:24 |
| amotoki | hichihara: np | 07:24 |
| *** tflynn_ has joined #openstack-neutron | 07:24 | |
| *** thorst has joined #openstack-neutron | 07:26 | |
| *** ildikov has quit IRC | 07:27 | |
| *** hynekm has joined #openstack-neutron | 07:27 | |
| *** lajos-katona has joined #openstack-neutron | 07:28 | |
| *** tflynn has quit IRC | 07:28 | |
| *** chlong_ has quit IRC | 07:30 | |
| *** korzen has joined #openstack-neutron | 07:31 | |
| *** thorst has quit IRC | 07:33 | |
| *** jhershbe has quit IRC | 07:33 | |
| *** w_verdugo has quit IRC | 07:34 | |
| *** javeriak has quit IRC | 07:36 | |
| openstackgerrit | Hynek Mlnarik proposed openstack/neutron: WIP: Squash initialization of OVS bridge flows https://review.openstack.org/282636 | 07:36 |
| *** jhershbe has joined #openstack-neutron | 07:39 | |
| *** salv-orlando has joined #openstack-neutron | 07:40 | |
| *** yfried_ has joined #openstack-neutron | 07:45 | |
| *** javeriak has joined #openstack-neutron | 07:46 | |
| *** shashank_hegde has quit IRC | 07:47 | |
| *** tflynn_ has quit IRC | 07:47 | |
| openstackgerrit | Lujin Luo proposed openstack/neutron: Add a composite unique key to floatingip table in Neutron database https://review.openstack.org/276018 | 07:47 |
| *** shashank_hegde has joined #openstack-neutron | 07:47 | |
| openstackgerrit | ZhaoBo proposed openstack/neutron: Add timestamp for neutron core resources https://review.openstack.org/213586 | 07:48 |
| *** ivar-lazzaro has joined #openstack-neutron | 07:48 | |
| *** nmagnezi has joined #openstack-neutron | 07:50 | |
| *** numans has joined #openstack-neutron | 07:50 | |
| *** kobis has quit IRC | 07:50 | |
| *** tflynn has joined #openstack-neutron | 07:51 | |
| *** nplanel has joined #openstack-neutron | 07:51 | |
| *** subscope has joined #openstack-neutron | 07:51 | |
| *** subscope has quit IRC | 07:51 | |
| *** jamielennox is now known as jamielennox|away | 07:52 | |
| *** scheuran has joined #openstack-neutron | 07:53 | |
| *** kobis has joined #openstack-neutron | 07:53 | |
| *** shwetaap2 has quit IRC | 07:56 | |
| *** shwetaap has joined #openstack-neutron | 07:56 | |
| *** xianghui has quit IRC | 07:57 | |
| *** dosaboy has quit IRC | 07:57 | |
| *** kickinz1 has quit IRC | 07:57 | |
| *** wolsen has quit IRC | 07:57 | |
| *** ijw has joined #openstack-neutron | 07:59 | |
| *** irenab has quit IRC | 07:59 | |
| *** hdaniel has joined #openstack-neutron | 08:00 | |
| *** wolsen has joined #openstack-neutron | 08:00 | |
| *** kickinz1 has joined #openstack-neutron | 08:01 | |
| *** sudipto_ has joined #openstack-neutron | 08:01 | |
| *** kickinz1 has quit IRC | 08:01 | |
| *** kickinz1 has joined #openstack-neutron | 08:01 | |
| *** shwetaap has quit IRC | 08:01 | |
| *** ildikov has joined #openstack-neutron | 08:01 | |
| *** jamielennox|away is now known as jamielennox | 08:02 | |
| *** tmorin has joined #openstack-neutron | 08:02 | |
| openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Add tags for neutron resources https://review.openstack.org/273881 | 08:03 |
| *** dosaboy has joined #openstack-neutron | 08:04 | |
| *** xianghui has joined #openstack-neutron | 08:04 | |
| *** ijw has quit IRC | 08:04 | |
| *** sudipto has quit IRC | 08:04 | |
| *** dedery has quit IRC | 08:05 | |
| *** pcaruana has joined #openstack-neutron | 08:05 | |
| *** subscope has joined #openstack-neutron | 08:07 | |
| *** javeriak has quit IRC | 08:09 | |
| *** ildikov has quit IRC | 08:09 | |
| *** hichihara has quit IRC | 08:09 | |
| *** hoangcx has quit IRC | 08:09 | |
| *** hdaniel has quit IRC | 08:09 | |
| *** ociuhandu has quit IRC | 08:11 | |
| *** ivar-lazzaro has quit IRC | 08:13 | |
| *** matrohon has joined #openstack-neutron | 08:15 | |
| *** javeriak has joined #openstack-neutron | 08:16 | |
| *** subscope has quit IRC | 08:17 | |
| *** fzdarsky has joined #openstack-neutron | 08:20 | |
| *** yfried_ has quit IRC | 08:21 | |
| *** yfried_ has joined #openstack-neutron | 08:21 | |
| *** apuimedo has joined #openstack-neutron | 08:22 | |
| *** ildikov has joined #openstack-neutron | 08:22 | |
| *** hdaniel has joined #openstack-neutron | 08:22 | |
| *** kawa2014 has joined #openstack-neutron | 08:23 | |
| *** Sukhdev has quit IRC | 08:24 | |
| *** salv-orlando has quit IRC | 08:25 | |
| *** subscope has joined #openstack-neutron | 08:26 | |
| *** jpena has joined #openstack-neutron | 08:26 | |
| *** jlanoux has joined #openstack-neutron | 08:28 | |
| *** changzhi has quit IRC | 08:29 | |
| *** changzhi has joined #openstack-neutron | 08:29 | |
| *** thorst has joined #openstack-neutron | 08:30 | |
| *** ihrachys has joined #openstack-neutron | 08:31 | |
| *** jistr has joined #openstack-neutron | 08:37 | |
| *** gampel has joined #openstack-neutron | 08:38 | |
| *** thorst has quit IRC | 08:38 | |
| *** subscope has quit IRC | 08:40 | |
| *** achanda has quit IRC | 08:42 | |
| *** yfried_ has quit IRC | 08:43 | |
| *** yfried has joined #openstack-neutron | 08:43 | |
| *** nyechiel_ has joined #openstack-neutron | 08:44 | |
| *** javeriak has quit IRC | 08:44 | |
| openstackgerrit | Reedip proposed openstack/python-neutronclient: Raise error for duplicate parameters https://review.openstack.org/230321 | 08:44 |
| *** achanda has joined #openstack-neutron | 08:46 | |
| *** claudiub has joined #openstack-neutron | 08:46 | |
| *** jschwarz has joined #openstack-neutron | 08:53 | |
| *** nyechiel_ has quit IRC | 08:53 | |
| *** safchain has joined #openstack-neutron | 08:54 | |
| *** javeriak has joined #openstack-neutron | 08:55 | |
| openstackgerrit | Reedip proposed openstack/python-neutronclient: Improve help messages for NeutronClient https://review.openstack.org/255697 | 08:57 |
| *** achanda has quit IRC | 08:58 | |
| *** miyagishi_t has quit IRC | 08:59 | |
| *** rkukura has joined #openstack-neutron | 09:00 | |
| *** tflynn has quit IRC | 09:00 | |
| openstackgerrit | Reedip proposed openstack/python-neutronclient: Improve Help message of VPN Update CLI https://review.openstack.org/271191 | 09:01 |
| *** ijw has joined #openstack-neutron | 09:01 | |
| *** subscope has joined #openstack-neutron | 09:01 | |
| *** garyk has quit IRC | 09:01 | |
| *** garyk has joined #openstack-neutron | 09:02 | |
| gsagie | garyk: sup | 09:03 |
| *** rvba has quit IRC | 09:03 | |
| reedip | kevinbenton: ping ? | 09:03 |
| *** javeriak has quit IRC | 09:04 | |
| *** mgoddard has joined #openstack-neutron | 09:04 | |
| *** ijw has quit IRC | 09:05 | |
| *** amotoki has quit IRC | 09:06 | |
| *** watanabe_isao has quit IRC | 09:07 | |
| *** MCoLo has quit IRC | 09:07 | |
| *** MCoLo has joined #openstack-neutron | 09:08 | |
| *** dedery has joined #openstack-neutron | 09:10 | |
| *** rkukura has quit IRC | 09:10 | |
| *** markvoelker has joined #openstack-neutron | 09:12 | |
| *** dedery has quit IRC | 09:14 | |
| *** amotoki has joined #openstack-neutron | 09:14 | |
| *** markvoelker has quit IRC | 09:16 | |
| garyk | gsagie: gutters and strikes | 09:16 |
| *** mfuruta has quit IRC | 09:17 | |
| *** baojg has quit IRC | 09:19 | |
| *** oanson has quit IRC | 09:20 | |
| *** saggi has quit IRC | 09:20 | |
| *** gsagie has quit IRC | 09:22 | |
| *** rvba has joined #openstack-neutron | 09:23 | |
| *** rvba has quit IRC | 09:23 | |
| *** rvba has joined #openstack-neutron | 09:23 | |
| *** hoangcx has joined #openstack-neutron | 09:23 | |
| *** gongysh has quit IRC | 09:23 | |
| scheuran | hi ihrachys, good morning! Would you mind taking this on your review list? https://review.openstack.org/275306 It's the first other exploiter of the common agent thing we did a couple of weeks ago | 09:23 |
| *** sudipto__ has joined #openstack-neutron | 09:24 | |
| *** efoley has joined #openstack-neutron | 09:24 | |
| ihrachys | scheuran: probably won't be able to get to it this cycle. I guess you want it in M, so better reach other folks. | 09:25 |
| ihrachys | scheuran: do we have approvers for the work? | 09:25 |
| *** subscope has quit IRC | 09:25 | |
| *** claudiub has quit IRC | 09:26 | |
| ihrachys | scheuran: for BPs, we have those; what about RFEs? | 09:26 |
| scheuran | ihrachys, ok, no problem. as it's an rfe not really | 09:26 |
| ihrachys | hm, that's illogical to my taste | 09:26 |
| ihrachys | that we have no approvers for RFEs | 09:26 |
| ihrachys | but have those for BPs | 09:26 |
| *** mickeys has joined #openstack-neutron | 09:26 | |
| ihrachys | there is no essential difference from my POV | 09:27 |
| jschwarz | kevinbenton, top of the morning to you, sir | 09:27 |
| scheuran | maybe there are in offical approvers :) | 09:27 |
| ihrachys | scheuran: lemme check policy docs on the matter. | 09:27 |
| jschwarz | ihrachys, did you hear the news? | 09:27 |
| *** sudipto_ has quit IRC | 09:28 | |
| scheuran | ihrachys, I had a quick talk to kevinbenton on the summit - so I probably will reach out to him... | 09:28 |
| ihrachys | jschwarz: tell me | 09:28 |
| *** ociuhandu has joined #openstack-neutron | 09:28 | |
| jschwarz | ihrachys, I'll see you in Austin and tell you then | 09:28 |
| *** neiljerram has joined #openstack-neutron | 09:28 | |
| jschwarz | ihrachys, ;-) | 09:28 |
| openstackgerrit | steve proposed openstack/neutron-specs: Add support for external vxlan encapsulation to neutron router https://review.openstack.org/282180 | 09:28 |
| ihrachys | jschwarz: WOOT | 09:28 |
| jschwarz | ihrachys, yep | 09:28 |
| ihrachys | jschwarz: will be happy to see you there. well deserved. | 09:29 |
| jschwarz | ihrachys, thank you, thank you | 09:29 |
| *** subscope has joined #openstack-neutron | 09:29 | |
| jschwarz | excited to finally come as well | 09:29 |
| ihrachys | jschwarz: gotta crawl bars ;) I hope you book with some time for that. | 09:32 |
| *** _oanson has joined #openstack-neutron | 09:33 | |
| jschwarz | ihrachys, so I'm arriving late Sunday and leaving early Saturday | 09:33 |
| jschwarz | ihrachys, but anything in between is fair game :D | 09:34 |
| *** saggi has joined #openstack-neutron | 09:34 | |
| jschwarz | ihrachys, when are you arriving/leaving? | 09:34 |
| *** amotoki has quit IRC | 09:35 | |
| ihrachys | jschwarz: I think that's same for me. gotta crawl Fri. :) | 09:35 |
| *** thorst has joined #openstack-neutron | 09:35 | |
| *** ildikov has quit IRC | 09:36 | |
| *** saggi has quit IRC | 09:36 | |
| jschwarz | ihrachys, my flight out from Austin leaves 8am on Saturday, so I gotta be up by 4:30am-ish | 09:37 |
| jschwarz | ihrachys, the alternative is not sleeping | 09:37 |
| jschwarz | can do | 09:37 |
| ihrachys | jschwarz: or you just don't sleep :D | 09:37 |
| jschwarz | XD | 09:37 |
| ihrachys | you'll have plenty of time for that on the plane | 09:37 |
| jschwarz | ihrachys, we'll prolly have some time mid-week for some early-bar-crawling and still get some sleep | 09:37 |
| jschwarz | ie. go out drinking 8pm and get to the hotel early enough to get some shut-eye | 09:38 |
| *** mvk has joined #openstack-neutron | 09:38 | |
| *** vishwanathj has quit IRC | 09:39 | |
| *** hoangcx has quit IRC | 09:39 | |
| *** _oanson has quit IRC | 09:39 | |
| *** claudiub has joined #openstack-neutron | 09:41 | |
| openstackgerrit | steve proposed openstack/neutron-specs: Add support for external vxlan encapsulation to neutron router https://review.openstack.org/282180 | 09:41 |
| *** thorst has quit IRC | 09:43 | |
| *** davidsha has joined #openstack-neutron | 09:45 | |
| *** ildikov has joined #openstack-neutron | 09:50 | |
| *** claudiub has quit IRC | 09:51 | |
| *** _oanson has joined #openstack-neutron | 09:52 | |
| *** saggi has joined #openstack-neutron | 09:52 | |
| *** claudiub has joined #openstack-neutron | 09:52 | |
| *** efoley_ has joined #openstack-neutron | 09:53 | |
| kevinbenton | jschwarz: yo | 09:54 |
| *** javeriak has joined #openstack-neutron | 09:54 | |
| jschwarz | kevinbenton, yo yo | 09:54 |
| kevinbenton | reedip: pong | 09:54 |
| kevinbenton | jschwarz: was thinking about the ALLOCATING patch | 09:54 |
| jschwarz | kevinbenton, so I've discovered at least one race condition in the l3 agent where the same router is both created and removed in the same cycle | 09:54 |
| jschwarz | kevinbenton, so it processes the "create" event when down the line there's also "remove"... looking at the code to find a fix | 09:54 |
| jschwarz | kevinbenton, what about the ALLOCATING patch? | 09:55 |
| kevinbenton | jschwarz: even if we hide the router from the agent during create, we can't help the case during delete | 09:55 |
| *** efoley has quit IRC | 09:55 | |
| jschwarz | kevinbenton, I agree | 09:55 |
| kevinbenton | jschwarz: where the l3 agent learns there is a router, fetches it, and then tries to lookup its interface | 09:56 |
| kevinbenton | jschwarz: well there is one thing we can do | 09:56 |
| jschwarz | kevinbenton, I think that one is a different problem alltogether | 09:56 |
| kevinbenton | jschwarz: it's a different problem, but the solution for it would also like make the ALLOCATING patch unnecessary | 09:56 |
| kevinbenton | jschwarz: and that's to teach the l3 agent how to handle the missing interface | 09:57 |
| jschwarz | kevinbenton, how do you mean? | 09:57 |
| kevinbenton | jschwarz: if the l3 agent looks up an HA interface after it finds out about a router, that interface can be gone because of a server delete | 09:58 |
| kevinbenton | jschwarz: so the agent will always need to be robust to a missing interface as long as the interface retrieval is separate from the sync_router | 09:58 |
| *** mickeys has quit IRC | 09:58 | |
| *** vikram has joined #openstack-neutron | 09:59 | |
| jschwarz | kevinbenton, so you want the l3 agent to say "hey, I don't have an HA port here - I'm gonna ask the controller to create it for me"? | 10:00 |
| *** saggi has quit IRC | 10:01 | |
| *** _oanson has quit IRC | 10:01 | |
| kevinbenton | jschwarz: nah, it just needs to make sure it gracefully skips trying to process that router on the agent side | 10:01 |
| *** efoley_ is now known as efoley | 10:02 | |
| jschwarz | kevinbenton, that won't do for the case where there is an HA router but no HA network (because it was deleted) | 10:02 |
| jschwarz | kevinbenton, in other words, the server might need to actually re-create the HA network (or prevent it from being deleted) | 10:03 |
| kevinbenton | jschwarz: right, but that's a server-side race that needs to be fixed | 10:03 |
| jschwarz | kevinbenton, I think that once the server-side race is fixed, the l3 agent will fix itself (or will simply retry) and that will be enough | 10:04 |
| kevinbenton | jschwarz: i see. so the endless looping is just because of a missing network? | 10:05 |
| kevinbenton | jschwarz: i'm not familiar enough with the ha stuff on the agent side to know what causes that behavior | 10:05 |
| jschwarz | kevinbenton, I think it is. I'm looking at the code as we speak to see if I can track down the cause | 10:06 |
| *** chlong_ has joined #openstack-neutron | 10:06 | |
| jschwarz | kevinbenton, the gist of it is "the l3 agent tries to delete a router, throws an exception which causes it to try and delete it again" | 10:06 |
| kevinbenton | jschwarz: right, and i thought the delete exception was because of a missing port or something | 10:07 |
| jschwarz | kevinbenton, have a look at http://pastebin.com/9n9ExAjs | 10:08 |
| jschwarz | this is what loops for all times | 10:08 |
| jschwarz | kevinbenton, actually looking at it again it's nothing related to the neutron-server but something internal | 10:08 |
| jschwarz | kevinbenton, if the namespace can't be deleted because it doens't exist - job done. | 10:08 |
| jschwarz | kevinbenton, though the real issue is "how did it get to this state"? | 10:09 |
| kevinbenton | jschwarz: right | 10:09 |
| scheuran | kevinbenton, would you mind having a look at this patch https://review.openstack.org/275306? It's the macvtap stuff we briefly discussed at the last summit... | 10:10 |
| scheuran | kevinbenton, I'm not sure who's the approver for this, but maybe it's you... | 10:10 |
| *** dedery has joined #openstack-neutron | 10:11 | |
| openstackgerrit | Victor Stinner proposed openstack/neutron: Reuse to_utf8() and jsonutils.dump_as_bytes() https://review.openstack.org/280732 | 10:11 |
| kevinbenton | scheuran: i'll probably get to it tomorrow. working on this l3 ha stuff a bit and then heading to bed | 10:12 |
| scheuran | kevinbenton, sure, no problem - I'll add you to the review (if not yet done) thank you! | 10:12 |
| kevinbenton | scheuran: sounds good | 10:12 |
| scheuran | kevinbenton, have a good night :) | 10:12 |
| *** saggi has joined #openstack-neutron | 10:15 | |
| *** _oanson has joined #openstack-neutron | 10:15 | |
| *** EinstCrazy has quit IRC | 10:16 | |
| *** dedery has quit IRC | 10:16 | |
| *** tiagogomes has joined #openstack-neutron | 10:18 | |
| hdaniel | ihrachys: hi, have a sec re: https://review.openstack.org/#/c/250081/23/neutron/db/rbac_db_mixin.py@333 ? | 10:18 |
| tiagogomes | Hi, does anyone knows why sr-iov vnic types don't support security groups? | 10:19 |
| openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: RPC Callback rolling upgrades reporting, and integration https://review.openstack.org/268040 | 10:21 |
| *** ArchiFle1 has quit IRC | 10:21 | |
| scheuran | tiagogomes, sriov makes a passthrough of a pci virtual function to the instance. - there's no vswitch inbetween like ovs | 10:22 |
| ihrachys | tiagogomes: because sec groups are implemented on a linuxbridge attached between tap device passed into VM and integration bridge. and since you pass the device directly into VM, there are no bridges. | 10:22 |
| scheuran | tiagogomes, in other words, the sriov adapter is the "vswitch" - so if you want sec groups - you would have to implement it there... | 10:23 |
| scheuran | tiagogomes, seems like adapters do not have this kind of functionality... | 10:23 |
| *** ArchiFleKs has joined #openstack-neutron | 10:24 | |
| *** slaweq has joined #openstack-neutron | 10:24 | |
| slaweq | ihrachys: hello | 10:25 |
| slaweq | can You check once again https://review.openstack.org/#/c/248938/ | 10:25 |
| ihrachys | slaweq: will do, yes. | 10:25 |
| ihrachys | hdaniel: I will get to the patch, yes. | 10:25 |
| slaweq | on friday I made some small changes there | 10:25 |
| slaweq | thx a lot :) | 10:25 |
| hdaniel | ihrachys: thanks, since it looks like kevinbenton didn't appreciate the metaclass approach, and I wanted to discuss it with you, whenever possible | 10:26 |
| *** salv-orlando has joined #openstack-neutron | 10:26 | |
| *** ArchiFleKs has quit IRC | 10:26 | |
| *** ArchiFleKs has joined #openstack-neutron | 10:27 | |
| jschwarz | kevinbenton, btw, the agent is still getting stuff like "Unable to process HA router b1758bdd-3905-4c6b-a81f-2f491b80189b without HA port" | 10:27 |
| jschwarz | kevinbenton, but that seems likely because the router has already been deleted | 10:27 |
| jschwarz | ie. it's a symptom, not a cause | 10:27 |
| ihrachys | hdaniel: yeah. I think it's not a fundamental concern but more about how we track original methods. I think the proper way to handle multiple metaclasses case is to 'closure' original methods inside other methods, instead of maintaining a 'global' dict with original methods. | 10:27 |
| ihrachys | hdaniel: I need to look more closely to give more concrete suggestion. | 10:28 |
| hdaniel | ihrachys: great | 10:28 |
| *** yamamoto has quit IRC | 10:28 | |
| tiagogomes | thanks scheuran ihrachys | 10:28 |
| kevinbenton | hdaniel: +1 to what ihrachys said. if you can write it like a decorator where the reference is kept in a closure i would be okay with it | 10:28 |
| *** dedery has joined #openstack-neutron | 10:29 | |
| kevinbenton | jschwarz: right | 10:29 |
| kevinbenton | jschwarz: i'm working on a quick patch to make sure the ha network is always there | 10:30 |
| *** salv-orlando has quit IRC | 10:30 | |
| jschwarz | kevinbenton, awesome | 10:30 |
| davidsha | ihrachys: ping | 10:30 |
| ihrachys | davidsha: pong | 10:31 |
| davidsha | ihrachys: feeling better? | 10:31 |
| ihrachys | davidsha: kinda. if that's for review, yes, it's in queue | 10:31 |
| ihrachys | davidsha: may take a bit of time since I recover after not reading emails for several days. | 10:31 |
| davidsha | ihrachys:kk, just wondering about an odd error in the python34 test, I don't get this when I run the test locally so a bit confused: http://logs.openstack.org/91/267591/25/check/gate-neutron-python34-constraints/936bdfe/testr_results.html.gz | 10:33 |
| *** pradiprwt has joined #openstack-neutron | 10:34 | |
| *** jlibosva has joined #openstack-neutron | 10:35 | |
| pradiprwt | Hi everyone, I am trying to delete router using openstack api, but I am unable to delete can any one suggest me what are the methods I need to call for deleting a router. | 10:36 |
| openstackgerrit | Hynek Mlnarik proposed openstack/neutron: Squash initialization of OVS bridge flows https://review.openstack.org/282636 | 10:37 |
| ihrachys | davidsha: ok, that's because you still inherit from NeutronObject. | 10:39 |
| ihrachys | davidsha: please revert back to object | 10:39 |
| ihrachys | davidsha: since we don't want it to be versioned object anymort | 10:39 |
| ihrachys | *anymore | 10:39 |
| davidsha | ihrachys: ok, thanks I'll have the PS up in the next half an hour. | 10:40 |
| *** pradiprwt has quit IRC | 10:40 | |
| *** thorst has joined #openstack-neutron | 10:40 | |
| *** moshele has quit IRC | 10:41 | |
| *** elo has quit IRC | 10:41 | |
| *** pradiprwt has joined #openstack-neutron | 10:41 | |
| kevinbenton | jschwarz: you ready to test some code? | 10:42 |
| jschwarz | kevinbenton, I'm always ready, but I'm heading for lunch in a few minutes | 10:42 |
| kevinbenton | jschwarz: have time for a quick run before leaving? | 10:42 |
| jschwarz | kevinbenton, sure | 10:42 |
| kevinbenton | jschwarz: k, uploading now | 10:43 |
| *** nyechiel_ has joined #openstack-neutron | 10:43 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: make l3ha interface creation network delete safe https://review.openstack.org/283031 | 10:44 |
| kevinbenton | jschwarz: ^^ | 10:44 |
| kevinbenton | jschwarz: checkout that branch and it will have the previous fixes including the safe network one | 10:44 |
| jschwarz | kevinbenton, ack | 10:45 |
| davidsha | Quick question, when you use run_tests.sh it runs the unit tests, is there anyway to run the functional tests? | 10:46 |
| kevinbenton | davidsha: no, i don't think that has support for the other test envs | 10:47 |
| davidsha | kevinbenton: Ok, thanks. | 10:47 |
| kevinbenton | davidsha: to run functional tests you just have to do 'tox -efunctional' | 10:47 |
| *** thorst has quit IRC | 10:48 | |
| *** mgoddard has quit IRC | 10:48 | |
| *** irenab has joined #openstack-neutron | 10:48 | |
| davidsha | kevinbenton: Hmmm, just get a maximum recursion error... thanks anyways. | 10:49 |
| kevinbenton | davidsha: ah, i think that means one of your local tools needs to be updated, tox IIRC | 10:50 |
| jschwarz | kevinbenton, looks good | 10:50 |
| jschwarz | it still throws off the odd exception but no endless loops | 10:51 |
| kevinbenton | jschwarz: ack | 10:51 |
| kevinbenton | jschwarz: i'm going to cleanup this patch and add tests and then head to bed | 10:51 |
| jschwarz | kevinbenton, wait, I was too quick to speak | 10:51 |
| kevinbenton | jschwarz: leave any comments on there if you find issues | 10:51 |
| jschwarz | kevinbenton, one of the agents has reached an endless loop | 10:52 |
| kevinbenton | jschwarz: is it still the namespace missing issues? | 10:52 |
| *** nplanel has quit IRC | 10:52 | |
| jschwarz | kevinbenton, yeah. I think it's something local to the agent that's wrong | 10:52 |
| kevinbenton | jschwarz: i agree | 10:52 |
| kevinbenton | jschwarz: somewhere is assuming the namespace exists incorrectly | 10:52 |
| kevinbenton | jschwarz: can you send me the trace? | 10:52 |
| jschwarz | kevinbenton, sure | 10:53 |
| kevinbenton | jschwarz: and then go to lunch! | 10:53 |
| davidsha | kevinbenton: tox IIRC? | 10:54 |
| kevinbenton | davidsha: i think 'tox' is the tool that needs to be updated that was leading to that recursion error if i recall correctly | 10:54 |
| *** nyechiel_ has quit IRC | 10:55 | |
| davidsha | kevinbenton: Ya I'm seeing a bug filed for it about a month ago, thanks. | 10:55 |
| kevinbenton | davidsha: no prob | 10:55 |
| jschwarz | kevinbenton, there ya' go: http://pastebin.com/9af7fnyX | 10:55 |
| *** claudiub has quit IRC | 10:55 | |
| jschwarz | the first exception happens only once and the second one then loops until I restart the process | 10:56 |
| *** wenjianhn has quit IRC | 10:56 | |
| *** nyechiel_ has joined #openstack-neutron | 10:56 | |
| jschwarz | kevinbenton, I'm in the process of surrounding the keepalived_manager in a property to see where it's being nullified from... once I get somewhere I'll propose new patchsets where relevant in your patches | 10:57 |
| jschwarz | kevinbenton, nighty nighty | 10:57 |
| kevinbenton | jschwarz: thx ttyl | 10:57 |
| *** subscope has quit IRC | 10:57 | |
| *** rossella_s has quit IRC | 10:58 | |
| *** shwetaap has joined #openstack-neutron | 10:58 | |
| *** rossella_s has joined #openstack-neutron | 10:58 | |
| *** moshele has joined #openstack-neutron | 11:00 | |
| *** ijw has joined #openstack-neutron | 11:02 | |
| *** shwetaap has quit IRC | 11:02 | |
| openstackgerrit | Victor Stinner proposed openstack/neutron: Reuse to_utf8() and jsonutils.dump_as_bytes() https://review.openstack.org/280732 | 11:04 |
| *** hanchao has quit IRC | 11:05 | |
| *** nyechiel_ has quit IRC | 11:05 | |
| *** sdague has joined #openstack-neutron | 11:06 | |
| vikram | kevinbenton: ping | 11:06 |
| *** mgoddard has joined #openstack-neutron | 11:08 | |
| *** yamamoto has joined #openstack-neutron | 11:10 | |
| *** neiljerram has quit IRC | 11:11 | |
| rossella_s | ihrachys, ajo maybe you know...how can I get an url to share the logstash query? | 11:12 |
| *** claudiub has joined #openstack-neutron | 11:12 | |
| ihrachys | rossella_s: I knew it before they upgrades the installation :-| | 11:12 |
| ihrachys | rossella_s: now I don't | 11:12 |
| *** markvoelker has joined #openstack-neutron | 11:12 | |
| ihrachys | *upgraded | 11:12 |
| rossella_s | ihrachys, same for me, it's frustrating | 11:12 |
| *** yamamoto has quit IRC | 11:14 | |
| *** stanzgy has quit IRC | 11:15 | |
| *** tfukushima has quit IRC | 11:16 | |
| *** changzhi has quit IRC | 11:16 | |
| *** markvoelker has quit IRC | 11:17 | |
| *** annp has quit IRC | 11:20 | |
| *** djan_ has quit IRC | 11:22 | |
| vikram | ihrachys: ping | 11:22 |
| vikram | ihrachys: do we have the support for dhcp relay in neutron? | 11:23 |
| *** rotbeard has joined #openstack-neutron | 11:24 | |
| *** subscope has joined #openstack-neutron | 11:30 | |
| *** neiljerram has joined #openstack-neutron | 11:31 | |
| *** salv-orlando has joined #openstack-neutron | 11:33 | |
| *** jaypipes has joined #openstack-neutron | 11:36 | |
| *** tpsilva has joined #openstack-neutron | 11:37 | |
| *** gongysh has joined #openstack-neutron | 11:39 | |
| *** mrvinti has quit IRC | 11:39 | |
| *** mgagne has quit IRC | 11:39 | |
| *** electrocucaracha has quit IRC | 11:39 | |
| *** wasmum has quit IRC | 11:39 | |
| *** ujjain has quit IRC | 11:39 | |
| *** ujjain has joined #openstack-neutron | 11:39 | |
| *** ujjain has quit IRC | 11:39 | |
| *** ujjain has joined #openstack-neutron | 11:39 | |
| *** insequent has quit IRC | 11:40 | |
| *** ryanpetrello has quit IRC | 11:40 | |
| *** rackertom has quit IRC | 11:40 | |
| *** zhenguo_ has quit IRC | 11:40 | |
| *** mrvinti has joined #openstack-neutron | 11:40 | |
| *** raorn has quit IRC | 11:40 | |
| *** saisriki has quit IRC | 11:40 | |
| *** krtaylor has quit IRC | 11:40 | |
| *** zigo has quit IRC | 11:40 | |
| *** rstarr has quit IRC | 11:40 | |
| *** clev has quit IRC | 11:40 | |
| *** carl_baldwin has quit IRC | 11:40 | |
| *** zacksh has quit IRC | 11:40 | |
| *** dhellmann has quit IRC | 11:41 | |
| *** zigo has joined #openstack-neutron | 11:41 | |
| *** zacksh has joined #openstack-neutron | 11:41 | |
| *** raorn has joined #openstack-neutron | 11:42 | |
| *** salv-orlando has quit IRC | 11:43 | |
| *** zhenguo_ has joined #openstack-neutron | 11:43 | |
| *** rackertom has joined #openstack-neutron | 11:43 | |
| *** carl_baldwin has joined #openstack-neutron | 11:44 | |
| *** subscope has quit IRC | 11:45 | |
| *** saisriki has joined #openstack-neutron | 11:45 | |
| *** subscope has joined #openstack-neutron | 11:45 | |
| *** mgagne has joined #openstack-neutron | 11:45 | |
| *** thorst has joined #openstack-neutron | 11:45 | |
| *** ryanpetrello has joined #openstack-neutron | 11:46 | |
| *** clev has joined #openstack-neutron | 11:46 | |
| *** mgagne is now known as Guest51435 | 11:46 | |
| *** wasmum has joined #openstack-neutron | 11:46 | |
| *** dhellmann has joined #openstack-neutron | 11:46 | |
| *** Bofu2U has joined #openstack-neutron | 11:46 | |
| openstackgerrit | Artur Korzeniewski proposed openstack/neutron: WIP Introducing the Subnet OVO https://review.openstack.org/264273 | 11:46 |
| openstackgerrit | Artur Korzeniewski proposed openstack/neutron: Objects DB api: added composite key to handle multiple primary key https://review.openstack.org/275790 | 11:46 |
| openstackgerrit | Artur Korzeniewski proposed openstack/neutron: Create a hook in base object to modify the fields before DB operations https://review.openstack.org/281850 | 11:46 |
| openstackgerrit | David Shaughnessy proposed openstack/neutron: Added agent specific API support to L2 extensions https://review.openstack.org/267591 | 11:51 |
| *** thorst has quit IRC | 11:53 | |
| *** sudipto__ has quit IRC | 11:53 | |
| *** krtaylor has joined #openstack-neutron | 11:53 | |
| *** links has quit IRC | 11:53 | |
| *** afaranha has joined #openstack-neutron | 11:53 | |
| *** insequent has joined #openstack-neutron | 11:54 | |
| *** bharathk has quit IRC | 11:55 | |
| *** shwetaap has joined #openstack-neutron | 11:58 | |
| *** efoley has quit IRC | 11:58 | |
| *** javeriak has quit IRC | 12:00 | |
| *** itzikb_ has quit IRC | 12:01 | |
| *** shwetaap has quit IRC | 12:02 | |
| *** javeriak has joined #openstack-neutron | 12:04 | |
| ihrachys | vikram: not that I saw. | 12:05 |
| vikram | ihrachys: I found a BP about it and got some doubt | 12:05 |
| vikram | ihrachys: Thanks for your help | 12:06 |
| vikram | ihrachys: https://blueprints.launchpad.net/neutron/+spec/dhcp-relay | 12:06 |
| *** links has joined #openstack-neutron | 12:07 | |
| *** baoli has joined #openstack-neutron | 12:08 | |
| *** itzikb has joined #openstack-neutron | 12:08 | |
| *** baoli has quit IRC | 12:10 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Make L3 HA interface creation concurrency safe https://review.openstack.org/282876 | 12:15 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Issue retry on router db reference error https://review.openstack.org/283031 | 12:15 |
| korzen | Hi rossella_s, ihrachys, do you have any time to look at: https://review.openstack.org/#/c/275790/ ? | 12:16 |
| rossella_s | korzen, I will try to look at it before the meeting | 12:16 |
| *** yfried has quit IRC | 12:17 | |
| *** chlong_ has quit IRC | 12:17 | |
| *** hynekm has quit IRC | 12:17 | |
| jschwarz | kevinbenton, any news? | 12:18 |
| * jaypipes thinks this is the 6th time he has read russellb's review on the Tacker governance patch... | 12:22 | |
| jaypipes | kevinbenton: what are you doing up? :) | 12:22 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: router may not have keepalive during cleanup https://review.openstack.org/283057 | 12:23 |
| kevinbenton | jschwarz: give that one a try | 12:23 |
| kevinbenton | jaypipes: getting ready for bed! :) | 12:23 |
| jaypipes | kevinbenton: figured as much :) | 12:23 |
| kevinbenton | jaypipes: i like to code on l3 ha to get sleepy ;) | 12:23 |
| jschwarz | kevinbenton, in a few - running some traceback-including runs to get some more info about the race condition | 12:23 |
| kevinbenton | jschwarz: it seems that the l3 cleanup code always assumes the presense of the namespace | 12:24 |
| jschwarz | kevinbenton, aye, and that can just be surrounded with a try-except | 12:24 |
| jschwarz | kevinbenton, i'm interested in why it gets to the first exception in the creation | 12:24 |
| kevinbenton | jschwarz: yeah, that patch i just pushed up was specific to that first exception | 12:25 |
| kevinbenton | jschwarz: the NoneType one | 12:25 |
| jschwarz | kevinbenton, this: https://review.openstack.org/#/c/282876/5/neutron/db/l3_hamode_db.py ? | 12:25 |
| kevinbenton | jschwarz: https://review.openstack.org/283057 | 12:25 |
| jschwarz | kevinbenton, ah I just saw this | 12:25 |
| kevinbenton | jschwarz: the router object is not initialized with a keepalive manager | 12:26 |
| jschwarz | kevinbenton, it's always supposed to be initialized by one - it's being deleted concurrently by something else | 12:26 |
| jschwarz | it's pretty straightforward, the keepalived_manager was initialized a few function calls back | 12:27 |
| kevinbenton | jschwarz: but if you hit that 'if not ha_port' thing the keepalived_manager will be None | 12:27 |
| openstackgerrit | vikram.choudhary proposed openstack/neutron: BGP Dynamic Routing: introduce BgpDriver https://review.openstack.org/207635 | 12:27 |
| kevinbenton | jschwarz: do you see the "Unable to process HA router" message before that? | 12:27 |
| jschwarz | kevinbenton, what 'if not ha_port'? | 12:27 |
| jschwarz | kevinbenton, yep | 12:28 |
| *** rotbeard has quit IRC | 12:28 | |
| kevinbenton | jschwarz: if you see that log, it means the object was initialized without setting up the keepalive manager | 12:28 |
| jschwarz | kevinbenton, look here: https://github.com/openstack/neutron/blob/master/neutron/agent/l3/agent.py#L346 | 12:29 |
| jschwarz | this calls https://github.com/openstack/neutron/blob/master/neutron/agent/l3/ha_router.py#L86 which initialzies the keepalived | 12:29 |
| *** davidsha has quit IRC | 12:30 | |
| jschwarz | kevinbenton, then later on in _router_added's call to process_router_add, it complains that no keepalived_manager is found (NoneType) | 12:30 |
| jschwarz | so it's deleted concurrently | 12:30 |
| kevinbenton | jschwarz: no | 12:30 |
| kevinbenton | jschwarz: look here: https://github.com/openstack/neutron/blob/master/neutron/agent/l3/ha_router.py#L77-L98 | 12:30 |
| kevinbenton | jschwarz: _init_keepalived_manager is what sets self.keepalived_manager | 12:31 |
| jschwarz | kevinbenton, ah. I missed that | 12:31 |
| jschwarz | that 'if not ha_port' check | 12:31 |
| kevinbenton | right | 12:31 |
| jschwarz | right, I'll give it a go then :) | 12:31 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: raise on bad l3 ha router https://review.openstack.org/283060 | 12:33 |
| kevinbenton | jschwarz: actually can you try this patch instead ^^ | 12:33 |
| jschwarz | kevinbenton, I just started a run :< | 12:33 |
| jschwarz | kevinbenton, why the raise? | 12:33 |
| kevinbenton | jschwarz: hmmm. nevermind, raise not be a good idea | 12:34 |
| *** rotbeard has joined #openstack-neutron | 12:34 | |
| kevinbenton | jschwarz: i was thinking maybe it would be better to completely bail | 12:34 |
| jschwarz | I think that making sure the keepalived_manager is there will mean that it will create the namespace correctly, and then the remove will not fail | 12:34 |
| openstackgerrit | vikram.choudhary proposed openstack/neutron: BGP Dynamic Routing: introduce BgpDriver https://review.openstack.org/207635 | 12:34 |
| jschwarz | kevinbenton, nah, it still fails but on something else | 12:35 |
| jschwarz | kevinbenton, I can go from here if you want to go to bed | 12:35 |
| kevinbenton | jschwarz: but we can't create the keepalived manager without an ha_port, right? | 12:36 |
| jschwarz | kevinbenton, right | 12:36 |
| jschwarz | kevinbenton, so not it doesn't fail on the keepalived manager but on something else | 12:36 |
| jschwarz | kevinbenton, actually it's still on the keepalived manager but on something else | 12:37 |
| *** vikram has quit IRC | 12:37 | |
| jschwarz | (some other piece of code) | 12:37 |
| jschwarz | gah, pastebin will talk much better than I | 12:37 |
| kevinbenton | jschwarz: ok | 12:37 |
| jschwarz | kevinbenton, http://pastebin.com/tJAe8dup | 12:37 |
| jschwarz | kevinbenton, if we keep this up, we'll just wrap all the calls to keepalived manager with some 'if self.keepalived_manager and ...' calls | 12:38 |
| *** ijw has quit IRC | 12:39 | |
| kevinbenton | jschwarz: yeah, that's why i was thinking of just throwing the exception completely | 12:39 |
| jschwarz | kevinbenton, aye... so | 12:39 |
| kevinbenton | jschwarz: if we convince the l3 agent the router is broken, what will it do though | 12:39 |
| jschwarz | kevinbenton, so lets raise an exception and up there we can just re-add the router to the queue | 12:39 |
| jschwarz | kevinbenton, actually that's a bad idea | 12:40 |
| *** salv-orlando has joined #openstack-neutron | 12:40 | |
| jschwarz | kevinbenton, the whole 'creation failed' thing should lead to a 'check that the router is really there' request | 12:40 |
| jschwarz | kevinbenton, then, depending on the result, the agent can either re-create it or delete it if it doesn't exist anymore | 12:40 |
| kevinbenton | jschwarz: right, that's exactly what we want | 12:41 |
| *** yamamoto has joined #openstack-neutron | 12:41 | |
| kevinbenton | jschwarz: because the router no longer exists, right? | 12:41 |
| jschwarz | kevinbenton, yes, OR it might exist but the HA network/ports have not been created *yet* | 12:41 |
| jschwarz | kevinbenton, but if they didn't exist yet, the agent shouldn't see the router anyway | 12:42 |
| jschwarz | kevinbenton, because of the ALLOCATING business | 12:42 |
| kevinbenton | jschwarz: right | 12:42 |
| kevinbenton | jschwarz: i'm actually confused about why this goes into an infinite loop now | 12:42 |
| *** efoley has joined #openstack-neutron | 12:42 | |
| *** nmagnezi has quit IRC | 12:42 | |
| jschwarz | kevinbenton, it's still the whole "the namespace wasn't created, can't delete it" from before | 12:43 |
| kevinbenton | jschwarz: ah, i see the infinite loop was happening on the delete | 12:43 |
| kevinbenton | jschwarz: a little different from this issue | 12:43 |
| jschwarz | yes | 12:43 |
| openstackgerrit | Merged openstack/os-vif: Use names() method of ExtensionManager insted of keys() https://review.openstack.org/282345 | 12:43 |
| *** markvoelker has joined #openstack-neutron | 12:43 | |
| jschwarz | kevinbenton, so a naive solution will be to make sure that the namespace exists before we try to delete it, and this will solve the infinite loop | 12:43 |
| kevinbenton | jschwarz: yeah, i'm looking at the code right now to see what else it tries to do | 12:44 |
| jschwarz | kevinbenton, the question is whether or not we want the naive solution, or a smart solution where the agent queries the actual state of the router | 12:44 |
| jschwarz | ? | 12:44 |
| *** gsagie has joined #openstack-neutron | 12:44 | |
| *** salv-orlando has quit IRC | 12:44 | |
| *** thorst has joined #openstack-neutron | 12:44 | |
| *** rtheis has joined #openstack-neutron | 12:44 | |
| kevinbenton | jschwarz: well in this case the agent knows the correct state of the router, it's deleted | 12:44 |
| jschwarz | kevinbenton, yeah.. one could claim that since the ALLOCATING stuff the agent shouldn't even be processing the router until the ports were created | 12:45 |
| jschwarz | kevinbenton, so it's just the case of create-delete | 12:45 |
| kevinbenton | jschwarz: right | 12:46 |
| *** claudiub has quit IRC | 12:47 | |
| *** nyechiel_ has joined #openstack-neutron | 12:47 | |
| *** markvoelker has quit IRC | 12:47 | |
| jschwarz | kevinbenton, https://github.com/openstack/neutron/blob/master/neutron/agent/l3/router_info.py#L404 | 12:49 |
| jschwarz | kevinbenton, "if not ip_wrapper.exists(): return []" | 12:50 |
| jschwarz | that should suffice IMO | 12:50 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Make L3 HA interface creation concurrency safe https://review.openstack.org/282876 | 12:51 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Add ALLOCATING state to routers https://review.openstack.org/257059 | 12:51 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Make router cleanup safer with premature router https://review.openstack.org/283057 | 12:51 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Issue retry on router db reference error https://review.openstack.org/283031 | 12:51 |
| jaypipes | kevinbenton: what happened to sleeping? :) | 12:52 |
| *** tmorin has quit IRC | 12:52 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Make router cleanup safer with premature router https://review.openstack.org/283057 | 12:53 |
| kevinbenton | jschwarz: that's what i put in that patch ^^ | 12:53 |
| kevinbenton | jaypipes: soon :) | 12:53 |
| jschwarz | kevinbenton, the last one? | 12:54 |
| kevinbenton | jschwarz: yeah: https://review.openstack.org/#/c/283057/3/neutron/agent/l3/router_info.py | 12:55 |
| jschwarz | kevinbenton, yep. I just wrote something very similar | 12:55 |
| jschwarz | kevinbenton, it's running as we speak | 12:55 |
| *** tmorin has joined #openstack-neutron | 12:55 | |
| kevinbenton | jschwarz: cool | 12:55 |
| jschwarz | kevinbenton, it's failing on something else now | 12:56 |
| jschwarz | kevinbenton, http://pastebin.com/T7khrRu5 | 12:57 |
| *** nmagnezi has joined #openstack-neutron | 12:57 | |
| *** jlibosva has quit IRC | 12:57 | |
| *** gvrangan has joined #openstack-neutron | 12:58 | |
| *** yfried has joined #openstack-neutron | 12:58 | |
| *** shwetaap has joined #openstack-neutron | 12:58 | |
| kevinbenton | jschwarz: getting closer | 12:58 |
| *** jlibosva has joined #openstack-neutron | 12:59 | |
| jschwarz | kevinbenton, I'm thinking we should get at the cause and not treat the symptoms | 12:59 |
| *** changzhi has joined #openstack-neutron | 13:00 | |
| *** neeti has quit IRC | 13:01 | |
| jschwarz | kevinbenton, I think that if we teach the agent to treat "Unable to process HA router bb1843f9-d870-4a9e-910c-f27790ddef77 without HA port" properly, everything else will be fixed | 13:02 |
| jschwarz | kevinbenton, because it all always starts there | 13:02 |
| jschwarz | everything else is just hotfixes | 13:02 |
| kevinbenton | jschwarz: yes | 13:03 |
| *** shwetaap has quit IRC | 13:03 | |
| *** NamNH has quit IRC | 13:03 | |
| jschwarz | kevinbenton, so raise an error and teach the l3 agent to re-discover if the router exists or not? | 13:04 |
| kevinbenton | jschwarz: nah, it's a little trickier. the issue is that the agent knows the router doesn't exist, that's why it's trying to delete it | 13:05 |
| jschwarz | kevinbenton, I think it's trying to delete it because it has received the 'router deleted' event from the server | 13:05 |
| jschwarz | it never deletes something on its own accord | 13:06 |
| *** davidsha has joined #openstack-neutron | 13:06 | |
| kevinbenton | jschwarz: yes, that's true | 13:06 |
| kevinbenton | jschwarz: what i'm saying is that it is in the correct state WRT what happened on the server | 13:06 |
| jschwarz | kevinbenton, yes. | 13:06 |
| *** amuller has joined #openstack-neutron | 13:06 | |
| jschwarz | kevinbenton, so lets change the tactic. we can assume that if the agent sees the router it's because it was created and it received an event from the controller about it | 13:07 |
| *** smalleni has quit IRC | 13:07 | |
| jschwarz | kevinbenton, if the router no longer exists (ie. ha_port == None), then it MUST have been deleted concurrently (because the port is created before the event is even sent) | 13:07 |
| jschwarz | kevinbenton, so raise the exception and stop the 'creation' | 13:08 |
| *** links has quit IRC | 13:08 | |
| *** amotoki has joined #openstack-neutron | 13:08 | |
| jschwarz | kevinbenton, ah, and that leads us to 'graceful cleanup' which you said you wanted to do | 13:08 |
| *** bee2502_ has joined #openstack-neutron | 13:09 | |
| jschwarz | kevinbenton, righto. I'll catch the exceptions and see where that lands us | 13:09 |
| *** yamamoto has quit IRC | 13:10 | |
| bee2502_ | Hi, I am Bhagyashree(Bee). I am interested in the Metering Agent project for Outreachy. I was hoping someone could help me with getting started :) | 13:10 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Make router cleanup safer with premature router https://review.openstack.org/283057 | 13:11 |
| kevinbenton | jschwarz: can you try out ^^ | 13:11 |
| jschwarz | kevinbenton, sure | 13:11 |
| kevinbenton | jschwarz: i think the issue is that the agent hasn't had a chance to create everything | 13:11 |
| kevinbenton | jschwarz: it has no namespace for the router, which tells me that it doesn't exist yet | 13:12 |
| jschwarz | kevinbenton, aye - the creation failed midway because the haport disappeared | 13:12 |
| bee2502_ | rosella_s : ^^^^^^ | 13:12 |
| kevinbenton | jschwarz: wait, have you confirmed that's the cause? | 13:14 |
| kevinbenton | jschwarz: do you see a portnotfound trace or something? | 13:14 |
| jschwarz | kevinbenton, we wrapped the portnotfound yesterday | 13:14 |
| jschwarz | so not anymore | 13:14 |
| jschwarz | kevinbenton, but the creation of the namespaces, etc - happen after the keepalived_manager's failed initializaiton IMO | 13:14 |
| *** bdemers has quit IRC | 13:15 | |
| *** bdemers has joined #openstack-neutron | 13:16 | |
| kevinbenton | jschwarz: where does the agent make a separate call to retrieve these interfaces? | 13:16 |
| jschwarz | kevinbenton, let me look | 13:16 |
| jschwarz | kevinbenton, so I assume you've decided to skip sleep altogether? ;-) | 13:16 |
| *** gongysh has quit IRC | 13:16 | |
| *** yamamoto has joined #openstack-neutron | 13:16 | |
| kevinbenton | jschwarz: not completely :) | 13:16 |
| jschwarz | kevinbenton, from the sync_routers I believe | 13:17 |
| *** amotoki has quit IRC | 13:18 | |
| kevinbenton | jschwarz: wait, if it's part of sync_routers, then we should be able to prevent the agent from ever getting a router without an interface | 13:19 |
| *** javeriak has quit IRC | 13:19 | |
| jschwarz | kevinbenton, we are though - ALLOCATING is what it's all about | 13:19 |
| jschwarz | kevinbenton, I might be mistaken then | 13:19 |
| kevinbenton | jschwarz: then how does the agent have a router without interface info? | 13:19 |
| jschwarz | kevinbenton, your latest patchset still ends up looping | 13:20 |
| kevinbenton | jschwarz: right, it seems somehow the agent is getting a router without an interface | 13:20 |
| kevinbenton | jschwarz: i think i need to revisit the patch to set the status back to ALLOCATING before a delete | 13:21 |
| kevinbenton | jschwarz: that way we can be sure that sync_routers will exclude a router that is about to have its interfaces removed | 13:21 |
| *** links has joined #openstack-neutron | 13:22 | |
| kevinbenton | jschwarz: hmmm. i need to see the order that interfaces are collected. maybe sync_routers does the db lookup of interfaces after a delay that opens up this window | 13:22 |
| jschwarz | kevinbenton, I'm not sure that will fix it | 13:22 |
| *** numans has quit IRC | 13:23 | |
| jschwarz | kevinbenton, I think that the problem is that when the agent queries the controller for the router (that was deleted), the controller returns some data that indicates that it was removed, but the agent doesn't have a check for that indication | 13:23 |
| jschwarz | marking the router as 'ALLOCATING' when it's deleting won't stop the router from being deleted - we just need to teach the agent to see that it's going to be deleted | 13:24 |
| kevinbenton | jschwarz: the agent will determine a router is deleted if it's not returned in sync_routers | 13:24 |
| jschwarz | kevinbenton, exactly. so why is it returned? | 13:24 |
| kevinbenton | jschwarz: it's not, the issue is that the agent somehow has a copy of the router already | 13:25 |
| kevinbenton | jschwarz: in a broken state | 13:25 |
| jschwarz | kevinbenton, agreed. so lets look at the sync_routers call and see what gives. | 13:25 |
| jschwarz | :) | 13:25 |
| *** rotbeard has quit IRC | 13:27 | |
| *** markvoelker has joined #openstack-neutron | 13:29 | |
| *** yamamoto has quit IRC | 13:33 | |
| *** jlibosva has quit IRC | 13:33 | |
| *** jlibosva has joined #openstack-neutron | 13:34 | |
| *** ijw has joined #openstack-neutron | 13:34 | |
| *** ParsectiX has joined #openstack-neutron | 13:34 | |
| *** edmondsw has joined #openstack-neutron | 13:35 | |
| *** mkolesni_ has quit IRC | 13:35 | |
| openstackgerrit | Moshe Levi proposed openstack/os-vif: VIFDirect: replace dev_name with dev_address https://review.openstack.org/283081 | 13:36 |
| *** javeriak has joined #openstack-neutron | 13:37 | |
| jschwarz | kevinbenton, so sync_routers returns an actual router for the cases where it ends up in an endless loop | 13:38 |
| jschwarz | ... | 13:38 |
| kevinbenton | jschwarz: but does it have HA_INTERFACE_KEY set? | 13:38 |
| jschwarz | kevinbenton, nope | 13:39 |
| *** mgoddard_ has joined #openstack-neutron | 13:39 | |
| kevinbenton | jschwarz: ok, working on a fix for that right now | 13:39 |
| jschwarz | kevinbenton, the status is ACTIVE but no keys with HA | 13:39 |
| *** ijw has quit IRC | 13:39 | |
| kevinbenton | jschwarz: what's happening is that the interface is deleted when get_sync_data gets the router but before it gets the interfaces | 13:39 |
| *** claudiub has joined #openstack-neutron | 13:40 | |
| *** dedery has quit IRC | 13:40 | |
| kevinbenton | jschwarz: or we are missing some case where the router is created but doesn't get its interface and goes to ACTIVE anyway | 13:41 |
| *** mgoddard has quit IRC | 13:42 | |
| *** jckasper has joined #openstack-neutron | 13:43 | |
| *** javeriak has quit IRC | 13:44 | |
| jschwarz | kevinbenton, so the get_sync_data is called concurrently to the deletion of the router on the server side? | 13:44 |
| jschwarz | kevinbenton, when the server started deleting the router but didn't reach the ha network yet? | 13:44 |
| *** brad_behle has joined #openstack-neutron | 13:44 | |
| *** salv-orlando has joined #openstack-neutron | 13:45 | |
| kevinbenton | jschwarz: i was thinking the get_sync_data is working on collecting the info, so it has retrieved the router, but not the interface info | 13:45 |
| *** jpena is now known as jpena|lunch | 13:46 | |
| kevinbenton | jschwarz: and the router gets deleted between when it has retrieved the router and that interface info | 13:46 |
| *** ildikov_ has joined #openstack-neutron | 13:46 | |
| jschwarz | kevinbenton, there really should be a per-router lock on the server side | 13:47 |
| kevinbenton | jschwarz: unfortunately we can't lock between servers | 13:47 |
| kevinbenton | jschwarz: otherwise yeah, that would be helpful | 13:47 |
| jschwarz | kevinbenton, mmap? :) | 13:47 |
| jschwarz | there are ways to have a multi-processes lock | 13:48 |
| jschwarz | kevinbenton, anyway if the end-case is what we think it is, setting ALLOCATING on removal won't solve the problem | 13:49 |
| *** ParsectiX has quit IRC | 13:49 | |
| jschwarz | kevinbenton, since if get_sync_data starts before the ALLOCATING was set, we end up in the same place | 13:49 |
| *** vishwanathj has joined #openstack-neutron | 13:50 | |
| *** links has quit IRC | 13:50 | |
| *** ildikov has quit IRC | 13:50 | |
| *** yfried has quit IRC | 13:51 | |
| kevinbenton | jschwarz: it's a lock across servers | 13:51 |
| kevinbenton | jschwarz: not threads | 13:51 |
| kevinbenton | jschwarz: it requires distributed coordination algorithm or service that provides it | 13:51 |
| jschwarz | kevinbenton, aye you're right | 13:52 |
| jschwarz | kevinbenton, still there are ways to do that as well | 13:52 |
| *** salv-orlando has quit IRC | 13:52 | |
| kevinbenton | jschwarz: yes, i'm working on moving the ALLOCATING filtering logic | 13:52 |
| jschwarz | kevinbenton, to where? | 13:52 |
| kevinbenton | jschwarz: to after all of the interface info and whatnot is collected | 13:52 |
| openstackgerrit | Evgeny Fedoruk proposed openstack/python-neutronclient: L7 capability implementation for lbaas v2 https://review.openstack.org/217276 | 13:53 |
| *** fzdarsky has quit IRC | 13:55 | |
| *** numans has joined #openstack-neutron | 13:55 | |
| *** korzen_ has joined #openstack-neutron | 13:56 | |
| *** neelashah has joined #openstack-neutron | 13:56 | |
| scheuran | Hi garyk, quick question regarding your review comment here: https://review.openstack.org/#/c/275306/9/neutron/plugins/ml2/drivers/macvtap/agent/config.py | 13:56 |
| *** electrocucaracha has joined #openstack-neutron | 13:56 | |
| scheuran | garyk, it's about if the the review sections are in lower or upper case | 13:56 |
| *** subscope has quit IRC | 13:56 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Make L3 HA interface creation concurrency safe https://review.openstack.org/282876 | 13:56 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Add ALLOCATING state to routers https://review.openstack.org/257059 | 13:56 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Make router cleanup safer with premature router https://review.openstack.org/283057 | 13:56 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Issue retry on router db reference error https://review.openstack.org/283031 | 13:56 |
| kevinbenton | jschwarz: ok, can you test from this patch? https://review.openstack.org/283031 | 13:57 |
| jschwarz | kevinbenton, ack | 13:57 |
| scheuran | garyk, I had a look at the lb agent and the ovs agent, both have their section in upper case (e.g. OVS, LINUXBRIDGE) | 13:57 |
| *** changzhi has quit IRC | 13:57 | |
| *** korzen has quit IRC | 13:57 | |
| scheuran | garyk, not sure what's the right way to go now... | 13:57 |
| openstackgerrit | Moshe Levi proposed openstack/os-vif: Change network vlan to integer https://review.openstack.org/283087 | 13:57 |
| jschwarz | kevinbenton, not on this: https://review.openstack.org/#/c/283057 ? | 13:57 |
| garyk | scheuran: ji prefer lower case if possible. just causes the OVS and LB are upper does not mean it correct | 13:57 |
| kevinbenton | jschwarz: nah, none of that one should be necessary if we stop this from getting to the agent | 13:58 |
| kevinbenton | jschwarz: if my understanding of the issue is correct now | 13:58 |
| jschwarz | kevinbenton, okies | 13:58 |
| *** amotoki has joined #openstack-neutron | 13:58 | |
| scheuran | garyk, ok - I don't have a problem with it - will change | 13:58 |
| scheuran | garyk, something else, is it possible to have a docimpact flag with a multiline description? | 13:58 |
| kevinbenton | jschwarz: btw, what are you running that makes this easy to reproduce. i was using a bash script to create and destroy routers over and over but didn't have any luck | 13:58 |
| *** shwetaap has joined #openstack-neutron | 13:58 | |
| jschwarz | kevinbenton, rally ;-p | 13:59 |
| *** ranjithd has joined #openstack-neutron | 13:59 | |
| kevinbenton | jschwarz: is that all rally is doing? or is it doing a mix of other user ports too? | 13:59 |
| garyk | scheuran: yes, sure you can have that | 13:59 |
| jschwarz | kevinbenton, all rally, a specific test | 13:59 |
| scheuran | garyk, just indent 4 spaces? | 14:00 |
| jschwarz | kevinbenton, this is the script i'm using: http://pastebin.com/QgJz5G40 | 14:00 |
| garyk | scheuran: sotty you have lost me | 14:00 |
| garyk | what do you mean 4 spaces | 14:00 |
| kevinbenton | jschwarz: ah, i suspect it's the concurrency that i'm missing | 14:00 |
| kevinbenton | jschwarz: i was only doing two concurrent things | 14:00 |
| kevinbenton | jschwarz: not 10 | 14:00 |
| scheuran | garyk, like you do in python if something does not fit into one line :) | 14:01 |
| jschwarz | kevinbenton, still no luck | 14:01 |
| jschwarz | kevinbenton, :< | 14:01 |
| garyk | scheuran: then you can use () | 14:01 |
| kevinbenton | jschwarz: so do you see a sync_routers sending an ha router without an HA interface somewhere? | 14:01 |
| scheuran | garyk, just to make sure, I'm talking about the doc impact flag in the commit message | 14:01 |
| *** fzdarsky has joined #openstack-neutron | 14:01 | |
| jschwarz | kevinbenton, I'll put the trace back in and run it again, sec | 14:01 |
| scheuran | garyk, so its's: DocImpact (blalbalba \n bla bla) ? | 14:02 |
| garyk | scheuran: for the DocImpact just write under that you have introduced a configuration varaible and the name. | 14:02 |
| garyk | you do not need all of the values | 14:02 |
| garyk | scheuran: DocImpact\n | 14:02 |
| garyk | Bla bla\n | 14:03 |
| *** dshaw has joined #openstack-neutron | 14:03 | |
| scheuran | garyk, ok got it! | 14:03 |
| scheuran | garyk, thank you! | 14:03 |
| *** shwetaap has quit IRC | 14:03 | |
| *** chlong_ has joined #openstack-neutron | 14:06 | |
| *** jistr has quit IRC | 14:06 | |
| garyk | scheuran: ok. | 14:06 |
| *** afaranha has quit IRC | 14:07 | |
| *** localloop127 has joined #openstack-neutron | 14:07 | |
| *** jistr has joined #openstack-neutron | 14:07 | |
| jschwarz | kevinbenton, keys for sync_router for deb5d0e1-c6fe-465a-925f-9b39b60a1dab: [u'status', u'external_gateway_info', u'availability_zone_hints', u'availability_zones', u'ha_vr_id', u'name', u'gw_port_id', u'admin_state_up', u'tenant_id', u'distributed', u'routes', u'ha', u'id'] | 14:09 |
| *** subscope has joined #openstack-neutron | 14:09 | |
| jschwarz | kevinbenton, so no _ha_interfaces | 14:09 |
| jschwarz | status == ACTIVE | 14:09 |
| kevinbenton | jschwarz: is it 'ha'==True? | 14:10 |
| kevinbenton | jschwarz: in that dict? | 14:10 |
| jschwarz | kevinbenton, yes | 14:10 |
| *** sc68cal has joined #openstack-neutron | 14:10 | |
| jschwarz | kevinbenton, received sync_router for deb5d0e1-c6fe-465a-925f-9b39b60a1dab: [{u'status': u'ACTIVE', u'external_gateway_info': None, u'availability_zone_hints': [], u'availability_zones': [u'nova'], u'ha_vr_id': 11, u'name': u's_rally_a4219df5_ygJiCvtB', u'gw_port_id': None, u'admin_state_up': True, u'tenant_id': u'9e5c45411c8a439f9ca9561bd08c7684', u'distributed': False, u'routes': [], u'ha': True, u'id': u'deb5d0e1-c6fe-465a-9 | 14:11 |
| jschwarz | 25f-9b39b60a1dab'}] | 14:11 |
| amuller | at least distributed is False, we got that going for us | 14:11 |
| kevinbenton | jschwarz: oh crap. hang on i got the order of operations wrong with the inheritance | 14:11 |
| kevinbenton | jschwarz: still a race | 14:11 |
| jschwarz | amuller, lol, good morning | 14:11 |
| kevinbenton | jschwarz: one sec | 14:11 |
| *** dshaw has left #openstack-neutron | 14:11 | |
| jschwarz | kevinbenton, ok | 14:11 |
| *** rotbeard has joined #openstack-neutron | 14:13 | |
| openstackgerrit | Brent Eagles proposed openstack/neutron: Adding data model for trunk ports https://review.openstack.org/279251 | 14:13 |
| *** claudiub|2 has joined #openstack-neutron | 14:13 | |
| *** dave-mccowan has joined #openstack-neutron | 14:14 | |
| openstackgerrit | Andreas Scheuring proposed openstack/neutron: macvtap: Macvtap L2 Agent https://review.openstack.org/275306 | 14:14 |
| *** claudiub has quit IRC | 14:14 | |
| scheuran | garyk, ok, pushed a new version - thanks! | 14:15 |
| jschwarz | kevinbenton, btw I can't see the code you changed in the last changeset for the safeguard | 14:16 |
| kevinbenton | jschwarz: it's in one it depends on | 14:16 |
| kevinbenton | jschwarz: the first in the chain with ALLOCATING logic | 14:16 |
| garyk | ok, thanks | 14:16 |
| jschwarz | kevinbenton, _filter_allocating_and_missing_routers? | 14:17 |
| *** openstackgerrit has quit IRC | 14:17 | |
| kevinbenton | yeah | 14:17 |
| *** openstackgerrit has joined #openstack-neutron | 14:18 | |
| *** banix has joined #openstack-neutron | 14:18 | |
| *** rickyrem has joined #openstack-neutron | 14:20 | |
| openstackgerrit | Merged openstack/os-vif: VIFDirect: replace dev_name with dev_address https://review.openstack.org/283081 | 14:20 |
| *** tiswanso has joined #openstack-neutron | 14:21 | |
| slaweq | ajo: jschwarz: hello, can You maybe also take a look on https://review.openstack.org/#/c/248938/29 | 14:22 |
| slaweq | thx :) | 14:22 |
| ajo | slaweq, ack | 14:23 |
| slaweq | ajo: thx a lot | 14:23 |
| *** afaranha has joined #openstack-neutron | 14:24 | |
| *** korzen_ has quit IRC | 14:25 | |
| *** julim has joined #openstack-neutron | 14:25 | |
| *** korzen has joined #openstack-neutron | 14:25 | |
| kevinbenton | jschwarz: in that dict response to any of them have that key set? | 14:27 |
| kevinbenton | jschwarz: i just noticed that we don't include that in all paths :/ | 14:27 |
| kevinbenton | jschwarz: depending on whether or not it went through the scheduler | 14:28 |
| jschwarz | kevinbenton, yay | 14:28 |
| jschwarz | kevinbenton, no, the key set changes | 14:28 |
| kevinbenton | jschwarz: in the same payload for different routers, right? | 14:29 |
| jschwarz | kevinbenton, yes | 14:29 |
| jschwarz | kevinbenton, the full set seems to be: [u'status', u'_interfaces', u'availability_zone_hints', u'availability_zones', u'ha_vr_id', u'name', u'gw_port_id', u'admin_state_up', u'_ha_state', u'_ha_interface', u'distributed', u'tenant_id', u'routes', u'external_gateway_info', u'ha', u'id'] | 14:29 |
| jschwarz | kevinbenton, and sometimes they lack certain keys, specifically '_ha_interface' and '_interfaces' | 14:29 |
| *** pradk has joined #openstack-neutron | 14:30 | |
| *** superdan is now known as dansmith | 14:30 | |
| *** kawa2014 has quit IRC | 14:31 | |
| *** breton has joined #openstack-neutron | 14:31 | |
| *** mfranc213 has joined #openstack-neutron | 14:31 | |
| *** kbringard has joined #openstack-neutron | 14:31 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Make L3 HA interface creation concurrency safe https://review.openstack.org/282876 | 14:33 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Add ALLOCATING state to routers https://review.openstack.org/257059 | 14:33 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Issue retry on router db reference error https://review.openstack.org/283031 | 14:33 |
| jschwarz | kevinbenton, ack | 14:33 |
| kevinbenton | jschwarz: ack, try with ^^ | 14:33 |
| jschwarz | XD | 14:33 |
| kevinbenton | 283031 | 14:33 |
| kevinbenton | is the hed | 14:33 |
| kevinbenton | head* | 14:33 |
| *** mubirru has quit IRC | 14:34 | |
| jschwarz | kevinbenton, it's running, I'll brb | 14:35 |
| kevinbenton | jschwarz: kk | 14:35 |
| kevinbenton | amuller: yo | 14:35 |
| amuller | kevinbenton: oy | 14:35 |
| kevinbenton | amuller: can you check out https://review.openstack.org/#/c/282876/ when you get a chance? | 14:36 |
| kevinbenton | amuller: i made the handling of the HA network during interface creation safer WRT to concurrent deletes | 14:36 |
| ajo | nice :) | 14:37 |
| kevinbenton | too much looking, not enough leaping :) | 14:37 |
| amuller | kevinbenton: this is for the create/delete rally test | 14:37 |
| openstackgerrit | Brent Eagles proposed openstack/neutron: Add helper methods for trunk port model https://review.openstack.org/279253 | 14:37 |
| *** kawa2014 has joined #openstack-neutron | 14:37 | |
| kevinbenton | amuller: i'm not sure if we are hitting that right now, but i noticed it can happen when i was stepping through the code earlier | 14:38 |
| jschwarz | amuller, yes | 14:38 |
| jschwarz | amongst others ;p | 14:39 |
| *** doug-fish has joined #openstack-neutron | 14:39 | |
| jschwarz | kevinbenton, so the good news or the bad news? | 14:39 |
| kevinbenton | both! | 14:39 |
| amuller | kevinbenton: (Making sure I'm up to date) the 'ALLOCATING' patch still has an issue where in some condition (That we don't yet understand) the agent can still receive an HA router without an HA port | 14:39 |
| kevinbenton | multiplexed into one incoherent sentence | 14:39 |
| jschwarz | kevinbenton, the good news are that there are no bad news; the endless loop didn't trigger | 14:40 |
| kevinbenton | amuller: yes, that's what we are trying to fix right now | 14:40 |
| amuller | alright | 14:40 |
| kevinbenton | (or maybe just fixed :) ) | 14:40 |
| jschwarz | XD | 14:40 |
| jschwarz | running it again now | 14:40 |
| *** claudiub|2 has quit IRC | 14:40 | |
| *** vhoward has joined #openstack-neutron | 14:41 | |
| *** salv-orlando has joined #openstack-neutron | 14:41 | |
| *** claudiub has joined #openstack-neutron | 14:41 | |
| jschwarz | kevinbenton, I can't even find the infamous "Unable to find HA port" error | 14:41 |
| kevinbenton | jschwarz: ok, i put in a sanity cheat that i want you to check to see if was hit | 14:42 |
| kevinbenton | jschwarz: L661 of https://review.openstack.org/#/c/257059/19/neutron/db/l3_hamode_db.py | 14:42 |
| jschwarz | aye, I see it | 14:43 |
| kevinbenton | jschwarz: does that string show up anywhere? | 14:43 |
| kevinbenton | jschwarz: if not, the ALLOCATING stuff was enough | 14:43 |
| jschwarz | kevinbenton, nope | 14:43 |
| jschwarz | (it doesn't show up) | 14:43 |
| *** zhhuabj_ has joined #openstack-neutron | 14:43 | |
| kevinbenton | jschwarz: excellent. i'll pull that out and then can you do one more run? | 14:44 |
| *** dane_leblanc has joined #openstack-neutron | 14:44 | |
| jschwarz | kevinbenton, aye | 14:44 |
| *** shwetaap has joined #openstack-neutron | 14:44 | |
| *** claudiub|2 has joined #openstack-neutron | 14:45 | |
| jschwarz | kevinbenton, is this needed now: https://review.openstack.org/#/c/283031/5 ? | 14:45 |
| *** josecastroleon has quit IRC | 14:46 | |
| *** apuimedo has quit IRC | 14:46 | |
| kevinbenton | jschwarz: yes, i think so. that will still die if a router is being scheduled to an agent and the router is deleted | 14:47 |
| kevinbenton | jschwarz: so we just restart the sync and it will be recognized as a non-existent router | 14:47 |
| *** claudiub|3 has joined #openstack-neutron | 14:47 | |
| *** claudiub has quit IRC | 14:48 | |
| kevinbenton | jschwarz: does that make sense? | 14:48 |
| jschwarz | kevinbenton, right I remembered | 14:48 |
| jschwarz | kevinbenton, it does | 14:48 |
| kevinbenton | jschwarz: that one isn't a race in the creation, just the scheduling | 14:48 |
| *** srampal has joined #openstack-neutron | 14:49 | |
| jschwarz | kevinbenton, perhaps we should squash the second and the third patches together, then? | 14:49 |
| jschwarz | kevinbenton, both deal with create-delete concurrency issues | 14:50 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Make L3 HA interface creation concurrency safe https://review.openstack.org/282876 | 14:50 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Add ALLOCATING state to routers https://review.openstack.org/257059 | 14:50 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Issue retry on router db reference error https://review.openstack.org/283031 | 14:50 |
| kevinbenton | jschwarz: no, only because Liu has another patch in flight to deal with that 3rd patch | 14:50 |
| *** claudiub|2 has quit IRC | 14:50 | |
| kevinbenton | jschwarz: it solves it a different way IIRC | 14:50 |
| kevinbenton | jschwarz: and we should review that one. i just put mine up for us to test because it's a really simple solution that i know would work | 14:51 |
| kevinbenton | jschwarz: but it's kind a hammer since it restarts the whole sync_routers operation at the API level | 14:51 |
| jschwarz | kevinbenton, i'll give the latest a try | 14:51 |
| *** claudiub|3 has quit IRC | 14:51 | |
| kevinbenton | jschwarz: cool | 14:52 |
| *** claudiub|3 has joined #openstack-neutron | 14:52 | |
| ajo | slaweq, I'm reading https://review.openstack.org/#/c/248938 slowly :) | 14:53 |
| kevinbenton | amuller: is the functional job broken? | 14:53 |
| jschwarz | kevinbenton, Liu's patch simply ignores the error, no retires | 14:53 |
| jschwarz | retries* | 14:53 |
| *** sridhar_ram has joined #openstack-neutron | 14:53 | |
| jschwarz | kevinbenton, https://review.openstack.org/#/c/265685/7/neutron/db/l3_hamode_db.py | 14:53 |
| *** nmagnezi has quit IRC | 14:53 | |
| jschwarz | actually I might be looking at the wrong patch... | 14:54 |
| slaweq | ajo: ok :) | 14:54 |
| openstackgerrit | Artur Korzeniewski proposed openstack/neutron: WIP Introducing the Subnet OVO https://review.openstack.org/264273 | 14:54 |
| jschwarz | ajo, do we have an upgrades meeting today? | 14:55 |
| kevinbenton | jschwarz: this was the one https://review.openstack.org/#/c/260303/ | 14:55 |
| *** jpena|lunch is now known as jpena | 14:55 | |
| *** krtaylor has quit IRC | 14:55 | |
| kevinbenton | jschwarz: it ignores the error as well, which is the more correct response but has to be more careful about branching to handle the failure case | 14:55 |
| ajo | jschwarz, I don't know, I'm a bit sick, so if there's meeting I'd probably back off :/ | 14:56 |
| jschwarz | ajo, :< | 14:56 |
| *** zhhuabj_ has quit IRC | 14:57 | |
| ajo | slaweq, why do you use the safe_* cleanups ? | 14:57 |
| ajo | does it fail sometimes? | 14:57 |
| *** rossella_s has quit IRC | 14:58 | |
| slaweq | yes | 14:58 |
| ajo | slaweq, any idea of why? | 14:58 |
| *** evgenyf has joined #openstack-neutron | 14:58 | |
| *** rossella_s has joined #openstack-neutron | 14:58 | |
| slaweq | please give me 5 minutes | 14:58 |
| slaweq | I will check it in code and then I will remind details :) | 14:58 |
| ajo | ack slaweq , I keep reading | 14:58 |
| jschwarz | kevinbenton, anyway, everything is good on my end | 14:59 |
| ihrachys | korzen: sc68cal: roaet_: ajo: jschwarz: friendly reminder about the upgrades meeting in #openstack-meeting-alt in 2 mins | 14:59 |
| jschwarz | hdaniel, ^ | 14:59 |
| ajo | ihrachys, I will probably lurk, but not sure if I'll stay, I'm feeling sicker | 14:59 |
| kevinbenton | jschwarz: ok. i'm out for now! | 14:59 |
| jschwarz | kevinbenton, yey, nighty night! :) | 15:00 |
| *** afaranha_ has joined #openstack-neutron | 15:00 | |
| sc68cal | ihrachys: here | 15:00 |
| *** zhhuabj_ has joined #openstack-neutron | 15:00 | |
| slaweq | ok, if You are talking about methods in LinuxBridgeFixture I think that reason of it was that bridges were created in namespace "Host-XXX" and later namespace was deleted before bridges | 15:00 |
| *** zhhuabj_ has quit IRC | 15:01 | |
| *** moshele has quit IRC | 15:01 | |
| *** afaranha has quit IRC | 15:01 | |
| *** hichihara has joined #openstack-neutron | 15:03 | |
| *** nmagnezi has joined #openstack-neutron | 15:03 | |
| *** ollivier has joined #openstack-neutron | 15:03 | |
| *** singhj has joined #openstack-neutron | 15:04 | |
| *** hichihara has quit IRC | 15:04 | |
| *** ollivier has left #openstack-neutron | 15:04 | |
| *** javeriak has joined #openstack-neutron | 15:08 | |
| *** rpothier has joined #openstack-neutron | 15:08 | |
| openstackgerrit | Richard Theis proposed openstack/python-neutronclient: Devref Update: Transition to OpenStack Client https://review.openstack.org/282555 | 15:08 |
| *** oshvartz has quit IRC | 15:09 | |
| *** javeriak has quit IRC | 15:09 | |
| *** javeriak has joined #openstack-neutron | 15:09 | |
| Sam-I-Am | rtheis: moo | 15:09 |
| rtheis | Sam-I-Am: hi | 15:10 |
| *** javeriak_ has joined #openstack-neutron | 15:12 | |
| *** yfried has joined #openstack-neutron | 15:13 | |
| *** javeriak has quit IRC | 15:14 | |
| *** baoli has joined #openstack-neutron | 15:16 | |
| *** dguitarbite has quit IRC | 15:18 | |
| *** haleyb has joined #openstack-neutron | 15:18 | |
| *** dguitarbite has joined #openstack-neutron | 15:18 | |
| *** sridharg has quit IRC | 15:21 | |
| *** ijw has joined #openstack-neutron | 15:21 | |
| *** z_kassab has joined #openstack-neutron | 15:22 | |
| garyk | scheuran: two more nits and then i am fine with it | 15:24 |
| scheuran | garyk, ok, let me have a look. thanks | 15:24 |
| *** lbzhao has joined #openstack-neutron | 15:24 | |
| *** crose has joined #openstack-neutron | 15:24 | |
| *** ijw has quit IRC | 15:26 | |
| *** singhj1 has joined #openstack-neutron | 15:26 | |
| *** singhj has quit IRC | 15:28 | |
| *** lbzhao has quit IRC | 15:28 | |
| amuller | kevinbenton: it was last night yes | 15:29 |
| amuller | kevinbenton: didn't check this morning | 15:29 |
| *** mgoddard has joined #openstack-neutron | 15:31 | |
| *** mgoddard_ has quit IRC | 15:31 | |
| garyk | scheuran: you woulc do them with your eyes closed :) | 15:32 |
| scheuran | garyk, let's give it a try ;) | 15:32 |
| *** achanda has joined #openstack-neutron | 15:34 | |
| *** rotbeard has quit IRC | 15:37 | |
| *** azbiswas has joined #openstack-neutron | 15:40 | |
| *** wwriverrat has quit IRC | 15:40 | |
| *** mgoddard_ has joined #openstack-neutron | 15:40 | |
| *** changbl has quit IRC | 15:41 | |
| *** localloop127 has quit IRC | 15:41 | |
| *** mgoddard has quit IRC | 15:41 | |
| openstackgerrit | Andreas Scheuring proposed openstack/neutron: macvtap: Macvtap L2 Agent https://review.openstack.org/275306 | 15:41 |
| *** ekuris has quit IRC | 15:42 | |
| scheuran | garyk, please see ^ | 15:42 |
| *** azbiswas_ has joined #openstack-neutron | 15:42 | |
| openstackgerrit | Hong Hui Xiao proposed openstack/neutron: Use floating IP to connect different address scopes https://review.openstack.org/283134 | 15:43 |
| garyk | scheuran: done | 15:43 |
| scheuran | garyk, nice, thanks! | 15:43 |
| garyk | thank you. hope this manages to get in. | 15:44 |
| *** achanda has quit IRC | 15:45 | |
| *** azbiswas has quit IRC | 15:45 | |
| *** krtaylor has joined #openstack-neutron | 15:46 | |
| scheuran | I'm confident it will.. | 15:47 |
| *** dingboopt has quit IRC | 15:47 | |
| *** jaugustine has joined #openstack-neutron | 15:47 | |
| *** jlanoux has quit IRC | 15:49 | |
| *** ildikov_ has quit IRC | 15:49 | |
| *** mdih has joined #openstack-neutron | 15:50 | |
| *** Sukhdev has joined #openstack-neutron | 15:50 | |
| *** jprovazn has joined #openstack-neutron | 15:50 | |
| *** cbouch has joined #openstack-neutron | 15:51 | |
| *** mgoddard_ has quit IRC | 15:52 | |
| *** mgoddard has joined #openstack-neutron | 15:52 | |
| openstackgerrit | Haim Daniel proposed openstack/neutron: Qos policy RBAC DB setup and migration https://review.openstack.org/250081 | 15:52 |
| *** fedexo has joined #openstack-neutron | 15:53 | |
| openstackgerrit | Jakub Libosvar proposed openstack/neutron: devref: Update ovs-firewall with rules explanation https://review.openstack.org/283137 | 15:56 |
| *** nyechiel_ has quit IRC | 15:56 | |
| *** changbl has joined #openstack-neutron | 15:56 | |
| *** nyechiel_ has joined #openstack-neutron | 15:56 | |
| *** numans has quit IRC | 15:58 | |
| *** ajmiller has joined #openstack-neutron | 15:59 | |
| openstackgerrit | Hong Hui Xiao proposed openstack/neutron: Revert the unused code for address scope https://review.openstack.org/283139 | 16:00 |
| *** regXboi has joined #openstack-neutron | 16:00 | |
| *** lajos-katona has quit IRC | 16:00 | |
| openstackgerrit | Victor Morales proposed openstack/neutron: Add sphinx extension to generate db diagram https://review.openstack.org/274874 | 16:01 |
| *** itzikb has quit IRC | 16:02 | |
| *** anilvenkata has quit IRC | 16:02 | |
| *** lezbar has joined #openstack-neutron | 16:03 | |
| *** wwriverrat has joined #openstack-neutron | 16:03 | |
| regXboi | carl_baldwin: ping | 16:04 |
| *** zhhuabj_ has joined #openstack-neutron | 16:04 | |
| *** sridhar_ram has quit IRC | 16:05 | |
| carl_baldwin | regXboi: pong | 16:05 |
| *** jhershbe has quit IRC | 16:07 | |
| regXboi | carl_baldwin: I'm trying to make sense out of https://review.openstack.org/283134 and since your name pops up, I wondered if you could tell me about it | 16:07 |
| *** Oku_OS has joined #openstack-neutron | 16:07 | |
| carl_baldwin | regXboi: It might take me a bit to refresh my memory on that one. | 16:08 |
| *** tiagogomes has left #openstack-neutron | 16:09 | |
| *** elopez has joined #openstack-neutron | 16:10 | |
| *** anshul has quit IRC | 16:10 | |
| regXboi | carl_baldwin: no worries, I've got time :) | 16:10 |
| *** changbl has quit IRC | 16:10 | |
| *** armax has joined #openstack-neutron | 16:10 | |
| carl_baldwin | regXboi: Are you going to be in Rochester? I'll be traveling this afternoon. | 16:10 |
| *** changbl has joined #openstack-neutron | 16:11 | |
| *** bharath has joined #openstack-neutron | 16:12 | |
| regXboi | carl_baldwin: unfortunately, I will only be there remotely - dealing with some potential health issues with my daughter, so I need to be around here in case we have crises... | 16:12 |
| *** sudipto__ has joined #openstack-neutron | 16:12 | |
| carl_baldwin | regXboi: Understood. Take good care of her. | 16:12 |
| carl_baldwin | regXboi: I'll see if I can get to that review today. If so, I'll ping you. Maybe I can add some clarifying notes to the review. | 16:13 |
| *** zhhuabj_ has quit IRC | 16:15 | |
| *** hdaniel has quit IRC | 16:16 | |
| regXboi | carl_baldwin: sounds good - travel safe | 16:16 |
| carl_baldwin | regXboi: Thanks | 16:18 |
| *** jlanoux has joined #openstack-neutron | 16:19 | |
| *** korzen has quit IRC | 16:21 | |
| *** sbalukoff has quit IRC | 16:21 | |
| *** mvk has quit IRC | 16:22 | |
| *** dosaboy has quit IRC | 16:23 | |
| *** Swami has joined #openstack-neutron | 16:23 | |
| *** bharathk has joined #openstack-neutron | 16:25 | |
| *** bharath has quit IRC | 16:26 | |
| dasm | good day all o/ | 16:27 |
| *** pcaruana has quit IRC | 16:27 | |
| *** ildikov_ has joined #openstack-neutron | 16:29 | |
| *** josecastroleon has joined #openstack-neutron | 16:29 | |
| *** nmagnezi has quit IRC | 16:29 | |
| *** gampel has quit IRC | 16:29 | |
| *** fedexo has quit IRC | 16:31 | |
| *** tbachman has joined #openstack-neutron | 16:31 | |
| *** wolsen has quit IRC | 16:34 | |
| *** iyamahat has joined #openstack-neutron | 16:34 | |
| *** mfranc213 has quit IRC | 16:36 | |
| *** rickyrem has quit IRC | 16:37 | |
| *** rickyrem has joined #openstack-neutron | 16:38 | |
| *** sleviim has quit IRC | 16:40 | |
| *** ijw has joined #openstack-neutron | 16:41 | |
| *** wolsen has joined #openstack-neutron | 16:42 | |
| *** john-davidge has joined #openstack-neutron | 16:42 | |
| *** dosaboy has joined #openstack-neutron | 16:43 | |
| *** john-davidge has quit IRC | 16:44 | |
| *** localloop127 has joined #openstack-neutron | 16:44 | |
| amuller | armax: kevinbenton: Looks like the functional job is belly up | 16:44 |
| amuller | I'm not aware of anyone working on it | 16:44 |
| armax | amuller: any idea what happened? | 16:44 |
| amuller | I looked at it briefly last night and nothing immediate jumped to me | 16:44 |
| sc68cal | ihrachys: Dumb question - when we talk about partial vs. non-partial upgrades | 16:44 |
| sc68cal | ihrachys: do we mean - upgrade all components at the same time (non-partial) | 16:45 |
| sc68cal | ihrachys: upgrade controller, then computes = partial? | 16:45 |
| amuller | armax: if you look at the console output tox is running the tests, passing all of them, then failing the job, I'm not clear why | 16:45 |
| *** ijw has quit IRC | 16:45 | |
| armax | amuller: link? | 16:45 |
| *** amotoki has quit IRC | 16:45 | |
| amuller | armax: any patch :) | 16:46 |
| armax | amuller: still a link would be handy :) | 16:46 |
| jschwarz | armax, amuller,l | 16:46 |
| jschwarz | http://logs.openstack.org/59/257059/20/check/gate-neutron-dsvm-functional/9ba0247/ | 16:46 |
| amuller | armax: http://logs.openstack.org/59/257059/20/check/gate-neutron-dsvm-functional/9ba0247/console.html | 16:46 |
| ihrachys | sc68cal: yes, that's what I mean using those terms. | 16:46 |
| ihrachys | sc68cal: though someone may have a different view | 16:46 |
| ihrachys | sc68cal: that's like a problem of rolling upgrade definition itself | 16:47 |
| ihrachys | sc68cal: f.e. we are about to claim support for old compute vs. new controller scenario | 16:47 |
| sc68cal | ihrachys: ok - I just wanted to make sure I was on same page as you, and yes I agree with your definition | 16:47 |
| *** jckasper_ has joined #openstack-neutron | 16:47 | |
| ihrachys | sc68cal: but we are not there in mixing controller versions | 16:47 |
| *** bharath has joined #openstack-neutron | 16:47 | |
| ihrachys | sc68cal: so can you claim we support rolling upgrades? depends on definition | 16:47 |
| *** bharathk has quit IRC | 16:47 | |
| *** araji has joined #openstack-neutron | 16:48 | |
| ihrachys | sc68cal: if you want each service instance to be upgradable separately, then we are not there; if it's about each common node ROLE being upgraded separately, then we are there | 16:48 |
| *** jckasper has quit IRC | 16:48 | |
| sc68cal | ihrachys: gotcha. I think we will need to document that - you *must* upgrade all the controllers at the same time | 16:49 |
| sc68cal | if you are doing HA | 16:49 |
| sc68cal | at this point in time | 16:49 |
| sc68cal | until we get the o.vo work complete | 16:50 |
| sc68cal | then maybe it'll be possible | 16:50 |
| *** zhhuabj_ has joined #openstack-neutron | 16:50 | |
| *** john-davidge has joined #openstack-neutron | 16:50 | |
| *** Leo_ has joined #openstack-neutron | 16:51 | |
| armax | amuller: thanks | 16:52 |
| *** todin_ is now known as todin | 16:52 | |
| ihrachys | sc68cal: it's somewhat documented. let me show you some links. | 16:53 |
| armax | amuller: I wonder if this is the culprit | 16:53 |
| *** haplo37 has joined #openstack-neutron | 16:53 | |
| armax | https://review.openstack.org/#/c/282337/1/.testr.conf | 16:53 |
| armax | amuller: but maybe not | 16:53 |
| ihrachys | sc68cal: first, ops side, we have this doc update proposed that should clarify the matter: https://review.openstack.org/#/c/276335/ | 16:53 |
| *** jwarendt has joined #openstack-neutron | 16:54 | |
| ihrachys | sc68cal: also we have devref section that covers that http://docs.openstack.org/developer/neutron/devref/upgrade.html#server-upgrade | 16:54 |
| *** eddima has quit IRC | 16:54 | |
| sc68cal | ihrachys: gotcha. yeah i've read the doc in devref for upgrades | 16:55 |
| *** zhhuabj_ has quit IRC | 16:55 | |
| *** mgoddard_ has joined #openstack-neutron | 16:55 | |
| *** hdaniel has joined #openstack-neutron | 16:55 | |
| *** mgoddard has quit IRC | 16:55 | |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: DVR: Handle unbound allowed_address_pair port with FIP https://review.openstack.org/254439 | 16:56 |
| amuller | armax: it's possible, it seems like the functional job error is not 100% | 16:56 |
| ihrachys | sc68cal: actually, I think we don't mention that requirement in networking-guide change proposed | 16:56 |
| ihrachys | sc68cal: I will leave the comment there :) | 16:56 |
| *** jschwarz has quit IRC | 16:56 | |
| *** zhhuabj_ has joined #openstack-neutron | 16:57 | |
| *** sridhar_ram1 has joined #openstack-neutron | 16:57 | |
| *** jlibosva has quit IRC | 16:57 | |
| armax | amuller: interesting | 16:57 |
| sc68cal | ihrachys: sounds good | 16:58 |
| *** Leo_ has quit IRC | 16:58 | |
| *** xianghui has quit IRC | 16:58 | |
| *** jwarendt has quit IRC | 16:58 | |
| *** kobis has quit IRC | 16:58 | |
| *** josecastroleon has quit IRC | 16:59 | |
| *** azbiswas_ has quit IRC | 16:59 | |
| ihrachys | armax: ping me when you are available for a quick heads-up from upgrades team on landing object patches in Mitaka | 17:00 |
| *** jwarendt has joined #openstack-neutron | 17:01 | |
| *** zhhuabj_ has quit IRC | 17:02 | |
| *** vishwana_ has joined #openstack-neutron | 17:02 | |
| ihrachys | armax: ok I guess I better just post and you read and comment later; we discussed how we can proceed during the remaining cycle time on objects without affecting code stability, and we decided that we'll still try to land some pieces, but without their integration into the production code. so we'll f.e. land port object but won't switch db classes to use it. and then we'll work on adopting the objects in production code early | 17:03 |
| ihrachys | in N. obviously, that has a drawback that the pieces landed won't have extensive validation in gate, just unit tests. does it sound like a reasonable plan? | 17:03 |
| *** vhosakot has joined #openstack-neutron | 17:05 | |
| *** jwarendt has quit IRC | 17:05 | |
| *** irenab_ has joined #openstack-neutron | 17:05 | |
| *** vishwanathj has quit IRC | 17:05 | |
| *** jwarendt has joined #openstack-neutron | 17:06 | |
| *** jreeves has quit IRC | 17:08 | |
| *** irenab has quit IRC | 17:08 | |
| *** irenab_ is now known as irenab | 17:08 | |
| *** scheuran has quit IRC | 17:10 | |
| *** jreeves has joined #openstack-neutron | 17:11 | |
| *** minwang2 has joined #openstack-neutron | 17:13 | |
| *** nlahouti has joined #openstack-neutron | 17:15 | |
| *** bharath has quit IRC | 17:15 | |
| carl_baldwin | regXboi: ping | 17:16 |
| carl_baldwin | regXboi: I added some reviewer notes to https://review.openstack.org/#/c/283134 . I had to swap in some old memories to remember what that patch was getting at. | 17:17 |
| *** Sukhdev has quit IRC | 17:17 | |
| regXboi | ack | 17:17 |
| *** fawadkhaliq has joined #openstack-neutron | 17:18 | |
| *** ijw has joined #openstack-neutron | 17:19 | |
| *** fawadkhaliq has quit IRC | 17:19 | |
| *** daneyon has joined #openstack-neutron | 17:21 | |
| *** tmorin has quit IRC | 17:21 | |
| *** daneyon_ has joined #openstack-neutron | 17:21 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron-specs: Agent specific API for L2 agent extensions https://review.openstack.org/263819 | 17:22 |
| *** ijw_ has joined #openstack-neutron | 17:22 | |
| *** ijw has quit IRC | 17:23 | |
| *** bharath has joined #openstack-neutron | 17:24 | |
| *** daneyon has quit IRC | 17:25 | |
| *** amuller is now known as amuller_afk | 17:27 | |
| *** reedip_ has joined #openstack-neutron | 17:27 | |
| *** tidwellr1 has quit IRC | 17:30 | |
| *** mickeys has joined #openstack-neutron | 17:32 | |
| *** ivar-lazzaro has joined #openstack-neutron | 17:32 | |
| *** alhu__ has joined #openstack-neutron | 17:33 | |
| *** alhu__ has quit IRC | 17:33 | |
| *** alwaysatthenoc has joined #openstack-neutron | 17:34 | |
| *** ihrachys has quit IRC | 17:35 | |
| *** matrohon has quit IRC | 17:36 | |
| *** vishwanathj has joined #openstack-neutron | 17:37 | |
| *** josecastroleon has joined #openstack-neutron | 17:38 | |
| *** eil397 has joined #openstack-neutron | 17:38 | |
| dasm | mestery: hey. can you add me to neutron-bugs team? i'd like to start mu bug deputy :) | 17:39 |
| *** spandhe has joined #openstack-neutron | 17:39 | |
| *** vishwana_ has quit IRC | 17:40 | |
| *** allan_h has joined #openstack-neutron | 17:40 | |
| dasm | thanks, for anyone who did it ^ :) | 17:41 |
| *** jlanoux has quit IRC | 17:41 | |
| *** fawadkhaliq has joined #openstack-neutron | 17:42 | |
| *** alwaysatthenoc has quit IRC | 17:43 | |
| *** kickinz1 has quit IRC | 17:43 | |
| *** eil397 has quit IRC | 17:43 | |
| *** eil397 has joined #openstack-neutron | 17:43 | |
| *** reedip_ has quit IRC | 17:46 | |
| *** srampal has quit IRC | 17:46 | |
| *** z_kassab has quit IRC | 17:46 | |
| *** subscope has quit IRC | 17:47 | |
| openstackgerrit | German Eichberger proposed openstack/neutron-fwaas: [WIP] Implements Versioned Objects https://review.openstack.org/274914 | 17:48 |
| *** Aish has joined #openstack-neutron | 17:50 | |
| *** dslevin has joined #openstack-neutron | 17:50 | |
| *** wolverineav has quit IRC | 17:50 | |
| *** fawadkhaliq has quit IRC | 17:51 | |
| *** bharath has quit IRC | 17:51 | |
| *** dslevin has quit IRC | 17:51 | |
| *** thumpba has joined #openstack-neutron | 17:51 | |
| *** wolverineav has joined #openstack-neutron | 17:52 | |
| *** abhiraut has joined #openstack-neutron | 17:55 | |
| *** jistr has quit IRC | 17:55 | |
| *** wolverineav has quit IRC | 17:56 | |
| *** banix has quit IRC | 17:56 | |
| *** davidsha has quit IRC | 17:59 | |
| *** shivrao has joined #openstack-neutron | 18:02 | |
| manjeets | is neutron functional job still broken or fixed ? | 18:03 |
| manjeets | i don't see any test failure here http://logs.openstack.org/46/124946/22/check/gate-neutron-dsvm-functional/e6f18b7/console.html#_2016-02-22_17_04_37_348 but jobs ends up in failure | 18:03 |
| openstackgerrit | Merged openstack/neutron-fwaas: Add testresources to neutron-fwaas test requirements. https://review.openstack.org/281535 | 18:04 |
| *** yfried has quit IRC | 18:05 | |
| *** hdaniel has quit IRC | 18:07 | |
| *** sudipto__ has quit IRC | 18:08 | |
| *** shashank_hegde has quit IRC | 18:08 | |
| *** gangil has joined #openstack-neutron | 18:08 | |
| *** gangil has quit IRC | 18:08 | |
| *** gangil has joined #openstack-neutron | 18:08 | |
| openstackgerrit | Merged openstack/neutron-vpnaas: Updated from global requirements https://review.openstack.org/278698 | 18:09 |
| *** sambetts is now known as sambetts|afk | 18:10 | |
| *** josecastroleon has quit IRC | 18:10 | |
| *** ijw_ has quit IRC | 18:10 | |
| *** yfried has joined #openstack-neutron | 18:11 | |
| *** Leom has joined #openstack-neutron | 18:14 | |
| *** azbiswas has joined #openstack-neutron | 18:14 | |
| *** Leom_ has joined #openstack-neutron | 18:15 | |
| *** ivar-lazzaro has quit IRC | 18:16 | |
| *** shivrao has quit IRC | 18:16 | |
| *** jpena has quit IRC | 18:18 | |
| *** Leom has quit IRC | 18:19 | |
| *** kawa2014 has quit IRC | 18:21 | |
| *** banix has joined #openstack-neutron | 18:22 | |
| *** banix has quit IRC | 18:22 | |
| *** josecastroleon has joined #openstack-neutron | 18:23 | |
| *** gangil has quit IRC | 18:23 | |
| *** banix has joined #openstack-neutron | 18:23 | |
| *** neiljerram has quit IRC | 18:23 | |
| *** Leom_ has quit IRC | 18:26 | |
| *** srampal has joined #openstack-neutron | 18:30 | |
| *** davideagnello has joined #openstack-neutron | 18:31 | |
| *** john-davidge has quit IRC | 18:31 | |
| *** sridhar_ram1 has quit IRC | 18:33 | |
| *** gvrangan has quit IRC | 18:33 | |
| *** dslevin has joined #openstack-neutron | 18:34 | |
| *** dslevin has quit IRC | 18:34 | |
| *** david-lyle_ has quit IRC | 18:34 | |
| *** wolverineav has joined #openstack-neutron | 18:36 | |
| *** dslevin has joined #openstack-neutron | 18:36 | |
| *** dslevin has quit IRC | 18:36 | |
| *** dslevin has joined #openstack-neutron | 18:37 | |
| *** dslevin has quit IRC | 18:37 | |
| openstackgerrit | Merged openstack/neutron: IPAM: add missing translation https://review.openstack.org/282858 | 18:37 |
| *** shivrao has joined #openstack-neutron | 18:38 | |
| *** dslevin has joined #openstack-neutron | 18:38 | |
| *** dslevin has quit IRC | 18:38 | |
| *** dslevin has joined #openstack-neutron | 18:39 | |
| openstackgerrit | Merged openstack/neutron: Functional test for address scope https://review.openstack.org/272474 | 18:39 |
| *** sc68cal has quit IRC | 18:39 | |
| *** dslevin has quit IRC | 18:39 | |
| *** dslevin has joined #openstack-neutron | 18:40 | |
| *** dslevin has quit IRC | 18:40 | |
| *** dslevin has joined #openstack-neutron | 18:40 | |
| *** Aish has quit IRC | 18:41 | |
| *** salv-orl_ has joined #openstack-neutron | 18:42 | |
| *** gberginc has quit IRC | 18:43 | |
| openstackgerrit | Carl Baldwin proposed openstack/neutron: Correlate address scope with network https://review.openstack.org/282942 | 18:44 |
| *** yamahata has quit IRC | 18:44 | |
| *** salv-orlando has quit IRC | 18:44 | |
| *** yamahata has joined #openstack-neutron | 18:45 | |
| *** sc68cal has joined #openstack-neutron | 18:45 | |
| *** rkukura has joined #openstack-neutron | 18:45 | |
| *** neelashah has quit IRC | 18:46 | |
| *** tflynn has joined #openstack-neutron | 18:46 | |
| *** john-davidge has joined #openstack-neutron | 18:47 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: BGP Dynamic Routing: introduce BgpDriver https://review.openstack.org/207635 | 18:49 |
| *** thorst is now known as thorst_afk | 18:50 | |
| openstackgerrit | Merged openstack/neutron: Deprecate 'force_gateway_on_subnet' configuration option https://review.openstack.org/277303 | 18:52 |
| *** shashank_hegde has joined #openstack-neutron | 18:52 | |
| *** thorst_afk is now known as thorst | 18:52 | |
| *** josecastroleon has quit IRC | 18:52 | |
| *** Aish has joined #openstack-neutron | 18:53 | |
| *** itzikb has joined #openstack-neutron | 18:53 | |
| *** Marga_ has quit IRC | 18:55 | |
| *** gvrangan has joined #openstack-neutron | 18:56 | |
| *** nlahouti has quit IRC | 19:00 | |
| *** ihrachys has joined #openstack-neutron | 19:01 | |
| *** abhiraut has quit IRC | 19:01 | |
| *** nlahouti has joined #openstack-neutron | 19:01 | |
| *** krtaylor has quit IRC | 19:01 | |
| *** tflynn has quit IRC | 19:04 | |
| *** subscope has joined #openstack-neutron | 19:04 | |
| *** amuller_afk is now known as amuller | 19:04 | |
| amuller | manjeets: no progress I think | 19:04 |
| *** tflynn has joined #openstack-neutron | 19:04 | |
| amuller | armax: any clues while I was gone? | 19:05 |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Add Queries For BGP Route Lookups https://review.openstack.org/242192 | 19:05 |
| armax | amuller: nope, I was packing between one thing and another | 19:05 |
| amuller | armax: ok I'll spend some time on this. before lunch I sampled the current review queue and it looks like around 50% failure rate | 19:06 |
| *** klindgren has joined #openstack-neutron | 19:06 | |
| armax | amuller: I had this one open at one point and I can concur | 19:06 |
| armax | http://grafana.openstack.org/dashboard/db/neutron-failure-rate?panelId=5&fullscreen | 19:06 |
| *** ZZelle_ has joined #openstack-neutron | 19:06 | |
| armax | it’s probably closer to 70 | 19:06 |
| armax | % | 19:06 |
| armax | but it’s not edging any farther | 19:07 |
| amuller | armax: let's just delete all the tests... accelerate development and conserve resources, win win! | 19:07 |
| *** neelashah has joined #openstack-neutron | 19:07 | |
| armax | amuller: +2 | 19:07 |
| *** elo has joined #openstack-neutron | 19:07 | |
| amuller | armax: great in the short term and who cares about the long term right? | 19:07 |
| armax | amuller: long term? I don’t even know what that means | 19:08 |
| armax | do you? | 19:08 |
| amuller | armax: sorry I'm not a native english speaker :( | 19:08 |
| amuller | those arcane concepts elude me | 19:08 |
| armax | well, me neither | 19:08 |
| armax | amuller: btw fwiw | 19:09 |
| armax | amuller: you know what those means don’t you? | 19:09 |
| armax | the failure looks like some ucaught exception that makes testr freak out | 19:09 |
| amuller | uhum | 19:09 |
| *** josecastroleon has joined #openstack-neutron | 19:10 | |
| *** evgenyf has quit IRC | 19:10 | |
| *** s3wong has joined #openstack-neutron | 19:10 | |
| amuller | armax: could be a random patch that reshuffled the test execution grouping then | 19:10 |
| *** psargent has quit IRC | 19:10 | |
| armax | amuller: it might, salv-orl_ might know how to troubleshoot these types of failures | 19:10 |
| *** dileepr has joined #openstack-neutron | 19:10 | |
| *** jobewan has joined #openstack-neutron | 19:11 | |
| amuller | armax: testr has an option that isolates failures, it finds you the minimal set of tests that reproduces the failure | 19:11 |
| amuller | armax: I'll start running that, it can take time... dunno if it reproduces locally also | 19:11 |
| armax | amuller: ok, the ft83.1: process-returncode_StringException: returncode 1 are always ‘fun’ to root cause | 19:11 |
| *** ivar-lazzaro has joined #openstack-neutron | 19:12 | |
| *** ivar-lazzaro has quit IRC | 19:13 | |
| *** ivar-lazzaro has joined #openstack-neutron | 19:14 | |
| *** sbalukoff has joined #openstack-neutron | 19:15 | |
| armax | amuller: did you pinpoint whether this is master only? | 19:15 |
| salv-orl_ | armax, amuller ? | 19:15 |
| *** ivar-lazzaro has quit IRC | 19:15 | |
| armax | functional job is failing withrocess-returncode_StringException: returncode 1 | 19:15 |
| amuller | salv-orl_: not every run though | 19:15 |
| armax | salv-orl_: word of advice? | 19:15 |
| salv-orl_ | armax: and no appparent failure right | 19:15 |
| *** ivar-lazzaro has joined #openstack-neutron | 19:15 | |
| armax | salv-orl_: correct | 19:16 |
| *** safchain has quit IRC | 19:16 | |
| salv-orl_ | I saw one of them this morning, I thought some test might be killing the test runner | 19:16 |
| salv-orl_ | I don't think we're still liable to wild sys.exit in the code thought | 19:16 |
| armax | salv-orl_: that’s what I figured | 19:16 |
| armax | some uncaught exception like that | 19:17 |
| salv-orl_ | armax: I'm afraid the log did not give absolutely any information | 19:17 |
| armax | salv-orl_: indeed | 19:17 |
| armax | amuller: btw, this looks to be master only | 19:17 |
| *** changbl has quit IRC | 19:18 | |
| salv-orl_ | what's worse is that I'm not sure here we play that game where we revert patches until the error goes away | 19:18 |
| armax | salv-orl_: this merged at about the time the issue arose: https://review.openstack.org/#/c/260440/ | 19:20 |
| HenryG | I hope it's not that pipe to cat thing I approved | 19:20 |
| armax | HenryG: I thought so, but then ruled it out | 19:20 |
| *** jprovazn has quit IRC | 19:21 | |
| *** jhershbe has joined #openstack-neutron | 19:22 | |
| *** sridhar_ram has joined #openstack-neutron | 19:22 | |
| openstackgerrit | Armando Migliaccio proposed openstack/neutron: Revert "Pecan: get rid of member action hook" https://review.openstack.org/283218 | 19:23 |
| *** neelashah1 has joined #openstack-neutron | 19:25 | |
| *** neelashah has quit IRC | 19:25 | |
| *** gangil has joined #openstack-neutron | 19:25 | |
| *** gangil has quit IRC | 19:25 | |
| *** gangil has joined #openstack-neutron | 19:25 | |
| *** efoley has quit IRC | 19:26 | |
| *** afaranha_ has left #openstack-neutron | 19:26 | |
| *** Leom has joined #openstack-neutron | 19:29 | |
| *** jhershbe has quit IRC | 19:29 | |
| *** mvk has joined #openstack-neutron | 19:29 | |
| *** neophy has joined #openstack-neutron | 19:30 | |
| *** Leom_ has joined #openstack-neutron | 19:30 | |
| openstackgerrit | Ryan Tidwell proposed openstack/neutron: Add BGP Callback and agent RPC notifcation implementations https://review.openstack.org/216994 | 19:31 |
| *** Swami has quit IRC | 19:32 | |
| *** abhiraut has joined #openstack-neutron | 19:33 | |
| *** Leom has quit IRC | 19:33 | |
| *** itzikb has quit IRC | 19:34 | |
| *** safchain has joined #openstack-neutron | 19:35 | |
| *** ivar-lazzaro has quit IRC | 19:37 | |
| *** irenab_ has joined #openstack-neutron | 19:37 | |
| *** mickeys has quit IRC | 19:37 | |
| *** irenab has quit IRC | 19:38 | |
| *** irenab_ is now known as irenab | 19:38 | |
| *** tiswanso has quit IRC | 19:38 | |
| *** josecastroleon has quit IRC | 19:39 | |
| *** gberginc has joined #openstack-neutron | 19:43 | |
| *** krtaylor has joined #openstack-neutron | 19:44 | |
| *** ivar-lazzaro has joined #openstack-neutron | 19:45 | |
| *** yamamoto has joined #openstack-neutron | 19:48 | |
| dasm | rossella_s: ping. i'd like to ask about this bug, triaged by you: https://bugs.launchpad.net/neutron/+bug/1548193 and this one: https://bugs.launchpad.net/neutron/+bug/1548466 added by infra. | 19:48 |
| openstack | Launchpad bug 1548193 in neutron "Remove 'force_gateway_on_subnet' option" [Low,New] - Assigned to Sreekumar S (sreesiv) | 19:48 |
| openstack | Launchpad bug 1548466 in neutron " Deprecate 'force_gateway_on_subnet' configuration option" [Undecided,New] | 19:48 |
| dasm | For me it's duplication, but i'd like to confirm, if both mean the same. | 19:49 |
| *** ijw has joined #openstack-neutron | 19:49 | |
| *** jaugustine has quit IRC | 19:50 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/282485 | 19:51 |
| *** john-davidge has quit IRC | 19:51 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-fwaas: Updated from global requirements https://review.openstack.org/282486 | 19:51 |
| rossella_s | dasm, it's not a duplicate, the second one is about documentation | 19:52 |
| dasm | rossella_s: mhm. True. | 19:52 |
| *** reedip_ has joined #openstack-neutron | 19:53 | |
| amuller | armax: it's possible that https://review.openstack.org/#/c/277813/ is to blame | 19:53 |
| *** safchain has quit IRC | 19:54 | |
| armax | amuller: ok | 19:54 |
| *** localloop127 has quit IRC | 19:54 | |
| amuller | armax: I'll propose a revert, I have a theory that explains this | 19:54 |
| amuller | time to experiment! | 19:54 |
| armax | do you think there’s something that breaks in the collect_debug_info | 19:54 |
| armax | and sends the testrunner nuts? | 19:54 |
| armax | what if you wrap the method into a try-except? | 19:55 |
| armax | just as a way to understand this further? | 19:55 |
| *** baoli has quit IRC | 19:55 | |
| *** Marga_ has joined #openstack-neutron | 19:56 | |
| armax | amuller: have you considered it? | 19:56 |
| openstackgerrit | Reedip proposed openstack/neutron: Compress extended ipv6 address https://review.openstack.org/267283 | 19:56 |
| amuller | armax: let's revert, recheck a few times, I'll work on a solution locally in the mean time in parallel | 19:56 |
| armax | ok | 19:56 |
| *** mlavalle has joined #openstack-neutron | 19:57 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Revert "tests: Collect info on failure of conn_tester" https://review.openstack.org/283228 | 19:58 |
| amuller | armax: this is actually easy to verify. I'll just raise an exception during self.addOnException and see if it kills testr | 19:58 |
| *** hdaniel has joined #openstack-neutron | 19:59 | |
| armax | right, that’s what I suspect | 19:59 |
| armax | and by wrapping the collect_debug_info we should be able to see what’s going on | 19:59 |
| *** ivar-lazzaro has quit IRC | 20:00 | |
| *** localloop127 has joined #openstack-neutron | 20:00 | |
| haleyb | amuller: case of conntrack returning 1 ? | 20:00 |
| amuller | haleyb: I don't know why collect_debug_info is being run but I suspect that it's raising an exception which is making testr go on LSD and fail the run | 20:02 |
| *** john-davidge has joined #openstack-neutron | 20:02 | |
| ihrachys | amuller: do we even install conntrack for func tests? | 20:02 |
| haleyb | amuller: ok. i was just remembering we added extra_ok_codes=[1] in the conntrack manager code | 20:03 |
| *** tiswanso has joined #openstack-neutron | 20:04 | |
| amuller | ihrachys: devstack installs conntrack | 20:04 |
| amuller | ihrachys: not just for tests | 20:04 |
| *** changbl has joined #openstack-neutron | 20:04 | |
| ihrachys | amuller: afaiu only when q-l3 is on | 20:04 |
| ihrachys | amuller: since it's defined under files/rpms/q-l3 | 20:05 |
| amuller | ihrachys: I added it on that condition but IIRC nova required it before I added it | 20:05 |
| ihrachys | hm ok | 20:05 |
| *** tiswanso has quit IRC | 20:06 | |
| *** reedip_ has quit IRC | 20:06 | |
| amuller | ihrachys: https://github.com/openstack-dev/devstack/blob/d56b3ff028c192db0bbd0b07117ee7526bb1d4d1/files/rpms/nova#L1 | 20:06 |
| *** tiswanso has joined #openstack-neutron | 20:06 | |
| mlavalle | carl_baldwin: ping | 20:07 |
| amuller | armax: so yes, raising an exception in collect_debug_info causes testr to print that process-returncode error | 20:07 |
| amuller | armax: verified locally | 20:07 |
| amuller | armax: so the quickest fix would be to approve the revert then reintroduce it with a fix | 20:08 |
| *** ivar-lazzaro has joined #openstack-neutron | 20:11 | |
| *** abhiraut has quit IRC | 20:13 | |
| *** Sukhdev has joined #openstack-neutron | 20:14 | |
| *** ivar-lazzaro has quit IRC | 20:14 | |
| *** ivar-lazzaro has joined #openstack-neutron | 20:15 | |
| Sukhdev | Folks, Neutron meeting is today or tomorrow? | 20:18 |
| *** gberginc has quit IRC | 20:18 | |
| *** Leom_ has quit IRC | 20:20 | |
| haleyb | amuller: maybe i'm just grasping at straws, but is that second argument to iptables valid as "-nv" ? should it be '-n', '-v' ? maybe execute() is picky | 20:20 |
| *** lbragstad_ has quit IRC | 20:20 | |
| *** dasm has quit IRC | 20:20 | |
| *** gridinv has quit IRC | 20:20 | |
| *** cgoncalves has quit IRC | 20:20 | |
| *** itsuugo has quit IRC | 20:20 | |
| *** bharathm has quit IRC | 20:20 | |
| *** mnaser has quit IRC | 20:20 | |
| *** SpamapS has quit IRC | 20:20 | |
| *** _fortis has quit IRC | 20:20 | |
| *** sgran has quit IRC | 20:20 | |
| *** jklare has quit IRC | 20:20 | |
| *** doude has quit IRC | 20:20 | |
| *** yalie has quit IRC | 20:20 | |
| *** ababich has quit IRC | 20:20 | |
| *** leifmadsen has quit IRC | 20:20 | |
| *** apatil has quit IRC | 20:20 | |
| *** Guest40848 has quit IRC | 20:20 | |
| *** reedip has quit IRC | 20:20 | |
| *** mestery has quit IRC | 20:20 | |
| *** markmcclain has quit IRC | 20:20 | |
| *** partner has quit IRC | 20:20 | |
| *** ekarlso has quit IRC | 20:20 | |
| *** beagles has quit IRC | 20:20 | |
| *** dnovosel has quit IRC | 20:20 | |
| *** ritesh_ has quit IRC | 20:20 | |
| *** isq_ has quit IRC | 20:20 | |
| *** jamespd has quit IRC | 20:20 | |
| *** mestery has joined #openstack-neutron | 20:20 | |
| *** partner has joined #openstack-neutron | 20:20 | |
| *** doude has joined #openstack-neutron | 20:20 | |
| *** sgran has joined #openstack-neutron | 20:20 | |
| *** lbragstad_ has joined #openstack-neutron | 20:20 | |
| *** jklare has joined #openstack-neutron | 20:21 | |
| *** b3nt_pin has joined #openstack-neutron | 20:21 | |
| *** cgoncalves has joined #openstack-neutron | 20:21 | |
| *** apatil has joined #openstack-neutron | 20:21 | |
| *** ekarlso has joined #openstack-neutron | 20:21 | |
| *** leifmadsen has joined #openstack-neutron | 20:21 | |
| *** isq_ has joined #openstack-neutron | 20:21 | |
| *** ababich has joined #openstack-neutron | 20:21 | |
| *** Guest40848 has joined #openstack-neutron | 20:21 | |
| *** markmcclain has joined #openstack-neutron | 20:21 | |
| *** yalie has joined #openstack-neutron | 20:21 | |
| *** leifmadsen has quit IRC | 20:21 | |
| *** leifmadsen has joined #openstack-neutron | 20:21 | |
| *** bee2502_ has left #openstack-neutron | 20:21 | |
| *** mnaser has joined #openstack-neutron | 20:21 | |
| *** dnovosel has joined #openstack-neutron | 20:22 | |
| *** jamespd has joined #openstack-neutron | 20:22 | |
| *** itsuugo has joined #openstack-neutron | 20:22 | |
| *** SpamapS has joined #openstack-neutron | 20:22 | |
| amuller | haleyb: are you referring to code in collect_debug_info? | 20:22 |
| *** bharathm has joined #openstack-neutron | 20:22 | |
| *** gridinv has joined #openstack-neutron | 20:22 | |
| *** reedip has joined #openstack-neutron | 20:22 | |
| amuller | haleyb: the issue in that method is that the namespace needed to run iptables in is gone by the time the method is executed | 20:23 |
| haleyb | oh, that is bad as well :) | 20:23 |
| carl_baldwin | mlavalle: pong | 20:23 |
| mlavalle | carl_baldwin: when are you flying to Minneapolis? | 20:23 |
| haleyb | amuller: but yes, i was referring to that code | 20:24 |
| *** _fortis has joined #openstack-neutron | 20:24 | |
| amuller | haleyb: we use addOnException, I'm thinking maybe cleanUps are run before that | 20:24 |
| amuller | haleyb: so the namespace fixtures clean up... | 20:24 |
| carl_baldwin | mlavalle: pong | 20:24 |
| *** dedery has joined #openstack-neutron | 20:24 | |
| mlavalle | carl_baldwin: when are you flying to the mid-cycle? | 20:25 |
| carl_baldwin | mlavalle: 5:30 Mst | 20:25 |
| mlavalle | carl_baldwin: currently sitting at DEN. My flight is delayed. I am leaving in about an hour. Have a nice trip! | 20:26 |
| *** Leo_ has joined #openstack-neutron | 20:26 | |
| carl_baldwin | mlavalle: you too. See you there. | 20:26 |
| *** Leo_ has quit IRC | 20:27 | |
| *** john51_ has quit IRC | 20:27 | |
| *** nyechiel_ has quit IRC | 20:27 | |
| *** john51 has joined #openstack-neutron | 20:28 | |
| *** Leo_ has joined #openstack-neutron | 20:33 | |
| *** dasm has joined #openstack-neutron | 20:35 | |
| *** krotscheck has quit IRC | 20:37 | |
| *** jwarendt has quit IRC | 20:37 | |
| *** jobewan has quit IRC | 20:38 | |
| *** anteaya has quit IRC | 20:38 | |
| *** jwarendt has joined #openstack-neutron | 20:38 | |
| *** Swami has joined #openstack-neutron | 20:44 | |
| *** ajo_ has joined #openstack-neutron | 20:44 | |
| *** doug-fish has quit IRC | 20:45 | |
| *** krotscheck has joined #openstack-neutron | 20:46 | |
| *** openstackgerrit has quit IRC | 20:47 | |
| *** abhiraut has joined #openstack-neutron | 20:47 | |
| *** neophy has quit IRC | 20:48 | |
| *** ociuhandu has quit IRC | 20:48 | |
| *** openstackgerrit has joined #openstack-neutron | 20:48 | |
| *** anteaya has joined #openstack-neutron | 20:48 | |
| *** doug-fish has joined #openstack-neutron | 20:48 | |
| *** dosaboy_ has joined #openstack-neutron | 20:49 | |
| *** dosaboy_ has quit IRC | 20:49 | |
| *** dosaboy has quit IRC | 20:50 | |
| pc_m | Sukhdev: neither. IT was supposed to be today, but was canceled because of people traveling for the mid-cycle. | 20:50 |
| *** ajo_ has quit IRC | 20:50 | |
| *** dosaboy has joined #openstack-neutron | 20:50 | |
| *** ajo_ has joined #openstack-neutron | 20:50 | |
| *** javeriak_ has quit IRC | 20:51 | |
| *** doug-fish has quit IRC | 20:53 | |
| *** john-davidge has quit IRC | 20:53 | |
| *** nlahouti1 has joined #openstack-neutron | 20:53 | |
| *** nlahouti has quit IRC | 20:53 | |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron-specs: Agent specific API for L2 agent extensions https://review.openstack.org/263819 | 20:54 |
| *** thumpba has quit IRC | 20:55 | |
| *** thumpba has joined #openstack-neutron | 20:56 | |
| *** nlahouti1 has quit IRC | 20:56 | |
| *** Kiall has quit IRC | 21:00 | |
| *** Kiall has joined #openstack-neutron | 21:00 | |
| *** thumpba has quit IRC | 21:00 | |
| *** baoli has joined #openstack-neutron | 21:00 | |
| *** Kiall has quit IRC | 21:00 | |
| *** Kiall has joined #openstack-neutron | 21:01 | |
| *** baoli has quit IRC | 21:01 | |
| *** iyamahat has quit IRC | 21:01 | |
| *** baoli has joined #openstack-neutron | 21:02 | |
| *** yamahata has quit IRC | 21:02 | |
| *** vhoward has quit IRC | 21:08 | |
| *** changbl has quit IRC | 21:08 | |
| *** vhoward has joined #openstack-neutron | 21:08 | |
| *** vhoward has quit IRC | 21:09 | |
| *** Kiall has quit IRC | 21:09 | |
| *** vhoward has joined #openstack-neutron | 21:09 | |
| *** Kiall has joined #openstack-neutron | 21:10 | |
| *** Sukhdev has quit IRC | 21:13 | |
| *** mlavalle has quit IRC | 21:14 | |
| *** gberginc has joined #openstack-neutron | 21:15 | |
| *** zhhuabj_ has joined #openstack-neutron | 21:16 | |
| *** ijw has quit IRC | 21:17 | |
| *** crose has quit IRC | 21:17 | |
| amuller | armax: am I crazy or...? https://review.openstack.org/#/c/282559/3/lib/neutron-legacy | 21:17 |
| *** ajo_ has quit IRC | 21:17 | |
| *** slaweq_ has joined #openstack-neutron | 21:17 | |
| *** Nakato has quit IRC | 21:17 | |
| *** ajo_ has joined #openstack-neutron | 21:18 | |
| *** jamielennox has quit IRC | 21:19 | |
| *** ijw has joined #openstack-neutron | 21:19 | |
| *** iyamahat has joined #openstack-neutron | 21:19 | |
| *** Nakato has joined #openstack-neutron | 21:19 | |
| *** jamielennox has joined #openstack-neutron | 21:20 | |
| *** yamahata has joined #openstack-neutron | 21:20 | |
| *** ihrachys has quit IRC | 21:20 | |
| *** baoli has quit IRC | 21:21 | |
| *** doug-fish has joined #openstack-neutron | 21:22 | |
| armax | amuller yes, you’re crazy | 21:31 |
| *** doug-fish has quit IRC | 21:31 | |
| armax | :) | 21:31 |
| amuller | armax: I knew it :( | 21:32 |
| *** mickeys has joined #openstack-neutron | 21:41 | |
| *** baoli has joined #openstack-neutron | 21:42 | |
| *** rpothier has quit IRC | 21:43 | |
| *** rickyrem has quit IRC | 21:44 | |
| *** zhhuabj_ has quit IRC | 21:44 | |
| *** MCoLo has quit IRC | 21:44 | |
| *** ajo_ has quit IRC | 21:46 | |
| *** ajo_ has joined #openstack-neutron | 21:47 | |
| *** changbl has joined #openstack-neutron | 21:47 | |
| *** edmondsw has quit IRC | 21:48 | |
| *** gberginc has quit IRC | 21:49 | |
| *** MCoLo has joined #openstack-neutron | 21:50 | |
| *** itisha has joined #openstack-neutron | 21:51 | |
| *** thorst has quit IRC | 21:53 | |
| *** nlahouti has joined #openstack-neutron | 21:53 | |
| *** ajo_ has quit IRC | 21:54 | |
| *** ajo_ has joined #openstack-neutron | 21:55 | |
| openstackgerrit | Claudiu Belu proposed openstack/neutron: WIP: Adds NVGRE network type https://review.openstack.org/283276 | 21:55 |
| openstackgerrit | Madhusudhan Kandadai proposed openstack/neutron-fwaas: Modify tox.ini to run dsvm-functional tests https://review.openstack.org/283277 | 21:57 |
| openstackgerrit | Madhusudhan Kandadai proposed openstack/neutron-fwaas: Select the right DB to store user and data in each backend https://review.openstack.org/279339 | 21:59 |
| *** thorst has joined #openstack-neutron | 21:59 | |
| *** Sukhdev has joined #openstack-neutron | 21:59 | |
| *** apuimedo has joined #openstack-neutron | 22:01 | |
| *** thumpba has joined #openstack-neutron | 22:03 | |
| *** tflynn has quit IRC | 22:03 | |
| *** cbouch has quit IRC | 22:03 | |
| *** thorst has quit IRC | 22:04 | |
| *** sc68cal has quit IRC | 22:04 | |
| *** ParsectiX has joined #openstack-neutron | 22:05 | |
| *** ParsectiX has quit IRC | 22:05 | |
| *** ajo_ has quit IRC | 22:05 | |
| *** ajo_ has joined #openstack-neutron | 22:05 | |
| Sukhdev | pc_m: thanks for the update - | 22:06 |
| *** changbl has quit IRC | 22:06 | |
| *** armax has quit IRC | 22:06 | |
| *** rtheis has quit IRC | 22:07 | |
| *** vhoward has quit IRC | 22:09 | |
| *** ajo_ has quit IRC | 22:10 | |
| *** matrohon has joined #openstack-neutron | 22:10 | |
| openstackgerrit | Slawek Kaplonski proposed openstack/neutron: Add support for QoS for LinuxBridge agent https://review.openstack.org/236210 | 22:13 |
| *** thumpba has quit IRC | 22:13 | |
| *** thumpba has joined #openstack-neutron | 22:13 | |
| *** hdaniel has quit IRC | 22:15 | |
| *** thorst has joined #openstack-neutron | 22:16 | |
| *** krtaylor has quit IRC | 22:17 | |
| *** thorst has quit IRC | 22:17 | |
| *** julim has quit IRC | 22:18 | |
| *** thorst has joined #openstack-neutron | 22:18 | |
| *** abhiraut has quit IRC | 22:19 | |
| *** regXboi has quit IRC | 22:19 | |
| *** abhiraut has joined #openstack-neutron | 22:23 | |
| *** john-davidge has joined #openstack-neutron | 22:23 | |
| *** thumpba has quit IRC | 22:23 | |
| *** neelashah1 has quit IRC | 22:26 | |
| *** tpsilva has quit IRC | 22:27 | |
| *** wolverineav has quit IRC | 22:27 | |
| *** thumpba has joined #openstack-neutron | 22:28 | |
| *** wolverineav has joined #openstack-neutron | 22:29 | |
| *** localloop127 has quit IRC | 22:30 | |
| *** sc68cal has joined #openstack-neutron | 22:30 | |
| *** sc68cal has quit IRC | 22:35 | |
| *** irenab_ has joined #openstack-neutron | 22:35 | |
| *** insequent has quit IRC | 22:36 | |
| *** irenab has quit IRC | 22:37 | |
| *** irenab_ is now known as irenab | 22:37 | |
| *** dingboopt has joined #openstack-neutron | 22:39 | |
| *** gangil1 has joined #openstack-neutron | 22:40 | |
| *** gangil has quit IRC | 22:41 | |
| *** banix has quit IRC | 22:43 | |
| *** gberginc has joined #openstack-neutron | 22:46 | |
| *** Sukhdev has quit IRC | 22:46 | |
| *** Leo_ has quit IRC | 22:47 | |
| *** marcusvrn_ has quit IRC | 22:47 | |
| *** Leo_ has joined #openstack-neutron | 22:47 | |
| *** azbiswas has quit IRC | 22:49 | |
| *** azbiswas has joined #openstack-neutron | 22:49 | |
| *** haplo37 has quit IRC | 22:49 | |
| *** allan_h has quit IRC | 22:50 | |
| *** allan_h has joined #openstack-neutron | 22:52 | |
| *** araji has quit IRC | 22:53 | |
| *** azbiswas has quit IRC | 22:54 | |
| *** sc68cal has joined #openstack-neutron | 22:55 | |
| *** kbringard has quit IRC | 22:57 | |
| *** krtaylor has joined #openstack-neutron | 22:58 | |
| *** singhj1 has quit IRC | 23:01 | |
| *** eil397 has quit IRC | 23:01 | |
| *** Sukhdev has joined #openstack-neutron | 23:03 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Revert "tests: Collect info on failure of conn_tester" https://review.openstack.org/283228 | 23:03 |
| *** slaweq_ has quit IRC | 23:03 | |
| *** jckasper_ has quit IRC | 23:03 | |
| *** jckasper has joined #openstack-neutron | 23:04 | |
| *** dims has joined #openstack-neutron | 23:04 | |
| *** Leo_ has quit IRC | 23:04 | |
| amuller | haleyb: HenryG: ^ | 23:04 |
| amuller | I'd like to merge that ASAP to unwedge the gate | 23:05 |
| haleyb | amuller: looking | 23:05 |
| HenryG | amuller: done | 23:05 |
| *** shwetaap has quit IRC | 23:07 | |
| haleyb | amuller: done, assuming jenkins likes it | 23:07 |
| amuller | HenryG: haleyb: Thanks fellas | 23:07 |
| *** shwetaap has joined #openstack-neutron | 23:07 | |
| *** shwetaap has quit IRC | 23:07 | |
| *** eil397 has joined #openstack-neutron | 23:07 | |
| *** salv-orl_ has quit IRC | 23:08 | |
| *** jckasper has quit IRC | 23:08 | |
| amuller | sc68cal: what do you think about adding l3 agent processes to netns-cleanup? | 23:10 |
| amuller | sc68cal: like radvd and keepalived | 23:10 |
| HenryG | amuller: who runs devstack with more than four tenants? ;) | 23:10 |
| sc68cal | lol | 23:11 |
| *** salv-orlando has joined #openstack-neutron | 23:11 | |
| amuller | CI I guess =p | 23:11 |
| *** haleyb has quit IRC | 23:11 | |
| sc68cal | amuller: not familiar with netns-cleanup | 23:11 |
| sc68cal | not finding anything with git grep | 23:11 |
| amuller | sc68cal: neutron/cmd/netns_cleanup | 23:12 |
| amuller | sc68cal: we use it in RDO | 23:12 |
| sc68cal | ah | 23:12 |
| amuller | sc68cal: pretty handy for development also | 23:12 |
| * sc68cal was looking in neutron/tools | 23:12 | |
| sc68cal | amuller: yeah if you want to replace all the devstack goop with calls to netns_cleanup or other neutron commands - that'd be great | 23:13 |
| *** allan_h has quit IRC | 23:14 | |
| *** allan_h has joined #openstack-neutron | 23:16 | |
| *** eil397 has quit IRC | 23:16 | |
| *** srampal has quit IRC | 23:16 | |
| *** allan_h has quit IRC | 23:17 | |
| *** gberginc has quit IRC | 23:19 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Fix generate_records_for_existing in migrations https://review.openstack.org/281533 | 23:20 |
| *** john-davidge has quit IRC | 23:22 | |
| *** jckasper has joined #openstack-neutron | 23:26 | |
| *** eil397 has joined #openstack-neutron | 23:27 | |
| *** jckasper has quit IRC | 23:31 | |
| *** eil397 has quit IRC | 23:31 | |
| *** eil397 has joined #openstack-neutron | 23:32 | |
| *** vhosakot has quit IRC | 23:32 | |
| openstackgerrit | Merged openstack/neutron: Updated from global requirements https://review.openstack.org/282485 | 23:33 |
| *** armax has joined #openstack-neutron | 23:34 | |
| *** pradk has quit IRC | 23:36 | |
| *** djan has joined #openstack-neutron | 23:37 | |
| *** mdih has quit IRC | 23:38 | |
| *** sdague has quit IRC | 23:39 | |
| amuller | armax: ping, I have a Q about RFEs, specs and blueprints | 23:39 |
| *** john-davidge has joined #openstack-neutron | 23:40 | |
| armax | amuller: shoot | 23:40 |
| *** allan_h has joined #openstack-neutron | 23:40 | |
| amuller | armax: if I want a list of features merged so far during Mitaka, would a launchpad query for rfes suffice? Is it guaranteed that any blueprint also had an RFE bug, and that any spec also has an RFE bug? | 23:41 |
| amuller | armax: or is there another way to get this list? | 23:41 |
| armax | amuller: https://launchpad.net/neutron/+milestone/mitaka-? with ? being 1 or 2 should give you what you look for | 23:42 |
| armax | as for M-3, there’s stuff that’s either implemented/released | 23:42 |
| armax | what are you looking for exactly | 23:42 |
| armax | ? | 23:42 |
| amuller | armax: a list of features merged during the Mitaka cycle, regardless if we used an RFE bug to track it, a blueprint, or a spec | 23:43 |
| amuller | armax: or any combination thereof | 23:43 |
| armax | amuller: it’s funny you mention that…I was actually working on something like this myself | 23:43 |
| armax | amuller: in preparation of the RC window | 23:43 |
| armax | I don’t have anything worth sharing yet…but it was going to be something posted against the specs repo | 23:44 |
| amuller | armax: if we track every single feature with an RFE bug, a launchpad query would suffice... the question is then if there's any features tracked via blueprint/spec but does not have a corresponding RFE bug | 23:44 |
| armax | amuller: btw yes | 23:44 |
| amuller | armax: we really just need a consistent way to track features | 23:44 |
| armax | amuller: I think there may still be exceptions of bp’s without an associated rfe bug | 23:44 |
| armax | but that’s for the stuff that was in flight prior | 23:45 |
| kevinbenton | amuller: WRT to that revert, did you confirm that the failure to collect diagnostics was because the namespace was deleted already? | 23:45 |
| amuller | kevinbenton: the error was always that the namespace does not exist | 23:45 |
| amuller | kevinbenton: in every trace I saw | 23:45 |
| kevinbenton | amuller: ok | 23:45 |
| kevinbenton | amuller: i suspect the easy fix is to move the addOnException above the super setup call | 23:45 |
| kevinbenton | amuller: but i'm confirming some internals in testtools right | 23:45 |
| kevinbenton | now | 23:45 |
| amuller | kevinbenton: you don't have to deal with that | 23:46 |
| amuller | kevinbenton: Jakub will :) | 23:46 |
| kevinbenton | ok, i'm going to push a patch that he can put his on top of | 23:46 |
| kevinbenton | amuller: this addOnException is unsafe | 23:47 |
| kevinbenton | amuller: so i'm going to override it to make it safe in our base test class | 23:47 |
| armax | amuller: what are you trying to do wrt the rfe's? | 23:47 |
| armax | amuller: I wonder if I should sync up with you before I go ahead doing what I am doing | 23:47 |
| *** miyagishi_t has joined #openstack-neutron | 23:47 | |
| amuller | armax: just look at the list for now, see that I didn't miss anything | 23:48 |
| amuller | armax: preparing for a meeting with red hat qa tbh | 23:48 |
| armax | amuller: another view that might be of interest is this one: | 23:48 |
| armax | https://blueprints.launchpad.net/neutron/mitaka/+assignments | 23:48 |
| armax | even though it doesn’t include the RFE bugs | 23:48 |
| *** subscope has quit IRC | 23:50 | |
| amuller | kevinbenton: you're truly obsessive | 23:50 |
| *** azbiswas has joined #openstack-neutron | 23:50 | |
| amuller | kevinbenton: I mean that in the sweetest way possible :) | 23:50 |
| amuller | armax: thank you | 23:50 |
| *** matrohon has quit IRC | 23:50 | |
| kevinbenton | amuller: well i'm upset because i told him to use that function :) | 23:51 |
| kevinbenton | amuller: and it turns out that function makes assumptions i didn't think it made | 23:51 |
| amuller | kevinbenton: and I approved it didn't I? | 23:51 |
| amuller | kevinbenton: yeah it's surprising behavior definitely | 23:51 |
| *** ociuhandu has joined #openstack-neutron | 23:53 | |
| *** Sukhdev has quit IRC | 23:54 | |
| *** azbiswas has quit IRC | 23:54 | |
| *** Sukhdev has joined #openstack-neutron | 23:55 | |
| *** dane_leblanc has quit IRC | 23:56 | |
| *** Leo_ has joined #openstack-neutron | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!