Friday, 2011-08-12

*** vladimir3p has quit IRC00:06
openstackgerritZiad Sawalha proposed a change to openstack/keystone: Legacy auth fix and doc, wadl, and xsd updates  https://review.openstack.org/21800:18
openstackgerritA change was merged to openstack/keystone: Legacy auth fix and doc, wadl, and xsd updates  https://review.openstack.org/21800:21
mtaylornotmyname: I know ttx definitely agrees with you00:34
*** jakedahn has quit IRC00:38
notmynamemtaylor: I'm sure he does ;-)00:40
*** mszilagyi has quit IRC00:46
mtaylornotmyname: chatted with some launchpad devs ... have a few ideas to try00:46
notmynamegreat00:46
mtaylorof course ... I think if I don't get the kombu packaging sorted out before I poke at that, jaypipes might kill me dead00:47
openstackjenkinsProject nova build #1,238: SUCCESS in 3 min 17 sec: https://jenkins.openstack.org/job/nova/1238/00:50
openstackjenkinsTarmac: Support for management of security groups in OS API as a new extension.00:50
*** HugoKuo has joined #openstack-dev00:55
*** heckj has quit IRC00:59
*** Tushar has quit IRC01:08
*** jdurgin has quit IRC01:16
*** jakedahn has joined #openstack-dev01:28
*** mattray has joined #openstack-dev01:51
*** jakedahn has quit IRC01:52
*** HugoKuo has quit IRC02:31
*** HugoKuo has joined #openstack-dev02:31
*** mattray has quit IRC02:31
*** HugoKuo has quit IRC02:32
*** HugoKuo has joined #openstack-dev02:32
*** HugoKuo has quit IRC02:34
*** HugoKuo1 has joined #openstack-dev02:35
*** anotherjesse has quit IRC02:44
*** chomping has joined #openstack-dev02:52
*** novemberstorm has joined #openstack-dev03:14
*** novemberstorm has quit IRC03:20
*** mattray has joined #openstack-dev03:26
*** mdomsch has joined #openstack-dev03:41
openstackgerritVish Ishaya proposed a change to openstack/keystone: fix ec2 and add keystone-manage command for creating credentials  https://review.openstack.org/21904:00
*** martine has joined #openstack-dev04:24
*** HugoKuo has joined #openstack-dev04:29
*** HugoKuo1 has quit IRC04:29
*** HugoKuo has quit IRC04:32
*** HugoKuo has joined #openstack-dev04:32
*** HugoKuo has quit IRC04:44
*** HugoKuo has joined #openstack-dev04:44
*** zaitcev has quit IRC04:44
*** martine has quit IRC04:44
*** mattray has quit IRC04:53
*** martine has joined #openstack-dev05:23
*** martine has quit IRC05:39
*** tsuzuki has joined #openstack-dev05:47
*** HugoKuo has quit IRC05:54
*** zul has joined #openstack-dev06:38
*** zul has quit IRC06:42
*** anotherjesse has joined #openstack-dev06:52
openstackgerritYuriy Taraday proposed a change to openstack/keystone: Removed ldap names import from fakeldap module.  https://review.openstack.org/22007:09
*** zul has joined #openstack-dev07:16
*** anotherjesse has quit IRC07:45
*** truijllo has joined #openstack-dev07:53
*** zul has quit IRC08:01
*** nickon has joined #openstack-dev08:01
*** teamrot has quit IRC08:12
*** zul has joined #openstack-dev08:41
*** rods has joined #openstack-dev08:58
*** darraghb has joined #openstack-dev08:59
*** chomping has quit IRC09:28
*** chomping has joined #openstack-dev09:32
*** chomping has quit IRC09:35
*** chomping has joined #openstack-dev09:36
*** medberry is now known as med_out10:06
*** med_out has left #openstack-dev10:06
*** tsuzuki has quit IRC10:27
*** lorin1 has joined #openstack-dev10:51
*** zul has quit IRC10:59
*** AhmedSoliman has joined #openstack-dev11:14
*** chomping has quit IRC11:27
*** martine has joined #openstack-dev11:28
*** mfer has joined #openstack-dev11:46
*** AhmedSoliman has quit IRC12:00
*** dprince has joined #openstack-dev12:28
*** mancdaz has quit IRC12:30
*** mancdaz has joined #openstack-dev12:30
*** nmistry has joined #openstack-dev12:42
openstackgerritJoe Savak proposed a change to openstack/keystone: Replacing tokens with the dummy tokens from sampledata.sh  https://review.openstack.org/22112:51
*** bsza has joined #openstack-dev12:57
openstackgerritYuriy Taraday proposed a change to openstack/keystone: Fixed service-bound roles implementation in LDAP backend.  https://review.openstack.org/22212:58
*** mdomsch has quit IRC13:04
*** lts has joined #openstack-dev13:05
*** kbringard has joined #openstack-dev13:17
*** nmistry has quit IRC13:23
*** mfer has quit IRC13:41
*** bcwaldon has joined #openstack-dev14:01
*** amccabe has joined #openstack-dev14:20
*** mattray has joined #openstack-dev14:30
openstackjenkinsProject nova build #1,239: SUCCESS in 3 min 44 sec: https://jenkins.openstack.org/job/nova/1239/14:45
openstackjenkinsTarmac: API needs virtual_interfaces.instance joined when pulling instances from the DB.  Updated instance_get_all() to match instance_get_all_by_filters() even though the former is only used by nova-manage now.  (The latter is used by the API).14:45
*** rnirmal has joined #openstack-dev14:51
*** nmistry has joined #openstack-dev14:51
*** mfer has joined #openstack-dev14:55
*** dragondm has joined #openstack-dev15:04
*** cp16net has joined #openstack-dev15:07
openstackjenkinsProject nova build #1,240: SUCCESS in 3 min 38 sec: https://jenkins.openstack.org/job/nova/1240/15:10
openstackjenkinsTarmac: Fixes pep8 issues in test_keypairs.py15:10
*** nmistry has quit IRC15:12
*** jkoelker has quit IRC15:15
*** jkoelker has joined #openstack-dev15:15
*** jkoelker has quit IRC15:18
*** jkoelker has joined #openstack-dev15:20
*** truijllo has quit IRC15:26
*** heckj has joined #openstack-dev15:28
*** vladimir3p has joined #openstack-dev15:29
*** heckj has quit IRC15:29
openstackjenkinsProject nova build #1,241: SUCCESS in 4 min 19 sec: https://jenkins.openstack.org/job/nova/1241/15:31
openstackjenkinsTarmac: Allows for a tunable number of SQL connections to be maintained between services and the SQL server using new configuration flags. Only applies when using the MySQLdb dialect in SQLAlchemy.15:31
*** heckj has joined #openstack-dev15:40
heckjsoren: ping15:41
*** nickon has quit IRC15:44
*** zaitcev has joined #openstack-dev15:53
ttxnotmyname: With travel, I completely forgot to come back with you with a date suggestion for 1.4.3. Will have something ready for you on Monday.15:55
ttxback *to you, even15:56
notmynamettx: no rush :-)15:56
*** mgius has joined #openstack-dev15:59
*** lpatil has joined #openstack-dev16:03
openstackgerritA change was merged to openstack/keystone: Removed ldap names import from fakeldap module.  https://review.openstack.org/22016:08
*** dolph_ has joined #openstack-dev16:10
openstackgerritA change was merged to openstack/glance: Adds syslog support  https://review.openstack.org/21716:15
openstackjenkinsProject nova build #1,242: SUCCESS in 3 min 36 sec: https://jenkins.openstack.org/job/nova/1242/16:15
openstackjenkinsTarmac: Fixing a 500 error when -1 is supplied for flavorRef on server create.16:15
pvojaypipes: ping?16:29
jaypipespvo: pung16:30
openstackjenkinsProject swift build #307: SUCCESS in 30 sec: https://jenkins.openstack.org/job/swift/307/16:32
openstackjenkinsTarmac: added client_ip to all proxy log lines not otherwise containing it16:32
*** HugoKuo has joined #openstack-dev16:38
*** jdurgin has joined #openstack-dev16:38
*** anotherjesse has joined #openstack-dev16:39
*** tsuzuki has joined #openstack-dev16:41
mtaylordprince: it's your branch for glance that needs python-kombu, yeah?16:54
dprincemtaylor: no sir. I'm just the one making noise about it.16:54
openstackjenkinsProject nova build #1,243: SUCCESS in 3 min 20 sec: https://jenkins.openstack.org/job/nova/1243/16:55
openstackjenkinsTarmac: Fix a typo that causes ami images to launch with a kernel as ramdisk when using xen.16:55
dprincemtaylor: is it ready?16:55
mtaylordprince: ok. well, kombu is in the ppa now - I'm adding it to the packaging requires which should trigger it to get installed on the build slave16:55
mtaylordprince: so a few more minutes and we'll be there16:55
dprincemtaylor: So will it eventually show up here: http://ppa.launchpad.net/glance-core/trunk/ubuntu/dists/maverick/main/binary-amd64/Packages16:56
dprincemtaylor: Also. Shouldn't we have a custom version of webob in the PPA now too?16:57
mtaylordprince: yes16:57
mtaylordprince: and should we?16:57
dprincemtaylor: Yes. But it isn't there.16:57
mtaylordprince: do we need the same one that's in nova?16:58
*** mszilagyi has joined #openstack-dev16:59
dprincemtaylor: I think they can use the same version so long as its rather newish. Sorry. don't have the versions offhand that we require.16:59
mtaylordprince: ok - well, I'm going to copy the one from nova for now16:59
mtaylordprince: and then I'm going to go make a list of all backports and custom packages we use, where they come from and how we upload them17:00
mtaylorbecause this is a little silly17:00
dprincemtaylor: SUre. Ping jay on the version numbers. He researched that throughly.17:00
mtaylordprince: great.17:00
mtaylorjaypipes: ^^^^^ PING17:00
mtaylorjaypipes: talk to me17:00
mtaylorjaypipes: tell me words17:00
*** tsuzuki has quit IRC17:00
mtaylordprince: https://code.launchpad.net/~mordred/glance/add-kombu-dep/+merge/7140317:06
mtaylordprince: wait - nevermind. that's screwed up17:07
*** jtran has joined #openstack-dev17:07
jtrancan i get a code review?  https://code.launchpad.net/~jtran/nova/lp817178/+merge/7019617:07
mtaylordprince: https://code.launchpad.net/~mordred/glance/add-kombu-dep/+merge/7140417:08
dprincemtaylor: Yep. That do it once its in the PPA.17:13
*** mattray has quit IRC17:13
*** BK_man has quit IRC17:14
dprincemtaylor: One clarification. When I say 'in the PPA' I mean the actual deb file shows up in the Apt repo. I think I've been confusing you guys with the whole 'in the PPA' thing.17:14
mtaylordprince: no - totally. that's what I mean by in the PPA17:16
mtaylorDaviey: I can haz review? https://code.launchpad.net/~mordred/glance/add-kombu-dep/+merge/7140417:17
jtranmtaylor: I put in a review for you :)  not that it carries any weight17:18
jtranmtaylor: can you review me?17:19
jtranhttps://code.launchpad.net/~jtran/nova/lp817178/+merge/7019617:19
bcwaldonttx: ping ping ping17:20
kbringardjtran, I keep trying to answer you in the other channel, but you keep logging out17:20
jtrankbringard: my bad!17:20
kbringardno worries17:20
jtranit's the ocd17:20
*** lpatil has quit IRC17:23
*** darraghb has quit IRC17:24
*** dolph_ has quit IRC17:24
openstackjenkinsProject nova build #1,244: SUCCESS in 3 min 24 sec: https://jenkins.openstack.org/job/nova/1244/17:25
openstackjenkinsTarmac: Fixes to the OSAPI floating API extension DELETE.17:25
openstackjenkinsUpdated to use correct args for self.disassociate (don't sweep exceptions which should cause test cases to fail under the rug).17:25
openstackjenkinsAdditionally updated to pass network_api.release_floating_ip the address instead of a dict.17:25
*** amccabe has quit IRC17:25
*** amccabe has joined #openstack-dev17:39
*** lpatil has joined #openstack-dev17:42
*** HugoKuo1 has joined #openstack-dev17:44
*** HugoKuo has quit IRC17:45
*** YorikSar has joined #openstack-dev17:51
*** HugoKuo1 has left #openstack-dev17:54
jaypipesmtaylor: sorry, I had to run a couple chores...17:55
jaypipesreading back..17:55
heckjmtaylor: found a bug with trunk & milestone3 on python-manage when running Nova on Natty - https://bugs.launchpad.net/nova/+bug/82486618:02
uvirtbotLaunchpad bug 824866 in nova "nova-common (milestone3, trunk) error on installing on Ubuntu 11.04 Natty" [Undecided,New]18:02
heckjmtaylor: I wrote to the upstream maintainer, but wasn't sure how best to resolve other than that.18:02
mtaylorheckj: ok. I'll get 0.6.1 in the ppa18:05
jaypipesdprince, mtaylor: negative on custom version of webob. cactus requires 0.9.8 and diablo requires 1.0.8.18:05
jtrananyone care to review?:  https://code.launchpad.net/~jtran/nova/lp817178/+merge/7019618:06
heckjmtaylor: If there's something I can do with a package or helping, let me know - mostly lack of knowledge on how to submit a merge request18:06
mtaylorheckj: yeah - I'm working on fixing that process... but for now I'll just take care of it18:06
dprincejaypipes: So by custom I thought we just meant specific. If diablo requires 1.0.8 that should be in the PPA right?18:07
heckjmtaylor: thanks18:07
dprincegotta go...18:07
mtaylordprince: only if it's not already in the archive18:07
jaypipesdprince: ah, sorry, I thought you meant custom like we meant custom for eventlet (patched version of eventlet)18:07
jaypipesmtaylor: well, your answer is above. 0.9.8 for cactus and 1.0.8 for diablo :)18:08
jaypipeswebob, that is.18:08
*** mattray has joined #openstack-dev18:09
jaypipesVek: great job on the documentation of auth in Glance and shared image groups.18:09
jaypipesVek: doing full review now.18:10
*** dprince has quit IRC18:11
*** dolph_ has joined #openstack-dev18:11
*** dprince has joined #openstack-dev18:13
*** jtran has quit IRC18:31
*** mattray has quit IRC18:36
openstackgerritVerification of a change to openstack/glance failed: Add some documentation for notifications  https://review.openstack.org/20018:39
openstackgerritVerification of a change to openstack/glance failed: Add notifications for uploads, updates and deletes  https://review.openstack.org/19218:39
openstackgerritA change was merged to openstack/keystone: Fixed service-bound roles implementation in LDAP backend.  https://review.openstack.org/22218:49
openstackgerritA change was merged to openstack/keystone: Replacing tokens with the dummy tokens from sampledata.sh  https://review.openstack.org/22118:51
mtaylornotmyname: around?18:52
*** mattray has joined #openstack-dev18:53
*** dprince has quit IRC18:57
*** anotherjesse has quit IRC18:59
openstackjenkinsProject swift build #308: SUCCESS in 29 sec: https://jenkins.openstack.org/job/swift/308/19:02
vishyjaypipes: ping19:02
openstackjenkins* Tarmac: Updated the swift tool so that when uploading an object giving the full path, such as e.g: /etc/vim/vimrc , it will strip the initial "/" from the object name.19:02
openstackjenkins* Tarmac: Minor fix of quarantine naming19:02
jaypipesvishy: pung19:02
vishyso it looks like glance client requires gettext19:02
vishyException Value: name '_' is not defined Exception Location:/usr/lib/pymodules/python2.6/glance/common/exception.py in UnknownScheme, line 5919:03
creihtheh19:03
vishyjaypipes: so are we expected to install gettext prior to importing glance client?19:03
*** rnorwood has joined #openstack-dev19:08
*** dragondm has quit IRC19:09
notmynamemtaylor: yo19:10
mtaylornotmyname: I wanted to get your opinion on a gerrit config setting and/or workflow thing19:11
mtaylornotmyname: currently, we have it set to "merge if necessary" - so it tries to FF, and if that fails it merges - and of course if there is a conflict it vomits back at you19:12
notmynamemtaylor: what do you need?19:12
mtaylornotmyname: there is another setting we could choose, which is to have it cherry-pick out approved changes and apply them that way - pro: completely clean master history - con: you'd REALLY have to discard that topic branch after it's merged19:13
mtaylornotmyname: any thoughts or opinions on which approach is better?19:13
notmynamemtaylor: we abandon branches after they merge anyway. especially true, i'd think, if we have to rebase and squash to submit it19:14
mtaylornotmyname: that's sort of what I was thinking19:15
notmynamemtaylor: what's the cost of making the wrong choice now and changing later?19:15
mtaylornotmyname: nothing really19:15
mtaylornotmyname: it only applies for approved changes moving forward19:15
mtaylorand it's a radio-button ... so changing it is pretty easy :)19:16
notmynamemtaylor: practicality above all. sounds like the cherry-pick version would be a good start. if it gives us problems, we should be able to find something better19:16
redbolook, I'm not paying for a branch every time I commit code19:16
notmynameyes, branches in git are especially expensive. ;-)19:17
mtaylorhehe19:17
notmynamemtaylor: redbo: honestly I don't really care. whatever the team prefers is what I want19:18
jaypipesvishy: hmm...19:18
mtaylornotmyname: it's not a huge thing either way to me either - I was mainly wondering which thing felt more "right"19:18
jaypipescreiht: something funny?19:18
creihtjaypipes: was going to say something, but then decided it was best I didn't19:19
jaypipesvishy: I'll file a bug on it.19:19
jaypipesvishy: I assume your inclination is to say "no, gettext should not be required for th eclient..."19:20
notmynamemtaylor: just to confirm, it cherry-picks on merge, not on submit? it's not something crazy like you can't edit the branch while it's being reviewed?19:20
mtaylornotmyname: correct. only on merge19:20
mtaylornotmyname: on submit would be MADNESS! :)19:20
notmynamenot madness. this is openstack :-)19:20
jaypipes*sigh*19:21
mtaylornotmyname: :)19:21
notmynamemtaylor: so if merge == don't do any more work on that branch, I think we're ok with that19:21
notmyname(until we're not, and then we'll let you know) ;-)19:21
creihtlol19:21
mtaylork. thanks notmyname !19:21
notmynamemtaylor: what else do you need?19:21
mtaylornotmyname: that's it19:22
mtaylorand now ... burgers19:22
jaypipesbcwaldon: want to chat about the subresource vs. top-level resource question on image memberships?19:22
notmynamemtaylor: this may be getting close to ttx's date for "final release before diablo", but the code move would be good for us if we can do it on or after 8/2319:22
bcwaldonjaypipes: sure, skype?19:23
jaypipesbcwaldon: sure.19:23
*** bcwaldon has quit IRC19:47
*** jdurgin has quit IRC19:47
jaypipes_cerberus_: moved your comments to https://bugs.launchpad.net/glance/+bug/825419 and marked your bug as duplicate.19:59
uvirtbotLaunchpad bug 825419 in glance "Functional tests for private and shared images" [High,Confirmed]19:59
_cerberus_jaypipes: cool19:59
* ttx tries to pong a missing bcwaldon20:00
jaypipesttx: brian stepped out for a short while.20:00
jaypipesttx: anything I can help with?20:00
ttxjaypipes: he triple-pinged me earlier20:00
ttxno clue why though ;)20:00
jaypipesah20:01
*** mattray has quit IRC20:14
*** anotherjesse has joined #openstack-dev20:25
*** YorikSar has quit IRC20:29
*** anotherjesse_ has joined #openstack-dev20:30
*** YorikSar has joined #openstack-dev20:30
*** jtran has joined #openstack-dev20:31
openstackgerritJohannes Erdfelt proposed a change to openstack/glance: Add some documentation for notifications  https://review.openstack.org/20020:33
*** mfer has quit IRC20:33
*** anotherjesse has quit IRC20:33
*** anotherjesse_ is now known as anotherjesse20:33
*** mfer has joined #openstack-dev20:33
*** lorin1 has quit IRC20:35
*** ameade has quit IRC20:35
*** lpatil has left #openstack-dev20:35
jtrananyone know if openstack nova is now fully supported under python 2.7?20:37
*** ameade has joined #openstack-dev20:38
*** dragondm has joined #openstack-dev20:40
openstackgerritVerification of a change to openstack/glance failed: Add notifications for uploads, updates and deletes  https://review.openstack.org/19220:40
openstackgerritJohannes Erdfelt proposed a change to openstack/glance: Add notifications for uploads, updates and deletes  https://review.openstack.org/19220:40
*** mfer has quit IRC20:47
*** cp16net has quit IRC20:57
openstackgerritZiad Sawalha proposed a change to openstack/keystone: Doc changes  https://review.openstack.org/22320:57
*** AhmedSoliman has joined #openstack-dev20:58
*** anotherjesse has quit IRC20:59
*** anotherjesse has joined #openstack-dev20:59
*** bcwaldon has joined #openstack-dev21:00
jaypipesbcwaldon: ttx ponged you earlier...21:01
bcwaldonttx: around?21:01
vishymtaylor: ping21:05
vishyjaypipes: i would think that if the client needs it, it should install it when it is imported...21:05
vishyjaypipes: I think we're doing it in __init__.py in nova now21:06
openstackgerritJames E. Blair proposed a change to openstack/openstack-ci: Add a script to close Github pull requests.  https://review.openstack.org/22421:07
jaypipesvishy: https://bugs.launchpad.net/bugs/82549321:08
uvirtbotLaunchpad bug 825493 in glance "Glance client requires gettext module" [High,Triaged]21:08
anotherjessemtaylor: around?21:09
anotherjessemtaylor: wanted to talk about keystone packaging - if you are the person21:10
jaypipesVek: ping21:12
*** anotherjesse_ has joined #openstack-dev21:14
*** anotherjesse has quit IRC21:17
*** anotherjesse_ is now known as anotherjesse21:17
*** lts has quit IRC21:18
Vekpong...21:26
bcwaldonVek: jaypipes was asking for ye21:26
bcwaldonVek: but he appears to have stepped away (temporarily?)21:26
*** jdurgin has joined #openstack-dev21:27
jaypipesVek: no I'm here. :)21:27
jaypipesVek: so, I'm really not sure about the tenant and user thing... I was under the impression that the owner of an image was the user, not the tenant.21:28
jaypipesVek: could you explain how the tenant should be the owner? you mentioned backups, but I'm not following you...21:28
VekI use an "owner" abstraction; you could replace the context and *make* it use the user, if you wanted.21:28
*** rnirmal has quit IRC21:28
Vekbut the default context uses the tenant instead of the user21:29
jaypipesVek: yes, I understand that, but I'm asking *why* :)21:29
Vekanyway, as I understand it, the tenant is the charged entity, in terms of hosting situations like rackspace21:30
jaypipesVek: right, which is why it doesn't make sense to have rackspace own all the images. Then a user couldn't share their own image (backup or otherwise) with another user...21:30
Vekthey may have one or more users with access, but what they have access to is the images for that tenant, and those images shared with that tenant...21:30
Vekmake sense?21:33
*** kbringard has quit IRC21:33
*** jtran has quit IRC21:33
* Vek wonders if maybe he should have selected different tenant names in his docstrings to make this clearer...21:33
jaypipesVek: doesn't that mean that I, as a user of Rackspace Cloud, or some other public cloud, cannot share my own image with some other user?21:34
Vekwhy would that be?21:34
Vekthe tenant is the chargable entity.21:34
jaypipesVek: in other words, the way it is now is almost exclusively the way a service provider sets things up, but not how a private cloud sets things up...21:34
Vekif I'm an individual user klmitch, there'll be a tenant for me, let's say with the same name.21:35
jaypipesand by "way it is now" I mean your current merge proposal, sorry21:35
Vekthere'll likewise be one set up for you.21:35
Vekthen, if I want to share an image with you, I share it with your tenant21:35
jaypipesno, there won't. that's what I'm saying... in the service provider model, the project would all be "rackspace", no?21:35
Vekno21:36
jaypipesthen I'm completely lost21:36
Vekthe tenant is the chargable entity.21:36
jaypipesI don't see the point of a tenant and a user being distinct if they are always the same thing...21:36
Vekso individual users aren't charged21:36
Vekthat only applies for small hostees21:36
jaypipesbut individual users don't have the ability to share images with other individual users in this patch...21:37
Veklet's say we've got big_company as a tenant21:37
Vekthey may have several dozen users, each with their own specific roles to play21:37
Vekbut there's only one chargable entity21:37
Vekand the images belong to that one entity, the tenant; not to an individual user.21:37
jaypipeswhy not to an individual user, that's what I'm asking...21:38
Veklet's take the multi-user tenant case.  We don't want a circumstance where a disgruntled employee could block all access to a critical image; the image should be owned by the tenant.  (Of course, I'm glossing over what *other* things a rogue user could do, but...)21:39
Vekin the case where we have a single-user tenant, the user and tenant functionally act the same; they would probably by default have the same name, but that could probably be changed.21:40
*** martine_ has joined #openstack-dev21:41
jaypipesVek: we're going in circles here :) What I'm saying is that the way you've coded the functionality is biased towards a large hosting company, but people will want to install Glance on a server and have *individual* users be able to share images with each other, not just tenants sharing with each other..21:42
Vekand all it takes is overriding RequestContext, which can be done with an option in the paste .conf files21:43
jaypipesVek: hmm.21:43
jaypipesVek: that so, eh? :)21:43
Vekchange the 'owner' property to return self.user instead of self.tenant and you should be golden.21:44
*** martine has quit IRC21:44
Vekfor that matter, we could augment RequestContext to make that decision based on a configuration option21:44
jaypipesVek: hmm, ok, then that might be the solution I'm looking for... although it *might* be nicer to have21:45
jaypipesjinks. you typed it before I did.21:45
Vekhaha :)21:45
jaypipesyeah, I would prefer a config option :)21:45
jaypipesVek: and that would solve that major issue for me.21:45
jaypipesVek: something like user_can_share_images?21:45
Vekowner_is_user vs. owner_is_tenant ?21:46
jaypipesVek: and the default?21:46
VekI'll defer that to your judgement, oh PTL... :)21:46
jaypipesVek: I would go with: owner_is_tenant = True21:46
jaypipesVek: and in the comments for that say "Set to False to allow individual users to share images"21:47
jaypipesVek: ok with you?21:47
VekPerfectly fine by me :)21:47
jaypipesVek: rock.21:47
openstackgerritDolph Mathews proposed a change to openstack/keystone: Doc changes  https://review.openstack.org/22321:48
jaypipesVek: all the other comments looked to be addressed. bcwaldon addressed the versions question you had.21:48
*** jdurgin has quit IRC21:48
Vek*nod* I saw.  I need to go looking to understand what I need to modify how in order to satisfy him, though...21:48
*** martine_ has quit IRC21:49
Veks/him/you/, too :)21:49
jaypipesVek: about a 2-line addition to glance/api/versions.py :)21:49
jaypipesVek: just adding another version struct returned in the list of versions...21:49
Vekah, in version_objs ?21:49
jaypipesVek: yup!21:49
Vekhmmm...need to figure out the status, too...21:50
jaypipesVek: that allows clients to detect if a server supports 1.1 API (shared images and auth)21:50
Vekmake v1.1 status be CURRENT, but what should v1.0 be?  SUPPORTED?21:50
*** jdurgin has joined #openstack-dev21:50
jaypipesbcwaldon: what do you think? I think SUPPORTED is fine.21:50
jaypipesVek: meh, just make it SUPPORTED. We can always comment on the change request...21:51
bcwaldonyeah21:52
Vekoh, and what order should they be in?  v1.1 first or last?21:52
jaypipesOK, now I have to go build a friggin lamp for my wife... back in a while.21:52
bcwaldondoesn't matter21:52
jaypipesVek: I would think CURRENT first...21:52
Veklamps are trivial.  Try projection screen controllers sometime...21:53
Vek<--- son of an electrician21:53
* Vek vanishes again...21:56
*** rnorwood has quit IRC21:57
*** amccabe has quit IRC21:59
*** bsza has quit IRC22:02
*** bcwaldon has quit IRC22:09
openstackgerritJames E. Blair proposed a change to openstack/openstack-ci: Add a script to close Github pull requests.  https://review.openstack.org/22422:15
*** eday has quit IRC22:17
openstackgerritJames E. Blair proposed a change to openstack/openstack-ci-puppet: Add cron job for closing pull requests.  https://review.openstack.org/22522:20
Davieymtaylor: will look shortly, sorry - been travelling22:24
*** eday has joined #openstack-dev22:28
openstackgerritJustin Shepherd proposed a change to openstack/glance: Bug #825493  https://review.openstack.org/22622:33
uvirtbotLaunchpad bug 825493 in glance "Glance client requires gettext module" [High,Triaged] https://launchpad.net/bugs/82549322:33
*** rnorwood has joined #openstack-dev22:43
*** bsza has joined #openstack-dev22:44
openstackgerritDolph Mathews proposed a change to openstack/keystone: Added check_password to abstract backend user API  https://review.openstack.org/22722:47
openstackgerritKevin L. Mitchell proposed a change to openstack/glance: Add support for shared images  https://review.openstack.org/20122:49
*** mattray has joined #openstack-dev22:49
*** mattray has quit IRC22:49
*** mattray has joined #openstack-dev22:49
*** mattray1 has joined #openstack-dev22:52
*** vladimir3p has quit IRC22:52
*** mszilagyi_ has joined #openstack-dev22:53
*** bsza has quit IRC22:54
*** mattray has quit IRC22:54
*** mszilagyi has quit IRC22:55
*** mszilagyi_ is now known as mszilagyi22:55
*** mattray1 has quit IRC22:57
*** rnorwood has quit IRC22:58
*** mszilagyi_ has joined #openstack-dev23:04
*** mszilagyi has quit IRC23:07
*** mszilagyi_ is now known as mszilagyi23:07
*** anotherjesse has quit IRC23:07
*** anotherjesse has joined #openstack-dev23:07
*** dolph_ has quit IRC23:07
*** AhmedSoliman has quit IRC23:21
*** heckj has quit IRC23:36
*** jdurgin has quit IRC23:46
*** anotherjesse has quit IRC23:54
*** martine_ has joined #openstack-dev23:59

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