| *** hbetts has quit IRC | 00:06 | |
| openstackgerrit | A change was merged to openstack/openstack-ci-puppet: Add apt-get update so that we get latest packages. https://review.openstack.org/518 | 00:06 |
|---|---|---|
| *** jeffjapan has quit IRC | 00:11 | |
| *** jeffjapan has joined #openstack-dev | 00:12 | |
| *** lorin1 has joined #openstack-dev | 00:16 | |
| *** zns has joined #openstack-dev | 00:20 | |
| *** lorin11 has joined #openstack-dev | 00:21 | |
| *** lorin12 has joined #openstack-dev | 00:21 | |
| *** lorin1 has quit IRC | 00:21 | |
| *** gabrielhurley has quit IRC | 00:22 | |
| *** wwkeyboard has joined #openstack-dev | 00:27 | |
| openstackgerrit | A change was merged to openstack/keystone: Changes to support auth catalog as per new format. https://review.openstack.org/525 | 00:38 |
| *** lorin12 has left #openstack-dev | 00:44 | |
| *** hugokuo has joined #openstack-dev | 00:47 | |
| *** HugoKuo_ has quit IRC | 00:50 | |
| *** anotherjesse has joined #openstack-dev | 00:51 | |
| *** bcwaldon has joined #openstack-dev | 00:53 | |
| *** bcwaldon has quit IRC | 01:00 | |
| *** nati has quit IRC | 01:15 | |
| *** jdurgin has quit IRC | 01:16 | |
| *** dolphm has joined #openstack-dev | 01:25 | |
| dolphm | anotherjesse: just realized i wasn't on irc.. | 01:25 |
| *** hbetts has joined #openstack-dev | 01:28 | |
| *** anotherjesse has quit IRC | 01:31 | |
| *** nati has joined #openstack-dev | 01:32 | |
| *** stuntmachine has joined #openstack-dev | 01:34 | |
| stuntmachine | Is Diablo still on track for release tomorrow? I'm not seeing much about it on the OpenStack website. | 01:35 |
| dolphm | stuntmachine: the release is under threat of delay, but i'm working my *** off :) | 01:44 |
| stuntmachine | hahaha, well that's appreciated :) | 01:44 |
| stuntmachine | I work for a company that is looking to use OpenStack for our network, and since we're so close to Diablo we're really excited about it, that's all. | 01:45 |
| stuntmachine | I know there are some big changes and merges coming along with it. | 01:45 |
| dolphm | cool! yeah some of those big changes lost sync with the rest of the community | 01:46 |
| stuntmachine | Sadly, I'm a sysadmin, not a developer, so I can't offer much in the way of code... | 01:46 |
| *** halfss has joined #openstack-dev | 01:46 | |
| *** yeming has joined #openstack-dev | 01:48 | |
| stuntmachine | That said, I'm excited about the prospect of multi-nic support in instances. It would make running a management network a lot simpler, though I'm not sure if I'm envisioning the networking goals of OpenStack in the same way the community is. | 01:49 |
| openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: POST /tokens: Added missing data https://review.openstack.org/516 | 01:50 |
| dolphm | stuntmachine: hopefully that's the last missing feature! ^^ | 01:50 |
| *** nci has left #openstack-dev | 01:50 | |
| stuntmachine | dolphm: Are you heavily involved with development? I know that's a very open-ended question, but I ask because I'm genuinely curious how something like multiple NICs was overlooked. It almost makes me wonder if we're envisioning this incorrectly. | 01:52 |
| dolphm | stuntmachine: i'm one of the core developers on keystone (identity/authentication)... but i don't even know what openstack project would be responsible for that | 01:53 |
| stuntmachine | Probably nova, since nova-network handles all networking management. | 01:54 |
| dolphm | ah okay | 01:54 |
| dolphm | as an openstack *user*, I'm only familiar with swift | 01:55 |
| *** hbetts has quit IRC | 01:56 | |
| *** joonwon has joined #openstack-dev | 02:01 | |
| openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: POST /tokens: Added missing data https://review.openstack.org/516 | 02:11 |
| dolphm | added docs for my last change, not sure if my XSD revision is valid, if anyone wants to look at it ^^ | 02:15 |
| *** dolphm has quit IRC | 02:15 | |
| *** zns has quit IRC | 02:27 | |
| *** halfss has quit IRC | 02:34 | |
| *** hugokuo has quit IRC | 02:43 | |
| *** hugokuo has joined #openstack-dev | 02:44 | |
| *** nci has joined #openstack-dev | 02:55 | |
| *** zns has joined #openstack-dev | 02:58 | |
| *** yeming has quit IRC | 03:07 | |
| *** yeming has joined #openstack-dev | 03:07 | |
| rmk | Hmm some interesting issues with my rabbit and mysql servers being on the opposite side of a stateful firewall from the compute nodes. | 03:08 |
| *** zns1 has joined #openstack-dev | 03:16 | |
| *** zns has quit IRC | 03:19 | |
| *** zns1 has quit IRC | 03:21 | |
| *** stanchan has quit IRC | 03:29 | |
| *** openstackjenkins has quit IRC | 03:32 | |
| *** joonwon has quit IRC | 03:33 | |
| *** lorin1 has joined #openstack-dev | 03:45 | |
| *** markvoelker1 has quit IRC | 03:57 | |
| *** zns has joined #openstack-dev | 03:58 | |
| *** hugokuo has quit IRC | 04:02 | |
| *** zns has quit IRC | 04:02 | |
| *** hugokuo has joined #openstack-dev | 04:02 | |
| *** lorin1 has left #openstack-dev | 04:03 | |
| *** stuntmachine has quit IRC | 04:06 | |
| *** openstackjenkins has joined #openstack-dev | 04:09 | |
| *** wwkeyboard has quit IRC | 04:13 | |
| *** Rajaram has joined #openstack-dev | 04:15 | |
| *** martine_ has joined #openstack-dev | 04:24 | |
| *** Rajaram has quit IRC | 04:27 | |
| *** stanchan has joined #openstack-dev | 04:54 | |
| *** zaitcev has quit IRC | 05:06 | |
| *** mjs has joined #openstack-dev | 05:10 | |
| *** martine_ has quit IRC | 05:12 | |
| *** mjs has quit IRC | 05:15 | |
| *** naehring has joined #openstack-dev | 05:37 | |
| *** rcc has joined #openstack-dev | 05:49 | |
| *** rcc1 has quit IRC | 05:50 | |
| *** jdag has joined #openstack-dev | 06:32 | |
| *** tsuzuki_ has joined #openstack-dev | 06:59 | |
| * ttx yawns | 07:07 | |
| *** reidrac has joined #openstack-dev | 07:10 | |
| *** nati has quit IRC | 07:18 | |
| *** xavicampa has joined #openstack-dev | 07:20 | |
| *** jakedahn has joined #openstack-dev | 07:21 | |
| ttx | soren, mtaylor, jeblair: jenkins is busted | 07:28 |
| ttx | "Service Temporarily Unavailable" -- bad timing | 07:28 |
| *** tr3buchet has quit IRC | 07:35 | |
| *** tr3buchet has joined #openstack-dev | 07:35 | |
| openstackgerrit | sleepsonthefloor proposed a change to openstack/keystone: Fix for bug 855226. In create token response, provide user and roles. https://review.openstack.org/526 | 07:38 |
| uvirtbot | Launchpad bug 855226 in keystone "Missing user and roles in create token response" [Undecided,New] https://launchpad.net/bugs/855226 | 07:38 |
| *** AhmedSoliman has joined #openstack-dev | 07:57 | |
| *** xavicampa has quit IRC | 07:58 | |
| *** rbasak has joined #openstack-dev | 08:14 | |
| soren | ttx: Erk. | 08:23 |
| soren | ttx: /me investigates | 08:23 |
| ttx | soren: probably the same disk space thing | 08:23 |
| ttx | soren: I just need it to work today. Hell can break loose tomorrow :) | 08:24 |
| *** openstackjenkins has quit IRC | 08:24 | |
| soren | ttx: On itsway back. | 08:24 |
| *** joonwon has joined #openstack-dev | 08:25 | |
| soren | No,plentyof space. | 08:25 |
| *** openstackjenkins has joined #openstack-dev | 08:25 | |
| soren | Hm...my spacebaaris actingup. | 08:25 |
| soren | Darn int. | 08:25 |
| ttx | soren: any particular reason outside of Murphy's law ? | 08:26 |
| soren | cmvvvvvvvvvvxx ,,,c x | 08:27 |
| soren | whoops | 08:27 |
| soren | There. All better, i think. | 08:27 |
| ttx | lsdddddddd. | 08:27 |
| *** rbasak has quit IRC | 08:27 | |
| ttx | jenkins has a start-up time ? Crazy Java. | 08:28 |
| soren | ttx: It spends a lot of time scanning through build history (which is kept on the filesystem rather than in something like sqlite) | 08:29 |
| soren | ttx: It seems to have died last night (3:25 UTC) | 08:30 |
| naehring | memrci | 08:31 |
| *** doude has joined #openstack-dev | 08:31 | |
| soren | Oh! | 08:31 |
| soren | ttx: There's a crapload of: Caused by: java.lang.OutOfMemoryError: Java heap space | 08:31 |
| * ttx sobs. Why does anyone use Java. | 08:31 | |
| ttx | soren, mtaylor: then we should probably bump up the JVM heap parameters, maybe they weren't adjusted to account for extra mem | 08:33 |
| *** AhmedSoliman has quit IRC | 08:34 | |
| soren | ttx: I'll leave that to mtaylor. He's java-enabled. He'll know how to do this in a flash. | 08:34 |
| ttx | poor soul | 08:35 |
| soren | ttx: My Java Certified Engineer certification expired years ago. I shouldn't even be looking at these logs. | 08:35 |
| ttx | soren: do they make your head hurt ? | 08:35 |
| soren | ttx: The logs? Only from shaking my head so much. | 08:36 |
| ttx | vishy: Jenkins is back on line, approve at will. | 08:36 |
| ttx | soren: thanks for saving the day again | 08:36 |
| soren | \o/ | 08:36 |
| *** rbasak has joined #openstack-dev | 08:36 | |
| * ttx ponders some kind of release sprint for OpenStack | 08:36 | |
| soren | I didn't realise we had https-enabled Jenkins. That's great. (and about time) | 08:36 |
| ttx | having people "around" and in the same timezone actually removes a lot of the stress | 08:37 |
| soren | ttx: I was thinking the same thing last night, actually. | 08:37 |
| ttx | and could double up as a design summit prep meeting | 08:37 |
| soren | ttx: I'm here for the rest of the day. Sorry about not picking up my phone. | 08:37 |
| ttx | cool, thx | 08:37 |
| *** jdag has quit IRC | 08:39 | |
| ttx | Better release today, apparently the press says it's already released. | 08:43 |
| * ttx ponders a one-week delay for fun. | 08:43 | |
| zykes- | :p | 08:44 |
| *** darraghb has joined #openstack-dev | 08:53 | |
| *** Jaesang has quit IRC | 09:06 | |
| openstackgerrit | benzwt proposed a change to openstack/glance: adaptive pagination https://review.openstack.org/433 | 09:20 |
| openstackgerrit | benzwt proposed a change to openstack/glance: adaptive pagination https://review.openstack.org/433 | 09:22 |
| *** rods has joined #openstack-dev | 09:39 | |
| * ttx disables Glance BugTeam's bug mail while he fixreleases 84 bugs | 09:46 | |
| * ttx reenabled Glance bugmail | 09:52 | |
| *** yeming has quit IRC | 09:56 | |
| soren | ttx: Good idea! | 09:58 |
| ttx | that won't save direct subscribers, but... | 09:58 |
| ttx | for Nova, I'll move 450 bugs | 09:58 |
| soren | Holy crap. | 10:00 |
| soren | Excusez mon français. | 10:00 |
| ttx | Looking at the list of Nova direct subscribers, this is going to hurt. | 10:03 |
| *** jeffjapan has quit IRC | 10:04 | |
| ttx | soren: btw, i was wondering if we should have oneiric packages in the 2011.3 release PPA | 10:05 |
| ttx | soren: for the 2011.2 one, we didn't have natty packages | 10:05 |
| ttx | because natty shipped with later versions anyway | 10:06 |
| ttx | that said... there may be value in providing "pure" Diablo packages for Oneiric | 10:06 |
| ttx | before Daviey adds one-month-worth of patches on top of it | 10:07 |
| ttx | dunno. | 10:08 |
| *** joonwon has quit IRC | 10:19 | |
| soren | ttx: I think we should. Just for good measure. | 10:36 |
| ttx | works for me | 10:37 |
| *** naehring has quit IRC | 11:01 | |
| Daviey | ttx: >:o | 11:03 |
| Daviey | ttx: providing you respect versioning, i don't mind :) | 11:04 |
| Daviey | not that i see the benefit.. but yes | 11:04 |
| Daviey | ttx: won't be over the moon if it gets large adoption, and people start raising bugs that are solved in oneiric. | 11:04 |
| *** wwkeyboard has joined #openstack-dev | 11:09 | |
| ttx | Daviey: that's your competitive advantage ! "We ship a Diablo that actually works" | 11:12 |
| Daviey | ttx: Looking at the current state, even i wouldn't make such a claim. | 11:13 |
| Daviey | marketing might. | 11:13 |
| Daviey | "we ship a diablo that is slightly less fragile", sounds more honest. | 11:14 |
| nhm | I'm somewhat embarrased to admit I'm still running cactus. | 11:17 |
| *** naehring has joined #openstack-dev | 11:34 | |
| *** tsuzuki_ has quit IRC | 11:34 | |
| zul | ttx: actually ill be doing the one-month-worth of patches on top of it ;) | 11:37 |
| ttx | zul: you're the man! | 11:38 |
| zul | ttx: bah | 11:38 |
| zul | just call me rodney | 11:38 |
| *** hugokuo has quit IRC | 11:46 | |
| *** bsza has joined #openstack-dev | 11:47 | |
| *** rcc1 has joined #openstack-dev | 11:49 | |
| *** rcc has quit IRC | 11:49 | |
| openstackjenkins | Project nova-milestone build #73: SUCCESS in 19 sec: https://jenkins.openstack.org/job/nova-milestone/73/ | 12:01 |
| openstackjenkins | Tarmac: Final versioning for Diablo | 12:01 |
| ttx | yeeha | 12:05 |
| *** rcc1 has quit IRC | 12:07 | |
| *** rcc has joined #openstack-dev | 12:08 | |
| *** Vek has quit IRC | 12:17 | |
| *** wwkeyboard has quit IRC | 12:20 | |
| *** rohitkarajgi has joined #openstack-dev | 12:29 | |
| *** pontid has quit IRC | 12:38 | |
| *** rohitkarajgi has quit IRC | 12:41 | |
| *** jahor has joined #openstack-dev | 12:42 | |
| ttx | WARNING: bugmail avalanche coming up. If you're directly subscribed to Nova bugs, consider disabling your subscription temporarily | 12:42 |
| *** mancdaz has quit IRC | 12:52 | |
| ttx | "Nova bugs" team bugmail temporarily disabled. | 12:52 |
| ttx | (a bit late) | 12:53 |
| *** stuntmachine has joined #openstack-dev | 12:58 | |
| *** rohitkarajgi has joined #openstack-dev | 13:00 | |
| *** joesavak has joined #openstack-dev | 13:03 | |
| *** amccabe has joined #openstack-dev | 13:04 | |
| *** amccabe_ has joined #openstack-dev | 13:05 | |
| * ttx continues bugmail avalanche. | 13:09 | |
| zul | is there a specific version for novaclient you have to use with nova? | 13:14 |
| *** amccabe has quit IRC | 13:18 | |
| *** lorin1 has joined #openstack-dev | 13:18 | |
| *** mdomsch has joined #openstack-dev | 13:22 | |
| *** bsza has quit IRC | 13:23 | |
| *** kbringard has joined #openstack-dev | 13:24 | |
| *** martine_ has joined #openstack-dev | 13:25 | |
| *** lorin1 has quit IRC | 13:26 | |
| *** amccabe_ has quit IRC | 13:33 | |
| *** bsza has joined #openstack-dev | 13:42 | |
| ttx | ok, bugmail avalanche completed, "Nova bugs" team bugmail restored. | 13:45 |
| * ttx pauses for a bit | 13:47 | |
| *** lts has quit IRC | 13:48 | |
| *** martine_ has quit IRC | 13:51 | |
| mtaylor | ttx, soren: java | 13:51 |
| *** wwkeyboard has joined #openstack-dev | 13:57 | |
| *** amccabe_ has joined #openstack-dev | 13:58 | |
| *** lts has joined #openstack-dev | 13:58 | |
| sandywalsh | vishy, ping? | 14:00 |
| sandywalsh | zul, this is the latest https://github.com/rackspace/python-novaclient | 14:01 |
| soren | mtaylor: What? | 14:01 |
| *** bcwaldon has joined #openstack-dev | 14:02 | |
| vishy | sandywalsh: sup | 14:04 |
| sandywalsh | vishy, hey, just sent an email, but if you a minute for a quick skype? | 14:05 |
| *** rohitkarajgi has quit IRC | 14:05 | |
| sandywalsh | mtaylor, so, what's the latest on that pymox thing? | 14:07 |
| sandywalsh | mtaylor, I have the other two branches merged (novaclient and nova) ... just need keystone | 14:07 |
| mtaylor | sandywalsh: uhm. entirely possible I've dropped a ball on something here - is the open task just adding it to the keystone build deps? | 14:09 |
| sandywalsh | mtaylor, to my understanding, yes | 14:10 |
| sandywalsh | mtaylor, specifically so jenkins can run the tests | 14:10 |
| sandywalsh | mtaylor, import stubout fails on jenkins | 14:10 |
| mtaylor | sandywalsh: ok. done. | 14:11 |
| sandywalsh | mtaylor, thanks ... will try to get it kicked again | 14:11 |
| sandywalsh | mtaylor, unless that's something you can do? | 14:11 |
| ttx | mtaylor: java? | 14:14 |
| *** Vek has joined #openstack-dev | 14:17 | |
| openstackgerrit | Dan Prince proposed a change to openstack/openstack-chef: Update raxkey extension paste config in keystone.conf. Add glance endpoint. Glance keystone auth support. Also add keystone::creds recipe. Cleanup: https://review.openstack.org/527 | 14:20 |
| openstackgerrit | A change was merged to openstack/openstack-chef: Update raxkey extension paste config in keystone.conf. Add glance endpoint. Glance keystone auth support. Also add keystone::creds recipe. Cleanup: https://review.openstack.org/527 | 14:21 |
| mtaylor | sandywalsh: looking | 14:22 |
| openstackgerrit | Verification of a change to openstack/keystone failed: Get Service Catalog from token https://review.openstack.org/359 | 14:24 |
| sandywalsh | thanks mtaylor ... at least now I have a legit bug to fix :) | 14:25 |
| mtaylor | sandywalsh: yay! | 14:25 |
| openstackgerrit | Anne Gentle proposed a change to openstack/image-api: Corrects API example returns URI fields as per bug 854094 https://review.openstack.org/528 | 14:28 |
| uvirtbot | Launchpad bug 854094 in glance "API documentation not current" [Medium,Triaged] https://launchpad.net/bugs/854094 | 14:28 |
| vishy | sandywalsh: I might have some time to chat on skype in a few hours | 14:28 |
| *** mfer has joined #openstack-dev | 14:29 | |
| openstackgerrit | A change was merged to openstack/image-api: Corrects API example returns URI fields as per bug 854094 https://review.openstack.org/528 | 14:30 |
| uvirtbot | Launchpad bug 854094 in glance "API documentation not current" [Medium,Triaged] https://launchpad.net/bugs/854094 | 14:30 |
| sandywalsh | vishy, k, ideally you can make the 11am CST? (see email) | 14:30 |
| vishy | sandywalsh: i will try | 14:32 |
| *** hbetts has joined #openstack-dev | 14:38 | |
| openstackgerrit | Anne Gentle proposed a change to openstack/image-api: Changes pom build file for new naming conventions with version of api https://review.openstack.org/529 | 14:39 |
| *** stanchan has quit IRC | 14:40 | |
| openstackgerrit | A change was merged to openstack/image-api: Changes pom build file for new naming conventions with version of api https://review.openstack.org/529 | 14:40 |
| *** vladimir3p has joined #openstack-dev | 14:42 | |
| *** lorin1 has joined #openstack-dev | 14:47 | |
| *** lorin1 has left #openstack-dev | 14:47 | |
| *** cp16net has joined #openstack-dev | 14:51 | |
| *** nhm has quit IRC | 14:52 | |
| *** dolphm has joined #openstack-dev | 14:53 | |
| *** nhm has joined #openstack-dev | 14:54 | |
| *** jdag has joined #openstack-dev | 14:56 | |
| *** amccabe_ has quit IRC | 14:56 | |
| *** zul has quit IRC | 14:58 | |
| *** rnirmal has joined #openstack-dev | 15:00 | |
| *** heckj has joined #openstack-dev | 15:01 | |
| *** zul has joined #openstack-dev | 15:02 | |
| *** stuntmachine has quit IRC | 15:03 | |
| *** stuntmachine has joined #openstack-dev | 15:03 | |
| openstackgerrit | Anne Gentle proposed a change to openstack/identity-api: Changes pom.xml to create a docs.openstack.org/api/ directory structure for readable URLs plus gives the readme an rst extension https://review.openstack.org/530 | 15:05 |
| *** reidrac has quit IRC | 15:18 | |
| vishy | jaypipes: have the link to the release notes etherpad handy? | 15:34 |
| jaypipes | vishy: http://wiki.openstack.org/ReleaseNotes/Diablo | 15:34 |
| vishy | ah it is a wiki, no wonder | 15:35 |
| bcwaldon | jaypipes, vishy: what's the value in releasing multiple swift versions within Diablo? | 15:35 |
| bcwaldon | just looked at the wiki and noticed it's different than nova/glance | 15:35 |
| jaypipes | bcwaldon: it was just cat'd from the Swift Changelog... | 15:36 |
| *** neogenix has quit IRC | 15:36 | |
| bcwaldon | Ok, it's just weird to say we released all these versions when really only 1.4.3 is useful | 15:36 |
| bcwaldon | jaypipes: does glance acutally have notifications? | 15:38 |
| notmyname | bcwaldon: all the intermediate versions were full prod-ready releases. that's what's happened between cactus and diablo | 15:38 |
| bcwaldon | notmyname: ok, so it's more of a regrouping of releases for swift, while it *is* the release for nova/glance | 15:39 |
| openstackjenkins | Project nova build #1,409: SUCCESS in 3 min 53 sec: https://jenkins.openstack.org/job/nova/1409/ | 15:39 |
| openstackjenkins | Tarmac: Update exception.wrap_exception so that all exceptions (not just Error and NovaException types) get logged correctly. | 15:39 |
| openstackgerrit | Anne Gentle proposed a change to openstack/identity-api: Renames README to README.rst https://review.openstack.org/531 | 15:40 |
| jaypipes | bcwaldon: yes, johan_-_ added them in D4 | 15:40 |
| notmyname | bcwaldon: it's openstack packaging a certain version of the swift code. those other releases are the diff since the last time openstack made an official package | 15:40 |
| bcwaldon | jaypipes: hmm, must have missed that one ;) | 15:41 |
| bcwaldon | notmyname: ok, thanks for explaining it for me :) | 15:41 |
| *** ameade has joined #openstack-dev | 15:42 | |
| vishy | By default (in the ubuntu packages), the --use_deprecated_auth' flag in /etc/nova/nova.conf | 15:46 |
| vishy | do our packages do this? | 15:46 |
| vishy | bcwaldon: * TODO: point out imageRef flavorRef change | 15:49 |
| vishy | any idea what that is referring to? | 15:49 |
| bcwaldon | vishy: yeah, where did that come from | 15:49 |
| vishy | release notes | 15:50 |
| vishy | can you give me a paragraph on it? | 15:50 |
| bcwaldon | vishy: it's really not that big of a change, but sure | 15:50 |
| vishy | well a sentence then | 15:50 |
| bcwaldon | vishy: fine! | 15:50 |
| openstackjenkins | Project nova build #1,410: SUCCESS in 3 min 54 sec: https://jenkins.openstack.org/job/nova/1410/ | 15:51 |
| openstackjenkins | * Tarmac: Add next links for server lists in OSAPI 1.1. This adds servers_links to the json responses, and an extra atom:link element to the servers node in the xml response. | 15:51 |
| openstackjenkins | * Tarmac: Update exception.wrap_exception so that all exceptions (not just Error and NovaException types) get logged correctly. | 15:51 |
| openstackjenkins | * Tarmac: This patch adds instance progress which is used by the OpenStack API to indicate how far along the current executing action is (BUILD/REBUILD, MIGRATION/RESIZE). | 15:52 |
| openstackjenkins | For the first cut, we decided to keep it simple and compute progress by counting discrete steps. This is not ideal since some steps, in particular, steps which involve transferring large amounts of data over the network, take *much* longer than others. A better approximation would account for the data-transferred to the destination host, since in most cases, this dominates the time spent. | 15:52 |
| openstackjenkins | In addition to adding progress, this patch: | 15:52 |
| openstackjenkins | - Allows resizes to use same host for source and destination which is useful for dev environments without a second host. This is enabled by the --allow_resize_to_same_host flag. | 15:52 |
| openstackjenkins | - Fixes a bug in the glance and migration XenAPI plugins where the VHDs were being copied into the SR in the wrong order. Before the base-copy was copied first meaning it was possible for snapwatchd to see the base-copy before the dependent cow was present. It was treat the base_copy as an unreferenced parent, and GC it. | 15:52 |
| openstackjenkins | - Additional refactoring and cleanups. | 15:52 |
| openstackgerrit | Anne Gentle proposed a change to openstack/compute-api: Modifies pom.xml to use a docs.openstack.org/api directory structure https://review.openstack.org/532 | 15:52 |
| *** SpamapS has quit IRC | 15:54 | |
| *** SpamapS has joined #openstack-dev | 15:54 | |
| *** SpamapS has joined #openstack-dev | 15:54 | |
| *** zaitcev has joined #openstack-dev | 16:00 | |
| rmk | Is there anything which would prevent the Cactus dash from working with Diablo for now? | 16:00 |
| *** dragondm has joined #openstack-dev | 16:01 | |
| sandywalsh | vishy, able to join us? | 16:01 |
| vishy | sandywalsh: I have another meeting right now, but hopefully it will be quick | 16:02 |
| vishy | sandywalsh: I will call in after | 16:02 |
| sandywalsh | vishy, excellent | 16:02 |
| kbringard | rmk: you may need to turn on the deprecated auth options in diablo, or the dashboard will display everything for everyone | 16:02 |
| kbringard | possibly… I'm not entirely sure, though | 16:03 |
| rmk | ubuntu packages ship with deprecated auth enabled, so cool | 16:03 |
| rmk | I'll give it a try today | 16:03 |
| kbringard | ah, well there you go :-) | 16:03 |
| *** CatKiller has quit IRC | 16:04 | |
| rmk | cheers on diablo guys | 16:04 |
| vishy | tr3buchet, pvo, westmaas, if any of you have things to add to the release notes, especially in regards to the feature disparity between KVM and XenServer, please let me know | 16:04 |
| vishy | tr3buchet, pvo, westmaas: http://wiki.openstack.org/ReleaseNotes/Diablo | 16:05 |
| rmk | I tried using OS with Xenserver a few weeks back and ended up giving up. Maybe it's worth another shot now. | 16:06 |
| bcwaldon | devcamcar: can you re-review this today -> https://code.launchpad.net/~rackspace-titan/nova/images-next/+merge/75367 | 16:07 |
| openstackgerrit | A change was merged to openstack/compute-api: Modifies pom.xml to use a docs.openstack.org/api directory structure https://review.openstack.org/532 | 16:07 |
| tr3buchet | i've had no issues with openstack and xenserver.. | 16:07 |
| *** adiantum_ has joined #openstack-dev | 16:07 | |
| rmk | tr3buchet: I think it's an issue of documentation really | 16:07 |
| rmk | There's nothing beyond the one wiki page referencing what any of the config should be | 16:08 |
| tr3buchet | could be, i haven't looked at the openstack xs wiki in a while | 16:08 |
| westmaas | vishy: added some os api 1.1 notes | 16:09 |
| tr3buchet | we maintain our own | 16:09 |
| rmk | tr3buchet: docs? | 16:09 |
| tr3buchet | rmk: yes | 16:09 |
| tr3buchet | for xenserver specifically | 16:09 |
| rmk | any intent to publish them ? | 16:09 |
| *** adiantum has quit IRC | 16:10 | |
| rmk | I was compelled by the combination but was concerned there wasn't much support for it | 16:10 |
| tr3buchet | rmk: i'm not sure, i'd have to find out. suffice to say we use it without regularly and generally without issue | 16:11 |
| tr3buchet | s/without// | 16:11 |
| rmk | nice | 16:12 |
| *** mfer has quit IRC | 16:14 | |
| *** mfer has joined #openstack-dev | 16:15 | |
| annegentle | tr3buchet: we should talk at the Summit about xenserver docs | 16:16 |
| tr3buchet | vishy: i'm not familiar enough with kvm.. i'll dig into the code and see if i can come up with some more differences | 16:17 |
| *** zns has joined #openstack-dev | 16:17 | |
| tr3buchet | annegentle: be glad to help out where i can | 16:17 |
| rmk | I think a lot of folks would be interested in OS/XS if there were more detail around it | 16:18 |
| annegentle | tr3buchet: cool, I've been meaning to talk to you more about it | 16:18 |
| kbringard | rmk: agreed, part of the reason I've been holding off is because Ubuntu is supporting it again in 11.10 | 16:18 |
| *** lorin1 has joined #openstack-dev | 16:19 | |
| *** lorin1 has left #openstack-dev | 16:19 | |
| annegentle | kbringard: good point | 16:20 |
| rmk | kbringard: Supporting what? xapi? | 16:20 |
| kbringard | rmk: yeppers | 16:20 |
| kbringard | well, xenserver in general | 16:20 |
| kbringard | soren knows much about this | 16:21 |
| tr3buchet | yeah i was about to say, he'd be the one to ask | 16:21 |
| soren | Sorry, about what? | 16:21 |
| kbringard | xenserver in 11.10 | 16:21 |
| soren | I have *no* clue about that | 16:21 |
| soren | Whatsoever. | 16:22 |
| soren | Nada. | 16:22 |
| kbringard | haha | 16:22 |
| soren | Zilch. | 16:22 |
| soren | Nothing. | 16:22 |
| openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: POST /tokens: Added missing data https://review.openstack.org/516 | 16:22 |
| openstackgerrit | Anne Gentle proposed a change to openstack/object-api: Changes extension of README, adds xml:id to the book file, changes pom.xml to docs.openstack.org/api structure https://review.openstack.org/533 | 16:22 |
| openstackgerrit | A change was merged to openstack/object-api: Changes extension of README, adds xml:id to the book file, changes pom.xml to docs.openstack.org/api structure https://review.openstack.org/533 | 16:23 |
| *** zns has quit IRC | 16:23 | |
| rmk | Is there a 2011.3 ppa now? | 16:25 |
| *** jdurgin has joined #openstack-dev | 16:26 | |
| openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: POST /tokens: Added missing data https://review.openstack.org/516 | 16:27 |
| kbringard | rmk: I think it'll be under /release when it's done, but I don't think it's been pushed yet, although I'm not 100% positive of this | 16:28 |
| rmk | Just tried adding it and it seems to be up | 16:29 |
| *** bsza has quit IRC | 16:29 | |
| *** adiantum_ has quit IRC | 16:30 | |
| ttx | vishy: around ? | 16:31 |
| vishy | soren: are you aware of any other feature differences KVM vs. XenServer? | 16:31 |
| ttx | I see you are | 16:31 |
| vishy | I'm trying to add all the known ones to the release notes | 16:31 |
| *** nati has joined #openstack-dev | 16:31 | |
| ttx | vishy: I'll send the announcement in less than 15min, will the releasenotes be readable ? | 16:31 |
| vishy | blamar: did you see the error I mentioned a few days ago about exceptions and logging | 16:32 |
| vishy | ttx: they are now | 16:32 |
| ttx | ok, cool | 16:32 |
| *** bsza has joined #openstack-dev | 16:32 | |
| vishy | ttx: I was just trying to collect any more info from others | 16:32 |
| openstackgerrit | Yogeshwar Srikrishnan proposed a change to openstack/keystone: Minor Changes to extension WADL. https://review.openstack.org/534 | 16:32 |
| ttx | vishy: you can continue to improve the release notes over the next hours and days | 16:33 |
| ttx | that's one advantage of linking to a wikipage :) | 16:33 |
| vishy | good point | 16:33 |
| vishy | :) | 16:33 |
| blamar | vishy: think I missed that one, link? | 16:34 |
| openstackgerrit | A change was merged to openstack/keystone: Fixes bug 855823 https://review.openstack.org/517 | 16:35 |
| uvirtbot | Launchpad bug 855823 in keystone "KeyError: 'roleRefs' in auth_token.py" [Undecided,New] https://launchpad.net/bugs/855823 | 16:35 |
| vishy | blamar: no link i just mentioned it. Our exceptions don't print nicely in Nose | 16:35 |
| vishy | the deal is, nose logger tries to add extra info to the exception and then reconstruct it passing in a new message | 16:36 |
| vishy | but our exceptions don't take a message parameter | 16:36 |
| vishy | so they try to do the % kwargs magic | 16:36 |
| vishy | and then fallback to printing the message out with no params | 16:36 |
| vishy | so you get very unhelpful exception messages | 16:37 |
| *** CatKiller has joined #openstack-dev | 16:37 | |
| vladimir3p | Folks, is there any draft plan for essex in terms of number of releases? Will it be possible to mark particular revisions as "relatively stable" (especially prior to big & risky merges) or will it be enough to have E1-E4? | 16:38 |
| vishy | sandywalsh: done with my call, still talking? | 16:38 |
| vishy | vladimir3p: we should be following a very similar milestone plan | 16:38 |
| openstackgerrit | A change was merged to openstack/keystone: Minor Changes to extension WADL. https://review.openstack.org/534 | 16:38 |
| ttx | vladimir3p: http://wiki.openstack.org/EssexReleaseSchedule ? | 16:39 |
| ttx | vladimir3p: still tbd at the summit | 16:39 |
| ttx | vishy: sending the emali now | 16:40 |
| vishy | ttx: woo! | 16:40 |
| vishy | i guess we have to do the github move now | 16:40 |
| vladimir3p | ttx: thanks. Yes, I'm actually trying to understand how frequently we will need to merge our code with trunk. The goal it to do it more frequently and not to wait for mid-november (e1), but at the same time there is a big risk that we will hit same issues as we had previously | 16:41 |
| vladimir3p | if we could tell that rev X was relatively stable - it might be good enough | 16:41 |
| vishy | ttx: oh, i think the comment about use_deprecated_auth is false | 16:42 |
| ttx | vishy: or drown ourselves in beer | 16:42 |
| ttx | vishy: fix fix | 16:42 |
| vishy | i need to find out | 16:42 |
| vishy | so someone set use_deprecated_auth to true in the packages | 16:45 |
| vishy | but didn't change the paste config to turn on deprecated auth | 16:45 |
| vishy | so I would suspect the packages are broken | 16:45 |
| vishy | do we want deprecated auth in the packages? | 16:46 |
| vishy | soren, Daviey, smoser, ttx, mtaylor ^^ | 16:46 |
| *** hbetts has quit IRC | 16:46 | |
| kbringard | it makes the upgrade path a lot smoother | 16:47 |
| kbringard | since people coming straight from cactus likely don't have keystone setup, or their configs setup to use it | 16:47 |
| *** CatKiller has quit IRC | 16:47 | |
| *** CatKiller has joined #openstack-dev | 16:48 | |
| mtaylor | vishy: I defer to everyone else on the decision of using deprecated_auth or not, but am happy to make whatever changes are needed so that the packages are not broken | 16:48 |
| vishy | if no one else has an opinion I'm going to ask for it to be changed back | 16:48 |
| vishy | to false | 16:49 |
| vishy | so it matches the code | 16:49 |
| *** jahor has quit IRC | 16:49 | |
| vishy | I'll give 10 minutes for other opinions | 16:49 |
| openstackjenkins | Project nova build #1,411: SUCCESS in 3 min 44 sec: https://jenkins.openstack.org/job/nova/1411/ | 16:49 |
| openstackjenkins | * Tarmac: This patch adds flavor filtering, specifically the ability to flavor on minRam, minDisk, or both, per the 1.1 OSAPI spec. | 16:49 |
| openstackjenkins | In addition, this patch refactors instance_type_get_all to return a *list* of instance_types instead of a *dict*. This makes it more consistent with the rest of the DB API. | 16:49 |
| openstackjenkins | * Tarmac: Add next links for server lists in OSAPI 1.1. This adds servers_links to the json responses, and an extra atom:link element to the servers node in the xml response. | 16:49 |
| smoser | i woudl think we want deprecated_auth turned on in in the packages if the alternative is failed upgrade | 16:49 |
| *** lorin1 has joined #openstack-dev | 16:49 | |
| smoser | but i'm not clear as to if you were saying it *was* turned on o rnot. | 16:50 |
| vishy | smoser: someone turned on the flag | 16:50 |
| vishy | smoser: but didn't change the paste.config | 16:50 |
| vishy | so we are not on or off at the moment | 16:50 |
| kbringard | doesn't it just return everything, as if they were an admin, if you just add the flag but not the correct pipelines? | 16:50 |
| *** cp16net has quit IRC | 16:50 | |
| vishy | kbringard: you will be doing everything as the "access_key" user | 16:51 |
| openstackgerrit | Anne Gentle proposed a change to openstack/identity-api: Modifies pom.xml to create the /api folder https://review.openstack.org/535 | 16:51 |
| mtaylor | also - are we actually expecting a smooth upgrade from people running cactus packages at this point? | 16:51 |
| *** blamar has quit IRC | 16:51 | |
| kbringard | ah, that seems suboptimal :-) | 16:51 |
| *** lorin1 has quit IRC | 16:52 | |
| vishy | kbringard: which will probably break quite a few things | 16:52 |
| smoser | zul, do yo uhave thoughts on above ? ^ it seems to me we need to then open a bug against ubuntu and fix the packages there. | 16:53 |
| kbringard | yea, wasn't saying we should leave it, was just musing slightly off topic | 16:53 |
| kbringard | my only input is the easier upgrade path to have it (properly) set to deprecated by default | 16:53 |
| zul | we have deprecated_auth turned on | 16:53 |
| * kbringard goes back to his hole | 16:53 | |
| vishy | zul: it isn't just a flag | 16:53 |
| vishy | zul: you need a different middleware stack in the paste config | 16:54 |
| mtaylor | well - there's two questions - what will the packages in ubuntu do (I thought they decided on deprecated_auth) and then what will the packages released by us do | 16:54 |
| vishy | mtaylor: seems like it will make more sense to do what ubuntu does, although that isn't the default in the source code, which is confusing | 16:54 |
| zul | well if the packaging needs to be updated in ubuntu then yes open a bug in ubuntu and fix packages | 16:55 |
| rmk | Hmm I thought I read that the bug around rabbitmq connections dying when idle and not recovering gracefully was resolved but it doesn't appear to be | 16:55 |
| openstackgerrit | Yogeshwar Srikrishnan proposed a change to openstack/keystone: Updating guide wrt wadl changes. https://review.openstack.org/536 | 16:55 |
| vishy | mtaylor: ok lets turn it back on in the packaging | 16:55 |
| vishy | mtaylor: that means we have to add our own paste deploy file to the packaging, I'm not completely sure how to do it | 16:56 |
| vishy | mtaylor: but if you grab the one from trunk, there are commented lines for the necessary ones for deprecated auth | 16:56 |
| mtaylor | vishy: ok. which way are you talking here - you do want to enable --use_deprecated_auth | 16:56 |
| mtaylor | yeah? | 16:57 |
| vishy | mtaylor: the flag is already set | 16:57 |
| openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: POST /tokens: Added missing data https://review.openstack.org/516 | 16:57 |
| vishy | mtaylor: we just need the middleware stack | 16:57 |
| mtaylor | correct. just making sure I was grokking which direction "let's turn it back on" was :) | 16:57 |
| *** lorin1 has joined #openstack-dev | 16:57 | |
| vishy | I will update the release notes | 16:57 |
| vishy | ok release notes updated | 17:02 |
| *** chemikadze has quit IRC | 17:02 | |
| vishy | mtaylor: please get that done quickly because ttx is/has announced the release | 17:03 |
| mtaylor | vishy, zul: https://code.launchpad.net/~mordred/nova/enable-deprecated-auth/+merge/76617 | 17:04 |
| *** lorin1 has quit IRC | 17:04 | |
| vishy | mtaylor: looks good | 17:06 |
| zul | ack | 17:07 |
| mtaylor | zul: ok, I'll push that to the diablo branch so you can cut the ubuntu packages for oneiric and then I'll figure out how to get that into the release ppa | 17:07 |
| zykes- | why oneiric ? isn | 17:09 |
| zykes- | isn't that like beta ? | 17:09 |
| *** nati_ has joined #openstack-dev | 17:10 | |
| mtaylor | zykes-: that's where new packages for ubuntu are uploaded if we're talking about the upcoming ubuntu release | 17:10 |
| mtaylor | our ppa contains packages for lucid-oneiric | 17:11 |
| *** lorin1 has joined #openstack-dev | 17:11 | |
| zul | mtaylor: you shouldnt need dnsmasq in your ppa for oneiric | 17:12 |
| mtaylor | zul: yes. you are right | 17:13 |
| *** nati has quit IRC | 17:13 | |
| *** martines has quit IRC | 17:16 | |
| *** martines has joined #openstack-dev | 17:17 | |
| zykes- | what's used instead of dnsmasq? | 17:18 |
| *** stanchan has joined #openstack-dev | 17:21 | |
| openstackgerrit | Dolph Mathews proposed a change to openstack/identity-api: Renames README to README.rst https://review.openstack.org/531 | 17:21 |
| *** CatKiller has quit IRC | 17:22 | |
| *** lorin1 has quit IRC | 17:23 | |
| openstackgerrit | Dolph Mathews proposed a change to openstack/identity-api: Renames README to README.rst https://review.openstack.org/531 | 17:24 |
| *** lorin1 has joined #openstack-dev | 17:24 | |
| *** CatKiller has joined #openstack-dev | 17:24 | |
| *** stanchan has quit IRC | 17:24 | |
| *** stanchan has joined #openstack-dev | 17:25 | |
| *** lorin1 has quit IRC | 17:25 | |
| mtaylor | zykes-: oneiric already has the version that we've backported in to the other release versions of our ppa | 17:27 |
| openstackgerrit | Anne Gentle proposed a change to openstack/identity-api: Changes pom.xml to create a docs.openstack.org/api/ directory structure for readable URLs plus gives the readme an rst extension https://review.openstack.org/530 | 17:27 |
| openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: POST /tokens: Added missing data https://review.openstack.org/516 | 17:28 |
| *** lorin1 has joined #openstack-dev | 17:29 | |
| *** lorin1 has quit IRC | 17:29 | |
| *** lorin1 has joined #openstack-dev | 17:30 | |
| *** lorin1 has quit IRC | 17:32 | |
| openstackgerrit | A change was merged to openstack/identity-api: Changes pom.xml to create a docs.openstack.org/api/ directory structure for readable URLs plus gives the readme an rst extension https://review.openstack.org/530 | 17:35 |
| *** lorin1 has joined #openstack-dev | 17:40 | |
| *** lorin1 has quit IRC | 17:41 | |
| *** martine has quit IRC | 17:41 | |
| *** lorin1 has joined #openstack-dev | 17:46 | |
| *** zns has joined #openstack-dev | 17:47 | |
| zul | markmc: ping why is the iscsi-tgtadm-choice branch marked pending? | 17:47 |
| *** rcc1 has joined #openstack-dev | 17:48 | |
| markmc | zul, https://code.launchpad.net/~markmc/nova/iscsi-tgtadm-choice/+merge/75906 | 17:49 |
| markmc | zul, I see "Status:Needs review" | 17:49 |
| *** lorin1 has quit IRC | 17:49 | |
| zul | markmc: heh...maybe im going blind in my old age | 17:49 |
| markmc | zul, oh wait, it's Pending in the "reviewer" table | 17:49 |
| * markmc has no idea what the difference is | 17:49 | |
| *** rcc has quit IRC | 17:50 | |
| vishy | mtaylor: were you able to get the release ppa updated? | 17:50 |
| openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: POST /tokens: Added missing data https://review.openstack.org/516 | 17:50 |
| mtaylor | vishy: I just uploaded the package changes to milestone-proposed. waiting to hear from ttx if it's ok that I push that to the release ppa | 17:50 |
| *** blamar has joined #openstack-dev | 17:51 | |
| openstackgerrit | A change was merged to openstack/keystone: Updating guide wrt wadl changes. https://review.openstack.org/536 | 17:51 |
| dolphm | (i hope everyone is enjoying my review spam) | 17:51 |
| *** lorin1 has joined #openstack-dev | 17:52 | |
| joesavak | spam is the sign of progress? | 17:52 |
| annegentle | yeah mine too | 17:53 |
| sandywalsh | vishy, sorry we missed you ... I still have a few questions for you if you're available? | 17:54 |
| vishy | yeah lets do this | 17:54 |
| *** darraghb has quit IRC | 17:56 | |
| openstackgerrit | Yogeshwar Srikrishnan proposed a change to openstack/keystone: Initial changes to move roles to extensions. https://review.openstack.org/537 | 17:57 |
| openstackgerrit | Yogeshwar Srikrishnan proposed a change to openstack/keystone: Initial set of changes to move role operations to extensions. https://review.openstack.org/538 | 17:57 |
| zykes- | what's going to be core then dolphm ? | 17:58 |
| *** lorin1 has quit IRC | 17:58 | |
| rmk | mtaylor: I forgot to file a bug for this but nova-compute should depend on the vlan package | 17:58 |
| dolphm | zykes-: not sure what you're referring to? | 17:58 |
| mtaylor | rmk: yes. I believe that has been fixed | 17:59 |
| rmk | ok cool | 17:59 |
| openstackgerrit | Verification of a change to openstack/glance failed: Adding git-specific Authors file test https://review.openstack.org/451 | 18:00 |
| openstackgerrit | Verification of a change to openstack/glance failed: Implementing changes-since param in api & registry https://review.openstack.org/386 | 18:00 |
| *** zns has quit IRC | 18:00 | |
| openstackgerrit | Yogeshwar Srikrishnan proposed a change to openstack/keystone: Initial set of changes to move role operations to extensions. https://review.openstack.org/539 | 18:02 |
| zykes- | everything as extensions ? | 18:03 |
| jaypipes | dolphm: this may be a stupid question, but can I ask why roles are extensions in Keystone? Seems to me they are a pretty critical part of the API, no? | 18:04 |
| *** RobertLaptop has quit IRC | 18:05 | |
| dolphm | jaypipes: uhhh.... yeah, that's news to me | 18:06 |
| *** jpipes has joined #openstack-dev | 18:06 | |
| jeblair | vishy: are you ready to move nova to git? | 18:06 |
| *** jaypipes has quit IRC | 18:06 | |
| jpipes | dolphm: sorry, power supply on my laptop is dying. random shutoffs... :( | 18:07 |
| jpipes | dolphm: did you answer my question above while I was down? | 18:07 |
| dolphm | <dolphm> jaypipes: uhhh.... yeah, that's news to me | 18:07 |
| dolphm | investigating now | 18:07 |
| openstackgerrit | Mark Gius proposed a change to openstack/swift: Add more specific error messages to swift-ring-builder https://review.openstack.org/540 | 18:07 |
| dolphm | jpipes: yogi just left too | 18:07 |
| jpipes | dolphm: https://review.openstack.org/#change,539,patchset=1 | 18:07 |
| *** heckj has quit IRC | 18:09 | |
| dolphm | jpipes: https://blueprints.launchpad.net/keystone/+spec/identity-api <-- these are the only calls identifies as "core" | 18:09 |
| vishy | jeblair: boy, am I! | 18:10 |
| dolphm | jpipes: so, it looks like all administration of roles is non-core (which makes sense to me), but the concept of roles is still definitely core | 18:10 |
| jeblair | vishy: :) okay, i'll get started on that. | 18:10 |
| jpipes | dolphm: hmm... ok | 18:10 |
| joesavak | jpipes: the crud of roles (add, delete, etc) is extension. Get wasn't. I think it was setup this way as different companies would want their own customized crud to fit their structure | 18:11 |
| vishy | jpipes: you're missing an (ay) | 18:11 |
| jpipes | vishy: ya, freenode still thinks I'm logged in as jaypipes since my power randomly shut off a few minutes ago... | 18:11 |
| vishy | hehe | 18:12 |
| vishy | ttx: are you back yet? | 18:13 |
| jpipes | joesavak: OK. I guess I would say there are differences in implementation of CRUD for Roles, but not so much on the API for it... I mean POST /roles should create a new role, PUT /roles/<ID> would modify an existing role or create a role with a known role identifier, DELETE /roles/<ID> would delete a role, etc... | 18:13 |
| jpipes | joesavak: whether behind the scenes the implementation inserts a role record in an LDAP repo or a record in a database I think is irrelevant to whether the API is in core, right? | 18:14 |
| jpipes | s/core/Identity Core API/ :) | 18:14 |
| jpipes | mtaylor: :( https://jenkins.openstack.org/job/glance/89243/console | 18:15 |
| vishy | jpipes: joesavak: the reason the roles (and user/tenant stuff) is extensions | 18:15 |
| vishy | is because not all backends are writable | 18:15 |
| mtaylor | jpipes: do you know about /msg nickserv ghost jaypipes ? | 18:15 |
| joesavak | jpipes: I agree - same for user crud (also in an that same extension). The issue i think was agreeing on the standard way of doing it. We're looking into SCIM standards for essex' | 18:16 |
| mtaylor | jpipes: it allows you to make nickserv kick your old ghosted login | 18:16 |
| jpipes | vishy: yes, but that just means the API for non-writeable endpoints returns a HTTP Not Implemented. | 18:16 |
| vishy | you don't want keystone writing to AD, enterprises have set ways of doing that | 18:16 |
| *** neogenix has joined #openstack-dev | 18:16 | |
| jpipes | vishy: that's an implementation detail, though, not a reason for having it as an extension IMHO. | 18:16 |
| mtaylor | jpipes: it's actually /msg nickserv ghost jaypipies YOURNICKSERVPASSWORD | 18:16 |
| vishy | jpipes: interesting, that sort of goes agianst the idea of a core api doesn't it. the whole point is that backends have to implement all core methods | 18:16 |
| jpipes | mtaylor: ah, cheers, good to know! | 18:16 |
| dolphm | joesavak: anthony (sleepsonthefloor) is happy with my review, but it still lacks LDAP support (2 test failures i can't explain) | 18:17 |
| vishy | jpipes: btw this is not necessarily my opinion, but that is the general approach that is being taken with the apis | 18:17 |
| jpipes | vishy: I guess I think of it more in terms of "what resources are core to this API" vs "What should every single implementation have to implement in the API" | 18:17 |
| joesavak | dolphm: ok cool. Still morally opposed to turning off ldap tests for now? | 18:18 |
| vishy | jpipes: I think the definition we have been using is: To be compatible, everything in core must be implemented | 18:18 |
| vishy | dolphm: are you using the fakeldap from nova? | 18:18 |
| jpipes | vishy: fair enough. I understand that view. | 18:18 |
| * jpipes makes vishy an extension. ;P | 18:18 | |
| vishy | haha | 18:18 |
| dolphm | vishy: i didn't know nova had a fakeldap -- yuriy contributed "a" fakeldap to keystone | 18:19 |
| *** jpipes is now known as jaypipes | 18:19 | |
| vishy | dolphm: hmm let me look at it | 18:19 |
| dolphm | jaypipes: congrats | 18:19 |
| *** nati_ has quit IRC | 18:19 | |
| vishy | dolphm: yup that is ours | 18:20 |
| joesavak | * makes vishy part of core for he should always be implemented. | 18:20 |
| jaypipes | mtaylor: see the msg above about the GitPython dep on Glance builder? | 18:20 |
| jaypipes | joesavak: :) | 18:20 |
| dolphm | joesavak: i'm only morally opposed to disabling tests as long as we're "supporting" ldap - as soon as we disable the tests (like we did for memcache) it becomes an untested, unmaintained, legacy POS | 18:20 |
| *** nati has joined #openstack-dev | 18:20 | |
| vishy | dolphm: note the us government copyringt header :p | 18:20 |
| mtaylor | jaypipes: yes | 18:20 |
| jaypipes | mtaylor: ok, thx | 18:20 |
| joesavak | dolphm: I agree. I think jesse et al is ok running off of review until we get LDAP fixed | 18:21 |
| jaypipes | bcwaldon: looks like the changes-since branch needs another quick rebase... | 18:21 |
| *** code_franco has joined #openstack-dev | 18:22 | |
| openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: POST /tokens: A chronicle of missing features https://review.openstack.org/516 | 18:22 |
| *** code_franco has quit IRC | 18:22 | |
| dolphm | LDAP testing disabled ^^^ the above WILL merge successfully | 18:22 |
| vishy | dolphm: if i wanted to grab the branch to see test failures, where would I get it? | 18:22 |
| *** naehring has quit IRC | 18:23 | |
| *** cp16net has joined #openstack-dev | 18:23 | |
| dolphm | vishy: checkout https://review.openstack.org/516 , open run_tests.py in root, un-comment LDAPTest | 18:23 |
| vishy | dolphm: hmm, i seem to be hanging after Running tests... | 18:24 |
| vishy | dolphm: do the tests run on osx? | 18:24 |
| jaypipes | Rackermail 503'ing unavailable yet again... | 18:24 |
| mtaylor | jaypipes: thanks... the continued non-existence of that helped me find a bug in the system | 18:24 |
| dolphm | vishy: if you have all the dependencies installed, sure | 18:24 |
| rmk | Hmm isn't nova supposed to detect when a hypervisor and associated VMs go down? | 18:24 |
| jaypipes | mtaylor: you are SO welcome! :) | 18:24 |
| vishy | i did pip install -r tools/pip-requires | 18:25 |
| rmk | My HV has been down for an hour and it still has no idea | 18:25 |
| vishy | rmk: no, it detects when vms go down | 18:25 |
| dolphm | vishy: test progress is suppressed, i guess for jenkins... anyway, you can use ./run_tests.sh -N --with-progress | 18:25 |
| vishy | but not hv | 18:25 |
| rmk | vishy: yeah they're still showing up | 18:25 |
| dolphm | vishy: which will show you execution progress, obviously | 18:25 |
| jaypipes | rmk: yes. that is the get_a_nast_call_from_customer.py module. | 18:25 |
| rmk | I'm intentionally trying to break it -- shutdown a compute node and the VMs all still show as active | 18:26 |
| zykes- | when you gonn get that merged dolphm ? ;) | 18:26 |
| dolphm | zykes-: it's capable of being merged now, unless vishy thinks he can kick fakeldap into shape.. | 18:26 |
| vishy | dolphm: go ahead I was just going to see if anything jumped out | 18:27 |
| dolphm | all in favor of abandoning support for keystone's ldap backend, say `aye` | 18:27 |
| zykes- | why abondon? | 18:28 |
| dolphm | zykes-: abandoning *support* - because we're turning off tests for it | 18:28 |
| dolphm | jaypipes: rackermail works for me, at the moment | 18:30 |
| vishy | dolphm: since I am behind on gerrit usage. Is there a magic set of commands to pull that branch for testing | 18:30 |
| vishy | ? | 18:30 |
| dolphm | vishy: yes, but it's a secret | 18:31 |
| vishy | ah i see it | 18:31 |
| vishy | there is a git fetch link | 18:31 |
| dolphm | vishy: https://review.openstack.org/#change,516 , scroll down to patch set 13, select Checkout and SSH, then copy the command it gives you | 18:31 |
| mtaylor | vishy: if you look on that page, you'll see a bit in the middle with a git pull command ... you found it | 18:31 |
| jaypipes | dolphm, joesavak: thought you would like this: http://thedailywtf.com/Articles/Python-Charmer.aspx | 18:32 |
| jaypipes | dolphm: it's the Outlook Web Access that is down, look like... | 18:32 |
| openstackgerrit | Brian Waldon proposed a change to openstack/glance: Adding git-specific Authors file test https://review.openstack.org/451 | 18:33 |
| dolphm | jaypipes: holy crap i thought that was just ascii art at first | 18:33 |
| jaypipes | lol | 18:33 |
| vishy | dolphm: tests running I'll tell you if anything looks fancy | 18:33 |
| dolphm | vishy: did you re-enable LDAPTest? | 18:33 |
| vishy | yup | 18:34 |
| vishy | i see two failures | 18:34 |
| bcwaldon | jaypipes: authors test merge still failing... | 18:34 |
| dolphm | vishy: that's what i see | 18:34 |
| *** Gordonz has quit IRC | 18:34 | |
| jaypipes | bcwaldon: yes, I know :) mtaylor is on it. | 18:34 |
| mtaylor | bcwaldon: yes. I'm working on it -- I broke something | 18:34 |
| bcwaldon | mtaylor: thanks | 18:34 |
| bcwaldon | jaypipes: I'm assuming we'll need this test for nova, too | 18:34 |
| jaypipes | bcwaldon: yup | 18:34 |
| vishy | dolphm: can i just run one test? | 18:36 |
| vishy | dolphm: or one class of tests? | 18:36 |
| dolphm | vishy: you can run it manually... fire up ./bin/keystone | 18:36 |
| dolphm | vishy: run ./bin/sampledata (i think) | 18:37 |
| * mtaylor is so happy that all of the packages are GREEN in the trunk ppa finally | 18:37 | |
| mtaylor | the lucid dh_python2 cantrip makes that so much prettier | 18:37 |
| dolphm | vishy: and then you can do `unittest2 discover keystone.test.functional.whatever` | 18:37 |
| vishy | mtaylor: not sure what happened to ttx? | 18:37 |
| mtaylor | vishy: perhaps he went and got drunk? | 18:37 |
| openstackgerrit | Brian Waldon proposed a change to openstack/glance: Implementing changes-since param in api & registry https://review.openstack.org/386 | 18:37 |
| *** mgius has quit IRC | 18:38 | |
| mtaylor | vishy: in any case, updated packages are in milestone-proposed: https://launchpad.net/~nova-core/+archive/milestone-proposed | 18:38 |
| mtaylor | vishy: I can move them over to release any time | 18:38 |
| *** mgius has joined #openstack-dev | 18:38 | |
| vishy | dolphm: unittest2 doesn't exist | 18:39 |
| rmk | Does nova regularly check the status of VMs it thinks should be up? | 18:39 |
| rmk | This is scaring me now | 18:39 |
| vishy | rmk: yes | 18:39 |
| vishy | the compute worker does | 18:39 |
| rmk | What if the compute worker is down? | 18:39 |
| vishy | then it doesn't | 18:39 |
| rmk | The one responsible for launching those VMs | 18:39 |
| vishy | but it will know that the compute worker is broken | 18:39 |
| vishy | it won't change the status of the vms though | 18:40 |
| dolphm | vishy: try `unit2` | 18:40 |
| rmk | ok so recovering from a state like this is something I need to script | 18:40 |
| bcwaldon | jaypipes: my merge was bad, btw | 18:40 |
| bcwaldon | jaypipes: didnt notice migration numbers | 18:40 |
| jaypipes | bcwaldon: crap. | 18:41 |
| jaypipes | bcwaldon: just approved... | 18:41 |
| bcwaldon | jaypipes: yeah, I know | 18:41 |
| jaypipes | heh | 18:41 |
| bcwaldon | jaypipes: I was going to tell you it was good then ran the tests and realized i screwed up | 18:41 |
| jaypipes | well, it will fail anyway. no worries | 18:41 |
| bcwaldon | jaypipes: running tests then I'll push and approve the review | 18:41 |
| openstackgerrit | Verification of a change to openstack/glance failed: Implementing changes-since param in api & registry https://review.openstack.org/386 | 18:41 |
| bcwaldon | jaypipes: don't worry about it anymoer | 18:41 |
| jaypipes | Zis is why we haz zie tests! | 18:41 |
| joesavak | jaypipes: python charmer = headache (but potential code re-use) ;) | 18:41 |
| jaypipes | joesavak: hehe | 18:42 |
| mtaylor | bcwaldon, jaypipes: ok. python-git is now there | 18:42 |
| bcwaldon | mtaylor: thank you, sir | 18:42 |
| jaypipes | mtaylor: rock. thx mate. BTW, what was the bug it exposed? | 18:42 |
| mtaylor | jaypipes: I had a broken crontab on some of the slaves which we thought we'd fixed across the board a while ago, but had missed a couple of slaves | 18:43 |
| mtaylor | jaypipes: so puppet wasn't updating on those hosts | 18:43 |
| jaypipes | mtaylor: gotcha. k. | 18:43 |
| openstackgerrit | Brian Waldon proposed a change to openstack/glance: Implementing changes-since param in api & registry https://review.openstack.org/386 | 18:44 |
| dolphm | vishy: unit2 keystone.test.functional.test_auth.TestAdminAuthenticationNegative | 18:44 |
| openstackgerrit | Verification of a change to openstack/glance failed: Adding git-specific Authors file test https://review.openstack.org/451 | 18:44 |
| dolphm | vishy: ... Ran 3 tests in 0.591s | 18:44 |
| openstackjenkins | Project nova-docs build #1,412: SUCCESS in 4 min 0 sec: https://jenkins.openstack.org/job/nova-docs/1412/ | 18:44 |
| bcwaldon | mtaylor: are you sure you installed the right python-git package? | 18:45 |
| vishy | dolphm, now i just have to get it to start with the ldap backend | 18:45 |
| vishy | :) | 18:45 |
| bcwaldon | mtaylor: I need GitPython from pypi | 18:45 |
| bcwaldon | mtaylor: looks like there are two python+git packages and they both install a 'git' module | 18:45 |
| rmk | Just so I am clear on this, if I lose a compute-node+hypervisor, I need to query nova regularly to notice, then manually iterate through the VMs associated with that compute node to migrate them or start them elsewhere? | 18:45 |
| mtaylor | bcwaldon: I'm guessing now that GitPython from PyPI is different from python-git which is in ubuntu | 18:45 |
| mtaylor | bcwaldon: god, that makes me SO happy | 18:45 |
| dolphm | vishy: lol uhh... reference the ldap.conf.template from keystone/test/etc | 18:45 |
| bcwaldon | mtaylor: unfortunately...yes | 18:45 |
| vishy | yeah, tried that | 18:45 |
| *** mfer has quit IRC | 18:46 | |
| vishy | gotta make the substitutions manually | 18:46 |
| mtaylor | heaven forbid anybody work together... grumble. | 18:46 |
| mtaylor | bcwaldon: ok. I'm on it | 18:46 |
| openstackgerrit | A change was merged to openstack/glance: Implementing changes-since param in api & registry https://review.openstack.org/386 | 18:46 |
| bcwaldon | mtaylor: thanks. sorry! | 18:46 |
| mtaylor | bcwaldon: it will be a few - I'll need to do some packaging here | 18:46 |
| dolphm | mtaylor: EVERY PROJECT FOR ITSELF | 18:46 |
| bcwaldon | mtaylor: yeah, it can wait a bit longer... | 18:46 |
| vishy | got it | 18:46 |
| vishy | keystone.backends.ldap.fakeldap: ERROR FakeLDAP modify item failed: dn 'cn=40f7a37e-8508-4a43-ad7e-de6aae7907b7,cn=977895a4-eac5-4381-a41c-bb91ac7f77f5,ou=Groups,dc=example,dc=com' not found. | 18:47 |
| * vishy found the error at least! | 18:47 | |
| *** mfer has joined #openstack-dev | 18:47 | |
| * dolphm has no idea what that entire string means | 18:47 | |
| openstackgerrit | sleepsonthefloor proposed a change to openstack/keystone: Fixes auth_token middleware to allow admin users in nova https://review.openstack.org/541 | 18:47 |
| jeblair | mtaylor: tarmac config for nova is: bash run_tests.sh -N && python setup.py sdist && pep8 --repeat nova | 18:47 |
| vishy | dolphm: seems like the tests should capture output from keystone in verbose mode and display it | 18:47 |
| dolphm | distinguished name... something something something example.com! | 18:48 |
| mtaylor | jeblair: yeah - we should not do that | 18:48 |
| jeblair | mtaylor: i'm assuming we want 'nova' job to 'bash run_tests.sh -N' | 18:48 |
| mtaylor | jeblair: correct | 18:48 |
| mtaylor | jeblair: and then we should have a pep8 job | 18:48 |
| jeblair | mtaylor: and a pep8 job | 18:48 |
| vishy | dolphm: is there any way to get keystone to log a stack trace on error? | 18:48 |
| jeblair | mtaylor: what's with the sdist? | 18:48 |
| dolphm | vishy: stack traces are going to the log defined in the conf file | 18:48 |
| mtaylor | vishy: is there any way to get run_tests.sh to _not_ run pep8? | 18:48 |
| dolphm | vishy: err backtraces, in pythonland | 18:49 |
| mtaylor | jeblair: I think it was a reaction to a time that someone did something that broke sdist | 18:49 |
| mtaylor | jeblair: we have enough other stuff that counts on that working now I'm fine with not doing it at the moment | 18:49 |
| vishy | dolphm: wow just about all of the modify items are not working | 18:49 |
| jeblair | mtaylor: so if we want it, it should be an sdist job. but we don't want it. okay. :) | 18:49 |
| mtaylor | jeblair: yes to both | 18:49 |
| dolphm | vishy: IT'S A PATTERN! | 18:49 |
| vishy | dolphm: and btw, no stack traces, just error | 18:50 |
| * vishy goes to get stack traces | 18:50 | |
| jaypipes | bcwaldon: https://jenkins.openstack.org/job/glance/89246/console | 18:51 |
| mtaylor | bcwaldon: http://pypi.python.org/pypi/GitPython/0.3.2.RC1 is what you want, right? | 18:51 |
| dolphm | vishy: isn't keystone awesome? | 18:51 |
| vishy | it is very odd there are no stack traces | 18:52 |
| vishy | perhaps they are falling through to syslog | 18:52 |
| rmk | I'm finding it a bit hard to believe the compute node/hypervisor dying use case and gracefully recovering by shuffling VMs around isn't accounted for in any way | 18:53 |
| dolphm | vishy: i sort of doubt that... there's a trend in keystone to catch and suppress errors during development... because, you know, developers want "friendly" error messages and all | 18:53 |
| jaypipes | mtaylor: yes, the error is unrelated AFAICT | 18:53 |
| vishy | dolphm: i don't really see where they are getting surpressed though | 18:54 |
| vishy | there is a logging.exception | 18:54 |
| vishy | which should print a stack trace | 18:54 |
| vishy | hmm, still nothing | 18:55 |
| bcwaldon | mtaylor: yes | 18:55 |
| dolphm | fakeldap.py:222-224 | 18:55 |
| bcwaldon | jaypipes: yeah, wrong git python package | 18:56 |
| * mtaylor starts crying | 18:56 | |
| jaypipes | oink. | 18:56 |
| mtaylor | "As this version of GitPython depends on GitDB, which in turn needs Async to work," | 18:56 |
| bcwaldon | mtaylor: want me to reimplement using the other lib? | 18:58 |
| vishy | ah thx | 18:58 |
| vishy | dolphm: those should probably be log.exception | 18:58 |
| mtaylor | bcwaldon: the other lib is just an older version | 18:58 |
| mtaylor | bcwaldon: I'll get it - just will take a few more moments | 18:58 |
| bcwaldon | mtaylor: kk | 18:58 |
| dolphm | vishy: isn't error is *for* exceptions (?) | 18:59 |
| vishy | logging.exception logs at error and prints stack trace | 19:00 |
| ttx | vishy: back | 19:00 |
| ttx | vishy: not too drunk | 19:00 |
| vishy | logging.error just logs at error | 19:00 |
| ttx | wassup ? | 19:00 |
| vishy | ttx: we had to fix the packaging | 19:00 |
| vishy | ttx: just wanted to make sure it was ok to push the new packages to release | 19:00 |
| ttx | vishy: to the release ppa ? sure | 19:00 |
| ttx | as long as we don't change the tarball... | 19:00 |
| ttx | ad we do it early... | 19:01 |
| ttx | mtaylor: you changed all ppas ? | 19:01 |
| dolphm | vishy: ah, got it | 19:01 |
| ttx | mtaylor: milestone-proposed, milestone and release should get it | 19:01 |
| vishy | cool | 19:01 |
| ttx | mtaylor: ideally you would just retrigger nova-milestone-ppa and then use sync_ppa to sync to milestone, then release | 19:02 |
| jeblair | mtaylor, ttx: i have disabled all nova jobs other than the ones i've converted to git. | 19:03 |
| jeblair | (nova bzr->git migration in progress) | 19:03 |
| ttx | nothing like doing everything at the same time | 19:03 |
| jeblair | theoretically this was to be after. but the lines have been a bit fuzzy today. | 19:04 |
| ttx | vishy: we went back to the creperie we were in on Tuesday. | 19:04 |
| ttx | jeblair: the lines are always fuzzy o nrelease day | 19:04 |
| dolphm | ttx: are you in SF? | 19:04 |
| zykes- | all projects @ git soon ? | 19:04 |
| ttx | I started releasing at 9am CET and finished at 6pm :) | 19:04 |
| ttx | dolphm: in Paris right now | 19:05 |
| * dolphm is jealous. | 19:05 | |
| ttx | dolphm: as Vish is. | 19:05 |
| zykes- | urk, i want to go there again ttx :( | 19:05 |
| bcwaldon | vishy: is devin around today? | 19:05 |
| jeblair | zykes-: all core diablo projects on git _very_ soon. | 19:05 |
| *** RobertLaptop has joined #openstack-dev | 19:05 | |
| mtaylor | ttx: I was not sure that the nova-milestone-ppa job would do that right thing in this context | 19:05 |
| openstackgerrit | James E. Blair proposed a change to openstack/nova: Add rfc.sh to help with gerrit workflow. https://review.openstack.org/542 | 19:06 |
| ttx | mtaylor: it should have, but manual works too | 19:06 |
| ttx | mtaylor: all set ? | 19:07 |
| ttx | what was the change about btw ? | 19:07 |
| rmk | Just a confirmation, I need to script dealing with a compute/hypervisor failure myself, yes? | 19:07 |
| mtaylor | ttx: yes. it's in milestone-proposed right now | 19:07 |
| mtaylor | ttx: the packaging change to use deprecated_auth was only half done | 19:07 |
| rmk | No way to get nova to auto restart VMs on other compute nodes in the zone? | 19:07 |
| ttx | mtaylor: ? | 19:08 |
| ttx | mtaylor: that needed a packaging change ? | 19:08 |
| mtaylor | ttx: yes. it was broken as shipped | 19:08 |
| ttx | cool | 19:08 |
| ttx | mtaylor: if it's ok now, please sync up asap | 19:08 |
| dolphm | vishy: how goes it? | 19:08 |
| mtaylor | ttx: the nova.conf that was shipped had an option that required a change in the paste config - but the paste config was not changed | 19:08 |
| vishy | looking | 19:08 |
| vishy | still | 19:09 |
| mtaylor | ttx: I think I can just copy the new package versions from the milestone-proposed ppa to milestone and release, yeah? | 19:09 |
| ttx | sure | 19:09 |
| zul | does anyone know why we are using iscsitarget rather than tgtadmin i was just wondering what the historical reasons are | 19:10 |
| vishy | dolphm, interesting, i bet it is the : in ANOTHER:TENANT | 19:11 |
| dolphm | vishy: that would be awesome | 19:11 |
| vishy | dolphm: darn | 19:12 |
| vishy | :( | 19:12 |
| mtaylor | ttx, vishy: done | 19:13 |
| openstackgerrit | Verification of a change to openstack/nova failed: Add rfc.sh to help with gerrit workflow. https://review.openstack.org/542 | 19:14 |
| jeblair | File "/home/jenkins/workspace/nova/nova/tests/test_misc.py", line 70, in test_authors_up_to_date | 19:15 |
| jeblair | import git | 19:15 |
| jeblair | ImportError: No module named git | 19:15 |
| dolphm | vishy: that doesn't seem to help for me | 19:16 |
| vishy | dolphm: yeah i tried it hence the darn :( above | 19:16 |
| dolphm | vishy: ah | 19:16 |
| jeblair | mtaylor: nova now needs a python git package to pass tests | 19:17 |
| sandywalsh | ok ... gerrit is being a pain in the ass | 19:18 |
| sandywalsh | Hint: A potential Change-Id was found, but it was not in the footer of the commit message. | 19:18 |
| sandywalsh | how do I fix that? | 19:18 |
| dolphm | sandywalsh: make sure the Change-Id is the last line in the commit? | 19:19 |
| sandywalsh | did that | 19:19 |
| sandywalsh | nothing | 19:19 |
| dolphm | sandywalsh: and that you only have one Change-Id? | 19:19 |
| sandywalsh | that I can see | 19:19 |
| vishy | dolphm: interesting, it is this line that is failing: api.USER.get_by_tenant(user_id, tenant_id) | 19:19 |
| ameade | sandywalsh: if you have a conflict message below that, delete the conflict message | 19:19 |
| mtaylor | jeremyb: working on it | 19:20 |
| mtaylor | jeblair: working on it right now | 19:20 |
| dolphm | vishy: eek, both ldap.api.user and sqlalchemy.api.user have a different implementation of that! | 19:20 |
| sandywalsh | Here's what I've got http://paste.openstack.org/show/2527/ ameade dolphm | 19:20 |
| dolphm | sandywalsh: git rid of the Conflicts section | 19:21 |
| dolphm | sandywalsh: also, there's two Change-Id's in there | 19:21 |
| *** zns has joined #openstack-dev | 19:21 | |
| sandywalsh | how? where? that's what 'git review' is spitting out | 19:21 |
| jeblair | sandywalsh: that's your commit message, edit it with 'git commit --amend' | 19:22 |
| sandywalsh | no, this is my commit message: http://paste.openstack.org/show/2528/ | 19:22 |
| dolphm | sandywalsh: (and the second Change-Id is "wrong" - use the first) | 19:22 |
| sandywalsh | (from within nano) | 19:23 |
| dolphm | sandywalsh: ^^ swap your Change-Id out for Ied52b5c561a6f07d499044dabe9dcb455bf1e7eb | 19:23 |
| dolphm | sandywalsh: which you can find at the top of the review: https://review.openstack.org/#change,359 | 19:23 |
| sandywalsh | :/ | 19:23 |
| dolphm | mtaylor: the magical-do-everything-for-you version of `git review` is a bit of a headache.. | 19:24 |
| mtaylor | dolphm: yeah? | 19:24 |
| sandywalsh | same problem http://paste.openstack.org/show/2529/ | 19:24 |
| dolphm | mtaylor: i'm really not a fan of the auto-rebasing | 19:24 |
| dolphm | mtaylor: if it's going to do that, i at least want it to stop and confirm... "hey, you probably need to rebase this on master, would you like me to run `git rebase master` for you?" | 19:25 |
| mtaylor | dolphm: k. there are several directions we can go with that | 19:25 |
| sandywalsh | I'd like some direction | 19:25 |
| ameade | mtaylor: I agree that it should be explicit, i got so confused one time | 19:25 |
| jeblair | sandywalsh: can you paste the commit message as it appears from 'git log' ? | 19:25 |
| sandywalsh | jeblair, http://paste.openstack.org/show/2530/ | 19:26 |
| dolphm | sandywalsh: can you push your change as-is to some other repo we can access? github.com/you/keystone or something? | 19:27 |
| jeblair | the last two changes have the same change id in their commit messages, and the 7048a0e3 change doesn't have a commit message other than the change-id | 19:28 |
| *** martine has joined #openstack-dev | 19:28 | |
| jeblair | should the top two changes be squashed? | 19:28 |
| jeblair | oh! | 19:29 |
| dolphm | jeblair: how can you tell that? sandywalsh: can you post your recent git log? | 19:29 |
| jeblair | dolphm: http://paste.openstack.org/show/2530/ | 19:29 |
| mtaylor | jeblair: looks to me like everything after 10f4511037bacbd1f80f4846acb0baa45751bbc7 should get squashed | 19:29 |
| jaypipes | looks like wwkeyboard has just put up his first branch for review. let's pounce! | 19:30 |
| jeblair | it's commit 2c5cd63d6fef1d95f94b09d7f663d444361bdb00 that it's complaining about | 19:30 |
| mtaylor | ++ | 19:30 |
| *** zns has quit IRC | 19:30 | |
| jeblair | sandywalsh: you're uploading many commits to gerrit. is that your intention? | 19:30 |
| wwkeyboard | jaypipes: fire away! | 19:31 |
| jeblair | each commit will become a change, and if it has an existing change-id, will update that change. | 19:31 |
| sandywalsh | jeblair, are you kiddin? | 19:31 |
| sandywalsh | this thing is beyond brain dead | 19:31 |
| dolphm | sandywalsh: you need to squash: git rebase -i HEAD~4 | 19:31 |
| jaypipes | wwkeyboard: :) | 19:31 |
| * jaypipes grants sandywalsh the Elixir of Serendipity | 19:32 | |
| jeblair | sandywalsh: the commit you describe as "get service catalog from token - pep8" is 2c5cd63d6fef1d95f94b09d7f663d444361bdb00 in your history | 19:33 |
| jeblair | gerrit doesn't have that commit | 19:33 |
| jeblair | so when you push your tree, that's a new commit to gerrit. | 19:33 |
| zykes- | git issues in openstack land ? | 19:33 |
| sandywalsh | jeblair, I have no idea wtf is going on here. I'm tempted to blow away my repo and recode the thing | 19:33 |
| sandywalsh | franken-git | 19:34 |
| jeblair | sandywalsh: i can't tell you how your repo got into that state, all I can see from this is that you have changes that aren't in gerrit. that may be because of a merge, rebase, ammended commit, etc. | 19:34 |
| jeblair | sandywalsh: from your git log, is there one change that you want to submit to gerrit? | 19:35 |
| sandywalsh | jeblair, best I can tell, this repo is golden https://github.com/SandyWalsh/keystone/tree/token_2_service_catalog | 19:36 |
| *** martine has quit IRC | 19:36 | |
| sandywalsh | ignore that ... it's still fucked | 19:37 |
| mtaylor | sandywalsh: there are 5 commits in that branch that are not in the upstream branch. what is your intent here - is your intent that there is one set of changes you are wanting to submit for review? | 19:38 |
| sandywalsh | guys, of course this is not my intent. My intent was a one-line change and a push. I'm sorry if this resulted in an avalanche of nonsense. | 19:39 |
| sandywalsh | I think the larger question is, how does one recover from these messes | 19:39 |
| mtaylor | sandywalsh: I'm not trying to imply that - I'm just making sure we're on the same page so that I can help you fix it and also explain what's going on so you don't get there again | 19:39 |
| jeblair | sandywalsh: which commit has what you want to submit? | 19:40 |
| sandywalsh | well, right now I have to get back to a branch that works again ... stay tuned. My repo is fried. | 19:40 |
| openstackgerrit | A change was merged to openstack/keystone: Fixes auth_token middleware to allow admin users in nova https://review.openstack.org/541 | 19:46 |
| openstackgerrit | Jay Pipes proposed a change to openstack/glance: Adds SSL configuration params to the client https://review.openstack.org/190 | 19:47 |
| openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: Get Service Catalog from token https://review.openstack.org/359 | 19:47 |
| dolphm | sandywalsh: i think i got you taken care of... review that ^^ | 19:47 |
| dolphm | mtaylor: ^^ | 19:48 |
| mtaylor | dolphm: ? | 19:49 |
| *** rcc1 has left #openstack-dev | 19:49 | |
| dolphm | sandywalsh: does that include all your latest changes | 19:49 |
| dolphm | mtaylor: i think i got a patch for sandy that will merge, in case you were doing the same | 19:49 |
| mtaylor | dolphm: awesome. thanks... no, I'm off in the land of packaging all of the things that git-python needs to operate | 19:50 |
| dolphm | mtaylor: cool | 19:50 |
| openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: Get Service Catalog from token - pep8 https://review.openstack.org/359 | 19:53 |
| sandywalsh | the rebase --continue is cycling on the same thing and reverting my changes each time | 19:54 |
| sandywalsh | here's where I stand: http://paste.openstack.org/show/2531/ | 19:56 |
| dolphm | sandywalsh: open the review- i pulled your code, rebased it, fixed a couple pep8 issues, and put it back up for review | 19:57 |
| *** mfer has quit IRC | 19:58 | |
| openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: Get Service Catalog from token https://review.openstack.org/359 | 19:58 |
| dolphm | and just changed pymox to mox ^ | 19:58 |
| *** ameade has quit IRC | 19:58 | |
| sandywalsh | dolphm, can I grab that branch and work from there? | 19:58 |
| sandywalsh | this branch is screwed | 19:59 |
| dolphm | absolutely | 19:59 |
| sandywalsh | the offending test still has to get fixed | 19:59 |
| dolphm | sandywalsh: use the checkout link on gerrit | 19:59 |
| *** mfer has joined #openstack-dev | 19:59 | |
| dolphm | sandywalsh: err, command, not link | 19:59 |
| dolphm | sandywalsh: hit Checkout, and SSH, and then copy the command it shows | 20:00 |
| dolphm | sandywalsh: after you make your changes, stage them, and then just `git commit --amend` to avoid creating new commits :) | 20:01 |
| sandywalsh | hmm, I want a new repo ... wouldn't I need to clone first? | 20:02 |
| dolphm | sandywalsh: you shouldn't need to? but yes | 20:02 |
| dolphm | shouldn't need a new repo* | 20:02 |
| sandywalsh | mine is pretty screwed | 20:02 |
| dolphm | when you checkout, it'll put you into a clean branch | 20:03 |
| dolphm | mostly because i cleaned it :) | 20:03 |
| rmk | Doesn't seem like the rabbit python bindings implement tcp keepalives | 20:04 |
| sandywalsh | well, that didn't go so smooth http://paste.openstack.org/show/2532/ | 20:04 |
| sandywalsh | aborted the last rebase | 20:04 |
| sandywalsh | better now | 20:04 |
| dolphm | sandywalsh: i've never seen git do anything like that after a checkout lol | 20:05 |
| sandywalsh | as I was saying | 20:06 |
| rmk | what's milestone-proposed representing now? it isn't essex yet is it? | 20:06 |
| sandywalsh | dolphm, strange ... tests all pass for me. perhaps an ldap test failing | 20:07 |
| vishy | rmk: no it should be == milestone == release | 20:07 |
| sandywalsh | dolphm, do they run for you in that branch? | 20:07 |
| rmk | vishy: No it's not essex then? | 20:08 |
| vishy | correct | 20:08 |
| dolphm | sandywalsh: did you fix the tests in one of your commits? | 20:08 |
| vishy | trunk is essex atm | 20:08 |
| sandywalsh | dolphm, nope | 20:08 |
| rmk | right I guess I'm not familiar with what the next milestone is | 20:08 |
| rmk | diablo+? | 20:09 |
| dolphm | well, are all your changes that you want included? | 20:09 |
| vishy | dolphm: any idea what the purpose of use_dumb_member is? | 20:09 |
| vishy | rmk: next milestone will be e1 | 20:09 |
| dolphm | vishy: to obfuscate fakeldap a little more? | 20:09 |
| sandywalsh | dolphm, see I'm getting a similar problem now. Changed one file. 'git commit --amend' ... then 'git review' and swalsh@novadev:~/github/keystone$ vi etc/keystone.conf | 20:10 |
| sandywalsh | swalsh@novadev:~/github/keystone$ git commit --amend | 20:10 |
| sandywalsh | [tryagain bf0c432] Get Service Catalog from token | 20:10 |
| sandywalsh | 6 files changed, 102 insertions(+), 2 deletions(-) | 20:10 |
| sandywalsh | swalsh@novadev:~/github/keystone$ git review | 20:10 |
| sandywalsh | etc/keystone.conf: needs update | 20:10 |
| sandywalsh | Working tree is dirty | 20:10 |
| sandywalsh | swalsh@novadev:~/github/keystone$ | 20:10 |
| sandywalsh | whoops | 20:10 |
| sandywalsh | was supposed to be paste | 20:10 |
| sandywalsh | http://paste.openstack.org/show/2533/ | 20:10 |
| dolphm | lol | 20:10 |
| vishy | hmm, perhaps it is because you can't have a group without a member | 20:10 |
| dolphm | sandywalsh: stage etc/keystone.conf before you `git commit --amend` | 20:11 |
| mtaylor | (you might need to do git add etc/keystone.conf ; git commit --amend _or_ do git commit -a --amend) | 20:11 |
| dolphm | sandywalsh: git add etc/keystone.conf` | 20:11 |
| openstackgerrit | Sandy Walsh proposed a change to openstack/keystone: Get Service Catalog from token https://review.openstack.org/359 | 20:12 |
| sandywalsh | hmm | 20:12 |
| sandywalsh | why did I need to do that? | 20:12 |
| dolphm | sandywalsh: git doesn't want to assume exactly what you want to commit | 20:12 |
| sandywalsh | it would normally | 20:12 |
| *** dysinger has joined #openstack-dev | 20:13 | |
| dolphm | sandywalsh: do you use `git commit -a` normally? | 20:13 |
| dolphm | i.e. git commit --all | 20:13 |
| sandywalsh | not that time ... normally I would, but the instructions were to --amend | 20:13 |
| dysinger | heads up: the new milestone packages on nova-core ppa seem broken (from 4 hours ago) | 20:14 |
| dolphm | sandywalsh: right, so normally you're telling git to stage EVERYTHING (which is quite explicit, and git doesn't have to assume) | 20:14 |
| mtaylor | dysinger: in which way? | 20:14 |
| dysinger | mysql interaction | 20:14 |
| dysinger | tons of entries in the mysql slow.log | 20:15 |
| *** neogenix has quit IRC | 20:15 | |
| dolphm | sandywalsh: `git commit -a` is a shortcut for `git add . && git commit` (roughly speaking) | 20:15 |
| dysinger | massive queries | 20:15 |
| dysinger | mysql disconnect | 20:15 |
| mtaylor | ah, well, that is not the bugfix I updated earlier | 20:15 |
| mtaylor | vishy: ^^ | 20:15 |
| dysinger | I'm trying the milestone-release ppa now | 20:15 |
| dolphm | sandywalsh: anyway, let's see what jenkins says :) | 20:16 |
| vishy | dysinger: they should be the same package | 20:16 |
| sandywalsh | dolphm, I thought the --amend was just 'reuse the last commit message' | 20:16 |
| dysinger | sorry milestone-proposed | 20:16 |
| mtaylor | dysinger: yes. they should all be the same | 20:16 |
| dysinger | ok | 20:16 |
| mtaylor | dysinger: release, milestone and milestone-proposed are identical at the moment | 20:16 |
| dolphm | sandywalsh: more specifically, reuse the last *commit* | 20:16 |
| sandywalsh | well, yes | 20:17 |
| dolphm | sandywalsh: it goes back and rewrites your commit history | 20:17 |
| vishy | dolphm: I may be making progress. It appears the role is being added to a uuid tenant_id but then checked on a string tenant_id | 20:17 |
| dolphm | sandywalsh: instead of appending to it | 20:17 |
| sandywalsh | -a acting as 'add .' is news to me though. I've never seen that behavior in the wild. | 20:17 |
| dolphm | vishy: i'm rooting for you :) | 20:18 |
| sandywalsh | I've always explicitly done 'add' | 20:18 |
| dolphm | sandywalsh: if you explicitly `git add` your files, you don't need to `git commit -a` | 20:18 |
| sandywalsh | ah, -a = "all tracked, modified files" ... as I expected | 20:18 |
| sandywalsh | so it should have been 'git commit -a --amend' | 20:19 |
| openstackgerrit | A change was merged to openstack/keystone: Get Service Catalog from token https://review.openstack.org/359 | 20:19 |
| dolphm | yeah, `git commit -a` is sort of like mercurial's `hg addremove`, i guess | 20:19 |
| dolphm | success! | 20:19 |
| sandywalsh | let's see what jenkins has to say | 20:19 |
| sandywalsh | merged | 20:20 |
| sandywalsh | hmm | 20:20 |
| sandywalsh | I changed nothing | 20:20 |
| sandywalsh | dolphm, did you happen to fix any tests? | 20:20 |
| dolphm | sandywalsh: nope | 20:20 |
| dolphm | sandywalsh: i only fixed pep8 and stuff for you | 20:21 |
| sandywalsh | curiouser and curiouser | 20:21 |
| dysinger | mtaylor: vishy: I just looked an hour ago or so - the nova-core/release packages are older | 20:21 |
| sandywalsh | well, I have to say I'm stumped | 20:21 |
| dysinger | maybe there's a buffer time to get it up there | 20:21 |
| dolphm | sandywalsh: you turned off verbose & debug lol... maybe that was it :P | 20:21 |
| sandywalsh | well, the tests all passed for me | 20:21 |
| sandywalsh | well, whatever happened I have no idea ... but thanks dolphm (and jeblair ameade mtaylor) | 20:24 |
| sandywalsh | sorry for the frustration ... perhaps some magical lightbulb will go on for me with this (but don't hold your breathe ;) | 20:25 |
| dolphm | sandywalsh: no frustration over here :) | 20:25 |
| *** stuntmachine has quit IRC | 20:25 | |
| vishy | dolphm: is the sql layer smart enough to search tenants by name or id? | 20:26 |
| *** RobertLaptop has quit IRC | 20:27 | |
| dolphm | vishy: it's not really supposed to, but the sql user API does... look at sqlalchemy.api.user line 48 | 20:27 |
| vishy | dolphm, beause i think that is the problem | 20:28 |
| dolphm | vishy: explain? | 20:28 |
| dolphm | vishy: oh wait... | 20:30 |
| vishy | well it is authenticating with the tenant_id of 'admin' | 20:30 |
| vishy | instead of the uuid | 20:30 |
| vishy | trying to figure out where that id is set now | 20:30 |
| *** nati has quit IRC | 20:30 | |
| *** nati has joined #openstack-dev | 20:30 | |
| vishy | dolphm: where do the uuids come from? | 20:32 |
| dolphm | vishy: only Tokens get UUID's | 20:33 |
| vishy | dolphm: curious | 20:33 |
| dolphm | vishy: in service.py line 130 | 20:33 |
| vishy | no common test makes them | 20:34 |
| vishy | optional_str = lambda x: x if x is not None else unique_str() | 20:34 |
| westmaas | but it should be authing with id, not name | 20:34 |
| westmaas | whether id is a uuid, or not. | 20:34 |
| * westmaas jumps in in the middle without reading the whole convo | 20:34 | |
| vishy | ok still looking | 20:35 |
| * vishy goes back to sniffing | 20:35 | |
| jeblair | vishy: just an update - | 20:37 |
| jeblair | the migration is mostly done, but we're waiting on the python-git package to be ready for the nova jenkins slaves so that the test suite passes (since python-git is now a requirement for that) | 20:37 |
| jeblair | vishy: i'm using the change that adds rfc.sh to test that everything works, so i'd like to hold of on telling devs to clone github, etc, until that goes through. if we need to push that change through while we continue to work so that devs can start work, lemme know. | 20:39 |
| tr3buchet | so uh, like, trunk... | 20:39 |
| vishy | k | 20:39 |
| dolphm | vishy: that optional_str is just creating a unique string for testing... those are names | 20:41 |
| tr3buchet | yeah... | 20:41 |
| *** troytoman-away is now known as troytoman | 20:46 | |
| pvo | jeblair: what happens to code in midmerge? thinks that are already propped but waiting for approvals? | 20:46 |
| pvo | if it was sent to the ML, please excuse the question | 20:46 |
| *** mgius has quit IRC | 20:47 | |
| dysinger | seriously the latest drop of nova looks busted | 20:49 |
| dysinger | :( | 20:49 |
| dysinger | we are thinking maybe it's sqlalchemy bump or something | 20:49 |
| dysinger | queries that work directly on mysql aren't working through sqlalchemy | 20:50 |
| dysinger | investigating | 20:50 |
| vishy | dolphm: ok i think i found this | 20:51 |
| jeblair | pvo: i'm not sure if vishy addressed that -- i don't want to contradict him. but i'd recommend taking a diff from the change, applying it to the git repo and committing / reviewing (after vishy gives the go-ahead to start using the new repo). i'm not aware of any plans to move those over automatically. | 20:51 |
| vishy | dolphm: but i need you to tell me how it is supposed to work | 20:51 |
| kbringard | dysinger: are they timing out? | 20:52 |
| *** mgius has joined #openstack-dev | 20:52 | |
| dolphm | vishy: found what? | 20:52 |
| vishy | dolphm: get_by_tenant is only checking for members, not associated roles | 20:52 |
| vishy | dolphm: in the ldap driver | 20:52 |
| dolphm | and it's doing both for SQL? | 20:52 |
| vishy | dolphm: i assume based on the comment | 20:53 |
| vishy | if get_user_by_tenant returns none | 20:54 |
| vishy | than it returns unauthorized | 20:54 |
| dolphm | vishy: well, the comments don't make much sense to me, but it's obviously checking to see if the user has a default tenant id, or *any* role whatsoever on the tenant, which makes sense | 20:54 |
| vishy | yes that second check doesn't exist in the ldap code | 20:55 |
| dysinger | kbringard: yes they are through sqlalchemy | 20:55 |
| dysinger | not through mysql directly | 20:55 |
| vishy | it just finds users who are members of the tenant | 20:55 |
| dolphm | vishy: is it easy to implement in ldap? | 20:56 |
| vishy | dolphm: i think so | 20:56 |
| vishy | let me take a crack at it | 20:56 |
| dolphm | make it so! | 20:56 |
| jk0 | so like | 20:56 |
| jk0 | is nova moving to gerrit right now? | 20:56 |
| jeblair | jk0: yes, held up on the python-git package needed on the jenkins slaves. | 20:57 |
| jk0 | was there anything announced on the ML? | 20:57 |
| comstud | nothing that I saw | 20:57 |
| vishy | dolphm: it should return user if it as any roles on the tenant right? | 20:58 |
| comstud | we fail at communication | 20:58 |
| dolphm | vishy: yes | 20:58 |
| jk0 | seems like this is something we should have communicated | 20:58 |
| dolphm | vishy: that's called when users want a scoped token for a tenant, if they have *some* mapping to the tenant, they can be issued a scoped token | 20:58 |
| *** rbasak has quit IRC | 20:59 | |
| openstackgerrit | Ying Liu proposed a change to openstack/quantum: fix bug856564 https://review.openstack.org/543 | 21:04 |
| dysinger | https://skitch.com/dysinger/f58dw/2-tig-tig O_o | 21:09 |
| *** bcwaldon has quit IRC | 21:09 | |
| zykes- | dysinger: isn't that wrong ? | 21:10 |
| *** vladimir3p has quit IRC | 21:10 | |
| vishy | dolphm: first test fixed | 21:11 |
| *** zul has quit IRC | 21:11 | |
| vishy | dolphm: on two second | 21:11 |
| vishy | * too | 21:11 |
| vishy | * to | 21:11 |
| dolphm | awesome! | 21:13 |
| vishy | dolphm: same issue get_user_tenants is not returning linked roles | 21:15 |
| vishy | dolphm: who wrote the ldap code? I wonder if it is supposed to be adding the user as a member to the tenant when you add a role? | 21:16 |
| vishy | dolphm: I don't understand the purpose of add_user if you are adding roles? | 21:16 |
| *** joesavak has quit IRC | 21:18 | |
| dolphm | vishy: also, i think yuriy (sp?) wrote all the ldap code | 21:20 |
| dolphm | vishy: and it's my understanding that he's no longer contributing to openstack | 21:21 |
| *** harlowja has joined #openstack-dev | 21:23 | |
| dolphm | vishy: I started wondering why LDAP implements 'add_user' for roles and tenants, but 'add_user' is not defined the BaseAPI's for either? and then i discovered that the baseApi is so confusing that yuriy implemented User.user_role_add() by calling his own api.role.add_user() which makes way more sense lol | 21:27 |
| vishy | well i managed to create a new bug! | 21:28 |
| dolphm | ssh it's a feature! | 21:28 |
| *** bsza has quit IRC | 21:29 | |
| vishy | whoot passed | 21:31 |
| dolphm | :D | 21:31 |
| zykes- | oh, fixed ldap ? ^^ | 21:32 |
| dolphm | zykes-: sounds like it! | 21:33 |
| *** neogenix has joined #openstack-dev | 21:33 | |
| zykes- | isn't it mainted as actively as sql ? | 21:33 |
| *** mfer has quit IRC | 21:34 | |
| dolphm | zykes-: not at all :P | 21:34 |
| zykes- | :/, why not ? p | 21:35 |
| dolphm | i assume because no one uses it? | 21:35 |
| zykes- | :/ | 21:37 |
| dolphm | vishy: can you put your change up for review? | 21:38 |
| vishy | jeblair, mtaylor the rfc.sh seems to blow up badly if origin is not gerrit | 21:39 |
| vishy | btw | 21:39 |
| mtaylor | vishy: it _should_ work ok if it's either gerrit or github, and _should_ give an informative error message if not - I'm taking it you did not find any of that to be the case? | 21:40 |
| vishy | well my origin was cloudbuilders/keystone | 21:40 |
| vishy | so it set the gerrit alias to cloudbuilders/keystone instead of openstack/keystone | 21:40 |
| vishy | also: In commit b58a8b3b6195a2a09718b7e5a2e1836d3913c498 | 21:41 |
| vishy | remote: ERROR: author email address sleepsonthefloor@gmail.com | 21:41 |
| vishy | remote: ERROR: does not match your user account. | 21:41 |
| vishy | sorry for the three line paste | 21:41 |
| dolphm | add your email to gerrit? | 21:41 |
| jeblair | vishy: you should clone from the main repo on github. http://wiki.openstack.org/GerritWorkflow#Project_Setup | 21:41 |
| dolphm | and edit .git/config ? | 21:41 |
| *** mfer has joined #openstack-dev | 21:42 | |
| vishy | jeblair: yes it seems like it should be able to work if i cloned from a different repo though | 21:42 |
| vishy | sigh | 21:42 |
| jeblair | vishy: it's a script that helps automate the common workflow documented in the wiki. if you want to go "off-trail" as it were, you can push to gerrit directly | 21:42 |
| mtaylor | vishy: I hadn't considered the cloudbuilders/keystone case. | 21:42 |
| vishy | yes, well i fixed the remote by hand | 21:43 |
| mtaylor | vishy: an easier thing may be to just set your gerrit remote by hand - rather than recloning or whatnot | 21:43 |
| mtaylor | there it is. :) | 21:43 |
| vishy | its cool, but i don't know why it thinks i'm pushing a commit by anthony | 21:43 |
| jeblair | vishy: likely that is a commit that is not in the main repo | 21:43 |
| mtaylor | vishy: do you have any commits in your tree that were in cloudbuilders that weren't in trunk? | 21:43 |
| vishy | mtaylor: ah that must be it | 21:43 |
| *** zul has joined #openstack-dev | 21:44 | |
| mtaylor | jeblair: ok. FINALLY uploading python-git to ppa | 21:44 |
| mtaylor | jeblair: we need to add two people to the Authors file - you and dan@nicira.com | 21:45 |
| mtaylor | vishy: any idea who dan@nicira.com is? | 21:45 |
| troytoman | mtaylor: dan wendlandt | 21:46 |
| *** mfer has quit IRC | 21:46 | |
| mtaylor | troytoman: awesome. thanks | 21:46 |
| troytoman | mtaylor: danwent when he is here | 21:46 |
| mtaylor | jeblair: do you want to edit your change to include that? (otherwise the test is going to fail) | 21:47 |
| jaypipes | johan_-_: ping | 21:47 |
| jeblair | mtaylor: are the slaves updated? | 21:47 |
| *** lts has quit IRC | 21:47 | |
| mtaylor | jeblair: not yet. the package needs updating. oh, I need to put this as a nova build-dep - and I'm going to need to install by hand right now - chicken and egg migration issue | 21:48 |
| jeblair | mtaylor: you only updated rfc.sh in openstack-ci. I'm pretty sure the other 30 copies of rfc.sh still don't have the nice "you didn't clone from the right place" message. | 21:48 |
| mtaylor | jeblair: AH. duh | 21:49 |
| jeblair | mtaylor: do you want the new version in nova? and do you have a plan to update the rest of them? | 21:49 |
| mtaylor | jeblair: yeah. we should update them. yeah, if you could use the new version that would be great. and yeah, we should really sort out a new plan for that | 21:49 |
| openstackgerrit | Verification of a change to openstack/glance failed: Adding git-specific Authors file test https://review.openstack.org/451 | 21:50 |
| vishy | mtaylor: yeah ptl for quantum | 21:50 |
| jeblair | mtaylor: yes. just to be clear, i'm expecting you to update them. | 21:50 |
| mtaylor | jeblair: and if you wanted to get really fancy, http://paste.openstack.org/show/2534/ cleans up the pep8 warnings - but I could also do that as a follow up | 21:50 |
| mtaylor | jeblair: yes. totally. and fair. | 21:50 |
| jeblair | mtaylor: when will the slaves be updated? | 21:51 |
| mtaylor | jeblair: shortly after the package finishes building and is published | 21:51 |
| jeblair | i'll update rfc.sh in nova and the Authors file then. | 21:51 |
| jaypipes | mtaylor: hmm, Git Authors patch still failing on builder... do I need to wait a bit longer then? | 21:51 |
| openstackgerrit | Vish Ishaya proposed a change to openstack/keystone: Fixes issues with ldap tests https://review.openstack.org/544 | 21:51 |
| mtaylor | jaypipes: yes. it is still building | 21:51 |
| *** zns has joined #openstack-dev | 21:52 | |
| jaypipes | mtaylor: oh, ok. sorry about that | 21:52 |
| vishy | jeblair: it seems to require that origin is actually the right repo. I assume it is something to do with the rebasing that happens | 21:52 |
| vishy | mtaylor: ^^ | 21:52 |
| mtaylor | vishy: yes. now that you say it - it is expecting to be able to rebase on origin/master and have that be the right thing | 21:52 |
| dolphm | vishy: not sure how to get your change and mine in, and test them both | 21:53 |
| vishy | dolphm: https://review.openstack.org/#change,544 | 21:53 |
| jeblair | mtaylor: shall I add "Dan Wendlandt <dan@nicira.com>" to Authors ? | 21:53 |
| mtaylor | dolphm: are you talking about changes to nova and keystone at the same time? | 21:53 |
| mtaylor | jeblair: yes please | 21:53 |
| jeblair | er | 21:53 |
| jeblair | he's there. | 21:53 |
| dolphm | mtaylor: no, i think i got it worked out... | 21:53 |
| vishy | dolphm, can't you just merge mine first? | 21:53 |
| dolphm | vishy: yeah, for some reason i was thinking mine had to go first | 21:54 |
| vishy | or merge yours then mine, then re-enable the ldap tests | 21:54 |
| vishy | either way | 21:54 |
| vishy | anyway i'm going to bed | 21:54 |
| dolphm | /salute | 21:54 |
| dolphm | vishy: enjoy paris | 21:54 |
| mtaylor | jeblair: ah | 21:54 |
| mtaylor | jeblair: http://paste.openstack.org/show/2535/ | 21:54 |
| mtaylor | jeblair: at some point he has what appears to be a bogus email with extra < > | 21:55 |
| jeblair | i agree. | 21:55 |
| mtaylor | jeblair: might need to put an entry for that into the .mailmap ? | 21:55 |
| mtaylor | lemme check that that works | 21:56 |
| jeblair | that doesn't show up in 'git log'. | 21:56 |
| openstackgerrit | A change was merged to openstack/keystone: Fixes issues with ldap tests https://review.openstack.org/544 | 21:57 |
| dolphm | YAY! | 21:57 |
| dolphm | thanks again vishy | 21:57 |
| mtaylor | jeblair: so - adding it to .mailmap does no good - but adding an extra line in Authors for Dan Wendlandt <<dan@nicira.com>> does | 21:58 |
| openstackgerrit | Dolph Mathews proposed a change to openstack/keystone: POST /tokens: A chronicle of missing features https://review.openstack.org/516 | 21:58 |
| jeblair | mtaylor: can you identify which commit it is? | 21:58 |
| openstackgerrit | A change was merged to openstack/keystone: POST /tokens: A chronicle of missing features https://review.openstack.org/516 | 21:59 |
| mtaylor | jeblair: nope | 21:59 |
| mtaylor | jeblair: hrm. one sec... might be a bug in the test | 22:00 |
| jeblair | if email.split(' ')[-1] == '<>': | 22:00 |
| jeblair | email = email.split(' ')[-2] | 22:00 |
| openstackgerrit | A change was merged to openstack/keystone: Initial set of changes to move role operations to extensions. https://review.openstack.org/539 | 22:02 |
| dolphm | mtaylor: that's it, keystone is diablo-sufficient | 22:02 |
| *** hbetts has joined #openstack-dev | 22:02 | |
| mtaylor | dolphm: yay! | 22:03 |
| dolphm | mtaylor: i know you wanted a git tag -- is it time for that? who needs to do it? should it be signed? | 22:04 |
| jeblair | dolphm: i believe ttx is handling that | 22:04 |
| mtaylor | dolphm: for now if you could just email me the git revid | 22:04 |
| dolphm | mtaylor: will do | 22:05 |
| mtaylor | dolphm: and we'll get something cut with that rev | 22:05 |
| mtaylor | jaypipes: glance builder updated | 22:08 |
| dolphm | mtaylor: thanks, sent to to your @rs address | 22:08 |
| openstackgerrit | aaron-lee proposed a change to openstack/nova: bug lp845714 https://review.openstack.org/545 | 22:08 |
| jaypipes | mtaylor: cheerio | 22:08 |
| jeblair | awesome | 22:08 |
| dolphm | i'm running home for now, will check in later | 22:08 |
| jeblair | people are submitting changes before we're done with the move. | 22:09 |
| jeblair | mtaylor: you still working on the authors test? | 22:09 |
| mtaylor | jeblair: ok. nova slaves have python-git | 22:09 |
| jeblair | yes, but you say it won't pass, right? | 22:09 |
| mtaylor | jeblair: I have not been able to do anything appreciably useful with the authors test | 22:09 |
| mtaylor | jeblair: I kind of suggest putting in the bogus extra Dan entry into the Authors file and then file a bug to fix it right | 22:10 |
| jeblair | mtaylor: would you like us to fix it, or should i push through the change without code review and leave the test failing for the nova project to fix? | 22:10 |
| jeblair | (push the rfc.sh change, that is) | 22:10 |
| *** dolphm has quit IRC | 22:11 | |
| mtaylor | I don't think that we should leave the repo in a state where changes unrelated to the one being reviewed cause things to fail | 22:11 |
| *** vladimir3p has joined #openstack-dev | 22:11 | |
| jeblair | i'll add the bogus dan entry then. | 22:12 |
| jeblair | mtaylor: will you file a bug for it? | 22:12 |
| openstackgerrit | sleepsonthefloor proposed a change to openstack/keystone: Fix for bug 856846 - cast ints to string in users_get_by_tenant_get_page so that they can be joined https://review.openstack.org/546 | 22:13 |
| uvirtbot | Launchpad bug 856846 in keystone "TypeError: sequence item 0: expected string in users_get_by_tenant_get_page" [Undecided,New] https://launchpad.net/bugs/856846 | 22:13 |
| openstackgerrit | James E. Blair proposed a change to openstack/nova: Add rfc.sh to help with gerrit workflow. https://review.openstack.org/542 | 22:13 |
| mtaylor | jeblair: found it | 22:15 |
| mtaylor | jeblair: http://paste.openstack.org/show/2536/ | 22:16 |
| jeblair | ah | 22:17 |
| openstackgerrit | A change to openstack/nova has been rejected: Add rfc.sh to help with gerrit workflow. https://review.openstack.org/542 | 22:18 |
| *** troytoman is now known as troytoman-away | 22:18 | |
| jeblair | i've blocked my change. | 22:18 |
| openstackgerrit | Verification of a change to openstack/nova failed: Add rfc.sh to help with gerrit workflow. https://review.openstack.org/542 | 22:18 |
| *** reed has joined #openstack-dev | 22:18 | |
| jeblair | mtaylor: so i'll apply that diff and remove the extra <<>> | 22:19 |
| mtaylor | jeblair: excellent | 22:19 |
| openstackgerrit | James E. Blair proposed a change to openstack/nova: Add rfc.sh to help with gerrit workflow. https://review.openstack.org/542 | 22:21 |
| *** cp16net has quit IRC | 22:22 | |
| mtaylor | jeblair: you may also hit the pep8 warnings. I've got a branch which fixes them. | 22:24 |
| *** kbringard has quit IRC | 22:26 | |
| jeblair | mtaylor: there is a yellow ball on pep8. i think that's okay for the rfc.sh change. | 22:26 |
| mtaylor | ok | 22:26 |
| jeblair | hrm | 22:27 |
| jeblair | does unstable block? | 22:27 |
| mtaylor | think so | 22:28 |
| jeblair | why wasn't it blocking when in bzr? | 22:28 |
| mtaylor | we weren't using jenkins jobs as blockers with bzr | 22:28 |
| jeblair | it was running as part of tarmac... | 22:28 |
| mtaylor | yes. but unstable is a jenkins concept. | 22:28 |
| mtaylor | now, why pep8 wasn't causing the tarmac job to fail is a mystery | 22:28 |
| jeblair | what would have caused ... yes. exactly. :) | 22:29 |
| mtaylor | but I gave up on trying to solve it because we were moving off of tarmac | 22:29 |
| jeblair | so when this is done, assuming it fails, i'm going to bypass code review for rfc.sh, and let you submit a separate change to fix pep8. | 22:29 |
| openstackgerrit | A change was merged to openstack/nova: Add rfc.sh to help with gerrit workflow. https://review.openstack.org/542 | 22:31 |
| jeblair | mtaylor: done. the only thing that was wrong was pep8. why don't you submit your pep8 change now. | 22:31 |
| openstackgerrit | sleepsonthefloor proposed a change to openstack/keystone: Fix for bug 856857 - add user.name to User() constructor to re-align param https://review.openstack.org/547 | 22:35 |
| uvirtbot | Launchpad bug 856857 in keystone "Incorrect invocation of types.user.User constructor" [Undecided,New] https://launchpad.net/bugs/856857 | 22:35 |
| openstackgerrit | Monty Taylor proposed a change to openstack/nova: Fix outstanding pep8 errors for a clean trunk. https://review.openstack.org/548 | 22:35 |
| mtaylor | jeblair: ^^ | 22:35 |
| mtaylor | jeblair: I also added a little bit to run_tests.sh so that we can use it in jenkins without having it ALSO run pep8 | 22:35 |
| *** wwkeyboard has quit IRC | 22:48 | |
| *** aa___ has joined #openstack-dev | 22:50 | |
| *** harlowja has left #openstack-dev | 22:51 | |
| openstackgerrit | Monty Taylor proposed a change to openstack/nova: Fix outstanding pep8 errors for a clean trunk. https://review.openstack.org/548 | 22:56 |
| openstackgerrit | Monty Taylor proposed a change to openstack/nova: Fix outstanding pep8 errors for a clean trunk. https://review.openstack.org/548 | 22:58 |
| openstackgerrit | Monty Taylor proposed a change to openstack/nova: Fix outstanding pep8 errors for a clean trunk. https://review.openstack.org/548 | 22:58 |
| mtaylor | gah. sorry for the spam - I suck | 22:58 |
| jeblair | mtaylor: i can't review them fast enough! | 22:58 |
| mtaylor | jeblair: sorry - I noticed a couple of typos | 22:59 |
| *** nati_ has joined #openstack-dev | 23:00 | |
| *** gigi has joined #openstack-dev | 23:02 | |
| *** gigi is now known as Guest23596 | 23:02 | |
| openstackgerrit | sleepsonthefloor proposed a change to openstack/keystone: Fix for bug 856857 - add user.name to User() constructor to re-align param https://review.openstack.org/547 | 23:02 |
| uvirtbot | Launchpad bug 856857 in keystone "Incorrect invocation of types.user.User constructor" [Undecided,New] https://launchpad.net/bugs/856857 | 23:02 |
| *** Guest23596 has quit IRC | 23:03 | |
| *** nati has quit IRC | 23:03 | |
| *** dwalleck has joined #openstack-dev | 23:07 | |
| *** mgius has quit IRC | 23:08 | |
| *** dragondm has quit IRC | 23:13 | |
| openstackgerrit | Monty Taylor proposed a change to openstack/nova: Fix outstanding pep8 errors for a clean trunk. https://review.openstack.org/548 | 23:13 |
| *** rnirmal has quit IRC | 23:15 | |
| *** aa___ has quit IRC | 23:24 | |
| *** dwalleck has quit IRC | 23:26 | |
| *** dwalleck has joined #openstack-dev | 23:27 | |
| *** dwalleck has quit IRC | 23:30 | |
| *** vladimir3p has quit IRC | 23:37 | |
| *** jeffjapan has joined #openstack-dev | 23:41 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!