Thursday, 2011-08-18

vladimir3pFolks, have anoybody else seen these issues with nova-dhcpbridge? In previous versions there was a problem with hostname. Now it is removed, but it still crashes on unknown var "interface"00:04
Davieymwhooker: bug 827386 seems pretty bitesize, and is kinda importiant. Not currently possible to see who uploaded an image, kinda bad on a shared cloud - as there is no way of gaining confidence of the person that uploaded it, and made it public.00:07
uvirtbotLaunchpad bug 827386 in nova "DescribeImages does not fill imageOwnerId" [Undecided,New] https://launchpad.net/bugs/82738600:07
mwhookerDaviey: Thanks. I'll check it out00:07
Davieygroovy00:07
vishyhuh? imageOwnerId is not there?00:10
vishyhow did you upload the image daviey?00:10
vishyDaviey: ^^00:10
*** ohnoimdead has quit IRC00:14
Davieyvishy: euca-upload-bundle i think00:18
Davieyvishy: ah no, i used the wrapper, uec-publish-tarball00:20
vishyok00:21
vishywe were returrning owner id so maybe it broke recently00:21
Davieyvishy: i'm lazy and use this, http://pb.daviey.com/hkDi/00:21
Davieymwhooker: If you are not able to reproduce it, i'll provide an xml dump of what i am getting back.00:24
Davieybut right now, BED.. nn folks.00:24
mwhookerthanks00:24
*** anotherjesse has quit IRC00:35
*** anotherjesse has joined #openstack-dev00:42
*** rods has quit IRC00:47
*** jakedahn has quit IRC00:50
*** anotherjesse has quit IRC00:53
*** mattray has joined #openstack-dev00:54
*** mattray has quit IRC00:54
*** mfer has quit IRC01:08
*** tushar_ has quit IRC01:13
*** lorin1 has joined #openstack-dev01:21
*** lorin1 has left #openstack-dev01:21
*** hisaharu has quit IRC01:23
*** vladimir3p has quit IRC01:25
*** jakedahn has joined #openstack-dev01:33
*** dragondm has quit IRC01:39
*** tsuzuki_ has joined #openstack-dev01:58
*** mfer has joined #openstack-dev02:16
*** mfer has quit IRC02:16
*** mnaser has quit IRC02:36
*** deshantm is now known as deshantm_away03:09
*** anotherjesse has joined #openstack-dev03:18
*** bengrue has quit IRC03:22
*** martine has joined #openstack-dev03:39
*** RobertLaptop has quit IRC05:14
*** RobertLaptop has joined #openstack-dev05:17
*** martine has quit IRC05:22
*** RobertLaptop has quit IRC05:27
*** RobertLaptop has joined #openstack-dev05:30
*** anotherjesse has quit IRC05:46
*** reidrac has joined #openstack-dev07:00
*** phil121 has quit IRC08:58
*** mnour has joined #openstack-dev09:07
*** tsuzuki_ has quit IRC09:19
*** rods has joined #openstack-dev09:46
*** YorikSar has joined #openstack-dev09:50
openstackjenkinsProject nova build #1,261: SUCCESS in 3 min 36 sec: https://jenkins.openstack.org/job/nova/1261/10:05
openstackjenkinsTarmac: Adds the enabled status of a host when XenServer reports its host's capabilities. This allows the scheduler to ignore hosts whose enabled is False when considering where to place a new instance.10:05
*** mancdaz has quit IRC11:05
*** markvoelker has quit IRC11:40
*** mnour has quit IRC11:45
*** markvoelker has joined #openstack-dev11:48
*** lorin1 has joined #openstack-dev11:51
*** lorin1 has left #openstack-dev11:53
*** mancdaz has joined #openstack-dev11:53
*** mancdaz has quit IRC11:54
*** mancdaz has joined #openstack-dev11:55
*** mfer has joined #openstack-dev12:03
*** lts has joined #openstack-dev12:09
*** mnour has joined #openstack-dev12:10
*** chipc has joined #openstack-dev12:19
*** chipc has quit IRC12:24
*** bsza has joined #openstack-dev12:27
*** martine has joined #openstack-dev12:29
*** chipc has joined #openstack-dev12:34
*** zul has joined #openstack-dev12:44
*** deshantm_away is now known as deshantm12:46
*** nmistry has joined #openstack-dev12:51
*** jaypipes has quit IRC13:00
*** dprince has joined #openstack-dev13:03
*** zns has joined #openstack-dev13:08
znscomstud: Could you help us with an eventlet issue? We've had this reported (http://pastie.org/2387012) and also been told you're the eventlet guru. Any quick pointers?13:16
*** joesavak has joined #openstack-dev13:21
openstackjenkinsProject nova build #1,262: SUCCESS in 3 min 33 sec: https://jenkins.openstack.org/job/nova/1262/13:25
openstackjenkinsTarmac: Fixed several logical errors in the scheduling process. Renamed the 'ZoneAwareScheduler' to 'AbstractScheduler', since the zone-specific designation is no longer relevant. Created a BaseScheduler class that has basic filter_hosts() and weigh_hosts() capabilities. Moved the filters out of one large file and into a 'filters' subdirectory of nova/scheduler.13:25
openstackjenkinsSince I had to do a lot of editing of the files to change the naming and references, I cleaned up much of the inconsistent formatting. Normally formatting changes don't belong in a code change merge prop, but as I was making large-scale changes to the files, it was simplest to do it all at once.13:25
*** amccabe has joined #openstack-dev13:30
*** kbringard has joined #openstack-dev13:37
*** nmistry has quit IRC13:37
*** joesavak has quit IRC13:52
*** nmistry has joined #openstack-dev14:03
*** joesavak has joined #openstack-dev14:05
*** jsavak has joined #openstack-dev14:07
*** joesavak has quit IRC14:10
*** zul has quit IRC14:12
*** zul has joined #openstack-dev14:13
*** timr has joined #openstack-dev14:18
*** mattray has joined #openstack-dev14:18
*** anp has joined #openstack-dev14:24
*** mnaser has joined #openstack-dev14:33
*** bcwaldon has joined #openstack-dev14:43
*** rnirmal has joined #openstack-dev14:49
*** zul has quit IRC14:49
sorenzns: Are you throwing a lot of stuff at it? I.e. is it likely to run into EMFILE?14:53
sorenzns: ...or does it rather smell like it's not closing file handles properly?14:54
kbringardhey soren: have you seen this before?14:55
kbringardhttp://paste.openstack.org/show/2201/14:55
kbringardit happens when I launch a VM in a security group that has a different security group set as a --source-group14:55
kbringardso like, the web group has -o default14:55
kbringardthen I launch a VM in web, and it craps itself14:55
kbringardI didn't see a bug, but I wanted to ask if you'd seen that before before I start digging into it14:56
*** dragondm has joined #openstack-dev15:00
*** dragondm has quit IRC15:01
*** heckj has joined #openstack-dev15:01
*** mnour has quit IRC15:02
*** _cerberus_ has joined #openstack-dev15:05
*** Gordonz has quit IRC15:05
*** ChanServ sets mode: +v _cerberus_15:06
*** reidrac has quit IRC15:06
*** alekibango has quit IRC15:11
sorenkbringard: This might be what vishy talked about a couple of days ago.15:11
*** dragondm has joined #openstack-dev15:11
sorenkbringard: Which version of euca2ools are you using?15:12
*** Gordonz has joined #openstack-dev15:12
kbringardI've tried it with 1.2 and 1.315:14
kbringardthe 1.2 has the source-port = None default thing, right?15:14
kbringardit doesn't seem to be doing it anymore on the newest trunk, but it seems that the source groups have gone back to not working (at least using euca2ools 1.2)15:16
*** yogirackspace has joined #openstack-dev15:18
sorenERr... I didn't even realise the source groups stuff had been merged.15:18
sorenHeh :)15:19
sorenI guess it has.15:19
kbringardI didn't think it had either… I just happened to notice that when I tried to launch VMs in a specific group, they all failed :-)15:19
kbringardah, yea, same thing when I add the rule with euca2ools 1.315:20
sorenOk.15:21
sorenCan you file a bug about it? I'm about to call it a day.15:21
kbringardsure thing, I'm in meetings all day but I'll file the bug now and try to find some time to look at it myself as well15:23
kbringardthanks, soren. Have a good night, sir!15:23
*** kbringard1 has joined #openstack-dev15:39
*** kbringard has quit IRC15:39
*** kbringard1 has quit IRC15:40
*** vladimir3p has joined #openstack-dev15:45
*** jsavak has quit IRC15:46
*** heckj has quit IRC15:49
*** heckj has joined #openstack-dev15:49
*** heckj has quit IRC15:50
*** jsavak has joined #openstack-dev15:53
*** mattfarina has joined #openstack-dev15:56
*** mfer has quit IRC15:56
*** mattfarina is now known as mfer15:56
*** mnaser has quit IRC16:05
*** mfer has quit IRC16:06
*** ohnoimdead has joined #openstack-dev16:10
znssoren: frankly, not sure how to replicate or why it is happening. The pastie is from anotherjesse. But it does look like an EMFILE ("[Errno 24] Too many open files"...16:24
znsanotherjesse: how can we replicate this? http://pastie.org/238701216:25
*** jakedahn has quit IRC16:25
*** mnaser has joined #openstack-dev16:27
znssoren: is there cleanup we should be doing somewhere? Or a flag we can turn on to debug deeper?16:28
znsWhat we do is pretty basic:16:28
znsserver = wsgi.Server()16:28
zns        server.start(app, int(conf['service_port']), conf['service_host'])16:28
anpwhen I install Openstack-dashboard from repo it installs openstack-compute-2.0a1-1.noarch.rpm16:32
anpbut I already have openstack-nova-compute-2011.3-0.20110727.1143.10.el6.noarch installed16:33
anpwill this conflict?16:33
anpHi16:43
anpI installed openstack-dashboard16:43
anpI am able to relog16:43
anpbut I get this error after log in16:43
anpError16:43
anpUnable to get service info: This server could not verify that you are authorized to access the document you requested. Either you supplied the wrong credentials (e.g., bad password), or your browser does not understand how to supply the credentials required.16:43
*** heckj has joined #openstack-dev16:46
anpAlso I dont see any options to launch a new instance from there16:49
anpbut I can see that services are up16:50
anpnever mind I found the option by logging in a normal user16:52
anpfor all the users it shows:16:53
anpErrorUnable to get usage info: This server could not verify that you are authorized to access the document you requested. Either you supplied the wrong credentials (e.g., bad password), or your browser does not understand how to supply the credentials required.16:53
anpeven if I am logged in succesfully16:53
heckjttx: ping?16:56
*** nmistry has quit IRC16:56
*** bcwaldon_ has joined #openstack-dev17:01
*** bcwaldon has quit IRC17:03
*** alekibango has joined #openstack-dev17:03
*** zns has quit IRC17:04
*** mfer has joined #openstack-dev17:05
*** jakedahn has joined #openstack-dev17:07
*** jsavak has quit IRC17:10
*** zns has joined #openstack-dev17:10
*** jakedahn has quit IRC17:13
*** dprince has quit IRC17:14
*** bcwaldon_ has quit IRC17:17
*** joesavak has joined #openstack-dev17:19
*** mattray1 has joined #openstack-dev17:20
*** mattray has quit IRC17:22
vishysoren, kbringard: haven't seen that one17:23
vishyblamar_: ping17:24
blamar_vishy: hey hey17:24
*** zul has joined #openstack-dev17:24
vishyblamar_: wanted to chat with you about some of the stuff in your review17:24
openstackjenkinsProject nova build #1,263: SUCCESS in 3 min 32 sec: https://jenkins.openstack.org/job/nova/1263/17:25
openstackjenkinsTarmac: Fix to return 413 for over limit exceptions with instances, metadata and personality17:25
vishyblamar_: gettext thing is clear, I guess it just wasn't removed from the other binaries, I'll pull it from all of them17:25
vishyblamar_: interested in how we would use entrypoints for the binary loading17:25
vishyblamar_: in the mean time, should I switch it to explicitly pass in all of the parameters to Service() ?17:26
*** jsavak has joined #openstack-dev17:27
blamar_vishy: incremental changes here are fine/better IMO if you want17:27
vishyblamar_: also, what exactly is the problem you were trying to solve with multiprocessing?  Efficient use of cores?  Can't we just start multiple copies of the api if we want to use the cores?17:27
blamar_vishy: ec2 and osapi are two services, why are they sharing a file when every other service gets to run by itself?17:28
vishyblamar_: I get your point there, no problem separating them.  I think it makes sense to include a combined one as well for compatibility with existing scripts17:29
*** zul has quit IRC17:29
vishyblamar_: I was just trying to understand if there is some hidden value of multiprocessing that I'm missing17:30
blamar_vishy: sure, so thats the one point, the other point is kind of processor-related, it's a lot easier to debug an eventlet API for performance if it's running in it's own process17:30
*** joesavak has quit IRC17:31
vishyblamar_: so that is solved by having the separate binary files?17:32
blamar_vishy: yeah17:32
vishyblamar_: ok I will add the separate files then17:33
vishyI spent about a week early in the process debugging an issue with twisted + multiprocessing17:33
blamar_vishy: ugh, twisted, sorry you had to do that17:34
vishyand it seems like odd issues like this one keep coming up so I prefer not to use event loops and multiprocessing together17:34
blamar_vishy: I swear I've never had any issues in the past using gevent + multiprocessing to solve problems related to cpu-utilization so I'll admit to jumping to that solution a little too quickly17:35
vishyblamar_: well it looks like it is actually a twisted bug17:35
vishys/twisted/eventlet17:35
vishyblamar_: wrt signal handling17:35
blamar_vishy: and I'm looking into how we might be able to use entry_points in the future, not a good answer for the "how" on that one right now, but I know it's possible17:35
*** troytoman-away is now known as troytoman17:35
vishyblamar_: my entry point experience is kind of limited17:36
vishyblamar_: so there is no need to have a custome signal handler at this point?17:36
blamar_vishy: you should be able to trash all signal handling17:37
vishyk17:40
*** zul has joined #openstack-dev17:43
*** zul has quit IRC17:43
*** zul has joined #openstack-dev17:44
*** yogirackspace has quit IRC17:48
*** nickon has joined #openstack-dev17:54
*** jakedahn has joined #openstack-dev17:56
creihtentry points++17:57
creiht:)17:57
*** zns has quit IRC18:01
*** yogirackspace has joined #openstack-dev18:02
*** jsavak has quit IRC18:02
*** zns has joined #openstack-dev18:02
*** jtanner has joined #openstack-dev18:07
*** joesavak has joined #openstack-dev18:07
*** zul has quit IRC18:15
*** rods has quit IRC18:17
*** bcwaldon has joined #openstack-dev18:20
vishyblamar_: check the new version of the code I just pushed?18:22
blamar_vishy: checking18:22
vishyblamar_: I tried to address your comments and clean up things further18:22
vishyvladimir3p: ping18:23
vladimir3pvishy: pong18:23
vishyso my partner here and I are doing volumes all day today18:23
vishyhow should we split this up18:23
vladimir3pvishy: are you doing actual code work / planning ?18:24
vishyactual code18:25
vladimir3pvishy: great... let me know if there are any potions of the code that you would like me to take. I'm prety much available today18:25
blamar_vishy: think the nova-network Server line might be wrong (line 327)18:27
vladimir3pvishy: also, we can probably do VolumeTypes + ExtraSpecs now and leave scheduler for the future ...18:27
vishyblamar_ nice one18:27
blamar_also nova-scheduler18:28
vishyoops18:28
vishydidn't push the last version18:28
vishyrev 1457 is coming up give it a sec18:28
vishy:)18:28
vishy* 145618:29
*** zns has quit IRC18:29
*** mattray1 is now known as mattray18:36
*** mattray has joined #openstack-dev18:36
ironcamelvishy: do you know about the "attrs" field of a servers response in the api?18:40
ironcamela stack i built via nova.sh is adding attrs to the servers response18:40
ironcamelhttp://paste.openstack.org/show/2204/18:49
vishyironcamel: not sure18:51
ironcamelvishy: do you see what i'm talking about?18:51
vishyironcamel: but i don't usually dump the response to see what is in there18:51
vishyyeah18:51
ironcameli'm wondering if that is some magic cloudbuilders added18:51
ironcamelbecause that is not in the spec18:51
ironcameland another stack i deployed using opentstack_vpc does not have the attrs stuff18:51
vishyironcamel: could be an addition via openstackx18:52
ironcameli'm not using bindings18:52
ironcameli'm basically using straight curl18:52
Davieyironcamel: Hey, did you get any further with kombu?18:52
ironcameli have not18:52
Davieyironcamel: ok.18:52
ironcamelDaviey: how about you?18:52
vishyironcamel: yes i think it is to support one of the extensions18:53
Davieyironcamel: nope.18:53
ironcamelvishy: openstackx is injecting that attrs stuff?18:53
Davieyironcamel: not touched it since i last spoke about it18:53
vishyironcamel: yeah, there is a bunch of stuff missing in the response to make dashboard work, we are trying to get all of the stuff added as extensions18:54
vishybut first pass was just throw the needed data into a attrs as a workaround18:54
Davieyis python-novaclient not using jenkins?  Or at least testsuite on merge?18:56
ironcamelvishy: when you add stuff via extensions, aren't they supposed to be in their own namespace?19:01
vishyextra attributes aren't19:02
ironcamelyou guys just plopped that stuff right into the top level. so dirty19:02
vishyextra endpoints19:02
vishy:)19:02
* ironcamel shakes his head in disapproval19:02
vishyironcamel: yes, quite dirty.  All of this stuff needs to be added to the spec though19:02
vishythat is why we didn't propose it as-is :)19:03
*** dprince has joined #openstack-dev19:04
*** bengrue has joined #openstack-dev19:06
vladimir3pvishy: pls let me know if I can help with anything re volume types/extra-data19:07
vishyvladimir3p: we're working on cleanup first19:12
vishyvladimir3p: so if you want to start on the extra data types stuff19:12
vishyit will take us at least a couple hours to get to that part19:13
*** dolphm has joined #openstack-dev19:13
vladimir3pvishy: ok. will take extra-specs19:14
vladimir3pvishy: could you prepare DB schemas for types & extra-specs?19:14
vishyvladimir3p: just copy from instance-types19:16
vishythere are currently no extra fields19:16
vishyso it is just id and name at the moment19:16
mtaylorDaviey: no, python-novaclient is not being managed currently - it's on my list to bring up with someone at some point :)19:21
vladimir3pok, will add the extraspecs a-la instance extraspecs19:21
Davieymtaylor: might be a good idea, current trunk seems to fail the test suite.19:22
*** alekibango has quit IRC19:23
mtaylorDaviey: oh, it's definitely a good idea. :)19:25
mtaylordolphm: I get this: http://paste.openstack.org/show/2205/ everytime I run the keystone tests locally ... any ideas?19:25
dolphmmtaylor: it looks like you're running without --with-coverage?19:26
mtaylordolphm: yes19:26
dolphmmtaylor: then you probably have all kinds of failures...19:27
mtaylorreally?19:27
mtaylorwe don't run --with-coverage on the main jenkins job for merges19:27
dolphmmtaylor: running tests *without* --with-coverage causes it to fail-fast... so you only see one arbitrary failure19:27
mtaylor./run_tests.sh -N is all we do there19:27
mtaylorREALLY - that's rather odd - how are merges happening then19:27
dolphmmtaylor: that's fine19:27
dolphm./run_tests.sh -N # aborts on first failure19:28
dolphm./run_tests.sh -N --with-coverage # runs through all failures19:28
mtayloroh. gotcha19:29
dolphmsort of makes sense to me, but i didn't come up with it...19:29
mtaylorsorry, I thought you were saying something different :)19:29
dolphmi guess it's convenient considering stdout is suppressed (grr)19:29
dolphmanyway, you probably have bigger problems than that unit test19:29
mtaylorok. I'll turn on coverage and see what happens19:29
dolphmpaste your git diff if you want ;)19:33
mtaylordolphm: oh, this is me working on packaging, so now that I can see more things failing - I think it has to do with versions of dependencies19:33
* Daviey perks up19:38
dolphmmtaylor: that would make sense19:39
dolphmmtaylor: anything in keystone.sql.log?19:39
mtaylordolphm: TypeError: 'ItemNotFoundFault' object is not callable19:44
dolphmwhere's that being thrown?19:46
dolphmmtaylor:  ^19:46
vladimir3pvishy: ping19:47
vishyvladimir3p: sup?19:47
vladimir3pvishy: prior to starting on volume types could you pls let me know. it is quite problematic to test extra-specs alone, so I added some volume types staff as well19:47
vladimir3pvishy: probably you could take my tree when you will start your part19:48
vladimir3p(or we can merge them...)19:48
mtaylordolphm: wait - new error (I can fresh so I could track where it was starting to go nuts)19:48
mtaylordolphm:     raise TemplateError('Template %s not found.' % repr(name))19:48
mtaylorTypeError: __init__() takes at least 3 arguments (2 given)19:48
vishyvladimir3p: you can do types as well at least in the db19:48
mtaylordolphm: keystone/common/template.py line 9519:48
vladimir3pvishy: yep, ok19:48
dolphmmtaylor:  that's fixable... looking into it19:49
dolphmmtaylor: try changing it to...19:55
dolphmraise TemplateError('Template %s not found.' % repr(name), sys.exc_info())19:55
dolphmmtaylor: just a wild guess... i get errors just trying to import paste.util.template, so i can't even poke at TemplateError19:55
mtaylordolphm: aaaah. I may have found a thing ...20:03
sorenI'm staring at the XenAPI driver. I'm having trouble grokking the rescue code.20:04
sorenI can't for the life of me figure out where the rescue disk image gets attached.20:05
sorenI mean... Of course it gets attached as the first disk, but which part of the code makes that happen?20:06
sorenThe fact that spawn_rescue just calls spawn with identical arguments really weirds me out.20:06
sorenI vaguely remember commenting on a merge proposal about that, but I forget what the rationale was :(20:07
*** mnour has joined #openstack-dev20:07
mtaylordolphm: heh. I'm uncovering some issues because I'm running 2.6 on this maverick box - and we're doing testing on natty with 2.7 right now.20:11
* mtaylor really needs to get both up and going...20:11
*** jsavak has joined #openstack-dev20:12
mtaylordolphm: keystone/test/functional/test_ext_rs_key_auth.py line 24: SyntaxError: expected path separator ([)20:13
sorenjk0: You wrote some of that code. Can you lend me a hand, perhaps?20:14
jk0sure20:14
soren\o/20:14
mtaylordolphm: http://paste.openstack.org/show/2208/ fwiw20:15
mtaylordolphm: we're getting MUCH further now though20:15
jk0so bascially a rescue instance has the same attributes as the original instance. it just attached the original instance's disk to it20:15
sorenjk0: Right.20:15
sorenjk0: What I'm wondering is where it gets the rescue image.20:15
*** joesavak has quit IRC20:16
jk0soren: it uses the image the original uses20:17
jk0so if the original was ubuntu, the rescue image would use the same ubuntu base image20:17
sorenjk0: Oh, so not a special rescue image?20:17
jk0doing it this was got us around having to prep and maintain a specific rescue image for each distro20:17
jk0nope20:17
openstackgerritMonty Taylor proposed a change to openstack/keystone: Cleaned up come issues with python2.6.  https://review.openstack.org/27020:17
jk0s/was/way/20:18
sorenjk0: Ok, gotcha. That explains why I couldn't find this magic rescue image :)20:18
jk0haha, yeah that would do it :)20:18
sorenjk0: Great, thanks a lot!20:18
jk0no problem20:18
openstackgerritYogeshwar Srikrishnan proposed a change to openstack/keystone: Changes to allow test to work on python 2.6.*.  https://review.openstack.org/27120:23
mwhookersmoser: hi. I'm working on bug # 827386 https://bugs.launchpad.net/nova/+bug/827386. what image service were you using?20:23
uvirtbotLaunchpad bug 827386 in nova "DescribeImages does not fill imageOwnerId" [Undecided,New]20:23
mtaylordolphm: woot! I found all the issues... updated patch coming... and haha, yogi seemed to have been working on the same thing :)20:23
mwhookersmoser: Looking at the code, glance + euca2ools w/ boto should display the owner id20:23
Davieymwhooker: He was using nova-objectstore20:23
mwhookerthanks Daviey20:24
Davieymwhooker: But you don't think it's an issue with glance?20:24
*** jsavak has quit IRC20:24
mwhookerI've only done a quick scan of the code, but I see the owner_id from the database being sent20:24
Davieymwhooker: I don't have complete confidence about that string being empty.20:26
mwhookergood point20:26
mwhookerit would be helpful if I could see some http sessions20:26
vladimir3pvishy: First cut on volume types & extra-data - lp:~vladimir.p/nova/volume_type_extradata (primarely DB schemas, DB APIs, migration)20:27
sorenDaviey: Even if you use nova-objectstore, the image lands in glance.20:27
vishyvladimir3p: coolness20:28
vishychecking20:28
sorenDaviey: nova-objectstore is where you push it to (if you're using cloud-utils). Once you call RegisterImage, it goes and grabs the image, deciphers it, stitches it together and pushes it to glance.20:28
openstackgerritA change was merged to openstack/openstack-ci-puppet: Cleanup.  https://review.openstack.org/26720:29
mwhookerhmm, actually api/ec2/cloud.py._format_image seems to be trying to get 'owner_id', but in glance the field is simply "owner"20:29
vishyvladimir3p: looking good20:30
mwhookerlet me see if I can test that20:30
vishyDaviey, mwhooker: it was using a custom owner_id set in properties, because glance didn't have an owner field until recently20:31
vladimir3pvishy: I will add tests for extra-specs & types ... (at least on DB level)20:31
vladimir3pvishy: next steps might be APIs for volume types, Openstack APIs for everything, integration with volume code20:32
vishyagreed20:34
mwhookervishy: grepping for "owner_id" in nova/glance turns up little. can you point me in the right direction?20:34
*** YorikSar has quit IRC20:35
*** dprince has quit IRC20:35
vishymwhooker: what do you need pointed at?20:35
*** jsavak has joined #openstack-dev20:35
*** YorikSar has joined #openstack-dev20:35
mwhookerwhere in the code the "customer owner_id" is set in properties20:36
*** zns has joined #openstack-dev20:36
vishymwhooker: it is set in properties in nova/image/s3.py i believe20:36
vishyand also in nova-manage register commands20:36
vishys/register/image register/20:37
mwhookerlooks like owner is being set as "project_id", not "owner_id"20:40
openstackgerritJames E. Blair proposed a change to openstack/openstack-ci: Updates to blueprint/topics.  https://review.openstack.org/27220:42
*** zns has quit IRC20:42
Davieymwhooker: that seems most likely based on the output, oneiric-20110813212842/oneiric-server-cloudimg-amd64.img20:43
Davieymwhooker: I uploaded the image using, http://pb.daviey.com/hkDi/ . so the project_id == bucket?20:44
Davieythe project_id certainly was not the same value as bucket btw.20:44
mwhookerinteresting. i have to research before I know what that means20:45
Davieyurgh, i might be being daft20:45
mwhookerso it seems like perhaps 2 things need to change to resolve this: 1. make sure owner is set to owner_id. 2. check for 'owner' from glance, and not 'owner_id'20:46
*** jsavak has quit IRC20:46
Davieymwhooker: I am being daft, it looks like it should have been directly following manifest.xml.. in human form.20:47
mwhookerinteresting20:47
openstackgerritA change was merged to openstack/openstack-ci: Updates to blueprint/topics.  https://review.openstack.org/27220:49
openstackjenkinsProject nova build #1,264: SUCCESS in 3 min 50 sec: https://jenkins.openstack.org/job/nova/1264/20:50
openstackjenkinsTarmac: Remove instances.admin_pass column.20:50
openstackgerritJames E. Blair proposed a change to openstack/openstack-ci-puppet: Update rfc.sh.  https://review.openstack.org/27320:51
mwhookerafk for lunch. See anyone at tonight's openstack meet-up in SF?20:51
*** jsavak has joined #openstack-dev20:52
*** joesavak has joined #openstack-dev20:56
*** jsavak has quit IRC20:56
*** mfer has quit IRC20:59
*** dolphm_ has joined #openstack-dev21:02
*** yogirackspace has left #openstack-dev21:04
openstackjenkinsProject nova build #1,265: SUCCESS in 3 min 33 sec: https://jenkins.openstack.org/job/nova/1265/21:20
openstackjenkinsTarmac: Cleanup the '_base' directory in libvirt tests.21:20
*** Tushar has joined #openstack-dev21:22
*** troytoman is now known as troytoman-away21:22
sorenvishy: The doc for a number of the methods in nova.virt.fake say that they're meant to return "a task". Any idea what sort of task that is?21:24
vishyhuh?21:24
*** jsavak has joined #openstack-dev21:24
vishyhow did that get in21:25
vishysoren: ah, ewan added those docstrings21:25
*** lts has quit IRC21:25
vishyprobably was never implemented21:25
sorenDidn't think so. All those things are synchronous, are they not?21:26
*** joesavak has quit IRC21:27
sorenvishy: I seem to remember some discussion about putting the docs in the fake rather then the abstract driver class. I don't recall why we ended up putting them in the fake. It seems odd.21:29
*** chipc has quit IRC21:30
vishysoren: there wasn't initially an abstract driver21:30
vishyjustinsb added it, used to be just a fake21:31
vishyso fake was the deinition21:31
sorenvishy: That would explain it :)21:31
vishy* definition21:31
*** martine has quit IRC21:31
sorenThe discussion must have been whether to add an abstract class just for the docs.21:31
sorenMeh.21:31
* soren transplants the docs from the fake into the abstract21:32
jk0bcwaldon: responded21:33
bcwaldonjk0: RE: testing - I was just thinking you could check that a 'rescue' action mapped to a compute api call21:34
bcwaldonjk0: and that error handling was working21:34
jk0I think that's what the extensions suite looks at now21:35
bcwaldonjk0: maybe I just missed that point, let me look real quick21:35
jk0I looked around, there are no unit tests for specific extensions21:35
jk0other than the generic extension test suite21:35
bcwaldonjk0: nova/tests/api/openstack/contrib/21:36
jk0ahah21:36
jk0woops21:36
bcwaldonjk0: yep, I'm 100% behind you once I see a few tests ;)21:36
jk0nice catch, I'm not sure how I didn't see that21:37
bcwaldonthey're hidden back there, no biggie21:37
bcwaldonTrust me, I always look for tests. We wouldn't have this many extensions with no test coverage21:37
jk0ok, hang on21:37
zykes-is there any hints on when resource accounting will be in ?21:39
vladimir3pvishy: seems like I need your help...21:40
vladimir3pvishy: unittests for volume types are failing on dictionary comparisons (strings vs unicode strings). What I don't understand is how test_instance_types_extra_specs performing exactly same thing and succeeding there ...21:42
*** amccabe has quit IRC21:42
vishyinteresting is it in your current code push?21:42
vladimir3ppushed it a min ago ...21:43
vladimir3padded logs there and printed dicts in both test cases - same string vs unistring compare, but mine is failing21:43
*** jsavak has quit IRC21:44
bcwaldons1rp: is lp:~jtran/nova/lp824008 good to go in? Noticed you were the second core approve21:44
s1rpbcwaldon: yep, think it's good to go21:47
bcwaldons1rp: Ok, I can push the button if you're busy. Otherwise, I'll just let you do it21:48
s1rpbcwaldon: yeah, ill go ahead and do that21:49
bcwaldonjk0: gotta head out for a bit. I'll check back later on your rescue branch21:49
bcwaldons1rp: thank you, sir21:49
jk0bcwaldon: thanks man21:49
bcwaldonjk0: no problem :)21:50
*** bsza has quit IRC21:51
vladimir3pvishy: what do you think about adding also volume extra_data (not type_extra_data, but volume). or we can call it meta-data in analogue to instance metadata21:54
vishyi think volume metadata is good too21:55
vladimir3pok, will add it as well. This one we definitely require21:56
*** bcwaldon has quit IRC21:59
*** rnirmal has quit IRC21:59
openstackjenkinsProject nova build #1,266: SUCCESS in 3 min 36 sec: https://jenkins.openstack.org/job/nova/1266/22:02
openstackjenkins* Tarmac: Fixed bug in which DescribeInstances was returning deleted instances.  Added tests for pertinent api methods.22:02
openstackjenkins* Tarmac: Accept binary user_data in radix-64 format when you launch a new server using OSAPI. This user_data would be stored along with the other server properties in the database. Once the VM instance boots you can query for the user-data to do any custom installation of applications/servers or do some specific job like setting up networking route table.22:02
openstackjenkinsYou can query for user-data using curl with the URL:-22:02
openstackjenkinscurl http://169.254.169.254/latest/user-data22:02
heckjjaypipes: ping22:02
*** rods has joined #openstack-dev22:06
vishyvladimir3p: looking at your code now22:13
*** Guest77784 has quit IRC22:15
vladimir3pvishy: let me know if you will find a reason of test failure ... I gave up22:17
vladimir3pvishy: in 10-15 min I will push another version with VolumeMetadata22:18
vishyvladimir3p: name of the tests file?22:18
vladimir3pvishy: nova.tests.test_volume_types_extra_specs vs nova.tests.test_instance_types_extra_specs22:19
*** zns has joined #openstack-dev22:19
openstackjenkinsProject nova build #1,267: SUCCESS in 3 min 32 sec: https://jenkins.openstack.org/job/nova/1267/22:19
openstackjenkinsTarmac: Fixes lp82820722:19
openstackjenkinsThe Host API reboot action was broken due to addition of a parameter to a utility method in compute/api.py22:19
vishyvladimir3p: it is int vs string22:20
vishyvladimir3p: you have to add specs as a string22:22
vladimir3pvishy: yeah... let me check22:22
*** nickon has quit IRC22:22
*** markvoelker has quit IRC22:25
vladimir3pvishy: ok, seems right. Can't test it right now. will do it a bit later22:26
creihtvishy: quick question about volumes22:29
creihtIs it the desired behavior that you have to restart the instance when you attach a volume to it for it to show up on that instance?22:30
*** zns has quit IRC22:30
*** Gordonz has quit IRC22:37
*** dragondm has quit IRC22:37
*** mattray has quit IRC22:38
vladimir3pvishy: need to go. just added code for metadata to the same tree (lp:~vladimir.p/nova/volume_type_extradata). ttyl22:46
vishyvladimir3p: awesome22:46
openstackjenkinsProject swift build #309: SUCCESS in 29 sec: https://jenkins.openstack.org/job/swift/309/22:47
openstackjenkinsTarmac: Add support for pulling quarantine stats.22:47
*** vladimir3p has quit IRC22:47
*** zns has joined #openstack-dev22:50
*** nmistry has joined #openstack-dev22:54
openstackgerritJames E. Blair proposed a change to openstack/openstack-ci: Update rfc.sh to assert divergence.  https://review.openstack.org/27422:55
openstackgerritJames E. Blair proposed a change to openstack/openstack-ci-puppet: Update rfc.sh to assert divergence.  https://review.openstack.org/27522:56
openstackgerritDolph Mathews proposed a change to openstack/keystone: Refactored manage.py to be both testable and useful for testing  https://review.openstack.org/27623:05
*** nmistry has quit IRC23:10
*** mwhooker has quit IRC23:11
*** heckj has quit IRC23:11
openstackjenkinsProject nova build #1,268: SUCCESS in 3 min 25 sec: https://jenkins.openstack.org/job/nova/1268/23:11
openstackjenkins* Tarmac: Updated docs for the recent scheduler class changes.23:11
openstackjenkins* Tarmac: Passes empty string instead of None to MySQLdb driver if the DB password isn't set.23:11
*** mfer has joined #openstack-dev23:15
*** ohnoimdead has quit IRC23:22
*** mwhooker has joined #openstack-dev23:22
*** mwhooker has quit IRC23:44
openstackjenkinsProject nova build #1,269: SUCCESS in 3 min 24 sec: https://jenkins.openstack.org/job/nova/1269/23:50
openstackjenkinsTarmac: Fixes issue where ServersXMLSerializer was missing a method for update actions23:50
*** bcwaldon has joined #openstack-dev23:54

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!