Tuesday, 2011-08-02

*** martine_ has joined #openstack-dev00:11
*** jdurgin has quit IRC00:11
*** medberry is now known as med_out00:13
*** mszilagyi_ has joined #openstack-dev00:16
*** mszilagyi has quit IRC00:18
*** martine_ has quit IRC00:27
*** heckj has quit IRC00:30
*** rods has quit IRC00:57
*** chomping has joined #openstack-dev01:00
*** martine_ has joined #openstack-dev02:56
*** thickski_ has joined #openstack-dev03:40
*** thickskin has quit IRC03:41
*** martine_ has quit IRC04:03
*** cp16net has joined #openstack-dev04:33
*** chomping has quit IRC04:50
*** zaitcev has quit IRC04:55
*** RobertLaptop has joined #openstack-dev05:16
*** cp16net has quit IRC06:28
*** reidrac has joined #openstack-dev07:11
*** mnour has quit IRC07:13
*** kennett85 has joined #openstack-dev07:16
kennett85I'm trying to do a virtual tour but I can't get it to work.07:17
kennett85:http://dojotoolkit.org07:17
kennett85sorry here is my code.   http://pastebin.com/RspU53NG07:17
*** AhmedSoliman has joined #openstack-dev07:46
*** duker has quit IRC07:54
*** tudamp has joined #openstack-dev08:05
*** darraghb has joined #openstack-dev08:11
*** kennett85 has quit IRC08:22
*** mnour has joined #openstack-dev08:22
*** mnour has quit IRC08:23
*** mnour1 has joined #openstack-dev08:23
*** RobertLaptop has quit IRC08:31
*** RobertLaptop has joined #openstack-dev08:35
*** alekibango has joined #openstack-dev09:11
*** alekibango has quit IRC09:14
*** mnour1 has quit IRC09:14
*** mnour has joined #openstack-dev09:14
*** alekibango has joined #openstack-dev09:15
*** mnour has quit IRC09:19
*** mnour has joined #openstack-dev09:20
*** mnour has quit IRC09:28
*** mnour1 has joined #openstack-dev09:28
*** mnour1 has quit IRC09:33
*** mnour has joined #openstack-dev09:33
*** RobertLaptop has quit IRC09:41
*** mnour has quit IRC09:42
*** mnour has joined #openstack-dev09:42
*** mnour has quit IRC09:50
*** mnour has joined #openstack-dev09:50
*** RobertLaptop has joined #openstack-dev09:54
*** mnour has quit IRC09:55
*** mnour1 has joined #openstack-dev09:55
*** mnour1 has quit IRC09:56
*** mnour has joined #openstack-dev09:56
*** mnour has quit IRC09:59
*** mnour has joined #openstack-dev09:59
*** AhmedSoliman has quit IRC10:00
*** mnour has quit IRC10:00
*** mnour has joined #openstack-dev10:00
*** mnour has quit IRC10:02
*** mnour has joined #openstack-dev10:05
*** throughnothing has quit IRC10:05
*** throughnothing has joined #openstack-dev10:06
*** mnour has quit IRC10:35
*** mnour has joined #openstack-dev10:35
*** tudamp has quit IRC10:40
*** rods has joined #openstack-dev10:47
*** JStoker has quit IRC10:49
*** tudamp has joined #openstack-dev10:59
*** yamahata_ has quit IRC10:59
*** yamahata__ has quit IRC10:59
*** markvoelker has joined #openstack-dev11:19
openstackjenkinsProject nova build #1,175: SUCCESS in 4 min 18 sec: http://jenkins.openstack.org/job/nova/1175/11:21
openstackjenkinsTarmac: Adds the auth token to nova's RequestContext.  This will allow for delegation, i.e., use of a nova user's credentials when accessing other services such as glance, or perhaps for zones.11:21
*** mnour has quit IRC11:23
*** mnour1 has joined #openstack-dev11:23
openstackjenkinsProject nova build #1,176: SUCCESS in 4 min 34 sec: http://jenkins.openstack.org/job/nova/1176/11:26
openstackjenkinsTarmac: Better error handling for resizing.11:26
*** mnour has joined #openstack-dev11:27
*** mnour1 has quit IRC11:28
openstackjenkinsProject nova build #1,177: SUCCESS in 4 min 17 sec: http://jenkins.openstack.org/job/nova/1177/11:34
openstackjenkins* Tarmac: code was checking for key in sqlalchemy instance and will ignore if value is None, but wasn't working if floating_ip was a non-sqlalchemy dict obj.  Therefore, updated the error checking to work in both caes.11:34
openstackjenkins* Tarmac: While we currently trap JSON encoding exceptions and bail out, for error notification it's more important that *some* form of the message gets out. So, we take complex notification payloads and convert them to something we know can be expressed in JSON.11:34
openstackjenkinsProject nova build #1,178: SUCCESS in 4 min 23 sec: http://jenkins.openstack.org/job/nova/1178/11:43
*** martine_ has joined #openstack-dev11:47
*** mnour has quit IRC11:48
*** mnour1 has joined #openstack-dev11:48
*** mfer has joined #openstack-dev11:48
*** mnour1 has quit IRC11:57
*** mnour has joined #openstack-dev11:57
*** jeremyb has quit IRC12:02
*** jeremyb has joined #openstack-dev12:02
sandywalshsorry vishy, was a civic holiday here yesterday ... ping back at ya12:10
*** mnour has quit IRC12:17
*** mnour has joined #openstack-dev12:17
*** mnour has quit IRC12:19
*** mnour has joined #openstack-dev12:19
*** yamahata has joined #openstack-dev12:22
jaypipessandywalsh: damn Canadians with their civic holidays.12:27
jaypipesyamahata: ping12:28
yamahatajaypipes, pong12:28
sandywalshjaypipes ... hey Don Beret! Long time!12:28
jaypipesyamahata: sorry, I totally messed up on that meeting request. I wrote one time, but the link said another :(12:28
jaypipessandywalsh: :)12:29
*** mnour has quit IRC12:29
*** mnour has joined #openstack-dev12:29
yamahatajaypipes, that's okay. Let's try again tomorrow.12:29
jaypipesyamahata: you sure? 9pm is late :( (though 7am is early, I suppose...)12:30
jaypipesyamahata: never easy to get a good time...12:30
yamahatajaypipes, 9pm is okay for me. Yes, it's really difficult.12:31
jaypipesyamahata: ok, then tomorrow morning (my time). I will transfer all your notes and add my own to an etherpad for our meeting. thx for your patience!12:32
yamahatajaypipes, thank you for arranging the meeting.12:32
jaypipesno prob12:33
*** mnour has quit IRC12:33
*** mnour has joined #openstack-dev12:34
*** martine_ has quit IRC12:42
*** mnour has quit IRC12:52
*** mnour has joined #openstack-dev12:52
*** bsza has joined #openstack-dev12:55
*** yamahata has quit IRC12:56
*** bcwaldon has joined #openstack-dev12:58
*** ameade has joined #openstack-dev13:03
sandywalsh:q13:03
sandywalsh:/13:04
*** mnour has quit IRC13:15
*** mnour has joined #openstack-dev13:15
*** martine_ has joined #openstack-dev13:21
*** bcwaldon has quit IRC13:34
*** bcwaldon has joined #openstack-dev13:36
*** zul has quit IRC13:38
*** mnour has quit IRC13:40
*** mnour1 has joined #openstack-dev13:40
*** mnour has joined #openstack-dev13:40
*** zul has joined #openstack-dev13:40
*** mnour has quit IRC13:41
*** mnour1 has joined #openstack-dev13:41
*** mnour has joined #openstack-dev13:41
*** kbringard has joined #openstack-dev13:41
*** mnour has quit IRC13:42
*** mnour has joined #openstack-dev13:42
*** zul has quit IRC13:42
*** mnour has quit IRC13:42
*** mnour1 has joined #openstack-dev13:42
*** zul has joined #openstack-dev13:43
*** mnour has joined #openstack-dev13:44
*** mnour2 has joined #openstack-dev13:45
*** mnour has joined #openstack-dev13:45
*** mnour has quit IRC13:46
*** mnour2 has joined #openstack-dev13:46
*** mnour1 has quit IRC13:47
*** mnour has joined #openstack-dev13:47
*** mnour has quit IRC13:48
*** mnour has joined #openstack-dev13:48
*** mnour1 has joined #openstack-dev13:50
*** mnour1 has quit IRC13:50
*** mnour has joined #openstack-dev13:50
*** zul has quit IRC13:54
*** cp16net has joined #openstack-dev13:55
*** amccabe has joined #openstack-dev13:55
*** zul has joined #openstack-dev14:02
*** kbringard has quit IRC14:03
*** kbringard has joined #openstack-dev14:03
*** statik has joined #openstack-dev14:08
*** yamahata has joined #openstack-dev14:08
*** yamahata has quit IRC14:09
bcwaldonvishy: Can you look at this again? Had to add xattr to get tests to pass: https://code.launchpad.net/~rackspace-titan/nova/remove-twistd/+merge/6986514:25
*** amccabe has quit IRC14:28
*** jkoelker has joined #openstack-dev14:28
jaypipesbcwaldon, jkoelker: so, I'm pretty sure the glance delayed delete functionality is what's blowing up test runs. I know that something isn't being cleaned up in the delayed_delete functional test case, and I think it may have something to do with the registry SQLite database file possibly being destroyed before the last loop in the delay test.14:39
jaypipesbcwaldon, jkoelker: still trying to figure out a good way to functionally test the delayed delete stuff...14:39
*** amccabe has joined #openstack-dev14:43
*** stiekes has joined #openstack-dev14:44
jkoelkerjaypipes: where is it blowing up at? During packaging again?14:49
jaypipesjkoelker: yeah. it gets hung up in test_scrubber, but the symptom can be the Failed to start servers thing, or it sometimes appears as a ECONNREFUSED to the registry server...14:50
jaypipesjkoelker: unfortunately, none of us can reproduce it locally :(14:50
jkoelkerhrm, I'll resetup a buildd server again and see if I can reproduce it from there, do you know where the tarball/repo is they are using for the build? I assume its not just the milestone branch14:53
*** dragondm has joined #openstack-dev14:59
jaypipesjkoelker: no, it's the proposed branch. python setup.py build...15:00
jkoelkerroger15:01
bcwaldonmtaylor: ping15:02
*** mattray has joined #openstack-dev15:05
*** dprince has joined #openstack-dev15:07
*** reidrac has quit IRC15:08
*** heckj has joined #openstack-dev15:09
*** mdomsch has joined #openstack-dev15:13
dprincesoren, mtaylor: Can you guys merge this one: https://code.launchpad.net/~dan-prince/glance/ubuntu-glance-cache2/+merge/7001915:26
mtaylordprince: yes. will do in a few minutes15:44
mtaylorbcwaldon: whazzup?15:44
bcwaldonmtaylor: Hey. I'm working on moving python-novaclient from github.com/rackspace to github.com/openstack and adding v1.1 support. Should I be added to the openstack org on github, or what?15:46
*** rnirmal has joined #openstack-dev15:47
mtaylorbcwaldon: ooh! potentially another victim...15:47
bcwaldonmtaylor: whaaaaat?15:47
mtaylorbcwaldon: one sec, and I'll innundate you with a bunch of thoughts15:48
ttxbcwaldon: escape while you still can15:51
bcwaldonttx: well I don't really want to get up...15:51
mtaylorbwahahaha15:52
mtaylorbcwaldon: so, my basic thought is that if we're moving you to github.com/openstack, it might be a good time to move you over to being managed by gerrit/jenkins - but I'm checking with the other folks on the ci team to make sure we're ready for another project15:53
bcwaldonmtaylor: I wouldn't be against that15:56
mtaylordprince: I'm adding the entry during the merge - but fwiw, next time you do a packaging patch (and thanks, btw) - could you add your commit message as an entry in debian/changelog using dch and then use debcommit to do the commit?15:56
mtaylorbcwaldon: great!15:56
bcwaldonmtaylor: I asked Jay earlier and he was unsure whether I should be added to the openstack org. Can you tell me that that org is designed for?15:57
mtaylorbcwaldon: well, right now it's a sort of half and half thing and is in transition15:57
mtaylorbcwaldon: the org is eventually going to be there so that it's easy to find the official "trunk" repos for each project15:58
mtaylorbcwaldon: we haven't fully put together the model for membership in the org - although really there is next to no reason for anyone to be members of it, since branch push access is managed through gerrit15:59
bcwaldonmtaylor: ok, once I have the code ready can we get it all set up in gerrit?16:01
mtaylorbcwaldon: totally. it's really just more of a coordination thing ... I'll have some info for you real-soon-now16:02
dprincemtaylor: Sure. Thanks man. I'll add my commit message next time.16:02
bcwaldonmtaylor: sounds good16:02
*** tudamp has left #openstack-dev16:05
mtaylordprince: sweet. you are merged/pushed16:06
*** mnour has joined #openstack-dev16:06
mtaylorbcwaldon: can we hold off for a few days? we're still sorting a few final things16:10
*** mdomsch has quit IRC16:12
*** mnour has quit IRC16:15
jaypipesblamar, bcwaldon, Vek, jkoelker, johan_-_: OK for a move to Gerrit/GitHub on Thursday morning? jeblair and mtaylor will be sending out a detailed plan for the move.16:20
blamarjaypipes: Whatever you say boss :) Just as long as it works!16:20
jaypipesjkoelker: and sorry, I forgot to include you in the follow-up email. :(16:20
jaypipesblamar: k.16:20
jaypipesblamar: but please don't call me boss :)16:20
blamarhaus?16:21
jkoelkerjkoelker: works for me16:23
*** mnour has joined #openstack-dev16:23
openstackgerritKevin L. Mitchell proposed a change to openstack/keystone: Pass auth_token to nova's RequestContext, for later delegation  https://review.openstack.org/12216:24
jaypipesblamar: :)16:25
jaypipesjkoelker: talkin' to yourself again? ;P16:25
jkoelkerSorry, its a bad habbit16:26
jaypipeshehe16:26
pvojaypipes: this is the novaclient move?16:26
jaypipespvo: Glance.16:26
pvojaypipes: cool.16:26
jaypipespvo: :)16:26
jaypipespvo: use Glancers are jumping in the pool.16:26
jaypipess/use/us16:26
jaypipespvo: I'll let you know if the water is freezing or not ;)16:27
pvohopefully no shrinkage16:27
jaypipespvo: lol16:27
jaypipesblamar, s1rp, jkoelker, Vek: if you have a spare 15 minutes... https://code.launchpad.net/~jaypipes/glance/refactor-stores/+merge/7009616:28
blamarjaypipes: wrapping your strings in parenthesis in preparation for i18n? (line 72)16:29
openstackjenkinsProject nova build #1,179: SUCCESS in 4 min 41 sec: http://jenkins.openstack.org/job/nova/1179/16:31
openstackjenkinsTarmac: Removing the xenapi_image_service flag in favor of image_service.16:31
*** jmckenty_ has joined #openstack-dev16:31
jaypipesblamar: yup.16:31
*** mnour has quit IRC16:42
*** mnour has joined #openstack-dev16:46
openstackgerritMonty Taylor proposed a change to openstack/openstack-ci: Instructions for running the git-bzr-ng migration.  https://review.openstack.org/12316:47
jaypipesjkoelker: thought that ImageAddResult would slip by people's reviews... :)16:48
jaypipesjkoelker: but no, you just had to go and notice it ;)16:48
heckjGerritWorkflow question - how do you vote with Gerrit? I'm a little confused by the UI. Do I implicitly give a +1 by adding my name to the reviewers?16:52
heckjtrying to do a review on https://review.openstack.org/12316:53
mtaylorheckj: nope. click the "review" button16:53
heckjmtalylor: Ahhh! Thanks16:53
*** mattray1 has joined #openstack-dev16:53
mtaylorheckj: although that certainly explains how you wound up on the reviewers list :) I was starting to worry16:53
openstackgerritA change was merged to openstack/openstack-ci: Instructions for running the git-bzr-ng migration.  https://review.openstack.org/12316:55
*** mattray has quit IRC16:57
heckjmtaylor: What's the difference between the check mark and the +1 at https://review.openstack.org/#change,12316:59
*** jdurgin has joined #openstack-dev17:00
*** mattray1 has quit IRC17:00
mtaylorheckj: the check mark is the definitive +2 approval (the thing that is similar to lp merge proposals going from "Needs Review" to "Approved"17:00
*** Tv_ has joined #openstack-dev17:01
heckjmtaylor: Got it - so the same actions I took is what ended up with the check mark - just that james blair did it.17:01
mtaylorheckj: very similar. when he hit review, since he's in the core team for that project, he had the option to vote +2 instead of +117:02
mtaylorand when he did that, it made the change Approved, and then jenkins tested it, which allowed gerrit to merge it17:03
heckjmtaylor: where is the core team setup getting pulled from? Is that the team in Launchpad, or something else maintained in Gerrit?17:09
*** mszilagyi has joined #openstack-dev17:12
jeblairheckj: it's pulled from launchpad, and should be kept closely in sync (starting real soon now)17:13
heckjAh - makes sense17:13
jaypipesblamar, jkoelker, Vek: another review on https://code.launchpad.net/~rackspace-titan/glance/registry-marker-lp819551/+merge/70094 would be great, too. thx in advance!17:14
*** mattray has joined #openstack-dev17:15
*** darraghb has quit IRC17:19
chemikadze Hi everyone! Does nova console client support keystone auth?17:20
chemikadzeLast time i looked on it all failed with redirect error.17:22
westmaaslp80966317:27
westmaasawww that didn't work17:27
westmaashttps://bugs.launchpad.net/glance/+bug/80966317:27
uvirtbotLaunchpad bug 809663 in glance "Glance milestone and integrated packages should automatically be uploaded to PyPI" [High,Confirmed]17:27
westmaasmtaylor: anything we can do to move this guy along? or is this not you?17:28
mtaylorwestmaas: it is me, and I'm going to get it done today17:29
mtaylorwestmaas: also, bug 80966317:29
uvirtbotLaunchpad bug 809663 in glance "Glance milestone and integrated packages should automatically be uploaded to PyPI" [High,Confirmed] https://launchpad.net/bugs/80966317:29
mtaylorshould trigger uvirtbot17:29
westmaasmtaylor: cool thanks17:31
*** dolphm has quit IRC17:31
westmaasjaypipes: you mind manually uploading glance to pypi once right now so we can move forward on some other branches that depend on it?17:34
jaypipeswestmaas: yep, lemme do that right now.17:35
jk0thanks17:35
westmaasty sir17:35
jaypipeswestmaas: the D3 milestone release, right?17:35
westmaasdo you know if Vek had his keystone integration in before then?17:36
westmaasVek: ^^17:36
jaypipeswestmaas: yes, he did. not the functional tests, but the basic context and middleware, yes.17:37
westmaasthen yeah that's what we should need17:37
westmaasassuming no vital bug fixes!17:37
jaypipeswestmaas: there are a few URI parsing fixes that went in afterwards...17:39
bcwaldonjaypipes, westmaas: Let's just go with trunk17:40
vishysandywalsh: https://blueprints.launchpad.net/nova/+spec/different-schedulers-per-call17:40
bcwaldonassuming you trust refactor-stores ;)17:40
jaypipeswestmaas, bcwaldon: OK, one thing to note if I do a trunk package to PyPI...17:40
*** dolphm has joined #openstack-dev17:41
jaypipeswestmaas, bcwaldon: anyone who upgrades is likely to run into the issue this guy ran into: https://bugs.launchpad.net/bugs/81498117:41
uvirtbotLaunchpad bug 814981 in openstack-live "glance-api fails on image delivery: AttributeError: context" [Medium,Confirmed]17:41
sandywalshvishy, looking17:41
westmaasuntil they update the glance conf?17:41
Vekexactly.17:41
jaypipeswestmaas: right.17:41
* westmaas hasn't actually looked yet17:41
westmaasyeah17:41
westmaashrm.17:41
jaypipeswestmaas: we can deal with it, but it's an issue...17:42
jk0that's something they would run into eventually anyway17:42
jaypipesjk0: yup.17:42
westmaassame for D3 though?17:42
jaypipeswestmaas: yes.17:42
jaypipeswestmaas: but there were a couple fixes after D3 that are useful...17:42
jaypipeswestmaas: so might as well do a trunk I guess17:42
westmaasyeah17:42
westmaasthat's what I'm thinking17:42
jaypipeswestmaas: k. gimme 10.17:42
vishysandywalsh: I was wondering if you wanted to tackle that one.  Seems like you could bang it out faster than anyone else.17:43
vishysandywalsh: I'm undecided on if it is best to run separate workers, or separate greenthreads listening on new topics, or just keep a list of drivers in the scheduler for each call.17:44
sandywalshvishy, wouldn't be hard, but I'm not sure the existing scheduler framework couldn't handle it17:44
dprincejaypipes, westmaas: If we get the paste stuff out of the conf we'd be better off: https://bugs.launchpad.net/glance/+bug/81520817:44
uvirtbotLaunchpad bug 815208 in glance "remove paste configuration from the config files" [Low,Triaged]17:44
vishysandywalsh: it could with option 3, I'm just thinking that we might want to take the opportunity to improve it a little if we feel like the architecture is wrong.17:45
dprincejaypipes: Sorry. I see you already commented. Anyway. Something nice to have to head off issues.17:45
sandywalshvishy, the way the scheduler works is: 1. it looks for schedule_<command> (like "schedule_run_instance") and if not found 2. call the generic schedule() method. I think we just need schedule_create_volume() and some refactoring17:45
*** dolphm has quit IRC17:46
*** dolphm has joined #openstack-dev17:46
jaypipeswestmaas, jk0: latest on PyPI now. should be glance-2011.3.tar.gaz17:47
jk0thanks man17:47
*** dolphm has joined #openstack-dev17:47
jaypipesnp17:47
vishysandywalsh: hmm, the idea is that you can keep a different backend scheduler for each call.  I guess we can solve it by keeping a map of {call: SchedulerDriver) and just picking the right one17:47
vishy17:47
westmaasjaypipes: ty sir17:47
jaypipeswestmaas: welcome sir17:47
vishysandywalsh: do you have time to do it?  Or should I find someone else?17:47
sandywalshvishy lemme check and see what the priority is ... sec17:48
vishysandywalsh: if not, its fine.  I can bang out the simple version pretty quick.  Just wanted to give you a chance to rearchitect it a little if you thought it was needed.17:49
sandywalshvishy I know Ozone is priming for alpha, lemme confirm with the higher powers first17:50
sandywalsh(just waiting to hear back now)17:50
openstackgerritMonty Taylor proposed a change to openstack/openstack-ci: Added cache flushing and db backups.  https://review.openstack.org/12417:54
sandywalshvishy how would you specify which scheduler to use without busting the public API?17:55
sandywalshvishy or is it --scheduler_driver becomes --scheduler_driver_compute/--scheduler_driver_volume/etc. ?17:56
jaypipesGlancers: w00t. the Jenkins job no longer seems to be blocking on random server startup failures... unclogging our review queue.17:57
jaypipesGlancers: also, I added a param builder that fires python setup.py test instead of ./run_tests.sh -N so you can check your work against the package builder, too: https://jenkins.openstack.org/view/Glance/job/glance-param-setuppy-test/17:57
bcwaldonjaypipes: how do you feel about this bug: https://bugs.launchpad.net/glance/+bug/81993818:00
uvirtbotLaunchpad bug 819938 in glance "setup.py needs to define dependencies" [Undecided,New]18:00
bcwaldonjaypipes: Seems like such a simple thing, I wonder how we got this far18:01
jaypipesbcwaldon: I will defer to mtaylor on that one. I'm used to the debian control way of managing deps...18:01
bcwaldonjaypipes: or maybe I'm wrong18:01
bcwaldonjaypipes: right, but when you 'pip install' something, its platform independent18:02
jaypipesbcwaldon: understood. I defer to mtaylor :)18:02
bcwaldonjaypipes: sounds good :)18:02
mtaylorbcwaldon: so, I originally thought that setup.py should enforce deps - but then apparently that gets wonky somewhere for some reason that I don't understand (I think, as usual, something in setuptools is only half-assed)18:02
bcwaldonmtaylor: not surprised, but then it makes our glance pip-requires file bleed over into our nova pip-requires file18:03
mtaylorbcwaldon: in any case, I believe I had that going in nova for like a day before someone made me remove it :)18:03
bcwaldonmtaylor: Ha, well I feel like you should be able to pip-install glance and not have to track down every dependency18:03
mtaylorbcwaldon: yeah. other than that - I actually don't pay any attention to pip-requires at all, because all of the builders use apt-get to install deps18:03
bcwaldonmtaylor: maybe glance doesn't belong in pypi?18:04
mtaylorbcwaldon: yes. I agree with that. I believe if you are using pip and you pip install glance, it should work18:04
mtaylorbcwaldon: it TOTALLY belongs in PyPI ... we just run the builders using packages because that's the harder thing18:04
bcwaldonmtaylor: I feel like pip is geared more towards libraries, while glance is an application18:04
mtaylorbcwaldon: it's really easy to add a bunch of deps from pypi that aren't in distros18:04
bcwaldonmtaylor: ok, so do we have a clear path forward on fixing the issue at hand?18:05
mtaylorbcwaldon: yes. pip itself is more geared towards libraries, but it still should work, really18:05
mtaylorbcwaldon: nope. :)18:05
bcwaldonson of a...18:05
bcwaldonmtaylor: I'm really curious what problem you ran into before, it seems like such a simple thing to me18:06
mtaylorbcwaldon: so, I _think_ what we might want to do perhaps is to define two builders in jenkins - one that builds against packages and one that builds in a virualenv using pip18:06
bcwaldonjay actually just did that18:07
bcwaldonjaypipes: ^^18:07
bcwaldonor maybe not...18:07
bcwaldonhe did for glance, are you talking about a new builder for nova?18:07
bcwaldonmtaylor: ^18:08
mtaylorhe's talking about a param-build builder that allows you to run something in the context of jenkins18:08
mtaylorI'm talking about builders that are part of the trunk gating18:08
bcwaldonmtaylor: gotcah18:08
bcwaldongotcha*18:09
mtaylorso that we make sure that both pip-requires and apt-get depends are solid before we merge something18:09
bcwaldonmtaylor: that sounds great18:09
mtayloralso ... jaypipes ... we're using run_tests.sh -N in the other jenkins builders - should we be using python setup.py test? last time I tried that in nova at least it was ultimate fail18:09
*** jmckenty_ has quit IRC18:10
*** glenc_ has joined #openstack-dev18:10
*** glenc has quit IRC18:12
mtaylorbcwaldon: this will all be easier to tackle after we transition on thursday18:15
bcwaldonmtaylor: glance transition is officially happening thursday?18:15
mtaylorbcwaldon: yeah.18:17
bcwaldonmtaylor: Okay, that'll work for me. I'm adding the same xattr dep into nova pip-requires for now18:18
mtaylorbcwaldon: at that point, it actually becomes WAY easier to set up multiple conditions/jobs that get tested18:18
mtaylorbcwaldon: just for the record - I am certainly not opposed to adding in deps to setup.py - we just need to test it and make sure it works - and that it _doesn't_ screw up package-based installs (I think there might have been something along those lines)18:19
bcwaldonmtaylor: okay, I'm with you 100%18:19
*** dprince has quit IRC18:23
bcwaldonmtaylor: packaging needs to be updated for https://code.launchpad.net/~rackspace-titan/nova/remove-twistd/+merge/69865 once it gets merged18:28
bcwaldonmtaylor: can I help out, or do you want to take care of it18:28
bcwaldonI would love to get some feedback on this branch. It's updating the HACKING file which affects everybody! https://code.launchpad.net/~rackspace-titan/nova/update-HACKING/+merge/6982918:35
mtaylorbcwaldon: if twisted is gone with that patch - that means that python setup.py test should work in nova now, yeah?18:37
sandywalshjaypipes, you seeing this on keystone unit tests? http://paste.openstack.org/show/2006/18:38
mtaylorbcwaldon: or, more specifically, that nosetest should work directly rather than needing the twisted additions in run_test.py18:38
mtaylor?18:38
dolphmsandywalsh: delete 'keystone.token.db' from the cwd as well as 'keystone.db'18:39
sandywalshah, thanks dolphm18:39
dolphmsandywalsh: a revised test script is in the works to fix that issue18:40
sandywalshdolphm, ah, good to know18:40
mwhookerhi all. Is there any nova code you would like to see cleaned up? I'm getting up to speed in the codebase, and trying to help out in the process.18:41
jk0pylint is always a great place to start for that kind of stuff18:42
jk0(I'm sure there is a lot that could be done with that)18:42
mwhookerjk0: I've done a pass at pylint, I'll check for more but there may not be much left18:42
ameadeyeah most of them are the "blank does not have blank member"18:43
mwhookerand most of those are false positives18:43
ameadeI'm sure there are a ton of goodies still hidden in there18:43
ameadeyeah18:43
jk0mwhooker: have a look at this: https://jenkins.openstack.org/job/nova-pylint/violations/?18:43
openstackgerritMonty Taylor proposed a change to openstack/openstack-ci: Ensured that projects are in a valid set.  https://review.openstack.org/12518:43
ameademwhhoker, jk0: I usually use this job; https://jenkins.openstack.org/job/nova-pylint-errors/violations/18:44
mwhookergood resources. I'll check both those out18:45
mwhookerthanks18:45
* mtaylor == ++ for cleaning pylint18:45
ameade:)18:45
bcwaldonmtaylor: not exactly sure, I'm not 100% on top of the venv stuff18:45
mtaylorok18:45
bcwaldonmtaylor: the packaging differences are Twisted and some removed files. I know nova-instancemonitor was listed in the package script18:45
openstackjenkinsProject nova build #1,180: SUCCESS in 4 min 30 sec: http://jenkins.openstack.org/job/nova/1180/18:46
openstackjenkinsTarmac: - Remove Twisted dependency from pip-requires18:46
openstackjenkins- Remove Twisted patch from tools/install_venv.py18:46
openstackjenkins- Remove eventlet patch from tools/install_venv.py18:46
openstackjenkins- Remove tools/eventlet-patch18:46
openstackjenkins- Remove nova/twistd.py18:46
openstackjenkins- Remove nova/tests/test_twistd.py18:46
openstackjenkins- Remove bin/nova-instancemonitor18:46
openstackjenkins- Remove nova/compute/monitor.py18:46
openstackjenkins- Add xattr to pip-requires until glance setup.py installs it correctly18:46
openstackjenkins- Remove references to removed files from docs/translations/code18:46
*** zaitcev has joined #openstack-dev18:49
ttxvishy: ping18:51
*** glenc has joined #openstack-dev18:52
*** glenc_ has quit IRC18:54
jaypipesmtaylor: sorry, stepped away for a bit..18:56
jaypipesmtaylor: no, I just added the setup.py test one because that's where we were seeing issues...18:56
mtaylorjaypipes: k18:56
jaypipessandywalsh: no, haven't seen those.18:56
sandywalshjaypipes, k, resolved by nuking all the .db's18:58
*** hub_cap has joined #openstack-dev18:58
jaypipessandywalsh: yeah. I'm actually working on a keystone branch that adds in a bunch of nova/glance standard run_tests.sh/py stuff as well as cleans up the setup.py to use nose.collector...18:58
sandywalshnice ... a nose cleaner :)18:59
dolphmjaypipes: looking forward to that! i'm working on testing multiple backends from run_tests.py18:59
*** zedas has joined #openstack-dev19:00
zedashey folks19:00
jaypipesdolphm: should be done today. got most of it done.19:00
jaypipessandywalsh: :)19:00
zedashey folks, me and mwhooker dumped a list of nova .py files ordered by when they were last modified: http://codepad.org/cRzcK7sy  I'm planning on starting the top (oldest) and doing code review/cleanup/improvement.19:01
zedaslooking at this list, nova/twistd.py isn't being used, and nova/tests/test_twistd.py too.  can we just delete those?19:01
jaypipesVek, bcwaldon, mtaylor: so... it looks like the nova jenkins slave may need to have the latest glance package installed, right?19:02
Vekit would appear so.19:03
Vekironically, it wouldn't have been a problem if I hadn't reverted the 'hasattr' test on 'set_auth_token' :)19:03
jaypipesJenkins is a very busy girl today.19:03
mtaylorjaypipes: should happen natually if the glance package gets uploaded to the ppa19:03
jaypipesVek: heh, indeed.19:03
jaypipesmtaylor: not sure. I uploaded to PyPI, not the PPA. I thought the PPA stuff happened automagically, no? the glance-ppa jenkins job?19:04
mtaylorjaypipes: yes. it should happen automagically - and then the jenkins slave _should_ automagically ... oh, wait. hrm19:04
mtaylorok. looking19:05
sandywalshzedas, I think twistd dependencies were mostly removed in the switch to eventlet, but something in the back of my head says objectstore still wants it?19:05
westmaaszedas: last commit was actually to remove those19:05
westmaashttps://code.launchpad.net/~rackspace-titan/nova/remove-twistd/+merge/6986519:05
zedaswestmaas: odd i just did a pull.  lemme pull again...19:05
westmaas35 mins ago! :)19:05
jaypipessandywalsh: instance-monitor19:05
sandywalshah19:05
zedasjaypipes: yeah instant monitor is using twisted, but not nova.twstid19:05
Vek(that spelling is rather twisted)19:06
jaypipeszedas: latest patch removed instance monitor IIRC19:06
jaypipeslol19:06
zedasjaypipes: ha yeah, 35 minutes too late.19:06
zedasalright, lemme find some more culprits...19:06
jaypipeszedas: :)19:06
openstackgerritA change was merged to openstack/openstack-ci: Ensured that projects are in a valid set.  https://review.openstack.org/12519:10
openstackgerritA change was merged to openstack/openstack-ci: Added cache flushing and db backups.  https://review.openstack.org/12419:10
openstackjenkinsProject nova build #1,181: SUCCESS in 6 min 29 sec: http://jenkins.openstack.org/job/nova/1181/19:13
openstackjenkinsTarmac: Moves image creation from POST /images to POST /servers/<id>/action19:13
zedasok, next two:  nova/tests/runtime_flags.py and nova/tests/declare_flags.py don't test anything, or at least don't obviously test something.19:15
zedasso, would like to either remove those or make them actually do something.  any thoughts?19:16
zedasand can someone explain what nova/tests/test_localization.py is supposed to do?  it seems to just be scanning the files and doing the localization manually.19:20
zedasahh there we go, those two files are used in test_flags.py19:23
*** ameade has quit IRC19:28
ttxMeeting in #openstack-meeting in 90 minutes.19:30
openstackgerritMonty Taylor proposed a change to openstack/openstack-ci: Added script to force set project watches.  https://review.openstack.org/12619:34
Vekjaypipes/mtaylor: my glance-private-images merge-prop failed due to glance version skew on jenkins.  Can we get that updated so it can go through?19:37
openstackgerritVish Ishaya proposed a change to openstack/keystone: Adds support for authenticating via ec2 signatures  https://review.openstack.org/12719:42
*** hazmat has left #openstack-dev19:44
bcwaldonsoren: hey, I just added a MP for the ubuntu package. Looks like there are a couple of instancemonitor things that still need to be removed. Feel free to delete my MP (the changelog isn't relevant anymore)19:45
notmynametermie: does git-bzr work for you in Lion? seems that I can't push new branches to LP19:46
*** Tv_ has quit IRC19:47
mtaylorVek: ok. glance should be updated now19:50
*** RobertLaptop has quit IRC19:53
*** bcwaldon has quit IRC19:53
openstackgerritJames E. Blair proposed a change to openstack/openstack-ci-puppet: Move all node configuration into site.pp, with node identifiers based on hostnames so that puppet self-identifies which node config to use.  https://review.openstack.org/12819:59
openstackgerritsleepsonthefloor proposed a change to openstack/keystone: nova context should be initialized with user_id and project_id, not refs.  additionally, is_admin needs to be set to behave as expected.  https://review.openstack.org/12920:03
openstackgerritJames E. Blair proposed a change to openstack/openstack-ci: Use "site.pp" in launch script.  https://review.openstack.org/13020:05
Vekmtaylor: Thanks...20:05
*** med_out is now known as med20:07
*** med is now known as medberry20:07
Vekmtaylor: doesn't appear that jenkins picked up the right version of glance20:08
mtaylorVek: hrm20:08
jkoelkerjaypipes: I got glance to build successfully under pdebuild20:08
jkoelkerhttp://paste.openstack.org/show/2013/20:08
mtaylorVek: 2011.3~d4~20110802.176-0ubuntu0ppa1~natty1 from http://ppa.launchpad.net/nova-core/trunk/ubuntu/20:08
Vekah, this version of glance may not yet be in the ppa20:09
jkoelkerhad to make those changes to the build repo though, it appears it was building off a 07/09 snapshot still20:09
Vekjaypipes pushed it up into pypi20:09
jaypipesjk0: doh! that was it.20:09
jk0ahah20:09
mtaylorah.20:10
*** Tushar has joined #openstack-dev20:10
mtaylorVek: well that's the thing then... let me check what's up with the ppa20:10
TusharVishy: Are you there?20:11
vishyTushar: yes20:11
TusharI have just seen that you have assigned security group BP to sleepsonfloor..20:11
TusharI have almost finish with the implementation and writing unit testcases.20:12
Tusharhttps://blueprints.launchpad.net/nova/+spec/os-security-groups20:12
vishydoh20:12
vishywe have an implementation also20:12
mtaylorVek: the latest trunk version of glance (2011.3~d4~20110802.176-0ubuntu0ppa1~natty1) seems to be in the nova-core ppa20:12
vishyTushar: do you have a branch up20:12
TusharI am going to create one today.20:13
Vekmtaylor: check if glance/client.py has a def for 'set_auth_context'20:13
vishyTushar: ok i guess we can look at both versions and see which we want to use?20:13
Veker, actually, glance/common/client.py, and it's a def for 'set_auth_token'20:13
* Vek looks for his "bang-head-here" mousepad20:14
TusharVishy: Ok20:14
mtaylorVek: no.20:15
mtaylorVek: ok. so that's vexing20:15
Vekindeed :)  jaypipes, any insights?20:15
jaypipesVek: one sec, in PPB meeting...20:15
Vek'k20:16
sleepsonthefloorTushar, Vishy: check out https://github.com/cloudbuilders/openstackx/blob/master/extensions/admin.py#L821 and https://github.com/cloudbuilders/openstackx/blob/master/extensions/admin.py#L90220:17
openstackjenkinsProject swift build #300: SUCCESS in 43 sec: http://jenkins.openstack.org/job/swift/300/20:17
openstackjenkinsTarmac: added Gaurav B. Gangalwar to AUTHORS file20:17
vishyTushar: if you want to look at the code that we have and integrate it with yours, I can assign it back to you20:21
vishyTushar: we haven't done any unittests yet so yours will probably be ready sooner20:22
TusharVishy: Please assign it to me. I will check your branch and integrate with my branch as needed20:26
vishyTushar:  Done.  I hope that your layout is similar to sleepsonthefloor's above, so we don't have to change our dash code too much.20:27
TusharVishy: I will take care of it.20:28
vishyTushar: thx20:28
sleepsonthefloorcool - thanks Tushar20:28
mtaylorsoren: awake?20:30
Tusharsleepsonthefloor: You are welcome20:30
sorenmtaylor: Mostly.20:30
notmynametermie: are you using Lion yet?20:31
sorenmtaylor: My allergy meds are making rather slow, though.20:31
mtaylorsoren: apparently the version of glance that's in the nova-core ppa is unacceptable to Vek and jaypipes in some way ... which is weird, because everything _seems_ to be working20:31
sorenmtaylor: ...and bad at typing full sentences.20:31
mtaylorsoren: allergy meds are lovley20:31
sorenMy usual ones are fine, they were just in my basement, so I took some other ones. I now remember why I got the new kind.20:31
sorenaaaaanyway.. Can you elaborate?20:32
sorenmtaylor: What are the unacceptable symptoms?20:32
mtaylorsoren: apparently it doesn't work20:33
mtaylorsoren: is missing something from glance/common/client.py20:33
mtaylorwhich may indicate oldness?20:33
sorenbug report?20:33
sorenor any other source of information?20:33
mtaylornot sure there is one - they have a merge prop that isn't passing because it's expecting something from glance which ain't there20:33
sorenI have IRC logs, too, if that helps.20:33
mtaylorI think IRC logs might be the key20:33
sorenWhat's the merge prop?20:34
jk0the problem is glance trunk isn't in the PPA20:34
* soren has a guess as to what the problem is, but would like more details first.20:34
vishynotmyname: I am20:34
openstackgerritKevin L. Mitchell proposed a change to openstack/keystone: Determine is_admin based on 'Admin' role; remove dead project_ref code; pass auth_token into request context; pass user_id/project_id into request context instead of their refs  https://review.openstack.org/13120:34
jk0(even though it says it is)20:34
jaypipesjeblair: whatcha think about sleepsonthefloor's last comment on https://review.openstack.org/#change,122? What is the best practice or protocol there?20:34
sorenjk0: What says what is where, and what disagrees?20:34
jeblairjaypipes: looking20:35
notmynamevishy: are you using git-bzr?20:35
termienotmyname: not using lion yet20:35
sorenI seem to be copmletely surrounded by no information at all :)20:35
jk0soren: hang tight20:35
sorenjk0: I always do.20:35
vishynotmyname: not regularly, I have used it though20:35
jk0soren: so apparently set_auth_token() (as is here: http://bazaar.launchpad.net/~hudson-openstack/glance/trunk/view/head:/glance/common/client.py ) isn't showing up in the PPA20:36
jk0which is causing jenkins to fail a MP because it depends on that new method20:36
notmynametermie: I was having an issue with pushing a new branch to lp earlier today (I solved it by reverting to pure bzr)20:36
termienotmyname: did you open an issue on the gh page about it?20:37
termienotmyname: easiest for me to track bits there20:37
sorenjk0: Jenkins doesn't give a #¤&!"#¤ about what's in the PPA.20:37
soren...unless something's changed that I'm not aware of.20:37
jk0soren: from my understanding mtaylor upgraded glance from the PPA on the jenkins server20:38
sorenAha!20:38
sorenmtaylor: What have you been up to?20:38
ttxTeam meeting in 20 minutes in #openstack-meeting20:38
mtaylorsoren: stuff20:38
mtaylorsoren: it's probably my fault20:38
jeblairjaypipes, sleepsonthefloor: i think uploading that patch for illustration is fine.  in this case, kevin should probably just change his patch, and we should consider the other patch feedback, and abandon it [more...]20:39
jaypipesjeblair: ok.20:39
mtaylorsoren: the build slave for nova has the apt build-deps for nova installed20:40
mtaylorsoren: and the nova-core/trunk ppa in its sources.list20:40
jeblairif it had been a bigger change, you could approve sleepsonthefloor's change, then ask kevin to rebase his patch on that commit20:40
jaypipesjeblair: ok.20:41
jaypipesjeblair: for some reason, I was thinking you could edit the original patch inline like you can in a fork-and-edit pull request on GH. I'm losing my mind obviously.20:41
Vekjeblair, jaypipes: I just submitted a different patch that does what mine does and what sleepsonthefloor's patch does, plus kills some then-dead code20:42
jeblaira third possibility would be for sleepsonthefloor to pull kevin's patch, change it, then upload a new patchset for the change.  i'm not certain about the permissions required to do this (ie, update someone else's change), or whether that would violate code attribution norms (ie, who the Author: is)20:42
jaypipesVek: yes, I noticed :)20:42
Vek:)20:42
Vekmtaylor: my patch depends on set_auth_token in the glance client, which was recently added.20:42
jeblairso it's fine to either update changes, or create new changes, whichever you feel is more suitable.  the main reason to update a change with a new patchset is for continuity of review comments.20:43
VekI can't really remove it from my patch.20:43
jaypipesjeblair: gotcha.20:43
notmynametermie: https://github.com/termie/git-bzr-ng/issues/3020:43
jeblairif you do create a new change (rather than a new patchset for an existing change) that supercedes an existing one, just click "abandon change" on the old one.20:44
Vekjeblair: was just waiting to see what people said on my new submission before I did that.20:44
jeblairvek: sounds good20:44
sorenerk..20:45
Vekgot a +1 from jaypipes, so if it meets with sleepsonfloor's approval, I'll go ahead and abandon the old one.20:45
jaypipesVek: oh, gah, I thought you'd updated the original...20:46
jaypipesVek: but whatever, yeah, just abandon the original if you didn't rebase...20:46
jeblairvek: you can add sleepsonfloor as a reviewer to your patch to notify him that you want him in particular to review it20:46
Vekyeah, I haven't played with rebase yet :)20:46
Vekjeblair: good idea...20:46
sorenjk0, mtaylor: I'm not sure what you guys are on about. The client.py from the PPA absolutely has a set_auth_token method in it.20:47
mtaylorsoren: really?20:47
termienotmyname: that error is the bane of my existence20:47
jk0ah, weird20:47
sorenI'm looking at it right now.20:47
notmynametermie: :-)20:47
sorenmtaylor: It's really boring.20:47
jaypipesblamar, s1rp: if you got a chance, this MP's dependent branch is now in trunk. Please review: https://code.launchpad.net/~rackspace-titan/glance/glance-cli-filters/+merge/7019720:47
mtaylorsoren: you are right. I thought it was set_auth_context, which is what I looked for earlier20:47
mtaylorVek: ok. it should work now20:47
sorenmtaylor: What's the MP that fails?20:47
termienotmyname: it pops up over and over again depending on different installation criteria, and i can't duplicate it in any tests20:48
mtaylorsoren: and sorry for wasting your time :)20:48
jk0soren: https://code.launchpad.net/~klmitch/nova/glance-private-images/+merge/6966120:48
Vekmtaylor: OK...20:48
sorenmtaylor: Well, it made me discover another quite serious bug, so it's all good.20:48
termienotmyname: that's where my extensive test suite came from, is trying to nail down this error20:48
* Vek looks at jk0--ball's in your court20:48
notmynametermie: any more info that I can provide to help?20:48
Vekmtaylor: and yeah, I had mistyped earlier; sorry about that...I corrected myself, but didn't draw attention to all parts of the correction...20:49
jk0I'll reapprove20:49
jk0cross digits20:49
mtaylorVek: hehe20:49
mtaylorsoren: yay!20:50
*** negronjl has quit IRC20:50
termienotmyname: if you turn on debug logging in git-bzr it should dump out the contents of your mapping files, if this is for a project with a lot of commits it will be pretty huge though and it just prints them to stdout20:50
Vekjk0: all my crossable digits are crossed...20:50
notmynametermie: how do I turn on debug?20:50
termienotmyname: if it is for a relatively low (less than a few hundred) number it could be helpful though20:50
termienotmyname: i think there is a commented out line at the top or bottom of the script20:51
termiejust vim your git-bzr20:51
termiei've been trying to get bzr to fix at least the symptom of this for some time20:51
termiethey have a subclass that doesn't implement the protected method that it is trying to execute20:51
termiehttps://bugs.launchpad.net/bzr/+bug/54162620:52
sorenmtaylor: How do you ensure that the build deps are installed?20:52
uvirtbotLaunchpad bug 541626 in bzr "'BTreeBuilder' object has no attribute '_find_ancestors'" [High,In progress]20:52
mtaylorsoren: apt-get build-dep nova20:52
mtaylorsoren: driven by a puppet config which is itself driven by cron20:53
zykes-in keystone, is there any diff between groups and tenants ?20:53
dolphmzykes-: groups were recently removed from keystone20:54
*** hub_cap has quit IRC20:54
mtaylorsoren: there is a bit of a bug in there in that I do not currently having anything doing upgrade- so I think I need to install cron-apt or the like20:54
zykes-dolphm: i still see code for it in models...20:54
dolphmzykes-: link20:54
dolphmzykes-: ?20:54
mtaylorsoren: the puppet stuff is in openstack/openstack-ci-puppet20:55
zykes-oh, old pull :p20:55
sorenmtaylor: I thought these were ephemeral?20:56
dolphmzykes-: when i said recently, i meant like very recently :) lol20:56
mtaylorsoren: they _will_ be ephemeral20:56
mtaylorsoren: right now we're just working on ensuring that they can be ephemeral20:56
sorenmtaylor: So where is this being run? On jenkins.openstack.org?20:56
mtaylorsoren: so, they are ephemeral in theory, but in practice they will stick around until the jclouds plugin is finished and we can have jenkins launch and delete them for us20:56
mtaylorsoren: where is what being run?20:57
sorenThe failing tests.20:57
mtayloron the nova build slave20:57
soren...which is ephemeral?20:57
mtaylorsoren: which is currently 173.203.108.12420:57
sorenOr can I go an look at it?20:57
mtayloryes20:57
mtaylorgo look at it!20:57
mwhookershould multiple calls to nova.db.sqlalchemy.session.get_session return the same session object?20:57
mtaylorsoren: but if you need to configure or install anything there, we should do it via the puppet module20:58
mwhookerit looks like it's returning a new session each time20:58
jk0Vek / soren / mtaylor: https://code.launchpad.net/~klmitch/nova/glance-private-images/+merge/69661 failed again20:58
openstackgerritA change was merged to openstack/openstack-ci-puppet: Move all node configuration into site.pp, with node identifiers based on hostnames so that puppet self-identifies which node config to use.  https://review.openstack.org/12820:58
Vek*argh*20:59
mwhookerso I'm wondering if there's some idiom for working with ORM objects accross session objects20:59
Vekexact same failures.20:59
sorenmtaylor: >>> import glance.client20:59
soren>>> glance.client.V1Client.set_auth_token20:59
mtaylorhah!20:59
soren<unbound method V1Client.set_auth_token>20:59
zykes-is jenkins kind of the same stuff as buildboT?20:59
mtaylorfor some reason nova-tarmac is running on master20:59
mtaylorzykes-: yes20:59
mwhookerfor ex., just uncovering a bug in nova.db.sqlalchemy.api. instance_type_extra_specs_update_or_create/instance_type_extra_specs_get_item20:59
zykes-diff being ?21:00
Vekcreiht: it did, it just threw me for a loop, because I knew it wasn't in my own code.21:00
ttxMeeting starts now, please join #openstack-meeting21:00
mwhookerbesides being completely broken, instance_type_extra_specs_update_or_create is trying to operate on an ORM object returned by instance_type_extra_specs_get_item which is bound to a different session21:00
mwhookerso once you remove the "diaper" except, you get this error InvalidRequestError: Object '<InstanceTypeExtraSpecs at 0x104f43990>' is already attached to session '4378062928' (this is '4378063824')21:01
mtaylorzykes-: jenkins looks like this: https://jenkins.openstack.org21:01
mwhookerwhich speaks of a deeper design problem to me21:01
mtaylorzykes-: it's two different projects that tackled a similar problem space with different approaches21:01
mtaylorzykes-: jenkins has through-the-web config, buildbot has editable python config files21:02
zykes-ah21:02
mtayloramongst many other things21:02
mwhookerI guess the idiom is to pass around the session21:03
mtaylorzykes-: I loved buildbot until someone made me use jenkins (which was hudson at the time) haven't looked back since21:03
zykes-what's openstack ci ?21:03
*** jakedahn has joined #openstack-dev21:03
mtaylorzykes-: openstack continuous integration21:04
*** mgius has joined #openstack-dev21:04
heckjzykes-: all the guts of the jenkins, build, etc setup21:04
mtaylorzykes-: it's just a place where we've been sticking the scripts/docss we're using to run/configure things21:04
zykes-ah21:04
*** JStoker has joined #openstack-dev21:04
vishymwhooker: that is bad21:04
vishymwhooker: yeah generally if you are making a separate query you pass the session21:05
mwhookervishy: thanks. I think I found a workable solution21:06
mwhookerthough I hope there's a critical eye trained on all that sqlalchemy code21:06
openstackgerritA change was merged to openstack/openstack-ci: Use "site.pp" in launch script.  https://review.openstack.org/13021:06
Tusharsleepsonthefloor: Are you there?21:08
sleepsonthefloorHey Tushar21:08
Tusharsleepsonthefloor: I checked your github code. I have one question21:08
openstackgerritA change was merged to openstack/openstack-ci: Added script to force set project watches.  https://review.openstack.org/12621:09
Tusharsleepsonthefloor: any reason why you have rules and security group into separate controller?21:09
sleepsonthefloorTushar - it seemed restier :)21:09
Vekmaybe we need to sponge the/a virtual environment somewhere to make it pull in the right glance when running the tests?21:10
Vek^^mtaylor21:11
uvirtbotVek: Error: "^mtaylor" is not a valid command.21:11
* Vek just stares at uvirtbot21:11
sleepsonthefloorTushar - did you do authorize/revoke actions on security groups?21:11
Tusharsleepsonthefloor: Yes21:12
mtaylorVek: ok. have that part fixed. now there is another bit I've got to sort21:12
Vek'k...21:12
Tusharsleepsonthefloor: IMO, it makes sense to have just one controller to manage everything related to secuirty group. what do you think?21:13
Tusharsleepsonthefloor: *security21:13
sleepsonthefloorTushar - I was torn on this one, but felt that treating the rules as proper resources was more in the spirit of rest.  But I don't feel super strongly either way.21:14
kbringardin the not too distant future isn't a lot of that going to be outsourced to quantum?21:14
*** stiekes has quit IRC21:15
*** stiekes has joined #openstack-dev21:16
*** martine_ has quit IRC21:17
sleepsonthefloorkbringard - yes i believe so.  but we still  need an interim solution.  without some security group control it is hard to make the cloud work with the osapi.  The default in nova is to block all instance traffic.21:18
kbringardsure, of course21:18
*** stiekes_ has joined #openstack-dev21:18
vishyit is more than not too distant future as well21:19
vishythey punted on security groups initially21:19
kbringardmy point, for lack of a better word, was: if you've done it one way already and it works, is it worth it to re-factor it if it's going to be replaced in 6 months, or whatever21:19
kbringardand I don't know the answer, just tossing it out there21:19
jaypipessoren: can I assign you to that new Glance bug? not sure why that is happening. I believe the test dir is excluded in the MANIFEST.in...21:20
sorenjaypipes: I can take a peek, sure.21:21
jaypipessoren: appreciated.21:21
*** stiekes has quit IRC21:22
Tusharsleepsonthefloor: Ok, I will add rules and security group as separate resources21:22
*** jakedahn_ has joined #openstack-dev21:23
openstackgerritDolph Mathews proposed a change to openstack/keystone: Issue #115: Added support for testing multiple keystone configurations (sql-only, memcache, ldap)  https://review.openstack.org/13221:24
sleepsonthefloorTushar - ok sounds good.  thanks again21:25
*** cp16net has quit IRC21:27
openstackgerritMonty Taylor proposed a change to openstack/openstack-ci-puppet: Added tarmac launchpad authentication info.  https://review.openstack.org/13321:27
openstackgerritMonty Taylor proposed a change to openstack/openstack-ci-puppet: Removed duplicate packages. Changed manifest in cron job.  https://review.openstack.org/13421:27
*** jakedahn has quit IRC21:27
*** jakedahn_ is now known as jakedahn21:27
*** kbringard has quit IRC21:29
openstackgerritA change was merged to openstack/openstack-ci-puppet: Removed duplicate packages. Changed manifest in cron job.  https://review.openstack.org/13421:30
openstackgerritA change was merged to openstack/openstack-ci-puppet: Added tarmac launchpad authentication info.  https://review.openstack.org/13321:30
Tusharsleepsonthefloor: In EC2 API, it has provision to add/delete multiple rules  to/from a security group in single API request. but here we are restricting to add/delete just one rule at a time. Is this acceptable?21:33
openstackgerritMonty Taylor proposed a change to openstack/openstack-ci-puppet: Added launchpad login info to bazaar.conf.  https://review.openstack.org/13521:37
sleepsonthefloorTushar - I thought so, and was thinking that bulk operations could be coordinated at a higher level.21:37
openstackgerritA change was merged to openstack/openstack-ci-puppet: Added launchpad login info to bazaar.conf.  https://review.openstack.org/13521:37
sorenjaypipes: I found the problem.21:39
sorenjaypipes: But I'd actually much rather not fix it this way.. Would you be wildly opposed to moving the tests under the glance namespace?21:39
jaypipessoren: actually, no that's fine with me.21:40
jaypipessoren: nova and keystone do that, too.21:40
jaypipesdolphm: mind joining #openstack-meeting, pls?21:40
dolphmjaypipes: sure21:41
jaypipesdolphm: cheers21:41
Tusharsleeponthefloor: Ok21:41
sorenjaypipes: Wicked.21:42
sorenjaypipes: /me does that instead21:42
sorenjaypipes: In case you're interested, the problem was that you only excluded 'tests' from find_packages. It's not recursive, so you should also have excluded 'tests.*'.21:42
jaypipessoren: ah!21:43
*** yogirackspace has joined #openstack-dev21:43
*** yogirackspace has left #openstack-dev21:43
mtaylorVek: ok. I think your thing just merged21:46
mtaylorVek: there were a couple of related problems in there- but thanks, actually, because something wasn't happening that I thought WAS happening21:46
Vekthanks :)21:47
*** bsza has quit IRC21:49
openstackjenkinsProject nova build #1,182: SUCCESS in 5 min 52 sec: http://jenkins.openstack.org/job/nova/1182/21:52
openstackjenkinsTarmac: Glance has been updated for integration with keystone.  That means that nova needs to forward the user's credentials (the auth token) when it uses the glance API.  This patch, combined with a forth-coming patch for nova_auth_token.py in keystone, establishes that for nova itself and for xenapi; other hypervisors will need to set up the appropriate hooks for their use of glance.21:52
*** sandywalsh has quit IRC21:54
*** asomya has joined #openstack-dev21:58
*** mfer has quit IRC22:11
zedashey, just confirming, but is test_localizations.py supposed to be looking for strings that have formats that are not %() style?  like _("This is %s." % stuff)?22:21
zedasbecause using the ast module finds a whole bunch more _() calls, and a whole bunch of strings that will fail this test.  i think.22:22
*** mszilagyi has quit IRC22:27
zedasok i should have a rewrite for that file which just uses the ast22:27
openstackjenkinsProject nova build #1,183: SUCCESS in 3 min 25 sec: http://jenkins.openstack.org/job/nova/1183/22:30
openstackjenkinsTarmac: Add support for 300 Multiple Choice responses when no version identifier is used in the URI (or no version header is present)22:30
openstackjenkinsAlso adds support for server detail json/xml/atom requests.22:30
tr3buchetblamar: are you working on lp815957?22:31
*** medberry is now known as med_brb22:33
*** mattray has quit IRC22:34
vishyplease to review: https://code.launchpad.net/~soren/nova/source-group-filtering/+merge/6933822:38
vishyalso, any other opinions on this before i fire it off? https://code.launchpad.net/~rackspace-titan/nova/update-HACKING/+merge/6982922:38
*** med_brb is now known as med22:40
*** med is now known as medberry22:40
*** nati has joined #openstack-dev22:47
vishymtaylor: ping22:53
mtaylorvishy: I didn't do it122:53
vishymtaylor: if we need lxml for tests, we need to add it to build-deps right?22:53
mtaylorvishy: yes. to the build-deps of the nova package ... in a pinch, we can add it directly to the puppet module - but build-deps would be better22:54
vishymtaylor: mind adding it to the build-deps of the packaging branch?22:54
mtaylorvishy: of course, adding it to build-deps means that a version of the package has to actually get built with those build-deps ... so it's entirely possible that direct puppet addition is the 'right' way22:54
mtaylorvishy: not at all22:55
vishyi assume the package is python-lxml but not sure22:55
vishyit is lxml in pip22:55
mtaylorvishy: done22:56
vishythanks mtaylor, you are a sexy beast22:56
mtaylorvishy: I do what I can22:56
mtaylorvishy: if you can make sure another branch other than the one that needs lxml goes through first, it should all sort itself out22:57
vishythere is one ahead of it in the queue :)22:57
vishywe'll see what happens22:57
*** rnirmal has quit IRC23:00
*** jkoelker has quit IRC23:02
zedashow's this grab people: http://codepad.org/YTODudUt  rewrote the localization test to use the python ast module, and cleaning it up in general.23:05
openstackjenkinsProject nova build #1,184: SUCCESS in 3 min 25 sec: http://jenkins.openstack.org/job/nova/1184/23:05
openstackjenkinsTarmac: fix for lp816713: In instance creation, when nova-api is passed imageRefs generated by itself, strip the url down to an id so that default glance connection params are used23:05
zedasso then after doing all that, it seems like a lot just to check that % formats are right.  shouldn't that also actually check that those are working with gettext somehow?23:08
*** markvoelker has quit IRC23:10
zedasactually i can't see what this test really does. i mean it doesn't check the string is in gettext, doesn't check that it's format is really correct, just makes sure theres <= 1 % sign23:15
openstackjenkinsProject nova build #1,185: SUCCESS in 3 min 24 sec: http://jenkins.openstack.org/job/nova/1185/23:16
openstackjenkinsTarmac: Fix for bug #79829823:16
openstackjenkins* Removes restriction of have to have an IPv4 network to create and use an IPv6 network23:16
uvirtbotLaunchpad bug 798298 in nova "IPv6 currently requires IPv4 addresses to be present of faked out" [Medium,In progress] https://launchpad.net/bugs/79829823:16
openstackjenkins* Modifies nova-manage network to require either an IPv4 or IPv6 network be passed23:16
openstackjenkins* Modifies nova-manage network to ignore FLAGS.use_ipv6 (what is passed on the command line is what you get)23:16
openstackjenkins* Modifies nova-manage network to list IPv4 and IPv6 on list, no longer lists IPv4 netmask as it is implied by the cidr for the network23:16
openstackjenkins* Modifies network/manage.py to allow for empty IPv4 assignments.23:16
*** asomya has quit IRC23:17
*** nati has quit IRC23:32
*** hub_cap has joined #openstack-dev23:33
openstackjenkinsProject nova build #1,186: SUCCESS in 5 min 37 sec: http://jenkins.openstack.org/job/nova/1186/23:38
openstackjenkinsTarmac: Fixes lp81952323:38
openstackjenkinsOS API Migrate is now an admin-only method23:38
*** deshantm has joined #openstack-dev23:54
*** rods has quit IRC23:58

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