Thursday, 2018-03-22

*** salv-orlando has joined #openstack-oslo00:27
*** salv-orlando has quit IRC00:32
*** tovin07_ has joined #openstack-oslo00:43
*** yamamoto has joined #openstack-oslo00:52
*** yamamoto has quit IRC00:57
*** crushil has joined #openstack-oslo01:02
*** salv-orlando has joined #openstack-oslo01:27
*** harlowja has quit IRC01:31
*** salv-orlando has quit IRC01:32
*** yamamoto has joined #openstack-oslo01:53
*** hoonetorg has quit IRC01:56
*** hoonetorg has joined #openstack-oslo01:57
*** eck` is now known as eck`gone01:59
*** yamamoto has quit IRC01:59
*** salv-orlando has joined #openstack-oslo02:28
*** salv-orlando has quit IRC02:34
*** yamamoto has joined #openstack-oslo02:37
*** edmondsw has joined #openstack-oslo02:42
*** yamamoto has quit IRC02:46
*** edmondsw has quit IRC02:49
*** yamamoto has joined #openstack-oslo02:56
*** dave-mccowan has quit IRC03:11
*** AlexeyAbashkin has joined #openstack-oslo03:17
*** AlexeyAbashkin has quit IRC03:21
*** salv-orlando has joined #openstack-oslo03:30
*** harlowja has joined #openstack-oslo03:32
*** salv-orlando has quit IRC03:35
*** d0ugal_ has joined #openstack-oslo03:39
*** d0ugal has quit IRC03:39
*** yikun has joined #openstack-oslo03:44
*** harlowja has quit IRC03:54
*** edmondsw has joined #openstack-oslo04:05
*** edmondsw has quit IRC04:10
*** chhagarw has joined #openstack-oslo04:13
*** salv-orlando has joined #openstack-oslo04:31
*** salv-orlando has quit IRC04:36
*** crushil has quit IRC04:46
*** janzian has quit IRC05:22
*** gyankum has joined #openstack-oslo05:26
*** jaosorior_ is now known as jaosorior05:26
*** salv-orlando has joined #openstack-oslo05:29
*** rcernin has quit IRC05:34
*** rcernin has joined #openstack-oslo05:46
*** links has joined #openstack-oslo05:53
*** edmondsw has joined #openstack-oslo05:54
*** edmondsw has quit IRC05:59
*** rcernin has quit IRC06:06
*** rcernin has joined #openstack-oslo06:08
*** e0ne has joined #openstack-oslo06:15
*** AlexeyAbashkin has joined #openstack-oslo06:16
*** lucas-afk has quit IRC06:20
*** AlexeyAbashkin has quit IRC06:21
*** lucasagomes has joined #openstack-oslo06:23
*** threestrands has quit IRC06:25
*** msimonin has joined #openstack-oslo06:32
*** e0ne has quit IRC06:33
*** msimonin has quit IRC06:33
*** e0ne has joined #openstack-oslo06:46
*** e0ne has quit IRC06:52
*** msimonin has joined #openstack-oslo07:06
*** d0ugal_ has quit IRC07:14
*** d0ugal has joined #openstack-oslo07:14
*** d0ugal has quit IRC07:14
*** d0ugal has joined #openstack-oslo07:14
*** links has quit IRC07:15
*** links has joined #openstack-oslo07:23
*** jaosorior has quit IRC07:25
*** slaweq has quit IRC07:25
*** sambetts|afk has quit IRC07:25
*** HenryG has quit IRC07:25
*** Anticimex has quit IRC07:25
*** jungleboyj has quit IRC07:25
*** HenryG_ has joined #openstack-oslo07:26
*** jaosorior has joined #openstack-oslo07:26
*** slaweq has joined #openstack-oslo07:26
*** jungleboyj has joined #openstack-oslo07:27
*** gagehugo has quit IRC07:28
*** sambetts_ has joined #openstack-oslo07:29
*** rcernin has quit IRC07:31
*** salv-orlando has quit IRC07:33
*** gagehugo has joined #openstack-oslo07:35
*** edmondsw has joined #openstack-oslo07:42
*** pcaruana has joined #openstack-oslo07:42
*** pcaruana has quit IRC07:44
*** pcaruana has joined #openstack-oslo07:44
*** pcaruana has quit IRC07:45
*** pcaruana has joined #openstack-oslo07:45
*** edmondsw has quit IRC07:46
*** pcaruana has quit IRC07:47
*** pcaruana has joined #openstack-oslo07:47
*** pcaruana has quit IRC07:48
*** pcaruana has joined #openstack-oslo07:48
*** pcaruana has quit IRC07:50
*** pcaruana has joined #openstack-oslo07:50
*** pcaruana has quit IRC07:51
*** pcaruana has joined #openstack-oslo07:51
*** pcaruana has quit IRC07:53
*** pcaruana has joined #openstack-oslo07:53
*** pcaruana has quit IRC07:54
*** pcaruana has joined #openstack-oslo07:55
*** AlexeyAbashkin has joined #openstack-oslo07:56
*** pcaruana has quit IRC07:56
*** aspiers[m] has quit IRC07:58
*** msimonin has quit IRC07:58
*** pblaho has joined #openstack-oslo08:05
*** dstanek has quit IRC08:06
*** pcaruana has joined #openstack-oslo08:06
*** aspiers[m] has joined #openstack-oslo08:07
*** pcaruana has quit IRC08:07
*** pcaruana has joined #openstack-oslo08:08
*** pcaruana has quit IRC08:09
*** pcaruana has joined #openstack-oslo08:09
*** pcaruana has quit IRC08:10
*** pcaruana has joined #openstack-oslo08:11
*** tesseract has joined #openstack-oslo08:11
*** dstanek has joined #openstack-oslo08:11
*** pcaruana has quit IRC08:12
*** pcaruana has joined #openstack-oslo08:13
*** pcaruana has quit IRC08:15
*** pcaruana has joined #openstack-oslo08:15
*** pcaruana has quit IRC08:16
*** AlexeyAbashkin has quit IRC08:17
*** pcaruana has joined #openstack-oslo08:18
*** AlexeyAbashkin has joined #openstack-oslo08:18
*** pcaruana has quit IRC08:20
*** pcaruana has joined #openstack-oslo08:20
*** pcaruana has quit IRC08:21
*** pcaruana has joined #openstack-oslo08:21
*** pcaruana has quit IRC08:22
*** pblaho has quit IRC08:24
*** snapiri has joined #openstack-oslo08:25
*** pcaruana has joined #openstack-oslo08:29
*** msimonin has joined #openstack-oslo08:29
*** pcaruana has quit IRC08:30
*** salv-orlando has joined #openstack-oslo08:34
*** pcaruana has joined #openstack-oslo08:36
*** pcaruana has quit IRC08:37
*** pcaruana has joined #openstack-oslo08:38
*** pcaruana has quit IRC08:39
*** pcaruana has joined #openstack-oslo08:40
*** salv-orlando has quit IRC08:40
*** pcaruana has quit IRC08:40
*** pcaruana has joined #openstack-oslo08:41
*** pcaruana has quit IRC08:41
*** e0ne has joined #openstack-oslo08:56
*** pcaruana has joined #openstack-oslo09:02
*** pcaruana has quit IRC09:03
*** pcaruana has joined #openstack-oslo09:03
*** pcaruana has quit IRC09:05
*** e0ne has quit IRC09:07
*** e0ne has joined #openstack-oslo09:11
*** e0ne has quit IRC09:15
*** edmondsw has joined #openstack-oslo09:30
*** edmondsw has quit IRC09:35
*** salv-orlando has joined #openstack-oslo09:35
openstackgerritStephen Finucane proposed openstack/oslo.cache master: Revert "Follow the new PTI for document build"  https://review.openstack.org/55486609:36
*** salv-orlando has quit IRC09:38
*** salv-orlando has joined #openstack-oslo09:38
*** pblaho has joined #openstack-oslo09:44
*** dkehn_ has quit IRC10:06
*** e0ne has joined #openstack-oslo10:10
*** shardy has joined #openstack-oslo10:15
*** tovin07_ has quit IRC10:25
*** pbourke has joined #openstack-oslo10:43
openstackgerritMerged openstack/oslo.config master: Remove MultiConfigParser  https://review.openstack.org/55461810:51
openstackgerritMerged openstack/oslo.middleware master: Updated from global requirements  https://review.openstack.org/55006210:55
*** AlexeyAbashkin has quit IRC11:00
*** AlexeyAbashkin has joined #openstack-oslo11:00
*** ansmith has joined #openstack-oslo11:38
openstackgerritMerged openstack/oslo.config master: Follow the new PTI for document build  https://review.openstack.org/55503911:43
*** sambetts_ is now known as sambetts11:54
*** pblaho has quit IRC12:02
*** pblaho has joined #openstack-oslo12:02
*** zhouyaguo has joined #openstack-oslo12:03
*** edmondsw has joined #openstack-oslo12:15
*** lucasagomes is now known as lucas-hungry12:23
*** yassine has quit IRC12:24
*** yassine has joined #openstack-oslo12:25
*** e0ne has quit IRC12:31
*** e0ne has joined #openstack-oslo12:45
*** kgiusti has joined #openstack-oslo12:48
*** yamamoto has quit IRC12:54
openstackgerritDoug Hellmann proposed openstack/automaton master: add lower-constraints job  https://review.openstack.org/55529113:06
*** bobh has joined #openstack-oslo13:06
*** crushil has joined #openstack-oslo13:06
openstackgerritDoug Hellmann proposed openstack-dev/pbr master: add lower-constraints job  https://review.openstack.org/55529713:07
*** eck`gone is now known as eck`13:13
*** shardy_ has joined #openstack-oslo13:14
*** shardy has quit IRC13:16
*** yamamoto has joined #openstack-oslo13:21
*** lucas-hungry is now known as lucasagomes13:24
*** janzian has joined #openstack-oslo13:51
*** yamamoto has quit IRC13:53
*** tblakes has joined #openstack-oslo13:54
tblakesjlvillal: Is https://github.com/openstack/oslo.serialization/commit/c1a7079c26d27a2e46cca26963d3d9aa040bdbe8 necessary to fix a bug?13:56
tblakesPreviously exceptions were serialized using exception.__dict__, but now they are serialized using repr(exception).13:56
tblakesrepr(exception) by default returns a string that doesn't contain any kwargs included in the exception, whereas exception.__dict__ nicely returned the kwargs as a dict.13:57
*** shardy_ has quit IRC14:03
*** shardy has joined #openstack-oslo14:05
kgiustimsimonin: ping - I'm back14:07
kgiustimsimonin: has the meeting started?14:07
*** yamamoto has joined #openstack-oslo14:08
*** links has quit IRC14:10
*** yamamoto has quit IRC14:13
*** zhouyaguo has quit IRC14:13
*** spilla has joined #openstack-oslo14:17
*** yamamoto has joined #openstack-oslo14:24
*** yamamoto has quit IRC14:28
*** eck` is now known as eck`gone14:32
*** eck`gone is now known as eck`14:34
*** yamamoto has joined #openstack-oslo14:39
*** yamamoto has quit IRC14:43
*** yamamoto has joined #openstack-oslo14:46
*** yamamoto has quit IRC14:46
*** bobh has quit IRC14:57
*** gyankum has quit IRC14:58
*** msimonin has quit IRC14:58
*** msimonin has joined #openstack-oslo15:02
*** dave-mccowan has joined #openstack-oslo15:15
bnemectblakes: Prior to that change calling jsonutils.dumps on an Exception failed (at least some Exceptions).  There's some discussion on the review: https://review.openstack.org/#/c/508287/15:24
bnemecOne option would be to implement __repr__ on the exceptions that take extra kwargs.15:25
*** bobh has joined #openstack-oslo15:28
tblakesbnemec: Okay. The only problem with that is that repr() can only return a str so we need to then reparse the exception string to extract the kwargs. It's doable but it was nice when it was formatted as a dict.15:29
*** bobh has quit IRC15:30
*** bobh has joined #openstack-oslo15:30
bnemectblakes: Strictly speaking, a proper repr implementation is supposed to satisfy "eval(repr(c))==c", although depending on whether you can trust the input, running eval on the serialized data may not be safe.15:38
tblakesbnemec: That makes sense and gives me a good starting point for implementing repr(). Thank you!15:41
*** pcaruana has joined #openstack-oslo15:42
*** pcaruana has quit IRC15:44
bnemectblakes: np, let us know if you run into any other issues.  This has unfortunately turned out to be a somewhat disruptive change, as evidenced by the fact that a revert was already proposed once: https://review.openstack.org/#/c/534736/15:44
*** yamamoto has joined #openstack-oslo15:47
*** pcaruana has joined #openstack-oslo15:48
tblakesbnemec: I was originally going to propose a fix in nova, and one of the reviewers noted that the change was going to result in kwargs being stripped from exceptions in all of the components not just nova. So there will need to be a repr() method implemented for all exceptions.15:48
tblakesAll exceptions that have kwargs at least.15:49
*** yamamoto has quit IRC15:53
bnemectblakes: Yeah, I have to admit I didn't realize that serializing exceptions ever worked before this.  My local testing suggested it was just broken, IIRC.15:53
bnemecI would be open to alternative fixes for the circular reference, although we'd need to coordinate with the affected projects to make sure it works for them.15:54
tblakesbnemec: I will reach out to some people on the nova side to start and see what they say. I don't know enough about the circular reference to propose an alternative solution. I guess we can weigh how hard it is to implement repr() verse getting a new fix for the circular reference.15:56
bnemectblakes: Sounds good15:56
jlvillaltblakes, As a note. If don't convert it to a repr in the future it would be broken a different way. According to code comments.16:00
jlvillalhttps://github.com/openstack/oslo.serialization/blob/0ddc7f9a2e441dcaa198fd50b1f26644fe4b190c/oslo_serialization/jsonutils.py#L190-L19216:00
jlvillalBut yeah we were seeing these circular reference errors without the change. Which broke things.16:01
tblakesjlvillal: So this code will be removed in version 3.0? https://github.com/openstack/oslo.serialization/blob/0ddc7f9a2e441dcaa198fd50b1f26644fe4b190c/oslo_serialization/jsonutils.py#L178-L18116:03
tblakesBecause that's the path we were going down prior to your change.16:04
jlvillaltblakes, got me16:04
jlvillaltblakes, I think at https://github.com/openstack/oslo.serialization/blob/0ddc7f9a2e441dcaa198fd50b1f26644fe4b190c/oslo_serialization/jsonutils.py#L190-L19216:04
jlvillaltblakes, It will raise a ValueError()16:04
*** harlowja has joined #openstack-oslo16:04
jlvillaltblakes, Which if I recall it did get there at some point. At least months ago when I was working on this.16:05
jlvillaltblakes, Who knows if they actually do that though.16:05
*** msimonin has quit IRC16:05
tblakesjlvillal: Okay - is the summary of this that we really don't want to rollback your change? I'm trying to figure out if that option is off the table before I start talking with other projects like nova.16:07
*** msimonin has joined #openstack-oslo16:07
jlvillaltblakes, I don't want to rollback the change as then we get circular reference errors.16:07
jlvillaltblakes, How is that useful?16:07
tblakesjlvillal: Fair enough. I'll check with nova about implementing repr(). I wanted to do my due diligence before I went to them because I know they're going to push back because this change affects a fair number of projects.16:09
*** harlowja has quit IRC16:09
jlvillaltblakes, Thanks16:10
*** tblakes has quit IRC16:16
*** yamamoto has joined #openstack-oslo16:17
*** yamamoto has quit IRC16:22
openstackgerritBen Nemec proposed openstack-dev/pbr master: Explicitly read setup.cfg as utf-8 on Python 3  https://review.openstack.org/55536616:24
*** bobh has quit IRC16:29
*** yamamoto has joined #openstack-oslo16:32
*** AlexeyAbashkin has quit IRC16:34
*** yamamoto has quit IRC16:36
*** crushil has quit IRC16:37
msimoninkgiusti: https://bugs.launchpad.net/oslo.messaging/+bug/1758100 :-)16:38
openstackLaunchpad bug 1758100 in oslo.messaging "amqp1.0 driver - memory increase in RPC client " [Undecided,New]16:38
kgiustimsimonin: great - thanks I'll look into it.16:39
*** msimonin has quit IRC16:39
*** msimonin has joined #openstack-oslo16:44
*** yamamoto has joined #openstack-oslo16:47
*** msimonin has quit IRC16:48
*** yamamoto has quit IRC16:52
*** yamamoto has joined #openstack-oslo17:02
*** yamamoto has quit IRC17:06
*** yamamoto has joined #openstack-oslo17:17
*** yamamoto has quit IRC17:22
*** sambetts is now known as sambetts|afk17:27
*** yamamoto has joined #openstack-oslo17:32
*** yamamoto has quit IRC17:37
*** e0ne has quit IRC17:39
*** lucasagomes is now known as lucas-afk17:40
*** yamamoto has joined #openstack-oslo17:47
*** Anticimex has joined #openstack-oslo17:50
*** pblaho has quit IRC17:50
*** yamamoto has quit IRC17:52
*** yamamoto has joined #openstack-oslo18:02
*** yamamoto has quit IRC18:06
*** yamamoto has joined #openstack-oslo18:06
*** yamamoto has quit IRC18:06
*** boris_42_ has joined #openstack-oslo18:13
boris_42_zzzeek: hi there18:13
boris_42_zzzeek: we at Rally are facing some strange issue with sqlalchemy/alembic18:14
*** harlowja has joined #openstack-oslo18:15
zzzeek boris_42_ I'm actually on the road right now can you send an email either to me or alembic or openstack list18:15
*** AlexeyAbashkin has joined #openstack-oslo18:15
*** harlowja_ has joined #openstack-oslo18:17
*** harlowja has quit IRC18:19
*** AlexeyAbashkin has quit IRC18:20
*** dave-mccowan has quit IRC18:20
openstackgerritDoug Hellmann proposed openstack/debtcollector master: add lower-constraints job  https://review.openstack.org/55540918:26
*** tesseract has quit IRC18:32
*** msimonin has joined #openstack-oslo18:46
*** yamamoto has joined #openstack-oslo19:07
*** yamamoto has quit IRC19:13
*** yamamoto has joined #openstack-oslo20:09
*** yamamoto has quit IRC20:14
openstackgerritKen Giusti proposed openstack/oslo.messaging master: DO NOT MERGE - debugging a gate failure  https://review.openstack.org/55465820:24
openstackgerritDoug Hellmann proposed openstack/futurist master: add lower-constraints job  https://review.openstack.org/55544720:25
*** syjulian has quit IRC20:33
*** rarora has quit IRC20:33
*** boris_42_ has quit IRC20:41
*** AlexeyAbashkin has joined #openstack-oslo20:45
openstackgerritKen Giusti proposed openstack/oslo.messaging master: DO NOT MERGE - debugging a gate failure  https://review.openstack.org/55465820:48
*** AlexeyAbashkin has quit IRC20:52
*** edmondsw has quit IRC20:56
*** edmondsw has joined #openstack-oslo20:56
*** spilla has quit IRC20:57
*** spilla has joined #openstack-oslo20:57
*** spilla has quit IRC20:58
*** edmondsw has quit IRC21:01
*** kgiusti has left #openstack-oslo21:07
*** yamamoto has joined #openstack-oslo21:10
*** chhagarw has quit IRC21:12
*** yamamoto has quit IRC21:16
*** AlexeyAbashkin has joined #openstack-oslo21:17
*** salv-orlando has quit IRC21:19
*** salv-orlando has joined #openstack-oslo21:19
*** AlexeyAbashkin has quit IRC21:21
*** raildo has quit IRC21:24
*** salv-orlando has quit IRC21:24
*** salv-orlando has joined #openstack-oslo21:26
*** ansmith has quit IRC21:27
*** pcaruana has quit IRC21:53
*** yamamoto has joined #openstack-oslo21:55
*** ihrachys has left #openstack-oslo22:17
*** crushil has joined #openstack-oslo22:25
*** crushil has left #openstack-oslo22:26
*** rcernin has joined #openstack-oslo22:34
*** edmondsw has joined #openstack-oslo22:46
*** AlexeyAbashkin has joined #openstack-oslo23:16
*** ansmith has joined #openstack-oslo23:19
*** AlexeyAbashkin has quit IRC23:20
*** e0ne has joined #openstack-oslo23:22
*** e0ne_ has joined #openstack-oslo23:54
*** e0ne has quit IRC23:57

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