Tuesday, 2011-07-26

*** jhtran has quit IRC00:06
*** donald650 has joined #openstack-dev00:18
*** mnour has joined #openstack-dev00:41
*** mnour has quit IRC00:57
*** jmckenty has joined #openstack-dev00:59
*** jdurgin has quit IRC01:01
*** vladimir3p has quit IRC01:03
*** jmckenty has quit IRC01:09
*** dragondm has quit IRC01:15
*** dantoni has quit IRC01:16
*** Tushar has joined #openstack-dev01:29
*** nmistry has joined #openstack-dev01:33
*** deshantm_away is now known as deshantm01:39
*** nmistry has quit IRC01:54
*** nmistry_ has joined #openstack-dev01:54
*** Tushar has quit IRC02:03
*** medberry is now known as med_out02:13
*** nmistry_ has quit IRC02:29
*** LoriC has quit IRC02:33
jaypipess1rp: w00t. cache is now in trunk.02:37
jaypipesttx: glance D3 looking pretty good to go. Still want to get the S3 bug fix in, though. Shouldn't be more than an hour of work in the morning...02:41
s1rpjaypipes: nice!02:43
*** Tushar has joined #openstack-dev03:42
*** Tushar has quit IRC03:45
*** martine has quit IRC04:23
*** chomping has joined #openstack-dev04:30
*** openpercept_ has joined #openstack-dev04:37
*** donald650 has quit IRC05:55
*** donald650 has joined #openstack-dev05:55
HugoKuo__Well , compute-node seems lost mysql connection while the controller host restarted06:17
HugoKuo__I'm not sure if this is a normal condition06:17
*** rupakg has quit IRC06:21
*** reidrac has joined #openstack-dev06:58
*** AhmedSoliman has joined #openstack-dev07:11
*** jaypipes has quit IRC07:11
*** jaypipes has joined #openstack-dev07:24
*** chomping has quit IRC07:31
openstackjenkinsProject nova build #1,148: SUCCESS in 4 min 4 sec: http://jenkins.openstack.org/job/nova/1148/07:56
openstackjenkinsTarmac: Moved the VIF network connectivity logic('ensure_bridge' and 'ensure_vlan_bridge') from the network managers to the virt layer.  In addition, VIF driver class is added to allow customized VIF configurations for various types of VIFs and underlying network technologies.07:56
*** ahmed_ has joined #openstack-dev08:18
*** AhmedSoliman has quit IRC08:20
vishyhttps://bugs.launchpad.net/nova/+bug/81623608:24
uvirtbot`Launchpad bug 816236 in nova "Initial 'nova db sync' migration failure on mysql due to foreign key reference" [Critical,Triaged]08:24
vishythat bug i've marked critical, if someone wants to investigate it08:24
vishyalso, if someone wants to try and fix the issue with boto 2.0 that is causing the oneiric builds to fail, that would rock08:26
openstackjenkinsProject nova build #1,149: SUCCESS in 4 min 32 sec: http://jenkins.openstack.org/job/nova/1149/08:36
openstackjenkinsTarmac: I'm sorry, for my fail with rebasing. Any way previous branch grew to many other futures, so I supersede it.08:36
openstackjenkins1. Used optparse for parsing arg string08:36
openstackjenkins2. Added decorator for describe method params08:36
openstackjenkins3. Added option for assigning network to certain project.08:36
openstackjenkins4. Added field to "network list" for showing which project owns network08:36
*** shehjart has joined #openstack-dev08:37
*** mnour has joined #openstack-dev08:40
*** rods has joined #openstack-dev08:54
*** mnour has quit IRC08:57
*** mnour has joined #openstack-dev08:57
*** darraghb has joined #openstack-dev08:58
*** mnour has quit IRC09:20
*** mnour has joined #openstack-dev09:20
*** ahmed_ has quit IRC09:29
HugoKuo__is there any approach to avoid ARP table lost while the nova-network host  restart09:34
*** darraghb has quit IRC09:46
*** darraghb has joined #openstack-dev09:47
ttxDevelopment switched to Swift 1.4.3, Nova diablo-4 and Glance diablo-410:22
ttxjaypipes: I cut the milestone-proposed branches -- The S3 fix (if any) will be backported there10:23
chemikadzettx: you mean diablo-3 freezed?10:24
ttxchemikadze: yes, the limit was yesterday night.10:24
chemikadzettx: you mean 00:00 UTC?10:25
ttxI mean, as long as it's yesterday somewhere. Like Hawaii time10:26
BK_manttx: do you have a tarballs ready to download?10:26
ttxBK_man: that's only proposed ones -- they will bake in testing for a few days, release on Thursday10:27
ttxwe need a few bugfixes in there10:27
ttxemail coming up10:27
ttxas soon as I unblock the toolchain10:27
*** AhmedSoliman has joined #openstack-dev10:27
BK_manttx: ok. thanks, we'll starting prepare our RPMs10:27
ttxBK_man: final will be on Thursday, but you should be able to test that the proposed one is ok for you10:28
ttxjaypipes: should keystone integration in Glance be considered completed by rev158 ?10:30
openstackjenkinsProject nova-milestone build #5: SUCCESS in 41 sec: http://jenkins.openstack.org/job/nova-milestone/5/10:37
*** mnour1 has joined #openstack-dev10:43
*** mnour has quit IRC10:44
jaypipesttx: morning11:02
jaypipesttx: no, keystone not done. Vek is still working on things...11:03
jaypipesttx: good progress, but will have to be D4.11:03
jaypipesttx: it is about 75% done11:03
jaypipesttx: I will move that11:03
jaypipesttx: moved.11:05
BK_manttx: which bzr rev your tarballs based on? I see 1134 for nova - it's very old11:10
*** markvoelker has joined #openstack-dev11:17
*** lorin1 has joined #openstack-dev11:31
*** gaitan has joined #openstack-dev11:33
*** timr has joined #openstack-dev11:45
*** mfer has joined #openstack-dev11:49
ttxbeh -- glance broken in ppa. Wil investigate11:52
ttxBK_man: r1134 is a revision from milestone-proposed branch, not from trunk11:52
*** duker has joined #openstack-dev11:53
ttxactually equivalent to "1322" at this point11:53
ttx(using the "r" you can tell which branch the tarball was built from)11:53
ttxBK_man: See bottom of http://wiki.openstack.org/BranchModel for explanation11:54
ttxjaypipes: some tests failing on the glance PPA: https://launchpadlibrarian.net/75994217/buildlog_ubuntu-natty-i386.glance_2011.3~d3~20110726.r143-0ubuntu0ppa1~natty1_FAILEDTOBUILD.txt.gz11:57
ttxsame in trunk. For some reasons the tests don't pass on the PPA builders11:58
jaypipesttx: yup, noticed. I will email jason koelker about it.11:59
ttxThe tests pass on Jenkins, so it might be some sensitivity to the env11:59
ttxlike a missing build-dep, but the error doesn't really hint12:00
jaypipesttx: https://bugs.launchpad.net/glance/+bug/81638612:08
uvirtbot`Launchpad bug 816386 in glance "test_scrubber functional tests fail on package build" [High,Confirmed]12:08
jaypipesttx: I've asked Jason to take a look at it12:08
ttxok12:08
*** soren has joined #openstack-dev12:16
*** ChanServ sets mode: +v soren12:16
*** martine has joined #openstack-dev12:29
*** nmistry has joined #openstack-dev12:32
*** lts has joined #openstack-dev12:32
*** duker has quit IRC12:35
zulttx: im trying to track down the copyrights for the openwrt images in the smoketests do you know where they came from?12:38
ttxzul: no. Maybe vishy knows. Maybe they should just be out of the codebase12:39
zulttx: ack im asuming that they are from the openwrt project though12:40
*** ameade has joined #openstack-dev12:42
*** nmistry has quit IRC12:45
BK_manis that a regression or I missed something? http://paste.openstack.org/show/1946/12:49
BK_manthis is on bzr130912:49
*** LoriC has joined #openstack-dev12:50
*** bsza has joined #openstack-dev12:50
*** bsza has quit IRC12:52
*** duker has joined #openstack-dev12:57
*** bsza has joined #openstack-dev13:04
jaypipeszul: ask devcamcar.13:17
zuljaypipes: k13:18
ttxjaypipes: I'm trying to reproduce the Glance test error locally... naively, I run ./run_tests.sh -N ... but I only get failures -- any hint before I debug ?13:18
jaypipesBK_man: looks like it might be due to new auth middleware?13:18
jaypipesttx: paste the error for me?13:18
BK_manjaypipes: yep. now trying to revert it. thanks13:19
*** dolphm has joined #openstack-dev13:19
ttxjaypipes: still in progress13:19
jaypipesdolphm: welcome Dolphin.13:19
ttxjaypipes: was missing xattr. solved.13:21
jaypipesttx: gotcha13:22
*** bcwaldon has joined #openstack-dev13:25
dolphmthank you, but only girls call me dolphin13:27
jaypipesdolphm: OK, sorry. welcome, Lundgren.13:28
jaypipes:P13:28
ttxjaypipes: for the test failure, I suspect the 15sec wait is optimistic on the PPA builders13:32
ttxtrying to see if something would ensure that delayed_delete is necessarily under 15 seconds13:32
jaypipesttx: hmm, yes, that could definitely be it...13:34
ttxjaypipes: the second failure is due to the leftover from previous test13:34
jaypipesttx: ooh, ok, that makes sense too.  side effects--13:35
ttxbasically the thing is still pending_delete when the next test ensures nothing is13:35
jaypipesright13:35
jaypipesbcwaldon: ok, don't hate me, but I need one more round of testing with the S3 branch... :(13:37
jaypipesbcwaldon: I added a functional test case for the delete non-existing scenario13:37
bcwaldonjaypipes: already on it13:39
jaypipesbcwaldon: u rock.13:39
ttxjaypipes: but 15sec should be more than enough, wakeup_time in test is 2 seconds.13:40
ttx(and scrub_time is 5)13:41
*** troytoman-away is now known as troytoman13:44
bcwaldonjaypipes: you can't have a plus sign in your bucket name13:48
bcwaldonjaypipes: seeing some weirdness, got a sec?13:54
*** vladimir3p has joined #openstack-dev13:55
bcwaldonjaypipes: comments on the MP13:59
ttxsoren: could you have a look at bug 816386 -- does time go that slowly on buildds ?14:03
uvirtbot`Launchpad bug 816386 in glance "test_scrubber functional tests fail on package build" [High,Confirmed] https://launchpad.net/bugs/81638614:03
*** mattray has joined #openstack-dev14:05
jaypipesbcwaldon: k, looking now..14:06
jaypipesbcwaldon: the zero size bug is known: https://bugs.launchpad.net/bugs/794582. I'll fix up the + thing and the typo in error.14:09
uvirtbot`Launchpad bug 794582 in glance "glance sets image size to 0 for http images -> yields connection reset error on GET" [Medium,Confirmed]14:09
bcwaldonjaypipes: awesome14:09
jaypipesbcwaldon: you have no idea how happy I am going to be to get bug713154 in trunk...14:13
bcwaldonjaypipes: I've got an idea :)14:13
jaypipes:)14:13
bcwaldonI'm tired of testing the darn thing, too!14:13
jaypipesbcwaldon: I bet.14:14
bcwaldonjaypipes: anything else you need me to look at?14:16
jaypipesbcwaldon: another round of s3 testing in a few minutes ;)14:16
bcwaldonjaypipes: I'll be ready14:16
jaypipesbcwaldon: :) thx man14:17
jaypipesbcwaldon: you can also have a look at johan_-_ 's notifier merge proposal14:17
bcwaldonjaypipes: Sounds good. Today is my Nova review day, but glance needs some love, too14:17
jaypipesgotcha.14:17
*** jkoelker has joined #openstack-dev14:19
vladimir3pbcwaldon: good luck. Btw there is our VSA merge proposal waiting for more reivews ;-)14:25
bcwaldonvladimir3p: I saw. It's a big one! I'll try to get some feedback for you today.14:25
vladimir3pbcwaldon: big portion of it is tests14:26
*** cp16net has joined #openstack-dev14:26
bcwaldonjaypipes: good to go!14:36
jaypipesbcwaldon: w00t.14:37
jaypipesbcwaldon: feel free to approve the MP.14:37
bcwaldonjaypipes: Let's not get too hasty, here. I'm going to get one more set of eyes on it first :)14:38
bcwaldonjaypipes: blamar is going to take a look momentarily14:38
ttxjaypipes, bcwaldon: if it makes it into Glance trunk, please propose it to milestone-proposed as well14:39
jaypipesbcwaldon: ok, thx14:39
bcwaldonjaypipes: can you set to NR14:40
*** tomeff has joined #openstack-dev14:45
*** dragondm has joined #openstack-dev14:45
jaypipesbcwaldon: yep, one sec14:45
jaypipesbcwaldon: done14:45
blamarugh, too many comments in that bug bcwaldon...can I get a TLDR? :P14:50
ttxany database migration expert that would like the glory of solving bug 816236 ?14:52
uvirtbot`Launchpad bug 816236 in nova "Initial 'nova db sync' migration failure on mysql due to foreign key reference" [Critical,Triaged] https://launchpad.net/bugs/81623614:52
ttx Fixing release-critical bugs gives you good karma.14:52
bcwaldonblamar: TL;DR: shut your mouth14:55
* blamar smacks bcwaldon with a trout14:55
*** dolphm has quit IRC14:55
*** dolphm has joined #openstack-dev14:56
*** rnirmal has joined #openstack-dev15:00
jaypipesttx: notice my lack of eagerness to approach that migrate bug.15:07
ttxjaypipes: it's probably not that difficult -- but it will take me ages to bridge the gaps in my SQL / sqlalchemy-migrate knowledge15:08
ttxI'm sure it's quite easy for people who already did it :)15:09
dolphmon gerrit, how do you push a new commit (a revision) to an existing review?15:09
* ttx tries to get to the bottom of bug 81436515:09
uvirtbot`Launchpad bug 814365 in nova "Should support boto 2.0 server-side (was: EC2 API fails with >=boto2.0)" [Wishlist,Confirmed] https://launchpad.net/bugs/81436515:09
*** rnirmal has quit IRC15:09
jaypipesdolphm: git review should push the new commit, no?15:10
*** reidrac has quit IRC15:10
dolphmwon't that just open a new review?15:10
jaypipesdolphm: hmm, I wouldn't think so, as the history should know it's related to the prior commit...15:11
jaypipesdolphm: one sec, asking mtaylor and jeblair15:11
jaypipesdolphm: I assume you're in the same branch in the repo as the original review push?15:11
dolphmwell, it's not me, but yes15:12
mtaylordolphm: so - if you grabbed the commit hook and installed it, it will attach it to the same review15:12
mtaylordolphm: http://wiki.openstack.org/GerritWorkflow#Change-Id_Hook15:12
dolphmso it just happens automagically? e.g. I've opened 3 reviews for distinct changes, but if one of those changes was dependent on a prior, it would have revised an existing review instead?15:13
mtayloryes15:13
dolphmfreaking awesome15:13
*** cp16net has quit IRC15:14
mtaylorI'm _sure_ there are some semantics in there somewhere, but yes. additionally - if you rebase and squash commits, as long as you keep the ChangeId comment from the original commit, it will still be able to keep track of it all15:15
*** med_out is now known as medberry15:22
jeblairdolph: it looks like #50 didn't have a change-id header15:22
jeblairdolphm: so it made a new change instead of attaching to the previous one (but it did note the previous one as a dependency)15:23
jeblairif you don't have the change-id hook installed, you can manually add the header to a commit message after looking up the value in gerrit (eg, Change-Id: I9af68c5c12da28b39972d653dbb288baf5b9a181)15:24
openstackjenkinsProject nova build #1,150: SUCCESS in 4 min 13 sec: http://jenkins.openstack.org/job/nova/1150/15:26
openstackjenkinsTarmac: Updates /servers requests to follow the v1.1 spec. Except for implementation of uuids replacing ids and access ips both of which are not yet implemented. Also, does not include serialized xml responses.15:26
jeblairdolphm: here's an example: https://review.openstack.org/#patch,sidebyside,42,1,/COMMIT_MSG15:28
bcwaldonjaypipes: we need to update glance package builder15:28
*** jhtran has joined #openstack-dev15:29
dolphmdoes the change-id need to be last like that?15:31
dolphmi.e. why didn't this work: https://review.openstack.org/#patch,sidebyside,53,1,/COMMIT_MSG15:31
bcwaldonjaypipes: missing python-xattr15:31
dolphmjeblair: ^^15:31
*** openpercept_ has quit IRC15:31
jeblairdolphm: i believe it does need to be at the end15:32
dolphmjeblair: k, i'll try that15:32
jeblairdolphm: "To be picked up by Gerrit, a Change-Id line must be in the bottom portion (last paragraph) of a commit message, and may be mixed together with the Signed-off-by, Acked-by, or other such footers. For example:..."15:32
jeblairdolphm: https://review.openstack.org/Documentation/user-changeid.html15:32
dolphmjeblair: ah, so it's a Footer lol15:33
blamarjaypipes: bug713154 approved, just have a note or two for you to read before flipping the switch15:33
jeblairdolphm: yes, i suppose i should not call it a header. :)15:33
dolphmdamn, now it wants me to squash commits locally... even though I have unrelated commits in between?15:36
jaypipesbcwaldon: yup, noted.15:40
jaypipesblamar: k, reading.15:40
mtaylordolphm: I've run in to this before myself - and I think it's one of those things that will incite us to use branches for different unrelated topics15:41
sorenttx: /me looks15:41
*** chomping has joined #openstack-dev15:41
*** lorin1 has quit IRC15:51
jaypipesmtaylor: we need to update the version of boto on the jenkins build box... to latest from 1.915:57
mtaylorjamshid: for which project?15:57
mtaylorjamshid: sorry - bad tab complete15:58
jaypipesmtaylor: glance15:58
mtaylorjaypipes: for which project is it a depend?15:58
*** mnour1 has quit IRC15:59
*** mnour has joined #openstack-dev15:59
mtaylorjaypipes: do we have more recent packages?15:59
mtaylorjaypipes: I think we may have to make some15:59
jaypipesmtaylor: for boto?15:59
mtayloryeah. 1.9 seems to be the latest in repo15:59
jaypipesmtaylor: we need to add dependency for boto>=2 and xattr to the glance packaging15:59
jaypipessoren: ^^16:00
jaypipesI was going to create a bug for that...16:00
jaypipesmtaylor: the loveliness of maintaining our own versions of packages..16:00
mtaylorjaypipes: python-xattr yeah?16:02
jaypipesmtaylor: ya16:02
mtaylork. that one's already there16:02
mtaylorwe'll need to wait on a new package version of boto to hit our repos for the other16:03
*** johnpur has quit IRC16:12
dolphmjeblair: mtaylor: i completely gave up on revising that review, abandoning a review and opening a new one seems easier, but of course, you lose context in the new review :/16:12
mtaylordolphm: I think it gets much easier when you have the change-id hook installed for the first one as well, so in _this_ case it was wonky, but I think once we're in a rythym it will actually work more naturally16:14
jeblairdolphm: and if topic branches are used, we probably won't end up with unrelated changes in the middle of a series16:15
dolphmmtaylor: hmm, I thought I installed it, but I don't see the footer on any of my commits16:15
mtaylordefinitely collecting tips/practices will be key as we go16:16
jeblairdolphm: it's per-repository, so it should be under your keystone repo in ".git/hooks/commit-msg"16:17
dolphmjeblair: oooh, i put it in ~16:17
jeblairdolphm: i'll try to clarify the wiki16:18
mtaylorannegentle: ping?16:20
dolphmjeblair: probably my bad, I'm used to mercurial paying attention to ~/.hgrc16:21
dolphmmtaylor: what's the scoop on enforcing pylint and coverage with gerrit?16:23
mtaylordolphm: pylint is easy to enforce - although you probably want to get clean first :)16:24
dolphmcan you do a (new => old) for pylint? i've been working on improving our score16:24
mtaylordolphm: coverage is on the todo list - need to make a change to the jenkins coverage plugin to understand "be unhappy if the number falls" (right now it only has a static number)16:24
dolphmmtaylor: or (new >= old - .05) or something16:24
mtaylordolphm: I don't think so - it's a similar problem to coverage16:24
mtaylordolphm: we could go in from time to time and set the lower limit to the current number while you're working on it16:25
dolphmmtaylor: that'd be a good start, if it's not a pain (weekly updates?)... also, what are other projects scoring in pylint, and is there a standard pylint config?16:27
dolphmmtaylor: openstack standard*16:27
mtaylordolphm: so currently you have 1415 violations ... I can set that to be the pain threshold for now - and yeah, let's do weekly updates16:28
jaypipesmtaylor: only 1415?16:28
mtaylordolphm: there is a pylintrc in lp:nova ... and the goal is 0 violations, but I don't think anybody is on that at the moment (since we haven't been enforcing it)16:29
mtaylorI believe projects _are_ pretty clean with pep8 though16:29
mtayloryeah - nova was pep8 clean, and is now up to 10 violations (which will be easier to deal with when we move them to gerrit and can enforce pylint counts)16:30
dolphmmtaylor: when you say violations, are you including notices?16:31
dolphmmtaylor: err, non-errors/warnings16:31
mtaylorI don't think so16:32
mtaylorhere's how we're running it: pylint --rcfile=.pylintrc -f parseable keystone > pylint.txt16:32
mtaylordolphm: ok. you are now set that if you have more than 1415 pylint responses that your merges will fail16:34
mtaylorhappy hunting!16:34
mtaylordolphm: and you seem to have the same pylintrc as nova, so that's good16:36
*** jdurgin has joined #openstack-dev16:41
dolphmmtaylor: I hadn't noticed our pylintrc file... after playing with it, it seems pretty sensible16:45
dolphmmtaylor: although I'm getting 939 'messages' total... where are you getting 1415 'violations'? (i know we're looking at slightly different code bases, but that's a big difference, unless we're looking at different things)16:47
*** mnour has quit IRC16:54
*** mnour has joined #openstack-dev16:54
sorenglance needs boto>=2?17:05
sorenjaypipes: ^17:05
jaypipessoren: yes, with the introduction of the S3 fully functional backend it does.17:05
sorennova doesn't work with boto 2, apparently. https://launchpad.net/~nova-core/+archive/trunk/+build/2646612/+files/buildlog_ubuntu-oneiric-i386.nova_2011.3~d4~20110726.1323-0ubuntu0ppa1~oneiric1_FAILEDTOBUILD.txt.gz17:05
jaypipessoren: gah.17:06
sorenI'm fairly sure that's why it fails on Oneiric.17:06
sorenjaypipes: ...so it's something we need to fix for sure.17:06
jaypipessoren: well, the folks who wrote boto changed the most basic method signature for the S3Connection.__init__() method from 1.9 to 2.0...17:06
jaypipessoren: adding in a kwarg headers that doesn't exist in 1.9...17:07
jaypipessoren: so they are incompatible...17:07
creihtboto changed quite a bit from 1.9 to 2.017:07
sorenjaypipes: Sounds like fun.17:07
jaypipescreiht: any suggestion on this? My inclination is to standardize on 2.0...17:07
* soren concurs17:07
* soren is all about the new hawtness17:08
jaypipesand fix Nova's issue with it...17:08
creihtwhen writing the s3 middleware, I used 1.9 since that was what nova used... :)17:08
jaypipespoopy pants.17:08
sorenWe'll have to switch sooner or later. Might as well do it sooner.17:08
creihtI really don't know what the community in general is using17:08
sorenOr support both, of course.17:08
sorenI don't know how much work that'll entail.17:09
creihtI know, there should be a best-boto lib, that translates the calls to boto ;)17:09
soren...but it seems like a reasonable assumption that the percentage that uses 1.9 vs. 2.0 will eventually tip in 2.0's favour.17:09
creihta boto abstraction layer :)17:09
jaypipeshehe17:09
jaypipessoren: yes, that is a reasonable assumption I would assume. Though looks like 1.9 is standard for Natty...17:10
creihtboto usage in swift is very small with the s3 compat stuff, so it shouldn't be a big deal either way17:10
jaypipessorry guys, off to dentist. back in 1.5 hours... :(17:10
creihtmight be worth asking jessie, vish, since it seemed like a concious decision to use 1.9, since 2.0 has been around for a while17:11
creihtlunchtime here17:11
*** kbringard has joined #openstack-dev17:11
openstackjenkinsProject nova build #1,151: SUCCESS in 4 min 5 sec: http://jenkins.openstack.org/job/nova/1151/17:11
openstackjenkinsTarmac: queries in the models.Instance context need to reference the table by name (fixed_ips) however queries in the models.FloatingIp context alias the tables out properly and return the data as fixed_ip (which is why you need to reference it by fixed_ip in that context).17:11
openstackjenkinsIdeally we should probably consider adding a fixed_ip alias to the models.Instance query context so that the api code is consistent, but for now this should resolve the outstanding issues.17:11
kbringardhey, just in time to see my merge17:11
*** darraghb has quit IRC17:14
openstackjenkinsProject keystone build #2: STILL FAILING in 29 sec: http://jenkins.openstack.org/job/keystone/2/17:21
openstackjenkinsdolph.mathews: Simple change to test gerrit17:21
openstackjenkinsProject nova build #1,152: SUCCESS in 4 min 12 sec: http://jenkins.openstack.org/job/nova/1152/17:22
openstackjenkinsTarmac: Add OpenStack API support for block_device_mapping.17:22
openstackjenkinsThis patch enables boot from volume feature already implemented in EC217:22
openstackjenkinsAPI, as an OpenStack API extension.17:22
dolphmmtaylor: is keystone on the PYTHONPATH for gerrit?17:23
mtaylordolphm: not sure what you mean - there should be no installed version of keystone17:24
mtaylordolphm: this is what's being run: python keystone/test/run_tests.py17:25
dolphmmtaylor: http://docs.python.org/tutorial/modules.html#the-module-search-path17:26
dolphmmtaylor: I'm guessing keystone.logic.types is the first time the unit tests import from the keystone package17:27
*** duker has quit IRC17:28
mtaylordolphm: yes... sorry, I phrased my response poorly - currently jenkins is pulling a fresh copy of the repo and doing nothing other than running python keystone/test/run_tests.py without doing any additional setup.17:28
mtaylordolphm: if we need to do something for right now (since we're still getting you added in to the system), we can do that - but in general, the expectation is that there is a test script in the tree which can be run without needing to set up any env vars17:29
mtaylordolphm: is there a different way you normally run that test script?17:29
dolphmin dev, i would expect any projects i'm working with to be on the PYTHONPATH17:30
*** mgius has joined #openstack-dev17:30
dolphmmtaylor: i'm wondering if there's a way to do it on the command line, like `python --append-path=. keystone/test/run_tests.py17:31
mtaylordolphm: really? fascinating. I expect the exact opposite17:31
mtaylordolphm: there is - I'm just saying, there should be no need to do that ... lemme take a more specific look real quick17:32
*** donald650 has quit IRC17:33
dolphmmtaylor: i guess the workaround is `possible_topdir = os.path.normpath(os.path.join(os.path.abspath(sys.argv[0])` ... but I think that's really poor python, as the environment should be configured by the sysadmin, not by the app17:34
openstackjenkinsProject nova build #1,153: SUCCESS in 4 min 19 sec: http://jenkins.openstack.org/job/nova/1153/17:35
openstackjenkinsTarmac: Updates to the compute API and manager so that rebuild, reboot, snapshots, and password resets work with the most recent versions of novaclient.17:35
mtaylordolphm: I agree - I do not think we need to munge the path like that17:36
dolphmmtaylor: http://stackoverflow.com/questions/4580101/python-add-pythonpath-during-command-line-module-run17:37
mtaylordolphm: what I'm saying is that the need to munge the path at all indicates a more fundamental problem ... and as we do not set pythonpath for any of the other projects, I want to figure out why it is that the test runner here is wanting us to17:38
dolphmmtaylor: other projects munge the path: https://github.com/openstack/nova/blob/master/bin/nova-api17:39
vishysoren! wb17:39
dolphmmtaylor: the file i believe is failing does not: https://github.com/openstack/keystone/blob/master/keystone/test/unit/test_keystone.py17:39
mtaylordolphm: fair enough - so, I'm going to have to punt this one up the line to jaypipes and/or vishy ... project standards for how to deal with this are out of my scope for deciding17:40
dolphmmtaylor: alrighty17:40
mtaylordolphm: I just don't want to add pythonpath munging to your jenkins jobs and nobody elses if that's going to cause unintended consequences later17:41
dolphmmtaylor: just for sanity, can you try running a build with the same command you tried before, prefixed by 'PYTHONPATH=. ' (or whatever path is appropriate)17:41
mtaylordolphm: sure. and I'm happy to add that to the job for the moment ... I just want to make sure we know what we're doing long-term here17:41
dolphmmtaylor: agree17:42
openstackjenkinsProject keystone build #3: STILL FAILING in 3.5 sec: http://jenkins.openstack.org/job/keystone/3/17:42
openstackjenkinsdolph.mathews: Simple change to test gerrit17:42
mtaylordolphm: ok, well, at least that's a DIFFERENT error :)17:43
dolphmmtaylor: previous *.db file was not deleted17:43
mtaylordolphm: something seems to be trying and failing to remove it ... rm: cannot remove `keystone/test/keystone.db': No such file or directory17:43
dolphmmtaylor: we have an open issue to run tests using an in-memory database, but we can't do that as long as we're using sampledata.sh as a crutch for automated tests17:44
mtaylordolphm: so what needs to be deleted?17:44
dolphmkeystone/keystone.db and keystone/keystone.token.db17:44
mtaylordolphm: fwiw, those wind up in the top src dir in my branch17:45
dolphmand run_tests.py needs to be fixed, to delete from the cwd, not from keystone/test17:45
mtaylorgotcha.17:45
dolphmmtaylor: or to tell keystone to create db's in the keystone/test directory17:45
*** donald650 has joined #openstack-dev17:49
dolphmmtaylor: as a workaround, can you cd into keystone/test before running run_tests.py? yogi is working on a new run_tests.py approach that will test with multiple configurations, and hopefully address this issue17:51
sorenvishy: ahoy17:54
mtaylordolphm: https://review.openstack.org/8917:55
mtaylordolphm: oh, well, sure17:55
dolphmmtaylor: the only problem with your change is that i think it's risky for a production system with a default config... if someone executes run_tests.py, they delete their data17:57
openstackjenkinsYippie, build fixed!17:57
openstackjenkinsProject keystone build #4: FIXED in 1 min 4 sec: http://jenkins.openstack.org/job/keystone/4/17:57
openstackjenkinsdolph.mathews: Simple change to test gerrit17:57
mtaylordolphm: fair enough17:57
dolphmmtaylor: all good for now?17:58
mtaylordolphm: seems like it!17:58
mtaylordolphm: and we can figure out long-term solutions for these things moving forward17:58
dolphmmtaylor: A) determine future of PYTHONPATH / possible_topdir stuff, B) rework configuration management for automated tests17:59
mtaylordolphm: yes18:00
dolphmmtaylor: deal18:00
dolphmmtaylor: https://github.com/rackspace/keystone/issues/12218:00
mtaylordolphm: I'd add a tentative c) align run_tests with nova18:00
mtaylordolphm: sweet18:00
mtaylordolphm: but I don't need to die on C18:00
dolphmmtaylor: lol fair enough18:01
*** jamshid has quit IRC18:02
annegentlemtaylor: pong18:09
mtaylorannegentle: just wanted to give you a heads up that I'm going to move the docs stuff on to a new server some time soon (I know you shell in some times and do things)18:09
dolphmmtaylor: the parens make that a little creepy18:10
annegentleok, thanks for the heads up - I was in there Friday in fact :) battling wiki spam, which has gone to zero but nobody else can create accounts either. Sigh.18:10
*** mattray has quit IRC18:11
*** AhmedSoliman has quit IRC18:11
mtaylorannegentle: and when I'm done - wiki will be on its own box, and $project.openstack.org and docs.openstack.org will be in the same place but on a different machine18:11
annegentlemtaylor: ok, nice.18:12
*** cp16net has joined #openstack-dev18:15
*** mfer has quit IRC18:16
*** mfer has joined #openstack-dev18:16
jhtranwhat's the best library for using OS api?18:19
bcwaldonjhtran: novaclient18:19
jhtranok i'll check that out.  is there a url?18:19
jhtranhttps://launchpad.net/python-novaclient ?18:20
bcwaldonhttp://pypi.python.org/pypi/python-novaclient/2.5.818:20
jhtrancool thanks!18:20
bcwaldonit's developed on github, btw18:20
*** mattray has joined #openstack-dev18:20
tr3buchetvishy: add_fixed_ip_to_instance() is broken unless you have a new way for it to work, any ideas?18:27
*** mnour has quit IRC18:28
vishydid it break with the l2 networking branch?18:28
vishytr3buchet: ^18:28
tr3buchetvishy: no it's passing host to add_fixed_ip_to_instance(), where does this host come from?18:31
vishyhmm18:32
vishylooking18:32
bcwaldonjaypipes: Can you propose the S3 bugfix for merging into milestone-proposed?18:33
vishyoh weird18:34
vishythe host got removed18:34
ttxjkoelker, jaypipes: test still failing on buildds.18:36
ttx(for bug 816386)18:37
uvirtbot`Launchpad bug 816386 in glance "test_scrubber functional tests fail on package build" [High,In progress] https://launchpad.net/bugs/81638618:37
vishytr3buchet: there are a couple easy ways to fix this18:40
tr3buchetvishy: i don't know why adding an ip to an instance requires knowing the host18:42
tr3buchethost should just be removed from that function18:42
vishytr3buchet: network needs to know where to send the message for multi_host networks18:43
vishyhost is passed so network doesn't have to look up the host from the instance_ref in the db18:44
vishytr3buchet: http://pastie.org/227527418:46
vishya little bit inefficient in multi_host mode but that should work18:47
glencjaypipes - FYI, https://blueprints.launchpad.net/glance/+spec/pluggable-auth18:52
ttxvishy: I had a try at bug 814365 -- will have to try out more tomorrow if noone beats me to it18:57
uvirtbot`Launchpad bug 814365 in nova "Should support boto 2.0 server-side (was: EC2 API fails with >=boto2.0)" [Wishlist,Confirmed] https://launchpad.net/bugs/81436518:57
ttxvishy: you might need to address bug 816236 -- I haven't found any candidate to fix it so far18:57
uvirtbot`Launchpad bug 816236 in nova "Initial 'nova db sync' migration failure on mysql due to foreign key reference" [Critical,Triaged] https://launchpad.net/bugs/81623618:57
ttxtr3buchet: any chance you can propose a fix for bug 810563 before D3 release ?18:58
uvirtbot`Launchpad bug 810563 in nova "nova-manage lets you create broken networks (was: trying to add VLAN #100 to IF -:None:- error: No such device)" [Medium,Confirmed] https://launchpad.net/bugs/81056318:58
*** cp16net has quit IRC19:03
*** cp16net has joined #openstack-dev19:05
*** cp16net_ has joined #openstack-dev19:06
jkoelkerttx: just saw that19:06
*** cp16net_ has quit IRC19:07
*** cp16net__ has joined #openstack-dev19:07
*** cp16net has quit IRC19:07
*** cp16net__ is now known as cp16net19:07
jkoelkerttx: can I get a tarball of the attempted build? And or a way to trigger builds?19:07
*** cp16net has quit IRC19:08
*** cp16net has joined #openstack-dev19:08
ttxhmm, it's not very easy, I think19:08
ttxsoren: any idea how to do that ?19:09
tr3buchetttx: yes. had some laptop issues earlier, i'll probably have a merge proposed today19:09
*** lorin1 has joined #openstack-dev19:10
tr3buchetvishy: why does compute need to know this if in multi-host the network is on the same host?19:11
*** nati has joined #openstack-dev19:11
sorenjkoelker: What would a "tarball of an attempted build" contain?19:12
kbringardI just came into the conversation, but from the question my guess is because of live-migration19:12
tr3buchetvishy: more like this: http://pastie.org/227541619:12
jkoelkerthe test.sqlite db19:12
vishytr3buchet: the request goes to network19:12
vishynetwork needs to know where to send it19:12
tr3buchetvishy: so your requests still go to the generic network topic?19:13
jkoelkersoren: its the only way i can think of how to track down why its failing on the build system but no where else19:13
vishyyes it has to19:13
tr3buchetvishy: in that case yeah i think you're right19:13
vishybecause the vm is attached to multiple networks19:13
vishysome might be multihost and some may not19:13
vishywe could shortcut a little in the add_fixed_ip19:14
vishybecause we know we are just allocating from just one network19:14
vishyso we could load the network in network.api and send the message to the right host there19:14
vishybut i think that is a little harder to follow19:15
tr3buchetah i see, and flat just ignores it19:15
vishybecause we're checking for 'multi_host' in another place19:15
tr3buchetyeah you're right. it's easy for compute to just append what it knows19:16
jaypipesbcwaldon: only once the packaging issues are fixed...19:17
jaypipesttx: looking..19:17
tr3buchetvishy: i see your logic there in the RPCAllocateFixedIP mixin.19:17
vishytr3buchet: you want to file a quick bug and merge proposal off of the milestone-proposed?19:17
tr3buchetsure yeah19:17
tr3buchetyou want to do the merge prop or shall i?19:18
vishyyou can do it19:18
tr3buchetk19:18
kbringardjaypipes: have you seen this one? http://paste.openstack.org/show/1954/19:18
vishyi'm going to try and sort out this boto bug19:18
tr3bucheti'll knock that out right quick19:18
kbringardjaypipes: or is that related to the packaging stuff19:18
jaypipeskbringard: yep19:19
kbringardcool, I figured19:19
kbringardthen I'll just leave it be for now19:19
kbringardthanks :-D19:19
jaypipeskbringard: https://bugs.launchpad.net/bugs/81498119:19
uvirtbot`Launchpad bug 814981 in glance "glance-api fails on image delivery: AttributeError: context" [Medium,Confirmed]19:19
jaypipeskbringard: there is a known workaround/fix... see bug.19:20
kbringardah, rad, thanks19:20
kbringardkick ass, works great19:21
kbringardyoda man19:21
*** mgius has quit IRC19:27
claygjaypipes: I ran into that error as well, work around was of course adding the new context middleware19:30
*** dolphm has quit IRC19:31
jaypipesclayg: yup... actually, dprince had suggested separating config files from paste.deploy to make things like this easier to upgrade properly... not sure I completely agree with that, though.19:31
claygbut I wonder why not just if not hasattr(req, 'context'): req.context = DefaultAdminContext19:31
jaypipesclayg: I added a blueprint to the openstack-ci project for this general problem... https://blueprints.launchpad.net/openstack-ci/+spec/glance-upgrade19:31
claygi don't know splitting your paste config from the app config, paste deploy .ini configs are pretty standard these days19:31
claygi don't really see it as an upgrade problem as much as a "required middleware" problem19:32
claygwhy should the app assume that something upstream added a non-standard attribute to the request?19:32
jaypipesclayg: I gather you are suggesting the best way would to have not created required middleware :)19:32
claygI just mean the app could make a default decision if context attribute isn't there19:33
jaypipesclayg: ya, I hear you.19:33
claygok19:33
jaypipesclayg: or even add it to wsgi.environ['context'] instead, making it environ.get('context')'able...19:34
jaypipesclayg: which I think may have been how nova implemented it... anyways, point taken :)19:34
claygidk, namespacing problems apply either way, when you add an attribute to a request webob's just putting it in environ['addhoc-attrs']['attr_name']19:35
creihtwsgi.environ['glance.context']19:35
creiht:)19:35
creihtis typical for that type of stuff19:35
jaypipescreiht: ah, yes, thx19:35
claygI presonally always liked adding a attribute with the name of the app - which itself is a dict i.e. req.glance.get('context', default_context)19:35
jaypipesclayg, creiht: cool, thx for input. I'm going to paste the above into the bug...19:36
*** troytoman is now known as troytoman-away19:37
tr3buchetvishy: just to double check, remove fixed ip shouldn't have any need for host info19:37
vishygood question19:40
*** dolphm has joined #openstack-dev19:41
ttxmtaylor: any idea why buildds choke on bug 816386 ?19:48
uvirtbot`Launchpad bug 816386 in glance "test_scrubber functional tests fail on package build" [High,In progress] https://launchpad.net/bugs/81638619:48
tr3buchetvishy: i remember going through it the first time, deciding it didn't, but you may have changed something19:48
claygjaypipes: I'm still having problems with swift+http backend (lp bug # 771849)19:48
openstackjenkinsProject keystone build #5: FAILURE in 2.9 sec: http://jenkins.openstack.org/job/keystone/5/19:48
openstackjenkinskevin.mitchell: Add Admin API tests for v2 authentication19:48
claygas before the adds/puts are ok, but gets blow up cause the auth_url isn't right: http://paste.openstack.org/show/1955/19:49
*** Tushar has joined #openstack-dev19:49
claygi got a patch, http://paste.openstack.org/show/1956/19:49
claygjaypipes: do you want a new bug?  or should I use 77184919:50
jaypipesclayg: hmm. the second URI isn't a valid URI, though...19:50
claygyeah, that's the current code :)19:51
jaypipesclayg: but it may be in the registry database already in the location field, eh?19:51
claygwith my patch: http://paste.openstack.org/show/1957/19:51
claygjaypipes: that's a good point, I'm not sure how my code would parse an already screwed up uri - please hold ;)19:52
*** dolphm has quit IRC19:52
jaypipesclayg: sure, I'm not disagreeing with you. just saying that "swift+http://user:pass@http://myswift.com/auth/v1.0/glance/img" is not a valid store URI. I think the source is the old registry location entries..19:52
claygoddly, it still works19:53
claygso... the "correct" uri would be: swift+http://user:pass@myswift.com/auth/v1.0/glance/img19:53
clayg??19:53
jaypipesclayg: the thing is, the store URI is not meant to be seen outside the API/registry server communication. It's just an oddity that it is today (useful in testing only..)19:53
jaypipesclayg: yes, that's the correct store URI.19:53
jaypipesclayg: regardless, looks like a few more tests need to be added...19:54
*** dolphm has joined #openstack-dev19:54
jaypipesclayg: and definitely a SQL migrate thing to adjust existing URIs.19:54
claygok, yeah but that's not what glance does today if your config specifies the swift_store_auth_address as http://auth.com:port/auth/v1.019:54
claygwhich to my knowledge is the only place you can specify your store is swift+http instead of just "swift"19:55
vishyi don't think i changed that19:56
jaypipesclayg: you can actually register an image in Glance using the X-Image-Meta-Location: swift+http(s)://blah.com/container/obj19:56
jaypipesclayg: but I don't think that's the issue here. just needs some more tests and a migration of existing entries I believe.19:57
claygjaypipes: existing entries my need more tests - but I'm talking about a fresh upload into swift+http - if you don't speicify the Location trunk glace will construct it for you as "swift+http://user:pass@http://myswift.com/auth/v1.0/glance/img"19:58
jeblairdolphm: have you had a chance to look at http://jenkins.openstack.org/job/keystone/5/console19:59
claygthen when you go to get that image it will pass the swift connection "http:/myswift.com/auth/v1.0" as the auth url, and the "http:/" instead of "http://" blows up19:59
jaypipesclayg: ok. I'll add that as a new bug.19:59
claygI haven't tried overriding the Location with meta?19:59
jaypipesclayg: sorry about this.. I thought I'd added a crapton of tests for that... :(19:59
jaypipesclayg: what version of Python you running?20:01
clayg2.620:03
jaypipesclayg: minor version?20:03
jaypipesclayg: please don't say 2.6.1...20:04
jaypipesclayg: MacOSX?20:04
*** dolphm has quit IRC20:05
claygubuntu 2.6.5 and 2.6.620:06
bcwaldonjaypipes: whew!20:06
jaypipesclayg: phew indeed :)20:06
jaypipesclayg, bcwaldon: though, more than likely fixing osx screwed other versions of urlparse :)20:07
jaypipesclayg: OK, duly noted. I will look into it shortly.20:07
jaypipesclayg: thx for your patience!20:07
claygnp20:07
openstackjenkinsProject keystone build #6: STILL FAILING in 4 sec: http://jenkins.openstack.org/job/keystone/6/20:07
openstackjenkinskevin.mitchell: Add Admin API tests for v2 authentication20:07
jaypipeshmmm... ^^ is my code, not kevin's...20:08
kbringardcode theif!@20:08
jaypipesVek: did you propose the above?20:08
jaypipeshehe, no, I'm just trying to figure out gerritbot...20:09
jaypipesor rather gerritbot + hudsonbot20:09
vishywhoot i think i got boto 2.0 workin20:09
*** nmistry has joined #openstack-dev20:10
ttxvishy: rha, I tried http://paste.ubuntu.com/652509/ but it still fails20:10
ttxvishy: would be interested in your solution, for my education20:11
vishyttx: there were quite a few bugs in the code20:11
vishyfor processing the rules20:11
ttx(I'm stuck at AssertionError: '' != '0.0.0.0/0')20:11
ttxvishy: in the test code ?20:11
vishyno in cloud.py20:12
ttxoh20:12
ttxso that last one was actually covering a real bug :)20:12
vishyyeah20:12
vishylooks like i have one test still failing20:12
vishyalmost there :)20:12
*** nati has quit IRC20:14
*** Daviey has quit IRC20:14
*** dolphm has joined #openstack-dev20:16
*** Tushar has quit IRC20:17
jeblairdolphm: ping20:18
*** Daviey has joined #openstack-dev20:18
dolphmjeblair: reading it now20:19
jeblairit looks like that's a change that predates the keystone/test directory20:19
jeblairseems to me like we should just approve those and get back to the current state20:20
claygjaypipes: so xattr is cool for image meta data, swift is all about the xattrs, but now now that it20:21
dolphmjeblair: where is the output of pwd?20:21
*** User784 has joined #openstack-dev20:21
claygs a top level import in glance.utils - the client tool requires it as well?20:21
clayg#justsayin20:21
*** Tushar has joined #openstack-dev20:21
*** Daviey has quit IRC20:22
dolphmjeblair: it looks like it's starting from the wrong directory? it's trying to cd into keystone/test so it can use run_tests.py from the cwd, to get around a flaw in run_tests.py (see https://github.com/rackspace/keystone/issues/122 )20:23
jeblairpwd is used here: PYTHONPATH=/var/lib/jenkins/jobs/keystone/workspace20:23
jeblairexport PYTHONPATH=`pwd`20:23
jeblairdolphm: i think it's the right directory20:24
jeblairthere's just no test directory because it didn't exist a month ago20:24
jeblairhttp://jenkins.openstack.org/job/keystone/ws/keystone/20:24
dolphmjeblair: then how can the file not exist?20:24
dolphmis this committing to an old branch or something?20:25
jeblairyes20:25
ttxReminder: Meeting in 35 minutes in #openstack-meeting20:25
jeblairziad pushed a bunch of old changes20:25
*** cp16net has quit IRC20:25
dolphmjeblair: why? lol20:25
jeblairi'm not sure?  i think he was trying to push the changes he merged to rackspace/keystone after the cutover?20:26
dolphmwell, the change commits to test_authn_v2.py, which the current test script doesn't even use20:27
dolphmjeblair: i'd like to remove that file entirely, but i want to rewrite the tests first20:27
*** cp16net has joined #openstack-dev20:29
dolphmjeblair: can we just let the change be rejected?20:29
jeblairdolphm: sure.  i thought ziad wanted all of the changes he pushed this morning to be merged.20:31
dolphmjeblair: i don't know anything about what he pushed :/20:31
dolphmjeblair: seems very weird that it's from june 5th though20:32
jeblairdolphm: i'll wait till i hear from you or ziad if you want me to do anything (like bypass jenkins)20:32
*** Daviey has joined #openstack-dev20:32
dolphmjeblair: are there a bunch more changes pending too?20:33
jeblairhttps://review.openstack.org/#q,status:open+project:openstack/keystone,n,z20:34
dolphmjeblair: holy crap20:34
*** nmistry has quit IRC20:39
jaypipesclayg: no, shouldn't be. but we'll deal with that in a later patch...20:47
jaypipesclayg: I'd like to do totally separate packages for client and server(s) in Glance.20:48
jaypipesclayg: we've got a bug logged for that, just havent' gotten around to it :(20:48
*** lorin1 has quit IRC20:51
*** Tushar has quit IRC20:52
openstackjenkinsProject nova build #1,154: SUCCESS in 4 min 16 sec: http://jenkins.openstack.org/job/nova/1154/20:56
openstackjenkinsTarmac: compute now appends self.host to the call to add an additional fixed ip to an instance20:56
*** asomya has joined #openstack-dev20:59
ttxMeeting starts, join #openstack-meeting now !21:01
Davieyyes sir!21:02
*** troytoman-away is now known as troytoman21:02
ttxDaviey: i'm not your boss anymore, yanow21:02
DavieyThank goodness :)21:05
*** dolphm has quit IRC21:09
*** cp16net has quit IRC21:12
*** martine has quit IRC21:15
*** ameade has quit IRC21:17
*** kbringard has quit IRC21:36
*** kbringard has joined #openstack-dev21:37
*** mwhooker has joined #openstack-dev21:43
mwhookerHello. I got a 404 for http://nova.openstack.org/Twisted-10.0.0Nova.tar.gz when running python ./tools/install_venv.py in Nova21:43
mwhookeranyone know where that file might have gone?21:43
vishymwhooker: are you installing a very old version?21:46
mwhookerfresh checkout21:46
mwhookerhttp://bazaar.launchpad.net/~hudson-openstack/nova/trunk/view/head:/tools/install_venv.py#L3321:46
*** lts has quit IRC21:51
openstackjenkinsYippie, build fixed!22:00
openstackjenkinsProject keystone build #7: FIXED in 1 min 8 sec: http://jenkins.openstack.org/job/keystone/7/22:00
openstackjenkinskevin.mitchell: Document how to allow anonymous access22:00
* Vek raises an eyebrow22:01
openstackjenkinsProject keystone build #8: SUCCESS in 1 min 6 sec: http://jenkins.openstack.org/job/keystone/8/22:02
openstackjenkinsdolph.mathews: Issue #13: Added support for Accept-appropriate 404 responses w/ tests for json & xml.22:02
openstackjenkinsProject keystone build #9: FAILURE in 2.9 sec: http://jenkins.openstack.org/job/keystone/9/22:02
openstackjenkinskevin.mitchell: Update the sample to reflect some minor enhancements to the base framework.22:02
*** bcwaldon has quit IRC22:02
openstackjenkinsYippie, build fixed!22:03
openstackjenkinsProject keystone build #10: FIXED in 1 min 3 sec: http://jenkins.openstack.org/job/keystone/10/22:03
openstackjenkinsdolph.mathews: PEP8 fixes for system tests22:03
openstackjenkinsProject keystone build #11: SUCCESS in 1 min 3 sec: http://jenkins.openstack.org/job/keystone/11/22:04
openstackjenkinsdolph.mathews: Making the API version configurable per API request22:04
openstackjenkinsProject keystone build #12: FAILURE in 2.7 sec: http://jenkins.openstack.org/job/keystone/12/22:04
openstackjenkins* jaypipes: Add test verifying a missing tenantId key in the password creds works properly in JSON22:04
openstackjenkins* jaypipes: Add Admin API tests for v2 authentication22:04
openstackjenkins* jaypipes: Add success test for GET /v2.0/tokens/<TOKEN_ID> in json and xml22:04
openstackjenkins* anotherjesse: keystone repo is now at github.com/rackspace/keystone22:04
openstackjenkinsProject keystone build #13: STILL FAILING in 2.7 sec: http://jenkins.openstack.org/job/keystone/13/22:04
openstackjenkinskevin.mitchell: Add test verifying a missing tenantId key in the password creds works properly in JSON22:04
openstackjenkinsYippie, build fixed!22:05
openstackjenkinsProject keystone build #14: FIXED in 1 min 6 sec: http://jenkins.openstack.org/job/keystone/14/22:05
openstackjenkins* robin.norwood: Update README with instructions to fix segfault.22:05
openstackjenkins* robin.norwood: Sigh. Proofreading...22:05
openstackjenkinsProject keystone build #15: SUCCESS in 1 min 4 sec: http://jenkins.openstack.org/job/keystone/15/22:06
openstackjenkinskevin.mitchell: Document how to allow anonymous access22:06
sorenErr..22:06
openstackjenkinsProject keystone build #16: SUCCESS in 1 min 4 sec: http://jenkins.openstack.org/job/keystone/16/22:07
openstackjenkins* robin.norwood: Update README with instructions to fix segfault.22:07
openstackjenkins* robin.norwood: Sigh. Proofreading...22:07
openstackjenkinsProject keystone build #17: SUCCESS in 1 min 4 sec: http://jenkins.openstack.org/job/keystone/17/22:08
openstackjenkinsrobin.norwood: Sigh. Proofreading...22:08
openstackjenkinsProject keystone build #18: SUCCESS in 1 min 3 sec: http://jenkins.openstack.org/job/keystone/18/22:09
openstackjenkinsyoga80: #66 Change in variable cases.22:09
openstackjenkinsProject keystone build #19: SUCCESS in 1 min 7 sec: http://jenkins.openstack.org/job/keystone/19/22:10
openstackjenkinsyoga80: #66 Change in variable cases.22:10
*** mfer has quit IRC22:10
openstackjenkinsProject keystone build #20: FAILURE in 2.7 sec: http://jenkins.openstack.org/job/keystone/20/22:10
openstackjenkinskevin.mitchell: The API is a moving target; update the test22:10
openstackjenkinsProject keystone build #21: STILL FAILING in 3.6 sec: http://jenkins.openstack.org/job/keystone/21/22:11
openstackjenkinskevin.mitchell: Split the Keystone service from the Admin service so we can test both22:11
openstackjenkinsYippie, build fixed!22:12
openstackjenkinsProject keystone build #22: FIXED in 1 min 6 sec: http://jenkins.openstack.org/job/keystone/22/22:12
openstackjenkins* kevin.mitchell: The API is a moving target; update the test22:12
openstackjenkins* kevin.mitchell: Split the Keystone service from the Admin service so we can test both22:12
openstackjenkins* kevin.mitchell: Try to use an admin credential to revoke the token.22:12
openstackjenkins* kevin.mitchell: Use un-spaced exception names...22:12
openstackjenkins* kevin.mitchell: Bah, somehow my sample data failed to include Admin as admin's role22:12
openstackjenkins* kevin.mitchell: Get and revoke both admin and user tokens...22:12
openstackjenkins* kevin.mitchell: Don't need to fiddle around with user tokens here, just admin tokens22:12
openstackjenkins* kevin.mitchell: Save expiration data for later comparison22:12
openstackjenkins* kevin.mitchell: Add test for validate_token22:12
openstackjenkins* kevin.mitchell: Update the sample to reflect some minor enhancements to the base framework.22:12
openstackjenkins* kevin.mitchell: It's possible to authenticate through the Admin API.22:12
openstackjenkins* kevin.mitchell: Add test verifying a missing tenantId key in the password creds works properly in JSON22:12
openstackjenkins* kevin.mitchell: Add Admin API tests for v2 authentication22:12
openstackjenkins* kevin.mitchell: Add success test for GET /v2.0/tokens/<TOKEN_ID> in json and xml22:12
openstackjenkins* kevin.mitchell: keystone repo is now at github.com/rackspace/keystone22:12
openstackjenkins* kevin.mitchell: Add middleware for glance integration22:12
openstackjenkinsProject keystone build #23: SUCCESS in 1 min 4 sec: http://jenkins.openstack.org/job/keystone/23/22:13
openstackjenkinsrobin.norwood: Update README with instructions to fix segfault.22:13
openstackjenkinsProject keystone build #24: SUCCESS in 1 min 3 sec: http://jenkins.openstack.org/job/keystone/24/22:14
openstackjenkinsyoga80: Changes to store tokens using memcache #66.22:14
openstackjenkinsProject keystone build #25: SUCCESS in 1 min 6 sec: http://jenkins.openstack.org/job/keystone/25/22:15
openstackjenkinskevin.mitchell: These changes make no sense--I didn't do them, and I'm in sync!22:15
*** gaitan has quit IRC22:16
openstackjenkinsProject keystone build #26: SUCCESS in 1 min 3 sec: http://jenkins.openstack.org/job/keystone/26/22:16
openstackjenkinskevin.mitchell: Add middleware for glance integration22:16
openstackjenkinsProject keystone build #27: FAILURE in 3.1 sec: http://jenkins.openstack.org/job/keystone/27/22:16
openstackjenkins* kevin.mitchell: Add test verifying a missing tenantId key in the password creds works properly in JSON22:16
openstackjenkins* kevin.mitchell: Add Admin API tests for v2 authentication22:16
openstackjenkins* kevin.mitchell: Add success test for GET /v2.0/tokens/<TOKEN_ID> in json and xml22:16
openstackjenkins* kevin.mitchell: keystone repo is now at github.com/rackspace/keystone22:16
openstackjenkinsProject keystone build #28: STILL FAILING in 2.8 sec: http://jenkins.openstack.org/job/keystone/28/22:16
openstackjenkins* yoga80: Support transforming service catalog22:16
openstackjenkins* yoga80: Changes to support service catalog.22:16
openstackjenkins* yoga80: Updating tests and sample data.22:16
openstackjenkins* yoga80: Changes to return service urls for Auth1.0 style calls.22:16
openstackjenkins* yoga80: Changes to return service urls for Auth1.0 style calls.22:16
openstackjenkins* yoga80: Removing debug print22:16
openstackjenkins* yoga80: Introducing new frontend component to handle rackspace legacy calls.22:16
openstackjenkins* yoga80: Introducing new frontend component to handle rackspace legacy calls.22:16
openstackjenkins* yoga80: Changing to legacy auth to standard wsgi middleware.22:16
openstackjenkins* yoga80: Changing to legacy auth to standard wsgi middleware.Name change of some of the files.22:16
openstackjenkins* yoga80: Removing redundant files.22:16
openstackjenkins* yoga80: PEP8 changes.22:16
openstackjenkins* yoga80: Code cleanup.22:16
openstackjenkins* yoga80: Minor fixes.22:16
openstackjenkinsYippie, build fixed!22:17
openstackjenkinsProject keystone build #29: FIXED in 59 sec: http://jenkins.openstack.org/job/keystone/29/22:17
*** bsza has quit IRC22:17
openstackjenkins* dolph.mathews: Correcting typo22:17
openstackjenkins* dolph.mathews: Added Routes and httplib2 to production dependencies22:17
openstackjenkins* dolph.mathews: Updated docs: sampledata.sh can't be executed outside of bin/22:17
openstackjenkins* dolph.mathews: Updated paths to unit/function tests in README22:17
openstackjenkins* dolph.mathews: Improved README formatting/consistency22:17
openstackjenkins* yoga80: Seperating user calls from tenants.22:17
openstackjenkins* github: Test fixes22:17
openstackjenkins* github: Changed BaseURLs to OpenStack names22:17
openstackjenkins* yoga80: Introduced a method to get all users @Users resource.Also moved the method to get user groups out of tenant scope.22:17
openstackjenkins* yoga80: Removing unused references to UserTenantAssociation.22:17
openstackjenkins* jaypipes: Adds Sphinx build ability and RST documentation.22:17
openstackjenkins* dolph.mathews: Grammar corrections22:17
openstackjenkins* dolph.mathews: Added pip requirements file for testing environments.22:17
openstackjenkins* yoga80: Adding call to modify tenant.Adding more tests and fixing minor issue.22:17
openstackjenkins* xtoddx: Fix symlinks after docs -> doc rename.22:17
openstackjenkins* xtoddx: Update the baseURL data pushed into glance.22:17
openstackjenkins* anotherjesse: nova needs 1.0 api currently22:17
openstackjenkins* sleepsonthefloor: dash needs both 1.0 and 1.1 compatability - need to fix that!22:17
openstackjenkins* sleepsonthefloor: for got to change a 1.1 to 1.022:17
openstackjenkins* yoga80: Changes to allow user creation without a tenant.22:17
openstackjenkins* yoga80: Pep8 changes.22:17
openstackjenkins* dolph.mathews: Refactored DB API into modules by model22:17
openstackjenkins* yoga80: Changes to wadl to support user operations.22:17
openstackjenkins* dolph.mathews: Refactored api function names to avoid redundancy with new module names22:17
openstackjenkins* dolph.mathews: Fixed broken unit test code22:17
openstackjenkins* yoga80: Fixing existing methods on wadl.22:17
openstackjenkins* yoga80: Changes to include support for paginations.22:18
openstackjenkins* dolph.mathews: Fixed spelling error22:18
openstackjenkins* dolph.mathews: Fix for keystone issue 41: https://github.com/rackspace/keystone/issues/4122:18
openstackjenkins* dolph.mathews: Removed unused imports in bin/22:18
openstackjenkins* dolph.mathews: Removed redundant sentence in dev guide22:18
openstackjenkins* dolph.mathews: Removed unused import22:18
openstackjenkins* yoga80: Merging changes22:18
openstackjenkins* dolph.mathews: Restored tools.tracer to bin/ scripts; included fix for empty frames22:18
openstackjenkins* anotherjesse: Edited keystone/auth_protocols/nova_auth_token.py via GitHub22:18
openstackjenkins* github: Readme fix22:18
openstackjenkins* yoga80: Bringing back the changes to support endpointTemplates and endpoints.22:18
openstackjenkins* github: Fixed requires for development and in readme22:18
openstackjenkins* github: Fixed readme instructions for Nova - Issue #5522:18
openstackjenkins* dolph.mathews: Issue 31: Switching default ports to 5000/5001 (public/admin)22:18
openstackjenkins* xtoddx: Swift-specific middleware.22:18
openstackjenkins* yoga80: Code changes to support global endpointTemplates.22:18
openstackjenkins* yoga80: Changes on Sample data.22:18
openstackjenkins* xtoddx: Remove swift-y bits from generic token auth.22:18
openstackjenkins* xtoddx: Make swift middleware live where it should.22:18
openstackjenkins* yoga80: Changes to support endpoints and endpointemplates (renaming BaseUrls and BaseURLRefs).22:18
openstackjenkins* github: Removed unnecessary symlink22:18
openstackjenkins* github: Recompiled devguide with endpoints and templates22:18
openstackjenkins* github: Fixed identity.wadl response - issue #71#22:18
openstackjenkins* yoga80: Initial changes to support multiple backends.22:18
openstackjenkins* yoga80: Adding list of todos.22:18
openstackjenkins* yoga80: Changes to support dynamic loading of models.22:18
openstackjenkins* dolph.mathews: Refactored URL extensions handling (for .json/.xml)22:18
openstackjenkins* dolph.mathews: Minor comment change22:18
openstackjenkins* dolph.mathews: Renamed exthandler to urlrewritefilter to better illustrate it's purpose22:18
openstackjenkins* dolph.mathews: Removed redundant action mappings (for version controller)22:18
openstackjenkins* dolph.mathews: Fixed default content type behavior (was defaulting to XML)22:18
openstackjenkins* github: Fixing bug reported using with swift22:18
openstackjenkins* github: Pylint an pep8 fixes22:18
openstackjenkins* github: pep822:18
openstackjenkins* dolph.mathews: Improved readme consistency22:18
openstackjenkins* dolph.mathews: Organized imports22:18
openstackjenkins* dolph.mathews: Added note about run_tests.py to readme22:18
openstackjenkins* dolph.mathews: Slightly simplified base wsgi router22:18
openstackjenkins* dolph.mathews: Removed unused import22:18
openstackjenkins* dolph.mathews: Fixed spelling22:18
openstackjenkins* dolph.mathews: Improved PEP8 compliance22:18
openstackjenkins* dolph.mathews: Fixed doc string22:18
openstackjenkins* dolph.mathews: Refactored routers and controllers into their own modules (issue #44)22:18
openstackjenkins* dolph.mathews: Removed unused logger22:18
openstackjenkins* dolph.mathews: Added bootstrap configuration script (with admin user assigned an Admin role)22:18
openstackjenkins* dolph.mathews: Added run_tests.py to keystone.test.system, which uses bootstrap db script22:18
openstackjenkins* dolph.mathews: Abstracted underlying HTTP behavior away from RestfulTestCase22:18
openstackjenkins* dolph.mathews: Improved readability a bit22:18
openstackjenkins* dolph.mathews: Improved dict formatting22:18
openstackjenkins* dolph.mathews: Added missing imports and fixed a few pylint issues22:18
openstackjenkins* dolph.mathews: Was this a typo or an incredibly lame joke?22:18
openstackjenkins* dolph.mathews: Formatting change22:18
openstackjenkins* dolph.mathews: Wrote test case for github issue #8522:18
openstackjenkins* dolph.mathews: Commented out failing request, until it's review22:18
openstackjenkins* github: Move dev guide to OpenStack22:19
openstackjenkins* dolph.mathews: Removed tenant id from admin user22:19
openstackjenkins* dolph.mathews: Fixed minor pylint issues22:19
openstackjenkins* dolph.mathews: Clarifying test case22:19
openstackjenkins* dolph.mathews: - Added 'automatic' admin authentication to KeystoneTestCase using bootstrapped user22:19
openstackjenkins* dolph.mathews: Merging keystone.auth_protocols package into keystone.middleware22:19
openstackjenkinsProject keystone build #30: SUCCESS in 1 min 3 sec: http://jenkins.openstack.org/job/keystone/30/22:19
openstackjenkins* yorik.sar: Add exception throwing and logging to keystone-manage.22:19
antonymheh...22:19
openstackjenkinsProject keystone build #31: SUCCESS in 1 min 3 sec: http://jenkins.openstack.org/job/keystone/31/22:19
openstackjenkins* dolph.mathews: Added missing import22:19
openstackjenkins* dolph.mathews: Specified python-ldap version, which appears to avoid the packaging issues we've experienced22:19
openstackjenkins* dolph.mathews: Fixing indentation22:19
openstackjenkins* dolph.mathews: Avoiding overloading of built-in: type()22:19
openstackjenkins* dolph.mathews: Improving variable naming consistency22:19
openstackjenkins* dolph.mathews: Fixing module-level variable naming issues22:19
openstackjenkins* mgius7096: Removes disabled checks from get_user and update_user22:19
openstackjenkins* dolph.mathews: Cleaning up unit tests22:19
openstackjenkins* dolph.mathews: Adding missing class variable declaration22:19
openstackjenkins* dolph.mathews: Assigned Base API classes so downstream code knows what to expect22:19
openstackjenkins* dolph.mathews: Noted potential issue, but I'm not sure if this is dead code or not anyway?22:19
openstackjenkins* dolph.mathews: Updating a disabled user should now succeed22:19
openstackjenkins* dolph.mathews: Updating a disabled user (via xml) should now succeed22:19
openstackjenkins* dolph.mathews: Disabled users should now be returned by GET /users/{user_id}22:19
openstackjenkins* yoga80: Changes to allow password updates even when the user is disabled.Also fixed failing tests.22:19
openstackjenkins* mgius7096: Fixes failing test introduced after disabled check remove.22:19
openstackjenkins* mgius7096: This makes the use of set_enabled more clear22:19
openstackjenkinsProject keystone build #32: SUCCESS in 1 min 5 sec: http://jenkins.openstack.org/job/keystone/32/22:20
openstackjenkins* dolph.mathews: Fixing naming conflict with builtin function next()22:20
openstackjenkins* yoga80: #3 Preventing creation of users with empty user id and pwds.22:20
openstackjenkinsProject keystone build #33: FAILURE in 2.7 sec: http://jenkins.openstack.org/job/keystone/33/22:20
openstackjenkinskevin.mitchell: keystone repo is now at github.com/rackspace/keystone22:20
openstackjenkinsProject keystone build #34: STILL FAILING in 2.7 sec: http://jenkins.openstack.org/job/keystone/34/22:20
openstackjenkinskevin.mitchell: Add success test for GET /v2.0/tokens/<TOKEN_ID> in json and xml22:20
openstackjenkinsProject keystone build #35: STILL FAILING in 2.6 sec: http://jenkins.openstack.org/job/keystone/35/22:20
openstackjenkins* github: Support for listing roles in keystone-manage22:20
openstackjenkins* yoga80: Changes to also return role references as a part of user when get token call is made for a specific tenant.22:20
openstackjenkins* yoga80: Meging changes22:20
openstackjenkins* yoga80: Merging changes22:20
openstackjenkins* yoga80: Merging changes22:20
openstackjenkins* yoga80: Merging changes22:20
openstackjenkins* yoga80: Changes on auth basic middleware component to return roles.Also changes on the application to return roles not tied to a tenant.22:21
openstackjenkins* jaypipes: Add test case for verifying GET /v2.0/tokens returns 404 Not Found22:21
openstackjenkins* jaypipes: Add more test cases for v2 authentication for bad requests and unauthorized results22:21
openstackjenkins* yoga80: Changes to support getTenants call for user with admin privelage and regular user.22:21
openstackjenkins* yoga80: Adding roles as comma seperated values on a single header.22:21
openstackjenkins* yoga80: Removing remerged comments.22:21
openstackjenkins* github: Support for listing BaseURL refs in keystone-manage22:21
openstackjenkins* github: Added URLs to sampledata22:21
openstackjenkins* github: pep822:21
openstackjenkinsProject keystone build #36: STILL FAILING in 2.6 sec: http://jenkins.openstack.org/job/keystone/36/22:21
openstackjenkinskevin.mitchell: It's possible to authenticate through the Admin API.22:21
openstackjenkinsProject keystone build #37: STILL FAILING in 2.7 sec: http://jenkins.openstack.org/job/keystone/37/22:21
openstackjenkinskevin.mitchell: Add test for validate_token22:21
openstackjenkinsProject keystone build #43: STILL FAILING in 2.7 sec: http://jenkins.openstack.org/job/keystone/43/22:21
openstackjenkinskevin.mitchell: Try to use an admin credential to revoke the token.22:21
tr3buchetwow22:21
tr3buchetKABOOOOOOOM22:21
kbringardthat was crazy22:24
*** glenc_ has joined #openstack-dev22:25
*** glenc has quit IRC22:27
*** mattray has quit IRC22:29
*** kbringard_ has joined #openstack-dev22:31
*** kbringard has quit IRC22:31
*** kbringard_ is now known as kbringard22:31
*** kbringard has quit IRC22:32
*** glenc_ is now known as glenc22:34
vishysoren, tr3buchet: test_cloud fixed22:48
vishyanyone working on libvirt tests?22:49
vishyI might start banging them out22:49
vishysoren, ttx, tr3buchet: i didn't realize that VMWare is actually broken.  I think it is worth delaying the milestone until we can verify that VMWare actually works with the multi_nic changes22:53
Davieyvishy: Interesting.. when did the s/mac_address/host/ change happen?22:54
vishyDaviey: mac_addresses were moved out of instances with the multinic changes22:55
Davieyah!22:55
*** troytoman is now known as troytoman-away22:58
*** alekibango has quit IRC22:59
*** medberry is now known as med_out22:59
*** User441 has joined #openstack-dev23:00
*** Tushar has joined #openstack-dev23:00
*** nati has joined #openstack-dev23:01
*** asomya has quit IRC23:01
*** nati has quit IRC23:05
vishymaking progress on vmware changes23:06
*** jkoelker has quit IRC23:08
vishytr3buchet: here?23:12
*** tomeff has quit IRC23:16
*** dragondm has quit IRC23:33
tr3buchetvishy: yes23:33
tr3buchetvishy: was lost in a world of hating flags23:33
vishyattempting to fix vmware23:34
vishyit is a little screwy23:34
vishya bunch of new code was added23:34
tr3buchetnot surprising23:34
tr3buchetsince multi-nic?23:34
*** dragondm has joined #openstack-dev23:35
vishyi think it was the vif plugging stuff23:35
vishythat midokura did23:36
tr3buchetah23:36
tr3buchetyeah their vif driver23:36
vishybasically the tests are all fubar23:39
tr3bucheti'd really like to default flat_interface to None like vlan interface23:39
tr3buchetis why they are skipped :(23:39
tr3buchetyou are serious about delaying milestone it sounds like23:40
*** markvoelker has quit IRC23:40
vishyi had to fake plug_vifs23:40
vishynow i'm just stuck on get_network_with_the_name23:40
vishylooks like the fake isn't good enough to support the new stuff23:41
tr3buchet:(23:42
tr3buchetvishy: vif.py ?23:46
vishynetwork_utils.py23:46
Davieyis nova trunk not pep8 clean for others?23:49
Davieybug 816725?23:49
uvirtbot`Launchpad bug 816725 in nova "pep8 failures" [Undecided,New] https://launchpad.net/bugs/81672523:49
Tusharhttps://bugs.launchpad.net/bugs/816555: Attach volume fails with NameError: global name'vol' is not defined23:50
tr3buchetDaviey: only the doc/ and tools/ directories for me23:50
uvirtbot`Launchpad bug 816555 in nova "Attach volume fails with NameError: global name 'vol' is not defined" [Undecided,New]23:50
TusharThis problem occurs in revision132623:50
*** zaitcev has joined #openstack-dev23:51
* Daviey is using using pep8 0.5.023:51
Davieytr3buchet: What pep8 are you using?23:51
tr3buchet0 6 123:51
Daviey*sigh*.. looks like we might need a more recent on in ubuntu oneiric.23:53
tr3buchetyuh oh23:55
vishywoot23:58
vishydown to 3 fails23:58
Daviey\o/23:58

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