Tuesday, 2016-06-14

*** ihrachys has joined #openstack-oslo00:01
amrithharlowja, enough for today; will try and make sense of imperfection tomorrow .... ARGH!00:05
harlowja:)00:06
*** claudiub has quit IRC00:15
*** ihrachys has quit IRC00:17
*** mtanin___ has joined #openstack-oslo00:30
*** yamahata has quit IRC00:30
*** mtanino has quit IRC00:31
*** yamahata has joined #openstack-oslo00:31
*** zz_dimtruck is now known as dimtruck00:41
*** browne has quit IRC00:41
*** sputnik13_ has quit IRC00:46
*** sputnik13 has joined #openstack-oslo00:47
*** dimtruck is now known as zz_dimtruck00:57
*** amotoki has joined #openstack-oslo01:06
*** ozamiatin has quit IRC01:16
*** amotoki has quit IRC01:21
*** amotoki has joined #openstack-oslo01:39
*** zz_dimtruck is now known as dimtruck01:41
*** jecarey has joined #openstack-oslo01:44
*** jecarey has quit IRC01:49
*** dimtruck is now known as zz_dimtruck02:06
*** zz_dimtruck is now known as dimtruck02:06
*** amotoki has quit IRC02:07
*** amotoki has joined #openstack-oslo02:07
*** yamahata has quit IRC02:12
*** mtanino has joined #openstack-oslo02:16
*** mtanin___ has quit IRC02:18
*** jecarey has joined #openstack-oslo02:25
*** mtanino has quit IRC02:37
*** amotoki has quit IRC02:40
*** yamamoto_ has joined #openstack-oslo02:50
*** amotoki has joined #openstack-oslo02:54
*** amotoki has quit IRC02:57
*** yamamoto_ has quit IRC03:02
*** yamamoto_ has joined #openstack-oslo03:05
*** links has joined #openstack-oslo03:08
*** openstackgerrit has quit IRC03:11
stevemardims: have time to review https://review.openstack.org/#/c/328899/ ?03:11
patchbotstevemar: patch 328899 - requirements - Bump dogpile.cache to 0.6.103:11
*** openstackgerrit has joined #openstack-oslo03:11
*** dimtruck is now known as zz_dimtruck03:18
*** rama_y has quit IRC03:18
*** yamahata has joined #openstack-oslo03:29
*** yamamoto_ has quit IRC03:30
*** yamamoto_ has joined #openstack-oslo03:36
*** sputnik13 has quit IRC03:44
openstackgerritSachi King proposed openstack-dev/pbr: Restore warnerrors behavior  https://review.openstack.org/22995104:24
stevemarthanks dims04:35
*** jecarey has quit IRC04:38
*** pcaruana has quit IRC04:39
*** lpetrut has joined #openstack-oslo05:09
*** salv-orl_ has quit IRC05:17
*** salv-orlando has joined #openstack-oslo05:18
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.cache: Updated from global requirements  https://review.openstack.org/32926905:24
*** yamamoto_ has quit IRC05:28
*** salv-orl_ has joined #openstack-oslo05:29
*** salv-orlando has quit IRC05:31
*** salv-orl_ has quit IRC05:34
*** salv-orlando has joined #openstack-oslo05:34
openstackgerritJoshua Harlow proposed openstack/taskflow: Rename engine analyzer to be named selector  https://review.openstack.org/32921105:40
*** rcernin has joined #openstack-oslo06:11
*** salv-orlando has quit IRC06:13
*** salv-orlando has joined #openstack-oslo06:14
*** pcaruana has joined #openstack-oslo06:16
*** yamamoto_ has joined #openstack-oslo06:17
*** yamahata has quit IRC06:23
*** yamahata has joined #openstack-oslo06:26
*** jecarey has joined #openstack-oslo06:38
*** ihrachys has joined #openstack-oslo06:41
*** jecarey has quit IRC06:43
*** openstackgerrit has quit IRC06:48
*** openstackgerrit has joined #openstack-oslo06:48
*** tesseract has joined #openstack-oslo06:56
*** amoralej|off is now known as amoralej06:58
*** jpena|off is now known as jpena07:26
openstackgerritMerged openstack/oslo.cache: Updated from global requirements  https://review.openstack.org/32926907:30
*** yamahata has quit IRC07:41
*** lpetrut has quit IRC07:42
*** salv-orlando has quit IRC07:44
*** salv-orlando has joined #openstack-oslo07:44
*** eezhova has joined #openstack-oslo07:48
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-oslo08:00
*** eezhova has quit IRC08:02
*** eezhova has joined #openstack-oslo08:04
*** shardy has joined #openstack-oslo08:04
*** e0ne has joined #openstack-oslo08:05
*** mhickey has joined #openstack-oslo08:20
openstackgerritKirill Bespalov proposed openstack/oslo.messaging: To set qos_prefetch_count value to 64 (max_workers)  https://review.openstack.org/32244608:28
*** cdent has joined #openstack-oslo08:37
*** yassine has joined #openstack-oslo08:48
*** dtardivel has joined #openstack-oslo08:54
*** dukhlov has joined #openstack-oslo09:07
*** lpetrut has joined #openstack-oslo09:10
*** eezhova has quit IRC09:40
*** eezhova has joined #openstack-oslo09:50
*** AJaeger has joined #openstack-oslo09:59
AJaegeroslo cores, could you import translations again, please? See https://review.openstack.org/#/q/status:open+topic:zanata/translations,n,z for all open changes09:59
*** links has quit IRC10:07
*** ozamiatin has joined #openstack-oslo10:18
*** links has joined #openstack-oslo10:20
*** jdandrea_ has joined #openstack-oslo10:22
*** ozamiatin has quit IRC10:22
*** jdandrea has quit IRC10:25
*** sdague has joined #openstack-oslo10:31
*** amotoki_ has joined #openstack-oslo10:34
*** amotoki_ has quit IRC10:37
*** eezhova has quit IRC10:38
*** salv-orlando has quit IRC10:38
*** salv-orlando has joined #openstack-oslo10:39
*** cdent has quit IRC10:43
*** jpena is now known as jpena|off10:48
*** jpena|off is now known as jpena10:48
*** yassine has quit IRC10:55
*** salv-orlando has quit IRC10:58
*** salv-orlando has joined #openstack-oslo10:58
*** permalac has quit IRC11:09
*** cdent has joined #openstack-oslo11:17
*** eezhova has joined #openstack-oslo11:20
*** salv-orl_ has joined #openstack-oslo11:28
openstackgerritamrith proposed openstack/oslo.utils: un-deprecate (resurrect) timeutils.isotime  https://review.openstack.org/32938411:29
*** gordc has joined #openstack-oslo11:30
*** salv-orlando has quit IRC11:32
amrith^^ harlowja, dhellmann, bknudson, dims ^^11:34
amrith^^ jd__ ^^11:34
openstackgerritMerged openstack/oslo.privsep: PrivContext: Sets client_mode to False on Windows  https://review.openstack.org/32822611:41
*** ozamiatin has joined #openstack-oslo11:44
*** jpena is now known as jpena|lunch11:52
*** takedakn has joined #openstack-oslo11:58
*** ozamiatin has quit IRC11:58
*** ozamiatin has joined #openstack-oslo12:01
openstackgerritMerged openstack/oslo.messaging: [Trival] fix a typo nit  https://review.openstack.org/32890012:04
openstackgerritMerged openstack/oslo.config: Required Option  https://review.openstack.org/30852412:11
*** yamamoto_ has quit IRC12:11
*** pradk has joined #openstack-oslo12:12
*** kgiusti has joined #openstack-oslo12:15
*** cdent has quit IRC12:16
*** cdent has joined #openstack-oslo12:17
*** eezhova has quit IRC12:23
*** eezhova has joined #openstack-oslo12:28
openstackgerritStephen Finucane proposed openstack/oslo.config: Add support for a deprecated version  https://review.openstack.org/30767112:31
*** salv-orl_ has quit IRC12:33
*** salv-orlando has joined #openstack-oslo12:34
*** yassine has joined #openstack-oslo12:40
*** amoralej is now known as amoralej|lunch12:48
*** jpena|lunch is now known as jpena12:51
bknudsonamrith: for keystone we need a function that generates timestamps like 2013-02-27T18:30:59.999999Z -- per the spec http://specs.openstack.org/openstack/keystone-specs/api/v3/identity-api-v3.html#tokens12:51
*** yamamoto has joined #openstack-oslo12:53
*** claudiub has joined #openstack-oslo12:53
*** takedakn has quit IRC12:53
*** salv-orlando has quit IRC13:00
*** salv-orlando has joined #openstack-oslo13:00
*** ozamiatin has quit IRC13:01
*** amakarov_away is now known as amakarov13:06
*** yamamoto has quit IRC13:07
*** yamamoto has joined #openstack-oslo13:07
*** ozamiatin has joined #openstack-oslo13:10
*** yamamoto has quit IRC13:12
openstackgerritGevorg Davoian proposed openstack/oslo.messaging: [zmq] Remove redundant Envelope class  https://review.openstack.org/32825713:13
*** amotoki has joined #openstack-oslo13:24
*** ericksonsantos has joined #openstack-oslo13:29
*** amoralej|lunch is now known as amoralej13:37
*** edmondsw has joined #openstack-oslo13:41
*** mtanino has joined #openstack-oslo13:43
*** jecarey has joined #openstack-oslo13:51
*** jecarey has quit IRC13:55
*** yamamoto has joined #openstack-oslo13:56
*** jecarey has joined #openstack-oslo13:57
*** mhickey has quit IRC14:02
*** zz_dimtruck is now known as dimtruck14:03
*** links has quit IRC14:14
*** mhickey has joined #openstack-oslo14:17
*** rlrossit has joined #openstack-oslo14:19
*** dukhlov has quit IRC14:21
openstackgerritBrian Stajkowski proposed openstack/oslo.config: Advanced Option  https://review.openstack.org/30435514:29
openstackgerritMichael Bayer proposed openstack/oslo.db: Memoize sys.exc_info() before attempting a savepoint rollback  https://review.openstack.org/32921514:35
*** mhickey has quit IRC14:35
openstackgerritAlexey Yelistratov proposed openstack/osprofiler: Add MongoDB driver  https://review.openstack.org/31053014:41
*** mhickey has joined #openstack-oslo14:47
*** amotoki has quit IRC14:55
*** _stowa has joined #openstack-oslo14:58
*** yamahata has joined #openstack-oslo15:05
*** mhickey has quit IRC15:07
*** pradk has quit IRC15:11
*** rcernin has quit IRC15:11
*** _stowa has quit IRC15:12
*** jecarey has quit IRC15:13
*** jecarey_ has joined #openstack-oslo15:22
*** jecarey__ has joined #openstack-oslo15:24
*** salv-orlando has quit IRC15:25
*** salv-orlando has joined #openstack-oslo15:26
*** jecarey__ has quit IRC15:26
*** jecarey_ has quit IRC15:27
*** jecarey has joined #openstack-oslo15:27
*** salv-orlando has quit IRC15:30
*** salv-orlando has joined #openstack-oslo15:30
*** permalac has joined #openstack-oslo15:38
*** eezhova has quit IRC15:39
*** yamamoto has quit IRC15:43
*** pradk has joined #openstack-oslo15:44
*** yamamoto has joined #openstack-oslo15:44
*** ozamiatin has quit IRC15:45
*** eezhova has joined #openstack-oslo15:46
*** yamamoto has quit IRC15:47
*** yamamoto has joined #openstack-oslo15:48
*** yamamoto has quit IRC15:50
*** yamamoto has joined #openstack-oslo15:55
*** gcb has quit IRC15:55
*** gcb has joined #openstack-oslo15:56
*** yassine has quit IRC15:58
*** e0ne has quit IRC16:00
*** yamamoto has quit IRC16:01
*** yamamoto has joined #openstack-oslo16:10
*** yamamoto_ has joined #openstack-oslo16:11
*** yamamoto has quit IRC16:12
*** rlrossit has quit IRC16:13
*** yamahata has quit IRC16:14
*** yamamoto_ has quit IRC16:15
*** permalac has quit IRC16:16
*** rlrossit has joined #openstack-oslo16:17
*** rlrossit has quit IRC16:17
*** rlrossit has joined #openstack-oslo16:17
*** yamahata has joined #openstack-oslo16:18
*** pradk has quit IRC16:26
*** nkrinner is now known as nkrinner_afk16:34
*** zancas has joined #openstack-oslo16:34
*** yamamoto has joined #openstack-oslo16:36
harlowjaamrith i'm sure jd__  will love all of this :-P16:38
harlowja;)16:38
amrithharlowja, yes. I copied him on it and added him to the review after I noticed that he is one of the maintainers of the iso8601 library that we use :)16:39
harlowja;)16:39
amrithbknudson, the code I proposed DOES do just what you want16:39
amrithisotime() the legacy one DOES NOT. it produces  2013-02-27T18:30:59.99999916:40
amrithno Z at the end16:40
amrithI have highlighted this difference specifically in my review16:40
*** eezhova has quit IRC16:40
amrithbknudson ^^16:40
bknudsonamrith: great, so we could use that code now and continue to use it going forward. We would not use isoformat.16:42
*** tesseract has quit IRC16:43
*** e0ne has joined #openstack-oslo16:43
*** e0ne has quit IRC16:44
amrith++16:45
*** salv-orlando has quit IRC16:49
openstackgerritAlexey Yelistratov proposed openstack/osprofiler: Add MongoDB driver  https://review.openstack.org/31053016:50
*** salv-orlando has joined #openstack-oslo16:50
*** sputnik13 has joined #openstack-oslo16:50
*** browne has joined #openstack-oslo16:56
amrithrbradfor, yt?16:57
amrithwill you be pushing up a new change set for https://review.openstack.org/#/c/296667/?16:57
patchbotamrith: patch 296667 - python-troveclient - [WIP] Replace Oslo Incubator apiclient usage16:57
*** ozamiatin has joined #openstack-oslo17:01
AJaegeroslo cores, could you import translations again, please? See https://review.openstack.org/#/q/status:open+topic:zanata/translations,n,z for all open changes17:02
*** lpetrut has quit IRC17:05
*** e0ne has joined #openstack-oslo17:12
harlowjawill do17:15
*** pradk has joined #openstack-oslo17:17
AJaegerthanks, harlowja !17:20
harlowjanp17:20
*** ankur-gupta-f has quit IRC17:24
*** miqui has joined #openstack-oslo17:26
*** salv-orlando has quit IRC17:26
*** yamamoto has quit IRC17:26
*** salv-orlando has joined #openstack-oslo17:26
*** salv-orl_ has joined #openstack-oslo17:29
*** salv-orlando has quit IRC17:32
*** eezhova has joined #openstack-oslo17:40
*** e0ne has quit IRC17:41
*** tonytan4ever has joined #openstack-oslo17:41
*** zancas has quit IRC17:42
*** ihrachys has quit IRC17:46
*** mtanino has quit IRC17:51
*** jpena is now known as jpena|off17:51
*** ankur-gupta-f has joined #openstack-oslo17:51
*** AJaeger has left #openstack-oslo17:53
*** mhickey has joined #openstack-oslo17:54
*** Rockyg has joined #openstack-oslo17:58
*** zancas has joined #openstack-oslo17:59
openstackgerritMerged openstack/oslo.privsep: Imported Translations from Zanata  https://review.openstack.org/32859618:01
amrithharlowja, got a second to discuss your comment (line 99, timeutils.py)18:02
openstackgerritamrith proposed openstack/oslo.utils: un-deprecate (resurrect) timeutils.isotime  https://review.openstack.org/32938418:06
*** mtanino has joined #openstack-oslo18:08
*** salv-orl_ has quit IRC18:10
*** salv-orlando has joined #openstack-oslo18:11
*** salv-orlando has quit IRC18:11
*** salv-orlando has joined #openstack-oslo18:12
openstackgerritMichael Bayer proposed openstack/oslo.db: Memoize sys.exc_info() before attempting a savepoint rollback  https://review.openstack.org/32921518:13
openstackgerritMerged openstack/oslo.reports: Imported Translations from Zanata  https://review.openstack.org/32859418:13
*** eezhova has quit IRC18:14
openstackgerritMerged openstack/oslo.cache: Imported Translations from Zanata  https://review.openstack.org/32858818:15
openstackgerritMerged openstack/oslo.concurrency: Imported Translations from Zanata  https://review.openstack.org/32860818:15
openstackgerritMerged openstack/oslo.utils: Imported Translations from Zanata  https://review.openstack.org/32860618:17
openstackgerritMerged openstack/oslo.versionedobjects: Imported Translations from Zanata  https://review.openstack.org/32859518:20
*** eezhova has joined #openstack-oslo18:23
*** mhickey has quit IRC18:23
*** mhickey has joined #openstack-oslo18:24
*** jecarey has quit IRC18:24
*** lpetrut has joined #openstack-oslo18:26
*** jecarey has joined #openstack-oslo18:26
harlowjaamrith  in a few, gotta do a phone call18:30
*** harlowja has quit IRC18:30
*** rlrossit has quit IRC18:31
*** harlowja has joined #openstack-oslo18:32
jamielennoxhey oslos, https://review.openstack.org/#/c/326782/ is a fairly simple patch adding oslo.config support where it was missed - i'm pushing this middleware to a couple of services and would really like to get it in before we get too far along18:33
patchbotjamielennox: patch 326782 - oslo.middleware - Expose sample config opts for http-proxy-to-wsgi18:33
*** e0ne has joined #openstack-oslo18:35
*** bzurawski has joined #openstack-oslo18:42
amrithok harlowja18:43
*** salv-orlando has quit IRC18:45
*** sdague has quit IRC18:46
*** salv-orlando has joined #openstack-oslo18:46
*** lifeless has quit IRC18:46
*** lifeless has joined #openstack-oslo18:47
*** zancas has quit IRC18:50
*** e0ne has quit IRC18:53
*** bzurawski has quit IRC18:53
*** ankur-gupta-f has left #openstack-oslo18:55
*** eezhova has quit IRC18:56
*** eezhova has joined #openstack-oslo18:56
*** amoralej is now known as amoralej|off18:57
*** eezhova has quit IRC18:59
*** jecarey has quit IRC19:00
*** eezhova has joined #openstack-oslo19:01
*** jecarey has joined #openstack-oslo19:02
*** jecarey has quit IRC19:02
*** jecarey has joined #openstack-oslo19:03
*** amakarov is now known as amakarov_away19:03
*** eezhova has quit IRC19:08
*** eezhova has joined #openstack-oslo19:08
*** tonytan4ever has quit IRC19:25
*** dtardivel has quit IRC19:27
*** eezhova has quit IRC19:27
*** jecarey has quit IRC19:28
*** jecarey has joined #openstack-oslo19:28
*** rlrossit has joined #openstack-oslo19:29
*** jecarey has quit IRC19:34
*** zancas has joined #openstack-oslo19:36
*** eezhova has joined #openstack-oslo19:39
*** jecarey has joined #openstack-oslo19:41
*** salv-orlando has quit IRC19:43
*** salv-orlando has joined #openstack-oslo19:44
*** eezhova has quit IRC19:44
*** dimtruck is now known as zz_dimtruck19:44
*** harlowja_ has joined #openstack-oslo19:45
*** harlowja has quit IRC19:49
*** jsavak has joined #openstack-oslo19:50
*** sdague has joined #openstack-oslo19:56
*** e0ne has joined #openstack-oslo19:58
harlowja_amrith ok i'm around if u got time20:00
amrithi have time, the tc meeting will be starting shortly and I'd like to listen in on that; happy to multitask if that's ok with you.20:01
harlowja_ah, tc meeting, wonder if anything interesting happening there?20:01
amriththere usually is an informative discussion20:01
harlowja_*sometimes*20:02
harlowja_lol20:02
amrithwell, the bar on what I consider informative is likely different from yours :)20:02
harlowja_:-P20:02
harlowja_fair enough20:02
amrithso I posted a response to your comment re: force utc20:02
harlowja_ya, i'm just wondering if that's peoples resistance to the adding this back20:03
harlowja_in that it makes those kinds of assumptions (by default)20:03
*** jsavak has quit IRC20:03
*** eezhova has joined #openstack-oslo20:03
* harlowja_ just was a thought20:03
amrithit may be, but these assumptions are the same ones that isoformat() makes20:03
harlowja_fair point20:04
harlowja_dims can u expand more on your thought about that review?20:05
*** yamahata has quit IRC20:05
harlowja_just to much pain for this? something else?20:05
amrithdims is at the tc meeting20:05
harlowja_oh20:05
harlowja_welllll20:05
amrithhe and I talked about this a while ago20:06
*** zz_dimtruck is now known as dimtruck20:06
*** ozamiatin_ has joined #openstack-oslo20:06
amrithI won't attempt to represent his point of view, but I understand it (though I don't agree with it).20:06
*** ozamiatin has quit IRC20:06
harlowja_i honestly (imho) don't see the big problem with having it, doesn't seem like it violates any standard, it does make an assumption about things that lack timezones  but thats about it :-P20:06
amrithI understood it to be that having this in oslo added little value given that it can be done fairly painlessly with isoformat().20:07
amrithIn an ideal world, I think that would be meaningful. just like it does not make sense for oslo to have a wrapper on very unix system call.20:07
harlowja_ya, meh, alot of the oslo_utils code u could say the same thing :-P20:07
amrithbut in this case, I believe that the issue is a bit more convoluted.20:07
harlowja_ya20:08
amriththere is a matter of standardization at issue here, and an uninteded consequence of the apparently obvious path of just telling people to use UTC and isoformat().20:08
amrithsetting aside the (IMHO infeasible) complications arising out of being prescriptive about the use of UTC, I think there's also the real issue of a change in behavior that people don't want.20:09
amriththe replacement of "Z" with +00:00.20:09
harlowja_right the isoformat() change forced that20:09
harlowja_(which even though it doesn't seem like it should matter i guess it does for folks)20:09
amrithnot speaking for bknudson but I think that was the thing that may have driven the decision(s) he made.20:09
harlowja_i'm a little unclear why it mattered though20:10
harlowja_it seems like "Z" or +00:00 is == so not sure it matters?20:10
harlowja_or maybe i don't know something :-P20:10
amrithdownstream parsers which expected the Z and didn't expect the timezone20:10
amriththere would always be a Z20:10
amrithand never a +/-NN:NN20:10
*** eezhova has quit IRC20:10
harlowja_iso8601 lib that jd__ maintains :-P?20:10
bknudsonidentity never documented that we'd ever present +00:00. Only Z20:11
harlowja_did u say u'd present a iso8601 string?20:11
bknudsonNo, we didn't say it was iso860120:11
amrithwell, iso8601 is another issue ... see: http://pyiso8601.readthedocs.io/en/latest/#where-it-differs-from-iso-860120:11
harlowja_k20:11
harlowja_bknudson  okie, thats interesting20:11
amrithZ is iso8601 and rfc 3339 compliant20:12
harlowja_if the docs just said "iso8601" compliant string here, then it wouldn't i guess have mattered Z or 00:00 or ... (both seem compliant)20:12
*** rlrossit has quit IRC20:12
harlowja_amrith  ya, i'm unsure why that lib differs from the spec (weird)20:12
amrithyes, I don't know for sure either.20:13
amrithI looked through the code and commit histories for the project to see if there was an indication of why20:13
amrithbut I could find none.20:13
harlowja_probably jd__ fault, lol20:13
harlowja_ha20:14
harlowja_sneaky guy20:14
harlowja_bknudson did the docs ever get updated to say 'iso8601 formatted string <here>'20:14
*** kgiusti has quit IRC20:15
amrithI did want to ask jd__ about the comment he made that he'd rather not maintain isotime() when isoformat() is available. I wondered whether he felt the same way about the iso8601 library and whether it'd change his mind if I volunteered to manage it.20:15
bknudsonharlowja_: the docs don't actually define the format. The only place is in examples.20:15
harlowja_bknudson k20:15
bknudsonhttp://specs.openstack.org/openstack/keystone-specs/api/v3/identity-api-v3.html is the spec20:15
bknudsonharlowja_: actually, there's one mention: updated: A string with the time when the specification status last changed in ISO8601 format. For example, “2013-03-06T00:00:00Z”.20:16
harlowja_right most of those say 'A string with the time when the specification status last changed in ISO8601 format.'20:16
harlowja_ah, ya, same one, ha20:16
bknudsonso apparently one field is iso860120:16
bknudsonthe important ones are the ones in the token.20:17
harlowja_expires_at (string, ISO 8601 extended format date time with microseconds)20:17
harlowja_i guess those20:17
harlowja_which also says ISO8601 format20:17
bknudsonhmm, ok, maybe then we can just change it.20:17
bknudsonwe'd have to fix all the tests, and probably tempest20:17
amrithbknudson, there are changes all over the place if I have to change that20:18
*** cdent has quit IRC20:18
amrithall of our API tests (for example)20:18
harlowja_perhaps, anything doing that strict of validation to not be able to use "Z" or "00:00"20:18
bknudsonanyways, we're not going to change our code to switch from Z to 00:0020:18
harlowja_right, but then something seems iffy in those api tests (i guess its just doing raw string == or something?)20:18
harlowja_bknudson fair, just thinking that in i guess an ideal world it wouldn't matter (cause both appear to be valid and equivalent)20:19
bknudsonyes, since we've doc'd it's iso8601 then anything in iso8601 would work.20:20
bknudsoncould just put "1" there.20:20
harlowja_so then jd__ amrith would people be more 'happy' if say we did a upstream python patch to put this in https://hg.python.org/cpython/file/tip/Lib/datetime.py#l1211 and then maintained the oslo_utils code (until said python change arrived/we cna use it, whenever openstack can use py3.6++++)20:21
harlowja_that code we can use in about 25 years if its in 3.6 (but thats a different problem, lol)20:22
amrith1s, timesliced by conversation on smaug20:23
harlowja_k20:23
*** zancas has quit IRC20:25
*** zancas has joined #openstack-oslo20:26
dimsharlowja_ : hosting isotime will repeat behavior of API's being busted as folks use it without thinking through. i'd rather they take the blame of their APIs being busted on their own accord20:27
bknudsonanyways I don't know what the iso8601 format is since I don't want to pay ISO to read their docs.20:27
harlowja_bknudson  :)20:27
amritham back20:28
harlowja_dims ok, so the only busted case is the assumption that no_timezone == 'UTC' (for some definition of busted)?20:28
amrithfor what definition of busted? that's an assumption ... :)20:29
*** e0ne has quit IRC20:29
* amrith tries to parse dims' concern20:30
harlowja_i need to know defintion first :-P20:30
amrithharlowja_, I don't think this is as much a matter of people liking or not but rather of OpenStack providing a common representation of timestamps. delegating everything down to projects is challenging in two situations; one for clients who are using openstack and second for projects that rely on other projects.20:31
*** pradk has quit IRC20:31
amrithif keystone for example changed its timestamp representation, the ripple could affect other projects as well.20:32
amrithespecially if they followed the path of cloning isotime() with its existing warts.20:32
amrithactually, not just isotime(). there are other methods/functions being deprecated as well.20:32
harlowja_likely20:32
amrithI think bknudson has a very good point when he says that we need a standard timestamp stringizing format for openstack20:33
harlowja_iso8601 ?20:33
amrithwell iso8601 is a specification20:33
amrithnot a format20:33
harlowja_k20:33
amrithit provides many formats that are all valid20:33
harlowja_ya, pick the least crappy one of those20:33
amrithand is permissive to a fault.20:33
harlowja_agreed20:33
bknudsoniso8601 actually allows lots of formats, for example "1" is a valid iso8601 string.20:33
harlowja_ya, thats a weird spec, lol20:34
bknudsonit also supports durations20:34
amrithQED20:34
bknudsonwhich is cool but way too lenient20:34
harlowja_def20:34
bknudsonsomething like https://www.w3.org/TR/NOTE-datetime would be better20:34
harlowja_def20:35
harlowja_pick one of the 6 'levels' and that should do it20:35
harlowja_or even 3 levels20:35
harlowja_which i guess amrith why u are thinking the TC should perhaps take this topic up?20:36
*** rlrossit has joined #openstack-oslo20:36
harlowja_(or that's why i think u thought that)20:36
harlowja_brb20:36
bknudsonmaybe the API working group.20:36
bknudson(if they don't have a recommendation already)20:36
amrithharlowja_, not really. I think someone else suggested it, maybe bknudson. but I think it makes sense that this is a formal decision, not an oslo/trove/keystone choice20:37
bknudsonso I would prefer if the identity API spec referenced https://www.w3.org/TR/NOTE-datetime rather than saying iso 8601.20:37
bknudsonand that could probably be done anyways20:37
bknudsonto take a wider view -- all of openstack should use the same format for timestamps.20:38
amriththat would make sense20:38
amrithand given that today there are few who use isoformat() we should stick to the current isotime() :>20:39
*** jecarey has quit IRC20:40
*** jecarey has joined #openstack-oslo20:40
bknudsonsaves a few bytes to use Z rather than 00:0020:49
harlowja_:-/20:50
*** edmondsw has quit IRC20:52
*** shardy has quit IRC20:53
*** eezhova has joined #openstack-oslo20:58
zancasHiya channel!20:58
zancasI'm wondering if it makes more sense to correct the docs here:20:58
zancasquired Option20:58
zancas  22                                                                                     │                with 'min/max'.                                                 │* |   bb0f7e3 (tag: 3.11.0) Merge "generator: format string default value for L20:58
zancas  23     from oslo_config import cfg                                                     │                                                                                │ist type properly"20:58
zancas:(   Oooppsss...20:58
zancasSorry the docs here:20:59
zancashttps://github.com/openstack/oslo.config/blob/master/oslo_config/cfg.py#L4220:59
zancasTo assert that the value of the type parameter can be other-than-a-string?20:59
zancasSince e.g. types.Integer can accept ints?21:00
*** jd__ has quit IRC21:01
zancasor would it be better to make the "types" more strict to match the implication of the doc...21:01
zancasi.e. make it so that non-string arguments raise an exception?21:01
*** jd__ has joined #openstack-oslo21:01
zancasThe current behavior is that e.g. x = types.Integer(1,4)21:02
zancasy = x(3)21:02
zancasWorks...  which is inconsistent with the (poorly) linked doc...  :(21:02
zancasAgain sorry for the paste dump!21:03
zancasI could see reasonable arguments for either behavior...  on the one hand...21:04
zancasIt's pretty obvious what was intended when a user passed 3 as an option to a types.Integer21:04
zancasvalidated parameter...21:04
zancasOn the other hand, when would we expect non-strings to be passed to an option parser?21:05
zancasPerhaps after options are parsed from config files?  <-- In answer to my own question.21:05
*** e0ne has joined #openstack-oslo21:06
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.config: Updated from global requirements  https://review.openstack.org/32257021:09
openstackgerritOpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements  https://review.openstack.org/32484821:09
*** yamahata has joined #openstack-oslo21:12
openstackgerritzancas proposed openstack/oslo.config: Correct doc to include arbitrary inputs to `types`  https://review.openstack.org/32968421:25
*** miqui has quit IRC21:26
*** jecarey_ has joined #openstack-oslo21:27
elarsondoes anyone here have any insight into who is responsible for the openstack dev list server?21:29
*** jecarey has quit IRC21:30
*** eezhova has quit IRC21:30
elarsonI've been getting disconnected from the list due to bounces and the folks managing my email could use some help figuring out why it is happening.21:30
bknudsonelarson: I suggest taking that question to the #openstack-infra channel21:34
*** eezhova has joined #openstack-oslo21:34
elarsonbknudson: thanks! will do!21:34
*** pradk has joined #openstack-oslo21:39
*** eezhova has quit IRC21:41
*** e0ne has quit IRC21:42
*** eezhova has joined #openstack-oslo21:45
*** sileht has quit IRC21:51
*** sileht has joined #openstack-oslo21:52
*** jecarey_ has quit IRC21:57
*** dimtruck is now known as zz_dimtruck21:58
*** browne has quit IRC22:08
*** rlrossit has quit IRC22:10
*** Rockyg has quit IRC22:12
*** zancas has quit IRC22:18
*** eezhova has quit IRC22:19
harlowja_lxsli just a quick question on the spooling stuff, when does log msg <-> value interpolation happen?22:19
*** mhickey has quit IRC22:20
harlowja_is it when added to the spool, or when the spool flushes?22:20
harlowja_because it looks like 'The record sits in the spool until the context manager exits,' would be weird22:20
harlowja_for example22:20
harlowja_since python is by reference :-/22:20
harlowja_(except for value types)22:20
harlowja_so if i was doing something like22:21
harlowja_initial_users = ['a']22:21
harlowja_LOG.debug(my initial users, initial_users)22:21
harlowja_then the_new_uers = initial_users22:22
harlowja_the_new_uers .append("b")22:22
harlowja_LOG.debug(my after modification users,    the_new_uers)22:22
harlowja_but if interpolation is delayed, both would have the same list :-/22:22
harlowja_(which would make for a weird log message)22:23
*** sigmavirus24 is now known as sigmavirus24_awa22:23
*** openstackstatus has quit IRC22:25
*** openstack has joined #openstack-oslo22:25
*** openstackstatus has joined #openstack-oslo22:26
*** ChanServ sets mode: +v openstackstatus22:26
*** browne has joined #openstack-oslo22:27
*** rcernin has joined #openstack-oslo22:28
*** mtanino has quit IRC22:38
*** mtanino has joined #openstack-oslo22:43
*** salv-orlando has quit IRC23:14
*** salv-orlando has joined #openstack-oslo23:14
*** lpetrut has quit IRC23:22
*** salv-orl_ has joined #openstack-oslo23:29
*** browne has quit IRC23:30
*** salv-orlando has quit IRC23:32
*** browne has joined #openstack-oslo23:35
*** rcernin has quit IRC23:40
*** sdague has quit IRC23:53

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