| *** ywu_ has quit IRC | 00:00 | |
| *** ywu_ has joined #openstack-dev | 00:00 | |
| *** nati_ueno has quit IRC | 00:01 | |
| *** nati_ueno has joined #openstack-dev | 00:01 | |
| *** jakedahn is now known as jakedahn_zz | 00:03 | |
| *** ywu_ has quit IRC | 00:04 | |
| *** ywu_ has joined #openstack-dev | 00:04 | |
| *** ywu_ has quit IRC | 00:06 | |
| *** ywu_ has joined #openstack-dev | 00:07 | |
| *** markmc has quit IRC | 00:11 | |
| *** mestery has joined #openstack-dev | 00:12 | |
| openstackgerrit | A change was merged to openstack/glance: Add filesystem store driver to new func testing https://review.openstack.org/12912 | 00:13 |
|---|---|---|
| *** ywu_ has quit IRC | 00:15 | |
| *** ywu_ has joined #openstack-dev | 00:15 | |
| gongysh | Ping danwent | 00:15 |
| danwent | gongysh: pong | 00:15 |
| gongysh | sorry for not joining today's meeting. | 00:16 |
| gongysh | Where Can I get the meeting log? | 00:16 |
| *** reed has quit IRC | 00:16 | |
| danwent | gongysh: http://eavesdrop.openstack.org/meetings/quantum/2012/quantum.2012-09-17-21.00.log.html | 00:17 |
| danwent | gongysh: probably most interesting points where around metadata service | 00:17 |
| openstackgerrit | A change was merged to openstack/nova: Add man pages https://review.openstack.org/13004 | 00:18 |
| jgriffith | bcwaldon: still around? | 00:19 |
| bcwaldon | jgriffith: yep | 00:20 |
| jgriffith | So... there's an interesting issue with those nova-manage commands | 00:20 |
| *** mestery has quit IRC | 00:20 | |
| jgriffith | The problem is they're direct db calls (should've been fixed in the seperation blue print) | 00:20 |
| jgriffith | The volume.delete is no big deal, just deperecate and note that there's an extension to cover it now | 00:21 |
| jgriffith | re-attach however, I'm a bit stumped | 00:21 |
| *** vinay_ has quit IRC | 00:21 | |
| jgriffith | We grab the volume info from the DB directly just to make sure it was actually attached before | 00:21 |
| jgriffith | I thought I'd just modify this to an API call, but apparantly the service catalog needs updating if I want nova-manage to be able to make calls to cinderclient | 00:22 |
| jgriffith | It seems like a similar admin extension to nova would be the right answer here | 00:22 |
| bcwaldon | jgriffith: that all sounds sane to me | 00:23 |
| *** mestery has joined #openstack-dev | 00:23 | |
| *** Guest__ has quit IRC | 00:23 | |
| *** ncode has quit IRC | 00:24 | |
| jgriffith | So the question is, given where things are at in the release cycle... just mark *both* delete and reattach as deprecated and open a bug to add an extension in Nova for the re-attach? | 00:24 |
| jgriffith | bcwaldon: Or try and figure out how to make the cinderclient call work? | 00:24 |
| bcwaldon | jgriffith: when you say 'deprecated', do you mean they dont work or that they still work but will be removed shortly? | 00:25 |
| vishy | jgriffith: i think deprecating the nova-manage commands is fine | 00:25 |
| vishy | * meaning remove them | 00:25 |
| bcwaldon | vishy: this late? | 00:25 |
| vishy | bcwaldon: they don't work | 00:25 |
| vishy | why leave them in | 00:25 |
| bcwaldon | ah, ok, I didnt get that piece earlier | 00:26 |
| bcwaldon | if they dont work at all, drop them | 00:26 |
| bcwaldon | deprecation doesnt help anybody | 00:26 |
| jgriffith | vishy: that was my first thought, except they do work if you're still using nova-vol | 00:26 |
| vishy | i mean i guess they still...yeah | 00:26 |
| *** amotoki has quit IRC | 00:26 | |
| jgriffith | So I thought deprecate as nova-vol only | 00:26 |
| vishy | just check the api setting in nova-manage and fail if it is set to cinder | 00:26 |
| bcwaldon | jgriffith: hrm, so I'm too confused to help at this point... | 00:26 |
| jgriffith | Works for me | 00:26 |
| jgriffith | vishy: We'll have to do something else for the re-attach but I'll leave it as a new bug | 00:27 |
| vishy | jgriffith: we just found a bunch of errors in the nova migration which converted to snapshot uuids | 00:27 |
| jgriffith | vishy: That's bad news | 00:27 |
| jgriffith | vishy: I was fixing the ones in the nova-manage at least | 00:27 |
| jgriffith | vishy: Is there a bug logged on the errors? | 00:27 |
| vishy | jgriffith: the result of the issue is that all the foreign keys to volumes table are gone | 00:29 |
| vishy | mikal is adding a new migration to bring them back | 00:29 |
| jgriffith | vishy: Oh crap | 00:29 |
| jgriffith | vishy: What the heck did I miss? | 00:30 |
| jgriffith | vishy: I never mapped the relationship in the migration! | 00:31 |
| vishy | jgriffith: bug 1052244 | 00:31 |
| uvirtbot | Launchpad bug 1052244 in nova "DB migration 91 breaks snapshots.volume_id -> volumes.id relation" [Critical,In progress] https://launchpad.net/bugs/1052244 | 00:31 |
| vishy | or maybe bug 1052220 | 00:32 |
| uvirtbot | Launchpad bug 1052220 in nova "DB migration 91 fails on precise with mysql" [Critical,In progress] https://launchpad.net/bugs/1052220 | 00:32 |
| vishy | we are fixing them, but anyway, I suppose we will need a similar migration in cinder | 00:32 |
| jgriffith | Alright, I'll port it over tonight | 00:32 |
| jgriffith | vishy: err.. actually | 00:32 |
| jgriffith | vishy: shouldn't affect cinder as there's no id-->uuid conversion | 00:33 |
| openstackgerrit | A change was merged to openstack/nova: Include volume_metadata with object on vol create https://review.openstack.org/10461 | 00:35 |
| vishy | jgriffith: well the dbs should match | 00:36 |
| jgriffith | vishy: Yeah, if it's a FK issue it'll need fixed there as well | 00:37 |
| jgriffith | vishy: I'll make sure it's synced up | 00:37 |
| *** s0mik has quit IRC | 00:38 | |
| *** gongysh has quit IRC | 00:38 | |
| *** rnirmal has joined #openstack-dev | 00:56 | |
| *** johnpur has quit IRC | 00:57 | |
| *** pvo has quit IRC | 00:57 | |
| *** pvo has joined #openstack-dev | 00:57 | |
| *** littleidea has joined #openstack-dev | 00:58 | |
| *** xchu has joined #openstack-dev | 00:59 | |
| *** pvo has joined #openstack-dev | 00:59 | |
| *** pvo has quit IRC | 00:59 | |
| *** pvo has joined #openstack-dev | 01:00 | |
| *** ChanServ sets mode: +v pvo | 01:00 | |
| *** datsun180b has quit IRC | 01:00 | |
| *** datsun180b has joined #openstack-dev | 01:02 | |
| *** edygarcia has joined #openstack-dev | 01:09 | |
| arosen1 | hi vishy would you mind checking this out when you get a sec (https://review.openstack.org/#/c/13043/) ? Would it be possible to merge this in before the next rc? | 01:09 |
| *** danwent has quit IRC | 01:14 | |
| *** mestery has quit IRC | 01:14 | |
| *** ncode has joined #openstack-dev | 01:16 | |
| *** amotoki has joined #openstack-dev | 01:18 | |
| *** PotHix has quit IRC | 01:19 | |
| *** chrisfer has quit IRC | 01:22 | |
| openstackgerrit | A change was merged to openstack/nova: Add admin actions extension https://review.openstack.org/13049 | 01:26 |
| openstackgerrit | A change was merged to openstack/nova: Allows waiting timers in libvirt to raise NotFound https://review.openstack.org/12819 | 01:27 |
| *** jog0 has quit IRC | 01:27 | |
| *** Guest___ has joined #openstack-dev | 01:28 | |
| *** nati_ueno has quit IRC | 01:31 | |
| *** wenjianhn has joined #openstack-dev | 01:35 | |
| *** mnaser has quit IRC | 01:39 | |
| *** xchu has quit IRC | 01:41 | |
| *** amotoki has quit IRC | 01:47 | |
| *** xchu has joined #openstack-dev | 01:47 | |
| vishy | arosen1: commented, it can get in if it merges today | 01:48 |
| arosen1 | vishy: thanks for the review i'll make those changes . | 01:49 |
| mnewby | vishy: question about security vulnerabilities. Is there any special procedure to follow to submit a fix for a vulnerability through gerrit? | 01:49 |
| vishy | mnewby: is it a public vulnerability? | 01:50 |
| mnewby | public in what sense? | 01:50 |
| mnewby | The bug in launchpad is not public, if that's what you mean. | 01:50 |
| vishy | mnewby: in that case do not submit through gerrit | 01:52 |
| mnewby | vishy: ok. What do I do? | 01:52 |
| vishy | mnewby: post a patch in the bug for preapproval | 01:52 |
| mnewby | Ah, ok. | 01:52 |
| mnewby | Are there any docs on the protocol to follow? | 01:53 |
| *** jimp has joined #openstack-dev | 01:53 | |
| *** colinmcnamara has joined #openstack-dev | 01:54 | |
| *** jimp has quit IRC | 01:55 | |
| *** jimp has joined #openstack-dev | 01:57 | |
| *** alexpilotti has quit IRC | 02:00 | |
| rmk | vishy: Got a few minutes? | 02:07 |
| *** jimp has quit IRC | 02:07 | |
| *** stanchan has quit IRC | 02:07 | |
| *** jimp has joined #openstack-dev | 02:08 | |
| *** zaitcev has quit IRC | 02:12 | |
| *** jimp has quit IRC | 02:17 | |
| *** rpedde_away is now known as rpedde | 02:20 | |
| *** jimp has joined #openstack-dev | 02:20 | |
| *** ayoung has quit IRC | 02:20 | |
| *** tgall_foo has joined #openstack-dev | 02:21 | |
| *** tgall_foo has quit IRC | 02:21 | |
| *** tgall_foo has joined #openstack-dev | 02:21 | |
| *** jimp has quit IRC | 02:22 | |
| *** novas0x2a|laptop has quit IRC | 02:22 | |
| *** ncode has quit IRC | 02:30 | |
| *** spn has joined #openstack-dev | 02:32 | |
| *** jtran has left #openstack-dev | 02:32 | |
| *** mnewby has quit IRC | 02:34 | |
| *** tgall_foo has quit IRC | 02:35 | |
| *** jimp has joined #openstack-dev | 02:38 | |
| *** zhuadl has quit IRC | 02:45 | |
| *** mattstep has joined #openstack-dev | 02:46 | |
| *** danwent has joined #openstack-dev | 02:48 | |
| *** spn has quit IRC | 02:50 | |
| *** mattstep has quit IRC | 02:51 | |
| *** johnpur has joined #openstack-dev | 02:58 | |
| *** johnpur has quit IRC | 02:58 | |
| *** datsun180b has quit IRC | 03:00 | |
| *** Mandell has quit IRC | 03:01 | |
| *** mattstep has joined #openstack-dev | 03:01 | |
| *** adalbas has quit IRC | 03:04 | |
| *** zhuadl has joined #openstack-dev | 03:07 | |
| *** zhuadl has quit IRC | 03:12 | |
| *** zhuadl has joined #openstack-dev | 03:14 | |
| *** markmcclain has joined #openstack-dev | 03:15 | |
| *** otter2 has joined #openstack-dev | 03:16 | |
| *** otter3 has joined #openstack-dev | 03:17 | |
| *** otter2 has quit IRC | 03:17 | |
| *** otter1 has quit IRC | 03:17 | |
| *** zbitter has joined #openstack-dev | 03:18 | |
| *** jimp has quit IRC | 03:19 | |
| *** s0mik has joined #openstack-dev | 03:20 | |
| *** jog0 has joined #openstack-dev | 03:20 | |
| *** stanchan has joined #openstack-dev | 03:21 | |
| *** zaneb has quit IRC | 03:21 | |
| *** xchu has quit IRC | 03:25 | |
| *** reed has joined #openstack-dev | 03:31 | |
| *** littleidea has quit IRC | 03:36 | |
| *** lifeless has quit IRC | 03:36 | |
| *** zhuadl_ has joined #openstack-dev | 03:36 | |
| *** edygarcia has quit IRC | 03:38 | |
| *** zhuadl has quit IRC | 03:39 | |
| *** zhuadl_ is now known as zhuadl | 03:39 | |
| *** nunosantos has quit IRC | 03:41 | |
| *** jimp has joined #openstack-dev | 03:43 | |
| *** lifeless has joined #openstack-dev | 03:44 | |
| *** littleidea has joined #openstack-dev | 03:45 | |
| *** jimp has quit IRC | 03:52 | |
| *** adjohn has quit IRC | 03:52 | |
| *** almaisan-away is now known as al-maisan | 03:53 | |
| *** hub_cap has quit IRC | 03:59 | |
| *** vinay_ has joined #openstack-dev | 04:06 | |
| *** vanchester has joined #openstack-dev | 04:06 | |
| *** pixelbeat has quit IRC | 04:07 | |
| *** vanchester has quit IRC | 04:11 | |
| *** vanchester has joined #openstack-dev | 04:12 | |
| *** torandu has quit IRC | 04:13 | |
| arosen1 | hey, vishy any chance you can check out that patch again? :) https://review.openstack.org/#/c/13043/ | 04:13 |
| *** vanchester has quit IRC | 04:13 | |
| arosen1 | hoping it can get in for the rc | 04:13 |
| *** markmcclain has quit IRC | 04:14 | |
| *** colinmcnamara has quit IRC | 04:14 | |
| *** markmcclain has joined #openstack-dev | 04:15 | |
| *** xchu has joined #openstack-dev | 04:15 | |
| *** torandu has joined #openstack-dev | 04:19 | |
| *** adjohn has joined #openstack-dev | 04:22 | |
| *** al-maisan is now known as almaisan-away | 04:24 | |
| *** cp16net is now known as cp16net|away | 04:31 | |
| *** lloydde has joined #openstack-dev | 04:36 | |
| *** markmcclain has quit IRC | 04:39 | |
| *** jimp has joined #openstack-dev | 04:39 | |
| *** arosen1 has quit IRC | 04:39 | |
| *** jtran has joined #openstack-dev | 04:40 | |
| *** jtran has left #openstack-dev | 04:40 | |
| *** Tross has quit IRC | 04:43 | |
| *** vinay_ has quit IRC | 04:45 | |
| *** garyk has quit IRC | 04:46 | |
| *** rpedde is now known as rpedde_away | 04:47 | |
| *** ywu_ has quit IRC | 04:48 | |
| *** ywu_ has joined #openstack-dev | 04:48 | |
| *** jimp has quit IRC | 04:49 | |
| *** ywu_ has quit IRC | 04:51 | |
| *** ywu_ has joined #openstack-dev | 04:51 | |
| *** vanchester has joined #openstack-dev | 04:54 | |
| *** reed has quit IRC | 04:55 | |
| *** ywu_ has quit IRC | 04:56 | |
| *** Guest___ has quit IRC | 04:56 | |
| *** ywu_ has joined #openstack-dev | 04:56 | |
| *** ywu_ has quit IRC | 05:00 | |
| *** ywu_ has joined #openstack-dev | 05:00 | |
| *** danwent has quit IRC | 05:00 | |
| *** garyk has joined #openstack-dev | 05:00 | |
| jgriffith | exit | 05:01 |
| *** hattwick has quit IRC | 05:01 | |
| *** Guest__ has joined #openstack-dev | 05:01 | |
| *** Mandell has joined #openstack-dev | 05:03 | |
| vishy | arosen1: commented. Still needs a docstring | 05:04 |
| *** koolhead17 has joined #openstack-dev | 05:06 | |
| *** ywu_ has quit IRC | 05:06 | |
| *** ywu_ has joined #openstack-dev | 05:07 | |
| *** Tross has joined #openstack-dev | 05:07 | |
| *** alobbs has joined #openstack-dev | 05:07 | |
| *** xchu has quit IRC | 05:11 | |
| *** thovden has joined #openstack-dev | 05:12 | |
| *** hattwick has joined #openstack-dev | 05:12 | |
| *** torandu has quit IRC | 05:13 | |
| *** thovden has quit IRC | 05:14 | |
| *** ywu_ has quit IRC | 05:15 | |
| *** salv-orlando has joined #openstack-dev | 05:15 | |
| *** ywu_ has joined #openstack-dev | 05:15 | |
| *** adjohn has quit IRC | 05:15 | |
| *** xchu has joined #openstack-dev | 05:19 | |
| *** samkottler is now known as samkottler|zzz | 05:26 | |
| *** armaan has joined #openstack-dev | 05:27 | |
| *** armaan has left #openstack-dev | 05:27 | |
| *** ywu_ has quit IRC | 05:30 | |
| *** ywu_ has joined #openstack-dev | 05:30 | |
| *** armaan1 has joined #openstack-dev | 05:33 | |
| *** danwent has joined #openstack-dev | 05:36 | |
| *** armaan1 has quit IRC | 05:38 | |
| *** tserong has quit IRC | 05:38 | |
| *** arosen1 has joined #openstack-dev | 05:40 | |
| arosen1 | vishy: do you know why jenkins is giving me a -1 with: http://logs.openstack.org/13043/8/check/gate-nova-merge/8822 : UNSTABLE | 05:41 |
| *** ywu_ has quit IRC | 05:45 | |
| *** ywu_ has joined #openstack-dev | 05:45 | |
| *** armaan has joined #openstack-dev | 05:45 | |
| *** armaan has left #openstack-dev | 05:46 | |
| *** ywu_ has quit IRC | 05:51 | |
| *** ywu_ has joined #openstack-dev | 05:51 | |
| *** ywu_ has quit IRC | 05:56 | |
| *** ywu_ has joined #openstack-dev | 05:56 | |
| *** ywu_ has quit IRC | 06:00 | |
| *** ywu_ has joined #openstack-dev | 06:00 | |
| *** soren_ has quit IRC | 06:00 | |
| *** creiht has quit IRC | 06:00 | |
| *** _0x44 has quit IRC | 06:00 | |
| *** pmyers_ has quit IRC | 06:00 | |
| *** creiht has joined #openstack-dev | 06:00 | |
| *** ChanServ sets mode: +v creiht | 06:00 | |
| *** shang has quit IRC | 06:01 | |
| *** pballand has quit IRC | 06:01 | |
| *** chalupaul has quit IRC | 06:01 | |
| *** pmyers has joined #openstack-dev | 06:01 | |
| *** Guest5487 has quit IRC | 06:01 | |
| *** soren has joined #openstack-dev | 06:01 | |
| *** ChanServ sets mode: +v soren | 06:01 | |
| *** _0x44 has joined #openstack-dev | 06:01 | |
| *** jgb has joined #openstack-dev | 06:01 | |
| *** chalupaul has joined #openstack-dev | 06:01 | |
| *** jgb is now known as Guest78056 | 06:02 | |
| *** tserong has joined #openstack-dev | 06:02 | |
| *** tserong has joined #openstack-dev | 06:02 | |
| *** zhuadl has quit IRC | 06:02 | |
| *** GheRivero has joined #openstack-dev | 06:04 | |
| *** ywu_ has quit IRC | 06:04 | |
| *** Guest__ has quit IRC | 06:04 | |
| *** ywu_ has joined #openstack-dev | 06:04 | |
| *** cp16net|away is now known as cp16net | 06:04 | |
| *** EmilienM has joined #openstack-dev | 06:05 | |
| *** ywu_ has quit IRC | 06:06 | |
| *** ywu_ has joined #openstack-dev | 06:07 | |
| *** EmilienM has quit IRC | 06:09 | |
| *** EmilienM_ has joined #openstack-dev | 06:10 | |
| *** EmilienM_ is now known as EmilienM | 06:10 | |
| *** zhuadl has joined #openstack-dev | 06:10 | |
| *** EmilienM_ has joined #openstack-dev | 06:10 | |
| *** EmilienM_ is now known as EmilienM | 06:11 | |
| *** EmilienM has quit IRC | 06:11 | |
| *** EmilienM_ has joined #openstack-dev | 06:11 | |
| *** EmilienM_ is now known as EmilienM | 06:12 | |
| *** pballand has joined #openstack-dev | 06:12 | |
| *** eglynn has joined #openstack-dev | 06:12 | |
| *** EmilienM has quit IRC | 06:12 | |
| *** shang has joined #openstack-dev | 06:13 | |
| vishy | rmk: any idea which versions of libvirt support attachDeviceFlags? | 06:13 |
| vishy | arosen1: not sure | 06:13 |
| *** ywu_ has quit IRC | 06:14 | |
| rmk | vishy: Good question. What do you think of that approach? | 06:14 |
| *** ywu_ has joined #openstack-dev | 06:15 | |
| *** EmilienM has joined #openstack-dev | 06:16 | |
| *** mindpixel has joined #openstack-dev | 06:17 | |
| *** EmilienM has joined #openstack-dev | 06:17 | |
| *** ywu_ has quit IRC | 06:17 | |
| *** ywu_ has joined #openstack-dev | 06:17 | |
| *** lloydde has quit IRC | 06:18 | |
| *** ywu_ has quit IRC | 06:20 | |
| *** ywu_ has joined #openstack-dev | 06:20 | |
| arosen | vishy: hrm i'll I guess I'll just have to git commit --amend (and change something and try again tomorrow). It seems like several other patches are hitting the same message from jenkins. | 06:23 |
| vishy | arosen: you can post a message with recheck to send it in again | 06:24 |
| vishy | arosen: you saw my comment about updating the docstring? | 06:25 |
| *** s0mik has quit IRC | 06:26 | |
| arosen | vishy: yes, I did. I added the docstring. | 06:27 |
| arosen | vishy: sorry this took some many revisions... | 06:27 |
| vishy | arosen: i was actually asking for a docstring in db/api.py not db/sqlalchemy/api.py | 06:28 |
| vishy | since that is the interface | 06:28 |
| vishy | also the commit message still refers to the negative | 06:28 |
| vishy | which is very confusing | 06:28 |
| vishy | ensure_default_security_group() does not call sgh | 06:28 |
| vishy | is a description of the bug not a description of the patch | 06:28 |
| vishy | makes ensure_default_security_group call sgh | 06:29 |
| vishy | would be a description of the patch | 06:29 |
| * vishy is nitpicking :( | 06:29 | |
| *** ywu_ has quit IRC | 06:30 | |
| *** ywu_ has joined #openstack-dev | 06:30 | |
| vishy | arosen: the merge is definitely broken, hopefully someone from openstack-infra can fix it :) | 06:30 |
| arosen | vishy: no worries :) | 06:30 |
| rmk | vishy: Looks like attachDeviceFlags has been there at least since early 2010. Maybe sooner, still looking through libvirt commits. | 06:31 |
| vishy | rmk: in that case it is probably fine :) | 06:31 |
| vishy | 2010 should be 9.0ish i think | 06:31 |
| *** koolhead17 has quit IRC | 06:32 | |
| arosen | vishy: I'm still not exactly clear what you are saying about the doc string "still missing a docstring describing the return values in db/api.py | 06:32 |
| arosen | " I didn't change the return values in db/api.py. I just added the call to sgh there. | 06:32 |
| arosen | i did change the return in sqlalchemy.api.py where i added the doc string. | 06:33 |
| vishy | arosen: db/api.py is the definition for the interface that is implemented by sqlalchemy/api.py | 06:33 |
| *** Mandell has quit IRC | 06:34 | |
| arosen | ah i see thanks | 06:34 |
| vishy | so if someone was righting for example mongodb/api.py they would look at db/api.py to figure out what the method should return | 06:34 |
| *** wenjianhn has quit IRC | 06:34 | |
| vishy | so the return value should be documented there | 06:34 |
| vishy | arosen: sorry for the confusion. :) | 06:35 |
| rmk | vishy: What's the oldest supported version of libvirt? | 06:35 |
| rmk | Yeah we're good for sure, it's been there for a long time. | 06:38 |
| vishy | rmk: most things work in 9.2 | 06:41 |
| vishy | which is oneiric version | 06:41 |
| vishy | older than that uknown | 06:41 |
| vishy | *unknown | 06:41 |
| *** ywu_ has quit IRC | 06:45 | |
| *** ywu_ has joined #openstack-dev | 06:45 | |
| *** mrunge has joined #openstack-dev | 06:49 | |
| *** e1mer has quit IRC | 06:51 | |
| *** ywu_ has quit IRC | 06:51 | |
| *** ywu_ has joined #openstack-dev | 06:51 | |
| *** rafaduran has joined #openstack-dev | 06:54 | |
| *** littleidea has quit IRC | 06:54 | |
| *** ywu_ has quit IRC | 06:56 | |
| *** ywu_ has joined #openstack-dev | 06:56 | |
| *** ywu_ has quit IRC | 07:00 | |
| *** ywu_ has joined #openstack-dev | 07:00 | |
| *** vanchester has quit IRC | 07:01 | |
| *** ywu_ has quit IRC | 07:06 | |
| *** ywu_ has joined #openstack-dev | 07:07 | |
| rmk | vishy: I see what you meant now about to_xml() | 07:09 |
| vishy | rmk: oh? | 07:09 |
| rmk | well, we were discussing always using to_xml to generate libvirt xml files rather than ever reading them from disk | 07:10 |
| rmk | if create_domain_and_network() called that every time, the attach code wouldn't even need to run | 07:10 |
| *** jog0 has quit IRC | 07:11 | |
| *** reidrac has joined #openstack-dev | 07:11 | |
| *** ogelbukh has quit IRC | 07:12 | |
| vishy | rmk: yeah :| | 07:12 |
| * ttx waves | 07:12 | |
| rmk | This could simply things a lot across the board | 07:12 |
| * vishy hides | 07:12 | |
| rmk | lol | 07:12 |
| * ttx looks for vishy | 07:13 | |
| ttx | rmk: seen him ? | 07:13 |
| rmk | ttx: he's pretty stealthy | 07:13 |
| *** reidrac has quit IRC | 07:13 | |
| ttx | vishy: that bugcount doesn't seem to want to go down anytime | 07:14 |
| vishy | ttx: yeah as soon as i think we have it nailed some new nasty one rears its head | 07:14 |
| ttx | vishy: when we are down to 5 I'd like to cut the branch, will give us better control | 07:14 |
| vishy | ttx: the migration ones are all critical | 07:15 |
| rmk | any I can help with? | 07:15 |
| *** ywu_ has quit IRC | 07:15 | |
| *** ywu_ has joined #openstack-dev | 07:15 | |
| vishy | rmk: I think everything is being worked on | 07:15 |
| *** xj has joined #openstack-dev | 07:15 | |
| *** mrunge has quit IRC | 07:15 | |
| ttx | vishy: should #1052157 (resource_tracker limits Ram even when RamFilter is diabled) be assigned to Brian Elliott too ? | 07:16 |
| vishy | ttx: not sure, but I don't think that one is release critical | 07:16 |
| *** ogelbukh has joined #openstack-dev | 07:16 | |
| vishy | ttx: right now the merge gate is broken so nothing is merging | 07:16 |
| xj | Hi,all why there needs two topic queues for nova-compute?The queue with routing key:"compute.host" is obvious,but what about the queue with routing key "compute"?what is this for?when does it use this queue? | 07:16 |
| ttx | uh | 07:16 |
| *** spn has joined #openstack-dev | 07:16 | |
| *** sandywalsh_ has joined #openstack-dev | 07:18 | |
| *** markmc has joined #openstack-dev | 07:19 | |
| *** sandywalsh has quit IRC | 07:20 | |
| *** mrunge has joined #openstack-dev | 07:20 | |
| *** reidrac has joined #openstack-dev | 07:21 | |
| rmk | I actually can't think of a reason why we ever care about anything other than what comes back from to_xml | 07:22 |
| rmk | Would fix so many problems | 07:22 |
| *** apevec has joined #openstack-dev | 07:24 | |
| *** apevec has joined #openstack-dev | 07:24 | |
| *** zbitter has quit IRC | 07:27 | |
| *** zbitter has joined #openstack-dev | 07:27 | |
| *** lloydde has joined #openstack-dev | 07:28 | |
| *** pixelbeat has joined #openstack-dev | 07:29 | |
| *** ywu_ has quit IRC | 07:30 | |
| *** ywu_ has joined #openstack-dev | 07:30 | |
| *** m4xmr has joined #openstack-dev | 07:32 | |
| *** zing_ has joined #openstack-dev | 07:32 | |
| *** lloydde has quit IRC | 07:33 | |
| *** jimp has joined #openstack-dev | 07:33 | |
| *** m4xmr has left #openstack-dev | 07:37 | |
| *** e1mer has joined #openstack-dev | 07:38 | |
| *** zhuadl has quit IRC | 07:39 | |
| *** e1mer has joined #openstack-dev | 07:40 | |
| jeblair | ttx: the gate is fixed | 07:41 |
| ttx | jeblair: cool | 07:42 |
| *** ywu_ has quit IRC | 07:45 | |
| *** ywu_ has joined #openstack-dev | 07:45 | |
| *** jkyle has quit IRC | 07:48 | |
| *** winston-d has joined #openstack-dev | 07:51 | |
| *** ywu_ has quit IRC | 07:51 | |
| *** ywu_ has joined #openstack-dev | 07:51 | |
| winston-d | markmc, hi | 07:51 |
| markmc | winston-d, hi | 07:51 |
| winston-d | markmc, i saw your comment about rpc version compatibility on the nova-volume bug fix. | 07:52 |
| *** e1mer has quit IRC | 07:53 | |
| winston-d | a bug fix to one version make version # increase .1? | 07:53 |
| markmc | no, it's about API compatibility | 07:54 |
| markmc | if you update the scheduler with your fix | 07:54 |
| markmc | the API service should keep working, even without your fix applied | 07:54 |
| openstackgerrit | A change was merged to openstack/nova: Fixes error handling during schedule_run_instance https://review.openstack.org/13159 | 07:55 |
| markmc | in the case, the API service is sending RPC messages with version=2.0 and no reservations param | 07:55 |
| markmc | your patch should allow that to continue to work | 07:55 |
| markmc | when the client is sending a reservations param, it should send version=2.1 | 07:55 |
| markmc | it's a protocol version | 07:55 |
| winston-d | i see | 07:55 |
| *** ywu_ has quit IRC | 07:56 | |
| *** ywu_ has joined #openstack-dev | 07:56 | |
| winston-d | so a change is needed to scheduler/rpcapi.py as well? | 07:57 |
| markmc | yes | 07:58 |
| markmc | to pass version=2.1 when calling create_volume | 07:58 |
| *** janisg has joined #openstack-dev | 07:58 | |
| winston-d | ok, got it. I'll update the patch right away. thx for the review. | 07:59 |
| rmk | markmc: hey there - updated my volume/resize patch based on your comments | 07:59 |
| rmk | markmc: https://review.openstack.org/13071 | 07:59 |
| markmc | rmk, yeah, saw that - haven't had a chance to review yet | 07:59 |
| *** ywu_ has quit IRC | 08:00 | |
| rmk | np was just about to head to bed so figured I'd ping and see if anything else needed to be changed | 08:00 |
| *** ywu_ has joined #openstack-dev | 08:00 | |
| *** derekh has joined #openstack-dev | 08:00 | |
| openstackgerrit | A change was merged to openstack/quantum: modify the wrong phy_brs into phys_brs. https://review.openstack.org/13179 | 08:02 |
| *** ywu_ has quit IRC | 08:04 | |
| *** ywu_ has joined #openstack-dev | 08:04 | |
| spn | trivial change in https://review.openstack.org/13188/ | 08:05 |
| *** zhuadl has joined #openstack-dev | 08:07 | |
| *** ywu_ has quit IRC | 08:07 | |
| *** ywu_ has joined #openstack-dev | 08:07 | |
| *** jimp has quit IRC | 08:10 | |
| *** markmc is now known as mcaway | 08:11 | |
| openstackgerrit | A change was merged to openstack/cinder: Fix volume deletion when device mapper is used https://review.openstack.org/12396 | 08:12 |
| *** adjohn has joined #openstack-dev | 08:13 | |
| *** zbitter has quit IRC | 08:14 | |
| *** zaneb has joined #openstack-dev | 08:14 | |
| zing_ | is there an equivalent to recheck for smokestack ? | 08:14 |
| *** ywu_ has quit IRC | 08:14 | |
| *** ywu_ has joined #openstack-dev | 08:14 | |
| *** almaisan-away is now known as al-maisan | 08:15 | |
| *** darraghb has joined #openstack-dev | 08:18 | |
| *** zhuadl_ has joined #openstack-dev | 08:18 | |
| *** olamalaf has joined #openstack-dev | 08:19 | |
| *** wenjianhn has joined #openstack-dev | 08:19 | |
| *** zhuadl has quit IRC | 08:20 | |
| *** zhuadl_ is now known as zhuadl | 08:20 | |
| *** adjohn has quit IRC | 08:20 | |
| *** adjohn has joined #openstack-dev | 08:21 | |
| *** mawagon1 has quit IRC | 08:22 | |
| *** adjohn has quit IRC | 08:25 | |
| zykes- | danwent: ping | 08:25 |
| *** ywu_ has quit IRC | 08:29 | |
| *** ywu_ has joined #openstack-dev | 08:29 | |
| *** ywu_ has quit IRC | 08:30 | |
| *** ywu_ has joined #openstack-dev | 08:30 | |
| *** danpb has joined #openstack-dev | 08:37 | |
| *** ywu_ has quit IRC | 08:45 | |
| *** ywu_ has joined #openstack-dev | 08:45 | |
| *** mcaway is now known as markmc | 08:46 | |
| *** Kiall is now known as zz_Kiall | 08:48 | |
| *** ywu_ has quit IRC | 08:51 | |
| *** mnewby has joined #openstack-dev | 08:51 | |
| *** ywu_ has joined #openstack-dev | 08:51 | |
| *** maploin has joined #openstack-dev | 08:55 | |
| *** maploin has quit IRC | 08:55 | |
| *** maploin has joined #openstack-dev | 08:55 | |
| *** ywu_ has quit IRC | 08:56 | |
| *** winston-d has quit IRC | 08:56 | |
| *** zz_Kiall is now known as Kiall | 08:56 | |
| *** ywu_ has joined #openstack-dev | 08:56 | |
| *** ywu_ has quit IRC | 09:00 | |
| *** ywu_ has joined #openstack-dev | 09:00 | |
| *** sandywalsh_ has quit IRC | 09:01 | |
| *** Dr_Who has quit IRC | 09:06 | |
| *** Dr_Who has joined #openstack-dev | 09:06 | |
| *** Dr_Who has quit IRC | 09:06 | |
| *** Dr_Who has joined #openstack-dev | 09:06 | |
| *** ywu_ has quit IRC | 09:06 | |
| *** ywu_ has joined #openstack-dev | 09:07 | |
| *** ywu_ has quit IRC | 09:15 | |
| *** ywu_ has joined #openstack-dev | 09:15 | |
| *** salv-orlando_ has joined #openstack-dev | 09:19 | |
| *** eglynn has quit IRC | 09:20 | |
| *** kpepple_ has joined #openstack-dev | 09:22 | |
| *** salv-orlando has quit IRC | 09:22 | |
| *** glenc has joined #openstack-dev | 09:22 | |
| *** salv-orlando_ is now known as salv-orlando | 09:22 | |
| *** glenc_ has quit IRC | 09:22 | |
| *** kpepple has quit IRC | 09:22 | |
| *** zhuadl has quit IRC | 09:22 | |
| *** timello has quit IRC | 09:23 | |
| *** zhuadl has joined #openstack-dev | 09:27 | |
| *** ywu_ has quit IRC | 09:30 | |
| *** ywu_ has joined #openstack-dev | 09:30 | |
| *** danwent has quit IRC | 09:30 | |
| *** timello has joined #openstack-dev | 09:34 | |
| *** k4n0 has joined #openstack-dev | 09:39 | |
| *** alexpilotti has joined #openstack-dev | 09:39 | |
| *** ywu_ has quit IRC | 09:45 | |
| *** ywu_ has joined #openstack-dev | 09:45 | |
| *** ywu_ has quit IRC | 09:51 | |
| *** ywu_ has joined #openstack-dev | 09:51 | |
| *** xchu has quit IRC | 09:53 | |
| *** ywu_ has quit IRC | 09:56 | |
| *** ywu_ has joined #openstack-dev | 09:56 | |
| *** eglynn has joined #openstack-dev | 09:59 | |
| *** ywu_ has quit IRC | 10:00 | |
| *** ywu_ has joined #openstack-dev | 10:00 | |
| *** oneiroi has quit IRC | 10:00 | |
| *** \etc\bin has joined #openstack-dev | 10:02 | |
| *** \etc\bin has joined #openstack-dev | 10:02 | |
| *** oneiroi has joined #openstack-dev | 10:02 | |
| *** ywu_ has quit IRC | 10:04 | |
| *** ywu_ has joined #openstack-dev | 10:04 | |
| *** ywu_ has quit IRC | 10:07 | |
| *** ywu_ has joined #openstack-dev | 10:07 | |
| *** ywu_ has quit IRC | 10:15 | |
| *** ywu_ has joined #openstack-dev | 10:15 | |
| *** ywu_ has quit IRC | 10:17 | |
| *** ywu_ has joined #openstack-dev | 10:17 | |
| *** xj has left #openstack-dev | 10:19 | |
| *** ywu_ has quit IRC | 10:22 | |
| *** ywu_ has joined #openstack-dev | 10:22 | |
| *** armaan has joined #openstack-dev | 10:26 | |
| *** armaan has left #openstack-dev | 10:26 | |
| *** zhuadl has quit IRC | 10:26 | |
| *** al-maisan is now known as almaisan-away | 10:28 | |
| *** ywu_ has quit IRC | 10:30 | |
| *** ywu_ has joined #openstack-dev | 10:30 | |
| *** vanchester has joined #openstack-dev | 10:42 | |
| *** ywu_ has quit IRC | 10:45 | |
| *** ywu_ has joined #openstack-dev | 10:45 | |
| *** ywu_ has quit IRC | 10:51 | |
| *** ywu_ has joined #openstack-dev | 10:51 | |
| *** larsbutler has joined #openstack-dev | 10:52 | |
| *** ywu_ has quit IRC | 10:56 | |
| *** ywu_ has joined #openstack-dev | 10:56 | |
| *** sandywalsh has joined #openstack-dev | 10:56 | |
| *** ywu_ has quit IRC | 11:00 | |
| *** ywu_ has joined #openstack-dev | 11:00 | |
| *** wenjianhn has quit IRC | 11:02 | |
| *** ywu_ has quit IRC | 11:06 | |
| *** ywu_ has joined #openstack-dev | 11:07 | |
| *** kbringard has joined #openstack-dev | 11:07 | |
| *** alexpilotti has quit IRC | 11:10 | |
| *** ywu_ has quit IRC | 11:15 | |
| *** ywu_ has joined #openstack-dev | 11:15 | |
| *** adalbas has joined #openstack-dev | 11:17 | |
| *** vanchester has quit IRC | 11:21 | |
| *** chrisfer has joined #openstack-dev | 11:24 | |
| *** ywu_ has quit IRC | 11:30 | |
| *** ywu_ has joined #openstack-dev | 11:30 | |
| *** olamalaf is now known as mawagon1 | 11:32 | |
| *** pixelbeat has quit IRC | 11:38 | |
| *** maurosr has joined #openstack-dev | 11:44 | |
| *** ywu_ has quit IRC | 11:45 | |
| *** ywu_ has joined #openstack-dev | 11:45 | |
| *** markvoelker has joined #openstack-dev | 11:49 | |
| *** ywu_ has quit IRC | 11:51 | |
| *** ywu_ has joined #openstack-dev | 11:51 | |
| *** ywu_ has quit IRC | 11:56 | |
| *** ywu_ has joined #openstack-dev | 11:56 | |
| *** kbringard has quit IRC | 11:56 | |
| *** ywu_ has quit IRC | 12:00 | |
| *** ywu_ has joined #openstack-dev | 12:00 | |
| *** ywu_ has quit IRC | 12:04 | |
| *** ywu_ has joined #openstack-dev | 12:04 | |
| *** kbringard has joined #openstack-dev | 12:04 | |
| *** ywu_ has quit IRC | 12:06 | |
| *** ywu_ has joined #openstack-dev | 12:07 | |
| *** EmilienM has quit IRC | 12:07 | |
| *** EmilienM has joined #openstack-dev | 12:08 | |
| *** ywu_ has quit IRC | 12:15 | |
| *** waa has joined #openstack-dev | 12:15 | |
| *** ywu_ has joined #openstack-dev | 12:15 | |
| *** derekh is now known as derekh_afk | 12:15 | |
| *** alexpilotti has joined #openstack-dev | 12:16 | |
| *** ywu_ has quit IRC | 12:16 | |
| *** fc__ has quit IRC | 12:16 | |
| *** alunduil has quit IRC | 12:21 | |
| *** timello has quit IRC | 12:34 | |
| *** mrunge has quit IRC | 12:34 | |
| *** armaan has joined #openstack-dev | 12:35 | |
| *** armaan has left #openstack-dev | 12:35 | |
| *** timello has joined #openstack-dev | 12:35 | |
| *** derekh_afk is now known as derekh | 12:39 | |
| *** eglynn_ has joined #openstack-dev | 12:40 | |
| *** eglynn has quit IRC | 12:40 | |
| *** pixelbeat has joined #openstack-dev | 12:43 | |
| *** sandywalsh_ has joined #openstack-dev | 12:49 | |
| *** sandywalsh has quit IRC | 12:49 | |
| zykes- | salv-orlando: hi, where you involved with the l3 agent ? | 12:50 |
| zykes- | or n0ano ? | 12:50 |
| salv-orlando | zykes-: l3 agent is not my little boy, but I've looked after it when its father (danwent) is away | 12:51 |
| salv-orlando | so I know a thing or two about it | 12:52 |
| zykes- | salv-orlando: any reason it does n api calls for each router ? | 12:52 |
| openstackgerrit | A change was merged to openstack/nova: Make DeregisterImage respect AWS EC2 specification https://review.openstack.org/13008 | 12:52 |
| zykes- | I mean I got a basic deployment of services with 1 compute node and 1 head node that runs the l3 plugin and my db has like an average of 150 queries a second | 12:53 |
| zykes- | or 140 sorry | 12:53 |
| zykes- | Queries per second avg: 139.806 | 12:53 |
| *** Guest__ has joined #openstack-dev | 12:55 | |
| zykes- | salv-orlando: does the agent pr loop iteration or what ever it does do 2 calls pr router? | 12:56 |
| salv-orlando | The l3 agent polls the quantum server each second. But it does use the API rather then accessing the DB. Now if those API calls result in more than 100 db calls | 12:57 |
| salv-orlando | that would be weird. | 12:57 |
| zykes- | salv-orlando: http://pastebin.com/RWUMus9i | 12:57 |
| zykes- | it seems to me that each time it iterates it's doing this once pr router cause my logs are spammed like no tomorrow | 12:58 |
| *** spn has quit IRC | 12:59 | |
| zykes- | I think there's a serious performance bug there salv-orlando ;) | 12:59 |
| zykes- | or maybe i'm wrong.. | 12:59 |
| *** vanchester has joined #openstack-dev | 12:59 | |
| salv-orlando | Can you paste also some detail about the queries performed on the db? | 13:00 |
| *** dprince has joined #openstack-dev | 13:00 | |
| zykes- | uhm, salv-orlando in l3_agent.py there's process_router() any reason for it to call self.qclient.list_ports() for each router? | 13:01 |
| *** jgriffith has quit IRC | 13:01 | |
| zykes- | as far as my eyes can tell that's horrible | 13:01 |
| salv-orlando | I am sure this could be optimized; we need list ports anyway for knowing which subnets are attached to the router. | 13:02 |
| zykes- | salv-orlando: yeah, but what if you have like 50++ routers? | 13:02 |
| zykes- | that would be a horrible number of api calls each second ? | 13:02 |
| salv-orlando | this is why I said I am sure it can be optimized | 13:02 |
| zykes- | hehe, I wouldn't wanna have this in production ;p | 13:03 |
| zykes- | and I only have 10 networks added at the moment | 13:03 |
| *** jgriffith has joined #openstack-dev | 13:03 | |
| *** littleidea has joined #openstack-dev | 13:05 | |
| zykes- | need any info salv-orlando ? | 13:05 |
| zykes- | I can report a bug if you want ;9 | 13:05 |
| *** kbringard has quit IRC | 13:05 | |
| salv-orlando | that would be useful. Some details on the db queries performed would be useful. Another aspect of the problem is that we're making too many queries for each call probably | 13:05 |
| zykes- | how can I get the query info ? | 13:07 |
| *** tgall_foo has joined #openstack-dev | 13:10 | |
| *** tgall_foo has quit IRC | 13:10 | |
| *** tgall_foo has joined #openstack-dev | 13:10 | |
| salv-orlando | you said you were seeing 139 db/queries per second. I though you had some sort of instrumentation | 13:12 |
| *** jgriffith has quit IRC | 13:14 | |
| *** k4n0 has left #openstack-dev | 13:14 | |
| *** jgriffith has joined #openstack-dev | 13:15 | |
| zykes- | salv-orlando: just status; in the mysql cmdline | 13:18 |
| salv-orlando | ok. So perhaps report the bug, and we'll have someone working on it as soon as possible. Performance optimization is high on the roadmap anyway. | 13:18 |
| zykes- | well, I mean in my case I have only 10 subnets with routers | 13:19 |
| mnewby | salv-orlando: Hi | 13:19 |
| mnewby | zykes-: Hi | 13:19 |
| mnewby | I have a question totally related... | 13:19 |
| openstackgerrit | A change was merged to openstack/python-cinderclient: Fix some pep8 issues. https://review.openstack.org/13174 | 13:20 |
| zykes- | It would be horrible for a large deployment doing hundreds or more then thousands of subnets with routers and n api calls pr router if you poll every second | 13:20 |
| mnewby | Is there a spec for how list_ports is supposed to accept filter args? | 13:20 |
| mnewby | I have a proposed fix for broken filtering, but I don't have a definitive spec for how it should work. | 13:20 |
| mnewby | salv-orlando: Any pointers? | 13:21 |
| mnewby | In any case, the intent is for list_ports to be properly filterable and use db indexes to do it. | 13:21 |
| zykes- | https://bugs.launchpad.net/quantum/+bug/1052485 | 13:21 |
| uvirtbot | Launchpad bug 1052485 in quantum "L3 agent doing an extreme number of calls pr loop iteration" [Undecided,New] | 13:21 |
| mnewby | zykes: So same question then. How to optimize. | 13:22 |
| mnewby | zikes-, salv-orlando: I think the starting point is knowing what filter convention we want to support. | 13:22 |
| *** vanchester has left #openstack-dev | 13:22 | |
| zykes- | mnewby: I guess it would allmost be better to get all subnets / ports in one swoop or something | 13:23 |
| zykes- | instead of doing 1 call pr router? | 13:23 |
| salv-orlando | mnewby, zykes-: guys I will join you back in about 10 minutes. Filters are pretty simple so far. They are just passed as kvps to the query string and converted into a (possibly optimizable) where clause. | 13:23 |
| mnewby | salv-orlando: mostly true, except for list_ports garyk implemented something infinitely more complex. | 13:23 |
| salv-orlando | The where clause is built always like a IN (values on query string for a specific value) | 13:23 |
| mnewby | salv-orlando: That's not the way list_ports works at all, but that's my suggested implementation. | 13:24 |
| mnewby | salv-orlando: Is that how it should work, to the best of your knowledge? | 13:24 |
| *** edygarcia has joined #openstack-dev | 13:24 | |
| mnewby | markmcclain was suggesting that the nec and nicira plugins actually depended on the more complex filtering semantics currently in list_pots | 13:25 |
| mnewby | list_ports, sorry | 13:25 |
| salv-orlando | I understand list_ports has some differences especially as concern fixed_ips. | 13:26 |
| salv-orlando | We're aware of those :) | 13:26 |
| mnewby | salv-orlando: I'm working on fixing list_ports - do you have any pointers on how it is supposed to work? | 13:26 |
| mnewby | Allowing arbitrary or/and clauses seems like a bad idea, and I haven't seen any good use cases. | 13:27 |
| salv-orlando | you mean internal or at the api interface? | 13:27 |
| mnewby | both | 13:27 |
| mnewby | API interface just accepts lists of filter arguments that have to be interpreted internally | 13:27 |
| salv-orlando | no we support only and clauses. If there's support for "or" clauses, you should probably talk to gary to understand why he did it. | 13:27 |
| mnewby | very bad practice | 13:27 |
| mnewby | salv-orlando: ah, ok. I'm guessing there was a use-case at one point, but I can't see a good argument for not changing it to be standard. | 13:28 |
| garyk | mnewby: i am not sure that it was more complex. the solution was quick and dirty. ;) | 13:29 |
| *** samkottler|zzz is now known as samkottler | 13:30 | |
| *** nunosantos has joined #openstack-dev | 13:30 | |
| mnewby | garyk: ah, ok | 13:30 |
| mnewby | garyk: So is it acceptable/desirable to to support OR'd values for the same key (using in_) and AND filters across keys? | 13:30 |
| *** ncode has joined #openstack-dev | 13:31 | |
| *** nunosantos has quit IRC | 13:31 | |
| garyk | mnewby: tough question. | 13:31 |
| *** nunosantos has joined #openstack-dev | 13:32 | |
| garyk | mnewby: are we talking about the get_ports method? | 13:32 |
| mnewby | garyk: yes. I | 13:33 |
| mnewby | garyk: I can see now that the intent was to allow arbitrarily complex filtering on fixed_ips. | 13:34 |
| mnewby | garyk: Do you remember why this was necessary? | 13:34 |
| garyk | mnewby: i recall that yong needed support for nova to be able to get the port id for a specific ip address. | 13:35 |
| mnewby | garyk: Ah, so maybe I should be bugging him? | 13:35 |
| garyk | no, i implemented the code. i was just trying to recall why it was needed. | 13:36 |
| garyk | mnewby: i am not really sure what we should return. there may be a case where more than one port has the same ip addresses (on a different network). | 13:38 |
| garyk | this may also be filtered afterwads by the tenant id (or before for that matter) | 13:39 |
| mnewby | garyk: I don't think that's a problem. | 13:39 |
| *** mrodden has joined #openstack-dev | 13:39 | |
| garyk | if so then i think that i am missing something | 13:40 |
| mnewby | garyk: Filtering is possible by ip and subnet at presnt. | 13:40 |
| mnewby | Adding support for filtering by network would be pretty trivial. | 13:40 |
| mnewby | Though that would be out-of-scope for the security fix. | 13:40 |
| garyk | mnewby: i agree | 13:41 |
| *** PotHix has joined #openstack-dev | 13:41 | |
| garyk | from what i recall the original port_list would enable filtering by network. there was just the problem of filtering with entries that were not part of the entry - for example ip's | 13:41 |
| *** torandu has joined #openstack-dev | 13:42 | |
| *** mestery has joined #openstack-dev | 13:42 | |
| salv-orlando | garyk: I now remember this discussion. It was early June if I'm not wrong. | 13:43 |
| salv-orlando | It was a huge thread on the mailing list about how to filter on composite attributes. | 13:43 |
| garyk | salv-orlando: correct | 13:43 |
| garyk | i think that it is something that we never did resolve on the list. | 13:44 |
| garyk | salv-orlando: i think that mnewby's fix is very good. i have yet to test it, but the code looks kosher and fits in very nice | 13:45 |
| mnewby | garyk: I have an update to that patch that allows multiple keys per value. | 13:45 |
| mnewby | garyk: Waiting on a test run. | 13:46 |
| *** jimp has joined #openstack-dev | 13:46 | |
| garyk | mnewby: does it resolve salv-orlando concerns? | 13:46 |
| mnewby | garyk: I understand from mark that the change breaks nec and nicira, though. The apparently rely on the more complex filtering behaviour. | 13:46 |
| mnewby | The => They | 13:46 |
| garyk | mnewby: sorry i am not really familiar with these implementations. maybe akihiro can chime in from nec. | 13:47 |
| garyk | mnewby: the question is, is the patch backward compatible? | 13:47 |
| mnewby | garyk: That's the question. It changes filtering behaviour, so likely not. | 13:48 |
| mnewby | garyk: I'm waiting on a tox install to verify. | 13:48 |
| *** nunosantos_ has joined #openstack-dev | 13:48 | |
| *** mtreinish has joined #openstack-dev | 13:48 | |
| salv-orlando | I am not sure about complex filtering on the nicira plugin. I don't recall us doing anything particular, but if that breaks the tests something fishy is going on | 13:49 |
| *** nunosantos_ is now known as nsantos | 13:49 | |
| *** nsantos is now known as nunofsantos | 13:49 | |
| garyk | mnewby: my bad * 2. first the eval and then automatic tests for the specific code. i tested manually. i ran into json formatting problems | 13:49 |
| mnewby | salv-orlando: Unfortunately the comment from mark wasn't detailed enough to repro. | 13:50 |
| mnewby | garyk: No worries, these things happen. Your reviewers are equally culpable, too. ;) | 13:51 |
| salv-orlando | Not a problem. I'd suggest to split this patch into a possibly small patch that fixes the bigger issue, and than into another patch that we can mull on a little bit more that reworks the filtering logic. | 13:51 |
| salv-orlando | Or the two things are tightly coupled? | 13:51 |
| mnewby | salv-orlando: They are coupled. | 13:51 |
| zykes- | Shouldn't the OVS agent be tagging interfaces when doing VLANs ? | 13:51 |
| mnewby | Security hole and functionality go hand in hand. | 13:51 |
| mnewby | salv-orlando: when i get the *@@!#ing tests to run. *sigh* tox is so slow. | 13:52 |
| salv-orlando | mneweby: /run_test.sh is still there, if you need it | 13:53 |
| salv-orlando | mnewby: then doing something like I suggested this morning won't help at all? | 13:54 |
| *** zyluo has joined #openstack-dev | 13:54 | |
| *** eharney has joined #openstack-dev | 13:54 | |
| mnewby | salv-orlando: Hmmmm | 13:54 |
| *** markmcclain has joined #openstack-dev | 13:55 | |
| *** edygarcia has quit IRC | 13:55 | |
| mnewby | salv-orlando: Maybe, if the intent was to keep the ability to provide dictionaries on the command line to filter things. | 13:55 |
| mnewby | salv-orlando: We could just as easily eval as json, but same idea. | 13:55 |
| mnewby | salv-orlando: I'd really like to see that behaviour just go away, though, unless the intent is to document it properly and use it in more places. | 13:56 |
| salv-orlando | mnewby: ok. I got your point. | 13:56 |
| salv-orlando | mnewby: I do apologize, but in these days I wish everything could be done with a one-liner | 13:56 |
| zyluo | nmewby: When is the next Quantum-Tempest meeting? | 13:56 |
| mnewby | salv-orlando: I guess i should seek out that mailing list thread you mentioned. | 13:56 |
| mnewby | salv-orlando: So does that imply that you would like to be able to supply dicts to filter? | 13:57 |
| mnewby | salv-orlando: I can see the appeal of the power, don't get me wrong. | 13:57 |
| *** GheRivero is now known as Ghe_Rivero | 13:58 | |
| zykes- | Hmm, is there a reason for my OVS not actually tagging interfaces with correct vlans ? | 13:58 |
| *** datsun180b has joined #openstack-dev | 13:58 | |
| zykes- | I have set 50:100 in the ovs agent conf but it seems to be tagging interfaces with tag 1-2 | 13:59 |
| salv-orlando | mnewby: I do not have a strong opinion about dicts on the query string. To use gary's words, this does not sound a lot kosher :) | 14:00 |
| mnewby | salv-orlando: hah, fair enough. :) | 14:00 |
| *** glenc_ has joined #openstack-dev | 14:00 | |
| salv-orlando | also because api users are meant to be rest users which probably do not give a ____ about python dicts | 14:00 |
| *** xchu has joined #openstack-dev | 14:01 | |
| mnewby | salv-orlando: I thought you meant one-liner on the command line, but I'm guessing you actually meant a one-line fix to the bug. | 14:01 |
| salv-orlando | mnewby: yes, that's what I meant | 14:01 |
| *** AlanClark has joined #openstack-dev | 14:02 | |
| mnewby | salv-orlando: True re: python dicts, though python dicts are basically json dicts. | 14:02 |
| salv-orlando | I'd like to avoid non-trivial patches in this stage of the release process. | 14:02 |
| * salv-orlando is shivering at the though of what could be on the query string once we support xml | 14:02 | |
| mnewby | salv-orlando: Understood. Especially given the backwards incompatibile nature of the proposed change. | 14:02 |
| mnewby | salv-orlando: heh | 14:02 |
| *** devananda has quit IRC | 14:03 | |
| *** edygarcia has joined #openstack-dev | 14:03 | |
| *** mrodden1 has joined #openstack-dev | 14:04 | |
| n0ano | zykes-, ACK | 14:04 |
| *** mrodden1 has quit IRC | 14:04 | |
| garyk | salv-orlando: xml is so 80's | 14:05 |
| salv-orlando | mnewby: backward incompatibility is actually what would push me to change it now. | 14:05 |
| *** mrodden1 has joined #openstack-dev | 14:05 | |
| salv-orlando | I would hate breaking compatibility in grizzly for this. | 14:05 |
| mnewby | salv-orlando: Well, we're all good then :) | 14:05 |
| *** devananda has joined #openstack-dev | 14:05 | |
| salv-orlando | I should have probably have a better look at your patch, and see about | 14:06 |
| salv-orlando | Mark's comments on plugin breakage. | 14:06 |
| salv-orlando | I think Gary's already ok with it. Dan will probably be able to chime in on it later on. And then we'll make a final call later on tonight. | 14:07 |
| *** mrodden has quit IRC | 14:07 | |
| mnewby | salv-orlando: Ok. I'll try to clear up the breakage if they're caught by the tests. Will submit a revised patch shortly that supports multiple values per key. | 14:08 |
| garyk | salv-orlando: mnewby yes, i am fine with it. tomorrow morning i'll do some testing (have some other things to do at the moment). sorry | 14:08 |
| salv-orlando | yep I meant "tonight" in California terms | 14:08 |
| salv-orlando | so it should be tomorrow morning for all of us | 14:09 |
| *** mestery has quit IRC | 14:09 | |
| zykes- | can anyone tell me if the openvswitch agent should re-create interfaces on the head / controller node if they don't exist or which plugin has that responsability ? | 14:10 |
| *** dolphm has joined #openstack-dev | 14:11 | |
| *** japage has joined #openstack-dev | 14:14 | |
| *** ywu_ has joined #openstack-dev | 14:16 | |
| *** s0mik has joined #openstack-dev | 14:17 | |
| EmilienM | zykes-: L3 agent creates br-ex and its interfaces | 14:18 |
| zykes- | EmilienM: so dhcp agent and l3 should be on the same box ? | 14:18 |
| EmilienM | zykes-: OVS agent create TAP interfaces & GW also | 14:18 |
| EmilienM | zykes-: not really | 14:18 |
| zykes- | uh, which thing makes the .1 address or the address where it interacts ? | 14:19 |
| EmilienM | zykes-: that's a good question. Maybe they consider that .1 may be used by something else, like router or something like that | 14:19 |
| *** ywu_ has quit IRC | 14:20 | |
| salv-orlando | .1 is the address taken by the qr-xxxxx interfaces. It corresponds to the gateway for the subnet associated with that qr-xxxxx port | 14:29 |
| *** Tross has quit IRC | 14:30 | |
| zykes- | salv-orlando: any idea on why the ovs plugin tags interfaces with 1-10 instead of using the allocated vlans ? | 14:31 |
| *** dolphm has quit IRC | 14:32 | |
| *** alunduil has joined #openstack-dev | 14:33 | |
| *** xchu has quit IRC | 14:33 | |
| *** stanchan has quit IRC | 14:33 | |
| Kiall | rmk, just getting a chance to test out your latest patchset re resize with volumes - before I do though - you're not about to update it again or anything? (Don't want to go through it twice!) | 14:33 |
| *** EmilienM has left #openstack-dev | 14:34 | |
| *** lifeless has quit IRC | 14:35 | |
| *** titankiller has joined #openstack-dev | 14:35 | |
| *** alunduil has quit IRC | 14:36 | |
| *** alunduil has joined #openstack-dev | 14:36 | |
| *** EmilienM has joined #openstack-dev | 14:37 | |
| *** jimp has quit IRC | 14:38 | |
| *** Tross has joined #openstack-dev | 14:38 | |
| *** torandu has quit IRC | 14:39 | |
| *** jaypipes has joined #openstack-dev | 14:40 | |
| *** Mandell has joined #openstack-dev | 14:40 | |
| salv-orlando | zykes-: are you seeing these tags on instance ports br-int or on the port to the br-ethX bridge used as an uplink? | 14:41 |
| *** s0mik has quit IRC | 14:41 | |
| EmilienM | zykes-: I've seen your bug https://bugs.launchpad.net/quantum/+bug/1052522 | 14:41 |
| uvirtbot | Launchpad bug 1052522 in quantum "L3 plugin exists / dies when external bridge isn't up" [Undecided,New] | 14:41 |
| EmilienM | I have the same | 14:41 |
| zykes- | ehm, exits :p | 14:41 |
| EmilienM | I think it's because of Ubuntu Packaging : | 14:41 |
| EmilienM | OVS starts after L3 agent | 14:41 |
| zykes- | EmilienM: ok, really ? | 14:41 |
| EmilienM | I've said it to adam_g | 14:41 |
| zykes- | EmilienM: even so, should the plugin not run even of the bridge is missing ? | 14:41 |
| zykes- | or should it fail ? | 14:41 |
| *** mestery has joined #openstack-dev | 14:42 | |
| EmilienM | in fact, I think the agent need br-ex | 14:42 |
| EmilienM | if OVS is shut off, no br-ex, no L3 agent running | 14:42 |
| zykes- | ok | 14:42 |
| zykes- | :) | 14:42 |
| EmilienM | that's why you need OVS running before | 14:42 |
| *** jimp has joined #openstack-dev | 14:43 | |
| zykes- | that can probably be moved to packaging then | 14:43 |
| EmilienM | https://bugs.launchpad.net/quantum/+bug/1052522/comments/1 | 14:43 |
| uvirtbot | Launchpad bug 1052522 in quantum "L3 plugin exists / dies when external bridge isn't up" [Undecided,New] | 14:43 |
| zykes- | salv-orlando: I'm seeing tagged ports on my head node on the tap / qr ifaces | 14:44 |
| zykes- | http://paste.ubuntu.com/1213044/ | 14:44 |
| zykes- | ovs-vsctl show output | 14:44 |
| zykes- | http://paste.ubuntu.com/1213046/ < is the config present on both head node and compute | 14:45 |
| *** reed has joined #openstack-dev | 14:45 | |
| EmilienM | zykes-: a lot of networks / taps :p | 14:46 |
| zykes- | EmilienM: I only got 10 networks with routers attached | 14:46 |
| *** \etc\bin has quit IRC | 14:47 | |
| salv-orlando | zykes-: are you running a single compute node? | 14:48 |
| zykes- | salv-orlando: currently yes | 14:48 |
| zykes- | should that matter ? :) | 14:48 |
| salv-orlando | those tags you see are "local" tags - then each of these tags is mapped to the actual allocated tag when the data are forwarded into the uplink port | 14:49 |
| salv-orlando | since you're running a single host, you do not see the point where the "local" tag is stripped and replaced with the allocated one | 14:49 |
| *** crandquist has quit IRC | 14:50 | |
| ttx | salv-orlando: see pm | 14:50 |
| zykes- | salv-orlando: I got two host, 1 controller node which has the l3 + dhcp agent as well as a OVS switch and one compute node with ovs agent and ovs | 14:51 |
| *** kobier has joined #openstack-dev | 14:52 | |
| *** eglynn_ has quit IRC | 14:54 | |
| *** eglynn_ has joined #openstack-dev | 14:55 | |
| salv-orlando | zykes-: I meant hosts where the ova plugin agent is runnign. | 14:55 |
| salv-orlando | running => running | 14:55 |
| zykes- | salv-orlando: yes I got 2 hosts 1 controller and 1 controller node | 14:56 |
| zykes- | ehm, compute and controller | 14:56 |
| *** torandu has joined #openstack-dev | 14:58 | |
| *** mindpixel has quit IRC | 15:00 | |
| *** cp16net is now known as cp16net|away | 15:00 | |
| salv-orlando | zykes-: have you set bridge_mappings into the plugin's ini file? | 15:00 |
| zykes- | salv-orlando: http://paste.ubuntu.com/1213046/ | 15:00 |
| zykes- | bridge_mappings = physnet1:br-virtual | 15:00 |
| salv-orlando | ok - I must be dump. I thought that br-virtual was something else | 15:01 |
| *** oneiroi has quit IRC | 15:01 | |
| zykes- | On the br-virtual shouldn't I be seeing some tagged stuff there ? | 15:02 |
| salv-orlando | then on br-int you have this untagged port: int-br-virtual | 15:02 |
| zykes- | yes I do | 15:02 |
| salv-orlando | this is the uplink port I was talking about | 15:02 |
| *** ociuhandu has joined #openstack-dev | 15:02 | |
| zykes- | and there's a phy-br-virtual | 15:03 |
| zykes- | on br-virtual | 15:03 |
| salv-orlando | if you do ovs-ofctl dump-flow br-int and ovs-ofctl dump-flow br-virtual | 15:03 |
| salv-orlando | you should see the entries that do the tagging | 15:03 |
| *** kbringard has joined #openstack-dev | 15:03 | |
| zykes- | Ah, I see | 15:04 |
| zykes- | what interface can I dump on salv-orlando to see that i'm getting traffic between controller >< computes ? | 15:06 |
| *** danwent has joined #openstack-dev | 15:06 | |
| *** oneiroi has joined #openstack-dev | 15:06 | |
| salv-orlando | salv-orlando is not bridge :) | 15:07 |
| zykes- | ;p | 15:07 |
| *** Mandell has quit IRC | 15:07 | |
| salv-orlando | you mean traffic between the compute node and the controller node where dhcp agent and l3 agent are running? | 15:08 |
| *** Mandell has joined #openstack-dev | 15:08 | |
| zykes- | yes salv-orlando | 15:08 |
| salv-orlando | from what you pasted traffic should be leaving the compute node on eth1 | 15:09 |
| salv-orlando | but you can even dump br-virtual I guess | 15:09 |
| *** cp16net|away is now known as cp16net | 15:12 | |
| *** danwent has quit IRC | 15:12 | |
| *** Mandell has quit IRC | 15:12 | |
| *** rpedde_away is now known as rpedde | 15:12 | |
| *** mnewby has quit IRC | 15:13 | |
| *** mnewby has joined #openstack-dev | 15:14 | |
| *** rnirmal has joined #openstack-dev | 15:16 | |
| *** salv-orlando has quit IRC | 15:28 | |
| *** joesavak has joined #openstack-dev | 15:31 | |
| *** salv-orlando has joined #openstack-dev | 15:31 | |
| rmk | Kiall: there shouldn't be any other changes | 15:36 |
| *** hemna has joined #openstack-dev | 15:39 | |
| *** heckj has joined #openstack-dev | 15:43 | |
| Kiall | rmk, K.. I already actually tested it anyway | 15:44 |
| Kiall | Looks good | 15:44 |
| rmk | cool thanks | 15:46 |
| *** koolhead17 has joined #openstack-dev | 15:46 | |
| *** derekh has quit IRC | 15:50 | |
| openstackgerrit | A change was merged to openstack/nova: Fix volume deletion when device mapper is used https://review.openstack.org/12434 | 15:50 |
| *** Ghe_Rivero is now known as GheRivero | 15:50 | |
| *** mnewby has quit IRC | 15:51 | |
| *** GheRivero is now known as GheAway | 15:53 | |
| *** derekh has joined #openstack-dev | 15:55 | |
| *** mnewby has joined #openstack-dev | 15:56 | |
| *** reidrac has quit IRC | 15:57 | |
| *** oneiroi has quit IRC | 15:57 | |
| *** rafaduran has quit IRC | 15:57 | |
| *** colinmcnamara has joined #openstack-dev | 15:58 | |
| *** joesavak has quit IRC | 16:01 | |
| *** oneiroi has joined #openstack-dev | 16:01 | |
| *** mestery has quit IRC | 16:03 | |
| *** colinmcnamara has left #openstack-dev | 16:04 | |
| *** joesavak has joined #openstack-dev | 16:05 | |
| mnewby | salv-orlando: fyi, I still can't add you on gerrit. | 16:09 |
| salv-orlando | mnewby: just paste the link here :) | 16:09 |
| mnewby | https://review.openstack.org/#/c/13205/ | 16:09 |
| openstackgerrit | A change was merged to openstack/nova: Fixes snat rules in complex networking configs https://review.openstack.org/12727 | 16:09 |
| mnewby | :) | 16:09 |
| salv-orlando | I added my self and starred the change | 16:11 |
| salv-orlando | for some reason gerrit still thinks I am salvatore.orlando@eu.citrix.com | 16:11 |
| *** edygarcia has quit IRC | 16:13 | |
| notmyname | ttx: I won't be at the project meeting today. I have a schedule conflict. I think everything is preparing for folsom. Is there anything you need from me/swift for that? | 16:17 |
| *** ywu_ has joined #openstack-dev | 16:17 | |
| *** gatuus has joined #openstack-dev | 16:19 | |
| *** mestery has joined #openstack-dev | 16:20 | |
| *** jsavak has joined #openstack-dev | 16:20 | |
| *** ywu_ has quit IRC | 16:21 | |
| *** littleidea has quit IRC | 16:23 | |
| *** joesavak has quit IRC | 16:25 | |
| *** Ryan_Lane has quit IRC | 16:25 | |
| *** adjohn has joined #openstack-dev | 16:26 | |
| *** apevec has quit IRC | 16:28 | |
| *** edygarcia has joined #openstack-dev | 16:28 | |
| *** Trev_ has joined #openstack-dev | 16:28 | |
| *** s0mik has joined #openstack-dev | 16:28 | |
| *** jog0 has joined #openstack-dev | 16:28 | |
| openstackgerrit | A change was merged to openstack/nova: Add console output extension to API samples test. https://review.openstack.org/12957 | 16:29 |
| openstackgerrit | A change was merged to openstack/cinder: Clean up db.volume_create() https://review.openstack.org/13161 | 16:30 |
| *** jtran has joined #openstack-dev | 16:30 | |
| *** Mandell has joined #openstack-dev | 16:31 | |
| maurosr | reviews welcomed https://review.openstack.org/#/c/12704/ tks! =) | 16:31 |
| openstackgerrit | A change was merged to openstack/nova: Fix volume id conversion in nova-manage volume https://review.openstack.org/13181 | 16:31 |
| *** janisg has quit IRC | 16:33 | |
| *** lloydde has joined #openstack-dev | 16:33 | |
| *** ewindisch has joined #openstack-dev | 16:33 | |
| *** andrewbogott is now known as andrewbogott_afk | 16:34 | |
| *** winston-d has joined #openstack-dev | 16:34 | |
| winston-d | russellb, hi | 16:34 |
| russellb | hi | 16:35 |
| *** littleidea has joined #openstack-dev | 16:35 | |
| *** edygarcia_ has joined #openstack-dev | 16:36 | |
| *** littleidea has quit IRC | 16:37 | |
| winston-d | russellb, i read your comment on my patch here. https://review.openstack.org/#/c/13175/3/nova/scheduler/rpcapi.py | 16:38 |
| *** edygarcia has quit IRC | 16:38 | |
| *** edygarcia_ is now known as edygarcia | 16:38 | |
| *** stanchan has joined #openstack-dev | 16:39 | |
| winston-d | I thought keeping reservations argument is to make sure we are compatible with v2.0 API? | 16:40 |
| russellb | you have to leave it on the server side - manager.py | 16:41 |
| russellb | but you can remove it from the rpcapi.py module, the client side | 16:41 |
| russellb | as long as all of the current code has been converted to no longer pass it in | 16:41 |
| winston-d | ah, i got it. thx! will update the patch right away. | 16:42 |
| russellb | np | 16:43 |
| jk0 | dtroyer: thanks for pushing that thru, makes my life much easier :) | 16:44 |
| jk0 | dtroyer: do you guys have a process for getting core status on that? | 16:45 |
| *** maploin has quit IRC | 16:46 | |
| *** armaan has joined #openstack-dev | 16:47 | |
| *** armaan has left #openstack-dev | 16:47 | |
| *** winston-d has quit IRC | 16:48 | |
| ttx | notmyname: no, that's cool. I was planning to ask you about any red flag with 1.7.0... that answers my question | 16:50 |
| *** ev0ldave has joined #openstack-dev | 16:52 | |
| *** s0mik has quit IRC | 16:53 | |
| *** s0mik has joined #openstack-dev | 16:53 | |
| *** jdurgin has joined #openstack-dev | 16:54 | |
| *** samkottler is now known as samkottler|lunch | 16:55 | |
| *** joesavak has joined #openstack-dev | 16:56 | |
| *** jsavak has quit IRC | 16:58 | |
| *** winston-d has joined #openstack-dev | 16:59 | |
| *** jakedahn_zz is now known as jakedahn | 17:01 | |
| *** dhellmann has joined #openstack-dev | 17:01 | |
| openstackgerrit | A change was merged to openstack/quantum: Fix for DB error in the Cisco plugin https://review.openstack.org/13186 | 17:01 |
| *** darraghb has quit IRC | 17:02 | |
| openstackgerrit | A change was merged to openstack/python-openstackclient: Update compute client bits https://review.openstack.org/11741 | 17:03 |
| *** zing_ has quit IRC | 17:04 | |
| *** utlemming has quit IRC | 17:04 | |
| *** epim has joined #openstack-dev | 17:04 | |
| *** z has quit IRC | 17:06 | |
| *** joesavak has quit IRC | 17:06 | |
| *** gyee has joined #openstack-dev | 17:08 | |
| *** joesavak has joined #openstack-dev | 17:08 | |
| *** zaneb has quit IRC | 17:09 | |
| *** zaneb has joined #openstack-dev | 17:10 | |
| *** Kiall is now known as zz_Kiall | 17:10 | |
| *** utlemming has joined #openstack-dev | 17:11 | |
| openstackgerrit | A change was merged to openstack/nova: Adds new volume API extensions https://review.openstack.org/13016 | 17:11 |
| arosen1 | join #geni | 17:13 |
| * arosen1 oops | 17:14 | |
| *** danpb has quit IRC | 17:16 | |
| *** ev0ldave has quit IRC | 17:16 | |
| zul | vishy: websockify and novnc made it into quantal im working on the nova bits and will backport it | 17:17 |
| *** ev0ldave has joined #openstack-dev | 17:17 | |
| salv-orlando | mnewby: are you still around? | 17:19 |
| mnewby | salv-orlando: yes | 17:19 |
| salv-orlando | I have a question for you on metadata (as you've found the other issue we were talking about yesterday) | 17:19 |
| mnewby | Shoot. I'm actually working on that right now. | 17:19 |
| salv-orlando | Would the patch currently under review be invalidated by this change? | 17:20 |
| mnewby | Which patch? | 17:20 |
| mnewby | hmm | 17:20 |
| salv-orlando | the patch is: https://review.openstack.org/13096 | 17:20 |
| openstackgerrit | A change was merged to openstack/nova: add get_location method for images https://review.openstack.org/13017 | 17:20 |
| openstackgerrit | A change was merged to openstack/nova: add ability to clone images https://review.openstack.org/13018 | 17:22 |
| mnewby | salv-orlando: I can't see how the eval patch would affect the review you just pointed out. | 17:22 |
| *** Ryan_Lane has joined #openstack-dev | 17:22 | |
| mnewby | salv-orlando: am I missing something? | 17:22 |
| *** samkottler|lunch is now known as samkottler | 17:22 | |
| *** jsavak has joined #openstack-dev | 17:22 | |
| mnewby | salv-orlando: I see it | 17:23 |
| mnewby | salv-orlando: Doesn't look like it would affect it. The only use of list_ports doesn't use filter by a fixed_ip parameter (ip_address/subnet_id). | 17:24 |
| mnewby | salv_orlando: Should be fine. | 17:24 |
| *** mnewby has left #openstack-dev | 17:25 | |
| *** mnewby has joined #openstack-dev | 17:25 | |
| *** winston-d has quit IRC | 17:25 | |
| *** joesavak has quit IRC | 17:26 | |
| *** jimp has quit IRC | 17:28 | |
| salv-orlando | mnewby: cool. Actually - I meant if you were working on the metadata issue you brought up yesterday, the one for which you filed bug 1052196 | 17:28 |
| uvirtbot | Launchpad bug 1052196 in nova "The metadata service cannot identity an instance when using the Quantum v2 network API" [Undecided,New] https://launchpad.net/bugs/1052196 | 17:28 |
| *** derekh is now known as derekh_afk | 17:28 | |
| *** zz_Kiall is now known as Kiall | 17:28 | |
| salv-orlando | but since that is a nov bug I do not thing there will be any interference. | 17:28 |
| salv-orlando | Just wanted to check with you to confirm. | 17:28 |
| mnewby | salv-orlando: Ah, sorry. Yes, I am working on the metadata issue. | 17:29 |
| *** zyluo has quit IRC | 17:29 | |
| salv-orlando | mnewby: ok | 17:29 |
| mnewby | mnewby: nearly done, just doing some manual integration testing. | 17:29 |
| openstackgerrit | A change was merged to openstack/cinder: Bump version to 2013.1 https://review.openstack.org/13195 | 17:30 |
| *** nati_ueno has joined #openstack-dev | 17:32 | |
| *** nati_ueno has quit IRC | 17:32 | |
| *** WormMan has quit IRC | 17:33 | |
| *** jog0 has quit IRC | 17:34 | |
| *** alexpilotti has quit IRC | 17:34 | |
| *** Ryan_Lane has quit IRC | 17:36 | |
| *** rnirmal_ has joined #openstack-dev | 17:38 | |
| salv-orlando | zykes-: you still around? I have a question on bug 1051679 | 17:38 |
| uvirtbot | Launchpad bug 1051679 in quantum "Error running ip netns command with l3-agent" [High,Confirmed] https://launchpad.net/bugs/1051679 | 17:38 |
| *** Trev_ has quit IRC | 17:41 | |
| *** rnirmal has quit IRC | 17:42 | |
| *** rnirmal_ is now known as rnirmal | 17:42 | |
| *** mestery has quit IRC | 17:42 | |
| *** dolphm has joined #openstack-dev | 17:43 | |
| zykes- | salv-orlando: ping | 17:43 |
| arosen1 | Hey vishy can I get you to +2 this again when you get a chance https://review.openstack.org/#/c/13043/ ? I changed the text working in the commit in order to get the smoke tests to run again. | 17:45 |
| *** dolphm has quit IRC | 17:45 | |
| *** WormMan has joined #openstack-dev | 17:45 | |
| *** dolphm has joined #openstack-dev | 17:45 | |
| *** rnirmal has quit IRC | 17:46 | |
| *** rnirmal has joined #openstack-dev | 17:46 | |
| salv-orlando | zykes-: hey, about that bug | 17:46 |
| *** apevec has joined #openstack-dev | 17:46 | |
| *** apevec has joined #openstack-dev | 17:46 | |
| *** colinmcnamara has joined #openstack-dev | 17:47 | |
| salv-orlando | I am trying to reproduce it. Did it occur to you after changing the name for the external bridge and then restarting quantum l3 agent or before starting the agent for the first time? | 17:47 |
| *** koolhead17 has quit IRC | 17:50 | |
| zykes- | salv-orlando: I changed the name before starting the agent for the first time | 17:51 |
| salv-orlando | zykes-: ok thanks | 17:51 |
| *** armaan has joined #openstack-dev | 17:55 | |
| *** armaan has left #openstack-dev | 17:55 | |
| *** zaitcev has joined #openstack-dev | 17:56 | |
| *** samkottler has quit IRC | 17:58 | |
| *** samkottler has joined #openstack-dev | 17:59 | |
| *** zing_ has joined #openstack-dev | 18:01 | |
| *** ksiu has joined #openstack-dev | 18:01 | |
| *** Ryan_Lane has joined #openstack-dev | 18:04 | |
| *** mestery has joined #openstack-dev | 18:05 | |
| *** jog0 has joined #openstack-dev | 18:06 | |
| *** cp16net is now known as cp16net|away | 18:14 | |
| *** ywu_ has joined #openstack-dev | 18:17 | |
| *** ywu_ has quit IRC | 18:21 | |
| *** negronjl has quit IRC | 18:22 | |
| *** negronjl has joined #openstack-dev | 18:24 | |
| rmk | markmc: Had a chance to review the updated patch? https://review.openstack.org/13071 | 18:29 |
| *** Ryan_Lane has quit IRC | 18:29 | |
| rmk | I'd really like to get this into Folsom, even if I need to make more changes to do so. | 18:29 |
| *** Ryan_Lane has joined #openstack-dev | 18:34 | |
| *** cp16net|away is now known as cp16net | 18:34 | |
| *** markmcclain has quit IRC | 18:35 | |
| *** tott has joined #openstack-dev | 18:36 | |
| *** johnpur has joined #openstack-dev | 18:36 | |
| *** ChanServ sets mode: +v johnpur | 18:36 | |
| *** novas0x2a|laptop has joined #openstack-dev | 18:37 | |
| *** otter3 has quit IRC | 18:37 | |
| *** mestery has quit IRC | 18:38 | |
| *** jimp has joined #openstack-dev | 18:40 | |
| *** otter1 has joined #openstack-dev | 18:44 | |
| *** zing_ has quit IRC | 18:44 | |
| openstackgerrit | A change was merged to openstack/nova: Adds API sample testing for Extended server attributes extension https://review.openstack.org/13028 | 18:52 |
| openstackgerrit | A change was merged to openstack/swift: builder.add_devs gets next id if not provided https://review.openstack.org/13047 | 18:53 |
| *** colinmcnamara has quit IRC | 18:54 | |
| openstackgerrit | A change was merged to openstack/swift: Fix bug where serialization_format is ignored https://review.openstack.org/13215 | 18:54 |
| *** dolphm has quit IRC | 19:05 | |
| *** zing_ has joined #openstack-dev | 19:05 | |
| openstackgerrit | A change was merged to openstack/nova: Fix doc/README.rst to render properly https://review.openstack.org/13216 | 19:10 |
| mnewby | salv-orlando: https://review.openstack.org/#/c/13218/ | 19:12 |
| salv-orlando | mnewby: thanks | 19:12 |
| openstackgerrit | A change was merged to openstack/glance: Generalize remote image functional test https://review.openstack.org/12913 | 19:14 |
| *** derekh_afk is now known as derekh | 19:21 | |
| *** lifeless has joined #openstack-dev | 19:22 | |
| *** andrewbogott_afk is now known as andrewbogott | 19:24 | |
| *** utlemming has quit IRC | 19:29 | |
| *** GheAway has quit IRC | 19:29 | |
| *** markmcclain has joined #openstack-dev | 19:29 | |
| *** GheRivero has joined #openstack-dev | 19:30 | |
| *** GheRivero is now known as Ghe_Rivero | 19:30 | |
| mnewby | smokestack broken? | 19:30 |
| mnewby | ah, nmind. the error is just unhelpful. | 19:31 |
| *** littleidea has joined #openstack-dev | 19:31 | |
| *** tott has quit IRC | 19:32 | |
| *** tott has joined #openstack-dev | 19:37 | |
| openstackgerrit | A change was merged to openstack/glance: Expand cache middleware unit tests https://review.openstack.org/13051 | 19:40 |
| *** Ryan_Lane has quit IRC | 19:47 | |
| openstackgerrit | A change was merged to openstack/glance: Don't WARN if trying to add a scheme which exists. https://review.openstack.org/13005 | 19:47 |
| *** dprince has quit IRC | 19:48 | |
| *** Ryan_Lane has joined #openstack-dev | 19:49 | |
| *** Ryan_Lane has quit IRC | 19:49 | |
| *** Ryan_Lane has joined #openstack-dev | 19:50 | |
| *** mnewby has left #openstack-dev | 19:50 | |
| *** colinmcnamara has joined #openstack-dev | 19:56 | |
| *** utlemming has joined #openstack-dev | 19:58 | |
| *** zing_ has quit IRC | 19:59 | |
| openstackgerrit | A change was merged to openstack/glance: Add unit tests for size_checked_iter https://review.openstack.org/12999 | 20:00 |
| sandywalsh_ | anyone else getting nfs test failures in trunk? | 20:00 |
| *** eglynn_ has quit IRC | 20:01 | |
| *** gatuus has quit IRC | 20:02 | |
| *** gatuus has joined #openstack-dev | 20:02 | |
| *** ev0ldave has quit IRC | 20:02 | |
| *** jsavak has quit IRC | 20:03 | |
| *** eglynn_ has joined #openstack-dev | 20:03 | |
| *** mestery has joined #openstack-dev | 20:05 | |
| *** ksiu has quit IRC | 20:05 | |
| notmyname | ttx: turns out something did come up today. I need to merge a patch to milestone-proposed (that's the right place, right?) | 20:06 |
| ttx | hmm | 20:06 |
| *** almaisan-away is now known as al-maisan | 20:06 | |
| ttx | notmyname: actually you'll need to do a 1.7.1 | 20:06 |
| notmyname | hmm | 20:06 |
| ttx | notmyname: so you could base it off master | 20:06 |
| notmyname | no, we've merged several things to master since 1.7.0 | 20:07 |
| *** rnirmal_ has joined #openstack-dev | 20:07 | |
| *** al-maisan is now known as almaisan-away | 20:07 | |
| *** jimp has quit IRC | 20:07 | |
| notmyname | ttx: this is something that needs to be in folsom | 20:08 |
| *** rnirmal has quit IRC | 20:08 | |
| *** rnirmal_ is now known as rnirmal | 20:08 | |
| ttx | notmyname: complicates things a bit, but I guess we could do 1.7.1 on top of milestone-proposed | 20:08 |
| ttx | let me check if it still exists | 20:08 |
| ttx | yes, we still have it | 20:09 |
| notmyname | ttx: https://bugs.launchpad.net/swift/+bug/1052644 and https://review.openstack.org/#/c/13217/ | 20:09 |
| uvirtbot | Launchpad bug 1052644 in swift "serialization_format is ignored" [Critical,New] | 20:09 |
| ttx | so.. master could jump to 1.8.0 while milestone-proposed gets 1.7.1 | 20:09 |
| notmyname | well, master is already at 1.7.1-dev. why would we need 1.8? | 20:11 |
| ttx | a bit dirty since we built 1.7.1 per-commit preversions out of master until now | 20:11 |
| notmyname | 1.7.0.1 ? | 20:11 |
| notmyname | ;-) | 20:11 |
| ttx | notmyname: ew. a bit ugly, not even sure the code supports it :P | 20:11 |
| ttx | notmyname: any reason why you don't want to jump to 1.8 on master ? | 20:13 |
| openstackgerrit | A change was merged to openstack/cinder: Add update to volume and snapshot controllers https://review.openstack.org/12067 | 20:13 |
| notmyname | ttx: there hasn't been a code reason to do so. what about 1.7.2 on master? why not do that? | 20:14 |
| ttx | notmyname: I fear we may repeat the ugliness again | 20:14 |
| ttx | notmyname: we did generate 1.7.1 preversions that contained stuff that will not be in 1.7.1 final | 20:15 |
| ttx | (since we are using that for milestone-proposed now) | 20:15 |
| notmyname | ya | 20:15 |
| ttx | If there is another issue... we'll do that again | 20:15 |
| ttx | a bit confusing, but I guess i can live with that | 20:16 |
| ttx | about the same as reverting all commits and pushing only one bugfix instead | 20:16 |
| notmyname | ttx: eww | 20:16 |
| notmyname | ttx: let me think about it for a little bit | 20:16 |
| ttx | if you do 1.8 jump on master, at least we won't do ugly again if you ever need to push 1.7.2 in folsom | 20:17 |
| *** jimp has joined #openstack-dev | 20:18 | |
| *** ywu_ has joined #openstack-dev | 20:18 | |
| vishy | ttx: ping | 20:19 |
| ttx | vishy: yes? | 20:19 |
| *** eglynn__ has joined #openstack-dev | 20:19 | |
| vishy | i think we can downgrade https://bugs.launchpad.net/nova/+bug/1052252 to rc-potential | 20:19 |
| uvirtbot | Launchpad bug 1052252 in nova "Migration 90 removes foreign keys but does not re-add them" [Critical,In progress] | 20:19 |
| *** eglynn_ has quit IRC | 20:20 | |
| vishy | i think it is actually just 1 foreign key and fixing it will require an addition to cinder and nova | 20:20 |
| vishy | the lack of the foreign key doesn't seem to be causing any issues, so i think putting in a known issue is fine | 20:20 |
| ttx | vishy: ok | 20:21 |
| *** ywu_ has quit IRC | 20:22 | |
| openstackgerrit | A change was merged to openstack/nova: Correct db migration 91 https://review.openstack.org/13203 | 20:24 |
| notmyname | ttx: here's what I think would be best: patch to milestone-proposed and milestone-proposed turns into 1.7.1-final. master then gets updated from 1.7.1-dev to 1.7.2-dev and work continues | 20:32 |
| ttx | notmyname: ok | 20:32 |
| ttx | notmyname: still having a meeting next hour ? | 20:33 |
| *** mtreinish has quit IRC | 20:33 | |
| ttx | notmyname: will push a 1.7.1 milestone under swift/folsom in LP | 20:33 |
| notmyname | wait | 20:33 |
| notmyname | ttx: wait. I misspoke. milestone-proposed should be 1.7.2 and master is 1.7.3. that way the current 1.7.1-dev will avoid confusion with the actual 1.7.1 | 20:34 |
| notmyname | so therefore swift will "skip" a 1.7.1 release | 20:34 |
| ttx | notmyname: well, you'd still have things in preversions of 1.7.1 that won't end up in 1.7.2, so the difference is not massive | 20:34 |
| ttx | not sure it's worth skipping | 20:35 |
| ttx | but you decide | 20:35 |
| notmyname | ttx: right, but a *-dev "release" should never be expected to turn into *-final. eg 1.4.9-dev turned into 1.5.0 | 20:35 |
| ttx | ah. ok | 20:36 |
| ttx | prior art I see | 20:36 |
| ttx | will create 1.7.2 under folsom | 20:36 |
| notmyname | and that wasn't the first time | 20:36 |
| notmyname | thansk | 20:36 |
| notmyname | ttx: shall I go ahead and merge the outstanding milestone-proposed patch and a version bump patch? | 20:37 |
| *** adjohn has quit IRC | 20:37 | |
| ttx | yes | 20:37 |
| *** adjohn has joined #openstack-dev | 20:37 | |
| ttx | notmyname: merge the fix to master first | 20:37 |
| notmyname | already done | 20:38 |
| ttx | ok then | 20:38 |
| ttx | yes, would be good to to the 1.7.2 version bump in the same commit | 20:40 |
| openstackgerrit | A change was merged to openstack/glance: Clean up pep8 E502, E711 violations https://review.openstack.org/13177 | 20:41 |
| *** nunofsantos has quit IRC | 20:42 | |
| openstackgerrit | A change was merged to openstack/python-glanceclient: Fixes glance add / update / image-create / image-update on Windows https://review.openstack.org/12936 | 20:44 |
| notmyname | ttx: sorry it will be 2 commits. waiting on jenkins now then I'll version bump | 20:45 |
| ttx | notmyname: that will produce a 1.7.0 tarball after the released one, which is also ugly. But if the current one can't be stopped... | 20:47 |
| ttx | (the idea is to try to avoid oddly-versioned artefacts) | 20:47 |
| *** nunosantos has quit IRC | 20:47 | |
| notmyname | ttx: can you stop jenkins? | 20:48 |
| *** colinmcnamara1 has joined #openstack-dev | 20:48 | |
| ttx | notmyname: I think if you -1 the patch it won't merge | 20:48 |
| ttx | even if tests succeed | 20:48 |
| *** colinmcnamara has quit IRC | 20:48 | |
| notmyname | done | 20:48 |
| *** gugl has joined #openstack-dev | 20:54 | |
| jgriffith | mikal: ping | 20:56 |
| *** mestery has quit IRC | 20:56 | |
| *** japage has quit IRC | 20:57 | |
| vishy | jgriffith: are you pinging about the fk re-add? | 20:57 |
| *** joesavak has joined #openstack-dev | 20:57 | |
| jgriffith | vishy: Yes, I have a patch for it, just writing tests but didn't know if he was done or in progress | 20:58 |
| *** gugl has quit IRC | 20:58 | |
| mikal | jgriffith: pong | 20:59 |
| *** Vek has quit IRC | 20:59 | |
| *** eharney has quit IRC | 20:59 | |
| jgriffith | mikal: Hey... just wanted to see where you were at on https://bugs.launchpad.net/nova/+bug/1052252 | 20:59 |
| uvirtbot | Launchpad bug 1052252 in nova "Migration 90 removes foreign keys but does not re-add them" [Medium,In progress] | 20:59 |
| mikal | jgriffith: I haven't got that far with it, so if you have a patch steal the bug and run with it | 20:59 |
| jgriffith | I have a fix for the FK migrations | 20:59 |
| mikal | jgriffith: I gift you the bug then | 21:00 |
| jgriffith | K.. the patch itself is ready, but I need to write a test for it | 21:00 |
| *** Vek has joined #openstack-dev | 21:00 | |
| jgriffith | mikal: Isn't that sort of *re-gifting* | 21:00 |
| jgriffith | :) | 21:00 |
| mikal | Heh | 21:00 |
| mikal | What's that US ad? Regifting of champions or something? | 21:01 |
| jgriffith | LOL... yeah, I kinda remember something like that | 21:01 |
| vishy | jgriffith: is it just the one foreign key? | 21:01 |
| vishy | I'm thinking that it isn't really vital for folsom | 21:01 |
| vishy | since the lack of a foreign key isn't causing problems | 21:02 |
| *** joesavak has quit IRC | 21:02 | |
| zykes- | hmm, with quantum v2 how does one get the metadata api to work ? | 21:02 |
| vishy | mikal, jgriffith: ^^ I untargeted it for rc-1 based on that reasoning | 21:02 |
| zykes- | salv-orlando: ? | 21:02 |
| salv-orlando | zykes-: pong | 21:02 |
| *** colinmcnamara1 has quit IRC | 21:03 | |
| zykes- | salv-orlando: you got a clue ? | 21:03 |
| jgriffith | vishy: It's all of the FK's I believe based on what the code was doing | 21:03 |
| openstackgerrit | A change was merged to openstack/nova: Make sure to return an empty subnet list for a network without sunbet. https://review.openstack.org/12195 | 21:03 |
| vishy | zykes-: based on bug reports, it appears that there is a bug in it | 21:03 |
| *** gabrielhurley has joined #openstack-dev | 21:03 | |
| jgriffith | it was trying to re-add them by reading them in from the table after they were dropped | 21:03 |
| jgriffith | It's your call whether to bother putting it in or not but I at least wanted to fix my mistake | 21:03 |
| salv-orlando | not yet - tried several options, but all failed to reproduce so far. I am stubborn guy though. | 21:04 |
| vishy | zykes-: https://bugs.launchpad.net/nova/+bug/1052196 | 21:04 |
| jgriffith | Really doesn't matter to me if it goes in or not | 21:04 |
| uvirtbot | Launchpad bug 1052196 in nova "The metadata service cannot identity an instance when using the Quantum v2 network API" [Medium,In progress] | 21:04 |
| salv-orlando | can you confirm you were running without namespace? | 21:04 |
| vishy | jgriffith: verified that only the snapshot -> volume_id foreign key is missing | 21:04 |
| zykes- | salv-orlando: me or ? | 21:05 |
| vishy | jgriffith: the other ones seem to work. It appears that it only reloads the data from the table for the primary keys or something | 21:05 |
| zykes- | I'm running with namespacing | 21:05 |
| *** markvoelker has quit IRC | 21:05 | |
| jgriffith | vishy: Well then it's just dumb luck, but like I said. If there are more pressing issues right now let it slide | 21:05 |
| salv-orlando | ok I saw in your report ip link being called without ns, but that's fine. namespaces should not be related to the issue you're seeing | 21:05 |
| jgriffith | vishy: It won't impact cinder so it's just up to what Nova wants to do with it | 21:06 |
| *** maurosr has quit IRC | 21:06 | |
| zykes- | vishy: shouldn't there be some route or fw rule for the metadata api to work ? | 21:06 |
| vishy | jgriffith: fair enough. I want to keep the dbs the same so if we can live without a foreign key for now | 21:06 |
| vishy | zykes-: I don't know, but you can look at the but and the patch | 21:07 |
| jgriffith | vishy: cool | 21:07 |
| vishy | jgriffith: https://review.openstack.org/#/c/13229 | 21:08 |
| vishy | jgriffith: not sure if you have rpc versioning in your scheduler but if so ^^ | 21:08 |
| jgriffith | vishy: :( | 21:08 |
| zykes- | :p | 21:08 |
| zykes- | salv-orlando: are you looking at the issue or ? | 21:09 |
| salv-orlando | yes, I am | 21:09 |
| salv-orlando | I am now in a meeting - will come back at in half a hour or so | 21:09 |
| salv-orlando | zykes-: any hint you might have, just post in on the bug report | 21:10 |
| *** Ghe_Rivero has quit IRC | 21:10 | |
| bcwaldon | eglynn__: ping | 21:11 |
| *** Guest78056 is now known as jgb | 21:11 | |
| *** jgb is now known as Guest66762 | 21:11 | |
| *** dolphm has joined #openstack-dev | 21:12 | |
| *** Guest66762 is now known as jaygeebee | 21:12 | |
| *** rnirmal has quit IRC | 21:14 | |
| *** rnirmal has joined #openstack-dev | 21:14 | |
| bcwaldon | russellb: https://bugs.launchpad.net/glance/+bug/1050661 | 21:15 |
| uvirtbot | Launchpad bug 1050661 in quantum "Default for qpid_heartbeat should be increased" [High,Fix committed] | 21:15 |
| bcwaldon | russellb: can you propose this fix to milestone-proposed for glance? | 21:15 |
| russellb | yup | 21:16 |
| *** dolphm has quit IRC | 21:16 | |
| bcwaldon | russellb: thanks! | 21:16 |
| *** dolphm has joined #openstack-dev | 21:16 | |
| russellb | bcwaldon: https://review.openstack.org/13230 | 21:17 |
| bcwaldon | russellb: fantastic | 21:19 |
| *** torandu has quit IRC | 21:20 | |
| bcwaldon | russellb: did you want to explore fixing the other bug you filed? | 21:20 |
| bcwaldon | russellb: I wanted to see if eglynn__ wanted to look at it, but he's afk | 21:20 |
| russellb | bcwaldon: i pinged eglynn__ about it earlier and it sounded like he was going to dig into it | 21:20 |
| bcwaldon | ok, cool | 21:20 |
| russellb | it's kinda late for him now | 21:21 |
| bcwaldon | yeah, I realize that now | 21:21 |
| bcwaldon | it can wait until tomorrow for sure | 21:21 |
| eglynn__ | bcwaldon: yep, I'm looking at it now | 21:21 |
| bcwaldon | eglynn__: fantastic! | 21:21 |
| russellb | eglynn__: nice | 21:21 |
| eglynn__ | bcwaldon: may be tomorrow by the time I have a fix though | 21:21 |
| eglynn__ | bcwaldon, russellb: see latest comment on the bug | 21:22 |
| eglynn__ | (the issue is not limited to copy-from) | 21:22 |
| bcwaldon | eglynn__: no worries, we have a couple of days for rc2 | 21:22 |
| eglynn__ | coolio | 21:22 |
| russellb | eglynn__: don't see a comment, launchpad or bz? | 21:22 |
| eglynn__ | russellb: mid-air collision! one sec ... | 21:23 |
| russellb | ah | 21:23 |
| *** maoy_ has joined #openstack-dev | 21:24 | |
| *** edehde has joined #openstack-dev | 21:30 | |
| *** edehde has left #openstack-dev | 21:32 | |
| russellb | eglynn__: thanks for digging ... sounds like a fun one :) | 21:33 |
| *** samkottler is now known as samkotter|afk | 21:35 | |
| *** samkotter|afk is now known as samkottler|afk | 21:35 | |
| *** n0ano has quit IRC | 21:36 | |
| *** jimp has quit IRC | 21:38 | |
| bcwaldon | eglynn__: might want to investigate with multiprocessing on as well | 21:38 |
| openstackgerrit | A change was merged to openstack/nova: Add deserialization for multiple create and az https://review.openstack.org/13221 | 21:42 |
| *** armaan has joined #openstack-dev | 21:44 | |
| *** gabrielhurley has quit IRC | 21:45 | |
| eglynn__ | bcwaldon: yep | 21:46 |
| *** armaan has quit IRC | 21:49 | |
| *** jecarey has joined #openstack-dev | 21:51 | |
| *** cmagina has joined #openstack-dev | 21:51 | |
| rmk | vishy: If I were to refactor a bunch of the libvirt code for G, would that be a blueprint? I basically want to eliminate ever using on-disk or live-domain XML data, as a starting point. | 21:57 |
| *** rnirmal has quit IRC | 21:58 | |
| bcwaldon | rmk: yes, blueprint would be great | 21:59 |
| rmk | I also think eliminating any (reasonable) expectation of system state needs to occur. Example: suspend a VM, reboot the hypervisor. Suspend operation thinks the networking is in place and doesn't bring any of it up. | 21:59 |
| zykes- | what can I do to get the metadata api working ? | 21:59 |
| rmk | Either everything should be re-applicable or we should validate expected state and correct where we can. | 21:59 |
| *** kbringard has quit IRC | 22:01 | |
| bcwaldon | rmk: sure, that sounds like a positive thing to me | 22:01 |
| *** heckj has quit IRC | 22:02 | |
| rmk | Great, I'll submit a BP | 22:04 |
| *** ywu_ has joined #openstack-dev | 22:04 | |
| vishy | rmk: sounds like a blueprint | 22:05 |
| openstackgerrit | A change was merged to openstack/horizon: Fixes error when clicking OpenStack logo. https://review.openstack.org/13231 | 22:08 |
| openstackgerrit | A change was merged to openstack/nova: Update scheduler rpc API version. https://review.openstack.org/13229 | 22:08 |
| *** EmilienM has left #openstack-dev | 22:10 | |
| rmk | I need to open a bug about the suspend thing. | 22:10 |
| *** s0mik has quit IRC | 22:10 | |
| *** datsun180b has quit IRC | 22:10 | |
| *** cmagina has quit IRC | 22:11 | |
| *** Gordonz has quit IRC | 22:13 | |
| *** markmc has quit IRC | 22:18 | |
| *** alunduil has quit IRC | 22:19 | |
| *** jecarey has quit IRC | 22:22 | |
| *** markmcclain has quit IRC | 22:23 | |
| *** dubsquared has joined #openstack-dev | 22:33 | |
| *** epim has quit IRC | 22:35 | |
| vishy | rmk: question on your migration | 22:36 |
| vishy | rmk: if you are resizing to the same host won't your reattach fail? | 22:36 |
| rmk | It shouldn't make a difference | 22:38 |
| rmk | What situation are you envisioning? Maybe I am not seeing it. | 22:39 |
| vishy | rmk: does the code detach all volume before migrate? | 22:39 |
| rmk | It only does it on the finish or revert_finish | 22:40 |
| rmk | finsih_revert rather | 22:40 |
| *** e1mer has joined #openstack-dev | 22:40 | |
| *** e1mer has joined #openstack-dev | 22:40 | |
| openstackgerrit | A change was merged to openstack/nova: Add user-data extension to API samples tests https://review.openstack.org/12987 | 22:40 |
| *** jaypipes has quit IRC | 22:41 | |
| openstackgerrit | A change was merged to openstack/nova: Makes scheduler hints and disk config xml correct https://review.openstack.org/13222 | 22:41 |
| vishy | rmk: yeah i don't see where the volumes are detached | 22:42 |
| rmk | Looking at it now.. | 22:43 |
| vishy | rmk: so it doesn't seem like your patch is particularly useful without that... | 22:43 |
| vishy | :) | 22:43 |
| vishy | rmk: I would expect you would need block_device_mapping in migrate_disk_and_power_off as well | 22:44 |
| rmk | when is migrate_disk_and_power_off called? Before the finish, right? | 22:45 |
| rmk | So I'm not tearing down the iscsi connections but that shouldn't actually impact anything | 22:46 |
| *** PotHix has quit IRC | 22:47 | |
| rmk | The old instance is destroyed before the new one comes up, so there should never be a case where both have the volume attached even if the iscsi connection is present in both places | 22:48 |
| rmk | Granted destroying the connections is a simple change | 22:48 |
| *** pixelbeat has quit IRC | 22:48 | |
| openstackgerrit | A change was merged to openstack/nova: sample api testing for os-floating-ips extension https://review.openstack.org/13156 | 22:48 |
| rmk | Depending on when migrate_disk_and_power_off is called. If it's called after the new instance is running on the same host, then yeah, you'll be destroying active iscsi connections and need to account for that. | 22:49 |
| rmk | It's harmless to keep them around for now imo | 22:49 |
| *** cmagina has joined #openstack-dev | 22:50 | |
| notmyname | ttx: single backport patch landing for swift 1.7.2-final https://review.openstack.org/#/c/13235/ | 22:52 |
| *** ncode has quit IRC | 22:53 | |
| *** sn0wb1rd has joined #openstack-dev | 22:54 | |
| *** rpedde is now known as rpedde_away | 22:55 | |
| rmk | vishy: I can add the detache code right now if you're still considering it for inclusion into folsom | 22:57 |
| rmk | detach | 22:57 |
| vishy | rmk: please add it. Might be too big, but definitely worth a backport to stable/folsom if it doesn't go | 22:57 |
| *** mikal has quit IRC | 22:58 | |
| *** mikal has joined #openstack-dev | 23:00 | |
| *** mikal has quit IRC | 23:00 | |
| *** mikal_ has joined #openstack-dev | 23:00 | |
| *** mikal_ is now known as mikal | 23:00 | |
| *** yb has joined #openstack-dev | 23:06 | |
| *** derekh has quit IRC | 23:08 | |
| *** dubsquared has quit IRC | 23:09 | |
| *** lloydde has quit IRC | 23:10 | |
| *** cmagina has quit IRC | 23:11 | |
| vishy | jog0: that is going to have to be a new migration | 23:11 |
| jog0 | vishy: thanks wasn't sure about that. Still working on adding new tests too | 23:12 |
| *** apevec has quit IRC | 23:15 | |
| *** edygarcia has quit IRC | 23:18 | |
| *** mrodden1 has quit IRC | 23:18 | |
| *** cmagina has joined #openstack-dev | 23:19 | |
| openstackgerrit | A change was merged to openstack/swift: 1.7.2. changelog and version bump to 1.7.3-dev https://review.openstack.org/13240 | 23:20 |
| *** lifeless has quit IRC | 23:22 | |
| *** ywu_ has quit IRC | 23:24 | |
| *** ywu_ has joined #openstack-dev | 23:24 | |
| *** jimp has joined #openstack-dev | 23:28 | |
| *** JStoker has quit IRC | 23:29 | |
| *** JStoker has joined #openstack-dev | 23:29 | |
| *** mrodden has joined #openstack-dev | 23:29 | |
| *** jimp has quit IRC | 23:30 | |
| *** ywu_ has quit IRC | 23:31 | |
| *** ywu_ has joined #openstack-dev | 23:31 | |
| *** gatuus has quit IRC | 23:32 | |
| *** Guest__ has quit IRC | 23:34 | |
| *** ywu_ has quit IRC | 23:34 | |
| *** ywu_ has joined #openstack-dev | 23:34 | |
| rmk | vishy: patch updated | 23:35 |
| *** ywu_ has quit IRC | 23:36 | |
| *** ywu_ has joined #openstack-dev | 23:36 | |
| *** waa has quit IRC | 23:37 | |
| *** lloydde has joined #openstack-dev | 23:39 | |
| *** lloydde_ has joined #openstack-dev | 23:40 | |
| *** zaneb has quit IRC | 23:40 | |
| *** zaneb has joined #openstack-dev | 23:41 | |
| *** ywu_ has quit IRC | 23:41 | |
| *** ywu_ has joined #openstack-dev | 23:41 | |
| arosen1 | vishy: mine too :) https://review.openstack.org/#/c/13043/ I only changed the commit message to get the smoke tests to run | 23:41 |
| vishy | rmk: looks good. Might want to update the commit message explaining that it is now detaching and attaching volumes on resize | 23:42 |
| rmk | Will do | 23:42 |
| *** salv-orlando has quit IRC | 23:43 | |
| *** lloydde has quit IRC | 23:43 | |
| *** cmagina has quit IRC | 23:43 | |
| *** AlanClark has quit IRC | 23:44 | |
| arosen1 | thanks vishy :), hopefully I'll get another core to review it soon. | 23:45 |
| *** lloydde has joined #openstack-dev | 23:47 | |
| *** edygarcia has joined #openstack-dev | 23:48 | |
| *** alunduil has joined #openstack-dev | 23:49 | |
| *** ywu_ has quit IRC | 23:49 | |
| *** ywu_ has joined #openstack-dev | 23:49 | |
| *** lloydde_ has quit IRC | 23:50 | |
| sn0wb1rd | What is the process that runs on controller node to allow connections from compute node on port 4000? | 23:51 |
| *** huats has quit IRC | 23:51 | |
| sn0wb1rd | Could someone help me with the above question? | 23:51 |
| *** ywu_ has quit IRC | 23:52 | |
| harlowja | are there any plans to make the clients, possibly starting with novaclient use its extension mechanism internally, right now it seems there is a disconnect between internal and external extensions when everything could just be in contrib? | 23:52 |
| *** ywu_ has joined #openstack-dev | 23:52 | |
| *** sandywalsh_ has quit IRC | 23:55 | |
| *** huats has joined #openstack-dev | 23:55 | |
| *** huats has quit IRC | 23:55 | |
| *** huats has joined #openstack-dev | 23:55 | |
| *** n0ano has joined #openstack-dev | 23:55 | |
| *** lifeless has joined #openstack-dev | 23:55 | |
| *** lloydde has quit IRC | 23:58 | |
| *** JStoker has joined #openstack-dev | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!