*** martine_ has joined #openstack-dev | 00:11 | |
*** jdurgin has quit IRC | 00:11 | |
*** medberry is now known as med_out | 00:13 | |
*** mszilagyi_ has joined #openstack-dev | 00:16 | |
*** mszilagyi has quit IRC | 00:18 | |
*** martine_ has quit IRC | 00:27 | |
*** heckj has quit IRC | 00:30 | |
*** rods has quit IRC | 00:57 | |
*** chomping has joined #openstack-dev | 01:00 | |
*** martine_ has joined #openstack-dev | 02:56 | |
*** thickski_ has joined #openstack-dev | 03:40 | |
*** thickskin has quit IRC | 03:41 | |
*** martine_ has quit IRC | 04:03 | |
*** cp16net has joined #openstack-dev | 04:33 | |
*** chomping has quit IRC | 04:50 | |
*** zaitcev has quit IRC | 04:55 | |
*** RobertLaptop has joined #openstack-dev | 05:16 | |
*** cp16net has quit IRC | 06:28 | |
*** reidrac has joined #openstack-dev | 07:11 | |
*** mnour has quit IRC | 07:13 | |
*** kennett85 has joined #openstack-dev | 07:16 | |
kennett85 | I'm trying to do a virtual tour but I can't get it to work. | 07:17 |
---|---|---|
kennett85 | :http://dojotoolkit.org | 07:17 |
kennett85 | sorry here is my code. http://pastebin.com/RspU53NG | 07:17 |
*** AhmedSoliman has joined #openstack-dev | 07:46 | |
*** duker has quit IRC | 07:54 | |
*** tudamp has joined #openstack-dev | 08:05 | |
*** darraghb has joined #openstack-dev | 08:11 | |
*** kennett85 has quit IRC | 08:22 | |
*** mnour has joined #openstack-dev | 08:22 | |
*** mnour has quit IRC | 08:23 | |
*** mnour1 has joined #openstack-dev | 08:23 | |
*** RobertLaptop has quit IRC | 08:31 | |
*** RobertLaptop has joined #openstack-dev | 08:35 | |
*** alekibango has joined #openstack-dev | 09:11 | |
*** alekibango has quit IRC | 09:14 | |
*** mnour1 has quit IRC | 09:14 | |
*** mnour has joined #openstack-dev | 09:14 | |
*** alekibango has joined #openstack-dev | 09:15 | |
*** mnour has quit IRC | 09:19 | |
*** mnour has joined #openstack-dev | 09:20 | |
*** mnour has quit IRC | 09:28 | |
*** mnour1 has joined #openstack-dev | 09:28 | |
*** mnour1 has quit IRC | 09:33 | |
*** mnour has joined #openstack-dev | 09:33 | |
*** RobertLaptop has quit IRC | 09:41 | |
*** mnour has quit IRC | 09:42 | |
*** mnour has joined #openstack-dev | 09:42 | |
*** mnour has quit IRC | 09:50 | |
*** mnour has joined #openstack-dev | 09:50 | |
*** RobertLaptop has joined #openstack-dev | 09:54 | |
*** mnour has quit IRC | 09:55 | |
*** mnour1 has joined #openstack-dev | 09:55 | |
*** mnour1 has quit IRC | 09:56 | |
*** mnour has joined #openstack-dev | 09:56 | |
*** mnour has quit IRC | 09:59 | |
*** mnour has joined #openstack-dev | 09:59 | |
*** AhmedSoliman has quit IRC | 10:00 | |
*** mnour has quit IRC | 10:00 | |
*** mnour has joined #openstack-dev | 10:00 | |
*** mnour has quit IRC | 10:02 | |
*** mnour has joined #openstack-dev | 10:05 | |
*** throughnothing has quit IRC | 10:05 | |
*** throughnothing has joined #openstack-dev | 10:06 | |
*** mnour has quit IRC | 10:35 | |
*** mnour has joined #openstack-dev | 10:35 | |
*** tudamp has quit IRC | 10:40 | |
*** rods has joined #openstack-dev | 10:47 | |
*** JStoker has quit IRC | 10:49 | |
*** tudamp has joined #openstack-dev | 10:59 | |
*** yamahata_ has quit IRC | 10:59 | |
*** yamahata__ has quit IRC | 10:59 | |
*** markvoelker has joined #openstack-dev | 11:19 | |
openstackjenkins | Project nova build #1,175: SUCCESS in 4 min 18 sec: http://jenkins.openstack.org/job/nova/1175/ | 11:21 |
openstackjenkins | Tarmac: 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 IRC | 11:23 | |
*** mnour1 has joined #openstack-dev | 11:23 | |
openstackjenkins | Project nova build #1,176: SUCCESS in 4 min 34 sec: http://jenkins.openstack.org/job/nova/1176/ | 11:26 |
openstackjenkins | Tarmac: Better error handling for resizing. | 11:26 |
*** mnour has joined #openstack-dev | 11:27 | |
*** mnour1 has quit IRC | 11:28 | |
openstackjenkins | Project 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 |
openstackjenkins | Project nova build #1,178: SUCCESS in 4 min 23 sec: http://jenkins.openstack.org/job/nova/1178/ | 11:43 |
*** martine_ has joined #openstack-dev | 11:47 | |
*** mnour has quit IRC | 11:48 | |
*** mnour1 has joined #openstack-dev | 11:48 | |
*** mfer has joined #openstack-dev | 11:48 | |
*** mnour1 has quit IRC | 11:57 | |
*** mnour has joined #openstack-dev | 11:57 | |
*** jeremyb has quit IRC | 12:02 | |
*** jeremyb has joined #openstack-dev | 12:02 | |
sandywalsh | sorry vishy, was a civic holiday here yesterday ... ping back at ya | 12:10 |
*** mnour has quit IRC | 12:17 | |
*** mnour has joined #openstack-dev | 12:17 | |
*** mnour has quit IRC | 12:19 | |
*** mnour has joined #openstack-dev | 12:19 | |
*** yamahata has joined #openstack-dev | 12:22 | |
jaypipes | sandywalsh: damn Canadians with their civic holidays. | 12:27 |
jaypipes | yamahata: ping | 12:28 |
yamahata | jaypipes, pong | 12:28 |
sandywalsh | jaypipes ... hey Don Beret! Long time! | 12:28 |
jaypipes | yamahata: sorry, I totally messed up on that meeting request. I wrote one time, but the link said another :( | 12:28 |
jaypipes | sandywalsh: :) | 12:29 |
*** mnour has quit IRC | 12:29 | |
*** mnour has joined #openstack-dev | 12:29 | |
yamahata | jaypipes, that's okay. Let's try again tomorrow. | 12:29 |
jaypipes | yamahata: you sure? 9pm is late :( (though 7am is early, I suppose...) | 12:30 |
jaypipes | yamahata: never easy to get a good time... | 12:30 |
yamahata | jaypipes, 9pm is okay for me. Yes, it's really difficult. | 12:31 |
jaypipes | yamahata: 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 |
yamahata | jaypipes, thank you for arranging the meeting. | 12:32 |
jaypipes | no prob | 12:33 |
*** mnour has quit IRC | 12:33 | |
*** mnour has joined #openstack-dev | 12:34 | |
*** martine_ has quit IRC | 12:42 | |
*** mnour has quit IRC | 12:52 | |
*** mnour has joined #openstack-dev | 12:52 | |
*** bsza has joined #openstack-dev | 12:55 | |
*** yamahata has quit IRC | 12:56 | |
*** bcwaldon has joined #openstack-dev | 12:58 | |
*** ameade has joined #openstack-dev | 13:03 | |
sandywalsh | :q | 13:03 |
sandywalsh | :/ | 13:04 |
*** mnour has quit IRC | 13:15 | |
*** mnour has joined #openstack-dev | 13:15 | |
*** martine_ has joined #openstack-dev | 13:21 | |
*** bcwaldon has quit IRC | 13:34 | |
*** bcwaldon has joined #openstack-dev | 13:36 | |
*** zul has quit IRC | 13:38 | |
*** mnour has quit IRC | 13:40 | |
*** mnour1 has joined #openstack-dev | 13:40 | |
*** mnour has joined #openstack-dev | 13:40 | |
*** zul has joined #openstack-dev | 13:40 | |
*** mnour has quit IRC | 13:41 | |
*** mnour1 has joined #openstack-dev | 13:41 | |
*** mnour has joined #openstack-dev | 13:41 | |
*** kbringard has joined #openstack-dev | 13:41 | |
*** mnour has quit IRC | 13:42 | |
*** mnour has joined #openstack-dev | 13:42 | |
*** zul has quit IRC | 13:42 | |
*** mnour has quit IRC | 13:42 | |
*** mnour1 has joined #openstack-dev | 13:42 | |
*** zul has joined #openstack-dev | 13:43 | |
*** mnour has joined #openstack-dev | 13:44 | |
*** mnour2 has joined #openstack-dev | 13:45 | |
*** mnour has joined #openstack-dev | 13:45 | |
*** mnour has quit IRC | 13:46 | |
*** mnour2 has joined #openstack-dev | 13:46 | |
*** mnour1 has quit IRC | 13:47 | |
*** mnour has joined #openstack-dev | 13:47 | |
*** mnour has quit IRC | 13:48 | |
*** mnour has joined #openstack-dev | 13:48 | |
*** mnour1 has joined #openstack-dev | 13:50 | |
*** mnour1 has quit IRC | 13:50 | |
*** mnour has joined #openstack-dev | 13:50 | |
*** zul has quit IRC | 13:54 | |
*** cp16net has joined #openstack-dev | 13:55 | |
*** amccabe has joined #openstack-dev | 13:55 | |
*** zul has joined #openstack-dev | 14:02 | |
*** kbringard has quit IRC | 14:03 | |
*** kbringard has joined #openstack-dev | 14:03 | |
*** statik has joined #openstack-dev | 14:08 | |
*** yamahata has joined #openstack-dev | 14:08 | |
*** yamahata has quit IRC | 14:09 | |
bcwaldon | vishy: 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/69865 | 14:25 |
*** amccabe has quit IRC | 14:28 | |
*** jkoelker has joined #openstack-dev | 14:28 | |
jaypipes | bcwaldon, 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 |
jaypipes | bcwaldon, jkoelker: still trying to figure out a good way to functionally test the delayed delete stuff... | 14:39 |
*** amccabe has joined #openstack-dev | 14:43 | |
*** stiekes has joined #openstack-dev | 14:44 | |
jkoelker | jaypipes: where is it blowing up at? During packaging again? | 14:49 |
jaypipes | jkoelker: 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 |
jaypipes | jkoelker: unfortunately, none of us can reproduce it locally :( | 14:50 |
jkoelker | hrm, 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 branch | 14:53 |
*** dragondm has joined #openstack-dev | 14:59 | |
jaypipes | jkoelker: no, it's the proposed branch. python setup.py build... | 15:00 |
jkoelker | roger | 15:01 |
bcwaldon | mtaylor: ping | 15:02 |
*** mattray has joined #openstack-dev | 15:05 | |
*** dprince has joined #openstack-dev | 15:07 | |
*** reidrac has quit IRC | 15:08 | |
*** heckj has joined #openstack-dev | 15:09 | |
*** mdomsch has joined #openstack-dev | 15:13 | |
dprince | soren, mtaylor: Can you guys merge this one: https://code.launchpad.net/~dan-prince/glance/ubuntu-glance-cache2/+merge/70019 | 15:26 |
mtaylor | dprince: yes. will do in a few minutes | 15:44 |
mtaylor | bcwaldon: whazzup? | 15:44 |
bcwaldon | mtaylor: 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-dev | 15:47 | |
mtaylor | bcwaldon: ooh! potentially another victim... | 15:47 |
bcwaldon | mtaylor: whaaaaat? | 15:47 |
mtaylor | bcwaldon: one sec, and I'll innundate you with a bunch of thoughts | 15:48 |
ttx | bcwaldon: escape while you still can | 15:51 |
bcwaldon | ttx: well I don't really want to get up... | 15:51 |
mtaylor | bwahahaha | 15:52 |
mtaylor | bcwaldon: 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 project | 15:53 |
bcwaldon | mtaylor: I wouldn't be against that | 15:56 |
mtaylor | dprince: 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 |
mtaylor | bcwaldon: great! | 15:56 |
bcwaldon | mtaylor: 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 |
mtaylor | bcwaldon: well, right now it's a sort of half and half thing and is in transition | 15:57 |
mtaylor | bcwaldon: the org is eventually going to be there so that it's easy to find the official "trunk" repos for each project | 15:58 |
mtaylor | bcwaldon: 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 gerrit | 15:59 |
bcwaldon | mtaylor: ok, once I have the code ready can we get it all set up in gerrit? | 16:01 |
mtaylor | bcwaldon: totally. it's really just more of a coordination thing ... I'll have some info for you real-soon-now | 16:02 |
dprince | mtaylor: Sure. Thanks man. I'll add my commit message next time. | 16:02 |
bcwaldon | mtaylor: sounds good | 16:02 |
*** tudamp has left #openstack-dev | 16:05 | |
mtaylor | dprince: sweet. you are merged/pushed | 16:06 |
*** mnour has joined #openstack-dev | 16:06 | |
mtaylor | bcwaldon: can we hold off for a few days? we're still sorting a few final things | 16:10 |
*** mdomsch has quit IRC | 16:12 | |
*** mnour has quit IRC | 16:15 | |
jaypipes | blamar, 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 |
blamar | jaypipes: Whatever you say boss :) Just as long as it works! | 16:20 |
jaypipes | jkoelker: and sorry, I forgot to include you in the follow-up email. :( | 16:20 |
jaypipes | blamar: k. | 16:20 |
jaypipes | blamar: but please don't call me boss :) | 16:20 |
blamar | haus? | 16:21 |
jkoelker | jkoelker: works for me | 16:23 |
*** mnour has joined #openstack-dev | 16:23 | |
openstackgerrit | Kevin L. Mitchell proposed a change to openstack/keystone: Pass auth_token to nova's RequestContext, for later delegation https://review.openstack.org/122 | 16:24 |
jaypipes | blamar: :) | 16:25 |
jaypipes | jkoelker: talkin' to yourself again? ;P | 16:25 |
jkoelker | Sorry, its a bad habbit | 16:26 |
jaypipes | hehe | 16:26 |
pvo | jaypipes: this is the novaclient move? | 16:26 |
jaypipes | pvo: Glance. | 16:26 |
pvo | jaypipes: cool. | 16:26 |
jaypipes | pvo: :) | 16:26 |
jaypipes | pvo: use Glancers are jumping in the pool. | 16:26 |
jaypipes | s/use/us | 16:26 |
jaypipes | pvo: I'll let you know if the water is freezing or not ;) | 16:27 |
pvo | hopefully no shrinkage | 16:27 |
jaypipes | pvo: lol | 16:27 |
jaypipes | blamar, s1rp, jkoelker, Vek: if you have a spare 15 minutes... https://code.launchpad.net/~jaypipes/glance/refactor-stores/+merge/70096 | 16:28 |
blamar | jaypipes: wrapping your strings in parenthesis in preparation for i18n? (line 72) | 16:29 |
openstackjenkins | Project nova build #1,179: SUCCESS in 4 min 41 sec: http://jenkins.openstack.org/job/nova/1179/ | 16:31 |
openstackjenkins | Tarmac: Removing the xenapi_image_service flag in favor of image_service. | 16:31 |
*** jmckenty_ has joined #openstack-dev | 16:31 | |
jaypipes | blamar: yup. | 16:31 |
*** mnour has quit IRC | 16:42 | |
*** mnour has joined #openstack-dev | 16:46 | |
openstackgerrit | Monty Taylor proposed a change to openstack/openstack-ci: Instructions for running the git-bzr-ng migration. https://review.openstack.org/123 | 16:47 |
jaypipes | jkoelker: thought that ImageAddResult would slip by people's reviews... :) | 16:48 |
jaypipes | jkoelker: but no, you just had to go and notice it ;) | 16:48 |
heckj | GerritWorkflow 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 |
heckj | trying to do a review on https://review.openstack.org/123 | 16:53 |
mtaylor | heckj: nope. click the "review" button | 16:53 |
heckj | mtalylor: Ahhh! Thanks | 16:53 |
*** mattray1 has joined #openstack-dev | 16:53 | |
mtaylor | heckj: although that certainly explains how you wound up on the reviewers list :) I was starting to worry | 16:53 |
openstackgerrit | A change was merged to openstack/openstack-ci: Instructions for running the git-bzr-ng migration. https://review.openstack.org/123 | 16:55 |
*** mattray has quit IRC | 16:57 | |
heckj | mtaylor: What's the difference between the check mark and the +1 at https://review.openstack.org/#change,123 | 16:59 |
*** jdurgin has joined #openstack-dev | 17:00 | |
*** mattray1 has quit IRC | 17:00 | |
mtaylor | heckj: 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-dev | 17:01 | |
heckj | mtaylor: Got it - so the same actions I took is what ended up with the check mark - just that james blair did it. | 17:01 |
mtaylor | heckj: 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 +1 | 17:02 |
mtaylor | and when he did that, it made the change Approved, and then jenkins tested it, which allowed gerrit to merge it | 17:03 |
heckj | mtaylor: 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-dev | 17:12 | |
jeblair | heckj: it's pulled from launchpad, and should be kept closely in sync (starting real soon now) | 17:13 |
heckj | Ah - makes sense | 17:13 |
jaypipes | blamar, 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-dev | 17:15 | |
*** darraghb has quit IRC | 17:19 | |
chemikadze | Hi everyone! Does nova console client support keystone auth? | 17:20 |
chemikadze | Last time i looked on it all failed with redirect error. | 17:22 |
westmaas | lp809663 | 17:27 |
westmaas | awww that didn't work | 17:27 |
westmaas | https://bugs.launchpad.net/glance/+bug/809663 | 17:27 |
uvirtbot | Launchpad bug 809663 in glance "Glance milestone and integrated packages should automatically be uploaded to PyPI" [High,Confirmed] | 17:27 |
westmaas | mtaylor: anything we can do to move this guy along? or is this not you? | 17:28 |
mtaylor | westmaas: it is me, and I'm going to get it done today | 17:29 |
mtaylor | westmaas: also, bug 809663 | 17:29 |
uvirtbot | Launchpad bug 809663 in glance "Glance milestone and integrated packages should automatically be uploaded to PyPI" [High,Confirmed] https://launchpad.net/bugs/809663 | 17:29 |
mtaylor | should trigger uvirtbot | 17:29 |
westmaas | mtaylor: cool thanks | 17:31 |
*** dolphm has quit IRC | 17:31 | |
westmaas | jaypipes: 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 |
jaypipes | westmaas: yep, lemme do that right now. | 17:35 |
jk0 | thanks | 17:35 |
westmaas | ty sir | 17:35 |
jaypipes | westmaas: the D3 milestone release, right? | 17:35 |
westmaas | do you know if Vek had his keystone integration in before then? | 17:36 |
westmaas | Vek: ^^ | 17:36 |
jaypipes | westmaas: yes, he did. not the functional tests, but the basic context and middleware, yes. | 17:37 |
westmaas | then yeah that's what we should need | 17:37 |
westmaas | assuming no vital bug fixes! | 17:37 |
jaypipes | westmaas: there are a few URI parsing fixes that went in afterwards... | 17:39 |
bcwaldon | jaypipes, westmaas: Let's just go with trunk | 17:40 |
vishy | sandywalsh: https://blueprints.launchpad.net/nova/+spec/different-schedulers-per-call | 17:40 |
bcwaldon | assuming you trust refactor-stores ;) | 17:40 |
jaypipes | westmaas, bcwaldon: OK, one thing to note if I do a trunk package to PyPI... | 17:40 |
*** dolphm has joined #openstack-dev | 17:41 | |
jaypipes | westmaas, bcwaldon: anyone who upgrades is likely to run into the issue this guy ran into: https://bugs.launchpad.net/bugs/814981 | 17:41 |
uvirtbot | Launchpad bug 814981 in openstack-live "glance-api fails on image delivery: AttributeError: context" [Medium,Confirmed] | 17:41 |
sandywalsh | vishy, looking | 17:41 |
westmaas | until they update the glance conf? | 17:41 |
Vek | exactly. | 17:41 |
jaypipes | westmaas: right. | 17:41 |
* westmaas hasn't actually looked yet | 17:41 | |
westmaas | yeah | 17:41 |
westmaas | hrm. | 17:41 |
jaypipes | westmaas: we can deal with it, but it's an issue... | 17:42 |
jk0 | that's something they would run into eventually anyway | 17:42 |
jaypipes | jk0: yup. | 17:42 |
westmaas | same for D3 though? | 17:42 |
jaypipes | westmaas: yes. | 17:42 |
jaypipes | westmaas: but there were a couple fixes after D3 that are useful... | 17:42 |
jaypipes | westmaas: so might as well do a trunk I guess | 17:42 |
westmaas | yeah | 17:42 |
westmaas | that's what I'm thinking | 17:42 |
jaypipes | westmaas: k. gimme 10. | 17:42 |
vishy | sandywalsh: I was wondering if you wanted to tackle that one. Seems like you could bang it out faster than anyone else. | 17:43 |
vishy | sandywalsh: 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 |
sandywalsh | vishy, wouldn't be hard, but I'm not sure the existing scheduler framework couldn't handle it | 17:44 |
dprince | jaypipes, westmaas: If we get the paste stuff out of the conf we'd be better off: https://bugs.launchpad.net/glance/+bug/815208 | 17:44 |
uvirtbot | Launchpad bug 815208 in glance "remove paste configuration from the config files" [Low,Triaged] | 17:44 |
vishy | sandywalsh: 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 |
dprince | jaypipes: Sorry. I see you already commented. Anyway. Something nice to have to head off issues. | 17:45 |
sandywalsh | vishy, 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 refactoring | 17:45 |
*** dolphm has quit IRC | 17:46 | |
*** dolphm has joined #openstack-dev | 17:46 | |
jaypipes | westmaas, jk0: latest on PyPI now. should be glance-2011.3.tar.gaz | 17:47 |
jk0 | thanks man | 17:47 |
*** dolphm has joined #openstack-dev | 17:47 | |
jaypipes | np | 17:47 |
vishy | sandywalsh: 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 one | 17:47 |
vishy | 17:47 | |
westmaas | jaypipes: ty sir | 17:47 |
jaypipes | westmaas: welcome sir | 17:47 |
vishy | sandywalsh: do you have time to do it? Or should I find someone else? | 17:47 |
sandywalsh | vishy lemme check and see what the priority is ... sec | 17:48 |
vishy | sandywalsh: 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 |
sandywalsh | vishy I know Ozone is priming for alpha, lemme confirm with the higher powers first | 17:50 |
sandywalsh | (just waiting to hear back now) | 17:50 |
openstackgerrit | Monty Taylor proposed a change to openstack/openstack-ci: Added cache flushing and db backups. https://review.openstack.org/124 | 17:54 |
sandywalsh | vishy how would you specify which scheduler to use without busting the public API? | 17:55 |
sandywalsh | vishy or is it --scheduler_driver becomes --scheduler_driver_compute/--scheduler_driver_volume/etc. ? | 17:56 |
jaypipes | Glancers: w00t. the Jenkins job no longer seems to be blocking on random server startup failures... unclogging our review queue. | 17:57 |
jaypipes | Glancers: 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 |
bcwaldon | jaypipes: how do you feel about this bug: https://bugs.launchpad.net/glance/+bug/819938 | 18:00 |
uvirtbot | Launchpad bug 819938 in glance "setup.py needs to define dependencies" [Undecided,New] | 18:00 |
bcwaldon | jaypipes: Seems like such a simple thing, I wonder how we got this far | 18:01 |
jaypipes | bcwaldon: I will defer to mtaylor on that one. I'm used to the debian control way of managing deps... | 18:01 |
bcwaldon | jaypipes: or maybe I'm wrong | 18:01 |
bcwaldon | jaypipes: right, but when you 'pip install' something, its platform independent | 18:02 |
jaypipes | bcwaldon: understood. I defer to mtaylor :) | 18:02 |
bcwaldon | jaypipes: sounds good :) | 18:02 |
mtaylor | bcwaldon: 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 |
bcwaldon | mtaylor: not surprised, but then it makes our glance pip-requires file bleed over into our nova pip-requires file | 18:03 |
mtaylor | bcwaldon: in any case, I believe I had that going in nova for like a day before someone made me remove it :) | 18:03 |
bcwaldon | mtaylor: Ha, well I feel like you should be able to pip-install glance and not have to track down every dependency | 18:03 |
mtaylor | bcwaldon: 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 deps | 18:03 |
bcwaldon | mtaylor: maybe glance doesn't belong in pypi? | 18:04 |
mtaylor | bcwaldon: yes. I agree with that. I believe if you are using pip and you pip install glance, it should work | 18:04 |
mtaylor | bcwaldon: it TOTALLY belongs in PyPI ... we just run the builders using packages because that's the harder thing | 18:04 |
bcwaldon | mtaylor: I feel like pip is geared more towards libraries, while glance is an application | 18:04 |
mtaylor | bcwaldon: it's really easy to add a bunch of deps from pypi that aren't in distros | 18:04 |
bcwaldon | mtaylor: ok, so do we have a clear path forward on fixing the issue at hand? | 18:05 |
mtaylor | bcwaldon: yes. pip itself is more geared towards libraries, but it still should work, really | 18:05 |
mtaylor | bcwaldon: nope. :) | 18:05 |
bcwaldon | son of a... | 18:05 |
bcwaldon | mtaylor: I'm really curious what problem you ran into before, it seems like such a simple thing to me | 18:06 |
mtaylor | bcwaldon: 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 pip | 18:06 |
bcwaldon | jay actually just did that | 18:07 |
bcwaldon | jaypipes: ^^ | 18:07 |
bcwaldon | or maybe not... | 18:07 |
bcwaldon | he did for glance, are you talking about a new builder for nova? | 18:07 |
bcwaldon | mtaylor: ^ | 18:08 |
mtaylor | he's talking about a param-build builder that allows you to run something in the context of jenkins | 18:08 |
mtaylor | I'm talking about builders that are part of the trunk gating | 18:08 |
bcwaldon | mtaylor: gotcah | 18:08 |
bcwaldon | gotcha* | 18:09 |
mtaylor | so that we make sure that both pip-requires and apt-get depends are solid before we merge something | 18:09 |
bcwaldon | mtaylor: that sounds great | 18:09 |
mtaylor | also ... 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 fail | 18:09 |
*** jmckenty_ has quit IRC | 18:10 | |
*** glenc_ has joined #openstack-dev | 18:10 | |
*** glenc has quit IRC | 18:12 | |
mtaylor | bcwaldon: this will all be easier to tackle after we transition on thursday | 18:15 |
bcwaldon | mtaylor: glance transition is officially happening thursday? | 18:15 |
mtaylor | bcwaldon: yeah. | 18:17 |
bcwaldon | mtaylor: Okay, that'll work for me. I'm adding the same xattr dep into nova pip-requires for now | 18:18 |
mtaylor | bcwaldon: at that point, it actually becomes WAY easier to set up multiple conditions/jobs that get tested | 18:18 |
mtaylor | bcwaldon: 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 |
bcwaldon | mtaylor: okay, I'm with you 100% | 18:19 |
*** dprince has quit IRC | 18:23 | |
bcwaldon | mtaylor: packaging needs to be updated for https://code.launchpad.net/~rackspace-titan/nova/remove-twistd/+merge/69865 once it gets merged | 18:28 |
bcwaldon | mtaylor: can I help out, or do you want to take care of it | 18:28 |
bcwaldon | I 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/69829 | 18:35 |
mtaylor | bcwaldon: if twisted is gone with that patch - that means that python setup.py test should work in nova now, yeah? | 18:37 |
sandywalsh | jaypipes, you seeing this on keystone unit tests? http://paste.openstack.org/show/2006/ | 18:38 |
mtaylor | bcwaldon: or, more specifically, that nosetest should work directly rather than needing the twisted additions in run_test.py | 18:38 |
mtaylor | ? | 18:38 |
dolphm | sandywalsh: delete 'keystone.token.db' from the cwd as well as 'keystone.db' | 18:39 |
sandywalsh | ah, thanks dolphm | 18:39 |
dolphm | sandywalsh: a revised test script is in the works to fix that issue | 18:40 |
sandywalsh | dolphm, ah, good to know | 18:40 |
mwhooker | hi 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 |
jk0 | pylint is always a great place to start for that kind of stuff | 18:42 |
jk0 | (I'm sure there is a lot that could be done with that) | 18:42 |
mwhooker | jk0: I've done a pass at pylint, I'll check for more but there may not be much left | 18:42 |
ameade | yeah most of them are the "blank does not have blank member" | 18:43 |
mwhooker | and most of those are false positives | 18:43 |
ameade | I'm sure there are a ton of goodies still hidden in there | 18:43 |
ameade | yeah | 18:43 |
jk0 | mwhooker: have a look at this: https://jenkins.openstack.org/job/nova-pylint/violations/? | 18:43 |
openstackgerrit | Monty Taylor proposed a change to openstack/openstack-ci: Ensured that projects are in a valid set. https://review.openstack.org/125 | 18:43 |
ameade | mwhhoker, jk0: I usually use this job; https://jenkins.openstack.org/job/nova-pylint-errors/violations/ | 18:44 |
mwhooker | good resources. I'll check both those out | 18:45 |
mwhooker | thanks | 18:45 |
* mtaylor == ++ for cleaning pylint | 18:45 | |
ameade | :) | 18:45 |
bcwaldon | mtaylor: not exactly sure, I'm not 100% on top of the venv stuff | 18:45 |
mtaylor | ok | 18:45 |
bcwaldon | mtaylor: the packaging differences are Twisted and some removed files. I know nova-instancemonitor was listed in the package script | 18:45 |
openstackjenkins | Project nova build #1,180: SUCCESS in 4 min 30 sec: http://jenkins.openstack.org/job/nova/1180/ | 18:46 |
openstackjenkins | Tarmac: - Remove Twisted dependency from pip-requires | 18:46 |
openstackjenkins | - Remove Twisted patch from tools/install_venv.py | 18:46 |
openstackjenkins | - Remove eventlet patch from tools/install_venv.py | 18:46 |
openstackjenkins | - Remove tools/eventlet-patch | 18:46 |
openstackjenkins | - Remove nova/twistd.py | 18:46 |
openstackjenkins | - Remove nova/tests/test_twistd.py | 18:46 |
openstackjenkins | - Remove bin/nova-instancemonitor | 18:46 |
openstackjenkins | - Remove nova/compute/monitor.py | 18:46 |
openstackjenkins | - Add xattr to pip-requires until glance setup.py installs it correctly | 18:46 |
openstackjenkins | - Remove references to removed files from docs/translations/code | 18:46 |
*** zaitcev has joined #openstack-dev | 18:49 | |
ttx | vishy: ping | 18:51 |
*** glenc has joined #openstack-dev | 18:52 | |
*** glenc_ has quit IRC | 18:54 | |
jaypipes | mtaylor: sorry, stepped away for a bit.. | 18:56 |
jaypipes | mtaylor: no, I just added the setup.py test one because that's where we were seeing issues... | 18:56 |
mtaylor | jaypipes: k | 18:56 |
jaypipes | sandywalsh: no, haven't seen those. | 18:56 |
sandywalsh | jaypipes, k, resolved by nuking all the .db's | 18:58 |
*** hub_cap has joined #openstack-dev | 18:58 | |
jaypipes | sandywalsh: 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 |
sandywalsh | nice ... a nose cleaner :) | 18:59 |
dolphm | jaypipes: looking forward to that! i'm working on testing multiple backends from run_tests.py | 18:59 |
*** zedas has joined #openstack-dev | 19:00 | |
zedas | hey folks | 19:00 |
jaypipes | dolphm: should be done today. got most of it done. | 19:00 |
jaypipes | sandywalsh: :) | 19:00 |
zedas | hey 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 |
zedas | looking at this list, nova/twistd.py isn't being used, and nova/tests/test_twistd.py too. can we just delete those? | 19:01 |
jaypipes | Vek, bcwaldon, mtaylor: so... it looks like the nova jenkins slave may need to have the latest glance package installed, right? | 19:02 |
Vek | it would appear so. | 19:03 |
Vek | ironically, it wouldn't have been a problem if I hadn't reverted the 'hasattr' test on 'set_auth_token' :) | 19:03 |
jaypipes | Jenkins is a very busy girl today. | 19:03 |
mtaylor | jaypipes: should happen natually if the glance package gets uploaded to the ppa | 19:03 |
jaypipes | Vek: heh, indeed. | 19:03 |
jaypipes | mtaylor: not sure. I uploaded to PyPI, not the PPA. I thought the PPA stuff happened automagically, no? the glance-ppa jenkins job? | 19:04 |
mtaylor | jaypipes: yes. it should happen automagically - and then the jenkins slave _should_ automagically ... oh, wait. hrm | 19:04 |
mtaylor | ok. looking | 19:05 |
sandywalsh | zedas, 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 |
westmaas | zedas: last commit was actually to remove those | 19:05 |
westmaas | https://code.launchpad.net/~rackspace-titan/nova/remove-twistd/+merge/69865 | 19:05 |
zedas | westmaas: odd i just did a pull. lemme pull again... | 19:05 |
westmaas | 35 mins ago! :) | 19:05 |
jaypipes | sandywalsh: instance-monitor | 19:05 |
sandywalsh | ah | 19:05 |
zedas | jaypipes: yeah instant monitor is using twisted, but not nova.twstid | 19:05 |
Vek | (that spelling is rather twisted) | 19:06 |
jaypipes | zedas: latest patch removed instance monitor IIRC | 19:06 |
jaypipes | lol | 19:06 |
zedas | jaypipes: ha yeah, 35 minutes too late. | 19:06 |
zedas | alright, lemme find some more culprits... | 19:06 |
jaypipes | zedas: :) | 19:06 |
openstackgerrit | A change was merged to openstack/openstack-ci: Ensured that projects are in a valid set. https://review.openstack.org/125 | 19:10 |
openstackgerrit | A change was merged to openstack/openstack-ci: Added cache flushing and db backups. https://review.openstack.org/124 | 19:10 |
openstackjenkins | Project nova build #1,181: SUCCESS in 6 min 29 sec: http://jenkins.openstack.org/job/nova/1181/ | 19:13 |
openstackjenkins | Tarmac: Moves image creation from POST /images to POST /servers/<id>/action | 19:13 |
zedas | ok, 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 |
zedas | so, would like to either remove those or make them actually do something. any thoughts? | 19:16 |
zedas | and 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 |
zedas | ahh there we go, those two files are used in test_flags.py | 19:23 |
*** ameade has quit IRC | 19:28 | |
ttx | Meeting in #openstack-meeting in 90 minutes. | 19:30 |
openstackgerrit | Monty Taylor proposed a change to openstack/openstack-ci: Added script to force set project watches. https://review.openstack.org/126 | 19:34 |
Vek | jaypipes/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 |
openstackgerrit | Vish Ishaya proposed a change to openstack/keystone: Adds support for authenticating via ec2 signatures https://review.openstack.org/127 | 19:42 |
*** hazmat has left #openstack-dev | 19:44 | |
bcwaldon | soren: 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 |
notmyname | termie: does git-bzr work for you in Lion? seems that I can't push new branches to LP | 19:46 |
*** Tv_ has quit IRC | 19:47 | |
mtaylor | Vek: ok. glance should be updated now | 19:50 |
*** RobertLaptop has quit IRC | 19:53 | |
*** bcwaldon has quit IRC | 19:53 | |
openstackgerrit | James 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/128 | 19:59 |
openstackgerrit | sleepsonthefloor 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/129 | 20:03 |
openstackgerrit | James E. Blair proposed a change to openstack/openstack-ci: Use "site.pp" in launch script. https://review.openstack.org/130 | 20:05 |
Vek | mtaylor: Thanks... | 20:05 |
*** med_out is now known as med | 20:07 | |
*** med is now known as medberry | 20:07 | |
Vek | mtaylor: doesn't appear that jenkins picked up the right version of glance | 20:08 |
mtaylor | Vek: hrm | 20:08 |
jkoelker | jaypipes: I got glance to build successfully under pdebuild | 20:08 |
jkoelker | http://paste.openstack.org/show/2013/ | 20:08 |
mtaylor | Vek: 2011.3~d4~20110802.176-0ubuntu0ppa1~natty1 from http://ppa.launchpad.net/nova-core/trunk/ubuntu/ | 20:08 |
Vek | ah, this version of glance may not yet be in the ppa | 20:09 |
jkoelker | had to make those changes to the build repo though, it appears it was building off a 07/09 snapshot still | 20:09 |
Vek | jaypipes pushed it up into pypi | 20:09 |
jaypipes | jk0: doh! that was it. | 20:09 |
jk0 | ahah | 20:09 |
mtaylor | ah. | 20:10 |
*** Tushar has joined #openstack-dev | 20:10 | |
mtaylor | Vek: well that's the thing then... let me check what's up with the ppa | 20:10 |
Tushar | Vishy: Are you there? | 20:11 |
vishy | Tushar: yes | 20:11 |
Tushar | I have just seen that you have assigned security group BP to sleepsonfloor.. | 20:11 |
Tushar | I have almost finish with the implementation and writing unit testcases. | 20:12 |
Tushar | https://blueprints.launchpad.net/nova/+spec/os-security-groups | 20:12 |
vishy | doh | 20:12 |
vishy | we have an implementation also | 20:12 |
mtaylor | Vek: the latest trunk version of glance (2011.3~d4~20110802.176-0ubuntu0ppa1~natty1) seems to be in the nova-core ppa | 20:12 |
vishy | Tushar: do you have a branch up | 20:12 |
Tushar | I am going to create one today. | 20:13 |
Vek | mtaylor: check if glance/client.py has a def for 'set_auth_context' | 20:13 |
vishy | Tushar: ok i guess we can look at both versions and see which we want to use? | 20:13 |
Vek | er, actually, glance/common/client.py, and it's a def for 'set_auth_token' | 20:13 |
* Vek looks for his "bang-head-here" mousepad | 20:14 | |
Tushar | Vishy: Ok | 20:14 |
mtaylor | Vek: no. | 20:15 |
mtaylor | Vek: ok. so that's vexing | 20:15 |
Vek | indeed :) jaypipes, any insights? | 20:15 |
jaypipes | Vek: one sec, in PPB meeting... | 20:15 |
Vek | 'k | 20:16 |
sleepsonthefloor | Tushar, 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#L902 | 20:17 |
openstackjenkins | Project swift build #300: SUCCESS in 43 sec: http://jenkins.openstack.org/job/swift/300/ | 20:17 |
openstackjenkins | Tarmac: added Gaurav B. Gangalwar to AUTHORS file | 20:17 |
vishy | Tushar: if you want to look at the code that we have and integrate it with yours, I can assign it back to you | 20:21 |
vishy | Tushar: we haven't done any unittests yet so yours will probably be ready sooner | 20:22 |
Tushar | Vishy: Please assign it to me. I will check your branch and integrate with my branch as needed | 20:26 |
vishy | Tushar: 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 |
Tushar | Vishy: I will take care of it. | 20:28 |
vishy | Tushar: thx | 20:28 |
sleepsonthefloor | cool - thanks Tushar | 20:28 |
mtaylor | soren: awake? | 20:30 |
Tushar | sleepsonthefloor: You are welcome | 20:30 |
soren | mtaylor: Mostly. | 20:30 |
notmyname | termie: are you using Lion yet? | 20:31 |
soren | mtaylor: My allergy meds are making rather slow, though. | 20:31 |
mtaylor | soren: 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 working | 20:31 |
soren | mtaylor: ...and bad at typing full sentences. | 20:31 |
mtaylor | soren: allergy meds are lovley | 20:31 |
soren | My 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 |
soren | aaaaanyway.. Can you elaborate? | 20:32 |
soren | mtaylor: What are the unacceptable symptoms? | 20:32 |
mtaylor | soren: apparently it doesn't work | 20:33 |
mtaylor | soren: is missing something from glance/common/client.py | 20:33 |
mtaylor | which may indicate oldness? | 20:33 |
soren | bug report? | 20:33 |
soren | or any other source of information? | 20:33 |
mtaylor | not sure there is one - they have a merge prop that isn't passing because it's expecting something from glance which ain't there | 20:33 |
soren | I have IRC logs, too, if that helps. | 20:33 |
mtaylor | I think IRC logs might be the key | 20:33 |
soren | What's the merge prop? | 20:34 |
jk0 | the problem is glance trunk isn't in the PPA | 20:34 |
* soren has a guess as to what the problem is, but would like more details first. | 20:34 | |
vishy | notmyname: I am | 20:34 |
openstackgerrit | Kevin 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/131 | 20:34 |
jk0 | (even though it says it is) | 20:34 |
jaypipes | jeblair: whatcha think about sleepsonthefloor's last comment on https://review.openstack.org/#change,122? What is the best practice or protocol there? | 20:34 |
soren | jk0: What says what is where, and what disagrees? | 20:34 |
jeblair | jaypipes: looking | 20:35 |
notmyname | vishy: are you using git-bzr? | 20:35 |
termie | notmyname: not using lion yet | 20:35 |
soren | I seem to be copmletely surrounded by no information at all :) | 20:35 |
jk0 | soren: hang tight | 20:35 |
soren | jk0: I always do. | 20:35 |
vishy | notmyname: not regularly, I have used it though | 20:35 |
jk0 | soren: 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 PPA | 20:36 |
jk0 | which is causing jenkins to fail a MP because it depends on that new method | 20:36 |
notmyname | termie: I was having an issue with pushing a new branch to lp earlier today (I solved it by reverting to pure bzr) | 20:36 |
termie | notmyname: did you open an issue on the gh page about it? | 20:37 |
termie | notmyname: easiest for me to track bits there | 20:37 |
soren | jk0: 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 |
jk0 | soren: from my understanding mtaylor upgraded glance from the PPA on the jenkins server | 20:38 |
soren | Aha! | 20:38 |
soren | mtaylor: What have you been up to? | 20:38 |
ttx | Team meeting in 20 minutes in #openstack-meeting | 20:38 |
mtaylor | soren: stuff | 20:38 |
mtaylor | soren: it's probably my fault | 20:38 |
jeblair | jaypipes, 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 |
jaypipes | jeblair: ok. | 20:39 |
mtaylor | soren: the build slave for nova has the apt build-deps for nova installed | 20:40 |
mtaylor | soren: and the nova-core/trunk ppa in its sources.list | 20:40 |
jeblair | if it had been a bigger change, you could approve sleepsonthefloor's change, then ask kevin to rebase his patch on that commit | 20:40 |
jaypipes | jeblair: ok. | 20:41 |
jaypipes | jeblair: 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 |
Vek | jeblair, jaypipes: I just submitted a different patch that does what mine does and what sleepsonthefloor's patch does, plus kills some then-dead code | 20:42 |
jeblair | a 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 |
jaypipes | Vek: yes, I noticed :) | 20:42 |
Vek | :) | 20:42 |
Vek | mtaylor: my patch depends on set_auth_token in the glance client, which was recently added. | 20:42 |
jeblair | so 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 |
Vek | I can't really remove it from my patch. | 20:43 |
jaypipes | jeblair: gotcha. | 20:43 |
notmyname | termie: https://github.com/termie/git-bzr-ng/issues/30 | 20:43 |
jeblair | if 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 |
Vek | jeblair: was just waiting to see what people said on my new submission before I did that. | 20:44 |
jeblair | vek: sounds good | 20:44 |
soren | erk.. | 20:45 |
Vek | got a +1 from jaypipes, so if it meets with sleepsonfloor's approval, I'll go ahead and abandon the old one. | 20:45 |
jaypipes | Vek: oh, gah, I thought you'd updated the original... | 20:46 |
jaypipes | Vek: but whatever, yeah, just abandon the original if you didn't rebase... | 20:46 |
jeblair | vek: you can add sleepsonfloor as a reviewer to your patch to notify him that you want him in particular to review it | 20:46 |
Vek | yeah, I haven't played with rebase yet :) | 20:46 |
Vek | jeblair: good idea... | 20:46 |
soren | jk0, 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 |
mtaylor | soren: really? | 20:47 |
termie | notmyname: that error is the bane of my existence | 20:47 |
jk0 | ah, weird | 20:47 |
soren | I'm looking at it right now. | 20:47 |
notmyname | termie: :-) | 20:47 |
soren | mtaylor: It's really boring. | 20:47 |
jaypipes | blamar, 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/70197 | 20:47 |
mtaylor | soren: you are right. I thought it was set_auth_context, which is what I looked for earlier | 20:47 |
mtaylor | Vek: ok. it should work now | 20:47 |
soren | mtaylor: What's the MP that fails? | 20:47 |
termie | notmyname: it pops up over and over again depending on different installation criteria, and i can't duplicate it in any tests | 20:48 |
mtaylor | soren: and sorry for wasting your time :) | 20:48 |
jk0 | soren: https://code.launchpad.net/~klmitch/nova/glance-private-images/+merge/69661 | 20:48 |
Vek | mtaylor: OK... | 20:48 |
soren | mtaylor: Well, it made me discover another quite serious bug, so it's all good. | 20:48 |
termie | notmyname: that's where my extensive test suite came from, is trying to nail down this error | 20:48 |
* Vek looks at jk0--ball's in your court | 20:48 | |
notmyname | termie: any more info that I can provide to help? | 20:48 |
Vek | mtaylor: 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 |
jk0 | I'll reapprove | 20:49 |
jk0 | cross digits | 20:49 |
mtaylor | Vek: hehe | 20:49 |
mtaylor | soren: yay! | 20:50 |
*** negronjl has quit IRC | 20:50 | |
termie | notmyname: 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 stdout | 20:50 |
Vek | jk0: all my crossable digits are crossed... | 20:50 |
notmyname | termie: how do I turn on debug? | 20:50 |
termie | notmyname: if it is for a relatively low (less than a few hundred) number it could be helpful though | 20:50 |
termie | notmyname: i think there is a commented out line at the top or bottom of the script | 20:51 |
termie | just vim your git-bzr | 20:51 |
termie | i've been trying to get bzr to fix at least the symptom of this for some time | 20:51 |
termie | they have a subclass that doesn't implement the protected method that it is trying to execute | 20:51 |
termie | https://bugs.launchpad.net/bzr/+bug/541626 | 20:52 |
soren | mtaylor: How do you ensure that the build deps are installed? | 20:52 |
uvirtbot | Launchpad bug 541626 in bzr "'BTreeBuilder' object has no attribute '_find_ancestors'" [High,In progress] | 20:52 |
mtaylor | soren: apt-get build-dep nova | 20:52 |
mtaylor | soren: driven by a puppet config which is itself driven by cron | 20:53 |
zykes- | in keystone, is there any diff between groups and tenants ? | 20:53 |
dolphm | zykes-: groups were recently removed from keystone | 20:54 |
*** hub_cap has quit IRC | 20:54 | |
mtaylor | soren: 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 like | 20:54 |
zykes- | dolphm: i still see code for it in models... | 20:54 |
dolphm | zykes-: link | 20:54 |
dolphm | zykes-: ? | 20:54 |
mtaylor | soren: the puppet stuff is in openstack/openstack-ci-puppet | 20:55 |
zykes- | oh, old pull :p | 20:55 |
soren | mtaylor: I thought these were ephemeral? | 20:56 |
dolphm | zykes-: when i said recently, i meant like very recently :) lol | 20:56 |
mtaylor | soren: they _will_ be ephemeral | 20:56 |
mtaylor | soren: right now we're just working on ensuring that they can be ephemeral | 20:56 |
soren | mtaylor: So where is this being run? On jenkins.openstack.org? | 20:56 |
mtaylor | soren: 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 us | 20:56 |
mtaylor | soren: where is what being run? | 20:57 |
soren | The failing tests. | 20:57 |
mtaylor | on the nova build slave | 20:57 |
soren | ...which is ephemeral? | 20:57 |
mtaylor | soren: which is currently 173.203.108.124 | 20:57 |
soren | Or can I go an look at it? | 20:57 |
mtaylor | yes | 20:57 |
mtaylor | go look at it! | 20:57 |
mwhooker | should multiple calls to nova.db.sqlalchemy.session.get_session return the same session object? | 20:57 |
mtaylor | soren: but if you need to configure or install anything there, we should do it via the puppet module | 20:58 |
mwhooker | it looks like it's returning a new session each time | 20:58 |
jk0 | Vek / soren / mtaylor: https://code.launchpad.net/~klmitch/nova/glance-private-images/+merge/69661 failed again | 20:58 |
openstackgerrit | A 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/128 | 20:58 |
Vek | *argh* | 20:59 |
mwhooker | so I'm wondering if there's some idiom for working with ORM objects accross session objects | 20:59 |
Vek | exact same failures. | 20:59 |
soren | mtaylor: >>> import glance.client | 20:59 |
soren | >>> glance.client.V1Client.set_auth_token | 20:59 |
mtaylor | hah! | 20:59 |
soren | <unbound method V1Client.set_auth_token> | 20:59 |
zykes- | is jenkins kind of the same stuff as buildboT? | 20:59 |
mtaylor | for some reason nova-tarmac is running on master | 20:59 |
mtaylor | zykes-: yes | 20:59 |
mwhooker | for ex., just uncovering a bug in nova.db.sqlalchemy.api. instance_type_extra_specs_update_or_create/instance_type_extra_specs_get_item | 20:59 |
zykes- | diff being ? | 21:00 |
Vek | creiht: it did, it just threw me for a loop, because I knew it wasn't in my own code. | 21:00 |
ttx | Meeting starts now, please join #openstack-meeting | 21:00 |
mwhooker | besides 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 session | 21:00 |
mwhooker | so 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 |
mtaylor | zykes-: jenkins looks like this: https://jenkins.openstack.org | 21:01 |
mwhooker | which speaks of a deeper design problem to me | 21:01 |
mtaylor | zykes-: it's two different projects that tackled a similar problem space with different approaches | 21:01 |
mtaylor | zykes-: jenkins has through-the-web config, buildbot has editable python config files | 21:02 |
zykes- | ah | 21:02 |
mtaylor | amongst many other things | 21:02 |
mwhooker | I guess the idiom is to pass around the session | 21:03 |
mtaylor | zykes-: I loved buildbot until someone made me use jenkins (which was hudson at the time) haven't looked back since | 21:03 |
zykes- | what's openstack ci ? | 21:03 |
*** jakedahn has joined #openstack-dev | 21:03 | |
mtaylor | zykes-: openstack continuous integration | 21:04 |
*** mgius has joined #openstack-dev | 21:04 | |
heckj | zykes-: all the guts of the jenkins, build, etc setup | 21:04 |
mtaylor | zykes-: it's just a place where we've been sticking the scripts/docss we're using to run/configure things | 21:04 |
zykes- | ah | 21:04 |
*** JStoker has joined #openstack-dev | 21:04 | |
vishy | mwhooker: that is bad | 21:04 |
vishy | mwhooker: yeah generally if you are making a separate query you pass the session | 21:05 |
mwhooker | vishy: thanks. I think I found a workable solution | 21:06 |
mwhooker | though I hope there's a critical eye trained on all that sqlalchemy code | 21:06 |
openstackgerrit | A change was merged to openstack/openstack-ci: Use "site.pp" in launch script. https://review.openstack.org/130 | 21:06 |
Tushar | sleepsonthefloor: Are you there? | 21:08 |
sleepsonthefloor | Hey Tushar | 21:08 |
Tushar | sleepsonthefloor: I checked your github code. I have one question | 21:08 |
openstackgerrit | A change was merged to openstack/openstack-ci: Added script to force set project watches. https://review.openstack.org/126 | 21:09 |
Tushar | sleepsonthefloor: any reason why you have rules and security group into separate controller? | 21:09 |
sleepsonthefloor | Tushar - it seemed restier :) | 21:09 |
Vek | maybe we need to sponge the/a virtual environment somewhere to make it pull in the right glance when running the tests? | 21:10 |
Vek | ^^mtaylor | 21:11 |
uvirtbot | Vek: Error: "^mtaylor" is not a valid command. | 21:11 |
* Vek just stares at uvirtbot | 21:11 | |
sleepsonthefloor | Tushar - did you do authorize/revoke actions on security groups? | 21:11 |
Tushar | sleepsonthefloor: Yes | 21:12 |
mtaylor | Vek: ok. have that part fixed. now there is another bit I've got to sort | 21:12 |
Vek | 'k... | 21:12 |
Tushar | sleepsonthefloor: IMO, it makes sense to have just one controller to manage everything related to secuirty group. what do you think? | 21:13 |
Tushar | sleepsonthefloor: *security | 21:13 |
sleepsonthefloor | Tushar - 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 |
kbringard | in the not too distant future isn't a lot of that going to be outsourced to quantum? | 21:14 |
*** stiekes has quit IRC | 21:15 | |
*** stiekes has joined #openstack-dev | 21:16 | |
*** martine_ has quit IRC | 21:17 | |
sleepsonthefloor | kbringard - 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 |
kbringard | sure, of course | 21:18 |
*** stiekes_ has joined #openstack-dev | 21:18 | |
vishy | it is more than not too distant future as well | 21:19 |
vishy | they punted on security groups initially | 21:19 |
kbringard | my 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 whatever | 21:19 |
kbringard | and I don't know the answer, just tossing it out there | 21:19 |
jaypipes | soren: 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 |
soren | jaypipes: I can take a peek, sure. | 21:21 |
jaypipes | soren: appreciated. | 21:21 |
*** stiekes has quit IRC | 21:22 | |
Tushar | sleepsonthefloor: Ok, I will add rules and security group as separate resources | 21:22 |
*** jakedahn_ has joined #openstack-dev | 21:23 | |
openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: Issue #115: Added support for testing multiple keystone configurations (sql-only, memcache, ldap) https://review.openstack.org/132 | 21:24 |
sleepsonthefloor | Tushar - ok sounds good. thanks again | 21:25 |
*** cp16net has quit IRC | 21:27 | |
openstackgerrit | Monty Taylor proposed a change to openstack/openstack-ci-puppet: Added tarmac launchpad authentication info. https://review.openstack.org/133 | 21:27 |
openstackgerrit | Monty Taylor proposed a change to openstack/openstack-ci-puppet: Removed duplicate packages. Changed manifest in cron job. https://review.openstack.org/134 | 21:27 |
*** jakedahn has quit IRC | 21:27 | |
*** jakedahn_ is now known as jakedahn | 21:27 | |
*** kbringard has quit IRC | 21:29 | |
openstackgerrit | A change was merged to openstack/openstack-ci-puppet: Removed duplicate packages. Changed manifest in cron job. https://review.openstack.org/134 | 21:30 |
openstackgerrit | A change was merged to openstack/openstack-ci-puppet: Added tarmac launchpad authentication info. https://review.openstack.org/133 | 21:30 |
Tushar | sleepsonthefloor: 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 |
openstackgerrit | Monty Taylor proposed a change to openstack/openstack-ci-puppet: Added launchpad login info to bazaar.conf. https://review.openstack.org/135 | 21:37 |
sleepsonthefloor | Tushar - I thought so, and was thinking that bulk operations could be coordinated at a higher level. | 21:37 |
openstackgerrit | A change was merged to openstack/openstack-ci-puppet: Added launchpad login info to bazaar.conf. https://review.openstack.org/135 | 21:37 |
soren | jaypipes: I found the problem. | 21:39 |
soren | jaypipes: 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 |
jaypipes | soren: actually, no that's fine with me. | 21:40 |
jaypipes | soren: nova and keystone do that, too. | 21:40 |
jaypipes | dolphm: mind joining #openstack-meeting, pls? | 21:40 |
dolphm | jaypipes: sure | 21:41 |
jaypipes | dolphm: cheers | 21:41 |
Tushar | sleeponthefloor: Ok | 21:41 |
soren | jaypipes: Wicked. | 21:42 |
soren | jaypipes: /me does that instead | 21:42 |
soren | jaypipes: 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 |
jaypipes | soren: ah! | 21:43 |
*** yogirackspace has joined #openstack-dev | 21:43 | |
*** yogirackspace has left #openstack-dev | 21:43 | |
mtaylor | Vek: ok. I think your thing just merged | 21:46 |
mtaylor | Vek: there were a couple of related problems in there- but thanks, actually, because something wasn't happening that I thought WAS happening | 21:46 |
Vek | thanks :) | 21:47 |
*** bsza has quit IRC | 21:49 | |
openstackjenkins | Project nova build #1,182: SUCCESS in 5 min 52 sec: http://jenkins.openstack.org/job/nova/1182/ | 21:52 |
openstackjenkins | Tarmac: 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 IRC | 21:54 | |
*** asomya has joined #openstack-dev | 21:58 | |
*** mfer has quit IRC | 22:11 | |
zedas | hey, 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 |
zedas | because 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 IRC | 22:27 | |
zedas | ok i should have a rewrite for that file which just uses the ast | 22:27 |
openstackjenkins | Project nova build #1,183: SUCCESS in 3 min 25 sec: http://jenkins.openstack.org/job/nova/1183/ | 22:30 |
openstackjenkins | Tarmac: Add support for 300 Multiple Choice responses when no version identifier is used in the URI (or no version header is present) | 22:30 |
openstackjenkins | Also adds support for server detail json/xml/atom requests. | 22:30 |
tr3buchet | blamar: are you working on lp815957? | 22:31 |
*** medberry is now known as med_brb | 22:33 | |
*** mattray has quit IRC | 22:34 | |
vishy | please to review: https://code.launchpad.net/~soren/nova/source-group-filtering/+merge/69338 | 22:38 |
vishy | also, any other opinions on this before i fire it off? https://code.launchpad.net/~rackspace-titan/nova/update-HACKING/+merge/69829 | 22:38 |
*** med_brb is now known as med | 22:40 | |
*** med is now known as medberry | 22:40 | |
*** nati has joined #openstack-dev | 22:47 | |
vishy | mtaylor: ping | 22:53 |
mtaylor | vishy: I didn't do it1 | 22:53 |
vishy | mtaylor: if we need lxml for tests, we need to add it to build-deps right? | 22:53 |
mtaylor | vishy: 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 better | 22:54 |
vishy | mtaylor: mind adding it to the build-deps of the packaging branch? | 22:54 |
mtaylor | vishy: 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' way | 22:54 |
mtaylor | vishy: not at all | 22:55 |
vishy | i assume the package is python-lxml but not sure | 22:55 |
vishy | it is lxml in pip | 22:55 |
mtaylor | vishy: done | 22:56 |
vishy | thanks mtaylor, you are a sexy beast | 22:56 |
mtaylor | vishy: I do what I can | 22:56 |
mtaylor | vishy: if you can make sure another branch other than the one that needs lxml goes through first, it should all sort itself out | 22:57 |
vishy | there is one ahead of it in the queue :) | 22:57 |
vishy | we'll see what happens | 22:57 |
*** rnirmal has quit IRC | 23:00 | |
*** jkoelker has quit IRC | 23:02 | |
zedas | how'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 |
openstackjenkins | Project nova build #1,184: SUCCESS in 3 min 25 sec: http://jenkins.openstack.org/job/nova/1184/ | 23:05 |
openstackjenkins | Tarmac: 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 used | 23:05 |
zedas | so 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 IRC | 23:10 | |
zedas | actually 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 % sign | 23:15 |
openstackjenkins | Project nova build #1,185: SUCCESS in 3 min 24 sec: http://jenkins.openstack.org/job/nova/1185/ | 23:16 |
openstackjenkins | Tarmac: Fix for bug #798298 | 23:16 |
openstackjenkins | * Removes restriction of have to have an IPv4 network to create and use an IPv6 network | 23:16 |
uvirtbot | Launchpad bug 798298 in nova "IPv6 currently requires IPv4 addresses to be present of faked out" [Medium,In progress] https://launchpad.net/bugs/798298 | 23:16 |
openstackjenkins | * Modifies nova-manage network to require either an IPv4 or IPv6 network be passed | 23: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 network | 23:16 |
openstackjenkins | * Modifies network/manage.py to allow for empty IPv4 assignments. | 23:16 |
*** asomya has quit IRC | 23:17 | |
*** nati has quit IRC | 23:32 | |
*** hub_cap has joined #openstack-dev | 23:33 | |
openstackjenkins | Project nova build #1,186: SUCCESS in 5 min 37 sec: http://jenkins.openstack.org/job/nova/1186/ | 23:38 |
openstackjenkins | Tarmac: Fixes lp819523 | 23:38 |
openstackjenkins | OS API Migrate is now an admin-only method | 23:38 |
*** deshantm has joined #openstack-dev | 23:54 | |
*** rods has quit IRC | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!