| *** neiljerram_bb has joined #openstack-neutron | 00:01 | |
| *** achanda_ has joined #openstack-neutron | 00:02 | |
| *** aranjan_ has quit IRC | 00:02 | |
| *** achanda has quit IRC | 00:05 | |
| *** aranjan has joined #openstack-neutron | 00:05 | |
| *** shashank_hegde has joined #openstack-neutron | 00:06 | |
| *** achanda_ has quit IRC | 00:07 | |
| DWfuturetec | ikeyman, i just setup an openstack environment a few weeks okay | 00:07 |
|---|---|---|
| ikeyman | ok | 00:08 |
| DWfuturetec | as far as i know, the documentation from the site you refer to is outdated .. e.g. the keystone cli tool and some of the config parameters shouldn’t be used anymore | 00:08 |
| DWfuturetec | I can’t give you the exact reason why your neutron agent-list fails, but I would s | 00:08 |
| openstackgerrit | Ryan Moats proposed openstack/neutron: l3_db, l3_dvr_db: separate _process_floating_{ips, interfaces}() https://review.openstack.org/124696 | 00:09 |
| *** nlahouti has joined #openstack-neutron | 00:09 | |
| DWfuturetec | I would suggest you to have a look at this guide: http://docs.openstack.org/kilo/install-guide/install/apt/content/ | 00:09 |
| ikeyman | i have fixed that! | 00:09 |
| ikeyman | thanks | 00:09 |
| DWfuturetec | ikeyman, okay great! | 00:09 |
| *** ex-parrot has quit IRC | 00:09 | |
| *** padkrish has joined #openstack-neutron | 00:09 | |
| ikeyman | what i need to do now? | 00:10 |
| ikeyman | i need to create the “outgoing network” | 00:10 |
| *** nlahouti has quit IRC | 00:10 | |
| *** nlahouti has joined #openstack-neutron | 00:10 | |
| DWfuturetec | ikeyman, if you mean with „outgoing network“ the external network (with e.g. connectivity to the public internet), you could have a look at this part: http://docs.openstack.org/kilo/install-guide/install/apt/content/neutron_initial-external-network.html | 00:11 |
| *** regXboi has quit IRC | 00:11 | |
| *** lilynn2 has joined #openstack-neutron | 00:13 | |
| *** lilynn has quit IRC | 00:13 | |
| *** rawl has joined #openstack-neutron | 00:14 | |
| *** tiswanso has joined #openstack-neutron | 00:14 | |
| ikeyman | i get: Unrecognized attribute(s) 'router:External' | 00:16 |
| *** shashank_hegde has quit IRC | 00:16 | |
| *** gangil has quit IRC | 00:16 | |
| ikeyman | I change the “External” to “external” and now works! | 00:17 |
| *** banix has quit IRC | 00:19 | |
| openstackgerrit | Miguel Lavalle proposed openstack/neutron: Add dns_label processing for Ports https://review.openstack.org/200952 | 00:20 |
| *** minwang2 has quit IRC | 00:20 | |
| *** minwang2 has joined #openstack-neutron | 00:20 | |
| *** achippa_ has joined #openstack-neutron | 00:22 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Make DHCP agent scheduler physical_network aware https://review.openstack.org/205631 | 00:22 |
| Sam-I-Am | amuller: ping | 00:22 |
| *** shadower has quit IRC | 00:23 | |
| amuller | Sam-I-Am: heya | 00:23 |
| Sam-I-Am | amuller: someone decided it was a great idea to merge my un-finished l3ha patch | 00:23 |
| *** shadower has joined #openstack-neutron | 00:23 | |
| Sam-I-Am | amuller: https://review.openstack.org/#/c/211785/ | 00:23 |
| amuller | hah | 00:23 |
| Sam-I-Am | my intention - fix the ovs part, let you review it | 00:24 |
| Sam-I-Am | then go fix the lb part | 00:24 |
| Sam-I-Am | aaaaaand... fai | 00:24 |
| Sam-I-Am | l | 00:24 |
| Sam-I-Am | sort of like my typing | 00:24 |
| amuller | Sam-I-Am: ok, please add me as a reviewer to the follow up then | 00:24 |
| amuller | Sam-I-Am: sorry I didn't review it for so long | 00:24 |
| amuller | lots of crap going on | 00:24 |
| Sam-I-Am | yeah | 00:24 |
| mlavalle | armax, kevinbenton: implemented validator as indicated by Kevin. Just pushed next revision. I also a couple of unit tests, to test this new validator | 00:24 |
| Sam-I-Am | can you take a look at the patch that went in? | 00:25 |
| amuller | Sam-I-Am: or did you want me to just review what was merged? | 00:25 |
| amuller | Sam-I-Am: oh yeah ok | 00:25 |
| *** tiswanso has quit IRC | 00:25 | |
| mlavalle | kevinbenton: I tweaked the validator a little bit, but it is essentially what your recommended | 00:25 |
| *** minwang2 has quit IRC | 00:25 | |
| amuller | Sam-I-Am: ok I'll leave it as the sole open tab for tomorrow morning :) | 00:25 |
| Sam-I-Am | amuller: i think i hit all your points, no contention (except the ml2 thing) | 00:25 |
| Sam-I-Am | amuller: thx | 00:25 |
| amuller | Sam-I-Am: off to dinner! toodles | 00:26 |
| Sam-I-Am | amuller: enjoy, thx | 00:26 |
| *** gangil has joined #openstack-neutron | 00:26 | |
| *** amuller has quit IRC | 00:26 | |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port https://review.openstack.org/194441 | 00:27 |
| *** claudiub has quit IRC | 00:28 | |
| *** shashank_hegde has joined #openstack-neutron | 00:33 | |
| *** sridhar_ram has quit IRC | 00:34 | |
| *** salv-orl_ has quit IRC | 00:34 | |
| *** abhiraut has quit IRC | 00:35 | |
| *** abhiraut has joined #openstack-neutron | 00:35 | |
| *** Piet has joined #openstack-neutron | 00:36 | |
| *** shoutm has quit IRC | 00:36 | |
| *** shoutm has joined #openstack-neutron | 00:37 | |
| *** mlavalle has quit IRC | 00:38 | |
| *** shivrao has quit IRC | 00:38 | |
| *** _cjones_ has quit IRC | 00:39 | |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent https://review.openstack.org/194446 | 00:42 |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Cleanup the fip agent gateway port delete routines https://review.openstack.org/194455 | 00:44 |
| *** Swami has quit IRC | 00:46 | |
| *** dims has joined #openstack-neutron | 00:46 | |
| *** njohnston is now known as nate_gone | 00:46 | |
| *** DWfuturetec has quit IRC | 00:47 | |
| *** dims_ has quit IRC | 00:47 | |
| *** gangil has quit IRC | 00:49 | |
| *** baoli has joined #openstack-neutron | 00:49 | |
| *** shivrao has joined #openstack-neutron | 00:50 | |
| *** smccully has joined #openstack-neutron | 00:51 | |
| *** tiswanso has joined #openstack-neutron | 00:52 | |
| *** gildub has joined #openstack-neutron | 00:53 | |
| *** shivrao has quit IRC | 00:53 | |
| *** achippa_ has quit IRC | 00:54 | |
| *** baoli has quit IRC | 00:54 | |
| *** tiswanso has quit IRC | 00:54 | |
| *** achippa has joined #openstack-neutron | 00:54 | |
| *** tiswanso has joined #openstack-neutron | 00:54 | |
| *** gildub has quit IRC | 00:55 | |
| *** Piet has quit IRC | 00:55 | |
| *** nlahouti has quit IRC | 00:55 | |
| openstackgerrit | Merged openstack/neutron: Redundant tests removed from ovs-lib unit tests: https://review.openstack.org/208860 | 00:55 |
| *** dims has quit IRC | 00:56 | |
| *** dims has joined #openstack-neutron | 00:56 | |
| *** piyanai has joined #openstack-neutron | 00:57 | |
| *** achippa has quit IRC | 00:58 | |
| *** dlundquist1 has quit IRC | 01:00 | |
| openstackgerrit | Pengtao Huang proposed openstack/neutron: Check ICMP port code in range [0,255] https://review.openstack.org/212220 | 01:02 |
| *** thorst has quit IRC | 01:02 | |
| *** piyanai has quit IRC | 01:02 | |
| *** liudong has joined #openstack-neutron | 01:05 | |
| *** jckasper has joined #openstack-neutron | 01:06 | |
| *** madhu_ak has quit IRC | 01:07 | |
| *** madhu_ak has joined #openstack-neutron | 01:07 | |
| *** aranjan has quit IRC | 01:07 | |
| *** mickeys has quit IRC | 01:08 | |
| *** madhu_ak_ has joined #openstack-neutron | 01:08 | |
| *** zhenguo has joined #openstack-neutron | 01:09 | |
| *** SumitNaiksatam has joined #openstack-neutron | 01:09 | |
| *** SumitNaiksatam has left #openstack-neutron | 01:09 | |
| *** sthillma has quit IRC | 01:11 | |
| *** banix has joined #openstack-neutron | 01:11 | |
| *** madhu_ak has quit IRC | 01:12 | |
| *** y_sawai has joined #openstack-neutron | 01:12 | |
| openstackgerrit | Pengtao Huang proposed openstack/neutron: Check ICMP port code in range [0,255] https://review.openstack.org/212220 | 01:14 |
| *** saksham has quit IRC | 01:15 | |
| *** ikeyman has quit IRC | 01:16 | |
| *** s3wong has quit IRC | 01:16 | |
| *** y_sawai has quit IRC | 01:17 | |
| *** SourabhP has quit IRC | 01:17 | |
| *** jwy has quit IRC | 01:17 | |
| *** SourabhP has joined #openstack-neutron | 01:17 | |
| *** SourabhP has quit IRC | 01:17 | |
| *** yamamoto has joined #openstack-neutron | 01:18 | |
| *** stanzgy has joined #openstack-neutron | 01:18 | |
| *** madhu_ak_ has quit IRC | 01:19 | |
| *** madhu_ak_ has joined #openstack-neutron | 01:19 | |
| *** padkrish has quit IRC | 01:20 | |
| *** moshele has joined #openstack-neutron | 01:22 | |
| openstackgerrit | Merged openstack/neutron: Final decomposition of ML2 Nexus Driver https://review.openstack.org/213266 | 01:23 |
| *** gildub has joined #openstack-neutron | 01:23 | |
| *** achippa has joined #openstack-neutron | 01:23 | |
| *** erkules_ has joined #openstack-neutron | 01:23 | |
| *** y_sawai has joined #openstack-neutron | 01:23 | |
| *** julim has joined #openstack-neutron | 01:24 | |
| *** erkules has quit IRC | 01:26 | |
| *** liangbo has joined #openstack-neutron | 01:26 | |
| openstackgerrit | Merged openstack/neutron: Stop using quota reservations on base controller https://review.openstack.org/214282 | 01:26 |
| openstackgerrit | Merged openstack/neutron-fwaas: Make tests/unit/extensions discoverable https://review.openstack.org/213927 | 01:27 |
| openstackgerrit | Merged openstack/neutron-vpnaas: Rename a test method in test_ipsec.py https://review.openstack.org/210989 | 01:28 |
| *** y_sawai has quit IRC | 01:28 | |
| *** yamamoto has quit IRC | 01:30 | |
| openstackgerrit | Pengtao Huang proposed openstack/neutron: Check ICMP codes in range [0,255] https://review.openstack.org/212220 | 01:30 |
| *** regXboi has joined #openstack-neutron | 01:30 | |
| *** liangbo has quit IRC | 01:31 | |
| *** banix has quit IRC | 01:32 | |
| *** madhu_ak has joined #openstack-neutron | 01:34 | |
| *** dave-mccowan has quit IRC | 01:36 | |
| *** madhu_ak_ has quit IRC | 01:37 | |
| *** cdl has joined #openstack-neutron | 01:37 | |
| *** cdl has left #openstack-neutron | 01:38 | |
| *** jwy has joined #openstack-neutron | 01:38 | |
| *** madhu_ak_ has joined #openstack-neutron | 01:39 | |
| *** lyanchih has joined #openstack-neutron | 01:41 | |
| *** madhu_ak has quit IRC | 01:42 | |
| *** garyk has quit IRC | 01:42 | |
| *** madhu_ak__ has joined #openstack-neutron | 01:43 | |
| *** ivar-lazzaro has joined #openstack-neutron | 01:44 | |
| *** yalie has joined #openstack-neutron | 01:44 | |
| *** madhu_ak_ has quit IRC | 01:44 | |
| *** madhu_ak has joined #openstack-neutron | 01:44 | |
| *** madhu_ak__ has quit IRC | 01:45 | |
| *** lazy_prince has joined #openstack-neutron | 01:48 | |
| *** britthouser has joined #openstack-neutron | 01:48 | |
| *** britthou_ has joined #openstack-neutron | 01:50 | |
| *** vivek-ebay has quit IRC | 01:52 | |
| openstackgerrit | Henry Gessau proposed openstack/neutron: Final decomposition of ML2 Cisco UCSM driver https://review.openstack.org/213668 | 01:52 |
| *** madhu_ak has quit IRC | 01:53 | |
| *** achippa has quit IRC | 01:53 | |
| *** madhu_ak has joined #openstack-neutron | 01:53 | |
| *** britthouser has quit IRC | 01:53 | |
| openstackgerrit | Henry Gessau proposed openstack/neutron: Rename args for alembic 0.8.0 https://review.openstack.org/212853 | 01:55 |
| *** y_sawai has joined #openstack-neutron | 01:56 | |
| *** liangbo has joined #openstack-neutron | 01:58 | |
| *** vivek-ebay has joined #openstack-neutron | 02:00 | |
| openstackgerrit | Merged openstack/neutron: Update sub projects git urls https://review.openstack.org/214364 | 02:01 |
| *** madhu_ak_ has joined #openstack-neutron | 02:01 | |
| *** y_sawai has quit IRC | 02:02 | |
| yalie | bug list in xi'an hackthon | 02:02 |
| yalie | https://etherpad.openstack.org/p/neutron-xian-hackathon | 02:02 |
| *** madhu_ak has quit IRC | 02:04 | |
| *** Piet has joined #openstack-neutron | 02:04 | |
| *** moshele has quit IRC | 02:06 | |
| regXboi | who has the care and feeding of the rally non voting job? | 02:06 |
| regXboi | it looks like it is going off to insanity | 02:07 |
| *** gongysh has joined #openstack-neutron | 02:07 | |
| *** redbeard has joined #openstack-neutron | 02:07 | |
| *** madhu_ak__ has joined #openstack-neutron | 02:08 | |
| *** y_sawai has joined #openstack-neutron | 02:08 | |
| yalie | https://etherpad.openstack.org/p/neutron-xian-hackathon | 02:08 |
| *** yuntongjin has joined #openstack-neutron | 02:09 | |
| HenryG | regXboi: It might settle now that https://review.openstack.org/214282 merged | 02:09 |
| *** changzhi has joined #openstack-neutron | 02:09 | |
| * regXboi looks | 02:09 | |
| *** madhu_ak_ has quit IRC | 02:11 | |
| abhiraut | HenryG: it still is bombing | 02:11 |
| *** madhu_ak__ has quit IRC | 02:12 | |
| HenryG | Then it's probably due to the enabling of pymysql + API workers | 02:12 |
| abhiraut | salv_orlando got rid of it to not complicate further | 02:12 |
| abhiraut | he thinks that could be the problem | 02:13 |
| abhiraut | creation of default security groups is hogging the logs | 02:13 |
| abhiraut | it could be that pymysql + API workers exposed a bug in neutron | 02:13 |
| *** yamahata has quit IRC | 02:14 | |
| *** shaohe_feng has joined #openstack-neutron | 02:14 | |
| *** piyanai has joined #openstack-neutron | 02:14 | |
| *** baojg has joined #openstack-neutron | 02:15 | |
| *** gongysh has quit IRC | 02:16 | |
| *** yamamoto has joined #openstack-neutron | 02:16 | |
| *** vivek-ebay has quit IRC | 02:17 | |
| HenryG | abhiraut: I think you may be right. IIRC it was observed in rally the last time we enabled pymysql + API workers too. | 02:18 |
| HenryG | I wonder if we can get rally to run its neutron job with API workers = 0 for a while, to see if that changes things | 02:19 |
| HenryG | We should isolate pymysql vs. API workers | 02:20 |
| abhiraut | yeah.. that sounds like a good start | 02:20 |
| HenryG | abhiraut: I have to quit for the day, but maybe you can find someone in the rally team to help do that? | 02:20 |
| boris-42 | HenryG: yep that happens=) | 02:21 |
| boris-42 | HenryG: I remember that* | 02:21 |
| boris-42 | HenryG: abhiraut I am from Rally team | 02:21 |
| *** lazy_prince has quit IRC | 02:21 | |
| abhiraut | oh great :) | 02:21 |
| abhiraut | boris-42: can we do something like that | 02:21 |
| boris-42 | abhiraut: so what we need to do? | 02:22 |
| abhiraut | boris-42: set the api_workers = 0 in neutron.conf as suggested by HenryG | 02:23 |
| *** yuntongjin1 has joined #openstack-neutron | 02:24 | |
| *** yuntongjin has quit IRC | 02:24 | |
| HenryG | abhiraut: boris-42: here are the two settings we want to isolate, https://review.openstack.org/#/c/199672/4/devstack-vm-gate.sh | 02:24 |
| boris-42 | abhiraut: you can do this | 02:25 |
| boris-42 | abhiraut: with depends-on feature | 02:25 |
| HenryG | abhiraut: boris-42: sorry I have to leave now, it is night here. I will check back tomorrow | 02:25 |
| boris-42 | HenryG: sure see you | 02:25 |
| *** annegentle has quit IRC | 02:25 | |
| boris-42 | abhiraut: so you should make a patch to devstack-gate | 02:26 |
| boris-42 | abhiraut: after that you should make dummy patch to neutron that depends on it | 02:26 |
| abhiraut | ok got it | 02:26 |
| *** annegentle has joined #openstack-neutron | 02:26 | |
| abhiraut | thanks for your help! | 02:27 |
| *** gildub has quit IRC | 02:27 | |
| *** shashank_hegde has quit IRC | 02:28 | |
| *** gongysh has joined #openstack-neutron | 02:29 | |
| openstackgerrit | Ivar Lazzaro proposed openstack/neutron: handle gw_info outside of the db transaction on router creation https://review.openstack.org/214424 | 02:32 |
| *** annegentle has quit IRC | 02:32 | |
| *** achanda has joined #openstack-neutron | 02:32 | |
| boris-42 | abhiraut: you are welcome | 02:33 |
| *** baojg has quit IRC | 02:35 | |
| abhiraut | boris-42 pushed up a patch on devstack-gate | 02:35 |
| abhiraut | https://review.openstack.org/#/c/214426/ | 02:36 |
| openstackgerrit | Ryan Moats proposed openstack/neutron: l3_db, l3_dvr_db: separate _process_floating_{ips, interfaces}() https://review.openstack.org/124696 | 02:36 |
| *** dane_leblanc_ has quit IRC | 02:36 | |
| *** minwang2 has joined #openstack-neutron | 02:36 | |
| *** regXboi has quit IRC | 02:41 | |
| *** vivek-ebay has joined #openstack-neutron | 02:42 | |
| *** yuntongjin1 has quit IRC | 02:42 | |
| *** yuntongjin1 has joined #openstack-neutron | 02:43 | |
| *** gongysh has quit IRC | 02:44 | |
| openstackgerrit | Abhishek Raut proposed openstack/neutron: [Do not merge]: Dummy patch to test rally jobs https://review.openstack.org/214428 | 02:44 |
| *** dane_leblanc has joined #openstack-neutron | 02:44 | |
| *** shaohe_feng1 has joined #openstack-neutron | 02:47 | |
| *** gildub has joined #openstack-neutron | 02:47 | |
| *** piyanai has quit IRC | 02:47 | |
| *** baojg has joined #openstack-neutron | 02:48 | |
| *** shaohe_feng has quit IRC | 02:48 | |
| jlvillal | HenryG, The patch that was supposed to fix Ironic did merge. Rechecks I've done so far are still failing. Not sure if there is a delay for it to hit the gate or if the patch didn't fix the issue. | 02:49 |
| jlvillal | https://review.openstack.org/#/c/214282/ is the patch. | 02:49 |
| jlvillal | Just an FYI | 02:49 |
| *** salv-orlando has joined #openstack-neutron | 02:49 | |
| *** shwetaap has joined #openstack-neutron | 02:51 | |
| *** yuntongjin has joined #openstack-neutron | 02:51 | |
| *** hakimo_ has joined #openstack-neutron | 02:52 | |
| *** abhiraut has quit IRC | 02:52 | |
| *** yuntongjin1 has quit IRC | 02:54 | |
| *** hakimo has quit IRC | 02:54 | |
| *** piyanai has joined #openstack-neutron | 02:55 | |
| *** dims has quit IRC | 02:55 | |
| *** changzhi has quit IRC | 02:56 | |
| *** gongysh has joined #openstack-neutron | 02:56 | |
| *** cdl has joined #openstack-neutron | 02:58 | |
| *** tiny-hands has quit IRC | 02:59 | |
| *** yuntongjin2 has joined #openstack-neutron | 02:59 | |
| openstackgerrit | willy lin proposed openstack/neutron: Remove VIF_TYPES constant https://review.openstack.org/214430 | 03:00 |
| *** dave-mccowan has joined #openstack-neutron | 03:00 | |
| *** y_sawai has quit IRC | 03:01 | |
| *** yuntongjin has quit IRC | 03:01 | |
| *** gberginc has joined #openstack-neutron | 03:01 | |
| *** salv-orlando has quit IRC | 03:05 | |
| *** shaohe_feng1 has quit IRC | 03:05 | |
| *** yuntongjin has joined #openstack-neutron | 03:08 | |
| *** liangbo has quit IRC | 03:09 | |
| *** agireud has joined #openstack-neutron | 03:09 | |
| openstackgerrit | Marga Millet proposed openstack/neutron: Support dhcp metadata service for all networks https://review.openstack.org/211963 | 03:10 |
| *** yuntongjin2 has quit IRC | 03:11 | |
| *** redbeard has quit IRC | 03:15 | |
| *** tiny-hands has joined #openstack-neutron | 03:16 | |
| *** rodrigods has quit IRC | 03:16 | |
| *** jwy has quit IRC | 03:17 | |
| *** piyanai has quit IRC | 03:17 | |
| *** vivek-ebay has quit IRC | 03:19 | |
| *** rodrigods has joined #openstack-neutron | 03:19 | |
| *** redbeard has joined #openstack-neutron | 03:21 | |
| *** tflynn_ has joined #openstack-neutron | 03:23 | |
| *** tiswanso has quit IRC | 03:25 | |
| *** glavni_ninja has quit IRC | 03:27 | |
| *** raginbajin has quit IRC | 03:27 | |
| *** vivek-ebay has joined #openstack-neutron | 03:27 | |
| *** vivek-ebay has quit IRC | 03:28 | |
| *** boltR has quit IRC | 03:28 | |
| *** glavni_ninja has joined #openstack-neutron | 03:28 | |
| *** boltR has joined #openstack-neutron | 03:29 | |
| *** raginbajin has joined #openstack-neutron | 03:29 | |
| *** logan2 has quit IRC | 03:31 | |
| *** shaohe_feng has joined #openstack-neutron | 03:32 | |
| *** dlundquist has joined #openstack-neutron | 03:33 | |
| *** agireud has quit IRC | 03:33 | |
| *** yuntongjin has quit IRC | 03:34 | |
| *** kiran-r has joined #openstack-neutron | 03:36 | |
| *** kiran-r has quit IRC | 03:36 | |
| *** yamamoto has quit IRC | 03:37 | |
| *** agireud has joined #openstack-neutron | 03:38 | |
| *** vivek-ebay has joined #openstack-neutron | 03:40 | |
| *** yamahata has joined #openstack-neutron | 03:40 | |
| *** shaohe_feng1 has joined #openstack-neutron | 03:40 | |
| *** shashank_hegde has joined #openstack-neutron | 03:41 | |
| *** yamamoto has joined #openstack-neutron | 03:41 | |
| *** shaohe_feng has quit IRC | 03:41 | |
| *** ajmiller has quit IRC | 03:43 | |
| *** dane_leblanc has quit IRC | 03:43 | |
| *** shoutm has quit IRC | 03:45 | |
| *** shaohe_feng has joined #openstack-neutron | 03:46 | |
| *** vivek-ebay has quit IRC | 03:46 | |
| *** haigang has joined #openstack-neutron | 03:47 | |
| *** shaohe_feng1 has quit IRC | 03:48 | |
| *** rawl has quit IRC | 03:48 | |
| *** diga has joined #openstack-neutron | 03:49 | |
| *** vivek-ebay has joined #openstack-neutron | 03:53 | |
| *** redbeard has quit IRC | 03:55 | |
| *** shoutm has joined #openstack-neutron | 03:56 | |
| *** mkolesni_ has joined #openstack-neutron | 03:57 | |
| *** redbeard has joined #openstack-neutron | 03:57 | |
| *** redbeard has quit IRC | 03:58 | |
| *** redbeard has joined #openstack-neutron | 03:58 | |
| *** julim has quit IRC | 03:59 | |
| *** yalie has quit IRC | 04:01 | |
| *** gangil has joined #openstack-neutron | 04:04 | |
| *** salv-orlando has joined #openstack-neutron | 04:04 | |
| *** apuimedo has joined #openstack-neutron | 04:05 | |
| *** dave-mccowan has quit IRC | 04:07 | |
| *** shwetaap has quit IRC | 04:10 | |
| *** raginbajin has quit IRC | 04:10 | |
| *** ivar-lazzaro has quit IRC | 04:11 | |
| *** Ramanjaneya has joined #openstack-neutron | 04:18 | |
| *** lyanchih has quit IRC | 04:18 | |
| *** vivek-ebay has quit IRC | 04:19 | |
| *** salv-orlando has quit IRC | 04:21 | |
| *** moshele has joined #openstack-neutron | 04:22 | |
| *** zhhuabj has quit IRC | 04:24 | |
| *** iwamoto has joined #openstack-neutron | 04:25 | |
| *** diga has quit IRC | 04:26 | |
| *** hoangcx has joined #openstack-neutron | 04:28 | |
| *** raginbajin has joined #openstack-neutron | 04:28 | |
| *** sdake has quit IRC | 04:29 | |
| *** shaohe_feng has quit IRC | 04:32 | |
| *** yamamoto has quit IRC | 04:33 | |
| *** minwang2 has quit IRC | 04:33 | |
| *** diga has joined #openstack-neutron | 04:35 | |
| *** mohankumar has joined #openstack-neutron | 04:36 | |
| *** y_sawai has joined #openstack-neutron | 04:36 | |
| *** zhhuabj has joined #openstack-neutron | 04:37 | |
| *** ivar-lazzaro has joined #openstack-neutron | 04:39 | |
| *** yamamoto has joined #openstack-neutron | 04:41 | |
| *** vivek-ebay has joined #openstack-neutron | 04:42 | |
| *** y_sawai has quit IRC | 04:42 | |
| *** shwetaap has joined #openstack-neutron | 04:45 | |
| *** coolsvap|away is now known as coolsvap | 04:46 | |
| *** y_sawai has joined #openstack-neutron | 04:48 | |
| *** agireud has quit IRC | 04:48 | |
| *** VikasC has joined #openstack-neutron | 04:53 | |
| *** tflynn_ has quit IRC | 04:55 | |
| *** agireud has joined #openstack-neutron | 04:55 | |
| *** irenab has quit IRC | 04:58 | |
| openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Fix tenant access to qos policies https://review.openstack.org/214030 | 04:58 |
| *** baojg has quit IRC | 04:59 | |
| *** mohankumar has quit IRC | 04:59 | |
| *** tflynn has joined #openstack-neutron | 04:59 | |
| ajo | armax, dougwig ^ (sorry, I saw it failed on the gate and I couldn't resist to fix 2 commit message typos) :/ I was unsure if gerrit-editing the commit message cleared the votes, and indeed, it did | 05:02 |
| *** shwetaap has quit IRC | 05:06 | |
| *** carl_baldwin has joined #openstack-neutron | 05:07 | |
| *** coolsvap is now known as coolsvap|away | 05:08 | |
| *** carl_baldwin has quit IRC | 05:08 | |
| *** vivek-ebay has quit IRC | 05:09 | |
| *** gongysh has quit IRC | 05:10 | |
| *** moshele has quit IRC | 05:11 | |
| *** y_sawai has quit IRC | 05:11 | |
| *** liangbo has joined #openstack-neutron | 05:12 | |
| *** gberginc has quit IRC | 05:13 | |
| *** redbeard is now known as Guest95303 | 05:14 | |
| *** JoshNang has quit IRC | 05:17 | |
| *** boltR has quit IRC | 05:17 | |
| *** ivar-lazzaro has quit IRC | 05:18 | |
| *** ivar-lazzaro has joined #openstack-neutron | 05:18 | |
| *** y_sawai has joined #openstack-neutron | 05:19 | |
| *** boris-42 has quit IRC | 05:20 | |
| *** y_sawai has quit IRC | 05:20 | |
| *** mickeys has joined #openstack-neutron | 05:23 | |
| *** shausy has joined #openstack-neutron | 05:23 | |
| *** abregman has joined #openstack-neutron | 05:23 | |
| *** coolsvap|away is now known as coolsvap | 05:23 | |
| *** JoshNang has joined #openstack-neutron | 05:25 | |
| *** yalie has joined #openstack-neutron | 05:25 | |
| *** liangbo has quit IRC | 05:27 | |
| *** yalie has quit IRC | 05:27 | |
| *** tflynn has quit IRC | 05:28 | |
| *** salv-orlando has joined #openstack-neutron | 05:28 | |
| *** gongysh has joined #openstack-neutron | 05:29 | |
| *** boltR has joined #openstack-neutron | 05:30 | |
| *** irenab has joined #openstack-neutron | 05:30 | |
| *** apuimedo has quit IRC | 05:31 | |
| *** shausy has quit IRC | 05:36 | |
| *** shausy has joined #openstack-neutron | 05:36 | |
| *** sudipto has joined #openstack-neutron | 05:38 | |
| *** gampel has joined #openstack-neutron | 05:38 | |
| *** trinaths has joined #openstack-neutron | 05:39 | |
| *** ivar-lazzaro has quit IRC | 05:41 | |
| *** sudipto has quit IRC | 05:43 | |
| *** emagana has joined #openstack-neutron | 05:45 | |
| *** amotoki has joined #openstack-neutron | 05:50 | |
| *** jprovazn has joined #openstack-neutron | 05:53 | |
| ajo | ping emagana :) | 05:53 |
| emagana | ajo: Que pasa Tio? | 05:54 |
| *** ihrachyshka has joined #openstack-neutron | 05:54 | |
| ajo | eh :D :-) muy buenas ;) | 05:54 |
| *** gberginc has joined #openstack-neutron | 05:54 | |
| emagana | ajo: catalan o castellano? | 05:54 |
| ajo | emagana: what's the way to document the qos api properly? api-site/api-ref/src/wadls/netconn-api/src/ | 05:54 |
| ajo | emagana: castellano :) | 05:55 |
| *** mohankumar has joined #openstack-neutron | 05:55 | |
| emagana | ajo: ohhhhhh ok... english! | 05:55 |
| emagana | api-ref is the bext way | 05:55 |
| *** claudiub has joined #openstack-neutron | 05:55 | |
| emagana | that gets linked to other docs | 05:55 |
| ajo | emagana, but it's that project? api-site? | 05:56 |
| ajo | so I'm doomed to write XML... :'( | 05:56 |
| ajo | looks painful after getting used to rst :) | 05:56 |
| emagana | There is the api guide | 05:56 |
| emagana | http://developer.openstack.org/api-ref-networking-v2.html | 05:57 |
| *** moshele has joined #openstack-neutron | 05:57 | |
| *** baojg has joined #openstack-neutron | 05:58 | |
| *** lukasa has joined #openstack-neutron | 05:59 | |
| *** Fdaisuke has joined #openstack-neutron | 06:00 | |
| *** Fdaisuke has left #openstack-neutron | 06:01 | |
| *** Fdaisuke has joined #openstack-neutron | 06:01 | |
| *** lyanchih has joined #openstack-neutron | 06:02 | |
| *** sudipto has joined #openstack-neutron | 06:02 | |
| *** Fdaisuke has left #openstack-neutron | 06:03 | |
| *** SridharG has joined #openstack-neutron | 06:03 | |
| *** Fdaisuke has joined #openstack-neutron | 06:04 | |
| *** agireud has quit IRC | 06:04 | |
| *** gampel has quit IRC | 06:04 | |
| *** lpetrut has joined #openstack-neutron | 06:05 | |
| *** armax has quit IRC | 06:05 | |
| *** afazkas has joined #openstack-neutron | 06:06 | |
| openstackgerrit | Moshe Levi proposed openstack/neutron: SR-IOV: deprecate agent_required option https://review.openstack.org/214324 | 06:07 |
| *** coolsvap is now known as coolsvap|away | 06:10 | |
| *** Murali has joined #openstack-neutron | 06:10 | |
| *** ihrachyshka has quit IRC | 06:13 | |
| *** yalie has joined #openstack-neutron | 06:13 | |
| *** mickeys has quit IRC | 06:16 | |
| *** shashank_hegde has quit IRC | 06:17 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex https://review.openstack.org/214470 | 06:19 |
| *** miyagishi_t has joined #openstack-neutron | 06:19 | |
| *** britthou_ has quit IRC | 06:21 | |
| *** britthouser has joined #openstack-neutron | 06:22 | |
| *** yalie1 has joined #openstack-neutron | 06:24 | |
| *** shashank_hegde has joined #openstack-neutron | 06:25 | |
| *** yalie has quit IRC | 06:25 | |
| *** y_sawai has joined #openstack-neutron | 06:27 | |
| *** dlundquist has left #openstack-neutron | 06:29 | |
| *** sridhar_ram has joined #openstack-neutron | 06:31 | |
| *** Qlawy has joined #openstack-neutron | 06:31 | |
| *** ddepaoli has joined #openstack-neutron | 06:32 | |
| openstackgerrit | Kenji Yasui proposed openstack/neutron-fwaas: Add REJECT rule on Neutron FWaaS https://review.openstack.org/148416 | 06:32 |
| *** achanda has quit IRC | 06:34 | |
| *** klindgren has joined #openstack-neutron | 06:35 | |
| *** sridhar_ram has quit IRC | 06:35 | |
| *** lazy_prince has joined #openstack-neutron | 06:36 | |
| *** neiljerram_bb has quit IRC | 06:36 | |
| *** SourabhP has joined #openstack-neutron | 06:37 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: l2pop: check port mac in pre-commit to stop change https://review.openstack.org/214474 | 06:38 |
| *** scheuran has joined #openstack-neutron | 06:40 | |
| openstackgerrit | Pengtao Huang proposed openstack/neutron: ignore the same variable of service_plugins https://review.openstack.org/212597 | 06:40 |
| *** SourabhP_ has joined #openstack-neutron | 06:40 | |
| *** salv-orlando has quit IRC | 06:40 | |
| irenab | ajo: hi | 06:40 |
| openstackgerrit | Pengtao Huang proposed openstack/neutron: ignore the same variable of service_plugins https://review.openstack.org/212597 | 06:41 |
| *** aranjan has joined #openstack-neutron | 06:41 | |
| *** devvesa has joined #openstack-neutron | 06:42 | |
| ajo | hi irenab , good morning :) | 06:42 |
| *** SourabhP has quit IRC | 06:42 | |
| *** SourabhP_ is now known as SourabhP | 06:42 | |
| *** numan has joined #openstack-neutron | 06:42 | |
| irenab | ajo: how are you? | 06:43 |
| ajo | irenab: good, good :-) happier now that I resolved the policies for qos, and jumping around to see if I can unveil any other bug :) | 06:44 |
| irenab | ajo: -:) | 06:44 |
| ajo | irenab: and you? :) | 06:44 |
| irenab | ajo: I am ok, jumping between tones of tasks, as usual | 06:45 |
| irenab | ajo: I want to keep you a bit busy with future plans regarding QoS, can you please look and see if you can add some content https://etherpad.openstack.org/p/qos-nova-neutron | 06:46 |
| irenab | ajo: I wanted us to start discussion on this today during the meeting | 06:46 |
| *** nmagnezi has joined #openstack-neutron | 06:46 | |
| ajo | irenab: nice :) | 06:47 |
| ajo | irenab: looking | 06:47 |
| irenab | ajo: just started, but the use case is real, providers want to defined qos and associate with flavors, less management overhead | 06:48 |
| *** Ramanjaneya has quit IRC | 06:49 | |
| ajo | irenab: makes total sense | 06:49 |
| ajo | irenab: that way nova can request the port with an specific qos profile id | 06:50 |
| irenab | ajo: exactly | 06:50 |
| ajo | irenab: and it would work on the current case, as long as the qos policy id is shared by admin, | 06:51 |
| ajo | irenab: currently, the only thing, is that tenant could go down to the port and modify the policy_id on it | 06:51 |
| openstackgerrit | Kevin Benton proposed openstack/neutron: l2pop: check port mac in pre-commit to stop change https://review.openstack.org/214474 | 06:51 |
| ajo | irenab: I guess we can solve that via RBAC | 06:51 |
| irenab | ajo: right, and admin can ‘publish’ flavors to tenants based on their needs | 06:52 |
| ajo | irenab: and I'm also guessing that nova resquests the port to neutron with the tenant credentials | 06:52 |
| ajo | well, as it's an admin thing, with the policies not being shared | 06:53 |
| ajo | nova could make a second request, with admin context, to set the qos policy to the port | 06:53 |
| irenab | ajo: agree, this maybe not as trivial | 06:53 |
| ajo | so tenant cannot switch policies | 06:53 |
| irenab | ajo: for a min hoped we can leverage nova with minimal if at all changes :-) | 06:53 |
| ajo | irenab: it could be, we just need to check | 06:54 |
| irenab | ajo: Lets chat on this during the mneeting if we have time. You also mentioned last week that you started to look into nova aspects as well | 06:55 |
| *** aranjan has quit IRC | 06:55 | |
| openstackgerrit | Pengtao Huang proposed openstack/neutron: ignore the same values of service_plugins https://review.openstack.org/212597 | 06:55 |
| *** sthillma has joined #openstack-neutron | 06:56 | |
| ajo | irenab, yes, more for the guarantee's things. But I believe this topic is equally important (or more) | 06:56 |
| irenab | ajo: complimentary :-) | 06:56 |
| ajo | I talked to nikola dipanov recently | 06:57 |
| *** Murali has quit IRC | 06:57 | |
| ajo | and to.. somebody else before, but cannot recall his name :( ajo-- | 06:57 |
| *** jlvillal has quit IRC | 06:58 | |
| *** jlvillal has joined #openstack-neutron | 06:58 | |
| *** StevenK has quit IRC | 06:58 | |
| *** StevenK has joined #openstack-neutron | 06:59 | |
| *** VikasC has quit IRC | 06:59 | |
| *** y_sawai has quit IRC | 07:00 | |
| *** Murali has joined #openstack-neutron | 07:01 | |
| *** kiran-r has joined #openstack-neutron | 07:06 | |
| *** shashank_hegde has quit IRC | 07:10 | |
| *** shashank_hegde has joined #openstack-neutron | 07:15 | |
| *** ronis has joined #openstack-neutron | 07:16 | |
| *** shashank_hegde has quit IRC | 07:16 | |
| *** shashank_hegde has joined #openstack-neutron | 07:16 | |
| *** kawa2014 has joined #openstack-neutron | 07:16 | |
| *** jpena has joined #openstack-neutron | 07:17 | |
| *** ronis_ has joined #openstack-neutron | 07:18 | |
| *** jschwarz has joined #openstack-neutron | 07:18 | |
| *** ronis__ has joined #openstack-neutron | 07:19 | |
| *** anshul has quit IRC | 07:19 | |
| *** anshul has joined #openstack-neutron | 07:20 | |
| *** ronis has quit IRC | 07:21 | |
| *** fawadkhaliq has joined #openstack-neutron | 07:23 | |
| *** devvesa has quit IRC | 07:23 | |
| *** matrohon has joined #openstack-neutron | 07:23 | |
| *** scheuran has quit IRC | 07:23 | |
| *** lpetrut has quit IRC | 07:26 | |
| *** alex_klimov has joined #openstack-neutron | 07:26 | |
| *** y_sawai has joined #openstack-neutron | 07:27 | |
| *** lyanchih has quit IRC | 07:27 | |
| *** rubasov has joined #openstack-neutron | 07:27 | |
| *** matrohon has quit IRC | 07:27 | |
| *** nmagnezi_ has joined #openstack-neutron | 07:28 | |
| *** nmagnezi__ has joined #openstack-neutron | 07:29 | |
| *** alex_klimov has quit IRC | 07:29 | |
| *** alex_klimov has joined #openstack-neutron | 07:29 | |
| *** kiran-r has quit IRC | 07:29 | |
| *** apuimedo has joined #openstack-neutron | 07:31 | |
| *** mohankumar has quit IRC | 07:33 | |
| *** kiran-r has joined #openstack-neutron | 07:33 | |
| *** SourabhP has quit IRC | 07:33 | |
| *** ronis_ has quit IRC | 07:38 | |
| openstackgerrit | yalei wang proposed openstack/neutron: Add support for unaddressed port https://review.openstack.org/200809 | 07:39 |
| *** nyechiel_ has joined #openstack-neutron | 07:40 | |
| *** yog_ has joined #openstack-neutron | 07:40 | |
| *** lyanchih has joined #openstack-neutron | 07:40 | |
| *** nmagnezi__ has quit IRC | 07:40 | |
| *** nmagnezi has quit IRC | 07:40 | |
| *** nmagnezi has joined #openstack-neutron | 07:41 | |
| *** salv-orlando has joined #openstack-neutron | 07:41 | |
| *** nmagnezi has quit IRC | 07:45 | |
| *** yamamoto has quit IRC | 07:46 | |
| *** nmagnezi has joined #openstack-neutron | 07:47 | |
| *** yamamoto has joined #openstack-neutron | 07:48 | |
| *** vikram has joined #openstack-neutron | 07:49 | |
| *** matrohon has joined #openstack-neutron | 07:50 | |
| *** ronis__ has quit IRC | 07:52 | |
| *** shaohe_feng has joined #openstack-neutron | 07:54 | |
| *** klindgren has quit IRC | 07:54 | |
| vikram | amotoki: ping | 07:55 |
| amotoki | vikram: pong | 07:55 |
| vikram | amotoki: Can please have a look at "https://review.openstack.org/#/c/214193/" | 07:55 |
| vikram | we need these changes for networking_sfc unit testing | 07:56 |
| amotoki | vikram: sure | 07:56 |
| vikram | amotoki: thanks for your instant help | 07:56 |
| *** tflynn has joined #openstack-neutron | 07:57 | |
| *** matrohon has quit IRC | 07:58 | |
| *** boris-42 has joined #openstack-neutron | 07:58 | |
| *** shaohe_feng has quit IRC | 08:03 | |
| *** shaohe_feng has joined #openstack-neutron | 08:03 | |
| *** yog_ has quit IRC | 08:05 | |
| *** shaohe_feng1 has joined #openstack-neutron | 08:09 | |
| *** tflynn has quit IRC | 08:10 | |
| openstackgerrit | vikram.choudhary proposed openstack/python-neutronclient: [WIP] BGP Dynamic Routing: neutronclient changes https://review.openstack.org/205756 | 08:11 |
| *** shaohe_feng has quit IRC | 08:12 | |
| *** gampel has joined #openstack-neutron | 08:15 | |
| *** shashank_hegde has quit IRC | 08:17 | |
| *** lyanchih has quit IRC | 08:18 | |
| *** apuimedo has quit IRC | 08:19 | |
| *** gildub has quit IRC | 08:19 | |
| *** saikrishna has joined #openstack-neutron | 08:19 | |
| *** bharath_ has quit IRC | 08:20 | |
| openstackgerrit | vikram.choudhary proposed openstack/python-neutronclient: Support Command line changes for Address Scope https://review.openstack.org/194635 | 08:21 |
| *** timbyr_ has joined #openstack-neutron | 08:21 | |
| *** bharath_ has joined #openstack-neutron | 08:21 | |
| *** nmagnezi has quit IRC | 08:22 | |
| *** nmagnezi_ has quit IRC | 08:22 | |
| *** markvoelker has quit IRC | 08:22 | |
| amotoki | vikram: I just commented the review you requested. I commented another idea. thoguht? | 08:22 |
| vikram | amotoki: thanks will check | 08:23 |
| vikram | amotoki: Looks good to me | 08:24 |
| vikram | amotoki: Will test ! | 08:24 |
| *** shaohe_feng1 has quit IRC | 08:24 | |
| openstackgerrit | yalei wang proposed openstack/neutron: Add missing tenant_id validation in RESOURCE_ATTRIBUTE_MAP https://review.openstack.org/214506 | 08:25 |
| *** sthillma has quit IRC | 08:25 | |
| *** liangbo has joined #openstack-neutron | 08:25 | |
| *** park_heijlong has joined #openstack-neutron | 08:25 | |
| *** park_heijlong has quit IRC | 08:25 | |
| *** bharath_ has quit IRC | 08:26 | |
| *** nmagnezi_ has joined #openstack-neutron | 08:27 | |
| *** nmagnezi_ has quit IRC | 08:27 | |
| *** saikrishna has quit IRC | 08:31 | |
| *** Ramanjaneya has joined #openstack-neutron | 08:31 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Add object status transition barrier mechanism https://review.openstack.org/181674 | 08:33 |
| *** jistr has joined #openstack-neutron | 08:34 | |
| *** nmagnezi has joined #openstack-neutron | 08:34 | |
| *** bharath_ has joined #openstack-neutron | 08:36 | |
| *** fawadkhaliq has quit IRC | 08:36 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-vpnaas: Updated from global requirements https://review.openstack.org/212262 | 08:36 |
| *** ihrachyshka has joined #openstack-neutron | 08:38 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-neutronclient: Updated from global requirements https://review.openstack.org/204422 | 08:39 |
| *** lpetrut has joined #openstack-neutron | 08:39 | |
| *** shoutm has quit IRC | 08:40 | |
| openstackgerrit | Kevin Benton proposed openstack/neutron: Add object status transition barrier mechanism https://review.openstack.org/181674 | 08:42 |
| *** yalie1 has quit IRC | 08:43 | |
| *** iwamoto has quit IRC | 08:45 | |
| *** yalie has joined #openstack-neutron | 08:46 | |
| *** yalie1 has joined #openstack-neutron | 08:47 | |
| *** yalie2 has joined #openstack-neutron | 08:48 | |
| *** yalie1 has quit IRC | 08:48 | |
| *** ihrachyshka has quit IRC | 08:48 | |
| *** ihrachyshka has joined #openstack-neutron | 08:48 | |
| *** achanda has joined #openstack-neutron | 08:48 | |
| *** kiran-r has quit IRC | 08:49 | |
| *** Murali has quit IRC | 08:49 | |
| *** yalie has quit IRC | 08:51 | |
| *** haigang has quit IRC | 08:52 | |
| *** haigang has joined #openstack-neutron | 08:52 | |
| *** haigang has quit IRC | 08:52 | |
| *** haigang has joined #openstack-neutron | 08:53 | |
| *** haigang has quit IRC | 08:53 | |
| *** haigang has joined #openstack-neutron | 08:53 | |
| *** aix has joined #openstack-neutron | 08:54 | |
| *** nijaba has quit IRC | 09:00 | |
| *** saikrishna has joined #openstack-neutron | 09:04 | |
| openstackgerrit | John Schwarz proposed openstack/python-neutronclient: Support CLI changes for QoS (2/2). https://review.openstack.org/198277 | 09:06 |
| openstackgerrit | John Schwarz proposed openstack/python-neutronclient: Support QoS neutron-client (1/2). https://review.openstack.org/189655 | 09:06 |
| jschwarz | ihrachyshka, ajo, ^^ :) | 09:06 |
| jschwarz | all rebased | 09:06 |
| jschwarz | will fix the server-side error when I get a chance to | 09:06 |
| ajo | jschwarz, ack :) | 09:06 |
| ajo | jschwarz, which server side error? | 09:06 |
| ihrachyshka | jschwarz, oh cool | 09:07 |
| ajo | jschwarz the one we talked about? | 09:07 |
| ihrachyshka | jschwarz, yeah, please reports bugs for any qos issues, and mark with 'qos' tag | 09:07 |
| jschwarz | ajo, some filters we're missing, the one moshele discovered and ihrachyshka already fixed in his mind ;-) | 09:07 |
| jschwarz | ihrachyshka, will do | 09:07 |
| ihrachyshka | jschwarz, I think someone was already looking into it | 09:07 |
| ihrachyshka | maybe moshele | 09:07 |
| ajo | ack: jschwarz check if it's registered https://bugs.launchpad.net/neutron/+bugs?field.tag=qos | 09:07 |
| ihrachyshka | jschwarz, so check with LP first | 09:08 |
| ihrachyshka | ok, I feel redundant | 09:08 |
| jschwarz | ihrachyshka, the one with the bandwidth-limit-rule-list showing all rules? | 09:08 |
| ihrachyshka | ajo, tell him :) | 09:08 |
| ajo | jschwarz: I guess it's: https://bugs.launchpad.net/neutron/+bug/1486053 | 09:08 |
| openstack | Launchpad bug 1486053 in neutron "get get_policy_bandwidth_limit_rules_for_policy return all bw rules and not filter by qos policy" [Undecided,In progress] - Assigned to Moshe Levi (moshele) | 09:08 |
| ajo | may be moshele is already at it | 09:08 |
| ajo | if that's the bug | 09:08 |
| moshele | ajo : you mean this https://review.openstack.org/#/c/214212/ | 09:08 |
| ajo | yey | 09:08 |
| jschwarz | moshele, yey XD | 09:09 |
| ajo | jschwarz: there's a similar bug: https://bugs.launchpad.net/neutron/+bug/1485993 | 09:09 |
| openstack | Launchpad bug 1485993 in neutron "Tenants could potentially modify rules from not owned policies" [Undecided,New] - Assigned to yong sheng gong (gongysh) | 09:09 |
| ajo | if you want to tackle it ;) | 09:09 |
| jschwarz | moshele, can I ask you for an API test in addition? | 09:09 |
| jschwarz | ajo, aye, we talked about it a while back :) | 09:09 |
| ajo | hmm gongysh took it 10 minutes ago | 09:09 |
| jschwarz | I'll need to add myself as a reviewer for those | 09:09 |
| *** dr_gogeta86_ is now known as dr_gogeta86 | 09:09 | |
| *** dr_gogeta86 has joined #openstack-neutron | 09:10 | |
| ajo | gongysh, are you working on 1485993 ? :) | 09:10 |
| *** hoangcx has quit IRC | 09:10 | |
| jschwarz | right so I'm off for lunch | 09:10 |
| ajo | gongysh, to verify it, you may need to modify /etc/neutron/policies.json | 09:10 |
| jschwarz | ajo, ihrachyshka, please have a good serious review of the neutronclient code | 09:10 |
| ajo | to allow tenants to create/modify rules/policies | 09:10 |
| ajo | :) | 09:10 |
| jschwarz | it's about time we push for it to merge :) | 09:10 |
| *** jschwarz is now known as jschwarz|lunch | 09:10 | |
| ajo | jschwarz: ack | 09:10 |
| ihrachyshka | jschwarz, it's in todo for today (that usually does not complete at the end of the day though) | 09:11 |
| ajo | jschwarz|lunch: I also believe the enhancement to auto-lookup for rule policies can be done in a follow up too | 09:11 |
| *** hoangcx has joined #openstack-neutron | 09:11 | |
| jschwarz|lunch | ajo, you'll have to be more specific\ | 09:11 |
| jschwarz|lunch | what enhancement? :) | 09:11 |
| ajo | jschwarz|lunch, to not need to specify the policy of a rule | 09:12 |
| jschwarz|lunch | off to food before mkolesni kills me | 09:12 |
| jschwarz|lunch | ahh | 09:12 |
| ajo | jschwarz|lunch: by listing policies, looking for the ID of the rule, using that policy id | 09:12 |
| *** abregman is now known as abregman|lunch | 09:12 | |
| jschwarz|lunch | ajo, then have a look at... | 09:12 |
| ajo | jschwarz|lunch: already done? :) | 09:12 |
| ajo | oh, you're my hero then ;D | 09:12 |
| jschwarz|lunch | ajo, https://review.openstack.org/#/c/207053/ | 09:12 |
| jschwarz|lunch | this | 09:12 |
| jschwarz|lunch | was a WIP before the mergeback but we decided not to | 09:12 |
| ajo | no | 09:12 |
| ajo | that doesn't work | 09:13 |
| jschwarz|lunch | have a look, it does what you want | 09:13 |
| jschwarz|lunch | going to food | 09:13 |
| ihrachyshka | ajo, let's not, at least not now | 09:13 |
| ihrachyshka | it introduces inconsistency that we should think better | 09:13 |
| ajo | we can auto discover from client without crushing the API | 09:13 |
| *** yanxb_ has joined #openstack-neutron | 09:13 | |
| ajo | ihrachyshka, I'm not talking about changing the API | 09:13 |
| *** sudipto has quit IRC | 09:13 | |
| *** lucasagomes has joined #openstack-neutron | 09:13 | |
| ajo | ihrachyshka, I mean, the client lists available policies, in the listing it has the rules... and auto pickups the right policy | 09:14 |
| ajo | I'll send a patch for it | 09:14 |
| ajo | :) | 09:14 |
| yanxb_ | test for hello | 09:14 |
| lucasagomes | hi all, there's a gate problem in Ironic testing postgres and it seems to be due an error in neutron | 09:14 |
| lucasagomes | http://logs.openstack.org/56/213856/4/check/gate-tempest-dsvm-ironic-pxe_ssh-postgres/91c7a5c/logs/screen-q-svc.txt.gz#_2015-08-19_03_23_38_802 | 09:14 |
| *** jlibosva has joined #openstack-neutron | 09:14 | |
| ihrachyshka | oh well... so you actually get all policies | 09:14 |
| *** yanxb has joined #openstack-neutron | 09:14 | |
| ihrachyshka | that's a pricy thing | 09:14 |
| ajo | ihrachyshka, may be you're right | 09:14 |
| ajo | good point | 09:14 |
| ajo | we don't expect to have thousands of policies, but... yes, it's expensive for just changing a rule.. | 09:15 |
| ajo | ihrachyshka: may be we could somehow filter policies by rule-id ? ? | 09:15 |
| ihrachyshka | lucasagomes, that's probably broken by: https://review.openstack.org/#/c/163659/ | 09:15 |
| ihrachyshka | lucasagomes, please report a bug, and assign to salv-orlando | 09:15 |
| *** yanxb__ has joined #openstack-neutron | 09:16 | |
| ajo | it's a subobject, ... so filtering needs an to lookup into the binding table.. | 09:16 |
| ajo | ok... | 09:16 |
| ajo | let's think about it later | 09:16 |
| ajo | I wanted to avoid changes into the client interface at a later time | 09:16 |
| ajo | but I guess we can just make the 2nd arg (policy) optional | 09:16 |
| ajo | in a 2nd version | 09:16 |
| ajo | yep | 09:16 |
| ajo | we could | 09:16 |
| *** achanda has quit IRC | 09:17 | |
| *** lyanchih has joined #openstack-neutron | 09:18 | |
| lucasagomes | ihrachyshka, will do | 09:18 |
| openstackgerrit | Jakub Libosvar proposed openstack/neutron: fullstack: Skip NotFound in safe_client cleanup https://review.openstack.org/214236 | 09:18 |
| *** gberginc has quit IRC | 09:18 | |
| *** yanxb has quit IRC | 09:19 | |
| *** yanxb_ has quit IRC | 09:19 | |
| *** yanxb__ has quit IRC | 09:19 | |
| *** gongysh has quit IRC | 09:19 | |
| *** liangbo has quit IRC | 09:19 | |
| *** john-davidge has joined #openstack-neutron | 09:20 | |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Template for ModelMigrationTest for external repos https://review.openstack.org/213194 | 09:22 |
| *** markvoelker has joined #openstack-neutron | 09:23 | |
| lucasagomes | ihrachyshka, https://bugs.launchpad.net/neutron/+bug/1486467 can you please escalate it? Ironic gate is broken due that error | 09:23 |
| openstack | Launchpad bug 1486467 in neutron "Postgres error: column "reservations.expiration" must appear in the GROUP BY clause or be used in an aggregate function" [Undecided,New] | 09:23 |
| lucasagomes | so we can't merge any patches | 09:23 |
| *** bharath__ has joined #openstack-neutron | 09:24 | |
| *** bharath_ has quit IRC | 09:24 | |
| jlibosva | ajo: hi | 09:24 |
| *** yanxb_ has joined #openstack-neutron | 09:24 | |
| ajo | jlibosva: hi :) | 09:24 |
| jlibosva | ajo: I don't get what you want in https://review.openstack.org/#/c/214215/1/neutron/db/qos/models.py :) | 09:24 |
| jlibosva | ajo: does it mean I should go with what is written in the patch? Or in the comment? | 09:24 |
| ajo | I mean | 09:24 |
| ajo | your current implementation in the patch | 09:25 |
| ajo | sorry | 09:25 |
| ajo | it was misleading | 09:25 |
| ajo | compared to the alternative you were proposing | 09:25 |
| jlibosva | ajo: ok :) thanks | 09:25 |
| ihrachyshka | lucasagomes, ack, as much as I can do for you | 09:25 |
| jlibosva | ajo: yeah, I found the alternative later when reading sqlalchemy doc | 09:25 |
| lucasagomes | ihrachyshka, thank you very much! | 09:25 |
| ihrachyshka | akamyshnikova, ping re postgres failure | 09:25 |
| *** yalie2 has quit IRC | 09:25 | |
| *** devvesa has joined #openstack-neutron | 09:26 | |
| akamyshnikova | ihrachyshka, hi | 09:26 |
| akamyshnikova | akamyshnikova, what failure? | 09:27 |
| ihrachyshka | akamyshnikova, ironic gate is busted because of reservations support patch from salv-orlando. the failure can be seen in: http://logs.openstack.org/56/213856/4/check/gate-tempest-dsvm-ironic-pxe_ssh-postgres/91c7a5c/logs/screen-q-svc.txt.gz#_2015-08-19_03_23_38_802 | 09:27 |
| ihrachyshka | I wonder whether you have quick ideas on what's going on there | 09:27 |
| *** yalie has joined #openstack-neutron | 09:27 | |
| *** ajo is now known as ajo\afk | 09:27 | |
| *** ddepaoli has quit IRC | 09:27 | |
| *** fawadkhaliq has joined #openstack-neutron | 09:28 | |
| *** miyagishi_t has quit IRC | 09:28 | |
| *** SumitNaiksatam has joined #openstack-neutron | 09:28 | |
| *** markvoelker has quit IRC | 09:28 | |
| ihrachyshka | lucasagomes, is it just postgres? do you have mysql gate? | 09:29 |
| lucasagomes | ihrachyshka, yeah mysql seems grand | 09:29 |
| lucasagomes | ihrachyshka, https://review.openstack.org/#/c/209605/ i.e | 09:29 |
| ihrachyshka | lucasagomes, yeah, we don't have psql gate in neutron since some time | 09:30 |
| *** mohankumar has joined #openstack-neutron | 09:30 | |
| openstackgerrit | Jakub Libosvar proposed openstack/neutron: qos: Delete bw limit rule when policy is deleted https://review.openstack.org/214215 | 09:30 |
| lucasagomes | yeah, I was looking to see if other projects were broken by the same problem but apparently not many projects runs postgres in the gate | 09:30 |
| lucasagomes | we have only 1 job with it | 09:30 |
| openstackgerrit | Jakub Libosvar proposed openstack/neutron: qos: Delete bw limit rule when policy is deleted https://review.openstack.org/214215 | 09:32 |
| *** kiran-r has joined #openstack-neutron | 09:34 | |
| *** DWfuturetec has joined #openstack-neutron | 09:35 | |
| Fdaisuke | ihrachyshka: Hi, | 09:35 |
| ihrachyshka | Fdaisuke, what's up | 09:37 |
| Fdaisuke | ihrachyshka:So, about the following bugs which you take over that you want to modify it before, | 09:38 |
| Fdaisuke | would you tell me the current situation? | 09:38 |
| Fdaisuke | https://bugs.launchpad.net/neutron/+bug/1466476 | 09:38 |
| openstack | Launchpad bug 1466476 in neutron "Change of the default value of user_identity in the log format" [Undecided,Incomplete] - Assigned to Ihar Hrachyshka (ihar-hrachyshka) | 09:38 |
| Fdaisuke | https://bugs.launchpad.net/neutron/+bug/1468723 | 09:38 |
| openstack | Launchpad bug 1468723 in neutron "Resource uuid is not logged when a resource is created" [Undecided,New] - Assigned to Ihar Hrachyshka (ihar-hrachyshka) | 09:38 |
| *** gberginc has joined #openstack-neutron | 09:38 | |
| ihrachyshka | Fdaisuke, oh that one. I was distracted and probably won't get to it in L | 09:38 |
| ihrachyshka | Fdaisuke, wanna take over? I think we had a plan with oslo team that required patches there. | 09:39 |
| ihrachyshka | aye, it's in comments for the first bug | 09:39 |
| *** _kiran_ has joined #openstack-neutron | 09:40 | |
| *** Ramanjaneya has quit IRC | 09:40 | |
| *** kiran-r has quit IRC | 09:41 | |
| Fdaisuke | ihrachyshka: Oh, okey. It seems that you are busy, I'll take over these. | 09:42 |
| akamyshnikova | ihrachyshka, may be this appeared because postgres expect to have pk in GROUP BY | 09:42 |
| *** ddepaoli has joined #openstack-neutron | 09:43 | |
| *** bharath__ has quit IRC | 09:43 | |
| ihrachyshka | Fdaisuke, thanks. please add me to reviewer lists | 09:44 |
| *** zhhuabj has quit IRC | 09:44 | |
| *** jaypipes has quit IRC | 09:46 | |
| Fdaisuke | ihrachyshka: okey! :) | 09:46 |
| salv-orlando | ihrachyshka: I have pushed a revert uesterday | 09:47 |
| salv-orlando | did it now work? | 09:47 |
| *** killer_prince has joined #openstack-neutron | 09:48 | |
| salv-orlando | ihrachyshka: it merge about 8 hours ago: https://review.openstack.org/#/c/214282/ | 09:49 |
| akamyshnikova | akamyshnikova, according this http://www.postgresql.org/message-id/200402271700.28133.dev@archonet.com we may need to have reservations.expiration in GROUP BY | 09:50 |
| ihrachyshka | salv-orlando, oh, I missed that | 09:51 |
| ihrachyshka | sorry to hear | 09:51 |
| salv-orlando | akamyshnikova: no worries I am fixing that already | 09:51 |
| ihrachyshka | salv-orlando, thanks! lucasagomes: does it mean you are unblocked? ^^ | 09:51 |
| salv-orlando | ihrachyshka: i did that because of ironic and pymsql | 09:51 |
| lucasagomes | ihrachyshka, salv-orlando lemme check | 09:51 |
| salv-orlando | ihrachyshka: pymsql does not like the resrvtion code either | 09:51 |
| lucasagomes | I will recheck a commit there | 09:51 |
| *** lazy_prince has quit IRC | 09:52 | |
| ihrachyshka | lucasagomes, ok. please update the bug once it's checked, and hopefully close | 09:52 |
| lucasagomes | ihrachyshka, salv-orlando will do! Thanks folks | 09:52 |
| akamyshnikova | salv-orlando, OK! | 09:53 |
| salv-orlando | lucasagomes: np, and sorry about the inconvenience | 09:53 |
| *** _kiran_ is now known as kiran-r | 09:53 | |
| lucasagomes | salv-orlando, np at all. I mean, that's what gate's are for | 09:53 |
| salv-orlando | akamyshnikova: in my opinion this is possibly a sqlalchemy bug, but we can fix it on the neutron side either | 09:53 |
| *** kiran-r has quit IRC | 09:53 | |
| *** kiran-r has joined #openstack-neutron | 09:54 | |
| salv-orlando | I think sqlalchemy should provide abstraction and therefore it should automatically generate postgres queries putting in group_by everything that is not in the agrregate func | 09:54 |
| ihrachyshka | salv-orlando, I am sure zzzeek will be happy to see a bug report | 09:55 |
| *** tfukushima has joined #openstack-neutron | 09:55 | |
| *** apuimedo has joined #openstack-neutron | 09:57 | |
| *** killer_prince has quit IRC | 09:58 | |
| akamyshnikova | salv-orlando, seems so | 09:59 |
| *** ushkalim has joined #openstack-neutron | 09:59 | |
| *** ddepaoli has quit IRC | 09:59 | |
| *** anilvenkata has joined #openstack-neutron | 10:00 | |
| *** anilvenkata has left #openstack-neutron | 10:00 | |
| *** saikrishna has quit IRC | 10:00 | |
| *** DWfuturetec has quit IRC | 10:00 | |
| *** zhhuabj has joined #openstack-neutron | 10:01 | |
| *** anilvenkata has joined #openstack-neutron | 10:01 | |
| *** yamamoto has quit IRC | 10:04 | |
| *** piyanai has joined #openstack-neutron | 10:05 | |
| *** Samos123 has quit IRC | 10:06 | |
| *** clev-away has quit IRC | 10:06 | |
| *** clev has joined #openstack-neutron | 10:07 | |
| openstackgerrit | Pavel Bondar proposed openstack/neutron: Move tests for non pluggable ipam backend https://review.openstack.org/211507 | 10:10 |
| *** Samos123 has joined #openstack-neutron | 10:11 | |
| tfukushima | diga, apuimedo: Hi diga, how's your network? Did you get the access to Gerrit finally? :-) | 10:12 |
| *** tfukushima has quit IRC | 10:15 | |
| *** tfukushima has joined #openstack-neutron | 10:16 | |
| *** lazy_prince has joined #openstack-neutron | 10:16 | |
| *** shoutm has joined #openstack-neutron | 10:18 | |
| *** Fdaisuke has quit IRC | 10:19 | |
| *** abregman|lunch is now known as abregman | 10:22 | |
| *** _kiran_ has joined #openstack-neutron | 10:23 | |
| *** y_sawai has quit IRC | 10:24 | |
| *** kiran-r has quit IRC | 10:24 | |
| *** baojg has quit IRC | 10:26 | |
| *** baojg has joined #openstack-neutron | 10:27 | |
| *** _kiran_ has quit IRC | 10:29 | |
| apuimedo | diga: only one port should be created for a container | 10:29 |
| apuimedo | s/a container/endpoint/ | 10:29 |
| *** jschwarz|lunch is now known as jschwarz | 10:29 | |
| apuimedo | and it will be bound to the veth side that does not go inside the container | 10:30 |
| diga | apuimedo: ok | 10:30 |
| diga | got it | 10:30 |
| *** baojg has quit IRC | 10:31 | |
| *** yamahata has quit IRC | 10:32 | |
| *** kiran-r has joined #openstack-neutron | 10:43 | |
| *** bharath_ has joined #openstack-neutron | 10:43 | |
| *** _kiran_ has joined #openstack-neutron | 10:45 | |
| *** _kiran_ has quit IRC | 10:45 | |
| *** _kiran_ has joined #openstack-neutron | 10:45 | |
| *** _kiran_ has quit IRC | 10:46 | |
| *** _kiran_ has joined #openstack-neutron | 10:47 | |
| *** kiran-r has quit IRC | 10:48 | |
| *** bharath_ has quit IRC | 10:48 | |
| *** _kiran_ has quit IRC | 10:48 | |
| *** kiran-r has joined #openstack-neutron | 10:49 | |
| apuimedo | tfukushima: diga: please set a topic to the patches according to feature: vif-binding in this case | 10:49 |
| *** nmagnezi_ has joined #openstack-neutron | 10:49 | |
| *** vthapar has joined #openstack-neutron | 10:49 | |
| *** nmagnezi_ has quit IRC | 10:50 | |
| *** stanzgy has quit IRC | 10:52 | |
| *** salv-orlando has quit IRC | 10:53 | |
| lucasagomes | salv-orlando, ihrachyshka unfortunately the problem seems to persist http://logs.openstack.org/56/213856/4/check/gate-tempest-dsvm-ironic-pxe_ssh-postgres/4edf470/logs/screen-q-svc.txt.gz#_2015-08-19_10_28_35_245 | 10:53 |
| lucasagomes | the other tests running against mysql seems fine | 10:53 |
| *** fawadkhaliq has quit IRC | 10:53 | |
| *** dims has joined #openstack-neutron | 10:56 | |
| *** mwagner_lap has quit IRC | 10:57 | |
| *** yamamoto_ has joined #openstack-neutron | 10:58 | |
| *** pal has joined #openstack-neutron | 10:59 | |
| *** ronis has joined #openstack-neutron | 10:59 | |
| *** killer_prince has joined #openstack-neutron | 10:59 | |
| pal | Hi. i tried to create a 2 node setup with devstack and tried to ping instance from controller but I am unable to do that | 11:00 |
| pal | can ayone help me on that | 11:00 |
| ihrachyshka | lucasagomes, meh. I guess it was not disabled completely | 11:00 |
| diga | apuimedo: sure | 11:01 |
| ihrachyshka | we need salv to take a look again | 11:01 |
| *** shoutm has quit IRC | 11:01 | |
| *** lazy_prince has quit IRC | 11:03 | |
| *** dims_ has joined #openstack-neutron | 11:03 | |
| *** jagter has quit IRC | 11:04 | |
| ihrachyshka | akamyshnikova, fyi we still have pgsql problem ^^ | 11:04 |
| *** baoli has joined #openstack-neutron | 11:05 | |
| akamyshnikova | ihrachyshka, salv-orlando is going to fix it, no? | 11:07 |
| *** dims has quit IRC | 11:07 | |
| *** hoangcx has quit IRC | 11:07 | |
| *** Murali has joined #openstack-neutron | 11:08 | |
| *** ddepaoli has joined #openstack-neutron | 11:08 | |
| *** thorst has joined #openstack-neutron | 11:09 | |
| *** garyk has joined #openstack-neutron | 11:10 | |
| *** britthouser has quit IRC | 11:10 | |
| *** baoli has quit IRC | 11:10 | |
| *** saikrishna has joined #openstack-neutron | 11:11 | |
| *** pc_m has joined #openstack-neutron | 11:11 | |
| *** gampel has quit IRC | 11:11 | |
| *** sudipto has joined #openstack-neutron | 11:13 | |
| *** gampel has joined #openstack-neutron | 11:14 | |
| openstackgerrit | Gal Sagie proposed openstack/neutron-specs: [WIP] Add Kuryr spec to Neutron https://review.openstack.org/213490 | 11:14 |
| akamyshnikova | ihrachyshka, we can try something like this http://paste.openstack.org/show/421749/ | 11:14 |
| *** anshul has quit IRC | 11:15 | |
| *** ddepaoli has quit IRC | 11:15 | |
| akamyshnikova | ihrachyshka, do you want me to push this change or we'll look savl-orlando? | 11:16 |
| *** anshul has joined #openstack-neutron | 11:16 | |
| ihrachyshka | akamyshnikova, I trust your judgement here, for I know nothing about that specific aspect of neutron :) not sure salv works on immediate fix or a general one to fix it for pymysql | 11:16 |
| ihrachyshka | akamyshnikova, you can at least push that as WIP to see what gate has to say to it. ironic people can depends-on it to validate the path. | 11:17 |
| akamyshnikova | ihrachyshka, OK, will do this | 11:17 |
| ihrachyshka | akamyshnikova, thanks! | 11:17 |
| ihrachyshka | since neutron is breaking another project, I think it's worth to validate the fix as soon as we have some idea. | 11:18 |
| *** vthapar has quit IRC | 11:20 | |
| *** boris-42 has quit IRC | 11:20 | |
| akamyshnikova | ihrachyshka, is there a bug for this? to add in commit message | 11:20 |
| *** pal has quit IRC | 11:21 | |
| ihrachyshka | akamyshnikova, https://bugs.launchpad.net/neutron/+bug/1486467 | 11:24 |
| openstack | Launchpad bug 1486467 in neutron "Postgres error: column "reservations.expiration" must appear in the GROUP BY clause or be used in an aggregate function" [Critical,New] | 11:24 |
| *** markvoelker has joined #openstack-neutron | 11:24 | |
| akamyshnikova | ihrachyshka, thanks! | 11:24 |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Fix ironic gates https://review.openstack.org/214564 | 11:25 |
| *** gampel has quit IRC | 11:26 | |
| *** shoutm has joined #openstack-neutron | 11:26 | |
| *** DWfuturetec has joined #openstack-neutron | 11:26 | |
| ihrachyshka | akamyshnikova++ | 11:26 |
| ihrachyshka | lucasagomes, ^^ can you depends-on an ironic patch so that we can validate the fix in gate? | 11:27 |
| lucasagomes | ihrachyshka, +1, doing it now | 11:27 |
| *** ddepaoli has joined #openstack-neutron | 11:27 | |
| ihrachyshka | lucasagomes, thanks a lot for help. please post a link once it's done so that I can monitor the gate in zuul | 11:28 |
| *** slaweq has joined #openstack-neutron | 11:28 | |
| *** markvoelker has quit IRC | 11:28 | |
| *** dims has joined #openstack-neutron | 11:29 | |
| *** tiny-hands has quit IRC | 11:29 | |
| lucasagomes | ihrachyshka, will do, the patch is https://review.openstack.org/#/c/214566/ | 11:29 |
| lucasagomes | I will keep an eye on it | 11:29 |
| *** mohankumar has quit IRC | 11:30 | |
| * ihrachyshka too | 11:30 | |
| *** dims_ has quit IRC | 11:31 | |
| *** dims_ has joined #openstack-neutron | 11:31 | |
| ajo\afk | ihrachyshka, jlibosva , added a question here:https://review.openstack.org/#/c/214215/3/neutron/db/qos/models.py | 11:32 |
| ajo\afk | could you check? | 11:32 |
| ajo\afk | I admit my ignorance with sqlalchemy, but I thought the DB SQL constraint in the migration should be enough | 11:32 |
| ihrachyshka | ajo\afk, I think relationships or cascade delete are sqlalchemy things | 11:33 |
| *** roeyc has joined #openstack-neutron | 11:33 | |
| ajo\afk | ihrachyshka, sql has on delete cascade | 11:33 |
| ajo\afk | when you define a foreign key | 11:33 |
| ajo\afk | at low level | 11:33 |
| ihrachyshka | ok, maybe. I am more ignorant than you | 11:34 |
| *** lyanchih has quit IRC | 11:34 | |
| ihrachyshka | :) | 11:34 |
| openstackgerrit | Paul Michali proposed openstack/python-neutronclient: Create hooks for running functional test https://review.openstack.org/210021 | 11:34 |
| ajo\afk | may be alembic isnt generating those for some compatibility reason? | 11:34 |
| * ajo\afk mysql-dumps and checks | 11:34 | |
| *** dims has quit IRC | 11:34 | |
| *** Kiall_ is now known as Kiall | 11:34 | |
| ihrachyshka | ajo\afk, you better ask zzzeek I guess | 11:34 |
| *** yamamoto_ has quit IRC | 11:35 | |
| ajo\afk | ihrachyshka: http://paste.fedoraproject.org/256674/39984110/ | 11:35 |
| ajo\afk | there it is, the ON DELETE CASCADE | 11:35 |
| *** fawadkhaliq has joined #openstack-neutron | 11:35 | |
| ajo\afk | ihrachyshka: I'll ask him | 11:35 |
| ajo\afk | ihrachyshka, jlibosva , did we actually check that this was indeed happening or it was a suspicion? | 11:35 |
| *** yamamoto has joined #openstack-neutron | 11:36 | |
| *** saikrishna has quit IRC | 11:36 | |
| ajo\afk | may be it's important to tell sqlalchemy model so it will trigger some delete method you can capture in the object.. but, otherwise I'm missing something important | 11:36 |
| ihrachyshka | ajo\afk, oh sorry I think I replied to wrong question | 11:36 |
| ihrachyshka | re: why do we need it in models.py... | 11:36 |
| ihrachyshka | that's because fullstack does not rely on migration scripts | 11:37 |
| ajo\afk | ihrachyshka: yes my question is why do we need it in models | 11:37 |
| ihrachyshka | but on create_all() | 11:37 |
| *** yamamoto has quit IRC | 11:37 | |
| *** yamamoto has joined #openstack-neutron | 11:37 | |
| ihrachyshka | which can be a problem in itself, but still | 11:37 |
| ajo\afk | yikes | 11:37 |
| jlibosva | the real question is why fullstack rely on create_all() when production code doesn't use it ... | 11:37 |
| ajo\afk | why do we do such thing? | 11:37 |
| ajo\afk | yes | 11:37 |
| ajo\afk | we may be sticking to production code as much as we can IMHO | 11:37 |
| ihrachyshka | jlibosva, it was easier to do it that way I guess ;) | 11:37 |
| ajo\afk | but may be amuller / jschwarz can tell us, I guess there's a reason | 11:38 |
| *** ajo\afk is now known as ajo | 11:38 | |
| ihrachyshka | I think we should fix the fullstack for us but report a bug to switch to migrations | 11:38 |
| ihrachyshka | that would be solved outside of qos scope | 11:38 |
| ajo | I guess yes | 11:38 |
| ajo | let's go on with the patch | 11:38 |
| ajo | so we'll be good with fullstack until they fix it | 11:38 |
| ajo | since the patch is done.. ;) | 11:38 |
| jlibosva | has anybody reported bug? If not, I can | 11:39 |
| ihrachyshka | jlibosva, go on | 11:40 |
| jlibosva | ajo: ihrachyshka the second concern is why it wasn't caught by functional db tests | 11:40 |
| * jlibosva going on | 11:40 | |
| ihrachyshka | jlibosva, the one that compares models with scripts? | 11:40 |
| jlibosva | yes, I haven't gone through that deeply | 11:40 |
| *** lucasagomes is now known as lucas-hungry | 11:40 | |
| ajo | ihrachyshka: may be unit tests also use create_all I guess | 11:40 |
| ajo | I don't see unit tests calling migrations all the time :) | 11:41 |
| ihrachyshka | ajo, they do use it | 11:41 |
| ajo | and for unit tests I guess it makes sense for speed | 11:41 |
| ajo | ok, mistery solved | 11:41 |
| ihrachyshka | ajo, as long as we validate models reflect migrations, I don't care much. but it seems validation has holes | 11:41 |
| ajo | ihrachyshka, yes, may be some automated verification of such thing could be good | 11:42 |
| ajo | run migrations, run create_all, compare for diffs... | 11:42 |
| ihrachyshka | ajo, there is such thing | 11:42 |
| ihrachyshka | neutron/tests/functional/db/test_migrations.py | 11:42 |
| ajo | hm | 11:42 |
| ihrachyshka | but apparently there are holes | 11:42 |
| ajo | hmm | 11:42 |
| *** tfukushima has quit IRC | 11:42 | |
| ajo | I guess it's not checking that type of constraint? | 11:43 |
| ihrachyshka | it uses alembic.autogenerate.compare_metadata | 11:43 |
| ajo | interesting, so that's what you meant with "functional tests" | 11:43 |
| ajo | yikes, alembic-- ;) | 11:44 |
| *** saikrishna has joined #openstack-neutron | 11:46 | |
| ihrachyshka | ajo, ok I think I know why it fails | 11:46 |
| ajo | ihrachyshka++ | 11:46 |
| ihrachyshka | see _compare_foreign_keys in alembic/autogenerate/compare.py | 11:46 |
| * ajo looks | 11:47 | |
| ihrachyshka | if I read it correctly, it only applies to new and removed constraints | 11:47 |
| ajo | ihrachyshka: ahaa | 11:48 |
| *** amuller has joined #openstack-neutron | 11:48 | |
| ajo | I guess we will have to ask zzzeek if it's possible to incorporate a check for creation | 11:48 |
| ihrachyshka | ajo, yeap | 11:48 |
| ihrachyshka | jlibosva, ^^ re test_migrations validation | 11:49 |
| jschwarz | ajo, what can I do you for? | 11:50 |
| ajo | jschwarz: I was going to propose you this: https://bugs.launchpad.net/neutron/+bug/1485993 | 11:52 |
| openstack | Launchpad bug 1485993 in neutron "Tenants could potentially modify rules from not owned policies" [Undecided,New] - Assigned to yong sheng gong (gongysh) | 11:52 |
| ajo | but gongysh took it | 11:52 |
| *** jaypipes has joined #openstack-neutron | 11:53 | |
| jschwarz | :< | 11:53 |
| jschwarz | ajo, wasn't that your bug? | 11:53 |
| ajo | jschwarz, well, I reported it, but didn't take ownership | 11:54 |
| jschwarz | gah | 11:54 |
| jschwarz | IRC is gong haywire | 11:55 |
| jschwarz | restarting client | 11:55 |
| *** jschwarz has quit IRC | 11:55 | |
| *** diga has quit IRC | 11:55 | |
| *** jschwarz has joined #openstack-neutron | 11:55 | |
| *** irenab has quit IRC | 11:55 | |
| *** slaweq has quit IRC | 11:56 | |
| *** pc_m_ has joined #openstack-neutron | 11:57 | |
| *** saikrishna has quit IRC | 11:58 | |
| jschwarz | ajo, I remembered you really looked into fixing that | 11:59 |
| jschwarz | talking with salv and stuff about how the policy.json works | 11:59 |
| jschwarz | but oh well :) | 11:59 |
| ajo | jschwarz, yes, but different stuff, that's api/service handler related | 11:59 |
| ajo | some permission checks are leveraged to the business logic due to some policy.json limitations | 12:00 |
| jschwarz | ohh | 12:00 |
| jschwarz | so yeah, that would have been right up my ally | 12:00 |
| jschwarz | alley* | 12:00 |
| jschwarz | I'll add myself as a subscriber and be sure to followup on that fix | 12:00 |
| ajo | for example, when you attach a port to a qos_policy, we're not supposed to grab out the policy from the DB too, and check it at the time policy.json is checked... | 12:00 |
| ajo | so we do it later in the neutron objects, when grabbing the policy | 12:00 |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Added initial devstack plugin https://review.openstack.org/214249 | 12:02 |
| jschwarz | ajo, I'll subscribe to a bunch of qos launchpad bugs and make sure I follow up on them | 12:02 |
| *** pc_m has quit IRC | 12:03 | |
| *** afazkas has quit IRC | 12:04 | |
| *** thiagop has joined #openstack-neutron | 12:06 | |
| *** dave-mccowan has joined #openstack-neutron | 12:06 | |
| *** mohankumar has joined #openstack-neutron | 12:06 | |
| *** julim has joined #openstack-neutron | 12:10 | |
| *** killer_prince has quit IRC | 12:10 | |
| ihrachyshka | ajo, ^^ I tried to enable qos in the plugin patch, let's see whether it actually does what we need. if so, I'll squash qos tests enablement patch there too | 12:12 |
| ajo | ihrachyshka: ack :) thanks | 12:13 |
| ihrachyshka | ajo, seems like we can do all we need in neutron tree these days | 12:13 |
| jschwarz | ihrachyshka++ | 12:13 |
| jschwarz | neutron-tree stuff ftw | 12:13 |
| openstackgerrit | Marga Millet proposed openstack/neutron: Support dhcp metadata service for all networks https://review.openstack.org/211963 | 12:14 |
| ajo | :) | 12:14 |
| *** marios_ has joined #openstack-neutron | 12:14 | |
| *** marios_ has quit IRC | 12:15 | |
| *** Ramanjaneya has joined #openstack-neutron | 12:16 | |
| ihrachyshka | akamyshnikova, I see the ironic depends-on patch succeeded with your patch | 12:17 |
| *** amotoki_ has joined #openstack-neutron | 12:17 | |
| akamyshnikova | ihrachyshka, great! | 12:17 |
| ihrachyshka | lucas-hungry, ^^ | 12:17 |
| jlibosva | ihrachyshka: In settings, we default to q-qos but not any agent extension. I don't know how plugins work yet, so maybe the question doesn't make sense. But if we will devstack with defaults, does it mean server will run with qos while agent will run without qos extension? | 12:19 |
| jlibosva | ihrachyshka: talking about devstack plugin ... | 12:19 |
| ihrachyshka | jlibosva, q-qos enablement will add the extension and put it in the conf file. I think it's just that you can define some additional extensions from your localrc before you call to enable_plugin for neutron | 12:20 |
| *** nmagnezi_ has joined #openstack-neutron | 12:21 | |
| ihrachyshka | so if L2_AGENT_EXTENSIONS is already provided by localrc, we don't reset the list | 12:21 |
| *** nmagnezi_ has quit IRC | 12:21 | |
| ihrachyshka | I wonder whether we need to have Q_ prefix for all knobs.. | 12:21 |
| jlibosva | ihrachyshka: right, but what if L2_AGENT_EXTENSIONS is not provided but we still enable q-qos by default | 12:21 |
| ihrachyshka | jlibosva, the settings makes sure it's enabled | 12:23 |
| ihrachyshka | and then we only configure qos extension | 12:24 |
| openstackgerrit | John Davidge proposed openstack/neutron: L3 agent changes and reference implementation for IPv6 PD https://review.openstack.org/185977 | 12:24 |
| *** salv-orlando has joined #openstack-neutron | 12:25 | |
| openstackgerrit | Paul Michali proposed openstack/python-neutronclient: Enable VPN test cases https://review.openstack.org/214587 | 12:26 |
| *** matrohon has joined #openstack-neutron | 12:26 | |
| *** chlong has joined #openstack-neutron | 12:26 | |
| *** yanxb_ has quit IRC | 12:27 | |
| *** piyanai has quit IRC | 12:27 | |
| *** amotoki_ has quit IRC | 12:27 | |
| *** edmondsw has joined #openstack-neutron | 12:27 | |
| *** britthouser has joined #openstack-neutron | 12:28 | |
| *** jpena is now known as jpena|lunch | 12:30 | |
| *** yanxb_ has joined #openstack-neutron | 12:31 | |
| *** piyanai has joined #openstack-neutron | 12:31 | |
| *** yamamoto has quit IRC | 12:32 | |
| *** salv-orl_ has joined #openstack-neutron | 12:32 | |
| *** salv-orlando has quit IRC | 12:33 | |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Fix query in get_reservations_for_resources https://review.openstack.org/214564 | 12:33 |
| *** bharath has joined #openstack-neutron | 12:33 | |
| *** mwagner_lap has joined #openstack-neutron | 12:33 | |
| *** kawa2014 has quit IRC | 12:35 | |
| ihrachyshka | akamyshnikova, some commit message nits for ^^ | 12:35 |
| *** erlarese has joined #openstack-neutron | 12:37 | |
| akamyshnikova | ihrachyshka, thanks! about what link? | 12:37 |
| *** bharath has quit IRC | 12:38 | |
| ihrachyshka | akamyshnikova, http://www.postgresql.org/message-id/200402271700.28133.dev@archonet.com ? | 12:38 |
| openstackgerrit | Paul Michali proposed openstack/python-neutronclient: Enable VPN test cases https://review.openstack.org/214587 | 12:38 |
| akamyshnikova | ihrachyshka, that one, thanks! | 12:38 |
| *** lucas-hungry is now known as lucasagomes | 12:39 | |
| lucasagomes | ihrachyshka, akamyshnikova great!!! Thanks | 12:39 |
| *** logan2 has joined #openstack-neutron | 12:39 | |
| *** haigang has quit IRC | 12:39 | |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron: Fix query in get_reservations_for_resources https://review.openstack.org/214564 | 12:40 |
| *** markvoelker has joined #openstack-neutron | 12:40 | |
| *** julim has quit IRC | 12:41 | |
| *** erlarese has quit IRC | 12:42 | |
| *** erlarese has joined #openstack-neutron | 12:42 | |
| *** julim has joined #openstack-neutron | 12:42 | |
| ihrachyshka | HenryG, are you already in you GateMan costume? | 12:42 |
| *** yamamoto has joined #openstack-neutron | 12:43 | |
| ihrachyshka | *your | 12:43 |
| ihrachyshka | HenryG, we broke ironic and they seem to feel better with https://review.openstack.org/#/c/214564/ | 12:43 |
| ihrachyshka | yamamoto, ^^ | 12:43 |
| *** tfukushima has joined #openstack-neutron | 12:43 | |
| * HenryG steps into phone booth | 12:43 | |
| *** gildub has joined #openstack-neutron | 12:43 | |
| *** GateMan has joined #openstack-neutron | 12:45 | |
| * GateMan emerges from phonebooth | 12:45 | |
| *** liudong has quit IRC | 12:46 | |
| *** fawadkhaliq has quit IRC | 12:46 | |
| salv-orl_ | HenryG: are you some sort of superhero now? | 12:47 |
| *** amotoki_ has joined #openstack-neutron | 12:47 | |
| HenryG | salv-orl_: Me? No. That's GateMan | 12:47 |
| *** yamamoto has quit IRC | 12:47 | |
| *** tfukushima has quit IRC | 12:48 | |
| salv-orl_ | ihrachyshka: so the revert did not work for ironic? | 12:48 |
| ihrachyshka | salv-orl_, apparently not, they got the same trace | 12:48 |
| *** kawa2014 has joined #openstack-neutron | 12:49 | |
| ihrachyshka | salv-orl_, it does not seem like it was a revert really, no? | 12:49 |
| ihrachyshka | salv-orl_, more like disabling | 12:49 |
| *** rpothier has joined #openstack-neutron | 12:49 | |
| salv-orl_ | ihrachyshka: indeed I wonder how do you get a failure from some code you're not calling | 12:49 |
| salv-orl_ | anyway | 12:49 |
| ihrachyshka | salv-orl_, trace is there, you can check | 12:49 |
| salv-orl_ | ihrachyshka: I see. I am apparently too dumb to fix anything nowaydays | 12:51 |
| salv-orl_ | I need to checkin into some elder people house | 12:51 |
| *** GateMan has left #openstack-neutron | 12:52 | |
| *** pal has joined #openstack-neutron | 12:52 | |
| ihrachyshka | salv-orl_, they say it's good to move into management after 40s or smth! | 12:52 |
| salv-orl_ | ihrachyshka: I need to look into something like that. | 12:53 |
| ihrachyshka | salv-orl_, yeah, that's how they call elder people houses for IT folks these days. | 12:53 |
| ihrachyshka | anyway, I seem to step into unsafe area ;) | 12:54 |
| *** lyanchih has joined #openstack-neutron | 12:54 | |
| HenryG | salv-orl_: Are you going to submit a new patch to avoid calling get_reservations_for_resources? Or something like that? | 12:56 |
| HenryG | salv-orl_: Or are we safe now with akamyshnikova's fix? | 12:56 |
| salv-orl_ | HenryG: we should be safe with the patch from akamyshnikova, because that's the only place where grouping qqueries are made. But for the other issues obverved with reservations and pymysql I'm going to push a patch anyway | 12:58 |
| HenryG | salv-orl_: OK | 12:59 |
| salv-orl_ | ihrachyshka: very unsafe. Especially if your manager is over 40 | 13:01 |
| *** tiny-hands has joined #openstack-neutron | 13:01 | |
| *** irenab has joined #openstack-neutron | 13:01 | |
| *** erkules_ is now known as erkules | 13:01 | |
| *** erkules has joined #openstack-neutron | 13:01 | |
| ihrachyshka | salv-orl_, she is not, so that's a relief! | 13:02 |
| ihrachyshka | HenryG, that won't fix the world, but at least ironic | 13:02 |
| salv-orl_ | ihrachyshka: why, is there any other outstanding gate issue? | 13:03 |
| HenryG | salv-orl_: rally | 13:04 |
| ihrachyshka | salv-orl_, nah I mean it won't fix pymysql issues. | 13:04 |
| HenryG | salv-orl_: But that is not due to reservations | 13:04 |
| *** lindycoder has joined #openstack-neutron | 13:06 | |
| salv-orl_ | HenryG: yeah I know the default sec group | 13:06 |
| salv-orl_ | one of my teammates is trying to help | 13:06 |
| salv-orl_ | HenryG: I disabled reservations because they crashed rally indeed. Before the ironic issue was reported. | 13:07 |
| ajo | amuller | 13:07 |
| amuller | ajo | 13:08 |
| salv-orl_ | so at least you have one less issue to chase while I deal with that in my local environment | 13:08 |
| ajo | about your qos etherpad comment of not being verbose enough: | 13:08 |
| ajo | neutron qos-bandwidth-limit-rule-create B --max_kbps 100 | 13:08 |
| ajo | Failed to create a duplicate QosBandwidthLimitRule object for fields {'max_kbps': 100, 'qos_policy_id': '2ec2b6e2-8427-4ffd-8ab1-f4a5d577e49b', 'max_burst_kbps': 0, 'id': 'f59e75dc-6e36-4a63-99f3-a84622bca7c6'} | 13:08 |
| ajo | does that sound better? | 13:08 |
| ajo | ihrachyshka ^ ? | 13:08 |
| ajo | I guess it's more similar to other duplication/conflict messages | 13:08 |
| amuller | that sounds better | 13:08 |
| *** kawa2014 has quit IRC | 13:08 | |
| HenryG | salv-orl_: yes, thanks | 13:08 |
| ihrachyshka | indeed it does | 13:08 |
| *** kawa2014 has joined #openstack-neutron | 13:09 | |
| HenryG | salv-orl_: The default SG issue has been around for a long time (since the beginning?) - it would be good to figure it out | 13:10 |
| *** DWfuturetec has quit IRC | 13:10 | |
| HenryG | salv-orl_: Last time I looked into the SG usages of DB I got very lost | 13:10 |
| salv-orl_ | HenryG: yes | 13:11 |
| salv-orl_ | historical memory is my only value in this project now ;) | 13:11 |
| ajo | HenryG, salv-orl_ , which issue? | 13:11 |
| *** trinaths has quit IRC | 13:11 | |
| salv-orl_ | I am a sort of augmented git history | 13:11 |
| openstackgerrit | Salvatore Orlando proposed openstack/neutron: Do not query reservations table when counting resources https://review.openstack.org/214602 | 13:11 |
| ajo | lol salv-orl_ | 13:11 |
| *** kbringard has joined #openstack-neutron | 13:12 | |
| ajo | salv-orl_: a neural network addition to git history, with the ability to talk via IRC | 13:12 |
| salv-orl_ | yeah, but if you want we can add another git hook so that when you do git log --with-human-history a bot will ping me on irc | 13:13 |
| *** pal has quit IRC | 13:13 | |
| openstackgerrit | Neil Jerram proposed openstack/neutron: DHCP agent: allow using gateway IPs instead of uniquely allocated https://review.openstack.org/206078 | 13:13 |
| salv-orl_ | HenryG, the patch above is the other one we were talking about. | 13:13 |
| *** annegentle has joined #openstack-neutron | 13:13 | |
| HenryG | salv-orl_: ack, looking | 13:13 |
| *** pal has joined #openstack-neutron | 13:14 | |
| amuller | salv-orl_: so you're both a neutron plugin and a git history implementation :) | 13:14 |
| *** banix has joined #openstack-neutron | 13:14 | |
| ajo | HenryG, salv-orl_ , what's the issue around SG & DB ? | 13:14 |
| *** pc_m_ has quit IRC | 13:15 | |
| *** pc_m has joined #openstack-neutron | 13:15 | |
| HenryG | ajo: take a look at the q-svc log for any recent rally job | 13:15 |
| *** tiswanso has joined #openstack-neutron | 13:15 | |
| salv-orl_ | ajo: it seems that pymysql don't like it - triggering a lock wait timeout | 13:16 |
| ajo | ouch | 13:16 |
| salv-orl_ | and the situation if then exacerbate by having multiple workers | 13:16 |
| salv-orl_ | as one worker holds the lock and causes the others to time out as well | 13:16 |
| salv-orl_ | so your rally log becomes redder than a ripe tomato | 13:17 |
| salv-orl_ | anyway, talking about logs | 13:17 |
| salv-orl_ | have you noticed not all log lines report PID numbers? Is there somebody already looking at that? | 13:17 |
| salv-orl_ | with multiple workers we really need PID numbers on every line | 13:17 |
| openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Make NeutronObjectDuplicateEntry exception more verbose https://review.openstack.org/214608 | 13:18 |
| ajo | no, I haven't noticed, but it makes sense, otherwise it's very difficult to trace anything | 13:18 |
| ajo | definitely we need to find out what's happening with the SGs DB... | 13:19 |
| *** jckasper has quit IRC | 13:20 | |
| *** pal has quit IRC | 13:20 | |
| ajo | salv-orl_, do we have anything to get oslo_log to punch a PID on every log line ? | 13:20 |
| salv-orl_ | ajo: abhishek from my team is spending some time on it. He started yesterday, I hope he might have some more details today | 13:21 |
| salv-orl_ | ajo: i am looking | 13:21 |
| ajo | nice | 13:21 |
| openstackgerrit | Mohankumar proposed openstack/python-neutronclient: Adding registration interface for non_admin_status_resources https://review.openstack.org/214193 | 13:21 |
| ajo | oslo_log.log.set_defaults(logging_context_format_string=None, default_log_levels=None) ? | 13:22 |
| ajo | ok, abhishek will tell us soon :) | 13:22 |
| ajo | amuller, ihrachyshka : https://review.openstack.org/#/c/214608/1/neutron/objects/base.py | 13:25 |
| ajo | I guess the unit test we have for forcing a duplicate entry exercises it, so I didn't find change detecting at the other side the final message | 13:25 |
| ajo | useful | 13:26 |
| salv-orl_ | ajo: for logging it seems that only some loggers report the pid | 13:27 |
| salv-orl_ | eg.: neutron.wsgi | 13:27 |
| *** roeyc has quit IRC | 13:28 | |
| ajo | salv-orl_: how does neutron.wsgi report the pid? | 13:28 |
| ajo | I see a normal logging.getLogger(__name__) , what do you mean? | 13:28 |
| *** dims_ has quit IRC | 13:29 | |
| *** tfukushima has joined #openstack-neutron | 13:29 | |
| *** dims has joined #openstack-neutron | 13:29 | |
| salv-orl_ | ajo: I'm trying to find that out as well, this is what I see from upstream job logs | 13:30 |
| *** vikram has quit IRC | 13:30 | |
| ajo | salv-orl_, let's let abhishek do his work, we're too curious ;D | 13:31 |
| ajo | ajo-- | 13:31 |
| *** mkolesni_ has quit IRC | 13:31 | |
| *** prad_ has joined #openstack-neutron | 13:32 | |
| salv-orl_ | abhishek is looking at the db issue. I noticed the log issue yesterday and was wondering if anybody has a clue already | 13:33 |
| *** Ramanjaneya has quit IRC | 13:33 | |
| ihrachyshka | sc68cal, anything that blocks us from merging https://review.openstack.org/#/c/212811/ ? | 13:34 |
| *** pal has joined #openstack-neutron | 13:35 | |
| *** roeyc has joined #openstack-neutron | 13:36 | |
| sc68cal | looks like SridharK just did | 13:37 |
| *** annegentle has quit IRC | 13:37 | |
| HenryG | salv-orl_: something like this again? https://review.openstack.org/160112 | 13:38 |
| salv-orl_ | ajo: it's just the logging_context_format_string option | 13:38 |
| openstackgerrit | Miguel Angel Ajo proposed openstack/neutron: Common l2_extension test code moved to the right directory https://review.openstack.org/214610 | 13:39 |
| *** jpena|lunch is now known as jpena | 13:40 | |
| salv-orl_ | HenryG: it's even simpler in this case. It seems it's just the log formatter not including the pid because we explicitly tell it to not do so. | 13:40 |
| openstackgerrit | Sandhya Dasu proposed openstack/neutron: Final decomposition of ML2 Cisco UCSM driver https://review.openstack.org/213668 | 13:40 |
| salv-orl_ | Only loggers which are configured before the configuraiton is parsed instead print the pid because they probably have a default formatter | 13:40 |
| ihrachyshka | sc68cal, cool, thanks! | 13:40 |
| ajo | salv-orl_: nice :) but where do we tell it not to do it? | 13:41 |
| openstackgerrit | Paul Michali proposed openstack/neutron-vpnaas: Endpoint groups: Create API and database https://review.openstack.org/212692 | 13:41 |
| salv-orl_ | if you look at this bug https://bugs.launchpad.net/neutron/+bug/1433687 from ihrachyshka it's explictly set in devstack because of colorization | 13:42 |
| openstack | Launchpad bug 1433687 in devstack "devstack logs do not contain pid information for log messages with context" [Undecided,In progress] - Assigned to Ihar Hrachyshka (ihar-hrachyshka) | 13:42 |
| salv-orl_ | guess who did colorization for neutron? | 13:42 |
| ihrachyshka | salv-orl_, ajo so there is a bug to make us reuse the default format string that includes the pid | 13:42 |
| ihrachyshka | oh man, I am too slow. feeling so redundant these days. | 13:42 |
| ajo | ihrachyshka: management | 13:42 |
| *** kiran-r has quit IRC | 13:42 | |
| ajo | :P | 13:42 |
| ajo | :) | 13:42 |
| ajo | ;-) | 13:42 |
| * ajo reuses jokes... | 13:43 | |
| ihrachyshka | oh yeah, definitely. but I think this space is used by amuller now! | 13:43 |
| ajo | this ball keeps bouncing, you never know where it's going to end.. | 13:43 |
| ihrachyshka | don't scare me with my future | 13:44 |
| ajo | lol ':) | 13:44 |
| *** lazy_prince has joined #openstack-neutron | 13:44 | |
| akamyshnikova | pc_m, hi | 13:44 |
| salv-orl_ | ihrachyshka: https://review.openstack.org/#/c/172509/ should show again the PID in neutron then | 13:45 |
| *** tkatarki has joined #openstack-neutron | 13:45 | |
| pc_m | akamyshnikova: hi | 13:45 |
| jlibosva | jschwarz: do you plan to update https://review.openstack.org/#/c/210830/4 or can I steal it from you? | 13:46 |
| *** mohankumar has quit IRC | 13:46 | |
| *** yamamoto has joined #openstack-neutron | 13:46 | |
| akamyshnikova | pc_m, have you seen this bug that I filed yesterday https://bugs.launchpad.net/neutron/+bug/1486077? | 13:46 |
| openstack | Launchpad bug 1486077 in neutron "VPNAAS: dsvm-functional doesn't have installed database backends" [Undecided,New] | 13:46 |
| *** yamamoto has quit IRC | 13:46 | |
| *** yamamoto has joined #openstack-neutron | 13:46 | |
| *** yamamoto has quit IRC | 13:46 | |
| *** dboik has joined #openstack-neutron | 13:46 | |
| pc_m | akamyshnikova: no, not yet. Haven't got to doing reviews yet this morning. | 13:47 |
| ihrachyshka | salv-orl_, yeah, but it's a complex pile. to be able to do it, we need to make oslo.log to log user names and project names for nova and others too | 13:47 |
| ajo | jschwarz: https://review.openstack.org/#/c/214610/ | 13:47 |
| ajo | :-) | 13:47 |
| ihrachyshka | and then this bug starts to interact: https://bugs.launchpad.net/neutron/+bug/1466476 | 13:47 |
| openstack | Launchpad bug 1466476 in neutron "Change of the default value of user_identity in the log format" [Undecided,Incomplete] - Assigned to Ihar Hrachyshka (ihar-hrachyshka) | 13:47 |
| ajo | you may like it | 13:47 |
| ajo | :) | 13:47 |
| jschwarz | jlibosva, feel free | 13:47 |
| *** yamamoto has joined #openstack-neutron | 13:47 | |
| ihrachyshka | and there, you can see a separate etherpad that me and dhellmann came up on how to solve it. it's not an easy bite. | 13:47 |
| jschwarz | jlibosva, though the comments on https://review.openstack.org/#/c/210830/4/neutron/tests/fullstack/resources/environment.py will take some thinking | 13:47 |
| jschwarz | ajo++ ++ | 13:48 |
| jschwarz | XD | 13:48 |
| salv-orl_ | ihrachyshka: I am looking indeed at the stack. But I think the complexity is about correctly represent the user info? | 13:48 |
| *** anilvenkata has left #openstack-neutron | 13:48 | |
| salv-orl_ | perhaps we can just take the pid bits out and start merging them? | 13:48 |
| pc_m | akamyshnikova: What does that test do? | 13:48 |
| ihrachyshka | salv-orl_, yeah, we can probably just patch the existing devstack strings with pid for now and think about getting rid of the option rewrite in devstack later. | 13:49 |
| *** killer_prince has joined #openstack-neutron | 13:49 | |
| akamyshnikova | pc_m, check that result of running migrations equal with models. We have it in Neutron, and this test will be implemented sub-projects, I implement it in VNPaaS for example | 13:50 |
| pc_m | akamyshnikova: OK, so are you saying that the functional jobs are not setting up the databases? | 13:51 |
| akamyshnikova | pc_m, yes | 13:51 |
| *** dane_leblanc has joined #openstack-neutron | 13:52 | |
| pc_m | akamyshnikova: The functional tests for VPN do NOT do a stack.sh (run devstack), they only configure devstack. Not sure if that is the same as what Neutron does. | 13:52 |
| *** lazy_prince has quit IRC | 13:52 | |
| akamyshnikova | pc_m, can be added installing mysql and postgres? | 13:54 |
| pc_m | akamyshnikova: I don't see why not. It may be worth while seeing what exactly Neutron does for the test setup, and then see if VPN is doing similar steps and if so, what is missing. | 13:55 |
| salv-orl_ | ihrachyshka: if that helps logging, why not. Even I should be able to do that | 13:55 |
| *** yamamoto has quit IRC | 13:56 | |
| *** prad_ is now known as pradk | 13:56 | |
| *** yamamoto has joined #openstack-neutron | 13:56 | |
| *** yamamoto has quit IRC | 13:56 | |
| *** sdake has joined #openstack-neutron | 13:56 | |
| *** yamamoto has joined #openstack-neutron | 13:57 | |
| *** nate_gone is now known as njohnston | 13:57 | |
| akamyshnikova | pc_m, in logs of gate-neutron-vpnaas-dsvm-functional I see that there are mysql.txt.gz and postgres.txt.gz | 13:57 |
| *** sdake_ has joined #openstack-neutron | 13:58 | |
| ihrachyshka | salv-orl_, come on... you should already have an army of minions to make such silly changes while you do the real breakage worth your status | 13:58 |
| akamyshnikova | pc_m, I'll try to find time to look deeper here, as not installing backends was not done intentionally. | 13:59 |
| *** dane_leblanc has quit IRC | 13:59 | |
| *** rubasov has quit IRC | 14:00 | |
| pc_m | akamyshnikova: great. The desire (from infra and others) for VPN was to not make tests heavy and do a full stacking of devstack. Instead, the minimal was done, so that the test could run. | 14:01 |
| pc_m | akamyshnikova: It's quite possible some needed setup is missing. | 14:01 |
| salv-orl_ | kevinbenton: when do you reckon you might be able to look at the pecan feature work | 14:01 |
| *** erlarese has quit IRC | 14:01 | |
| *** sdake has quit IRC | 14:01 | |
| *** dboik has quit IRC | 14:02 | |
| *** gildub has quit IRC | 14:02 | |
| ajo | ihrachyshka: do we have meeting today? | 14:02 |
| akamyshnikova | pc_m, I looked in console and find this http://logs.openstack.org/43/209943/4/check/gate-neutron-vpnaas-dsvm-functional/e355a1f/console.html#_2015-08-17_11_46_10_191 and http://logs.openstack.org/43/209943/4/check/gate-neutron-vpnaas-dsvm-functional/e355a1f/console.html#_2015-08-17_11_46_19_974 | 14:02 |
| *** yamamoto has quit IRC | 14:03 | |
| ajo | irenab, ihrachyshka , moshele , jlibosva , jschwarz #openstack-meeting-3 ? | 14:03 |
| irenab | ajo: sure | 14:03 |
| *** dane_leblanc has joined #openstack-neutron | 14:03 | |
| *** pal has quit IRC | 14:04 | |
| ihrachyshka | ajo, ouch | 14:04 |
| *** piyanai has quit IRC | 14:04 | |
| *** phil_h has joined #openstack-neutron | 14:04 | |
| pc_m | akamyshnikova: So seems like it is setting them up | 14:05 |
| *** lukasa has quit IRC | 14:06 | |
| akamyshnikova | pc_m, yes... may be I missed something when I implement test, will check this, thanks for your time! | 14:06 |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron-vpnaas: Test change for template of test_migrations in for out-of-tree plugin/driver https://review.openstack.org/209943 | 14:07 |
| *** kbyrne has quit IRC | 14:07 | |
| pc_m | akamyshnikova: Sure. Not sure I helped much :) | 14:07 |
| *** timbyr_ has quit IRC | 14:07 | |
| *** ajmiller has joined #openstack-neutron | 14:08 | |
| *** jckasper has joined #openstack-neutron | 14:08 | |
| *** shoutm has quit IRC | 14:10 | |
| *** killer_prince is now known as lazy_prince | 14:11 | |
| *** kbyrne has joined #openstack-neutron | 14:11 | |
| *** roeyc has quit IRC | 14:11 | |
| openstackgerrit | Brent Eagles proposed openstack/neutron-vpnaas: Manage cleanup of .ctl/.pid files for LibreSwan https://review.openstack.org/212673 | 14:12 |
| *** sdake_ is now known as sdake | 14:13 | |
| sc68cal | Hey, does anyone know how the names for 3rd party CI jobs are defined? I crap my pants every time I see something like this in gertty - http://i.imgur.com/FCXMUQ3.png - because the Cisco FwaaS ci and VPNaaS CI jobs are just showing up as "fwaas" and "vpnaas" respectively | 14:15 |
| *** thumpba has joined #openstack-neutron | 14:15 | |
| *** boris-42 has joined #openstack-neutron | 14:15 | |
| *** shwetaap has joined #openstack-neutron | 14:16 | |
| *** garyk has quit IRC | 14:17 | |
| *** busterswt has joined #openstack-neutron | 14:17 | |
| salv-orl_ | sc68cal: they look scary in gertty because the names have been "designed" for the gerrit UI | 14:18 |
| salv-orl_ | afaict there are no official guidelines for 3rd party job names | 14:18 |
| salv-orl_ | when you post a vote you also post the name of the job that's voting | 14:19 |
| *** thiagop has left #openstack-neutron | 14:20 | |
| *** pavel_bondar has joined #openstack-neutron | 14:21 | |
| *** bharath has joined #openstack-neutron | 14:22 | |
| *** edmondsw has quit IRC | 14:22 | |
| sc68cal | salv-orl_: ah. guess I'll have to write to the ci owners and ask | 14:22 |
| *** absubram has joined #openstack-neutron | 14:23 | |
| *** dave-mccowan has quit IRC | 14:24 | |
| ihrachyshka | sc68cal, it may be easier to make gertty to show it so that you don't mess with your clothes. it's one place to fix instead of multiple external | 14:24 |
| *** bharath has quit IRC | 14:26 | |
| sc68cal | ihrachyshka: good point | 14:26 |
| *** yamamoto has joined #openstack-neutron | 14:27 | |
| *** yamamoto has quit IRC | 14:27 | |
| *** yamamoto has joined #openstack-neutron | 14:28 | |
| *** lazvon has joined #openstack-neutron | 14:29 | |
| *** vivek-ebay has joined #openstack-neutron | 14:29 | |
| *** dmartls1 has joined #openstack-neutron | 14:33 | |
| *** Murali has quit IRC | 14:34 | |
| matrohon | salv-orl_: hi. I've tried to explain the bgpvpn context on the ML. thanks for your answer. | 14:35 |
| *** thurloat_isgone is now known as thurloat | 14:35 | |
| ihrachyshka | matrohon, re your policy.d bug... it does not look an easy thing, I will need to rework base test class to use oslo.config fixture I guess. | 14:36 |
| *** edmondsw has joined #openstack-neutron | 14:37 | |
| matrohon | ihrachyshka : thanks for handling it! | 14:37 |
| *** crose has joined #openstack-neutron | 14:37 | |
| *** dave-mccowan has joined #openstack-neutron | 14:38 | |
| openstackgerrit | Pengtao Huang proposed openstack/neutron: add a default value for the variable core_plugin https://review.openstack.org/214383 | 14:41 |
| *** sdake_ has joined #openstack-neutron | 14:41 | |
| *** jprovazn has quit IRC | 14:42 | |
| *** zzzeek has joined #openstack-neutron | 14:43 | |
| *** garyk has joined #openstack-neutron | 14:44 | |
| *** sdake has quit IRC | 14:44 | |
| openstackgerrit | Marga Millet proposed openstack/neutron: Support dhcp metadata service for all networks https://review.openstack.org/211963 | 14:44 |
| *** cdl has quit IRC | 14:44 | |
| openstackgerrit | Ann Kamyshnikova proposed openstack/neutron-vpnaas: Test change for template of test_migrations in for out-of-tree plugin/driver https://review.openstack.org/209943 | 14:44 |
| *** Murali has joined #openstack-neutron | 14:46 | |
| *** annegentle has joined #openstack-neutron | 14:46 | |
| *** annegentle has quit IRC | 14:47 | |
| *** busterswt has quit IRC | 14:47 | |
| *** annegentle has joined #openstack-neutron | 14:47 | |
| *** diga has joined #openstack-neutron | 14:49 | |
| ihrachyshka | pc_m, hey | 14:50 |
| ihrachyshka | re gate hooks | 14:50 |
| pc_m | ihrachyshka: ho | 14:50 |
| ihrachyshka | pc_m, for qos, I look into enabling the qos plugin for (some) neutron tempest jobs | 14:50 |
| *** gangil has quit IRC | 14:50 | |
| *** pal has joined #openstack-neutron | 14:50 | |
| * pc_m slow to respond as in meeting. | 14:51 | |
| ihrachyshka | pc_m, so I added enable_plugin in gate_hook.sh in neutron tree, and it works for api | 14:51 |
| ihrachyshka | pc_m, but I see that in jenkins/jobs/devstack-gate.yaml, the gate_hook is not triggered, so no plugin enabled for tempest | 14:51 |
| ihrachyshka | am I right that to utilize the hook, I need to modify gate jobs in project-config to use the hook for tempest? | 14:52 |
| *** phil_h has quit IRC | 14:52 | |
| *** fedexo has joined #openstack-neutron | 14:53 | |
| openstackgerrit | Brent Eagles proposed openstack/neutron-vpnaas: Manage cleanup of .ctl/.pid files for LibreSwan https://review.openstack.org/212673 | 14:54 |
| openstackgerrit | Ihar Hrachyshka proposed openstack/neutron: Added initial devstack plugin https://review.openstack.org/214249 | 14:54 |
| ihrachyshka | ajo, that one should enable qos api tests ^^ no tempest so far. | 14:55 |
| *** regXboi has joined #openstack-neutron | 14:55 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Make DHCP agent scheduler physical_network aware https://review.openstack.org/205631 | 14:56 |
| *** Murali has quit IRC | 14:56 | |
| *** emagana has quit IRC | 14:57 | |
| pc_m | ihrachyshka: Most of the devstack based jobs will end up invoking safe-devstack-vm-gate-wrap.sh to do the stacking | 14:57 |
| *** yapeng has joined #openstack-neutron | 14:58 | |
| pc_m | this script will either run a default stack, or, if it exists, run the repo's gate-hook (which needs to do setup and then run the same script to stack). | 14:58 |
| *** yapeng has left #openstack-neutron | 14:59 | |
| pc_m | ihrachyshka: which particular job template are you referring to? | 14:59 |
| pc_m | so there is context. | 14:59 |
| *** vivek-ebay has quit IRC | 14:59 | |
| ihrachyshka | pc_m, f.e. I don't see hook triggered at https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/devstack-gate.yaml#L414 | 15:00 |
| ihrachyshka | it's only in jenkins/jobs/neutron.yaml that we trigger it | 15:01 |
| ihrachyshka | for fullstack ,api, functional | 15:01 |
| *** dims has quit IRC | 15:01 | |
| ihrachyshka | but that's all | 15:01 |
| *** logan2 has quit IRC | 15:01 | |
| *** dims has joined #openstack-neutron | 15:01 | |
| *** agireud has joined #openstack-neutron | 15:02 | |
| * ajo runs and cooks | 15:02 | |
| pc_m | Right, so that test is not setup for doing any hooks (gate or post-test) | 15:02 |
| *** yfried|afk has joined #openstack-neutron | 15:03 | |
| jschwarz | obondarev, ping, re: https://review.openstack.org/#/c/199032/ | 15:03 |
| jschwarz | I think it introduced a regression in snat scheduling | 15:03 |
| obondarev | jschwarz: pong | 15:03 |
| obondarev | jschwarz: ? | 15:04 |
| pc_m | ihrachyshka: so you are looking to have QoS enabled for the tempest run? | 15:04 |
| jschwarz | obondarev, so if you create a router, attach it to some tenant network, and only then provide it a gateway interface - _get_candidates() will return an empty list | 15:04 |
| ihrachyshka | pc_m, so the proper order would be to make the hook just call to devstack-gate script, then update jobs to trigger the hook, and then we can mess with hook-based configuration however we want | 15:05 |
| *** dims_ has joined #openstack-neutron | 15:05 | |
| ihrachyshka | pc_m, yes, that's the goal | 15:05 |
| jschwarz | obondarev, then _schedule_router won't call plugin.schedule_snat_router() | 15:05 |
| *** klindgren has joined #openstack-neutron | 15:05 | |
| *** logan2 has joined #openstack-neutron | 15:05 | |
| *** ajmiller_ has joined #openstack-neutron | 15:05 | |
| pc_m | ihrachyshka: You may want to consult with infra folks on the best path forward. For separate repos, like VPN, we've been making repo specific jobs to test things (VPN specific functional tests, someday grenade, and soon? API). | 15:05 |
| obondarev | jschwarz: interesting, let me check why _get_candidates() won't return anything | 15:06 |
| pc_m | ihrachyshka: In your case, QoS is part of neutron right? | 15:06 |
| ihrachyshka | pc_m, I believe we don't want to have more jobs in neutron gate | 15:06 |
| ihrachyshka | pc_m, it is | 15:06 |
| *** dims has quit IRC | 15:06 | |
| ihrachyshka | pc_m, so I assume it's the same as for metering | 15:06 |
| jschwarz | obondarev, okay. from my debugging it seems that it gets dropped at the 'candidates = list(new_l3agentset - old_l3agentset)' line | 15:06 |
| ihrachyshka | meaning, always enabled. | 15:06 |
| *** shaohe_feng has joined #openstack-neutron | 15:07 | |
| jschwarz | (both l3_agents and new_l3agents contain the same agents, no reference to whether snat is scheduled or not is done) | 15:07 |
| obondarev | jschwarz: is there a bug for this? | 15:07 |
| jschwarz | obondarev, not yet, will open one in a few minutes | 15:08 |
| pc_m | ihrachyshka: You are correct though, the process in general is to add a gate hook, have the job template use the hook, and have the hook do the needed enabling. However a few things... | 15:08 |
| jschwarz | wanted to hear what you thought first :) | 15:08 |
| obondarev | jschwarz: cool, thanks | 15:08 |
| pc_m | ihrachyshka: One is that I've done that by introducing new jobs (as experimental) to test out the new hook, so as not to interfere with existing jobs. | 15:08 |
| *** ajmiller has quit IRC | 15:09 | |
| ihrachyshka | pc_m, meh. what could go wrong, right?! :)) | 15:09 |
| ihrachyshka | could -> can | 15:09 |
| pc_m | ihrachyshka: Another is, I don't know how keen infra will be on modifying tempest job operation. They don't like to make gate-hooks conditional (as I was told on my latest changes). | 15:09 |
| ihrachyshka | pc_m, what do you mean? | 15:10 |
| ihrachyshka | I mean, re conditional | 15:10 |
| ihrachyshka | the VENV check in gate_hook? | 15:10 |
| pc_m | ihrachyshka: One way to handle hooks in the job template is to use it only if the hook file exists in the repo, if not, then don't. That is not desired, because it can lead to silent failures/passes, if hooks are missing/moved/etc. | 15:11 |
| ihrachyshka | pc_m, yeah, but why would I want to make it conditional? | 15:12 |
| *** mickeys has joined #openstack-neutron | 15:12 | |
| ihrachyshka | I will just first make sure the hook does no difference for all unknown VENVs, then get it on in gate, then specialize as I want | 15:12 |
| pc_m | As a result, you either use the hook or don't. | 15:13 |
| *** dannywilson has joined #openstack-neutron | 15:13 | |
| amuller | obondarev: jschwarz: The fix to the bug should include more comprehensive unit/functional tests for DVR scheduling. For example https://review.openstack.org/#/c/188157/ (added DVR scheduler functional tests) are all passing | 15:13 |
| pc_m | ihrachyshka: One way would be to add the conditional hook logic in job, and then add the hook in neutron to test. Since this is not desired. | 15:13 |
| pc_m | you have to create a new job. Otherwise you risk breaking things. | 15:14 |
| *** dannywilson has quit IRC | 15:14 | |
| *** dannywilson has joined #openstack-neutron | 15:14 | |
| pc_m | e.g. create hook (bad one), switch job to use hook (it would pass it's tests), find out it is broken next time someone commits. | 15:14 |
| pc_m | There's no way to have a "test" neutron commit use the new job with gate hook, until that upstreams. So... you have to create new job. | 15:15 |
| ihrachyshka | pc_m, pfff. that sucks | 15:16 |
| *** chadix has quit IRC | 15:16 | |
| *** agireud has quit IRC | 15:16 | |
| pc_m | ihrachyshka: Probably a realistic way would be to make the hook conditional, add it to neutron, test it out, then go back and make it unconditional. Not something infra wants though. | 15:17 |
| ihrachyshka | I need to talk to mestery I guess. I don't like the idea of separate jobs to test that a script is actually triggered. | 15:18 |
| *** coolsvap|away is now known as coolsvap | 15:18 | |
| ihrachyshka | but meh, maybe I haven't broken enough things yet | 15:18 |
| *** yamahata has joined #openstack-neutron | 15:18 | |
| pc_m | So, the alternative is... make the hook, create a new job that uses the hook (experimental), test it out with a dummy neutron fix (or run on any commit), make the job voting and get rid of the old job. | 15:18 |
| pc_m | this is what I'm doing with neutronclient use of VPN plugin. | 15:18 |
| *** ronis has quit IRC | 15:19 | |
| pc_m | ihrachyshka: 210021, 209887, and 214587. | 15:19 |
| *** pal has quit IRC | 15:19 | |
| obondarev | amuller: definitely the fix should be covered with a correct test | 15:19 |
| *** dims_ has quit IRC | 15:19 | |
| pc_m | It's really awkward, IMHO. | 15:19 |
| *** dims has joined #openstack-neutron | 15:19 | |
| ihrachyshka | ajo, ^^ I start to feel bad, physically, even considering I will need to mess with the infra configs to enable tempest qos | 15:19 |
| *** SridharG has left #openstack-neutron | 15:21 | |
| obondarev | jschwarz: I see the root cause, feel free to assign the bug to me | 15:21 |
| pc_m | ihrachyshka: come on! you can do it! We're right behind you :) | 15:21 |
| openstackgerrit | Salvatore Orlando proposed openstack/neutron: WIP: Fix reservation engine https://review.openstack.org/214660 | 15:22 |
| ihrachyshka | pc_m, ready to punch? :) | 15:22 |
| *** coolsvap is now known as coolsvap|away | 15:22 | |
| *** irenab has quit IRC | 15:22 | |
| *** mohankumar has joined #openstack-neutron | 15:22 | |
| *** daneyon has joined #openstack-neutron | 15:22 | |
| pc_m | ihrachyshka: ready to use you as a human shield :) | 15:22 |
| *** moshele has quit IRC | 15:23 | |
| ihrachyshka | pc_m, it sounds so boring, and I am bad at doing something that does not provide enough fun for my battery to run | 15:23 |
| pc_m | ihrachyshka: Take a look at the three reviews I posted above for the sequence of steps. | 15:23 |
| pc_m | ihrachyshka: boring until gate fails and then you go into panic mode. | 15:24 |
| ihrachyshka | pc_m, that's when the fun starts | 15:24 |
| pc_m | yup | 15:24 |
| ihrachyshka | you are in the center of attention! all spot on you! | 15:24 |
| pc_m | oh the joy... | 15:24 |
| openstackgerrit | Pengtao Huang proposed openstack/neutron: Use directly neutron.common.constants constants in l3_dvr_db https://review.openstack.org/213563 | 15:24 |
| ihrachyshka | which move will that poor guy make next? | 15:24 |
| *** wwriverrat has quit IRC | 15:24 | |
| *** yfried|afk has quit IRC | 15:25 | |
| *** mlavalle has joined #openstack-neutron | 15:25 | |
| *** daneyon_ has quit IRC | 15:26 | |
| *** jlibosva has quit IRC | 15:26 | |
| pc_m | ihrachyshka: stongly suggest giving infra a heads up on your plan, to make sure they are comfortable (and hopefully will gain some confidence in us). Right now, I think things are overly restrictive, because wounds are still fresh. | 15:26 |
| ihrachyshka | pc_m, I get it's wrong time to mess with infra :) | 15:27 |
| pc_m | yes | 15:27 |
| *** clev is now known as clev-away | 15:27 | |
| pc_m | don't poke the bear :) | 15:27 |
| ihrachyshka | maybe I'll need to discuss that with infra liaisons first | 15:27 |
| *** ajmiller_ is now known as ajmiller | 15:27 | |
| *** wwriverrat has joined #openstack-neutron | 15:27 | |
| *** jbell8 has joined #openstack-neutron | 15:27 | |
| ihrachyshka | now that we have them :) | 15:27 |
| openstackgerrit | Merged openstack/neutron: Support delegation of bind_port to networking-odl backend driver. https://review.openstack.org/205102 | 15:28 |
| *** wwriverrat has left #openstack-neutron | 15:28 | |
| *** ushkalim has quit IRC | 15:29 | |
| *** TrevorV has joined #openstack-neutron | 15:29 | |
| *** agireud has joined #openstack-neutron | 15:30 | |
| *** irenab has joined #openstack-neutron | 15:31 | |
| ihrachyshka | amuller, we need gate_hook to trigger on tempest jobs to enable qos there. just a heads up that we consider it, and I may have a plan ready for that in several days to discuss with infra liaisons and infra and all. | 15:31 |
| amuller | ihrachyshka: at the end of the day you're talking about adding 'qos' to the service_plugins list, adding 'qos' to the l2 agent extensions file, and adding 'qos' to the ml2 extensions list? | 15:33 |
| ihrachyshka | amuller, yes, but for that, I need to enable devstack plugin, and that is done in gate_hook | 15:33 |
| jschwarz | obondarev, out of interest - what's the root cause? | 15:33 |
| ihrachyshka | which is not triggered for tempest jobs | 15:34 |
| amuller | ihrachyshka: ok I understand | 15:34 |
| ihrachyshka | and pc_m told me it's a pain to make such changes in infra, especially now that we lost some trust | 15:34 |
| amuller | ihrachyshka: from my perspective this is all so stupid, but I understand infra's PoV also | 15:34 |
| ihrachyshka | the benefit of being at the core of gate | 15:34 |
| *** vivek-ebay has joined #openstack-neutron | 15:35 | |
| openstackgerrit | Merged openstack/neutron-vpnaas: Updated from global requirements https://review.openstack.org/212262 | 15:35 |
| *** britthouser has quit IRC | 15:35 | |
| obondarev | jschwarz: router was sheduled to all controllers but didn't have gw that time, so snat was not scheduled. Once router got external gw, get_candidates() returned None since router was already scheduled to all snat nodes | 15:36 |
| jschwarz | obondarev, https://bugs.launchpad.net/neutron/+bug/1486627 (can't assign it to someone else) | 15:36 |
| openstack | Launchpad bug 1486627 in neutron "DVR doesn't always schedule SNAT routers" [Undecided,New] | 15:36 |
| jschwarz | obondarev, aye. do you have a fix in mind for that? | 15:36 |
| *** pal has joined #openstack-neutron | 15:37 | |
| openstackgerrit | Sandhya Dasu proposed openstack/neutron: Final decomposition of ML2 Cisco UCSM driver https://review.openstack.org/213668 | 15:37 |
| obondarev | jschwarz: yeah, we just shouldn't return in case of no candidates in dvr router | 15:37 |
| obondarev | jschwarz: always need to check if snat was scheduled or not | 15:37 |
| jschwarz | obondarev, sounds like fun :) | 15:38 |
| *** klindgren has quit IRC | 15:38 | |
| *** changzhi has joined #openstack-neutron | 15:39 | |
| obondarev | jschwarz: so it's one of many confusions with dvr scheduling which whould be handled with https://review.openstack.org/#/c/175237/ | 15:40 |
| obondarev | jschwarz: only in M I guess | 15:40 |
| *** scheuran has joined #openstack-neutron | 15:40 | |
| *** Swami has joined #openstack-neutron | 15:41 | |
| obondarev | jschwarz: thanks for finding and reporting the bug :) | 15:41 |
| jschwarz | obondarev, M is a long way away me thinks. we probably want a fix beforehand (for L)? | 15:41 |
| jschwarz | obondarev, sure :) | 15:41 |
| *** minwang2 has joined #openstack-neutron | 15:41 | |
| jschwarz | seems to me pushing the 'if not candidates: return' part down to after the 'if router_distributed' bracket | 15:41 |
| obondarev | jschwarz: yeah, I will upload fix asap, the hardes part will be to write correct test :) | 15:41 |
| *** moha_hunt has quit IRC | 15:41 | |
| jschwarz | (that should fix it, I meant) | 15:42 |
| jschwarz | obondarev, I'm working on a fullstack test that will check that, but it depends on a bunch of other fullstack-infra patches that are still on the queue | 15:42 |
| jschwarz | so that should help in the long run | 15:42 |
| *** Piet has quit IRC | 15:42 | |
| obondarev | jschwarz: cool, good to know | 15:43 |
| openstackgerrit | Pengtao Huang proposed openstack/neutron: Check ICMP codes in range [0,255] https://review.openstack.org/212220 | 15:43 |
| *** pal has quit IRC | 15:44 | |
| *** busterswt has joined #openstack-neutron | 15:44 | |
| *** pal has joined #openstack-neutron | 15:46 | |
| *** emagana has joined #openstack-neutron | 15:46 | |
| *** numan has quit IRC | 15:48 | |
| *** lpetrut has quit IRC | 15:48 | |
| *** dims has quit IRC | 15:49 | |
| *** sdake_ is now known as sdake | 15:49 | |
| *** nmagnezi has quit IRC | 15:49 | |
| *** shaohe_feng1 has joined #openstack-neutron | 15:49 | |
| *** dims has joined #openstack-neutron | 15:49 | |
| *** scheuran has quit IRC | 15:50 | |
| *** minwang2 has quit IRC | 15:50 | |
| *** shaohe_feng has quit IRC | 15:51 | |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port https://review.openstack.org/194441 | 15:51 |
| jschwarz | amuller, are you planning on rebasing the fullstack patch chain? | 15:52 |
| amuller | jschwarz: naw, just the first patch | 15:52 |
| *** alex_klimov has quit IRC | 15:52 | |
| *** pc_m has quit IRC | 15:52 | |
| amuller | jschwarz: I didn't have a need to rebase the chain | 15:52 |
| *** kbringard has quit IRC | 15:52 | |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent https://review.openstack.org/194446 | 15:52 |
| amuller | ihrachyshka: did you see I updated the first patch in the series? | 15:52 |
| jschwarz | amuller, found a bug halfway through | 15:53 |
| *** pc_m has joined #openstack-neutron | 15:53 | |
| *** kbringard has joined #openstack-neutron | 15:53 | |
| ihrachyshka | amuller, nope. I am lagging on what I have in todo these days. :) | 15:53 |
| *** dims_ has joined #openstack-neutron | 15:53 | |
| ihrachyshka | :) -> :( | 15:53 |
| *** DWfuturetec has joined #openstack-neutron | 15:53 | |
| jschwarz | can't remember for the life of me what it was but I have it committed locally | 15:53 |
| amuller | ihrachyshka: https://review.openstack.org/#/c/191197/, kuba +1'd | 15:53 |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Cleanup the fip agent gateway port delete routines https://review.openstack.org/194455 | 15:53 |
| jschwarz | one of the fullstack tests fail because of an upstream change someplace | 15:53 |
| *** nlahouti has joined #openstack-neutron | 15:53 | |
| *** shaohe_feng1 has quit IRC | 15:53 | |
| amuller | jschwarz: with what fullstack patch? what test? | 15:54 |
| jschwarz | amuller, erm.. O.o | 15:54 |
| jschwarz | I'll need to check | 15:54 |
| jschwarz | one of the last 2 iirc | 15:54 |
| *** dims has quit IRC | 15:54 | |
| *** Sukhdev has joined #openstack-neutron | 15:54 | |
| amuller | jschwarz: the chain isn't rebased though | 15:54 |
| *** yapeng has joined #openstack-neutron | 15:55 | |
| ihrachyshka | amuller, ok, if kuba did it, looking | 15:55 |
| openstackgerrit | Elena Ezhova proposed openstack/neutron-vpnaas: [OpenSwan] Enable usage of the MTU value of an IPSec connection https://review.openstack.org/206539 | 15:55 |
| *** nlahouti has quit IRC | 15:55 | |
| jschwarz | amuller, if you made no changes to that code I can just rebase it myself and it will contain the fix ;-) | 15:55 |
| jschwarz | amuller, ah! the one with the local_ip stuff we talked about much earlier this week | 15:56 |
| jschwarz | where local_ip needs to be an actual IP of one of the interfaces... | 15:56 |
| *** kbringard has quit IRC | 15:57 | |
| amuller | ihrachyshka: From my pov I want to just start merging these patches, the chain is too long and there's many efforts riding on that series, and we want a lot of new patches for DVR, QoS... | 15:57 |
| *** lazvon has quit IRC | 15:57 | |
| amuller | ihrachyshka: we can polish it as we go, imo it doesn't have to be 'perfect', but that's just me | 15:57 |
| amuller | jschwarz: weren't you going to send a patch that adds an 'if' there? | 15:58 |
| jschwarz | amuller, https://review.openstack.org/#/c/190786/12/neutron/tests/fullstack/resources/environment.py should produce an actual local_ip if tunneling is not allowed | 15:58 |
| jschwarz | amuller, I was, got side-tracked with DVR ;-) | 15:58 |
| amuller | jschwarz: full stack shouldn't need any changes | 15:58 |
| amuller | jschwarz: for the local_ip stuff | 15:58 |
| amuller | jschwarz: the OVS agent should only perform that check if tunneling is enabled | 15:58 |
| *** TrevorV has quit IRC | 15:58 | |
| jschwarz | so fullstack will have to depend on that change until it's merged? | 15:59 |
| *** tidwellr has joined #openstack-neutron | 15:59 | |
| jschwarz | amuller, anyway I'll submit the patches soon(tm) | 15:59 |
| amuller | jschwarz: that change is a one liner? we can merge it today | 15:59 |
| *** nlahouti has joined #openstack-neutron | 15:59 | |
| jschwarz | you're optimistic ;-) | 15:59 |
| *** yapeng has quit IRC | 15:59 | |
| openstackgerrit | Elena Ezhova proposed openstack/neutron-vpnaas: [DO NOT MERGE] Test https://review.openstack.org/214688 | 15:59 |
| amuller | jschwarz: not at all | 15:59 |
| *** _cjones_ has joined #openstack-neutron | 16:00 | |
| jschwarz | you's with the +2 votes... | 16:00 |
| *** fedexo has quit IRC | 16:02 | |
| jschwarz | amuller, ahh, now I remembered | 16:02 |
| jschwarz | I need to rebase the fullstack patches anyway | 16:02 |
| jschwarz | to make sure I have the host_desc and env_desc at the last patch | 16:02 |
| jschwarz | so... | 16:02 |
| *** garyk1 has joined #openstack-neutron | 16:02 | |
| *** lyanchih has quit IRC | 16:02 | |
| *** moha_hunt has joined #openstack-neutron | 16:03 | |
| *** kiran-r has joined #openstack-neutron | 16:03 | |
| jschwarz | amuller, you mind if I do some rebasing for all the patches (except the first?) | 16:03 |
| *** garyk has quit IRC | 16:03 | |
| amuller | jschwarz: not at all... | 16:04 |
| *** kbringard has joined #openstack-neutron | 16:04 | |
| *** mestery has joined #openstack-neutron | 16:04 | |
| *** daneyon_ has joined #openstack-neutron | 16:05 | |
| *** jbell8_ has joined #openstack-neutron | 16:05 | |
| *** jbell8 has quit IRC | 16:05 | |
| *** lindycoder has quit IRC | 16:06 | |
| *** irenab has quit IRC | 16:07 | |
| *** clev-away is now known as clev | 16:07 | |
| *** fawadkhaliq has joined #openstack-neutron | 16:08 | |
| *** daneyon has quit IRC | 16:08 | |
| *** irenab has joined #openstack-neutron | 16:08 | |
| *** carl_baldwin has joined #openstack-neutron | 16:08 | |
| *** carl_baldwin has quit IRC | 16:08 | |
| *** Piet has joined #openstack-neutron | 16:08 | |
| *** klindgren has joined #openstack-neutron | 16:10 | |
| ihrachyshka | amuller, agreed. check my silly comments at https://review.openstack.org/#/c/191197/ and say whether you want to respin and get +2 or just +2 and a follow up | 16:10 |
| *** sthillma has joined #openstack-neutron | 16:10 | |
| *** bharath has joined #openstack-neutron | 16:10 | |
| *** busterswt has quit IRC | 16:12 | |
| *** ajmiller has quit IRC | 16:13 | |
| amuller | ihrachyshka: I'll push a new PS real quick | 16:14 |
| ihrachyshka | amuller, and I'll +2 real quick after | 16:14 |
| ihrachyshka | agreed we should get it in | 16:14 |
| *** bharath has quit IRC | 16:15 | |
| ihrachyshka | zzzeek, re https://review.openstack.org/214564 is it some sqla limitation? we were thinking that maybe sqla should do the right thing for pgsql. | 16:15 |
| *** ddepaoli has quit IRC | 16:15 | |
| *** mohankumar has quit IRC | 16:16 | |
| *** Guest95303 has quit IRC | 16:16 | |
| zzzeek | ihrachyshka: IMO the “right thing” is query(x, y, func(z)).group_by(x) emits “SELECT x, y, func(z) FROM .. GROUP BY x” , no ? | 16:16 |
| zzzeek | ihrachyshka: are you saying it shoudl implicitly add “y” to GROUP BY ? | 16:17 |
| *** cdl has joined #openstack-neutron | 16:17 | |
| *** kawa2014 has quit IRC | 16:17 | |
| ihrachyshka | zzzeek, if we know pgsql requires it, then why not | 16:17 |
| zzzeek | ihrachyshka: the SQL standard requires it. its MySQL that’s broken | 16:18 |
| zzzeek | ihrachyshka: SQLalchemy writes the SQL you give it, it would be disastrous if it attempted to guess intent | 16:18 |
| *** ajmiller has joined #openstack-neutron | 16:18 | |
| zzzeek | ihrachyshka: if that were supported, it woudl need to be a new method like “group_by_non_funcs()” or something | 16:19 |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Add a fullstack fake VM, basic connectivity test https://review.openstack.org/191197 | 16:19 |
| zzzeek | ihrachyshka: it would amke no sense if you said, “group_by(x)” and it emits “GROUP BY x, y” | 16:19 |
| amuller | ihrachyshka: ^ | 16:19 |
| salv-orl_ | zzzeek: I get your point | 16:19 |
| ihrachyshka | zzzeek, ok, whatever. I am not a guy to ask about sql details, if only to make a fool of him :) I just wondered that if some code works for one backend but not another then maybe there is space for consistency | 16:19 |
| *** ronis has joined #openstack-neutron | 16:19 | |
| *** amuller is now known as amuller_afk | 16:20 | |
| zzzeek | salv-orl_ / ihrachyshka : it is possible to make a method that does “group by” everything you’re selecting from that isn’t a funciton, the Query is introspectable in that regard | 16:20 |
| zzzeek | ihrachyshka: yes, there’s a MySQL setting that causes it to do standard GROUP BY rules | 16:20 |
| salv-orl_ | zzzeek: good point | 16:20 |
| *** lpetrut has joined #openstack-neutron | 16:21 | |
| ihrachyshka | amuller_afk, +2 | 16:21 |
| *** moshele has joined #openstack-neutron | 16:21 | |
| zzzeek | ihrachyshka: SQLite may or may not be silently passing on this too, not sure, guess it has to be if this method doesnt raise for a SQLite test suite ? | 16:21 |
| ihrachyshka | zzzeek, should we have it in gate? | 16:21 |
| ihrachyshka | zzzeek, well maybe it's not covered by sqlite tests ;) | 16:21 |
| zzzeek | ihrachyshka: you mean running this against SQLite ? | 16:21 |
| zzzeek | ihrachyshka: probably isnt’. I’d stick to MySQL/Postgresql | 16:21 |
| ihrachyshka | zzzeek, no, against mysql with std group_by | 16:21 |
| zzzeek | ihrachyshka: oh. HMM | 16:22 |
| zzzeek | ihrachyshka: im not even sure if we turn on that setting, we have a thing in oslo.db that sets some of this | 16:22 |
| *** pritesh has joined #openstack-neutron | 16:22 | |
| zzzeek | https://dev.mysql.com/doc/refman/5.0/en/sql-mode.html#sqlmode_only_full_group_by | 16:22 |
| zzzeek | ihrachyshka: so this is part ot SQL MODE | 16:22 |
| salv-orl_ | zzzeek: what would oslo_db need to do? I think ihrachyshka was referring to a configuration change for gare tests | 16:23 |
| zzzeek | salv-orl_: oslo.db sets up MySQL MODE on connect | 16:23 |
| ihrachyshka | salv-orl_, options are per connection | 16:23 |
| salv-orl_ | zzzeek: ah I thought this was something you'd set in my.cnf never mind | 16:23 |
| zzzeek | salv-orl_: you probably can but we do it client side | 16:23 |
| ihrachyshka | so if we would have it on in oslo.db, we would not allow it in neutron in the first place and wouldn't break ironic | 16:24 |
| zzzeek | salv-orl_: oslo.db is looking for either TRADITIONAL or STRICT_ALL_TABLES right now as presenet in SQL_MODE | 16:24 |
| zzzeek | salv-orl_: but i think it’s a bitflag kind of thing where ONLY_FULL_GROUP_BY could be added in | 16:24 |
| zzzeek | salv-orl_: though I woudl think just having Postgresql in the gate was well as MySQL is enough, the ONLY_FULL_GROUP_BY flag is something I never see anyone setting, only because mysql’s behavior is so widely accepted | 16:25 |
| zzzeek | salv-orl_: but its a client setting, so maybe its OK | 16:25 |
| salv-orl_ | zzzeek: I agree, it seems to me the best thing to avoid issues like this in the future | 16:26 |
| ihrachyshka | zzzeek, we don't gate neutron with pgsql | 16:26 |
| zzzeek | ihrachyshka: OK | 16:26 |
| salv-orl_ | the others is to hang developers that can't write a query | 16:26 |
| ihrachyshka | we did it before | 16:26 |
| *** lazy_prince has quit IRC | 16:26 | |
| *** shashank_hegde has joined #openstack-neutron | 16:27 | |
| *** _kiran_ has joined #openstack-neutron | 16:27 | |
| ihrachyshka | zzzeek, another db question, now alembic | 16:27 |
| zzzeek | salv-orl_: so note that MySQL’s nonstadnard GROUP BY does have an intent of allowing better performance in some circumstances. These are cirucmstances that IMO are avoidable in other ways, though it kind of links into how MySQL is bad at subqueries also | 16:27 |
| ihrachyshka | zzzeek, we hit today that even though we have a functional test that makes sure models are in sync with alembic scripts, it does not catch all differences: http://git.openstack.org/cgit/openstack/neutron/tree/neutron/tests/functional/db/test_migrations.py#n138 | 16:28 |
| *** minwang2 has joined #openstack-neutron | 16:28 | |
| salv-orl_ | zzzeek: from the client perspective - neutron in this case - this is the first case afaict where a group by clause has been used | 16:28 |
| salv-orl_ | so probably I would not be too much worried about impacting performance in other parts of the neutron server | 16:29 |
| ihrachyshka | zzzeek, specifically, it does not catch ondelete=CASCADE for foreign keys, and from looking into alembic code, it seems to me that compare_metadata just checks the keys are there, but not their properties. | 16:29 |
| zzzeek | salv-orl_: yes there’s no issue here, MySQL’s rationale is for when someone wants to GROUP BY the whole table it offers the option to only consider the primary key. we dont do that :) | 16:29 |
| *** jistr has quit IRC | 16:30 | |
| zzzeek | ihrachyshka: that is likely accurate | 16:30 |
| ihrachyshka | zzzeek, so we were probably too optimistic when assuming compare_metadata is the answer to keep alembic and models in sync :) | 16:30 |
| zzzeek | ihrachyshka: the things alembic compares are an ongoing process. | 16:30 |
| *** kiran-r has quit IRC | 16:31 | |
| *** clev is now known as clev-away | 16:31 | |
| openstackgerrit | Moshe Levi proposed openstack/neutron: SR-IOV: deprecate agent_required option https://review.openstack.org/214324 | 16:31 |
| zzzeek | ihrachyshka: we can add support for that to Alembic though it relies upon the backend dialects also reflecting it in SQLAlchemy | 16:31 |
| zzzeek | ihrachyshka: it’s likely that MySQL/PG both include these | 16:31 |
| ihrachyshka | zzzeek, what should I do to make it eventually happen? report a bug? | 16:31 |
| zzzeek | ihrachyshka: a bug…blah. pull request w tests :) | 16:31 |
| * ihrachyshka hides | 16:32 | |
| zzzeek | ihrachyshka: the FK comparison was written by akamyshnikova in the first place :) | 16:32 |
| ihrachyshka | oh! I see whom to nag now ;) | 16:32 |
| *** thumpba has quit IRC | 16:32 | |
| zzzeek | ihrachyshka: this featrue would be very easy to add | 16:33 |
| ihrachyshka | akamyshnikova is a kind of an engineer, isn't she? she does not report bugs, she sends PRs. /me is embarassed. | 16:33 |
| *** jpena is now known as jpena|away | 16:34 | |
| openstackgerrit | Zhi Chang proposed openstack/neutron-vpnaas: Add log to IPSec pluto process https://review.openstack.org/214701 | 16:34 |
| ihrachyshka | zzzeek, I won't get there in next month, but ok, point taken | 16:34 |
| *** lindycoder has joined #openstack-neutron | 16:34 | |
| zzzeek | ihrachyshka: there’s an object _fk_constraint_sig() which is a hashable “signature” of an FK, we would just need to add these flags to it. I think the reason they arent there is that this would now generate diffs on a backend that doesnt support reflecting those flags | 16:34 |
| zzzeek | ihrachyshka: so we might want to add some kind of check to see that reflection of these is supported, based on SQLA version / backend possible | 16:35 |
| ihrachyshka | I believe in neutron we run the test on mysql only | 16:35 |
| zzzeek | ihrachyshka: this is more about, releasing alembic 0.8.1 and people run it on oracle, MSSQL , etc. and expect no diffs :) | 16:39 |
| *** dims_ has quit IRC | 16:39 | |
| *** dims has joined #openstack-neutron | 16:39 | |
| *** pc_m_ has joined #openstack-neutron | 16:40 | |
| ihrachyshka | bloody people! they always expect something unreasonable! but yeah, jokes apart, there should be a check for versions and backends. | 16:40 |
| *** changzhi has quit IRC | 16:40 | |
| *** pc_m has quit IRC | 16:42 | |
| ihrachyshka | ajo, jschwarz, woot: https://jenkins04.openstack.org/job/gate-neutron-dsvm-api/717/console that's the devstack plugin run that shows qos tests run and pass in master | 16:43 |
| zzzeek | ihrachyshka: let me see if we even reflect these right now | 16:43 |
| jschwarz | ihrachyshka, woot woot | 16:44 |
| jschwarz | ihrachyshka++ | 16:44 |
| jschwarz | much appreciation was given to ihrachyshka at this day | 16:44 |
| *** nyechiel_ has quit IRC | 16:44 | |
| *** aranjan has joined #openstack-neutron | 16:44 | |
| ihrachyshka | dougwig, have a look, that's the qos API gate enablement and it passes with qos API tests enabled: https://review.openstack.org/#/c/214249/ for tempest, it will require more fiddling with project-config and gate_hook | 16:44 |
| zzzeek | ihrachyshka: yeah looks like mysql + postgresql do, though not in older versions, sqlite/mssql/oracle nope. | 16:45 |
| ihrachyshka | who runs with mssql? | 16:45 |
| zzzeek | ihrachyshka: however, I might be able to just poke in the dictionary and see if an “options” entry is present or not, determines if the backend reflects them | 16:45 |
| zzzeek | ihrachyshka: alembic is not part of openstack :) | 16:45 |
| ihrachyshka | who needs anything apart from openstack? it's so 90s | 16:46 |
| *** Marga_ has quit IRC | 16:47 | |
| *** SourabhP has joined #openstack-neutron | 16:48 | |
| salv-orl_ | yes but also openstack is kind of legacy nowadays | 16:48 |
| zzzeek | ihrachyshka: plus mysql’s dialect on SQLalchemy has no tests for it. the PG dialect does | 16:48 |
| *** madhu_ak has joined #openstack-neutron | 16:49 | |
| *** erkules_ has joined #openstack-neutron | 16:50 | |
| *** SourabhP_ has joined #openstack-neutron | 16:50 | |
| *** SourabhP has quit IRC | 16:52 | |
| *** SourabhP_ is now known as SourabhP | 16:52 | |
| *** erkules has quit IRC | 16:53 | |
| dougwig | ihrachyshka: sweet, looking | 16:53 |
| *** thurloat is now known as thurloat_isgone | 16:54 | |
| *** thurloat_isgone is now known as thurloat | 16:54 | |
| *** lpetrut has quit IRC | 16:54 | |
| *** erkules has joined #openstack-neutron | 16:55 | |
| *** mohankumar has joined #openstack-neutron | 16:56 | |
| *** erkules_ has quit IRC | 16:56 | |
| *** tflynn has joined #openstack-neutron | 16:56 | |
| *** vivek-ebay has quit IRC | 16:57 | |
| zzzeek | ihrachyshka: its easy and this is RH-paid work so I am doing https://bitbucket.org/zzzeek/alembic/issues/317/implement-autogen-detection-for-onupdate righ tnow | 16:58 |
| ihrachyshka | zzzeek, huh! kudos to Red Hat then ;) | 16:59 |
| *** cdl has left #openstack-neutron | 16:59 | |
| *** _kiran_ has quit IRC | 16:59 | |
| *** tflynn_ has joined #openstack-neutron | 17:00 | |
| *** britthouser has joined #openstack-neutron | 17:00 | |
| *** emagana has quit IRC | 17:02 | |
| *** lpetrut has joined #openstack-neutron | 17:02 | |
| *** diga has quit IRC | 17:02 | |
| *** tflynn has quit IRC | 17:03 | |
| *** ritesh_ has quit IRC | 17:03 | |
| *** emagana has joined #openstack-neutron | 17:03 | |
| *** ivar-lazzaro has joined #openstack-neutron | 17:04 | |
| *** ritesh_ has joined #openstack-neutron | 17:04 | |
| *** pal has quit IRC | 17:05 | |
| *** apuimedo has quit IRC | 17:05 | |
| *** kbringard1 has joined #openstack-neutron | 17:05 | |
| *** abhiraut has joined #openstack-neutron | 17:06 | |
| yamamoto | mestery: ping | 17:07 |
| mestery | yamamoto: pongalongdingdong | 17:07 |
| yamamoto | i have a few questions about L release of neutron subprojects | 17:08 |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent https://review.openstack.org/194446 | 17:08 |
| mlavalle | mestery: Lol | 17:08 |
| *** kbringard has quit IRC | 17:08 | |
| mestery | yamamoto: I'm at Linuxcon, and may be in and out, so maybe email is better? Would that be ok? | 17:09 |
| yamamoto | ok, i'll email. | 17:09 |
| mestery | yamamoto: thanks :) | 17:10 |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Cleanup the fip agent gateway port delete routines https://review.openstack.org/194455 | 17:10 |
| openstackgerrit | John Schwarz proposed openstack/neutron: Uniquely identify tunnel interfaces https://review.openstack.org/193770 | 17:10 |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add L3 HA fullstack failover test https://review.openstack.org/196393 | 17:10 |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add tunneling support to full stack tests https://review.openstack.org/190786 | 17:10 |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add l2pop support to full stack tests https://review.openstack.org/190832 | 17:10 |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add EnvironmentDescription, pass it down https://review.openstack.org/210830 | 17:10 |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add a fullstack fake VM, basic connectivity test https://review.openstack.org/191197 | 17:10 |
| openstackgerrit | John Schwarz proposed openstack/neutron: Add DVR fullstack test https://review.openstack.org/214715 | 17:10 |
| jschwarz | amuller_afk, ^ | 17:10 |
| jschwarz | obondarev, for some reason I can't add you as a reviewer but have a look at https://review.openstack.org/214715 - the first DVR fullstack test :) | 17:12 |
| *** moshele has quit IRC | 17:12 | |
| *** amuller_afk is now known as amuller | 17:13 | |
| *** abhiraut has quit IRC | 17:13 | |
| amuller | jschwarz: you rebased an approved patch | 17:13 |
| amuller | https://review.openstack.org/#/c/191197/ | 17:13 |
| jschwarz | gah | 17:13 |
| jschwarz | it wasn't supposed to do that :< | 17:13 |
| *** mestery has quit IRC | 17:14 | |
| *** tidwellr has quit IRC | 17:15 | |
| *** SumitNaiksatam has quit IRC | 17:15 | |
| *** vivek-ebay has joined #openstack-neutron | 17:15 | |
| amuller | jschwarz: when you use git review you can check the commit hash of the first patch it's pushing. If it's not the same as the commit hash of the patch in review then you're about to rebase it. | 17:15 |
| *** sthillma has quit IRC | 17:16 | |
| amuller | ihrachyshka: can you approve https://review.openstack.org/#/c/191197/? | 17:16 |
| *** gsagie has joined #openstack-neutron | 17:16 | |
| jschwarz | amuller, yeah, but usually it's smart and knows not to submit what I didn't explicitly rebase :< | 17:17 |
| *** sdake_ has joined #openstack-neutron | 17:17 | |
| jschwarz | amuller, sorry for the delay this causes | 17:17 |
| *** SourabhP has quit IRC | 17:17 | |
| amuller | jschwarz: I'm guessing you checked one revision of a patch just before I updated it | 17:17 |
| *** abhiraut has joined #openstack-neutron | 17:18 | |
| jschwarz | amuller, yeah, but then I rebased on top of the last one you uploaded | 17:18 |
| *** claudiub has quit IRC | 17:18 | |
| jschwarz | so it shouldn't have done that | 17:18 |
| openstackgerrit | Merged openstack/python-neutronclient: Create hooks for running functional test https://review.openstack.org/210021 | 17:18 |
| amuller | jschwarz: it was rebased because of a change in the parent patch | 17:19 |
| *** shashank_hegde has quit IRC | 17:19 | |
| *** clev-away is now known as clev | 17:19 | |
| amuller | jschwarz: check out the parent patch for PS 15 and PS 16, they're not the same | 17:19 |
| *** shausy has quit IRC | 17:19 | |
| *** sthillma has joined #openstack-neutron | 17:20 | |
| *** shausy has joined #openstack-neutron | 17:20 | |
| *** lodgenbd has joined #openstack-neutron | 17:20 | |
| jschwarz | amuller, aye, but PS15 and PS14 has the same parent | 17:20 |
| jschwarz | it must have explicitly merged it during git review :\ | 17:20 |
| *** fawadk has joined #openstack-neutron | 17:20 | |
| *** sdake has quit IRC | 17:20 | |
| *** jckasper has quit IRC | 17:20 | |
| jschwarz | explicitly rebased, that is | 17:21 |
| amuller | jschwarz: I know that PS14 and 15 have the same parent because those are two PSs where I did some minor work and pushed it | 17:21 |
| amuller | jschwarz: git review doesn't explicitly rebase anything for you | 17:21 |
| amuller | jschwarz: you rebased on a different local master | 17:21 |
| jschwarz | amuller, but I didn't rebase on master | 17:21 |
| *** fawadkhaliq has quit IRC | 17:21 | |
| amuller | jschwarz: what command did you use to rebase? did you check out PS 15 and cherry pick the rest of the series? Because that would have resulted in pushing only the other patches but not the first one | 17:22 |
| *** dsneddon_brb is now known as dsneddon | 17:22 | |
| jschwarz | amuller, I've checked out PS 15 and rebased the chain on top of that one - normally it results in a submission of the others in the chain | 17:22 |
| jschwarz | https://github.com/openstack/neutron/commit/a5b0e6eaf9a9b4e8b274a3f3e124c27bba40116e - this is the parent and it's the last patch merged according to github | 17:23 |
| jschwarz | I didn't pull the repo since Monday | 17:23 |
| amuller | jschwarz: can you describe your work flow after you checked out PS 15? | 17:23 |
| amuller | jschwarz: The reason I'm asking is that this happens from time to time (Just ask carl_baldwin he's mega pissed about this stuff) and we can never seem to nail down why *sometimes* a patch is magically rebased | 17:24 |
| *** sbalukoff has quit IRC | 17:24 | |
| jschwarz | amuller, git checkout <last patch hash>; git rebase -i a464caf, and made sure to pick only patches starting after https://review.openstack.org/#/c/210830/ | 17:25 |
| jschwarz | amuller, then git review | 17:25 |
| jschwarz | amuller, oh, it did complain of a merge conflict *during* git review of one of the last patches (which I had to resolve and 'git rebase --continue' and then 'git review' afterwards) | 17:26 |
| jschwarz | so that probably was my mistake | 17:26 |
| amuller | yes that is when it rebased | 17:26 |
| amuller | ok, I'm not sure what you could have done to avoid that | 17:26 |
| jschwarz | :\ | 17:26 |
| amuller | sometimes I think we should stop using git review | 17:27 |
| *** britthouser has quit IRC | 17:27 | |
| jschwarz | could've waited for tomorrow and submit it after it was merged | 17:27 |
| *** rpothier has quit IRC | 17:27 | |
| jschwarz | but too late now :< | 17:27 |
| jschwarz | anyways I'm off to home-sweet-home | 17:27 |
| *** saksham has joined #openstack-neutron | 17:27 | |
| jschwarz | amuller, sorry again | 17:27 |
| jschwarz | nn all | 17:27 |
| *** britthouser has joined #openstack-neutron | 17:28 | |
| *** DWfuturetec has quit IRC | 17:28 | |
| *** tiswanso has quit IRC | 17:28 | |
| *** shwetaap1 has joined #openstack-neutron | 17:28 | |
| *** tiswanso has joined #openstack-neutron | 17:28 | |
| *** dave-mcc_ has joined #openstack-neutron | 17:29 | |
| *** lucasagomes is now known as lucas-hungry | 17:29 | |
| *** tflynn_ has quit IRC | 17:29 | |
| amuller | jschwarz: night! | 17:30 |
| amuller | jschwarz: np | 17:30 |
| *** dave-mccowan has quit IRC | 17:30 | |
| *** shwetaap has quit IRC | 17:30 | |
| *** jschwarz has quit IRC | 17:32 | |
| *** mohankumar has quit IRC | 17:32 | |
| *** minwang2 has quit IRC | 17:33 | |
| *** yamahata has quit IRC | 17:34 | |
| *** Piet has quit IRC | 17:35 | |
| *** jbell8_ has quit IRC | 17:35 | |
| *** minwang2 has joined #openstack-neutron | 17:36 | |
| *** Piet has joined #openstack-neutron | 17:36 | |
| *** sridhar_ram has joined #openstack-neutron | 17:36 | |
| *** sdake has joined #openstack-neutron | 17:37 | |
| *** krtaylor has quit IRC | 17:37 | |
| *** tflynn has joined #openstack-neutron | 17:38 | |
| *** sthillma has quit IRC | 17:38 | |
| ihrachyshka | amuller, I allow you to approve once jenkins is happy :) | 17:39 |
| ihrachyshka | amuller, I think without it, we may push a broken patch in the gate and reset the queue | 17:39 |
| *** sdake_ has quit IRC | 17:40 | |
| *** Piet has quit IRC | 17:41 | |
| *** achanda has joined #openstack-neutron | 17:42 | |
| amuller | ihrachyshka: roger that | 17:42 |
| amuller | ihrachyshka: on to approve my own patch! | 17:42 |
| *** dane_leblanc has quit IRC | 17:43 | |
| *** jwy has joined #openstack-neutron | 17:43 | |
| *** Piet has joined #openstack-neutron | 17:45 | |
| *** abregman has quit IRC | 17:45 | |
| *** sdake has quit IRC | 17:47 | |
| *** padkrish has joined #openstack-neutron | 17:47 | |
| *** ihrachyshka has quit IRC | 17:48 | |
| openstackgerrit | Merged openstack/python-neutronclient: Support Command line changes for Address Scope https://review.openstack.org/194635 | 17:49 |
| *** armax has joined #openstack-neutron | 17:50 | |
| *** sdake has joined #openstack-neutron | 17:51 | |
| openstackgerrit | Aishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario tests using Rally https://review.openstack.org/213781 | 17:51 |
| *** ameybhide has joined #openstack-neutron | 17:51 | |
| *** breitz has quit IRC | 17:51 | |
| *** breitz has joined #openstack-neutron | 17:52 | |
| openstackgerrit | Aishwarya Thangappa proposed openstack/neutron-vpnaas: Basic VPNaaS Scenario Test https://review.openstack.org/213819 | 17:52 |
| *** SourabhP has joined #openstack-neutron | 17:52 | |
| *** ekuris has quit IRC | 17:53 | |
| *** anshul has quit IRC | 17:53 | |
| *** mickeyspiegel has joined #openstack-neutron | 17:54 | |
| *** achanda_ has joined #openstack-neutron | 17:54 | |
| *** Sukhdev has quit IRC | 17:54 | |
| *** ronis_ has joined #openstack-neutron | 17:54 | |
| *** afazkas has joined #openstack-neutron | 17:55 | |
| *** aix has quit IRC | 17:55 | |
| *** yanxb__ has joined #openstack-neutron | 17:55 | |
| *** achanda has quit IRC | 17:55 | |
| *** tidwellr has joined #openstack-neutron | 17:55 | |
| *** kiran-r has joined #openstack-neutron | 17:55 | |
| *** ronis has quit IRC | 17:55 | |
| *** Sukhdev has joined #openstack-neutron | 17:56 | |
| *** yamahata has joined #openstack-neutron | 17:56 | |
| *** mickeys has quit IRC | 17:57 | |
| *** yanxb_ has quit IRC | 17:57 | |
| *** DWfuturetec has joined #openstack-neutron | 17:58 | |
| *** pritesh has quit IRC | 17:59 | |
| *** pritesh has joined #openstack-neutron | 18:00 | |
| *** shashank_hegde has joined #openstack-neutron | 18:00 | |
| *** gangil has joined #openstack-neutron | 18:02 | |
| *** jbell8 has joined #openstack-neutron | 18:03 | |
| *** annegentle has quit IRC | 18:04 | |
| *** kiran-r has quit IRC | 18:05 | |
| *** bharath has joined #openstack-neutron | 18:06 | |
| *** jckasper has joined #openstack-neutron | 18:06 | |
| *** rushil has joined #openstack-neutron | 18:07 | |
| SourabhP | abhiraut: ping | 18:07 |
| abhiraut | SourabhP: pong | 18:08 |
| *** dane_leblanc has joined #openstack-neutron | 18:08 | |
| *** njohnston is now known as nate_gone | 18:09 | |
| Swami | obondarev: ping | 18:10 |
| *** krtaylor has joined #openstack-neutron | 18:13 | |
| *** shausy has quit IRC | 18:16 | |
| *** emagana has quit IRC | 18:16 | |
| *** Piet has quit IRC | 18:16 | |
| *** klindgren has quit IRC | 18:16 | |
| *** tflynn has quit IRC | 18:18 | |
| *** tidwellr has quit IRC | 18:20 | |
| *** emagana has joined #openstack-neutron | 18:22 | |
| *** TrevorV has joined #openstack-neutron | 18:27 | |
| *** anshul has joined #openstack-neutron | 18:30 | |
| *** bapalm has quit IRC | 18:31 | |
| *** sthillma has joined #openstack-neutron | 18:32 | |
| *** yamamoto has quit IRC | 18:32 | |
| *** bapalm has joined #openstack-neutron | 18:32 | |
| *** tfukushima has quit IRC | 18:33 | |
| *** banix has quit IRC | 18:33 | |
| *** emagana has quit IRC | 18:34 | |
| *** lori is now known as lori|away | 18:34 | |
| *** emagana has joined #openstack-neutron | 18:34 | |
| *** Marga_ has joined #openstack-neutron | 18:35 | |
| john-davidge | regXboi: The dvr multinote tests failed in a different and unrelated way the second time https://review.openstack.org/#/c/185977/ | 18:35 |
| john-davidge | regXboi: So I think it’s safe to say that patch isn’t causing it | 18:35 |
| john-davidge | regXboi: But we can run jenkins again to be sure if you prefer | 18:36 |
| *** rm_work is now known as rm_work|away | 18:36 | |
| regXboi | john-davidge: let me look | 18:37 |
| * regXboi looks | 18:37 | |
| regXboi | I agree - that's completely different ... i'll pull my -1 | 18:38 |
| regXboi | john-davidge: thanks for checking! | 18:39 |
| *** Piet has joined #openstack-neutron | 18:41 | |
| john-davidge | regXboi: No worries, cheers | 18:41 |
| *** tfukushima has joined #openstack-neutron | 18:42 | |
| *** claudiub has joined #openstack-neutron | 18:43 | |
| *** tflynn has joined #openstack-neutron | 18:45 | |
| *** sbalukoff has joined #openstack-neutron | 18:46 | |
| *** tfukushima has quit IRC | 18:46 | |
| *** klindgren has joined #openstack-neutron | 18:47 | |
| *** gaud has joined #openstack-neutron | 18:48 | |
| *** padkrish has quit IRC | 18:48 | |
| *** padkrish has joined #openstack-neutron | 18:48 | |
| *** gaud has left #openstack-neutron | 18:51 | |
| *** rkukura has quit IRC | 18:51 | |
| *** padkrish has quit IRC | 18:52 | |
| *** rkukura has joined #openstack-neutron | 18:52 | |
| *** padkrish has joined #openstack-neutron | 18:52 | |
| *** banix has joined #openstack-neutron | 18:53 | |
| *** nate_gone is now known as njohnston | 18:54 | |
| *** rm_work|away is now known as rm_work | 18:57 | |
| *** tflynn has quit IRC | 18:57 | |
| *** pal has joined #openstack-neutron | 18:58 | |
| *** tflynn has joined #openstack-neutron | 18:58 | |
| mlavalle | armax: when you have a chance, please take a look at https://review.openstack.org/#/c/200952/. It's good to go. Incorporated kevinbenton advice | 18:58 |
| *** tflynn has quit IRC | 18:59 | |
| *** liudong has joined #openstack-neutron | 18:59 | |
| *** john-davidge has quit IRC | 19:00 | |
| *** rkukura has quit IRC | 19:01 | |
| *** rushil has quit IRC | 19:01 | |
| *** sudipto has quit IRC | 19:02 | |
| *** tflynn has joined #openstack-neutron | 19:03 | |
| *** rushil has joined #openstack-neutron | 19:03 | |
| *** lpetrut has quit IRC | 19:03 | |
| *** fawadk has quit IRC | 19:04 | |
| *** lpetrut has joined #openstack-neutron | 19:04 | |
| *** tflynn_ has joined #openstack-neutron | 19:05 | |
| openstackgerrit | Brent Eagles proposed openstack/neutron-vpnaas: Manage cleanup of .ctl/.pid files for LibreSwan https://review.openstack.org/212673 | 19:07 |
| *** jpena|away has quit IRC | 19:07 | |
| *** tflynn has quit IRC | 19:08 | |
| *** annegentle has joined #openstack-neutron | 19:13 | |
| *** lpetrut has quit IRC | 19:13 | |
| *** afazkas has quit IRC | 19:13 | |
| *** minwang2 has quit IRC | 19:14 | |
| *** afazekas has joined #openstack-neutron | 19:14 | |
| *** nlahouti has quit IRC | 19:16 | |
| *** slaweq has joined #openstack-neutron | 19:16 | |
| *** emagana has quit IRC | 19:17 | |
| *** klindgren has quit IRC | 19:18 | |
| *** anshul has quit IRC | 19:18 | |
| *** Piet has quit IRC | 19:19 | |
| *** TrevorV has quit IRC | 19:19 | |
| *** blogan_ has joined #openstack-neutron | 19:19 | |
| *** lennyb has quit IRC | 19:20 | |
| *** nlahouti has joined #openstack-neutron | 19:21 | |
| *** Sukhdev has quit IRC | 19:23 | |
| *** tkatarki has quit IRC | 19:29 | |
| *** shashank_hegde has quit IRC | 19:29 | |
| *** regXboi has quit IRC | 19:32 | |
| *** abhiraut has quit IRC | 19:32 | |
| *** yamamoto has joined #openstack-neutron | 19:32 | |
| *** gsagie has quit IRC | 19:33 | |
| *** abhiraut has joined #openstack-neutron | 19:34 | |
| *** rkukura has joined #openstack-neutron | 19:34 | |
| *** devvesa has quit IRC | 19:34 | |
| *** pal has quit IRC | 19:37 | |
| *** achanda_ has quit IRC | 19:37 | |
| *** yamamoto has quit IRC | 19:38 | |
| *** piyanai has joined #openstack-neutron | 19:38 | |
| *** tflynn_ has quit IRC | 19:40 | |
| openstackgerrit | Aishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario tests using Rally https://review.openstack.org/213781 | 19:41 |
| *** clev is now known as clev-away | 19:42 | |
| *** Sukhdev has joined #openstack-neutron | 19:43 | |
| *** rkukura has quit IRC | 19:44 | |
| *** blogan_ has quit IRC | 19:45 | |
| openstackgerrit | James Arendt proposed openstack/python-neutronclient: WIP: Add Neutron flavor framework CLI https://review.openstack.org/214776 | 19:45 |
| *** blogan_ has joined #openstack-neutron | 19:45 | |
| *** afazekas has quit IRC | 19:46 | |
| *** nlahouti has quit IRC | 19:46 | |
| *** abhiraut has quit IRC | 19:48 | |
| *** mickeyspiegel has quit IRC | 19:49 | |
| *** ronis_ has quit IRC | 19:51 | |
| *** achanda has joined #openstack-neutron | 19:51 | |
| *** alex_klimov has joined #openstack-neutron | 19:51 | |
| *** minwang2 has joined #openstack-neutron | 19:56 | |
| *** alejandrito has joined #openstack-neutron | 19:59 | |
| *** annegentle has quit IRC | 19:59 | |
| *** alejandrito_ has joined #openstack-neutron | 20:00 | |
| *** alejandrito has quit IRC | 20:00 | |
| *** jpena has joined #openstack-neutron | 20:00 | |
| *** TrevorV has joined #openstack-neutron | 20:01 | |
| *** zhhuabj has quit IRC | 20:02 | |
| *** zhhuabj has joined #openstack-neutron | 20:03 | |
| *** thurloat is now known as thurloat_isgone | 20:05 | |
| *** clev-away is now known as clev | 20:09 | |
| *** aranjan has quit IRC | 20:09 | |
| *** aranjan has joined #openstack-neutron | 20:11 | |
| *** dane_leblanc has quit IRC | 20:11 | |
| *** Piet has joined #openstack-neutron | 20:11 | |
| *** smccully has quit IRC | 20:13 | |
| openstackgerrit | Merged openstack/neutron: Fix query in get_reservations_for_resources https://review.openstack.org/214564 | 20:14 |
| *** rushil has quit IRC | 20:16 | |
| *** slaweq has quit IRC | 20:16 | |
| *** alejandrito_ has quit IRC | 20:18 | |
| *** blogan_ has quit IRC | 20:20 | |
| *** lpetrut has joined #openstack-neutron | 20:20 | |
| *** blogan_ has joined #openstack-neutron | 20:20 | |
| *** _cjones_ has quit IRC | 20:21 | |
| *** piyanai has quit IRC | 20:22 | |
| *** tflynn has joined #openstack-neutron | 20:24 | |
| *** r-daneel has joined #openstack-neutron | 20:24 | |
| *** julim has quit IRC | 20:25 | |
| *** abhiraut has joined #openstack-neutron | 20:27 | |
| *** emagana has joined #openstack-neutron | 20:28 | |
| *** Piet has quit IRC | 20:29 | |
| *** neophy has joined #openstack-neutron | 20:29 | |
| *** klindgren has joined #openstack-neutron | 20:30 | |
| *** Piet has joined #openstack-neutron | 20:32 | |
| *** azbiswas has joined #openstack-neutron | 20:34 | |
| *** rkukura has joined #openstack-neutron | 20:37 | |
| *** pritesh has quit IRC | 20:38 | |
| *** padkrish has quit IRC | 20:42 | |
| *** padkrish has joined #openstack-neutron | 20:43 | |
| *** annegentle has joined #openstack-neutron | 20:45 | |
| *** nlahouti has joined #openstack-neutron | 20:45 | |
| *** padkrish has quit IRC | 20:45 | |
| *** padkrish has joined #openstack-neutron | 20:45 | |
| *** lpetrut has quit IRC | 20:46 | |
| *** achanda has quit IRC | 20:47 | |
| *** blogan_ has quit IRC | 20:47 | |
| *** blogan_ has joined #openstack-neutron | 20:48 | |
| *** shashank_hegde has joined #openstack-neutron | 20:50 | |
| *** armax has quit IRC | 20:51 | |
| openstackgerrit | Hirofumi Ichihara proposed openstack/neutron: Enable to update external network subnet's gateway-ip https://review.openstack.org/194939 | 20:51 |
| *** achanda has joined #openstack-neutron | 20:52 | |
| *** annegentle has quit IRC | 20:53 | |
| *** annegentle has joined #openstack-neutron | 20:54 | |
| *** lilynn2 has quit IRC | 20:55 | |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Revert "DVR: fix router scheduling" https://review.openstack.org/214790 | 20:56 |
| *** TrevorV has quit IRC | 20:58 | |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port https://review.openstack.org/194441 | 20:58 |
| *** armax has joined #openstack-neutron | 20:59 | |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Revert "DVR: fix router scheduling" https://review.openstack.org/214790 | 20:59 |
| *** debedb has joined #openstack-neutron | 21:01 | |
| *** dave-mcc_ has quit IRC | 21:01 | |
| debedb | hello | 21:01 |
| Swami | haleyb: ping | 21:02 |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent https://review.openstack.org/194446 | 21:02 |
| debedb | I am running openstack with linuxbridge and am trying to understand where exactly is the tap interface created that then is used in the vm | 21:03 |
| openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron: Cleanup the fip agent gateway port delete routines https://review.openstack.org/194455 | 21:03 |
| haleyb | Swami: pong, I see some DVR reverts :( | 21:03 |
| *** neophy has quit IRC | 21:03 | |
| *** piyanai has joined #openstack-neutron | 21:03 | |
| debedb | I just see this tapce0c516f-4c for example appear | 21:03 |
| Swami | haleyb: yes, this patch has introduced an issue where the snat namespace is not created. | 21:03 |
| *** neiljerram_bb has joined #openstack-neutron | 21:04 | |
| *** mickeys has joined #openstack-neutron | 21:04 | |
| Swami | haleyb: I was trying to nail down the issue from yesterday since in my devstack environment I found that the snat namespace is not created. | 21:04 |
| *** regXboi has joined #openstack-neutron | 21:05 | |
| *** piyanai has quit IRC | 21:05 | |
| Swami | obondarev: Can you take a look at this patch #link https://review.openstack.org/#/c/214790/ | 21:06 |
| *** liudong has quit IRC | 21:06 | |
| *** Marga_ has quit IRC | 21:06 | |
| *** rpothier has joined #openstack-neutron | 21:06 | |
| haleyb | Swami: so we keep moving that code around, it would be good to fix both bugs :) | 21:06 |
| *** mickeyspiegel has joined #openstack-neutron | 21:06 | |
| *** achanda has quit IRC | 21:06 | |
| Swami | haleyb: the bug that is mentioned in that patch was seen in Juno and was not seen anymore on kilo | 21:08 |
| *** mickeys has quit IRC | 21:08 | |
| Swami | haleyb: the double binding issue. | 21:08 |
| Swami | haleyb: eventually we need to figure out the best option | 21:08 |
| *** rushil has joined #openstack-neutron | 21:09 | |
| *** piyanai has joined #openstack-neutron | 21:10 | |
| *** _cjones_ has joined #openstack-neutron | 21:12 | |
| amuller | Swami: obondarev was made aware of the issue today by jschwarz | 21:12 |
| Swami | amuller: thanks | 21:12 |
| amuller | Swami: he identified the root cause | 21:13 |
| amuller | Swami: can you wait with the revert until you talk to him tomorrow? | 21:13 |
| *** rushil_ has joined #openstack-neutron | 21:13 | |
| Swami | amuller: ok, I will | 21:13 |
| amuller | Swami: thanks | 21:13 |
| amuller | haleyb: ^ | 21:13 |
| *** amotoki_ has quit IRC | 21:14 | |
| haleyb | amuller: thanks. Swami: i haven't updated my devstack since last week to notice | 21:15 |
| *** tkatarki has joined #openstack-neutron | 21:15 | |
| *** rushil_ has quit IRC | 21:16 | |
| Swami | haleyb: good you have a working one from last week, since I came back from vacation I need to redo my devstack, then I found this issue. | 21:16 |
| *** rushil has quit IRC | 21:16 | |
| *** rushil has joined #openstack-neutron | 21:16 | |
| haleyb | i'm stacking now, but of course pip is barfing over something | 21:17 |
| Swami | amuller: I added a comment in the patch as waiting for oleg's input and marked it as WIP. | 21:17 |
| Swami | haleyb: good luck. | 21:18 |
| *** krtaylor has quit IRC | 21:18 | |
| *** azbiswas has quit IRC | 21:19 | |
| *** azbiswas has joined #openstack-neutron | 21:19 | |
| openstackgerrit | Aishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario Test - test-vpn-connectivity https://review.openstack.org/213819 | 21:20 |
| *** tflynn has quit IRC | 21:20 | |
| openstackgerrit | Aishwarya Thangappa proposed openstack/neutron-vpnaas: VPNaaS Scenario Test - test-vpn-connectivity https://review.openstack.org/213819 | 21:21 |
| *** blogan_ has quit IRC | 21:21 | |
| *** blogan_ has joined #openstack-neutron | 21:21 | |
| *** jpena has quit IRC | 21:23 | |
| amuller | Swami: haleyb: Can we close https://bugs.launchpad.net/neutron/+bug/1450604, or what's left there? | 21:23 |
| openstack | Launchpad bug 1450604 in neutron "Fix DVR multinode upstream CI testing" [Medium,In progress] - Assigned to Ryan Moats (rmoats) | 21:23 |
| *** azbiswas has quit IRC | 21:24 | |
| *** tiswanso has quit IRC | 21:25 | |
| amuller | haleyb: also this: https://review.openstack.org/#/c/178880/ is good to go on my end | 21:27 |
| *** tflynn has joined #openstack-neutron | 21:27 | |
| *** shivrao has joined #openstack-neutron | 21:27 | |
| haleyb | amuller: looking | 21:28 |
| *** abhiraut has quit IRC | 21:29 | |
| *** pc_m_ has quit IRC | 21:29 | |
| Swami | amuller: This bug was created to track any patches that are required to make multinode job stable. | 21:30 |
| amuller | Swami: it's not? | 21:30 |
| amuller | Swami: ok I see it isn't http://goo.gl/EAugSi | 21:31 |
| amuller | generally 12.5% is considered stable | 21:31 |
| amuller | actually AIO DVR job also has a higher than desired failure rate | 21:31 |
| *** thorst has quit IRC | 21:32 | |
| Swami | amuller: ok, if it is within 12.5% then you can close the bug | 21:32 |
| *** abhiraut has joined #openstack-neutron | 21:32 | |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Remove VIF_TYPES constant https://review.openstack.org/213905 | 21:33 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Move in-tree vendor VIF_TYPE_* constants https://review.openstack.org/214805 | 21:33 |
| openstackgerrit | Cedric Brandily proposed openstack/neutron: Remove out-of-tree vendor VIF_TYPE_* constants https://review.openstack.org/214806 | 21:33 |
| *** azbiswas has joined #openstack-neutron | 21:33 | |
| *** rpothier has quit IRC | 21:36 | |
| *** njohnston is now known as nate_gone | 21:37 | |
| *** tflynn has quit IRC | 21:38 | |
| *** Piet has quit IRC | 21:38 | |
| amuller | haleyb: I'll push a new PS without the unused 'HOST' variable | 21:40 |
| *** Sukhdev has quit IRC | 21:43 | |
| haleyb | amuller: i am a nit-picker :) What about carl's comment? | 21:43 |
| amuller | haleyb: well of course :) | 21:43 |
| *** s3wong has joined #openstack-neutron | 21:44 | |
| *** aranjan has quit IRC | 21:44 | |
| *** blogan_ has quit IRC | 21:46 | |
| *** piyanai has quit IRC | 21:47 | |
| *** blogan_ has joined #openstack-neutron | 21:47 | |
| *** piyanai has joined #openstack-neutron | 21:47 | |
| *** tflynn has joined #openstack-neutron | 21:47 | |
| *** minwang2 has quit IRC | 21:49 | |
| *** signed8bit has joined #openstack-neutron | 21:49 | |
| *** tflynn has quit IRC | 21:49 | |
| *** aranjan has joined #openstack-neutron | 21:49 | |
| *** piyanai has quit IRC | 21:49 | |
| *** piyanai has joined #openstack-neutron | 21:51 | |
| *** sambetts has quit IRC | 21:51 | |
| *** tflynn has joined #openstack-neutron | 21:51 | |
| *** sambetts has joined #openstack-neutron | 21:51 | |
| *** klindgren has quit IRC | 21:53 | |
| *** aranjan_ has joined #openstack-neutron | 21:54 | |
| openstackgerrit | Salvatore Orlando proposed openstack/neutron: WIP: Fix reservation engine https://review.openstack.org/214660 | 21:56 |
| *** mattt has joined #openstack-neutron | 21:57 | |
| *** aranjan has quit IRC | 21:57 | |
| *** dmartls1 has quit IRC | 21:58 | |
| *** aranjan_ has quit IRC | 21:59 | |
| *** krtaylor has joined #openstack-neutron | 22:00 | |
| *** shwetaap1 has quit IRC | 22:00 | |
| abhiraut | ping salv_orl_ | 22:00 |
| abhiraut | ping salv-orl | 22:00 |
| *** rushil_ has joined #openstack-neutron | 22:01 | |
| *** mchestr has joined #openstack-neutron | 22:01 | |
| abhiraut | ping salv-orl_ | 22:01 |
| *** mchestr has quit IRC | 22:01 | |
| openstackgerrit | Assaf Muller proposed openstack/neutron: Add high-level functional/integration DVR tests https://review.openstack.org/178880 | 22:02 |
| amuller | haleyb: ^ | 22:02 |
| *** dave-mccowan has joined #openstack-neutron | 22:02 | |
| *** Swami has quit IRC | 22:02 | |
| openstackgerrit | James Arendt proposed openstack/python-neutronclient: WIP: Add Neutron flavor framework CLI https://review.openstack.org/214776 | 22:02 |
| *** y_sawai has joined #openstack-neutron | 22:02 | |
| salv-orl_ | aloha abhiraut | 22:03 |
| openstackgerrit | Paul Michali proposed openstack/python-neutronclient: Enable VPN test cases https://review.openstack.org/214587 | 22:03 |
| *** crose has quit IRC | 22:03 | |
| salv-orl_ | HenryG, ajo, kevinbenton: abhiraut has been spending some time chasing default security group rally failures | 22:03 |
| abhiraut | salv-orl_ : so i was able to reproduce the rally job failure on my devstack machine | 22:03 |
| *** absubram has quit IRC | 22:03 | |
| abhiraut | salv-orl_ : simply tried to create ten networks concurrently and lo it fails | 22:04 |
| salv-orl_ | with a lock wait timeour or a dbdeadlock? | 22:04 |
| abhiraut | lockwaittimeout | 22:04 |
| *** rushil has quit IRC | 22:04 | |
| *** lilynn2 has joined #openstack-neutron | 22:04 | |
| *** lilynn has joined #openstack-neutron | 22:04 | |
| abhiraut | DBError: (pymysql.err.InternalError) (1205, u'Lock wait timeout exceeded; try restarting transaction') [SQL: u'INSERT INTO securitygroups (tenant_id, id, name, description) VALUES (%s, %s, %s, %s)'] [parameters: (u'd2e3b08690d74014afb097b21ebfc2ee', 'bd4a79e6-e467-448d-93e0-65e4f2bdb4f5', 'default', u'Default security group')] | 22:04 |
| *** lilynn2 has quit IRC | 22:05 | |
| salv-orl_ | abhiraut: how many workers do you have in your env? | 22:05 |
| salv-orl_ | api workers | 22:05 |
| abhiraut | just 2 | 22:05 |
| abhiraut | this is of course with the ml2 plugin | 22:05 |
| salv-orl_ | abhiraut: do you see the lock wait timeout on both workers or only one | 22:05 |
| *** minwang2 has joined #openstack-neutron | 22:05 | |
| *** tiny-hands has quit IRC | 22:07 | |
| *** gangil has quit IRC | 22:08 | |
| abhiraut | dumb question: how do i verify that? | 22:08 |
| *** y_sawai has quit IRC | 22:08 | |
| salv-orl_ | error traces should have a pid too in the logs | 22:08 |
| salv-orl_ | abhiraut: the pid is usally found after the timestamp | 22:09 |
| abhiraut | no i dont see it | 22:09 |
| abhiraut | yea i was looking for it | 22:09 |
| abhiraut | but didnt see a pid | 22:09 |
| salv-orl_ | abhiraut: ah you can reconfigured neutron to print it | 22:09 |
| salv-orl_ | I mean change neutron configuration | 22:09 |
| abhiraut | oh ok | 22:09 |
| salv-orl_ | there are some logging format parameters in neutron.conf. Idk which ones are in yours | 22:09 |
| salv-orl_ | but there should be %(process)d in the format string | 22:10 |
| *** y_sawai has joined #openstack-neutron | 22:10 | |
| salv-orl_ | if there isn't, add it | 22:10 |
| *** emagana has quit IRC | 22:10 | |
| abhiraut | yeah its missing .. let me add and retry | 22:11 |
| *** padkrish has quit IRC | 22:11 | |
| *** padkrish has joined #openstack-neutron | 22:11 | |
| *** y_sawai_ has joined #openstack-neutron | 22:11 | |
| *** lindycoder has quit IRC | 22:11 | |
| openstackgerrit | Merged openstack/neutron-vpnaas: Killed existing downgrade rules in migration scripts https://review.openstack.org/206990 | 22:11 |
| *** Sukhdev has joined #openstack-neutron | 22:13 | |
| *** amotoki_ has joined #openstack-neutron | 22:14 | |
| *** y_sawai has quit IRC | 22:14 | |
| mattt | is there a straightforward way of knowing, after upgrading neutron, if there are pending db migrations to apply? | 22:15 |
| *** mickeyspiegel has quit IRC | 22:15 | |
| *** y_sawai_ has quit IRC | 22:16 | |
| *** mickeyspiegel has joined #openstack-neutron | 22:16 | |
| *** mickeyspiegel has quit IRC | 22:16 | |
| *** annegentle has quit IRC | 22:17 | |
| *** mickeys has joined #openstack-neutron | 22:17 | |
| *** amuller has quit IRC | 22:18 | |
| *** amotoki_ has quit IRC | 22:19 | |
| abhiraut | salv-orl_ same api worker in the first run | 22:20 |
| *** kbringard1 has quit IRC | 22:21 | |
| *** aranjan has joined #openstack-neutron | 22:22 | |
| *** gildub has joined #openstack-neutron | 22:23 | |
| *** pradk has quit IRC | 22:23 | |
| *** gangil has joined #openstack-neutron | 22:23 | |
| *** dave-mccowan has quit IRC | 22:24 | |
| salv-orl_ | abhiraut: ok. so we concurrent network create operations for a given tenant. they both try and create a security group. I think the code should already be resilient to concurrent creation, so I would look whether there is a condition where evenetlet would yield in the wrong place thus causing the lock wait timeout issue | 22:25 |
| salv-orl_ | in the transaction for creating a network is there any query that does SELECT... FOR UPDATE? | 22:26 |
| *** tkatarki has quit IRC | 22:26 | |
| *** Piet has joined #openstack-neutron | 22:28 | |
| abhiraut | this time around i am getting SAVEPOINT sa_savepoint_3 does not exist error.. again in the context of create_sec_grp | 22:28 |
| abhiraut | and in both the api workers | 22:28 |
| salv-orl_ | ok so you havve real nested transactions, even if I recall a recent patch that removed those | 22:29 |
| abhiraut | also i tend to get these errors when i first delete the default security group for that tenant | 22:29 |
| abhiraut | and then perform create_networks | 22:30 |
| salv-orl_ | abhiraut: about nested trans, they should have been removed, unless https://review.openstack.org/#/c/207034/ has been reverted | 22:30 |
| abhiraut | yeah i have that patch | 22:31 |
| *** mwagner_lap has quit IRC | 22:32 | |
| *** signed8bit is now known as signed8bit_ZZZzz | 22:32 | |
| *** agireud has quit IRC | 22:34 | |
| abhiraut | salv-orl_ not sure if it is worthwhile checking out this patch and try | 22:35 |
| abhiraut | https://review.openstack.org/#/c/203384/8 | 22:35 |
| *** mickeys has quit IRC | 22:35 | |
| *** mickeys has joined #openstack-neutron | 22:36 | |
| *** shashank_hegde has quit IRC | 22:37 | |
| *** sridhar_ram1 has joined #openstack-neutron | 22:38 | |
| *** sridhar_ram has quit IRC | 22:39 | |
| *** signed8bit_ZZZzz is now known as signed8bit | 22:39 | |
| openstackgerrit | Miguel Lavalle proposed openstack/neutron: External DNS driver reference implementation https://review.openstack.org/212213 | 22:40 |
| *** padkrish has quit IRC | 22:41 | |
| *** padkrish has joined #openstack-neutron | 22:41 | |
| salv-orl_ | abhiraut: this was pointed out to me as well, what do you reckon? I guess trying won't do any harm | 22:42 |
| abhiraut | i just tried | 22:42 |
| abhiraut | passed | 22:42 |
| abhiraut | also i checked that rally jobs are happy for that patch | 22:43 |
| abhiraut | ill try a few more times.. and let you know | 22:43 |
| salv-orl_ | abhiraut: do a recheck on that patch maybe | 22:43 |
| abhiraut | yeah | 22:43 |
| salv-orl_ | the rally run is from aug 7th I don't think multiple api workers were enabled | 22:44 |
| abhiraut | yeah.. recheck done. *fingers crossed* | 22:44 |
| jlvillal | salv-orl_: Just an FYI. The Ironic gate is working once again. Thanks for all the help from the Neutron team and getting it working. | 22:44 |
| *** VikasC has joined #openstack-neutron | 22:45 | |
| salv-orl_ | jlvillal: yes sorry about the fact that the first revert I pushed did not disable all the logic | 22:45 |
| jlvillal | salv-orl_: It happens. I don't want to think about all of my errors ;) Thanks again. | 22:45 |
| salv-orl_ | we fixed the postgres query thanks to akamyshnikova and then we also ensured that bit of logic is not used anyway | 22:45 |
| salv-orl_ | so you should be good forever | 22:45 |
| salv-orl_ | well, not forever, but at least until I push another patch ;) | 22:45 |
| *** ajmiller_ has joined #openstack-neutron | 22:46 | |
| *** mickeyspiegel has joined #openstack-neutron | 22:46 | |
| *** claudiub has quit IRC | 22:46 | |
| jlvillal | hehe | 22:46 |
| boris-42 | salv-orl_: there were other bugs* | 22:46 |
| boris-42 | salv-orl_: e.g. swithicng to new mysql lib and so on | 22:47 |
| salv-orl_ | yes boris-42 I am the man of the bugs. I spread them worse than an open sewer in full summer ;) | 22:47 |
| salv-orl_ | we are aware of all of this | 22:47 |
| salv-orl_ | well at least I am | 22:47 |
| salv-orl_ | abhiraut is looking at the failures concerning the securitty group issues | 22:47 |
| salv-orl_ | and it seems one patch from enikanorov might fix those | 22:47 |
| boris-42 | salv-orl_: ;) | 22:47 |
| *** dave-mccowan has joined #openstack-neutron | 22:47 | |
| salv-orl_ | for the issues concerning quota reservations, I have temporarily disabled that code until I fix it | 22:48 |
| salv-orl_ | as we did not see any other issue in rally logs for neutron we think that these are the ones we need to chase | 22:48 |
| *** r-daneel has quit IRC | 22:49 | |
| *** mickeys has quit IRC | 22:49 | |
| *** ajmiller has quit IRC | 22:49 | |
| *** srilatha has joined #openstack-neutron | 22:51 | |
| *** changzhi has joined #openstack-neutron | 22:51 | |
| *** baoli has joined #openstack-neutron | 22:51 | |
| *** srilatha has joined #openstack-neutron | 22:51 | |
| *** Piet has quit IRC | 22:53 | |
| *** signed8bit has quit IRC | 22:54 | |
| *** baoli has quit IRC | 22:56 | |
| *** zzzeek has quit IRC | 22:56 | |
| regXboi | is there a way within logstash to compare values from two different records? | 22:56 |
| regXboi | compare in this case translates as match | 22:56 |
| *** britthouser has quit IRC | 22:56 | |
| clarkb | regXboi: you want to say field1 == field2 ? | 22:58 |
| abhiraut | salv-orl_ so far so good with enikanorov patch | 22:58 |
| clarkb | I don't think you can with a single query at least maybe not without a newer version of elasticsearch | 22:58 |
| clarkb | I want ot say that newer es supports something similar to that | 22:58 |
| regXboi | clarkb: too bad - looks like I'll have to do this the hard way | 22:58 |
| abhiraut | lets just wait to see how responds to it after the recheck | 22:58 |
| abhiraut | rally* | 22:58 |
| *** VikasC has quit IRC | 23:00 | |
| *** mickeyspiegel has quit IRC | 23:00 | |
| *** mickeys has joined #openstack-neutron | 23:00 | |
| *** dannywil_ has joined #openstack-neutron | 23:01 | |
| *** banix has quit IRC | 23:02 | |
| *** tfukushima has joined #openstack-neutron | 23:03 | |
| *** dannywilson has quit IRC | 23:03 | |
| *** Sukhdev has quit IRC | 23:05 | |
| *** lucas-hungry has quit IRC | 23:08 | |
| *** DWfuturetec has quit IRC | 23:09 | |
| *** lodgenbd has quit IRC | 23:10 | |
| *** rushil_ has quit IRC | 23:10 | |
| *** rushil has joined #openstack-neutron | 23:10 | |
| salv-orl_ | abhiraut: I think I'll see that tomorrow morning, goodnight! | 23:11 |
| abhiraut | sleep well | 23:11 |
| *** Piet has joined #openstack-neutron | 23:12 | |
| *** aranjan has quit IRC | 23:13 | |
| *** dave-mccowan has quit IRC | 23:13 | |
| *** amotoki_ has joined #openstack-neutron | 23:15 | |
| *** aranjan has joined #openstack-neutron | 23:17 | |
| *** mwagner_lap has joined #openstack-neutron | 23:19 | |
| *** amotoki_ has quit IRC | 23:19 | |
| *** piyanai has quit IRC | 23:26 | |
| *** piyanai has joined #openstack-neutron | 23:27 | |
| *** klindgren has joined #openstack-neutron | 23:28 | |
| *** chlong has quit IRC | 23:28 | |
| *** banix has joined #openstack-neutron | 23:31 | |
| *** tflynn_ has joined #openstack-neutron | 23:31 | |
| *** achanda has joined #openstack-neutron | 23:33 | |
| *** Piet has quit IRC | 23:33 | |
| *** dims_ has joined #openstack-neutron | 23:33 | |
| *** tflynn has quit IRC | 23:35 | |
| *** azbiswas has quit IRC | 23:35 | |
| *** azbiswas has joined #openstack-neutron | 23:35 | |
| *** tflynn_ has quit IRC | 23:35 | |
| *** dims has quit IRC | 23:36 | |
| *** gangil has quit IRC | 23:36 | |
| *** shoutm has joined #openstack-neutron | 23:37 | |
| *** fedexo has joined #openstack-neutron | 23:37 | |
| *** Piet has joined #openstack-neutron | 23:37 | |
| *** alex_klimov has quit IRC | 23:38 | |
| *** dims_ has quit IRC | 23:39 | |
| HenryG | abhiraut: ping | 23:39 |
| *** dims has joined #openstack-neutron | 23:39 | |
| abhiraut | HenryG: pong | 23:39 |
| *** azbiswas has quit IRC | 23:39 | |
| *** lilynn has quit IRC | 23:40 | |
| *** klindgren has quit IRC | 23:40 | |
| HenryG | abhiraut: https://review.openstack.org/#/c/203384/8/neutron/plugins/ml2/plugin.py | 23:40 |
| *** dave-mccowan has joined #openstack-neutron | 23:40 | |
| *** lilynn has joined #openstack-neutron | 23:40 | |
| HenryG | If you get time, can you try your testing without that part of the patch? | 23:40 |
| abhiraut | HenryG: aye aye | 23:41 |
| HenryG | abhiraut: thanks | 23:41 |
| abhiraut | i think the most relevant part of the patch is the sec_grp part | 23:42 |
| *** piyanai has quit IRC | 23:42 | |
| HenryG | yep, 2 < ∞ | 23:43 |
| HenryG | Even for small values of ∞ | 23:43 |
| abhiraut | actually i didnt even try to create ports | 23:43 |
| abhiraut | but let me give that a shot as well | 23:43 |
| abhiraut | :) | 23:43 |
| HenryG | thanks | 23:44 |
| *** dims has quit IRC | 23:44 | |
| *** shashank_hegde has joined #openstack-neutron | 23:44 | |
| *** gangil has joined #openstack-neutron | 23:44 | |
| *** debedb has quit IRC | 23:45 | |
| *** Fdaisuke has joined #openstack-neutron | 23:45 | |
| *** ameybhide has quit IRC | 23:46 | |
| *** azbiswas has joined #openstack-neutron | 23:47 | |
| *** Piet has quit IRC | 23:47 | |
| *** rpothier has joined #openstack-neutron | 23:48 | |
| *** ajmiller_ has quit IRC | 23:51 | |
| *** piyanai has joined #openstack-neutron | 23:52 | |
| *** tiswanso has joined #openstack-neutron | 23:53 | |
| *** Piet has joined #openstack-neutron | 23:53 | |
| *** bharath has quit IRC | 23:54 | |
| abhiraut | HenryG: looks fine to me | 23:55 |
| HenryG | abhiraut: you tested create ports? | 23:55 |
| HenryG | with or without @wrap_db_retry on _create_port_db()? | 23:56 |
| *** edmondsw has quit IRC | 23:56 | |
| abhiraut | yeah .. without the wrap | 23:57 |
| *** tiswanso has quit IRC | 23:58 | |
| abhiraut | tried create_port without the wrap and didnt see any issue so far.. although i must say that my testing is not as scaled as you would see on a rally job :) | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
