*** ihrachys has joined #openstack-oslo | 00:01 | |
amrith | harlowja, enough for today; will try and make sense of imperfection tomorrow .... ARGH! | 00:05 |
---|---|---|
harlowja | :) | 00:06 |
*** claudiub has quit IRC | 00:15 | |
*** ihrachys has quit IRC | 00:17 | |
*** mtanin___ has joined #openstack-oslo | 00:30 | |
*** yamahata has quit IRC | 00:30 | |
*** mtanino has quit IRC | 00:31 | |
*** yamahata has joined #openstack-oslo | 00:31 | |
*** zz_dimtruck is now known as dimtruck | 00:41 | |
*** browne has quit IRC | 00:41 | |
*** sputnik13_ has quit IRC | 00:46 | |
*** sputnik13 has joined #openstack-oslo | 00:47 | |
*** dimtruck is now known as zz_dimtruck | 00:57 | |
*** amotoki has joined #openstack-oslo | 01:06 | |
*** ozamiatin has quit IRC | 01:16 | |
*** amotoki has quit IRC | 01:21 | |
*** amotoki has joined #openstack-oslo | 01:39 | |
*** zz_dimtruck is now known as dimtruck | 01:41 | |
*** jecarey has joined #openstack-oslo | 01:44 | |
*** jecarey has quit IRC | 01:49 | |
*** dimtruck is now known as zz_dimtruck | 02:06 | |
*** zz_dimtruck is now known as dimtruck | 02:06 | |
*** amotoki has quit IRC | 02:07 | |
*** amotoki has joined #openstack-oslo | 02:07 | |
*** yamahata has quit IRC | 02:12 | |
*** mtanino has joined #openstack-oslo | 02:16 | |
*** mtanin___ has quit IRC | 02:18 | |
*** jecarey has joined #openstack-oslo | 02:25 | |
*** mtanino has quit IRC | 02:37 | |
*** amotoki has quit IRC | 02:40 | |
*** yamamoto_ has joined #openstack-oslo | 02:50 | |
*** amotoki has joined #openstack-oslo | 02:54 | |
*** amotoki has quit IRC | 02:57 | |
*** yamamoto_ has quit IRC | 03:02 | |
*** yamamoto_ has joined #openstack-oslo | 03:05 | |
*** links has joined #openstack-oslo | 03:08 | |
*** openstackgerrit has quit IRC | 03:11 | |
stevemar | dims: have time to review https://review.openstack.org/#/c/328899/ ? | 03:11 |
patchbot | stevemar: patch 328899 - requirements - Bump dogpile.cache to 0.6.1 | 03:11 |
*** openstackgerrit has joined #openstack-oslo | 03:11 | |
*** dimtruck is now known as zz_dimtruck | 03:18 | |
*** rama_y has quit IRC | 03:18 | |
*** yamahata has joined #openstack-oslo | 03:29 | |
*** yamamoto_ has quit IRC | 03:30 | |
*** yamamoto_ has joined #openstack-oslo | 03:36 | |
*** sputnik13 has quit IRC | 03:44 | |
openstackgerrit | Sachi King proposed openstack-dev/pbr: Restore warnerrors behavior https://review.openstack.org/229951 | 04:24 |
stevemar | thanks dims | 04:35 |
*** jecarey has quit IRC | 04:38 | |
*** pcaruana has quit IRC | 04:39 | |
*** lpetrut has joined #openstack-oslo | 05:09 | |
*** salv-orl_ has quit IRC | 05:17 | |
*** salv-orlando has joined #openstack-oslo | 05:18 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.cache: Updated from global requirements https://review.openstack.org/329269 | 05:24 |
*** yamamoto_ has quit IRC | 05:28 | |
*** salv-orl_ has joined #openstack-oslo | 05:29 | |
*** salv-orlando has quit IRC | 05:31 | |
*** salv-orl_ has quit IRC | 05:34 | |
*** salv-orlando has joined #openstack-oslo | 05:34 | |
openstackgerrit | Joshua Harlow proposed openstack/taskflow: Rename engine analyzer to be named selector https://review.openstack.org/329211 | 05:40 |
*** rcernin has joined #openstack-oslo | 06:11 | |
*** salv-orlando has quit IRC | 06:13 | |
*** salv-orlando has joined #openstack-oslo | 06:14 | |
*** pcaruana has joined #openstack-oslo | 06:16 | |
*** yamamoto_ has joined #openstack-oslo | 06:17 | |
*** yamahata has quit IRC | 06:23 | |
*** yamahata has joined #openstack-oslo | 06:26 | |
*** jecarey has joined #openstack-oslo | 06:38 | |
*** ihrachys has joined #openstack-oslo | 06:41 | |
*** jecarey has quit IRC | 06:43 | |
*** openstackgerrit has quit IRC | 06:48 | |
*** openstackgerrit has joined #openstack-oslo | 06:48 | |
*** tesseract has joined #openstack-oslo | 06:56 | |
*** amoralej|off is now known as amoralej | 06:58 | |
*** jpena|off is now known as jpena | 07:26 | |
openstackgerrit | Merged openstack/oslo.cache: Updated from global requirements https://review.openstack.org/329269 | 07:30 |
*** yamahata has quit IRC | 07:41 | |
*** lpetrut has quit IRC | 07:42 | |
*** salv-orlando has quit IRC | 07:44 | |
*** salv-orlando has joined #openstack-oslo | 07:44 | |
*** eezhova has joined #openstack-oslo | 07:48 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-oslo | 08:00 | |
*** eezhova has quit IRC | 08:02 | |
*** eezhova has joined #openstack-oslo | 08:04 | |
*** shardy has joined #openstack-oslo | 08:04 | |
*** e0ne has joined #openstack-oslo | 08:05 | |
*** mhickey has joined #openstack-oslo | 08:20 | |
openstackgerrit | Kirill Bespalov proposed openstack/oslo.messaging: To set qos_prefetch_count value to 64 (max_workers) https://review.openstack.org/322446 | 08:28 |
*** cdent has joined #openstack-oslo | 08:37 | |
*** yassine has joined #openstack-oslo | 08:48 | |
*** dtardivel has joined #openstack-oslo | 08:54 | |
*** dukhlov has joined #openstack-oslo | 09:07 | |
*** lpetrut has joined #openstack-oslo | 09:10 | |
*** eezhova has quit IRC | 09:40 | |
*** eezhova has joined #openstack-oslo | 09:50 | |
*** AJaeger has joined #openstack-oslo | 09:59 | |
AJaeger | oslo cores, could you import translations again, please? See https://review.openstack.org/#/q/status:open+topic:zanata/translations,n,z for all open changes | 09:59 |
*** links has quit IRC | 10:07 | |
*** ozamiatin has joined #openstack-oslo | 10:18 | |
*** links has joined #openstack-oslo | 10:20 | |
*** jdandrea_ has joined #openstack-oslo | 10:22 | |
*** ozamiatin has quit IRC | 10:22 | |
*** jdandrea has quit IRC | 10:25 | |
*** sdague has joined #openstack-oslo | 10:31 | |
*** amotoki_ has joined #openstack-oslo | 10:34 | |
*** amotoki_ has quit IRC | 10:37 | |
*** eezhova has quit IRC | 10:38 | |
*** salv-orlando has quit IRC | 10:38 | |
*** salv-orlando has joined #openstack-oslo | 10:39 | |
*** cdent has quit IRC | 10:43 | |
*** jpena is now known as jpena|off | 10:48 | |
*** jpena|off is now known as jpena | 10:48 | |
*** yassine has quit IRC | 10:55 | |
*** salv-orlando has quit IRC | 10:58 | |
*** salv-orlando has joined #openstack-oslo | 10:58 | |
*** permalac has quit IRC | 11:09 | |
*** cdent has joined #openstack-oslo | 11:17 | |
*** eezhova has joined #openstack-oslo | 11:20 | |
*** salv-orl_ has joined #openstack-oslo | 11:28 | |
openstackgerrit | amrith proposed openstack/oslo.utils: un-deprecate (resurrect) timeutils.isotime https://review.openstack.org/329384 | 11:29 |
*** gordc has joined #openstack-oslo | 11:30 | |
*** salv-orlando has quit IRC | 11:32 | |
amrith | ^^ harlowja, dhellmann, bknudson, dims ^^ | 11:34 |
amrith | ^^ jd__ ^^ | 11:34 |
openstackgerrit | Merged openstack/oslo.privsep: PrivContext: Sets client_mode to False on Windows https://review.openstack.org/328226 | 11:41 |
*** ozamiatin has joined #openstack-oslo | 11:44 | |
*** jpena is now known as jpena|lunch | 11:52 | |
*** takedakn has joined #openstack-oslo | 11:58 | |
*** ozamiatin has quit IRC | 11:58 | |
*** ozamiatin has joined #openstack-oslo | 12:01 | |
openstackgerrit | Merged openstack/oslo.messaging: [Trival] fix a typo nit https://review.openstack.org/328900 | 12:04 |
openstackgerrit | Merged openstack/oslo.config: Required Option https://review.openstack.org/308524 | 12:11 |
*** yamamoto_ has quit IRC | 12:11 | |
*** pradk has joined #openstack-oslo | 12:12 | |
*** kgiusti has joined #openstack-oslo | 12:15 | |
*** cdent has quit IRC | 12:16 | |
*** cdent has joined #openstack-oslo | 12:17 | |
*** eezhova has quit IRC | 12:23 | |
*** eezhova has joined #openstack-oslo | 12:28 | |
openstackgerrit | Stephen Finucane proposed openstack/oslo.config: Add support for a deprecated version https://review.openstack.org/307671 | 12:31 |
*** salv-orl_ has quit IRC | 12:33 | |
*** salv-orlando has joined #openstack-oslo | 12:34 | |
*** yassine has joined #openstack-oslo | 12:40 | |
*** amoralej is now known as amoralej|lunch | 12:48 | |
*** jpena|lunch is now known as jpena | 12:51 | |
bknudson | amrith: 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#tokens | 12:51 |
*** yamamoto has joined #openstack-oslo | 12:53 | |
*** claudiub has joined #openstack-oslo | 12:53 | |
*** takedakn has quit IRC | 12:53 | |
*** salv-orlando has quit IRC | 13:00 | |
*** salv-orlando has joined #openstack-oslo | 13:00 | |
*** ozamiatin has quit IRC | 13:01 | |
*** amakarov_away is now known as amakarov | 13:06 | |
*** yamamoto has quit IRC | 13:07 | |
*** yamamoto has joined #openstack-oslo | 13:07 | |
*** ozamiatin has joined #openstack-oslo | 13:10 | |
*** yamamoto has quit IRC | 13:12 | |
openstackgerrit | Gevorg Davoian proposed openstack/oslo.messaging: [zmq] Remove redundant Envelope class https://review.openstack.org/328257 | 13:13 |
*** amotoki has joined #openstack-oslo | 13:24 | |
*** ericksonsantos has joined #openstack-oslo | 13:29 | |
*** amoralej|lunch is now known as amoralej | 13:37 | |
*** edmondsw has joined #openstack-oslo | 13:41 | |
*** mtanino has joined #openstack-oslo | 13:43 | |
*** jecarey has joined #openstack-oslo | 13:51 | |
*** jecarey has quit IRC | 13:55 | |
*** yamamoto has joined #openstack-oslo | 13:56 | |
*** jecarey has joined #openstack-oslo | 13:57 | |
*** mhickey has quit IRC | 14:02 | |
*** zz_dimtruck is now known as dimtruck | 14:03 | |
*** links has quit IRC | 14:14 | |
*** mhickey has joined #openstack-oslo | 14:17 | |
*** rlrossit has joined #openstack-oslo | 14:19 | |
*** dukhlov has quit IRC | 14:21 | |
openstackgerrit | Brian Stajkowski proposed openstack/oslo.config: Advanced Option https://review.openstack.org/304355 | 14:29 |
openstackgerrit | Michael Bayer proposed openstack/oslo.db: Memoize sys.exc_info() before attempting a savepoint rollback https://review.openstack.org/329215 | 14:35 |
*** mhickey has quit IRC | 14:35 | |
openstackgerrit | Alexey Yelistratov proposed openstack/osprofiler: Add MongoDB driver https://review.openstack.org/310530 | 14:41 |
*** mhickey has joined #openstack-oslo | 14:47 | |
*** amotoki has quit IRC | 14:55 | |
*** _stowa has joined #openstack-oslo | 14:58 | |
*** yamahata has joined #openstack-oslo | 15:05 | |
*** mhickey has quit IRC | 15:07 | |
*** pradk has quit IRC | 15:11 | |
*** rcernin has quit IRC | 15:11 | |
*** _stowa has quit IRC | 15:12 | |
*** jecarey has quit IRC | 15:13 | |
*** jecarey_ has joined #openstack-oslo | 15:22 | |
*** jecarey__ has joined #openstack-oslo | 15:24 | |
*** salv-orlando has quit IRC | 15:25 | |
*** salv-orlando has joined #openstack-oslo | 15:26 | |
*** jecarey__ has quit IRC | 15:26 | |
*** jecarey_ has quit IRC | 15:27 | |
*** jecarey has joined #openstack-oslo | 15:27 | |
*** salv-orlando has quit IRC | 15:30 | |
*** salv-orlando has joined #openstack-oslo | 15:30 | |
*** permalac has joined #openstack-oslo | 15:38 | |
*** eezhova has quit IRC | 15:39 | |
*** yamamoto has quit IRC | 15:43 | |
*** pradk has joined #openstack-oslo | 15:44 | |
*** yamamoto has joined #openstack-oslo | 15:44 | |
*** ozamiatin has quit IRC | 15:45 | |
*** eezhova has joined #openstack-oslo | 15:46 | |
*** yamamoto has quit IRC | 15:47 | |
*** yamamoto has joined #openstack-oslo | 15:48 | |
*** yamamoto has quit IRC | 15:50 | |
*** yamamoto has joined #openstack-oslo | 15:55 | |
*** gcb has quit IRC | 15:55 | |
*** gcb has joined #openstack-oslo | 15:56 | |
*** yassine has quit IRC | 15:58 | |
*** e0ne has quit IRC | 16:00 | |
*** yamamoto has quit IRC | 16:01 | |
*** yamamoto has joined #openstack-oslo | 16:10 | |
*** yamamoto_ has joined #openstack-oslo | 16:11 | |
*** yamamoto has quit IRC | 16:12 | |
*** rlrossit has quit IRC | 16:13 | |
*** yamahata has quit IRC | 16:14 | |
*** yamamoto_ has quit IRC | 16:15 | |
*** permalac has quit IRC | 16:16 | |
*** rlrossit has joined #openstack-oslo | 16:17 | |
*** rlrossit has quit IRC | 16:17 | |
*** rlrossit has joined #openstack-oslo | 16:17 | |
*** yamahata has joined #openstack-oslo | 16:18 | |
*** pradk has quit IRC | 16:26 | |
*** nkrinner is now known as nkrinner_afk | 16:34 | |
*** zancas has joined #openstack-oslo | 16:34 | |
*** yamamoto has joined #openstack-oslo | 16:36 | |
harlowja | amrith i'm sure jd__ will love all of this :-P | 16:38 |
harlowja | ;) | 16:38 |
amrith | harlowja, 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 |
amrith | bknudson, the code I proposed DOES do just what you want | 16:39 |
amrith | isotime() the legacy one DOES NOT. it produces 2013-02-27T18:30:59.999999 | 16:40 |
amrith | no Z at the end | 16:40 |
amrith | I have highlighted this difference specifically in my review | 16:40 |
*** eezhova has quit IRC | 16:40 | |
amrith | bknudson ^^ | 16:40 |
bknudson | amrith: 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 IRC | 16:43 | |
*** e0ne has joined #openstack-oslo | 16:43 | |
*** e0ne has quit IRC | 16:44 | |
amrith | ++ | 16:45 |
*** salv-orlando has quit IRC | 16:49 | |
openstackgerrit | Alexey Yelistratov proposed openstack/osprofiler: Add MongoDB driver https://review.openstack.org/310530 | 16:50 |
*** salv-orlando has joined #openstack-oslo | 16:50 | |
*** sputnik13 has joined #openstack-oslo | 16:50 | |
*** browne has joined #openstack-oslo | 16:56 | |
amrith | rbradfor, yt? | 16:57 |
amrith | will you be pushing up a new change set for https://review.openstack.org/#/c/296667/? | 16:57 |
patchbot | amrith: patch 296667 - python-troveclient - [WIP] Replace Oslo Incubator apiclient usage | 16:57 |
*** ozamiatin has joined #openstack-oslo | 17:01 | |
AJaeger | oslo cores, could you import translations again, please? See https://review.openstack.org/#/q/status:open+topic:zanata/translations,n,z for all open changes | 17:02 |
*** lpetrut has quit IRC | 17:05 | |
*** e0ne has joined #openstack-oslo | 17:12 | |
harlowja | will do | 17:15 |
*** pradk has joined #openstack-oslo | 17:17 | |
AJaeger | thanks, harlowja ! | 17:20 |
harlowja | np | 17:20 |
*** ankur-gupta-f has quit IRC | 17:24 | |
*** miqui has joined #openstack-oslo | 17:26 | |
*** salv-orlando has quit IRC | 17:26 | |
*** yamamoto has quit IRC | 17:26 | |
*** salv-orlando has joined #openstack-oslo | 17:26 | |
*** salv-orl_ has joined #openstack-oslo | 17:29 | |
*** salv-orlando has quit IRC | 17:32 | |
*** eezhova has joined #openstack-oslo | 17:40 | |
*** e0ne has quit IRC | 17:41 | |
*** tonytan4ever has joined #openstack-oslo | 17:41 | |
*** zancas has quit IRC | 17:42 | |
*** ihrachys has quit IRC | 17:46 | |
*** mtanino has quit IRC | 17:51 | |
*** jpena is now known as jpena|off | 17:51 | |
*** ankur-gupta-f has joined #openstack-oslo | 17:51 | |
*** AJaeger has left #openstack-oslo | 17:53 | |
*** mhickey has joined #openstack-oslo | 17:54 | |
*** Rockyg has joined #openstack-oslo | 17:58 | |
*** zancas has joined #openstack-oslo | 17:59 | |
openstackgerrit | Merged openstack/oslo.privsep: Imported Translations from Zanata https://review.openstack.org/328596 | 18:01 |
amrith | harlowja, got a second to discuss your comment (line 99, timeutils.py) | 18:02 |
openstackgerrit | amrith proposed openstack/oslo.utils: un-deprecate (resurrect) timeutils.isotime https://review.openstack.org/329384 | 18:06 |
*** mtanino has joined #openstack-oslo | 18:08 | |
*** salv-orl_ has quit IRC | 18:10 | |
*** salv-orlando has joined #openstack-oslo | 18:11 | |
*** salv-orlando has quit IRC | 18:11 | |
*** salv-orlando has joined #openstack-oslo | 18:12 | |
openstackgerrit | Michael Bayer proposed openstack/oslo.db: Memoize sys.exc_info() before attempting a savepoint rollback https://review.openstack.org/329215 | 18:13 |
openstackgerrit | Merged openstack/oslo.reports: Imported Translations from Zanata https://review.openstack.org/328594 | 18:13 |
*** eezhova has quit IRC | 18:14 | |
openstackgerrit | Merged openstack/oslo.cache: Imported Translations from Zanata https://review.openstack.org/328588 | 18:15 |
openstackgerrit | Merged openstack/oslo.concurrency: Imported Translations from Zanata https://review.openstack.org/328608 | 18:15 |
openstackgerrit | Merged openstack/oslo.utils: Imported Translations from Zanata https://review.openstack.org/328606 | 18:17 |
openstackgerrit | Merged openstack/oslo.versionedobjects: Imported Translations from Zanata https://review.openstack.org/328595 | 18:20 |
*** eezhova has joined #openstack-oslo | 18:23 | |
*** mhickey has quit IRC | 18:23 | |
*** mhickey has joined #openstack-oslo | 18:24 | |
*** jecarey has quit IRC | 18:24 | |
*** lpetrut has joined #openstack-oslo | 18:26 | |
*** jecarey has joined #openstack-oslo | 18:26 | |
harlowja | amrith in a few, gotta do a phone call | 18:30 |
*** harlowja has quit IRC | 18:30 | |
*** rlrossit has quit IRC | 18:31 | |
*** harlowja has joined #openstack-oslo | 18:32 | |
jamielennox | hey 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 along | 18:33 |
patchbot | jamielennox: patch 326782 - oslo.middleware - Expose sample config opts for http-proxy-to-wsgi | 18:33 |
*** e0ne has joined #openstack-oslo | 18:35 | |
*** bzurawski has joined #openstack-oslo | 18:42 | |
amrith | ok harlowja | 18:43 |
*** salv-orlando has quit IRC | 18:45 | |
*** sdague has quit IRC | 18:46 | |
*** salv-orlando has joined #openstack-oslo | 18:46 | |
*** lifeless has quit IRC | 18:46 | |
*** lifeless has joined #openstack-oslo | 18:47 | |
*** zancas has quit IRC | 18:50 | |
*** e0ne has quit IRC | 18:53 | |
*** bzurawski has quit IRC | 18:53 | |
*** ankur-gupta-f has left #openstack-oslo | 18:55 | |
*** eezhova has quit IRC | 18:56 | |
*** eezhova has joined #openstack-oslo | 18:56 | |
*** amoralej is now known as amoralej|off | 18:57 | |
*** eezhova has quit IRC | 18:59 | |
*** jecarey has quit IRC | 19:00 | |
*** eezhova has joined #openstack-oslo | 19:01 | |
*** jecarey has joined #openstack-oslo | 19:02 | |
*** jecarey has quit IRC | 19:02 | |
*** jecarey has joined #openstack-oslo | 19:03 | |
*** amakarov is now known as amakarov_away | 19:03 | |
*** eezhova has quit IRC | 19:08 | |
*** eezhova has joined #openstack-oslo | 19:08 | |
*** tonytan4ever has quit IRC | 19:25 | |
*** dtardivel has quit IRC | 19:27 | |
*** eezhova has quit IRC | 19:27 | |
*** jecarey has quit IRC | 19:28 | |
*** jecarey has joined #openstack-oslo | 19:28 | |
*** rlrossit has joined #openstack-oslo | 19:29 | |
*** jecarey has quit IRC | 19:34 | |
*** zancas has joined #openstack-oslo | 19:36 | |
*** eezhova has joined #openstack-oslo | 19:39 | |
*** jecarey has joined #openstack-oslo | 19:41 | |
*** salv-orlando has quit IRC | 19:43 | |
*** salv-orlando has joined #openstack-oslo | 19:44 | |
*** eezhova has quit IRC | 19:44 | |
*** dimtruck is now known as zz_dimtruck | 19:44 | |
*** harlowja_ has joined #openstack-oslo | 19:45 | |
*** harlowja has quit IRC | 19:49 | |
*** jsavak has joined #openstack-oslo | 19:50 | |
*** sdague has joined #openstack-oslo | 19:56 | |
*** e0ne has joined #openstack-oslo | 19:58 | |
harlowja_ | amrith ok i'm around if u got time | 20:00 |
amrith | i 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 |
amrith | there usually is an informative discussion | 20:01 |
harlowja_ | *sometimes* | 20:02 |
harlowja_ | lol | 20:02 |
amrith | well, the bar on what I consider informative is likely different from yours :) | 20:02 |
harlowja_ | :-P | 20:02 |
harlowja_ | fair enough | 20:02 |
amrith | so I posted a response to your comment re: force utc | 20:02 |
harlowja_ | ya, i'm just wondering if that's peoples resistance to the adding this back | 20:03 |
harlowja_ | in that it makes those kinds of assumptions (by default) | 20:03 |
*** jsavak has quit IRC | 20:03 | |
*** eezhova has joined #openstack-oslo | 20:03 | |
* harlowja_ just was a thought | 20:03 | |
amrith | it may be, but these assumptions are the same ones that isoformat() makes | 20:03 |
harlowja_ | fair point | 20:04 |
harlowja_ | dims can u expand more on your thought about that review? | 20:05 |
*** yamahata has quit IRC | 20:05 | |
harlowja_ | just to much pain for this? something else? | 20:05 |
amrith | dims is at the tc meeting | 20:05 |
harlowja_ | oh | 20:05 |
harlowja_ | welllll | 20:05 |
amrith | he and I talked about this a while ago | 20:06 |
*** zz_dimtruck is now known as dimtruck | 20:06 | |
*** ozamiatin_ has joined #openstack-oslo | 20:06 | |
amrith | I 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 IRC | 20: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 :-P | 20:06 |
amrith | I understood it to be that having this in oslo added little value given that it can be done fairly painlessly with isoformat(). | 20:07 |
amrith | In 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 :-P | 20:07 |
amrith | but in this case, I believe that the issue is a bit more convoluted. | 20:07 |
harlowja_ | ya | 20:08 |
amrith | there 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 |
amrith | setting 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 |
amrith | the replacement of "Z" with +00:00. | 20:09 |
harlowja_ | right the isoformat() change forced that | 20:09 |
harlowja_ | (which even though it doesn't seem like it should matter i guess it does for folks) | 20:09 |
amrith | not 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 though | 20: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 :-P | 20:10 |
amrith | downstream parsers which expected the Z and didn't expect the timezone | 20:10 |
amrith | there would always be a Z | 20:10 |
amrith | and never a +/-NN:NN | 20:10 |
*** eezhova has quit IRC | 20:10 | |
harlowja_ | iso8601 lib that jd__ maintains :-P? | 20:10 |
bknudson | identity never documented that we'd ever present +00:00. Only Z | 20:11 |
harlowja_ | did u say u'd present a iso8601 string? | 20:11 |
bknudson | No, we didn't say it was iso8601 | 20:11 |
amrith | well, iso8601 is another issue ... see: http://pyiso8601.readthedocs.io/en/latest/#where-it-differs-from-iso-8601 | 20:11 |
harlowja_ | k | 20:11 |
harlowja_ | bknudson okie, thats interesting | 20:11 |
amrith | Z is iso8601 and rfc 3339 compliant | 20: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 IRC | 20:12 | |
harlowja_ | amrith ya, i'm unsure why that lib differs from the spec (weird) | 20:12 |
amrith | yes, I don't know for sure either. | 20:13 |
amrith | I looked through the code and commit histories for the project to see if there was an indication of why | 20:13 |
amrith | but I could find none. | 20:13 |
harlowja_ | probably jd__ fault, lol | 20:13 |
harlowja_ | ha | 20:14 |
harlowja_ | sneaky guy | 20:14 |
harlowja_ | bknudson did the docs ever get updated to say 'iso8601 formatted string <here>' | 20:14 |
*** kgiusti has quit IRC | 20:15 | |
amrith | I 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 |
bknudson | harlowja_: the docs don't actually define the format. The only place is in examples. | 20:15 |
harlowja_ | bknudson k | 20:15 |
bknudson | http://specs.openstack.org/openstack/keystone-specs/api/v3/identity-api-v3.html is the spec | 20:15 |
bknudson | harlowja_: 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, ha | 20:16 |
bknudson | so apparently one field is iso8601 | 20:16 |
bknudson | the 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 those | 20:17 |
harlowja_ | which also says ISO8601 format | 20:17 |
bknudson | hmm, ok, maybe then we can just change it. | 20:17 |
bknudson | we'd have to fix all the tests, and probably tempest | 20:17 |
amrith | bknudson, there are changes all over the place if I have to change that | 20:18 |
*** cdent has quit IRC | 20:18 | |
amrith | all 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 |
bknudson | anyways, we're not going to change our code to switch from Z to 00:00 | 20: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 |
bknudson | yes, since we've doc'd it's iso8601 then anything in iso8601 would work. | 20:20 |
bknudson | could 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 |
amrith | 1s, timesliced by conversation on smaug | 20:23 |
harlowja_ | k | 20:23 |
*** zancas has quit IRC | 20:25 | |
*** zancas has joined #openstack-oslo | 20:26 | |
dims | harlowja_ : 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 accord | 20:27 |
bknudson | anyways 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 |
amrith | am back | 20:28 |
harlowja_ | dims ok, so the only busted case is the assumption that no_timezone == 'UTC' (for some definition of busted)? | 20:28 |
amrith | for what definition of busted? that's an assumption ... :) | 20:29 |
*** e0ne has quit IRC | 20:29 | |
* amrith tries to parse dims' concern | 20:30 | |
harlowja_ | i need to know defintion first :-P | 20:30 |
amrith | harlowja_, 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 IRC | 20:31 | |
amrith | if keystone for example changed its timestamp representation, the ripple could affect other projects as well. | 20:32 |
amrith | especially if they followed the path of cloning isotime() with its existing warts. | 20:32 |
amrith | actually, not just isotime(). there are other methods/functions being deprecated as well. | 20:32 |
harlowja_ | likely | 20:32 |
amrith | I think bknudson has a very good point when he says that we need a standard timestamp stringizing format for openstack | 20:33 |
harlowja_ | iso8601 ? | 20:33 |
amrith | well iso8601 is a specification | 20:33 |
amrith | not a format | 20:33 |
harlowja_ | k | 20:33 |
amrith | it provides many formats that are all valid | 20:33 |
harlowja_ | ya, pick the least crappy one of those | 20:33 |
amrith | and is permissive to a fault. | 20:33 |
harlowja_ | agreed | 20:33 |
bknudson | iso8601 actually allows lots of formats, for example "1" is a valid iso8601 string. | 20:33 |
harlowja_ | ya, thats a weird spec, lol | 20:34 |
bknudson | it also supports durations | 20:34 |
amrith | QED | 20:34 |
bknudson | which is cool but way too lenient | 20:34 |
harlowja_ | def | 20:34 |
bknudson | something like https://www.w3.org/TR/NOTE-datetime would be better | 20:34 |
harlowja_ | def | 20:35 |
harlowja_ | pick one of the 6 'levels' and that should do it | 20:35 |
harlowja_ | or even 3 levels | 20:35 |
harlowja_ | which i guess amrith why u are thinking the TC should perhaps take this topic up? | 20:36 |
*** rlrossit has joined #openstack-oslo | 20:36 | |
harlowja_ | (or that's why i think u thought that) | 20:36 |
harlowja_ | brb | 20:36 |
bknudson | maybe the API working group. | 20:36 |
bknudson | (if they don't have a recommendation already) | 20:36 |
amrith | harlowja_, 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 choice | 20:37 |
bknudson | so I would prefer if the identity API spec referenced https://www.w3.org/TR/NOTE-datetime rather than saying iso 8601. | 20:37 |
bknudson | and that could probably be done anyways | 20:37 |
bknudson | to take a wider view -- all of openstack should use the same format for timestamps. | 20:38 |
amrith | that would make sense | 20:38 |
amrith | and given that today there are few who use isoformat() we should stick to the current isotime() :> | 20:39 |
*** jecarey has quit IRC | 20:40 | |
*** jecarey has joined #openstack-oslo | 20:40 | |
bknudson | saves a few bytes to use Z rather than 00:00 | 20:49 |
harlowja_ | :-/ | 20:50 |
*** edmondsw has quit IRC | 20:52 | |
*** shardy has quit IRC | 20:53 | |
*** eezhova has joined #openstack-oslo | 20:58 | |
zancas | Hiya channel! | 20:58 |
zancas | I'm wondering if it makes more sense to correct the docs here: | 20:58 |
zancas | quired Option | 20:58 |
zancas | 22 │ with 'min/max'. │* | bb0f7e3 (tag: 3.11.0) Merge "generator: format string default value for L | 20:58 |
zancas | 23 from oslo_config import cfg │ │ist type properly" | 20:58 |
zancas | :( Oooppsss... | 20:58 |
zancas | Sorry the docs here: | 20:59 |
zancas | https://github.com/openstack/oslo.config/blob/master/oslo_config/cfg.py#L42 | 20:59 |
zancas | To assert that the value of the type parameter can be other-than-a-string? | 20:59 |
zancas | Since e.g. types.Integer can accept ints? | 21:00 |
*** jd__ has quit IRC | 21:01 | |
zancas | or would it be better to make the "types" more strict to match the implication of the doc... | 21:01 |
zancas | i.e. make it so that non-string arguments raise an exception? | 21:01 |
*** jd__ has joined #openstack-oslo | 21:01 | |
zancas | The current behavior is that e.g. x = types.Integer(1,4) | 21:02 |
zancas | y = x(3) | 21:02 |
zancas | Works... which is inconsistent with the (poorly) linked doc... :( | 21:02 |
zancas | Again sorry for the paste dump! | 21:03 |
zancas | I could see reasonable arguments for either behavior... on the one hand... | 21:04 |
zancas | It's pretty obvious what was intended when a user passed 3 as an option to a types.Integer | 21:04 |
zancas | validated parameter... | 21:04 |
zancas | On the other hand, when would we expect non-strings to be passed to an option parser? | 21:05 |
zancas | Perhaps after options are parsed from config files? <-- In answer to my own question. | 21:05 |
*** e0ne has joined #openstack-oslo | 21:06 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.config: Updated from global requirements https://review.openstack.org/322570 | 21:09 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements https://review.openstack.org/324848 | 21:09 |
*** yamahata has joined #openstack-oslo | 21:12 | |
openstackgerrit | zancas proposed openstack/oslo.config: Correct doc to include arbitrary inputs to `types` https://review.openstack.org/329684 | 21:25 |
*** miqui has quit IRC | 21:26 | |
*** jecarey_ has joined #openstack-oslo | 21:27 | |
elarson | does anyone here have any insight into who is responsible for the openstack dev list server? | 21:29 |
*** jecarey has quit IRC | 21:30 | |
*** eezhova has quit IRC | 21:30 | |
elarson | I'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 |
bknudson | elarson: I suggest taking that question to the #openstack-infra channel | 21:34 |
*** eezhova has joined #openstack-oslo | 21:34 | |
elarson | bknudson: thanks! will do! | 21:34 |
*** pradk has joined #openstack-oslo | 21:39 | |
*** eezhova has quit IRC | 21:41 | |
*** e0ne has quit IRC | 21:42 | |
*** eezhova has joined #openstack-oslo | 21:45 | |
*** sileht has quit IRC | 21:51 | |
*** sileht has joined #openstack-oslo | 21:52 | |
*** jecarey_ has quit IRC | 21:57 | |
*** dimtruck is now known as zz_dimtruck | 21:58 | |
*** browne has quit IRC | 22:08 | |
*** rlrossit has quit IRC | 22:10 | |
*** Rockyg has quit IRC | 22:12 | |
*** zancas has quit IRC | 22:18 | |
*** eezhova has quit IRC | 22:19 | |
harlowja_ | lxsli just a quick question on the spooling stuff, when does log msg <-> value interpolation happen? | 22:19 |
*** mhickey has quit IRC | 22: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 weird | 22:20 |
harlowja_ | for example | 22:20 |
harlowja_ | since python is by reference :-/ | 22:20 |
harlowja_ | (except for value types) | 22:20 |
harlowja_ | so if i was doing something like | 22:21 |
harlowja_ | initial_users = ['a'] | 22:21 |
harlowja_ | LOG.debug(my initial users, initial_users) | 22:21 |
harlowja_ | then the_new_uers = initial_users | 22: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_awa | 22:23 | |
*** openstackstatus has quit IRC | 22:25 | |
*** openstack has joined #openstack-oslo | 22:25 | |
*** openstackstatus has joined #openstack-oslo | 22:26 | |
*** ChanServ sets mode: +v openstackstatus | 22:26 | |
*** browne has joined #openstack-oslo | 22:27 | |
*** rcernin has joined #openstack-oslo | 22:28 | |
*** mtanino has quit IRC | 22:38 | |
*** mtanino has joined #openstack-oslo | 22:43 | |
*** salv-orlando has quit IRC | 23:14 | |
*** salv-orlando has joined #openstack-oslo | 23:14 | |
*** lpetrut has quit IRC | 23:22 | |
*** salv-orl_ has joined #openstack-oslo | 23:29 | |
*** browne has quit IRC | 23:30 | |
*** salv-orlando has quit IRC | 23:32 | |
*** browne has joined #openstack-oslo | 23:35 | |
*** rcernin has quit IRC | 23:40 | |
*** sdague has quit IRC | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!