Friday, 2013-02-22

*** dhubbard has quit IRC00:01
*** dhubbard has joined #openstack-dev00:02
*** cp16net is now known as cp16net|away00:05
FatDarrelis there a good way to install openstack on ubuntu00:05
*** garyTh has quit IRC00:07
*** terryh has joined #openstack-dev00:07
*** ecarlin has quit IRC00:08
*** joshuamckenty has joined #openstack-dev00:08
vishyFatDarrel: https://www.devstack.com is a nice way to give it a try00:09
openstackgerritA change was merged to openstack-infra/devstack-gate: Set backdoor port to a random port.  https://review.openstack.org/2260300:09
*** jaybuff has joined #openstack-dev00:11
jaybuffhi, i'm trying to understand why jenkins isn't running some of the keystoneclient tests00:11
*** cp16net|away is now known as cp16net00:12
jaybuffspecifically, tests.test_auth_token_middleware is failing for me using the latest keystoneclient code from github.com00:12
*** garyTh has joined #openstack-dev00:12
henrynashyoung, dolphm: gyee: young said there was an issue with v2 tokens failing policy checks?00:12
jaybuffit looks like https://review.openstack.org/#/c/20427/ is the commit that got merged that caused my failures to happen00:12
openstackgerritA change was merged to openstack/nova: Remove function redefinitions  https://review.openstack.org/2258800:12
jaybuffi think the failure is related to bad test code00:13
jaybuffmy main concern is why jenkins didn't run that particular test when that review was submitted00:13
gyeehenrynash, what's the problem?00:13
clarkbjaybuff: can you link to the review/test that didn't run the test?00:13
jaybuffsure, i'll show the failure00:13
henrynashgyee: not sure, Adam said he thought I broke something :-) but will be back in a bit00:14
*** tomoe_ has quit IRC00:14
henrynashgyee: and I think I might have done…I think assert_admin will fail00:14
*** tomoe_ has joined #openstack-dev00:15
jaybuffclarkb: https://gist.github.com/jaybuff/500969600:15
clarkbjaybuff: I won't be much help sorting out why it fails locally but can look at the jenkins logs to figure out why jenkins didn't run the test00:15
gyeehenrynash, v2 API doesn't use policies right?00:16
*** melwitt has quit IRC00:16
jaybuffclarkb: http://logs.openstack.org/20427/3/gate/gate-python-keystoneclient-python26/727/console.html.gz shows that jenkins didn't run the test00:16
jaybuffbut the test existed in that commit00:17
henrynashgyee: yes, it requires everything to be admin…and asser_admin calls our policy front-end with an "illegal" rule….00:17
jaybuffclarkb: those tests were run because of https://review.openstack.org/#/c/20427/ 00:17
clarkbjaybuff: http://logs.openstack.org/20427/3/gate/gate-python-keystoneclient-python26/727/nose_results.html.gz shows why it did not run00:17
*** dolphm has joined #openstack-dev00:17
*** andrewbogott is now known as andrewbogott_afk00:17
henrynashgyee: since the old one used to force "rule:" on the front of very action00:17
clarkbjaybuff: nose does not fail if setupmodules fail00:18
clarkbit just moves on00:18
henrynashgyee: so actually, I *think* assert admin will fail for v3 and v2 !00:18
*** salv-orlando has quit IRC00:18
jaybuffclarkb: i see00:18
*** winston-d_ has joined #openstack-dev00:18
jaybuffthat nose_results.html complains about examples/pki/cms/auth_token_scoped.pem not existing00:19
jaybuffbut that file is checked in00:19
*** harlowja has quit IRC00:20
henrynashgyee: if I'm right, it's a trivial one-line fix to assert_admin in wsgi.py…..just call the enforce with "rules:admin_required" rather than just "admin_required"00:20
clarkbjaybuff: I am not sure why that happened.00:22
gyeehenrynash, we don't use assert admin in v3, we use controller.protected00:22
henrynashgyee: ah, right, true, but we would just to prices any v2 apis00:23
henrynashprocess....00:23
*** sgordon has quit IRC00:23
*** jbr_1 has joined #openstack-dev00:23
*** melwitt has joined #openstack-dev00:23
henrynashgyee: we really should have a test for that :-)00:24
* jaybuff looks to see if it is still happening00:24
ladquinvishy, ping?00:24
*** sgordon has joined #openstack-dev00:25
*** melwitt1 has joined #openstack-dev00:25
*** alszar has quit IRC00:26
*** joshuamckenty has quit IRC00:26
*** harlowja has joined #openstack-dev00:26
openstackgerritA change was merged to openstack/nova: Add missing ec2 security group quantum mixin  https://review.openstack.org/2262300:26
gyeehenrynash, yeah, good idea00:27
openstackgerritA change was merged to openstack/keystone: Change the default LDAP mapping for description.  https://review.openstack.org/2235200:27
openstackgerritA change was merged to openstack-infra/devstack-gate: Add statsd to pip-requires.  https://review.openstack.org/2124900:27
*** melwitt has quit IRC00:28
*** sacharya has joined #openstack-dev00:28
*** woodspa has quit IRC00:30
*** woodspa has joined #openstack-dev00:30
*** woodspa has quit IRC00:30
*** brucer_afk has joined #openstack-dev00:34
jaybuffhere's a change that happened today, and keystoneclient py26 is still not running AuthTokenMiddlewareTests (and some others):00:34
jaybuffhttp://logs.openstack.org/22476/1/gate/gate-python-keystoneclient-python26/757/nose_results.html00:34
*** jbr_1 has quit IRC00:35
*** aeperezt has quit IRC00:37
*** networkstatic has joined #openstack-dev00:38
*** brucer has quit IRC00:38
*** aeperezt has joined #openstack-dev00:40
*** troytoman-away is now known as troytoman00:40
clarkbjaybuff: now that I have looked at your original gist it looks like you an improper setUp() super call for that test class00:45
clarkbit keeps recursively calling itself00:45
*** dachary has quit IRC00:45
jaybuffyes, i agree the code is wrong00:45
jaybuffi am using the latest code00:46
jaybuffwith no changes00:46
clarkbah I see00:46
* clarkb looks at existing code00:46
*** roampune1 has quit IRC00:46
jaybuffthe author of that patch was josh harlow, and i looked at it with him00:46
jaybuffand he agreed it was broken00:46
jaybuffmy concern is that the patch passed CI00:46
clarkbit passed CI because nose is broken00:48
clarkband I see the problem now00:48
jaybuffawesome00:48
clarkbthere are two super calls in setUp for the base middle ware test class00:48
clarkbonly the first one should exist00:48
openstackgerritA change was merged to openstack-dev/devstack: Adding sysstat monitoring  https://review.openstack.org/2246700:49
jaybuffoh, yeah, i was able to hack around it and get the tests passing00:49
jaybuffjosh is going to submit a fix to gerrit00:50
openstackgerritA change was merged to openstack/nova: Fix baremetal migration skipping.  https://review.openstack.org/2261900:50
openstackgerritA change was merged to openstack/nova: Migration 135: Fix drop_column statement  https://review.openstack.org/2262500:51
clarkband the fix for nose is to stop using setUpModule then start using testr :)00:51
*** joshuamckenty has joined #openstack-dev00:54
*** tomoe_ has quit IRC00:54
*** zzs has left #openstack-dev00:58
jaybuffcool, thanks for looking at this clark00:59
*** jaybuff has left #openstack-dev01:00
*** hemna has quit IRC01:00
*** hemna has joined #openstack-dev01:01
henrynashgyee: so not so sure it is broken….since there are tests….but now I can't see how they are passing (unless of course the new policy engine allows you to pass in a rule without labelling it as such)01:01
*** colinmcnamara has quit IRC01:02
*** njoy_ has quit IRC01:03
henrynashgyee: ok, so I can't see how it's broken…you absolutely allow to pass in just the name of the rule on its own as an action (in fact you can't pass in "rule:action")….I'll wait to find out more Adam or others...01:09
*** thingee is now known as thingee_zzz01:09
henrynashayoung: let me know more about how v2 token & policy is broken (not saying it isn't, just been running some tests and haven't got it to fail)01:10
*** FatDarrel has quit IRC01:14
henrynashgyee: do we allow someone to get use a v2 token to all v3 apis?01:15
*** dolphm has quit IRC01:15
*** jog0 has quit IRC01:18
openstackgerritA change was merged to openstack/quantum: Imported Translations from Transifex  https://review.openstack.org/2264401:19
*** dolphm has joined #openstack-dev01:19
henrynashayoung: is the issue that we want to support a user with a v2 token getting access to v3 apis?  If so, then, yep, my building of the creeds in the protection wrapper isn't good enough01:19
*** soody has quit IRC01:22
*** nati_ueno has joined #openstack-dev01:23
*** boden has joined #openstack-dev01:25
*** FatDarrel has joined #openstack-dev01:25
*** reed has quit IRC01:27
*** joshuamckenty has quit IRC01:28
*** nandX64 has joined #openstack-dev01:31
*** FatDarrel has quit IRC01:31
*** obondarev has quit IRC01:32
*** renier has quit IRC01:32
*** renier_ has joined #openstack-dev01:32
*** bdpayne has quit IRC01:33
gyeehenrynash, we allow v2 to v3 tokens to be use interchangeable01:33
*** datsun180b has quit IRC01:34
*** obondarev has joined #openstack-dev01:34
*** FatDarrel has joined #openstack-dev01:35
*** gyee has quit IRC01:35
*** FatDarrel_ has joined #openstack-dev01:36
*** burris has joined #openstack-dev01:36
*** soody has joined #openstack-dev01:37
*** vkmc has quit IRC01:39
ayounghenrynash, yeah, the building of the creds is v3 specific.  The old one was v2 specific.  I just pulled them together, not a big deal.  But surprised it worked.01:40
*** boden has quit IRC01:40
*** danwent has quit IRC01:40
*** danwent has joined #openstack-dev01:41
*** bencherian has quit IRC01:41
*** yaguang has joined #openstack-dev01:41
*** anteaya has left #openstack-dev01:43
*** alexxu has joined #openstack-dev01:43
*** FatDarrel_ has quit IRC01:44
ayoungdolphm, so...the hold has been that henrynash 's latest change, which sets the policy path, seems to change the policy path for later tests01:44
dolphmayoung: policy path?01:45
ayoungdolphm, yeah...it is _POLICY_PATH inside the policy.py file, set by...01:45
dolphmpolicy_file in keystone.conf01:45
ayoungdolphm, test_v3_policy.py01:45
dolphmayoung: can be overridden in tests by self.opt(policy_file=etc('policy.json')) for example, which is done in keystone.test right now01:45
ayoungdolphm, test_v3_protect01:45
dolphmetcdir*01:46
ayoungyeah, so this in another variable01:46
dolphmayoung: i believe policy path is derived from keystone.conf01:46
ayoungyeah, he's doing an mktemp01:46
*** spzala has quit IRC01:46
*** jimfehlig has quit IRC01:46
ayoungdolphm, the thing is, it is inside on test, and affecting other tests01:46
dolphmayoung: it's a global variable01:47
dolphmayoung: set it through conf instead01:47
dolphmayoung: bbl01:47
*** dolphm has quit IRC01:47
*** alexxu has quit IRC01:47
*** garyTh has quit IRC01:48
*** monst_ has quit IRC01:49
*** esp1 has quit IRC01:51
*** CaptTofu has quit IRC01:51
*** CaptTofu has joined #openstack-dev01:51
*** nandX64 has quit IRC01:53
*** dolphm has joined #openstack-dev01:54
openstackgerritA change was merged to openstack/ceilometer: Support different publisher interval  https://review.openstack.org/2213201:55
*** CaptTofu has quit IRC01:56
ayoungdolphm, so...I think the bleed over is happening because we on't call tear down in the base class.01:58
dolphmayoung: link to patch?01:59
*** tomoe_ has joined #openstack-dev02:00
*** bing_bu has joined #openstack-dev02:02
*** CaptTofu has joined #openstack-dev02:04
*** cp16net is now known as cp16net|away02:04
*** Nachi has joined #openstack-dev02:05
*** CaptTofu has quit IRC02:06
*** CaptTofu has joined #openstack-dev02:06
*** alexxu has joined #openstack-dev02:08
*** shang has quit IRC02:09
*** topol has joined #openstack-dev02:10
*** CaptTofu has quit IRC02:11
*** unix has joined #openstack-dev02:13
ayoungdolphm, still looking, but02:14
ayoungthe v3 tests don't call the tear down...I'm still tracking down the root cause....02:15
ayoungCurrent hypothesis is that the policy CACHE needs to be flushed between tests02:15
ayoungdolphm, look in v3_test tearDown.  It doesn't call super.tearDown, but I am guessing that is intentional02:16
*** alexxu has quit IRC02:17
dolphmayoung: i doubt that's intentional02:18
ayoungand the base test does CONF.reset...but I suspect now that it is irrelevant, that the policy rules are already loaded02:18
*** nati_uen_ has joined #openstack-dev02:18
*** cp16net|away is now known as cp16net02:18
*** nati_uen_ has quit IRC02:18
ayoungdolphm, wel...you wrote it02:19
dolphmayoung: i didn't write any of the policy stuff02:19
ayoungclass RestfulTestCase(test_content_types.RestfulTestCase):02:19
dolphmayoung: i revised it to support conf, but that's it02:19
*** monst_ has joined #openstack-dev02:19
ayoungdolphm, no this is in the v3 test code02:19
*** maoy has joined #openstack-dev02:19
ayoungsetUp doesn't call super .setup, I assumed because you were trying to avoid the fixtures02:19
*** nati_uen_ has joined #openstack-dev02:19
*** almaisan-away has quit IRC02:20
dolphmayoung: ah, that's probably true02:20
*** kagan has quit IRC02:20
*** terryh has quit IRC02:20
ayoungdolphm, still, I don't think that is the problem.02:20
*** Nachi has quit IRC02:21
ayoungdolphm, right now I am explicitly resetting the policy path in a teardown, but it is still failing, I assume for the same reason02:21
ayoungbasically, my policy rules are not in the cache.02:21
*** almaisan-away has joined #openstack-dev02:22
*** ctracey` has quit IRC02:22
*** ctracey` has joined #openstack-dev02:22
*** jcmartin has quit IRC02:23
mattraehi folks, when i have multiple compute nodes, instances can not resolve eachother via dns. is there an openstack component that handles this? nova-dns maybe?02:24
mattraeoops, meant to ask in non-dev channel02:24
*** shang has joined #openstack-dev02:24
*** danwent has quit IRC02:25
ayoungdolphm, the test_v3_protection is writing its own rules to the file.  I think keystone.common.policy.reset() will do the trick02:26
dolphmayoung: does that already exist? lol02:26
ayoungdolphm, right, it just isn't called yet02:27
*** shang has quit IRC02:27
*** nati_uen_ has quit IRC02:27
ayoungdolphm, interesting.  Now, when I call reset in tearDown, my tests fail consistantly.02:28
ayoungnot the first one, but the second and third02:29
*** alexxu has joined #openstack-dev02:29
ayoungdolphm, the cache test does an mtime, which is why it wasn't rereading the file once set back to the original02:30
*** Nachi has joined #openstack-dev02:30
*** monst_ has quit IRC02:30
ayoungsince the cache was newer than the original file...02:30
*** Nachi has quit IRC02:31
*** Nachi has joined #openstack-dev02:32
*** Nachi has joined #openstack-dev02:33
*** dolphm has quit IRC02:34
*** dolphm has joined #openstack-dev02:35
*** shang has joined #openstack-dev02:36
ayoungdolphm, henrynash got it02:36
ayoungfrom keystone.policy.backends import rules02:36
ayoungrules.reset()02:36
*** CaptTofu has joined #openstack-dev02:37
dolphmayoung: henrynash: cool02:37
ayoungdolphm, I got it, irc just didn't put a comma after henrynash .  Don't want him stealing my glory.02:37
ayoungdolphm, so, do you like the latest V3 API changes?  I put in 'impersonate' and  I think we're good, at least content wise02:39
ayoungprobably can always keep cleaning up format etc02:39
*** terry7 has quit IRC02:39
ayoungdolphm, well, except the roles thing02:40
dolphmayoung: i didn't read the examples section in my last review02:40
dolphmayoung: did you remove the two lists in a response thing?02:40
dolphmayoung: and do you like the api better or worse after all this? lol02:41
ayoungdolphm,  the one thing is, I was resistant to putting the full roles in,  but it is probably too big a focus on performance on my part02:41
ayoungdolphm, API is definitely better02:41
dolphmayoung: http://teacherslovesmartboards.com/wp-content/uploads/2012/03/eat-grandma.jpg02:41
ayoungso..I probably will do one more revision with the roles objects.02:42
dolphmayoung: i think the flexibility we get down the line is worth the performance penalty02:42
ayoungyeah, and people hate the uuids02:42
ayoungrightfuly so02:42
ayoungso giving them the role names and the endpoint names is much friendlier02:43
*** CaptTofu has quit IRC02:43
ayoungOK...one more revision it is.02:43
*** CaptTofu has joined #openstack-dev02:44
dolphmayoung: i've been thinking about making the generated uuid length configurable, and defaulting to 6 like in the docs, and recommended the full 32 for a real deployment02:44
ayoungdolphm, gyee asked if impoersonate was going to be defaulted or if it should be mandatory to specify.  I think I like mandatory02:44
dolphmayoung: would you be in favor?02:44
ayoung6chars ids?  So long as we don't call them uuids02:44
*** yaguang has quit IRC02:44
dolphmayoung: your call on that one, as long as it's opt-in02:44
dolphmayoung: i never call them uuid's ;)02:44
ayoungdolphm, read up " making the generated uuid length configurable"02:45
ayoungAs Mr Connery learned, Never say Never02:45
ayoungopt in is a definite, I think mandatory02:45
ayoungcreating a trust is a rare event, and people should think hard about the permissions they are giving away.  Lets not hide anything from them\02:46
dolphmayoung: we could base-36 encode them to get even more variety in 6 chars02:46
ayoungdolphm, OK need to clean up pep 8 and then I'll ahve another patch to encode02:46
ayoungdolphm, I wonder if that is true.02:46
ayoungI wouldn't be surprised if that actually decreased the entropy02:47
ayoungsomehow02:47
*** Nachi has quit IRC02:47
*** CaptTofu has quit IRC02:48
dolphmayoung: i don't know how long 32-char base-16 UUID's would be in base-36, but it'd be the same value either way02:48
dolphmayoung: there's some math in there worth doing02:49
ayoungdolphm, I am sorely tempted to hack in an option to run_tests.sh for short pep802:49
dolphmayoung: a base-36 encoded uuid: 7kq687xxvyruynjf1fz8vuu6q02:49
dolphmshort pep8?02:50
dolphmayoung: oh, just write pep8 compliant code :)02:50
ayoungdolphm, without the explantions02:50
dolphmayoung: do that on wednesday :)02:50
*** giulivo has quit IRC02:50
ayoungdolphm, yeah...02:50
ayoungHavana is going to be no big ticket itmes for me, just all cleanup02:51
ayoungunit tests and LDAP and performance...02:51
ayoungand a pony and the moon on a stick02:51
*** amerine has quit IRC02:51
dolphmayoung: you really think it will stay that way? :P02:52
*** riskable has quit IRC02:53
ayoungdolphm, not for a moment02:53
ayoungI have too many dreams02:53
ayoungdolphm, but I sorely want to make HTTPD and all that gets us the default.  I do not trust Eventlet, and I want Kerberos, etc...02:54
ayoungX509 in place of tokens is also a nice dream.02:54
dolphmayoung: pow(36, 6) == 2,176,782,336 -- random enough for reliable small deployments?02:55
*** amerine has joined #openstack-dev02:55
ayoungdolphm, probably.  What happens if there is a clash?02:55
*** monst_ has joined #openstack-dev02:56
ayoungIt really only matters within the same entity type02:56
dolphmayoung: unnexpected 409 conflict that isn't the user's fault02:56
ayoungI assume uuids because you want to be able to generate on different machines without deconflicting02:56
dolphmayoung: yep02:56
ayoungdeconflicting on line that is02:56
dolphmayoung: and provide uniqueness among two different clouds to allow reliable federation02:56
ayoungdolphm, but if two users get the same UUID, from different keystone servers, it could be bad02:57
dolphmayoung: quite02:57
ayoungsame thing with, say glance images and so on02:57
*** andrewbogott has joined #openstack-dev02:57
*** andrewbogott has joined #openstack-dev02:57
ayoungdolphm, something to keep us up at night02:57
dolphmayoung: that's why it'd have to be configurable for big deployments, and it's also something you can reconfigure as you scale up02:58
ayoungdolphm, it would be nice if you could do a Nosql solution for identifying that duplication has occured.  Does't have to be right way, just "eventually"03:00
*** radez is now known as radez_g0n303:00
ayoungdolphm, OK,  finally got an updated patch.  The todo items I know about for it are03:01
ayoung1.  impersonate03:01
ayoung2.  objects for roels and endpoints03:01
*** garyk has quit IRC03:01
ayoung3.  v3 auth integration03:01
*** garyk has joined #openstack-dev03:01
ayoung4.  little bit of policy work03:01
ayoung5.  clean up the unit tests03:02
ayoungI think that is it03:02
dolphmayoung: mind if i check it out in the morning?03:03
ayoungdolphm, don't mind at all03:03
*** giulivo has joined #openstack-dev03:03
ayoungI'm going to start hackin on the API doc one more time and get the objects in there.03:04
ayoungdolphm, we are just putting in names and ids for roles and for endpoints, right?03:04
ayoungno other attributes?03:04
dolphmayoung: endpoints don't have names, but otherwise, yes03:04
dolphmayoung: eventually links should go in there03:04
ayoungdolphm, the URLS?03:04
*** unix has quit IRC03:04
*** baba has joined #openstack-dev03:04
ayoungdolphm, that is what I mean, the links03:04
dolphmayoung: i mean "links": {"self": "http://keystone:35357/v3/roles/{role_id}"}03:05
*** monst_ has quit IRC03:05
ayoungso  endspoint : [ {id:bah, links: [http://blah, http://blah]}]?03:05
dolphmayoung: the link back to keystone for that entity for restful hateos browseryness03:05
ayoungdolphm, yeah03:06
dolphmayoung: not the endpoint url03:06
*** n0ano has quit IRC03:06
dolphmayoung: but if you think the endpoint url should go in there, just stick it in the spec -- i wouldn't be opposed03:06
ayoungdolphm, yeah, I think it would be friendly03:07
ayounglist trusts could get huge03:07
dolphmayoung: exclude the url's then -- we can always add them later03:07
dolphmayoung: and we can't take them away later03:08
*** KurtMartin has joined #openstack-dev03:09
*** woodspa has joined #openstack-dev03:11
*** Mandell has quit IRC03:16
*** amerine has quit IRC03:18
*** woodspa has quit IRC03:20
*** jcmartin has joined #openstack-dev03:20
*** roampune has joined #openstack-dev03:21
*** roampune has quit IRC03:23
*** roampune has joined #openstack-dev03:24
*** jcmartin has quit IRC03:28
*** yaguang has joined #openstack-dev03:28
*** jcmartin has joined #openstack-dev03:29
*** shang_ has joined #openstack-dev03:31
*** monst_ has joined #openstack-dev03:32
*** shang has quit IRC03:33
*** BalleS_ has joined #openstack-dev03:34
*** susanne-balle has quit IRC03:38
*** aeperezt has quit IRC03:39
*** monst_ has quit IRC03:41
*** melwitt1 has quit IRC03:45
*** aeperezt has joined #openstack-dev03:46
*** roampune has quit IRC03:49
*** nati_ueno_2 has joined #openstack-dev03:53
*** nati_ueno has quit IRC03:54
*** otherwiseguy has quit IRC03:58
*** wwallnrr__ has quit IRC03:58
*** epim has quit IRC03:59
*** martine has joined #openstack-dev04:08
*** todo_stack has quit IRC04:08
*** monst_ has joined #openstack-dev04:08
*** sandywalsh_ has quit IRC04:11
*** bencherian has joined #openstack-dev04:12
*** monst_ has quit IRC04:17
*** otherwiseguy has joined #openstack-dev04:20
*** Tross has joined #openstack-dev04:24
*** jcmartin_ has joined #openstack-dev04:24
*** anniec has joined #openstack-dev04:24
*** jcmartin has quit IRC04:27
*** jcmartin_ is now known as jcmartin04:27
*** henrynash has quit IRC04:31
*** smoser has quit IRC04:35
*** crandquist has quit IRC04:36
*** henrynash has joined #openstack-dev04:37
*** smoser has joined #openstack-dev04:39
*** esp has joined #openstack-dev04:40
*** bencherian has quit IRC04:44
*** monst_ has joined #openstack-dev04:44
*** sandywalsh has joined #openstack-dev04:44
*** martine has quit IRC04:46
*** Mandell has joined #openstack-dev04:51
*** mikal has quit IRC04:52
*** mikal has joined #openstack-dev04:53
*** monst_ has quit IRC04:53
*** mohits has joined #openstack-dev04:54
*** nati_ueno has joined #openstack-dev04:55
*** ek6 has quit IRC04:56
*** andrewbogott has quit IRC04:58
*** dolphm has quit IRC05:04
*** gyee has joined #openstack-dev05:06
*** armaan has joined #openstack-dev05:07
*** armaan has left #openstack-dev05:07
*** topol has quit IRC05:13
*** topol has joined #openstack-dev05:14
*** nunosantos has quit IRC05:14
*** nati_ueno has quit IRC05:16
*** Tross has quit IRC05:16
*** monst_ has joined #openstack-dev05:20
*** zaitcev has quit IRC05:25
*** aeperezt has quit IRC05:28
*** monst_ has quit IRC05:29
*** dolphm has joined #openstack-dev05:35
*** navid has joined #openstack-dev05:40
*** almaisan-away is now known as al-maisan05:41
*** maoy has quit IRC05:43
*** dolphm has quit IRC05:43
*** brucer_afk has quit IRC05:45
*** al-maisan is now known as almaisan-away05:46
*** koolhead17 has joined #openstack-dev05:49
*** monst_ has joined #openstack-dev05:56
*** yaguang has quit IRC05:57
*** nati_ueno_2 has quit IRC06:02
*** nati_ueno has joined #openstack-dev06:02
*** cdub_ has quit IRC06:06
*** cdub_ has joined #openstack-dev06:06
*** monst_ has quit IRC06:07
*** gyee has quit IRC06:08
*** aswadrangnekar has left #openstack-dev06:10
*** markmc has joined #openstack-dev06:11
*** olaph has quit IRC06:13
*** olaph has joined #openstack-dev06:14
openstackgerritA change was merged to openstack/quantum: Provide a default api_extensions_path for nvp_plugin  https://review.openstack.org/2264106:16
*** esp has quit IRC06:21
*** yjiang5_away has quit IRC06:23
*** almaisan-away is now known as al-maisan06:25
*** Mandell has quit IRC06:29
*** Ryan_Lane has quit IRC06:30
*** KurtMartin has quit IRC06:30
*** sacharya has quit IRC06:31
*** monst_ has joined #openstack-dev06:33
*** anniec has quit IRC06:34
*** boris-42 has joined #openstack-dev06:41
*** alunduil has joined #openstack-dev06:43
*** monst_ has quit IRC06:43
*** topol has quit IRC06:44
*** llu has quit IRC06:44
*** alunduil has quit IRC06:45
*** soody has quit IRC06:49
*** soody has joined #openstack-dev06:53
*** ek6 has joined #openstack-dev06:56
*** PaulCarlton has joined #openstack-dev07:02
*** baba has quit IRC07:03
*** techlife has quit IRC07:05
*** techlife has joined #openstack-dev07:05
*** dedis has quit IRC07:07
*** afrittoli has joined #openstack-dev07:09
*** troytoman is now known as troytoman-away07:09
*** amerine has joined #openstack-dev07:09
*** monst_ has joined #openstack-dev07:09
*** andreaf has quit IRC07:10
*** unix has joined #openstack-dev07:12
*** morganfainberg has left #openstack-dev07:14
openstackgerritA change was merged to openstack/nova: import base_dir_name config option into vmwareapi  https://review.openstack.org/2257707:18
*** alunduil has joined #openstack-dev07:19
*** yaguang has joined #openstack-dev07:19
*** monst_ has quit IRC07:20
*** yolanda has joined #openstack-dev07:22
*** alunduil has quit IRC07:22
*** afazekas has joined #openstack-dev07:27
*** Ryan_Lane has joined #openstack-dev07:28
*** kbrierly has quit IRC07:29
*** esp has joined #openstack-dev07:31
*** kbrierly has joined #openstack-dev07:33
*** smulcahy has quit IRC07:34
*** llu has joined #openstack-dev07:35
*** smulcahy has joined #openstack-dev07:35
*** mindpixel has joined #openstack-dev07:38
*** alunduil has joined #openstack-dev07:43
*** unix has quit IRC07:43
*** unix has joined #openstack-dev07:43
*** boris-42 has quit IRC07:44
*** alunduil has quit IRC07:45
*** Mandell has joined #openstack-dev07:46
*** masumotok has joined #openstack-dev07:46
*** monst_ has joined #openstack-dev07:47
masumotokCurrently, I did "git-review", and jenkins raise error. It is not related to my patch, but my environment also raise same error. After I checked it, I realize library version problem of Jenkins, because if I try pyzmq==2.2.0.1 environment, it worsk, but pyzmq=13.0.0 environment, it didnt.07:49
masumotokDoes anyone know who should I ask about Jenkins?07:49
lifelessmasumotok: requires.txt describes the dependencies07:49
lifelessmasumotok: you can ask on #openstack-infra if you want details about jenkins07:50
*** soody has quit IRC07:50
masumotoklifeless: thank you. I'll visit openstack-infra.07:50
*** esp has left #openstack-dev07:51
*** masumotok has left #openstack-dev07:51
*** monst_ has quit IRC07:59
*** unix has quit IRC07:59
*** unix has joined #openstack-dev07:59
*** pnavarro has joined #openstack-dev08:00
*** tomoe_ has quit IRC08:02
*** tomoe_ has joined #openstack-dev08:02
*** rafaduran has joined #openstack-dev08:03
*** FlorianOtel has joined #openstack-dev08:05
*** xga_ has joined #openstack-dev08:05
*** tomoe_ has quit IRC08:07
*** koolhead17 has quit IRC08:09
*** adjohn has quit IRC08:09
*** FlorianOtel has quit IRC08:11
*** thouveng has joined #openstack-dev08:14
*** ondergetekende has joined #openstack-dev08:15
*** henrynash has quit IRC08:18
garykarosen: you around08:18
*** mmagr has joined #openstack-dev08:22
*** unix has quit IRC08:24
*** unix has joined #openstack-dev08:24
*** FlorianOtel has joined #openstack-dev08:24
*** monst_ has joined #openstack-dev08:24
*** salv-orlando has joined #openstack-dev08:28
*** zoresvit has joined #openstack-dev08:30
*** gael has joined #openstack-dev08:32
*** davidha has quit IRC08:33
*** psedlak has joined #openstack-dev08:34
*** monst_ has quit IRC08:36
*** zoresvit has quit IRC08:37
*** jgallard has joined #openstack-dev08:37
*** nsatterl_ has joined #openstack-dev08:40
*** adjohn has joined #openstack-dev08:40
*** dachary has joined #openstack-dev08:41
*** zing has joined #openstack-dev08:42
*** shardy_afk is now known as shardy08:49
*** adjohn has quit IRC08:49
*** torandu has quit IRC08:52
*** mattymo has joined #openstack-dev08:53
*** zoresvit has joined #openstack-dev08:53
ttxnijaba: cutting g3 for ceilometer08:54
*** torandu has joined #openstack-dev08:54
*** eglynn has joined #openstack-dev08:55
*** jpich has joined #openstack-dev08:55
*** winston-d_ has quit IRC08:57
*** pixelbeat has joined #openstack-dev09:00
*** darjeeling has quit IRC09:01
zykes-garyk: question: Did LBaaS land ?09:01
Davieyzykes-: Err, you know G-3 has only just been cut now?09:01
zykes-Daviey: yeh :)09:01
zykes-I was a bit over-eager09:02
Davieyzykes-: then give us a chance :P09:02
zykes-:P09:02
* zykes- brings out the whip, mosh!09:02
*** dosaboy has joined #openstack-dev09:02
*** monst_ has joined #openstack-dev09:02
*** jprovazn has joined #openstack-dev09:04
*** ndipanov has joined #openstack-dev09:05
*** dachary has quit IRC09:06
zykes-Daviey: btw, python-ceilometer takes in mongodb09:08
*** boris-42 has joined #openstack-dev09:08
*** koolhead17 has joined #openstack-dev09:08
zykes-ehm, ceilometer-agent-compute that is09:08
*** boden has joined #openstack-dev09:09
Davieyzykes-: Great!09:09
zykes-Daviey: where would you have me report bugs ? :p09:10
ttxnijaba: https://launchpad.net/ceilometer/grizzly/grizzly-3 <- you can announce it09:10
zykes-ttx: are there some features missing for quantum ?09:10
ttxbah, I'll just do it09:10
*** dachary has joined #openstack-dev09:11
ttxzykes-: for some definition of "missing" yes.09:11
ttxif by missing you mean "added on the last week and did not make it", then yes09:11
ttxalaso a number of features were granted an exception and will land next week09:11
*** derekh has joined #openstack-dev09:11
ttxlike lbaas09:12
Davieyzykes-: ubuntu-bug ceilometer .. or by hand, https://bugs.launchpad.net/ubuntu/+source/ceilometer/+filebug09:12
Davieyzykes-: the former also includes som data about package version etc.09:12
zykes-ttx: lbaas will be going in ?09:13
koolhead17al-maisan: slides of your heat demo needed :)09:14
ttxzykes-: that's what has been promised to me09:14
*** Mandell has quit IRC09:14
ttxzykes-: https://launchpad.net/quantum/+milestone/grizzly-rc109:14
*** darraghb has joined #openstack-dev09:14
*** monst_ has quit IRC09:15
*** afazekas has quit IRC09:15
zykes-koolhead17: which demo ? : p09:15
*** adjohn has joined #openstack-dev09:15
al-maisankoolhead17: https://github.com/al-maisan/p/blob/master/2013-02-19-chosug.zh.heat.pdf?raw=true09:15
koolhead17zykes-: did a nice heat demo at swiss OS meetup09:16
zykes-k09:16
koolhead17al-maisan: planning to blog with deployment instructions sir :P09:17
al-maisankoolhead17: ah, nice :-)09:18
*** jruzicka has joined #openstack-dev09:18
koolhead17al-maisan: i was poking you 4 that :P09:18
*** bing_bu has quit IRC09:19
al-maisanno problem :-) when you install/deploy on ubutu 12.04 the following may be handy: https://bugs.launchpad.net/heat/+bug/112788609:19
uvirtbotLaunchpad bug 1127886 in heat "Fix ubuntu 12.04 installation problems" [Undecided,New]09:19
*** adjohn has quit IRC09:20
*** nati_ueno has quit IRC09:23
*** nati_ueno has joined #openstack-dev09:24
*** doude has joined #openstack-dev09:27
*** networkstatic has quit IRC09:27
*** trapni has joined #openstack-dev09:28
*** amerine has quit IRC09:28
*** matelakat has joined #openstack-dev09:31
*** nati_ueno_2 has joined #openstack-dev09:32
*** trapnii has joined #openstack-dev09:32
*** nati_ueno has quit IRC09:34
*** trapni has quit IRC09:35
nijabattx: ack09:38
*** egallen has joined #openstack-dev09:39
*** monst_ has joined #openstack-dev09:41
ttxnijaba: I made a follow-up announcement09:42
nijabattx: where?09:42
ttxon the openstack list09:43
* nijaba looks09:43
ttxnijaba: https://lists.launchpad.net/openstack/msg21181.html09:43
nijabattx: nice.  thanks09:44
*** darjeeling has joined #openstack-dev09:44
*** al-maisan is now known as almaisan-away09:44
*** trapnii has quit IRC09:47
*** henrynash has joined #openstack-dev09:47
*** xga_ has quit IRC09:48
*** danpb has joined #openstack-dev09:51
*** nati_ueno_2 has quit IRC09:53
*** monst_ has quit IRC09:53
*** nati_ueno has joined #openstack-dev09:54
*** k4n0 has joined #openstack-dev09:54
*** nati_ueno has quit IRC09:58
*** trapnii has joined #openstack-dev10:00
*** alexxu has quit IRC10:10
*** adjohn has joined #openstack-dev10:16
*** unix has quit IRC10:18
*** unix has joined #openstack-dev10:19
*** monst_ has joined #openstack-dev10:19
*** mattmonkeymagic has joined #openstack-dev10:20
mattmonkeymagicanyone around for a Filesystem question on compute node??10:20
*** adjohn has quit IRC10:20
mattmonkeymagichaving a prob with compute node FS filling up with snapshots, but there doesn't seem to be anything in OpenStack monitoring if a nodes storage is reaching capacity. Is there a config setting I can change to warn on disk full??10:26
*** jprovazn has quit IRC10:27
*** gray-- has joined #openstack-dev10:29
*** monst_ has quit IRC10:32
*** hughsaunders has quit IRC10:35
mattmonkeymagicas in the node will still accept requests for new instances to fire up, even tho that node no longer has sufficient capacity to fulfil request10:35
*** dachary has quit IRC10:39
*** dachary has joined #openstack-dev10:41
*** darjeeling has quit IRC10:43
*** henrynash has quit IRC10:49
marohgongysh : hi10:50
*** ndipanov has quit IRC10:53
*** henrynash has joined #openstack-dev10:54
*** monst_ has joined #openstack-dev10:58
*** henrynash has quit IRC11:00
*** unix has quit IRC11:01
*** henrynash has joined #openstack-dev11:01
gongyshmaroh: hi11:04
*** unix has joined #openstack-dev11:05
*** mkollaro has joined #openstack-dev11:07
*** monst_ has quit IRC11:10
*** pcm__ has joined #openstack-dev11:19
*** pcm__ has quit IRC11:19
*** henrynash has quit IRC11:19
*** pcm_ has joined #openstack-dev11:20
*** mohits has quit IRC11:21
*** olaph has quit IRC11:23
*** olaph has joined #openstack-dev11:25
*** darjeeling has joined #openstack-dev11:26
*** unix has quit IRC11:27
*** unix has joined #openstack-dev11:32
*** zbitter has joined #openstack-dev11:37
*** zaneb has quit IRC11:37
*** monst_ has joined #openstack-dev11:37
*** ndipanov has joined #openstack-dev11:37
*** davidha has joined #openstack-dev11:39
*** martine has joined #openstack-dev11:39
*** yaguang has quit IRC11:40
*** sthaha has joined #openstack-dev11:43
*** vkmc has joined #openstack-dev11:45
*** adjohn has joined #openstack-dev11:47
*** gray-- has quit IRC11:48
*** monst_ has quit IRC11:49
*** adjohn has quit IRC11:51
*** dachary1 has joined #openstack-dev11:52
*** dachary has quit IRC11:55
openstackgerritA change was merged to openstack/tempest: Simplify xml-json inheritance in compute  https://review.openstack.org/2211211:59
smulcahysdague: thanks for that review12:00
sdaguesmulcahy: no prob12:00
sdaguethanks for that super quick turn around :)12:00
openstackgerritA change was merged to openstack/tempest: Have all test case to use a single base class  https://review.openstack.org/2068712:02
*** arbrandes has joined #openstack-dev12:06
*** dachary1 is now known as dachary12:08
*** monst_ has joined #openstack-dev12:16
openstackgerritA change was merged to openstack-dev/devstack: Update cinder to use rootwrap_config  https://review.openstack.org/2247412:16
openstackgerritA change was merged to openstack-dev/devstack: Fix EXTRA_BAREMETAL_OPTS  https://review.openstack.org/2228112:18
*** mkollaro has quit IRC12:22
openstackgerritA change was merged to openstack-dev/devstack: replace use of cirros 0.3.0 with 0.3.1  https://review.openstack.org/2245212:22
openstackgerritA change was merged to openstack-dev/devstack: Deprecate DATABASE_TYPE and use_database  https://review.openstack.org/2263512:26
*** romcheg has joined #openstack-dev12:26
*** monst_ has quit IRC12:28
*** romcheg1 has joined #openstack-dev12:28
*** maoy has joined #openstack-dev12:28
*** adalbas has joined #openstack-dev12:30
openstackgerritA change was merged to openstack-dev/devstack: Upload XenServer style ovf directly to glance  https://review.openstack.org/2135412:30
*** romcheg has quit IRC12:32
*** mkollaro has joined #openstack-dev12:35
*** unix has quit IRC12:35
*** tomoe_ has joined #openstack-dev12:36
smulcahysdague: keen to get it merged before the weekend12:36
smulcahyis jenkins normally that slow verifying changes or is it v busy?12:37
sdaguesmulcahy: it should go through12:37
sdaguewhat was the review again?12:37
smulcahysdague: https://review.openstack.org/#/c/2253312:39
sdaguesmulcahy: oh, well we need dtroyer to take a look again. even though it's just dean and I doing core reviews, we try to make sure we both +2 things before they go in12:39
smulcahyah, musta heard me :)12:39
sdaguesmulcahy: yeh, figure ~1 hr for the verify12:39
smulcahysdague: ah ok, devstack tests are bigger than cinder ones I guess, makes sense :)12:40
sdagueyeh, it brings up a full devstack node12:40
sdagueand runs the 700 tempest tests o nit12:40
sdaguewhich means spawning vms, creating volumes, all sorts of things12:40
*** johnthetubaguy1 has joined #openstack-dev12:41
*** timello has quit IRC12:41
smulcahysure12:41
smulcahyok, hopefully dean will get a chance to look at it today12:41
mordredsdague: guess what?12:43
sdaguemordred: .... oh no12:43
sdaguewhat?12:43
mordredsdague: I think that pyzmq made a new release, and I think that in places that we use it we're not pinned, and I think that in those places it can break us12:43
mordredsdague: http://logs.openstack.org/22523/3/check/gate-oslo-incubator-python27/1113/nose_results.html12:43
sdaguewell, at least it can only break oslo, right?12:44
mordreddunno - I have no idea what's actually going on here12:44
*** CaptTofu has joined #openstack-dev12:45
mordredother than pyzmq _did_ make a release yesterday, and now I've got a failing test case :(12:45
mordredI'm going to try to dig in further - but figured I should toss out a heads-up12:45
sdaguemordred: I'd suggest pinning in oslo, I think that's the only place it will splode12:45
mordredgreat. I've got a patch for that coming12:45
sdagueas far as I know nothing else exercises the pyzmq stuff12:46
sdagueewindisch would be the person to ask as well, I think they are basically the only ones that use zmq for real12:46
*** johnthetubaguy1 has left #openstack-dev12:47
*** DuncanT has joined #openstack-dev12:47
*** adjohn has joined #openstack-dev12:48
k4n0https://review.openstack.org/#/c/21415/ , please review this guys :)12:49
*** rods has joined #openstack-dev12:51
*** adjohn has quit IRC12:53
*** Guest86187 has quit IRC12:54
*** monst_ has joined #openstack-dev12:55
*** markvoelker has joined #openstack-dev12:56
*** yamahata has joined #openstack-dev12:59
openstackgerritA change was merged to openstack/nova: Fix launching libvirt instances with swap  https://review.openstack.org/2258613:02
*** yolanda has quit IRC13:03
*** davidha has quit IRC13:03
*** timello has joined #openstack-dev13:04
*** davidha has joined #openstack-dev13:04
*** aeperezt has joined #openstack-dev13:05
*** mattmonkeymagic has quit IRC13:07
*** monst_ has quit IRC13:08
*** yolanda has joined #openstack-dev13:09
*** unix has joined #openstack-dev13:12
sandywalshcan't get a venv going for tests ... anyone seen this: "No distributions at all found for oslo-config from http://tarballs.openstack.org/oslo-config/oslo-config-2013.1b4.tar.gz"13:18
*** adjohn has joined #openstack-dev13:19
*** adalbas has quit IRC13:21
*** danwent has joined #openstack-dev13:23
*** Tross has joined #openstack-dev13:24
*** sandywalsh has quit IRC13:24
*** adjohn has quit IRC13:24
mordredmarkmc: so, I think this: http://logs.openstack.org/22523/4/check/gate-oslo-incubator-python27/1118/nose_results.html13:28
mordredmarkmc: is uncovering an actual bug13:28
mordredmarkmc: but I don't understand the intent of the test13:28
mordredmarkmc: as best I can see - exception.wrap_exception should always only ever throw an Error - not the wrapped exception13:29
mordredI'm not really sure the previous use of the base Exception class was causing that test to pass - but I'm fairly certain it's a bad test13:30
mordredalso - I'd like to take this moment to register my EXTREME objection to catching one exception and then raising a different one13:30
mordredso, I think this chunk of code should diaf anyway - but that's a whole other kettle of fish to be sorted out after fixing the testing of the current code13:32
markmcmordred, well, the current code looks correct - i.e. the test should pass13:32
mordredmarkmc: really? can you explain why you think that?13:32
markmcmordred, but it does look like a big pile of bogusness more generally13:32
mordredmarkmc: as far as I can see - if your wrapped function raised an exception based on Error, it will re-raise13:33
markmcmordred, bad function throws exception, wrap exception catches and throws Error, Error is a subclass of Exception, we assertRaises(Exception)13:33
*** adalbas has joined #openstack-dev13:33
markmcmordred, actually, sorry13:34
*** monst_ has joined #openstack-dev13:34
mordredyah. k. that's why it passes - but I don't understand why it's testing that13:34
markmcmordred, bad function throws exception, wrap exception catches it and re-raises it because it's not an Error, we assertRaises(Exception)13:34
markmcmordred, it's testing its behaviour, whether the behaviour is useful ... :)13:35
mordredmarkmc: but no - wrap_exception doesn't re-raise - it raises a new Error13:35
markmcmordred, I'm all for just deleting the exception module from oslo13:35
mordredI think the test was intending to test that the function re-raises13:35
markmcmordred, only it caught an Error instance13:35
markmcmordred, you're right13:35
mordredbut because it used a test Exception that was also the base class of Error...13:35
markmcmordred, if it's not an Error, it raises an Error13:35
mordredyah. which is HORRIBLE, btw13:36
markmcagree13:36
mordredhow about I just delete this module :)13:36
markmclooks like wsgi is the only module using exceptions from it13:36
markmcmove those exceptions into wsgi.py and that's probably enough to delete it13:36
mordredconsider it done13:37
markmccheers13:37
*** anteaya has joined #openstack-dev13:37
*** sandywalsh has joined #openstack-dev13:37
*** Tross has quit IRC13:40
*** radez_g0n3 is now known as radez13:43
*** mlavalle has left #openstack-dev13:46
*** soody has joined #openstack-dev13:46
*** monst_ has quit IRC13:46
*** CaptTofu has quit IRC13:48
*** CaptTofu has joined #openstack-dev13:48
mordredmarkmc: https://review.openstack.org/22681 is a stab at it13:48
*** adjohn has joined #openstack-dev13:50
*** annegentle has joined #openstack-dev13:52
*** dprince has joined #openstack-dev13:52
mordredexcept that I suck.13:53
*** CaptTofu has quit IRC13:53
*** k4n0 has left #openstack-dev13:54
*** adjohn has quit IRC13:54
*** pasquier-s has joined #openstack-dev13:55
*** rkukura has joined #openstack-dev13:55
*** maoy has quit IRC13:56
*** kbringard has joined #openstack-dev13:58
*** sandywalsh has quit IRC14:00
*** almaisan-away is now known as al-maisan14:03
*** sandywalsh has joined #openstack-dev14:03
*** Tross has joined #openstack-dev14:04
*** henrynash has joined #openstack-dev14:04
*** martine has quit IRC14:06
*** ondergetekende has quit IRC14:08
*** martine has joined #openstack-dev14:09
*** monst_ has joined #openstack-dev14:12
dacharyjdurgin Do you know if https://wiki.openstack.org/wiki/BlockDeviceConfig has been discussed already ? During meetings / on lists. It looks like a change that will trigger a fair amount of discussion. It recommends adding a number of new fields. I'm very much in favor of giving more control to the user over the allocation of disks because it often frustrated me to not be able to fine tune disks allocation.14:12
*** FlorianOtel has quit IRC14:14
*** eharney has joined #openstack-dev14:15
*** al-maisan is now known as almaisan-away14:16
*** renier has joined #openstack-dev14:17
*** bknudson has quit IRC14:17
*** renier_ has quit IRC14:17
ewindischmordred: I'll look at the new pyzmq release & compat, but pinning sounds reasonable for now.14:19
mordredewindisch: ossum14:19
*** CaptTofu has joined #openstack-dev14:19
*** CaptTofu has quit IRC14:20
*** CaptTofu has joined #openstack-dev14:21
*** BLZbubba_ has joined #openstack-dev14:23
*** monst_ has quit IRC14:24
ewindischalso, this got a FFE but hasn't gotten any eyes: https://review.openstack.org/#/c/20434/14:24
*** CaptTofu has quit IRC14:25
*** henrynash has quit IRC14:27
*** henrynash has joined #openstack-dev14:29
*** maoy has joined #openstack-dev14:30
*** martine has quit IRC14:30
*** bknudson has joined #openstack-dev14:30
*** mindpixel has quit IRC14:31
*** davidkranz has joined #openstack-dev14:31
*** unix has quit IRC14:32
*** unix has joined #openstack-dev14:33
openstackgerritA change was merged to openstack/oslo-incubator: Fix "Needs to specify pyzmq version"  https://review.openstack.org/2267214:33
*** martine has joined #openstack-dev14:36
sandywalsham I reading this correctly, that a postgres client is required to run tests now?14:39
*** dolphm has joined #openstack-dev14:39
*** markmcclain has joined #openstack-dev14:41
*** CaptTofu has joined #openstack-dev14:41
*** woodspa has joined #openstack-dev14:45
*** armaan has joined #openstack-dev14:46
*** anniec has joined #openstack-dev14:46
*** READ10 has joined #openstack-dev14:47
ewindischsandywalsh: where are you reading that? I don't see postgres in pip-requires/test-requires14:49
danpbsandywalsh: yep, hit that the other day - needed to install postgresql-devel  to bootstrap14:50
*** jruzicka has quit IRC14:50
*** monst_ has joined #openstack-dev14:50
sandywalshthat's kinda shitty, no? Shouldn't it just pass the tests if it doesn't find the client?14:51
*** adjohn has joined #openstack-dev14:51
sandywalshand I'm seeing problems with getting from pypi lxml under python 2.6 ... works fine under 2.7 (when setting up a fresh venv)14:52
*** mtreinish has joined #openstack-dev14:52
sandywalshewindisch, it's the psycops (?) package that needs it ... Feb 4 merge14:52
danpbewindisch: tools/test-requires:psycopg214:52
*** anniec has quit IRC14:52
ewindischdanpb: I was looking in oslo-incubator… not there14:53
danpbcommit 5dcf4dd9b2e996913c8fe4bfa20a6186e9a066ab14:53
danpbAuthor: Kurt Taylor <krtaylor@us.ibm.com>14:53
danpbDate:   Thu Jan 17 15:15:59 2013 -050014:53
danpb    Added Postgres CI opportunistic test case14:53
danpbits in Nova14:53
sdaguesandywalsh: this remains the problem of no test optionals14:53
*** anniec has joined #openstack-dev14:53
*** jruzicka has joined #openstack-dev14:53
sandywalshsdague, how do the zeromq tests deal with it?14:53
sdaguewe can revisit again, but there is a desire that the installed set of software for ci is the same as for developers,14:54
ewindischsandywalsh: I do think that if postgresql is supported at all, it should be in the test-requires at a minimum14:54
sdaguesandywalsh: it might be a pure python driver14:54
ewindischsandywalsh: we now require installing zeromq and pyzmq14:54
ewindisch*libzmq14:54
sdagueewindisch: it is in test-requires14:54
sandywalshugh14:54
*** armaan has left #openstack-dev14:54
ewindischsandywalsh: although the latest pyzmq includes the source to libzmq and compiles it14:54
sandywalshewindisch, turning into "download the internet"14:55
sdaguesandywalsh: have you built a venv from scratch recently for nova? :)14:55
sdaguewe're about 1/2 way there already14:55
sandywalshsdague, that's what I'm trying to do now14:55
sdagueyou get really good at compiling numpy14:55
ewindischsandywalsh: no different than having to install mysqlclient.14:55
*** adjohn has quit IRC14:55
sandywalshsdague, can't get it to work under 2.6 though14:56
sdaguesandywalsh: really?14:56
sandywalshewindisch, yeah, I guess. Sort of depends on "what is the core deploy?"14:56
*** garyTh has joined #openstack-dev14:56
ewindischsandywalsh: well, if pyzmq wasn't in test-requires for oslo, that could would break. It *has* broken from not being tested.14:57
sdaguesandywalsh: so there is probably a good conversation for summit about test-optional as a 3rd list, now that we have more of these tools14:57
ewindischI assume that is a lesson that can be applied elsewhere to code that doesn't get tested.14:57
ewindischsdague: and what is the value? that people will only realize they broke something after it gets to Jenkins?14:58
sandywalshsdague, +114:58
sdagueewindisch: not if jenkins always installs it14:58
sandywalshewindisch, I guess the question becomes, when we add code are we responsible for ensuring every optional component still works14:58
ewindischsdague: I meant, people won't run some tests locally, then they'll hit the wall on jenkins, reevaluate, and run the wider local tests?14:59
sandywalshewindisch, for example, I add something to xen, do I need to ensure it works under libvirt even though I don't use it.14:59
sdagueewindisch: yeh14:59
sdaguesandywalsh: you need to ensure you don't break libvirt14:59
ewindischsdague: which might not sound too horrible, but  they'll have not had those libraries installed, so they'll now spend time setting up their environment, pissed off at those optional components14:59
sandywalshsdague, bad example, I'm thinking adding functionality, not changing it. :)15:00
sdaguesandywalsh: right, but that's why pg is in the requires list now15:00
sdagueI spent 2 months chasing pg regressions :)15:00
sdaguenow it's in ci, so you can't slide code in that breaks what we've got tested15:00
*** mrodden has joined #openstack-dev15:01
*** romcheg has joined #openstack-dev15:01
sandywalshsdague, I can see the argument, but I can also see the logical conclusion of all this.15:01
*** diogogmt has joined #openstack-dev15:01
danpbsandywalsh: yes if you add code whether a feature or bug, you are responsible for not breaking other drivers15:01
*** cloudchimp has joined #openstack-dev15:02
danpbi'm not saying you need to implement the feature for all drivers, just make sure you don't cause breakage in any existing work15:02
sandywalshdanpb, I guess it brings up the question of more formal interfaces15:02
sdaguesandywalsh: I'd personally be ok with test-optional (given that at least in the db case we are doing skips if the db isn't setup right), and then catch it in check15:02
sdagueand force devs to pull test-optional then to make it past jenkins check15:03
*** monst_ has quit IRC15:03
sdaguegiven that about 80% of them probably won't ever need to, but I can also see the argument that everyone should be on the same dev env15:03
*** romcheg1 has quit IRC15:03
ewindischand if you modify those other drivers attempting to implement it, it should either work, or not break it. Tests should run and catch it. Russell made good *looking* changes to the zeromq driver, but didn't actually run the tests, and the code was broken for a week. Then, for 2 months, the new feature still didn't even *work* until I wrote new tests that caught it.15:03
sandywalshdanpb, I'm cool with that. A stub that says "not implemented" ... but otherwise we'll get to the point we have to be experts in every unique snowflake in the system.15:04
ewindischthat was before the tests were gating in Jenkins, of course.15:04
danpbsdague: IMHO as long as the stuff the dev env is depending on is all opensource, then it isn't unreasonably onerous to have devs install the complete set of deps15:04
*** jimfehlig has joined #openstack-dev15:04
sdaguedanpb: yep, that's fair. it's status quo15:05
sandywalshI'd much rather see test jigs on the core interfaces and the drivers have to be compatible with that ... but nova core shouldn't need to worry about the details.15:05
ewindischand the fact the projects have broken up means that the dependencies are slightly better handled.15:05
danpbsdague: since it is really just a few more KB download / few more seconds build time when creating the .venv - which is counterbalanced by better quality for ethe community as a whole15:05
sdaguesandywalsh: I think we'd all like to be there eventually, it's just not clear that we're really ready to nail down those interfaces15:05
ewindischyou only need postgres when testing Nova, or zeromq when testing oslo-incubator.15:05
sandywalsh*until it affects a core interface ... then it's "feats of strength"15:05
danpbsandywalsh: raise NotImplementedError is basically what is already done with compute drivers15:06
*** romcheg has quit IRC15:06
*** mikal has quit IRC15:06
*** ron-slc_ has joined #openstack-dev15:07
sandywalshdanpb, theoretically yes, but should any driver be able to make it into core or should it be an optional install. Just like an incubated project15:08
*** sthaha has quit IRC15:08
*** dolphm has quit IRC15:08
*** mikal has joined #openstack-dev15:08
sdaguesandywalsh: we are a long way from that, look at how much change has happened just in this cycle15:08
sdagueI'm all for getting us there15:08
sandywalshsdague, good summit discussion I think.15:09
sdaguesandywalsh: absolutely15:09
*** navid has left #openstack-dev15:09
danpbsandywalsh: well for virt drivers, we've basically got all the interesting ones now anyway :-)15:09
sdagueespecially once you freeze an API, you also need to be able to version it.15:09
danpbi think we're a very long way from being able to freeze the internal driver APIs15:10
*** digitalsanctum has joined #openstack-dev15:10
sandywalshdanpb, true ... and the reality would be that I would be affected by that change in the negative (we're Xen and KVM is the thousand pound gorilla), so we'd be an optional component.15:10
sandywalshbut, if we're going through the effort on Oslo to share code, this is the next logical step.15:11
danpbsandywalsh: in libvirt we've basically accepted support for any hypervisor driver where someone has been prepared to serve as the maintainer15:11
sandywalshdanpb, until they don't15:12
danpbsandywalsh: libvirt devs doing internal refactoring are required to make sure every driver in tree continues to build15:12
sdaguedanpb: I agree, we're a long way on the virt api.15:12
danpbover the long term it has worked out fine and we've not had to consider freezing our internal APIs at all15:12
sandywalshdanpb, interesting ... certainly the easiest solution for the maintainers. Not so much for the developers. I need to stew on that a little.15:13
sandywalsh*general developers15:13
danpbwe prioritize the good of the project as a whole over an easy life for individual developrs15:14
*** terryh has joined #openstack-dev15:14
sandywalshyep, I can see the rationale. It's a interesting set of tradeoffs.15:14
sandywalshthe thing that sparked it for me was seeing I had to install postgres client  and thinking "great, but cruft on my dev laptop (since it's outside of the venv)"15:15
sandywalshand realizing it's only going to get worse15:15
sandywalshs/but/more/15:16
sandywalshhaha, funny slip15:16
sdague:)15:16
ewindischThere is something psychologically broken about test-optional. Instead of making everyone install all the dependencies, it becomes something that one needs to do to fix code they've already relegated as "optional" or unnecessary. I don't know if there is a term for it, but it will likely harbor resentment.15:16
ewindischinstead, people like Sandy can just bemoan the same pains as everyone else.15:17
*** wwallnrr__ has joined #openstack-dev15:18
*** sacharya has joined #openstack-dev15:18
sdagueewindisch: yeh. Being the brunt of the complaints for having introduced real db dependencies, I'm just sensitive to people's pain15:18
sdagueeven if most of them come from people developing on macs... :)15:18
*** rpedde_away is now known as rpedde15:20
sandywalshewindisch, good point. It's a question of letting the interfaces grow organically or making them become a gating/community vote thing early on.15:20
sandywalshand, where process kills, organic wins15:20
ewindischsdague: I think you and I are in the same boat. I'm surprised there haven't been more complaints about zeromq being required to test in oslo, but I think that can be attributed to few people actually doing work there. Also, it helped that I did warn the mailing list.15:21
sdaguesandywalsh: and realize, every time we've added those external dependencies it was because real bugs were found, and they were part of keeping them from regressing15:21
ewindischsandy: just wait until I add gpgme...15:22
sandywalshheh ... I'd even been happy with a distro package of the non-python dev requirements. Something I can just freshen up, rather than having to chase down pip failures15:23
*** lucasagomes has joined #openstack-dev15:23
sdaguesandywalsh: yeh, that's a good idea15:23
sandywalshsdague, might be the first step?15:24
*** wwallnrr__ has quit IRC15:24
sdaguesandywalsh: yeh, sure, making install_venv a little smarter15:24
*** topol has joined #openstack-dev15:25
*** wwallnrr__ has joined #openstack-dev15:25
*** davidha has quit IRC15:26
ewindischbtw, I have a draft patch for recursive dependency resolution of oslo requirements, if anyone wants a gander.15:26
sdaguesandywalsh: design / hack session on that would be reasonable in portland15:27
*** boris-42 has quit IRC15:27
*** cp16net is now known as cp16net|away15:27
ewindischanother thing is we could do better to link dependencies to the code that actually depends on them.15:27
*** sacharya has quit IRC15:27
BLZbubbaargh, is there no longer a way to specify the nova-api ports on the command line?15:27
sandywalshI was working on a tool called Signature, for checking method signatures between drivers/plugin modules https://github.com/sandywalsh/signature15:28
sandywalshmight be a handy set of tests too15:29
*** monst_ has joined #openstack-dev15:29
ayounghenrynash, http://logs.openstack.org/20289/20/check/gate-keystone-python26/2693/console.html.gz15:29
BLZbubbaif not, what is the more up-to-date way to do something like this: http://www.cybera.ca/tech-radar/putting-openstack-production-part-3-load-balancing-and-https15:29
henrynashayoung: looking at it15:31
*** sc68cal has joined #openstack-dev15:31
ayounghenrynash, same problem...policy is getting messed up.  My suggestion is to use the generated link on the page to pull the code as a branch.15:32
henrynashayoung: on your latest patch, you mean?15:32
ayounghenrynash, yeah15:32
ayounghttps://review.openstack.org/#/c/20289/15:33
henrynashayoung: will do15:33
*** danwent has quit IRC15:33
ayoungif you look at the section just above the diff, where it is labled Download, there are a range of options.  THe oe I found most useful is 'checkout'15:33
*** yidclare has quit IRC15:36
*** esheffield has joined #openstack-dev15:36
*** maoy has quit IRC15:36
*** mindpixel has joined #openstack-dev15:37
BLZbubbaugh do i really have to have a one line config file for each nova-api process instead of being able to specify --osapi_listen_port ?15:37
BLZbubbai hope you added those port options back in folsom15:37
*** arbrandes has quit IRC15:39
BLZbubbaor else have it fork off each request15:39
*** danwent has joined #openstack-dev15:39
*** danwent has quit IRC15:41
mordredmarkmc: got a sec to join in #openstack-infra?15:41
*** monst_ has quit IRC15:41
*** mmagr has quit IRC15:43
*** zoresvit has quit IRC15:44
henrynashayoung: ok, have it re-produced inside a single test file (you v3_trusts)…by including a fake test class ahead of yours that rewrite the policy file…so will debug now15:45
*** alszar has joined #openstack-dev15:46
*** markwash has quit IRC15:46
*** alunduil has joined #openstack-dev15:47
*** dolphm has joined #openstack-dev15:47
*** psedlak has quit IRC15:48
*** alunduil has quit IRC15:48
*** rnirmal has joined #openstack-dev15:49
*** alunduil has joined #openstack-dev15:49
*** radez is now known as radez_g0n315:51
*** soody has quit IRC15:51
*** adjohn has joined #openstack-dev15:52
*** soody has joined #openstack-dev15:53
*** gyee has joined #openstack-dev15:53
*** radez_g0n3 is now known as radez15:54
*** ecarlin has joined #openstack-dev15:55
*** adjohn has quit IRC15:57
*** jgallard has quit IRC15:57
*** datsun180b has joined #openstack-dev15:57
*** diogogmt has quit IRC15:59
*** sacharya has joined #openstack-dev15:59
*** jaypipes has quit IRC15:59
henrynashayoung: ok, got the fix.15:59
*** ecarlin has quit IRC16:00
henrynashayoung: it's because self.opt(policy_file=self.tmpfilename) changes the (effective_ CONF.policy file name16:00
*** diogogmt has joined #openstack-dev16:00
*** monst_ has joined #openstack-dev16:01
henrynashayoung: so teardown has to put it back.  Will post a fix shortly16:01
*** ecarlin has joined #openstack-dev16:01
*** rnirmal has quit IRC16:01
*** zoresvit has joined #openstack-dev16:02
*** gary_th has joined #openstack-dev16:03
*** cp16net|away is now known as cp16net16:04
ayounghenrynash, so, can you do it in setUp as well.16:04
ayoungProbably best for each test to be paranoid16:04
ayoungand ot assume anything about the state of the system coming in16:04
*** jgallard has joined #openstack-dev16:04
ayoungand 'not assume'16:04
henrynashjust seeing what self.opt() does…16:05
*** garyTh has quit IRC16:05
*** gongysh has quit IRC16:05
*** monst_ has quit IRC16:05
henrynashayoung: since CONF.Policyfile jas been changed by it…so unless you reload the the conf settings (or save it off and put it back in your teardown, you won;t have it right)16:06
ayounghenrynash, we should reload it16:06
henrynashayoung: yes, I think so16:06
ayounghenrynash, In setup we specify the conf file to lead16:07
ayoungload16:07
*** adalbas has quit IRC16:07
henrynashayoung: fyi, here's how I debugged it…added this fake test class ahead of yours in test_v3_trusts…as it is it now works, take out the reseting of policy_file in teardown and it fails.  http://fpaste.org/yAAX/16:09
*** zbitter is now known as zaneb16:09
*** CaptTofu has quit IRC16:09
*** CaptTofu has joined #openstack-dev16:10
*** CaptTofu has quit IRC16:10
*** CaptTofu has joined #openstack-dev16:11
*** CaptTofu has quit IRC16:11
*** CaptTofu has joined #openstack-dev16:11
*** CaptTofu has quit IRC16:11
*** CaptTofu has joined #openstack-dev16:12
*** monst_ has joined #openstack-dev16:15
*** esp1 has joined #openstack-dev16:15
*** monst_ has quit IRC16:15
*** jrodom has joined #openstack-dev16:15
topolYorkikSar, are you around?16:16
henrynashayoung: so maybe I'll post a patch that does the correct cleanup (since not good to leave things in a bad way)…do you want add the loading part?16:16
henrynash(in setup)16:16
*** monst_ has joined #openstack-dev16:16
*** nunosantos has joined #openstack-dev16:16
*** radez is now known as radez_g0n316:16
topolYorikSar, sorry for the wrong spelling. You there?16:16
BLZbubbaanyone here know anything about nova-api?16:17
*** ecarlin has quit IRC16:17
*** pabelanger has joined #openstack-dev16:17
*** jbr_ has quit IRC16:18
*** radez_g0n3 is now known as radez16:19
*** jaypipes has joined #openstack-dev16:19
BLZbubbaugh is nova-api configurable at all?  it is ignoring the port options in nova.conf and the undocumented flagfile option seems useless too16:20
*** flaper87 has joined #openstack-dev16:21
*** thingee_zzz is now known as thingee16:21
*** thingee has quit IRC16:22
*** adjohn has joined #openstack-dev16:23
*** maroh has quit IRC16:23
*** aloga has quit IRC16:23
*** adalbas has joined #openstack-dev16:23
*** yamahata has quit IRC16:24
*** jcmartin has quit IRC16:26
*** doude has left #openstack-dev16:26
ayounghenrynash, so my bad for that failing.  I removed the CONF.reset() prior to pushing, and of course, that was also required16:26
henrynashayoung: np…double whammy :-)16:27
*** adjohn has quit IRC16:27
henrynashayoung: so about to post a patch which fixes the tear downs of test_policy and test_v3_protection16:27
*** alunduil has quit IRC16:28
*** devoid has joined #openstack-dev16:29
*** thingee has joined #openstack-dev16:29
*** ecarlin has joined #openstack-dev16:30
*** ecarlin has quit IRC16:30
*** dansmith is now known as SteelyDan16:30
*** koolhead17 has quit IRC16:30
*** imsplitbit has joined #openstack-dev16:32
*** esp1 has quit IRC16:32
henrynashayoung: https://review.openstack.org/#/c/22705/16:34
ayounghenrynash, the change you make in v3 protection should be in the base class.  I think it should be  moved  to test_v316:35
*** networkstatic has joined #openstack-dev16:35
ayounghenrynash, but, yes, that looks about right16:36
henrynash…but not all v3 child test classes re-write policy files…but I guess it wouldn't harm16:36
*** monst_ has quit IRC16:36
ayounghenrynash, one problem is that you can't tell just from that if it is working....I've incorporated the changed into my own patch for now and can drive on.16:36
henrynashayoung: yes, agreed16:37
ayounghenrynash, so, yeah, each test should clean up after itself, but each test class should have a certain blank slate to work with.  Its like the live DB tests.  If you haven't cleaned up the db before run the test, they fail16:38
*** adjohn has joined #openstack-dev16:38
ayoungand that is a pain16:38
*** Tross has quit IRC16:39
*** anniec has quit IRC16:41
gyeeayoung, trust API goes to RC1 then?16:42
*** thouveng has quit IRC16:42
gyeethere was an announcement about G3 availability16:42
ayounggyee, yeah16:44
*** CaptTofu has quit IRC16:44
*** opilotte has quit IRC16:45
ayounggyee, s'alright.  Nothing depends on it yet.  I just want it there so people can build on it in the future.16:45
*** pnavarro has quit IRC16:46
gyeeayoung, k, I was confuse about the announcement and FFE16:46
ayounggyee, are you guys waiting on trusts?16:47
*** opilotte has joined #openstack-dev16:48
gyeeayoung, yeah, we have similar use case as HEAT16:49
ayounggyee, we'll have it soon.  BTW,  Can you take a look at the patch, specifically at the token/controller.py changes, and figure out how that should go into v3 auth?16:50
*** jbryce has joined #openstack-dev16:53
*** soody has quit IRC16:53
gyeeayoung, sure16:54
*** soody has joined #openstack-dev16:55
*** rnirmal has joined #openstack-dev16:56
*** edmund has joined #openstack-dev16:57
sandywalshstill getting test failures around test_xml_and_uri_rescue_* ... anyone run into this before?16:58
sandywalsh(nova trunk)16:59
*** nati_ueno has joined #openstack-dev16:59
*** nati_ueno has quit IRC17:02
*** monst_ has joined #openstack-dev17:03
*** monst_ has joined #openstack-dev17:04
*** Gordonz has joined #openstack-dev17:04
*** imsplitbit has quit IRC17:05
*** henrynash has quit IRC17:05
*** danwent has joined #openstack-dev17:05
*** nsatterl_ has quit IRC17:06
ewindischFFE review ping: https://review.openstack.org/#/c/20434/17:06
*** martine has quit IRC17:08
*** soody has quit IRC17:09
*** CaptTofu has joined #openstack-dev17:09
*** mrodden has quit IRC17:10
*** imsplitbit has joined #openstack-dev17:10
*** dolphm has quit IRC17:11
jdurgindachary: danpb created most of that page after some discussion on the list (http://lists.openstack.org/pipermail/openstack-dev/2013-January/thread.html#4842)17:11
vishygaryk: I need to chat more with you about the dhcp issue, but I'm on a bad connection right now. Will you be around in about an hour?17:11
openstackgerritA change was merged to openstack/python-openstackclient: Correct the version mapping to image service.  https://review.openstack.org/2222717:12
*** rafaduran has left #openstack-dev17:12
*** hemna has quit IRC17:14
*** spzala has joined #openstack-dev17:15
*** dolphm has joined #openstack-dev17:15
*** soody has joined #openstack-dev17:15
*** jgallard has quit IRC17:16
*** pnavarro has joined #openstack-dev17:17
*** jpich has quit IRC17:17
*** troytoman-away is now known as troytoman17:17
*** dhellmann has joined #openstack-dev17:18
*** pnavarro has quit IRC17:18
*** esp1 has joined #openstack-dev17:18
danpbjdurgin think i'm missing some conversation here...  most of what page ?17:19
*** Gordonz has quit IRC17:20
*** zaitcev has joined #openstack-dev17:21
*** gael has quit IRC17:22
*** flaper87 has quit IRC17:23
arosengaryk:  yup here17:24
*** cp16net is now known as cp16net|away17:25
*** mrodden has joined #openstack-dev17:26
*** amerine has joined #openstack-dev17:26
gyeeayoung, endpoint scoping is not supported yet right?17:27
*** cp16net|away is now known as cp16net17:27
*** Gordonz has joined #openstack-dev17:28
*** jruzicka has quit IRC17:28
*** lglenden has joined #openstack-dev17:28
*** amerine has quit IRC17:28
*** mindpixel has quit IRC17:29
*** TheEstablishment has joined #openstack-dev17:29
*** amerine has joined #openstack-dev17:29
nikhila doc question for nova: if i write an extension and there is changes in the /servers and /servers/detail response, does that sample response need to exist in doc/api_samples/<extension_name>/sample response17:29
*** roampune has joined #openstack-dev17:31
*** henrynash has joined #openstack-dev17:31
*** ndipanov has quit IRC17:33
dacharydanpb: jdurgin answered my question about the origins of https://wiki.openstack.org/wiki/BlockDeviceConfig :-) I updated https://blueprints.launchpad.net/nova/+spec/flexible-block-device-config with a pointer to http://lists.openstack.org/pipermail/openstack-dev/2013-January/thread.html#484217:33
*** jrodom has quit IRC17:33
*** almaisan-away is now known as al-maisan17:33
*** unix has quit IRC17:34
*** pasquier-s has quit IRC17:34
ayounggyee, that is correct.  THe trust supports it, but the tokens do not.17:35
ayounggyee, does v3 auth support endpoint scoping>17:35
gyeeayoung, not right now17:35
danpbdachary: ah, i just updated the wiki page to link back to the blueprint to avoid future confusion17:35
gyeejust want to know what's the expectation17:35
*** arbrandes has joined #openstack-dev17:36
*** markmc has quit IRC17:37
nikhilalaski: ^^17:38
*** TheEstablishment has quit IRC17:39
ayounggyee, when it comes down to it, it really is up to policy or auth_token middleware to support endpoint scoping.17:39
*** dachary has quit IRC17:39
*** jrodom has joined #openstack-dev17:39
gyeeayoung, what I mean is when we issue a trust token, do we filter the endpoints returned?17:40
*** arbrandes has quit IRC17:40
ayounggyee, ah, no17:41
gyeewhy not?17:41
*** Mandell has joined #openstack-dev17:41
ayounggyee, but...we can17:41
gyeewe should right?17:41
ayounggyee, really?  because it wasn't enforced yet.  Lazyiness, really17:41
*** njoy has joined #openstack-dev17:41
ayoungyeah,  I'll  add that in.17:41
gyeeif we filter them, that would make them effective17:42
*** john5223 has joined #openstack-dev17:42
*** AnilV4 has quit IRC17:43
*** njoy_ has joined #openstack-dev17:44
ayounggyee, we don't filter the service catalog based on user yet, so I can't confirm that the trustor has the endpoint in their catalog17:44
gyeeayoung, I have a blueprint to establish the relationship between project and endpoints17:45
ayounggyee, good stuff, but I don't think that will make it into rc117:45
gyeecouple that with trust, that would make endpoint scoping effective17:45
zykes-win 4017:45
ayounggyee, still needs enforcement17:45
ayoungotherwise, token gets passed to service and it gets through regardless of what is in it17:46
gyeeenforcement is in the client17:46
ayoungenforcement needs to be in auth_token middleware.  Or in policy17:46
gyeeno endpoint in the catalog, user have no access17:46
zykes-ayoung: will trust stuff get into g ?17:46
ayoungzykes-, yes17:47
*** njoy has quit IRC17:47
ayounggyee, so I need to wrap trust.create's input document in a tope level trust value?17:47
ayoungbecaus of XML?17:47
gyeeayoung, sadly yes17:48
ayounggyee, OK.  I think that is fine.  dumb, but fine17:48
gyeewith XML, only one root element is allowed17:48
*** Ryan_Lane has quit IRC17:48
zykes-ayoung: are there services using it as well ?17:48
ayoungzykes-, there will be soon, but not yet17:48
*** Ryan_Lane has joined #openstack-dev17:48
*** mrodden has quit IRC17:49
*** FatDarrel has joined #openstack-dev17:49
gyeeayoung, I was kind of joking about endpoint enforcement in the client17:50
*** devananda has quit IRC17:50
gyeesecurity 101, never trust the client :)17:50
*** danpb has quit IRC17:50
ayounggyee, I need to cut scope somewhere.  No endpoint enforcement in G17:50
*** devananda has joined #openstack-dev17:51
gyeeayoung, that's fine, lets do this in H17:51
*** trapnii has quit IRC17:54
*** matelakat has quit IRC17:54
*** CaptTofu has quit IRC17:56
ayounggyee, actually, I thine the API doc was wrong.  I thin I already had the top level trust groupoing in there.  The controller's require it17:56
*** CaptTofu has joined #openstack-dev17:56
ayounghttps://review.openstack.org/#/c/22715/1/tests/test_v3_trust.py17:56
ayoungline 19217:56
*** edmund has quit IRC17:57
*** tomoe_ has quit IRC17:59
*** davidh_ has joined #openstack-dev17:59
*** russellb is now known as rustlebee18:00
*** CaptTofu has quit IRC18:01
gyeeayoung, k, I'll wait for your next patch then18:01
*** lucasagomes has quit IRC18:02
*** yidclare has joined #openstack-dev18:02
ayounggyee, please don't wait18:02
*** derekh has quit IRC18:02
*** Tross has joined #openstack-dev18:03
ayoungI'm working on the endpoint and roles for dolphm 's suggestions, which is that they get returned as objects18:03
zykes-ayoung: as in G or ?18:03
ayoungbut the API doc is wrong, and I'm fixing that as well, to match {trust:{....18:03
alaskinikhil: yes.  An extensions samples should be in doc/api_samples/<extension_name>/<samples>, which in your case would be a superset of the servers response.  And the docs should be generated by templates in nova/tests/integrated/api_samples/<extension>/18:03
gyeeayoung, sure, I'll take a look at your code18:03
ayounggyee, just give me a plan for the v3 auth and tokens18:03
*** alunduil has joined #openstack-dev18:04
ayounggyee, ah, you meant my next patch on API...yes, I'll ping you when that is done, too.18:04
ayoungthanks18:04
gyeeayoung, we'll fix all the bugs in rc118:04
gyeefor the token inconsistent formats18:04
sdaguerustlebee: as markmc isn't around, what's the oslo stance on freeze? are we allowed to start factoring in modules that won't migrate out to projects until havana?18:04
openstackgerritA change was merged to openstack-dev/devstack: Added cinder-backup service (not enabled by default)  https://review.openstack.org/2253318:04
openstackgerritA change was merged to openstack-dev/devstack: set dhcp_agent_manager for ovs plugin  https://review.openstack.org/2269318:04
*** melwitt has joined #openstack-dev18:05
rustlebeesdague: same policy, oslo-incubator will have a stable/grizzly branch, so shouldn't be merging features without an exception right now18:06
*** mrodden has joined #openstack-dev18:06
sdaguerustlebee: ok18:06
*** CaptTofu has joined #openstack-dev18:06
*** markwash has joined #openstack-dev18:07
*** dontalton has joined #openstack-dev18:08
*** zing has quit IRC18:09
*** darraghb has quit IRC18:14
*** olaph has quit IRC18:14
*** dachary has joined #openstack-dev18:15
vishyjgriffith: ping18:16
*** dprince has quit IRC18:16
jgriffithvishy: pong18:18
*** zoresvit has quit IRC18:18
ayoungdolphm, what attributes do we want in roles and endpoint for the trust?  Is it OK if I go with a minimal set at first (role id,nam)  endpoint(id) ?18:19
ayoungwe can do links etc later?18:19
dolphmayoung: minimal is ideal18:19
ayoungdolphm, OK,  on list, I am thinking of removing most values18:20
ayoungI'll leave trustro and trustee and trust id18:20
dolphmayoung: wait, on GET /trusts?18:20
vishyjgriffith: so we have an odd test run where c-vol failed to start and doesn't log at all18:20
vishyi was wondering if you have ever seen that18:20
*** olaph has joined #openstack-dev18:20
ayoungdolphm, yeah.  You get all the values on GET trusts/(id)18:20
jgriffithvishy: yes18:20
ayounglist does one DB lookup18:20
dolphmayoung: i'm asking about GET /trusts (the collection)18:20
*** trapnii has joined #openstack-dev18:20
jgriffithvishy: It's not the same one we looked at last week?18:20
ayoungyeah,18:20
jgriffithvishy: the one we thought was screen issue?18:20
ayoungdolphm, on GET trusts I only want to do one DB lookup18:21
dolphmayoung: all the other collections return full resources18:21
ayoungdolphm, OK...I'll kep it18:21
*** andrewbogott_afk is now known as andrewbogott18:21
ayoungkeep18:21
dolphmayoung: wait, i see what you're saying18:21
*** devananda has quit IRC18:22
dolphmayoung: what attributes do you want to include? trustor_user_id and trustee_user_id?18:22
ayoungdolphm, yeah, and the trust id18:22
*** andrewbogott is now known as andrewbogott_afk18:22
*** andrewbogott_afk is now known as andrewbogott18:22
dolphmayoung: how about the impersonation flag?18:22
*** andrewbogott is now known as andrewbogott_afk18:22
ayoungdolphm, not on lit, I think18:22
ayounglist18:23
ayoungdolphm, but if you insist, sure18:23
*** andrewbogott_afk is now known as andrewbogott18:23
dolphmayoung: hrm, so ... how about splitting up the API a bit ...18:23
*** alunduil has quit IRC18:23
dolphmayoung: don't know why i didn't htink of this before, but do:18:23
dolphmayoung: /trusts/{trust_id}/roles and /trusts/{trust_id}/endpoints18:23
*** devananda has joined #openstack-dev18:23
dolphmayoung: err, but you want immutable...18:24
ayoungdolphm, that works immutable18:24
*** sandywalsh_ has joined #openstack-dev18:24
*** sandywalsh has quit IRC18:24
ayoungyou add them all on create.  THat api would be just query, but I don;t think it buys muh18:24
ayoungmuh18:24
ayoungmuch18:24
ayoungdolphm, so limiting values on list is OK?18:25
dolphmayoung: you'll have to implement GET /trusts/{trust_id}/roles (return the collection of full role resources), HEAD /trusts/{trust_id}/roles/{role_id} (check if role is included in trust), and GET /trusts/{trust_id}/roles/{role_id}18:25
dolphmayoung: /trusts/{trust_id}/projects too18:25
dolphmayoung: don't even include them on the list18:25
ayoungdolphm, no need for those up front, but they would be OK as RFEs18:25
*** trapniii has joined #openstack-dev18:25
jgriffithvishy: If it's not the same one then the only thing I can think of is to doulbe check the changes we made a few days ago18:25
ayoungdolphm, how about18:25
ayoungeach list gives you trustor, truste and a link to the GET18:26
dolphmayoung: you can keep POST /trusts basically the same, but then make it so that to get those collections back, you have to ask for them specifically18:26
ayoungthen follow the link if you want full details18:26
jgriffithvishy: service --> launcher18:26
sandywalsh_anyone else getting trunk libvirt build failures? http://paste.openstack.org/show/32353/18:26
dolphmayoung: trusts should include a "links" object that -- let me make a paste18:26
vishyjgriffith: it is a random failure18:27
ayoungdolphm, yeah,  the self link18:27
vishyjgriffith: i thought these kind of failures were due to screen_it using stuff but it is still happening18:27
*** trapnii has quit IRC18:27
jgriffithvishy: K, that's what I was wondering18:28
jgriffithvishy: recent gate logs?18:28
*** maoy has joined #openstack-dev18:28
jgriffithrealizing there's nothing there :(18:28
dolphmayoung: should be able to include stuff like this http://paste.openstack.org/raw/32357/18:29
*** PaulCarlton has quit IRC18:29
vishyjgriffith: http://logs.openstack.org/22590/4/gate/gate-tempest-devstack-vm-quantum/9667/logs/18:29
*** CaptTofu has quit IRC18:29
ayoungdolphm, how about just self for now?18:30
dolphmayoung: sure, and you'll get "self" for free by using the v3 controllers with the wrap_collection / wrap_member class methods18:31
ayoungdolphm, schweet!18:31
*** CaptTofu has joined #openstack-dev18:31
*** Ryan_Lane has quit IRC18:32
dolphmayoung: make sure you set collection_name = 'trusts' and member_name = 'trust' e.g. https://github.com/openstack/keystone/blob/master/keystone/identity/controllers.py#L65218:32
dolphmayoung: and then call TrustV3.wrap_member(context, ref) and TrustV3.wrap_collection(context, refs) as appropriate, e.g. https://github.com/openstack/keystone/blob/master/keystone/identity/controllers.py#L66418:33
*** trapniii has quit IRC18:34
*** nati_ueno has joined #openstack-dev18:35
openstackgerritA change was merged to openstack/nova: Migration 144: Fix drop index statement  https://review.openstack.org/2262818:35
*** nati_ueno has quit IRC18:36
jgriffithvishy: Working on a local setup to try something18:37
jgriffithvishy: I seem to recall hitting this once a while back when I enabled swift on my devstack locally18:38
jgriffithvishy: Maybe I can hit it again and get some more info18:38
vishyjgriffith: you think it could be a bad interaction with swift?18:38
jgriffithvishy: That's a super long shot... but it's interesting I've spun up devstack so many times lately to do reviews etc18:39
jgriffithnever hit it18:39
notmynamejgriffith: vishy: is something I need to look in to?18:39
ayoungdolphm, adding calls to wrap_member have astrange side effect: Conflict: Conflict occurred attempting to store tenant. Duplicate ID, bar.18:39
vishyjgriffith: infra is doing a search to see if the failure happens to other workers18:39
jgriffithnotmyname: don't think so, just trying to figure out how to repro something18:39
vishynotmyname: not yet, we are grasping at straws :)18:40
*** gary_th has quit IRC18:40
jgriffithvishy: cool18:40
ayoungthe might befrom wrap_collection actually18:40
notmynameok, let me know if something comes up18:40
jgriffithvishy: The other idea is if they can setup a trap on it18:40
jgriffithvishy: Then I can log on and try to trace through the startup maybe18:40
jgriffithI'll ask them..18:40
ayoungdolphm, maybe I misunderstood what it was doing, let me try something a little different18:41
dolphmayoung: you should call it on the return18:41
vishygaryk: back yet?18:42
*** CaptTofu has quit IRC18:43
*** CaptTofu has joined #openstack-dev18:43
openstackgerritA change was merged to openstack/nova: BM Migrations 2 & 3: Fix drop_column statements  https://review.openstack.org/2263018:44
*** jcmartin has joined #openstack-dev18:44
dolphmgyee: revised https://review.openstack.org/#/c/22696/18:45
openstackgerritA change was merged to openstack/nova: Fix migration snake-walk.  https://review.openstack.org/2263118:45
*** dprince has joined #openstack-dev18:46
ayoungdolphm, Nah, I got it.  Code error.18:47
openstackgerritA change was merged to openstack/nova: Add `post_downgrade` hook for migration tests.  https://review.openstack.org/2263318:48
*** nati_ueno has joined #openstack-dev18:49
openstackgerritA change was merged to openstack/nova: Migration 146: Execute delete call.  https://review.openstack.org/2263418:50
*** kagan has joined #openstack-dev18:50
*** dachary has quit IRC18:51
*** trapniii has joined #openstack-dev18:51
*** pixelbeat has quit IRC18:53
*** koolhead17 has joined #openstack-dev18:56
gyeedolphm, https://review.openstack.org/#/c/22696/ looks good18:57
*** cp16net is now known as cp16net|away18:57
gyeenow I need to adjust the code18:57
dolphmayoung: your turn https://review.openstack.org/#/c/22696/18:58
dolphmhenrynash: ^18:58
*** alunduil has joined #openstack-dev18:59
dolphmayoung: henrynash: see the referenced bug18:59
*** cp16net|away is now known as cp16net18:59
ayoungdolphm, DOes the API doc change  have a deadline?19:00
*** soody has quit IRC19:00
henrynashdolphm: looking at it19:00
*** capnkooc has joined #openstack-dev19:01
*** troytoman is now known as troytoman-away19:02
*** beagles has quit IRC19:03
*** al-maisan is now known as almaisan-away19:04
*** beagles has joined #openstack-dev19:07
*** networkstatic has quit IRC19:07
henrynashdolphm, young: I'm good with https://review.openstack.org/#/c/22696/ - but Adam, do you want to have a look before we push the buton19:08
henrynashdolphm, ayoung: I'm good with https://review.openstack.org/#/c/22696/ - but Adam, do you want to have a look before we push the buton19:08
ayounghenrynash, I'll take a look.19:09
*** afazekas has joined #openstack-dev19:10
*** riskable has joined #openstack-dev19:10
ayoungdolphm, is it correct to say tokesn do have ID attributes, or just that they are not embedded?19:10
*** novas0x2a|laptop has quit IRC19:10
ayoungactually, I think that statement is correct from an API perspective19:12
ayounghenrynash, dolphm pulled the trigger19:12
*** colinmcnamara has joined #openstack-dev19:14
*** FatDarrel has quit IRC19:15
*** dhellmann is now known as dhellmann-afk19:15
gyeeayoung, what's the branch for rc1?19:16
ayounggyee, no idea...19:16
topoldolphm, i there still a fix me on line 1145 i22696. Is that OK?19:17
*** soody has joined #openstack-dev19:17
gyeehenrynash, dolphm, any idea? how do I code against rc1?19:18
topoldolphm, err there was still a dolph fix me on line 1145 is what I was trying to say19:19
*** morganfainberg has joined #openstack-dev19:19
mtreinishbcwaldon: I just pushed the rebased migration tests19:20
*** alunduil has quit IRC19:20
*** cloudchimp has quit IRC19:20
*** Ryan_Lane has joined #openstack-dev19:22
*** jaypipes has quit IRC19:22
*** FatDarrel has joined #openstack-dev19:22
*** kagan has quit IRC19:24
*** psedlak has joined #openstack-dev19:27
henrynashgyee: i thought that master was still headed for rc1, no?19:28
*** devoid has quit IRC19:28
henrynashgyee: which sounds dangerous…so that can't be right!19:28
*** dubsquared1 has joined #openstack-dev19:30
dubsquared1I have a random question about openstack services from folsom to grizzly … does anyone know if nova-network and nova-volume are still present in the grizzly release?19:32
jgriffithdubsquared1: nova-volume is history19:32
dubsquared1so what was 'left' so volume, is now cinder … and the attach/detach commands are left up to compute, yes?19:33
*** dachary has joined #openstack-dev19:33
jgriffithdubsquared1: correct19:33
dubsquared1perfect, thanks19:33
dolphmayoung: as long as we're not adding new features to the API doc -- when havana opens up for dev, we should start considering new api features as v3.119:33
ayoungdolphm, sounds good19:34
dolphmayoung: i also just caught a typo in one of my json changes19:34
*** devoid has joined #openstack-dev19:35
dolphmayoung: henrynash: gyee: someone approve typo fix? https://review.openstack.org/#/c/22734/19:35
*** dprince has quit IRC19:36
ayoungdolphm, done19:36
sdaguehenrynash: master is headed to rc1, the final branching doesn't happen until later.19:36
dolphmayoung: thanks, sorry about that -- i normally validate json, but these changes were so trivial..19:36
ayoungdolphm, NP.  API docs will get a final scrub before GA19:37
gyeesdague, can you quantify "later"? If I need to make bug fix for rc1 now, which branch should I be using?19:38
sdaguemaster19:38
gyeesdague, cool, thanks for the info19:38
sdagueit's up to the cores on each project at this point to enforce freeze on their master19:38
*** novas0x2a|laptop has joined #openstack-dev19:38
sdagueand let in what they think is right19:38
gyeemake sense19:39
ladquinvishy, got a sec?19:39
vishyladquin: sup?19:40
openstackgerritA change was merged to openstack/glance: Document db_auto_create in default config files  https://review.openstack.org/2221419:40
sdaguejgriffith: you seeing this error in cinder local tests? - Exception: Versioning for this project requires either an sdist tarball, or access to an upstream git repository.19:40
ladquinvishy, hey, I working on a commit to document this https://review.openstack.org/#/c/17273/19:40
ladquinvishy, I'm sure the URI there is correct, but I don't understand exactly how it works... is this for the admin pass?19:41
jgriffithsdague: fixed that by bumping the oslo version19:43
openstackgerritA change was merged to openstack/python-openstackclient: Added compute agent support.  https://review.openstack.org/2233719:43
sdaguejgriffith: which oslo version?19:43
jgriffithsdague: https://review.openstack.org/#/c/22450/19:44
jgriffithb419:44
sdaguehmmm.... that doesn't fix it for me19:44
sdagueI have b419:44
jgriffith:(19:44
openstackgerritA change was merged to openstack/glance: Remove compat cfg wrapper  https://review.openstack.org/2271219:44
jgriffithThat was the exact same error IIRC19:44
jgriffithI had numerous problems with the version code however19:46
sdaguelet me re-re-replicate it on totally fresh repo19:46
ayoungdolphm, I just rebased and merged my trusts API changes.  I noticed that in the rebase it looks like a small section of the sample token data was removed.  Is that correct?19:46
jgriffithsdague: nova?19:46
sdaguecinder19:46
jgriffithsdague: Oh...19:46
sdaguethat's why I'm asking you :)19:47
jgriffith:)19:47
*** soody has quit IRC19:47
jgriffithThought maybe it was just broke somewhere else now (hoped)19:47
*** kagan has joined #openstack-dev19:47
jgriffithsdague: I'm not using venv either19:50
sdaguejgriffith: ok, let me try a couple more things19:50
jgriffithsdague: that may have something to do with it19:50
sdagueget back to you in a minute19:50
sdagueyeh, it might... :P19:50
jgriffithk19:50
*** esp1 has quit IRC19:52
sdaguejgriffith: you just use tox?19:53
jgriffithsdague: No... I do run_tests.sh outside of venv19:54
jgriffithtox, venv etc just causes me problems on a constant basis19:54
*** FatDarrel has quit IRC19:54
jgriffithsdague: I'm downloading venv and trying your case now19:54
*** esp1 has joined #openstack-dev19:54
jgriffithIt should only take 15 minutes19:54
jgriffith:(19:54
sdagueso, there is an install_venv_common issue as well, which I'm going to sync and fix19:55
*** FatDarrel has joined #openstack-dev19:56
sdaguebut then I tripped over this other thing19:56
jgriffithsdague: So we've had a number of issues with the version with the oslo activity the past week or so19:57
jgriffithI thought/hoped that was behind us19:57
*** gyee has quit IRC19:57
*** dachary has quit IRC20:00
*** mdomsch has joined #openstack-dev20:00
sdaguejgriffith: yep, still an issue. But let me at least push this install_venv_common fix into the queue20:01
jgriffithsounds good....20:01
*** arbrandes has joined #openstack-dev20:01
jgriffithI'm still downloading... maybe I'll get lunch and come back20:01
jgriffithOh crap... meeting!20:01
jgriffithsdague: I'll check in with ya when I get back20:02
sdaguecool20:02
*** numero8 has joined #openstack-dev20:05
*** cloudchimp has joined #openstack-dev20:07
*** sandywalsh_ has quit IRC20:08
*** sacharya has quit IRC20:08
sdaguejgriffith: it's this bug - https://bugs.launchpad.net/cinder/+bug/112541620:08
uvirtbotLaunchpad bug 1125416 in cinder "run_tests.sh fails with version_string errors" [Undecided,In progress]20:08
*** FatDarrel has quit IRC20:09
*** arbrandes has quit IRC20:11
*** arbrandes has joined #openstack-dev20:12
*** nsatterl_ has joined #openstack-dev20:14
openstackgerritA change was merged to openstack-infra/devstack-gate: Enable sysstat on devstack for investigate timeout  https://review.openstack.org/2265720:16
*** reed has joined #openstack-dev20:17
*** danwent has quit IRC20:19
*** dolphm has quit IRC20:20
*** n0ano has joined #openstack-dev20:21
*** ron-slc_ has quit IRC20:22
mtreinishmarkwash, bcwaldon: So CI found a bug with migration 018. The engine wasn't set to InnoDB when image_locations was created. Is it ok to just update the definition in the migration or should I make a new migration to switch it?20:23
*** arbrandes has quit IRC20:24
mtreinishalso I guess this question applies to the status length in https://review.openstack.org/#/c/22731/ too20:24
*** psedlak has quit IRC20:25
*** psedlak has joined #openstack-dev20:25
*** arbrandes has joined #openstack-dev20:25
YorikSartopol: Hi, here I am.20:27
YorikSarWe had OpenStack meetup here :)20:27
*** devoid has quit IRC20:28
*** ayoung is now known as ayoung-lunch20:28
YorikSartopol: I should've asked you in the Gerrit. How did you check if my code ran or not?20:28
topolHi YorikSar,   I need to finish something that should take about 15 minutes.   I ran your code against a live ldap and tried to test cases.  1st test case was to see if your code would create the member role.  It did that test case perfectly20:30
*** mkollaro has quit IRC20:31
*** gyee has joined #openstack-dev20:31
topolYorikSar,  I then tried a second test case where I put a user as a member attribute under a created project and the code did not move the user.20:32
*** devoid has joined #openstack-dev20:32
*** dolphm has joined #openstack-dev20:32
topolI put in a comment the ldap ldif file I was using if you want to remove it.  It did not appear to me that the migrate routine ran. Did I need to do something special for it to run?  In any case I need to finish something but can discuss more in bout 15 minutes20:33
topols/remove/try20:34
*** crandquist has joined #openstack-dev20:34
YorikSartopol: db_sync runs that migration. At least should.20:36
topolYorikSar, in 15 minutes I can pull up that test branch I had and try and do some debugging20:36
YorikSartopol: Ok, I'll check it again in my local LDAP in the mean time.20:37
topolk20:37
*** gyee has quit IRC20:38
*** Mathnerd314 has joined #openstack-dev20:38
*** cp16net is now known as cp16net|away20:39
*** lloydde has joined #openstack-dev20:40
*** zzs has joined #openstack-dev20:41
*** gyee has joined #openstack-dev20:41
*** radez is now known as radez_g0n320:41
*** dubsquared1 has quit IRC20:41
*** arbrandes_ has joined #openstack-dev20:44
*** arbrandes has quit IRC20:46
*** cp16net|away is now known as cp16net20:46
*** dubsquared1 has joined #openstack-dev20:46
YorikSartopol: Looks like I tested one thing then changed it.20:48
YorikSartopol: Fixed that problems. Let me add more tests and push it to Gerrit.20:49
zigo_https://paste.debian.net/237199/ <--- What's going on with the test suite of Nova???20:51
*** arbrandes_ has quit IRC20:52
*** arbrandes__ has joined #openstack-dev20:52
*** aeperezt has quit IRC20:53
topolYorikSar, sounds good,  When its ready I will pull it down and do some more live ldap tests.20:55
*** lglenden has quit IRC20:56
YorikSartopol: I actually did LDAP tests. But it looks like I did some optimization (smashing and breaking) afterwards20:56
ayoung-lunchdolphm, OK,  I forget exactly what I told you last night was on the todo list, but right now, all I can remember that is not done is: 1.  v3 auth, 2.  clean up unit tests, 3. match the list calls to the API spec.  4.  make role an object in the get call (just doing name right now.20:56
YorikSartopol: I mean, live LDAP20:56
bcwaldonmtreinish: we probably want a new migration20:57
topolYorikSar, understand. but always good to have the non code developer test it :-)20:57
YorikSartopol: Sure20:57
*** mnaser has quit IRC20:57
*** aeperezt has joined #openstack-dev20:58
clarkbzigo_ update or venv and or testr20:58
*** cloudchimp has quit IRC20:58
zigo_clarkb: I have the latest testr from upstream.20:59
*** esp1 has quit IRC20:59
mtreinishbcwaldon: ok, I pushed it20:59
zigo_clarkb: From the package I just made ...20:59
zigo_clarkb: I'm using -N, so it shouldn't be using virtualenv !20:59
clarkbit should install setup.py entrypoints created20:59
*** yolanda has quit IRC20:59
clarkblatest testr that is20:59
ayoung-lunchgyee, as far as V3 auth, I assume I need to 1) add a checkinto the auth controller for the trust field, and 2 extend the token-factory.21:00
clarkband that entrypoint isnt being found21:00
zigo_clarkb: I have 0.0.9-121:00
clarkbI think there is a newer version21:01
*** imsplitbit has quit IRC21:01
*** mnaser has joined #openstack-dev21:01
clarkb0.0.14 according to pypi21:02
gyeeayoung-lunch yes, token-factory does all the heavy lifting already21:02
zigo_clarkb: I'll try, thanks.21:03
*** ayoung-lunch is now known as ayoung21:03
ewindischdripton: take another look at 20434 - redis matchmaker ?21:03
gyeeayoung-lunch, all you have to do is filter the roles and set the user-id accordingly21:03
ayounggyee, there any direct unit tests for token-factory, or is it just tested through the other objects?21:03
driptonewindisch: I will21:03
ewindischthanks :)21:03
gyeetested through test_v3_auth21:04
*** zzs has left #openstack-dev21:04
*** dubsquared1 has quit IRC21:04
*** reed has quit IRC21:04
*** dachary has joined #openstack-dev21:06
*** FatDarrel has joined #openstack-dev21:06
*** monst__ has joined #openstack-dev21:08
*** cp16net is now known as cp16net|away21:08
*** esp1 has joined #openstack-dev21:09
*** martine_ has joined #openstack-dev21:09
*** nsatterl_ has quit IRC21:10
*** danwent has joined #openstack-dev21:10
YorikSartopol: Here it is.21:11
*** monst_ has quit IRC21:11
topolYorikSar, did you post a new patch?21:12
YorikSartopol: Yep21:12
topolYorikSar, OK21:12
*** arbrandes__ has quit IRC21:13
*** cp16net|away is now known as cp16net21:16
YorikSarayoung: I remember, you said there is some change request or bug about logging in keystone-manage. Can you point me at one?21:17
ayoungYorikSar, not sure what you meant.  What are you trying to do?21:17
*** reed has joined #openstack-dev21:18
YorikSarayoung: I want keystone-manage to setup logging so that, for example, if --debug is passed, it'll log to console.21:18
*** maoy has quit IRC21:18
ayoungYorikSar, not aware of anything that will do that for you. No bug that I know of.21:19
YorikSar(looks like Russian commas)21:19
YorikSarayoung: Ok, pushing new change request then.21:19
*** annegentle has quit IRC21:20
YorikSarayoung: Really simple: https://review.openstack.org/22745 :)21:21
*** pabelanger has quit IRC21:21
*** buzztroll_ has joined #openstack-dev21:22
*** FatDarrel has quit IRC21:23
*** hugokuo has joined #openstack-dev21:26
*** vkmc has quit IRC21:27
*** numero8 has quit IRC21:27
*** yidclare has quit IRC21:27
*** Tross has quit IRC21:28
*** eharney has quit IRC21:29
sdaguejgriffith: you about around?21:30
*** stevebaker has quit IRC21:31
*** psedlak has quit IRC21:31
*** koolhead17 has quit IRC21:31
*** FatDarrel has joined #openstack-dev21:32
*** iartarisi has joined #openstack-dev21:33
*** yidclare has joined #openstack-dev21:35
*** stevebaker has joined #openstack-dev21:38
*** digitalsanctum has quit IRC21:38
*** vipul is now known as vipul|away21:39
*** adalbas has quit IRC21:39
*** yidclare has quit IRC21:43
*** psedlak has joined #openstack-dev21:44
*** mdomsch has quit IRC21:46
ayoungYorikSar, that scares me.  What does it do?21:46
YorikSarayoung: It just creates all Handlers for logging infrastructure.21:47
YorikSarayoung: So that we can see, for example, LDAP iteractions if we want.21:47
ayoungYorikSar, if it changes the default output on a normal run, it will break a lot of tools21:47
YorikSarayoung: It might change stderr output, but not stdout.21:48
ayoungYorikSar, I'd prefer it if that were not the default behaviour21:48
YorikSarayoung: If keystone.conf is accessible (not Ubuntu) and has verbose=True or debug=True (not production)21:48
YorikSarayoung: It is not.21:49
ayoungYorikSar, That will go to the log files.  Not the right place21:49
YorikSarayoung: By default debug and verbose are False.21:49
ayoungYorikSar, it shoulkd be controlled by a command line switch, not from the config files21:50
ayoungand should log to stderr only, I think21:50
YorikSarayoung: If you pass --debug or --verbose, it will.21:50
*** mtreinish has quit IRC21:51
YorikSarayoung: In typical logging configuration it'll create /var/log/keystone/keystone-manage.log, I think/21:51
*** vipul|away is now known as vipul21:51
YorikSarayoung: Just like nova-manage.21:51
ayoungYorikSar, not sure that is appropriate, but lets let some others chime in.21:51
*** digitalsanctum has joined #openstack-dev21:52
ayoungAdded some other reviewers21:52
YorikSarayoung: And I think, management process should be logged since it's basically some direct poking to backends.21:53
ayoungYorikSar, I'm not saying it is a bad idea, just not ready to commit...21:53
YorikSarayoung: Oh, 'some' other reviewers :)21:54
ayoungYorikSar, I'm heads down in the trusts work.21:54
*** martine_ has quit IRC21:55
openstackgerritA change was merged to openstack/nova: Remove Print Statement  https://review.openstack.org/2274321:58
*** eharney has joined #openstack-dev21:59
*** eharney has quit IRC21:59
*** eharney has joined #openstack-dev21:59
*** mdomsch has joined #openstack-dev21:59
*** mdomsch has quit IRC22:01
ayounggyee, so I want to do auth_response = self.auth_v3_controller.authenticate({}, request_body)  but I see that there is auth_context and auth_data and I have to say, that is not necessarily self docuementing.  What is the scope of each?22:01
ayoungauth_info, not auth_data22:01
*** TheEstablishment has joined #openstack-dev22:02
*** monst__ has quit IRC22:02
ayoungoh I want authenticate_for_token don;t I22:03
*** monst_ has joined #openstack-dev22:03
gyeeayoung, are you using auth.controllers.Auth22:03
markwasheglynn: I was just taking a look at https://review.openstack.org/#/c/19547/, I think that approach is the right way to go and left a note on the (abandoned) review22:03
ayounggyee, yep22:04
markwasheglynn: let me know what you think, we can figure out how to move forward for rc122:04
gyeeyeah22:04
*** TheEstablishment has quit IRC22:04
gyeeyou need authenticate_for_toke22:04
*** dubsquared1 has joined #openstack-dev22:05
gyeeayoung, I purposely name it that way so we can have authenticate_for_x509 in the future22:05
*** jrodom has quit IRC22:06
*** jbryce has quit IRC22:07
openstackgerritA change was merged to openstack/tempest: update identity to handle new table attributes  https://review.openstack.org/2226922:08
*** mdomsch has joined #openstack-dev22:09
*** arbrandes__ has joined #openstack-dev22:09
*** dubsquared1 has quit IRC22:10
*** soody has joined #openstack-dev22:11
topolYorikSar, the latest version did not get through basic devstack keystone operations without failing.  Here is the keystone log fpaste.org/Q10k/22:11
*** sandywalsh has joined #openstack-dev22:11
*** ayoung has quit IRC22:11
*** sandywalsh has quit IRC22:12
YorikSartopol: Looks like smth wrong with sql.connection setting22:13
topolYorikSar, you mean something set by devstack or something in your patch I pulled down?22:15
YorikSartopol: I guess, devstack config file has 'connection=' line in [sql] section.22:15
*** esp1 has left #openstack-dev22:16
YorikSartopol: Not devstack config, of course, but Keystone one.22:16
openstackgerritA change was merged to openstack/tempest: Cleanup of identity/admin/test_users.py  https://review.openstack.org/2264922:17
*** john5223 has quit IRC22:17
*** buzztroll_ has quit IRC22:17
topolYorikSar, Oh, OK. I was testing using devstack with minimal services. I will retest turning on full services22:17
topolYorikSar, I will try and retest later tonight22:18
*** boden has quit IRC22:18
*** mdomsch has quit IRC22:19
YorikSarWow. Ran tests the first time after refactor and they worked.22:19
*** mdomsch has joined #openstack-dev22:19
*** dolphm has quit IRC22:22
*** n0ano has quit IRC22:23
*** crandquist has quit IRC22:23
YorikSartopol: Look - more LDAP backend cleanup :) https://review.openstack.org/2275222:26
*** topol has quit IRC22:29
*** giulivo has quit IRC22:29
*** dubsquared1 has joined #openstack-dev22:31
*** n0ano has joined #openstack-dev22:33
*** esheffield has quit IRC22:33
*** nati_ueno has quit IRC22:35
*** Nachi has joined #openstack-dev22:35
*** nati_ueno has joined #openstack-dev22:35
*** spzala has quit IRC22:37
*** networkstatic has joined #openstack-dev22:37
*** esp1 has joined #openstack-dev22:39
*** Nachi has quit IRC22:39
*** eharney has quit IRC22:42
*** hugokuo has quit IRC22:42
*** hugokuo has joined #openstack-dev22:44
*** arbrandes__ has quit IRC22:44
*** hugo_kuo has joined #openstack-dev22:44
*** hugo_kuo has left #openstack-dev22:45
*** pcm_ has quit IRC22:46
*** mdomsch has quit IRC22:46
*** Tross has joined #openstack-dev22:46
*** hugo__kuo has joined #openstack-dev22:46
*** soody has quit IRC22:47
*** eglynn has quit IRC22:47
*** soody has joined #openstack-dev22:47
*** iartarisi has quit IRC22:48
*** hugo__kuo has quit IRC22:50
*** dubsquared1 has quit IRC22:50
*** dubsquared1 has joined #openstack-dev22:51
*** pixelbeat has joined #openstack-dev22:51
clarkbjgriffith: cinder-api fails to start using the sample cinder.conf because it is missing a log_config specification. gate doesn't catch it because i believe we set it on the commandline22:51
clarkbjgriffith: is that something that should be a bug?22:52
*** Gordonz has quit IRC22:53
*** jcmartin has quit IRC22:54
clarkbhmm maybe that is not the problem22:55
*** bdpayne has joined #openstack-dev22:55
*** monst__ has joined #openstack-dev22:56
openstackgerritA change was merged to openstack/nova: Fix exception handling in baremetal API.  https://review.openstack.org/2262722:56
*** alunduil has joined #openstack-dev22:57
*** soody has quit IRC22:58
*** monst_ has quit IRC22:59
*** arbrandes__ has joined #openstack-dev23:00
*** kbringard has quit IRC23:00
*** dachary has quit IRC23:01
*** monst__ has quit IRC23:01
*** rnirmal has quit IRC23:01
bknudsonis there any way to break into the debugger during run_tests.sh?23:03
YorikSarbknudson: Add 'import pdb; pdb.set_trace()' to set a breakpoint in code and runs run_tests.sh with -s arg23:04
clarkbit also depends on your project23:04
*** dachary has joined #openstack-dev23:04
bknudsonnova.23:05
openstackgerritA change was merged to openstack/nova: Remove compat cfg wrapper  https://review.openstack.org/2271123:05
clarkbbknudson: yeah nova is going to be slightly different because it uses testr as a test runner runner23:05
openstackgerritA change was merged to openstack/nova: Add module prefix to exception types  https://review.openstack.org/2259123:05
*** networkstatic has quit IRC23:05
bknudsonI get stdout: {{{ ... (Pdb) }}} so it's hitting the break but not interactive.23:06
clarkbbknudson: add the pdb stuff to your code then source the venv. Then `python -m testtools.run path.to.test`23:06
clarkbbknudson: yeah testr captures stdout and stderr and logs then logs it all into subunit format. to avoid that use testtools.run directly23:06
*** jbr_ has joined #openstack-dev23:08
bknudsonclarkb: thanks. worked like a champ23:08
*** dolphm has joined #openstack-dev23:08
bknudsone.g., tools/with_venv.sh python -m testtools.run nova.tests.api.openstack.compute.contrib.test_cells.TestCellsXMLSerializer23:08
*** cloudchimp has joined #openstack-dev23:09
clarkbyeah I should add that as a thing to run_tests23:09
bknudsonclarkb: call it -s.23:09
*** Ryan_Lane has quit IRC23:09
*** Ryan_Lane has joined #openstack-dev23:09
*** ayoung has joined #openstack-dev23:09
*** FatDarrel has quit IRC23:11
clarkbwill do23:11
*** nati_ueno has quit IRC23:11
*** alunduil has quit IRC23:11
*** nati_ueno has joined #openstack-dev23:12
YorikSarWhen I don't find a way to pass stdin/out to pdb, I open another tab (tmux pane), run 'tty; sleep 10000' there and change breakpoint code to smth like "f1,f2=open('/dev/pts/XX','r'),open('/dev/pts/XX','w');import pdb; pdb.Pdb(stdin=f1,stdout=f2).set_trace()"23:12
clarkbheh of course run_tests.sh already has -s for something else23:12
YorikSarThen debug in separate pane as usual.23:12
*** dubsquared1 has quit IRC23:13
YorikSarclarkb: You can purge that old option there.23:14
YorikSarclarkb: virtualenv creates isolated environment by default.23:14
clarkbI think I am just giong to use the old flags before the testr switch23:15
clarkband reviewers can sort out what they want it to be long term23:15
bknudsonbtw - what does -s do? I didn't find it in the help.23:15
bknudsonfor keystone or nova23:15
clarkbbknudson: for nova it is --no-site-packages23:15
YorikSarFor Nova: -s, --no-site-packages      Isolate the virtualenv from the global Python environment23:15
bknudsonoh, there it is.23:16
YorikSarbknudson: For all other it's passed to nosetests23:16
bknudsonwhy would I want that for debug?23:16
YorikSarbknudson: And then nosetsts won't intercept stdin/out/err23:16
YorikSarbknudson: And Pdb is happy.23:17
clarkbYorikSar: nova does not use nose anymore23:17
YorikSarclarkb: Why?..23:17
clarkbbecause we wanted clean parallel testing and nose does some really weird things23:18
*** vipul is now known as vipul|away23:18
bknudsonclarkb: the parallel testing is great.23:19
clarkblike not fail tests if setUpModule() throws an exception23:19
clarkbor passing if 0 tests are run23:19
YorikSarThen I'll say it this way: Why aren't other project switched to testr? :)23:19
clarkbYorikSar: we have been trying to. its not a simple process as all intertest conflicts need to be sorted out23:19
clarkbwe have been working on it though.23:19
*** jimfehlig has quit IRC23:19
clarkbI think mordred has most of the python-*clients done but I may be mistaken23:20
clarkband at this point I think many projects would rather we wait for havana23:20
YorikSarI've seen some very scary monsters in nose source. Always wondered if there is something as powerful, but cleaner and better.23:20
clarkbI really like testr and subunit23:20
clarkbthe subunit logs are great I think and was something missing with nose23:20
*** davidh_ has quit IRC23:21
clarkb(testr keeps track of successive runs so taht you can run just failing tests and ask for statistics)23:21
bknudsonclarkb: so is there a similar command for keystone?23:21
bknudsonto tools/with_venv.sh python -m testtools.run ...23:21
clarkbbknudson: not yet23:22
clarkbit would be run_tests.sh -s as keystone still uses nose23:22
YorikSarbknudson: For keystone you can use -s23:22
jgriffithclarkb: yes, sounds like a but to me23:23
jgriffithclarkb: but it needs added in devstack23:23
YorikSarclarkb: Why is it run as 'setup.py testr', not 'setup.py test' that sounds more common?23:23
jgriffithOr I mean "it is set in devstack"23:23
clarkbYorikSar: because we needed to condense several testr specific commands into one and mordred went that route23:24
clarkbjgriffith: yeah I am not entirely sure what was going on. I think it was swift ran into a problem where we use the --log-config variable on the command line in devstack and oslo config changes affected swift.conf log parameters23:24
clarkbechohead: ^ say hello to jgriffith23:25
openstackgerritA change was merged to openstack-dev/devstack: Fixes Bug1130377  devstack ldap set values in keystone.conf  https://review.openstack.org/2235623:26
*** vipul|away is now known as vipul23:27
*** Tross has quit IRC23:27
jgriffithclarkb: all that I don't know why this is an issue?23:27
*** henrynash has quit IRC23:28
*** datsun180b has quit IRC23:29
*** bdpayne has quit IRC23:29
jgriffithsdague: back... what's up?23:30
*** bdpayne has joined #openstack-dev23:31
clarkbjgriffith: apparently using the sample cinder.conf and logging.conf (edited slightly for env specific things) tip of master cinder-api fails to start.23:31
*** alszar has quit IRC23:32
clarkbjgriffith: I think swift had a similar problem and it got through the gate because we start services in the gate with additional command line options23:32
zykes-hmm dolphm about ? I upgrade to latest keystone trunk and now I get sqlalchemy.exc.NoSuchTableError: `tenant`23:32
jgriffithclarkb: Maybe I'm not following, the default in all the projects I believe is log_config=<None> isn't it?23:32
zykes-http://pastebin.com/FNPF8Vjy23:33
jgriffithOr are you saying if folks over-ride it it breaks?23:33
*** digitalsanctum has quit IRC23:33
clarkbjgriffith: yeah I think it was overriden.23:34
*** dolphm has quit IRC23:34
*** soody has joined #openstack-dev23:35
clarkbI believe it was complaining about log-format specifically in oslo config23:35
*** trapniii has quit IRC23:36
*** lloydde has quit IRC23:36
*** lloydde has joined #openstack-dev23:36
*** dolphm has joined #openstack-dev23:37
echoheadhey jgriffith: i'm here now.23:38
jgriffithclarkb: well that's unfortunate, I think the real problem is cinder-api doesn't even register the FLAGS :(23:38
echoheadjgriffith: reading scrollback.23:38
jgriffithechohead: hi there23:38
echoheadhi23:38
jgriffithechohead: Assuming you discovered the issue clarkb has mentioned/23:38
jgriffith?23:38
echoheadright.23:38
echoheadone sec and i'll make a paste with my config and stuff.23:39
jgriffithK23:39
*** dolphm has quit IRC23:39
jgriffithalthough I think it's most likely just because cinder-api is ignoring your Flags anyway23:39
jgriffithIt's only reading command line options23:39
jgriffithSo as clarkb pointed out, works fine in the gates23:40
jgriffithand devstack23:40
echoheadi tried passing --config-file, --log-config, --config-dir via command line, to no avail.23:40
*** lloydde has quit IRC23:41
clarkbbknudson: vishy sdague https://review.openstack.org/#/c/22761/23:41
jgriffithechohead: oh, even echohead I would've expected --config-file to work23:42
*** rpedde is now known as rpedde_away23:42
*** dhellmann-afk has quit IRC23:43
*** psedlak has quit IRC23:43
echoheadjgriffith: http://paste.openstack.org/show/32379/23:44
sdaguejgriffith: just curious if you were able to confirm the bug23:44
jgriffithsdague: no, but found another one possibly23:44
echohead(sorry the line-endings got botched in the paste)23:44
sdaguejgriffith: with a clean venv you weren't able to confirm?23:45
*** dosaboy has quit IRC23:45
sdaguejgriffith: your source tree isn't a devstack source tree is it?23:45
jgriffithsdague: I can't get dam venv to install23:45
jgriffithno... on my laptop23:45
*** dosaboy has joined #openstack-dev23:45
sdaguejgriffith: oh, you need the fix I put in23:45
jgriffithand I just got back so haven't tried it again23:45
jgriffithcrap23:45
jgriffithyeah, I knew that23:45
jgriffithdang23:45
sdagueclarkb: what's the bug for the VOLUMES fail?23:46
jgriffithI made the set -u but forgot the bootstrapping one23:46
sdagueon quantum23:46
bknudsonclarkb: we should still support running without -d ;)23:46
clarkbbknudson: yeah my bad23:47
sdaguejgriffith: my install_venv_common.sh got gate killed by that same VOLUMES fail in quantum23:47
clarkbsdague: uh I recently restarted firefox and now I don't have it handy23:47
jgriffithsdague: So that's our friendly "instance fails to come up"23:47
echoheadjgriffith: clarkb: i have to run. msg me on irc if there's anything else i can do, will respond later.23:47
*** annegentle has joined #openstack-dev23:47
sdagueclarkb: have keywords?23:47
jgriffithsdague: Yeah, that's what I wsa pointing out, it's not really "volumes" that failed23:48
clarkbsdague: I think nati updated it23:48
sdagueclarkb: which project was it assigned to?23:48
*** trapniii has joined #openstack-dev23:48
jgriffithsdague: ERROR: No server with a name or ID of 'ex-vol-inst' exists.23:48
clarkbsdague: I think it may be under nova23:48
jgriffithoh wait... sorry23:48
*** dosaboy has quit IRC23:48
clarkbbecause it was there first when nati_ueno was upadting things23:49
nati_ueno>23:49
nati_ueno?23:49
clarkbnati_ueno: the bug that you added sysstat to debug23:50
clarkbnati_ueno: what is the bug number?23:50
sdaguehttps://bugs.launchpad.net/nova/+bug/1101147 ? or another one23:50
uvirtbotLaunchpad bug 1101147 in nova ""Instance didn't become active" in Devstack Gate Exercises" [Critical,Confirmed]23:50
nati_uenoclarkb: This one https://bugs.launchpad.net/nova/+bug/1131030 ?23:50
uvirtbotLaunchpad bug 1131030 in nova "echo 'volume didnt become available within 180 seconds'" [Undecided,Invalid]23:50
clarkbyeah23:50
clarkbty23:51
sdaguenati_ueno: cool, thanks23:51
sdaguejgriffith: well once the install_venv fix lands, you should be to a replication state23:51
nati_uenosdague: np. It looks cinder connection problem23:51
jgriffithsdague: I just pulled the change in running now23:51
jgriffithsdague: So this is the c-vol log empty thing again!23:52
sdagueI'm about to become a pumpkin, friday night and all. hopefully it's enough to get you running23:52
sdaguejgriffith: yeah, though honestly I just want unit tests to run locally :)23:52
jgriffithclarkb: fungi any chance that instance still exists?23:52
sdagueso I can finish the db unit tests for cinder23:52
clarkbjgriffith: no we checked after the most recent one23:52
jgriffithhttp://logs.openstack.org/22739/1/gate/gate-tempest-devstack-vm-quantum/9725/logs/23:52
clarkbthey get cleaned up pretty quickly23:52
jgriffithbummer23:53
*** dontalton has quit IRC23:53
clarkbjgriffith: we will probably need to have jenkins check for specific failures and not delete the instance if they occur23:53
clarkbbut that will take a little work to do23:53
*** jcmartin has joined #openstack-dev23:53
jgriffithclarkb: yes please23:53
jgriffith:)23:53
*** gongysh has joined #openstack-dev23:53
jgriffithclarkb: what about just adding a "check service" after screen_it ?23:53
*** mrodden has quit IRC23:53
jgriffithas it seems it could be one of a number of services that hit this23:54
*** colinmcnamara has quit IRC23:54
clarkbjgriffith: and error early with diagnostic info if the service is not up?23:54
jgriffithclarkb: exactly23:54
jgriffithfail fast23:54
clarkbyeah if you want to add that into devstack I would be for it :)23:54
jgriffithand think about what we might be able to collect23:54
jgriffithclarkb: Yeah, I will see about that after I sort the things echohead and sdague are seeing23:55
clarkbI think that adding the ability to capture arbitrary nodes to jenkins is a useful thing though so we should probably do that too23:55
jgriffithclarkb: that one I'll leave to you :)23:55
sdaguealright, well I'm out for the rest of the night. But if you guys come up with a devstack fix, I'll look in the morning over coffee. Otherwise monday.23:56
sdaguehave a good one folks23:56
clarkbyou too23:56
jgriffithsdague: take care23:56
jgriffithechohead: still there?23:57
*** soody has quit IRC23:58

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