*** SumitNaiksatam has quit IRC | 00:00 | |
*** yamamoto has joined #openstack-neutron | 00:01 | |
*** absubram has joined #openstack-neutron | 00:01 | |
*** srilatha has quit IRC | 00:01 | |
*** yongshenggong_ has quit IRC | 00:01 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: CONF.reset does not happen for all tests https://review.openstack.org/219878 | 00:02 |
---|---|---|
*** emagana has quit IRC | 00:03 | |
*** vikram__ has quit IRC | 00:04 | |
*** mchestr has quit IRC | 00:06 | |
*** thurloat_isgone is now known as thurloat | 00:11 | |
*** sthillma_ has joined #openstack-neutron | 00:11 | |
*** rushil_ has quit IRC | 00:13 | |
*** sthillma has quit IRC | 00:14 | |
*** sthillma_ is now known as sthillma | 00:14 | |
*** Swami has quit IRC | 00:14 | |
*** rushil has joined #openstack-neutron | 00:14 | |
*** Swami_ has quit IRC | 00:14 | |
*** pavel_bondar has quit IRC | 00:14 | |
*** Swami__ has joined #openstack-neutron | 00:14 | |
*** Swami has joined #openstack-neutron | 00:14 | |
*** pavel_bondar has joined #openstack-neutron | 00:15 | |
*** tobasco_ has quit IRC | 00:15 | |
*** tobasco has joined #openstack-neutron | 00:16 | |
*** vikram__ has joined #openstack-neutron | 00:16 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Deprecate external_network_bridge option in L3 agent https://review.openstack.org/219928 | 00:17 |
*** banix has quit IRC | 00:17 | |
*** padkrish has joined #openstack-neutron | 00:19 | |
*** goodygum has quit IRC | 00:19 | |
*** goodygum has joined #openstack-neutron | 00:21 | |
*** thorst has joined #openstack-neutron | 00:21 | |
*** padkrish has quit IRC | 00:23 | |
*** shadower has quit IRC | 00:23 | |
*** padkrish has joined #openstack-neutron | 00:23 | |
*** shadower has joined #openstack-neutron | 00:23 | |
*** thorst has quit IRC | 00:23 | |
*** outworlder has quit IRC | 00:24 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Retain logs from functional test runs https://review.openstack.org/219935 | 00:24 |
*** banix has joined #openstack-neutron | 00:25 | |
*** Swami__ has quit IRC | 00:25 | |
*** Swami has quit IRC | 00:25 | |
*** Aish has quit IRC | 00:26 | |
*** yalie has quit IRC | 00:26 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Retain logs from functional test runs https://review.openstack.org/219935 | 00:27 |
*** vikram__ has quit IRC | 00:27 | |
*** achanda has joined #openstack-neutron | 00:28 | |
*** abhiraut has quit IRC | 00:28 | |
*** jbell8 has joined #openstack-neutron | 00:29 | |
*** Swami has joined #openstack-neutron | 00:30 | |
*** moshele has quit IRC | 00:31 | |
*** miyagishi_t has joined #openstack-neutron | 00:31 | |
*** nlahouti has quit IRC | 00:32 | |
*** abhiraut has joined #openstack-neutron | 00:33 | |
*** sthillma_ has joined #openstack-neutron | 00:34 | |
*** sthillma has quit IRC | 00:37 | |
*** sthillma_ is now known as sthillma | 00:37 | |
*** nlahouti has joined #openstack-neutron | 00:37 | |
*** Sukhdev_ has joined #openstack-neutron | 00:37 | |
*** vikram__ has joined #openstack-neutron | 00:39 | |
*** tiswanso has joined #openstack-neutron | 00:39 | |
*** ameybhide has quit IRC | 00:40 | |
*** nlahouti has quit IRC | 00:40 | |
*** shivrao has quit IRC | 00:41 | |
*** nlahouti has joined #openstack-neutron | 00:42 | |
*** roeyc has joined #openstack-neutron | 00:42 | |
*** nlahouti has quit IRC | 00:42 | |
*** sthillma has quit IRC | 00:44 | |
*** armax has quit IRC | 00:46 | |
*** pritesh has quit IRC | 00:47 | |
*** armax has joined #openstack-neutron | 00:47 | |
*** achanda has quit IRC | 00:51 | |
*** achanda has joined #openstack-neutron | 00:52 | |
*** achanda has quit IRC | 00:52 | |
*** banix has quit IRC | 00:54 | |
openstackgerrit | stephen-ma proposed openstack/neutron: Descheduling DVR routers when ports are unbound from VM https://review.openstack.org/219067 | 00:54 |
*** shoutm_ has joined #openstack-neutron | 00:56 | |
*** shoutm has quit IRC | 00:56 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Retain logs from functional test runs https://review.openstack.org/219935 | 00:58 |
*** hichihara has joined #openstack-neutron | 00:59 | |
openstackgerrit | Emett Speer proposed openstack/neutron-specs: [Labs] Update links to Cloud Admin Guide https://review.openstack.org/219942 | 00:59 |
*** rushil has quit IRC | 01:02 | |
*** fzdarsky_ has joined #openstack-neutron | 01:02 | |
*** zzzeek has joined #openstack-neutron | 01:03 | |
*** vikram__ has quit IRC | 01:04 | |
*** fzdarsky has quit IRC | 01:05 | |
*** julim has joined #openstack-neutron | 01:05 | |
*** rossella_s has quit IRC | 01:06 | |
*** rossella_s has joined #openstack-neutron | 01:06 | |
*** ducnc has joined #openstack-neutron | 01:07 | |
*** SumitNaiksatam has joined #openstack-neutron | 01:08 | |
*** thurloat is now known as thurloat_isgone | 01:09 | |
*** namptit has joined #openstack-neutron | 01:12 | |
*** shivrao has joined #openstack-neutron | 01:13 | |
*** tiswanso has quit IRC | 01:13 | |
*** tiswanso has joined #openstack-neutron | 01:13 | |
*** namptit has quit IRC | 01:14 | |
*** namptit has joined #openstack-neutron | 01:15 | |
*** namptit has quit IRC | 01:15 | |
*** julim has quit IRC | 01:15 | |
*** banix has joined #openstack-neutron | 01:16 | |
*** shivrao_ has joined #openstack-neutron | 01:16 | |
*** vikram__ has joined #openstack-neutron | 01:16 | |
*** vivek-ebay has quit IRC | 01:17 | |
*** shivrao has quit IRC | 01:18 | |
*** shivrao_ is now known as shivrao | 01:18 | |
*** erkules has joined #openstack-neutron | 01:19 | |
*** jwarendt has quit IRC | 01:20 | |
*** ijw__ has quit IRC | 01:20 | |
*** julim has joined #openstack-neutron | 01:20 | |
*** erkules_ has quit IRC | 01:22 | |
*** yushiro has quit IRC | 01:23 | |
*** abhiraut has quit IRC | 01:23 | |
*** ociuhandu has joined #openstack-neutron | 01:25 | |
*** padkrish has quit IRC | 01:26 | |
*** padkrish has joined #openstack-neutron | 01:26 | |
*** tflynn has joined #openstack-neutron | 01:27 | |
*** nplanel has joined #openstack-neutron | 01:27 | |
*** tflynn has quit IRC | 01:31 | |
*** padkrish has quit IRC | 01:31 | |
*** sbalukoff has quit IRC | 01:32 | |
*** vikram__ has quit IRC | 01:34 | |
*** sdake has quit IRC | 01:36 | |
*** sridhar_ram has quit IRC | 01:36 | |
*** sballe has quit IRC | 01:36 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Retain logs from functional test runs https://review.openstack.org/219935 | 01:39 |
*** Swami has quit IRC | 01:44 | |
*** achanda has joined #openstack-neutron | 01:44 | |
*** Aish has joined #openstack-neutron | 01:45 | |
*** vikram__ has joined #openstack-neutron | 01:45 | |
*** julim has quit IRC | 01:47 | |
*** minwang2 has quit IRC | 01:48 | |
*** Aish has left #openstack-neutron | 01:49 | |
*** boris-42 has quit IRC | 01:50 | |
*** banix has quit IRC | 01:51 | |
*** jbell8 has quit IRC | 01:55 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Retain logs from functional test runs https://review.openstack.org/219935 | 01:56 |
*** ybabenko has joined #openstack-neutron | 01:57 | |
*** fzdarsky__ has joined #openstack-neutron | 01:59 | |
*** ybabenko has quit IRC | 02:01 | |
*** fzdarsky_ has quit IRC | 02:02 | |
*** vikram__ has quit IRC | 02:04 | |
*** SumitNaiksatam has quit IRC | 02:05 | |
*** baoli has quit IRC | 02:06 | |
*** baoli has joined #openstack-neutron | 02:06 | |
*** jwy has quit IRC | 02:06 | |
*** SumitNaiksatam has joined #openstack-neutron | 02:07 | |
*** baoli has quit IRC | 02:07 | |
*** nihilifer has quit IRC | 02:07 | |
*** baoli has joined #openstack-neutron | 02:08 | |
*** nihilifer has joined #openstack-neutron | 02:08 | |
*** zzzeek has quit IRC | 02:09 | |
*** mickeys has quit IRC | 02:10 | |
*** padkrish has joined #openstack-neutron | 02:10 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Retain logs from functional test runs https://review.openstack.org/219935 | 02:11 |
*** lyanchih has joined #openstack-neutron | 02:12 | |
*** armax has quit IRC | 02:12 | |
*** iwamoto has joined #openstack-neutron | 02:12 | |
*** Leonr_ has quit IRC | 02:14 | |
*** Leonr has joined #openstack-neutron | 02:14 | |
*** banix has joined #openstack-neutron | 02:16 | |
*** vikram__ has joined #openstack-neutron | 02:16 | |
*** spandhe has quit IRC | 02:17 | |
*** banix has quit IRC | 02:17 | |
*** bharathm has quit IRC | 02:19 | |
iwamoto | kevinbenton: ping | 02:20 |
*** padkrish has quit IRC | 02:21 | |
*** madhu_ak has quit IRC | 02:23 | |
kevinbenton | iwamoto: pong | 02:25 |
iwamoto | kevinbenton: hi | 02:25 |
kevinbenton | iwamoto: what's up? | 02:26 |
iwamoto | I'd ask your +2 on https://review.openstack.org/153946 | 02:26 |
*** shwetaap has joined #openstack-neutron | 02:26 | |
*** julim has joined #openstack-neutron | 02:26 | |
*** banix has joined #openstack-neutron | 02:27 | |
*** Leonr has quit IRC | 02:27 | |
*** vikram__ has quit IRC | 02:28 | |
*** sdake has joined #openstack-neutron | 02:28 | |
*** Kennan has quit IRC | 02:29 | |
*** Kennan has joined #openstack-neutron | 02:30 | |
*** sdake_ has joined #openstack-neutron | 02:31 | |
*** roeyc has quit IRC | 02:31 | |
*** Sukhdev_ has quit IRC | 02:32 | |
*** jbell8 has joined #openstack-neutron | 02:32 | |
kevinbenton | iwamoto: done | 02:33 |
iwamoto | kevinbenton: thanks a lot | 02:33 |
iwamoto | kevinbenton: I need another +2. To whom should I ask? Any idea? | 02:34 |
*** sdake has quit IRC | 02:34 | |
*** vivek-ebay has joined #openstack-neutron | 02:35 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: DO NOT MERGE: OVS-agent: Switch the default to "native" of_interface https://review.openstack.org/206329 | 02:38 |
*** vikram__ has joined #openstack-neutron | 02:40 | |
*** bharathm has joined #openstack-neutron | 02:41 | |
*** kiran-r has joined #openstack-neutron | 02:43 | |
*** saksham has quit IRC | 02:44 | |
*** shivrao has quit IRC | 02:44 | |
*** shivrao has joined #openstack-neutron | 02:45 | |
*** armax has joined #openstack-neutron | 02:49 | |
openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Add availability_zone support for Scheduler https://review.openstack.org/204436 | 02:52 |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Retain logs from functional test runs https://review.openstack.org/219935 | 02:53 |
*** hakimo_ has quit IRC | 02:54 | |
*** vikram__ has quit IRC | 02:55 | |
*** minwang2 has joined #openstack-neutron | 02:55 | |
*** hakimo has joined #openstack-neutron | 02:57 | |
*** kiran-r has quit IRC | 02:58 | |
*** rushil has joined #openstack-neutron | 03:07 | |
*** vikram__ has joined #openstack-neutron | 03:07 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Retain logs from functional test runs https://review.openstack.org/219935 | 03:11 |
*** tflynn has joined #openstack-neutron | 03:12 | |
*** vikram__ has quit IRC | 03:13 | |
*** fedexo has joined #openstack-neutron | 03:14 | |
*** rushil has quit IRC | 03:17 | |
kevinbenton | iwamoto: Miguel Ajo | 03:19 |
kevinbenton | iwamoto: or see if armax is willing to bump to +2 | 03:19 |
*** banix has quit IRC | 03:19 | |
iwamoto | kevinbenton: thanks. I'll ask them :) | 03:20 |
*** shoutm_ has quit IRC | 03:20 | |
iwamoto | ajo, armax: ping | 03:20 |
*** tiswanso has quit IRC | 03:23 | |
*** shashank_hegde has joined #openstack-neutron | 03:23 | |
*** vikram__ has joined #openstack-neutron | 03:25 | |
*** Sukhdev has quit IRC | 03:26 | |
*** Sukhdev has joined #openstack-neutron | 03:27 | |
*** mickeys has joined #openstack-neutron | 03:27 | |
*** Samos123 has quit IRC | 03:27 | |
*** vishwanathj has quit IRC | 03:28 | |
*** achanda has quit IRC | 03:28 | |
*** bobmel has joined #openstack-neutron | 03:29 | |
*** erhudy1 has quit IRC | 03:29 | |
*** dguerri` has quit IRC | 03:30 | |
*** jroll has quit IRC | 03:30 | |
*** lyanchih_ has joined #openstack-neutron | 03:30 | |
*** lyanchih has quit IRC | 03:31 | |
*** lyanchih_ is now known as lyanchih | 03:31 | |
*** clev-away has quit IRC | 03:31 | |
*** bobmel_ has quit IRC | 03:32 | |
*** fedexo has quit IRC | 03:32 | |
*** dguerri` has joined #openstack-neutron | 03:32 | |
*** clev has joined #openstack-neutron | 03:32 | |
*** dguerri` is now known as dguerri | 03:32 | |
*** dguerri has joined #openstack-neutron | 03:32 | |
*** vikram__ has quit IRC | 03:32 | |
*** fedexo has joined #openstack-neutron | 03:32 | |
*** jroll has joined #openstack-neutron | 03:33 | |
*** Samos123 has joined #openstack-neutron | 03:33 | |
*** adam_g has quit IRC | 03:33 | |
*** adam_g has joined #openstack-neutron | 03:35 | |
*** adam_g has quit IRC | 03:35 | |
*** adam_g has joined #openstack-neutron | 03:35 | |
*** dims has joined #openstack-neutron | 03:35 | |
*** shashank_hegde_ has joined #openstack-neutron | 03:40 | |
*** shashank_hegde has quit IRC | 03:40 | |
*** shashank_hegde_ is now known as shashank_hegde | 03:40 | |
*** fedexo has quit IRC | 03:41 | |
*** shoutm has joined #openstack-neutron | 03:41 | |
*** sdake_ is now known as sdake | 03:42 | |
*** deepthi has joined #openstack-neutron | 03:42 | |
*** shaohe_feng has joined #openstack-neutron | 03:44 | |
*** vikram__ has joined #openstack-neutron | 03:45 | |
*** tflynn_ has joined #openstack-neutron | 03:45 | |
*** tflynn has quit IRC | 03:48 | |
*** vikram__ has quit IRC | 03:50 | |
*** SridharG has joined #openstack-neutron | 03:52 | |
*** shivrao has quit IRC | 03:53 | |
*** julim has quit IRC | 03:54 | |
openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Add functional test for availability_zone support https://review.openstack.org/210977 | 03:55 |
*** boris-42 has joined #openstack-neutron | 03:59 | |
*** dims has quit IRC | 04:00 | |
*** links has joined #openstack-neutron | 04:02 | |
*** vikram__ has joined #openstack-neutron | 04:02 | |
openstackgerrit | Sridhar Gaddam proposed openstack/neutron: Add IPv6 Address Resolution protection https://review.openstack.org/201650 | 04:03 |
openstackgerrit | Merged openstack/neutron: DHCP agent: allow using gateway IPs instead of uniquely allocated https://review.openstack.org/206078 | 04:05 |
*** Sukhdev_ has joined #openstack-neutron | 04:06 | |
*** vikram__ has quit IRC | 04:08 | |
*** amotoki has joined #openstack-neutron | 04:11 | |
*** baojg has joined #openstack-neutron | 04:11 | |
*** baoli has quit IRC | 04:14 | |
*** shaohe_feng has quit IRC | 04:14 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements https://review.openstack.org/219496 | 04:15 |
*** mickeys has quit IRC | 04:16 | |
*** slaweq has joined #openstack-neutron | 04:16 | |
*** vikram__ has joined #openstack-neutron | 04:20 | |
openstackgerrit | Sergey Vilgelm proposed openstack/neutron: Fix a wrong condition for the _purge_metering_info function https://review.openstack.org/218885 | 04:21 |
yongshenggong__ | ping amotoki | 04:22 |
*** vishwanathj has joined #openstack-neutron | 04:22 | |
yongshenggong__ | https://review.openstack.org/#/c/139928/ the availability zone spec rest is not approved yet. | 04:23 |
*** gsagie_ has joined #openstack-neutron | 04:23 | |
*** minwang2 has quit IRC | 04:24 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Retain logs for functional test cases https://review.openstack.org/219935 | 04:25 |
*** slaweq has quit IRC | 04:25 | |
*** vikram__ has quit IRC | 04:26 | |
*** carl_baldwin has joined #openstack-neutron | 04:26 | |
*** SridharG has quit IRC | 04:26 | |
*** yfried has quit IRC | 04:31 | |
*** baojg has quit IRC | 04:31 | |
*** lyanchih has quit IRC | 04:32 | |
*** yalie has joined #openstack-neutron | 04:32 | |
*** garyk has quit IRC | 04:35 | |
hichihara | yongshenggong__: Please look at https://review.openstack.org/#/c/169612/ | 04:35 |
*** SridharG has joined #openstack-neutron | 04:35 | |
yongshenggong__ | hichihara: thanks, I will review that patch. | 04:36 |
hichihara | yongshenggong__: Thank you for your help. | 04:36 |
yongshenggong__ | hichihara: I like the ideas to improve the agents and related scheduler, which is my initial idea. thanks for the improvement. | 04:37 |
*** shoutm has quit IRC | 04:38 | |
*** vikram__ has joined #openstack-neutron | 04:38 | |
yongshenggong__ | hi guys, who know how to upgrade the pip version in tox create virtual env? | 04:39 |
hichihara | yongshenggong__: I'm happy :) | 04:40 |
yongshenggong__ | me too :) | 04:42 |
*** moshele has joined #openstack-neutron | 04:43 | |
*** vikram__ has quit IRC | 04:43 | |
*** carl_baldwin has quit IRC | 04:43 | |
*** kobayashi has joined #openstack-neutron | 04:44 | |
*** shivrao has joined #openstack-neutron | 04:44 | |
*** anilvenkata has joined #openstack-neutron | 04:44 | |
gus | yongshenggong__: .tox/$env/bin/pip install --upgrade pip | 04:46 |
*** shoutm has joined #openstack-neutron | 04:47 | |
yalie | ping armax | 04:48 |
yalie | armax: ping | 04:48 |
*** shwetaap has quit IRC | 04:48 | |
*** BhavyaM has joined #openstack-neutron | 04:48 | |
*** gsagie_ has quit IRC | 04:49 | |
*** vikram has joined #openstack-neutron | 04:49 | |
*** moshele has quit IRC | 04:50 | |
*** minwang2 has joined #openstack-neutron | 04:51 | |
HenryG | armax: ping | 04:52 |
*** anilvenkata has quit IRC | 04:52 | |
HenryG | CONF.reset ... | 04:52 |
*** dannywilson has joined #openstack-neutron | 04:52 | |
*** BhavyaM is now known as BhavyaM_afk | 04:53 | |
yongshenggong__ | gus, do u know why tox -e py27 will reset the virtual env's pip back into old version? | 04:53 |
gus | yongshenggong__: hrm, I don't think it should - unless the tox env is being recreated (tox -r, or a change to tox.ini, etc) | 04:54 |
gus | Does it say it is recreating the env? | 04:54 |
yongshenggong__ | no. I don't recreate it | 04:54 |
*** ducnc is now known as ducnguyen | 04:55 | |
*** vikram__ has joined #openstack-neutron | 04:55 | |
gus | yongshenggong__: tox can decide to recreate it itself automatically sometimes (modified tox.ini, new python, etc) - but the output should say it is recreating it when it does. | 04:56 |
*** BhavyaM_afk is now known as BhavyaM | 04:56 | |
yongshenggong__ | gus: I will create paste bin | 04:58 |
yongshenggong__ | gus: could you please have a look at http://paste.openstack.org/show/443588/ | 04:58 |
yongshenggong__ | gus: I am struggling with the pip version | 04:59 |
*** dims has joined #openstack-neutron | 04:59 | |
armax | HenryG: oing | 04:59 |
armax | HenryG: pong | 04:59 |
HenryG | armax: I just hit send on a reply in the comments | 04:59 |
* armax looks | 05:00 | |
*** srikanth_p has joined #openstack-neutron | 05:00 | |
armax | HenryG: that I know | 05:00 |
HenryG | armax: so what am I missing? | 05:00 |
armax | it’s more the ‘static’ decomposition of responsability | 05:00 |
srikanth_p | ca someone help me how to write CLI commands to neutronclient? | 05:00 |
armax | rather than the fact that eventually the cleanup function is called | 05:00 |
armax | but say for instance that we fail in the base setup | 05:01 |
armax | luckily the reset method is idempotent I think | 05:01 |
yongshenggong__ | srikanth_p: you can look at https://review.openstack.org/#/c/219455/ | 05:01 |
yongshenggong__ | to learn how to write neutron client commands | 05:02 |
yongshenggong__ | It is not difficult | 05:02 |
*** lyanchih has joined #openstack-neutron | 05:02 | |
HenryG | we already do things like mock.patch.stopall for everyone | 05:02 |
HenryG | the magic is rampant | 05:03 |
*** vikram__ has quit IRC | 05:03 | |
armax | that’s slightly different | 05:03 |
armax | but maybe I am arguing for nothing | 05:03 |
*** afazekas has joined #openstack-neutron | 05:04 | |
HenryG | you are usually right though | 05:04 |
armax | HenryG: at the moment the movement feels awkward | 05:05 |
*** rossella_s has quit IRC | 05:06 | |
*** dims has quit IRC | 05:06 | |
armax | my point is that I would like to see self.setup_config() and addcleanup closer together | 05:06 |
*** rossella_s has joined #openstack-neutron | 05:06 | |
gus | yongshenggong__: oh, try .tox/pu27/bin/pip install --upgrade pbr | 05:06 |
armax | if possible | 05:06 |
iwamoto | armax: hi. could you consider changing your +1 to +2 on the ryu patch? | 05:07 |
armax | no | 05:07 |
armax | why? | 05:07 |
armax | there would be no point | 05:08 |
armax | iwamoto: we’re cutting L3 any moment now | 05:08 |
armax | the ryu patch will have to be deferred to RC1 | 05:08 |
iwamoto | should I apply for FFE? | 05:08 |
armax | having said that, please reach out to anorher L2 Lt | 05:08 |
*** shaohe_feng has joined #openstack-neutron | 05:08 | |
armax | iwamoto: hold on | 05:08 |
iwamoto | it seems kevinbenton is the only L2 lieutenant | 05:10 |
armax | what about rossella_s? | 05:13 |
armax | she is the other core reviewer for L2 | 05:13 |
*** vivek-ebay has quit IRC | 05:13 | |
*** sdake_ has joined #openstack-neutron | 05:13 | |
*** kiran-r has joined #openstack-neutron | 05:14 | |
*** sbalukoff has joined #openstack-neutron | 05:14 | |
HenryG | armax: I think addCleanup(CONF.reset) must be done in Diet because test classes can derive from Diet and then change CONF in other ways than setup_config(). It seems that tests deriving from Diet do not want to do setup_config(). ? | 05:15 |
*** vikram__ has joined #openstack-neutron | 05:15 | |
armax | that’s fine | 05:15 |
HenryG | armax: but at this hour my brain is fried. You can educate me more tomorrow | 05:15 |
armax | we can leave the magic be | 05:16 |
*** shausy has joined #openstack-neutron | 05:16 | |
*** sdake has quit IRC | 05:17 | |
*** nlahouti has joined #openstack-neutron | 05:17 | |
armax | HenryG: feel free to nudge it in, it’ll be unlikely if it merges any time soon | 05:17 |
armax | HenryG: but the patch is speculative anyway | 05:17 |
*** sthillma has joined #openstack-neutron | 05:17 | |
HenryG | armax: no way, I am not approving that :) | 05:18 |
armax | HenryG: why? | 05:18 |
HenryG | You sow doubt in my mind | 05:18 |
*** n00btube has joined #openstack-neutron | 05:18 | |
armax | take it out | 05:18 |
armax | it’s not worth it | 05:19 |
*** nlahouti1 has joined #openstack-neutron | 05:19 | |
*** slaweq has joined #openstack-neutron | 05:19 | |
*** vikram__ has quit IRC | 05:21 | |
*** nlahouti has quit IRC | 05:21 | |
*** n00btube has quit IRC | 05:23 | |
*** lyanchih has quit IRC | 05:23 | |
*** Kennan2 has joined #openstack-neutron | 05:24 | |
*** Kennan has quit IRC | 05:25 | |
*** shausy has quit IRC | 05:27 | |
*** SridharG has quit IRC | 05:29 | |
*** neeti has joined #openstack-neutron | 05:30 | |
*** shaohe_feng has quit IRC | 05:30 | |
*** yfried has joined #openstack-neutron | 05:33 | |
*** vikram__ has joined #openstack-neutron | 05:33 | |
*** achanda has joined #openstack-neutron | 05:34 | |
*** shoutm has quit IRC | 05:36 | |
*** nlahouti1 has quit IRC | 05:37 | |
*** shoutm has joined #openstack-neutron | 05:37 | |
*** yfried_ has joined #openstack-neutron | 05:37 | |
*** yfried has quit IRC | 05:37 | |
*** vikram__ has quit IRC | 05:38 | |
*** mkolesni has joined #openstack-neutron | 05:38 | |
*** itzikb has joined #openstack-neutron | 05:39 | |
*** achanda has quit IRC | 05:40 | |
*** tflynn has joined #openstack-neutron | 05:41 | |
*** dannywilson has quit IRC | 05:41 | |
*** tflynn_ has quit IRC | 05:42 | |
*** kobayashi has quit IRC | 05:42 | |
yalie | armax: ping | 05:42 |
*** sthillma has quit IRC | 05:43 | |
*** kobayashi has joined #openstack-neutron | 05:43 | |
*** ducnguyen has quit IRC | 05:44 | |
*** kiran-r has quit IRC | 05:44 | |
armax | yalie: pong | 05:46 |
*** anilvenkata has joined #openstack-neutron | 05:46 | |
yalie | ha armax, I want to confirm that the unaddressed-port issue, | 05:47 |
yalie | is that because this port could spoofing any ip in the same network? | 05:47 |
armax | yalie: what do you mean? | 05:47 |
yalie | for example, a network with IP A, B | 05:48 |
armax | yalie: do you have a good understanding of the vulnerability? | 05:48 |
*** sdake has joined #openstack-neutron | 05:48 | |
armax | beacuse if you don’t this discussion is pointless | 05:48 |
yalie | I am not sure, and want to confirm with you | 05:48 |
armax | k | 05:48 |
yalie | if the unaddrss port not spoof A, B, would it be ok? | 05:48 |
*** moshele has joined #openstack-neutron | 05:49 | |
*** VikasC has joined #openstack-neutron | 05:49 | |
armax | let’s back out for a sec | 05:49 |
yalie | ok | 05:49 |
armax | your proposed solution prevented l3 security rules to be applied to the port | 05:50 |
armax | if it didn’t have an address | 05:50 |
armax | correct | 05:50 |
armax | ? | 05:50 |
yalie | yes | 05:50 |
armax | now imagine the scenario where you creatae a network | 05:50 |
armax | create a port | 05:50 |
armax | and only then create subnet | 05:50 |
yalie | yes | 05:50 |
armax | now the port will have no l3 rules correct? | 05:51 |
yalie | the port in the same broadcast domian with the new created subnet | 05:52 |
*** sdake_ has quit IRC | 05:52 | |
yalie | it could pretend to be any ip of the network? | 05:52 |
yalie | right | 05:52 |
yalie | the port still not have L3 rules | 05:53 |
*** tfukushima has joined #openstack-neutron | 05:53 | |
*** sdake_ has joined #openstack-neutron | 05:54 | |
yalie | it will accept all the traffic of other ip? | 05:54 |
armax | are you asking me? | 05:54 |
yalie | sorry, I don't understand | 05:55 |
yalie | do you think all the l3 rules should be need if the port is in a network includeing subnet? | 05:56 |
openstackgerrit | Oleg Bondarev proposed openstack/neutron: Avoid DB errors when deleting network's ports and subnets https://review.openstack.org/212110 | 05:57 |
*** sdake has quit IRC | 05:57 | |
*** shoutm has quit IRC | 05:58 | |
yalie | I dont' know if it 's adequate that add a mech that make unaddressport knows other ip in the network, and not spoof these ones. | 05:58 |
*** shoutm has joined #openstack-neutron | 05:58 | |
*** yfried_ is now known as yfried|afk | 06:00 | |
*** achanda has joined #openstack-neutron | 06:00 | |
*** lyanchih has joined #openstack-neutron | 06:01 | |
*** ekuris has joined #openstack-neutron | 06:01 | |
*** itzikb has quit IRC | 06:01 | |
*** yfried|afk is now known as yfried_ | 06:02 | |
*** itzikb has joined #openstack-neutron | 06:03 | |
*** vthapar has joined #openstack-neutron | 06:03 | |
*** shashank_hegde has quit IRC | 06:04 | |
*** shashank_hegde has joined #openstack-neutron | 06:05 | |
armax | yalie: I think in lieu of a good understanding of the problem it’s better to err on the side of caution and ensure that we don’t open up a huge hole into a shared network | 06:05 |
*** Sukhdev_ has quit IRC | 06:06 | |
*** garyk has joined #openstack-neutron | 06:07 | |
*** tflynn_ has joined #openstack-neutron | 06:07 | |
yalie | so, rules should be kept if the network include the subnet(mayshared)? | 06:07 |
yalie | or, we should say share network should not involved in unadressed port | 06:09 |
armax | both can perhaps work | 06:09 |
armax | check with kevinbenton, he may have other ideas too | 06:09 |
*** numans has joined #openstack-neutron | 06:10 | |
yalie | yes, I checked with kevin, he adviced that maybe need a new knob | 06:10 |
*** roeyc has joined #openstack-neutron | 06:10 | |
yalie | how do you think about it ? | 06:10 |
*** tflynn has quit IRC | 06:10 | |
*** qwebirc29265 has joined #openstack-neutron | 06:11 | |
*** abregman has joined #openstack-neutron | 06:11 | |
*** vivek-ebay has joined #openstack-neutron | 06:14 | |
*** baoli has joined #openstack-neutron | 06:14 | |
yalie | ok, thanks armax, I will check more about it. | 06:14 |
*** shivrao has quit IRC | 06:15 | |
*** vivek-ebay has quit IRC | 06:19 | |
*** baoli has quit IRC | 06:19 | |
*** achanda has quit IRC | 06:19 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex https://review.openstack.org/218742 | 06:21 |
*** slaweq has quit IRC | 06:22 | |
*** slaweq has joined #openstack-neutron | 06:23 | |
*** slaweq has quit IRC | 06:25 | |
*** shoutm_ has joined #openstack-neutron | 06:25 | |
*** dedery has joined #openstack-neutron | 06:25 | |
*** shoutm has quit IRC | 06:26 | |
*** slaweq has joined #openstack-neutron | 06:26 | |
*** slaweq has quit IRC | 06:28 | |
openstackgerrit | Li Ma proposed openstack/neutron: Implement external physical bridge mapping in linuxbridge https://review.openstack.org/209006 | 06:29 |
*** gampel has joined #openstack-neutron | 06:32 | |
*** SridharG has joined #openstack-neutron | 06:36 | |
*** minwang2 has quit IRC | 06:40 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Add support for PluginWorker and Process creation notification https://review.openstack.org/189391 | 06:40 |
*** gampel1 has joined #openstack-neutron | 06:41 | |
*** gampel has quit IRC | 06:44 | |
*** kiran-r has joined #openstack-neutron | 06:44 | |
*** maishsk has joined #openstack-neutron | 06:45 | |
maishsk | Good morning Neutron | 06:45 |
maishsk | Is it possible to set a domain suffix per tenant? | 06:45 |
maishsk | Or is is a general neutron setting done in dhcp_agent.ini (dhcp_domain)? | 06:46 |
*** achanda has joined #openstack-neutron | 06:48 | |
maishsk | And therefore only one per neutron? | 06:49 |
*** ybabenko has joined #openstack-neutron | 06:51 | |
*** alex_klimov has joined #openstack-neutron | 06:55 | |
*** ybabenko has quit IRC | 06:56 | |
*** ybabenko has joined #openstack-neutron | 06:56 | |
*** ushkalim has joined #openstack-neutron | 06:56 | |
*** jlibosva has joined #openstack-neutron | 06:56 | |
*** safchain has joined #openstack-neutron | 06:56 | |
*** armax has quit IRC | 06:58 | |
*** qwebirc29265 has quit IRC | 06:58 | |
*** shz has quit IRC | 06:59 | |
*** kiran-r has quit IRC | 06:59 | |
*** kawa2014 has joined #openstack-neutron | 07:02 | |
*** ildikov has joined #openstack-neutron | 07:05 | |
*** tmorin has joined #openstack-neutron | 07:06 | |
*** Murali has joined #openstack-neutron | 07:07 | |
*** karthiks has joined #openstack-neutron | 07:07 | |
*** srikanth_p has quit IRC | 07:09 | |
*** roeyc has quit IRC | 07:11 | |
*** tflynn_ has quit IRC | 07:13 | |
*** yfried_ is now known as yfried|afk | 07:14 | |
*** yfried|afk is now known as yfried_ | 07:14 | |
*** Marga_ has joined #openstack-neutron | 07:15 | |
*** Marga_ has quit IRC | 07:15 | |
*** Marga_ has joined #openstack-neutron | 07:16 | |
kevinbenton | maishsk: I believe a new feature just went in this cycle to allow dns for each tenant | 07:16 |
maishsk | kevinbenton: in Kilo? link? | 07:17 |
*** y_sawai has joined #openstack-neutron | 07:18 | |
kevinbenton | maishsk: no, in liberty | 07:19 |
kevinbenton | https://review.openstack.org/#/c/88624/ | 07:19 |
SridharG | kevinbenton: I raised a new bug and updated the commit message. Can you please take a look, thanks - https://review.openstack.org/#/c/201650/ | 07:20 |
*** gparaskevas has joined #openstack-neutron | 07:23 | |
SridharG | kevinbenton: thanks for reviewing the patch. | 07:23 |
kevinbenton | SridharG: np | 07:24 |
*** jpena has joined #openstack-neutron | 07:24 | |
*** shashank_hegde has quit IRC | 07:25 | |
*** ccard has quit IRC | 07:26 | |
*** ronis has joined #openstack-neutron | 07:26 | |
*** dedery has quit IRC | 07:26 | |
*** bharathm has quit IRC | 07:26 | |
*** dedery has joined #openstack-neutron | 07:26 | |
maishsk | kevinbenton: I am not sure how that bp addresses the fact that each tenant will need a different DNS? | 07:27 |
maishsk | or I should rather say - not need - but a domain suffix could be configured per tenant - or even more so - per subnet | 07:27 |
maishsk | aka network | 07:27 |
*** matrohon has joined #openstack-neutron | 07:28 | |
kevinbenton | maishsk: isn't a different dns domain per network what you want? | 07:29 |
*** evgenyf has joined #openstack-neutron | 07:30 | |
maishsk | kevinbenton: Yes. Today it is limited to one per neutron - if I understand correctly ? | 07:30 |
*** obondarev has joined #openstack-neutron | 07:30 | |
*** anshul has joined #openstack-neutron | 07:31 | |
*** tfukushima has quit IRC | 07:31 | |
kevinbenton | maishsk: yes | 07:31 |
*** tfukushima has joined #openstack-neutron | 07:32 | |
kevinbenton | maishsk: I haven't tried overriding with the extra dhcp options API though | 07:32 |
kevinbenton | maishsk: have you tried that? | 07:32 |
*** anshul has quit IRC | 07:33 | |
*** anshul has joined #openstack-neutron | 07:33 | |
*** afazekas__ has joined #openstack-neutron | 07:33 | |
maishsk | kevinbenton: I don’t that is available - honestly would not know really how to look for is | 07:34 |
maishsk | don’t think* | 07:34 |
kevinbenton | maishsk: I gtg. Try shooting an email to the list | 07:34 |
maishsk | from a neutron subnet-show I don’t see any dns information - except the nameservers. | 07:34 |
maishsk | kevinbenton: will do. | 07:35 |
maishsk | Thanks | 07:35 |
*** jlanoux has joined #openstack-neutron | 07:36 | |
*** mickeys has joined #openstack-neutron | 07:39 | |
hichihara | maishsk: I guess that we can configure dns name servers but cannot do domain-suffix | 07:40 |
*** vikram has quit IRC | 07:41 | |
*** rotbeard has joined #openstack-neutron | 07:41 | |
*** ociuhandu has quit IRC | 07:41 | |
*** lajos-katona has joined #openstack-neutron | 07:41 | |
*** jschwarz has joined #openstack-neutron | 07:41 | |
maishsk | hichihara: that is my understanding as well. | 07:43 |
maishsk | I am not sure that the bp that kevinbenton mentioned above covers this use case | 07:44 |
*** y_sawai has quit IRC | 07:45 | |
hichihara | I don't think so. That is external dns case. I guess that dnsmasq run with "--domain-suffix" which you want. | 07:46 |
maishsk | this looks more suitable https://blueprints.launchpad.net/neutron/+spec/dns-subsystem | 07:48 |
*** lyanchih has quit IRC | 07:48 | |
*** jbell8 has quit IRC | 07:49 | |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron-vpnaas: Drop for 'tenant_id' column for 'cisco_csr_identifier_map' table https://review.openstack.org/213174 | 07:49 |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron-vpnaas: Implement ModelMigrationSyncTest https://review.openstack.org/209943 | 07:49 |
hichihara | Yeah but it's big bp... and I cannot see the spec and code | 07:51 |
*** mickeys has quit IRC | 07:51 | |
*** Marga_ has quit IRC | 07:52 | |
*** ociuhandu has joined #openstack-neutron | 07:52 | |
hichihara | Well, you might want to send your question to ML as kevinbenton said. | 07:52 |
*** vikram has joined #openstack-neutron | 07:54 | |
*** Marga_ has joined #openstack-neutron | 07:56 | |
*** lyanchih has joined #openstack-neutron | 07:57 | |
*** dedery_ has joined #openstack-neutron | 07:59 | |
*** iwamoto has quit IRC | 07:59 | |
openstackgerrit | Merged openstack/neutron: Add tenant_id to flavor service profiles attributes https://review.openstack.org/217440 | 08:00 |
*** dedery has quit IRC | 08:02 | |
*** links has quit IRC | 08:02 | |
*** zhipeng has joined #openstack-neutron | 08:03 | |
*** y_sawai has joined #openstack-neutron | 08:03 | |
*** kobayashi has quit IRC | 08:04 | |
*** y_sawai has quit IRC | 08:08 | |
*** frobware_ has joined #openstack-neutron | 08:08 | |
*** y_sawai has joined #openstack-neutron | 08:10 | |
*** y_sawai has quit IRC | 08:10 | |
*** y_sawai has joined #openstack-neutron | 08:11 | |
openstackgerrit | Merged openstack/neutron: Enable to update external network subnet's gateway-ip https://review.openstack.org/194939 | 08:11 |
*** txomon|fon has quit IRC | 08:11 | |
openstackgerrit | Merged openstack/neutron: Make Neutron service flavor save service_type https://review.openstack.org/218534 | 08:12 |
openstackgerrit | Merged openstack/neutron: Deprecate --service option for neutron-db-manage https://review.openstack.org/219456 | 08:12 |
rossella_s | gongysh ping | 08:13 |
*** zhipeng has quit IRC | 08:14 | |
openstackgerrit | Merged openstack/neutron-fwaas: Removing unused dependency: discover https://review.openstack.org/218420 | 08:14 |
*** y_sawai has quit IRC | 08:15 | |
*** zhipeng has joined #openstack-neutron | 08:15 | |
*** zhipeng_ has joined #openstack-neutron | 08:23 | |
*** txomon|fon has joined #openstack-neutron | 08:24 | |
*** zhipeng has quit IRC | 08:25 | |
*** vikram has quit IRC | 08:25 | |
*** zhipeng_ has quit IRC | 08:27 | |
*** y_sawai has joined #openstack-neutron | 08:27 | |
*** zhipeng has joined #openstack-neutron | 08:29 | |
*** gampel has joined #openstack-neutron | 08:30 | |
*** yfried_ is now known as yfried|afk | 08:31 | |
*** gampel1 has quit IRC | 08:31 | |
*** ygbo has joined #openstack-neutron | 08:32 | |
*** l6unchpad has joined #openstack-neutron | 08:34 | |
*** feleouet has joined #openstack-neutron | 08:36 | |
*** achanda has quit IRC | 08:36 | |
*** jistr has joined #openstack-neutron | 08:37 | |
*** hichihara has quit IRC | 08:37 | |
*** zhipeng has quit IRC | 08:39 | |
*** zhipeng has joined #openstack-neutron | 08:41 | |
*** bharathm has joined #openstack-neutron | 08:41 | |
*** doude_ has quit IRC | 08:44 | |
*** y_sawai has quit IRC | 08:46 | |
*** tfukushima has quit IRC | 08:46 | |
*** y_sawai has joined #openstack-neutron | 08:49 | |
*** tfukushima has joined #openstack-neutron | 08:51 | |
*** yfried|afk is now known as yfried_ | 08:51 | |
*** bobmel has quit IRC | 08:51 | |
*** diga has joined #openstack-neutron | 08:52 | |
*** y_sawai has quit IRC | 08:54 | |
*** dedery_ has quit IRC | 08:55 | |
*** dedery has joined #openstack-neutron | 08:55 | |
*** neeti has quit IRC | 08:55 | |
openstackgerrit | Neil Jerram proposed openstack/neutron: Use US spelling for 'behavior', instead of 'behaviour' https://review.openstack.org/220039 | 08:59 |
*** anilvenkata has quit IRC | 09:00 | |
*** l6unchpad has quit IRC | 09:01 | |
yongshenggong__ | pong rossella_s> | 09:02 |
*** zhipeng has quit IRC | 09:02 | |
*** neeti has joined #openstack-neutron | 09:02 | |
rossella_s | yongshenggong__ so you are gongysh ? | 09:04 |
yongshenggong__ | yes | 09:05 |
yongshenggong__ | gongysh is taken by my machine at office. | 09:05 |
rossella_s | good to know...I wanted to bother you again regarding https://review.openstack.org/#/c/211191/ | 09:05 |
*** vikram has joined #openstack-neutron | 09:08 | |
*** ccard has joined #openstack-neutron | 09:08 | |
yongshenggong__ | rossella_s: it is good for the dependent patch. I think in this case it makes sense. but why not just merge these two patches? | 09:09 |
yongshenggong__ | but anyway, it is good to remove the local states keeping. | 09:09 |
rossella_s | yongshenggong__, I tried to make small reviewable patches | 09:09 |
yongshenggong__ | yes, but small patch sometimes does not make obvious point for change. thanks | 09:11 |
*** john-davidge has joined #openstack-neutron | 09:12 | |
*** yfried_ is now known as yfried|afk | 09:12 | |
yongshenggong__ | rossella_s: I scored +1 | 09:12 |
*** yamamoto has quit IRC | 09:13 | |
*** yongshenggong__ has quit IRC | 09:14 | |
*** john-davidge_ has joined #openstack-neutron | 09:15 | |
*** alex_klimov has quit IRC | 09:15 | |
*** alex_klimov has joined #openstack-neutron | 09:15 | |
*** SridharG has quit IRC | 09:16 | |
*** john-davidge has quit IRC | 09:16 | |
*** john-davidge_ is now known as john-davidge | 09:16 | |
ZZelle | Steap, hi | 09:17 |
*** Marga_ has quit IRC | 09:17 | |
*** Marga_ has joined #openstack-neutron | 09:17 | |
*** vthapar has quit IRC | 09:19 | |
Steap | ZZelle: hi | 09:20 |
*** jamespd has quit IRC | 09:22 | |
*** gongysh__ has joined #openstack-neutron | 09:22 | |
*** ihrachys has joined #openstack-neutron | 09:23 | |
*** l6unchpad has joined #openstack-neutron | 09:23 | |
*** Marga_ has quit IRC | 09:24 | |
*** Marga_ has joined #openstack-neutron | 09:24 | |
*** evgenyf has quit IRC | 09:26 | |
*** vthapar has joined #openstack-neutron | 09:26 | |
ZZelle | Steap, could you (or Victor?) look at https://review.openstack.org/219389? One test is still not passing ... and i have a poor understanding of unicode/utf8 en/decoding | 09:26 |
Steap | I was looking at this | 09:27 |
Steap | will ask Victor | 09:27 |
*** gongysh__ has quit IRC | 09:28 | |
*** jamespd has joined #openstack-neutron | 09:28 | |
Steap | worst case scenario, we just skip this test as well :) | 09:28 |
*** ivar-lazzaro has joined #openstack-neutron | 09:29 | |
ZZelle | Steap, i fear it could a bigger trouble with string encoding/decoding | 09:29 |
ZZelle | Steap, i fear it could HIDE a bigger trouble with string encoding/decoding | 09:29 |
*** Marga_ has quit IRC | 09:29 | |
*** gongysh__ has joined #openstack-neutron | 09:30 | |
Steap | this is usually a huge PITA indeed | 09:30 |
*** yfried|afk is now known as yfried_ | 09:32 | |
*** vthapar has quit IRC | 09:32 | |
*** vthapar_ has joined #openstack-neutron | 09:32 | |
*** SridharG has joined #openstack-neutron | 09:33 | |
*** gongysh__ has quit IRC | 09:34 | |
*** vthapar_ has quit IRC | 09:36 | |
*** gongysh__ has joined #openstack-neutron | 09:37 | |
*** jbell8 has joined #openstack-neutron | 09:37 | |
ihrachys | Steap: if it takes a long time, then I think it's better to merge the fix with the skip, and then follow-up, to avoid lack of test coverage while you investigate. | 09:38 |
ihrachys | ZZelle: btw thanks for looking into the patch, really appreciated, and great progress! | 09:38 |
Steap | would be nice to merge this today | 09:38 |
Steap | before thte FF | 09:38 |
*** gongysh__ has quit IRC | 09:42 | |
*** jbell8 has quit IRC | 09:42 | |
*** Marga_ has joined #openstack-neutron | 09:42 | |
*** gongysh__ has joined #openstack-neutron | 09:42 | |
*** apuimedo has joined #openstack-neutron | 09:42 | |
*** vthapar has joined #openstack-neutron | 09:44 | |
ZZelle | Steap, ihrachys, +1 to test skip | 09:45 |
ajo | ping moshele | 09:47 |
ajo | moshele: were you able to check the random failures of https://review.openstack.org/#/c/214218/? | 09:48 |
*** gongysh__ has quit IRC | 09:48 | |
ajo | I'm writing a patch to fix https://bugs.launchpad.net/neutron/+bug/1486039 that modifies the logic a bit | 09:48 |
openstack | Launchpad bug 1486039 in neutron "Setting a policy to a network will limit the router/dhcp/net-device ports, that's not expected" [High,New] - Assigned to Miguel Angel Ajo (mangelajo) | 09:48 |
ajo | I guess they will conflict a bit, but I can help to rebase yours | 09:49 |
ajo | or may be | 09:49 |
ajo | I can just build it on top of yours | 09:49 |
ajo | that's probably going to be easier | 09:49 |
ajo | no need to replicate the sriov logic | 09:49 |
*** gongysh__ has joined #openstack-neutron | 09:49 | |
*** y_sawai has joined #openstack-neutron | 09:50 | |
*** y_sawai_ has joined #openstack-neutron | 09:51 | |
*** miyagishi_t has quit IRC | 09:52 | |
*** gampel has quit IRC | 09:54 | |
*** gongysh__ has quit IRC | 09:54 | |
moshele | ajo: I am not sure I will be able today I want to finish the SR-IOV part in https://review.openstack.org/#/c/214055/, also yesterday my devstack env was ruined I just now able to recreate it :( | 09:54 |
*** y_sawai has quit IRC | 09:54 | |
*** gongysh__ has joined #openstack-neutron | 09:54 | |
*** bobmel has joined #openstack-neutron | 09:55 | |
ajo | moshele, ack, I will try to find the bug in your review once I finish my bugfix | 09:56 |
*** y_sawai_ has quit IRC | 09:56 | |
moshele | ajo: thanks :) I may continue also tomorrow because I have to leave early today | 09:57 |
ajo | moshele :) np, I just wanted to let you know, | 09:57 |
*** jlanoux has quit IRC | 09:58 | |
*** gampel has joined #openstack-neutron | 09:58 | |
*** gongysh__ has quit IRC | 10:00 | |
*** jlanoux has joined #openstack-neutron | 10:02 | |
*** Marga_ has quit IRC | 10:02 | |
*** Marga_ has joined #openstack-neutron | 10:02 | |
*** neeti has quit IRC | 10:07 | |
*** bharathm has quit IRC | 10:07 | |
*** neeti has joined #openstack-neutron | 10:07 | |
*** evgenyf has joined #openstack-neutron | 10:09 | |
apuimedo | gsagie: I sent some comments to your patch ;-) | 10:11 |
*** frobware_ has quit IRC | 10:12 | |
*** kbyrne has quit IRC | 10:12 | |
*** sc68cal has quit IRC | 10:13 | |
*** yamamoto has joined #openstack-neutron | 10:13 | |
*** tfukushima has quit IRC | 10:13 | |
openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Revert "Decentralize the managemement of service providers" https://review.openstack.org/220070 | 10:15 |
*** kbyrne has joined #openstack-neutron | 10:15 | |
*** vivek-ebay has joined #openstack-neutron | 10:16 | |
*** yamamoto has quit IRC | 10:18 | |
*** nyechiel has joined #openstack-neutron | 10:19 | |
*** vivek-ebay has quit IRC | 10:20 | |
*** l6unchpad has quit IRC | 10:20 | |
*** BhavyaM has quit IRC | 10:21 | |
*** BhavyaM has joined #openstack-neutron | 10:22 | |
*** ybabenko has quit IRC | 10:23 | |
rossella_s | ihrachys is it too late to approve patches? I am not sure how it work with the so close release cut | 10:25 |
ihrachys | rossella_s: in which patch have I violated something? | 10:26 |
*** ekuris has quit IRC | 10:26 | |
rossella_s | ihrachys I am not saying you violated anything...I am just asking you since you are an "old" core. I just want to know how I should behave...if I should approve patches or not | 10:27 |
*** ildikov has quit IRC | 10:28 | |
ihrachys | rossella_s: ok I see armax email now. | 10:28 |
rossella_s | ihrachys, I missed that actually, reading it now | 10:28 |
*** oomichi is now known as oomichi_away | 10:29 | |
ihrachys | rossella_s: well, I just pushed a test only patch for gate. I believe that one is ok. but generally, I guess now the armax email should be considered, I guess we should mind the gate (if queue is long, refrain to push without approval from armax), and invasiveness | 10:29 |
ihrachys | rossella_s: well, that's cool you notified me to check, I didn't see the email too ;) | 10:29 |
ihrachys | rossella_s: am I an old core?.. I believe I'm like 4 months in the position :) | 10:30 |
rossella_s | ihrachys and actually I didn't notice your patch. I picked you because as far as I know you are the "older" core online now | 10:30 |
ihrachys | ok, even if you picked me for the patch, it's absolutely fine to call out :) | 10:31 |
rossella_s | yes it's fine but that was not my intention... :) | 10:31 |
*** links has joined #openstack-neutron | 10:31 | |
*** pc_m has joined #openstack-neutron | 10:35 | |
*** l6unchpad has joined #openstack-neutron | 10:35 | |
*** yfried_ has quit IRC | 10:37 | |
*** nplanel_ has joined #openstack-neutron | 10:37 | |
*** mkolesni has quit IRC | 10:38 | |
*** jschwarz has quit IRC | 10:39 | |
*** anilvenkata has joined #openstack-neutron | 10:40 | |
*** SridharG1 has joined #openstack-neutron | 10:40 | |
openstackgerrit | Merged openstack/neutron: Add info to debug test_keepalived_respawns gate failure https://review.openstack.org/218578 | 10:42 |
*** claudiub has joined #openstack-neutron | 10:44 | |
*** Kennan2 is now known as Kennan_on_vacati | 10:44 | |
*** SridharG has quit IRC | 10:44 | |
*** Kennan_on_vacati is now known as Kennan_Vacation | 10:45 | |
*** aix has quit IRC | 10:46 | |
*** ZZelle has quit IRC | 10:48 | |
*** ZZelle has joined #openstack-neutron | 10:48 | |
*** tfukushima has joined #openstack-neutron | 10:48 | |
*** kawa2014 has quit IRC | 10:49 | |
*** kawa2014 has joined #openstack-neutron | 10:50 | |
*** evgenyf has quit IRC | 10:50 | |
*** ihrachys has quit IRC | 10:52 | |
*** neeti has quit IRC | 10:52 | |
*** itzikb has quit IRC | 10:53 | |
*** banix has joined #openstack-neutron | 10:54 | |
*** neeti has joined #openstack-neutron | 10:57 | |
*** mattt has left #openstack-neutron | 10:57 | |
*** vikschw has joined #openstack-neutron | 10:58 | |
openstackgerrit | Merged openstack/neutron-fwaas: Updated from global requirements https://review.openstack.org/218909 | 10:58 |
*** vikschw has quit IRC | 11:00 | |
*** hdaniel has joined #openstack-neutron | 11:00 | |
*** lazy_prince has joined #openstack-neutron | 11:02 | |
*** y_sawai has joined #openstack-neutron | 11:03 | |
apuimedo | gsagie: banix: diga: rossella_s: irenab: tfukushima: daneyon_: Irena and I have updated the virtual sprint etherpad | 11:03 |
apuimedo | https://etherpad.openstack.org/p/kuryr-virtual-sprint | 11:03 |
apuimedo | it's starting to be ready ;-) | 11:03 |
diga | apuimedo: Great! will look at it | 11:04 |
tfukushima | apuimedo: thanks | 11:04 |
*** y_sawai has quit IRC | 11:05 | |
diga | apuimedo: I need your inputs on this - https://review.openstack.org/#/c/214002/13/usr/libexec/kuryr/binding/ovs.py | 11:05 |
diga | apuimedo: blocked at this, How to proceed on this | 11:05 |
*** y_sawai has joined #openstack-neutron | 11:05 | |
apuimedo | diga: sorry, I thought I commented here on #neutron that I agreed with Irena | 11:06 |
apuimedo | I also tend to agree with ajo that this executable would be much simpler if it were just the bash call | 11:07 |
*** ivar-lazzaro has quit IRC | 11:07 | |
apuimedo | and it being a bash script | 11:07 |
apuimedo | it gets the input as args | 11:07 |
*** bharathm has joined #openstack-neutron | 11:08 | |
diga | okay | 11:08 |
apuimedo | and then the communication back via stdout, stderr | 11:08 |
*** gparaskevas has quit IRC | 11:08 | |
apuimedo | and return code | 11:08 |
apuimedo | for exceptions | 11:08 |
apuimedo | 0 -> fine | 11:08 |
diga | okay | 11:08 |
apuimedo | anything else it is an exception | 11:08 |
apuimedo | diga: thanks ;-) | 11:08 |
diga | can we import nova_vif library here ? | 11:09 |
apuimedo | no | 11:10 |
diga | ok | 11:10 |
*** y_sawai has quit IRC | 11:10 | |
diga | I will write bash script for this | 11:10 |
apuimedo | the executables should be as simple as possible | 11:10 |
diga | okay | 11:10 |
apuimedo | I added it as a review comment now | 11:11 |
* apuimedo going for lunch | 11:11 | |
*** apuimedo is now known as apuimedo|lunch | 11:11 | |
*** Marga_ has quit IRC | 11:12 | |
diga | apuimedo|lunch: thanks :) will take a look at it | 11:12 |
*** bharathm has quit IRC | 11:12 | |
banix | apuimedo|lunch: ok thx | 11:13 |
openstackgerrit | Sridhar Gaddam proposed openstack/neutron: Add IPv6 Address Resolution protection https://review.openstack.org/201650 | 11:13 |
*** aix has joined #openstack-neutron | 11:15 | |
*** banix has quit IRC | 11:15 | |
*** shoutm has joined #openstack-neutron | 11:17 | |
*** shoutm_ has quit IRC | 11:18 | |
*** y_sawai has joined #openstack-neutron | 11:18 | |
*** lyanchih has quit IRC | 11:19 | |
*** y_sawai has quit IRC | 11:23 | |
*** sayali has quit IRC | 11:26 | |
*** alex_klimov has quit IRC | 11:26 | |
*** yamamoto has joined #openstack-neutron | 11:29 | |
*** nyechiel has quit IRC | 11:31 | |
*** BhavyaM has quit IRC | 11:32 | |
*** ybabenko has joined #openstack-neutron | 11:32 | |
*** ybabenko has quit IRC | 11:32 | |
*** nplanel_ has quit IRC | 11:33 | |
openstackgerrit | YAMAMOTO Takashi proposed openstack/neutron: create_router: Handle the case with a surrounding transaction https://review.openstack.org/219215 | 11:33 |
openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: [neutron-db-manage] Add check for branches https://review.openstack.org/220091 | 11:34 |
*** yamamoto has quit IRC | 11:34 | |
*** Marga_ has joined #openstack-neutron | 11:35 | |
*** ociuhandu has quit IRC | 11:36 | |
*** julim has joined #openstack-neutron | 11:37 | |
*** thorst has joined #openstack-neutron | 11:37 | |
*** ildikov has joined #openstack-neutron | 11:39 | |
*** y_sawai has joined #openstack-neutron | 11:39 | |
*** ushkalim has quit IRC | 11:40 | |
*** ushkalim has joined #openstack-neutron | 11:40 | |
*** y_sawai has quit IRC | 11:43 | |
openstackgerrit | Moshe Levi proposed openstack/neutron: Qos SR-IOV: Refactor extension delete to get mac and pci slot https://review.openstack.org/214055 | 11:43 |
*** y_sawai has joined #openstack-neutron | 11:43 | |
*** ihrachys has joined #openstack-neutron | 11:44 | |
*** ihrachys has quit IRC | 11:44 | |
*** ihrachys has joined #openstack-neutron | 11:46 | |
*** nplanel_ has joined #openstack-neutron | 11:46 | |
*** y_sawai_ has joined #openstack-neutron | 11:47 | |
*** amotoki has quit IRC | 11:47 | |
*** neeti has quit IRC | 11:47 | |
*** y_sawai has quit IRC | 11:48 | |
*** nplanel_ has quit IRC | 11:50 | |
*** y_sawai_ has quit IRC | 11:51 | |
*** breitz has quit IRC | 11:51 | |
*** breitz has joined #openstack-neutron | 11:51 | |
*** shaohe_feng has joined #openstack-neutron | 11:52 | |
*** ociuhandu has joined #openstack-neutron | 11:55 | |
*** alexpilotti_ has joined #openstack-neutron | 11:58 | |
*** amuller has joined #openstack-neutron | 11:58 | |
*** akamyshnikova has quit IRC | 11:58 | |
moshele | ajo, ihrachys can you review again https://review.openstack.org/214055 ? | 11:59 |
*** mohankumar has joined #openstack-neutron | 12:01 | |
*** Marga_ has quit IRC | 12:01 | |
*** vthapar has quit IRC | 12:02 | |
ihrachys | moshele: hm, why isn't the blueprint scheduled for L? | 12:02 |
ihrachys | moshele: 'Is pci_lib used by anything outside the sr-iov driver?' ? | 12:04 |
moshele | ihrachys: no it is part of the sriov mech driver only | 12:05 |
moshele | ihrachys: I don't think so | 12:05 |
*** alex_klimov has joined #openstack-neutron | 12:05 | |
moshele | ihrachys: I update the bp for L | 12:06 |
*** thurloat_isgone is now known as thurloat | 12:10 | |
*** mohankumar has quit IRC | 12:11 | |
*** shaohe_feng has quit IRC | 12:11 | |
*** shaohe_feng has joined #openstack-neutron | 12:11 | |
ihrachys | moshele: yes, it should be included | 12:12 |
ihrachys | moshele: replied to the patch | 12:12 |
*** mohankumar has joined #openstack-neutron | 12:14 | |
*** jpena is now known as jpena|lunch | 12:15 | |
moshele | ihrachys: what should be included? | 12:15 |
ihrachys | moshele: the bp - in L release | 12:15 |
moshele | ihrachys: sure I updated to be in L | 12:16 |
irenab | garyk: hi | 12:17 |
moshele | ihrachys: I failed to understand you concern | 12:18 |
moshele | regrading the port_update | 12:18 |
*** deepthi has quit IRC | 12:19 | |
irenab | garyk: can you please take a look at https://review.openstack.org/#/c/199112/ , updates to the QoS spec | 12:21 |
ihrachys | moshele: so... before the change, all ports got into updated_devices, right? but now you seem to put there only those that have pci_slot set. I wonder whether you are sure it won't have some other unexpected impact on the code that uses the updated_devices set | 12:21 |
*** nyechiel has joined #openstack-neutron | 12:22 | |
garyk | irenab: ok, i'll try | 12:23 |
irenab | garyk: thanks | 12:23 |
moshele | ihrachys: I can double check, but we will get only direct/mavtap vnic_type which has to have pci_slot, unless there is bug in nova | 12:24 |
*** absubram has quit IRC | 12:24 | |
*** SridharG1 has left #openstack-neutron | 12:24 | |
garyk | irenab: is liberty no in freeze now? | 12:24 |
*** SridharG has joined #openstack-neutron | 12:24 | |
moshele | ihrachys: I added the check for safety is should not happen | 12:24 |
irenab | garyk: it is pec update to reflect changes to the code that is already merged for Liberty. I am not sure if freeze applies to spec repo. Is it? | 12:25 |
garyk | irenab: ah, ok | 12:26 |
garyk | irenab: we need one of the guys who is on the spec team to approve. | 12:26 |
irenab | garyk: it has already +2 from ajo and ihrachys | 12:27 |
irenab | ihrachys: can you approve the spec update? | 12:27 |
*** VikasC has quit IRC | 12:28 | |
*** shaohe_feng has quit IRC | 12:28 | |
garyk | irenab: it also has from me, but none of us can approve it. that can only be done by the drivers team. that is kind of like the a-team. i wonder who mt-t is. i think that armando could be | 12:28 |
irenab | garyk: ok, thanks! | 12:29 |
openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Add debugging to tox_install.sh https://review.openstack.org/203201 | 12:30 |
*** ivar-lazzaro has joined #openstack-neutron | 12:33 | |
*** baoli has joined #openstack-neutron | 12:33 | |
openstackgerrit | Oleg Bondarev proposed openstack/neutron: OVS agent: flush firewall rules for all deleted ports at once https://review.openstack.org/220115 | 12:34 |
*** baoli_ has joined #openstack-neutron | 12:37 | |
*** dedery has quit IRC | 12:38 | |
*** baoli has quit IRC | 12:38 | |
*** dims has joined #openstack-neutron | 12:41 | |
*** njohnston is now known as nate_gone | 12:42 | |
*** sayali has joined #openstack-neutron | 12:42 | |
*** y_sawai has joined #openstack-neutron | 12:43 | |
*** pradk has quit IRC | 12:45 | |
*** mohankumar has quit IRC | 12:46 | |
*** shoutm_ has joined #openstack-neutron | 12:47 | |
*** gildub has quit IRC | 12:47 | |
*** kairo has joined #openstack-neutron | 12:48 | |
*** shoutm has quit IRC | 12:48 | |
*** y_sawai has quit IRC | 12:48 | |
*** diga has quit IRC | 12:48 | |
*** anshul has quit IRC | 12:49 | |
*** dguitarbite has quit IRC | 12:51 | |
*** thurloat is now known as thurloat_isgone | 12:52 | |
*** sayali has quit IRC | 12:52 | |
*** dguitarbite has joined #openstack-neutron | 12:52 | |
*** akamyshnikova has joined #openstack-neutron | 12:52 | |
*** rpothier has joined #openstack-neutron | 12:52 | |
*** jagter has quit IRC | 12:54 | |
*** tbachman has quit IRC | 12:54 | |
*** ivar-lazzaro has quit IRC | 12:55 | |
*** chutwig has joined #openstack-neutron | 12:55 | |
*** sayali has joined #openstack-neutron | 12:56 | |
*** lazy_prince has quit IRC | 12:57 | |
*** nplanel has quit IRC | 12:58 | |
*** jagter has joined #openstack-neutron | 13:01 | |
*** rossella_s has quit IRC | 13:06 | |
*** rossella_s has joined #openstack-neutron | 13:06 | |
*** nyechiel has quit IRC | 13:07 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Retain logs for functional test cases https://review.openstack.org/219935 | 13:11 |
*** nplanel has joined #openstack-neutron | 13:12 | |
openstackgerrit | venkata anil proposed openstack/neutron: select router with subnet's gateway_ip for floatingip https://review.openstack.org/220135 | 13:12 |
*** tbh has joined #openstack-neutron | 13:12 | |
*** lazy_prince has joined #openstack-neutron | 13:13 | |
*** edmondsw has joined #openstack-neutron | 13:13 | |
*** yamamoto has joined #openstack-neutron | 13:14 | |
*** jckasper has joined #openstack-neutron | 13:16 | |
*** itlinux has joined #openstack-neutron | 13:16 | |
*** sayali has quit IRC | 13:16 | |
*** arajagopal has joined #openstack-neutron | 13:16 | |
*** evgenyf has joined #openstack-neutron | 13:17 | |
*** cbouch has joined #openstack-neutron | 13:19 | |
*** yamamoto has quit IRC | 13:19 | |
ihrachys | irenab: I am not a driver | 13:19 |
*** tbachman has joined #openstack-neutron | 13:20 | |
ihrachys | moshele: hm, but that port_update, who generates the notification? | 13:20 |
irenab | ihrachys: got it. Added armax to review. Hope this will be merged soon. I do not like the spec to be out of sync | 13:20 |
*** jpena|lunch is now known as jpena | 13:21 | |
cbouch | ihrachys: could you +2 instead of +1 on cherry pick for https://review.openstack.org/#/c/219836/? Only limited people can +2 there. | 13:21 |
*** sayali has joined #openstack-neutron | 13:22 | |
cbouch | iharachys: Just saw your note. I'll chat with gessau. | 13:22 |
ihrachys | cbouch: replied with the rationale of +1 and not +2. sorry. you can ask garyk. | 13:22 |
ihrachys | or someone else. I don't believe it's correct to vote with +2 to something that you are not sure about | 13:22 |
*** tiswanso has joined #openstack-neutron | 13:24 | |
HenryG | ihrachys: no problem, thanks for taking the time to look and comment | 13:24 |
*** sayali has quit IRC | 13:24 | |
cbouch | ihrachys: Yes thank you for the review | 13:24 |
amuller | ihrachys: fyi https://review.openstack.org/#/c/219935/ | 13:25 |
*** ivar-lazzaro has joined #openstack-neutron | 13:26 | |
*** sayali has joined #openstack-neutron | 13:27 | |
garyk | ihrachys: am i the stable rubber stamp? | 13:28 |
garyk | cbouch: i feel that due to the fact that this has the henry's thumbs up then i am fine with this. | 13:29 |
*** amotoki has joined #openstack-neutron | 13:29 | |
garyk | i feel that stable core privileges should be given to the decomposed teams. that is something we should talk about at a meeting when mestery is back | 13:30 |
*** lori|away is now known as lori | 13:30 | |
ihrachys | garyk: yes, teams should have at least one +2 stable core for the project | 13:31 |
ihrachys | another one is ok to have from horizontal team, to validate bureaucracy and general sanity | 13:31 |
ihrachys | I don't know much about any vendor repo, and frankly, I don't want to get involved too deep | 13:32 |
*** tbh has quit IRC | 13:33 | |
*** numans has quit IRC | 13:33 | |
cbouch | garyk: Thank you! | 13:34 |
openstackgerrit | Oleg Bondarev proposed openstack/neutron: OVS agent: handle deleted ports on each rpc_loop iteration https://review.openstack.org/220140 | 13:34 |
garyk | cbouch: np. thanks for doing the backports. | 13:34 |
garyk | my two cents that vendors doing the backports and making sure that the decomposed versions are stable worthy is a healthy move | 13:35 |
garyk | a luta continua | 13:35 |
*** links has quit IRC | 13:40 | |
*** l6unchpad has quit IRC | 13:41 | |
*** lyanchih has joined #openstack-neutron | 13:41 | |
*** hdaniel has quit IRC | 13:43 | |
openstackgerrit | Brent Eagles proposed openstack/neutron-vpnaas: Manage cleanup of .ctl/.pid files for LibreSwan https://review.openstack.org/212673 | 13:43 |
*** shaohe_feng has joined #openstack-neutron | 13:43 | |
*** y_sawai has joined #openstack-neutron | 13:44 | |
*** moshele has quit IRC | 13:44 | |
akamyshnikova | ihrachys: Hi, I answered on some your comments on https://review.openstack.org/#/c/220091/ | 13:46 |
*** abregman is now known as abregman|afk | 13:46 | |
*** ajmiller has joined #openstack-neutron | 13:47 | |
*** kbringard has joined #openstack-neutron | 13:47 | |
ihrachys | akamyshnikova: yeah, I saw, I believe I agree with all that is said :) | 13:48 |
*** y_sawai has quit IRC | 13:49 | |
*** lyanchih has quit IRC | 13:50 | |
*** anshul has joined #openstack-neutron | 13:51 | |
*** zzzeek has joined #openstack-neutron | 13:51 | |
akamyshnikova | ihrachys: OK, Thanks! | 13:51 |
*** ajmiller has quit IRC | 13:51 | |
*** l6unchpad has joined #openstack-neutron | 13:52 | |
*** alexpilotti_ is now known as alexpilotti | 13:53 | |
*** DanyC has joined #openstack-neutron | 13:53 | |
*** DanyC has left #openstack-neutron | 13:53 | |
*** pradk has joined #openstack-neutron | 13:54 | |
*** jroll has quit IRC | 13:54 | |
*** jroll has joined #openstack-neutron | 13:54 | |
*** sayali has quit IRC | 13:57 | |
*** nate_gone is now known as njohnston | 13:57 | |
*** rtheis has joined #openstack-neutron | 13:58 | |
*** nyechiel has joined #openstack-neutron | 13:58 | |
*** shaohe_feng1 has joined #openstack-neutron | 13:59 | |
*** shaohe_feng has quit IRC | 13:59 | |
*** korzen has joined #openstack-neutron | 13:59 | |
*** sayali has joined #openstack-neutron | 13:59 | |
*** clev is now known as clev-away | 14:00 | |
*** clev-away is now known as clev | 14:00 | |
*** clev is now known as clev-away | 14:00 | |
*** nplanel has quit IRC | 14:00 | |
*** clev-away is now known as clev | 14:00 | |
*** abregman|afk is now known as abregman | 14:01 | |
*** SridharG has quit IRC | 14:01 | |
*** rushil has joined #openstack-neutron | 14:04 | |
openstackgerrit | Moshe Levi proposed openstack/neutron: Qos SR-IOV: Refactor extension delete to get mac and pci slot https://review.openstack.org/214055 | 14:05 |
*** txomon|fon has quit IRC | 14:06 | |
*** apuimedo|lunch has quit IRC | 14:08 | |
*** rushil has quit IRC | 14:09 | |
*** bobmel has quit IRC | 14:09 | |
*** korzen has quit IRC | 14:10 | |
*** apuimedo|lunch has joined #openstack-neutron | 14:12 | |
*** n00btube has joined #openstack-neutron | 14:12 | |
*** regXboi has joined #openstack-neutron | 14:13 | |
*** n00btube has quit IRC | 14:13 | |
openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Add debugging to tox_install.sh https://review.openstack.org/203201 | 14:14 |
*** vivek-ebay has joined #openstack-neutron | 14:17 | |
*** dane_leblanc has joined #openstack-neutron | 14:17 | |
*** bobmel has joined #openstack-neutron | 14:18 | |
*** shoutm_ has quit IRC | 14:18 | |
*** dave-mccowan has quit IRC | 14:18 | |
*** txomon|fon has joined #openstack-neutron | 14:19 | |
*** mlavalle has joined #openstack-neutron | 14:20 | |
*** Murali has quit IRC | 14:21 | |
*** njohnston is now known as nate_gone | 14:22 | |
*** vivek-ebay has quit IRC | 14:22 | |
*** yamahata has joined #openstack-neutron | 14:22 | |
*** evgenyf has quit IRC | 14:23 | |
Steap | ZZelle: so, should we update that "last" py3 patch ? | 14:23 |
*** ajmiller has joined #openstack-neutron | 14:24 | |
*** kbringard has quit IRC | 14:26 | |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: QoS agent extension and driver refactoring https://review.openstack.org/214218 | 14:26 |
openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: No network devices on network attached qos policies https://review.openstack.org/220164 | 14:26 |
*** karthiks has quit IRC | 14:27 | |
*** kbringard has joined #openstack-neutron | 14:28 | |
*** rushil_ has joined #openstack-neutron | 14:28 | |
*** anilvenkata has left #openstack-neutron | 14:29 | |
*** Piet has quit IRC | 14:30 | |
*** dave-mccowan has joined #openstack-neutron | 14:31 | |
amuller | ihrachys: about tempest_set_log_file | 14:33 |
*** devlaps has quit IRC | 14:33 | |
amuller | ihrachys: any idea of an alternative way of accomplishing what we need without using a super duper private do-not-use mega method of doom? | 14:33 |
ihrachys | amuller: maybe passing --log-file in BaseTestCase.config_parse? | 14:34 |
amuller | ihrachys: let me try | 14:35 |
amuller | it | 14:35 |
mlavalle | pc_m: ping | 14:36 |
*** chutwig has quit IRC | 14:36 | |
pc_m | dougwig: ping | 14:36 |
pc_m | mlavalle: pong | 14:37 |
mlavalle | pc_m: I am digging into https://bugs.launchpad.net/neutron/+bug/1461172, that has been makin some noise lately | 14:37 |
openstack | Launchpad bug 1461172 in neutron "neutron.tests.functional.agent.test_l3_agent.MetadataL3AgentTestCase.test_access_to_metadata_proxy times out intermittently" [Critical,Confirmed] - Assigned to Miguel Lavalle (minsel) | 14:37 |
amuller | ihrachys: reading oslo logging code that should be doable | 14:37 |
*** markstur has quit IRC | 14:38 | |
*** prad- has joined #openstack-neutron | 14:38 | |
*** cbouch has quit IRC | 14:38 | |
*** sbelous has quit IRC | 14:39 | |
mlavalle | pc_m: it seems you have a patchset, https://review.openstack.org/#/c/196149, that creates a failure on purpose. This failure is what I found in the latest ocurrence of the bug in logstach | 14:39 |
*** sbelous_ has joined #openstack-neutron | 14:39 | |
*** sayali has quit IRC | 14:40 | |
mlavalle | pc_m: if you go to the latest comment in the bug, you can read more about my finding | 14:40 |
*** prad- has quit IRC | 14:41 | |
*** dmartls1 has joined #openstack-neutron | 14:41 | |
pc_m | mlavalle: 196149 is just a dummy commit that should fail some of the VPN functional tests. | 14:41 |
pc_m | mlavalle: I'm trying to test out a new experimental job that would run on Neutron patch sets, and invoke the VPN functional tests, to check cross repo breakages. | 14:42 |
mlavalle | pc_m: does thos VPN functional test involve L3 agent functional tests? | 14:42 |
mlavalle | that^^^ | 14:42 |
pc_m | mlavalle: Currently, when 196149 runs it is NOT using the actual Neutron patchset for the VPN test. | 14:42 |
*** Marga_ has joined #openstack-neutron | 14:43 | |
pc_m | mlavalle: It is a VPN functional job (pair actually) that sets up devstack, and then runs a VPN scenario test to make sure that VPN agent is working correctly. | 14:43 |
*** itlinux has quit IRC | 14:43 | |
*** sbelous has joined #openstack-neutron | 14:44 | |
mlavalle | pc_m: ok, I might be querying logstash wrongly.... will keep investigating | 14:44 |
pc_m | Now, if 196149 is run against neutron tests, it'll probably fail on tests that do rootwrap stuff (as it changes the return code to indicate an error). | 14:44 |
pc_m | mlavalle: please ignore any test results for 196149. | 14:45 |
pc_m | (hipefully it isn't throwing off the logstash queries) | 14:45 |
*** y_sawai has joined #openstack-neutron | 14:45 | |
*** sbelous has quit IRC | 14:45 | |
*** sbelous has joined #openstack-neutron | 14:45 | |
mlavalle | pc_m: yeah, but if I am querying logstash right, then the patchset is throwing it off signifcantly.... I'll just keep an eye on it and will confirm that I am queruing correctly | 14:46 |
mlavalle | pc_m: for the time being, you ignore me... thanks :-) | 14:46 |
pc_m | mlavalle: OK. If it is causing problems with you researching the bug, let me know and I can defer to next week (I'm on PTO tomorrow). | 14:48 |
*** phil_h has joined #openstack-neutron | 14:48 | |
mlavalle | pc_m: thanks. I'll mention this in the L3 meeting in a few minutes | 14:48 |
*** tbachman has quit IRC | 14:48 | |
*** dmartls1 has quit IRC | 14:49 | |
pc_m | see you there | 14:49 |
*** gampel has quit IRC | 14:49 | |
openstackgerrit | Assaf Muller proposed openstack/neutron: Retain logs for functional test cases https://review.openstack.org/219935 | 14:49 |
amuller | ihrachys: done | 14:50 |
amuller | ihrachys: now question is are we actually getting all of the logs | 14:50 |
*** y_sawai has quit IRC | 14:50 | |
amuller | ihrachys: the job is running 198 tests, I don't think we're getting 198 logs | 14:50 |
*** mickeys has joined #openstack-neutron | 14:51 | |
*** vikram__ has joined #openstack-neutron | 14:51 | |
*** dmartls1 has joined #openstack-neutron | 14:52 | |
*** carl_baldwin has joined #openstack-neutron | 14:52 | |
*** wznoinsk has quit IRC | 14:52 | |
amuller | mlavalle: I'll be joining the L3 meeting a bit late, but I'd love to get a mention on that bug :) | 14:52 |
*** amuller is now known as amuller_afk | 14:53 | |
*** anshul has quit IRC | 14:53 | |
*** txomon|fon has quit IRC | 14:54 | |
*** y_sawai has joined #openstack-neutron | 14:54 | |
*** maishsk has quit IRC | 14:57 | |
mlavalle | amuller_afk: if you mis it during the l3 meeting, ping me later and we can chat about it | 14:58 |
*** tbachman has joined #openstack-neutron | 14:58 | |
*** abregman is now known as abregman|afk | 14:58 | |
*** johnbelamaric has joined #openstack-neutron | 14:58 | |
*** y_sawai has quit IRC | 14:59 | |
*** emagana has joined #openstack-neutron | 15:01 | |
*** jistr is now known as jistr|call | 15:02 | |
*** ronis has quit IRC | 15:02 | |
*** Leonr has joined #openstack-neutron | 15:02 | |
*** armax has joined #openstack-neutron | 15:03 | |
*** rushil_ has quit IRC | 15:04 | |
*** kbringard has quit IRC | 15:04 | |
*** rotbeard has quit IRC | 15:04 | |
*** yamamoto has joined #openstack-neutron | 15:05 | |
*** Marga_ has quit IRC | 15:05 | |
*** sayali has joined #openstack-neutron | 15:05 | |
*** TrevorV has joined #openstack-neutron | 15:07 | |
openstackgerrit | gongysh proposed openstack/neutron: Add list routes https://review.openstack.org/218479 | 15:07 |
*** markvoelker has joined #openstack-neutron | 15:08 | |
*** txomon|fon has joined #openstack-neutron | 15:08 | |
*** txomon|fon has quit IRC | 15:08 | |
*** jbell8 has joined #openstack-neutron | 15:08 | |
*** arajagopal has quit IRC | 15:08 | |
*** sayali has quit IRC | 15:10 | |
*** moha_hunt has joined #openstack-neutron | 15:10 | |
*** tbachman has quit IRC | 15:12 | |
*** chutwig has joined #openstack-neutron | 15:14 | |
*** nate_gone is now known as njohnston | 15:14 | |
*** amuller_afk is now known as amuller | 15:14 | |
*** ivar-lazzaro has quit IRC | 15:15 | |
*** akamyshnikova has quit IRC | 15:15 | |
pc_m | armax: hi | 15:15 |
*** alexpilotti_ has joined #openstack-neutron | 15:16 | |
*** wwriverrat has quit IRC | 15:17 | |
*** alexpilotti has quit IRC | 15:17 | |
*** alexpilotti_ is now known as alexpilotti | 15:17 | |
*** shwetaap has joined #openstack-neutron | 15:18 | |
*** jprovazn has joined #openstack-neutron | 15:19 | |
*** rkukura has quit IRC | 15:20 | |
*** shaohe_feng1 has quit IRC | 15:21 | |
*** annegentle has joined #openstack-neutron | 15:21 | |
*** tbachman has joined #openstack-neutron | 15:22 | |
*** sayali has joined #openstack-neutron | 15:22 | |
*** SumitNaiksatam has quit IRC | 15:23 | |
*** SumitNaiksatam has joined #openstack-neutron | 15:23 | |
ihrachys | armax: hey. I have stuff to discuss around provider_configuration. | 15:23 |
armax | ihrachys: I was just going through the bug report | 15:23 |
armax | ihrachys: let me catch up | 15:23 |
ihrachys | armax: cool. we had discussion with dhellmann in #*-oslo too, you can read latest logs | 15:24 |
*** srilatha has joined #openstack-neutron | 15:24 | |
*** kbringard has joined #openstack-neutron | 15:25 | |
*** dims has quit IRC | 15:26 | |
armax | ihrachys: what is puzzling to me is how we regressed | 15:26 |
ihrachys | armax: easy: we don't gate neutron against puppet or delorean | 15:26 |
*** maishsk has joined #openstack-neutron | 15:26 | |
armax | ihrachys: since I haven’t touched the logic of loading the config | 15:26 |
armax | ihrachys: not that, that I get | 15:27 |
ihrachys | armax: haven't you? config_dir usage is in new code only | 15:27 |
armax | no | 15:27 |
armax | ihrachys: it was here: https://review.openstack.org/#/c/202207/23/neutron/common/repos.py | 15:27 |
armax | already | 15:27 |
ihrachys | ah nope, sorry, indeed it's in repos.py | 15:27 |
armax | ihrachys: something else must have happened | 15:28 |
*** neophy has joined #openstack-neutron | 15:28 | |
armax | ihrachys: so config_dir might be a red herring | 15:28 |
*** shivrao has joined #openstack-neutron | 15:28 | |
*** yamahata has quit IRC | 15:28 | |
ihrachys | EmilienM: re that puppet neutron regression: does it work in kilo though? | 15:28 |
*** matrohon has quit IRC | 15:29 | |
ihrachys | armax: the workaround in LP comments suggests it's not red herring, but maybe it's not that patch? | 15:29 |
armax | ihrachys: um that I don’t know yet, but it looks too much of a coincidence | 15:29 |
*** gampel has joined #openstack-neutron | 15:29 | |
*** arajagopal has joined #openstack-neutron | 15:30 | |
*** sthillma has joined #openstack-neutron | 15:30 | |
*** emagana has quit IRC | 15:30 | |
*** shivrao_ has joined #openstack-neutron | 15:30 | |
*** sthillma_ has joined #openstack-neutron | 15:31 | |
*** dannywilson has joined #openstack-neutron | 15:31 | |
EmilienM | ihrachys: yes | 15:31 |
*** emagana has joined #openstack-neutron | 15:32 | |
*** sayali has quit IRC | 15:32 | |
ihrachys | EmilienM: and it worked for L several days ago | 15:33 |
*** shivrao has quit IRC | 15:33 | |
*** shivrao_ is now known as shivrao | 15:33 | |
armax | ihrachys: going through the logs | 15:33 |
EmilienM | right | 15:33 |
*** tbachman has quit IRC | 15:33 | |
*** sthillma has quit IRC | 15:34 | |
*** sthillma_ is now known as sthillma | 15:34 | |
*** sballe has joined #openstack-neutron | 15:35 | |
armax | ihrachys: this part was removed | 15:35 |
armax | ihrachys: https://review.openstack.org/#/c/220070/1/neutron/services/provider_configuration.py line 75 | 15:35 |
armax | that’s what happens | 15:35 |
armax | I think | 15:36 |
armax | RDO must catch up perhaps? | 15:36 |
*** dims has joined #openstack-neutron | 15:36 | |
ihrachys | armax: delorean is always latest master | 15:36 |
armax | ihrachys: um... | 15:36 |
*** sayali has joined #openstack-neutron | 15:36 | |
ihrachys | (unless there are build issues) | 15:36 |
*** tbachman has joined #openstack-neutron | 15:36 | |
* armax keeps digging | 15:37 | |
*** y_sawai has joined #openstack-neutron | 15:37 | |
*** lajos-katona has quit IRC | 15:38 | |
*** y_sawai has quit IRC | 15:38 | |
ihrachys | armax: why don't we just drop that manual read of config files and rely on oslo.config? what's the limitation that made us avoid the lib? | 15:38 |
armax | ihrachys: the problem as I understand it is that we have the same config option | 15:39 |
armax | named the same way | 15:39 |
armax | in multiple files | 15:39 |
ihrachys | armax: does oslo.config override it instead of merging? I thought MultiStrOpt should merge. | 15:39 |
*** gampel1 has joined #openstack-neutron | 15:40 | |
ihrachys | but that's more common sense than actual knowledge | 15:40 |
armax | ihrachys: it doesn’t afaik | 15:40 |
*** amotoki has quit IRC | 15:40 | |
ihrachys | ok, then I get the rationale | 15:40 |
armax | ihrachys: still going through the logs | 15:41 |
*** gampel has quit IRC | 15:41 | |
*** markvoelker_ has joined #openstack-neutron | 15:42 | |
ihrachys | EmilienM: I think there could be recent changes in puppet for neutron. where can I check? I think I saw some patch recently that tried to adopt liberty. was it merged? | 15:44 |
*** jistr|call is now known as jistr | 15:44 | |
*** markvoelker has quit IRC | 15:45 | |
*** blogan_ has joined #openstack-neutron | 15:45 | |
*** wwriverrat has joined #openstack-neutron | 15:45 | |
*** garyk has quit IRC | 15:47 | |
*** gampel1 has quit IRC | 15:47 | |
*** gampel has joined #openstack-neutron | 15:47 | |
armax | ihrachys: see here: | 15:48 |
armax | http://logs.openstack.org/07/202207/23/check/gate-tempest-dsvm-neutron-full/b425da1/logs/screen-q-svc.txt.gz#_2015-09-01_04_03_27_841 | 15:48 |
armax | http://logs.openstack.org/50/216950/5/check/gate-puppet-neutron-puppet-beaker-rspec-dsvm-centos7/9e7e510/logs/neutron/server.txt.gz#_2015-09-01_12_40_22_734 | 15:48 |
armax | in the second log, the service providers are empty | 15:49 |
*** sthillma has quit IRC | 15:49 | |
*** shivrao has quit IRC | 15:49 | |
*** breitz has quit IRC | 15:49 | |
*** yamahata has joined #openstack-neutron | 15:50 | |
openstackgerrit | Dongcan Ye proposed openstack/neutron-vpnaas: Fix AH-ESP transform protocol in ipsec policy https://review.openstack.org/218788 | 15:50 |
*** absubram has joined #openstack-neutron | 15:51 | |
*** bharathm has joined #openstack-neutron | 15:51 | |
*** blogan__ has joined #openstack-neutron | 15:52 | |
ihrachys | armax: but in neutron_lbaas.conf, I still see "service_provider=LOADBALANCER:Haproxy:neutron_lbaas.services.loadbalancer.drivers.haproxy.plugin_driver.HaproxyOnHostPluginDriver:default" | 15:52 |
*** blogan_ has quit IRC | 15:52 | |
absubram | hi.. could someone please help me get a pointer to the code where the —config-dir, —config-file options are picked up from for the neutron server? | 15:53 |
armax | ihrachys: https://review.openstack.org/#/c/220070/1/neutron/services/provider_configuration.py something must have changed in parse_service_provider_opt | 15:53 |
armax | ihrachys: but not quite sure yet why and what | 15:54 |
HenryG | armax: sorry to interrupt | 15:55 |
HenryG | armax: I suspect it might be critical to get https://bugs.launchpad.net/bugs/1490767 into L-3 | 15:56 |
openstack | Launchpad bug 1490767 in neutron "DB migration of geneve type driver should be 'expand'" [High,In progress] - Assigned to Henry Gessau (gessau) | 15:56 |
*** rushil has joined #openstack-neutron | 15:56 | |
pc_m | absubram: I think it is in neutron/server/__init__.py | 15:56 |
*** SourabhP has joined #openstack-neutron | 15:56 | |
*** vikram has quit IRC | 15:56 | |
HenryG | armax: otherwise too many users will pick up a bad migration | 15:57 |
*** gampel1 has joined #openstack-neutron | 15:57 | |
absubram | pc_m: thanks! | 15:57 |
armax | HenryG: it’s too far down the queue | 15:57 |
armax | :( | 15:58 |
EmilienM | ihrachys: the only change was using rdo trunk | 15:58 |
*** nlahouti has joined #openstack-neutron | 15:58 | |
pc_m | absubram: I think it goes to common/config.py to parse config. | 15:58 |
amuller | armax: ihrachys: I just approved https://review.openstack.org/#/c/219935/ | 15:58 |
dhellmann | ihrachys, armax : afaict, oslo.config does merge multistr values from different sources: https://review.openstack.org/220210 | 15:58 |
*** SourabhP_ has joined #openstack-neutron | 15:58 | |
armax | dhellmann: you rock, thanks for confirming this publicly :) | 15:58 |
armax | dhellmann: sometimes else must be at play then | 15:59 |
dhellmann | armax: sure. next time ask! | 15:59 |
*** afazekas__ has quit IRC | 15:59 | |
*** gampel has quit IRC | 16:00 | |
*** jlibosva has quit IRC | 16:00 | |
*** sridhar_ram has joined #openstack-neutron | 16:00 | |
ihrachys | armax: well we don't use oslo.config to merge the values | 16:00 |
*** maishsk has quit IRC | 16:00 | |
*** SourabhP has quit IRC | 16:01 | |
*** SourabhP_ is now known as SourabhP | 16:01 | |
HenryG | armax: if it does not make L-3 we should probably abandon it instead. /cc ihrachys | 16:01 |
*** jlibosva has joined #openstack-neutron | 16:01 | |
armax | HenryG: why? | 16:01 |
absubram | pc_m: got it.. any idea is the —config-dir option has been removed?.. it’s what I’m looking for.. for some reason that config is not being picked up | 16:01 |
*** ihrachys_ has joined #openstack-neutron | 16:02 | |
HenryG | armax: because patching the migration timeline should only be done if we are fairly sure it will not affect anyone | 16:02 |
armax | HenryG: but L3 is not going to be the final release | 16:02 |
armax | HenryG: there is going to be an RC1, etc... | 16:02 |
ihrachys_ | armax: some people may want to rebase their distros to L3 | 16:03 |
armax | ihrachys_: they would be nuts, but ok | 16:03 |
armax | ihrachys_: it’s a possibility | 16:03 |
ihrachys_ | armax: I believe zigo once tracked milestones | 16:03 |
*** ihrachys has quit IRC | 16:05 | |
*** jlanoux has quit IRC | 16:05 | |
openstackgerrit | Oleg Bondarev proposed openstack/neutron: OVS agent: handle deleted ports on each rpc_loop iteration https://review.openstack.org/220140 | 16:05 |
openstackgerrit | Oleg Bondarev proposed openstack/neutron: OVS agent: flush firewall rules for all deleted ports at once https://review.openstack.org/220115 | 16:05 |
pc_m | absubram: I'm not really sure. | 16:06 |
*** jistr has quit IRC | 16:06 | |
*** jaypipes has joined #openstack-neutron | 16:07 | |
*** ijw_ has joined #openstack-neutron | 16:08 | |
*** blogan__ has quit IRC | 16:09 | |
*** abregman|afk is now known as abregman | 16:09 | |
*** blogan_ has joined #openstack-neutron | 16:09 | |
*** s3wong has joined #openstack-neutron | 16:10 | |
*** alex_klimov has quit IRC | 16:11 | |
*** slaweq has joined #openstack-neutron | 16:11 | |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Enable most unit tests for py34 job https://review.openstack.org/219389 | 16:11 |
*** aix has quit IRC | 16:12 | |
*** Piet has joined #openstack-neutron | 16:12 | |
*** VikasC has joined #openstack-neutron | 16:12 | |
*** ygbo has quit IRC | 16:13 | |
*** crose has joined #openstack-neutron | 16:14 | |
*** ihrachys_ has quit IRC | 16:14 | |
*** ihrachys has joined #openstack-neutron | 16:14 | |
*** ivar-lazzaro has joined #openstack-neutron | 16:15 | |
*** vikram__ has quit IRC | 16:17 | |
*** sbelous has quit IRC | 16:19 | |
*** sbelous has joined #openstack-neutron | 16:19 | |
HenryG | absubram: why are you asking about --config-dir? It seems armax and ihrachys are discussing some regression with it. | 16:20 |
*** ivar-lazzaro has quit IRC | 16:20 | |
*** neiljerr` has joined #openstack-neutron | 16:21 | |
*** l6unchpad has quit IRC | 16:21 | |
armax | ihrachys: this is from a build prior to the merge of my patch | 16:21 |
*** banix has joined #openstack-neutron | 16:21 | |
armax | ihrachys: http://logs.openstack.org/50/216950/4/check/gate-puppet-neutron-puppet-beaker-rspec-dsvm-centos7/cb6a577/logs/neutron/server.txt.gz#_2015-08-31_19_22_40_424 | 16:21 |
*** maishsk has joined #openstack-neutron | 16:21 | |
armax | clearly removing https://review.openstack.org/#/c/220070/1/neutron/services/provider_configuration.py | 16:21 |
armax | ihrachys line 95-91 | 16:21 |
armax | ihrachys: is upsetting things | 16:21 |
*** neiljerram has quit IRC | 16:21 | |
armax | ihrachys: now I wonder why it upsets RDO and not devstack | 16:22 |
ihrachys | armax: devstack does not use multiple --config-dirs | 16:23 |
*** tmorin has quit IRC | 16:23 | |
armax | ihrachys: gotcha | 16:23 |
*** ivar-lazzaro has joined #openstack-neutron | 16:23 | |
ihrachys | armax: remember I proposed to use it? you said it's yagni :) | 16:23 |
absubram | HenryG: thanks for pointing this out.. it is excalty the same issue.. | 16:24 |
absubram | the —config-dir option is not being picked up | 16:24 |
rossella_s | mlavalle if you need help with https://bugs.launchpad.net/neutron/+bug/1461172 feel free to ping me | 16:25 |
openstack | Launchpad bug 1461172 in neutron "neutron.tests.functional.agent.test_l3_agent.MetadataL3AgentTestCase.test_access_to_metadata_proxy times out intermittently" [Critical,Confirmed] - Assigned to Miguel Lavalle (minsel) | 16:25 |
ihrachys | absubram: can you tell what's your issue? | 16:25 |
*** sbelous has quit IRC | 16:25 | |
*** chutwig has quit IRC | 16:25 | |
absubram | we are trying out baremetal deployments along with using Heat templates and puppet.. we see the overcloud nodes being deployed | 16:26 |
absubram | and the ini files all being written correctly | 16:26 |
*** sayali has quit IRC | 16:26 | |
absubram | but neutron-server doesn’t seem to read the ini file we have in the —config-dir option | 16:26 |
armax | ihrachys: what’s weird though is that the option is not in here: | 16:27 |
armax | http://logs.openstack.org/50/216950/4/check/gate-puppet-neutron-puppet-beaker-rspec-dsvm-centos7/cb6a577/logs/etc/neutron/neutron.conf.txt.gz | 16:27 |
armax | absubram: we’re actively looking for a fix | 16:27 |
ihrachys | armax: it's passed in args | 16:27 |
absubram | ihrachys: ExecStart=/usr/bin/neutron-server --config-file /usr/share/neutron/neutron-dist.conf --config-dir /usr/share/neutron/server --config-file /etc/neutron/neutron.conf --config-file /etc/neutron/plugin.ini --config-dir /etc/neutron/conf.d/common --config-dir /etc/neutron/conf.d/neutron-server --log-file /var/log/neutron/server.log | 16:27 |
absubram | that’s how we start neutron | 16:27 |
ihrachys | absubram: we ? | 16:27 |
armax | ihrachys: what is? | 16:28 |
absubram | internally here in Cisco :) | 16:28 |
ihrachys | absubram: get it. I believe you base on RDO/Delorean | 16:28 |
absubram | yes | 16:28 |
*** chutwig has joined #openstack-neutron | 16:29 | |
ihrachys | so yes, that's the same issue we discuss. puppet-neutron CI was broken because they use delorean among other things | 16:29 |
ihrachys | armax: the config-dir option is not defined in neutron.conf, but in args to neutron-server | 16:29 |
ihrachys | as in snippet above | 16:30 |
absubram | ah.. ok.. thanks armax, ihrachys | 16:30 |
armax | ihrachys: I meant the service_providers option in neutron.conf | 16:30 |
*** vikram__ has joined #openstack-neutron | 16:30 | |
ihrachys | armax: I believe it should not be put there anymore, right? | 16:30 |
armax | ihrachys: correct | 16:30 |
armax | ihrachys: and indeed it isn't | 16:30 |
ihrachys | armax: I believe all the mess could be fixed by getting rid of provider_configuration. why not leaving it to oslo.config to read the value? are we still afraid to break someone? | 16:31 |
*** bdemers has joined #openstack-neutron | 16:32 | |
armax | ihrachys: at this point no | 16:32 |
*** vivek-ebay has joined #openstack-neutron | 16:32 | |
armax | ihrachys: let post a patch to see if I can clear this mess | 16:32 |
ihrachys | armax: ok. I honestly feel myself uncapable to come up with better ideas here, I still lack full understanding of the code. | 16:33 |
*** neiljerr` has quit IRC | 16:33 | |
*** ronis has joined #openstack-neutron | 16:34 | |
*** sayali has joined #openstack-neutron | 16:35 | |
*** sbelous has joined #openstack-neutron | 16:36 | |
armax | ihrachys: bear with me while I whip something up | 16:36 |
*** annegentle has quit IRC | 16:36 | |
*** SourabhP has quit IRC | 16:36 | |
*** annegentle has joined #openstack-neutron | 16:37 | |
*** minwang2 has joined #openstack-neutron | 16:39 | |
*** afazekas__ has joined #openstack-neutron | 16:41 | |
*** srilatha has quit IRC | 16:41 | |
*** vikram__ has quit IRC | 16:47 | |
*** SourabhP has joined #openstack-neutron | 16:47 | |
*** nyechiel has quit IRC | 16:48 | |
*** garyk has joined #openstack-neutron | 16:49 | |
armax | ihrachys: btw, you said that devstack doesn’t use config-dir | 16:50 |
ihrachys | armax: not two --config-dir args | 16:50 |
ihrachys | maybe one | 16:50 |
armax | ihrachys: not that I can se | 16:50 |
armax | e | 16:50 |
armax | http://logs.openstack.org/07/202207/23/check/gate-neutron-lbaasv2-dsvm-api/99778b6/logs/screen-q-svc.txt.gz | 16:50 |
armax | here for instance | 16:50 |
armax | we don’t pass the *aas conf file | 16:50 |
*** bharathm has quit IRC | 16:50 | |
ihrachys | armax: because it's loaded by provider_configuration, no? | 16:51 |
armax | ihrachys: right | 16:51 |
*** clev is now known as clev-away | 16:52 | |
*** Swami has joined #openstack-neutron | 16:52 | |
armax | so we can use config_dir, but for the time being (to avoid breaking devstack) we need to keep both logics | 16:52 |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Fix fip agent gateway port delete when migrating a fip https://review.openstack.org/180408 | 16:53 |
*** vikram has joined #openstack-neutron | 16:53 | |
*** maishsk has quit IRC | 16:54 | |
*** s3wong has quit IRC | 16:54 | |
*** s3wong has joined #openstack-neutron | 16:55 | |
*** hswayne77 has joined #openstack-neutron | 16:55 | |
*** klindgren_ has joined #openstack-neutron | 16:55 | |
klindgren_ | dougwig, ping | 16:55 |
ihrachys | armax: I feel like my brain fails to follow the logic. I will leave now and maybe return later today to check the status. if you don't complete the fix today, please update the bug with findings. | 16:56 |
armax | ihrachys: I am filing a fix shortly | 16:56 |
*** jpena has quit IRC | 16:56 | |
ihrachys | ok, I'll check it in 1-2h | 16:56 |
*** safchain has quit IRC | 16:56 | |
armax | ihrachys: for the time being I am thinking of allowing both scenarios where one can load neutron by passing config-dir or just a single config-file and let the modules load their providers automagically | 16:57 |
*** bharathm has joined #openstack-neutron | 16:57 | |
*** ihrachys has quit IRC | 16:57 | |
armax | ihrachys: if we intend to keep devstack work the way it does, that’s our best bet | 16:57 |
pc_m | armax: Casually following the discussion... doesn't VPN use multiple --config-file args? | 16:58 |
*** sbelous has quit IRC | 16:58 | |
armax | pc_m: it should, but can you find out a run from the gate? | 16:59 |
pc_m | armax: Will check | 16:59 |
*** maishsk has joined #openstack-neutron | 17:02 | |
ajmiller | garyk markmcclain could I please get reviews of this stable/kilo backport? https://review.openstack.org/#/c/216810/ | 17:03 |
pc_m | armax: tempest uses neutron.conf and ml2_conf.ini for neutron. For VPN, the functional tests don't stack, they just configure devstack. | 17:03 |
*** lazy_prince has quit IRC | 17:04 | |
pc_m | so guess it is not an issue for VPN tests. | 17:04 |
*** rossella_s has quit IRC | 17:05 | |
*** jlibosva has quit IRC | 17:06 | |
*** rossella_s has joined #openstack-neutron | 17:06 | |
*** jlibosva has joined #openstack-neutron | 17:06 | |
*** nyechiel has joined #openstack-neutron | 17:07 | |
*** blogan_ has quit IRC | 17:07 | |
*** afazekas__ has quit IRC | 17:08 | |
pc_m | armax: I see for grenade multiple config-file entries are used when starting the agent. | 17:12 |
armax | pc_m: looking | 17:12 |
pc_m | http://logs.openstack.org/49/196149/8/check/gate-grenade-dsvm-neutron/0bed589/logs/grenade.sh.txt.gz#_2015-09-01_12_12_49_457 | 17:12 |
*** afazekas__ has joined #openstack-neutron | 17:12 | |
*** maishsk has quit IRC | 17:12 | |
armax | pc_m: not here: | 17:13 |
armax | http://logs.openstack.org/49/196149/8/check/gate-grenade-dsvm-neutron/0bed589/logs/grenade.sh.txt.gz#_2015-09-01_12_01_48_647 | 17:13 |
armax | pc_m: that’s what we care about | 17:13 |
*** azbiswas has joined #openstack-neutron | 17:13 | |
armax | the server-side rather than the agent side | 17:13 |
pc_m | armax: gotcha | 17:13 |
pc_m | will that still have ml2 and neutron confs? | 17:14 |
armax | yes | 17:14 |
*** carl_baldwin has quit IRC | 17:14 | |
*** achanda has joined #openstack-neutron | 17:14 | |
*** jbell8 has quit IRC | 17:15 | |
*** kawa2014 has quit IRC | 17:15 | |
*** madhu_ak has joined #openstack-neutron | 17:16 | |
*** markvoelker_ has quit IRC | 17:16 | |
openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: VPNaaS: Add debugging to tox_install.sh https://review.openstack.org/203201 | 17:19 |
*** tflynn has joined #openstack-neutron | 17:20 | |
*** achanda has quit IRC | 17:21 | |
*** sthillma has joined #openstack-neutron | 17:21 | |
*** julim has quit IRC | 17:21 | |
*** abhiraut has joined #openstack-neutron | 17:21 | |
*** afazekas__ has quit IRC | 17:21 | |
*** kairo has quit IRC | 17:22 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port https://review.openstack.org/194441 | 17:22 |
dougwig | klindgren_: ack | 17:22 |
*** achanda has joined #openstack-neutron | 17:22 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent https://review.openstack.org/194446 | 17:22 |
pc_m | armax: dougwig: Can you guys review https://review.openstack.org/#/c/209887/ that adds experimental job for neutron-client. In Neutron meeting we agreed to have a job for core changes and one for adv. svcs. Infra folks would like to see one of you guys review 1st. This is to clean up devstack plugin and re-enable VPN tests. | 17:23 |
*** sthillma has quit IRC | 17:26 | |
openstackgerrit | Roman Bogorodskiy proposed openstack/neutron: sriov: update port state even if ip link fails https://review.openstack.org/195060 | 17:27 |
*** banix_ has joined #openstack-neutron | 17:29 | |
*** bharathm has quit IRC | 17:29 | |
*** banix has quit IRC | 17:30 | |
*** banix_ is now known as banix | 17:30 | |
garyk | ajmiller: lgtm | 17:31 |
ajmiller | garyk thanks! | 17:31 |
*** saksham has joined #openstack-neutron | 17:31 | |
*** abhiraut has quit IRC | 17:32 | |
*** annegentle has quit IRC | 17:33 | |
*** SourabhP has quit IRC | 17:34 | |
amuller | armax: I think we have a new failure: message:"Details: {u'message': u'Network not found', u'code': 404}" AND tags:"console" AND build_name:"gate-tempest-dsvm-neutron-full" | 17:34 |
armax | that’s new | 17:35 |
amuller | armax: 9 unique hits from today | 17:35 |
armax | amuller: the one about ports is gone | 17:35 |
amuller | it started today | 17:35 |
amuller | armax++ | 17:35 |
armax | amuller: but we have traded one for another? | 17:35 |
armax | oh joy | 17:35 |
armax | amuller: bug #? | 17:35 |
amuller | armax: not sure anyone reported a bug yet | 17:36 |
amuller | it's brand new :) | 17:36 |
*** abhiraut has joined #openstack-neutron | 17:36 | |
*** divyac has joined #openstack-neutron | 17:36 | |
*** vikram has quit IRC | 17:36 | |
divyac | Anand Shanmugam | 17:36 |
*** gampel1 has quit IRC | 17:38 | |
*** gampel has joined #openstack-neutron | 17:39 | |
*** luqas has joined #openstack-neutron | 17:39 | |
*** srilatha has joined #openstack-neutron | 17:39 | |
*** hswayne77 has quit IRC | 17:40 | |
*** annegentle has joined #openstack-neutron | 17:42 | |
*** xgerman is now known as the_real_blogan | 17:42 | |
*** the_real_blogan is now known as xgerman | 17:42 | |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: Make sure service providers can be loaded correctly https://review.openstack.org/220254 | 17:42 |
*** absubram has quit IRC | 17:43 | |
*** vishwanathj has quit IRC | 17:43 | |
*** jlibosva has quit IRC | 17:44 | |
*** gsagie_ has joined #openstack-neutron | 17:45 | |
*** jbell8 has joined #openstack-neutron | 17:46 | |
*** sballe is now known as sballe_ | 17:47 | |
*** abhiraut has quit IRC | 17:47 | |
*** ijw_ has quit IRC | 17:48 | |
*** sballe_ is now known as sballe | 17:49 | |
*** sayali has quit IRC | 17:50 | |
armax | amuller: you mean this one? | 17:50 |
armax | http://logs.openstack.org/46/153946/136/gate/gate-tempest-dsvm-neutron-full/05cfd06/logs/testr_results.html.gz | 17:50 |
armax | bummer | 17:50 |
amuller | armax: ya | 17:50 |
amuller | armax: reporting bug as we speak | 17:50 |
armax | amuller: cool | 17:50 |
armax | assign it to me, I’ll triage it | 17:50 |
amuller | armax: https://bugs.launchpad.net/neutron/+bug/1491965 | 17:51 |
openstack | Launchpad bug 1491965 in neutron "test_list_show_tenant_networks triggering error "itemNotFound": {"message": "Network not found", "code": 404}" [Critical,New] - Assigned to Armando Migliaccio (armando-migliaccio) | 17:51 |
armax | thanks | 17:51 |
*** azbiswas has quit IRC | 17:52 | |
*** kbringard has quit IRC | 17:52 | |
*** aranjan has joined #openstack-neutron | 17:52 | |
*** clev-away is now known as clev | 17:53 | |
*** kbringard has joined #openstack-neutron | 17:53 | |
*** jckasper has quit IRC | 17:53 | |
amuller | armax: looks like another test is deleting that network | 17:53 |
*** aranjan has quit IRC | 17:53 | |
*** sthillma has joined #openstack-neutron | 17:53 | |
armax | amuller: need to step away for a bit | 17:53 |
armax | but I’ll have time to look at it today | 17:53 |
amuller | armax: ok, I'll add info to the bug | 17:53 |
*** SourabhP has joined #openstack-neutron | 17:53 | |
*** sayali has joined #openstack-neutron | 17:53 | |
*** bradjones has joined #openstack-neutron | 17:53 | |
*** bradjones has quit IRC | 17:53 | |
*** bradjones has joined #openstack-neutron | 17:53 | |
*** aranjan has joined #openstack-neutron | 17:53 | |
*** emagana has quit IRC | 17:54 | |
*** bharathm has joined #openstack-neutron | 17:54 | |
*** carl_baldwin has joined #openstack-neutron | 17:54 | |
*** carl_baldwin has quit IRC | 17:54 | |
*** debedb has joined #openstack-neutron | 17:55 | |
*** carl_baldwin has joined #openstack-neutron | 17:55 | |
*** evgenyf has joined #openstack-neutron | 17:55 | |
*** gampel has quit IRC | 17:55 | |
*** VikasC has quit IRC | 17:55 | |
*** azbiswas has joined #openstack-neutron | 17:55 | |
*** nlahouti has quit IRC | 17:56 | |
*** aranjan has quit IRC | 17:56 | |
*** nlahouti has joined #openstack-neutron | 17:56 | |
*** aranjan has joined #openstack-neutron | 17:56 | |
*** VikasC has joined #openstack-neutron | 17:57 | |
*** nlahouti has quit IRC | 17:57 | |
kevinbenton | armax, amuller: this had come up before | 17:57 |
kevinbenton | that test was bad for neutron anyway | 17:57 |
kevinbenton | it assumes networks never change | 17:58 |
amuller | I'm looking at the test right now | 17:58 |
*** shivrao has joined #openstack-neutron | 17:58 | |
amuller | yeah it looks like a really stupid test? I don't get it | 17:58 |
kevinbenton | it's for nova net | 17:58 |
amuller | BTW observing the logs I see which (other) test is deleting the network | 17:58 |
amuller | ok | 17:58 |
amuller | kevinbenton: so, let's just skip it for Neutron | 17:58 |
kevinbenton | amuller: but it's this test that is wrong even though it doesn't immediately appear that way | 17:58 |
*** julim has joined #openstack-neutron | 17:58 | |
kevinbenton | amuller: https://review.openstack.org/#/c/195345/ | 17:58 |
*** nlahouti has joined #openstack-neutron | 18:01 | |
divyac | Hi i have a simple change for review (backport from master to kilo), Could any one please review this change? https://review.openstack.org/#/c/217843/ | 18:02 |
HenryG | kevinbenton: amuller: can we request tempest to merge the skip now? Salvatore can follow up later with a redesign of the test. | 18:03 |
*** tfukushima has quit IRC | 18:03 | |
amuller | that makes sense to me | 18:03 |
kevinbenton | HenryG: +1 | 18:04 |
amuller | I guess I'll propose another patch | 18:05 |
amuller | since salv is offline | 18:05 |
amuller | actually I just rebased the patch =D | 18:05 |
amuller | that removed the WIP heh | 18:05 |
HenryG | amuller: change commit msg to "Related-bug" | 18:05 |
*** sayali has quit IRC | 18:06 | |
*** wwriverrat has quit IRC | 18:07 | |
*** bobmel has quit IRC | 18:08 | |
*** carl_baldwin has quit IRC | 18:08 | |
*** rkukura has joined #openstack-neutron | 18:08 | |
markmcclain | ajmiller: approved | 18:13 |
*** abregman has quit IRC | 18:15 | |
*** abhiraut has joined #openstack-neutron | 18:17 | |
dougwig | pc_m: is that test failure good news? | 18:17 |
pc_m | In 169149? Yes. | 18:18 |
ajmiller | markmcclain thanks! | 18:18 |
pc_m | dougwig: I think I have a mechanism in 203201, that works for the VPN functional tests. Need you to look it over to see if it looks right, though. | 18:19 |
*** debedb has quit IRC | 18:20 | |
dougwig | pc_m: interesting. i think if you add a -e to that pip install, it'd be good. | 18:21 |
*** roeyc has joined #openstack-neutron | 18:21 | |
*** debedb has joined #openstack-neutron | 18:22 | |
*** ameybhide has joined #openstack-neutron | 18:22 | |
pc_m | dougwig: instead of the -U? | 18:22 |
pc_m | dougwig: or in addition? | 18:23 |
dougwig | pc_m: both | 18:23 |
pc_m | gotcha. | 18:23 |
pc_m | Will try to revise. | 18:24 |
*** sbelous has joined #openstack-neutron | 18:24 | |
pc_m | Do you think that will work for all cases? | 18:24 |
pc_m | (the script) | 18:24 |
pc_m | dougwig: Looks like the script is causing a problem with py34 test run. | 18:27 |
openstackgerrit | Armando Migliaccio proposed openstack/neutron: CONF.reset does not happen for all tests https://review.openstack.org/219878 | 18:29 |
*** rushil has quit IRC | 18:29 | |
armax | amuller: I have taken https://review.openstack.org/#/c/219878/ out of the gate unfortuntately | 18:29 |
armax | it was failing the py27 unit tests | 18:30 |
armax | failure here: http://logs.openstack.org/78/219878/3/gate/gate-neutron-python27/68420e4/ | 18:30 |
amuller | wat | 18:30 |
amuller | armax: ok | 18:30 |
*** sbelous has quit IRC | 18:30 | |
amuller | armax: dealing with the tempest failure now in openstack-qa | 18:30 |
armax | ok | 18:31 |
armax | just a heads-up | 18:31 |
*** harlowja has quit IRC | 18:31 | |
*** annegentle has quit IRC | 18:31 | |
*** localloop127 has joined #openstack-neutron | 18:31 | |
*** harlowja has joined #openstack-neutron | 18:32 | |
*** rkukura has quit IRC | 18:32 | |
amuller | armax: it was too simple :) | 18:32 |
*** roeyc has quit IRC | 18:33 | |
amuller | perhaps salv will have insight, he created the diet base class, and it looks like it was not resetting the conf purposely (According to the comments) | 18:33 |
amuller | I just have no idea why | 18:33 |
*** aditirajagopal has joined #openstack-neutron | 18:33 | |
*** mickeys has quit IRC | 18:34 | |
armax | let’s dig around to see whether that trace happened before | 18:35 |
debedb | hi all, question on the pluggable ipam… if an ipam driver plugin needs some configuration, what is the right way of getting it ? | 18:35 |
armax | it may not be rlated to your move at all | 18:35 |
openstackgerrit | James Arendt proposed openstack/neutron: WIP: Fix Neutron flavor framework https://review.openstack.org/219486 | 18:35 |
amuller | armax: maybe | 18:35 |
amuller | kevinbenton: HenryG: Did you happen to catch the convo in openstack-qa? Looks like the situation is a bit more complicated than we thought | 18:36 |
HenryG | amuller: yes, trying to grok | 18:37 |
*** ajmiller has quit IRC | 18:38 | |
*** arajagopal has left #openstack-neutron | 18:39 | |
*** aditirajagopal is now known as arajagopal_ | 18:39 | |
*** arajagopal_ has quit IRC | 18:40 | |
*** aditirajagopal has joined #openstack-neutron | 18:40 | |
*** aditirajagopal is now known as arajagopal_ | 18:40 | |
*** ajmiller has joined #openstack-neutron | 18:40 | |
pc_m | dougwig: Any idea what the py34 failure is with tox_install.sh? Error code 1, but I'm not sure what the failure is. | 18:41 |
*** emagana has joined #openstack-neutron | 18:46 | |
*** debedb has quit IRC | 18:48 | |
*** slaweq has quit IRC | 18:48 | |
*** minwang2 has quit IRC | 18:49 | |
*** ijw_ has joined #openstack-neutron | 18:50 | |
*** yamahata has quit IRC | 18:50 | |
*** tongli has joined #openstack-neutron | 18:51 | |
*** yamahata has joined #openstack-neutron | 18:51 | |
*** nlahouti has quit IRC | 18:51 | |
*** tongli has quit IRC | 18:52 | |
*** s3wong has quit IRC | 18:53 | |
*** nlahouti has joined #openstack-neutron | 18:54 | |
*** ijw_ has quit IRC | 18:55 | |
daneyon_ | gsagie gsagie_ It would be great to have a joint magnum/kuryr meeting at the design summit to work through integration details, roadmap, etc.. Does the Kuryr team have any plans established for the design summit? | 18:57 |
*** armax has quit IRC | 18:57 | |
*** debedb has joined #openstack-neutron | 18:58 | |
*** nlahouti has quit IRC | 18:58 | |
*** cbouch has joined #openstack-neutron | 18:58 | |
gsagie_ | daneyon_ : hi, yes we will probably have a time in the design summit for Neutron | 18:58 |
gsagie_ | Do you have one for Magnum networking as well? we can discuss it there as well | 18:59 |
*** nlahouti has joined #openstack-neutron | 18:59 | |
*** VikasC has quit IRC | 19:00 | |
*** jprovazn has quit IRC | 19:00 | |
*** annegentle has joined #openstack-neutron | 19:01 | |
*** amuller is now known as amuller_afk | 19:02 | |
*** dedery has joined #openstack-neutron | 19:04 | |
*** tbachman has quit IRC | 19:04 | |
*** srilatha has quit IRC | 19:06 | |
*** srilatha has joined #openstack-neutron | 19:06 | |
*** ivar-lazzaro has quit IRC | 19:09 | |
*** yfried_ has joined #openstack-neutron | 19:09 | |
*** srilatha has quit IRC | 19:10 | |
*** azbiswas has quit IRC | 19:11 | |
*** azbiswas has joined #openstack-neutron | 19:11 | |
*** pc_m is now known as pc_m_AWAY | 19:12 | |
*** aranjan has quit IRC | 19:12 | |
*** azbiswas has quit IRC | 19:16 | |
*** SourabhP has quit IRC | 19:16 | |
*** SourabhP has joined #openstack-neutron | 19:16 | |
openstackgerrit | Michael Bayer proposed openstack/neutron: Add non-model index names to autogen exclude filters https://review.openstack.org/213303 | 19:17 |
openstackgerrit | Michael Bayer proposed openstack/neutron: Implement expand/contract autogenerate extension https://review.openstack.org/203842 | 19:17 |
*** nyechiel has quit IRC | 19:17 | |
*** achanda has quit IRC | 19:18 | |
*** jobewan has joined #openstack-neutron | 19:19 | |
*** madhu_ak has quit IRC | 19:20 | |
*** abhiraut has quit IRC | 19:21 | |
*** nyechiel has joined #openstack-neutron | 19:21 | |
*** john-davidge has quit IRC | 19:21 | |
*** alex_klimov has joined #openstack-neutron | 19:23 | |
*** rtheis has quit IRC | 19:25 | |
*** rtheis has joined #openstack-neutron | 19:26 | |
*** alexpilotti has quit IRC | 19:26 | |
*** korzen has joined #openstack-neutron | 19:26 | |
*** arajagopal_ has quit IRC | 19:27 | |
*** tbachman has joined #openstack-neutron | 19:27 | |
*** Marga_ has joined #openstack-neutron | 19:29 | |
*** _cjones_ has joined #openstack-neutron | 19:29 | |
*** armax has joined #openstack-neutron | 19:29 | |
*** dguerri is now known as dguerri` | 19:29 | |
*** changbl has quit IRC | 19:30 | |
*** yfried_ has quit IRC | 19:31 | |
*** _cjones_ has quit IRC | 19:31 | |
*** _cjones_ has joined #openstack-neutron | 19:32 | |
*** gsagie_ has quit IRC | 19:32 | |
*** thumpba has joined #openstack-neutron | 19:32 | |
*** matrohon has joined #openstack-neutron | 19:33 | |
*** ijw_ has joined #openstack-neutron | 19:34 | |
*** kbringard has quit IRC | 19:34 | |
*** yfried_ has joined #openstack-neutron | 19:34 | |
*** _cjones_ has quit IRC | 19:36 | |
*** saksham has quit IRC | 19:39 | |
*** banix has quit IRC | 19:41 | |
*** ihrachys has joined #openstack-neutron | 19:41 | |
*** ihrachys has quit IRC | 19:41 | |
*** ihrachys has joined #openstack-neutron | 19:41 | |
*** yamamoto has quit IRC | 19:42 | |
daneyon_ | gsagie_ Yes, we will have a magnum networking design summit session. Can you keep me in the loop as you work out the details? I would like to make sure the 2 teams get together at the design summit. | 19:44 |
*** minwang2 has joined #openstack-neutron | 19:44 | |
*** amuller_afk is now known as amuller | 19:44 | |
*** madhu_ak has joined #openstack-neutron | 19:46 | |
zzzeek | ihrachys: ping | 19:47 |
ihrachys | zzzeek: pong | 19:47 |
zzzeek | “overall, we won't have new per-release branches” - means….. “expand” “contract”, but not “liberty/expand” “liberty/contract” ? | 19:47 |
zzzeek | im confused | 19:47 |
zzzeek | i see liberty there still | 19:47 |
armax | amuller: so is anyone looking into https://bugs.launchpad.net/tempest/+bug/1468523 | 19:48 |
openstack | Launchpad bug 1468523 in tempest "tempest.api.compute.test_tenant_networks should be skipped when neutron is enabled" [Undecided,In progress] - Assigned to Assaf Muller (amuller) | 19:48 |
armax | ? | 19:48 |
amuller | armax: I left a comment on the patch, and there's new comments on the bug by kevinbenton and mtreinish | 19:48 |
armax | ok | 19:48 |
*** vivek-ebay has quit IRC | 19:48 | |
amuller | it's not clear to me why it's happening | 19:48 |
amuller | it looks like a tenant is for some reason seeing an external network created by another tenant | 19:49 |
* armax looks | 19:49 | |
armax | kevinbenton: are you by any chance looking into it? | 19:49 |
*** kbringard has joined #openstack-neutron | 19:49 | |
ihrachys | zzzeek: it's in file names but not in script chain. | 19:49 |
zzzeek | ihrachys: if im to “expand” to liberty and “expand” to M, i just do “Expand” and the idea is, the source tree I’ve checked out is only one major release past the one I have anyway ? | 19:49 |
*** ivar-lazzaro has joined #openstack-neutron | 19:50 | |
zzzeek | ihrachys: are we going to update the blueprint? this is a big change? | 19:50 |
ihrachys | zzzeek: I think we updated devref with that. not sure it's worth updating a spec? | 19:50 |
zzzeek | ihrachys: jsut the spec is not what we’re doing anymore here. seems confusing. if the spec had been updated i woudl have noticed this aspect of the model is changing :) | 19:51 |
ihrachys | zzzeek: the rationale was noted at: https://review.openstack.org/#/c/215076/ | 19:51 |
*** azbiswas has joined #openstack-neutron | 19:51 | |
ihrachys | zzzeek: it's mostly the same, it's a tiny bit different. the core idea is left intact. | 19:52 |
zzzeek | ihrachys: feel free to link me on major changes to what I spec’ed :) reading now | 19:52 |
*** slaweq has joined #openstack-neutron | 19:53 | |
ihrachys | zzzeek: I actually thought you were looped in that one. But now I see we forgot to add you to the list there. sorry for that. | 19:53 |
*** ivar-lazzaro has quit IRC | 19:53 | |
zzzeek | ihrachys: what im not sure of is, is it safe to “Expand” from kilo->liberty->mitaka at once, then “contract” across all three at once | 19:53 |
ihrachys | I added you for the dep patch, and thought I did for that one too | 19:53 |
zzzeek | ihrachys: the reason for the leapfrogging was to prevent that | 19:53 |
*** ivar-lazzaro has joined #openstack-neutron | 19:53 | |
ihrachys | zzzeek: I believe it should work assuming we don't have depends_on from expand to contract in the row (that's one thing to tackle too - to guard against it somehow). | 19:54 |
*** evgenyf has quit IRC | 19:56 | |
*** nlahouti has quit IRC | 19:57 | |
*** arajagop_ has joined #openstack-neutron | 19:57 | |
zzzeek | ihrachys: im not sure either that the way we had it allows “liberty_expand@<some_revision_from_mitaka>” | 19:58 |
zzzeek | because mitaka and liberty are different bases | 19:58 |
zzzeek | they are only linked via depends_on | 19:58 |
zzzeek | not the same branches | 19:58 |
zzzeek | if that’s the only problem with how it was, we should just disallow complex versions like that from neutron-db-manage anyway | 19:58 |
zzzeek | neutron-db-manage should only allow simple, major targets | 19:59 |
*** emagana has quit IRC | 20:00 | |
ihrachys | zzzeek: if we would link with depends_on, we would have as much heads as we have releases * 2. doesn't sound ideal. so the idea was that we actually link them with down_revision instead. | 20:01 |
ihrachys | zzzeek: some people chase master, they want to upgrade in steps | 20:01 |
*** Marga_ has quit IRC | 20:01 | |
ihrachys | armax: I left silly comments in https://review.openstack.org/#/c/220254/, please have a look | 20:01 |
zzzeek | ihrachys: I understand that use case, I’d propose they use the alembic CLI directly | 20:01 |
armax | ihrachys: looking | 20:01 |
armax | ihrachys: all in all what do you think? | 20:02 |
zzzeek | ihrachys: its true, this doesnt matter that much, but I now see that ppl are no longer able to do an expand contract for a single version at a time, there is no more label or anything | 20:02 |
*** aranjan has joined #openstack-neutron | 20:02 | |
*** nlahouti has joined #openstack-neutron | 20:03 | |
amuller | armax: it's trivial to fix that test in tempest | 20:03 |
amuller | armax: in a million different ways | 20:03 |
armax | ihrachys: replied | 20:03 |
ihrachys | zzzeek: that's true. the assumption is that you want the version that you checked out, and that upgrade for multiple versions should work. | 20:03 |
amuller | armax: but wouldn't you like to figure out why one tenant is seeing an external network created by another tenant? | 20:04 |
*** chutwig is now known as erhud1 | 20:04 | |
*** erhud1 is now known as erhudy1 | 20:04 | |
armax | amuller: according to kevinbenton, this is supposed to be the design | 20:05 |
amuller | armax: I don't think he's right | 20:05 |
armax | amuller: I can’t recall on the top of my head if external networks are shared by default | 20:05 |
amuller | armax: they aren't | 20:05 |
* armax still going through the weeds | 20:05 | |
HenryG | zzzeek: We mark each release with a milestone name migration. So 'upgrade expand@liberty" | 20:05 |
amuller | armax: an external network that is not shared is not (in my devstack) and should not be visible to another tenant in a net-list command | 20:05 |
zzzeek | ihrachys: OK, I guess maybe it will, I dont know offhand if they are going to work across many major versions. example. Liberty replaces column A with column B. Mitaka replaces column B with column C. There is no version of the code that translates column A to C at once | 20:05 |
*** abhiraut has joined #openstack-neutron | 20:06 | |
armax | so both salv and kevinbenton are wrong? | 20:06 |
armax | something is up | 20:06 |
amuller | armax: http://paste.openstack.org/show/444565/ | 20:06 |
zzzeek | HenryG: OK, so you can expand/contract specific to a single major relasee? then its fine | 20:06 |
*** emagana has joined #openstack-neutron | 20:06 | |
*** dave-mccowan has quit IRC | 20:06 | |
zzzeek | HenryG / ihrachys : although, what if they just “upgrade expand”. now with the column A->B->C problem, they are sunk | 20:07 |
zzzeek | its a mistake that can now be made | 20:07 |
*** localloop127 has quit IRC | 20:07 | |
HenryG | zzzeek: yeah, that's not great | 20:08 |
amuller | armax: my instinct is that there's some sort of mix up in Tempest | 20:08 |
zzzeek | HenryG: it can be solved by just placing limitations on neutron-migrate-db | 20:08 |
ihrachys | zzzeek: that replace scenario, is it really expand? | 20:08 |
HenryG | zzzeek: moar checks | 20:08 |
armax | amuller: that’s what we gotta find out | 20:08 |
zzzeek | ihrachys: by “replace” I mean, liberty “adds” column B, and assumes the app will translate data from A to B. mitaka “Adds” columnn C, and assumes the app will translate from B to C | 20:09 |
zzzeek | in their “contract”, liberty drops A, mitaka drops B | 20:09 |
ihrachys | zzzeek: I believe you consider data migration, and we are still in schema migration phase. | 20:09 |
zzzeek | ihrachys: that is correct however the whole point is that data migration has to integrate :) | 20:09 |
zzzeek | ihrachys: without that, all this effort is a waste of time :) | 20:09 |
ihrachys | zzzeek: I believe we may need checkpoints for data migrations that are not allowed to pass with unmigrated data | 20:10 |
armax | amuller: the problem may be that other tests may be creating shared networks | 20:10 |
ihrachys | I think nova was looking into checkpoints too | 20:10 |
armax | and those would accidentally be visible when getting all networks from the offending test | 20:10 |
mtreinish | armax: the logs clearly show shared: false in the network create response | 20:10 |
zzzeek | ihrachys: nova is doing this witout any help from me :) | 20:10 |
ihrachys | we have sanity checks in neutron-db-manage, I believe those would belong there | 20:11 |
zzzeek | ihrachys: they’re in the “hey lets try this!” stage, i think there’s a high liklihood the whole thign isn’t going to work for them | 20:11 |
*** localloop127 has joined #openstack-neutron | 20:11 | |
zzzeek | ihrachys: then again i think the data migration aspect of this whole thing is going to be unmanagable anyway | 20:11 |
mtreinish | armax: I mean there is definitely a potential bug there with shared networks showing up in list responses which is why we try to guard against that in tests | 20:11 |
armax | amuller, mtreinish: I’m still going through all the data | 20:11 |
ihrachys | zzzeek: define 'this whole thing' | 20:11 |
amuller | armax: the test it interacts with is tempest.api.network.admin.test_external_network_extension.ExternalNetworksTestJSON.test_delete_external_networks_with_floating_ip | 20:11 |
ihrachys | nova? neutron? | 20:11 |
mtreinish | armax: that nova list test is poorly written for a neutron env, but I don't think that's the issue here | 20:12 |
zzzeek | ihrachys: the, “I want to install mitaka and do a full upgrade without shutting off my openstack service” | 20:12 |
mtreinish | that'll have to be fixed either way | 20:12 |
armax | amuller, mtreinish: I haven’t formulated an opinion yet, neither I am concuring with kevinbenton nor salv | 20:12 |
mtreinish | armax: I don't have a horse in the race either, since I don't know all the details of the api | 20:12 |
armax | mtreinish: if we can guard the test making it isolated as a result, fine…but yes, I would like to better understand what’s going on | 20:12 |
mtreinish | I'm just opposed to a skip | 20:12 |
*** baoli_ has quit IRC | 20:13 | |
armax | mtreinish: agreed | 20:13 |
amuller | aye | 20:13 |
ihrachys | zzzeek: I don't believe you can upgrade code without shutting off. it's not the problem at stake. reducing downtime is. | 20:13 |
*** mickeys has joined #openstack-neutron | 20:13 | |
armax | mtreinish: let me mull over this a bit | 20:13 |
amuller | mtreinish: running test_show_external_networks_attribute on its own also fails | 20:13 |
amuller | mtreinish: seems like a common occurence | 20:14 |
mtreinish | amuller: that seems unexpected, do you have a pastebin of the failure | 20:14 |
zzzeek | ihrachys: restate more accurately. “ i want to install mitaka and do rolling schema upgrades without downtime while the database moves from one version to another, transparently maintaining the data model in both logical versions of the schema simultanously" | 20:14 |
openstackgerrit | Cedric Brandily proposed openstack/neutron: Enable most unit tests for py34 job https://review.openstack.org/219389 | 20:15 |
ihrachys | zzzeek: that's absolutely doable! :) | 20:15 |
*** dedery has quit IRC | 20:16 | |
zzzeek | ihrachys: if i had started writing openstack for NASA however many years ago, it would have really really been not like this at all. but then again, it woudl not have created 5000 new programming jobs :) | 20:16 |
amuller | mtreinish: http://paste.openstack.org/show/444663/ | 20:16 |
ihrachys | zzzeek: job security woooot | 20:16 |
zzzeek | ihrachys: i woudl have thought, “web application that spawns VMs and tracks status in a database” | 20:17 |
ihrachys | so in any case, I hope we are ok with that silly change I made without asking you. we seem to have all needed, but we'll need more checks and probably check-points etc. work to do in M+. | 20:18 |
*** julim has quit IRC | 20:18 | |
*** dguerri` is now known as dguerri | 20:18 | |
mtreinish | amuller: that's weird it does a create and then does a get on it and it gets a 404 resp | 20:18 |
*** achanda has joined #openstack-neutron | 20:18 | |
amuller | mtreinish: it's created by the admin client and the show is with the normal client | 20:18 |
mtreinish | amuller: neutron returns 201 so I'd expect that to be a syncronous create | 20:18 |
amuller | it is synchronous yes | 20:18 |
mtreinish | amuller: ah | 20:18 |
mtreinish | yeah that owuld do it | 20:19 |
ihrachys | zzzeek: I don't believe that simplistic approach at the start would leave you anywhere, but whatever | 20:19 |
ihrachys | armax: another silly comment | 20:19 |
*** bobmel has joined #openstack-neutron | 20:19 | |
mtreinish | amuller: how is that ever passing then? | 20:20 |
armax | ihrachys: you’re being whimsical today aren’t you? | 20:20 |
amuller | mtreinish: I don't know | 20:20 |
*** dguerri is now known as dguerri` | 20:20 | |
amuller | mtreinish: maybe my tempest conf is off? | 20:20 |
zzzeek | ihrachys: the change is fine, im a little less certain about it but if you are good with it then im sure it will work out | 20:20 |
amuller | mtreinish: I really don't know | 20:20 |
mtreinish | amuller: that test seems to be there to verify that external nets can be seen by any user | 20:20 |
amuller | mtreinish: yeah, I'm lost at this point | 20:20 |
mtreinish | the comment seems to say the same thing | 20:20 |
ihrachys | armax: I learn from salv | 20:21 |
*** julim has joined #openstack-neutron | 20:21 | |
armax | ihrachys: replied | 20:23 |
ihrachys | armax: lol for the latter | 20:23 |
armax | :) | 20:23 |
*** achanda has quit IRC | 20:24 | |
ihrachys | armax: what is the format of service_provider that you match self.module_name against? | 20:24 |
armax | the module_name is something like neutron_lbaas | 20:24 |
armax | or networking_l2gw | 20:24 |
armax | or neutron_vpaas | 20:24 |
armax | it’s the module’s name | 20:24 |
*** ronis has quit IRC | 20:24 | |
armax | we could pass the Service Label | 20:24 |
armax | to have a better match | 20:25 |
*** cbouch has quit IRC | 20:25 | |
armax | ihrachys: but that would have to go through a tango of changes that involves all the projects that use the service provider stuff | 20:25 |
*** sdake has joined #openstack-neutron | 20:25 | |
ihrachys | so f.e. it's 'VPN:vyatta:neutron_vpnaas.services.vpn.service_drivers.vyatta_ipsec.VyattaIPsecDriver:default' | 20:25 |
openstackgerrit | Michael Bayer proposed openstack/neutron: Add non-model index names to autogen exclude filters https://review.openstack.org/213303 | 20:25 |
openstackgerrit | Michael Bayer proposed openstack/neutron: Implement expand/contract autogenerate extension https://review.openstack.org/203842 | 20:25 |
armax | ihrachys: yes | 20:25 |
ihrachys | you try to match against 'neutron_vpnaas.services.vpn.service_drivers.vyatta_ipsec.VyattaIPsecDriver:default' | 20:25 |
ihrachys | right? | 20:26 |
armax | no | 20:26 |
ihrachys | meh | 20:26 |
armax | just neutron_vpnaas | 20:26 |
armax | I mean... | 20:26 |
ihrachys | yeah, ok, sure. but I mean, that part, not e.g. VPN | 20:26 |
*** ZZelle_ has joined #openstack-neutron | 20:26 | |
armax | ihrachys: I filter all the drivers that contain the module_name being given to input | 20:26 |
armax | to NeutronModule | 20:26 |
armax | surely the provider can be a random thing that doesn’t belong to the neutron_vpnaas_project | 20:27 |
*** Marga_ has joined #openstack-neutron | 20:27 | |
ihrachys | armax: so that part is device_driver, right? and it's read thru stevedore, which allows any symbolic labels for drivers, that could have no module_name in the aliases at all | 20:27 |
armax | hence matching by service label, ie. VPN would be better…but that would require more tahn a single patch to fix this | 20:27 |
ihrachys | yeah, in the last comment of yours, I see that we start to converge | 20:28 |
*** zhhuabj_ has joined #openstack-neutron | 20:28 | |
*** jwy has joined #openstack-neutron | 20:28 | |
ihrachys | so yeah, my comment was mostly about 'what about drivers that are defined by aliases, as we actually should push *aas users to specify them in long run' | 20:29 |
*** korzen has quit IRC | 20:29 | |
*** sdake_ has quit IRC | 20:29 | |
ihrachys | using import paths is so.... wrong | 20:29 |
*** achanda has joined #openstack-neutron | 20:29 | |
armax | ihrachys: yes, at one point when I moved the NeutronModule class I thought about passing the service label | 20:29 |
armax | ihrachys: but didn’t see a need for it, I do now | 20:30 |
ihrachys | that said, I don't see any drivers in *aas trees that would define aliases. | 20:30 |
armax | ihrachys: and I can clean this up, once we untangle #1490990 | 20:30 |
ihrachys | (not that there can't be external drivers that do) | 20:30 |
armax | ihrachys: right | 20:30 |
*** zhhuabj has quit IRC | 20:31 | |
*** azbiswas has quit IRC | 20:31 | |
ihrachys | yeah, I agree we should fix it first and foremost even if the fix is not ideal from silly commenter's view | 20:31 |
armax | they are not silly at all | 20:31 |
armax | ihrachys: I had thought about that, as the module name is not ideal for the reasons why just discussed | 20:32 |
*** dave-mccowan has joined #openstack-neutron | 20:32 | |
*** julim has quit IRC | 20:33 | |
armax | I’ll clean this up as soon as I put out a few other fires. | 20:33 |
ihrachys | armax: put or set? | 20:33 |
ihrachys | :) | 20:33 |
ihrachys | the latter is more fun | 20:34 |
armax | ihrachys: that’s mean, but I guess I earned that one | 20:34 |
armax | :) | 20:34 |
*** azbiswas has joined #openstack-neutron | 20:36 | |
ihrachys | huh, not really :) | 20:37 |
*** aranjan has quit IRC | 20:37 | |
ihrachys | EmilienM: still here? | 20:38 |
EmilienM | yes Sir | 20:38 |
EmilienM | I should stop saying that | 20:38 |
EmilienM | ihrachys: what's up? | 20:38 |
*** aranjan has joined #openstack-neutron | 20:38 | |
ihrachys | EmilienM: so armax came up with a fix. I wonder whether we have an option to validate the fix before we merge it (f.e. thru Depends-On) | 20:38 |
*** achanda has quit IRC | 20:38 | |
EmilienM | oh | 20:38 |
ihrachys | EmilienM: of course I mean validate for puppet | 20:38 |
armax | amuller: how did you create that external network? which account? | 20:38 |
EmilienM | ihrachys: not in my (puppet) case | 20:38 |
ihrachys | EmilienM: that's sad. doesn't Depends-On work for you? | 20:39 |
EmilienM | ihrachys: neutron/oslo patches can't be tested without merged (rdo trunk build them) | 20:39 |
EmilienM | it does but we use packaging | 20:39 |
amuller | armax: I was talking to Kevin in #openstack-qa, turns out I'm just a dumbass and forgot I played around with Neutron's policy.json on the machine I was experimenting on | 20:39 |
EmilienM | we are not devstack | 20:39 |
ihrachys | EmilienM: ah right, indeed. | 20:39 |
amuller | armax: tenants can infact see external networks created by another tenant | 20:39 |
armax | amuller: ok | 20:39 |
armax | so we’re good? | 20:39 |
armax | besides I can see here ExternalNetworksTestJSON | 20:40 |
ihrachys | EmilienM: you should build your packages with delorean ;) | 20:40 |
armax | that the extenral network is admin created | 20:40 |
EmilienM | ihrachys: but if you provide me a fix, I can test it | 20:40 |
armax | and that’s definitely visible | 20:40 |
EmilienM | ihrachys: yeah :-) | 20:40 |
ihrachys | EmilienM: the fix: https://review.openstack.org/#/c/220254/ | 20:40 |
armax | think about the ‘public’ network created by devstack itself. | 20:40 |
armax | amuller: capisce? | 20:40 |
amuller | armax: the proper fix is to change test_list_show_tenant_networks to create its own network | 20:40 |
armax | amuller: aye | 20:40 |
armax | there’s a caveat though | 20:40 |
EmilienM | ihrachys: no depends-on? | 20:40 |
amuller | armax: BTW that test is utterly broken, if you run it on its own it fails | 20:40 |
armax | this test is actually a test for https://blueprints.launchpad.net/neutron/+spec/get-me-a-network ;) | 20:41 |
amuller | armax: regardless of any test interaction | 20:41 |
ihrachys | EmilienM: ? I presumed you will test it locally somehow? | 20:41 |
EmilienM | ihrachys: I mean, no dependency in oslo? | 20:41 |
ihrachys | EmilienM: no, not really. which is good. | 20:41 |
EmilienM | cool | 20:41 |
EmilienM | I'll test it | 20:41 |
ihrachys | EmilienM: thanks! | 20:42 |
EmilienM | thx for the fixes, armax++ | 20:42 |
*** srilatha has joined #openstack-neutron | 20:42 | |
*** yamamoto has joined #openstack-neutron | 20:42 | |
armax | EmilienM: well, apologies for the glitch | 20:42 |
*** julim has joined #openstack-neutron | 20:42 | |
ihrachys | EmilienM: armax: I will drop off now for a nap, please sync on testing results. cheers! | 20:42 |
armax | amuller: so we’re good then? | 20:42 |
*** kairo has joined #openstack-neutron | 20:43 | |
EmilienM | no need to apology, I'm happy we found it before the release | 20:43 |
armax | ihrachys: nap? | 20:43 |
amuller | armax: ask mtreinish | 20:43 |
armax | ihrachys: who’s got time for a nap? | 20:43 |
armax | ihrachys: slacker | 20:43 |
EmilienM | ihrachys: sleep well | 20:43 |
ihrachys | actually not a nap, but a full blown sleep! | 20:43 |
ihrachys | it's a bit late here already, I deserve it! | 20:43 |
* armax thought ihrachys didn’t sleep | 20:43 | |
EmilienM | it's feature freeze anyway, go sleep :-P | 20:43 |
amuller | ihrachys: 11pm is an early night for you :) | 20:43 |
armax | amuller: now that we have a settled on the matter | 20:43 |
armax | then we can go ahead and ‘fix’ the test rather than skip it | 20:44 |
armax | mtreinish: ^^^ | 20:44 |
*** localloop127 has quit IRC | 20:44 | |
amuller | yeah | 20:44 |
* armax goes and types on the keyboard like a monkey | 20:44 | |
amuller | armax: so, nova network doesn't have a create network API does it | 20:44 |
*** localloop127 has joined #openstack-neutron | 20:45 | |
armax | that’s the whole point of get-me-network monkey businees | 20:45 |
amuller | because that test is currently running against both nova network and neutron | 20:45 |
mtreinish | amuller: I laid out the basic steps in the bug on how to fix the test | 20:46 |
amuller | mtreinish: ack | 20:46 |
*** achanda has joined #openstack-neutron | 20:47 | |
armax | mtreinish: thanks for spelling it out for us | 20:47 |
*** yamamoto has quit IRC | 20:47 | |
mtreinish | armax: sure, np :) | 20:47 |
armax | mtreinish: I’ll get a patch to your attention in a few minutes | 20:47 |
armax | mtreinish: after a dozen patchsets we’ll probably be able to get it merged | 20:48 |
armax | well | 20:48 |
armax | approved | 20:48 |
armax | a few weeks from now we’ll get it merged | 20:48 |
openstackgerrit | Kevin Benton proposed openstack/neutron: Deprecate external_network_bridge option in L3 agent https://review.openstack.org/219928 | 20:49 |
mtreinish | armax: heh, I think we can get it done a bit quicker than that | 20:49 |
armax | mtreinish: you overestimate me | 20:49 |
armax | :) | 20:49 |
*** localloop127 has quit IRC | 20:49 | |
amuller | that whole set_network_resources() thing is so weird | 20:50 |
amuller | it's the most roundabout way to create a network =D | 20:50 |
armax | amuller, mtreinish: posting a patch shortly | 20:50 |
*** saksham has joined #openstack-neutron | 20:50 | |
*** l6unchpad has joined #openstack-neutron | 20:51 | |
mtreinish | amuller: well it's the method to create/access networking resources that're implicitly expected to be there | 20:51 |
mtreinish | it was needed because of the change in model from n-net where a network was always there for you | 20:52 |
mtreinish | salv and I took a lot of time to figure out an approach that worked there | 20:52 |
amuller | I see, it doesn't feel like code you'd right like that from the get go. I prefer test setup as close to the test as possible, not hidden away in some base class 5 levels separated | 20:52 |
amuller | you'd write* | 20:52 |
openstackgerrit | Kevin Benton proposed openstack/neutron: ovs agent resync may miss port remove event https://review.openstack.org/165023 | 20:52 |
mtreinish | amuller: fwiw, I originally wanted to just always have a network, subnet, etc always created | 20:53 |
mtreinish | but back in the icehouse days that actually crashed neutron | 20:53 |
mtreinish | amuller: well except that would basically mean every test class would copy and paste the same setup and teardown code | 20:54 |
*** julim has quit IRC | 20:54 | |
mtreinish | we already had the abstract mechanism to provide isolation | 20:54 |
mtreinish | so we just added networks to that | 20:55 |
mtreinish | fwiw, I think in the future we'll probably drop the explicit method and make it a class attr | 20:55 |
mtreinish | like we did for creds | 20:55 |
openstackgerrit | Stephen Balukoff proposed openstack/python-neutronclient: LBaaS updates to reflect independent pools feature https://review.openstack.org/218563 | 20:56 |
*** fzdarsky__ has quit IRC | 21:02 | |
*** julim has joined #openstack-neutron | 21:02 | |
*** abhiraut has quit IRC | 21:04 | |
*** thorst has quit IRC | 21:04 | |
*** rkukura has joined #openstack-neutron | 21:05 | |
*** abhiraut has joined #openstack-neutron | 21:05 | |
*** rkukura has quit IRC | 21:05 | |
*** tiswanso has quit IRC | 21:06 | |
*** vivek-ebay has joined #openstack-neutron | 21:07 | |
*** pc_m_AWAY is now known as pc_m | 21:09 | |
*** aranjan has quit IRC | 21:09 | |
*** aranjan has joined #openstack-neutron | 21:10 | |
*** Piet has quit IRC | 21:11 | |
*** ivar-lazzaro has quit IRC | 21:14 | |
*** slaweq has quit IRC | 21:15 | |
*** mlavalle has quit IRC | 21:15 | |
*** rtheis has quit IRC | 21:16 | |
*** slaweq has joined #openstack-neutron | 21:17 | |
*** crose has quit IRC | 21:18 | |
*** slaweq has quit IRC | 21:18 | |
*** ihrachys has quit IRC | 21:20 | |
*** jbell8 has quit IRC | 21:21 | |
*** flaviof is now known as flaviof_afk | 21:22 | |
*** aranjan has quit IRC | 21:22 | |
*** clev is now known as clev-away | 21:26 | |
*** pc_m is now known as pc_m_AWAY | 21:27 | |
*** arajagop_ has quit IRC | 21:28 | |
*** arajagop_ has joined #openstack-neutron | 21:28 | |
*** nyechiel has quit IRC | 21:31 | |
*** Leonr has quit IRC | 21:32 | |
*** arajagop_ has quit IRC | 21:32 | |
amuller | armax: going offline for now | 21:35 |
armax | ok | 21:35 |
armax | I got derailed with that tempest patch, pushing it imminently | 21:35 |
armax | leave it with me | 21:35 |
*** rpothier has quit IRC | 21:35 | |
armax | I won’t disappoint you | 21:35 |
amuller | armax: I trust you won't :) | 21:35 |
*** amuller has quit IRC | 21:38 | |
*** annegentle has quit IRC | 21:38 | |
*** l6unchpad has quit IRC | 21:38 | |
*** njohnston is now known as nate_gone | 21:43 | |
*** johnbelamaric has quit IRC | 21:43 | |
*** aranjan has joined #openstack-neutron | 21:46 | |
*** vivek-ebay has quit IRC | 21:46 | |
*** matrohon has quit IRC | 21:49 | |
*** aranjan has quit IRC | 21:50 | |
*** localloop127 has joined #openstack-neutron | 21:51 | |
*** flaviof_afk is now known as flaviof | 21:52 | |
*** sridhar_ram has quit IRC | 21:53 | |
*** sridhar_ram has joined #openstack-neutron | 21:54 | |
*** annegentle has joined #openstack-neutron | 21:57 | |
*** tbachman has quit IRC | 21:58 | |
*** localloop127 has quit IRC | 21:59 | |
*** nyechiel has joined #openstack-neutron | 22:00 | |
*** tflynn_ has joined #openstack-neutron | 22:00 | |
*** annegentle has quit IRC | 22:03 | |
*** tflynn has quit IRC | 22:04 | |
openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: Support for VPN functional tests on Neutron commits https://review.openstack.org/203201 | 22:05 |
*** tflynn_ has quit IRC | 22:05 | |
*** dane_leblanc has quit IRC | 22:05 | |
*** aranjan has joined #openstack-neutron | 22:06 | |
*** arajagop_ has joined #openstack-neutron | 22:07 | |
*** clev-away is now known as clev | 22:07 | |
*** moha_hunt has quit IRC | 22:08 | |
*** abhiraut has quit IRC | 22:09 | |
*** emagana has quit IRC | 22:10 | |
*** pc_m_AWAY has quit IRC | 22:10 | |
*** kbringard has quit IRC | 22:10 | |
*** wwriverrat has joined #openstack-neutron | 22:14 | |
*** jbell8 has joined #openstack-neutron | 22:15 | |
*** thorst has joined #openstack-neutron | 22:16 | |
*** abhiraut has joined #openstack-neutron | 22:16 | |
openstackgerrit | Kevin Benton proposed openstack/neutron: Process user iptables rules before INVALID https://review.openstack.org/218517 | 22:17 |
*** Piet has joined #openstack-neutron | 22:17 | |
*** mlavalle has joined #openstack-neutron | 22:19 | |
*** moha_hunt has joined #openstack-neutron | 22:20 | |
*** azbiswas has quit IRC | 22:23 | |
*** azbiswas has joined #openstack-neutron | 22:23 | |
*** dmartls1 has quit IRC | 22:25 | |
*** alex_klimov has quit IRC | 22:28 | |
*** Leonr has joined #openstack-neutron | 22:30 | |
*** Leon has joined #openstack-neutron | 22:31 | |
*** Leon is now known as Guest58233 | 22:31 | |
*** Leonr has quit IRC | 22:34 | |
*** nyechiel has quit IRC | 22:35 | |
*** Marga_ has quit IRC | 22:35 | |
*** regXboi has quit IRC | 22:35 | |
*** Marga_ has joined #openstack-neutron | 22:37 | |
*** tbachman has joined #openstack-neutron | 22:37 | |
*** thumpba has quit IRC | 22:40 | |
*** Marga_ has quit IRC | 22:43 | |
*** Marga_ has joined #openstack-neutron | 22:44 | |
*** markstur has joined #openstack-neutron | 22:46 | |
*** phil_h has quit IRC | 22:48 | |
*** shwetaap has quit IRC | 22:48 | |
*** edmondsw has quit IRC | 22:53 | |
*** vivek-ebay has joined #openstack-neutron | 22:55 | |
*** sridhar_ram has quit IRC | 22:55 | |
*** sridhar_ram has joined #openstack-neutron | 22:56 | |
*** vivek-ebay has quit IRC | 22:56 | |
*** abhiraut has quit IRC | 22:56 | |
*** srilatha has quit IRC | 22:59 | |
*** srilatha has joined #openstack-neutron | 22:59 | |
*** srilatha has quit IRC | 23:00 | |
*** srilatha has joined #openstack-neutron | 23:00 | |
*** n00btube has joined #openstack-neutron | 23:02 | |
*** sdake has quit IRC | 23:06 | |
*** zzzeek has quit IRC | 23:10 | |
*** arajagop_ has quit IRC | 23:12 | |
*** Marga_ has quit IRC | 23:12 | |
*** n00btube has quit IRC | 23:12 | |
*** markvoelker has joined #openstack-neutron | 23:13 | |
*** arajagopal has joined #openstack-neutron | 23:13 | |
*** moha_hunt has quit IRC | 23:14 | |
*** n00btube has joined #openstack-neutron | 23:15 | |
*** pradk has quit IRC | 23:16 | |
*** n00btube has quit IRC | 23:16 | |
*** mickeys has quit IRC | 23:17 | |
*** arajagopal has quit IRC | 23:17 | |
*** markvoelker has quit IRC | 23:17 | |
*** n00btube has joined #openstack-neutron | 23:21 | |
*** tidwellr1 has joined #openstack-neutron | 23:22 | |
*** gildub has joined #openstack-neutron | 23:22 | |
*** Sukhdev_ has joined #openstack-neutron | 23:23 | |
*** dannywilson has quit IRC | 23:23 | |
Sukhdev_ | armax: Ping | 23:24 |
armax | pong | 23:24 |
Sukhdev_ | armax: I noticed one patch got merged recently which is causing our CI to break - please see the paste here - http://paste.openstack.org/show/444792/ | 23:25 |
*** markvoelker has joined #openstack-neutron | 23:25 | |
Sukhdev_ | armax: I noticed you commented on the patch and approved it - so, I thought you will have context to the issue - | 23:25 |
armax | the paste alone is not enought for me to understand what;’s going on | 23:25 |
armax | what patch are you talking about? | 23:25 |
Sukhdev_ | this has to do something with the API workers - | 23:26 |
armax | which one? | 23:26 |
armax | this one? | 23:26 |
armax | https://review.openstack.org/#/c/189391/ | 23:26 |
armax | this hasn’t merged | 23:26 |
*** n00btube has quit IRC | 23:26 | |
*** moha_hunt has joined #openstack-neutron | 23:27 | |
Sukhdev_ | armax: thanks - yes, I think this one | 23:27 |
armax | your ci fails but neutron starts fine | 23:27 |
armax | it passed on PS17 | 23:27 |
*** abhiraut has joined #openstack-neutron | 23:27 | |
*** azbiswas has quit IRC | 23:27 | |
armax | and PS18 is a rebase | 23:28 |
*** azbiswas has joined #openstack-neutron | 23:28 | |
armax | Sukhdev_: so I am not sure you’re looking at the right thing | 23:28 |
Sukhdev_ | armax: hang on - let me paste more details | 23:28 |
*** azbiswas has quit IRC | 23:29 | |
*** annegentle has joined #openstack-neutron | 23:29 | |
Sukhdev_ | armax: actually you can look at this log - http://172.22.255.11:8080/job/Arista%20ML2%20Driver%20Testing/30425/console | 23:29 |
armax | it’s not opening up | 23:30 |
*** azbiswas has joined #openstack-neutron | 23:30 | |
Sukhdev_ | armax: the paste I pasted earlier was from the log gf q-svc (the link to the logs is also on that line) | 23:30 |
armax | the server must be behind firewall | 23:30 |
Sukhdev_ | armax: oh may be firewall issue :) | 23:30 |
Sukhdev_ | let me paste it and send it | 23:31 |
*** bradjones has quit IRC | 23:31 | |
*** achanda has quit IRC | 23:32 | |
Sukhdev_ | armax: here is the past of devstack failure - http://paste.openstack.org/show/444794/ | 23:32 |
*** y_sawai has joined #openstack-neutron | 23:32 | |
Sukhdev_ | armax: and here it the link to the logs - you should be able to open this - http://arista-ci.arista.com:8000/September-2015/Arista_Tempest_Test/56/219456/4/ | 23:33 |
* armax looks | 23:33 | |
*** n00btube has joined #openstack-neutron | 23:33 | |
*** jobewan has quit IRC | 23:33 | |
*** azbiswas has quit IRC | 23:34 | |
armax | the patch is actually this one | 23:35 |
armax | https://review.openstack.org/#/c/219456/ | 23:35 |
armax | this is very odd | 23:35 |
armax | because that patch is a noop | 23:35 |
*** clev is now known as clev-away | 23:35 | |
armax | HenryG: might be able to shed more light | 23:35 |
armax | use git bisect to figure out where things go south | 23:36 |
Sukhdev_ | armax: you may be right - | 23:36 |
*** n00btube has quit IRC | 23:36 | |
Sukhdev_ | armax: what is git bisect? | 23:36 |
*** neophy has quit IRC | 23:36 | |
armax | a tool command that lets you identify what git commit caused the regression | 23:37 |
armax | Sukhdev_: http://webchick.net/node/99 | 23:37 |
armax | http://git-scm.com/docs/git-bisect | 23:37 |
*** jhyangtw has joined #openstack-neutron | 23:38 | |
Sukhdev_ | armax: cool - thanks - let me try it | 23:38 |
*** oomichi_away is now known as oomichi | 23:39 | |
armax | the fact that the error says it can’t bind makes me wonder whether teh port is already open | 23:39 |
*** y_sawai has quit IRC | 23:41 | |
*** ZZelle_ has quit IRC | 23:42 | |
*** clev-away is now known as clev | 23:42 | |
*** jhyangtw has quit IRC | 23:42 | |
Sukhdev_ | armax: interesting information - this patch that you mentioned merged around 1:12 am, my CI is broken as of yesterday 6:40pm (apps) - | 23:42 |
armax | so it must be something else ;) | 23:43 |
Sukhdev_ | armax: every run is failing with exact same error | 23:43 |
*** dims has quit IRC | 23:44 | |
*** ijw_ has quit IRC | 23:45 | |
*** mickeys has joined #openstack-neutron | 23:45 | |
*** watanabe_isao has joined #openstack-neutron | 23:46 | |
*** changbl has joined #openstack-neutron | 23:48 | |
*** shoutm has joined #openstack-neutron | 23:50 | |
*** watanabe_isao has quit IRC | 23:51 | |
*** miyagishi_t has joined #openstack-neutron | 23:52 | |
*** dims has joined #openstack-neutron | 23:53 | |
*** Swami has quit IRC | 23:56 | |
*** tidwellr1 has quit IRC | 23:58 | |
*** aix has joined #openstack-neutron | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!