Monday, 2016-06-13

*** ozamiatin_ has quit IRC00:11
*** rcernin has quit IRC00:13
*** ozamiatin_ has joined #openstack-oslo01:08
*** yamamoto_ has joined #openstack-oslo02:05
*** zz_dimtruck is now known as dimtruck02:06
*** yamamoto_ has quit IRC02:33
*** yamamoto_ has joined #openstack-oslo02:37
*** ozamiatin_ has quit IRC02:44
*** yamamoto_ has quit IRC02:46
*** yamamoto_ has joined #openstack-oslo02:48
*** yamamoto_ has quit IRC02:50
*** gcb has quit IRC03:12
*** yamamoto_ has joined #openstack-oslo03:34
*** yamamoto_ has quit IRC03:36
*** dimtruck is now known as zz_dimtruck04:06
*** zz_dimtruck is now known as dimtruck04:16
*** lpetrut has joined #openstack-oslo04:19
*** dimtruck is now known as zz_dimtruck04:26
*** links has joined #openstack-oslo04:26
*** yamamoto_ has joined #openstack-oslo04:33
*** lpetrut has quit IRC05:10
*** salv-orlando has joined #openstack-oslo05:21
*** salv-orl_ has quit IRC05:21
*** salv-orl_ has joined #openstack-oslo05:29
*** salv-orlando has quit IRC05:31
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.cache: Imported Translations from Zanata  https://review.openstack.org/32858806:01
*** rcernin has joined #openstack-oslo06:14
*** rcernin has quit IRC06:18
*** rcernin has joined #openstack-oslo06:19
openstackgerrit赖寒雪 proposed openstack/oslo.db: Fix typos in comments and docstring  https://review.openstack.org/32885806:21
*** gcb has joined #openstack-oslo06:22
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.versionedobjects: Imported Translations from Zanata  https://review.openstack.org/32859506:42
*** jpena|off is now known as jpena06:51
*** e0ne has joined #openstack-oslo06:53
*** amotoki has joined #openstack-oslo07:00
*** tesseract has joined #openstack-oslo07:00
*** salv-orl_ has quit IRC07:07
*** salv-orlando has joined #openstack-oslo07:08
*** mhickey has joined #openstack-oslo07:12
*** nkrinner has joined #openstack-oslo07:19
*** ihrachys has joined #openstack-oslo07:23
*** e0ne has quit IRC07:37
*** gibi has joined #openstack-oslo07:44
*** e0ne has joined #openstack-oslo07:55
*** e0ne has quit IRC07:59
*** zzzeek has quit IRC08:00
*** permalac has joined #openstack-oslo08:00
*** zzzeek has joined #openstack-oslo08:00
*** shardy has joined #openstack-oslo08:18
*** shardy has quit IRC08:21
*** salv-orlando has quit IRC08:23
*** salv-orlando has joined #openstack-oslo08:24
openstackgerritliusheng proposed openstack/oslo.messaging: [Trival] fix a typo nit  https://review.openstack.org/32890008:35
*** lpetrut has joined #openstack-oslo08:38
*** yassine__ has joined #openstack-oslo08:45
*** cdent has joined #openstack-oslo08:59
openstackgerritliusheng proposed openstack/oslo.messaging: Remove deprecated "topic" in Notifier initialization  https://review.openstack.org/32891109:00
*** e0ne has joined #openstack-oslo09:22
*** pcaruana has joined #openstack-oslo09:27
*** amotoki has quit IRC09:31
*** yamamoto_ has quit IRC10:19
*** amotoki has joined #openstack-oslo10:19
*** ozamiatin has joined #openstack-oslo10:45
*** sputnik13_ has joined #openstack-oslo10:55
*** dukhlov has joined #openstack-oslo10:55
*** sputnik13_ has quit IRC11:00
amrithharlowja, ping!11:01
openstackgerritChangBo Guo(gcb) proposed openstack/oslo.utils: Raise ValueError when get_ipv6_addr_by_EUI64's parameters with empty string  https://review.openstack.org/32894911:02
amrithharlowja, at the OSLO meeting today I'd like to raise the issue of isotime() that I'd summarized in an email to the ML (http://markmail.org/message/xosf5h7gvhqdwtz7) and some code that is in https://review.openstack.org/#/c/326655/11:03
patchbotamrith: patch 326655 - trove - Handle isotime deprecation in oslo_utils.timeutils11:03
amriththanks! (and apologies in advance for bringing up everyones favorite topic).11:03
*** salv-orlando has quit IRC11:07
*** salv-orlando has joined #openstack-oslo11:08
openstackgerritChangBo Guo(gcb) proposed openstack/oslo.utils: Use is_valid_ipv4 in get_ipv6_addr_by_EUI64  https://review.openstack.org/32894911:18
*** dukhlov has quit IRC11:27
*** salv-orl_ has joined #openstack-oslo11:28
*** dukhlov has joined #openstack-oslo11:29
*** salv-orlando has quit IRC11:32
*** yamamoto has joined #openstack-oslo11:33
*** gordc has joined #openstack-oslo11:38
*** amoralej has joined #openstack-oslo11:41
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Start/Stop ZmqServer  https://review.openstack.org/32896311:42
openstackgerritChangBo Guo(gcb) proposed openstack/oslo.utils: Use is_valid_ipv4 in get_ipv6_addr_by_EUI64  https://review.openstack.org/32894911:42
*** yamamoto has quit IRC11:44
*** jpena is now known as jpena|lunch11:44
amoralejping jamielennox, could you take a look into https://review.openstack.org/#/c/328919/ when you have a chance, i'm not sure what's the best way to fix that11:44
patchbotamoralej: patch 328919 - keystonemiddleware - Fix circular import in auth_token11:44
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Start/Stop ZmqServer  https://review.openstack.org/32896311:46
*** amotoki has quit IRC11:49
*** sdague has joined #openstack-oslo11:49
*** fesp has joined #openstack-oslo11:52
*** yamamoto has joined #openstack-oslo11:53
*** pradk has joined #openstack-oslo12:06
*** fesp has quit IRC12:08
*** flaper87 has quit IRC12:09
*** mtanino has joined #openstack-oslo12:11
*** yamamoto has quit IRC12:13
*** flaper87 has joined #openstack-oslo12:15
*** flaper87 has quit IRC12:15
*** flaper87 has joined #openstack-oslo12:15
*** e0ne has quit IRC12:15
*** e0ne has joined #openstack-oslo12:15
*** sdague has quit IRC12:16
*** sdague has joined #openstack-oslo12:16
*** amotoki has joined #openstack-oslo12:16
*** kgiusti has joined #openstack-oslo12:21
*** amotoki_ has joined #openstack-oslo12:38
*** amotoki has quit IRC12:41
*** jpena|lunch is now known as jpena12:44
*** yamamoto has joined #openstack-oslo12:46
*** yamamoto has quit IRC12:47
*** amotoki_ has quit IRC12:48
*** e0ne has quit IRC12:48
*** e0ne has joined #openstack-oslo12:49
openstackgerritAlexis Lee proposed openstack/oslo.log: log: SpoolFilter and SpoolContextManager  https://review.openstack.org/32898812:51
openstackgerritAlexis Lee proposed openstack/oslo-specs: log: Spool request logs spec  https://review.openstack.org/22776612:52
*** yamamoto has joined #openstack-oslo12:56
*** yamamoto has quit IRC12:57
*** pradk has quit IRC13:04
*** yamamoto has joined #openstack-oslo13:13
*** edmondsw has joined #openstack-oslo13:15
*** yamamoto has quit IRC13:16
*** pblaho has joined #openstack-oslo13:19
*** yamamoto has joined #openstack-oslo13:28
*** yamamoto has quit IRC13:31
*** yamamoto has joined #openstack-oslo13:32
*** permalac has quit IRC13:37
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Properly stop ZmqServer  https://review.openstack.org/32896313:45
*** yamamoto has quit IRC13:48
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Properly stop ZmqServer  https://review.openstack.org/32896313:49
*** yamamoto has joined #openstack-oslo13:54
*** links has quit IRC13:54
*** tonytan4ever has joined #openstack-oslo13:57
*** pblaho has quit IRC14:01
*** jecarey has joined #openstack-oslo14:02
*** pblaho has joined #openstack-oslo14:03
*** e0ne has quit IRC14:04
*** e0ne has joined #openstack-oslo14:05
*** pradk has joined #openstack-oslo14:12
*** zz_dimtruck is now known as dimtruck14:14
*** dimtruck is now known as zz_dimtruck14:15
*** permalac has joined #openstack-oslo14:21
*** yamamoto has quit IRC14:24
openstackgerritBrian Stajkowski proposed openstack/oslo.config: Advanced Option  https://review.openstack.org/30435514:25
*** yamamoto has joined #openstack-oslo14:25
*** yamamoto has quit IRC14:28
*** dukhlov has quit IRC14:29
*** snizar_ has joined #openstack-oslo14:31
*** snizar has joined #openstack-oslo14:31
openstackgerritIvan Kolodyazhny proposed openstack/oslo.reports: Use file modification events instead of signal handler  https://review.openstack.org/26097614:32
*** snizar has quit IRC14:32
*** snizar_ has quit IRC14:32
*** yamamoto has joined #openstack-oslo14:32
*** sigmavirus24_awa is now known as sigmavirus2414:32
*** yamamoto has quit IRC14:35
*** zz_dimtruck is now known as dimtruck14:37
*** dtardivel has joined #openstack-oslo14:42
*** yamamoto has joined #openstack-oslo14:49
*** amotoki has joined #openstack-oslo14:51
*** yamamoto has quit IRC14:53
*** miqui has joined #openstack-oslo14:56
*** yamamoto has joined #openstack-oslo14:56
*** permalac has quit IRC14:58
*** tonytan4ever has quit IRC15:00
*** yamamoto has quit IRC15:05
*** yamamoto has joined #openstack-oslo15:07
*** yamamoto has quit IRC15:10
*** yamamoto has joined #openstack-oslo15:12
*** permalac has joined #openstack-oslo15:13
*** yamamoto has quit IRC15:17
*** lpetrut has quit IRC15:20
openstackgerritIvan Kolodyazhny proposed openstack/oslo.reports: Use file modification events instead of signal handler  https://review.openstack.org/26097615:25
*** mtanino has quit IRC15:28
*** mhickey has quit IRC15:28
*** tonytan4ever has joined #openstack-oslo15:34
*** harlowja_at_home has joined #openstack-oslo15:35
*** tesseract has quit IRC15:39
*** pblaho has quit IRC15:43
*** rama_y has joined #openstack-oslo15:44
*** xiangxinyong has quit IRC15:46
*** xiangxinyong has joined #openstack-oslo15:46
*** pblaho has joined #openstack-oslo15:57
openstackgerritAlexis Lee proposed openstack/oslo.log: log: SpoolFilter and SpoolContextManager  https://review.openstack.org/32898815:58
openstackgerritAlexis Lee proposed openstack/oslo.log: log: Spooling logger and manager  https://review.openstack.org/31616215:58
openstackgerritAlexis Lee proposed openstack/oslo.log: log: Extra spool overflow strategies  https://review.openstack.org/31821215:58
*** rcernin has quit IRC15:59
openstackgerritAlexis Lee proposed openstack/oslo.log: log: SpoolFilter and SpoolContextManager  https://review.openstack.org/32898816:01
openstackgerritAlexis Lee proposed openstack/oslo-specs: log: Spool request logs spec  https://review.openstack.org/22776616:04
openstackgerritAlexis Lee proposed openstack/oslo.log: log: SpoolFilter and 'spooling' context manager  https://review.openstack.org/32898816:05
*** mtanino has joined #openstack-oslo16:13
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Properly stop ZmqServer  https://review.openstack.org/32896316:14
*** salv-orl_ has quit IRC16:25
*** salv-orlando has joined #openstack-oslo16:25
*** e0ne has quit IRC16:29
*** rcernin has joined #openstack-oslo16:30
*** rcernin has quit IRC16:35
*** rcernin has joined #openstack-oslo16:36
*** sputnik13_ has joined #openstack-oslo16:38
*** pcaruana has quit IRC16:38
openstackgerritOleksii Zamiatin proposed openstack/oslo.messaging: [zmq] Properly stop ZmqServer  https://review.openstack.org/32896316:58
harlowja_at_homebbiab, commence time discussion now!16:58
harlowja_at_homelol16:58
amrithhello16:59
*** harlowja_at_home has quit IRC16:59
amrithbknudson, dhellmann ... courtesy ping16:59
amrithso, from what we all said, it sounds like we want standardization of timestamps in openstack16:59
amriththat sounds like a good thing16:59
amrithdhellmann said we should be able to parse any format and internally represent in a standard format.17:00
amrithI'll extend that and say any unambiguous timestamp format; with some assumptions (if we want to go here) for ambiguous formats should be parsed and represented internally in some known format17:00
amrithopenstack should always return timestamps in a standard format.17:01
amrithI'd like to propose that this format be the iso8601 format where "Z" suffix is used for +00:00 offset.17:01
dhellmannwhat did the old function produce, the one that was removed?17:01
amriththat would be what it produced17:01
amrithwith Z at the end ALWAYS (irrespective of the timezone unfortunately)17:02
amrithi.e. it was wilfully ignorant of timezone17:02
dhellmannah17:02
amrithI'm proposing that for UTC (all the good guys) we get no change.17:02
amrithZ at the end17:02
amrithfor non-UTC timestamps you get whatever isoformat() gives you17:02
amrithi.e. +NN:NN17:02
amrithsorry17:02
amrith+/- NN:NN17:02
amriththat would be ISO8601 compliant17:03
amrithand minimal change for projects already using isotime()17:03
amriths/minimal/NO/17:03
amrithand that's basically what I implemented for trove's version of timeutils.17:03
amrithit took bknudson's recommendation of use isoformat()17:03
amrithand tweaked it to replace +00:00 with Z to minimize change elsewhere.17:04
amriththe rest of the change is to call my own isotime() instead of oslo_utils.timeutils.isotime()17:04
amrithand add some tests.17:04
amrithIf we went the route I'm proposing, the change in Trove would be abandoned17:04
amriththe timeutils code would move to OSLO with the unit tests17:04
amriththen we can standardize on a timestamp format17:05
amrithand projects which had to clone the code could choose whether they want to unclone or not17:05
amrithif that makes sense, I'm happy to write it up as a change for oslo's review and a resolution for the TC to consider (if that's the route we want to go)17:05
*** ihrachys has quit IRC17:06
dhellmannI don't think we need a TC resolution, I hope that wasn't a serious suggestion17:06
dhellmannit makes sense to write it up more formally than this IRC discussion17:06
dhellmannthe key thing is to document in/near the function WHY we have it, so we don't go deleting it again17:07
amrithok, I'll write it up and propose a change in oslo with a comment nearby indicating why it is still there ...17:07
dhellmannthanks17:07
amrithbknudson, does that work for you?17:07
*** yamahata has joined #openstack-oslo17:15
amrithharlowja, please see AJaeger's comment in https://review.openstack.org/32908617:16
amrithah, never mind17:16
amrithmy mistake17:16
bknudsonnow that I see what isoformat does vs the w3c / ietf standards I think we should use a library that generates timestamps with Z rather than +00:0017:26
bknudsonI think this agrees with amrith's proposal.17:26
bknudsonamrith: so were you going to propose the new function to oslo?17:27
bknudsonor do we use an existing one?17:27
*** yamahata has quit IRC17:28
*** salv-orl_ has joined #openstack-oslo17:29
amrithbknudson, I will propose a change in oslo which does what I'm doing in Trove17:29
amrithor rather what I've proposed in Trove17:29
amrithI'll abandon the trove change in favor of the oslo change17:29
bknudsonIf it works as talked about here I expect keystone will be able to use it.17:30
amrithbknudson, would you like to test it out standalone or would you rather wait till it lands in oslo to test it?17:30
*** salv-orl_ has quit IRC17:32
*** salv-orlando has quit IRC17:32
*** salv-orlando has joined #openstack-oslo17:32
openstackgerritJulian proposed openstack/oslo.versionedobjects: JSON schema get_schema implementation for common fields  https://review.openstack.org/32835617:39
*** pcaruana has joined #openstack-oslo17:39
harlowjacool, u guys all agreed??? :-P17:44
*** mtanino has quit IRC17:47
bknudsonamrith: should be easy to try it out before it lands.17:47
*** jpena is now known as jpena|off17:53
amrithok, thanks bknudson17:54
amrithharlowja, I think we are.17:54
harlowjawoot17:54
harlowjaprogress++17:54
harlowjalol17:54
*** yamahata has joined #openstack-oslo17:55
harlowjakgiusti poor horse17:55
harlowjalol17:55
harlowjahorse just wants to go to horse-heaven17:55
kgiustiharlowja: where s/he can say hi to "Mr Ed"17:55
harlowjalol17:56
harlowjaman, if i understand (even the slightest that reference) that must mean something17:56
kgiustiharlowja: but no, it is an evil horse and we must continue to beat on it until everyone is completely dissatisfied :)17:56
kgiustiharlowja: this is dated, even for an old fart like myself: http://www.imdb.com/title/tt0054557/17:57
harlowjaya, i think i saw some reruns of that show somewhere on AMC once or something17:57
harlowja'my arm is tired - this is one big horse.'17:58
harlowjalol17:58
kgiustiharlowja: in any case, Mr. Ed has been pushing up daisies for awhile now.  Safe to assume he's in horse heaven.17:59
harlowjalol17:59
*** lpetrut has joined #openstack-oslo17:59
kgiustiharlowja: actually, no - he made that poor Mr Post's life a living hell - no horse heaven for you!!!17:59
harlowjaha18:00
harlowjahorse hell?18:00
harlowjain general kgiusti i agree with u, the meaning of 'ack' is messed up in MQ systems18:00
harlowjait just acks the message broker, lol18:01
* harlowja wonder if we have a better list of things that would have to change in oslo.messaging18:02
harlowjaor a scratchpad of that18:02
kgiustiharlowja: not necessarily - not sure what zmq does but if you read my oslo-spec (ahem) you'll see that message ack can be end to end18:02
harlowjaright, it gets confusing in the zmq case and amqp1.0 i would suspect18:02
kgiustiharlowja: sileht's right - it would mean big changes, not only to the library but the API.18:02
kgiustiharlowja: asynch acks require asynch RPC replies, since you'll likely ack first, reply later18:03
harlowjaright, which i guess gets into async all the way down18:04
harlowja(and/or more async stuff exposed to users)18:04
kgiustiharlowja: yes, much like turtles18:04
harlowjayup18:04
kgiustiharlowja: and we all know how much fun timing bugs are18:04
harlowjadef18:05
*** mtanino has joined #openstack-oslo18:05
kgiustiharlowja: but I agree with users - this capability makes sense in some cases.18:06
kgiustiharlowja: all the low level messaging apis I've dealt with offer async ack.  It's expected.18:06
kgiustiharlowja: but the way we've designed the RPC service really doesn't allow for it :(18:07
harlowjaasync in openstack == mind-blown18:07
harlowjaya18:07
*** dimtruck is now known as zz_dimtruck18:08
kgiustiharlowja: lucky for me my mind has been blown many times so it's basically gone.18:08
harlowjalol18:08
harlowjasame here18:08
*** browne has joined #openstack-oslo18:12
*** jecarey_ has joined #openstack-oslo18:15
*** jecarey has quit IRC18:18
openstackgerritMerged openstack/oslo.vmware: Imported Translations from Zanata  https://review.openstack.org/32861018:20
openstackgerritMerged openstack/oslo.vmware: Refactor VmdkWriteHandle and VmdkReadHandle  https://review.openstack.org/28125018:20
*** yassine__ has quit IRC18:22
*** ankur-gupta-f has joined #openstack-oslo18:27
ankur-gupta-fGood afternoon all. I am working on completing Feature Classification for the Neutron project (nearly identical to Nova's implementation). The discussion has come up that it should potentially go into a common repo/utility so that multiple projects can easily implement http://docs.openstack.org/developer/nova/support-matrix.html . I was wondering if Oslo is the right place to do that. P.S. I am new to OpenStack :)18:29
harlowjaankur-gupta-f quite possible that it could be a common thing, can u explain more of what it is18:30
ankur-gupta-fharlowja: So if you take a look at the link. What it is matrix that compares plugins/hypervisors/configs and if they can do some of the capabilities exposed by the projects API. For example:18:32
harlowjayup18:32
ankur-gupta-fAn end user can use many different Hypervisors. But not all of them can for example Attach a block volume to an instance.18:33
harlowjathat looks like maybe oslosphinx could have a helper to make that?18:33
ankur-gupta-fprobably. have not looked into it yet18:33
harlowjais that what u are thinking, maybe some sphinx plugin to help make such a table?18:33
harlowjafeature-matrix plugin or something18:33
ankur-gupta-fahh looking now. Yes.18:33
ankur-gupta-fcurrent implementation is using sphinx18:33
ankur-gupta-fso this probably is a good option to look into18:34
ankur-gupta-fharlowja: Going to play around with oslosphinx. If I were to move the files into oslo, what would the process be? Should I file a bug?18:35
harlowjathat's probably be useful (to file a bug)18:37
*** zz_dimtruck is now known as dimtruck18:38
ankur-gupta-fokay I will do that. Thank you for your help harlowja18:38
harlowjanp18:38
*** hecliunyx has quit IRC18:39
ankur-gupta-fharlowja: One last. oslosphinx would fall under Oslo in launchpad right :|18:39
harlowjaright18:39
ankur-gupta-fharlowja: woot. Thanks18:39
harlowjaor https://bugs.launchpad.net/oslosphinx18:39
harlowjai think either would be eok18:39
harlowja*be ok18:39
*** amoralej is now known as amoralej|off19:01
*** jecarey_ has quit IRC19:01
*** salv-orlando has quit IRC19:04
*** salv-orlando has joined #openstack-oslo19:05
*** jecarey_ has joined #openstack-oslo19:06
*** jecarey_ has quit IRC19:07
*** jecarey_ has joined #openstack-oslo19:07
*** rama_y has quit IRC19:10
*** e0ne has joined #openstack-oslo19:11
*** mhickey has joined #openstack-oslo19:26
*** salv-orlando has quit IRC19:31
*** salv-orlando has joined #openstack-oslo19:31
*** ihrachys has joined #openstack-oslo19:35
*** zancas has joined #openstack-oslo19:39
*** hecliunyx has joined #openstack-oslo19:48
*** e0ne has quit IRC19:48
*** claudiub has joined #openstack-oslo19:49
claudiubdims: hello. can you take a look at this pls? https://review.openstack.org/32822619:49
*** dtardivel has quit IRC19:57
*** hecliunyx has quit IRC19:59
zancasHi folks!  I'm consistently getting a single test failure against 2.7.11 when I run the test suite against master.20:01
zancasApologies if this is a known issue.20:01
zancasThe relevant commit hash is:20:01
zancasb694235dcc9b38517eb511b303795176b92db95d20:01
zancasI thought I'd ask here before looking for a more formal bug-report venue.20:01
zancasAnybody interested?20:01
zancasAhem, and more apologies if this is not the correct channel to discuss:20:02
zancashttps://github.com/openstack/oslo.config.git20:03
zancaserrr...  sorry... I meant:20:03
zancashttps://github.com/openstack/oslo.log20:03
* zancas examines: https://bugs.launchpad.net/oslo.log20:05
harlowjai haven't heard of anything20:05
*** cdent has quit IRC20:06
zancasharlowja: Should I paste the traceback in a pastebin?  Who is likely to be intereted?20:07
zancasinterested?20:07
harlowjaya, pastebin would be fine, not sure who might be, but just drop it here and maybe someone will be interested20:08
*** tonytan4ever has quit IRC20:12
zancasHere's a link to a paste:20:14
zancashttps://defuse.ca/b/0nyGlthpP0cMSgd57FdYto20:14
zancasThe relevant test is this one:20:14
zancashttps://github.com/openstack/oslo.log/blob/master/oslo_log/tests/unit/test_log.py#L113720:16
openstackgerritClaudiu Belu proposed openstack/oslo.privsep: PrivContext: Sets client_mode to False on Windows  https://review.openstack.org/32822620:19
*** kgiusti has quit IRC20:22
*** tonytan4ever has joined #openstack-oslo20:42
*** jecarey_ has quit IRC20:43
*** ihrachys has quit IRC20:47
*** ihrachys has joined #openstack-oslo20:50
*** ericksonsantos has quit IRC20:59
harlowjaif people are interested https://review.openstack.org/#/c/329186/21:04
patchbotharlowja: patch 329186 - releases - Oslo releases for week of june 13, 201621:04
stevemarharlowja: poke21:12
harlowjastevemar poke21:12
harlowjapoke poke21:12
stevemarharlowja: https://review.openstack.org/#/c/327503/1 merged but depends on dogpile 0.6.021:12
patchbotstevemar: patch 327503 - oslo.cache - Use entrypoints for backends (MERGED)21:12
stevemarwhich is here: https://review.openstack.org/#/c/328899/21:13
patchbotstevemar: patch 328899 - requirements - Bump dogpile.cache to 0.6.121:13
harlowjakk21:13
stevemarharlowja: anyway to not release oslo.cahce this week?21:13
harlowjaya, thats fine with me21:13
harlowjawill take off the listing21:13
stevemarotherwise we may release a bad version21:13
stevemar++21:13
harlowjayup, understood21:13
harlowjathx21:13
*** ihrachys has quit IRC21:14
harlowjaokie dokie, https://review.openstack.org/#/c/329186/ updated stevemar21:14
patchbotharlowja: patch 329186 - releases - Oslo releases for week of june 13, 201621:14
harlowjathx for letting me know21:14
*** ihrachys has joined #openstack-oslo21:14
openstackgerritJoshua Harlow proposed openstack/tooz: Add docs for new consul driver  https://review.openstack.org/31589021:16
stevemarharlowja: danke21:21
stevemarharlowja: next week should be fine, just waiting for a requirements patch to land, it can take a while :)21:21
harlowjayup yup21:22
harlowjaall good21:22
*** zancas has quit IRC21:22
*** zancas has joined #openstack-oslo21:32
*** lpetrut has quit IRC21:46
*** rcernin has quit IRC21:48
zancasHmmm...   is it the case that some tests are assuming the21:50
zancastest runner is not root?21:50
zancasI get one test failure in oslo.log and another in oslo.cfg21:51
zancasin both cases there's an assertion that a particular21:51
harlowjaunsure21:51
zancaserror will be raised21:51
zancasBut it's not.21:51
zancasIf I Understand Correctly IIUC the reason the error21:51
zancasis not being raised in my case is because I'm running21:51
zancasthe tests as root (in a docker container).21:52
*** mhickey has quit IRC21:52
*** ihrachys has quit IRC22:00
*** ihrachys has joined #openstack-oslo22:05
*** hoonetorg has quit IRC22:05
*** miqui has quit IRC22:06
*** edmondsw has quit IRC22:07
*** hoonetorg has joined #openstack-oslo22:08
* zancas begins prodding the mysterious oslo_config.cfg module....22:13
*** amotoki has quit IRC22:17
*** ihrachys_ has joined #openstack-oslo22:18
*** ihrachys has quit IRC22:20
*** tonytan4ever has quit IRC22:23
*** rama_y has joined #openstack-oslo22:24
*** gordc has quit IRC22:24
*** ihrachys_ has quit IRC22:25
*** ihrachys has joined #openstack-oslo22:27
*** ihrachys has quit IRC22:33
zancasI notice that oslo_config.types.Integer objects can accept int's as arguments...22:35
zancasBut the doc for oslo_config.cfg.Opt states:22:36
zancas"the option's type. Must be a callable object that takes string and returns converted and validated value"22:36
openstackgerritJoshua Harlow proposed openstack/taskflow: Rename engine analyzer to be named selector  https://review.openstack.org/32921122:37
*** dimtruck is now known as zz_dimtruck22:37
zancasHere's an example where the doc seems to suggest that `oslo_config.types` might strictly accept strings:22:38
zancashttp://docs.openstack.org/developer/oslo.config/cfg.html#option-types22:38
openstackgerritJoshua Harlow proposed openstack/taskflow: Rename engine analyzer to be named selector  https://review.openstack.org/32921122:38
*** ihrachys has joined #openstack-oslo22:39
zancasharlowja: Waddya' think, is it correct for oslo_config.types.Integer objects to take ints as input?22:44
*** jecarey has joined #openstack-oslo22:45
zancasI guess that's consistent with the principle-of-least-surprise for the user.22:45
zancasbut...   I was surprised that it worked because the above docs suggested to me that all input to "type" callables22:45
zancaswould have to be strings.22:46
harlowjazancas ya, my guess is its just doing a int(X) somewhere and X can be a string or int to that22:46
* zancas looks22:46
harlowjahttps://github.com/openstack/oslo.config/blob/master/oslo_config/types.py#L284 i guess22:47
harlowjaa little more sophosticated i guess22:47
zancas"        if not isinstance(value, int):"22:47
harlowjaya22:47
*** ihrachys has quit IRC22:48
zancasSo it checks if its an int, and if not assumes its something that can be converted to a string.22:48
harlowjaso seems so22:48
zancasThat all makes good sense I suppose, but seems a little inconsistent with the docs..22:48
zancasWould it be useful to ammend the docs?22:48
harlowjayup, that'd be cool22:48
* zancas would like to be helpful wilst learning the codebase22:49
*** jecarey has quit IRC22:49
harlowjathx22:49
* zancas opens CONTRIBUTING.rst ...22:49
* zancas notices the time... 22:51
*** salv-orlando has quit IRC22:51
zancasOK, I'll propse a doc change.22:51
zancasl8r \022:51
harlowjapeace22:52
*** zancas has left #openstack-oslo22:52
*** ihrachys has joined #openstack-oslo22:52
*** salv-orlando has joined #openstack-oslo22:52
openstackgerritMichael Bayer proposed openstack/oslo.db: Another non-working mysql rollback workaround.  https://review.openstack.org/32921522:56
openstackgerritJoshua Harlow proposed openstack/taskflow: Rename engine analyzer to be named selector  https://review.openstack.org/32921122:59
*** ihrachys has quit IRC23:01
*** ihrachys has joined #openstack-oslo23:01
*** sheeprine has quit IRC23:03
*** sheeprine has joined #openstack-oslo23:06
*** pradk has quit IRC23:07
*** yamahata has quit IRC23:11
openstackgerritMichael Bayer proposed openstack/oslo.db: Memoize sys.exc_info() before attempting a savepoint rollback  https://review.openstack.org/32921523:13
*** ihrachys has quit IRC23:17
*** ihrachys has joined #openstack-oslo23:22
*** salv-orl_ has joined #openstack-oslo23:28
*** salv-orlando has quit IRC23:31
*** ihrachys has quit IRC23:33
*** ihrachys has joined #openstack-oslo23:33
*** sdague has quit IRC23:38
*** amotoki has joined #openstack-oslo23:46
*** ihrachys has quit IRC23:47
*** ihrachys has joined #openstack-oslo23:47
amrithbknudson, yt?23:49
*** amotoki has quit IRC23:50
*** yamahata has joined #openstack-oslo23:51
amrithbknudson, we should chat; I think we have a problem. isotime() {the old one} and the one that was cloned by folks into projects (like keystone) actually returns a non-isotime :(23:52
amrithtrove didn't face this problem (thankfully)23:52
amrithbut keystone may notice it if I push up my changes23:52
amrithat issue is the fact that if subsecond=True, isotime() returns YYYY-MM-DDTHH:MM:SS.NNNNNN23:52
amrithwith no timezone information23:52
amrithno Z23:52
amrithno +/-NN:NN23:53
amrithwhich is bad23:53
amrithand if I 'fix' as we discussed23:53
amriththis could break something in keystone23:53
amrithwhich expects what isotime() called PERFECT_TIME_FORMAT23:53
amrithwhich I now realize is a cruel joke23:53
amrithsince it is neither perfect nor even ISO 8601 :(23:53
amrithdhellmann, bknudson, harlowja ^^ I will propose a change, this will not be exactly what I'd thought I would be proposing.23:54
harlowjalol23:54
harlowjaits a cruel joke23:55
*** ihrachys has quit IRC23:55
harlowjaNOT_PERFECT_TIME_FORMAT23:55
harlowjalol23:55
amrithit's like they taught me in marketing 101. when you want to lie, make it the headline. then no one cares. "Light Cigarettes", "Diet Mountain Dew", ...23:55

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