| vishy | yeah, but more of the pylint ones are spurious | 00:00 |
|---|---|---|
| vishy | here is a list of ones that are probably actually all bugs | 00:00 |
| vishy | http://pastie.org/2282072 | 00:00 |
| mwhooker | excellent, thanks | 00:01 |
| *** scottsanchez has quit IRC | 00:03 | |
| * Daviey learns about DTest | 00:07 | |
| vishy | adam_g: pushed the fix | 00:09 |
| vishy | i think we just wait for a new build now | 00:09 |
| adam_g | vishy: thanks | 00:10 |
| vishy | adam_g: looks like launchpad doesn't recognize your email | 00:12 |
| *** deshantm has quit IRC | 00:12 | |
| vishy | might want to add it to your account | 00:12 |
| adam_g | thats what i get for trying to keep changelog entries under 80 chars per line. | 00:13 |
| *** donald650 has joined #openstack-dev | 00:14 | |
| Daviey | adam_g: for DEBEMAIL it doesn't matter. :) | 00:14 |
| vishy | Daviey: if the glance ppa was updated, do i need to update glance packages in the nova ppa or will they be pulled in automatically? | 00:15 |
| Daviey | vishy: doesn't nova.sh add the glance PPA? | 00:16 |
| vishy | Daviey: no | 00:16 |
| *** rods has quit IRC | 00:17 | |
| Daviey | looks like jenkins pushes to the nova PPA? | 00:17 |
| vishy | ah, where do you see that? | 00:17 |
| Daviey | https://launchpad.net/~nova-core/+archive/trunk?field.series_filter=oneiric | 00:17 |
| Daviey | uploaded by OpenStack Hudson | 00:17 |
| Daviey | at 1hr ago | 00:17 |
| *** mnour has quit IRC | 00:18 | |
| vishy | ah nice | 00:18 |
| Daviey | hah | 00:18 |
| vishy | new one just came in | 00:18 |
| vishy | 2011.3~d4~20110727.164-0ubuntu0ppa1~oneiric2 | 00:18 |
| Daviey | adam_g is there | 00:18 |
| vishy | nifty-pie | 00:18 |
| adam_g | http://wiki.openstack.org/Packaging/Ubuntu | 00:18 |
| vishy | good deal i think we gots it | 00:18 |
| Daviey | It's a shame the changelog doesn't show the bzr commit msg as the changelog entry :) | 00:19 |
| Daviey | that WOULD be nifty-pie | 00:19 |
| *** tadhgred has joined #openstack-dev | 00:29 | |
| *** medberry is now known as med_out | 00:30 | |
| *** timr has quit IRC | 00:31 | |
| vishy | Daviey: hmm, it doesn't upload them to the milestone ppas though | 00:31 |
| Daviey | Is that intentionally manual? | 00:32 |
| vishy | probably | 00:33 |
| vishy | copying the new glance packages from trunk | 00:34 |
| Daviey | I think there is a promise of some stability there | 00:34 |
| * Daviey heads to bed.. nn peeps | 00:34 | |
| vishy | for milestone yes it should be updated when the milestone is cut | 00:34 |
| Daviey | is that a vishy or ttx job? | 00:34 |
| vishy | ttx, but i need to get them into proposed so people can test | 00:35 |
| Daviey | vishy: Yeah, sounds like a good idea to dput that then | 00:36 |
| vishy | dput? I just do it through the web-ui. I is a newb | 00:36 |
| Daviey | ah yes, lp can copy packages between PPA's.. my bad. | 00:36 |
| *** jdurgin has quit IRC | 00:38 | |
| Daviey | diablo-3 comes out today (28th), right? | 00:38 |
| vishy | well tomorrow now :) | 00:38 |
| Daviey | $ date | 00:39 |
| Daviey | Thu Jul 28 01:38:56 BST 2011 | 00:39 |
| Daviey | :) | 00:39 |
| Daviey | We should probably put d-3 in Oneiric when that is cut, rather than just a snapshot then. | 00:39 |
| Daviey | Our Alpha 3 is next week, so that times nicely. | 00:40 |
| Daviey | we'll need to fix bug #816972 first. | 00:42 |
| uvirtbot | Launchpad bug 816972 in glance "Packaging needs to be updated to take glance-scrubber into account" [High,Confirmed] https://launchpad.net/bugs/816972 | 00:42 |
| *** scottsanchez has joined #openstack-dev | 00:48 | |
| *** negronjl has quit IRC | 00:53 | |
| *** vladimir3p has quit IRC | 01:10 | |
| *** rjimenez has quit IRC | 01:10 | |
| *** nmistry has joined #openstack-dev | 01:16 | |
| *** nmistry has quit IRC | 01:26 | |
| mwhooker | vishy: I'v pushed a branch with fixes to some of those errors you pointed out: https://code.launchpad.net/~openstack/nova/pylint-fixes do I go to lp:nova and say "propose branch for merging" and add that branch? | 01:27 |
| *** dragondm has quit IRC | 01:31 | |
| *** thickskin has left #openstack-dev | 01:38 | |
| vishy | looks like you already proposed it | 01:41 |
| *** mwhooker has quit IRC | 01:43 | |
| *** mfer has joined #openstack-dev | 01:44 | |
| vishy | mwhooker: just gave you a review. BTW, have you signed the cla? | 01:45 |
| vishy | mwhooker: and did you make sure to ./run_tests.sh to verify that all tests still pass? | 01:46 |
| vishy | mwhooker: the next step would be to create some tests that fail. See if you can find one of those bugs and write a test for it that causes the failure. | 01:47 |
| *** cp16net has joined #openstack-dev | 02:02 | |
| *** thickskin has joined #openstack-dev | 02:04 | |
| tr3buchet | vishy: everything going alright? | 02:08 |
| cp16net | how does the nova code changes get merged into github? | 02:10 |
| *** HugoKuo has joined #openstack-dev | 02:10 | |
| *** markvoelker has quit IRC | 02:31 | |
| *** sriramkr has joined #openstack-dev | 02:56 | |
| *** lorin1 has joined #openstack-dev | 02:58 | |
| *** sriramkr has quit IRC | 03:01 | |
| *** sriramkr has joined #openstack-dev | 03:07 | |
| *** alekibango has quit IRC | 03:07 | |
| *** mfer has quit IRC | 03:17 | |
| *** martine has joined #openstack-dev | 03:21 | |
| *** sriramkr has quit IRC | 03:25 | |
| *** mdomsch has quit IRC | 03:39 | |
| *** lorin1 has quit IRC | 04:02 | |
| *** chomping has joined #openstack-dev | 04:03 | |
| *** vladimir3p has joined #openstack-dev | 04:09 | |
| *** winston-d has joined #openstack-dev | 04:09 | |
| *** martine has quit IRC | 04:15 | |
| *** chomping has quit IRC | 04:22 | |
| *** chomping has joined #openstack-dev | 04:25 | |
| *** timr has joined #openstack-dev | 04:35 | |
| *** tadhgred has quit IRC | 04:37 | |
| vishy | tr3buchet: i think we're good afaik | 04:48 |
| tr3buchet | vishy: awesome | 04:55 |
| *** openpercept_ has joined #openstack-dev | 04:59 | |
| *** tadhgred has joined #openstack-dev | 05:19 | |
| *** timr has quit IRC | 05:22 | |
| *** rjimenez has joined #openstack-dev | 05:30 | |
| *** alekibango has joined #openstack-dev | 06:26 | |
| *** rjimenez has quit IRC | 06:51 | |
| *** tomeff has joined #openstack-dev | 07:00 | |
| *** tomeff has quit IRC | 07:04 | |
| *** timr has joined #openstack-dev | 07:15 | |
| *** tadhgred has quit IRC | 07:17 | |
| HugoKuo | does any one knows the benefit with ChanceScheduler | 07:19 |
| ttx | hmm. It's lightweight ? | 07:19 |
| HugoKuo | not sure . why not using simplescheduler as default ? | 07:21 |
| HugoKuo | Actually, ZoneScheduler works like ChanceScheduler (does not use load information), but using availability zone name from hosts (provide ability to place nodes to different availability zones). | 07:22 |
| *** donald650 has quit IRC | 07:28 | |
| *** tomeff has joined #openstack-dev | 08:00 | |
| *** tomeff has left #openstack-dev | 08:11 | |
| *** med_out has quit IRC | 08:12 | |
| *** vladimir3p has quit IRC | 08:33 | |
| *** darraghb has joined #openstack-dev | 09:05 | |
| *** AhmedSoliman has joined #openstack-dev | 09:25 | |
| *** medberry has joined #openstack-dev | 09:28 | |
| *** medberry has joined #openstack-dev | 09:28 | |
| *** ahmed_ has joined #openstack-dev | 09:38 | |
| *** AhmedSoliman has quit IRC | 09:39 | |
| *** chomping has quit IRC | 09:46 | |
| chemikadze | Hi all. Does using of several hypervisord in one cloud supported? E.g. half hosts with libvirt/KVM, half with XenServer. | 10:13 |
| ttx | chemikadze: you can certainly run different ones -- the problem is using a scheduler that is aware of this | 10:15 |
| ttx | chemikadze: ISI planned some support for heterogeneous scheduling, not sure that landed yet | 10:16 |
| ttx | feel free to ping Lorin Hochstein on list if you're interested by that | 10:16 |
| chemikadze | ttx: thanks! | 10:17 |
| *** ahmed_ has quit IRC | 10:18 | |
| chemikadze | And can I have different sets of flavours on different hosts? | 10:23 |
| *** rohitk has joined #openstack-dev | 10:30 | |
| ttx | chemikadze: that's the idea behind heterogeneous scehduling. Being able to specify properties for the compute host that would handle your request | 10:32 |
| *** Binbin has joined #openstack-dev | 11:01 | |
| *** markvoelker has joined #openstack-dev | 11:23 | |
| *** tadhgred has joined #openstack-dev | 11:30 | |
| *** timr has quit IRC | 11:31 | |
| *** lorin1 has joined #openstack-dev | 11:36 | |
| *** mfer has joined #openstack-dev | 11:43 | |
| thickskin | is there someone who using Fusion io with nova? | 11:48 |
| *** thickskin has quit IRC | 11:51 | |
| *** thickskin has joined #openstack-dev | 11:52 | |
| *** LoriC has joined #openstack-dev | 12:01 | |
| *** LoriC has quit IRC | 12:05 | |
| *** darraghb has left #openstack-dev | 12:13 | |
| *** bsza has joined #openstack-dev | 12:29 | |
| *** tadhgred has quit IRC | 12:35 | |
| *** TimR has joined #openstack-dev | 12:37 | |
| *** bsza3 has joined #openstack-dev | 12:41 | |
| *** bsza has quit IRC | 12:41 | |
| *** darraghb has joined #openstack-dev | 12:49 | |
| *** rjimenez has joined #openstack-dev | 13:08 | |
| *** jaypipes has joined #openstack-dev | 13:17 | |
| *** TimR has quit IRC | 13:26 | |
| *** TimR has joined #openstack-dev | 13:27 | |
| *** sandywalsh has quit IRC | 13:34 | |
| *** sandywalsh has joined #openstack-dev | 13:36 | |
| *** kbringard has joined #openstack-dev | 13:38 | |
| *** dspano has joined #openstack-dev | 13:55 | |
| *** ameade has joined #openstack-dev | 13:58 | |
| *** lts has joined #openstack-dev | 14:14 | |
| jaypipes | clayg: if you have a moment, I've merged your URI parse fixes with some other fixes from another branch and proposed: https://code.launchpad.net/~jaypipes/glance/bug817121/+merge/69523. Please let me know if that branch is good. Thanks in advance. | 14:21 |
| johan_-_ | how does one get modules installed onto the jenkins server? | 14:24 |
| *** markvoelker has quit IRC | 14:25 | |
| jaypipes | johan_-_: ping mtaylor | 14:28 |
| jaypipes | johan_-_: and I've already pinged him about kombu ;) | 14:29 |
| *** dragondm has joined #openstack-dev | 14:29 | |
| johan_-_ | thanks | 14:29 |
| *** rnirmal has joined #openstack-dev | 14:30 | |
| mtaylor | johan_-_: in this case, kombu doesn't seem to have ubuntu packages, so we need to package it up real quick. at that point, it's dead easy to add it to the glance builder | 14:32 |
| *** rohitk has quit IRC | 14:33 | |
| johan_-_ | ahh. i thought the build server used pip for some reason | 14:34 |
| mtaylor | nope. not at all | 14:35 |
| mtaylor | reason being that the project has as one of its outputs ubuntu packages- so we make sure that everything can run/test on a server where all of the depends are installed from packages | 14:36 |
| mtaylor | it's a little bit of extra work, but makes sure that we don't break things for people who are unwilling to deploy pip-based things (most sysadmins) | 14:36 |
| *** rnorwood has joined #openstack-dev | 14:37 | |
| *** jkoelker has joined #openstack-dev | 14:39 | |
| *** vladimir3p has joined #openstack-dev | 14:41 | |
| ttx | jaypipes, vishy: ok to release ? | 14:42 |
| *** Binbin has quit IRC | 14:42 | |
| jaypipes | ttx: sorry, no. waiting on a couple reviews... | 14:42 |
| jaypipes | blamar: around? | 14:42 |
| blamar | jaypipes: whats up | 14:42 |
| jaypipes | blamar: hey, wondering if you could review https://code.launchpad.net/~jaypipes/glance/bug817121/+merge/69523? | 14:43 |
| jaypipes | blamar: after that, I need to merge that (if going into trunk) with the S3 branch and re-proposed for merge.. | 14:43 |
| blamar | kk on it | 14:43 |
| jaypipes | blamar: much appreciated. | 14:43 |
| jaypipes | blamar: also https://code.launchpad.net/~rconradharris/glance/lp817275/+merge/69577 looks like an easy one, too... | 14:44 |
| blamar | lets not push it... | 14:44 |
| blamar | :P | 14:44 |
| *** martine has joined #openstack-dev | 14:45 | |
| vladimir3p | Folks, any chance to get more reviews for https://code.launchpad.net/~vladimir.p/nova/vsa/+merge/68987 Thanks a lot to blamar for his comments | 14:46 |
| *** chomping has joined #openstack-dev | 14:49 | |
| *** zaitcev has joined #openstack-dev | 14:49 | |
| *** donald650 has joined #openstack-dev | 14:53 | |
| *** cp16net_ has joined #openstack-dev | 15:01 | |
| jk0 | can a nova c0re look at this pls: https://code.launchpad.net/~jk0/nova/lp805549/+merge/69518 | 15:02 |
| openstackjenkins | Project swift build #298: SUCCESS in 42 sec: http://jenkins.openstack.org/job/swift/298/ | 15:03 |
| openstackjenkins | Tarmac: Updated CHANGELOG with stuff I got confused about with the 1.4.2 release. | 15:03 |
| *** gaitan has joined #openstack-dev | 15:05 | |
| *** yogirackspace has joined #openstack-dev | 15:08 | |
| jaypipes | Vek: looks like there's merge conflicts in the Nova glance-private-image merge proposal... | 15:10 |
| *** statik has quit IRC | 15:15 | |
| zul | so you need openvswitch now? | 15:17 |
| jk0 | that's only for XenServer 5.6fp2 | 15:23 |
| jk0 | (comes with it) | 15:23 |
| jk0 | in dom0 | 15:23 |
| *** dolphm has quit IRC | 15:25 | |
| zul | apparently not so if you want to use libvirt with vif | 15:27 |
| jk0 | ah, didn't realize anything else used it atm | 15:28 |
| *** dolphm has joined #openstack-dev | 15:30 | |
| ttx | vishy: up yet ? | 15:31 |
| jeblair | yogirackspace: when replacing a change, put the Change-Id: on its own line at the bottom of the commit message, like https://review.openstack.org/#patch,sidebyside,42,1,/COMMIT_MSG | 15:31 |
| dolphm | jaypipes: when you get some time, i want to figure out your unit test issue | 15:32 |
| *** jeblair has quit IRC | 15:33 | |
| blamar | jaypipes: bug817121, lines 163/164 ... I think that raise was just added to fix another bug, any reason to remove? | 15:33 |
| jaypipes | blamar: it didn't make sense any more, or at least, I couldn't figure out where it was supposed to go after removing the if block... thoughts? | 15:34 |
| jaypipes | dolphm: I have time. well, as much time as ttx will give me ;) | 15:35 |
| *** mattray has joined #openstack-dev | 15:35 | |
| blamar | jaypipes: if the "else raise" isn't there then resp_headers never gets defined and can fail further down | 15:37 |
| jaypipes | blamar: hmm... | 15:37 |
| jaypipes | blamar: OK, lemme re-look at it. | 15:37 |
| jaypipes | blamar: I couldn't see where the raise was relevant after the code change, but I'll look again by comparing against trunk. prolly just a matter of the diff not being obvious. | 15:38 |
| *** markvoelker has joined #openstack-dev | 15:39 | |
| jaypipes | blamar: fixed and pushed. yeah, the diff was showing the wrong place... so I got confused. | 15:40 |
| dolphm | jaypipes: give me a few minutes, i'm working through the merge conflict on gerrit with yogi | 15:43 |
| jaypipes | dolphm: k, no prob | 15:43 |
| *** jeblair has joined #openstack-dev | 15:46 | |
| openstackjenkins | Project keystone build #54: SUCCESS in 1 min 12 sec: http://jenkins.openstack.org/job/keystone/54/ | 15:51 |
| openstackjenkins | yoga80: Trying to fix existing merge issue. | 15:51 |
| dolphm | YAY | 15:51 |
| dolphm | go jenkins go! | 15:51 |
| openstackjenkins | Project nova build #1,163: SUCCESS in 4 min 16 sec: http://jenkins.openstack.org/job/nova/1163/ | 15:51 |
| openstackjenkins | Tarmac: Add a flag to set the default file mode of logs. | 15:51 |
| jeblair | dolphm: that patch has been merged, yay! | 15:51 |
| jeblair | https://review.openstack.org/#change,57 | 15:52 |
| dolphm | is it going to power through all the rest now? | 15:52 |
| jeblair | there's one more that has a conflict (that we know of now): | 15:53 |
| jeblair | https://review.openstack.org/#change,58 | 15:53 |
| *** jpipes has joined #openstack-dev | 15:53 | |
| *** jaypipes has quit IRC | 15:53 | |
| dolphm | jeblair: alright, now we know how to handle it via gerrit | 15:54 |
| jeblair | dolphm: 1 min | 15:55 |
| jeblair | dolphm: it looks like all the patches that depended on that one now need to be rebased in order to be merged :( | 15:57 |
| jeblair | dolphm: http://osdir.com/ml/repo-discuss/2009-11/msg00151.html | 15:59 |
| *** hub_cap has joined #openstack-dev | 15:59 | |
| jeblair | perhaps we should consider temporarily changing the gerrit merge strategy to cherry pick to get through these | 15:59 |
| yogirackspace | thanks jeblair | 16:00 |
| dolphm | jeblair: would that automate the whole thing | 16:00 |
| jeblair | dolphm: that's my hope, but i don't know for sure | 16:00 |
| dolphm | jeblair: makes sense, is it easy to try? | 16:01 |
| jeblair | dolphm: yep. gimme another minute to look into it | 16:01 |
| *** jpipes has quit IRC | 16:01 | |
| *** dolphm has quit IRC | 16:02 | |
| *** dolphm has joined #openstack-dev | 16:02 | |
| jeblair | dolphm: http://osdir.com/ml/repo-discuss/2009-11/msg00155.html | 16:03 |
| jeblair | dolphm: that's the disadvantage of doing that -- the new tree will look different than that of the contributor, with different brand new commits for what look like old changes in contributors trees. | 16:04 |
| jeblair | dolphm: of course, our alternative may not be much different. | 16:04 |
| dolphm | jeblair: definitely not a strategy i would use forever | 16:04 |
| jeblair | dolphm: (yogi did just write a new patch) | 16:04 |
| jeblair | dolphm: lets give it a shot then and see if we can get this stuff merged | 16:05 |
| dolphm | jeblair: go for it | 16:05 |
| *** jaypipes has joined #openstack-dev | 16:07 | |
| kbringard | it looks like some (or all) of the VIF stuff got merged in at 1321? Are there any docs on that… I'd love to start playing with Open vSwitch integration | 16:15 |
| kbringard | ah, looks like some info here… http://openvswitch.org/openstack/ | 16:20 |
| *** bcwaldon has joined #openstack-dev | 16:23 | |
| jeblair | dolphm: that didn't work. monty and i think it's time for plan b: | 16:25 |
| jeblair | dolphm: i'm going to take rackspace/keystone, and apply all of the approved gerrit changes to it | 16:25 |
| ttx | vishy: moo? | 16:25 |
| jeblair | dolphm: that should get us the "correct" repository | 16:25 |
| jeblair | dolphm: we'll push that to gerrit and github:openstack/keystone, and abandon all existing gerrit changes | 16:25 |
| jeblair | dolphm: that may be a little disruptive for your repositories (you may need to do something like git reset, or just clone a new copy from github) | 16:26 |
| jeblair | dolpdm: but we think this is the easiest and sanest way to get working again | 16:26 |
| *** dolphm has quit IRC | 16:26 | |
| *** dolphm has joined #openstack-dev | 16:28 | |
| jeblair | dolphm: did you get all that before you left? | 16:29 |
| mtaylor | dolphm: looks like a network hiccup there- did you catch the stuff jeblair was just saying | 16:29 |
| mtaylor | ? | 16:29 |
| mtaylor | jeblair: jinx | 16:29 |
| jeblair | mtaylor: thank goodness chanserv doesn't understand jinx. :) | 16:29 |
| *** jdurgin has joined #openstack-dev | 16:35 | |
| *** scottsanchez has quit IRC | 16:35 | |
| clayg | jaypipes: this fix (https://code.launchpad.net/~rconradharris/glance/lp816817/+merge/69542) removes the need to prepend http:// to self.authurl | 16:43 |
| dolphm | jeblair: mtaylor: yeah, i got it all | 16:43 |
| jaypipes | clayg: authurl should not have any http(s):// in it at all... | 16:44 |
| clayg | from lp:~jaypipes/glance/bug817121 - http://paste.openstack.org/show/1970/ | 16:44 |
| clayg | jaypipes: correct | 16:44 |
| jeblair | dolphm: good, does that sound okay to you? | 16:44 |
| jaypipes | clayg: ugh. k. fixing yet again... | 16:44 |
| dolphm | jeblair: mtaylor: if a commit has only been pushed to gerrit, but hasn't been pushed *by* gerrit, will it need to be put up for review again? | 16:44 |
| clayg | previously get would use store_loc.authurl - now that it has .swift_auth_url there's no reason for the scheme to be in the authurl | 16:44 |
| jaypipes | clayg: such a mess. there's been like 12 fixes on the exact same lines of code in 4 different branches. | 16:45 |
| mtaylor | dolphm: we're going to apply those as well | 16:46 |
| clayg | that one by rick got merged in pretty quickly, and without tests :\ | 16:47 |
| mtaylor | dolphm: we're going to get the tree into the state it _should_ have been in the first place - so everything that got both approved and verified will go in | 16:47 |
| clayg | it looks obviously correct tho, I'm going off to do some testing | 16:47 |
| mtaylor | dolphm: and then we're going to re-inject unapproved things back in so that we can start this puppy up right | 16:47 |
| dolphm | mtaylor: awesome | 16:47 |
| dolphm | mtaylor: would it be possible/useful to make gerrit reject commits that don't have change-id's in the future? | 16:49 |
| mtaylor | dolphm: I would be in favor of that - I would have to check with jeblair to see how possible that is | 16:50 |
| mtaylor | dolphm: I do think we should be WAY stronger in our wording about installing that commit hook | 16:50 |
| jaypipes | clayg: no need. the swift_auth_url @property rick added had no test cases. I'm adding some that verify the bug you see. | 16:50 |
| dolphm | mtaylor: definitely... it seems like it would solve a lot of (perhaps simple) issues, but i'm wondering if it could also cause some issues down the line | 16:51 |
| jeblair | dolphm: good idea, and i don't know off the top of my head; i'll research that | 16:52 |
| clayg | jaypipes: I don't think the bug I see shows up in rick's branch - it was a side effect of his fix plus the fix you pulled in from my branch | 16:52 |
| mtaylor | dolphm: it makes it _slightly_ harder to get going - but really, we're all about enforcing sanity | 16:52 |
| clayg | it's just a mess | 16:52 |
| mtaylor | dolphm: although, that being said - really the current state is one that should be extremely hard to get to in normal usage moving forward | 16:52 |
| dolphm | mtaylor: :) | 16:52 |
| clayg | the backend fix still needs to get added in ;) | 16:52 |
| mtaylor | dolphm: even without change-id's | 16:53 |
| jaypipes | clayg: exactly. but there were no unit tests added for the swift_auth_url property when rick added his change, so the changes I made in my branch that broke it weren't caught... | 16:53 |
| jeblair | mtaylor, dolphm: i have a local repo starting with rackspace/keystone and all the changes merged by gerrit applied without incident | 16:53 |
| jaypipes | clayg: I've added unit tests and more fixes. pushing shortly. | 16:53 |
| dolphm | mtaylor: we're pretty good at finding hard problems | 16:53 |
| mtaylor | dolphm: since normally those would have come in and been dealt with not as a backlog of divergent stuff :) | 16:53 |
| jeblair | mtaylor, dolphm: now to apply the approved but not merged changes | 16:53 |
| mtaylor | jeblair: sweet. I have yogi's latest locally | 16:53 |
| clayg | jaypipes: ok cool | 16:53 |
| clayg | jaypipes: in the fake_auth_url - that auth url validation test would be better if checked startswith('http') and .count('://') == 1 | 16:55 |
| jaypipes | clayg: sure. though the .count('://') check is already done in parse_uri... | 16:56 |
| jaypipes | clayg: now throws a BadStoreUri exception. | 16:56 |
| clayg | but isn't that on the uri - not on the swift_auth_url? | 16:56 |
| jeblair | mtaylor, dolphm: those patches are now applied | 16:57 |
| clayg | otherwise that test would have blown up in your branch | 16:57 |
| jeblair | mtaylor, dolphm: i am now going to abandon all the changes in gerrit | 16:57 |
| *** negronjl has joined #openstack-dev | 17:00 | |
| *** rohitk has joined #openstack-dev | 17:03 | |
| jaypipes | clayg: would you mind trying https://code.launchpad.net/~jaypipes/glance/bug817121/+merge/69523 once more? thx in advance | 17:06 |
| jeblair | mtaylor, dolphm: done (yay gerrit ssh commands) | 17:08 |
| mtaylor | jeblair: sweet! | 17:08 |
| jeblair | mtaylor, dolphm: next, i'll push my tree to gerrit | 17:08 |
| dolphm | jeblair: awesome, so what's the next step for us? pull from openstack/keystone? or clone? | 17:08 |
| jeblair | dolphm: sit tight for a few more minutes | 17:09 |
| jeblair | dolphm, mtaylor: okay, i've force-pushed my tree to gerrit | 17:17 |
| jeblair | and it has synced to github | 17:17 |
| jeblair | https://github.com/openstack/keystone/ | 17:17 |
| jeblair | dolphm: i think cloning a new copy may be the safest way to go; i honestly don't know what would happen if you just pull. | 17:18 |
| dolphm | jeblair: i'll make a backup and try pulling | 17:19 |
| jeblair | dolphm, mtaylor: okay. now mtaylor is going to upload the change that was in the queue and not approved | 17:20 |
| *** bcwaldon has quit IRC | 17:20 | |
| mtaylor | jeblair: ok. I'm almost certain this will do the right thing... | 17:22 |
| dolphm | jeblair: pulling seems to have worked, i'm going to try a simply gerrit review | 17:22 |
| mtaylor | dolphm: one sec - let me do this first... | 17:22 |
| mtaylor | ok. that seems to have worked | 17:23 |
| mtaylor | dolphm: go for it | 17:24 |
| mtaylor | dolphm: also, check out https://review.openstack.org/#change,107 and make sure it looks ok to you and yogi | 17:24 |
| *** openstackgerrit has joined #openstack-dev | 17:24 | |
| mtaylor | welcome openstackgerrit | 17:25 |
| dolphm | jeblair: gerrit is complaining that i'm trying to upload merges | 17:25 |
| dolphm | jeblair: do i need to clone, then? | 17:25 |
| Daviey | mtaylor: code review bot? | 17:26 |
| mtaylor | dolphm: yeah. I think so. | 17:26 |
| mtaylor | Daviey: yup. we're getting fancy up in this joint | 17:26 |
| Daviey | mtaylor: sounds like it! | 17:26 |
| openstackjenkins | Project nova build #1,164: SUCCESS in 4 min 21 sec: http://jenkins.openstack.org/job/nova/1164/ | 17:26 |
| openstackjenkins | Tarmac: Make network_info truly optional | 17:26 |
| Daviey | mtaylor: automated reviews of MP's? | 17:26 |
| mtaylor | Daviey: hehe. nope. humans must review things :) | 17:26 |
| Daviey | bah | 17:27 |
| mtaylor | Daviey: but we will actually now tell you when there's a new thing to review - and as usual there is automated testing/merging :) | 17:27 |
| dolphm | mtaylor: 107 looks even better than the original | 17:27 |
| Daviey | mtaylor: So you added bzr support to gerrit? :)) | 17:27 |
| mtaylor | dolphm: great! | 17:27 |
| mtaylor | Daviey: nope | 17:27 |
| dolphm | mtaylor: jeblair: why are we allowed to +1 our own reviews? | 17:28 |
| mtaylor | Daviey: that might be ... hard | 17:28 |
| Daviey | mtaylor: shame | 17:28 |
| mtaylor | dolphm: not sure - I've been meaning to ask jeblair about that and see if we can make a change there | 17:28 |
| jeblair | dolphm: i turned off merge uploads since that inadvertently lead to the mess we were in. i suspect that most of your work _won't_ be merges so if we can get away without it, that would be nice. | 17:28 |
| dolphm | jeblair: fair enough | 17:29 |
| mtaylor | dolphm: be sure to either ping us with things you notice like that/and or file bugs - we took best-guess stab at several things, but certainly want policy feedback there | 17:29 |
| clayg | jaypipes: re: merge#69523 - code looks good, i'm testing in the lab now | 17:29 |
| jeblair | dolphm, mtaylor: i don't think that's tunable. but +1 doesn't get you approved, only +2. | 17:29 |
| jaypipes | clayg: cheers | 17:29 |
| mtaylor | jeblair: is it possible to disable +2'ing your own change? | 17:30 |
| mtaylor | (I don't know that it's necessary - just curious) | 17:30 |
| jeblair | dolphm, mtaylor: (and multiple +1's don't add up to a +2) | 17:30 |
| mtaylor | damn google geeks | 17:30 |
| jeblair | mtaylor: i don't think any of the permissions are variable with respect to whose patch it is, so you can change who can +1 and +2 in the context of specific projects, but not specific reviews | 17:31 |
| mtaylor | ok. well that's probably fine | 17:32 |
| *** bcwaldon has joined #openstack-dev | 17:38 | |
| mtaylor | dolphm: so you can wrangle the rest of the team and get them re-cloned with the commit-msg hook instaled? | 17:39 |
| dolphm | mtaylor: sure | 17:40 |
| mtaylor | dolphm: awesome | 17:40 |
| clayg | jaypipes: fwiw I put my approve on merge # 69523 | 17:53 |
| *** amccabe has joined #openstack-dev | 18:03 | |
| *** negronjl has quit IRC | 18:07 | |
| *** mwhooker has joined #openstack-dev | 18:12 | |
| *** mwhooker has quit IRC | 18:13 | |
| *** mwhooker has joined #openstack-dev | 18:13 | |
| *** openpercept_ has quit IRC | 18:26 | |
| dolphm | jeblair: mtaylor: good or bad idea?: forking openstack/keystone on github (e.g. dolph/keystone), and working with our fork until we're ready to submit to gerrit? | 18:30 |
| mtaylor | dolphm: do it! | 18:33 |
| mtaylor | dolphm: I'd just rebase/squash before you submit if you don't want to submit multiple reviews when you submit | 18:33 |
| dolphm | mtaylor: is it more likely to trigger rejected merges going into gerrit? | 18:33 |
| mtaylor | dolphm: nope | 18:33 |
| dolphm | mtaylor: alright | 18:33 |
| dolphm | mtaylor: thanks | 18:34 |
| mtaylor | dolphm: it will probably be the normal operating process | 18:34 |
| dolphm | ls | 18:34 |
| openstackgerrit | Monty Taylor proposed a change to openstack/openstack-ci: Added bogus email addresses. https://review.openstack.org/108 | 18:34 |
| mwhooker | how might people respond to patches which simply rewrite code to be more "pythonic"? | 18:35 |
| jaypipes | clayg: ty clay | 18:37 |
| jaypipes | blamar: the file:// filesystem:// thing is for convenience, and for backwards compat. We've allowed filesystem to be specified in the store. If there's no other concerns, would you approve that bug fix? | 18:37 |
| *** mdomsch has joined #openstack-dev | 18:39 | |
| blamar | jaypipes: meh, it's a pet peeve to have multiple ways to specify things, creates confusion and makes for convoluted code; that being said it already works that way so okie dokie | 18:39 |
| jaypipes | blamar: believe me, if I fixed all the things in the code base that were pet peeves of mine, I'd never get anything merged. Oh wait, that sounds exactly like my refactor-stores branch... that I've done 12 merges with trunk in the past week on... | 18:40 |
| *** vladimir3p has quit IRC | 18:41 | |
| blamar | jaypipes: hey, we're getting a lot of things in/done, doesn't mean I'm going to stop questioning things :) | 18:41 |
| jaypipes | blamar: not at all :) | 18:42 |
| vishy | guys: some eyes here would be good. https://code.launchpad.net/~vishvananda/nova/keystone-migration/+merge/68915 | 18:43 |
| vishy | it touches a lot of tests, and merging is starting to get painful | 18:43 |
| openstackjenkins | Project nova build #1,165: SUCCESS in 4 min 18 sec: http://jenkins.openstack.org/job/nova/1165/ | 18:46 |
| openstackjenkins | Tarmac: Catch DBError for duplicate projects. | 18:46 |
| *** donald650 has quit IRC | 18:49 | |
| *** BK_man_ has joined #openstack-dev | 18:52 | |
| BK_man_ | hi all. Does nova client support keystone or not? | 18:53 |
| BK_man_ | # nova list | 18:53 |
| BK_man_ | 'x-server-management-url' | 18:53 |
| ttx | jaypipes: so I'll release tomorrow early in my morning, so that it's still Thursday somewhere. I'll take whatever is in Glance milestone-proposed branch at that point. Does that work for you ? | 18:56 |
| *** lorin1 has left #openstack-dev | 19:00 | |
| *** darraghb has quit IRC | 19:03 | |
| *** sriramkr has joined #openstack-dev | 19:05 | |
| openstackjenkins | Project nova build #1,166: SUCCESS in 4 min 7 sec: http://jenkins.openstack.org/job/nova/1166/ | 19:06 |
| openstackjenkins | Tarmac: Updated deserialization of POST /servers in the OSAPI to match the latest v1.1 spec | 19:06 |
| *** bsza3 has quit IRC | 19:08 | |
| dolphm | jaypipes: i assume you're still having unit test issues? | 19:11 |
| openstackgerrit | A change was merged to openstack/openstack-ci: Added bogus email addresses. https://review.openstack.org/108 | 19:14 |
| *** vladimir3p has joined #openstack-dev | 19:16 | |
| *** bsza has joined #openstack-dev | 19:27 | |
| *** donald650 has joined #openstack-dev | 19:29 | |
| *** sriramkr has quit IRC | 19:32 | |
| adam_g | ttx: hey, when did python-xattr become a dependency? | 19:55 |
| mtaylor | adam_g: at some point | 19:57 |
| kbringard | I upgrade to the latest trunk almost every day, and I think it happened since yesterday morning | 19:57 |
| bcwaldon | adam_g: it is a requirement for Glance caching. The packages were updated within the last day or so | 19:57 |
| adam_g | right, i added it as a dependency of python-glance, but we need to file an MIR | 20:00 |
| johan_-_ | http://bazaar.launchpad.net/~hudson-openstack/glance/trunk/annotate/head:/tools/pip-requires | 20:00 |
| johan_-_ | there you go | 20:00 |
| *** lts has quit IRC | 20:05 | |
| *** lts has joined #openstack-dev | 20:06 | |
| johan_-_ | mtaylor: did you know when the jenkins environment will be kombu enabled? | 20:13 |
| mtaylor | johan_-_: no. waiting on kombu packaging ... and I was going to get soren to do that. if I get a free moment later today i'll try to knock it out | 20:13 |
| *** Tushar has joined #openstack-dev | 20:14 | |
| johan_-_ | mtaylor: that's ok. it's not a rush, i was just curious what i should expect | 20:19 |
| *** negronjl has joined #openstack-dev | 20:20 | |
| *** amccabe has quit IRC | 20:35 | |
| ttx | jaypipes: around ? | 20:45 |
| ttx | jaypipes: bah, see email (@gmail) | 20:48 |
| *** amccabe has joined #openstack-dev | 20:48 | |
| *** negronjl_ has joined #openstack-dev | 20:59 | |
| *** negronjl has quit IRC | 21:00 | |
| openstackjenkins | Project nova build #1,167: SUCCESS in 4 min 2 sec: http://jenkins.openstack.org/job/nova/1167/ | 21:06 |
| openstackjenkins | Tarmac: Fixes bug 816604, which is the problem that timeformat in server responses for updated and created are incorrect. This fix just converts the datetime into the correct format. | 21:06 |
| uvirtbot | Launchpad bug 816604 in nova "OSAPI: created and updated for /servers have incorrect time format" [Medium,Fix committed] https://launchpad.net/bugs/816604 | 21:06 |
| *** martine has quit IRC | 21:07 | |
| openstackjenkins | Project nova build #1,168: SUCCESS in 4 min 8 sec: http://jenkins.openstack.org/job/nova/1168/ | 21:17 |
| openstackjenkins | Tarmac: Removed unused Duplicate catch. | 21:17 |
| bcwaldon | vishy: https://code.launchpad.net/~rackspace-titan/nova/remove_xenapi_image_service_flag_lp708754/+merge/69193 | 21:20 |
| vishy | bcwaldon: that is what i mean | 21:24 |
| bcwaldon | vishy: excellent! | 21:24 |
| *** negronjl_ has quit IRC | 21:24 | |
| vishy | bcwaldon: but I wanted to make sure that no one is still trying to use the old objectstore plugin | 21:24 |
| bcwaldon | would the nova-objectstore package go away with this change? | 21:25 |
| *** yogirackspace has left #openstack-dev | 21:26 | |
| *** bsza has quit IRC | 21:27 | |
| *** mattray has quit IRC | 21:30 | |
| *** lts has quit IRC | 21:30 | |
| *** mfer has quit IRC | 21:32 | |
| *** alfred_mancx has joined #openstack-dev | 21:33 | |
| kbringard | when the objectstore stuff drops, does that mean the euca-upload-bundle stuff won't work anymore? or has that functionality been moved over to glance? | 21:33 |
| *** ameade has quit IRC | 21:38 | |
| *** cp16net_ has quit IRC | 21:41 | |
| vishy | kbringard, bcwaldon: no | 21:41 |
| *** gaitan has quit IRC | 21:42 | |
| vishy | that plugin was for the old objecttore, which did images as well as buckets | 21:42 |
| vishy | new one just does buckets | 21:42 |
| kbringard | ah, OK | 21:42 |
| vishy | and it won't go away so euca-upload will still work | 21:42 |
| kbringard | I asked because I was just working on an EC2 -> OpenStack migration solution | 21:42 |
| kbringard | which relied on that, hehe | 21:42 |
| kbringard | thanks for the info | 21:43 |
| alfred_mancx | is there an openstack-qa channel? | 21:44 |
| vishy | alfred: no | 21:44 |
| vishy | alfred_mancx: ^ | 21:44 |
| *** hub_cap has quit IRC | 21:44 | |
| vishy | depending on the topic it should go here or openstack | 21:44 |
| *** negronjl has joined #openstack-dev | 21:45 | |
| *** amccabe has quit IRC | 21:47 | |
| *** markvoelker has quit IRC | 21:48 | |
| openstackgerrit | Monty Taylor proposed a change to openstack/openstack-ci: Added gerrit user info syncing script. https://review.openstack.org/109 | 21:58 |
| *** mattray has joined #openstack-dev | 22:00 | |
| *** dspano has quit IRC | 22:00 | |
| kbringard | in FlatDHCP, the network controller should still show up in nova-manage service list, right? | 22:02 |
| vladimir3p | Folks, where can I find sources for smoketests? Is it all in smoktests/ dir? there are only few files ... thanks | 22:03 |
| *** negronjl has quit IRC | 22:05 | |
| *** negronjl has joined #openstack-dev | 22:05 | |
| openstackjenkins | Project nova build #1,169: SUCCESS in 4 min 4 sec: http://jenkins.openstack.org/job/nova/1169/ | 22:06 |
| openstackjenkins | Tarmac: Adds XML serialization for servers responses that match the current v1.1 spec | 22:06 |
| *** kbringard has quit IRC | 22:08 | |
| openstackgerrit | A change was merged to openstack/openstack-ci: Added gerrit user info syncing script. https://review.openstack.org/109 | 22:10 |
| *** mattray1 has joined #openstack-dev | 22:17 | |
| *** mattray has quit IRC | 22:19 | |
| *** bcwaldon has quit IRC | 22:22 | |
| *** mdomsch has quit IRC | 22:28 | |
| *** rnorwood has quit IRC | 22:36 | |
| *** richardkaufmann has joined #openstack-dev | 22:37 | |
| *** mattray1 has quit IRC | 22:49 | |
| *** rnirmal has quit IRC | 23:21 | |
| *** donald650 has quit IRC | 23:27 | |
| *** dragondm has quit IRC | 23:35 | |
| *** vladimir3p has quit IRC | 23:40 | |
| *** alfred_mancx has quit IRC | 23:43 | |
| *** jkoelker has quit IRC | 23:47 | |
| *** alfred_mancx has joined #openstack-dev | 23:49 | |
| *** negronjl has quit IRC | 23:53 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!