vishy | trebuchet: seen this one? ClassNotFound: Class xenapi_net could not be found | 00:02 |
---|---|---|
tr3buchet | vishy: i have not | 00:02 |
tr3buchet | vmware refers to xenapi? | 00:02 |
vishy | trying to renable test_spawn_vlanmanager | 00:02 |
jhtran | would any of you know off the top of your head, where the OSAPI tests in nova go? | 00:02 |
tr3buchet | jhtran: yes | 00:02 |
vishy | 'module' object has no attribute 'xenapi_net' | 00:03 |
tr3buchet | jhtran: trunk/nova/tests/api/openstack | 00:03 |
jhtran | ah i see. thanks! | 00:04 |
vishy | i guess that network manager got removed | 00:04 |
jhtran | cool! Lots of tests in there | 00:04 |
tr3buchet | vishy: i've never heard of it... | 00:04 |
vishy | got removed recently i remember | 00:05 |
vishy | oh yeah there is another typo | 00:05 |
vishy | in a lot of the xenapi tests | 00:06 |
vishy | it throws a KeyError['domid'] | 00:07 |
vishy | tr3buchet: any idea about domid? | 00:13 |
vishy | tr3buchet it seems to be getting a string back instead of a vm_rec | 00:13 |
tr3buchet | hmm | 00:14 |
tr3buchet | where is this? | 00:14 |
*** rods has quit IRC | 00:15 | |
vishy | _make_plugin_call | 00:16 |
vishy | in xenapi/vmops.py | 00:16 |
vishy | in wait_for_boot | 00:17 |
vishy | it throws it a whole bunch | 00:17 |
*** donald650 has quit IRC | 00:21 | |
tr3buchet | vishy: i ran into that | 00:24 |
tr3buchet | before | 00:24 |
tr3buchet | i forgot about that | 00:24 |
tr3buchet | where was it... | 00:24 |
tr3buchet | .... | 00:24 |
tr3buchet | injecting network stuff if i recall | 00:24 |
tr3buchet | vishy: there won't be a domid if the instance isn't running | 00:25 |
vishy | i guess | 00:25 |
tr3buchet | ? | 00:26 |
vishy | the fake is not returning properly though | 00:26 |
vishy | the fake is returning the string 'domid' as vm_rec | 00:26 |
vishy | which makes no sense | 00:26 |
tr3buchet | is it just a typo maybe? | 00:27 |
tr3buchet | cause yeah, no sense at all | 00:28 |
tr3buchet | shit my lp810563 fix + bridge ended up being 200 lines | 00:28 |
tr3buchet | i'll have to test it tomorrow, i'm starving | 00:30 |
tr3buchet | vishy: i'll see ya later | 00:30 |
openstackjenkins | Yippie, build fixed! | 00:40 |
openstackjenkins | Project keystone build #44: FIXED in 1 min 5 sec: http://jenkins.openstack.org/job/keystone/44/ | 00:40 |
openstackjenkins | dolph.mathews: Making the API version configurable per API request | 00:40 |
vishy | soren, ttx: I'm a test fixing machine...Got them all but libvirt. Working on libvirt tests later tonight | 00:48 |
*** dragondm has quit IRC | 01:09 | |
*** jdurgin has quit IRC | 01:16 | |
*** vladimir3p has quit IRC | 01:19 | |
*** jhtran has quit IRC | 01:25 | |
*** jhtran has joined #openstack-dev | 01:29 | |
*** martine has joined #openstack-dev | 01:33 | |
*** jhtran has quit IRC | 01:34 | |
Tushar | Vishy: Skipped libvirt unit testcases are fixed in this merge proposal: https://code.launchpad.net/~rohitkarajgi/nova/libvirt_unittests/+merge/68144 | 01:35 |
*** Tushar has quit IRC | 01:36 | |
*** mwhooker has quit IRC | 01:37 | |
*** tadhgred has joined #openstack-dev | 02:06 | |
*** timr has quit IRC | 02:08 | |
*** deshantm_laptop has joined #openstack-dev | 02:24 | |
*** lorin1 has joined #openstack-dev | 02:26 | |
*** dolph has joined #openstack-dev | 02:27 | |
*** dolphm has joined #openstack-dev | 02:29 | |
*** lorin1 has quit IRC | 02:38 | |
*** deshantm_laptop_ has joined #openstack-dev | 02:55 | |
*** User694 has joined #openstack-dev | 02:57 | |
*** deshantm_laptop has quit IRC | 02:58 | |
*** deshantm_laptop_ is now known as deshantm_laptop | 02:58 | |
*** Tushar has joined #openstack-dev | 02:58 | |
*** Tushar has quit IRC | 03:00 | |
*** deshantm_laptop has quit IRC | 03:01 | |
*** deshantm_laptop has joined #openstack-dev | 03:02 | |
*** timr has joined #openstack-dev | 03:50 | |
*** tadhgred has quit IRC | 03:52 | |
*** tadhgred has joined #openstack-dev | 04:05 | |
*** timr has quit IRC | 04:07 | |
*** Tushar has joined #openstack-dev | 04:07 | |
*** dolphm has quit IRC | 04:09 | |
*** nci has joined #openstack-dev | 04:51 | |
*** zaitcev has quit IRC | 05:05 | |
*** martine has quit IRC | 05:19 | |
*** deshantm_laptop has quit IRC | 05:36 | |
*** jhtran has joined #openstack-dev | 05:38 | |
*** timr has joined #openstack-dev | 05:54 | |
*** tadhgred has quit IRC | 05:56 | |
vishy | Please to be reviewing my fixes for all of the skipped tests: | 06:43 |
vishy | https://code.launchpad.net/~vishvananda/nova/fix-more-skips/+merge/69382 | 06:43 |
vishy | https://code.launchpad.net/~vishvananda/nova/fix-libvirt-skips/+merge/69398 | 06:43 |
vishy | https://code.launchpad.net/~vishvananda/nova/fix-test-cloud/+merge/69373 | 06:44 |
*** reidrac has joined #openstack-dev | 07:01 | |
*** tomeff has joined #openstack-dev | 07:09 | |
HugoKuo__ | while I upgrade glance , there's an error with db stuff | 07:33 |
HugoKuo__ | sqlalchemy.exc.OperationalError: (OperationalError) table images already exists u'\nCREATE TABLE images etc... | 07:34 |
*** openpercept_ has joined #openstack-dev | 07:37 | |
*** tadhgred has joined #openstack-dev | 08:05 | |
*** timr has quit IRC | 08:06 | |
*** duker has joined #openstack-dev | 08:08 | |
*** tadhgred has quit IRC | 08:19 | |
*** tadhgred has joined #openstack-dev | 08:20 | |
ttx | HugoKuo__: bug 779311 ? | 08:23 |
uvirtbot` | Launchpad bug 779311 in glance "Glance update for Ubuntu failing" [Medium,Confirmed] https://launchpad.net/bugs/779311 | 08:23 |
HugoKuo__ | ttx , checking | 08:35 |
adam_g | any reason why the latest build for oneiric available in the trunk ppa is dated 07-14? | 08:39 |
ttx | adam_g: due to the boto 2 migration in oneiric | 08:42 |
ttx | bug 814365 | 08:42 |
uvirtbot` | Launchpad bug 814365 in nova "Should support boto 2.0 server-side (was: EC2 API fails with >=boto2.0)" [High,In progress] https://launchpad.net/bugs/814365 | 08:42 |
HugoKuo__ | ttx , seems same | 08:46 |
HugoKuo__ | ttx, a question | 08:46 |
adam_g | ttx: gotcha, thanks | 08:46 |
HugoKuo__ | ttx , what if you want to upgrade glance . just apt-get upgrade ? or should I do anything before ?.......how could glance migrate the db to new version .....? I think the scheme is different in different version of glance | 08:47 |
openstackjenkins | Project nova-milestone build #6: SUCCESS in 13 sec: http://jenkins.openstack.org/job/nova-milestone/6/ | 08:47 |
openstackjenkins | Tarmac: Cherry-pick of tr3buchet's fix for add_fixed_ip_to_instance. | 08:47 |
ttx | HugoKuo__: looks like this bug needs to be more properly resurrected. Under its current form, it flies under the radar | 08:48 |
HugoKuo__ | while I copy the old sqlite file to new glance and then glance-manage db sync . got same error | 08:48 |
ttx | jaypipes: we should probably create a new bug to track it | 08:48 |
ttx | HugoKuo__: expected behavior is to just apt-get upgrade | 08:48 |
ttx | HugoKuo__: if it fails, it's a bug we need to fix | 08:49 |
HugoKuo__ | ttx , ok got it | 08:49 |
HugoKuo__ | ttx , thaks | 08:49 |
*** mnour has joined #openstack-dev | 08:50 | |
ttx | If any nova-core dev is nightsleeping, please consider approving https://code.launchpad.net/~vishvananda/nova/fix-boto-2/+merge/69361 | 08:50 |
HugoKuo__ | ttx , additional question . Is the migration of glance easy to do ? I nerver success ..with copy /images and glance.sqlite from a old version to new one..... I though glance-manage will help user to handle the db table stuff | 08:51 |
ttx | HugoKuo__: I admit not having looked at it from an upgrade perspective yet. You should try to corner Glance devs | 08:53 |
*** winston-d has quit IRC | 09:05 | |
ttx | adam_g: I admit having trouble to reproduce your mysql db sync issue | 09:05 |
ttx | adam_g: just tried a db sync on a virgin maverick setup, works well | 09:05 |
*** duker has quit IRC | 09:06 | |
*** tomeff1 has joined #openstack-dev | 09:08 | |
adam_g | ttx: havnen't tried maverick, just tried running migrations manually on oneiric with some success, tho downgrading the migrations failed for similar reasons. booting natty once more right now | 09:10 |
* ttx refreshed natty setup | 09:10 | |
*** tomeff1 has quit IRC | 09:11 | |
*** alekibango has joined #openstack-dev | 09:11 | |
*** duker has joined #openstack-dev | 09:11 | |
*** tomeff has quit IRC | 09:12 | |
*** tomeff1 has joined #openstack-dev | 09:12 | |
*** darraghb has joined #openstack-dev | 09:13 | |
ttx | adam_g: db sync works on natty for me | 09:16 |
ttx | adam_g: Am I missing anything ? | 09:17 |
adam_g | ttx: yeah, odd. seeing the same. trying with a remote database as i was earlier | 09:17 |
ttx | adam_g: did you have data in that table, perhaps ? | 09:17 |
adam_g | ttx: nope, fresh database | 09:18 |
ttx | adam_g: setting to Incomplete, until you can reproduce it | 09:19 |
ttx | adam_g: If you have specific reproduceable downgrade issues, would be good to file bug as well | 09:19 |
adam_g | ttx: sure thing. | 09:21 |
adam_g | ttx: if you still have a mysql setup going, try setting 'default_storage_engine = InnoDB' | 09:49 |
ttx | adam_g: my show create table shows "ENGINE=InnoDB" so I suppose it's already active ? | 09:50 |
adam_g | ttx: there is a migration that sets all tables to innodb, long after they've been created | 09:51 |
ttx | hm | 09:51 |
adam_g | by default i believe MyISAM is used | 09:52 |
* ttx respawns vm | 09:52 | |
adam_g | 022_set_engine_mysql_innodb.py | 09:52 |
ttx | adam_g: the 'default_storage_engine' line, I should add it in the [mysqld] section of my.cnf, right ? | 09:55 |
adam_g | yah | 09:55 |
ttx | adam_g: yep, reproduced | 09:59 |
adam_g | wonder if sqlalchemy/migrate is able to cope with switching storage engine so late in the game. | 10:02 |
adam_g | bedtime. cya | 10:04 |
ttx | cya | 10:05 |
ttx | thx for your help in reproducing | 10:05 |
*** thickskin has quit IRC | 10:19 | |
*** thickskin has joined #openstack-dev | 10:19 | |
*** anm3rt has joined #openstack-dev | 10:28 | |
*** anm3rt has left #openstack-dev | 10:29 | |
Daviey | soren: you were looking at the glance test failure fro scrub? | 10:50 |
*** rods has joined #openstack-dev | 11:11 | |
*** markvoelker has joined #openstack-dev | 11:12 | |
*** mfer has joined #openstack-dev | 11:51 | |
*** martine has joined #openstack-dev | 12:05 | |
*** sandywalsh has joined #openstack-dev | 12:09 | |
*** bsza has joined #openstack-dev | 12:22 | |
*** bsza has quit IRC | 12:24 | |
*** bsza has joined #openstack-dev | 12:24 | |
*** mnour has quit IRC | 12:31 | |
*** mnour1 has joined #openstack-dev | 12:31 | |
*** mfer has quit IRC | 12:34 | |
*** mfer has joined #openstack-dev | 12:36 | |
*** chomping has quit IRC | 12:43 | |
*** lorin1 has joined #openstack-dev | 12:47 | |
ttx | jaypipes, soren: ok, so I know what's failing, thanks to Daviey for his help debugging | 12:50 |
ttx | will update bug | 12:50 |
Daviey | ttx: It seems glance needs more verbosity.. it should have been that tricky to find the reasoning for the failing test. | 12:54 |
ttx | Daviey: glance-control is surprisingly quiet in that case, I agree | 12:54 |
*** duker has quit IRC | 12:55 | |
*** duker has joined #openstack-dev | 12:55 | |
Daviey | ofc, s/should/shouldn't/ | 12:55 |
ttx | Glance code devs, please see https://code.launchpad.net/~ttx/glance/lp816386/+merge/69458 | 13:03 |
ttx | In related news, that test case was failing for a good reason :) | 13:04 |
*** ameade has joined #openstack-dev | 13:11 | |
ttx | sandywalsh: around ? | 13:18 |
sandywalsh | yessir | 13:18 |
ttx | need nova-core approval on https://code.launchpad.net/~vishvananda/nova/fix-boto-2/+merge/69361 | 13:18 |
sandywalsh | got it | 13:18 |
ttx | already been reviewed by a number of people, missing 2nd core approval though | 13:18 |
*** mfer has quit IRC | 13:19 | |
*** mfer has joined #openstack-dev | 13:20 | |
ttx | would like to doublecheck early that it actually fixes building on Oneiric | 13:20 |
*** mnour1 has quit IRC | 13:20 | |
*** mnour has joined #openstack-dev | 13:20 | |
*** lts has joined #openstack-dev | 13:22 | |
ttx | sandywalsh: if you have a few more cycles, https://code.launchpad.net/~vishvananda/nova/lp816555/+merge/69400 is easy too (and blocking D3 as well) | 13:22 |
ttx | (as well as all the lp:nova reviews at https://bugs.launchpad.net/nova/+bug/816699, but those take a bit more time) | 13:23 |
uvirtbot` | Launchpad bug 816699 in nova "skipped tests need to be reenabled" [High,In progress] | 13:23 |
*** mnour has quit IRC | 13:23 | |
*** mnour has joined #openstack-dev | 13:24 | |
*** mfer has quit IRC | 13:24 | |
*** bcwaldon has joined #openstack-dev | 13:24 | |
*** mfer has joined #openstack-dev | 13:24 | |
*** gaitan has joined #openstack-dev | 13:30 | |
*** mnour has quit IRC | 13:35 | |
*** mnour has joined #openstack-dev | 13:35 | |
*** mnour has quit IRC | 13:36 | |
sandywalsh | ttx, k, moving onto 816555 | 13:37 |
*** mnour1 has joined #openstack-dev | 13:37 | |
openstackjenkins | Project nova build #1,155: SUCCESS in 5 min 38 sec: http://jenkins.openstack.org/job/nova/1155/ | 13:52 |
openstackjenkins | Tarmac: Makes security group rules with the newer version of the ec2 api and correctly supports boto 2.0 | 13:52 |
*** markvoelker has quit IRC | 14:02 | |
* ttx will soon start Swift 1.4.2 release process, unless someone objects (/cc notmyname) | 14:07 | |
*** jkoelker has joined #openstack-dev | 14:19 | |
ttx | jkoelker: looks like I found the issue on https://bugs.launchpad.net/glance/+bug/816386 | 14:22 |
uvirtbot` | Launchpad bug 816386 in glance "glance-scrubber missing from tarball (was: test_scrubber functional tests fail on package build)" [High,In progress] | 14:22 |
ttx | jkoelker: should we keep your looping fix ? | 14:22 |
jkoelker | ttx: probably don't need it | 14:24 |
jkoelker | ttx: but it shouldn't hurt | 14:24 |
ttx | jkoelker: right | 14:24 |
jkoelker | ttx: wow, I'm an idiot ;) | 14:25 |
* jkoelker jsut saw the patch | 14:25 | |
ttx | jkoelker: see my postmortem notes on how to avoid that situation in the future | 14:26 |
ttx | jkoelker: it was a bit too hard to debug. | 14:26 |
*** kbringard has joined #openstack-dev | 14:26 | |
ttx | jkoelker: glance-control reporting "everythign is fine" did not really help. | 14:26 |
dabo | got a meeting now - may not make it back in time for standup | 14:28 |
dabo | basically nothing different from yesterday. Making some progress in re-creating my env; gonna look into the zone error stuff after that. | 14:28 |
*** tomeff has joined #openstack-dev | 14:32 | |
jkoelker | ttx: yea, I'll see if I can track down why it didn't exit with non-zero and file a bug on it | 14:34 |
mtaylor | anybody have any idea of how to actually parse a date like '2011/06/01 08:27:30 -0700' in python? strptime in 2.6 doesn't seem to actually support %z for that offset... | 14:35 |
kbringard | I've seen a lot of stuff comparing different hypervisors, but does anyone know of any good studies comparing KVM/Xen/etc. to bare metal? | 14:36 |
kbringard | mtaylor: that for the dashboard? | 14:36 |
mtaylor | kbringard: nope. it's for a script I'm working on - but jeez you'd think that would be a basic thing | 14:36 |
kbringard | (sorry, I don't know the answer to your question off the top of my head though) | 14:37 |
*** tomeff1 has quit IRC | 14:37 | |
kbringard | one would think | 14:37 |
kbringard | strftime claims to support it | 14:38 |
lorin1 | kbringard: For what types of applications? | 14:40 |
kbringard | lorin1: any really, I'm just curious how KVM and/or Xen compare to bare metal when running a few standard things... like something memory intensive, something disk intensive, cpu intensive, etc | 14:41 |
*** vladimir3p has joined #openstack-dev | 14:41 | |
kbringard | I'm trying to accomplish a couple of things which are kind of related, determine what kind of overhead I should expect and how much capacity is supported per metal node | 14:42 |
mtaylor | kbringard: it does - it's lying | 14:43 |
lorin1 | kbringard: There was a paper in this year's IEEE Cloud 2011 conference that was looking at the impact on virtualization vs. bare metal for scientific computing applications, I think. I'd have to dig to find the reference. | 14:43 |
mtaylor | kbringard: I mean, it does claim to support it - however, it does not | 14:43 |
kbringard | mtaylor: sucks, sorry dude | 14:43 |
mtaylor | kbringard: it's ok - string slicing is working for now :) | 14:43 |
lorin1 | kbringard: "Analysis of Virtualization Technologies for High Performance Computing Environments" was the name of the paper, first author is Andrew Younge. Out of Indiana University. | 14:44 |
kbringard | I saw something about it tzname and daylight being true | 14:44 |
kbringard | Support for the %Z directive is based on the values contained in tzname and whether daylight is true. Because of this, it is platform-specific except for recognizing UTC and GMT which are always known (and are considered to be non-daylight savings timezones). | 14:44 |
kbringard | mtaylor: it may be related to named timezones though | 14:44 |
kbringard | sorry, I'm relatively new to python, so if I'm not helping, feel free to tell me to shtup | 14:44 |
kbringard | lorin1: awesome, thanks, I'll go googling for it | 14:44 |
*** dragondm has joined #openstack-dev | 14:46 | |
kbringard | one last quick question (for now :-)): I read that KVM doesn't yet support pinning... so in my mind that translates to mean that even though we specify the number of cores to provision it's not really enforcing it; is that true? | 14:53 |
*** openpercept_ has quit IRC | 14:58 | |
*** dolphm has joined #openstack-dev | 15:15 | |
*** jjm has joined #openstack-dev | 15:15 | |
*** mnour has joined #openstack-dev | 15:25 | |
*** mnour1 has quit IRC | 15:25 | |
*** reidrac has left #openstack-dev | 15:27 | |
*** dspano has joined #openstack-dev | 15:36 | |
ttx | ok, let's see if glance package building is now unfucked | 15:37 |
kbringard | if it's on the repos I'll be a guinea pig | 15:40 |
*** danishman has joined #openstack-dev | 15:40 | |
ttx | looks like it's building ok now | 15:41 |
ttx | https://launchpad.net/~glance-core/+archive/trunk/+packages | 15:41 |
*** lorin1 has quit IRC | 15:41 | |
* jkoelker does the dance of joy | 15:42 | |
jaypipes | s1rp: bug thief. | 15:42 |
*** mattray has joined #openstack-dev | 15:43 | |
kbringard | 100% unrelated to anything, but if anyone is on the fence about getting a new MacBook Air: just do it | 15:44 |
kbringard | unless, of course, you believe the rumors about a new 15" air, and that's your bag | 15:44 |
*** mnour has quit IRC | 15:45 | |
*** mnour has joined #openstack-dev | 15:45 | |
blamar | kbringard: getting one for the wife I think but I'll never see it :( | 15:45 |
kbringard | haha, yea, I bought her an iPad awhile back, in an attempt to distract her from the Air I knew I'd be getting soon | 15:45 |
kbringard | now she has an iPad and an Air | 15:46 |
blamar | kbringard: my wife? | 15:46 |
blamar | :P | 15:46 |
kbringard | lol | 15:46 |
kbringard | no commewnt | 15:46 |
blamar | fair enough | 15:46 |
kbringard | I'm perfectly fine with random people buying my wife expensive crap, so long as they don't expect anything in return | 15:46 |
ttx | ok, Swift 1.4.2 out. | 15:47 |
ttx | jaypipes: pushing the scrubber fix in milestone-proposed | 15:48 |
ttx | oh, you did it already | 15:48 |
ttx | vishy: ping | 15:51 |
creiht | woot | 15:51 |
*** mdomsch has joined #openstack-dev | 15:55 | |
*** donald650 has joined #openstack-dev | 15:56 | |
kbringard | hmmmm, I recall jay talking about Python 2.6.1 on OSX yesterday... is there any reason why pep8 wouldn't be running in that environment? | 15:59 |
bcwaldon | kbringard: the OSX/py2.6.1 thing was related to a urlparse bug, btw | 16:06 |
kbringard | ah, OK, then unrelated | 16:06 |
bcwaldon | urlparse python2.6.1 and python2.6.6 parse urls differently ;) | 16:06 |
kbringard | I'm not getting any output when I run pep8 tests, which is disconcerting because a merge of mine had some pep8 bugs | 16:06 |
kbringard | or "warnings" I should say | 16:07 |
bcwaldon | well somtimes a local pep8 v0.5.0 will complain while v0.6.1 on tarmac will not | 16:07 |
bcwaldon | just make sure you are using 0.6.1 and only checking the files indicated in run_tests.sh | 16:07 |
bcwaldon | some files are excluded | 16:08 |
kbringard | yea, I'm just running run_tests.sh -p | 16:08 |
kbringard | on my branch | 16:08 |
bcwaldon | right, and you don't get any output? | 16:08 |
kbringard | correct | 16:08 |
bcwaldon | run ./tools/with_venv.sh pep8 --version | 16:08 |
dolphm | is it intentional that you can +1 your own commits in gerrit? | 16:08 |
kbringard | 0.6.1 | 16:08 |
bcwaldon | kbringard: then you're fine | 16:08 |
kbringard | ok, any suggestions for how to respond to this bug? | 16:09 |
kbringard | https://bugs.launchpad.net/nova/+bug/816725 | 16:09 |
uvirtbot` | Launchpad bug 816725 in nova "pep8 failures" [Undecided,In progress] | 16:09 |
bcwaldon | kbringard: I added a comment | 16:10 |
*** markvoelker has joined #openstack-dev | 16:10 | |
kbringard | ah, yea, and I see that Dave noticed as well and updated | 16:11 |
bcwaldon | kbringard: just let it sit and ttx can invalidate it if he agrees | 16:11 |
kbringard | okie, cool, thanks! | 16:11 |
bcwaldon | ttx: ^ | 16:11 |
kbringard | makes me feel better too | 16:12 |
* ttx looks | 16:13 | |
ttx | right | 16:14 |
*** markvoelker has quit IRC | 16:14 | |
*** danishman has quit IRC | 16:15 | |
kbringard | cool, thanks... easiest bug fix ever :-p | 16:16 |
bcwaldon | indeed | 16:16 |
soren | mtaylor: I can't access http://jenkins.openstack.org/ right now.. Is that known? | 16:17 |
soren | mtaylor: It seems to keep reloading. | 16:17 |
mtaylor | soren: looking | 16:17 |
soren | (I got the openid thing from Launchpad first, got sent back to Jenkins and am now stuck) | 16:17 |
mtaylor | soren: works fine for me | 16:17 |
soren | mtaylor: Log out and back in? | 16:17 |
dolphm | soren: i'm betting you're in some kind of openid loop | 16:18 |
mtaylor | soren: give a shot at that | 16:18 |
* mtaylor blames soren's browser | 16:18 | |
soren | mtaylor: I can't. i'm suggesting *you* do it, so that you can reproduce it. | 16:18 |
Daviey | soren is stacking up blame today. | 16:18 |
mtaylor | soren: I did. | 16:18 |
soren | mtaylor: Can't log out, since I'm not logged in :) | 16:18 |
soren | Ah. | 16:18 |
mtaylor | soren: I was not logged in when I loaded the url | 16:18 |
soren | Oddness. | 16:19 |
mtaylor | is anybody else having problems getting to jenkins? or just soren? | 16:19 |
* soren tries something else | 16:19 | |
kbringard | worked for me | 16:19 |
soren | Worked in inkognito moe. | 16:20 |
soren | mode, even. | 16:20 |
soren | Meh. /me reboots web browser | 16:20 |
bcwaldon | works for me | 16:20 |
kbringard | that's what soren gets for trying to build broken merges as someone else ;-) | 16:21 |
*** markvoelker has joined #openstack-dev | 16:21 | |
soren | ?!? | 16:22 |
kbringard | haha, just messing with you | 16:22 |
dolphm | soren: log out of launchpad, gerrit, etc too | 16:23 |
soren | dolphm: Just tried that. No dice. | 16:26 |
soren | Weird. | 16:26 |
* soren nukes some more cookies | 16:28 | |
*** jdurgin has joined #openstack-dev | 16:30 | |
soren | Ok, I've removed all cookies from *.launchpad.net and *.openstack.org. Didn't help. Suggestions? | 16:31 |
openstackjenkins | Project nova build #1,156: SUCCESS in 4 min 2 sec: http://jenkins.openstack.org/job/nova/1156/ | 16:31 |
openstackjenkins | Tarmac: Re-enables and fixes test_cloud tests that broke from multi_nic. | 16:31 |
vishy | tr3buchet: how is the fix for that bug coming? | 16:33 |
tr3buchet | good. i've got the version for milestone coded, testing now | 16:34 |
vishy | tr3buchet: ok we're getting close on the milestone, I think we can put in the mysql one as a known issue for now | 16:34 |
tr3buchet | vishy: i tried to replicate that one, and couldn't | 16:35 |
vishy | tr3buchet: yeah it looks like it only happens if you initially create the db as innodb | 16:36 |
*** mdomsch has quit IRC | 16:36 | |
tr3buchet | ah | 16:37 |
openstackjenkins | Project nova-milestone build #7: SUCCESS in 14 sec: http://jenkins.openstack.org/job/nova-milestone/7/ | 16:37 |
openstackjenkins | Tarmac: Fix for boto2. | 16:37 |
soren | \o/ | 16:37 |
tr3buchet | yay | 16:37 |
soren | mtaylor: Aha! | 16:38 |
vishy | other two fix skipped test branches could use review | 16:38 |
mtaylor | soren: you got it? | 16:38 |
soren | mtaylor: If I set my local time to Portland time, I can't log in. | 16:39 |
mtaylor | soren: what was up? | 16:39 |
mtaylor | soren: WEIRD | 16:39 |
soren | mtaylor: Jenkins sends back a cookie that is already expired. | 16:39 |
mtaylor | beautiful | 16:39 |
soren | At least I think it's Jenkins. Let me check agian. | 16:39 |
vishy | https://code.launchpad.net/~vishvananda/nova/fix-libvirt-skips/+merge/69399 | 16:39 |
mtaylor | why wouldn't that just re-direct you to log in again? | 16:39 |
vishy | oops | 16:40 |
vishy | this one: https://code.launchpad.net/~vishvananda/nova/fix-libvirt-skips/+merge/69398 | 16:40 |
soren | mtaylor: Nope, it's not. | 16:40 |
soren | mtaylor: It's login.ubuntu.com. | 16:40 |
vishy | and this: https://code.launchpad.net/~vishvananda/nova/fix-more-skips/+merge/69382 | 16:40 |
soren | mtaylor: Darn it. | 16:40 |
vishy | and: https://code.launchpad.net/~vishvananda/nova/lp816555/+merge/69400 | 16:41 |
mtaylor | soren: beautiful. tell lifeless | 16:41 |
soren | mtaylor: I don't grok openid 100%. I end up in a redirect loop, not a "please log in" loop. Not sure why. | 16:41 |
vishy | sandywalsh: can i get a rereview on ^^ | 16:41 |
sandywalsh | vishy, looking at it now :) | 16:41 |
openstackjenkins | Project nova-milestone build #8: SUCCESS in 12 sec: http://jenkins.openstack.org/job/nova-milestone/8/ | 16:42 |
openstackjenkins | Tarmac: Re-enables and fixes test_cloud tests that broke from multi_nic. | 16:42 |
vishy | blamar: ping | 16:44 |
blamar | vishy: greetings | 16:44 |
*** mnour has quit IRC | 16:44 | |
*** mnour1 has joined #openstack-dev | 16:44 | |
vishy | blamar: did you do the exception refactor? | 16:45 |
blamar | vishy: bcwaldon I think, if I'm thinking of the refactor you're thinking of | 16:46 |
bcwaldon | vishy: yes, twas me | 16:46 |
tr3buchet | vishy: just realized I left my vpn token at the office, i'll need to drive there to do more testing | 16:46 |
tr3buchet | vishy: here's the branch for the milestone: https://code.launchpad.net/~tr3buchet/nova/lp810563-ms | 16:47 |
tr3buchet | be back in a few | 16:47 |
vishy | bcwaldon: ah in that case | 16:47 |
s1rp | jaypipes: heh, must have assigned at nearly the same time | 16:47 |
vishy | bcwaldon: do you mind the following added to your plate? https://bugs.launchpad.net/nova/+bug/817107 | 16:48 |
uvirtbot` | Launchpad bug 817107 in nova "Exceptions should be united and exception.wrap_exception should be updated or removed" [Undecided,New] | 16:48 |
bcwaldon | vishy: I'll take it | 16:48 |
s1rp | jaypipes: you want me to keep on with that, or do you have something already for it? | 16:48 |
vishy | bcwaldon: I don't think it is a huge amount of work. It seems that wrap_exception isn't really doing anything valuable at this point | 16:48 |
vishy | but we still have a few Error based exceptions. | 16:48 |
bcwaldon | vishy: yeah, I didn't feel comfortable blowing it all away at the time | 16:49 |
jaypipes | s1rp: please go for it. I have a couple other bugs I'm working on (boto2 stuff.) | 16:49 |
vishy | bcwaldon: I noticed it because I was adding a test for libvirt checking for a raise, and it was getting wrapped because it didn't derive from Error | 16:49 |
s1rp | jaypipes: k | 16:49 |
bcwaldon | vishy: gotcha. I'll take a closer look later this week | 16:49 |
jaypipes | s1rp: cheers | 16:49 |
vishy | bcwaldon: I added a quick change so it doesn't wrap NovaExceptions either, but i think the right move is probably just to kill wrap_exception | 16:50 |
vishy | bcwaldon: cool thanks | 16:50 |
*** rupakg has joined #openstack-dev | 16:51 | |
soren | uvirtbot`: nick uvirtbot | 16:52 |
*** uvirtbot` is now known as uvirtbot | 16:52 | |
sandywalsh | vishy, can't kill wrap_exception ... that's our only hook to the notification system for detecting failed operations | 16:53 |
sandywalsh | vishy, it could ultimately be the hook for orchestration I think? | 16:53 |
*** mnour1 has quit IRC | 16:53 | |
*** mnour has joined #openstack-dev | 16:53 | |
vishy | sandywalsh: ah, sorry didn't see that that was added | 16:53 |
sandywalsh | so vishy, will that other branch make it in for D3 and fix that failing test? | 16:54 |
vishy | sandywalsh, bcwaldon: well we just need to get rid of Error then | 16:54 |
bcwaldon | vishy: loud and clear | 16:54 |
sandywalsh | vishy that would be good | 16:54 |
vishy | sandywalsh: sorry? | 16:55 |
vishy | sandywalsh: the test i put in is passing in the branch | 16:55 |
vishy | sandywalsh: is that what you mean? | 16:55 |
sandywalsh | vishy, not for me, fails | 16:56 |
sandywalsh | vishy, the raise was removed and the test has an assertRaises is still there | 16:57 |
vishy | sandywalsh: oh? hmm perhaps it only passes on mac. let me retry on ubuntu | 16:57 |
vishy | sandywalsh: the raise is in the other method | 16:57 |
sandywalsh | vishy, merge diff +51 and +27 | 16:57 |
sandywalsh | hmm | 16:58 |
sandywalsh | vishy, oh, it was another test that failed | 16:58 |
vishy | sandywalsh: the removed test was redundant | 16:58 |
sandywalsh | test_compute.ComputeTestCase | 16:58 |
sandywalsh | lemme retry | 16:58 |
vishy | sandywalsh: ah might be that it was checking for a wrapped exception | 16:58 |
vishy | let me check it | 16:59 |
sandywalsh | could be heisenbug | 16:59 |
vishy | test_resize_same_source_fails | 16:59 |
vishy | that one? | 16:59 |
sandywalsh | http://paste.openstack.org/show/1959/ | 17:00 |
sandywalsh | yup | 17:00 |
vishy | yeah it is checking for Error | 17:02 |
vishy | i should figure out what exception it should be getting | 17:02 |
vishy | interesting i just uncovered a bug i think | 17:03 |
vishy | sandywalsh: it is getting InstanceNotFound and I don't think that is the exception that was being expected | 17:03 |
sandywalsh | hmm, looking now | 17:04 |
sandywalsh | vishy, it was expecting the exception to be wrapped ... so perhaps we just fix that test? | 17:08 |
vishy | sandywalsh: no actually | 17:08 |
vishy | it was passing because it got the wrong exception | 17:08 |
vishy | it was being passed instance_id instead of a uuid | 17:09 |
sandywalsh | ah, right | 17:09 |
vishy | but because it was checking for Error it was passing | 17:09 |
vishy | because of the wrap | 17:09 |
sandywalsh | yup, false positive | 17:09 |
vishy | I pushed a fix | 17:09 |
sandywalsh | nice, pulling | 17:09 |
sandywalsh | vishy, done | 17:17 |
*** mnour has quit IRC | 17:19 | |
vishy | soren: ping | 17:24 |
vishy | mtaylor: ping | 17:27 |
vishy | ttx: ping | 17:28 |
mtaylor | vishy: whazzup? | 17:30 |
tr3buchet | anyone seen ryu around? | 17:31 |
*** rupakg has quit IRC | 17:31 | |
vishy | mtaylor: so it looks like all of the launchpad builders blew up on the last merge | 17:32 |
mtaylor | vishy: yay! | 17:32 |
vishy | mtaylor: although hudson and local tests run fine | 17:32 |
*** rupakg has joined #openstack-dev | 17:32 | |
mtaylor | vishy: looking | 17:32 |
mtaylor | vishy: http://paste.openstack.org/show/1960/ | 17:34 |
vishy | mtaylor: two of them blew up with an odd looking error about a missing flag | 17:34 |
vishy | yeah that | 17:34 |
mtaylor | vishy: yeah | 17:34 |
vishy | the other two just look like they hung | 17:34 |
vishy | is it possible a corrupted tarball was uploaded or something? | 17:34 |
mtaylor | do we have different versions of libs? I mean, we're not using a weird version of gflags are we? | 17:35 |
mtaylor | corrupted tarball is unlikely | 17:35 |
vishy | the thing is nothing even remotely related to flags changed in that branch | 17:35 |
mtaylor | yeah. that's pretty vexing | 17:36 |
vishy | the only thing that changed was tests/test_cloud | 17:37 |
vishy | it was just the last build that failed right? | 17:38 |
mtaylor | vishy: https://launchpad.net/~nova-core/+archive/trunk/+files/nova_2011.3~d4~20110727.1329.orig.tar.gz is the tarball used for that build | 17:38 |
vishy | the one before that branch was all roses? | 17:38 |
mtaylor | and yeah - things look green before that | 17:38 |
vishy | pulled that tarball, tests run fine on my mac | 17:40 |
vishy | mtaylor: they run fine from the tarball on my maverick vm as well | 17:42 |
mtaylor | grrr | 17:43 |
vishy | mtaylor: so seems like a dependency. Did something change recently | 17:43 |
vishy | either a) un upstream dependency or b) a change in the packaging scripts? | 17:43 |
mtaylor | vishy: not _that_ recently, no | 17:44 |
mtaylor | as in - not between the last build and this one | 17:44 |
vishy | wow super weird | 17:45 |
vishy | i have no idea how to figure out what went wrong | 17:45 |
mtaylor | I don't either - I'm thinking about re-running to see if it was some ephemeral thing there | 17:46 |
vishy | mtaylor: we should at least try it | 17:47 |
mtaylor | doing now | 17:47 |
vishy | ok afk a few. I'll check back | 17:49 |
*** darraghb has quit IRC | 17:54 | |
mtaylor | devcamcar: were you the one talking to me about missing things for dashboard on jenkins? | 17:56 |
openstackjenkins | Project nova build #1,157: SUCCESS in 4 min 10 sec: http://jenkins.openstack.org/job/nova/1157/ | 18:16 |
openstackjenkins | Tarmac: Remove xenapi_inject_image flag. | 18:16 |
*** bsza has quit IRC | 18:20 | |
*** bsza has joined #openstack-dev | 18:22 | |
jaypipes | clayg: around? wondering if you could poke at https://code.launchpad.net/~jaypipes/glance/bug817121? Should fix the URI parsing and get_backend_class() problems you were experiencing, and adds unit tests to verify. | 18:22 |
dolphm | can anyone link me to an openstack implementation of a 300 multiple choice response? | 18:23 |
*** mwhooker has joined #openstack-dev | 18:26 | |
clayg | jaypipes: that won't fix the issue, see https://code.launchpad.net/~clay-gerrard/glance/fix_swift_http/+merge/69354 | 18:29 |
clayg | jaypipes: to be fair, my merge doesn't fix all the issues either - just the ones that I'm running into in my lab ;) | 18:30 |
clayg | bcwaldon: ping - I don't follow your last comment. I'd rather not create a glance speicific exception that is only raised once in a mock of non glance component. I'm hoping I misunderstood you. Might be better to focus on Jay's branch anyway. I'm happy to bow out. | 18:32 |
bcwaldon | clayg: I was thinking the actual implementation would use whatever exception you put in the mock | 18:32 |
bcwaldon | clayg: but it's not really a big deal | 18:33 |
*** mwhooker has quit IRC | 18:35 | |
*** mwhooker has joined #openstack-dev | 18:36 | |
*** bcwaldon has quit IRC | 18:38 | |
*** AhmedSoliman has joined #openstack-dev | 18:39 | |
*** jkoelker has quit IRC | 18:40 | |
*** bcwaldon has joined #openstack-dev | 18:40 | |
*** jkoelker has joined #openstack-dev | 18:40 | |
*** bcwaldon_ has joined #openstack-dev | 18:41 | |
*** bcwaldon_ has joined #openstack-dev | 18:42 | |
vishy | mtaylor: https://launchpad.net/~nova-core/+archive/trunk/+build/2651604/+files/buildlog_ubuntu-oneiric-i386.nova_2011.3%7Ed4%7E20110727.1330-0ubuntu0ppa1%7Eoneiric1_FAILEDTOBUILD.txt.gz | 18:42 |
vishy | mtaylor: looks like it is not getting glance properly? | 18:42 |
vishy | mtaylor: the other ones are still having the gflags errors | 18:44 |
*** bcwaldon has quit IRC | 18:44 | |
mtaylor | vishy: jesus | 18:45 |
mtaylor | vishy: landing - will look when I'm back online | 18:46 |
*** sriramkr has joined #openstack-dev | 18:58 | |
*** shehjart has quit IRC | 19:02 | |
*** HugoKuo__ has quit IRC | 19:15 | |
tr3buchet | can i get some eyes on this branch: https://code.launchpad.net/~tr3buchet/nova/lp810563-ms | 19:16 |
tr3buchet | specifically bang on nova-manage create network with whatever wacky configurations you have to make sure it handles everything correctly | 19:17 |
tr3buchet | i'm pretty sure it does | 19:17 |
tr3buchet | vishy: ttx: zulcss: any else interested: ^ | 19:18 |
tr3buchet | if it works to everyone's satisfaction we can merge it into milestone proposed, i'm going to merge trunk and push another branch to propose merging to trunk as well so you can see the diff | 19:19 |
tr3buchet | merge prop: https://code.launchpad.net/~tr3buchet/nova/lp810563/+merge/69536 | 19:25 |
*** lorin1 has joined #openstack-dev | 19:26 | |
*** rupakg has quit IRC | 19:34 | |
*** AhmedSoliman has quit IRC | 19:38 | |
vishy | tr3buchet: if you don't specify a bridge with network create what happens? | 19:40 |
vishy | tr3buchet: it should merge cleanly into trunk | 19:40 |
tr3buchet | vishy: if you are using flat or flatDHCP and have not set the flat_network_bridge flag, it will raise suggesting --bridge is necessary | 19:41 |
vishy | tr3buchet: also i think the dict() around net_v6 is unneccessary. Looks like it is always a dict | 19:41 |
tr3buchet | vishy: i've just changed that part anyway | 19:41 |
tr3buchet | dumb mistake there | 19:41 |
vishy | tr3buchet: can we keep the default as br100? | 19:42 |
vishy | a lot of install scripts don't specify one | 19:42 |
tr3buchet | vishy: i don't want to because then it will always be set no matter what, even if it is a wrong value, and then we can't catch it | 19:42 |
vishy | can we default in nova-manage then? | 19:43 |
vishy | i'd prefer not to break install scripts again | 19:43 |
tr3buchet | vishy: we can, but again, if that's an incorrect value we are in the same place | 19:44 |
vishy | hmm | 19:45 |
vishy | what makes it an incorrect value? | 19:45 |
vishy | in vlan mode it is autogenerated right? | 19:46 |
vishy | maybe we could put a message in | 19:46 |
tr3buchet | yes in vlan mode you don't need to pass in --bridge or use the flag | 19:46 |
tr3buchet | and the command works with that | 19:46 |
vishy | if the flag is None then set it to br100 but put a big message | 19:46 |
vishy | bridge not specified, using br100 to remain compatible with older scripts | 19:47 |
tr3buchet | but see for example we use xenbr0 | 19:47 |
vishy | we've already messed up a lot of people with the label parameter | 19:47 |
tr3buchet | so setting it to br100 and moving on then means we now have bad data in the db | 19:47 |
vishy | tr3buchet: so, you set it in a flag right? | 19:48 |
tr3buchet | right, i'm saying we should make the command enforce using the flag | 19:48 |
tr3buchet | instead of trying to suggest a value (br100) that might work and just moving on as though it will | 19:48 |
vishy | but that is exactly how it worked previously | 19:49 |
*** jaypipes has quit IRC | 19:49 | |
vishy | i don't understand the need to change the functionality | 19:49 |
tr3buchet | no, previously it used the flag, or the flag's default value | 19:49 |
tr3buchet | so even though i didn't set my flag, and br100 is wrong for my implementation i now have networks that don't work and have no idea why | 19:49 |
vishy | right, the flag defaulted to br100 | 19:50 |
tr3buchet | instead of having no networks and a reason why they couldn't be created | 19:50 |
tr3buchet | you are right about breaking scripts though | 19:50 |
vishy | it seems like you are fixing an issue that we haven't really hit | 19:50 |
tr3buchet | so are the scripts still broken if the only change is to add a flag to your conf? | 19:50 |
*** bcwaldon_ has quit IRC | 19:50 | |
*** bcwaldon has joined #openstack-dev | 19:51 | |
vishy | it means we have to add that flag to all of our instructions | 19:51 |
tr3buchet | i mostly just wanted to either have a default that always works, or not have a default at all | 19:51 |
vishy | tr3buchet: but there are already a bunch of defaults that don't always work | 19:51 |
vishy | the default libvirt driver won't work if you don't have hw vt | 19:52 |
vishy | or if you are using xen | 19:52 |
tr3buchet | true, and i suppose we can never fix them all | 19:52 |
tr3buchet | so can we leave the flag default to None | 19:52 |
tr3buchet | use your idea to point out a message | 19:52 |
vishy | tr3buchet: I agree with your usage of the flag, that is how it 'should' be, but i think we should leave the old behavior for compatibility | 19:52 |
tr3buchet | even though it's old and busted? | 19:53 |
vishy | i really wish we had done the same for --glance_host and for the added label in nova-manage | 19:53 |
tr3buchet | i figured that since this command was breaking scripts anyway | 19:53 |
tr3buchet | as did the lvov patch | 19:53 |
tr3buchet | that now was the perfect time | 19:53 |
vishy | the lvov patch actually didn't break things | 19:54 |
vishy | you can still specify params in order | 19:54 |
tr3buchet | well sure, but chances are you want to use the command as it is meant to be now | 19:54 |
vishy | but i see your point | 19:54 |
tr3buchet | this backwards compatibility for scripts just gets worse over time | 19:55 |
vishy | ugh, this one is going to break everyone's install | 19:57 |
vishy | who is using kvm | 19:57 |
vishy | and flat | 19:57 |
tr3buchet | mailing list about the flag? | 19:57 |
jk0 | vishy: https://code.launchpad.net/~jk0/nova/lp816236/+merge/69545 | 19:58 |
ttx | vishy: pong | 19:58 |
tr3buchet | if it's if it is a large enough issue, i'll concede to br100, but setting it in nova-manage, and using a deprecated warning that says you should set flat_network_bridge FLAG | 19:58 |
tr3buchet | i also want to stick a sleep(10) in there or something too so it's visible | 19:59 |
jk0 | ttx: you may also be interested in https://code.launchpad.net/~jk0/nova/lp816236/+merge/69545 if I recall, it is supposed to go into d3? | 20:08 |
ttx | jk0: yes | 20:08 |
vishy | jk0: nice | 20:12 |
vishy | ttx: do you have input on the tr3buchet issue? | 20:13 |
ttx | jk0: we'll need the same fix proposed to the milestone-proposed branch | 20:13 |
vishy | ttx: i was trying to track down some help | 20:13 |
jk0 | ttx: ok, hold on to your short | 20:13 |
vishy | for figuring out why the nova launchpad builders are all failing | 20:13 |
jk0 | *shorts even | 20:13 |
ttx | reading backlog | 20:14 |
tr3buchet | ttx: i removed the default value of the flat_network_bridge flag | 20:14 |
jk0 | ttx: https://code.launchpad.net/~jk0/nova/lp816236/+merge/69550 | 20:14 |
vishy | jk0: you need to remake the branch off of milestone proposed | 20:15 |
jk0 | ah, crap | 20:15 |
jk0 | I'll fix | 20:16 |
vishy | jk0: have you tried where mysql is not set to innodb as well | 20:16 |
ttx | someone had a magic bzr merge command to cherrypick the fix | 20:16 |
jk0 | vishy: yeah, that was the first thing I tried | 20:16 |
ttx | adam_g: could you check the fix from https://code.launchpad.net/~jk0/nova/lp816236/+merge/69545 ? | 20:16 |
vishy | bzr merge <branch> -c <revno> | 20:16 |
ttx | vishy: builds on trunk fail with missing python-xattr build dep apparently | 20:19 |
ttx | hmm that doesn't make sense | 20:22 |
jk0 | ttx / vishy: https://code.launchpad.net/~jk0/nova/lp816236-d3/+merge/69552 | 20:23 |
jk0 | python-xattr was added to glance recently as a dep for caching | 20:24 |
ttx | soren: around ? | 20:25 |
ttx | I just don't see why it would suddenly need xattr | 20:29 |
bcwaldon | ttx: that's how cached images are identified | 20:29 |
ttx | I mean why Nova would need it | 20:29 |
bcwaldon | because it needs glance ;) | 20:30 |
vishy | ah, that makes sense | 20:30 |
bcwaldon | glance client/server aren't in different packages (yet) | 20:30 |
bcwaldon | and server needs xattr | 20:30 |
vishy | yes it must be the use of glance client | 20:30 |
bcwaldon | vishy: exactly | 20:30 |
ttx | hm | 20:30 |
ttx | I can try to push a quick fix. An extra builddep never hurts anyway | 20:30 |
*** med_out is now known as medberry | 20:32 | |
vishy | awesome thanks ttx | 20:35 |
ttx | ...and trigger a new build | 20:36 |
ttx | The commit it started to fail to build didn't introduce any glance change though | 20:37 |
ttx | so I'm unsure of the fix | 20:37 |
ttx | vishy: could you propose your boto2 fix to milestone-proposed ? | 20:40 |
ttx | I know that one builds ok | 20:40 |
vishy | ttx: it is proposed and in | 20:41 |
ttx | ah, hm | 20:41 |
vishy | ttx: the test_cloud stuff may have re-enabled a glance client import that was disabled? | 20:41 |
ttx | maybe | 20:41 |
vishy | ttx: or maybe that was the first build since glance updated their packages? | 20:42 |
* ttx is missing some emails for some reason | 20:42 | |
ttx | vishy: at this hour, I'm ready to accept that explanation, if the fix fixes it | 20:42 |
* vishy crosses fingers | 20:43 | |
vishy | still have 3 branches that need one more review | 20:43 |
*** lorin1 has quit IRC | 20:45 | |
bcwaldon | vishy: I'll look at lp816555 | 20:45 |
openstackjenkins | Project nova build #1,158: SUCCESS in 4 min 20 sec: http://jenkins.openstack.org/job/nova/1158/ | 20:51 |
openstackjenkins | Tarmac: Drop FK before dropping instance_id column. | 20:51 |
adam_g | ttx: looking now | 20:52 |
*** timr has joined #openstack-dev | 20:52 | |
ttx | adam_g: bah, fix is in, now, so it has better work | 20:53 |
* vishy goes to revert it | 20:53 | |
vishy | :p | 20:53 |
ttx | suspense on the packagng fix is at a peak | 20:53 |
ttx | follow at home: https://launchpad.net/~nova-core/+archive/trunk/+packages | 20:53 |
*** LoriC has quit IRC | 20:54 | |
*** tadhgred has quit IRC | 20:54 | |
vishy | dum-bum-bum | 20:54 |
ttx | hooray | 20:55 |
ttx | natty built, so the others should be ok too | 20:55 |
ttx | vishy: things look good now ? | 20:56 |
vishy | woohoo | 20:56 |
vishy | i just popped in the mysql fix | 20:56 |
vishy | waiting on 4 fixes still | 20:57 |
vishy | tr3buchet: still alive? | 20:57 |
ttx | okay, will pick up the pieces tomorrow | 20:57 |
* ttx disappears | 20:57 | |
vishy | should be ready to go by the time you wake up | 20:57 |
vishy | :) | 20:57 |
*** martine has quit IRC | 21:00 | |
adam_g | jk0: was that migration patch tested on non-innodb databases as well? | 21:03 |
jk0 | yeah | 21:03 |
*** mdomsch has joined #openstack-dev | 21:04 | |
adam_g | jk0: ok, ill test as well. i was going to propose the same fix, but as far as i can tell, FKs are only named $table_ibfk_N when innodb is the default, and in other cases migrate is able to introspect the database and drop the FKs automatically as it needs | 21:04 |
jeblair | jay pipes would like people to know that his laptop has died, and he won't be able to be on irc for a while. | 21:05 |
vishy | adam_g, jk0: doesn't it just ignore in that case? | 21:05 |
jk0 | hm, I'm getting an error now in trunk | 21:06 |
jk0 | one sec | 21:06 |
adam_g | vishy: the database ignore the DROP query? ya, probably | 21:06 |
adam_g | er, ALTER | 21:07 |
jk0 | the DB does, but the migration fails | 21:07 |
jk0 | just needs to be wrapped in a try | 21:07 |
openstackjenkins | Project nova-milestone build #9: SUCCESS in 14 sec: http://jenkins.openstack.org/job/nova-milestone/9/ | 21:07 |
vishy | jk0: doh | 21:07 |
openstackjenkins | Tarmac: Drop FK before dropping instance_id column. | 21:07 |
vishy | new fix! | 21:07 |
jk0 | fix coming | 21:07 |
vishy | if you make it on milestone | 21:07 |
vishy | you can propose the same branch into both | 21:07 |
vishy | jk0: ^' | 21:08 |
jk0 | ok | 21:08 |
adam_g | wonder if it can be later fixed in sqlalchemy migrate. i believe workarounds exist there for similar cases with oracle | 21:08 |
openstackjenkins | Project nova build #1,159: SUCCESS in 4 min 7 sec: http://jenkins.openstack.org/job/nova/1159/ | 21:16 |
openstackjenkins | Tarmac: Fixes typo in attach volume. | 21:16 |
mwhooker | Hi guys. I'm fairly new to openstack and would like to make some contributions. Are there any bugs people could recommend I work on? | 21:21 |
vishy | mwhooker: we could always use more tests | 21:22 |
vishy | see http://wiki.openstack.org/SmallTestingGuide | 21:23 |
mwhooker | vishy: great, I'll look at the coverage reports. any modules which particularly need help? | 21:23 |
vishy | also general test cleanup would be useful | 21:23 |
vishy | a lot of the tests are throwing things into the database when they should be using mocks, etc. | 21:24 |
bcwaldon | mwhooker: Check this out for coverage: http://jenkins.openstack.org/job/nova-coverage/990/cobertura/? | 21:24 |
bcwaldon | mwhooker: you can execute run_tests.sh -c to get a local code-coverage report | 21:25 |
*** rjimenez has joined #openstack-dev | 21:25 | |
mwhooker | thanks | 21:26 |
openstackjenkins | Project nova build #1,160: SUCCESS in 4 min 9 sec: http://jenkins.openstack.org/job/nova/1160/ | 21:27 |
openstackjenkins | * Tarmac: Fix the skipped tests in vmwareapi and misc spots. The vmware networking stuff is stubbed out, so the tests can be improved there by fixing the fakes. | 21:27 |
openstackjenkins | * Tarmac: Fix all of the skipped libvirt tests. | 21:27 |
tr3buchet | vishy: yes still alive, been trying to get a laptop for a while | 21:30 |
vishy | looks like just your branch and the refix by jk0 to go in | 21:31 |
jk0 | vishy: coming in a few mins | 21:31 |
tr3buchet | vishy: sorry i got behind. what did you decide? | 21:37 |
jk0 | vishy: https://code.launchpad.net/~jk0/nova/lp816236-d3/+merge/69563 | 21:38 |
jk0 | vishy: and https://code.launchpad.net/~jk0/nova/lp816236/+merge/69565 | 21:42 |
vishy | tr3buchet, I thought we agreed on the deprecated option | 21:43 |
vishy | or you conceded to use it i suppose :) | 21:44 |
vishy | we can remove it later after the deprecated has been around for awhile and people have had a chance to upgrade scripts | 21:45 |
*** lts has quit IRC | 21:46 | |
jk0 | can we get one more set of eyes on https://code.launchpad.net/~jk0/nova/lp816236/+merge/69565 pls? | 21:46 |
openstackjenkins | Project nova-milestone build #10: SUCCESS in 17 sec: http://jenkins.openstack.org/job/nova-milestone/10/ | 21:49 |
openstackjenkins | Tarmac: Fixes typo in attach volume. | 21:49 |
tr3buchet | ok i'm in, i'll add that, and let you know | 21:49 |
*** dspano has quit IRC | 21:52 | |
openstackjenkins | Project nova-milestone build #11: SUCCESS in 16 sec: http://jenkins.openstack.org/job/nova-milestone/11/ | 21:52 |
openstackjenkins | Tarmac: Fix all of the skipped libvirt tests. | 21:52 |
tr3buchet | vishy: let's say that i put the deprecated message in there, with a sleep so they see it, and suggest that they need to start setting the flag, or pass the --bridge option | 21:53 |
tr3buchet | and that it will be removed in milestone d4 | 21:53 |
tr3buchet | giving everyone some weeks to update their "scripts" | 21:54 |
*** mfer has quit IRC | 21:55 | |
vishy | mwhooker: also, http://jenkins.openstack.org/job/nova-pylint-errors/violations/ some of these are real errors | 21:56 |
vishy | sure | 21:56 |
vishy | tr3buchet: sounds good | 21:56 |
openstackjenkins | Project nova-milestone build #12: SUCCESS in 17 sec: http://jenkins.openstack.org/job/nova-milestone/12/ | 21:57 |
openstackjenkins | Tarmac: Fix the skipped tests in vmwareapi and misc spots. The vmware networking stuff is stubbed out, so the tests can be improved there by fixing the fakes. | 21:57 |
*** negronjl has quit IRC | 21:58 | |
*** sriramkr has quit IRC | 22:03 | |
*** ecarlin has joined #openstack-dev | 22:04 | |
*** ecarlin has quit IRC | 22:06 | |
*** negronjl has joined #openstack-dev | 22:12 | |
*** ameade has quit IRC | 22:14 | |
*** tomeff has quit IRC | 22:19 | |
*** scottsanchez has quit IRC | 22:20 | |
*** negronjl has quit IRC | 22:20 | |
*** negronjl has joined #openstack-dev | 22:21 | |
jk0 | vishy: https://code.launchpad.net/~jk0/nova/lp816236/+merge/69565 is Approved now | 22:21 |
*** negronjl has quit IRC | 22:23 | |
*** negronjl has joined #openstack-dev | 22:23 | |
vishy | jk0: thx | 22:24 |
jk0 | vishy: mind having a quick look at https://code.launchpad.net/~jk0/nova/lp805549/+merge/69518 ? two lines :) | 22:27 |
vishy | seems fine although I thought the real fix would be to pull out passwords from the log somehow | 22:27 |
jk0 | the main desc of the bug was that the logs were 644 by default | 22:28 |
jk0 | but I agree, passwords shouldn't be tossed around in verbose output | 22:29 |
jk0 | but that's a whole 'nother blueprint | 22:29 |
vishy | seems fine, I don't really think anyone is ever going to change this, but i suppose it can't hurt | 22:29 |
jk0 | yeah, that one guy might use it, but I doubt many others will | 22:29 |
jk0 | lowers the open bug count if anything | 22:29 |
vishy | does this work if the log doesn't exist yet? | 22:30 |
clayg | hey ahale, I think the us authurl is hard-coded as a constant in woodchipper.auth - it even has a TODO about making it a config option :) | 22:30 |
jk0 | yeah, I tested it using new logs (I always run without --logfile) | 22:30 |
clayg | wrong channel :( | 22:30 |
jk0 | so it was forced to create new logs | 22:30 |
vishy | hmm i'm also unsure about how logrotation will work | 22:31 |
jk0 | do we use logging's rotation or the actual logrotate? | 22:31 |
jk0 | if it's logging's, the nova uid will have access to it, same for logrotate if running as root | 22:32 |
*** gaitan has quit IRC | 22:32 | |
tr3buchet | vishy: both branches are updated with setting to br100 + deprecation warning | 22:33 |
*** negronjl_ has joined #openstack-dev | 22:33 | |
vishy | tr3buchet | 22:33 |
vishy | thx | 22:33 |
tr3buchet | vishy: also set merge prop to milestone | 22:34 |
*** negronjl has quit IRC | 22:34 | |
tr3buchet | separate one | 22:34 |
tr3buchet | i hope i didn't do anything wrong there | 22:34 |
openstackjenkins | Project nova build #1,161: SUCCESS in 4 min 0 sec: http://jenkins.openstack.org/job/nova/1161/ | 22:36 |
openstackjenkins | Tarmac: Fail silently. | 22:36 |
vishy | tr3buchet: now we just need another reviewer | 22:36 |
* jk0 can | 22:37 | |
jk0 | URL? | 22:37 |
tr3buchet | jk0 tear it up! | 22:37 |
openstackjenkins | Project nova-milestone build #13: SUCCESS in 16 sec: http://jenkins.openstack.org/job/nova-milestone/13/ | 22:37 |
openstackjenkins | Tarmac: Fail silently. | 22:37 |
tr3buchet | https://code.launchpad.net/~tr3buchet/nova/lp810563-ms/+merge/69570 | 22:37 |
tr3buchet | that's to milestone | 22:37 |
tr3buchet | https://code.launchpad.net/~tr3buchet/nova/lp810563/+merge/69536 | 22:37 |
tr3buchet | that's the trunk merge | 22:37 |
jk0 | nice changes | 22:40 |
tr3buchet | thx | 22:40 |
*** bcwaldon has quit IRC | 22:41 | |
jk0 | done | 22:42 |
*** kbringard has quit IRC | 22:44 | |
vishy | nice as soon as those patches land we have all known milestone bugs fixed | 22:48 |
tr3buchet | awesome | 22:48 |
vishy | tr3buchet: test fail | 22:49 |
*** mattray has quit IRC | 22:49 | |
tr3buchet | ah shit wtf, i just ran those | 22:50 |
vishy | tr3buchet: hmm looks like an error in the init scripts | 22:50 |
vishy | for nose | 22:50 |
vishy | tests/__init__.py | 22:50 |
*** bsza has quit IRC | 22:50 | |
tr3buchet | i see what it is, retarded.. | 22:54 |
* vishy hands tr3buchet a samurai sword | 22:55 | |
vishy | you know what you need to do... | 22:55 |
jk0 | tr3buchet isn't allowed near sharp/pointy objects | 22:55 |
tr3buchet | jk0: that was a low blow | 22:58 |
* jk0 doesn't make the rules | 22:58 | |
adam_g | trunk failing to install on natty: http://paste.ubuntu.com/653410/ | 23:03 |
vishy | adam_g: it needs python-xattr | 23:03 |
vishy | adam_g: perhaps ttx just added it as a build dep and not a depends? | 23:04 |
adam_g | vishy: ill peak | 23:04 |
vishy | should be a dep of glance though, so you think glance would have pulled it in | 23:04 |
tr3buchet | vishy: jk0: alright looks like tests are passing | 23:05 |
*** mnour has joined #openstack-dev | 23:09 | |
tr3buchet | vishy: and i've got instances with ipv6 addresses so i did it right | 23:10 |
vishy | k | 23:10 |
donald650 | in swift source folder there is a test folder what is the difference between functional and functionalnosetest | 23:11 |
donald650 | i thought all tests can be ran through nose | 23:13 |
vishy | tr3buchet: approved, can you add the change to -ms as well? | 23:13 |
*** jkoelker has quit IRC | 23:14 | |
tr3buchet | vishy: already done | 23:15 |
vishy | tr3buchet: not showing here https://code.launchpad.net/~tr3buchet/nova/lp810563-ms/+merge/69570 | 23:16 |
tr3buchet | vishy: might help if i push | 23:20 |
tr3buchet | i'm scatterbrained today apparently | 23:20 |
vishy | aye, 'tmight | 23:20 |
tr3buchet | and i'm off! | 23:21 |
tr3buchet | i'll check in in a bit and make sure everything is ok | 23:22 |
openstackjenkins | Project keystone build #45: SUCCESS in 1 min 10 sec: http://jenkins.openstack.org/job/keystone/45/ | 23:23 |
openstackjenkins | dolph.mathews: pylint fixes for tenant-group unit tests | 23:23 |
openstackjenkins | Project keystone build #46: SUCCESS in 1 min 4 sec: http://jenkins.openstack.org/job/keystone/46/ | 23:24 |
openstackjenkins | dolph.mathews: pylint fixes for tenant-group unit tests | 23:24 |
openstackjenkins | Project keystone build #47: SUCCESS in 1 min 12 sec: http://jenkins.openstack.org/job/keystone/47/ | 23:25 |
openstackjenkins | yoga80: Changes to make cache time configurable. | 23:25 |
openstackjenkins | Project nova build #1,162: SUCCESS in 4 min 8 sec: http://jenkins.openstack.org/job/nova/1162/ | 23:26 |
openstackjenkins | Tarmac: updates handling of arguments in nova-manage network create. | 23:26 |
openstackjenkins | updates a few of the arguments to nova-manage and related help. | 23:26 |
openstackjenkins | updates nova-manage to raise proper exceptions. | 23:26 |
openstackjenkins | updates network manager create_networks to handle ipv6 more correctly and efficiently. | 23:26 |
openstackjenkins | flat_network_bridge FLAG now defaults to None. | 23:26 |
openstackjenkins | Project keystone build #48: SUCCESS in 1 min 15 sec: http://jenkins.openstack.org/job/keystone/48/ | 23:27 |
openstackjenkins | yoga80: Flow diagram to support keystone service registration. | 23:27 |
adam_g | vishy: python-xaddr added install depend for python-nova, https://code.launchpad.net/~gandelman-a/nova/ubuntu/+merge/69571 | 23:28 |
openstackjenkins | Project keystone build #49: SUCCESS in 1 min 4 sec: http://jenkins.openstack.org/job/keystone/49/ | 23:28 |
openstackjenkins | dolph.mathews: pylint fixes for server unit tests | 23:28 |
openstackjenkins | Project keystone build #50: SUCCESS in 1 min 7 sec: http://jenkins.openstack.org/job/keystone/50/ | 23:29 |
openstackjenkins | yoga80: Changes suggested by Ziad.Adding validateToken operation. | 23:29 |
vishy | adam_g: saw that. I was wondering if it should be a dependency of glance instead | 23:29 |
openstackjenkins | Project keystone build #51: SUCCESS in 1 min 13 sec: http://jenkins.openstack.org/job/keystone/51/ | 23:30 |
openstackjenkins | dolph.mathews: Removing attribute duplicated from superclass; causes an issue in py 2.7 | 23:30 |
openstackjenkins | Project keystone build #52: SUCCESS in 1 min 3 sec: http://jenkins.openstack.org/job/keystone/52/ | 23:31 |
openstackjenkins | dolph.mathews: pylint fixes for role api | 23:31 |
*** donald650 has quit IRC | 23:31 | |
openstackjenkins | Project keystone build #53: SUCCESS in 1 min 5 sec: http://jenkins.openstack.org/job/keystone/53/ | 23:32 |
openstackjenkins | dolph.mathews: Restored identity.wadl w/ system test | 23:32 |
adam_g | vishy: glance or python-glance? glance itself doesn't get pulled in when installing nova-common | 23:32 |
Daviey | adam_g: that is a bug | 23:35 |
Daviey | Hmm. Does glance *need* to run on the same host as nova? | 23:35 |
adam_g | Daviey: what is? | 23:36 |
Daviey | well nova fails without access to glance, right? | 23:36 |
Daviey | (not sure the *current* oneiric archive package does.. as that was a new coupling) | 23:37 |
vishy | adam_g: ah it depends on python-glance? well it should be a dep of python-glance then | 23:38 |
adam_g | AFAIK, local image service is deprecated in favor of glance running locally with local file backing? | 23:38 |
vishy | correct | 23:38 |
vishy | you have to run glance | 23:38 |
Daviey | vishy: Does nova need glance itself to be running on the same host as nova? | 23:38 |
vishy | so maybe it should just depend on the full glance | 23:38 |
* Daviey hasn't tried splitting them | 23:38 | |
vishy | Daviey: ah good point no | 23:38 |
vishy | Daviey: it just needs to be running somewhere | 23:38 |
Daviey | adam_g: in which case, it should be a Recommends. | 23:38 |
vishy | so sounds like python-glance needs the dependency | 23:39 |
Daviey | Does nova need python-glance? | 23:39 |
Daviey | I should just try this myself, but i'm being lazy. | 23:40 |
*** negronjl_ is now known as negronjl | 23:40 | |
vishy | Daviey: yes nova-needs python-glance | 23:40 |
vishy | because it uses glance-client | 23:40 |
Daviey | ah! thought so. | 23:41 |
Daviey | So.. python-glance should be a Depends of nova, and glance should be a Recommends (perhaps even a Suggests) of python-glance. | 23:41 |
vishy | so i think rather than nova > python-xattr we should have nova > python-glance > python-xattr | 23:41 |
adam_g | vishy: aggree | 23:42 |
openstackjenkins | Project nova-milestone build #14: SUCCESS in 18 sec: http://jenkins.openstack.org/job/nova-milestone/14/ | 23:42 |
openstackjenkins | Tarmac: updates handling of arguments in nova-manage network create. | 23:42 |
openstackjenkins | updates a few of the arguments to nova-manage and related help. | 23:42 |
openstackjenkins | updates nova-manage to raise proper exceptions. | 23:42 |
openstackjenkins | updates network manager create_networks to handle ipv6 more correctly and efficiently. | 23:42 |
openstackjenkins | flat_network_bridge FLAG now defaults to None. | 23:42 |
openstackjenkins | bug810563 | 23:42 |
openstackjenkins | changes mirrored in a merge prop to trunk | 23:42 |
Daviey | (noting that the default behavior in Ubuntu is to install Recommens by default, which is not the case in Debian) | 23:42 |
vishy | so looks like python-glance is already a dep of python-nova | 23:45 |
adam_g | yup | 23:45 |
vishy | so maybe we just need python-xattr as a dep of python-glance and we are done? | 23:45 |
vishy | yes it isn't there: http://bazaar.launchpad.net/~openstack-ubuntu-packagers/glance/ubuntu/view/head:/debian/control | 23:47 |
adam_g | yeah, adding now. i think we can wait on recommending glance itself till things have settled? | 23:47 |
vishy | adam_g: awesome thanks | 23:47 |
mwhooker | Hi, I'm trying to resolve some errors in nova/virt/vmwareapi/vim.py, but as I go through the code I wonder how it's even possible that this runs. Is it possibly being used by anything? | 23:47 |
vishy | mwhooker: why don't you think it runs? | 23:49 |
medberry | adam_g, are you working on a bug (w/r/t python-xattr)? Is this a new behavior? | 23:49 |
vishy | medberry: yes and yes | 23:49 |
mwhooker | it makes a call to self.RerieveServiceContent at the bottom of __init__, which isn't defined anywhere | 23:50 |
medberry | tx. | 23:50 |
mwhooker | and object.__getattr__ isn't a method | 23:51 |
mwhooker | which is the first statement in __getattr__ | 23:51 |
vishy | yes that does look broken, seems like it should be self.client.RetrieveServiceContent or something. May not be used | 23:51 |
mwhooker | vishy, or perhaps it's doing something very tricky with getattr | 23:52 |
vishy | mwhooker: I might try starting somewhere else since you will have trouble testing your fixes there | 23:52 |
mwhooker | good idea | 23:52 |
mwhooker | thanks for looking at that | 23:52 |
vishy | mwhooker, unless you happen to have installed vmware with openstack | 23:52 |
*** scottsanchez has joined #openstack-dev | 23:53 | |
vishy | mwhooker: another way to find some stuff that is broken: apt-get install pyflakes and something like: pyflakes nova | grep -v "undefined name '_'" | grep -v unable | 23:58 |
vishy | mwhooker: for example, just uncovered this bug: nova/api/openstack/create_instance_helper.py:191: undefined name 'ServerCreateRequestXMLDeserializer' | 23:58 |
mwhooker | good idea, though I see pylint captures that, too http://jenkins.openstack.org/job/nova-pylint-errors/violations/file/nova/api/openstack/create_instance_helper.py/? | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!