Wednesday, 2019-02-27

*** tosky has quit IRC00:02
*** edmondsw has quit IRC00:17
*** agopi has joined #openstack-oslo00:25
*** nsmeds has quit IRC00:27
*** lbragstad has quit IRC00:59
*** nsmeds has joined #openstack-oslo01:15
*** takamatsu_ has quit IRC01:39
*** edmondsw has joined #openstack-oslo01:43
*** takamatsu_ has joined #openstack-oslo01:45
*** lbragstad has joined #openstack-oslo01:58
*** rcernin has quit IRC02:32
*** takamatsu_ has quit IRC02:53
*** dkehn has quit IRC04:56
*** dave-mccowan has quit IRC05:05
*** njohnston has quit IRC06:27
*** Luzi has joined #openstack-oslo06:47
*** e0ne has joined #openstack-oslo07:09
*** aojea has joined #openstack-oslo07:16
*** e0ne has quit IRC07:16
*** takamatsu_ has joined #openstack-oslo07:21
*** e0ne has joined #openstack-oslo07:57
*** lbragstad has quit IRC08:01
*** pcaruana has joined #openstack-oslo08:13
*** pcaruana has quit IRC08:28
*** hberaud|gone is now known as hberaud08:31
*** takamatsu_ has quit IRC08:31
*** tosky has joined #openstack-oslo08:41
*** pcaruana has joined #openstack-oslo08:42
*** pcaruana has quit IRC08:51
*** e0ne has quit IRC08:57
*** pcaruana has joined #openstack-oslo08:58
*** pcaruana|afk| has joined #openstack-oslo09:01
*** pcaruana has quit IRC09:03
*** takamatsu has joined #openstack-oslo09:15
*** shachar has quit IRC09:28
*** Luzi has quit IRC09:36
*** takamatsu has quit IRC09:44
*** Luzi has joined #openstack-oslo09:51
*** snapiri has joined #openstack-oslo10:07
*** moguimar has joined #openstack-oslo11:16
*** e0ne has joined #openstack-oslo11:16
*** takamatsu has joined #openstack-oslo11:24
*** hberaud is now known as hberaud|lunch11:31
*** a-pugachev has joined #openstack-oslo11:46
*** a-pugachev has quit IRC11:46
*** a-pugachev has joined #openstack-oslo11:47
*** a-pugachev has quit IRC12:00
*** Luzi has quit IRC12:00
*** Luzi has joined #openstack-oslo12:02
*** a-pugachev has joined #openstack-oslo12:06
*** raildo has joined #openstack-oslo12:22
openstackgerritRodolfo Alonso Hernandez proposed openstack/oslo.privsep master: Convert dict keys received in _ClientChannel from byte to str  https://review.openstack.org/63821612:30
*** hberaud|lunch is now known as hberaud12:37
*** ansmith has quit IRC12:43
*** agopi has quit IRC12:59
*** njohnston has joined #openstack-oslo13:01
*** dave-mccowan has joined #openstack-oslo13:04
*** pcaruana|afk| has quit IRC13:09
*** kgiusti has joined #openstack-oslo13:20
*** a-pugachev has quit IRC13:29
*** a-pugachev has joined #openstack-oslo13:30
openstackgerritHervé Beraud proposed openstack-dev/pbr master: Fix error when keywords are defined as a list in cfg  https://review.openstack.org/63966113:31
*** agopi has joined #openstack-oslo13:35
*** ansmith has joined #openstack-oslo13:36
*** a-pugachev has quit IRC13:39
*** agopi has quit IRC13:40
*** a-pugachev has joined #openstack-oslo13:41
*** a-pugachev has quit IRC13:43
*** a-pugachev has joined #openstack-oslo13:44
openstackgerritHervé Beraud proposed openstack-dev/pbr master: Fix error when keywords are defined as a list in cfg  https://review.openstack.org/63966113:48
*** e0ne has quit IRC13:50
*** e0ne has joined #openstack-oslo14:04
*** hberaud has quit IRC14:06
*** hberaud has joined #openstack-oslo14:06
*** agopi has joined #openstack-oslo14:08
*** Luzi has quit IRC14:17
*** lbragstad has joined #openstack-oslo14:20
*** moguimar has quit IRC14:31
*** moguimar has joined #openstack-oslo14:48
*** pcaruana has joined #openstack-oslo14:57
openstackgerritHervé Beraud proposed openstack-dev/pbr master: Fix error when keywords are defined as a list in cfg  https://review.openstack.org/63966115:21
openstackgerritHervé Beraud proposed openstack-dev/pbr master: Fix error when keywords are defined as a list in cfg  https://review.openstack.org/63966115:39
*** dmellado has quit IRC15:42
*** dmellado has joined #openstack-oslo15:43
kmallochberaud: saw your email. i think i was mistaken about the dogpile test in oslo.15:48
kmallochberaud: it was sane to just delete it.15:49
hberaudack15:49
hberaudkmalloc: we have discuss about this yesterday with zzzeek and bnemec15:50
hberaudkmalloc: we want to fix and keep test in oslo.cache side in a first time15:50
hberaudkmalloc: and migrate test into dogpile.cache instead these tests are out of the oslo scope15:51
hberaudkmalloc: https://github.com/sqlalchemy/dogpile.cache/pull/14515:51
bnemecI think we would be covered to just remove it at this point too. dogpile covers the kwarg case in https://github.com/sqlalchemy/dogpile.cache/blob/master/tests/cache/test_decorator.py#L21915:51
hberaudbnemec: it's not really the same test15:52
hberaudbnemec: this test directly use dogpile.cache utils15:53
kmalloc++15:53
hberaudbnemec: and they don't cover the decorator problem15:53
bnemechberaud: It's verifying that the decorator raises when it gets kwargs.15:53
bnemecThe change here was that it doesn't get kwargs in some cases where it used to.15:53
kmallochberaud: i just wanted to be sure you knew I read the email, but here not just the @rh side ;)15:53
hberaudsure but not really in the same inception way15:53
hberaudkmalloc: thx :)15:55
*** a-pugachev has quit IRC15:58
*** a-pugachev has joined #openstack-oslo16:07
*** takamatsu has quit IRC16:09
openstackgerritRodolfo Alonso Hernandez proposed openstack/oslo.privsep master: Convert dict keys received in _ClientChannel from byte to str  https://review.openstack.org/63821616:17
*** aojea has quit IRC16:20
openstackgerritBen Nemec proposed openstack/oslo.cache master: Fix memcache pool client in monkey-patched environments  https://review.openstack.org/63945716:23
openstackgerritBen Nemec proposed openstack/oslo.cache master: Fix memcache issue due to the __new__ attribute reassign.  https://review.openstack.org/63445716:23
openstackgerritBen Nemec proposed openstack/oslo.cache master: Remove dogpile kwarg test  https://review.openstack.org/63873216:23
* bnemec crosses his fingers that he didn't screw up that rebase16:24
bnemecHey, looks good.16:24
bnemechberaud: I rebased everything in the order that I think they should merge^16:25
hberaudbnemec: great! thanks16:25
bnemecAll of the votes carried over so I don't think I hosed anything up. :-)16:25
hberaudack16:25
bnemecI did drop your WIP followup. It wasn't current with the latest patch set of the dependencies anyway.16:26
bnemecIt's still there of course, it's just not in the series.16:26
hberaudbnemec: ack16:27
*** dmellado has quit IRC16:32
*** dmellado has joined #openstack-oslo16:34
kmallocbnemec: i think i prefer pymemcache fix to eventlet-specific hacky-code16:39
kmallocbut i'll support either16:39
bnemeckmalloc: The hack is only for backportability.16:44
bnemecI want it to die in a fire ASAP.16:45
bnemecI rebased everything in order on https://review.openstack.org/#/q/topic:all-the-fixes+(status:open+OR+status:merged)16:46
*** pcaruana has quit IRC16:57
*** e0ne has quit IRC17:00
*** a-pugachev has quit IRC17:01
*** a-pugachev has joined #openstack-oslo17:04
kmallocWill +2 the stack shortly17:13
kmallocAt the doctor's office right now17:13
kmallocBack in 30-40m17:13
bnemecack, thanks17:28
openstackgerritHervé Beraud proposed openstack/oslo.cache master: Switch from python-memcached to pymemcache.  https://review.openstack.org/63445717:32
*** dims has quit IRC17:35
*** takamatsu has joined #openstack-oslo17:41
hberaudbnemec: changes on mine done17:41
*** dims has joined #openstack-oslo17:48
moguimarbnemec: should I update other oslo projects accortding to https://review.openstack.org/#/c/639655/ ?17:51
bnemecmoguimar: No. Did you see the latest responses to your thread?17:52
bnemecThere are still quite a few projects with py35 jobs running because of the bionic migration not being complete yet.17:52
bnemecSo I'd prefer to hold off on the Oslo libs so we don't accidentally break someone.17:53
bnemecI don't think we need to add py35 back to any projects that removed it, but let's not make any more of those changes for now.17:53
*** a-pugachev has quit IRC17:54
*** takamatsu has quit IRC18:01
moguimarok18:02
*** moguimar has quit IRC18:03
*** hberaud is now known as hberaud|gone18:29
openstackgerritMerged openstack/oslo.cache master: fix dogpile.cache issue due to the new usage of decorate in the lib  https://review.openstack.org/63878818:29
*** takamatsu has joined #openstack-oslo18:52
bnemec\o/18:57
*** e0ne has joined #openstack-oslo18:58
*** lbragstad has quit IRC19:39
*** lbragstad has joined #openstack-oslo19:41
openstackgerritEric Harney proposed openstack/oslo.privsep master: Add sample_default for thread_pool_size Opt  https://review.openstack.org/63980419:51
njohnstonbnemec: Quick question as I try to dig into https://bugs.launchpad.net/neutron/+bug/181795319:57
openstackLaunchpad bug 1817953 in neutron "oslopolicy-policy-generator does not work for neutron" [Medium,New] - Assigned to Nate Johnston (nate-johnston)19:58
*** e0ne has quit IRC19:59
njohnstonWhen called as `oslopolicy-policy-generator --namespace neutron`, when we get to line 87 of oslo_policy/generator.py it looks like mgr[namespace] returns a KeyError.  Does this mean it is using a different setup.cfg than I am aware of?20:00
*** e0ne has joined #openstack-oslo20:09
bnemecnjohnston: I don't see anything obviously wrong. Maybe lbragstad will have an idea?20:12
lbragstadbnemec njohnston i can dig in here in just a minute - wrapping something up20:15
bnemecnjohnston: Actually, that's working for me in my local copy of neutron. Is this related to some other change?20:15
njohnstonbnemec: I tried it out on a plain-jane devstack I just pulled down20:17
*** e0ne has quit IRC20:17
bnemecnjohnston: Does the tox target work for you?20:18
lbragstadlocally - i get this https://pasted.tech/pastes/d161c315bc0ed89481adb9f70de2aac9bdbf7265.raw20:22
lbragstadi'm running from 376fa8aef49fa45c7038be4648aecd6899fb8d4e20:22
openstackgerritHervé Beraud proposed openstack/oslo.cache master: Switch from python-memcached to pymemcache.  https://review.openstack.org/63445720:26
*** e0ne has joined #openstack-oslo20:30
njohnstonI am running from the same git hash20:30
bnemecnjohnston: Very strange. Are you running it out of the tox venv or directly on the devstack system?20:32
bnemecMaybe devstack is not installing something that is needed for policy generation?20:32
*** e0ne has quit IRC20:33
*** e0ne has joined #openstack-oslo20:35
njohnstonbnemec: devstack directly.  If I do a `tox -e genconfig` it works fine.20:35
njohnstonbnemec: could this occur if devstack is installing under python3 but oslopolicy-sample-generator is installing under python 2.7?20:35
bnemecnjohnston: I would think so. If it runs in a python environment where neutron was not installed then I could see it behaving like this.20:36
*** tosky has quit IRC20:36
lbragstadoh sure20:37
njohnstonbnemec: ok, let me try forcing oslopolicy-config-generator to py3 and see if it works20:37
*** dave-mccowan has quit IRC20:38
bnemecYep, if I install oslo.policy in a venv without neutron then I also get the KeyError.20:39
lbragstadfwiw - that error is familiar and i've seen it under similar circumstances20:42
lbragstadi wonder if it's worth adding some logging to make it a little more apparent to the user what's actually happening20:42
bnemecYeah, I'm working on a patch to provide a more useful error.20:47
lbragstadcool - i can review20:47
bnemecWriting unit tests takes so much longer than writing the actual code on simple fixes. :-)20:49
openstackgerritBen Nemec proposed openstack/oslo.policy master: Provide more specific error when namespace is missing  https://review.openstack.org/63982220:55
*** mmethot has quit IRC21:06
*** jaosorior has quit IRC21:08
*** redrobot has quit IRC21:25
*** ansmith has quit IRC21:33
*** Guest66940 has joined #openstack-oslo21:37
*** Guest66940 is now known as redrobot21:38
*** mmethot has joined #openstack-oslo21:42
*** e0ne has quit IRC21:54
*** agopi has quit IRC22:16
*** agopi has joined #openstack-oslo22:20
*** ansmith has joined #openstack-oslo22:30
*** agopi has quit IRC22:31
*** kgiusti has left #openstack-oslo22:34
*** raildo has quit IRC22:37
*** mmethot has quit IRC22:57
*** rcernin has joined #openstack-oslo23:06
*** dave-mccowan has joined #openstack-oslo23:18
*** harlowja has joined #openstack-oslo23:35

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