Thursday, 2016-08-18

*** thumpba_ has joined #openstack-oslo00:03
*** snizar_ has quit IRC00:03
*** snizar has quit IRC00:03
*** thumpba has quit IRC00:05
*** snizar has joined #openstack-oslo00:10
*** snizar_ has joined #openstack-oslo00:10
zzzeekBknudson: oh it's migrate ?  Ugh yeah much more quirks w engines... Big reason I wrote alembic00:12
*** code-R_ has quit IRC00:22
*** sdake has joined #openstack-oslo00:35
openstackgerritMerged openstack/oslo.messaging: Fix the help info format  https://review.openstack.org/35266700:37
*** dimtruck is now known as zz_dimtruck00:45
bknudsonzzzeek: migrate didn't like that the scripts had the same name... for some reason migrate was getting the wrong object from its object cache.00:48
*** jamielennox is now known as jamielennox|away00:48
bknudsoneasy workaround is to rename the scripts00:48
*** gus has joined #openstack-oslo00:48
bknudsonalso, yes, we should have switched to alembic a while ago but we don't take care of debt in keystone, only add more.00:48
*** itisha has quit IRC00:50
*** spzala has joined #openstack-oslo00:59
*** sdake has quit IRC00:59
*** sdake has joined #openstack-oslo01:02
*** spzala has quit IRC01:03
*** jamielennox|away is now known as jamielennox01:04
*** snizar has quit IRC01:04
*** snizar_ has quit IRC01:04
harlowjaok folks i proposed https://review.openstack.org/#/c/356822/01:05
patchbotharlowja: patch 356822 - releases - Oslo weekly release for week of aug 15th, 201601:05
harlowjait should have our needed stuff for this week (that isn't already being released via different reviews)01:06
*** jecarey has quit IRC01:08
harlowjasorry for being a little late there01:08
*** code-R has joined #openstack-oslo01:09
*** sdake has quit IRC01:11
*** spzala has joined #openstack-oslo01:15
*** jeckersb_gone is now known as jeckersb01:20
*** zz_dimtruck is now known as dimtruck01:31
*** code-R has quit IRC01:37
*** spzala has quit IRC01:38
*** spzala has joined #openstack-oslo01:39
*** edmondsw has quit IRC01:40
*** hockeynut has joined #openstack-oslo01:42
*** spzala has quit IRC01:43
*** takedakn has joined #openstack-oslo01:44
*** takedakn has quit IRC01:51
*** thumpba_ has quit IRC01:52
*** tbarron is now known as tbarron|afk02:00
*** thumpba has joined #openstack-oslo02:23
*** eandersson_ has joined #openstack-oslo02:40
*** jamielennox is now known as jamielennox|away02:42
*** takedakn has joined #openstack-oslo02:46
*** eandersson_ has quit IRC02:47
openstackgerritJinxing Fang proposed openstack/oslo.messaging: Change the rabbit_max_retries default value  https://review.openstack.org/35684002:49
openstackgerritJinxing Fang proposed openstack/oslo.messaging: Change the rabbit_max_retries default value  https://review.openstack.org/35684002:51
*** spzala has joined #openstack-oslo03:00
*** spzala has quit IRC03:05
*** hockeynut has quit IRC03:06
*** jamielennox|away is now known as jamielennox03:07
*** yamahata has quit IRC03:07
*** thumpba has quit IRC03:15
openstackgerritChangBo Guo(gcb) proposed openstack/oslo.log: Default use_stderr to False  https://review.openstack.org/35128303:16
*** tonytan4ever has joined #openstack-oslo03:36
*** code-R has joined #openstack-oslo03:38
*** code-R has quit IRC03:43
*** links has joined #openstack-oslo03:45
*** takedakn has quit IRC03:46
*** code-R has joined #openstack-oslo03:47
*** code-R_ has joined #openstack-oslo03:50
*** code-R has quit IRC03:52
*** spzala has joined #openstack-oslo04:01
*** code-R_ has quit IRC04:04
*** tonytan4ever has quit IRC04:05
*** spzala has quit IRC04:07
*** code-R has joined #openstack-oslo04:25
*** amotoki has joined #openstack-oslo04:33
*** amotoki has quit IRC04:38
*** code-R has quit IRC04:38
*** code-R has joined #openstack-oslo04:38
*** code-R has quit IRC04:59
*** spzala has joined #openstack-oslo05:00
*** amotoki has joined #openstack-oslo05:00
*** amotoki has quit IRC05:05
*** spzala has quit IRC05:06
*** tonytan4ever has joined #openstack-oslo05:06
*** tonytan4ever has quit IRC05:11
*** code-R has joined #openstack-oslo05:15
*** eandersson_ has joined #openstack-oslo05:24
*** amotoki has joined #openstack-oslo05:28
*** amotoki has quit IRC05:32
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.config: Updated from global requirements  https://review.openstack.org/35693505:55
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.log: Updated from global requirements  https://review.openstack.org/35446205:55
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements  https://review.openstack.org/35693605:55
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.service: Updated from global requirements  https://review.openstack.org/35693705:56
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.utils: Updated from global requirements  https://review.openstack.org/35693805:56
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.vmware: Updated from global requirements  https://review.openstack.org/35693905:56
*** spzala has joined #openstack-oslo06:03
*** code-R has quit IRC06:05
*** spzala has quit IRC06:07
*** eandersson_ has quit IRC06:07
*** code-R has joined #openstack-oslo06:08
*** rcernin has joined #openstack-oslo06:12
*** lpetrut has joined #openstack-oslo06:18
*** yamamoto has quit IRC06:21
*** amotoki has joined #openstack-oslo06:22
*** amotoki has quit IRC06:26
*** amotoki has joined #openstack-oslo06:29
*** code-R_ has joined #openstack-oslo06:34
*** code-R has quit IRC06:36
*** takedakn has joined #openstack-oslo06:44
*** takedakn has quit IRC06:49
*** amotoki has quit IRC06:51
*** pcaruana has joined #openstack-oslo06:52
*** code-R_ has quit IRC06:53
*** takedakn has joined #openstack-oslo06:56
*** amotoki has joined #openstack-oslo06:56
*** yamamoto has joined #openstack-oslo07:00
*** amotoki has quit IRC07:04
*** xek__ has quit IRC07:08
*** lpetrut has quit IRC07:11
*** tesseract- has joined #openstack-oslo07:17
*** e0ne has joined #openstack-oslo07:18
*** yamamoto has quit IRC07:38
*** nkrinner_afk has quit IRC07:39
*** nkrinner_afk has joined #openstack-oslo07:51
*** amotoki has joined #openstack-oslo07:55
*** amotoki_ has joined #openstack-oslo07:57
*** zzzeek has quit IRC08:00
*** amotoki has quit IRC08:00
*** zzzeek has joined #openstack-oslo08:01
*** cdent has joined #openstack-oslo08:04
*** spzala has joined #openstack-oslo08:04
*** ihrachys has joined #openstack-oslo08:08
*** spzala has quit IRC08:08
*** mhickey has joined #openstack-oslo08:09
*** eandersson_ has joined #openstack-oslo08:26
*** ihrachys has quit IRC08:36
*** takedakn has quit IRC08:37
*** yamamoto has joined #openstack-oslo08:39
openstackgerritJulien Danjou proposed openstack/tooz: file: move _read_{group,member}_id to staticmethod-s  https://review.openstack.org/34784108:40
openstackgerritJulien Danjou proposed openstack/tooz: file: move _load_and_validate to a method  https://review.openstack.org/34784208:40
openstackgerritJulien Danjou proposed openstack/tooz: file: return converted voluptuous data  https://review.openstack.org/34784308:40
openstackgerritJulien Danjou proposed openstack/tooz: file: update .metadata atomically  https://review.openstack.org/34784408:40
*** yamamoto has quit IRC08:52
*** yamamoto has joined #openstack-oslo08:56
*** yamamoto has quit IRC08:56
*** yamamoto has joined #openstack-oslo08:56
*** dkehn_ has quit IRC09:00
*** dkehn has quit IRC09:00
*** spzala has joined #openstack-oslo09:04
*** e0ne has quit IRC09:07
*** dkehn has joined #openstack-oslo09:07
*** d0ugal has quit IRC09:08
*** spzala has quit IRC09:09
*** yamamoto has quit IRC09:10
*** ihrachys has joined #openstack-oslo09:10
*** yamamoto has joined #openstack-oslo09:12
*** d0ugal has joined #openstack-oslo09:12
*** yamamoto has quit IRC09:12
*** d0ugal_ has joined #openstack-oslo09:17
*** d0ugal_ has quit IRC09:18
*** d0ugal_ has joined #openstack-oslo09:18
*** dkehn_ has joined #openstack-oslo09:18
*** dtardivel has joined #openstack-oslo09:19
*** d0ugal has quit IRC09:20
*** d0ugal_ has quit IRC09:20
*** d0ugal has joined #openstack-oslo09:20
*** ihrachys has quit IRC09:24
*** sambetts|afk is now known as sambetts09:25
*** lpetrut has joined #openstack-oslo09:29
*** yamamoto has joined #openstack-oslo09:30
*** yamamoto has quit IRC09:30
*** yamamoto has joined #openstack-oslo09:30
*** yamamoto has quit IRC09:30
*** yamamoto has joined #openstack-oslo09:31
openstackgerritMerged openstack-dev/pbr: Replace OpenStack LLC with OpenStack Foundation  https://review.openstack.org/34729909:31
openstackgerritGevorg Davoian proposed openstack/oslo.messaging: [zmq] Implement retries for unacknowledged CALLs  https://review.openstack.org/35352909:33
*** yamamoto has quit IRC09:36
*** ozamiatin has joined #openstack-oslo09:49
openstackgerritMerged openstack/oslo.config: Updated from global requirements  https://review.openstack.org/35693509:51
*** ihrachys has joined #openstack-oslo09:56
*** d0ugal has quit IRC09:58
openstackgerritMerged openstack/oslo.utils: Updated from global requirements  https://review.openstack.org/35693810:00
*** yamamoto has joined #openstack-oslo10:01
*** yamamoto has quit IRC10:01
*** gcb_ has quit IRC10:02
*** d0ugal has joined #openstack-oslo10:04
openstackgerritMerged openstack/oslo.log: Updated from global requirements  https://review.openstack.org/35446210:05
*** spzala has joined #openstack-oslo10:05
*** sdague has joined #openstack-oslo10:06
*** ihrachys has quit IRC10:07
*** spzala has quit IRC10:10
*** amotoki_ has quit IRC10:14
*** amotoki has joined #openstack-oslo10:18
openstackgerritMerged openstack/oslo.vmware: Updated from global requirements  https://review.openstack.org/35693910:26
*** ozamiatin_ has joined #openstack-oslo10:29
*** yamamoto has joined #openstack-oslo10:32
*** code-R has joined #openstack-oslo10:33
*** ociuhandu has quit IRC10:33
*** boden has joined #openstack-oslo10:33
*** yamamoto has quit IRC10:37
*** code-R_ has joined #openstack-oslo10:40
*** yamamoto has joined #openstack-oslo10:41
*** lpetrut1 has joined #openstack-oslo10:43
*** code-R has quit IRC10:43
*** lpetrut has quit IRC10:45
*** lpetrut1 is now known as lpetrut10:45
*** amotoki has quit IRC10:45
*** amotoki_ has joined #openstack-oslo10:46
openstackgerritMerged openstack/oslo.privsep: Use default value for undefined caps in fmt_caps  https://review.openstack.org/34052410:47
openstackgerritMerged openstack/oslo.vmware: Fix TypeError:six.StringIO(resp.content) must be str or None, not bytes.  https://review.openstack.org/35160110:53
*** code-R_ has quit IRC10:57
*** code-R has joined #openstack-oslo10:57
*** ozamiatin_ has quit IRC10:59
*** spzala has joined #openstack-oslo11:06
*** spzala has quit IRC11:11
*** lucasagomes is now known as lucas-hungry11:13
*** cdent has quit IRC11:17
*** spzala has joined #openstack-oslo11:23
*** spzala has quit IRC11:23
*** takedakn has joined #openstack-oslo11:34
*** code-R has quit IRC11:38
*** code-R has joined #openstack-oslo11:38
*** takedakn has quit IRC11:39
*** ociuhandu has joined #openstack-oslo11:39
*** cdent has joined #openstack-oslo11:48
*** code-R has quit IRC11:49
*** boden has quit IRC11:54
*** jpena is now known as jpena|lunch12:00
*** code-R has joined #openstack-oslo12:03
*** jeckersb is now known as jeckersb_gone12:04
*** boden has joined #openstack-oslo12:05
*** e0ne has joined #openstack-oslo12:09
*** edmondsw has joined #openstack-oslo12:15
*** ansmith has joined #openstack-oslo12:15
*** lucas-hungry is now known as lucasagomes12:19
*** ihrachys has joined #openstack-oslo12:20
*** spzala has joined #openstack-oslo12:24
*** code-R has quit IRC12:24
*** gordc has joined #openstack-oslo12:27
*** spzala has quit IRC12:29
*** pradk has joined #openstack-oslo12:31
*** ihrachys has quit IRC12:31
*** ihrachys has joined #openstack-oslo12:32
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.privsep: Updated from global requirements  https://review.openstack.org/35091412:39
*** jeckersb_gone is now known as jeckersb12:50
*** e0ne has quit IRC12:52
*** kgiusti has joined #openstack-oslo12:52
*** e0ne has joined #openstack-oslo12:55
*** jpena|lunch is now known as jpena12:55
*** rlrossit has joined #openstack-oslo12:59
*** ihrachys has quit IRC12:59
openstackgerritGevorg Davoian proposed openstack/oslo.messaging: [zmq] Implement retries for unacknowledged CALLs  https://review.openstack.org/35352912:59
*** amotoki_ has quit IRC13:00
openstackgerritKirill Bespalov proposed openstack/oslo.messaging: Fix socket getting in the set_transport_socket_timeout()  https://review.openstack.org/35718913:00
*** ihrachys has joined #openstack-oslo13:02
*** code-R has joined #openstack-oslo13:05
*** amotoki has joined #openstack-oslo13:06
*** jecarey has joined #openstack-oslo13:07
kgiustiharlowja: dude - I need a favor...13:07
*** code-R_ has joined #openstack-oslo13:09
*** code-R has quit IRC13:13
*** ihrachys has quit IRC13:13
*** mhickey has quit IRC13:14
*** ihrachys has joined #openstack-oslo13:16
*** links has quit IRC13:16
*** thumpba has joined #openstack-oslo13:17
*** ihrachys has quit IRC13:18
*** amotoki has quit IRC13:27
*** mhickey has joined #openstack-oslo13:28
openstackgerritGevorg Davoian proposed openstack/oslo.messaging: [zmq] Host name and target in socket identity  https://review.openstack.org/35645113:36
*** amotoki has joined #openstack-oslo13:37
*** dimtruck is now known as zz_dimtruck13:41
*** zz_dimtruck is now known as dimtruck13:41
*** spzala has joined #openstack-oslo13:42
*** ihrachys has joined #openstack-oslo13:47
*** ihrachys has quit IRC13:50
*** dimtruck is now known as zz_dimtruck13:51
*** cdent has quit IRC13:52
*** ihrachys has joined #openstack-oslo13:53
*** itisha has joined #openstack-oslo13:57
jschwarzjd__, hey13:58
jd__jschwarz: oy13:58
jschwarzjd__, how's life?13:58
jd__jschwarz: good and ya?13:58
jschwarzjd__, good, good13:59
jschwarzjd__, so I'm starting to look into tooz' adoption for Neutorn during Ocata13:59
jschwarzjd__, I had a couple of questions.. the first regards the API of tooz13:59
jschwarzjd__, if for example I wish to move all the exceptions from https://github.com/openstack/tooz/blob/master/tooz/coordination.py#L648 to a new file (exceptions.py), is that something that can be done?14:00
jd__jschwarz: possibly14:02
jschwarzjd__, yey :)14:02
jd__moving code around for moving code around is not my cup of tea14:02
jd__but if you got compelling argument and want to do it… I can be sold14:02
jschwarzjd__, it's not for the sake of moving around - most other projects have exceptions.py and we don't want coordination.py to be too crowded14:02
jschwarzjd__, so consistency and organisation are my selling points :)14:03
*** abashmak has left #openstack-oslo14:03
jschwarzjd__, the second question regards https://github.com/openstack/cinder/tree/master/cinder/coordination.py#L6014:03
jschwarzjd__, basically I'm thinking of copying that from cinder to tooz so that it can be used throughout (as it's quite generic)14:04
jschwarzjd__, although some of it could go into DriverManager14:04
jschwarzjd__, what do you think?14:04
jd__hm14:05
jd__most of it is already in tooz, the heartbeat thread part14:05
jschwarzjd__, really? where?14:05
jd__jschwarz: http://docs.openstack.org/developer/tooz/developers.html#tooz.coordination.CoordinationDriver.start14:06
jd__start_heart=True14:06
jd__that does it for ya14:06
jd__maybe the reconnect part of Cinder could be useful, I don't think we have that14:06
jschwarzjd__, I see14:07
jschwarzjd__, the Heart class is newer than the Cinder code so I think that's why14:08
jschwarzjd__, so adapting Cinder code instead of copy-pasting - roger that14:08
jschwarzjd__, what about Cinder's Lock and @synchronized (same file I linked earlier)?14:09
*** zz_dimtruck is now known as dimtruck14:10
*** e0ne has quit IRC14:10
*** ihrachys has quit IRC14:11
jd__jschwarz: it is newer indeed, Cinder should be updated agreed14:12
jd__jschwarz: let me check14:12
jd__jschwarz: I don't think Lock is useful, it's mainly a hack to have a global variable named COORDINATOR, we don't want that in a lib (nor in a program if you ask me lol)14:14
*** ihrachys has joined #openstack-oslo14:14
jd__jschwarz: synchronized, why not14:14
*** e0ne has joined #openstack-oslo14:14
jd__though I'm not sure having a decorator that can't rely on a global variable is that useful14:15
jschwarzjd__, though, Cinder's Lock also supports "Lock('foo-{volume.id}, {'volume': ...,})"14:15
openstackgerritDan Smith proposed openstack/oslo.versionedobjects: Fix remotable object change tracking  https://review.openstack.org/35598114:15
jschwarzwhich I'm not sure if yours do or not14:15
*** ihrachys has quit IRC14:16
jd__I don't follow, which feature is useful here?14:16
jd__the name of the lock? tooz has14:16
jschwarzjd__, the "inspect the lock name in order to retrieve variable lock names"14:16
openstackgerritGevorg Davoian proposed openstack/oslo.messaging: [zmq] Implement retries for unacknowledged CALLs  https://review.openstack.org/35352914:17
jschwarzjd__, so if I have a Lock on a volume with an id '111', it will produce a Lock that is automatically unique to that volume.id14:17
jd__so you mean lock_name.format(**lock_data) ?14:17
jschwarzjd__, yes14:17
jd__I don't think it's really useful in tooz TBH14:17
jschwarzjd__, why not?14:18
jd__everyone picks its naming convention14:18
jd__we don't really care14:18
jschwarzjd__, aye, but that naming convention is quite useful IMO14:18
jd__it's a one line function in each app that wants to implement its naming convention based on its workflow14:18
*** ihrachys has joined #openstack-oslo14:18
jschwarzand not providing a naming convention at all (except for some pre-determined string) will only cause each project to define its own naming scheme14:19
jschwarzjd__, which is code duplication that can be avoided in most cases14:19
jd__well, when we'll identify that code duplication, maybe14:19
jschwarz"hey, here's our default, useful naming convention you can use. if you don't want what most projects use, define your own"14:20
jd__until then, I can tell that it won't apply e.g to any of telemetry projects14:20
jschwarzjd__, I can identify the code duplication right now actually14:20
jd__why Cinder does not use the volume uuid for the lock name?14:20
jschwarzjd__, because if that naming scheme isn't in Tooz, I'm gonna copy it from Cinder to Neutron14:21
jschwarzjd__, I don't know. would it matter if they did?14:21
jd__well, I don't see what code you need in tooz if you do that, like we do in telemetry it's just self.coordinator.get_lock(resource.id)14:22
jd__and you get a lock for your resource14:22
jd__do they need anything more?14:22
jd__or Neutron14:22
jschwarzjd__, it would be extremely helpful to put that information (the lock name) as a decorator14:22
jd__as a decorator? WDYM?14:23
jschwarzjd__, but a decorator doesn't normally has access to 'resource', unless it can inspect the lock_data as Cinder does14:23
jschwarzjd__, let me find a code example14:23
jd__sure go ahead :)14:23
jschwarzjd__, https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/glusterfs.py#L18814:24
* jd__ eyes start bleeding14:25
jschwarzjd__, this is extremely usefull IMO - they can ensure the function executes only when it has the lock and they don't need to call "self.coordinator.get_lock()" and try-except blocks every time14:25
jschwarzjd__, no! don't bleed!14:25
jschwarz>.<14:25
*** mpotdar_ has joined #openstack-oslo14:26
jd__you don't need a try block, just a with self.coord.get_lock(volume.id)14:26
jd__it's just one line, like the decorator14:26
jschwarzjd__, well technically it's more than one14:26
jschwarzbecause you then need to release it14:27
jd__with does that…14:27
jd__the lock work as context managers14:27
jschwarzjd__, so you either use a 'with' block on the entire function (which is meh), or you add a 'lock.release()' before each return14:27
jd__no14:27
jschwarz?14:27
jd__google python context manager :P14:28
jd__let me show you14:28
jschwarzlol, I know how to use context managers14:28
jschwarzwell, I hope :P14:28
jd__the locks in tooz are defined as context managers: https://github.com/openstack/tooz/blob/master/tooz/locking.py#L3714:29
jd__so you can do with thelock: <anything>14:29
jd__as soon as you quit the with block in _any_ way: return, raise, whatever14:29
jschwarzjd__, yes14:29
jd__the __exit__ function is called14:29
jd__which does a release of the lock14:29
jschwarzjd__, but that increases the indentation of the entire function, which is meh :P14:29
jd__…14:29
jd__I have a rule, I stop arguing when it goes out of technical scope such as coding style, etc, I'm sorry :-)14:30
jschwarzjd__, consider a 30-lines function - you now have to increase its indentation (take care of pep8) or create a pass-through function, which is basically a decorator14:30
jd__jschwarz: example in Gnocchi https://github.com/openstack/gnocchi/blob/master/gnocchi/storage/_carbonara.py#L22214:30
jschwarzjd__, it's not strictly coding styles, it's usability as well14:31
jd__jschwarz: you're free to create a decorator wrapper with a "with lock:" (like @synchronized) if you wish – but without a global variable I think it'll be clunky14:31
jschwarzif you consider how people are going to use coordinator.get_lock(), a decorator that automatically acquires and releases it is acceptable14:31
jd__jschwarz: I won't break my rule, I'm sorry14:32
jd__:)14:32
jschwarzjd__, okay then14:32
jschwarzI'll find a way to make it non-clunky :)14:32
jd__jschwarz: but you can't do that without a global variable I think, what you want to do14:32
jd__which is a no-go14:32
jschwarzjd__, well14:32
jschwarzjd__, you could have it decorate a class method, so the inner function will accept 'self' as first argument and refer to 'self.coordinator'14:33
jd__we already suffered too much of oslo_config.cfg.CONF14:33
jschwarzit'll work but it'll suck ;-)14:33
jd__jschwarz: it is hackish indeed…14:33
*** mhickey has quit IRC14:34
jschwarzjd__, well, thanks for your time indeed :)14:35
jd__no problem jschwarz14:36
*** cdent has joined #openstack-oslo14:37
*** d0ugal has quit IRC14:41
*** mhickey has joined #openstack-oslo14:47
*** ihrachys has quit IRC14:47
*** eeiden has quit IRC14:47
*** d0ugal has joined #openstack-oslo14:48
*** d0ugal has quit IRC14:48
*** d0ugal has joined #openstack-oslo14:48
*** syjulian has quit IRC14:49
*** syjulian has joined #openstack-oslo14:50
*** dmellado|off is now known as dmellado14:52
*** tonytan4ever has joined #openstack-oslo14:57
*** kgiusti is now known as kgiusti|meeting14:59
*** tbarron|afk is now known as tbarron15:00
*** salv-orlando has joined #openstack-oslo15:00
*** yamamoto has quit IRC15:01
*** hockeynut has joined #openstack-oslo15:04
*** yamamoto has joined #openstack-oslo15:04
*** d0ugal has quit IRC15:09
*** yamamoto has quit IRC15:09
*** eeiden has joined #openstack-oslo15:15
*** ihrachys has joined #openstack-oslo15:21
*** rcernin has quit IRC15:22
*** ihrachys has quit IRC15:22
*** d0ugal has joined #openstack-oslo15:24
*** mhickey has quit IRC15:24
*** e0ne has quit IRC15:30
*** ozamiatin_ has joined #openstack-oslo15:32
*** mhickey has joined #openstack-oslo15:36
*** ozamiatin_ has quit IRC15:36
*** ozamiatin_ has joined #openstack-oslo15:40
*** kgiusti|meeting is now known as kgiusti15:41
*** mpotdar_ has quit IRC15:45
*** mpotdar_ has joined #openstack-oslo15:45
*** tonytan4ever has quit IRC16:00
*** tonytan4ever has joined #openstack-oslo16:01
*** yamamoto has joined #openstack-oslo16:06
*** yamamoto has quit IRC16:11
*** yamahata has joined #openstack-oslo16:13
*** ihrachys has joined #openstack-oslo16:16
*** mtanino has joined #openstack-oslo16:20
*** tesseract- has quit IRC16:26
*** sambetts is now known as sambetts|afk16:27
*** ihrachys has quit IRC16:36
*** thumpba_ has joined #openstack-oslo16:37
*** ihrachys has joined #openstack-oslo16:38
*** mhickey has quit IRC16:39
*** thumpba has quit IRC16:39
*** amotoki has quit IRC16:40
*** itisha has quit IRC16:40
*** ihrachys has quit IRC16:41
*** code-R_ has quit IRC16:41
*** lucasagomes is now known as lucas-dinner16:42
*** ihrachys has joined #openstack-oslo16:44
*** sdague has quit IRC16:49
*** srini has joined #openstack-oslo16:52
*** d34dh0r53 is now known as RichardLongus16:52
sriniIs there a way to force oslo_db to reconnect mysql connections with new config? For e.g. there is a password change. Are is there a better strategy for handling password changes in case of a security threat16:53
*** eandersson_ has quit IRC16:54
*** RichardLongus is now known as d34dh0r5316:58
*** ihrachys has quit IRC16:58
*** yamahata has quit IRC17:00
*** yamahata has joined #openstack-oslo17:00
*** mpotdar_ has quit IRC17:03
*** tonytan_brb has joined #openstack-oslo17:04
*** sdague has joined #openstack-oslo17:06
*** tonytan4ever has quit IRC17:06
kgiustidims: ping - would you be able to help me create a merge patch for the amqp1-dispatch-router feature branch?17:11
kgiustidims: I don't have the perms:  http://paste.openstack.org/show/560798/17:11
*** ozamiatin_ has quit IRC17:14
harlowjakgiusti let me make that17:14
kgiustiharlowja: you da man!17:14
harlowjanp sir17:15
*** tonytan_brb is now known as tonytan4ever17:25
*** gsilvis has joined #openstack-oslo17:31
*** lpetrut has quit IRC17:31
openstackgerritAlexander Makarov proposed openstack/oslo.cache: Store cache invalidation timestamps on region backend  https://review.openstack.org/35483117:36
harlowjalet see if i can do this, ha17:36
kgiustiharlowja: I used this: http://docs.openstack.org/infra/manual/drivers.html17:38
kgiustiharlowja: it was trivial, until I tried the git review -R, which bombed out17:39
harlowja:-P17:39
harlowjak17:39
kgiustiharlowja: take note of the .gitreview checkout in that manual - I initially missed that!17:39
gsilvisIn oslo-cache, 'get_memoization_decorator' takes a 'group' parameter, saying where in the config file it should look to get the caching configuration... but 'configure' only loads the configuration in the section named 'cache'17:39
gsilvisThis seems odd to me, but I'm not sure quite what was intended there17:40
harlowjakgiusti kk, almost to that step, ha17:40
* kgiusti crosses fingers, toes, eyes17:41
openstackgerritJoshua Harlow proposed openstack/oslo.messaging: Merge branch 'amqp-dispatch-router' into merge-amqp1-dispatch-router  https://review.openstack.org/35737017:41
harlowjakgiusti  ^17:41
kgiustiharlowja: YOWZA!17:41
harlowjahmmm17:41
harlowjais that right, ha17:41
harlowjaweird17:42
harlowjalol17:42
kgiustiharlowja: hrm - I would've expected a bit more :)17:42
harlowjalol17:42
harlowjaya17:42
kgiustione sec..17:43
harlowjak, pasting the commands i did17:43
*** hockeynut has quit IRC17:44
harlowjahttp://paste.openstack.org/show/560838/17:44
harlowjanothing special17:44
harlowjabut ya, the review seems sorta ummm, small17:45
harlowjalol17:45
kgiustiharlowja: ah, I did it a bit differently17:45
kgiustiharlowja: one sec...17:45
harlowjathough when i look at `git log` it seems to show all your stuff17:45
harlowjaodd, ha17:46
*** ekcs has left #openstack-oslo17:46
kgiustiharlowja: I did the merge of the upstream branch (not a local one) e.g.: git merge --no-ff origin/feature/amqp-dispatch-router17:48
harlowjaright17:48
harlowjashould be the same17:48
harlowjabut let me try that approach17:48
kgiustiharlowja: not aware of why there's no other stuff showing up17:48
harlowjasee if anything chages, ha17:48
kgiustiharlowja: hrm... when I resync the branch to master, the review only showed where I had to fix conflicts17:50
openstackgerritJoshua Harlow proposed openstack/oslo.messaging: Merge remote-tracking branch 'origin/feature/amqp-dispatch-router' into merge-amqp1-dispatch-router  https://review.openstack.org/35737617:50
kgiustiharlowja: not all the changes that merged in17:50
harlowjalol, ya doing what u did the same thing :-P17:50
kgiustiharlowja: crap17:50
harlowjamaybe this is how it works :-/17:50
harlowjalet me try one other approach, ha17:51
kgiustiharlowja: doesn't seem to helpful, review-wise, eh?17:51
*** rcernin has joined #openstack-oslo17:51
harlowjalol17:52
harlowjamaybe thats how its supposed to be?17:52
harlowjaso kgiusti https://review.openstack.org/#/c/357382/ is what i do without any git-review thing17:53
patchbotharlowja: patch 357382 - oslo.messaging (feature/amqp-dispatch-router) - Merge remote-tracking branch 'origin/feature/amqp-...17:53
harlowjapretty much a blank review17:53
harlowjathats via http://paste.openstack.org/show/560840/17:54
harlowjaso i'm thinking this is just the normal way this works17:54
harlowja@fungi u got a sec?17:54
kgiustiugh17:54
fungicatching up17:55
harlowjathx fungi , basically just a merge feature branch question17:55
harlowjawondering if the result of http://paste.openstack.org/show/560838/ (which ends up as https://review.openstack.org/#/c/357376/ ) is correct17:55
patchbotharlowja: patch 357376 - oslo.messaging - Merge remote-tracking branch 'origin/feature/amqp-...17:55
kgiustiharlowja: seems like the same wrt the pika merge: https://review.openstack.org/#/c/264768/17:55
patchbotkgiusti: patch 264768 - oslo.messaging - Merge remote-tracking branch 'origin/feature/pika'... (MERGED)17:55
fungii can probably answer this by saying that what gerrit's webui diff view shows is not the difference between the parents in the merge commit, but any merge conflict resolution involved in the merge commit17:55
harlowjaokie dokie17:55
harlowjafungi k17:56
harlowjawhich would be the .gitreview file i guess17:56
kgiusti'zactly what I saw when I resync the branch to master17:56
fungithe best way to validate it is to retrieve that from review.o.o yourself and git diff things like HEAD and the previous branch state, or the parent commits of the merge, or whatever you're worried about17:56
harlowjaright, looked at git log17:57
harlowjaand it seemed right17:57
harlowjaguess its just UI weirdness then17:57
fungiyeah, it's an intentional design choice on the part of gerrit's dev team, fwiw17:57
fungibut everyone i know finds it confusing17:57
harlowjaokie dokie17:59
harlowjathx fungi17:59
harlowjaalright kgiusti i guess https://review.openstack.org/#/c/357376/ is the winner then :-P17:59
patchbotharlowja: patch 357376 - oslo.messaging - Merge remote-tracking branch 'origin/feature/amqp-...17:59
kgiustiharlowja: well that's a quick review then17:59
kgiusti:P17:59
harlowjaha17:59
fungiany time harlowja!18:00
fungii'm full of fun stories about gerrit's intentional design shortcomings ;)18:00
harlowjafungi i bet18:03
harlowjai can tell u some stuff about jenkins pipelines now :-P18:04
harlowjacollecting my fun stories about that, ha18:04
harlowjaits intersting18:04
harlowjalike 80% functional imho18:04
fungiluckily i can enjoy jenkins horror stories now rather than spend my time checking whether we're appropriately mitigating them18:04
harlowja;)18:07
harlowjau can just listen in fungi18:07
harlowjaha18:07
*** mpotdar_ has joined #openstack-oslo18:09
*** yamahata has quit IRC18:10
kgiustiharlowja: thanks for the help.18:11
kgiustiharlowja: hopefully zuul won't be as grumpy as it has lately...18:11
harlowjakgiusti :)18:11
harlowjanp18:11
*** jecarey has quit IRC18:13
*** jecarey has joined #openstack-oslo18:15
harlowjaSpamapS why u not in #openstack-architecture18:21
harlowjajeez18:21
harlowjau mr.architect18:22
harlowjalol18:22
SpamapShahaha18:22
SpamapSthere's an architecture channel?18:22
SpamapSjoin #openstack-architecture18:22
harlowja:-p18:22
*** mpotdar_ has quit IRC18:24
*** mpotdar_ has joined #openstack-oslo18:25
dansmithharlowja: dims: I have my grubby fingers all over this, but it's good to go and we should get it for newton: https://review.openstack.org/#/c/355981/18:25
patchbotdansmith: patch 355981 - oslo.versionedobjects - Fix remotable object change tracking18:25
dansmithif you or others could/would +2, that would be cool18:26
harlowjadansmith cool, checking out18:32
harlowjadansmith u should write a little book on oslo.versionedobjects18:33
harlowjalike one of those pocket guides :-P18:33
harlowjai'd buy it18:33
dansmithI could write on microfiche and make it a nano guide18:33
harlowjathat'd work18:33
dansmithheh18:33
harlowjau must have really tiny fingers to write on microfiche18:33
harlowja:-P18:33
harlowjananofingers18:34
dansmithheh18:34
harlowjawith a really small pen18:34
dansmithI already do this: https://xkcd.com/378/18:34
dansmithneedles are small18:34
harlowja:-P18:35
*** hockeynut has joined #openstack-oslo18:36
*** mpotdar_ has quit IRC18:38
*** mpotdar_ has joined #openstack-oslo18:38
openstackgerritBrant Knudson proposed openstack/oslo.cache: Correct help text for backend option  https://review.openstack.org/35740718:40
*** yamahata has joined #openstack-oslo18:46
*** dtardivel has quit IRC18:47
*** lpetrut has joined #openstack-oslo18:47
*** ociuhandu has quit IRC18:47
openstackgerritGeorge Silvis, III proposed openstack/oslo.cache: Expand usage example (WIP)  https://review.openstack.org/35741218:52
*** rlrossit has quit IRC18:52
*** mpotdar_ has quit IRC18:54
*** mpotdar_ has joined #openstack-oslo18:54
*** rlrossi__ has joined #openstack-oslo18:55
openstackgerritGiridhar Jayavelu proposed openstack/oslo.vmware: Improve logging for task updates  https://review.openstack.org/35021818:58
*** jecarey has quit IRC19:05
*** ozamiatin_ has joined #openstack-oslo19:05
*** jecarey has joined #openstack-oslo19:11
*** jecarey has quit IRC19:11
*** jecarey has joined #openstack-oslo19:12
*** salv-orl_ has joined #openstack-oslo19:15
*** salv-orlando has quit IRC19:18
openstackgerritGeorge Silvis, III proposed openstack/oslo.cache: Add usage example to documentation  https://review.openstack.org/35741219:29
*** thumpba_ has quit IRC19:29
*** e0ne has joined #openstack-oslo19:34
*** thumpba has joined #openstack-oslo19:39
openstackgerritPatrick East proposed openstack/oslo.concurrency: Add info for shared lock_path config in description  https://review.openstack.org/35526919:42
*** ociuhandu has joined #openstack-oslo19:45
*** srini has quit IRC19:45
*** mpotdar_ has quit IRC19:56
*** mpotdar_ has joined #openstack-oslo19:57
openstackgerritGeorge Silvis, III proposed openstack/oslo.cache: Add usage example to documentation  https://review.openstack.org/35741219:59
*** thumpba has quit IRC20:04
*** mpotdar_ has quit IRC20:12
*** mpotdar_ has joined #openstack-oslo20:12
*** mpotdar_ has quit IRC20:12
*** mpotdar_ has joined #openstack-oslo20:13
harlowjasputnik13 do u know if the cue jobs that are failing @ http://logs.openstack.org/periodic/periodic-cue-py27-with-oslo-master/1949588/ will be fixed? should we turn off that periodic job?20:22
harlowjaseems oslo.context releated20:23
*** kgiusti has left #openstack-oslo20:25
openstackgerritGeorge Silvis, III proposed openstack/oslo.cache: Fix docstring for get_memoization_decorator  https://review.openstack.org/35746020:29
*** hemanthm is now known as hemanthm|afk20:29
*** jeckersb is now known as jeckersb_gone20:34
*** rlrossi__ has quit IRC20:44
*** ansmith has quit IRC20:49
*** rlrossit has joined #openstack-oslo20:55
*** spzala has quit IRC20:56
*** jecarey has quit IRC20:59
*** e0ne has quit IRC21:04
*** yamamoto has joined #openstack-oslo21:09
*** yamamoto has quit IRC21:09
*** boden has quit IRC21:11
*** hockeynut has quit IRC21:12
*** edmondsw has quit IRC21:25
*** rlrossit has quit IRC21:31
*** cdent has quit IRC21:33
*** ozamiatin has quit IRC21:37
*** ozamiatin_ has quit IRC21:38
*** pradk has quit IRC21:45
*** lpetrut has quit IRC21:55
*** tbarron is now known as tbarron|afk22:02
*** gordc has quit IRC22:02
*** jeckersb_gone is now known as jeckersb22:05
*** mpotdar_ has quit IRC22:11
*** mpotdar_ has joined #openstack-oslo22:11
*** john51 has quit IRC22:36
*** john51 has joined #openstack-oslo22:39
*** mpotdar_ has quit IRC22:40
*** yamamoto has joined #openstack-oslo22:42
*** tonytan4ever has quit IRC22:47
*** mpotdar_ has joined #openstack-oslo23:04
*** mpotdar__ has joined #openstack-oslo23:04
*** dimtruck is now known as zz_dimtruck23:06
*** mpotdar_ has quit IRC23:06
*** sdague has quit IRC23:14
sputnik13harlowja didn't know those were on...  yes they will be fixed23:15
sputnik13"soon"23:15
harlowjasputnik13 cool23:15
sputnik13:)23:15
sputnik13don't ask me what "soon" means23:15
sputnik13:)23:15
harlowjadepends on your timescale, ha23:16
openstackgerritJamie Lennox proposed openstack/oslo.context: Fix X-User and X-Tenant deprecated headers in from_environ  https://review.openstack.org/35753723:19
*** rcernin has quit IRC23:32
*** syjulian has quit IRC23:39
*** eeiden has quit IRC23:40
*** eeiden has joined #openstack-oslo23:40
*** tonytan4ever has joined #openstack-oslo23:47
*** tonytan4ever has quit IRC23:53

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