Wednesday, 2013-06-26

lbragstadjamielennox: yep, that sounds right. I believe it is within context switching00:01
*** galstrom is now known as galstrom_zzz00:01
lbragstadin the oslo-incubator code00:01
jamielennoxlbragstad: the ideal way would be to get oslo to stop assuming that eventlet is going to be present00:01
jamielennoxbut that's going to be a slow process00:02
lbragstadjamielennox: right, kinda what I was thinking00:02
*** zhuadl has joined #openstack-dev00:03
lbragstadjamielennox: specifically, https://github.com/openstack/oslo-incubator/blob/master/openstack/common/local.py#L2200:03
jamielennoxlbragstad: what that requires i've really got no idea, i like my environment separation but from a quick look through the RPC code it would be a pretty big change00:03
jamielennoxlbragstad: even looking at https://review.openstack.org/#/c/29803/5/keystone/openstack/common/rpc/amqp.py there is a lot of dependant stuff00:04
jamielennoxlbragstad: the other hack i've seen is to check if socket or os is patched and in which case import eventlet stuff00:05
lbragstadjamielennox: right, I just saw that. I know we have a blueprint in keystone for notifications as well. Which there doesn't seem to be a good way of getting around00:05
jamielennoxbut again looking at rpc they are specifically doing green thread pools00:05
lbragstadjamielennox: have there been other projects that have done that?00:05
jamielennoxnot as far as i'm aware, but there are a number of projects that are trying to rid themselves of eventlet00:06
jamielennoxswift i know is00:06
jamielennoxmind you i just grepped there source and there is eventlet scattered everywhere00:07
lbragstadkeystone source or other projects as well?00:08
jamielennoxas i said i was under the impression swift was trying to accomodate running without it but a quick look through the code shows a lot of hard dependencies on eventlet00:09
lbragstadmost of it in Keystone seems to be isolated in keystone/common/environment/*. I remember that going through00:11
jamielennoxlbragstad: yea, that was mine and that seperation is part of why adam -2ed the logging stuff00:11
*** markwash has quit IRC00:12
lbragstadright, the logging stuff would have introduced it into another place00:12
jamielennoxlbragstad: so i guess it would be interesting to know if you ignore the eventlet stuff of the oslo patch how much work is in writing our own RPC call00:13
jamielennoxlbragstad: i dont know what it would take, but again looking at rpc most of the complication seems to be related to using eventlet anyway00:13
jamielennoxif we are running synchronously then we don't need to worry about any thread pooling and return wait checking00:15
jamielennoxmaybe? interesting implications to that if you are doing a network call on every log00:15
*** epim has quit IRC00:16
jamielennoxno idea why they made a logging interface on top of a synchronous network call, that just seems dumb00:16
*** cliu has joined #openstack-dev00:16
lbragstadso at this point its worth it to consider writing our own unified logging implementation00:17
lbragstadand holding off on pulling in anything from oslo00:18
*** msmedved has left #openstack-dev00:18
*** msmedved has quit IRC00:18
lbragstadin which case we would either have to hold off on the notifier service as well, or implement our own00:18
jamielennoxlbragstad: i'm not sure, i don't claim to be an authority on this and i really don't know much about how the rpc works in oslo00:19
jamielennoxas we are only a sender and never a receiver or rpcs it might be easy to wrap simply the logging interface until there is some sort of eventlet consensus00:20
*** msmedved has joined #openstack-dev00:20
jamielennoxbut i've got no idea what this would take, there is also some talk about how keystone should be queing notifications about this like when a user is deleted00:21
jamielennoxso i guess if it can be interfaced fairly easily it's worth it, otherwise we have to suck it up and take eventlet back00:22
jamielennoxlbragstad: but how i became an authority on this i've got no idea :)00:22
lbragstadjamielennox: haha you'll have to talk to Adam on that one,00:23
lbragstadjamielennox: when you say interfaced you're talking about notification, logging or both?00:23
*** RajeshMohan has quit IRC00:24
jamielennoxlbragstad: so i'm not sure what is underpinning logging and notifications, i understand/think it is a synchronous rpc call, you would probably know better00:24
*** cliu has quit IRC00:25
jamielennoxlooking through oslo's rpc there is a lot of work with eventlet and format checking that is very generic00:26
*** RajeshMohan has joined #openstack-dev00:26
jamielennoxif all logging boils down to is call ampq with this json data then we can probably do that ourselves00:26
jamielennoxi think there is a fair bit of pressure to do notifications so that will probably have to happen soon anyway so we will need to either bring that in from oslo or look at it ourselves, but again i'm not really sure how that mechanism works00:28
lbragstadI think the problem with logging was it was using corolocal to switch between contexts00:28
lbragstadI don't seeing logging in oslo having deps on rpc or notifier00:28
lbragstadsee*00:29
*** portante has joined #openstack-dev00:30
*** msmedved has quit IRC00:30
jamielennoxok so i'm likely wrong and maybe the easiest way of handling this is just a hack in oslo that if eventlet isn't isntalled then don't use the corolocal stuff00:30
*** msmedved has joined #openstack-dev00:30
jamielennoxso i was looking at the failed review rather than the oslo code, why does brining in logging also bring in RPC?00:33
lbragstadwell, there are a couple ways to do it. After playing around with oslo and asking some questions you can either apply a module independently or try applying multiple oslo modules to a project at once00:34
lbragstadsometimes it is common to applie rpc and notifier in the same patch since notifier is dependent on rpc00:34
lbragstadI *think* in this case, if we fixed the context adapter issue with log.py https://github.com/openstack/oslo-incubator/blob/master/openstack/common/log.py#L258 in oslo then I think we could try and apply it to keystone separately instead of with other modules, like I did in the first patch. I figured I would bring them all in (rpc, log, local, notifier) at once since I was going to be stepping right into the notifications work right after I finishe00:36
jamielennoxyea it looks like 2 of the drivers rely on rpc00:36
*** bdpayne has quit IRC00:36
*** sleepsontheflo-1 has quit IRC00:37
*** msmedved has quit IRC00:37
jamielennoxright00:37
jamielennoxok00:37
jamielennoxSorry for talking about all the other stuff, it appears most of rpc can be irrelevant00:38
lbragstadafter having dims and markmc explain it to me, it sometimes goes by the content in which you're bringing things into a project, and what would happen if it needed to be backed out. No worries, it all needs to be addressed sooner or later00:38
*** ffio has quit IRC00:39
*** jasdeepH has quit IRC00:39
jamielennoxso what is the purpose of having a weak store here?00:39
*** cliu has joined #openstack-dev00:39
*** noslzzp has quit IRC00:39
*** ffio_ has joined #openstack-dev00:40
jamielennoxlbragstad: why is it doing this at all?00:41
lbragstadsomething pertaining to the scope of the context00:42
lbragstadI don't think I truely understood problem that was solved by this00:42
*** galstrom_zzz is now known as galstrom00:43
jamielennoxlbragstad: neither00:43
*** bdpayne has joined #openstack-dev00:44
jamielennoxmikal: if you are around, what is the purpose of the context in message logging? why do you need to use a corolocal?00:44
*** vipul is now known as vipul|away00:45
jamielennoxlbragstad: it would appear that it's just a way of storing weakrefs on eventlet00:45
openstackgerritA change was merged to openstack/ceilometer: api: enable v2 tests on SQLAlchemy & HBase  https://review.openstack.org/3393600:45
jamielennoxi guess that when you do green context switching if you have weakrefs it must mess with there scoping00:46
lbragstadmakes sense00:46
jamielennoxlbragstad: all that class would appear to be doing is storing the variable in a green thread local way to deal with the scoping00:46
jamielennoxso00:47
*** sthaha has joined #openstack-dev00:47
*** sthaha has joined #openstack-dev00:47
jamielennoxwhat i would suggest is to submit a patch to osol with a similar approach to what cms.py does in keystone00:47
*** yaguang has joined #openstack-dev00:47
lbragstadok00:48
lbragstadlooking00:48
jamielennoxcheck if we are running under eventlet, if we are then have the WeakLocal class inherit from corolocal00:48
jamielennoxotherwise define your own WeakLocal class which is simply an object with a __getattr__ stores weakrefs00:48
jamielennoxfrom memory there is a weakref dictionary that would make this really easy00:49
jamielennoxlbragstad: but try stepping through it because given that it would appear that context is simply a greenthread local global variable00:51
lbragstadok00:52
*** Mandell has quit IRC00:52
jamielennoxso it might work to simply replace it with a global context variable00:52
openstackgerritA change was merged to openstack/nova: Refresh volume connections when starting instances  https://review.openstack.org/3199200:53
*** cliu has quit IRC00:53
*** alexpilotti has quit IRC00:54
jamielennoxyea, local is  used in a number of places throughout oslo and it does appear to just be a way of doing a request specific global variable00:54
jamielennoxI would go the 'make your own weak_store and strong_store' route, that way it will fix all of them00:54
lbragstadok, then we don't have check for running on eventlet like in cms.py?00:55
jamielennoxyea you will, because this will be brought into other projects that will be using eventlet00:55
lbragstadahh ok00:55
*** vipul|away is now known as vipul00:55
jamielennoxthen i would just not import the RPC log handlers into keystone at all00:56
lbragstadjust see if we can get away with import log.py and local.py00:57
*** lloydde_ has quit IRC00:57
*** bknudson has quit IRC00:58
jamielennoxyea, so long as we are using oslo's logging interface then we can argue with them over how rpc works later00:58
jamielennoxthat'll be a tough fight and i would be trying to get others to have it00:59
lbragstadyeah, Nova is already using that implementation of notifier00:59
*** zhuadl has quit IRC00:59
*** Ryan_Lane has quit IRC01:03
openstackgerritA change was merged to openstack/nova: Add basic SecurityGroup model  https://review.openstack.org/3349401:04
openstackgerritA change was merged to openstack/nova: Add Instance.security_groups  https://review.openstack.org/3349501:04
openstackgerritA change was merged to openstack/nova: Make InstanceList filter non-column extra attributes  https://review.openstack.org/3370101:04
lbragstadjamielennox: one last question, where in cms.py do you actually detect eventlet? Or is that in environment?01:05
*** Ryan_Lane has joined #openstack-dev01:05
jamielennoxlbragstad: ah right because i would have converted that over to use environment, try this one: https://github.com/openstack/python-keystoneclient/blob/master/keystoneclient/common/cms.py01:06
*** SumitNaiksatam has joined #openstack-dev01:07
lbragstadjamielennox: ahh, _ensure_subprocess, makes sense, I was a little confused looking at https://github.com/openstack/keystone/blob/master/keystone/common/cms.py01:10
lbragstadthanks01:10
*** gyee has quit IRC01:10
jamielennoxlbragstad: before you go have a look at https://review.openstack.org/#/c/34484/ for me, it's breaking devstack so it will need to be fixed quickly01:11
lbragstadjamielennox: sure01:11
*** mikal has quit IRC01:13
lbragstadjamielennox: maybe a stupid question, but why from eventlet.green import httplib in keystone/middleware/ec2_token.py  and only import httplib in keystone/middleware/s3_token.py?01:13
jamielennoxno idea01:13
jamielennoxi reverted it back to its pre- environment state01:14
lbragstadgotcha, yeah looks good01:14
jamielennoxthere are so many inconsistencies with how eventlet is used which was part of the reason for  environment in the first place01:15
*** tjones has left #openstack-dev01:15
*** mikal has joined #openstack-dev01:15
jamielennoxi'll add a comment to explain that01:15
*** redbeard2 has joined #openstack-dev01:16
lbragstadYeah that makes sense,01:16
*** pabelanger has joined #openstack-dev01:18
*** comay has quit IRC01:18
*** tjones has joined #openstack-dev01:18
*** redbeard2 has quit IRC01:20
*** erkules_ has joined #openstack-dev01:21
*** ffio_ has quit IRC01:22
*** erkules has quit IRC01:23
*** tjones1 has joined #openstack-dev01:24
*** tjones1 has left #openstack-dev01:25
*** torandu has quit IRC01:26
*** tjones has quit IRC01:28
*** yuanz has quit IRC01:34
*** jasdeepH has joined #openstack-dev01:34
*** redbeard2 has joined #openstack-dev01:36
*** terry7 has quit IRC01:37
*** sungju has joined #openstack-dev01:40
*** galstrom is now known as galstrom_zzz01:40
*** bdpayne has quit IRC01:42
*** adalbas has quit IRC01:47
*** Tedster has quit IRC01:52
*** wenjianhn has joined #openstack-dev02:00
openstackgerritA change was merged to openstack/oslo-incubator: rpc: remove some unused serialization code  https://review.openstack.org/3431602:04
*** zigo has quit IRC02:09
*** zigo has joined #openstack-dev02:10
*** shang has joined #openstack-dev02:11
*** ctracey` has quit IRC02:11
*** noslzzp has joined #openstack-dev02:12
*** jbresnah has quit IRC02:12
*** ctracey|away has joined #openstack-dev02:13
*** ctracey|away is now known as ctracey02:13
openstackgerritA change was merged to openstack/oslo-incubator: Add IpFilter, IPNetnsExecFilter and EnvFilter  https://review.openstack.org/2963602:14
*** abhisri has joined #openstack-dev02:19
*** nati_ueno has quit IRC02:21
*** abhisri has quit IRC02:21
*** shang has quit IRC02:23
*** sdake has quit IRC02:23
*** bdpayne has joined #openstack-dev02:25
*** stevemar has joined #openstack-dev02:26
*** yanglyy has joined #openstack-dev02:30
*** portante_ has joined #openstack-dev02:30
*** portante has quit IRC02:34
*** matiu has quit IRC02:35
*** shang has joined #openstack-dev02:36
*** rcleere has joined #openstack-dev02:37
*** bing_bu has joined #openstack-dev02:42
*** jasdeepH has quit IRC02:43
*** melwitt has quit IRC02:43
*** jbresnah has joined #openstack-dev02:48
*** novas0x2a|laptop has quit IRC02:49
*** shang has quit IRC02:50
*** WindowsFanboi has joined #openstack-dev02:50
*** woodspa has quit IRC02:52
*** sleepsonthefloor has joined #openstack-dev02:53
*** cmark has joined #openstack-dev02:53
*** maheshp has joined #openstack-dev02:54
*** Tedster has joined #openstack-dev02:55
openstackgerritA change was merged to openstack/nova: Adds v3 API extension discovery filtering  https://review.openstack.org/3195302:56
openstackgerritA change was merged to openstack/nova: port agent API into v3 part1  https://review.openstack.org/3177102:57
jkyleis there an official or semi-official source for daily builds of ubuntu packages? (a ppa or something)03:01
*** ytwu has joined #openstack-dev03:02
*** shang has joined #openstack-dev03:03
*** ytwu has quit IRC03:04
clarkbjkyle: https://launchpad.net/~openstack-ubuntu-testing03:04
jkyleclarkb: thanks03:04
jkylegrizzly trunk would be 'master'?03:06
clarkbno havana is master03:08
*** leizhang has joined #openstack-dev03:08
jkylek, looking up repo policy. branch -a only showed master and stable/*03:09
jkyleI don't see a reference to testing branches, only milestones.03:11
*** markvoelker_ has quit IRC03:12
clarkbjkyle: we do all current development on master03:12
clarkbjkyle: once every six months a stable/* branch is forked off of master03:13
clarkbstable/havana will be the next branch forked off of master so https://launchpad.net/~openstack-ubuntu-testing/+archive/havana is the PPA to use to test current master03:13
*** litong has quit IRC03:15
*** timello_ has joined #openstack-dev03:16
jkylegot it, so what does the grizzly trunk represent? I'd like to get at the most current stable grizzly, though testing might be sufficient. I've noticed the ubuntu cloud archive debs are not current03:18
*** timello has quit IRC03:18
clarkbgrizzly is the current release. it gets periodic point releases which should end up on the cloud archive03:18
*** stevemar has quit IRC03:19
*** jasdeepH has joined #openstack-dev03:19
*** alexpilotti has joined #openstack-dev03:19
clarkbhttps://launchpad.net/~openstack-ubuntu-testing/+archive/grizzly-trunk-testing should be tracking the development between those point releases03:20
jkylegotcha03:20
jkylethat ought to work. the bug I just tracked was commited to stable/grizzly on may 10th. so probably need something tracking a bit better03:21
*** WindowsFanboi has quit IRC03:21
clarkbkeep in mind they are testing PPAs. I don't know how stable the grizzly test stuff is, but I know the one that tracks master can apparently be an adventure :)03:22
*** alexpilotti has quit IRC03:25
jkyleit may be more prudent to set up my own build server that tracks stable/grizzly03:29
*** tjones has joined #openstack-dev03:31
*** danjared has quit IRC03:34
*** danjared has joined #openstack-dev03:34
*** neelashah has joined #openstack-dev03:43
*** gongysh has joined #openstack-dev03:51
*** martine has joined #openstack-dev03:58
*** nati_ueno has joined #openstack-dev04:04
*** nati_ueno has quit IRC04:05
*** nati_ueno has joined #openstack-dev04:05
*** jbresnah has quit IRC04:07
*** bdpayne has quit IRC04:11
*** zhiyan has joined #openstack-dev04:11
*** martine_ has joined #openstack-dev04:14
*** cliu has joined #openstack-dev04:14
*** Aarti has joined #openstack-dev04:15
*** ffio has joined #openstack-dev04:16
*** ffio is now known as Guest1540804:17
zhiyanhi folks, i meet a strange issue about local variable scope issue, can somebody give a hand? ?04:17
*** prekarat has joined #openstack-dev04:17
*** Guest15408 has quit IRC04:18
lifelesszhiyan: just ask :)04:18
*** ffio_ has joined #openstack-dev04:18
*** martine has quit IRC04:18
*** ffio_ is now known as fire04:19
zhiyanyes, i have change the 'cinder/volume/manager.py' attach_host function to give some new function. add a new argument: 'host_name'. and run some testcase, but failed. the code like this:04:19
zhiyan    def attach_volume(self, context, volume_id, instance_uuid, host_name,04:19
zhiyan                      mountpoint):04:19
zhiyan        """Updates db to show volume is attached"""04:19
zhiyan        print "EXT VARS==>", locals()04:19
zhiyan        a=104:19
zhiyan        @utils.synchronized(volume_id, external=True)04:19
zhiyan        def do_attach():04:19
zhiyan            print "INT VARS==>", locals()04:19
zhiyanwhen i run the test case: './run_tests.sh cinder.tests.api.contrib.test_admin_actions.AdminActionsTest.test_attach_in_used_volume_by_host'04:20
zhiyani meet this situation:04:20
zhiyanEXT VARS==> {'instance_uuid': None, 'self': <cinder.volume.manager.VolumeManager object at 0x3c99c90>, 'host_name': 'fake_host', 'context': <cinder.context.RequestContext object at 0x3e4d7d0>, 'volume_id': 'fec306b5-cfbb-4041-b025-3b308c86e8a7', 'mountpoint': '/dev/vbd'}04:20
zhiyanINT VARS==> {'instance_uuid': None, 'mountpoint': '/dev/vbd', 'self': <cinder.volume.manager.VolumeManager object at 0x3c99c90>, 'context': <cinder.context.RequestContext object at 0x3e4d7d0>, 'volume_id': 'fec306b5-cfbb-4041-b025-3b308c86e8a7'}04:20
zhiyanso, 'host_name' argument lost in inner 'do_attach' function...it's strange to me....thanks04:21
zhiyanlifeless: can you give some input about this? thanks04:25
lifelesszhiyan: host_name isn't a local of do_attach.04:26
*** shang has quit IRC04:26
*** zaitcev has quit IRC04:26
lifelesszhiyan: it's in the closure, but not a local. Whats strange about that ?04:26
*** rcleere has quit IRC04:27
zhiyanlifeless: but other vars can be accessed in internal, like 'volume_id', 'mountpoint'...04:27
zhiyanand 'instance_uuid'..04:27
zhiyanwhy only 'host_name' can not be04:27
*** tjones has quit IRC04:27
*** afazekas has joined #openstack-dev04:29
*** vartom has joined #openstack-dev04:29
*** lloydde has joined #openstack-dev04:31
*** Mandell has joined #openstack-dev04:31
*** Hien has quit IRC04:32
*** tjones has joined #openstack-dev04:36
openstackgerritA change was merged to openstack/cinder: Update kombu requirement  https://review.openstack.org/3371704:39
openstackgerritA change was merged to openstack/swift: Local write affinity for object PUT requests.  https://review.openstack.org/3333304:39
*** tjones has left #openstack-dev04:42
*** Guest99955 has joined #openstack-dev04:44
*** benner has quit IRC04:44
*** martine_ has quit IRC04:44
*** martine has joined #openstack-dev04:45
*** maheshp has quit IRC04:46
*** Guest99955 has quit IRC04:47
*** Hien has joined #openstack-dev04:47
*** benner has joined #openstack-dev04:50
*** vartom has quit IRC04:50
*** koolhead11|away has joined #openstack-dev04:51
*** shang has joined #openstack-dev04:52
*** nati_uen_ has joined #openstack-dev04:53
*** nati_uen_ has joined #openstack-dev04:54
*** nati_ueno has quit IRC04:56
*** koolhead11|away is now known as koolhead1704:57
*** kaushikc has joined #openstack-dev04:58
*** Hien has quit IRC04:59
*** fire has quit IRC04:59
*** maheshp has joined #openstack-dev04:59
*** noslzzp has quit IRC05:00
*** ffio_ has joined #openstack-dev05:01
*** ffio_ is now known as fire05:01
*** Hien has joined #openstack-dev05:01
*** colinmcnamara has joined #openstack-dev05:01
*** neelashah has quit IRC05:02
*** jasdeepH has quit IRC05:04
*** abhishekkr_ has joined #openstack-dev05:07
*** alexxu has joined #openstack-dev05:10
*** SergeyLukjanov has joined #openstack-dev05:11
*** gongysh has quit IRC05:13
*** maheshp has quit IRC05:13
*** jasdeepH has joined #openstack-dev05:13
*** cmark has quit IRC05:15
*** martine has quit IRC05:18
lifelesszhiyan: it should be accessible, but not via locals()05:24
*** anfrolov has quit IRC05:25
*** colinmcnamara has quit IRC05:26
*** anfrolov has joined #openstack-dev05:27
*** cdub_ has quit IRC05:28
*** maheshp has joined #openstack-dev05:30
zhiyanlifeless: thanks for you response. i just resolved this issue :) . also, locals() can access it ('host_name').05:31
*** fire has quit IRC05:32
zhiyanthis situation seems because some python feature (py27 i used)05:35
*** kaushikc has quit IRC05:41
*** lloydde has quit IRC05:41
*** kaushikc has joined #openstack-dev05:41
*** giroro_ has quit IRC05:42
*** Ruetobas has joined #openstack-dev05:44
*** Ruetobas has joined #openstack-dev05:47
*** nati_ueno has joined #openstack-dev05:48
*** vartom has joined #openstack-dev05:48
*** nati_ueno has quit IRC05:49
*** nati_ueno has joined #openstack-dev05:51
*** nati_uen_ has quit IRC05:51
*** zhuadl has joined #openstack-dev05:51
*** devvesa has joined #openstack-dev05:52
*** nati_ueno has quit IRC05:52
*** jcoufal has joined #openstack-dev05:52
*** nati_ueno has joined #openstack-dev05:53
*** vartom has quit IRC05:53
*** zhuadl has quit IRC05:56
*** anfrolov has quit IRC05:59
*** jasdeepH has quit IRC06:01
*** jasdeepH has joined #openstack-dev06:01
*** lloydde has joined #openstack-dev06:02
*** zhuadl has joined #openstack-dev06:04
*** SergeyLukjanov has quit IRC06:04
*** kushal has joined #openstack-dev06:05
*** kushal has quit IRC06:08
*** cmark has joined #openstack-dev06:09
*** kushal has joined #openstack-dev06:09
*** cmark has joined #openstack-dev06:09
*** devvesa has quit IRC06:10
*** devvesa has joined #openstack-dev06:16
*** lsmola has joined #openstack-dev06:19
yanglyyDo you know, are there any document links of heat api?06:21
*** bashok has joined #openstack-dev06:23
*** aeperezt has quit IRC06:23
*** xqueralt-afk is now known as xqueralt06:24
*** maheshp has quit IRC06:25
*** cmark has quit IRC06:28
*** psedlak has joined #openstack-dev06:28
*** xga has joined #openstack-dev06:30
*** xga_ has joined #openstack-dev06:30
openstackgerritA change was merged to openstack/nova: Port quota API into v3 part2  https://review.openstack.org/3440106:30
*** egallen has joined #openstack-dev06:31
*** xga__ has joined #openstack-dev06:32
*** navid_ has joined #openstack-dev06:33
*** xga has quit IRC06:34
*** xga_ has quit IRC06:34
*** xga__ has quit IRC06:37
openstackgerritA change was merged to openstack/swift: Deleted account respond as non existing accounts  https://review.openstack.org/3256606:38
*** yanglyy has quit IRC06:39
*** maheshp has joined #openstack-dev06:40
*** xga has joined #openstack-dev06:41
*** yanglyy has joined #openstack-dev06:42
openstackgerritA change was merged to openstack/swift: updated changelog and authors update for 1.9.0 release  https://review.openstack.org/3449606:43
*** xga has quit IRC06:45
openstackgerritA change was merged to openstack/swift: version bump to 1.9.1  https://review.openstack.org/3449706:46
*** eglynn has joined #openstack-dev06:46
ianwzhiyan: it's because the variable isn't bound.  printing it or something like that would make it turn up in locals06:47
*** xga has joined #openstack-dev06:51
*** anfrolov has joined #openstack-dev06:51
*** kpavel has joined #openstack-dev06:52
*** odyssey4me has joined #openstack-dev06:53
*** Ryan_Lane has quit IRC06:54
*** eglynn has quit IRC06:54
*** Ryan_Lane has joined #openstack-dev06:54
*** vsergeyev has quit IRC06:55
zhiyanianw: thanks for you replay. but you see the 'host_name' not represent in the internal print output.06:55
*** abhishekkr_ has quit IRC06:57
*** torandu has joined #openstack-dev06:58
*** yolanda has joined #openstack-dev06:58
*** reidrac has joined #openstack-dev07:02
*** tkammer_ has joined #openstack-dev07:02
*** zoresvit has joined #openstack-dev07:07
*** mmagr has joined #openstack-dev07:08
*** xga has quit IRC07:08
*** timello_ has quit IRC07:09
*** timello_ has joined #openstack-dev07:10
*** gongysh has joined #openstack-dev07:12
*** cliu has quit IRC07:13
*** jtomasek has joined #openstack-dev07:13
gongyshgaryk: ping07:13
*** abhishekkr_ has joined #openstack-dev07:14
*** pnavarro has joined #openstack-dev07:15
*** xga has joined #openstack-dev07:16
*** egallen has quit IRC07:17
*** ffio has joined #openstack-dev07:18
*** egallen has joined #openstack-dev07:18
*** lloydde has quit IRC07:18
*** lloydde has joined #openstack-dev07:19
*** READ10 has joined #openstack-dev07:19
*** mrunge has joined #openstack-dev07:20
*** xga__ has joined #openstack-dev07:20
*** xga_ has joined #openstack-dev07:20
*** pasquier-s has joined #openstack-dev07:21
*** xga has quit IRC07:22
*** Alexei_987 has joined #openstack-dev07:23
*** koolhead17 has quit IRC07:23
*** sushils has joined #openstack-dev07:23
*** lloydde has quit IRC07:24
*** freedomhui has joined #openstack-dev07:24
*** ffio has quit IRC07:25
*** corXi has joined #openstack-dev07:27
*** kaushikc has quit IRC07:27
*** zhuadl has quit IRC07:29
*** kaushikc has joined #openstack-dev07:29
*** sthaha has quit IRC07:34
*** JordanP has joined #openstack-dev07:36
openstackgerritA change was merged to openstack/quantum: Update to use OSLO db  https://review.openstack.org/2726507:36
*** zhuadl has joined #openstack-dev07:37
*** Ryan_Lane has quit IRC07:37
*** jgallard has joined #openstack-dev07:43
*** Ryan_Lane has joined #openstack-dev07:44
*** maheshp has quit IRC07:44
*** iartarisi has joined #openstack-dev07:45
*** JordanP has quit IRC07:47
*** JordanP has joined #openstack-dev07:48
*** lloydde has joined #openstack-dev07:49
*** jgallard has quit IRC07:51
*** jgallard has joined #openstack-dev07:52
*** Ryan_Lane has quit IRC07:53
*** Ryan_Lane has joined #openstack-dev07:54
*** johnthetubaguy has joined #openstack-dev07:54
enikanorovwow. oslo db landed into quantum07:57
*** lloydde has quit IRC07:58
*** erkules_ is now known as erkules08:02
*** Aarti has quit IRC08:04
*** huats has joined #openstack-dev08:05
*** huats has quit IRC08:05
*** huats has joined #openstack-dev08:05
*** maheshp has joined #openstack-dev08:06
*** xga__ has quit IRC08:08
*** xga_ has quit IRC08:08
*** eglynn has joined #openstack-dev08:08
*** Mandell has quit IRC08:09
*** bashok has quit IRC08:10
*** bashok has joined #openstack-dev08:10
*** negronjl has quit IRC08:11
*** athomas has joined #openstack-dev08:11
*** lucasagomes has joined #openstack-dev08:12
*** huats has quit IRC08:12
*** huats has joined #openstack-dev08:13
*** huats has joined #openstack-dev08:13
*** Ryan_Lane has quit IRC08:13
*** xga has joined #openstack-dev08:16
*** xga_ has joined #openstack-dev08:16
*** Ryan_Lane has joined #openstack-dev08:16
*** abhishekkr__ has joined #openstack-dev08:17
*** derekh has joined #openstack-dev08:17
*** nati_ueno has quit IRC08:17
*** pixelbeat has joined #openstack-dev08:17
*** abhishekkr_ has quit IRC08:20
*** ffio has joined #openstack-dev08:22
*** afazekas_ has joined #openstack-dev08:23
*** afazekas_ has quit IRC08:23
*** afazekas_ has joined #openstack-dev08:24
*** kaushikc has quit IRC08:24
*** nati_ueno has joined #openstack-dev08:24
*** reed has joined #openstack-dev08:25
*** Ryan_Lane has quit IRC08:26
*** jclift has quit IRC08:29
*** negronjl has joined #openstack-dev08:31
*** jpich has joined #openstack-dev08:31
*** bswartz has quit IRC08:32
*** martyntaylor has joined #openstack-dev08:37
*** SergeyLukjanov has joined #openstack-dev08:49
*** jgallard has quit IRC08:49
*** jgallard has joined #openstack-dev08:49
*** lloydde has joined #openstack-dev08:49
*** salv-orlando_ has joined #openstack-dev08:54
*** salv-orlando has quit IRC08:54
*** salv-orlando_ is now known as salv-orlando08:54
*** lloydde has quit IRC08:54
*** mikal has quit IRC08:57
*** zodiak has quit IRC08:58
*** bashok has quit IRC08:58
*** mikal has joined #openstack-dev08:58
*** mkollaro has joined #openstack-dev09:02
*** bashok has joined #openstack-dev09:03
*** vartom has joined #openstack-dev09:04
*** romcheg has joined #openstack-dev09:06
*** WindowsFanboi has joined #openstack-dev09:07
*** bashok has quit IRC09:08
*** johnthetubaguy has quit IRC09:08
*** johnthetubaguy has joined #openstack-dev09:08
*** vartom has quit IRC09:09
*** bashok has joined #openstack-dev09:11
*** danpb has joined #openstack-dev09:11
*** xga has quit IRC09:16
*** xga_ has quit IRC09:16
*** nati_ueno has quit IRC09:17
*** mrunge has quit IRC09:21
*** romcheg has left #openstack-dev09:23
*** bing_bu has quit IRC09:24
*** athomas has quit IRC09:26
*** mrunge_ has joined #openstack-dev09:28
*** navid_ has quit IRC09:31
agorodnevlifeless: hi! =) I was trying to use MonkeyPatch from fixtures instead of ReplaceModule from my patch, but didn't have luck. It seems that MonkeyPatch doesn't work for modules. Any ideas?09:31
lifelessagorodnev: well, the main point was that you don't need to couple your thing to the test case.09:32
lifelessagorodnev: there are several ways you could do that.09:32
*** shang has quit IRC09:32
*** mrunge_ has quit IRC09:34
*** mrunge_ has joined #openstack-dev09:34
agorodnevlifeless:  ok =) I separated all that things from the testcase class. And I've read your article. Now I try to create more "concentrated" version of the patch09:36
openstackgerritA change was merged to openstack/horizon: Fix Further Pyflakes warnings  https://review.openstack.org/3138309:36
*** athomas has joined #openstack-dev09:37
openstackgerritA change was merged to openstack/horizon: python3: Introduce py33 to tox.ini  https://review.openstack.org/3139809:37
*** ffio has quit IRC09:38
*** boris-42 has quit IRC09:39
*** navid_ has joined #openstack-dev09:39
*** mihgen has joined #openstack-dev09:40
*** maheshp has quit IRC09:41
*** zhiyan has left #openstack-dev09:41
*** maheshp has joined #openstack-dev09:41
*** jgallard has quit IRC09:41
*** jgallard has joined #openstack-dev09:42
*** jgallard has quit IRC09:42
*** mrunge_ has quit IRC09:43
*** jgallard has joined #openstack-dev09:43
*** drusskikh has quit IRC09:43
*** lloydde has joined #openstack-dev09:50
*** alexisT has joined #openstack-dev09:51
*** dabo has joined #openstack-dev09:51
*** crazed has joined #openstack-dev09:51
*** briancline has joined #openstack-dev09:51
*** devx has joined #openstack-dev09:51
*** jjmb has joined #openstack-dev09:51
*** cyeoh has joined #openstack-dev09:52
*** lloydde has quit IRC09:55
*** READ10 has quit IRC09:55
*** READ10 has joined #openstack-dev09:59
*** yanglyy has quit IRC09:59
*** maheshp has quit IRC10:01
*** WindowsFanboi has quit IRC10:03
*** maheshp has joined #openstack-dev10:03
*** pcm___ has joined #openstack-dev10:04
*** pcm___ has quit IRC10:04
*** psedlak has quit IRC10:04
*** pcm__ has joined #openstack-dev10:05
*** markmc has joined #openstack-dev10:08
*** gmoro_ has joined #openstack-dev10:10
*** drusskikh_ has quit IRC10:13
*** mihgen has quit IRC10:13
*** fifieldt has joined #openstack-dev10:14
*** maheshp1 has joined #openstack-dev10:15
*** maheshp has quit IRC10:16
*** freedomhui has quit IRC10:17
*** shang has joined #openstack-dev10:18
*** martyntaylor has quit IRC10:20
*** yaguang has quit IRC10:21
*** xga_ has joined #openstack-dev10:25
*** xga has joined #openstack-dev10:25
*** romcheg1 has joined #openstack-dev10:26
*** zul has quit IRC10:27
*** zul has joined #openstack-dev10:29
*** maheshp1 has quit IRC10:32
*** psedlak has joined #openstack-dev10:33
*** leizhang has quit IRC10:42
*** martyntaylor has joined #openstack-dev10:48
*** gongysh has quit IRC10:50
*** lloydde has joined #openstack-dev10:50
*** martyntaylor has quit IRC10:52
*** martyntaylor has joined #openstack-dev10:53
*** jgallard has quit IRC10:53
*** portante_ is now known as portante10:53
*** rpodolyaka has joined #openstack-dev10:54
*** maheshp has joined #openstack-dev10:55
*** lloydde has quit IRC10:55
*** ffio has joined #openstack-dev10:58
*** timello_ has quit IRC10:58
*** timello_ has joined #openstack-dev10:59
*** pnavarro has quit IRC10:59
*** kaushikc has joined #openstack-dev11:01
*** sungju_ has joined #openstack-dev11:02
*** sungju_ has quit IRC11:02
*** ndipanov has quit IRC11:03
*** anfrolov has quit IRC11:09
*** anfrolov has joined #openstack-dev11:14
*** jruzicka has joined #openstack-dev11:14
*** zhuadl has quit IRC11:14
*** ifarkas has quit IRC11:16
*** ndipanov has joined #openstack-dev11:16
*** ifarkas has joined #openstack-dev11:16
*** bashok has quit IRC11:18
*** dkranz has quit IRC11:18
*** fifieldt has quit IRC11:19
*** HenryG has joined #openstack-dev11:20
*** prekarat has quit IRC11:20
*** romcheg has joined #openstack-dev11:21
*** boris-42 has joined #openstack-dev11:23
*** kaushikc has quit IRC11:24
*** romcheg1 has quit IRC11:25
*** sdake has joined #openstack-dev11:26
*** sdake has joined #openstack-dev11:26
*** maheshp has quit IRC11:28
*** maheshp has joined #openstack-dev11:28
*** jjmb has quit IRC11:29
*** rscottcoyle has joined #openstack-dev11:30
*** garyk has joined #openstack-dev11:30
*** abhishekkr__ has quit IRC11:31
*** jclift has joined #openstack-dev11:38
openstackgerritA change was merged to openstack/oslo-incubator: Make AMQP based RPC consumer threads more robust  https://review.openstack.org/3223511:40
*** abhishekkr__ has joined #openstack-dev11:40
*** eglynn has quit IRC11:40
*** eglynn has joined #openstack-dev11:41
openstackgerritA change was merged to openstack/oslo-incubator: Don't reconnect to exclusive fanout consumers.  https://review.openstack.org/3171811:41
*** vkmc has joined #openstack-dev11:41
*** vkmc has quit IRC11:41
*** vkmc has joined #openstack-dev11:41
openstackgerritA change was merged to openstack/oslo-incubator: Add network_utils.urlsplit  https://review.openstack.org/3363411:42
*** vkmc has quit IRC11:44
*** vkmc has joined #openstack-dev11:45
*** vkmc has quit IRC11:45
*** vkmc has joined #openstack-dev11:45
*** dkranz has joined #openstack-dev11:49
*** lloydde has joined #openstack-dev11:51
*** rnirmal has joined #openstack-dev11:51
openstackgerritA change was merged to openstack/oslo-incubator: Don't use mixture of cfg.Opt() deprecated args  https://review.openstack.org/3425911:54
*** tkammer_ has quit IRC11:55
*** jcoufal has quit IRC11:55
*** afazekas_ has quit IRC11:55
*** martine has joined #openstack-dev11:56
*** noslzzp has joined #openstack-dev11:59
*** eharney has joined #openstack-dev11:59
*** eharney has quit IRC11:59
*** eharney has joined #openstack-dev11:59
*** flaper87 has joined #openstack-dev11:59
*** drewlander has joined #openstack-dev12:01
*** psedlak_ has joined #openstack-dev12:01
*** mkollaro has quit IRC12:02
*** jcoufal has joined #openstack-dev12:02
*** mkollaro has joined #openstack-dev12:02
*** psedlak has quit IRC12:02
*** dprince has joined #openstack-dev12:05
*** lloydde has quit IRC12:05
*** freedomhui has joined #openstack-dev12:06
*** SergeyLukjanov has quit IRC12:07
*** lucasagomes is now known as lucas-hungry12:08
*** CaptTofu_ has quit IRC12:10
*** CaptTofu has joined #openstack-dev12:11
*** CaptTofu has quit IRC12:12
*** CaptTofu has joined #openstack-dev12:12
*** maheshp has quit IRC12:13
*** afazekas_ has joined #openstack-dev12:13
*** ndipanov has quit IRC12:13
*** maheshp has joined #openstack-dev12:14
zoresvitHi. According to https://wiki.openstack.org/wiki/UnifiedCLI/Mapping openstackclient should already support "user role list" command, but it's not: ERROR: cliff.app Unknown command ['user', 'role', 'add']12:14
zoresvitIs the doc outdated?12:14
*** galstrom_zzz is now known as galstrom12:15
*** SergeyLukjanov has joined #openstack-dev12:17
rpodolyakazoresvit: openstackclient is so mysterious that it's hard find the answer for your question even on openstack-dev channel :)12:17
*** rpodolyaka has quit IRC12:20
*** tkammer_ has joined #openstack-dev12:21
*** afazekas_ has quit IRC12:21
*** rpodolyaka has joined #openstack-dev12:23
*** maheshp has quit IRC12:24
garykmarkmc: i was chatting with HenryG and a question arose regarding the lowercase section names. would puppet, chef etc support this?12:24
*** ndipanov has joined #openstack-dev12:25
*** maheshp has joined #openstack-dev12:28
*** freedomhui has quit IRC12:29
*** maheshp has quit IRC12:29
*** morazi_ has quit IRC12:29
*** maheshp has joined #openstack-dev12:30
*** gordc has quit IRC12:33
*** zhuadl has joined #openstack-dev12:34
JordanPsdague, hi. A moment about the id vs UUID in tempest/quantum ? In fact the check that validate the identifier (whether it is a uuid or an id) is done at nova-api side (see api/OS/compute/contrib/security_groups*, which calls validate_id()). Quantum (nor the quantum-client) is not implied at all in the validation. It's just that the implementation of validate_id (in nova/network/security_group/quantum_drive.py for Quantum and nova/co12:37
JordanPmpute/api.py for Nova-Net)  is different12:37
JordanPso 2 weeks ago, you recommended to do some sort of exception translation not to raise the default/raw exception raised by quantum-client.  This recommendation doesn't stand, i think12:39
*** alexxu has quit IRC12:39
markmcgaryk, that's a very broad question12:39
markmcgaryk, the case we found yesterday was where a default value using the new name was taking precedence over a user-provided value using the deprecated name12:40
markmcgaryk, the fix for that is to make sure default values are commented out in default config files12:40
markmcgaryk, lowercase vs uppercase should work fine, it's actually more about the deprecated option names12:41
*** vsergeyev has joined #openstack-dev12:41
garykmarkmc: understood. gongysh is already in that task. thanks for the clarifications.12:41
*** pnavarro has joined #openstack-dev12:42
*** galstrom is now known as galstrom_zzz12:42
*** dhellmann-away is now known as dhellmann12:42
HenryGmarkmc: Thanks, looks like I should be fine then12:42
*** lbragstad has quit IRC12:43
*** sandywalsh has quit IRC12:45
*** SergeyLukjanov has quit IRC12:48
*** leizhang has joined #openstack-dev12:49
openstackgerritA change was merged to openstack/tempest: Remove dependency on MySQL-python.  https://review.openstack.org/3415212:51
openstackgerritA change was merged to openstack/glance: Do not raise NEW exceptions  https://review.openstack.org/3326712:51
*** noslzzp has quit IRC12:51
*** lloydde has joined #openstack-dev12:51
*** noslzzp has joined #openstack-dev12:56
*** lloydde has quit IRC12:56
*** jjmb has joined #openstack-dev12:58
*** gordc has joined #openstack-dev12:58
*** bing_bu has joined #openstack-dev12:58
*** kaushikc has joined #openstack-dev12:59
*** sandywalsh has joined #openstack-dev12:59
*** portante has quit IRC12:59
*** kaushikc has quit IRC13:00
sandywalshmarkmc, what's the scoop on oslo-config 1.2 ... I was hoping to merge oslo-common into ceilometer, but there's a 1.2 dependency in master it seems.13:00
*** kaushikc has joined #openstack-dev13:00
*** redbeard2 has quit IRC13:00
*** msmedved has joined #openstack-dev13:01
*** psedlak__ has joined #openstack-dev13:02
*** gargya has joined #openstack-dev13:02
*** bknudson has joined #openstack-dev13:03
*** litong has joined #openstack-dev13:03
*** psedlak_ has quit IRC13:03
*** mkollaro has quit IRC13:03
*** martine_ has joined #openstack-dev13:03
*** martine has quit IRC13:04
*** bdpayne has joined #openstack-dev13:05
*** tzumainn has joined #openstack-dev13:05
*** gongysh has joined #openstack-dev13:06
*** jtomasek has quit IRC13:06
*** jtomasek has joined #openstack-dev13:07
*** jergerber has joined #openstack-dev13:08
Alexei_987russellb: Hello, is there any list of extensions that should be ported to nova-v3-api? I would like to choose one to work on it.13:08
*** SergeyLukjanov has joined #openstack-dev13:09
*** maheshp has quit IRC13:09
*** rharwood has joined #openstack-dev13:10
*** ayoung has joined #openstack-dev13:11
*** lbragstad has joined #openstack-dev13:11
*** krtaylor has quit IRC13:12
openstackgerritA change was merged to openstack/tempest: Revert "Update a test to work with Quantum SecGroup"  https://review.openstack.org/3442813:13
openstackgerritA change was merged to openstack/glance: Import sql_connection option before using it.  https://review.openstack.org/3372113:13
openstackgerritA change was merged to openstack/tempest: Skip list_vifs test with Quantum  https://review.openstack.org/3443913:13
markmcsandywalsh, https://bugs.launchpad.net/oslo/+bug/1189889 should give you some pointers13:13
*** Aarti has joined #openstack-dev13:13
*** kbringard has joined #openstack-dev13:14
*** jonesld has joined #openstack-dev13:14
markmcsandywalsh, basically, include the full url in requirements.txt and make sure to require latest pbr13:14
*** olaph has joined #openstack-dev13:14
*** prad has joined #openstack-dev13:15
sandywalshmarkmc, looking ... thanks13:16
*** egallen has quit IRC13:16
sdagueJordanP: as long as nova returns the same status code for the same request regardless of whether nova-network or quantum is the backend, I don't really care what the intermediate implementation is. The important thing is the surface doesn't change when the backend changes.13:16
*** cp16net is now known as cp16net|away13:16
*** egallen has joined #openstack-dev13:16
*** egallen has quit IRC13:17
*** jjmb has quit IRC13:17
*** olaph_ has quit IRC13:18
*** jayg|g0n3 is now known as jayg13:19
JordanPsdague, so I think the only solution is to remove the validation, don't check whether the id (be it a regular ID or a uuid) is valid, because the moment you try to validate, as the validation method is different, you introduce a difference based on the backend13:19
*** gargya has quit IRC13:19
*** gargya has joined #openstack-dev13:20
*** joesavak has joined #openstack-dev13:20
*** dkranz has quit IRC13:21
sdagueJordanP: there is a difference between validating, and validating and returning the validation error result to the user. In nova net we're just returning 404 in an invalid id, right?13:22
JordanPno13:22
sdaguewe do this in the db layer already13:22
JordanPqe are returning BadRquest13:23
JordanPno it is done before the DB is even hit13:23
sdagueJordanP: link?13:23
JordanPa sec13:23
sdagueJordanP: no, I don't mean for this case, I mean in other cases13:23
sdaguewhere pg and mysql act differently on validation13:23
*** lucas-hungry is now known as lucasagomes13:24
*** afazekas_ has joined #openstack-dev13:25
*** rpodolyaka has left #openstack-dev13:25
JordanPi am not aware of what is done in for mysql vs db. In my case (uuid vs id) validation is done by nova-api before the call to the backend (nova net vs quantum) is even done : https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/contrib/security_groups.py#L25813:26
*** tkammer_ has quit IRC13:26
sdagueJordanP: ok, I have to go run to some meetings for the next 1.5 hrs, I'll look as soon as I get out13:27
*** abhishekkr__ has quit IRC13:27
JordanPok thanks13:27
*** AGiardini has joined #openstack-dev13:28
*** dolphm has joined #openstack-dev13:28
*** AGiardini has quit IRC13:28
*** neelashah has joined #openstack-dev13:28
*** tkammer has joined #openstack-dev13:29
openstackgerritA change was merged to openstack/tempest: Use os.path.join to form the cli command  https://review.openstack.org/3426513:30
*** dhellmann is now known as dhellmann-away13:31
*** yaguang has joined #openstack-dev13:31
*** yaguang has quit IRC13:34
zoresvitAlright, is there at least a way to validate policy.json file for keystone, anyone?13:34
*** afazekas_ has quit IRC13:34
dolphmzoresvit: validate against what?13:35
openstackgerritA change was merged to openstack/tempest: Skip test that is not relevant to Quantum  https://review.openstack.org/3440613:35
*** mkollaro has joined #openstack-dev13:35
zoresvitdolphm: validate that for example this syntax is meaningful for Keystone:13:36
zoresvit"admin_required": "(role:tenant_admin and tenant_id:%(target_tenant_id)s)"13:36
zoresvitBecause iven if I put an invalid json into policy.json file, I can't see any complains from Keystone whatsoever. So just wondered if there are any checks made on that.13:36
*** dkranz has joined #openstack-dev13:36
dolphmzoresvit: that should cause a backtrace as soon as it's parsed13:37
openstackgerritA change was merged to openstack/tempest: Skip fixed_ip_details test with Quantum  https://review.openstack.org/3443813:38
openstackgerritA change was merged to openstack/tempest: CLI doc missing colon for code block  https://review.openstack.org/3431113:40
bknudsonayoung dolphm: can I get a review of https://review.openstack.org/#/c/34475/ -- it should be quick because it re-does a fix we had earlier.13:40
dolphmbknudson: looking13:41
*** morazi has joined #openstack-dev13:41
zoresvitdolphm: Is that correct that to change policy, I modify /etc/keystone/policy.json file and than restart Keystone?13:41
bknudsonzoresvit: you shouldn't have to restart Keystone13:41
dolphmbknudson: does it re-read the policy file per request?13:42
bknudsondolphm: that's what I think happens.13:42
bknudsonI'll try it.13:42
ayoungbknudson, you move the code around, into a different function.  Why is that?13:43
zoresvitdolphm: okay, so it's read each time some action is performed, not on service startup. Got it, thanks :)13:44
bknudsonthe code in original at 872 didn't do anything. users.append(user_dn) couldn't raise UserNotFound13:44
bknudsonit's just adding a string to a list.13:44
*** jecarey_ has joined #openstack-dev13:44
bknudsonayoung: It's "self.user.get(user_id)" at line 373 in original that could raise UserNotFound13:45
ayoungbknudson, ah, so users.append(user_dn)  did nothing13:46
*** noslzzp has quit IRC13:48
bknudsonayoung: it doesn't check if the user exists, that's done when users.append(self.user.get(user_id))13:48
*** stevemar has joined #openstack-dev13:48
garykgongysh: ping13:48
*** andrewbogott_afk is now known as andrewbogott13:49
*** jjmb has joined #openstack-dev13:49
*** kevinconway has left #openstack-dev13:50
*** portante has joined #openstack-dev13:50
ayoungdolphm, one issue from last nights release meeting you should be aware of:13:50
ayoung21:28:04 <ayoung> but  are we OK with saying that all of the current clients change over to using keystoneclient as a dependency for auth?13:50
ayoung21:28:20 <ttx> ayoung: I think that would be a sane statement13:50
ayoungdolphm, a little background13:51
ayoungIs coming to you in an email13:51
*** krtaylor has joined #openstack-dev13:51
*** mtreinish has joined #openstack-dev13:51
*** lloydde has joined #openstack-dev13:52
*** noslzzp has joined #openstack-dev13:52
*** drewland_ has joined #openstack-dev13:53
russellbAlexei_987: talk to cyeoh, he's leading the v3 api effort13:54
russellbAlexei_987: oh nm, i see that you got the answer in another channel13:54
Alexei_987russellb: Yep, thanks13:55
*** jimfehlig has joined #openstack-dev13:55
*** drewlander has quit IRC13:55
*** kevinconway has joined #openstack-dev13:56
*** lloydde has quit IRC13:57
openstackgerritA change was merged to openstack/python-quantumclient: python3: Introduce py33 to tox.ini  https://review.openstack.org/3141013:58
dolphmayoung: what's the issue? that's just a goal lol13:59
*** rkukura has joined #openstack-dev13:59
ayoungdolphm, heh, still geting the email ready.  Launching it in 3. 2. 1.13:59
*** xga__ has joined #openstack-dev14:00
ayoungdolphm, it started when a couple people on my team started looking into doing additional forms of authentication to Nova and such.  For people looking for Kerberos and or X509 authentication.14:01
*** xga_ has quit IRC14:01
*** xga has quit IRC14:01
*** xga has joined #openstack-dev14:01
ayoungdolphm, Turns out, that making it work for Keystone is the easy part.14:01
*** garyk has quit IRC14:01
*** redbeard2 has joined #openstack-dev14:02
*** shang has quit IRC14:04
*** markmcclain has joined #openstack-dev14:06
*** mrodden has joined #openstack-dev14:07
*** shang has joined #openstack-dev14:09
*** odyssey4me has quit IRC14:10
*** cdub_ has joined #openstack-dev14:10
*** michchap has quit IRC14:12
*** zhuadl has quit IRC14:13
*** timello_ has quit IRC14:15
*** crobertsrh has joined #openstack-dev14:15
*** crobertsrh has left #openstack-dev14:15
*** timello_ has joined #openstack-dev14:15
*** tkammer has quit IRC14:15
*** pschaef has joined #openstack-dev14:16
*** tkammer has joined #openstack-dev14:16
*** aeperezt has joined #openstack-dev14:22
*** andrew_plunk has joined #openstack-dev14:22
*** rcleere has joined #openstack-dev14:23
*** zodiak has joined #openstack-dev14:23
sandywalshmarkmc, I have the link in requirements.txt and it seems to build the .tox env fine, but when the tests are running it complains it can't find the oslo module. http://paste.openstack.org/show/39250/ any ideas what I'm missing?14:26
markmcsandywalsh, there could be a problem here - apparently there was an issue with nova unit tests and tripleo are seeing an issue too14:27
markmcsandywalsh, could you confirm that 1.2.0a2 is actually installed in the venv?14:27
markmcsandywalsh, e.g.  pkg_resources.working_set.find(pkg_resources.Requirement.parse('oslo.config>=1.2.0a2'))14:27
*** dolphm has quit IRC14:28
sandywalshmarkmc, yep, sec14:28
markmcsandywalsh, after .tox/py27/bin/activate of course14:28
*** wenjianhn has quit IRC14:28
*** egallen has joined #openstack-dev14:28
*** adalbas has joined #openstack-dev14:29
*** kaushikc has quit IRC14:31
*** Aarti has quit IRC14:31
openstackgerritA change was merged to openstack/swift: Remove dead code and tests for staticweb.  https://review.openstack.org/3449914:32
*** Aarti has joined #openstack-dev14:32
*** dolphm has joined #openstack-dev14:32
sandywalshmarkmc, yes, it's there (along with 1.1 packages) http://paste.openstack.org/show/39254/14:32
markmcsandywalsh, ok, I've no idea why the unit tests are getting 1.1 then :(14:35
sandywalshmarkmc, ok, I'll dig deeper ... I'm sure there's some other CM dependency in there (perhaps nova or client) that's pulling it in14:36
*** woodspa has joined #openstack-dev14:37
markmcsandywalsh, I suspect it's some horrendous python packaging issue, like there not being .pth file to activate the egg or some other horror14:38
sandywalshmarkmc, I removed 1.1 and it's still complaining that it can't find 1.2.0a214:39
markmcsandywalsh, it's probably mordred's fault14:39
zoresvitProbably, final question, is there a way to implement "and" rule for Keystone policy? Is there a specific syntax for that? Thanks.14:39
sandywalshmarkmc, perhaps I'll take another tact and only pull over the oslo.rpc stuff and leave the DeprecatedOpt stuff out for now14:39
markmcsandywalsh, selective syncing should be what most people do IMHO https://wiki.openstack.org/wiki/Oslo#Syncing_Code_from_Incubator14:40
*** topol has joined #openstack-dev14:41
sandywalshmarkmc, cool ... I'll try that14:41
*** psedlak__ has quit IRC14:41
*** aelkikhia has joined #openstack-dev14:42
*** jtomasek has quit IRC14:42
*** jtomasek has joined #openstack-dev14:42
*** michchap has joined #openstack-dev14:42
*** kenperkins has quit IRC14:43
bknudsonzoresvit: in old-style, outer-list is ors, inner-lists are ands; empty string is "@".14:43
*** beagles is now known as beagles_brb14:44
*** carl_baldwin has joined #openstack-dev14:44
*** terryh has joined #openstack-dev14:45
*** spzala has joined #openstack-dev14:46
*** _TheDodd_ has joined #openstack-dev14:46
zoresvitbknudson: Thanks a lot for the tip, works!14:46
bknudsonzoresvit: not sure where this is documented... at one point I went through the code and wrote down how I thought it worked.14:47
*** vkmc has quit IRC14:47
zoresvitbknudson: Yeah, I had troubles finding the format.14:47
*** maheshp has joined #openstack-dev14:48
*** maheshp has quit IRC14:49
*** datsun180b has joined #openstack-dev14:49
*** maheshp has joined #openstack-dev14:49
*** FunnyLookinHat has joined #openstack-dev14:50
*** galstrom_zzz is now known as galstrom14:51
*** ifarkas has quit IRC14:51
*** romcheg has left #openstack-dev14:52
dolphmannegentle: is diane fleming on irc?14:52
*** michchap has quit IRC14:52
*** devvesa has quit IRC14:53
*** ffio_ has joined #openstack-dev14:53
*** ffio has quit IRC14:53
*** locke105 has quit IRC14:54
*** ifarkas has joined #openstack-dev14:54
*** mrodden has quit IRC14:54
*** xga_ has joined #openstack-dev14:55
*** aelkikhia has left #openstack-dev14:55
*** sneezezhang has joined #openstack-dev14:56
*** locke105 has joined #openstack-dev14:56
*** xga has quit IRC14:56
*** xga has joined #openstack-dev14:57
*** xga__ has quit IRC14:57
*** mkerrin has joined #openstack-dev14:57
*** leizhang has quit IRC14:58
openstackgerritA change was merged to openstack/python-swiftclient: Remove explicit distribute depend.  https://review.openstack.org/3260614:58
*** dhellmann-away is now known as dhellmann14:58
*** jvrbanac has joined #openstack-dev14:58
*** Samos123 has quit IRC14:59
*** Samos123 has joined #openstack-dev15:00
*** portante has quit IRC15:01
*** carl_baldwin has quit IRC15:02
spzaladolphm: good morning!15:02
*** sleepsonthefloor has quit IRC15:03
*** drewland_ has quit IRC15:03
*** drewlander has joined #openstack-dev15:04
sdagueJordanP: ok, finally back to https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/contrib/security_groups.py#L258, why doesn't that address the issue?15:04
sdaguebecause validate id is different on different implementations?15:04
JordanPyes15:05
*** psedlak has joined #openstack-dev15:05
JordanPone checks it's an ID, the other checks it's an UUID15:05
JordanPthus same input returns 2 differents results15:05
sdagueso that's sort of broken15:05
*** jasdeepH has joined #openstack-dev15:05
*** Samos123 has quit IRC15:06
JordanPi am not sure wa can call this broken, quatum decided to use UUID instead of ID, it's the origin of the issue15:06
*** reidrac has quit IRC15:07
sdagueJordanP: so when quantum does that UUID check, does it return an exception that we can ignore or translate?15:07
*** mrodden has joined #openstack-dev15:08
JordanPits not quantum that does the check, it's nova in :  nova/network/sec_group/quantum_driver.py15:08
JordanPbefore quantum (quantum server) is even called15:08
JordanPeuh no15:09
*** Samos123 has joined #openstack-dev15:09
JordanPthe validate_id() method is implemented in  nova/network/sec_group/quantum_driver.py for Quantum15:09
JordanPbut the check, ie the call to validate_id() is done by nova (see the github link you pasted)15:10
*** lloydde has joined #openstack-dev15:10
*** prekarat has joined #openstack-dev15:10
*** gargya has quit IRC15:12
*** jcoufal has quit IRC15:12
*** pmathews has joined #openstack-dev15:12
sdagueJordanP: so is the crux of the issue that we just need to remove the validate_id() code in nova for the quantum driver, and it's totally self inflicted because nova code is returning differently for different back ends?15:13
*** nati_ueno has joined #openstack-dev15:13
*** pmathews1 has joined #openstack-dev15:13
*** armax has joined #openstack-dev15:13
*** rcrit has quit IRC15:13
sdaguevalidate_id really seems like that needs to be nova generic code, and shouldn't be specific to secgroup implementation15:13
*** Samos123 has quit IRC15:14
*** jtomasek has quit IRC15:14
JordanPi am not sure I fully understand the sentence before the last one :) crux == heart ? and what do you mean by self-inflicted ?15:15
*** Samos123 has joined #openstack-dev15:15
*** otherwiseguy has joined #openstack-dev15:16
JordanPimo, validate_id can't be generic because the nature of an id depends on the backend. The validation should be done by the backend and not before (ie by the api)15:16
sdagueJordanP: that's the issue15:16
sdagueit really can't15:16
JordanPyep15:16
*** navid_ has quit IRC15:17
sdagueso a good instance on the db side15:17
*** pmathews has quit IRC15:17
*** iartarisi has quit IRC15:17
*** xga__ has joined #openstack-dev15:17
*** andrewbogott is now known as andrewbogott_afk15:17
sdaguemysql and pg have different max int lengths15:17
sdagueso it turns out that sending really big ints down act differently15:17
*** tkammer has quit IRC15:18
sdaguethey return different errors15:18
*** tkammer has joined #openstack-dev15:18
sdaguebut we turn them all back into 404s because we didn't want to enforce on the top side max length for those fields15:18
sdagueso I think in this case, it's time for the same approach15:18
_TheDodd_How do I modify associate my real name and email with my IRC nick? So that the information will show up for `whois`.15:19
*** bing_bu has quit IRC15:19
*** cliu has joined #openstack-dev15:19
*** michchap has joined #openstack-dev15:20
_TheDodd_* How do I associate my real name and email with my IRC nick? So that the information will show up for `whois`.15:20
*** maheshp has quit IRC15:20
lucasagomessandywalsh, I got stuck in this quantum problem as well, what seems to be causing it is the .pth file from the old version of quantum: http://fpaste.org/21056/15:20
sdaguehttps://github.com/openstack/nova/blob/master/nova/compute/api.py#L143615:20
lucasagomesold version of oslo*15:20
*** xga_ has quit IRC15:20
sdagueif you trace how we use InvalidID, you'll see it's thrown by a DataError exception in PG15:20
*** danwent has joined #openstack-dev15:20
*** xga has quit IRC15:20
JordanPyeah I believe i see how it works15:21
*** xga has joined #openstack-dev15:21
*** kaushikc has joined #openstack-dev15:22
_TheDodd_* How do I associate my real name and email with my IRC nick? So that the information will show up for `whois`.15:22
*** Samos123 has quit IRC15:23
*** kmartin has joined #openstack-dev15:25
JordanPsdague, in fact i don't see why the id validation is necessary, quantum deals fine with id that is not UUID15:25
*** kaushikc1 has joined #openstack-dev15:25
*** jasdeepH has quit IRC15:26
JordanPthe validate_id call in nova-api is just a way to bypass the call the to quantum backend, as an optimization15:26
sdagueJordanP: that's probably even a better solution then, just remove that15:26
*** rcrit has joined #openstack-dev15:26
*** jasdeepH has joined #openstack-dev15:27
sdagueyeh, that seems like the wrong place to put the optimization. I'd say just feed it to quantum, don't do the extra step, which is basically putting quantum policy in nova15:27
JordanPexactly !15:27
sdaguewill be interesting if that solves the issue, or there still needs to be a translation of errors to get them to line up15:27
sdaguewill look forward to the patch :)15:27
JordanPnova should not be aware of what is concidered valid by Quantum, and if quantum wants to validate data, it should do in in its own code15:28
*** _TheDodd_ has quit IRC15:28
*** ifarkas has quit IRC15:28
*** mmagr has quit IRC15:28
*** kaushikc1 has quit IRC15:28
*** kaushikc has quit IRC15:29
*** rwsu has joined #openstack-dev15:29
*** _TheDodd_ has joined #openstack-dev15:29
*** Ritz-Laptop has quit IRC15:29
bknudsondolphm: I finally got around to trying changing policy.json without restarting keystone. It does parse policy.json on every request.15:30
bknudson{"error": {"message": "An unexpected error prevented the server from fulfilling your request. Expecting , delimiter: line 29 column 35 (char 1272)", "code": 500, "title": "Internal Server Error"}}15:30
bknudsonI changed it so it's not valid.15:30
*** bing_bu has joined #openstack-dev15:31
*** michchap has quit IRC15:31
*** martyntaylor has quit IRC15:31
dolphmbknudson: finally got to your review, +2/approved15:33
bknudsondolphm: thank you15:33
mirrorboxlifeless: maybe you could share your thoughts on that subject in this bp: https://blueprints.launchpad.net/oslo/+spec/common-unit-tests ?15:33
mirrorboxerr15:34
dolphmbknudson: cool, i know i had seen policy.json syntax errors raised to the api, but wasn't sure if it reloaded on every request15:34
mirrorboxlifeless: sorry, please ignore15:34
openstackgerritA change was merged to openstack/python-keystoneclient: python3: Introduce py33 to tox.ini  https://review.openstack.org/3141115:34
bknudsondolphm: probably would be better if the text of the error wasn't sent to clients. Not sure if this is because of --debug.15:35
*** mihgen has joined #openstack-dev15:36
*** Ritz-Laptop has joined #openstack-dev15:37
dolphmbknudson: details of 500's aren't suppressed from the api in --debug15:38
dolphmbknudson: could make UnexpectedError extend SecurityError though to get that for free15:39
mordredmarkmc: I've sent a patch for the namespace_packages bug upstream, and have pinged the author directly to see if we can get a new release cut asap15:39
mordredmarkmc: https://github.com/iguananaut/d2to1/pull/32 if you want to track15:39
mordredmarkmc: if I don't hear back from him soonish, I'll see if I can sort a workaround in pbr itself15:40
*** anfrolov has quit IRC15:40
*** otherwiseguy has quit IRC15:40
*** Aarti has quit IRC15:41
*** maheshp has joined #openstack-dev15:42
openstackgerritA change was merged to openstack/nova: Refactor db.security_group_get() instance join behavior  https://review.openstack.org/3400015:42
*** corXi has quit IRC15:42
openstackgerritA change was merged to openstack/swift: Adding unit tests for direct client  https://review.openstack.org/3340515:42
*** Aarti has joined #openstack-dev15:42
openstackgerritA change was merged to openstack/oslo-incubator: Refactor KombuStubs fixture.  https://review.openstack.org/3380415:43
*** beagles_brb is now known as beagles15:43
*** wenjianhn has joined #openstack-dev15:45
*** bing_bu has quit IRC15:45
*** yidclare has joined #openstack-dev15:45
*** bing_bu has joined #openstack-dev15:45
*** matiu has joined #openstack-dev15:46
*** maheshp has quit IRC15:46
*** maheshp has joined #openstack-dev15:46
*** martyntaylor has joined #openstack-dev15:47
*** cliu has quit IRC15:48
*** Samos123 has joined #openstack-dev15:48
*** maheshp has quit IRC15:48
*** sandeepr has quit IRC15:49
JordanPsdague, I know you're against having different input (id or uuid) depending on the backend, in a Tempest test. But'is already the case in nova unit test : https://github.com/openstack/nova/blob/master/nova/tests/api/openstack/compute/contrib/test_security_groups.py#L12715:49
*** michchap has joined #openstack-dev15:50
*** sandeepr has joined #openstack-dev15:50
*** bing_bu has quit IRC15:50
markmcmordred, excellent, thanks15:50
*** michchap_ has joined #openstack-dev15:51
markmcmordred, proxying that to the bug15:51
*** Ritz-Laptop has quit IRC15:51
*** edmund has joined #openstack-dev15:51
*** pabelanger_ has joined #openstack-dev15:52
*** dprince has quit IRC15:52
mordredJordanP: yeah, but that's a unit test ... tempest is black-box testing what the api experience is15:52
*** michcha__ has joined #openstack-dev15:52
markmcmordred, I see you've some pull requests sitting there for some months :(15:52
*** enikanorov has quit IRC15:52
*** pabelanger_ has quit IRC15:52
*** pabelanger_ has joined #openstack-dev15:52
mordredJordanP: if the end user needs to know something about the implementation choice of the provider, something is broken in the interface15:52
mordredmarkmc: yeah, that's why I pinged him directly15:53
*** pabelanger has quit IRC15:53
markmcmordred, cool15:53
*** pabelanger_ is now known as pabelanger15:53
mordredmarkmc: I spoke with him when we were looking at using d2to1 about whether I should make pbr a fork of d2to1 or use it, and he said he was interested in continuing to maintain it as a library15:53
*** pabelanger_ has joined #openstack-dev15:53
*** pabelanger has quit IRC15:53
JordanPmordred, good point15:53
*** michch___ has joined #openstack-dev15:53
markmcmordred, ok15:53
*** pabelanger has joined #openstack-dev15:54
*** michchap has quit IRC15:54
mordredmarkmc: but, tbh, there isn't _that_ much code in there. If he goes awol, I'll just merge the functionality directly into pbr and be done with it15:54
markmcmordred, funny with this new github world, it's upload access to pypi being restricted to a single maintainer would be the bigger worry than commit access :)15:54
markmcmordred, yeah, that's an option too15:54
mordredmarkmc: I would prefer to find a path forward for sanity though15:55
*** michchap has joined #openstack-dev15:55
markmcmordred, yep15:55
mordredmarkmc: actually, you can add uploaders on pypi15:55
markmcmordred, right, you could ask him to add you :)15:55
*** sleepsonthefloor has joined #openstack-dev15:55
*** michchap_ has quit IRC15:55
*** michchap_ has joined #openstack-dev15:55
*** lloydde has quit IRC15:55
mordredmarkmc: just got a response, he's been on vacation for a few weeks and is working through post-vacation email backlog15:56
*** michcha__ has quit IRC15:56
mordredmarkmc: but he's going to bump that patch to the top of the list15:56
markmcmordred, great15:56
*** Alexei_987 has quit IRC15:56
*** redondos has quit IRC15:56
*** michcha__ has joined #openstack-dev15:57
ayoungdolphm, https://review.openstack.org/#/c/32510/  OK to merge?15:57
*** jgallard has joined #openstack-dev15:57
*** michch___ has quit IRC15:58
*** yidclare has quit IRC15:58
*** avishay has joined #openstack-dev15:58
*** michchap has quit IRC15:59
*** armax has left #openstack-dev15:59
*** electrichead has joined #openstack-dev16:00
*** jsavak has joined #openstack-dev16:00
*** michchap_ has quit IRC16:00
*** jdurgin1 has joined #openstack-dev16:00
*** michcha__ has quit IRC16:01
*** michchap has joined #openstack-dev16:01
*** Alla has joined #openstack-dev16:02
*** carl_baldwin has joined #openstack-dev16:03
*** michchap_ has joined #openstack-dev16:03
*** joesavak has quit IRC16:03
*** joesavak has joined #openstack-dev16:04
*** jasondotstar has joined #openstack-dev16:04
*** pcm__ has quit IRC16:04
*** jsavak has quit IRC16:05
*** michchap has quit IRC16:06
*** michchap_ has quit IRC16:07
*** garyk has joined #openstack-dev16:09
*** JordanP has quit IRC16:10
*** alop has joined #openstack-dev16:10
*** SumitNaiksatam has quit IRC16:11
*** devoid has joined #openstack-dev16:11
*** xga__ has quit IRC16:13
*** xga has quit IRC16:14
*** eglynn is now known as eglynn-afk16:14
sandywalshlucasagomes, thanks ... I'll try that (I wonder how we would patch that?)16:15
lucasagomessandywalsh, it's more like a workaround for now16:16
*** kbrierly has joined #openstack-dev16:17
markmclucasagomes, sorry, missed your comment earlier16:17
openstackgerritA change was merged to openstack/keystone: LDAP list group users not fail if user entry deleted  https://review.openstack.org/3447516:18
markmclucasagomes, we'll need to understand why the version in the venv doesn't override that16:18
markmclucasagomes, nasty16:18
*** psedlak has quit IRC16:18
markmcmordred, see this? http://fpaste.org/21056/16:18
lucasagomesmarkmc, yea it's pretty nasty :/16:18
markmcmordred, oslo.config 1.1 in /usr/lib, 1.2 in a venv, 1.1 somehow gets loaded16:18
markmcmordred, it's a --system-site-packages venv, but that shouldn't be an issue16:19
markmcmordred, presumably this was the problem with nova too16:19
lucasagomesin my case I don't even have the oslo-config 1.1 in my venv, the pth file making the venv to leak and use the system one16:20
markmcmordred, see https://review.openstack.org/33888 for the nova revert16:20
markmclucasagomes, yep16:20
markmcdansmith, got more details on that nova revert ?16:21
dansmithmarkmc: actually clarkb understood it more than I did16:21
markmcdansmith, ok16:21
dansmithmarkmc: but it was breaking tox runs for everyone, but working in CI16:21
markmcclarkb, more details ?16:21
*** markwash has joined #openstack-dev16:21
mordredyeek. how the heck does it work in CI but break locally?16:22
* markmc thinks we need a hadoop based irc -> launchpad proxy16:22
markmcit's a Big Data problem16:22
dansmithmordred: yeah, it was scary.. I think he said some sort of test ordering thing was making it pass erroneously16:22
russellbmarkmc: wat16:22
markmcuseful info on irc, need to extract it into launchpad16:22
*** lloydde has joined #openstack-dev16:22
*** electrichead has quit IRC16:22
mordredmarkmc: is it in scrollback?16:22
*** _TheDodd_ has quit IRC16:23
markmcmordred, no idea, I only saw the revert commit days after it was proposed16:23
dansmithit was in infra, and a week ago16:23
* markmc has no context other than "nova broke"16:23
mordredoy. I'll never find that in my scrollback16:23
markmcwhich isn't in eavesdrop either16:23
*** nati_ueno has quit IRC16:23
russellbwas also in -nova16:23
russellb(not logged either afaik)16:23
mordredI'll see if clarkb can fill me in on his understanding when he gets back online16:23
mordredwe should maybe log more channels16:23
markmccan we file bugs people, please? :)16:23
* markmc pleads16:23
russellbi don't remember, people were just like "omg it broke and if i revert it works, let's revert and we'll figure it out"16:23
*** jruzicka has quit IRC16:24
markmcrussellb, I'm totally down with that16:24
dansmithmarkmc: yeah, my bad, sorry, it was right after merge, and major breakage16:24
russellbwith a bug report?  :)16:24
markmcdansmith, anything you can dig up from scrollback or whatever would be great16:24
russellbre-apply patch and see what happens now?16:25
*** otherwiseguy has joined #openstack-dev16:25
markmcrussellb, yeah16:25
dansmithwell,16:25
dansmiththere was some analysis by clarkb16:25
dansmithI'll get some logs, just a sec16:25
markmcrussellb, in other words, "STFU and reproduce it yourself" :)16:25
russellbi meant that to dansmith16:25
*** electrichead has joined #openstack-dev16:25
*** xqueralt is now known as xqueralt-afk16:25
markmcrussellb, heh, ok16:25
* markmc can just reproduce it16:26
*** martyntaylor has quit IRC16:26
*** markwash_ has joined #openstack-dev16:27
*** garyk has quit IRC16:27
*** mkollaro has quit IRC16:28
*** boris-42 has quit IRC16:28
*** anfrolov has joined #openstack-dev16:29
dansmithmarkmc: http://paste.openstack.org/show/39258/16:29
dansmithmarkmc: that's from when I started the conversation through the end of the day16:29
*** markwash has quit IRC16:29
*** markwash_ is now known as markwash16:29
markmcdansmith, heh, ok - thanks16:29
*** kenperkins has joined #openstack-dev16:30
markmc<clarkb>this is my gripe with oslo.config the way we use it makes import order super important rather than putting the config stuff right at the beginning of the import order we mix it around16:30
bknudsonayoung: would be nice to get https://review.openstack.org/#/c/33102/ merged with https://review.openstack.org/#/c/32510/ , since both are fixing MyISAM.16:30
*** anfrolov has quit IRC16:30
* markmc can't parse that16:31
*** tkammer has quit IRC16:31
markmc<clarkb>geekinutah: instead this is a test run order conflict. run tests in one order and they pass. Run them in a different order and they fail16:31
markmcthat doesn't sound right16:31
*** Guest32596 is now known as med_16:32
*** med_ has quit IRC16:32
*** med_ has joined #openstack-dev16:32
*** gyee has joined #openstack-dev16:32
*** shang_ has joined #openstack-dev16:32
*** epim has joined #openstack-dev16:33
*** jgallard has quit IRC16:34
*** galstrom is now known as galstrom_zzz16:34
*** shang has quit IRC16:36
*** Aarti has quit IRC16:36
*** jcoufal has joined #openstack-dev16:36
*** michchap has joined #openstack-dev16:36
*** Aarti has joined #openstack-dev16:36
*** SergeyLukjanov has quit IRC16:36
*** eglynn-afk is now known as eglynn16:37
*** derekh has quit IRC16:37
*** leizhang has joined #openstack-dev16:37
*** galstrom_zzz is now known as galstrom16:37
*** sulrich has joined #openstack-dev16:38
*** Mandell has joined #openstack-dev16:38
*** dprince has joined #openstack-dev16:38
mordredmarkmc: I wonder if the problem is related to the namespace package problem16:39
*** martyntaylor has joined #openstack-dev16:39
*** gongysh has quit IRC16:39
*** martyntaylor has quit IRC16:39
*** sneezezhang has quit IRC16:39
*** SumitNaiksatam has joined #openstack-dev16:39
*** martyntaylor has joined #openstack-dev16:39
markmcmordred, maybe, can't really see how though16:40
markmcmordred, if it was the other way around, maybe - i.e. if there was a oslo/__init__.py in /usr/lib16:40
mordredyeah. good point16:40
markmcmordred, anything is possible with this stuff though16:40
mordredyup. also, the nova site-packages/libvirt bit makes the isolation we shoot for harder16:41
*** egallen has quit IRC16:42
*** colinmcnamara has joined #openstack-dev16:43
*** ajyounge_ has joined #openstack-dev16:43
*** colinmcnamara has quit IRC16:45
*** garyk has joined #openstack-dev16:45
*** _TheDodd_ has joined #openstack-dev16:45
*** michchap has quit IRC16:48
icchaannegentle: any leads on glance documentation ? (based on our previous conversation)16:49
*** freedomhui has joined #openstack-dev16:51
*** jyhc has joined #openstack-dev16:53
*** freedomhui has quit IRC16:54
*** romcheg has joined #openstack-dev16:55
*** nati_ueno has joined #openstack-dev16:55
sandywalshmarkmc, the update.py file seems to want to copy everything all the time. I've moved oslo-common.conf and issued http://paste.openstack.org/show/39260/, but it replaces all modules.16:56
sandywalshmarkmc, is there a different incantation I should be using?16:56
markmcsandywalsh, ugh, try --nodeps16:57
*** devoid has quit IRC16:57
* markmc knew that feature was a bad idea16:57
sandywalshmarkmc, thanks16:57
*** kushal has quit IRC17:03
*** jyhc has quit IRC17:06
*** comay has joined #openstack-dev17:09
*** lucasagomes has quit IRC17:09
*** mikal has quit IRC17:09
*** danpb has quit IRC17:10
*** mikal has joined #openstack-dev17:11
*** athomas has quit IRC17:11
*** esheffield1 has joined #openstack-dev17:13
*** terriyu has joined #openstack-dev17:13
*** esheffield has quit IRC17:14
*** michchap has joined #openstack-dev17:15
*** athomas has joined #openstack-dev17:17
*** wenjianhn has quit IRC17:17
mordredmarkmc: speaking of ...17:17
mordredmarkmc: when pip 1.4 comes out17:17
mordredmarkmc: I'd like to have a discussion with you about re-thinking a few things that we do17:17
mordredmarkmc: because I think the machinery will be different/better17:18
mordredmarkmc: it might not be possible to do _anything_ differently- but I think it's worth a mental exercise17:18
*** rkukura has quit IRC17:18
mordredmarkmc: (this is all around the new behavior of not installing pre-release things unless you explicitly ask for them)17:19
*** athomas has quit IRC17:20
*** pschaef has quit IRC17:20
*** xBytez has quit IRC17:21
*** jyhc has joined #openstack-dev17:21
*** jyhc has quit IRC17:21
*** jyhc has joined #openstack-dev17:21
*** sandywalsh has quit IRC17:21
*** sarob has joined #openstack-dev17:22
openstackgerritA change was merged to openstack/swift: Change ring partition size for SAIO  https://review.openstack.org/3457417:22
*** andrewbogott_afk is now known as andrewbogott17:22
*** sarob has quit IRC17:23
*** sarob has joined #openstack-dev17:23
*** jbresnah has joined #openstack-dev17:24
*** sarob has quit IRC17:24
*** sarob has joined #openstack-dev17:25
*** Aarti has quit IRC17:27
*** michchap has quit IRC17:27
*** Farooque has joined #openstack-dev17:27
*** sarob_ has joined #openstack-dev17:27
*** rods has joined #openstack-dev17:28
*** xBytez has joined #openstack-dev17:30
*** sarob__ has joined #openstack-dev17:30
*** ssurana has joined #openstack-dev17:31
*** sarob has quit IRC17:31
*** sarob_ has quit IRC17:31
*** kushal has joined #openstack-dev17:32
*** prekarat1 has joined #openstack-dev17:33
*** prekarat has quit IRC17:34
mordredmarkmc: also - workaround for namespace package issue until a fix is in to d2to1:17:35
mordredmarkmc: add namespace_packages = ['oslo'], into setup.py directly17:36
mordredmarkmc: it's not how we want to do that- but it will work until a new fix is released upstream17:36
*** mihgen has quit IRC17:36
*** GheRiver1 is now known as GheRivero17:38
*** sandywalsh has joined #openstack-dev17:38
*** johnthetubaguy has quit IRC17:38
*** sgordon has quit IRC17:38
*** sgordon has joined #openstack-dev17:38
*** prekarat1 has quit IRC17:39
*** zaitcev has joined #openstack-dev17:39
*** Ryan_Lane has joined #openstack-dev17:41
lbragstadmikal: jamielennox and I were discussing earlier what the purpose of the context being stored in a WeakLocal object was? We are trying to implement a unified logging solution into keystone from Oslo-incubator but Keystone is hesitant on bringing in dependencies on eventlet (corolocal.local). Just wondering if you had a take on what we could do on this?17:42
*** Ryan_Lane1 has joined #openstack-dev17:43
*** tmclaugh[work] has joined #openstack-dev17:43
*** Ryan_Lane1 has quit IRC17:44
*** Ryan_Lane1 has joined #openstack-dev17:44
*** Ryan_Lane has quit IRC17:44
*** Ryan_Lane1 is now known as Ryan_Lane17:44
*** rnirmal has quit IRC17:44
*** sandywalsh has quit IRC17:44
*** ndipanov is now known as ndipanov_gone17:45
*** otherwiseguy has quit IRC17:46
*** xmltok has joined #openstack-dev17:48
*** xmltok has quit IRC17:49
*** Tross has quit IRC17:50
*** xmltok_ has quit IRC17:50
*** galstrom is now known as galstrom_zzz17:50
*** jcoufal has quit IRC17:50
*** xmltok has joined #openstack-dev17:50
*** michchap has joined #openstack-dev17:53
*** romcheg has left #openstack-dev17:54
*** mikal has quit IRC17:55
*** mrodden has quit IRC17:56
*** mikal has joined #openstack-dev17:57
*** pcm___ has joined #openstack-dev17:58
*** sandywalsh has joined #openstack-dev17:58
*** pcm___ has quit IRC17:58
*** timello_ has quit IRC17:58
*** pcm__ has joined #openstack-dev17:58
*** timello_ has joined #openstack-dev17:59
*** lbragstad has quit IRC17:59
*** mrodden has joined #openstack-dev17:59
*** sdake has quit IRC17:59
*** ydacheville has joined #openstack-dev18:00
*** lbragstad has joined #openstack-dev18:00
*** ydacheville_ has quit IRC18:04
*** esheffield1 has quit IRC18:05
*** michchap has quit IRC18:06
*** _TheDodd_ has quit IRC18:06
*** Mandell has quit IRC18:07
*** _TheDodd_ has joined #openstack-dev18:09
*** otherwiseguy has joined #openstack-dev18:11
*** anfrolov has joined #openstack-dev18:13
*** anniec has joined #openstack-dev18:14
*** ctlaugh has quit IRC18:14
*** jpich has quit IRC18:16
*** redondos has joined #openstack-dev18:16
*** Farooque has left #openstack-dev18:16
*** sulrich has quit IRC18:17
openstackgerritA change was merged to openstack/cinder: Content-length missing in put_object  https://review.openstack.org/3363918:17
openstackgerritA change was merged to openstack/nova: Add basic InstanceFault model  https://review.openstack.org/3355118:17
openstackgerritA change was merged to openstack/nova: Add Instance.fault  https://review.openstack.org/3355218:18
*** sulrich has joined #openstack-dev18:18
openstackgerritA change was merged to openstack/nova: Make NovaObject support the 'in' operator  https://review.openstack.org/3387118:18
*** cp16net|away is now known as cp16net18:19
*** sarob has joined #openstack-dev18:19
*** Mandell has joined #openstack-dev18:20
*** cliu has joined #openstack-dev18:22
*** otherwiseguy has quit IRC18:22
*** sarob__ has quit IRC18:22
*** sushils has quit IRC18:23
*** dhellmann has quit IRC18:23
*** epim has quit IRC18:25
pcm__nati_ueno: Hi! Do you have time for a few questions on VPNaaS?18:25
nati_uenopcm__: absolutely :)18:25
openstackgerritA change was merged to openstack/nova: Fix instance object's use of a db query method parameter  https://review.openstack.org/3387218:26
pcm__nati_ueno: I set up two devstack nodes on bare metal. Have public net connection. Trying to setup connection18:26
nati_uenopcm__: Ah sorry, please use latest patch. I did same thing yesterday, and I found one bug. But newest patch works18:27
openstackgerritA change was merged to openstack/nova: Make info_cache handle when network_info is None  https://review.openstack.org/3387318:27
pcm__nati_ueno: Q: I was looking to see if config files are created. Doesn't look like we set, so default should be /var/lib/quantum/strongswan/, right?18:27
nati_uenopcm__: /opt/stack/data/quantum/strongswan is default for devestack18:27
*** galstrom_zzz is now known as galstrom18:28
pcm__nati_ueno: Oh, where does that get set?18:28
nati_uenopcm__: In devstack, it setup state_path.18:28
nati_uenopcm__: strongswan conf goes unser the state_path18:28
nati_uenos/unser/under/18:29
*** SumitNaiksatam has quit IRC18:29
pcm__nati_ueno: checking...18:29
nati_uenopcm__: here https://github.com/openstack-dev/devstack/blob/master/lib/quantum#L46318:30
pcm__nati_ueno: Yes. I see the directory - that solves one question.18:31
pcm__I did ikepolicy, ipsecpolicy, service and connection create on each side.18:31
nati_uenopcm__: ok18:31
*** michchap has joined #openstack-dev18:32
pcm__nati_ueno: I see entries with UUID of router.18:32
vishymarkmcclain, arosen: https://review.openstack.org/#/c/34589/18:33
*** jculp_ has joined #openstack-dev18:33
*** jculp_ has quit IRC18:33
vishymarkmcclain, arosen: i initially added that for compatibility with our nova-network install. It turns out that it isn't a show-stopper, but I think the patch still makes sense.18:33
pcm__nati_ueno: Just checked and it looks like service is ACTIVE18:34
*** markmc has quit IRC18:34
pcm__nati_ueno: However, the connection is saying "PENDING_CREATE"18:35
pcm__nati_ueno: Am I missing some step?18:35
nati_uenopcm__: That's part is not implemented yet.18:35
pcm__ah.18:36
nati_uenopcm__: I mentioned it in the commit message18:36
nati_uenopcm__: so We need to update connection status based on the ipsec status command18:36
pcm__OK. Didn't read the message closely. Sorry.18:36
nati_uenopcm__: Noop. The problem is connection statte could be dynamically changed18:36
nati_uenopcm__: so pediatric check might be needed18:37
*** novas0x2a|laptop has joined #openstack-dev18:37
*** diogogmt has joined #openstack-dev18:37
nati_uenopcm__: current patch is about 1k, so I thought it good to separate for another patch18:37
markmcclainvishy: looking18:38
vishymarkmcclain: I didn't think it was big enough for a blueprint and I'm not sure it is really a "bug", so i just went for straight patch on this one18:39
pcm__nati_ueno: Can I manually monitor/access the ipsec status?18:39
*** epim has joined #openstack-dev18:39
*** otherwiseguy has joined #openstack-dev18:39
nati_uenopcm__: yes. so please go to $state_path/quantum/strongswan/$ROUTER_ID/18:40
nati_uenothen18:40
nati_uenosudo ip netns exec qrouter-$ROUTER_ID18:40
nati_uenosorry typo18:40
nati_uenosudo ip netns exec qrouter-$ROUTER_ID bash18:40
nati_uenothen18:40
nati_uenoquantum-vpn-nswrap `pwd` ipsec status18:40
pcm__nati_ueno: Ah, so that runs the wrapper to run the ipsec command under the processes area. Saw that in the wiki, but wasn't sure how it worked.18:42
pcm__nati_ueno: Cool! I'll give it a try.18:42
*** lloydde has quit IRC18:42
nati_uenopcm__: Thanks!18:42
pcm__nati_ueno: Still trying to wrap my head around it all :)18:42
pcm__nati_ueno: Do have a bit of confusion on the stronswan sync() method.18:43
nati_uenopcm__:  thank you for your review. I'm working on it18:43
nati_uenopcm__: What's confusion?18:43
markmcclainvishy: change makes sense18:44
pcm__nati_ueno: It is getting a list of vpnservices. What are they?18:44
*** michchap has quit IRC18:44
pcm__nati_ueno: Getting a bit confused on who's talking to whom... it contacts agent_rpc to get_vpn_services_on_host(). Is that the OVS agent or other?18:45
*** eglynn has quit IRC18:45
nati_uenopcm__: get_vpn_services_on_host is in the service side ipsec driver code18:45
nati_uenopcm__: it gets assigned vpnservices for router18:46
*** esheffield has joined #openstack-dev18:46
pcm__nati_ueno: So is that IPSec>18:47
pcm__?18:47
nati_uenopcm__: yes whole configuration18:48
*** mrodden has quit IRC18:48
*** lloydde has joined #openstack-dev18:49
pcm__nati_ueno: Maybe I could step through the sync() function to get a better understanding of the pieces and flow?18:49
bnemecHmm, I'm running stack.sh and seeing the following error:18:50
bnemec++ nova flavor-create m1.nano 42 64 0 118:50
bnemecERROR: printt18:50
bnemecI get the same thing if I try to run any other nova commands.18:50
pcm__nati_ueno: Could I set a pdb breakpoint, restart the quantum process and then issue a command to step through the code in that area?18:50
bnemecI'll look into it, but just wondering if anyone else has run seen it.18:50
*** devoid has joined #openstack-dev18:51
nati_uenopcm__: yes it should work18:51
nati_uenonn18:51
nati_uenosorry typo18:51
*** portante has joined #openstack-dev18:52
pcm__nati_ueno: I'm trying to understand how the flow goes.18:52
*** bknudson has quit IRC18:53
nati_uenopcm__: here is bp https://docs.google.com/a/ntti3.com/presentation/d/1uoYMl2fAEHTpogAe27xtGpPcbhm7Y3tlHIw_G1Dy5aQ/edit#slide=id.gc2bc6c2e_024 This may help you to understand the code :)18:55
pcm__nati_ueno: Thanks! Will look at it. I'm having a bit of a hard time figuring out who's calling who...18:57
*** Tross has joined #openstack-dev18:57
nati_uenopcm__: yeah, it's complicated18:58
*** jjmb has quit IRC18:58
nati_uenopcm__: basically, service_driver calls device_driver18:58
pcm__nati_ueno: Looked like plugin updates DBase and then calls driver (vpnservice_updated?) which seemed to call IPSec Agent API. Is that right?18:58
nati_uenopcm__: However l3_agent also calls device_driver in case of router create and delete and restart18:58
nati_uenopcm__: yes18:59
pcm__nati_ueno: That seems to get l3_agents and then sends RPC?18:59
*** rnirmal has joined #openstack-dev18:59
pcm__nati_ueno: Does that then end up at the StrongSwan Driver RPC?19:00
nati_uenoso device driver consumes VPN_AGENT_TOPIC with host name19:00
nati_uenoIPSec Agent send message for the TOPIC19:00
nati_uenopcm__: yes19:00
*** avishay has quit IRC19:01
*** cp16net is now known as cp16net|away19:01
nati_uenopcm__: you can find topic value in the rpc call.19:01
*** cp16net|away is now known as cp16net19:01
nati_uenopcm__: you can also find topic value for comsumer code19:01
dolphmayoung: did you already start working bug 1194938?19:02
nati_uenopcm__: https://review.openstack.org/#/c/33148/8/quantum/services/vpn/service_drivers/ipsec_driver.py L87 is rpc sender19:02
ayoungdolphm, no19:03
nati_uenopcm__: https://review.openstack.org/#/c/33148/8/quantum/services/vpn/device_drivers/strongswan_driver.py L220 is consumer code19:03
ayoungdolphm, just want the bug in place for when we do, and to have a place to contain the discussion about it19:03
nati_uenopcm__: then you can know the flow19:03
dolphmayoung: i've actually already did the same last week, while playing with caching at the manager layer... happy to pull that out and propose19:03
*** jjmb has joined #openstack-dev19:04
pcm__nati_ueno: Great! That seems to match what I have so far in my notes.19:04
dolphms/did/done/19:04
nati_uenopcm__: Ok sounds good :)19:04
dolphms/grammar/more better/19:04
ayoungdolphm, oh, I wish I had known that before submitting this: https://review.openstack.org/#/c/34254/19:05
*** rkukura has joined #openstack-dev19:05
dolphmayoung: holy crap lol19:05
lifelessttx: should I put forward a proposal for TripleO at this point ?19:05
lifelessttx: (or perhaps 'Ops' or 'Deployments' - the other names seem to be sane rather than cute).19:06
dolphmayoung: can we agree to fix bug 1194938 before doing that?19:06
pcm__nati_ueno: OK, so then SS driver updates NAT and updates the config files as the last steps.19:06
ayoungdolphm, sure.19:07
ttxlifeless: it's a bit early still. We didn't approve the concept of "programs" yet. Next week ?19:07
ayounglifeless, calle it Trapelo19:07
lifelessayoung: heh :)19:07
lifelessttx: kk19:07
ttxlifeless: TripleO is fine, after all tere is Oslo19:07
lifelessttx: gives me time to work on some prose19:08
nati_uenopcm__: no, ss will do sync with servers. Then get vpnservices, then setup nat and setup conf19:08
dolphmayoung: assigning the bug to myself, i'll put up a patch in next 24 hours19:08
ayounghttps://en.wiktionary.org/wiki/trapelare#Italian19:08
*** dperaza has joined #openstack-dev19:08
*** n0ano has quit IRC19:08
ayoungdolphm, can you send me an early version?  I'll redo my patch ontop of what you have and rebase when the real thing comes through.19:09
lifelessayoung: nice19:09
pcm__nati_ueno: OK. I'll need to look through that part "sync with servers".19:09
ayounghttps://en.wiktionary.org/wiki/trape%C5%82o19:09
*** Tross has quit IRC19:09
ayoungGadget, Contraption.  OK lifeless we need to call it Trapelo19:10
dolphmayoung: what i have right now wouldn't be useful to rebase on top of, but i'll make it a priority19:10
ayoungdolphm, fair enough.19:10
nati_uenopcm__: OK I'm following same model of current l3-agent. so l3-agent code may also helpful19:10
dolphmayoung: mostly because i completely rewrote every manager, which i plan on undo-ing :)19:11
dolphmayoung: i went down a long dark road and changed my mind19:11
*** michchap has joined #openstack-dev19:11
pcm__nati_ueno: I don't know that code (yet) either :(19:11
ayoungdolphm, I am going to submit the "split Identity" in a scaled back version.  It won;'t allow multiple LDAP instance, just allow you to mix an identity provider and a assignment provider (assignments being projects, roles etc)19:11
ayoungbut the magix is in not breaking existing deployments19:11
nati_uenopcm__: :)19:11
ayoungdolphm, so if there is no assignment backend specified, it will match the backend to the identity backend19:12
*** garyk has quit IRC19:12
ayoungdolphm, Does that make sense?19:12
dolphmayoung: i think that makes sense19:13
pcm__nati_ueno: Can you clarify a bit on the unimplemented connection part? Are you saying more needs to be done in the SS driver as part of the sync() call or higher up at the IPsec driver?19:13
nati_uenopcm__: OK currently, only vpnservice status is updated19:14
*** networkstatic has joined #openstack-dev19:14
nati_uenopcm__: https://review.openstack.org/#/c/33148/8/quantum/services/vpn/device_drivers/strongswan_driver.py L29319:14
nati_uenopcm__: It checks ipsec status result only19:15
nati_uenopcm__: We need to update check connection status in future19:15
*** boris-42 has joined #openstack-dev19:16
*** jasdeepH has quit IRC19:17
pcm__nati_ueno: So that update_status() talks back to the IPSec driver, right?19:18
nati_uenopcm__: yes19:18
*** otherwiseguy has quit IRC19:20
*** martyntaylor has quit IRC19:20
pcm__nati_ueno: Does that end up calling RPCProxy base function? What does it update?19:21
*** woodspa has quit IRC19:21
nati_uenopcm__: yes it using cast method in base class. It update status19:22
*** atiwari has joined #openstack-dev19:22
*** kushal has quit IRC19:23
*** kenperkins has quit IRC19:23
*** michchap has quit IRC19:23
*** jyhc_ has joined #openstack-dev19:23
*** vipul is now known as vipul|away19:24
pcm__nati_ueno: So after it updates the status, it needs to update the connection status or check the connection status (sorry, did quite follow you sentence)19:24
*** jyhc_ has quit IRC19:24
nati_uenopcm__: so (1) ss driver should check ipsec status $VPNCONNECTION_ID19:24
*** jyhc_ has joined #openstack-dev19:25
nati_uenopcm__: (2) ss driver create status_update dict including status for each connection19:25
nati_ueno(3) ss driver sends the status update for ipsec driver19:25
nati_ueno(4) ipsec driver will update DB entry19:25
*** jjmb has quit IRC19:25
nati_uenopcm__: This is the flow19:25
*** jyhc has quit IRC19:26
ayoungatiwari, your link is not resolving19:27
atiwari1 sec19:27
stevemarayoung: over here now19:27
pcm__nati_ueno: Great! Thanks a bunch. I'll look at the flow diagram, try the manual commands you mentioned, and give a shot at stepping through the sync function so I better understand all the pieces.19:27
atiwariayoung, please try https://blueprints.launchpad.net/keystone/+spec/serviceid-binding-with-role-definition19:27
*** jclift has quit IRC19:28
pcm__nati_ueno: It's much less fuzzy now :)19:28
stevemarayoung is very popular right now :)19:28
nati_uenopcm__: OK I'll add more docstrings :P19:28
*** sdake has joined #openstack-dev19:28
*** sdake has joined #openstack-dev19:28
pcm__nati_ueno: Super. Helps us newbies :)19:28
ayoungatiwari, so you want to namesapce roles.  Fine by me.19:29
nati_uenopcm__: you are not newbies. Doing great review :)19:29
atiwaricorrect19:29
pcm__nati_ueno: Thanks! Trying my best. Learning a bunch just reviewing and seeing how it goes together.19:30
pcm__nati_ueno: Appreciate all the help!19:30
atiwariayoung, did you look the API changes in etherpad ?19:30
*** krtyyy has joined #openstack-dev19:30
*** jsavak has joined #openstack-dev19:30
*** krtyyy has left #openstack-dev19:31
*** olaph has quit IRC19:31
*** cliu has quit IRC19:31
*** jyhc has joined #openstack-dev19:33
*** jyhc has joined #openstack-dev19:33
*** joesavak has quit IRC19:34
*** ffio_ has quit IRC19:34
jyhcI am writing some python scripts to translate XML instance specifications into launching an instance (via the HTTP REST API)19:35
jyhcI am looking for a way to determine whether an instance has started. Any ideas?19:36
*** jjmb has joined #openstack-dev19:36
jyhcSome type of communication back to a Python daemon or script that the instance is up and running19:38
*** rony358 has joined #openstack-dev19:38
*** olaph has joined #openstack-dev19:39
*** jasdeepH has joined #openstack-dev19:39
_TheDodd_Is there a channel for 'Core Infrastructure' developers?19:40
*** leveldoc has joined #openstack-dev19:40
dolphm_TheDodd_: #openstack-infra19:40
_TheDodd_dolphm: Thanks for the timely response.19:40
leveldocis there support in horizon for managing ceph-based object store (radosgw)? I saw a screenshot, but that seemed to be for essex, we're deploying grizzly19:41
_TheDodd_dolphm: I know that you are quite busy, but is there any chance that you might have time for a code review at: https://review.openstack.org/#/c/34266/19:41
_TheDodd_?19:42
*** koolhead17 has joined #openstack-dev19:42
*** drewlander has quit IRC19:44
openstackgerritA change was merged to openstack/quantum: Imported Translations from Transifex  https://review.openstack.org/3453419:46
openstackgerritA change was merged to openstack/quantum: Require greenlet 0.3.2 (or later)  https://review.openstack.org/3284519:46
ayoungatiwari, so the big question I have "Is service the only namespace that makes sense for roles"19:46
ayoung?19:46
ayoungatiwari, because I would think that a namespace approach should work for anywhere we need to distinguish between to roles named the same thing.  We already sort of namespace the role by having the project name as a reference19:47
ayoungatiwari, domains are also treated this way19:48
*** dprince has quit IRC19:48
*** eglynn has joined #openstack-dev19:48
ayoungatiwari, so, maybe the problem isn't "we need to namespace roles for services" but rather "we need to add services to the set of objects by which we scope roles19:49
atiwariayoung, I think serivceId make sense because roles are basically derived from service19:49
ayoungatiwari, no they are not19:49
ayoungonly a subset are19:49
ayoungthe rest are service agnostic19:49
ayoungroles in domain and in projects are the same regardless of which service they are executed on19:49
*** michchap has joined #openstack-dev19:50
atiwarie.g. all the nova roles are derived from nove19:50
atiwarisame applies for other services19:50
*** tzumainn has quit IRC19:50
atiwarie.g. swift has its own roles19:50
atiwariso I think serviceId names pace make sense as attribute in role definition19:51
_TheDodd_I've been using 'tox' for my testing before I submit my code for review, are there any other tesing suites that anyone would recommend?19:51
_TheDodd_eatmydata maybe?19:51
atiwariayoung, not to mention I BP talks about role definitions19:52
ayoungatiwari, I am not disagreeing with that statement, I am stating that we already have a mechanism for role assignments, and we should extend it to cover your cases19:52
ayoungatiwari, the most common role name is "admin"19:52
ayoungwe want to be able to distinguish between "admin for nova" and "admin for demoproject"19:53
atiwariok, as mentined in the BP, it would be a challenge for PaaS services19:53
atiwarie.g. SQLDb service and MongoDB as service19:53
openstackgerritA change was merged to openstack/ceilometer: Pin the keystone client version  https://review.openstack.org/3457319:54
atiwariayoung, I think you are think in role assignment19:54
ayoungatiwari, well, again, unless you are administering them, the role should not be on the whole database, but on the particular namespace within the DBMS19:54
ayoungatiwari, yes, yes I do19:54
ayoungatiwari, I am not saying you are wrong, I am trying to map your BP to what we have now19:55
atiwariwe do not want to have name space in assignment19:55
atiwarithat looks good19:55
ayoungatiwari, what I thin you want is to have a separate section of the token with service roles in it like this:19:55
ayoung{service: 'nova';  'role','admin'}19:55
ayoungand then a user can specifically ask for a service token instead of a project token19:56
atiwarieventually19:56
ayoungdolphm, does my suggestion to atiwari match your view of role assignments?19:56
atiwaribut to go there we need a name space in role def19:56
ayoungatiwari, not really.19:57
ayoungatiwari, the role is "president'19:57
ayoungthere is a big difference between 'president of IBM' and 'president of the USofA19:57
ayoung'19:57
ayoungbut the role name is the same19:57
atiwarithat is correct19:58
ayoungatiwari, but what we don't have now is the concept of scoping roles to services.  Which is, I think, a valuable concept19:58
*** aelkikhia has joined #openstack-dev19:58
dolphmayoung: wouldn't the service just be in the token's scope?19:58
atiwariyou are adding a name space there with role19:58
atiwariUSA and IBM19:58
dolphmayoung: or even better, the token would be encrypted such that it can only be verified by service X?19:59
ayoungdolphm, yeah, just scoped to the service, not to the project.  To distinguish between a project admin and a service admin19:59
atiwariand that is what we want in role def entity19:59
ayoungdolphm, Bridge too far19:59
ayoungatiwari, yes, USA and IBM.  One is a Government, one is a corporation.  Two different types of groups, just like the distinction between projects and services20:00
atiwarinow the question is where to put that distinction20:01
atiwari?20:01
ayoungatiwari, we add a type of assignement:  service_role_assignemnt. I am currently splitting assignements off of identity, into a backend called assignments.  It would go into the assignments backend, and a user should be able to request a token with a service scoped role in it20:02
*** michchap has quit IRC20:02
ayoungatiwari, I think that fulfills your requirement.20:02
atiwariayoung, it seems I am not getting your idea20:04
*** mlavalle has joined #openstack-dev20:04
atiwarithere are more issues raised in the BP20:04
atiwarilike tooling issue how does assignment wd handle those20:05
ayoung"2. There is no way a middle ware can filter out unwanted roles from token while redirecting request to service (Nova, Swift, etc....), e.g. As mentioned in the bug below, token scoped to a project which has multiple services associated will have all service roles in it. A middle ware which operating on behalf of a service X has no way to filter out roles which not related to service X, unless it make some decision based on desc20:05
ayoungriptive role names."20:05
*** xBytez has quit IRC20:06
*** xBytez has joined #openstack-dev20:06
ayoungatiwari, so, aside from "admin" what roles do you really think are going to conflict?20:06
ayoungroles are quickly going to reflect the set of permissions they encapsulate, like "vm manager"20:07
ayoungor better yet20:07
ayoung"vm restarter"20:07
*** joesavak has joined #openstack-dev20:07
atiwariOK, suppose there are three db PaaS service on top of IaaS20:07
ayoung"a user with the role 'vm restarter' is allowed to restart virtual machines, but not just power them off."20:07
atiwariand every one want to have db-admin20:07
ayoungatiwari, then that role would be assigned to the service.20:08
*** networkstatic has quit IRC20:08
ayoungIt is just if you think a role needs to be scoped down to both the service and the project that there is an issue20:08
ayoungeither/or is fine20:08
ayoungdm-admin is still really the role 'admin' on the service 'postgres" or 'mongo'20:08
*** jsavak has quit IRC20:10
atiwariok, but how does some one distinguish admin or PG and mongo20:12
atiwarispecially in case of 2 in BP20:12
*** SumitNaiksatam has joined #openstack-dev20:12
ayoungatiwari, it would be the service id.  so in the token:20:13
*** rony358 has left #openstack-dev20:13
ayoung{service: 'postgres';  'role','admin'}20:13
ayoungor better yet20:13
ayoung{service: '1BDEEF145';  'role','admin'}20:13
ayoungObviously, "postgres" is too granular a term.  Probably needs to be able to be scoped to an endpoint as well as service.20:14
*** ajyounge_ has quit IRC20:15
*** terryh has quit IRC20:15
*** ajyounge has joined #openstack-dev20:16
ayoungatiwari, ponder it.   I think it is the right solution, and it suits your needs.  It is just adapting your blueprint to the pre-existing abstractions, not fundamentally changing it20:17
*** jbresnah has quit IRC20:18
*** vipul|away is now known as vipul20:18
*** jaybuff has joined #openstack-dev20:20
*** mkollaro has joined #openstack-dev20:22
*** jjmb has quit IRC20:24
atiwariI don't think the issue raised in the BP is addressed by having abstraction in role assignment20:24
atiwariyou are looking one part of the issue20:24
*** sushils has joined #openstack-dev20:28
*** michchap has joined #openstack-dev20:28
*** terryh has joined #openstack-dev20:29
*** lbragstad has quit IRC20:29
*** ffio has joined #openstack-dev20:29
*** SergeyLukjanov has joined #openstack-dev20:29
*** jbresnah has joined #openstack-dev20:30
*** sarob_ has joined #openstack-dev20:31
*** adalbas has quit IRC20:32
gyeeayoung, are you saying roles assigning will be a 4 party affair? i.e. assign a "role" to a "user" for a given "project" and "service"?20:34
ayounggyee, no20:34
ayounggyee, I was only suggesting that roles could *also* be assigned to services20:34
*** sarob has quit IRC20:34
gyeeayoung, that's what the BP's about, having a role "definition" tied to a service20:35
gyeewhat am I missing?20:35
ayounggyee, it is not the role "definition" just the role assignment that needs to change20:35
jyhcWhen Nova launches an instance, a notification is generated in JSON and then sent to an AMQP queue, correct?20:35
*** sarob_ has quit IRC20:36
*** jjmb has joined #openstack-dev20:36
ayounggyee, what they are suggesting is a de facto 4 way relationship, but I don;t think that is what they want20:36
ayounggyee, I think a role should be either on a project, or on a service, not both20:36
ayoungrole assignement that is20:36
*** dkranz has quit IRC20:37
gyeeayoung, right now is role assignment is  role - project/domain - user/group20:37
gyeeoh yeah, inheritance too20:37
gyeeare we saying we need to first assign a role to a service, then to a project?20:37
ayoungno20:38
*** ffio has quit IRC20:38
ayoungjust that the use cases they have put forth seem to all be "admin for db is different than admin for nova"20:38
ayoungwhich I agree with20:38
gyeeright, I understand the use case20:38
ayoungand thus the solution is "admin on <service>" separate from "admin on <project>"20:38
gyeethat's where I got lost20:39
atiwarime too20:39
gyeeso we are saying assign a role to a user for a given service, instead of a given project?20:40
gyeestill a 3 way affair20:40
atiwariand how it work with inherited role assignment20:40
*** michchap has quit IRC20:41
openstackgerritA change was merged to openstack/tempest: Add scenario test of instance snapshot and boot  https://review.openstack.org/3376620:42
*** otherwiseguy has joined #openstack-dev20:42
*** mrodden has joined #openstack-dev20:42
ayoungatiwari, my gut reaction is that inheritance is irrelevant to service roles.  THere is no grouping for services.  BUt I guess endpoints would inherit the roles assigned on the assocaited services20:42
*** enikanorov has joined #openstack-dev20:43
ayoungatiwari, the point you have to get clear is that  roles assignments are a 3 way tuple( user, project, rolename)20:44
ayoungservices are outside of that grouping20:44
*** tmclaugh[work] has left #openstack-dev20:44
gyeeproject is essentially a collection of services20:44
ayoungyou are suggesting a de factor20:44
atiwariI think we are complicating a simple thing here. Service owns role Def (Nova admin or Swift admin) and they are adding a name space by add service name in role def20:44
ayoungde facto (user, project, service, rolename)20:44
atiwarithen what is the issue adding an official name space20:45
ayoungNO!20:45
ayounggyee, that is wrong20:45
ayoungprojects are not collection of services20:45
*** jjmb has quit IRC20:45
ayoungprojects are collections of resources.  Services operate on the project resources, but those projects span multiple services20:45
ayoungif I have a project named demo (from devstack) there will be aspects of 'demo' stored in nova, glance, quantum, and cinder20:46
ayoungnot to mention keystone, of course20:46
ayoungNow, if I need to admin Nova, it is irrrelevant of project20:47
openstackgerritA change was merged to openstack/trove: Fixes my.cnf not reverting with a resize revert.  https://review.openstack.org/3151520:48
*** lbragstad has joined #openstack-dev20:48
openstackgerritA change was merged to openstack/nova: Change resource links when url has no project id  https://review.openstack.org/3163220:49
openstackgerritA change was merged to openstack/nova: Port flavors core API to v3 tree  https://review.openstack.org/2882520:49
atiwariayoung, what are the other resource a project can have ?20:49
openstackgerritA change was merged to openstack/nova: Prepare fake instance stubs for objects  https://review.openstack.org/3387420:49
atiwariapart from service20:49
openstackgerritA change was merged to openstack/nova: Make fake_network tolerant of objects  https://review.openstack.org/3387520:50
openstackgerritA change was merged to openstack/nova: Imported Translations from Transifex  https://review.openstack.org/3460320:50
ayoungatiwari, VMs,ports, Networks, images, partitions...20:50
openstackgerritA change was merged to openstack/nova: Cells: Add support for global cinder  https://review.openstack.org/3156120:50
gyeeayoung, those are service resources20:51
atiwariand why they are not a service ?20:51
atiwariNetworks is a service20:51
gyeekeystone does not have VMs, ports,20:51
gyeekeytone knows about quantum, nova, swift, etc20:52
atiwaribasically VM are managed by Nova20:52
atiwariand role maps to service20:52
atiwariimages are managed by glance20:53
*** woodspa has joined #openstack-dev20:53
atiwariso projects are nothing but collection of services20:53
gyeea collection of service endpoints to be exact :)20:54
atiwari+120:54
ayoungNo20:54
ayoungthe same endpoint spans multiple projects20:54
atiwarithe same way we describe domains20:54
atiwari"Domains represent collections of users, groups and projects"20:54
gyeeayoung, I didn't say they an endpoint cannot span multiple projects20:55
gyeeor service in that matter20:55
ayounggyee, I have work to get done.20:55
*** spzala has quit IRC20:55
*** joesavak has quit IRC20:55
_TheDodd_Does anyone use `eatmydata`, or is `tox` the preferred testing orchistrator?20:56
*** stevemar has quit IRC20:56
_TheDodd_Does anyone use `eatmydata`, or is `tox` the preferred testing orchistrator?20:56
ayoungI really have no desire to spend time quibbling.  Roles are just names.  If that role really needs to be deconflicted between what can be done on Nova and what can be done on Glance, then the organuization deploying it needs to deconflict.20:56
ayoungSo that argument is speciaous20:56
ayoungif you are trying to distinguish between "admin for nova" and "admin for glance" then we scope a role to a service and or and endpoint20:57
ayoungGo sleep on it, and we can discuss tomorrow20:57
*** martine_ has quit IRC20:58
dolphm_TheDodd_: tox20:59
_TheDodd_dolphm, that's what I thought, and that's  what I've been using. I noticed that `eatmydata ./run_test.sh` was mentioned somewhere and I wasn't sure if it was pertinent.21:00
gyeeayoung, define "scope a role to a service or an endpoint" so I have something to sleep on21:00
dolphm_TheDodd_: i've never actually heard of eatmydata21:00
*** jjmb has joined #openstack-dev21:00
*** sandywalsh has quit IRC21:01
*** jasondotstar has quit IRC21:01
_TheDodd_dolphm: I just came across it this morning. Do you know if there happens to be any doscumentation on 'tox' and out testing process? I've been trying to get a better feel of how our testing works and how to maximize it etc.21:02
dolphm_TheDodd_: https://wiki.openstack.org/wiki/ProjectTestingInterface21:02
*** bknudson has joined #openstack-dev21:02
*** bdpayne has quit IRC21:02
_TheDodd_dolphm: Thx.21:03
dolphm_TheDodd_: also checkout whatever commands are available in the tox.ini of the project you're looking at21:03
*** topol has quit IRC21:05
*** enikanorov has quit IRC21:05
dolphm_TheDodd_: eatmydata looks to be a completely different tool http://packages.debian.org/sid/eatmydata21:05
dolphm_TheDodd_: looks like you could run $ eatmydata tox21:06
*** pcm__ has quit IRC21:06
_TheDodd_dolphm: Interesting.21:06
_TheDodd_dolphm: Thanks for the links.21:07
*** michchap has joined #openstack-dev21:07
atiwariayoung, can you please add your thoughts how you want to associate roles with service ?21:08
atiwarimay be https://etherpad.openstack.org/serviceid-binding-with-role-definition21:08
atiwariis the right place21:08
*** rharwood has quit IRC21:11
*** enikanorov has joined #openstack-dev21:15
*** yolanda has quit IRC21:15
*** mkoderer has quit IRC21:16
dtroyer_zzdolphm: got a question on identity api version strings...21:17
*** HenryG has quit IRC21:17
*** sandywalsh has joined #openstack-dev21:17
*** cliu has joined #openstack-dev21:18
*** sarob has joined #openstack-dev21:19
*** sarob has quit IRC21:19
*** koolhead17 has quit IRC21:19
*** sarob has joined #openstack-dev21:19
*** michchap has quit IRC21:20
*** FunnyLookinHat has quit IRC21:20
*** torandu has quit IRC21:22
*** jayg is now known as jayg|g0n321:22
*** pnavarro has quit IRC21:23
*** pnavarro has joined #openstack-dev21:23
*** pnavarro has quit IRC21:23
*** zul has quit IRC21:24
*** kenperkins has joined #openstack-dev21:24
*** morazi has quit IRC21:24
*** FunnyLookinHat has joined #openstack-dev21:24
*** torandu has joined #openstack-dev21:26
*** nati_uen_ has joined #openstack-dev21:28
dolphmdtroyer_zz: sure21:28
dolphmbknudson: you might be interested in https://review.openstack.org/#/c/34339/2/keystone/policy/backends/rules.py21:28
bknudsonthat was fast.21:29
*** xqueralt-afk is now known as xqueralt21:29
dtroyer_zzdolphm: It looks like the string 'v3' is always going to be used in a v3 URI?  the version returned by curl http://10.130.50.20:35357/ has "id": "v3.0" in it.21:30
dtroyer_zzis that id supposed to represent the actual version being used and I should just hard-conde 'v3' into the URL?21:30
*** rnirmal has quit IRC21:30
*** nati_ueno has quit IRC21:31
dolphmdtroyer_zz: well that's completely awful...21:32
*** mrodden has quit IRC21:33
*** lbragstad has quit IRC21:33
dolphmdtroyer_zz: hmm... you do get back a link to the v3 endpoint21:33
*** cliu has quit IRC21:33
dtroyer_zzyes but two things: a) it has localhost rather than the actual IP as the v2.0 service catalog has, and b) it has 'v3' in it.  I don't want to have to parse that to match up the requested version to the available versions.21:35
dtroyer_zzthe url is this: "href": "http://localhost:35357/v3/"21:35
*** rscottcoyle has quit IRC21:35
dolphmdtroyer_zz: the 'localhost' bit is configurable via keystone.conf21:36
dtroyer_zzah, ok.  so it can account for proxies and so on21:36
dolphmdtroyer_zz: keystone can't reliably get that info from it's own service catalog21:36
*** krtaylor has quit IRC21:36
dolphmdtroyer_zz: exactly21:36
dolphmdtroyer_zz: i think it's keystone.conf [DEFAULT] admin_endpoint and public_endpoint21:37
dolphmdtroyer_zz: you don't have to parse the URL's though?21:38
dtroyer_zzdolphm: yup.  maybe devstack should set those?21:38
dolphmdtroyer_zz: +121:39
dtroyer_zzdolphm: I'm trying to figure out the multi-api version stuff in openstackclient, and one of the things I'm running in to is it using v2.0 for auth then v3, is that intentional?21:39
dtroyer_zzand alogn the way I noticed the '3.0' and went sideways21:40
dolphmdtroyer_zz: we had it defaulting the other way around, but ran into a bug with nova + auth_token, so it was reversed21:40
_TheDodd_I have noticed distincly that https://wiki.openstack.org/wiki/Main_Page does not, by any means, have an exaustive list of wiki pages and sources for developers to go to. Is there a particular location where one could see all of the pertinent wikis, as opposed to generating a massive - yet incomplete - bookmarks folder?21:40
dolphmdtroyer_zz: the URL should *always* be /v3/ (regardless of v3.0 or v3.1)21:40
dtroyer_zzdolphm: ok, that's good to know.  This also ties in to making devstack _use_ the v3 api rather than just enable it...21:41
dtroyer_zzv2.0 is hardcoded in stack.sh a couple of places21:41
jyhcIs there any wiki page that shows me how code is organized in, say, Nova?21:41
dolphmdtroyer_zz: and given a version-less endpoint http://localhost:35357/ keystoneclient should figure out whether v2.0, v3.0, or v3.1 is available and operate accordingly21:41
jyhcI'm looking at the github master branch and there's oodles of .py files21:41
jyhc(frankly I am very new to coding in Python)21:42
jyhcI'd just like to see an overview of the code structure of Nova and then I can take a closer look at each part alone21:42
*** cliu has joined #openstack-dev21:42
dolphmjyhc: http://docs.openstack.org/developer/nova/devref/index.html#module-reference21:42
*** zul has joined #openstack-dev21:42
dtroyer_zzdolphm: exactly…it is building the proper endpoint that I'm stumbling on.  so if I configure keystone properly the url in the api version structure should be usable.21:43
dolphmdtroyer_zz: should be!21:43
openstackgerritA change was merged to openstack/nova: Retry quota_reserve on DBDeadlock  https://review.openstack.org/3391521:44
*** timello_ has quit IRC21:44
*** mrodden has joined #openstack-dev21:44
dolphmdtroyer_zz: have you seen gabrielhurley's work on standardizing this stuff across clients?21:44
jyhcdolphm: Thanks. It is kinda lacking though...21:44
*** timello_ has joined #openstack-dev21:45
dolphmjyhc: it's not documenting much more than structural organization, but that's what you were looking for, no?21:46
dtroyer_zzdolphm: I have but haven't dug in to detail yet.  hopefully that falls below what I need to handle in osc; it should be in the libs21:46
*** mlavalle has quit IRC21:46
dolphmdtroyer_zz: definitely get his opinion if you start writing code around this :)21:46
*** michchap has joined #openstack-dev21:46
dolphmdtroyer_zz: i'm not sure what he's coded beyond the doc he wrote21:47
dtroyer_zzdolphm: I don't think much.  I'd like that to follow alessio's changes pretty closely though21:48
jyhcDoes Openstack use twisted in any way?21:48
dtroyer_zzthen we only do it once21:48
_TheDodd_I have noticed distincly that https://wiki.openstack.org/wiki/Main_Page does not, by any means, have an exaustive list of wiki pages and sources for developers to go to. Is there a particular location where one could see all of the pertinent wikis, as opposed to generating a massive - yet incomplete - bookmarks folder of pages that one has come across so far?21:49
*** nachi_ has joined #openstack-dev21:49
*** nachi__ has joined #openstack-dev21:49
*** nachi__ has quit IRC21:49
*** mtreinish has quit IRC21:50
dolphmjyhc: http://docs.openstack.org/developer/nova/devref/threading.html21:50
openstackgerritA change was merged to openstack/nova: Cells: Pass object for start/stop  https://review.openstack.org/3336421:50
openstackgerritA change was merged to openstack/swift: Allow SLOs to be made up of other SLOs  https://review.openstack.org/3095621:51
openstackgerritA change was merged to openstack/nova: Port quota classes extension to v3 API Part 1  https://review.openstack.org/3300321:51
*** mkollaro has quit IRC21:52
dolphm_TheDodd_: i've never seen the main page lol... googling "openstack [topic]" leads to some obscure wiki.openstack.org page 90% of the time for me21:52
*** mlavalle has joined #openstack-dev21:53
*** eglynn has quit IRC21:53
_TheDodd_dolphm: Nice. I was afraid that such was the case. I think it would be extremely beneficial to have a page wich would supply a hierarchical listing of topics and pertinent wikis.21:55
dolphm_TheDodd_: i suppose https://wiki.openstack.org/wiki/Special:AllPages21:55
dolphm_TheDodd_: they're sort of paginated (those are 5 pages listed)21:56
openstackgerritA change was merged to openstack/python-openstackclient: Fix py26 tests: assertDictEqual  https://review.openstack.org/3245321:56
*** gmurphy has quit IRC21:56
*** michchap_ has joined #openstack-dev21:57
*** michchap has quit IRC21:58
*** michcha__ has joined #openstack-dev21:58
*** michchap has joined #openstack-dev21:59
*** redbeard2 has quit IRC21:59
*** cliu has quit IRC22:01
*** michchap_ has quit IRC22:01
*** michchap_ has joined #openstack-dev22:02
*** michcha__ has quit IRC22:02
*** martyntaylor has joined #openstack-dev22:03
*** michcha__ has joined #openstack-dev22:03
ayoungdolphm, http://logs.openstack.org/34161/2/check/gate-python-keystoneclient-pep8/1252/console.html.gz adds a test dependency on httpPretty.  Is that OK?22:03
*** jergerber has quit IRC22:03
*** michchap has quit IRC22:03
*** aelkikhia has left #openstack-dev22:04
ayounghttp://falcao.it/HTTPretty/22:04
*** michchap has joined #openstack-dev22:04
*** cliu has joined #openstack-dev22:04
*** nati_uen_ has quit IRC22:05
openstackgerritA change was merged to openstack/python-keystoneclient: Fix the cache interface to use time= by default.  https://review.openstack.org/3384922:05
*** michch___ has joined #openstack-dev22:05
_TheDodd_Any keystone pros willing to do a code review at https://review.openstack.org/#/c/34266/?22:05
*** kbringard has quit IRC22:06
*** michchap_ has quit IRC22:06
ayoung_TheDodd_, oooh, I like22:07
*** michchap_ has joined #openstack-dev22:07
*** michcha__ has quit IRC22:07
*** michchap has quit IRC22:09
*** xqueralt is now known as xqueralt-afk22:09
*** michch___ has quit IRC22:10
*** spzala has joined #openstack-dev22:10
*** afazekas has quit IRC22:10
_TheDodd_ayoung, I appreciate that! It's been waiting for a final review for a while now.22:10
ayoung_TheDodd_, then why didn't you add me as a reviewer?22:10
_TheDodd_ayoung, lol...my bad.22:11
*** michchap_ has quit IRC22:11
*** _TheDodd_ has quit IRC22:12
ayoung_TheDodd_, btw, I think that Dolph's comment is the more correct solution, although yours seems to work on my machine as well22:12
*** martyntaylor has quit IRC22:13
*** jasdeepH has quit IRC22:14
*** diogogmt has quit IRC22:14
*** galstrom is now known as galstrom_zzz22:17
*** jasdeepH has joined #openstack-dev22:19
openstackgerritA change was merged to openstack/nova: Adds support for the Indigo Virtual Switch (IVS)  https://review.openstack.org/3173022:22
*** HenryG has joined #openstack-dev22:23
*** _TheDodd_ has joined #openstack-dev22:25
*** spzala has quit IRC22:25
openstackgerritA change was merged to openstack/swift: TempAuth: don't return 401/403 if .r without .rlistings  https://review.openstack.org/3433122:25
openstackgerritA change was merged to openstack/nova: Accept is_public=None when listing all flavors  https://review.openstack.org/3271322:26
*** lloydde has quit IRC22:27
*** jjmb has quit IRC22:29
*** sarob_ has joined #openstack-dev22:30
*** cliu has quit IRC22:31
*** FunnyLookinHat has quit IRC22:32
*** sarob_ has quit IRC22:32
*** sarob_ has joined #openstack-dev22:33
openstackgerritA change was merged to openstack/trove: GuestTimeout needed to be imported.  https://review.openstack.org/3462122:33
*** lloydde has joined #openstack-dev22:33
*** sarob has quit IRC22:34
*** SergeyLukjanov has quit IRC22:35
*** atiwari has quit IRC22:35
*** nati_ueno has joined #openstack-dev22:36
jaybuffwhen I delete an image from glance is the resource url supposed to eventually return 404?22:36
*** jvrbanac has quit IRC22:36
*** gmurphy has joined #openstack-dev22:37
*** michchap has joined #openstack-dev22:37
*** edmund has quit IRC22:38
openstackgerritA change was merged to openstack/trove: Fix few missed issues with the reddwarf -> trove rename  https://review.openstack.org/3445922:39
jaybufflooking at the code (and what my system does) it looks like it returns 200 with a json object that has status => DELETED in it.  But the tempest.api.compute.images.test_images_oneserver:ImagesOneServerTestJSON.test_create_delete_image tempest test seems to want the resource url to be 404.22:39
*** krtaylor has joined #openstack-dev22:40
*** neelashah has quit IRC22:44
*** jecarey_ has quit IRC22:44
*** gongysh has joined #openstack-dev22:45
*** dolphm has quit IRC22:45
*** zhuadl has joined #openstack-dev22:45
*** jjmb has joined #openstack-dev22:48
*** andrewbogott is now known as andrewbogott_afk22:48
*** michchap has quit IRC22:50
*** jimfehlig has quit IRC22:50
*** michchap has joined #openstack-dev22:52
*** michchap has quit IRC22:52
*** mlavalle has quit IRC22:53
*** mrodden has quit IRC22:53
*** michchap has joined #openstack-dev22:53
*** electrichead has quit IRC22:55
*** fifieldt has joined #openstack-dev22:56
*** cliu has joined #openstack-dev22:57
*** jjmb has quit IRC22:59
*** redbeard2 has joined #openstack-dev23:00
*** bknudson has quit IRC23:00
*** aelkikhia1 has joined #openstack-dev23:02
*** otherwiseguy has quit IRC23:02
*** datsun180b has quit IRC23:03
*** aelkikhia1 has quit IRC23:03
*** aelkikhia has joined #openstack-dev23:03
*** pmathews1 has quit IRC23:04
*** _TheDodd_ has quit IRC23:04
*** rcleere has quit IRC23:06
*** nati_ueno has quit IRC23:06
*** shang_ has quit IRC23:07
*** shang_ has joined #openstack-dev23:08
*** jjmb has joined #openstack-dev23:08
*** rcrit has quit IRC23:09
*** jjmb has quit IRC23:10
*** pixelbeat has quit IRC23:10
*** pixelbeat has joined #openstack-dev23:12
*** sarob_ has quit IRC23:12
*** lloydde has quit IRC23:12
*** sarob has joined #openstack-dev23:13
*** lbragstad has joined #openstack-dev23:15
*** nati_ueno has joined #openstack-dev23:18
*** Mandell has quit IRC23:19
*** aelkikhia has left #openstack-dev23:20
*** andrew_plunk has quit IRC23:21
*** noslzzp has quit IRC23:21
*** sdake has quit IRC23:23
*** rcrit has joined #openstack-dev23:23
*** portante has quit IRC23:25
*** cp16net is now known as cp16net|away23:28
*** Oneiroi has quit IRC23:28
*** cp16net|away is now known as cp16net23:29
*** lloydde has joined #openstack-dev23:31
*** lbragstad has left #openstack-dev23:31
*** leizhang has quit IRC23:34
*** devoid has quit IRC23:36
*** Oneiroi has joined #openstack-dev23:37
*** gyee has quit IRC23:37
*** markwash has quit IRC23:42
*** lbragstad has joined #openstack-dev23:43
*** jasdeepH has quit IRC23:46
*** crank_ is now known as crank23:49
*** melwitt has joined #openstack-dev23:51
*** crank is now known as crank_23:51
*** pabelanger has quit IRC23:52
*** crank_ is now known as crank23:52
*** crank is now known as crank_23:53
*** crank_ is now known as crank23:53
*** bing_bu has joined #openstack-dev23:54
*** dkranz has joined #openstack-dev23:55
*** jasdeepH has joined #openstack-dev23:58
*** salv-orlando has quit IRC23:59
*** jjmb has joined #openstack-dev23:59

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