| markmcclain | RajeshMohan: I've been working on the cleanup from the slow running gate | 00:00 |
|---|---|---|
| RajeshMohan | markmcclain: ok | 00:00 |
| markmcclain | so haven't had the opportunity for circle back to the review | 00:00 |
| RajeshMohan | markmcclain: I see | 00:01 |
| *** tongli has joined #openstack-neutron | 00:02 | |
| *** tongli has quit IRC | 00:03 | |
| *** dvorkinista has quit IRC | 00:04 | |
| *** dvorkinista has joined #openstack-neutron | 00:04 | |
| RajeshMohan | markmcclain: I also sent email to ML requesting FFE - I am not sure what the process is | 00:05 |
| RajeshMohan | markmcclain: http://lists.openstack.org/pipermail/openstack-dev/2014-March/029049.html | 00:05 |
| *** shivharis has quit IRC | 00:06 | |
| markmcclain | yes.. I saw | 00:06 |
| RajeshMohan | markmcclain: ok - thanks. | 00:07 |
| RajeshMohan | markmcclain: ... so does it sounds reasonable? | 00:08 |
| *** dims has joined #openstack-neutron | 00:10 | |
| *** crc32 has quit IRC | 00:12 | |
| *** andyhky has quit IRC | 00:12 | |
| *** sungju has quit IRC | 00:13 | |
| *** andyhky has joined #openstack-neutron | 00:17 | |
| *** alexpilotti_ has joined #openstack-neutron | 00:17 | |
| *** alexpilotti has quit IRC | 00:19 | |
| *** suresh12 has quit IRC | 00:21 | |
| *** alexpilotti_ has quit IRC | 00:21 | |
| *** markwash has quit IRC | 00:23 | |
| *** suresh12 has joined #openstack-neutron | 00:23 | |
| *** markmcclain has quit IRC | 00:25 | |
| *** packet has quit IRC | 00:26 | |
| *** yamahata has joined #openstack-neutron | 00:27 | |
| banix | markmcclain: can you have a look at this file: neutron/db/migration/alembic_migrations/versions/2eeaf963a447_floatingip_status.py | 00:27 |
| banix | there is a missing ' at the end of one of the lines in plugin (our plugin) | 00:27 |
| banix | is that ok? | 00:28 |
| SumitNaiksatam | banix: i think that will be a problem | 00:30 |
| banix | SumitNaiksatam: looks like it was generated yesterday | 00:31 |
| SumitNaiksatam | banix: yes | 00:31 |
| SumitNaiksatam | banix: let me file a bug to fix it | 00:31 |
| banix | thanks summit for responding; thanks | 00:31 |
| *** vkozhukalov has joined #openstack-neutron | 00:32 | |
| banix | sorry meant to say Sumit | 00:32 |
| SumitNaiksatam | banix: thanks for catching | 00:32 |
| banix | yeah noticed my tempest tests failing all of a sudden | 00:32 |
| SumitNaiksatam | banix: that is the reason | 00:32 |
| banix | I was about to add our plugin there and noticed it was there… incorrectly. | 00:33 |
| banix | will probably scrw up linux bridge plugin as well | 00:34 |
| SumitNaiksatam | banix: yes everything after that | 00:34 |
| kevinbenton | Freescale CI is sending lots of emails | 00:39 |
| *** openstackgerrit has joined #openstack-neutron | 00:40 | |
| openstackgerrit | Sumit Naiksatam proposed a change to openstack/neutron: Fix typo in migration script https://review.openstack.org/78479 | 00:41 |
| SumitNaiksatam | banix: ^^^ | 00:44 |
| banix | SumitNaiksatam: Thanks! | 00:45 |
| SumitNaiksatam | banix: base your test of the above and see if it still fails or goes through | 00:45 |
| banix | Will do | 00:46 |
| SumitNaiksatam | banix: thanks | 00:46 |
| banix | you prefer if I pull from this fix? I just modified the file locally and trying to restart. | 00:47 |
| SumitNaiksatam | banix: oh yeah sure | 00:47 |
| *** suresh12 has quit IRC | 00:47 | |
| *** shakayumi has joined #openstack-neutron | 00:48 | |
| *** suresh12 has joined #openstack-neutron | 00:49 | |
| *** suresh12 has quit IRC | 00:49 | |
| banix | SumitNaiksatam: yes that solved the problem. | 00:50 |
| *** dvorkinista has quit IRC | 00:50 | |
| *** dvorkinista has joined #openstack-neutron | 00:51 | |
| *** _cerberus_ has quit IRC | 00:53 | |
| *** fandi has quit IRC | 00:54 | |
| *** baoli has joined #openstack-neutron | 00:57 | |
| *** dvorkinista has quit IRC | 00:57 | |
| *** dvorkinista has joined #openstack-neutron | 00:58 | |
| *** vkozhukalov has quit IRC | 00:58 | |
| *** otherwiseguy has joined #openstack-neutron | 00:59 | |
| *** WackoRobie has joined #openstack-neutron | 00:59 | |
| hemanthravi | sumit: addressed your review comments, if you get a chance can you review https://review.openstack.org/#/c/69246/ again and https://review.openstack.org/#/c/77809/.....thanks | 01:00 |
| *** banix has quit IRC | 01:00 | |
| *** dvorkinista has quit IRC | 01:01 | |
| *** iwamoto has joined #openstack-neutron | 01:01 | |
| *** dvorkinista has joined #openstack-neutron | 01:01 | |
| *** yamahata_ has quit IRC | 01:01 | |
| *** yamahata_ has joined #openstack-neutron | 01:02 | |
| *** devlaps has quit IRC | 01:02 | |
| hemanthravi | SumitNaiksatam: addressed your review comments, if you get a chance can you review https://review.openstack.org/#/c/69246/ again and https://review.openstack.org/#/c/77809/.....thanks | 01:03 |
| *** SumitNaiksatam has quit IRC | 01:05 | |
| *** SumitNaiksatam has joined #openstack-neutron | 01:05 | |
| *** sfox has quit IRC | 01:06 | |
| *** hemanthravi has quit IRC | 01:06 | |
| *** dvorkinista has quit IRC | 01:08 | |
| *** dvorkinista has joined #openstack-neutron | 01:09 | |
| marun | wtf! the unit tests take use ~500mb per testr instance, so an 8 core machine eats 4gb memory! | 01:09 |
| *** fandi has joined #openstack-neutron | 01:09 | |
| *** mwagner_lap has joined #openstack-neutron | 01:10 | |
| *** dvorkini_ has joined #openstack-neutron | 01:11 | |
| marun | lifeless: Running the neutron unit tests on an 8 core machine, I'm seeing 8 testr subprocesses each consuming ~500mb of memory and maxing out the 4gb allocated to the vm. Crazy! | 01:11 |
| iwamoto | my experience is 4gb is too small even with 1 cpu machine. | 01:13 |
| marun | that's f'ing insane | 01:13 |
| marun | iwamoto: does testr parallelize on a single cpu machine? | 01:14 |
| iwamoto | marun: it seems not | 01:14 |
| marun | iwamoto: so a single unit test process eats that much memory? | 01:14 |
| *** dvorkinista has quit IRC | 01:14 | |
| marun | iwamoto: that's so very broken | 01:14 |
| marun | I wonder how much memory they are allocating to run in the gate | 01:15 |
| iwamoto | marun: i guess required mem calculation would be (shared resource (text, etc))+(memory per testr process) | 01:16 |
| marun | iwamoto: i think it means rethinking how we write unit tests. | 01:17 |
| marun | iwamoto: we're not doing anything that complicated. | 01:17 |
| *** yamamoto_ has quit IRC | 01:18 | |
| iwamoto | agreed | 01:19 |
| *** dvorkini_ has quit IRC | 01:19 | |
| *** fandi has quit IRC | 01:22 | |
| *** dvorkinista has joined #openstack-neutron | 01:23 | |
| *** dvorkini_ has joined #openstack-neutron | 01:25 | |
| *** dvorkinista has quit IRC | 01:25 | |
| *** manishg has quit IRC | 01:30 | |
| *** dvorkini_ has quit IRC | 01:31 | |
| *** dvorkinista has joined #openstack-neutron | 01:31 | |
| *** xuhanp has joined #openstack-neutron | 01:33 | |
| *** thuc has quit IRC | 01:36 | |
| lifeless | marun: testr using 500mb ? | 01:37 |
| *** thuc has joined #openstack-neutron | 01:37 | |
| lifeless | marun: testr fork+execs the test runners, it doesn't fork itself. | 01:37 |
| lifeless | marun: perhaps you're looking at threads of the same process? | 01:37 |
| *** dvorkinista has quit IRC | 01:41 | |
| *** dvorkinista has joined #openstack-neutron | 01:41 | |
| *** thuc has quit IRC | 01:41 | |
| *** vkozhukalov has joined #openstack-neutron | 01:44 | |
| *** SumitNaiksatam has quit IRC | 01:47 | |
| *** jorgem1 has quit IRC | 01:48 | |
| *** sn6i23a has quit IRC | 01:50 | |
| *** rms_13 has left #openstack-neutron | 01:52 | |
| openstackgerrit | Aaron Rosen proposed a change to openstack/neutron: Notify nova when ports are ready https://review.openstack.org/75253 | 01:54 |
| openstackgerrit | Aaron Rosen proposed a change to openstack/neutron: Send network-changed notifications to nova https://review.openstack.org/78041 | 01:54 |
| *** rwsu has quit IRC | 01:54 | |
| *** blogan has joined #openstack-neutron | 01:56 | |
| openstackgerrit | A change was merged to openstack/neutron: Updated from global requirements https://review.openstack.org/78377 | 01:56 |
| *** harlowja has quit IRC | 01:58 | |
| *** WackoRobie has quit IRC | 02:02 | |
| *** harlowja has joined #openstack-neutron | 02:04 | |
| *** thedodd has joined #openstack-neutron | 02:06 | |
| *** xianghui has joined #openstack-neutron | 02:07 | |
| *** SumitNaiksatam has joined #openstack-neutron | 02:10 | |
| *** manishg has joined #openstack-neutron | 02:14 | |
| *** morganfainberg is now known as morganfainberg_Z | 02:16 | |
| abhiraut | arosen: ping | 02:16 |
| arosen | pong | 02:16 |
| abhiraut | replied to you on -> https://review.openstack.org/#/c/75291/ | 02:16 |
| abhiraut | let me know if that sounds okay to you | 02:17 |
| *** dvorkinista has quit IRC | 02:19 | |
| *** dvorkinista has joined #openstack-neutron | 02:19 | |
| *** manishg has quit IRC | 02:20 | |
| *** chandan_kumar has joined #openstack-neutron | 02:21 | |
| *** ryu25 has joined #openstack-neutron | 02:22 | |
| anteaya | ryu25: hello | 02:23 |
| anteaya | are you ryu_ from last night? | 02:23 |
| ryu25 | anteaya: hi there, yes I am | 02:23 |
| anteaya | a man of many costumes | 02:23 |
| ryu25 | thanks for replying to the email. I'll do what you suggested | 02:23 |
| anteaya | great I just replied to your email | 02:24 |
| anteaya | great thank you | 02:24 |
| oda-g | nati_ueno: ping | 02:26 |
| *** nati_uen_ has quit IRC | 02:27 | |
| *** bada_ has joined #openstack-neutron | 02:28 | |
| *** bada has quit IRC | 02:28 | |
| *** tomoe_ has joined #openstack-neutron | 02:29 | |
| *** hemanthravi has joined #openstack-neutron | 02:38 | |
| abhiraut | arosen: thank you | 02:40 |
| arosen | np | 02:41 |
| *** banix has joined #openstack-neutron | 02:43 | |
| *** _cjones_ has quit IRC | 02:46 | |
| *** dvorkini_ has joined #openstack-neutron | 02:47 | |
| *** thuc has joined #openstack-neutron | 02:47 | |
| *** dvorkinista has quit IRC | 02:48 | |
| openstackgerrit | Lee Li proposed a change to openstack/python-neutronclient: Use six.moves.urllib.parse to replace urlparse https://review.openstack.org/73509 | 02:50 |
| *** bada has joined #openstack-neutron | 02:50 | |
| *** thuc has quit IRC | 02:52 | |
| *** shashank_ has quit IRC | 02:52 | |
| *** bada_ has quit IRC | 02:52 | |
| openstackgerrit | liusheng proposed a change to openstack/neutron: Remove vi modelines https://review.openstack.org/78500 | 02:53 |
| *** markmcclain has joined #openstack-neutron | 02:55 | |
| *** markmcclain has quit IRC | 02:56 | |
| *** sweston has quit IRC | 02:59 | |
| *** chandan_kumar has quit IRC | 02:59 | |
| *** SumitNaiksatam has quit IRC | 03:00 | |
| *** SumitNaiksatam has joined #openstack-neutron | 03:03 | |
| *** ramishra has joined #openstack-neutron | 03:09 | |
| *** dvorkini_ has quit IRC | 03:12 | |
| openstackgerrit | Abhishek Raut proposed a change to openstack/neutron: Use segment range from network profile tables for N1k plugin https://review.openstack.org/78506 | 03:13 |
| *** tomoe_ has quit IRC | 03:14 | |
| openstackgerrit | Aaron Rosen proposed a change to openstack/neutron: Notify nova when ports are ready https://review.openstack.org/75253 | 03:17 |
| openstackgerrit | Aaron Rosen proposed a change to openstack/neutron: Send network-changed notifications to nova https://review.openstack.org/78041 | 03:17 |
| openstackgerrit | Aaron Rosen proposed a change to openstack/neutron: Query for port before calling l3plugin.disassociate_floatingips() https://review.openstack.org/78399 | 03:17 |
| *** SumitNaiksatam has quit IRC | 03:22 | |
| *** SumitNaiksatam has joined #openstack-neutron | 03:23 | |
| *** otherwiseguy has quit IRC | 03:26 | |
| *** vkozhukalov has quit IRC | 03:27 | |
| *** otherwiseguy has joined #openstack-neutron | 03:28 | |
| openstackgerrit | Aaron Rosen proposed a change to openstack/neutron: NSX: only catch not found update_port https://review.openstack.org/77729 | 03:32 |
| *** ryu25 has quit IRC | 03:36 | |
| openstackgerrit | Aaron Rosen proposed a change to openstack/neutron: NSX: Make replication mode configurable https://review.openstack.org/76670 | 03:36 |
| *** sfox has joined #openstack-neutron | 03:37 | |
| *** sfox has quit IRC | 03:38 | |
| openstackgerrit | Aaron Rosen proposed a change to openstack/neutron: NSX: propagate network name changes to backend https://review.openstack.org/70968 | 03:38 |
| *** tomoe_ has joined #openstack-neutron | 03:39 | |
| *** sphoorti has joined #openstack-neutron | 03:46 | |
| openstackgerrit | Dhanashree Gosavi proposed a change to openstack/neutron: Validate multicast ip range in Cisco N1kv Plugin https://review.openstack.org/75291 | 03:52 |
| sphoorti | Hello folks, I am running neutron unit tests from /opt/stack/neutron - ./run_tests.sh . The tests however fail , so I try the individual test. Here is the output from the console. | 03:55 |
| sphoorti | http://paste.openstack.org/show/72745/ . It mentions ipv6 for gateway. What possibly could be going wrong ? I have cloned a fresh devstack with neutron | 03:55 |
| *** _cjones_ has joined #openstack-neutron | 03:57 | |
| openstackgerrit | A change was merged to openstack/neutron: VPNaaS Service Driver for Cisco CSR https://review.openstack.org/74144 | 03:59 |
| *** bada has quit IRC | 04:00 | |
| *** _cjones_ has quit IRC | 04:02 | |
| *** vu_ has quit IRC | 04:07 | |
| *** vu has quit IRC | 04:07 | |
| *** abhiraut has quit IRC | 04:08 | |
| *** sfox has joined #openstack-neutron | 04:11 | |
| *** julim has joined #openstack-neutron | 04:11 | |
| openstackgerrit | mark mcclain proposed a change to openstack/neutron: VPNaaS Device Driver for Cisco CSR https://review.openstack.org/74156 | 04:14 |
| *** blogan has quit IRC | 04:16 | |
| *** ramishra has quit IRC | 04:17 | |
| *** networkstatic has joined #openstack-neutron | 04:19 | |
| *** julim has quit IRC | 04:21 | |
| *** dvorkinista has joined #openstack-neutron | 04:23 | |
| *** dvorkinista has quit IRC | 04:27 | |
| *** baoli has quit IRC | 04:38 | |
| *** bvandenh has joined #openstack-neutron | 04:41 | |
| openstackgerrit | A change was merged to openstack/neutron: Adds the missing migration for gw_ext_mode https://review.openstack.org/78421 | 04:49 |
| *** harlowja is now known as harlowja_away | 04:49 | |
| *** shashank_ has joined #openstack-neutron | 04:52 | |
| *** devlaps has joined #openstack-neutron | 04:55 | |
| *** thuc has joined #openstack-neutron | 04:56 | |
| *** thuc_ has joined #openstack-neutron | 04:56 | |
| *** ramishra has joined #openstack-neutron | 04:57 | |
| *** garyk has quit IRC | 04:58 | |
| *** hemanthravi has quit IRC | 04:59 | |
| *** thuc has quit IRC | 05:01 | |
| *** ramishra has quit IRC | 05:01 | |
| *** harlowja_away is now known as harlowja | 05:02 | |
| openstackgerrit | Dazhao Yu proposed a change to openstack/neutron: Calculate stateless IPv6 address https://review.openstack.org/56184 | 05:10 |
| openstackgerrit | Dazhao Yu proposed a change to openstack/neutron: Create new IPv6 attributes for Subnets https://review.openstack.org/52983 | 05:10 |
| *** ramishra has joined #openstack-neutron | 05:15 | |
| openstackgerrit | Dazhao Yu proposed a change to openstack/neutron: Calculate stateless IPv6 address https://review.openstack.org/56184 | 05:16 |
| *** sweston has joined #openstack-neutron | 05:16 | |
| *** otherwiseguy has quit IRC | 05:19 | |
| marun | lifeless: I don't think so. The memory usage varies between the processes and they consume all available memory. | 05:20 |
| marun | lifeless: But maybe I'm doing something wrong. | 05:20 |
| *** yfried has quit IRC | 05:22 | |
| *** nati_ueno has joined #openstack-neutron | 05:22 | |
| *** yfried has joined #openstack-neutron | 05:22 | |
| *** yfried has quit IRC | 05:23 | |
| marun | lifeless: backing up, it looks like what's happening is the neutron test suite uses ~1gb ram, and running tox on an 8 core host would require 8gb of ram rather than the 4gb that I am currently stuck with. | 05:24 |
| marun | lifeless: so, I have to limit the number of test runners to 4 | 05:25 |
| *** shivharis has joined #openstack-neutron | 05:28 | |
| *** ryu25 has joined #openstack-neutron | 05:51 | |
| oda-g | nati_ueno: ping | 05:57 |
| *** sphoorti has quit IRC | 05:58 | |
| *** chandan_kumar has joined #openstack-neutron | 06:04 | |
| *** gdubreui has quit IRC | 06:06 | |
| *** sfox has quit IRC | 06:10 | |
| openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: Sync models with migrations https://review.openstack.org/55411 | 06:11 |
| *** sphoorti has joined #openstack-neutron | 06:12 | |
| *** shivharis has quit IRC | 06:12 | |
| *** arosen1 has joined #openstack-neutron | 06:16 | |
| *** arosen1 is now known as arosen-home | 06:17 | |
| openstackgerrit | Dazhao Yu proposed a change to openstack/neutron: Calculate stateless IPv6 address https://review.openstack.org/56184 | 06:18 |
| *** nati_uen_ has joined #openstack-neutron | 06:21 | |
| *** thuc has joined #openstack-neutron | 06:23 | |
| *** nati_ueno has quit IRC | 06:24 | |
| *** irenab has joined #openstack-neutron | 06:25 | |
| *** yfried has joined #openstack-neutron | 06:26 | |
| *** thuc_ has quit IRC | 06:26 | |
| *** thuc has quit IRC | 06:27 | |
| openstackgerrit | Jenkins proposed a change to openstack/neutron: Imported Translations from Transifex https://review.openstack.org/78114 | 06:29 |
| *** skraynev_afk is now known as skraynev | 06:30 | |
| *** thedodd has quit IRC | 06:34 | |
| *** sweston has quit IRC | 06:39 | |
| *** sphoorti has quit IRC | 06:45 | |
| *** banix has quit IRC | 06:48 | |
| *** hemanthravi has joined #openstack-neutron | 06:48 | |
| *** amritanshu_RnD has joined #openstack-neutron | 06:49 | |
| *** amritanshu_RnD is now known as Guest46316 | 06:50 | |
| *** garyk has joined #openstack-neutron | 06:51 | |
| *** dvorkinista has joined #openstack-neutron | 06:52 | |
| *** rotbeard has joined #openstack-neutron | 06:54 | |
| *** harlowja is now known as harlowja_away | 06:59 | |
| *** jpich has joined #openstack-neutron | 07:01 | |
| openstackgerrit | A change was merged to openstack/neutron: ovs-agent: use hexadecimal IP address in tunnel port name https://review.openstack.org/74442 | 07:03 |
| *** xianghui has quit IRC | 07:03 | |
| openstackgerrit | A change was merged to openstack/neutron: Fix typo in migration script https://review.openstack.org/78479 | 07:05 |
| lifeless | marun: thats more plausible ;). 1GB of ram suggests memory leaks somewhere | 07:06 |
| lifeless | marun: you can cap testr's concurrency fairly easily though | 07:06 |
| marun | lifeless: it's pushing 2gb :( | 07:07 |
| lifeless | marun: meep | 07:07 |
| marun | lifeless: yeah, I'm using concurrency=1 at this point on a 4gb box | 07:07 |
| *** xianghui has joined #openstack-neutron | 07:07 | |
| marun | lifeless: i'll be interested to see how much memory gets used. | 07:07 |
| *** jlibosva has joined #openstack-neutron | 07:07 | |
| marun | lifeless: few people are actually running the full suite anymore, apparently. | 07:07 |
| marun | lifeless: locally I mean. | 07:07 |
| marun | lifeless: 10k tests? 2gb+ memory usage? no wonder. :( | 07:08 |
| marun | lifeless: you think we were doing something interesting or somethingl. | 07:08 |
| marun | you -> you'd | 07:08 |
| openstackgerrit | A change was merged to openstack/neutron: BigSwitch: Auto re-sync on backend inconsistencies https://review.openstack.org/73575 | 07:09 |
| openstackgerrit | Aaron Rosen proposed a change to openstack/neutron: Rename DEVICE_OWNER_COMPUTE_PROBE to not start with compute https://review.openstack.org/78533 | 07:09 |
| *** sphoorti has joined #openstack-neutron | 07:10 | |
| openstackgerrit | Jakub Libosvar proposed a change to openstack/neutron: Add script for populating loadbalancer table after migration https://review.openstack.org/78293 | 07:10 |
| arosen-home | I have 32 gigs + 8 proc; running the neutron tests locally can take over 12 gigs some time. | 07:10 |
| arosen-home | I've also noticed some issues with using tox where processes keep running in the background. | 07:10 |
| openstackgerrit | A change was merged to openstack/neutron: BigSwitch: Add SSL Certificate Validation https://review.openstack.org/70906 | 07:12 |
| lifeless | arosen-home: definitely a leak then :) | 07:12 |
| lifeless | arosen-home: something like meliae may be useful for tracking that down | 07:13 |
| arosen-home | lifeless: yea i know we had several leaks before i'm having a hard time remembering what was exactly causing it. | 07:14 |
| ajo | phew, 12 GBs? | 07:14 |
| arosen-home | that's with 8 processes though running | 07:15 |
| ajo | It looks to me like testing would also benefit from memory optimizations :-) | 07:15 |
| ajo | aha | 07:15 |
| lifeless | so ~ 120KB per test leaked ? | 07:16 |
| ajo | ah, leaks | 07:17 |
| arosen-home | err i might be exaggerating a little on 12 gigs testing.... | 07:17 |
| ajo | arosen, if you could run a ps aux and paste it somewhere in the middle of a test | 07:18 |
| ajo | would help spotting out what are the memory-sucking processes | 07:18 |
| arosen-home | it would be nice if there was a memory profiler that could run as part of the tests. | 07:20 |
| arosen-home | to give you memory usage (in addition to time each test runs) | 07:20 |
| arosen-home | i'm sure there's something outthere that already does that. | 07:20 |
| ajo | arosen-home, totally | 07:20 |
| ajo | that would be awesome | 07:21 |
| ajo | I was thinking about that the last few days | 07:21 |
| *** harlowja_away has quit IRC | 07:21 | |
| ajo | it would be awesome if we could even profile neutron execution as we test | 07:21 |
| ajo | both CPU & RAM wise, although, I'm not sure how precise (and how cheap) would that be on our testing environment | 07:22 |
| *** saju_m has joined #openstack-neutron | 07:22 | |
| *** hemanthravi has quit IRC | 07:23 | |
| *** shashank_ has quit IRC | 07:31 | |
| *** thuc has joined #openstack-neutron | 07:34 | |
| *** hemanthravi has joined #openstack-neutron | 07:37 | |
| *** zhipeng has quit IRC | 07:38 | |
| *** thuc has quit IRC | 07:38 | |
| *** sweston has joined #openstack-neutron | 07:40 | |
| *** sweston has quit IRC | 07:44 | |
| *** ramishra has quit IRC | 07:52 | |
| *** djoreilly has joined #openstack-neutron | 07:56 | |
| openstackgerrit | Maru Newby proposed a change to openstack/neutron: Add script to migrate ovs or lb db to ml2 db https://review.openstack.org/76533 | 08:02 |
| *** jpich has quit IRC | 08:05 | |
| *** saju_m has quit IRC | 08:06 | |
| *** luqas has joined #openstack-neutron | 08:08 | |
| *** jistr has joined #openstack-neutron | 08:15 | |
| *** saju_m has joined #openstack-neutron | 08:18 | |
| *** devvesa has joined #openstack-neutron | 08:18 | |
| *** dvorkinista has quit IRC | 08:18 | |
| marun | ajo: ps aux doesn't tell you anything. the memory sucking processes are the test runners | 08:21 |
| *** shakayumi has quit IRC | 08:21 | |
| ajo | marun, unit testing? | 08:21 |
| marun | arosen: I think this is a pretty serious problem. Not everyone has a 32gb machine to run tests on. | 08:21 |
| marun | ajo: yes | 08:21 |
| ajo | ah, ok, understood | 08:21 |
| ajo | marun: you're right, even from the gate / CI perspective, 32GB is a lot | 08:22 |
| *** devvesa has quit IRC | 08:22 | |
| openstackgerrit | Darragh O'Reilly proposed a change to openstack/neutron: linuxbridge-agent: process port updates in the main loop https://review.openstack.org/62875 | 08:22 |
| marun | ajo: or 12gb, as the case may be | 08:22 |
| ajo | that'd almost require baremetal machines to test | 08:22 |
| *** ramishra has joined #openstack-neutron | 08:22 | |
| ajo | or 1/2 1/4 machine at most | 08:22 |
| *** saju_m has quit IRC | 08:24 | |
| ajo | marun, it's memory leak related?, or from the test suite?, the test themselves ? | 08:24 |
| ajo | tbd? | 08:24 |
| openstackgerrit | A change was merged to openstack/neutron: Fix DetachedInstanceError for Agent instance https://review.openstack.org/74962 | 08:25 |
| obondarev | amotoki: ping | 08:25 |
| *** ramishra has quit IRC | 08:28 | |
| *** ramishra has joined #openstack-neutron | 08:28 | |
| *** jgallard has joined #openstack-neutron | 08:28 | |
| openstackgerrit | Tomoe Sugihara proposed a change to openstack/neutron: Implement MidoNet plugin for Icehouse (Part 1) https://review.openstack.org/78543 | 08:29 |
| openstackgerrit | Akihiro Motoki proposed a change to openstack/neutron: NEC plugin: delete old OFC ID mapping tables https://review.openstack.org/77407 | 08:29 |
| *** devvesa has joined #openstack-neutron | 08:32 | |
| ryu25 | anteaya: hey Anita, FYI we just pushed the first patch. https://review.openstack.org/78543 We broke up the original one to three, so we have two more, but they are smaller changes. I will abandon the old patch. | 08:33 |
| *** iwamoto has quit IRC | 08:33 | |
| *** openstackgerrit has quit IRC | 08:34 | |
| *** openstackgerrit has joined #openstack-neutron | 08:34 | |
| *** saju_m has joined #openstack-neutron | 08:36 | |
| *** amuller has joined #openstack-neutron | 08:36 | |
| *** Guest46316 has quit IRC | 08:36 | |
| openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: WIP Add testing for database from oslo https://review.openstack.org/76519 | 08:37 |
| *** Guest46316 has joined #openstack-neutron | 08:40 | |
| *** sweston has joined #openstack-neutron | 08:41 | |
| *** harmw has quit IRC | 08:42 | |
| *** sweston has quit IRC | 08:45 | |
| *** jroovers has joined #openstack-neutron | 08:46 | |
| *** jroovers|afk has joined #openstack-neutron | 08:46 | |
| *** dave_tucker is now known as dave_tucker_zzz | 08:47 | |
| *** jroovers has quit IRC | 08:50 | |
| *** networkstatic has quit IRC | 08:52 | |
| *** harmw has joined #openstack-neutron | 08:52 | |
| openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: WIP Add testing for database from oslo https://review.openstack.org/76519 | 08:53 |
| *** ygbo has joined #openstack-neutron | 08:53 | |
| *** jpich has joined #openstack-neutron | 08:54 | |
| *** devvesa has quit IRC | 08:55 | |
| openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: WIP Add testing for database from oslo https://review.openstack.org/76519 | 09:01 |
| *** rotbeard has quit IRC | 09:03 | |
| *** evgenyf has joined #openstack-neutron | 09:03 | |
| *** luqas has quit IRC | 09:10 | |
| *** devvesa has joined #openstack-neutron | 09:11 | |
| *** luqas has joined #openstack-neutron | 09:11 | |
| *** saju_m has quit IRC | 09:12 | |
| *** tomoe_ has quit IRC | 09:13 | |
| *** gongysh has joined #openstack-neutron | 09:13 | |
| *** bada has joined #openstack-neutron | 09:14 | |
| openstackgerrit | enikanorov proposed a change to openstack/neutron: Fix unittest failure in radware lbaas driver https://review.openstack.org/78554 | 09:18 |
| *** rossella_s has joined #openstack-neutron | 09:18 | |
| *** leseb has joined #openstack-neutron | 09:25 | |
| *** safchain has joined #openstack-neutron | 09:26 | |
| *** ramishra has quit IRC | 09:34 | |
| *** yfried has quit IRC | 09:34 | |
| *** ramishra has joined #openstack-neutron | 09:35 | |
| *** SumitNaiksatam has quit IRC | 09:39 | |
| *** SumitNaiksatam has joined #openstack-neutron | 09:39 | |
| *** arosen-home has quit IRC | 09:41 | |
| *** ramishra has quit IRC | 09:45 | |
| *** tomoe_ has joined #openstack-neutron | 09:46 | |
| *** ramishra has joined #openstack-neutron | 09:48 | |
| *** dvorkinista has joined #openstack-neutron | 09:49 | |
| *** yfried has joined #openstack-neutron | 09:49 | |
| *** banix has joined #openstack-neutron | 09:50 | |
| *** banix has quit IRC | 09:50 | |
| *** dvorkinista has quit IRC | 09:54 | |
| *** yamahata has quit IRC | 09:56 | |
| *** saju_m has joined #openstack-neutron | 09:58 | |
| *** jp_at_hp has joined #openstack-neutron | 10:00 | |
| *** yfried has quit IRC | 10:06 | |
| *** morganfainberg_Z is now known as morganfainberg | 10:06 | |
| *** yfried has joined #openstack-neutron | 10:08 | |
| *** simon-AS559 has joined #openstack-neutron | 10:12 | |
| *** jistr has quit IRC | 10:13 | |
| *** xuhanp has quit IRC | 10:13 | |
| *** jroovers|afk has quit IRC | 10:13 | |
| *** enikanorov has quit IRC | 10:18 | |
| *** enikanorov has joined #openstack-neutron | 10:18 | |
| *** roeyc has joined #openstack-neutron | 10:21 | |
| *** evgenyf has quit IRC | 10:21 | |
| *** dave_tucker_zzz is now known as dave_tucker | 10:30 | |
| *** evgenyf has joined #openstack-neutron | 10:33 | |
| *** ramishra has quit IRC | 10:34 | |
| *** bvandenh has quit IRC | 10:36 | |
| *** jistr has joined #openstack-neutron | 10:37 | |
| *** ramishra has joined #openstack-neutron | 10:39 | |
| *** sweston has joined #openstack-neutron | 10:41 | |
| openstackgerrit | Tomoe Sugihara proposed a change to openstack/neutron: Implement MidoNet plugin for Icehouse (Part 2) https://review.openstack.org/78575 | 10:42 |
| *** amotoki has joined #openstack-neutron | 10:42 | |
| ryu25 | anteaya: heya, here's the second patch as promised: https://review.openstack.org/#/c/78575/ there is just one more | 10:45 |
| *** garyk has quit IRC | 10:45 | |
| *** sweston has quit IRC | 10:46 | |
| *** sphoorti has quit IRC | 10:46 | |
| *** saju_m has quit IRC | 10:46 | |
| *** yamahata has joined #openstack-neutron | 10:47 | |
| *** dvorkinista has joined #openstack-neutron | 10:50 | |
| openstackgerrit | shihanzhang proposed a change to openstack/python-neutronclient: Parameter support both id and name https://review.openstack.org/78577 | 10:51 |
| *** dvorkinista has quit IRC | 10:54 | |
| *** gdubreui has joined #openstack-neutron | 10:55 | |
| *** gdubreui has quit IRC | 10:56 | |
| openstackgerrit | Aleks Chirko proposed a change to openstack/neutron: Bugfix and refactoring for ovs_lib flow methods https://review.openstack.org/58533 | 10:58 |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack/neutron: Test (don't review) https://review.openstack.org/78580 | 10:58 |
| openstackgerrit | Darragh O'Reilly proposed a change to openstack/neutron: lb-agent: ensure removed devices get treated on resyncs https://review.openstack.org/77842 | 11:02 |
| *** bvandenh has joined #openstack-neutron | 11:03 | |
| *** saju_m has joined #openstack-neutron | 11:03 | |
| hemanthravi | salv-orlando: hi | 11:06 |
| salv-orlando | hi | 11:06 |
| hemanthravi | looking for some help with db migration scripts for https://review.openstack.org/#/c/69246/ | 11:07 |
| *** pbeskow has quit IRC | 11:07 | |
| hemanthravi | salv-orlando: got a review comment from mark mcclain for https://review.openstack.org/#/c/69246/ to add db migrations | 11:08 |
| *** rotbeard has joined #openstack-neutron | 11:08 | |
| salv-orlando | ok, markmcclain is the boss! if he says we need migrations, we must give him migrations :) | 11:09 |
| salv-orlando | there are two strategies here. | 11:09 |
| salv-orlando | the first is to add an initial migration which creates all the model for your plugin | 11:09 |
| hemanthravi | salv-orlando: do i need these if i don't have any schema changes?...i run neutron-db-mange...as documented but i get some upgrade/downgrade that dont' work | 11:09 |
| hemanthravi | :) | 11:09 |
| salv-orlando | whereas the other is to piggyback on existing migrations so that they are executed even when you run your plugin | 11:09 |
| salv-orlando | it's not just about schema changes. You should have a migration also for your initial schema. | 11:10 |
| salv-orlando | neutron has this 'auto-generation' mechanisms for db tables, but that's not one to be relied upon. | 11:10 |
| salv-orlando | indeed we proposed to remove that in the past, but we were held back as there was some work in progress to reconcile auto generation with migration. Anyway, this is another story | 11:11 |
| hemanthravi | how can i piggyback on existing scripts? | 11:12 |
| salv-orlando | did I already pointed you to this patch: https://review.openstack.org/#/c/66545/7 | 11:12 |
| salv-orlando | that's another plugin trying to do what you need to do | 11:12 |
| salv-orlando | add your plugin to folsom_initial | 11:13 |
| salv-orlando | also, specify there if your plugin is L3 capable or not | 11:13 |
| *** jgallard has quit IRC | 11:13 | |
| salv-orlando | unlike that patch, I would not advise to touch the havana_release migration: https://review.openstack.org/#/c/66545/7/neutron/db/migration/alembic_migrations/versions/havana_release.py | 11:14 |
| salv-orlando | that does not seem right to me. | 11:14 |
| hemanthravi | will add to folsom_initial, also tried to generate a script with neutron-db-manage, but didn't end up right upgrade/downgrade | 11:15 |
| salv-orlando | then either 1) add your plugin to all the migrations for extensions which are not included in folsom_initial | 11:15 |
| salv-orlando | or 2) add a single migration with all the schema changes you need specifically for your plugin | 11:15 |
| hemanthravi | ok, for eg if i need sec-grp add it to the existing sec-grp ? | 11:16 |
| salv-orlando | hemanthravi: yes, that would be fine for me. | 11:16 |
| salv-orlando | as your plugin is new. If it weren't, this would have not been ok because it would have broken upgrades. | 11:16 |
| salv-orlando | if you need to generate migrations and don't work, you can try at least to post somewhere the migration you tried to add and the error message you get. | 11:17 |
| hemanthravi | the issue might be with me running db-manage to generate it, i didn't manually add them | 11:18 |
| *** rossella_s has quit IRC | 11:18 | |
| *** rossella_s has joined #openstack-neutron | 11:19 | |
| hemanthravi | i'll delete the db, run stack.sh and see if it generates one that will work | 11:20 |
| *** xianghui has quit IRC | 11:20 | |
| openstackgerrit | Sylvain Afchain proposed a change to openstack/neutron: Introduces a keepalived manager for HA https://review.openstack.org/68142 | 11:20 |
| hemanthravi | salv-orlando: thanks | 11:20 |
| *** bashok has joined #openstack-neutron | 11:20 | |
| *** pbeskow has joined #openstack-neutron | 11:21 | |
| openstackgerrit | Dazhao Yu proposed a change to openstack/neutron: Calculate stateless IPv6 address https://review.openstack.org/56184 | 11:24 |
| *** gdubreui has joined #openstack-neutron | 11:31 | |
| *** dave_tucker is now known as dave_tucker_zzz | 11:32 | |
| *** tomoe_ has quit IRC | 11:32 | |
| openstackgerrit | A change was merged to openstack/neutron: Replaces network:* strings by constants https://review.openstack.org/67862 | 11:33 |
| *** pcm_ has joined #openstack-neutron | 11:36 | |
| *** tomoe_ has joined #openstack-neutron | 11:36 | |
| *** pcm_ has quit IRC | 11:38 | |
| *** pcm_ has joined #openstack-neutron | 11:39 | |
| *** itzikb has joined #openstack-neutron | 11:39 | |
| *** sweston has joined #openstack-neutron | 11:42 | |
| *** sweston has quit IRC | 11:47 | |
| *** gdubreui has quit IRC | 11:47 | |
| *** baoli has joined #openstack-neutron | 11:47 | |
| *** dvorkinista has joined #openstack-neutron | 11:51 | |
| *** ramishra has quit IRC | 11:55 | |
| *** dvorkinista has quit IRC | 11:55 | |
| *** devvesa has quit IRC | 11:55 | |
| *** baoli has quit IRC | 11:56 | |
| openstackgerrit | A change was merged to openstack/neutron: NEC plugin: delete old OFC ID mapping tables https://review.openstack.org/77407 | 11:58 |
| *** bashok has quit IRC | 11:58 | |
| *** gongysh has quit IRC | 11:59 | |
| *** devvesa has joined #openstack-neutron | 12:01 | |
| *** yfujioka has quit IRC | 12:06 | |
| *** devlaps has quit IRC | 12:11 | |
| *** devvesa has quit IRC | 12:11 | |
| *** irenab has quit IRC | 12:13 | |
| *** ramishra has joined #openstack-neutron | 12:16 | |
| *** bvandenh has quit IRC | 12:18 | |
| openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: Returned not nullable patameter for some columns https://review.openstack.org/78591 | 12:20 |
| *** pcm_ has quit IRC | 12:20 | |
| *** bada has quit IRC | 12:21 | |
| *** kedarkul has joined #openstack-neutron | 12:21 | |
| *** morganfainberg is now known as morganfainberg_Z | 12:27 | |
| *** mwagner_lap has quit IRC | 12:27 | |
| openstackgerrit | A change was merged to openstack/neutron: NSX: passing wrong security_group id mapping to nsx backend https://review.openstack.org/77727 | 12:30 |
| *** garyk has joined #openstack-neutron | 12:33 | |
| *** garyk has quit IRC | 12:37 | |
| *** baoli has joined #openstack-neutron | 12:38 | |
| *** baoli has quit IRC | 12:41 | |
| *** baoli has joined #openstack-neutron | 12:41 | |
| *** sweston has joined #openstack-neutron | 12:42 | |
| *** roeyc has quit IRC | 12:43 | |
| *** sweston has quit IRC | 12:47 | |
| *** dvorkinista has joined #openstack-neutron | 12:51 | |
| *** jroovers has joined #openstack-neutron | 12:52 | |
| *** xuhanp has joined #openstack-neutron | 12:53 | |
| *** markvoelker has quit IRC | 12:55 | |
| *** devvesa has joined #openstack-neutron | 12:55 | |
| *** dvorkinista has quit IRC | 12:56 | |
| *** mfink has quit IRC | 13:00 | |
| *** yamahata has quit IRC | 13:00 | |
| *** pcm_ has joined #openstack-neutron | 13:08 | |
| *** sfox has joined #openstack-neutron | 13:08 | |
| *** pcm_ has quit IRC | 13:08 | |
| *** pcm_ has joined #openstack-neutron | 13:09 | |
| openstackgerrit | Phil Day proposed a change to openstack/python-neutronclient: Allow user ID for authentication https://review.openstack.org/78596 | 13:09 |
| *** yamahata has joined #openstack-neutron | 13:13 | |
| openstackgerrit | Gastón Severina proposed a change to openstack/python-neutronclient: Per project user quotas support in client https://review.openstack.org/78598 | 13:14 |
| *** dims has quit IRC | 13:18 | |
| anteaya | ryu25: thanks for your posts, for the future, since it is good to keep the review history along with a patch, if you are taking a patch and breaking it into smaller pieces, edit the first patch so that it is smaller and has the original change id (which will perserve the history on gerrit) and add the removed pieces as dependencies onto that | 13:18 |
| anteaya | ryu25: don't try to do that now, but for next time | 13:18 |
| *** dims has joined #openstack-neutron | 13:19 | |
| anteaya | also for the first patch in the series, please add a comment to the patch that includes the url of the abandoned patch, to enable reviewers to find the history | 13:19 |
| anteaya | ryu25: and good work | 13:19 |
| *** jistr has quit IRC | 13:21 | |
| *** saju_m has quit IRC | 13:27 | |
| *** WackoRobie has joined #openstack-neutron | 13:27 | |
| ryu25 | anteaya: thanks for the advice. I wasn't sure about it since I thought perhaps the original patch could still be accepted but we just went ahead and abandoned it in the end. Will definitely do that next time. | 13:29 |
| anteaya | okay thanks | 13:29 |
| anteaya | and I re-read my above post | 13:29 |
| anteaya | ensure the change id is not changed in the original commit which would be the first of the dependency tree | 13:30 |
| anteaya | I want to make that clear | 13:30 |
| anteaya | the change id stays the same | 13:30 |
| openstackgerrit | Sylvain Afchain proposed a change to openstack/neutron: Introduces a keepalived manager for HA https://review.openstack.org/68142 | 13:31 |
| ryu25 | anteaya: right, to keep one stream of all the patch sets, including the original and all the broken-up ones. makes sense | 13:32 |
| *** jroovers has quit IRC | 13:34 | |
| *** mfink has joined #openstack-neutron | 13:34 | |
| *** madmike has joined #openstack-neutron | 13:35 | |
| anteaya | ryu25: great thanks, I had just re-read what I wrote and thought I needed to reiterate that for the benefit of whoelse comes afterward reading the logs | 13:35 |
| anteaya | speaking of people reading the logs (or backscroll) here is ttx' blog post about feature freeze: http://fnords.wordpress.com/2014/03/06/why-we-do-feature-freeze/ | 13:35 |
| anteaya | I suggest reading it as a refresher even if you know what feature freeze is | 13:36 |
| anteaya | so that means testing a bug fixes times | 13:36 |
| anteaya | testing and bug fixes time | 13:36 |
| anteaya | I still am typing with morning fingers | 13:36 |
| anteaya | if you are confused about how this affects you, do post in channel so that we can get things cleared up and you make the most efficient use of your time | 13:37 |
| *** mfink has quit IRC | 13:38 | |
| openstackgerrit | Ann Kamyshnikova proposed a change to openstack/neutron: Returned not nullable patameter for some columns https://review.openstack.org/78591 | 13:40 |
| *** yamahata has quit IRC | 13:41 | |
| anteaya | sc68cal: what is the status of the ipv6 code? I must admit I didn't read all the backscroll from yesterday | 13:43 |
| *** jgallard has joined #openstack-neutron | 13:44 | |
| *** pcm_ has quit IRC | 13:47 | |
| *** thuc has joined #openstack-neutron | 13:50 | |
| *** dvorkinista has joined #openstack-neutron | 13:52 | |
| *** jistr has joined #openstack-neutron | 13:52 | |
| *** ramishra has quit IRC | 13:53 | |
| *** dvorkinista has quit IRC | 13:57 | |
| *** jorgem has joined #openstack-neutron | 14:01 | |
| jorgem | Anyone know what the channel is for the weekly meeting for neutron lbaas? | 14:02 |
| enikanorov | jorgem: you're there | 14:04 |
| enikanorov | openstack-meeting | 14:04 |
| jorgem | just joined. Thanks! | 14:04 |
| hemanthravi | amotoki: hi | 14:04 |
| amotoki | hemanthravi: hi | 14:05 |
| amotoki | hemanthravi: unfortunately i am leaving the channel. I will connect after one or two hours after. | 14:06 |
| hemanthravi | amotoki: reg your review for https://review.openstack.org/#/c/77809/ .... I'm adding all the migrations in a patchhttps://review.openstack.org/#/c/69246/ | 14:06 |
| hemanthravi | once i post this patch (which it's dependent on), can you re-review this...will ping you in 2hrs | 14:07 |
| amotoki | hemanthravi: okay. generally speaking if so it is not a good idea to split the patch, but if mark says okay i am okay. | 14:07 |
| hemanthravi | this was my first checkin, wanted to make it easier for review...will keep that in mind in the future | 14:08 |
| xuhanp | anteaya, I noticed you asked about IPv6 code status, we still have many code review comments to address. | 14:08 |
| amotoki | hemanthravi: i understand the situation :-) | 14:08 |
| amotoki | hemanthravi: regarding test coverage, i hope you post another patch soon. | 14:08 |
| hemanthravi | amotoki: thanks for the review..will post the patch in next 30 min | 14:08 |
| *** blogan has joined #openstack-neutron | 14:09 | |
| amotoki | hemanthravi: your patch becomes pretty good shape :-) | 14:09 |
| *** amotoki has quit IRC | 14:09 | |
| hemanthravi | amotoki: will in the first changeid https://review.openstack.org/#/c/69246/ | 14:09 |
| *** thuc has quit IRC | 14:10 | |
| *** yamahata has joined #openstack-neutron | 14:10 | |
| mestery | anteaya: I missed your reply last night, are you around this morning by chance? | 14:11 |
| *** thuc has joined #openstack-neutron | 14:12 | |
| *** changbl has quit IRC | 14:14 | |
| *** yamahata has quit IRC | 14:17 | |
| *** yamahata has joined #openstack-neutron | 14:17 | |
| *** otherwiseguy has joined #openstack-neutron | 14:17 | |
| *** yamahata has quit IRC | 14:19 | |
| *** samuelbercovici has joined #openstack-neutron | 14:19 | |
| *** yamahata has joined #openstack-neutron | 14:19 | |
| *** pcm_ has joined #openstack-neutron | 14:20 | |
| *** julim has joined #openstack-neutron | 14:20 | |
| anteaya | xuhanp: thank you for the update, I won't keep you from your work | 14:21 |
| anteaya | mestery: here I am | 14:21 |
| *** tomoe_ has quit IRC | 14:21 | |
| mestery | anteaya: See qustions in #openstack-infra please :) | 14:21 |
| mestery | anteaya: Just posted. | 14:21 |
| mestery | Maybe more appropriate there. | 14:21 |
| mestery | Let me know if not | 14:21 |
| *** tomoe_ has joined #openstack-neutron | 14:21 | |
| *** pcm_ has quit IRC | 14:21 | |
| *** vu_ has joined #openstack-neutron | 14:22 | |
| *** pcm_ has joined #openstack-neutron | 14:22 | |
| *** itzikb has quit IRC | 14:23 | |
| *** tomoe_ has quit IRC | 14:25 | |
| *** thuc has quit IRC | 14:27 | |
| *** thuc has joined #openstack-neutron | 14:28 | |
| *** yamahata has quit IRC | 14:29 | |
| *** yamahata has joined #openstack-neutron | 14:30 | |
| *** WackoRobie has quit IRC | 14:30 | |
| *** thuc has quit IRC | 14:31 | |
| *** banix has joined #openstack-neutron | 14:32 | |
| *** HenryG has quit IRC | 14:34 | |
| *** peristeri has joined #openstack-neutron | 14:35 | |
| *** afazekas has joined #openstack-neutron | 14:39 | |
| *** ramishra has joined #openstack-neutron | 14:41 | |
| *** yamahata has quit IRC | 14:41 | |
| *** jobewan has joined #openstack-neutron | 14:42 | |
| openstackgerrit | Sylvain Afchain proposed a change to openstack/neutron: Add HA support to the l3 agent https://review.openstack.org/70700 | 14:45 |
| openstackgerrit | Sylvain Afchain proposed a change to openstack/neutron: Introduces a keepalived manager for HA https://review.openstack.org/68142 | 14:45 |
| openstackgerrit | Sylvain Afchain proposed a change to openstack/neutron: Add L3 VRRP HA base classes https://review.openstack.org/64553 | 14:45 |
| *** tomoe_ has joined #openstack-neutron | 14:51 | |
| *** carl_baldwin has joined #openstack-neutron | 14:54 | |
| *** HenryG has joined #openstack-neutron | 14:57 | |
| openstackgerrit | Hemanth Ravi proposed a change to openstack/neutron: One Convergence Neutron Plugin Implementation https://review.openstack.org/69246 | 14:58 |
| *** shakayumi has joined #openstack-neutron | 15:01 | |
| enikanorov | samuelbercovici: I'll be online in 1.5-2 hours if you need me | 15:01 |
| *** swat30 has joined #openstack-neutron | 15:03 | |
| samuelbercovici | jorgem: whare are you located? | 15:03 |
| *** banix has quit IRC | 15:03 | |
| jorgem | samuelbercovici: Rackspace | 15:03 |
| *** vjay has joined #openstack-neutron | 15:03 | |
| samuelbercovici | jorgem: I mean where in the world | 15:03 |
| jorgem | San Antonio. But Austin is probably where I'd set something up | 15:03 |
| jorgem | San Antonio, TX | 15:04 |
| jorgem | USA | 15:04 |
| *** mwagner_lap has joined #openstack-neutron | 15:04 | |
| jorgem | samuelbercovici: you? | 15:04 |
| samuelbercovici | jorgem: israel | 15:04 |
| jorgem | samuelbercovici: Will you be attending the Openstack conference? | 15:04 |
| samuelbercovici | jorgem: yes | 15:04 |
| *** thurloat has joined #openstack-neutron | 15:05 | |
| jorgem | samuelbercovici: Yeah, I wanted to get all the active participants together as email, IRC and not as rich as face to face communication. I feel like we could get a lot more done face-to-face. The openstack conference gives that but there are also distractions do to all the other Openstack projects. | 15:06 |
| jorgem | samuelbercovici: It's just an idea. I realize that Atlanta is only a few months away too | 15:06 |
| samuelbercovici | jorgem: will you be able to travel to the bay area? | 15:07 |
| *** alagalah has joined #openstack-neutron | 15:07 | |
| jorgem | samuelbercovici: Perhaps? But I wanted to gauge everyone's interest first before committing. | 15:08 |
| jorgem | I'm guessing you will be in SF sometime soon | 15:08 |
| samuelbercovici | jorgem: how did you know ;-) | 15:08 |
| jorgem | samuelbercovici: I'm psychic lol | 15:08 |
| jorgem | Anyways, I'll shot an email to everyone to see who's interested | 15:09 |
| samuelbercovici | jorgem: sure. finding a place to be hosted should not be too difficult | 15:10 |
| jorgem | samuelbercovici: Rackspace has an office there too so I can start asking | 15:10 |
| *** jobewan has quit IRC | 15:10 | |
| samuelbercovici | jorgem: will you send such an email, or do you want me to? | 15:11 |
| thurloat | meatspace meetup won't work for me, long travel from canada :P | 15:11 |
| jorgem | samuelbercovici: I'll go ahead and send one | 15:11 |
| xuhanp | carl_baldwin: there is no L3 sub-team today? | 15:12 |
| samuelbercovici | jorgem:: 10x | 15:12 |
| carl_baldwin | xuhanp: #openstack-meeting-3 | 15:12 |
| openstackgerrit | A change was merged to openstack/python-neutronclient: Print human friendly string as an error message https://review.openstack.org/54525 | 15:12 |
| jorgem | thurloat: The designate team just did one and they hooked up video chat to it | 15:12 |
| jorgem | :) | 15:12 |
| swat30 | jorgem: should have the ability to remote in as well, maybe via hangouts? | 15:12 |
| swat30 | :D | 15:12 |
| jorgem | swat30: correct | 15:12 |
| swat30 | you beat me to the punch | 15:12 |
| jorgem | :) | 15:12 |
| thurloat | great idea :) | 15:13 |
| jorgem | The designate mini-summit was pretty productive and I like to be productive :) | 15:13 |
| *** jaypipes has joined #openstack-neutron | 15:13 | |
| jorgem | as I'm sure we all are | 15:13 |
| jorgem | face2face is just a much more pleasant experience | 15:13 |
| jorgem | even if it is google hang outs | 15:13 |
| *** sfox has quit IRC | 15:13 | |
| *** jobewan has joined #openstack-neutron | 15:15 | |
| samuelbercovici | jorgem: nned to quit for now. will be back in a few minutes | 15:15 |
| samuelbercovici | by | 15:15 |
| samuelbercovici | e | 15:15 |
| *** samuelbercovici has quit IRC | 15:15 | |
| *** uma has joined #openstack-neutron | 15:17 | |
| thurloat | yeah, I'd love to get more involved in the discussion and execution too. Been working on some other openstack services in recently taking up a lot of focus. | 15:17 |
| openstackgerrit | Salvatore Orlando proposed a change to openstack/neutron: Add support for tenant-provided NSX gateways devices https://review.openstack.org/74209 | 15:18 |
| *** armax has joined #openstack-neutron | 15:19 | |
| *** Guest46316 has quit IRC | 15:22 | |
| *** amotoki has joined #openstack-neutron | 15:23 | |
| *** piyush has joined #openstack-neutron | 15:27 | |
| openstackgerrit | Sylvain Afchain proposed a change to openstack/neutron: Add a new scheduler for the l3 HA https://review.openstack.org/66347 | 15:28 |
| *** yfried has quit IRC | 15:29 | |
| openstackgerrit | Phil Day proposed a change to openstack/python-neutronclient: Allow user ID for authentication https://review.openstack.org/78596 | 15:29 |
| *** WackoRobie has joined #openstack-neutron | 15:34 | |
| anteaya | jorgem swat30 thurloat the more you have discussions on irc the more the rest of the community is aware of what you are doing | 15:34 |
| anteaya | while this may be percieved as "slowing you down" when it comes time to integrate whatever you are working on with the rest of openstack, the process is first possible and second smoother | 15:35 |
| anteaya | ask jgallard about lxc in devstack as an example | 15:35 |
| thurloat | anteaya that's why I'm here :) I'm completely onboard with the process. | 15:36 |
| anteaya | thurloat: great | 15:36 |
| thurloat | just wanting to get involved in a more significant way | 15:36 |
| anteaya | so I have missed your mission | 15:36 |
| anteaya | are you completely new to openstack and looking to get started? | 15:36 |
| anteaya | or do you have a mandate handed down from on high? | 15:36 |
| *** markmcclain has joined #openstack-neutron | 15:36 | |
| *** blogan has quit IRC | 15:38 | |
| thurloat | neither :) I am from high, and have been just been lurking for a long time while we've been deploying our public cloud. | 15:39 |
| *** thedodd has joined #openstack-neutron | 15:39 | |
| anteaya | oh lovely | 15:40 |
| anteaya | great, so being from high | 15:40 |
| *** juice has quit IRC | 15:40 | |
| anteaya | what would you like to accomplish? | 15:40 |
| jorgem | anteaya: Correct, my intentions are to be as open as possible. As thurloat alluded to a richer form of communication is more productive from time to time. | 15:40 |
| anteaya | or what have you done thus far in openstack contribution? | 15:41 |
| anteaya | jorgem: while that can feel like the case, that can also lead to a difficult situation | 15:41 |
| *** jecarey has joined #openstack-neutron | 15:41 | |
| anteaya | let me dig up a ml post | 15:41 |
| *** safchain has quit IRC | 15:42 | |
| anteaya | http://lists.openstack.org/pipermail/openstack-dev/2014-March/028624.html | 15:42 |
| jorgem | anteaya: The Designate team just had a mini summit and it was well received so I'm just trying to mimic that if possible | 15:42 |
| anteaya | ah okay | 15:42 |
| anteaya | we too had a code sprint | 15:42 |
| jorgem | anteaya: Thank you for the heads up though. I definitely don't want to dig myself a hole | 15:42 |
| anteaya | http://lists.openstack.org/pipermail/openstack-dev/2014-February/027163.html | 15:42 |
| *** safchain has joined #openstack-neutron | 15:42 | |
| *** banix has joined #openstack-neutron | 15:43 | |
| anteaya | jorgem: awesome thank you for understanding | 15:43 |
| anteaya | so jorgem thurloat are you paddling the same boat? or different boats? | 15:43 |
| *** sweston has joined #openstack-neutron | 15:43 | |
| jorgem | anteaya: I'm trying to get involved in Neutron LBaaS. | 15:44 |
| anteaya | great | 15:44 |
| anteaya | have you meet anyone on that project yet? | 15:44 |
| anteaya | sweston: morning | 15:44 |
| thurloat | I see what you're saying anteaya , it's definitely not my intention to be some jerk trying to circumvent processes. | 15:44 |
| jorgem | yes. but not in person. I was just on the weekly IRC meeting | 15:44 |
| anteaya | thurloat: awesome, thank you | 15:44 |
| thurloat | am on the same train as jorgem WRT lbaas :) | 15:44 |
| anteaya | great | 15:44 |
| jorgem | anteaya: Which part of Neutron are you participating in mostly? | 15:45 |
| anteaya | jorgem: communication and integration with openstack as a whole | 15:46 |
| anteaya | testing and gate stability | 15:46 |
| anteaya | I am actually with the infra team | 15:46 |
| anteaya | stuck my foot in my mouth last summit because I had had enough during havanna feature freeze | 15:47 |
| anteaya | and organized the neutron tempest code sprint in january | 15:47 |
| anteaya | so I know a lot about how to contribute effectively to openstack - any project | 15:47 |
| anteaya | and almost nothing about neutron code | 15:48 |
| *** sweston has quit IRC | 15:48 | |
| jorgem | lol, well process is what I'm trying to learn so I might pick your brain from time to time if that's okay with you | 15:48 |
| swat30 | anteaya: for the sake of transparency, I work with OpenStack every day to provide a public cloud service. I chose OpenStack because I believe in the open nature, and vendor neutrality. We want to contribute back to the project more and more as we progress, including the decisions that are made which affect all of us operators :) | 15:49 |
| thurloat | thanks for the background, I didn't know which gatekeeper I had stumbled upon with the interrogation ;) | 15:49 |
| anteaya | jorgem: pick away, that is why I am here | 15:50 |
| *** _cerberus_ has joined #openstack-neutron | 15:50 | |
| anteaya | jorgem: also tee them up, i am gone for next week | 15:50 |
| anteaya | swat30: awesome and thanks for transparency | 15:51 |
| thurloat | for the sake of greater transparency, swat30 and I are from the same team :) | 15:51 |
| openstackgerrit | Hemanth Ravi proposed a change to openstack/neutron: One Convergence Neutron Plugin Implementation https://review.openstack.org/69246 | 15:51 |
| anteaya | swat30: glad to have you aboard | 15:51 |
| anteaya | thurloat: yes, good to learn the gate keepers | 15:51 |
| thurloat | he literally sits next to me | 15:51 |
| anteaya | thurloat: ha ha ha, good to know that too | 15:51 |
| swat30 | anteaya: glad to be here! I've been here for some time in the darkness | 15:51 |
| anteaya | since we are putting our cards on the table | 15:51 |
| anteaya | I am an astrologer and love patterns and speak my mind so I end up a good fit here | 15:52 |
| swat30 | and a good gate keeper :) | 15:52 |
| anteaya | my technical skills need work which is what I want to focus on for juno | 15:52 |
| anteaya | thanks | 15:52 |
| anteaya | hp pays my paycheque, I haven't talked to my boss in months | 15:52 |
| swat30 | that sounds like a fantastic relationship lol | 15:53 |
| anteaya | I do as I see fit and get some applause or scowls from others depending on what side of the fence they are on | 15:53 |
| anteaya | I miss him actually, I really enjoy talking to him but he is busy with internal stuff which I have no interest in | 15:53 |
| *** xuhanp has quit IRC | 15:54 | |
| anteaya | that's about it | 15:54 |
| *** tomoe_ has quit IRC | 15:54 | |
| anteaya | I hang out with the infra team, mostly doing customer service when our testing structure has an issue and helping people learn our system | 15:54 |
| swat30 | well great to meet you anteaya ! I'm sure you'll be seeing lots of thurloat and I | 15:54 |
| anteaya | and want to focus on more code coming up | 15:54 |
| anteaya | swat30: glad to hear that | 15:55 |
| anteaya | okay from the top | 15:55 |
| anteaya | does everyone have launchpad accounts, gerrit accounts and has submitted at least one patchset? | 15:55 |
| anteaya | if you have submitted a patchset please post the url of one patch | 15:56 |
| anteaya | as an example of what I am looking for: https://review.openstack.org/#/c/77376/ | 15:57 |
| anteaya | if no, say no | 15:57 |
| anteaya | I just need to know where you are starting from | 15:57 |
| anteaya | don't want to skip anything but don't want to bore you either | 15:57 |
| *** dvorkinista has joined #openstack-neutron | 15:57 | |
| HenryG | amotoki: ping ... have question(s) greenthread switching | 15:57 |
| openstackgerrit | Jon Grimm proposed a change to openstack/neutron: Fix ml2 & nec plugins for allowedaddresspairs tests https://review.openstack.org/58896 | 15:58 |
| thurloat | have launchpad and gerrit accounts, have not signed the contributor agreement | 15:58 |
| swat30 | anteaya: https://launchpad.net/~jacobgodin haven't submitted a patchset, but have done a couple bug reports etc. I'm not a super-coder, that would be thurloat who has just joined the team. I'm more DevOps I guess you would say | 15:58 |
| anteaya | thurloat: great, can I help at all to support you doing so? | 15:58 |
| anteaya | swat30: great infra is all devops | 15:58 |
| *** xuhanp has joined #openstack-neutron | 15:59 | |
| anteaya | so the more you know the better you can contribute that knowledge to neutron | 15:59 |
| anteaya | swat30: do you have a gerrit account yet? | 15:59 |
| anteaya | if no, say no | 15:59 |
| swat30 | yup, just created one :) | 15:59 |
| anteaya | swat30: k | 15:59 |
| jgallard | anteaya, :) | 15:59 |
| amotoki | HenryG: hi | 15:59 |
| anteaya | log in, put in your username and ask any questions you have | 15:59 |
| anteaya | jgallard: how is the new direction coming? | 15:59 |
| thurloat | swat30 and I are a two headed beast. I bring the programming background and he brings the ops. | 16:00 |
| HenryG | amotoki: you mention in a review comment that greenthread switching occurs when "some I/O operations are included" | 16:00 |
| anteaya | jgallard: have you been able to convey your findings to the higher ups? | 16:00 |
| *** jroovers has joined #openstack-neutron | 16:00 | |
| anteaya | thurloat: great glad to have teams | 16:00 |
| thurloat | I assume that the " | 16:00 |
| anteaya | thurloat: the problem you are going to face is to have your conversations on irc rather than face to face | 16:00 |
| thurloat | DONT SIGN THIS" means, not to sign the agreement ? | 16:00 |
| HenryG | amotoki: which I/O operations? How do you find them? | 16:00 |
| amotoki | HenryG: I am not sure the detail about when greenthread is switched. | 16:01 |
| anteaya | thurloat: correct, you need to sign the _individual_ agreement | 16:01 |
| thurloat | bingo. | 16:01 |
| jgallard | we are studying all the third-party stuff | 16:01 |
| jgallard | anteaya, ^ | 16:01 |
| *** blogan has joined #openstack-neutron | 16:01 | |
| anteaya | thurloat: you have probably found the government one, that we can't remove and that people find and sign and then we have to clear the db | 16:01 |
| swat30 | lol | 16:02 |
| jgallard | anteaya, we are studying in which way we can improve nodepool for multinode | 16:02 |
| amotoki | HenryG: honestly speaking I would like to know what is the conditions where exception context is lost as described in save_and_reraise docstring... | 16:02 |
| openstackgerrit | Rob Pothier proposed a change to openstack/neutron: Enable delete_port to tolerate deadlocks https://review.openstack.org/78643 | 16:02 |
| anteaya | jgallard: great, do speak up if you need to confirm anything, I will be away next week so anything in infra or qa is a good place to ask questions | 16:02 |
| HenryG | amotoki: OK, but in the opendaylight code review you say "in this case ... no greenthread swithing occurs". How could you be sure in that case? | 16:02 |
| anteaya | jgallard: for nodepool, jeblair is your man | 16:02 |
| anteaya | jgallard: and mikal and jhesketh | 16:03 |
| amotoki | HenryG: in ODL driver case, there is no I/O operation and just call variable assignment. | 16:03 |
| *** jecarey has quit IRC | 16:03 | |
| *** jroovers|afk has joined #openstack-neutron | 16:03 | |
| amotoki | HenryG: eventlet patches several module which involves I/O operations. They triggers greenthread switching. | 16:04 |
| HenryG | amotoki: sendjson() is not I/O? | 16:04 |
| *** jroovers has quit IRC | 16:05 | |
| anteaya | in case you haven't seen this yet: https://wiki.openstack.org/wiki/Gerrit_Workflow | 16:05 |
| anteaya | this is very handy | 16:05 |
| jgallard | anteaya, thanks a lot for the contacts, we will continue to study this, when we will be ready (I hope quickly), we will come and ask/propose things :-) | 16:05 |
| *** WackoRobie has quit IRC | 16:05 | |
| amotoki | HenryG: only "self.out_of_sync = True" is called at the line I commented. | 16:05 |
| amotoki | HenryG: sendjson() is called before entering exception handling, so it is okay in my understanding. | 16:06 |
| anteaya | jgallard: great don't be shy on the ask/propose things | 16:06 |
| anteaya | jgallard: it is better for you that you let jeblair know and participate in the design of anything you are thinking of doing | 16:07 |
| anteaya | have some chats with him | 16:07 |
| *** juice has joined #openstack-neutron | 16:07 | |
| anteaya | if you do, your code stands a much better chance of being accepting when you do write it | 16:07 |
| anteaya | don't worry about the quickly | 16:08 |
| anteaya | multi-node is going to be comples | 16:08 |
| *** mlemay has joined #openstack-neutron | 16:08 | |
| anteaya | complex | 16:08 |
| jgallard | ohhh yes | 16:08 |
| openstackgerrit | Tomoe Sugihara proposed a change to openstack/neutron: Implement MidoNet plugin for Icehouse (Part 2) https://review.openstack.org/78575 | 16:08 |
| anteaya | and affect a lot of people, since what ever does get in has to work with everybody who needs multi-node | 16:08 |
| anteaya | are you in #openstack-infra? | 16:09 |
| jgallard | ok, I will do and speak with him, just want to understand the picture before to discuss | 16:09 |
| anteaya | do you follow the conversations about zuul? it comes up daily | 16:09 |
| jgallard | yes | 16:09 |
| anteaya | great | 16:09 |
| jgallard | I do my best to follow them | 16:09 |
| anteaya | great | 16:10 |
| anteaya | and speak up | 16:10 |
| anteaya | ask questions | 16:10 |
| openstackgerrit | Hemanth Ravi proposed a change to openstack/neutron: One Convergence Neutron Plugin Implementation https://review.openstack.org/69246 | 16:10 |
| jgallard | ok | 16:11 |
| anteaya | great | 16:11 |
| anteaya | and thanks for understanding | 16:11 |
| jgallard | thanks to you for your time to explain that! | 16:11 |
| *** carl_baldwin has quit IRC | 16:12 | |
| thurloat | anteaya agreement signed -- we're on our way! | 16:12 |
| *** xuhanp has quit IRC | 16:12 | |
| *** carl_baldwin has joined #openstack-neutron | 16:13 | |
| thurloat | anteaya: etiquette question: appropriate to hop on code reviews that you've not been a part of to date? | 16:13 |
| *** chandan_kumar has quit IRC | 16:13 | |
| HenryG | amotoki: thanks! | 16:15 |
| anteaya | jgallard: my pleasure | 16:15 |
| anteaya | thurloat: go you | 16:15 |
| anteaya | thurloat: define hop on? | 16:15 |
| anteaya | thurloat: do you mean review a patch? | 16:16 |
| anteaya | yes do review a patch | 16:16 |
| thurloat | yep | 16:16 |
| anteaya | http://anteaya.info/blog/2013/03/21/reviewing-an-openstack-patch/ | 16:16 |
| *** carl_baldwin has quit IRC | 16:16 | |
| thurloat | sounds like you've had this question asked before ;) | 16:16 |
| anteaya | I've asked this question before | 16:17 |
| anteaya | most of my blog posts are notes to myself | 16:18 |
| anteaya | but you can read them too | 16:18 |
| *** dvorkinista has quit IRC | 16:19 | |
| *** dfarrell07 has joined #openstack-neutron | 16:19 | |
| *** changbl has joined #openstack-neutron | 16:20 | |
| openstackgerrit | Darren Birkett proposed a change to openstack/neutron: Correctly inherit __table_args__ from parent class https://review.openstack.org/78649 | 16:20 |
| *** thuc has joined #openstack-neutron | 16:24 | |
| *** thuc has quit IRC | 16:25 | |
| *** thuc has joined #openstack-neutron | 16:25 | |
| openstackgerrit | Tomoe Sugihara proposed a change to openstack/neutron: Implement MidoNet plugin for Icehouse (Part 1) https://review.openstack.org/78543 | 16:25 |
| openstackgerrit | Aaron Rosen proposed a change to openstack/neutron: Notify nova when ports are ready https://review.openstack.org/75253 | 16:25 |
| openstackgerrit | Aaron Rosen proposed a change to openstack/neutron: Send network-changed notifications to nova https://review.openstack.org/78041 | 16:25 |
| openstackgerrit | Aaron Rosen proposed a change to openstack/neutron: Query for port before calling l3plugin.disassociate_floatingips() https://review.openstack.org/78399 | 16:25 |
| *** thuc has quit IRC | 16:26 | |
| *** jecarey has joined #openstack-neutron | 16:26 | |
| openstackgerrit | Tomoe Sugihara proposed a change to openstack/neutron: Implement MidoNet plugin for Icehouse (Part 2) https://review.openstack.org/78575 | 16:26 |
| *** thuc has joined #openstack-neutron | 16:26 | |
| amotoki | HenryG: i am studying eventlet too :-) | 16:26 |
| *** thuc has quit IRC | 16:27 | |
| *** thuc has joined #openstack-neutron | 16:27 | |
| *** thuc has quit IRC | 16:28 | |
| *** _cjones_ has joined #openstack-neutron | 16:28 | |
| *** thuc has joined #openstack-neutron | 16:28 | |
| *** vjay has quit IRC | 16:29 | |
| *** yfried has joined #openstack-neutron | 16:29 | |
| *** dvorkinista has joined #openstack-neutron | 16:32 | |
| *** mlavalle has joined #openstack-neutron | 16:35 | |
| *** vu_ has quit IRC | 16:35 | |
| *** afazekas has quit IRC | 16:37 | |
| *** vjay has joined #openstack-neutron | 16:43 | |
| *** sweston has joined #openstack-neutron | 16:44 | |
| *** SumitNaiksatam has quit IRC | 16:45 | |
| *** dims has quit IRC | 16:45 | |
| *** dvorkinista has quit IRC | 16:46 | |
| *** SumitNaiksatam has joined #openstack-neutron | 16:46 | |
| openstackgerrit | Ihar Hrachyshka proposed a change to openstack/neutron: Port to oslo.messaging (WIP) https://review.openstack.org/75861 | 16:49 |
| *** yamahata has joined #openstack-neutron | 16:50 | |
| *** carl_baldwin has joined #openstack-neutron | 16:50 | |
| *** sweston has quit IRC | 16:51 | |
| *** samuelbercovici has joined #openstack-neutron | 16:52 | |
| openstackgerrit | Tomoe Sugihara proposed a change to openstack/neutron: Implement MidoNet plugin for Icehouse (Part 1) https://review.openstack.org/78543 | 16:53 |
| *** packet has joined #openstack-neutron | 16:53 | |
| JoeHazzers | perhaps i'm missing something, but my extension doesn't seem to be actually extending attributes. | 16:55 |
| *** bada has joined #openstack-neutron | 16:56 | |
| *** dims has joined #openstack-neutron | 17:00 | |
| *** harlowja has joined #openstack-neutron | 17:02 | |
| *** bada has quit IRC | 17:05 | |
| *** rwsu has joined #openstack-neutron | 17:08 | |
| *** devvesa has quit IRC | 17:11 | |
| *** vjay has quit IRC | 17:11 | |
| *** yfried has quit IRC | 17:12 | |
| *** rotbeard has quit IRC | 17:15 | |
| *** vjay has joined #openstack-neutron | 17:16 | |
| *** jroovers|afk has quit IRC | 17:18 | |
| *** shashank_ has joined #openstack-neutron | 17:19 | |
| *** manishg has joined #openstack-neutron | 17:19 | |
| anteaya | thurloat: I see in backscroll you are canuck, where abouts are you freezing these days | 17:25 |
| *** nati_uen_ has quit IRC | 17:27 | |
| *** shashank_ has quit IRC | 17:35 | |
| *** sfox has joined #openstack-neutron | 17:35 | |
| *** rkukura has quit IRC | 17:35 | |
| *** jgallard has quit IRC | 17:35 | |
| *** shashank_ has joined #openstack-neutron | 17:37 | |
| *** dfarrell07 has quit IRC | 17:38 | |
| *** vkozhukalov has joined #openstack-neutron | 17:38 | |
| *** safchain has quit IRC | 17:39 | |
| *** jpich has quit IRC | 17:40 | |
| *** jistr has quit IRC | 17:40 | |
| *** ramishra has quit IRC | 17:40 | |
| *** jlibosva has quit IRC | 17:41 | |
| *** vu_ has joined #openstack-neutron | 17:42 | |
| openstackgerrit | A change was merged to openstack/neutron: Add OpenDaylight ML2 MechanismDriver https://review.openstack.org/69775 | 17:43 |
| openstackgerrit | A change was merged to openstack/neutron: Imported Translations from Transifex https://review.openstack.org/78114 | 17:44 |
| *** dvorkinista has joined #openstack-neutron | 17:45 | |
| *** dvorkinista has quit IRC | 17:45 | |
| *** vu_ has quit IRC | 17:46 | |
| *** dvorkinista has joined #openstack-neutron | 17:46 | |
| *** vu_ has joined #openstack-neutron | 17:46 | |
| *** rkukura has joined #openstack-neutron | 17:48 | |
| *** vjay has quit IRC | 17:52 | |
| *** vjay has joined #openstack-neutron | 17:52 | |
| thurloat | anteaya Halifax, NS :) | 17:56 |
| thurloat | not quite as cold as the rest of the country (--40 holy cow) though | 17:56 |
| *** banix has quit IRC | 17:57 | |
| *** jroovers has joined #openstack-neutron | 17:57 | |
| *** jroovers|afk has joined #openstack-neutron | 17:58 | |
| anteaya | thurloat: I was wondering if that was where you were | 18:00 |
| anteaya | somehow I pictured a coast | 18:00 |
| anteaya | and BC is not awake yet | 18:00 |
| anteaya | Ontario | 18:00 |
| *** sballe has joined #openstack-neutron | 18:01 | |
| anteaya | ohhh, just checked the weather | 18:01 |
| anteaya | t-shirt weather, -8 | 18:01 |
| *** sfox1 has joined #openstack-neutron | 18:01 | |
| *** jroovers has quit IRC | 18:01 | |
| anteaya | thurloat: how many potential openstack contributors have you there? | 18:02 |
| *** simon-AS559 has quit IRC | 18:02 | |
| *** simon-AS559 has joined #openstack-neutron | 18:02 | |
| *** evgenyf has quit IRC | 18:02 | |
| *** sfox has quit IRC | 18:03 | |
| *** sballe_ has joined #openstack-neutron | 18:03 | |
| *** sballe has quit IRC | 18:04 | |
| *** sballe_ has quit IRC | 18:04 | |
| *** sballe has joined #openstack-neutron | 18:04 | |
| *** sfox1 has quit IRC | 18:05 | |
| *** ygbo has quit IRC | 18:06 | |
| *** simon-AS559 has quit IRC | 18:07 | |
| *** chandan_kumar has joined #openstack-neutron | 18:07 | |
| *** sfox has joined #openstack-neutron | 18:07 | |
| *** ajo has quit IRC | 18:08 | |
| ihrachys | I get some test failures locally after recent ssl bigswitch change | 18:09 |
| *** nati_ueno has joined #openstack-neutron | 18:09 | |
| ihrachys | like http://pastebin.com/zJXq6CUX | 18:09 |
| anteaya | I think kevinbenton is bigswitch | 18:10 |
| ihrachys | it seems it tries to access certificates in /etc/. is it expected? | 18:10 |
| *** kvalena has joined #openstack-neutron | 18:10 | |
| anteaya | ihrachys: sounds like you did not expect that | 18:13 |
| *** vjay has quit IRC | 18:13 | |
| ihrachys | anteaya: I didn't expect that tests fail :) and that certs are accessed under /etc/neutron/plugins/bigswitch/ssl | 18:14 |
| anteaya | great, so bigswitch need to help you understand or meet your expectaions better | 18:15 |
| ihrachys | indeed :) | 18:15 |
| anteaya | if noone from bigswitch is around | 18:15 |
| anteaya | got to this list: https://review.openstack.org/#/admin/groups/91,members | 18:15 |
| anteaya | email bigswitch, cc'ing infra ml: openstack-infra@lists.openstack.org | 18:16 |
| ihrachys | anteaya: sorry, probably you understood me wrong. I get failures locally, not from Bigswitch bot | 18:16 |
| anteaya | ah | 18:16 |
| anteaya | why are you running bigswitch locally? | 18:16 |
| *** piyush has quit IRC | 18:16 | |
| anteaya | obviously I am not understanding yet | 18:16 |
| ihrachys | it just that it revealed after recent rebase including their ssl change | 18:16 |
| ihrachys | I run unit tests | 18:17 |
| ihrachys | and bigswitch tests fail due to the problem | 18:17 |
| anteaya | ah | 18:17 |
| anteaya | the bigswitch driver code in neutron master is failing in your unit tests that you are running locally | 18:17 |
| anteaya | yes? | 18:17 |
| ihrachys | yes :) | 18:18 |
| *** shakayumi has quit IRC | 18:18 | |
| ihrachys | so maybe this is my fault, but the error I get looks weird because it tries to access certificates under some weird system path | 18:18 |
| *** dfarrell07 has joined #openstack-neutron | 18:19 | |
| anteaya | k well the only bigswitch person I know is kevinbenton | 18:20 |
| ihrachys | ok, let's wait for him | 18:20 |
| anteaya | so hopefully he will respond and the two you can have a lovely chat | 18:20 |
| anteaya | k | 18:21 |
| openstackgerrit | Rob Pothier proposed a change to openstack/neutron: Enable delete_port to tolerate deadlocks https://review.openstack.org/78643 | 18:21 |
| *** dfarrell07 has quit IRC | 18:22 | |
| *** rms_13 has joined #openstack-neutron | 18:23 | |
| *** thedodd has quit IRC | 18:26 | |
| *** ajo has joined #openstack-neutron | 18:28 | |
| *** mlemay has quit IRC | 18:32 | |
| *** ryu25 has quit IRC | 18:32 | |
| *** thuc has quit IRC | 18:39 | |
| *** thuc has joined #openstack-neutron | 18:39 | |
| *** dfarrell07 has joined #openstack-neutron | 18:40 | |
| *** jorgem has quit IRC | 18:40 | |
| *** jorgem has joined #openstack-neutron | 18:41 | |
| openstackgerrit | A change was merged to openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/75588 | 18:41 |
| *** dfarrell07 has quit IRC | 18:41 | |
| kevinbenton | ihrachys: ping | 18:42 |
| *** dfarrell07 has joined #openstack-neutron | 18:42 | |
| *** thuc_ has joined #openstack-neutron | 18:42 | |
| *** dfarrell07 has quit IRC | 18:42 | |
| *** thuc has quit IRC | 18:43 | |
| *** thuc__ has joined #openstack-neutron | 18:43 | |
| *** thuc__ has quit IRC | 18:44 | |
| *** thuc has joined #openstack-neutron | 18:44 | |
| *** yfried has joined #openstack-neutron | 18:45 | |
| *** dfarrell07 has joined #openstack-neutron | 18:45 | |
| *** dfarrell07 has quit IRC | 18:45 | |
| rms_13 | <rms_13> salv-orlando: ping | 18:46 |
| *** dfarrell07 has joined #openstack-neutron | 18:46 | |
| kevinbenton | anteaya: ping | 18:46 |
| *** yfried has quit IRC | 18:47 | |
| *** kvalena has quit IRC | 18:47 | |
| *** mlemay has joined #openstack-neutron | 18:47 | |
| *** yfried has joined #openstack-neutron | 18:47 | |
| *** thuc_ has quit IRC | 18:47 | |
| *** sweston has joined #openstack-neutron | 18:47 | |
| *** sff has joined #openstack-neutron | 18:48 | |
| *** zzelle has joined #openstack-neutron | 18:49 | |
| *** ywu has joined #openstack-neutron | 18:49 | |
| *** dfarrell07 has quit IRC | 18:49 | |
| *** dfarrell07 has joined #openstack-neutron | 18:49 | |
| *** mwagner_lap has quit IRC | 18:51 | |
| openstackgerrit | Sandhya Dasu proposed a change to openstack/neutron: Implements a ML2 mechanism driver for Cisco UCS Manager https://review.openstack.org/74134 | 18:51 |
| *** banix has joined #openstack-neutron | 18:55 | |
| *** dfarrell07 has quit IRC | 18:56 | |
| anteaya | kevinbenton: hello there | 18:56 |
| kevinbenton | anteaya: ihrachys was having trouble with a BigSwitch UT. Do you know if that got resolved? | 18:57 |
| anteaya | so far my understanding is that it is not resolved | 18:58 |
| anteaya | however I am a couch potato on this one | 18:58 |
| *** dfarrell07 has joined #openstack-neutron | 18:58 | |
| anteaya | so hopefully ihrachys well be back shortly and will update you | 18:58 |
| kevinbenton | anteaya: ok. i'll wait to hear from ihrachys | 18:58 |
| anteaya | great | 18:59 |
| anteaya | thanks | 18:59 |
| *** dims has quit IRC | 19:00 | |
| *** ywu has quit IRC | 19:01 | |
| *** dims has joined #openstack-neutron | 19:02 | |
| JoeHazzers | would you guys be interested in me writing a full-blown guide/tutorial on writing a neutron extension? | 19:03 |
| *** jecarey_ has joined #openstack-neutron | 19:04 | |
| *** crc32 has joined #openstack-neutron | 19:05 | |
| *** jecarey has quit IRC | 19:06 | |
| anteaya | JoeHazzers: whose neutron extention do you like that you would like to follow their example? | 19:07 |
| JoeHazzers | oh well | 19:08 |
| JoeHazzers | that's a whole different kettle of fish | 19:08 |
| JoeHazzers | if only the extensions had agreed on a naming convention / structure convention | 19:08 |
| JoeHazzers | you see some with base files, you see some without database, you see some swapping RESOURCE_ATTRIBUTE_MAP and EXTENDED_ATTRIBUTES_2_0 around in their source files, some using and not using constants at the top of the extensions files | 19:09 |
| JoeHazzers | different methods of hooking into updating dicts for extended attributes | 19:09 |
| JoeHazzers | although i quite like the external_net at the moment | 19:09 |
| enikanorov | JoeHazzers: RESOURCE_ATTRIBUTE_MAP and EXTENDED_ATTRIBUTES_2_0 are just variable names | 19:11 |
| *** skraynev is now known as skraynev_afk | 19:11 | |
| JoeHazzers | enikanorov: that may be the case, but it's much easier if everyone uses the same variable names in the same contexts | 19:12 |
| JoeHazzers | (take a look at the get_extended_resources and get_resources methods, they share a lot of common code | 19:12 |
| enikanorov | it's not a big deal really. those variables are not directly exposed to the outside | 19:12 |
| JoeHazzers | but you do see a few minor differences, like returing [controller-create-method-here] or exts = controller-create-method-here; return [exts] | 19:13 |
| JoeHazzers | i'm not saying it's a big deal | 19:13 |
| enikanorov | JoeHazzers: they do and i believe some extensions were refactored recently so the duplication is removed | 19:13 |
| JoeHazzers | but it improves readability and reduces confusion | 19:13 |
| JoeHazzers | yes, i noticed, but a few are still swapped around | 19:13 |
| JoeHazzers | i'm not saying the code doesn't work, i'm just saying it's harder for an outsider to penetrate for the first time | 19:13 |
| *** alexpilotti has joined #openstack-neutron | 19:13 | |
| *** ywu has joined #openstack-neutron | 19:14 | |
| JoeHazzers | and the current approach of "if you want to know what extensions can do, read the tests" isn't very approachable | 19:14 |
| enikanorov | those few could be refactored in the same way, i suppose | 19:14 |
| *** alexpilotti has quit IRC | 19:15 | |
| enikanorov | JoeHazzers: i think that API extension is defined by it's RESOURCE_ATTRIBUTE_MAP | 19:15 |
| *** alexpilotti has joined #openstack-neutron | 19:15 | |
| enikanorov | that's the first place one should look to understand what extension does | 19:15 |
| JoeHazzers | i think some comprehensive documentation about extensions and how the API works would be quite nice to have | 19:15 |
| JoeHazzers | see: wishlist | 19:15 |
| *** blogan has quit IRC | 19:16 | |
| enikanorov | more documentation is always good, no doubt | 19:16 |
| JoeHazzers | might shove it up as a blueprint if it's not already | 19:16 |
| *** sballe has quit IRC | 19:17 | |
| *** piyush has joined #openstack-neutron | 19:17 | |
| JoeHazzers | i had to learn how the API worked (controllers, plugins, extensions, extending attributes, declaring attributes, mixins) by trial and error mostly | 19:17 |
| *** abhiraut has joined #openstack-neutron | 19:17 | |
| *** blogan has joined #openstack-neutron | 19:18 | |
| *** piyush1 has joined #openstack-neutron | 19:19 | |
| openstackgerrit | Ronak Shah proposed a change to openstack/neutron: Nuage plugin was missed in floatingip_status db migration script https://review.openstack.org/78720 | 19:19 |
| *** sweston has quit IRC | 19:20 | |
| *** thedodd has joined #openstack-neutron | 19:20 | |
| openstackgerrit | Kevin Benton proposed a change to openstack/neutron: Mock agent RPC for FWaaS tests to delete DB objs https://review.openstack.org/78457 | 19:21 |
| *** piyush has quit IRC | 19:21 | |
| *** luqas has quit IRC | 19:22 | |
| blogan | JoeHazzers: I agree with you its tough to get your bearings at first and figure out how everything fits in, i would +1 that wishlist item | 19:22 |
| JoeHazzers | I will have some time in the next few months (and a spare summer, i will return to being a student) to write some of this documentation | 19:22 |
| *** dvorkinista has quit IRC | 19:22 | |
| blogan | i'm currently writing up a code workflow document on neutron lbaas, it'll include the extension, plugin, drivers, and agent | 19:24 |
| markmcclain | JoeHazzers: I expect the way to write extensions will be changing in Juno | 19:24 |
| openstackgerrit | Ronak Shah proposed a change to openstack/neutron: Nuage plugin was missed in floatingip_status db migration script https://review.openstack.org/78720 | 19:24 |
| JoeHazzers | markmcclain: and what will happen to the existing extensions? | 19:24 |
| markmcclain | part of the process will be to convert them to new form | 19:25 |
| blogan | markmclain: any kind of documentation on how it will change? | 19:25 |
| rms_13 | salv-orlando: ping | 19:25 |
| markmcclain | blogan: that is inflight | 19:25 |
| markmcclain | I expect there will be more info later this month | 19:26 |
| JoeHazzers | the problem with neutron seems to be the lack of anything on paper :3 | 19:26 |
| markmcclain | JoeHazzers: that's part of what we'll be fixing | 19:26 |
| JoeHazzers | i'm more than happy to help with that | 19:27 |
| JoeHazzers | since i do need to twist neutron a bit to get it to work in our infrastructure, plus we'll be needing a few features | 19:27 |
| blogan | markmcclain: is it a discussion in the mailing list? i'm fairly new to openstack so I don't know if there is some committee that decides major changes like that or if it is just held in a public forum until a consensus is reached | 19:27 |
| *** bada has joined #openstack-neutron | 19:27 | |
| JoeHazzers | see: restricting subnets to hosts/agents, create a port specifying which host/agent it will be bound on, extensible dns management, external IP and MAC address selection | 19:28 |
| markmcclain | JoeHazzers: thanks.. I'll keep you in the loop when we start the discussion in public | 19:28 |
| salv-orlando | hi rms_13 | 19:28 |
| *** amuller has quit IRC | 19:28 | |
| markmcclain | blogan: we haven't picked up the discussion since the summit in HKG… mainly because I did not want to distract from the current Icehouse work | 19:28 |
| JoeHazzers | markmcclain: want to talk about it in atlanta? :3 | 19:29 |
| rms_13 | salv-orlando: hi, In floatingip_status migration that got in yesterday, nuage plugin was missed from the list, can you please review https://review.openstack.org/78720 | 19:29 |
| blogan | ah ok thanks, i think it is a good idea to simplify it and obviously more documentation is better | 19:29 |
| blogan | and standardize it | 19:30 |
| rms_13 | salv-orlando: Nuage CI is failing since than reporting unknown column "last_known_router_id", which was part of the new migration. The change fixes the problem | 19:30 |
| JoeHazzers | markmcclain: does the proposal for extensions include extensions relying on other extensions, "better" loading (no more mixin + supported aliases) and perhaps | 19:31 |
| JoeHazzers | EVEN | 19:31 |
| JoeHazzers | this would be my dream come true | 19:31 |
| JoeHazzers | allowing the client to dynamically support extensions by getting metadata from the server | 19:31 |
| markmcclain | blogan: agreed…better developer docs are an expect product of that work | 19:31 |
| *** manishg_ has joined #openstack-neutron | 19:32 | |
| markmcclain | JoeHazzers: I do expect a refactoring of how extensions are loaded | 19:33 |
| JoeHazzers | markmcclain: then i'm very interested indeed. | 19:33 |
| markmcclain | also I expect the process will be more explicit than the current process | 19:33 |
| *** sweston has joined #openstack-neutron | 19:33 | |
| salv-orlando | rms_13: patch is good, can I just ask for a few amendments to the commit message? | 19:34 |
| salv-orlando | sorry about the pedantry | 19:34 |
| *** manishg has quit IRC | 19:36 | |
| *** manishg_ is now known as manishg | 19:36 | |
| rms_13 | salv-orlando: No worries. Will make the change. Always good to learn something new :) | 19:36 |
| *** annashen_ has quit IRC | 19:36 | |
| JoeHazzers | markmcclain: in that case, i guess i can register the "better-everything" blueprint, set the milestone for icehouse, and set you as assignee? :D | 19:37 |
| markmcclain | JoeHazzers: I think I've submitted that design summit session for like 3 summits :) | 19:38 |
| *** jecarey_ has quit IRC | 19:38 | |
| *** jecarey has joined #openstack-neutron | 19:39 | |
| JoeHazzers | Well I'll come and say hi at the summit regardless | 19:40 |
| *** jp_at_hp has quit IRC | 19:40 | |
| JoeHazzers | which will be like 1 month before of my internship (which is the entire reason i'm working with openstack) | 19:40 |
| openstackgerrit | Ronak Shah proposed a change to openstack/neutron: Nuage plugin was missed in floatingip_status db migration script https://review.openstack.org/78720 | 19:40 |
| *** samuelbercovici has quit IRC | 19:42 | |
| blogan | JoeHazzers: where is your internship | 19:43 |
| *** dave_tucker_zzz is now known as dave_tucker | 19:43 | |
| JoeHazzers | CERN | 19:47 |
| *** bada has quit IRC | 19:48 | |
| blogan | awesome, bring proof of the higgs to atlanta | 19:50 |
| JoeHazzers | i shall bring my best CERN t-shirt and higgs boson plushie | 19:50 |
| blogan | thats proof enough for me! | 19:51 |
| *** ajo has quit IRC | 19:51 | |
| *** shashank_ has quit IRC | 19:53 | |
| *** jroovers|afk has quit IRC | 19:53 | |
| *** shashank_ has joined #openstack-neutron | 19:54 | |
| *** markwash has joined #openstack-neutron | 19:56 | |
| *** annashen_ has joined #openstack-neutron | 19:57 | |
| *** luqas has joined #openstack-neutron | 20:00 | |
| *** openstackgerrit has quit IRC | 20:00 | |
| *** openstackgerrit_ has joined #openstack-neutron | 20:00 | |
| *** openstackgerrit_ is now known as openstackgerrit | 20:01 | |
| *** hemanthravi has quit IRC | 20:01 | |
| *** echohead has joined #openstack-neutron | 20:03 | |
| *** harlowja has quit IRC | 20:03 | |
| *** echohead_ has quit IRC | 20:03 | |
| *** vu_ has quit IRC | 20:03 | |
| *** harlowja has joined #openstack-neutron | 20:04 | |
| *** circ-user-2FKUf has joined #openstack-neutron | 20:04 | |
| abhiraut | i have a naive question about port delete ... anyone on here who can help me? | 20:05 |
| ihrachys | kevinbenton: not solved, haven't spent more time on this | 20:06 |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: NSX: Propagate name updates for security profiles https://review.openstack.org/78744 | 20:07 |
| *** vkozhukalov has quit IRC | 20:08 | |
| *** sweston has quit IRC | 20:11 | |
| *** simon-AS559 has joined #openstack-neutron | 20:15 | |
| *** ihrachys is now known as ihrachys|afk | 20:15 | |
| *** dfarrell07 has quit IRC | 20:16 | |
| *** dfarrell07 has joined #openstack-neutron | 20:17 | |
| *** simon-AS559 has quit IRC | 20:19 | |
| *** vu_ has joined #openstack-neutron | 20:20 | |
| *** shivharis has joined #openstack-neutron | 20:21 | |
| *** simon-AS559 has joined #openstack-neutron | 20:22 | |
| openstackgerrit | Darren Birkett proposed a change to openstack/neutron: Correctly inherit __table_args__ from parent class https://review.openstack.org/78649 | 20:23 |
| *** zzelle has quit IRC | 20:29 | |
| *** vu_ has quit IRC | 20:31 | |
| *** luqas has quit IRC | 20:35 | |
| *** devlaps has joined #openstack-neutron | 20:36 | |
| openstackgerrit | Ivar Lazzaro proposed a change to openstack/neutron: Embrane LBaaS Driver https://review.openstack.org/54236 | 20:39 |
| *** rossella_s has quit IRC | 20:40 | |
| *** rossella_s has joined #openstack-neutron | 20:42 | |
| *** hemanthravi has joined #openstack-neutron | 20:43 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Makes the Extension loader behavior predictable https://review.openstack.org/78751 | 20:43 |
| *** vu_ has joined #openstack-neutron | 20:44 | |
| *** dave_tucker is now known as dave_tucker_zzz | 20:44 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Makes the Extension loader behavior predictable https://review.openstack.org/78751 | 20:44 |
| *** mlemay has quit IRC | 20:46 | |
| openstackgerrit | Armando Migliaccio proposed a change to openstack/neutron: Drop old nvp extension file https://review.openstack.org/78754 | 20:47 |
| *** sweston has joined #openstack-neutron | 20:49 | |
| *** vu_ has quit IRC | 20:49 | |
| *** otherwiseguy has quit IRC | 20:51 | |
| *** hemanthravi has quit IRC | 20:52 | |
| *** vu_ has joined #openstack-neutron | 20:52 | |
| *** jroovers has joined #openstack-neutron | 20:54 | |
| *** jroovers|afk has joined #openstack-neutron | 20:55 | |
| *** jroovers has quit IRC | 20:59 | |
| *** SumitNaiksatam has quit IRC | 21:00 | |
| *** jorgem1 has joined #openstack-neutron | 21:00 | |
| *** SumitNaiksatam has joined #openstack-neutron | 21:00 | |
| *** samuelbercovici has joined #openstack-neutron | 21:00 | |
| *** shakayumi has joined #openstack-neutron | 21:01 | |
| *** jorgem has quit IRC | 21:02 | |
| *** dvorkinista has joined #openstack-neutron | 21:02 | |
| openstackgerrit | A change was merged to openstack/neutron: One Convergence Neutron Plugin Implementation https://review.openstack.org/69246 | 21:03 |
| openstackgerrit | A change was merged to openstack/python-neutronclient: New exception when auth_url is not specified https://review.openstack.org/53461 | 21:03 |
| *** sweston has quit IRC | 21:04 | |
| *** vu_ has quit IRC | 21:05 | |
| *** jorgem1 has quit IRC | 21:08 | |
| *** manishg has quit IRC | 21:08 | |
| *** jorgem has joined #openstack-neutron | 21:08 | |
| *** manishg has joined #openstack-neutron | 21:09 | |
| *** sweston has joined #openstack-neutron | 21:09 | |
| jogo | salv-orlando markmcclain: Bug 1288427 has a lot of hits in check queue http://status.openstack.org/elastic-recheck/index.html | 21:16 |
| jogo | wendar discovered it ^ | 21:16 |
| wendar | Also, Hyper-V CI is reporting problems with it. | 21:17 |
| salv-orlando | thanks jogo… this is the first time I hear of this bug, and I won't be around for long. I can take a look at it in the morning, or someone else might fix it right away | 21:17 |
| jogo | salv-orlando: thank wendar not me | 21:17 |
| salv-orlando | thanks wendar then! | 21:18 |
| salv-orlando | wendar: this is the same issue as yesterday, isn't it? | 21:18 |
| markmcclain | jogo: we should ignore checks | 21:18 |
| wendar | markmcclain fixed it yesterday by resequencing the alembic merges | 21:18 |
| wendar | salv-orlando: yes, it's the same one | 21:19 |
| wendar | salv-orlando: but, that's a temporary fix, it doesn't have any affect on future merges | 21:19 |
| wendar | salv-orlando: TBH, I don't know that there is a code fix. | 21:19 |
| markmcclain | there is not a code to fix it | 21:20 |
| wendar | It's more of an alembic branch merging policy matter. | 21:20 |
| salv-orlando | wendar: that's the problem! we need a CI fix | 21:20 |
| *** peristeri has quit IRC | 21:20 | |
| *** simon-AS559 has quit IRC | 21:20 | |
| salv-orlando | I don't mean to say it's a CI bug, but we should have a check that either fails fast or snipes the patch out of the gate queue | 21:20 |
| wendar | Or, lifeless suggested doing some alembic sequencing behind the scenes automatically in CI. | 21:20 |
| salv-orlando | wendar: yes, something like that too | 21:20 |
| kevinbenton | salv-orlando: i could write a unit test that checks that | 21:20 |
| *** luqas has joined #openstack-neutron | 21:20 | |
| wendar | salv-orlando: I'm happy to work on a CI fix, but we do need to understand neutron's use of alembic better. | 21:21 |
| kevinbenton | so it would fail locally on the dev machine | 21:21 |
| markmcclain | kevinbenton: I've actually got a tox check written for it | 21:21 |
| salv-orlando | kevinbenton: the problem is when the patch goes in the gate | 21:21 |
| *** bjornar has joined #openstack-neutron | 21:21 | |
| salv-orlando | I thought even nova with sqlalchemy.migrate would suffer of this problem | 21:22 |
| kevinbenton | salv-orlando: right, but won't it get kicked out almost immediately if the failure is detected in the UT | 21:22 |
| kevinbenton | ? | 21:22 |
| salv-orlando | kevinbenton: the goal is to no cause gate resets | 21:22 |
| wendar | kevinbenton: to detect the issue locally, it would have to check if there were any pending alembic merges already in the gate | 21:22 |
| salv-orlando | whether you fail fast or note, that causes a failure in the gate queue, and that has a reset as a consequence | 21:22 |
| wendar | kevinbenton: and, you'd have a timing issue, since people could still submit simultaneously | 21:23 |
| kevinbenton | salv-orlando: right, but the reset would cause less pain | 21:23 |
| kevinbenton | it's def not a fix, but reduces the impact | 21:23 |
| kevinbenton | rather than during tempest tests 20 minutes in | 21:23 |
| kevinbenton | it would be within a few mins | 21:23 |
| salv-orlando | the way gate test run now, even if your unit test fails in 5 seconds, it will still run the rest of the checks. Is that right jogo? | 21:23 |
| sdague | salv-orlando: yes, but it will reset content behind it on first fail | 21:24 |
| kevinbenton | salv-orlando: yeah, but the children are reset right away | 21:24 |
| salv-orlando | wendar: do you know why this problem does not happen with sqlalchemy.migrate | 21:24 |
| markmcclain | ideally this check should run before running the other jobs | 21:24 |
| sdague | salv-orlando: because we merge conflict | 21:24 |
| sdague | because we have to pick a number | 21:24 |
| wendar | salv-orlando: I assume it does happen, just the case isn't as frequent | 21:24 |
| wendar | salv-orlando: (fewer db migrations) | 21:25 |
| sdague | the fail scenario is different | 21:25 |
| sdague | and much earlier in the process IIRC | 21:25 |
| jogo | salv-orlando: correct :/ | 21:25 |
| wendar | it's only a problem with multiple migrations in the gate at the same time | 21:25 |
| jogo | salv-orlando: patches welcome ! | 21:25 |
| *** shivharis has quit IRC | 21:25 | |
| *** sfox has quit IRC | 21:25 | |
| jogo | salv-orlando: although in this case the tests don't even run because neutron db sync fails | 21:26 |
| salv-orlando | so jogo, wendar: if we could generate a file with the migration history, and add it to git, then we would merge fail even with alembic I think | 21:26 |
| salv-orlando | the file will have just the list of migration uuids | 21:26 |
| sdague | salv-orlando: that's kind of an interesting idea | 21:27 |
| jogo | how do other alembic projects solve this? | 21:27 |
| sdague | jogo: which other projects are on alembic? | 21:27 |
| wendar | salv-orlando: sounds worth trying | 21:27 |
| openstackgerrit | rcurran proposed a change to openstack/neutron: ML2 Cisco Nexus MD: VM migration support https://review.openstack.org/78774 | 21:27 |
| jogo | sdague: no idea, just assumed they exist | 21:27 |
| sdague | quantum was first | 21:27 |
| sdague | ceilometer maybe? | 21:28 |
| sdague | I think glance might be part way converted | 21:28 |
| sdague | but it's not a lot | 21:28 |
| salv-orlando | yeah maybe, but markmcclain perhaps knows which ones use alembic for sure | 21:28 |
| *** thuc has quit IRC | 21:28 | |
| *** dfarrell07 has quit IRC | 21:28 | |
| *** thuc has joined #openstack-neutron | 21:29 | |
| markmcclain | I think that is the right set of users | 21:29 |
| jogo | sdague: Iwas thinking !openstack | 21:31 |
| markmcclain | seems like we want run a check just prior to starting the rest of the tests, so that is can fail quickly | 21:32 |
| sdague | jogo: so my guess is most of them don't have a flow where so much of the content is in flight | 21:32 |
| sdague | and not in patch series | 21:32 |
| jogo | sdague: agreed | 21:32 |
| sdague | markmcclain: so that idea has been off again / on again | 21:33 |
| *** thuc has quit IRC | 21:33 | |
| sdague | however, I actually kind of like salv-orlando's idea of making the alembic migration process also generate a file which is a list of all the migrations | 21:33 |
| *** thuc has joined #openstack-neutron | 21:34 | |
| sdague | because then you could use merge conflict to trigger that | 21:34 |
| *** sfox has joined #openstack-neutron | 21:34 | |
| *** thuc has quit IRC | 21:34 | |
| markmcclain | but now we're requiring the submitted to generate that file | 21:34 |
| salv-orlando | markmcclain: the issue with failing fast, as also suggested by kevinbenton, is that it will cause a gate reset anyway, unless we have a way to treat that failure differently | 21:34 |
| sdague | but you need to run some tool to generate the migration id anyway right? | 21:34 |
| sdague | how is this resolved manually when it fails? | 21:35 |
| markmcclain | sdague: yeah but it a wrapper for alembic | 21:35 |
| *** thuc has joined #openstack-neutron | 21:35 | |
| sdague | markmcclain: so just wrapper that | 21:36 |
| markmcclain | we resolve it manually by running neutron-db-manage branches command | 21:36 |
| salv-orlando | markmclain: worst case will end up putting a requirement similar to tempest.conf.sample (which I initially found a bit annoying but then got used to it) | 21:36 |
| sdague | ok, so make that build the fail | 21:36 |
| sdague | file | 21:36 |
| *** madmike has quit IRC | 21:36 | |
| openstackgerrit | Ronak Shah proposed a change to openstack/neutron: Nuage plugin was missed in floatingip_status db migration script https://review.openstack.org/78720 | 21:36 |
| *** manishg has quit IRC | 21:36 | |
| markmcclain | sdague: except that fixing merge conflicts requires manual sequencing of the ids | 21:37 |
| sdague | markmcclain: so alembic knows how to walk these things | 21:38 |
| markmcclain | yeah each file has a down revision | 21:38 |
| sdague | I can't believe it's impossible to write a generator to do that | 21:38 |
| sdague | and just write out that id sequence | 21:38 |
| *** otherwiseguy has joined #openstack-neutron | 21:38 | |
| *** julim has quit IRC | 21:39 | |
| sdague | yeh, so if they all have a down revision, then you can make a text file with the ids in order | 21:39 |
| sdague | which very clearly will merge conflict | 21:39 |
| kevinbenton | sdague: the new file might scare us developers :-) | 21:39 |
| markmcclain | I'm leery of having dev make files | 21:40 |
| *** shashank_ has quit IRC | 21:41 | |
| markmcclain | I | 21:41 |
| markmcclain | I'd rather we automate the walking | 21:41 |
| kevinbenton | could modify the script so the name of the migration script it generates is just the version number it's revising... | 21:42 |
| kevinbenton | conflict on merge then comes for free | 21:42 |
| kevinbenton | but we lose useful filenames | 21:42 |
| salv-orlando | This is perhaps unrelated, but sometimes I do this to look at the list of migrations: neutron-db-manage --config-file /etc/neutron/neutron.conf history | grep "Revision ID:" | 21:42 |
| markmcclain | except alembic prefixes the script name with it's revision id | 21:42 |
| kevinbenton | markmcclain: can't we change that behavior? | 21:43 |
| markmcclain | that would break convention | 21:43 |
| kevinbenton | markmcclain: ok | 21:44 |
| markmcclain | so sdague what step in the pipeline checks for git merge conflicts? | 21:45 |
| *** hemanthravi has joined #openstack-neutron | 21:45 | |
| markmcclain | would it not be possible to add a hook there? | 21:45 |
| sdague | markmcclain: when things enter the gate the refs all get calculated | 21:45 |
| sdague | there is not a hook there today | 21:46 |
| *** dave_tucker_zzz is now known as dave_tucker | 21:46 | |
| sdague | it's pretty deeply about git | 21:46 |
| markmcclain | k | 21:46 |
| salv-orlando | sdague: so at the moment there is no place where we could perform a check on the rebased refs before gate jobs start? | 21:47 |
| *** yamahata has quit IRC | 21:47 | |
| sdague | you can always talk to jeblair about other approaches in zuul. But the approach salv-orlando came up with you could implement in project and be done with it in a day | 21:47 |
| openstackgerrit | Ivar Lazzaro proposed a change to openstack/neutron: Embrane LBaaS Driver https://review.openstack.org/54236 | 21:47 |
| sdague | salv-orlando: correct. to the best of my understanding | 21:47 |
| *** yamahata has joined #openstack-neutron | 21:47 | |
| *** blogan has quit IRC | 21:48 | |
| openstackgerrit | Sergio Cazzolato proposed a change to openstack/neutron: Per user quotas https://review.openstack.org/78781 | 21:50 |
| openstackgerrit | Hemanth Ravi proposed a change to openstack/neutron: One Convergence Neutron Plugin l3 ext support https://review.openstack.org/77809 | 21:50 |
| wendar | sdague: doing it within one project has the downside of not benefitting others using alembic, but the upside that we could buy some time to coordinate across the other projects | 21:51 |
| markmcclain | sdague: ok… looking at log stash most of the errors are in the check queue | 21:51 |
| wendar | sdague: this might be a Summit topic | 21:51 |
| markmcclain | what if we atleast add a check | 21:51 |
| kevinbenton | earlier failures are much cheaper than later | 21:52 |
| *** itzikb has joined #openstack-neutron | 21:52 | |
| *** manishg has joined #openstack-neutron | 21:54 | |
| wendar | and in gate, the problem is solved if the problematic change can be kicked, instead of retried | 21:54 |
| *** sfox has quit IRC | 21:55 | |
| *** thuc has quit IRC | 21:56 | |
| *** thuc has joined #openstack-neutron | 21:56 | |
| *** bjornar has quit IRC | 21:58 | |
| *** sfox has joined #openstack-neutron | 21:59 | |
| markmcclain | wendar the trick is you dont know if you can kick it out | 22:01 |
| markmcclain | what if a conflict is caused by something in the pipeline that will fail? | 22:02 |
| sdague | markmcclain: so the check queue has 5x the volume of the gate | 22:02 |
| sdague | plus it's got (on average) a 2x failure rate, because tons of bad code | 22:02 |
| sdague | so if the incidence rate wasn't 5 - 10x in check vs. gate then I'd be concerned | 22:02 |
| *** manishg has quit IRC | 22:03 | |
| wendar | markmcclain: yes, we don't really have sufficient identification | 22:04 |
| oda-g | nati_ueno: ping | 22:05 |
| wendar | markmcclain: and, we do risk kicking all the pending migration changes, instead of getting one through, and kicking the rest | 22:05 |
| wendar | markmcclain: better than locking up the gate for 16 hours, but less-than-ideal | 22:06 |
| markmcclain | was this really the sole problem of locking up the gate for 16 hrs? | 22:07 |
| *** shashank_ has joined #openstack-neutron | 22:08 | |
| *** mlemay has joined #openstack-neutron | 22:09 | |
| sdague | markmcclain: there is rarely one bug that locks it up. It's the fact that we have 60+ ER bugs right now | 22:09 |
| wendar | markmcclain: yeah. we had some minor blockage with grenade yesterday, but only after the db migrations were cleared | 22:09 |
| sdague | so every little big helps | 22:10 |
| markmcclain | I pulled most of the migration errors out of the gate once the issue was raised | 22:10 |
| markmcclain | s/most/all/ | 22:10 |
| wendar | yup, and that's when the gate started catching up | 22:10 |
| wendar | jogo has some nice graphs | 22:11 |
| markmcclain | I think this is something we can tackle Atlanta | 22:11 |
| wendar | markmcclain: It's definitely worth more careful thought. | 22:11 |
| markmcclain | because we're only planning to merge 2 migrations during the exemption phase | 22:11 |
| *** uma has quit IRC | 22:11 | |
| markmcclain | wendar: I agree | 22:12 |
| *** piyush1 has left #openstack-neutron | 22:12 | |
| wendar | markmcclain: Ah, well, then we should be fine. | 22:12 |
| markmcclain | sadly alembic has few strange interfaces we'd need to get changed upstream | 22:12 |
| wendar | I mean, the "manual" fix is just coordinating in core to make sure only one migration hits the gate at a time. | 22:12 |
| *** packet has quit IRC | 22:12 | |
| wendar | That can get us through Atlanta. | 22:12 |
| jogo | jogo.github.io/gate | 22:13 |
| wendar | markmcclain: hmmm... there's probably still time to invite some alembic devs to Summit | 22:14 |
| kevinbenton | https://bitbucket.org/zzzeek/alembic/issue/167/multiple-heads-branch-resolution-support | 22:14 |
| *** samuelbercovici has quit IRC | 22:15 | |
| wendar | kevinbenton: nice, so he's thinking about the problem anyway | 22:18 |
| kevinbenton | wendar: yeah, i think it's pretty rare that there are two patches with alembic migrations that actually depend on each other. And even in those cases, the dependency would probably be explicitly set in the gerrit patch-sets | 22:20 |
| wendar | kevinbenton: aye, true dependencies are sensible as patch-sets | 22:21 |
| *** dvorkinista has quit IRC | 22:22 | |
| anteaya | one of the big problems with the gate yesterday is a zuul bug that times out a repo confirmation and considers the patch not merged | 22:24 |
| anteaya | reseting the gate | 22:24 |
| *** leseb has quit IRC | 22:25 | |
| *** sfox has quit IRC | 22:25 | |
| anteaya | this was the fix for that issue: https://review.openstack.org/#/c/78222/ | 22:25 |
| *** leseb has joined #openstack-neutron | 22:25 | |
| anteaya | nova is the only repo, that I saw, that triggered this leading to the conclusion this was due to its size | 22:26 |
| openstackgerrit | Salvatore Orlando proposed a change to openstack/python-neutronclient: CLI support for NSX gateway devices https://review.openstack.org/77164 | 22:27 |
| *** jorgem has quit IRC | 22:27 | |
| *** sfox has joined #openstack-neutron | 22:28 | |
| *** dims has quit IRC | 22:28 | |
| *** jecarey has quit IRC | 22:28 | |
| *** jorgem has joined #openstack-neutron | 22:29 | |
| *** leseb has quit IRC | 22:29 | |
| openstackgerrit | Claudiu Belu proposed a change to openstack/neutron: Fixes the Hyper-V agent individual ports metrics https://review.openstack.org/78795 | 22:36 |
| *** jecarey has joined #openstack-neutron | 22:38 | |
| *** leseb has joined #openstack-neutron | 22:39 | |
| *** morganfainberg_Z is now known as morganfainberg | 22:41 | |
| *** dims has joined #openstack-neutron | 22:41 | |
| *** vkozhukalov has joined #openstack-neutron | 22:43 | |
| jogo | markmcclain: I would like to note that this bug (migrations) will only hurt neutron in the check queue. | 22:55 |
| markmcclain | well nothing can exit the check queue to the gate with a -1 right? | 22:56 |
| markmcclain | so that will cut some of the items that move across right? | 22:56 |
| *** dvorkinista has joined #openstack-neutron | 22:56 | |
| markmcclain | long term developing a good way to fail fast and in a way that is easy to manage by the dev is must | 22:57 |
| openstackgerrit | Itzik Brown proposed a change to openstack/neutron: Migration support for Mellanox Neutron plugin https://review.openstack.org/66545 | 22:59 |
| *** dvorkinista has quit IRC | 23:02 | |
| *** mlemay has quit IRC | 23:02 | |
| openstackgerrit | Ivar Lazzaro proposed a change to openstack/neutron: Floatingip_status migration not including Embrane's plugin https://review.openstack.org/78805 | 23:03 |
| *** Qlawy has quit IRC | 23:04 | |
| *** mlemay has joined #openstack-neutron | 23:04 | |
| *** Qlawy has joined #openstack-neutron | 23:04 | |
| *** Qlawy has joined #openstack-neutron | 23:04 | |
| *** dvorkinista has joined #openstack-neutron | 23:05 | |
| *** dvorkinista has quit IRC | 23:06 | |
| *** luqas has quit IRC | 23:06 | |
| *** djoreilly has quit IRC | 23:07 | |
| nati_ueno | oda-g: pong | 23:12 |
| jogo | markmcclain: correct on both counts | 23:12 |
| oda-g | nati_ueno: Would you approve https://review.openstack.org/#/c/70060 ? | 23:13 |
| *** banix has quit IRC | 23:13 | |
| nati_ueno | oda-g: sure | 23:13 |
| *** armax has quit IRC | 23:14 | |
| oda-g | nati_ueno: I found already done, Thank you. | 23:14 |
| nati_ueno | oda-g: THank you for your fixing this | 23:14 |
| *** reaper has quit IRC | 23:15 | |
| oda-g | nati_ueno: I have three other fixes to be uploaded after 70060 get mereged. Please mind these fixes. | 23:17 |
| nati_ueno | oda-g: sure | 23:17 |
| *** mfink has joined #openstack-neutron | 23:18 | |
| *** leseb has quit IRC | 23:20 | |
| *** jobewan has quit IRC | 23:23 | |
| *** jorgem has quit IRC | 23:23 | |
| *** thedodd has quit IRC | 23:23 | |
| *** mlemay has quit IRC | 23:25 | |
| *** mlemay has joined #openstack-neutron | 23:27 | |
| *** manishg has joined #openstack-neutron | 23:28 | |
| openstackgerrit | Sourabh Patwardhan proposed a change to openstack/neutron: For Cisco Neutron plugin: Alembic migration from revision f44ab9871cd6 to 2eeaf963a447 fails because the floatingips table doesn't exist. https://review.openstack.org/78809 | 23:31 |
| *** wusWWW has joined #openstack-neutron | 23:35 | |
| *** shashank_ has quit IRC | 23:35 | |
| openstackgerrit | Sean M. Collins proposed a change to openstack/neutron: Create new IPv6 attributes for Subnets https://review.openstack.org/52983 | 23:36 |
| itzikb | Is there a need for a bug for the recent change ( " For Cisco Neutron plugin: Alembic migration from revision..")? | 23:36 |
| *** yfujioka has joined #openstack-neutron | 23:37 | |
| itzikb | I see there is .. | 23:38 |
| openstackgerrit | Sean M. Collins proposed a change to openstack/neutron: Create new IPv6 attributes for Subnets https://review.openstack.org/52983 | 23:45 |
| itzikb | If a plugin is in L3 capable in folsom migration file , should it be added to 2c4af419145b_l3_support.py ? | 23:48 |
| *** gdubreui has joined #openstack-neutron | 23:54 | |
| *** alexpilotti has quit IRC | 23:54 | |
| *** jroovers|afk has quit IRC | 23:55 | |
| *** jroovers has joined #openstack-neutron | 23:56 | |
| *** jp_at_hp has joined #openstack-neutron | 23:56 | |
| *** changbl has quit IRC | 23:56 | |
| *** shashank_ has joined #openstack-neutron | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!