| *** tsekiyam_ has joined #openstack-oslo | 00:12 | |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Use encodeutils for exception -> string function https://review.openstack.org/192458 | 00:14 |
|---|---|---|
| *** mtanino has quit IRC | 00:15 | |
| *** tsekiyama has quit IRC | 00:16 | |
| *** tsekiyam_ has quit IRC | 00:17 | |
| *** manishg has joined #openstack-oslo | 00:20 | |
| *** sputnik1_ has quit IRC | 00:21 | |
| *** manishg has quit IRC | 00:21 | |
| *** manishg has joined #openstack-oslo | 00:25 | |
| *** ViswaV has quit IRC | 00:26 | |
| *** peterstac has quit IRC | 00:27 | |
| *** ViswaV has joined #openstack-oslo | 00:28 | |
| *** sirushti has quit IRC | 00:28 | |
| *** sdake has quit IRC | 00:29 | |
| *** ViswaV has quit IRC | 00:32 | |
| openstackgerrit | Brant Knudson proposed openstack/oslo.cache: Docs for configure() https://review.openstack.org/197382 | 00:32 |
| *** dims has joined #openstack-oslo | 00:32 | |
| *** ViswaV has joined #openstack-oslo | 00:33 | |
| openstackgerrit | Brant Knudson proposed openstack/oslo.cache: Fix docs for get_memoization_decorator https://review.openstack.org/197383 | 00:33 |
| *** sirushti has joined #openstack-oslo | 00:33 | |
| openstackgerrit | Brant Knudson proposed openstack/oslo.cache: Remove useless docs from private functions https://review.openstack.org/197384 | 00:33 |
| *** peterstac has joined #openstack-oslo | 00:34 | |
| openstackgerrit | Brant Knudson proposed openstack/oslo.cache: Fix docstring for configure_cache_region https://review.openstack.org/197385 | 00:34 |
| openstackgerrit | Brant Knudson proposed openstack/oslo.cache: Make DebugProxy private https://review.openstack.org/197386 | 00:35 |
| *** dims_ has joined #openstack-oslo | 00:35 | |
| openstackgerrit | Merged openstack/oslo.config: fix fixture tests for fixtures 1.3.x https://review.openstack.org/197299 | 00:36 |
| krotscheck | lifeless: Tell ya what. You go work on the speed at which it takes to actually land code, and remove some of the barriers which force any meaningful work to take months to land, and I'll promise that it'll free up enough time for me to handle that change you want :) | 00:37 |
| *** manishg has quit IRC | 00:37 | |
| * krotscheck has already lost weeks in rechecks. | 00:38 | |
| krotscheck | Not to mention spec revisions. | 00:38 |
| *** dims has quit IRC | 00:39 | |
| lifeless | krotscheck: that is what I'm trying to accomplish | 00:49 |
| lifeless | krotscheck: the first cab off the rank is reducing gate wedges which break and stall everyone | 00:50 |
| *** ViswaV has quit IRC | 00:51 | |
| *** crc32 has quit IRC | 00:55 | |
| *** sigmavirus24 is now known as sigmavirus24_awa | 00:58 | |
| *** boris-42 has quit IRC | 01:02 | |
| *** stevemar has joined #openstack-oslo | 01:02 | |
| *** Rockyg has joined #openstack-oslo | 01:03 | |
| stevemar | dims_: bknudson ughhhh we privatized one thing too many | 01:09 |
| dims_ | ha :) | 01:09 |
| stevemar | dims_: ugh, give me a minute to fix this | 01:12 |
| openstackgerrit | Brant Knudson proposed openstack/oslo.cache: Don't use global conf https://review.openstack.org/197389 | 01:13 |
| openstackgerrit | Steve Martinelli proposed openstack/oslo.cache: make REGION public https://review.openstack.org/197391 | 01:17 |
| *** manishg has joined #openstack-oslo | 01:23 | |
| *** zzzeek has quit IRC | 01:23 | |
| *** Rockyg has quit IRC | 01:28 | |
| *** sputnik13 has joined #openstack-oslo | 01:30 | |
| *** sputnik13 has quit IRC | 01:31 | |
| *** sputnik13 has joined #openstack-oslo | 01:32 | |
| *** sputnik13 has quit IRC | 01:33 | |
| *** sputnik1_ has joined #openstack-oslo | 01:33 | |
| *** rjaiswal has quit IRC | 01:34 | |
| openstackgerrit | Merged openstack/oslo.service: Track created thread and disallow more than one start being active https://review.openstack.org/195358 | 01:34 |
| *** edleafe has quit IRC | 01:35 | |
| *** edleafe has joined #openstack-oslo | 01:37 | |
| *** sputnik1_ has quit IRC | 01:37 | |
| stevemar | i wonder what bknudson is trying to say in his comment... | 01:40 |
| *** sputnik13 has joined #openstack-oslo | 01:41 | |
| *** sputnik13 has quit IRC | 01:41 | |
| *** yamahata has quit IRC | 02:05 | |
| *** ajo has quit IRC | 02:08 | |
| openstackgerrit | Steve Martinelli proposed openstack/oslo.cache: make REGION public https://review.openstack.org/197391 | 02:09 |
| *** sdake has joined #openstack-oslo | 02:11 | |
| *** ajo has joined #openstack-oslo | 02:11 | |
| *** mriedem has quit IRC | 02:16 | |
| openstackgerrit | Merged openstack/oslo.cache: Fix docs for get_memoization_decorator https://review.openstack.org/197383 | 02:22 |
| openstackgerrit | Merged openstack/oslo.cache: Docs for configure() https://review.openstack.org/197382 | 02:23 |
| openstackgerrit | Merged openstack/oslo.cache: make REGION public https://review.openstack.org/197391 | 02:25 |
| openstackgerrit | Brant Knudson proposed openstack/oslo.cache: Don't use global conf https://review.openstack.org/197389 | 02:25 |
| openstackgerrit | Merged openstack/oslo.cache: Fix docstring for configure_cache_region https://review.openstack.org/197385 | 02:26 |
| openstackgerrit | Brant Knudson proposed openstack/oslo.cache: Don't use global conf https://review.openstack.org/197389 | 02:28 |
| *** tobe has joined #openstack-oslo | 02:29 | |
| *** manishg has quit IRC | 02:32 | |
| openstackgerrit | Brant Knudson proposed openstack/oslo.cache: Make DebugProxy private https://review.openstack.org/197386 | 02:32 |
| *** Kennan has quit IRC | 02:38 | |
| openstackgerrit | Merged openstack/oslo.config: Fixes example for cfg https://review.openstack.org/195536 | 02:38 |
| *** Kennan has joined #openstack-oslo | 02:38 | |
| openstackgerrit | Merged openstack/oslo.cache: Make DebugProxy private https://review.openstack.org/197386 | 02:51 |
| openstackgerrit | Merged openstack/oslo.cache: Remove useless docs from private functions https://review.openstack.org/197384 | 02:52 |
| openstackgerrit | Steve Martinelli proposed openstack/oslo.cache: Don't use global conf https://review.openstack.org/197389 | 02:56 |
| stevemar | dims_: last one ^ | 02:57 |
| dims_ | oops pep8 failure | 02:58 |
| dims_ | stevemar: ^^ | 02:59 |
| stevemar | gah | 02:59 |
| stevemar | ffs | 02:59 |
| openstackgerrit | Steve Martinelli proposed openstack/oslo.cache: Don't use global conf https://review.openstack.org/197389 | 03:00 |
| stevemar | dims_: ^ | 03:00 |
| stevemar | god damn, i need to update the keystone code now | 03:01 |
| stevemar | since we'll be taking conf objects | 03:01 |
| * stevemar shakes fist at brant | 03:01 | |
| openstackgerrit | Merged openstack/oslo.cache: Don't use global conf https://review.openstack.org/197389 | 03:10 |
| stevemar | hmmm, we should swap the order on one function | 03:11 |
| openstackgerrit | Steve Martinelli proposed openstack/oslo.cache: swap the order of conf for a function https://review.openstack.org/197403 | 03:13 |
| stevemar | dims_: brant ^ | 03:13 |
| openstackgerrit | Steve Martinelli proposed openstack/oslo.cache: swap the order of conf for a function https://review.openstack.org/197403 | 03:18 |
| *** zzzeek has joined #openstack-oslo | 03:26 | |
| *** zzzeek has quit IRC | 03:26 | |
| openstackgerrit | Steve Martinelli proposed openstack/oslo.cache: swap the order of conf for a function https://review.openstack.org/197403 | 03:33 |
| *** harlowja_at_home has joined #openstack-oslo | 03:40 | |
| *** jecarey has joined #openstack-oslo | 03:43 | |
| *** stevemar has quit IRC | 04:00 | |
| *** stevemar has joined #openstack-oslo | 04:01 | |
| *** dims_ has quit IRC | 04:03 | |
| *** harlowja_at_home has quit IRC | 04:15 | |
| *** achanda has joined #openstack-oslo | 04:19 | |
| *** jecarey has quit IRC | 04:21 | |
| *** morganfainberg is now known as caerbannograbbit | 04:24 | |
| *** caerbannograbbit is now known as CaerbannogRabbit | 04:24 | |
| openstackgerrit | Joshua Harlow proposed openstack/oslo.service: Ensure we set the event and wait on the timer in the test https://review.openstack.org/197415 | 04:25 |
| *** tobe has quit IRC | 04:26 | |
| *** kzaitsev_mb has joined #openstack-oslo | 04:32 | |
| *** achanda has quit IRC | 04:38 | |
| *** browne1 has joined #openstack-oslo | 04:40 | |
| *** browne has quit IRC | 04:40 | |
| *** nkrinner has joined #openstack-oslo | 04:41 | |
| *** kzaitsev_mb has quit IRC | 04:46 | |
| openstackgerrit | Steve Martinelli proposed openstack/oslo.utils: Add fileutils to oslo_utils https://review.openstack.org/154975 | 04:46 |
| *** rpodolyaka1 has joined #openstack-oslo | 04:47 | |
| *** eezhova has joined #openstack-oslo | 05:02 | |
| *** sreshetnyak has joined #openstack-oslo | 05:05 | |
| *** dims has joined #openstack-oslo | 05:07 | |
| *** exploreshaifali has joined #openstack-oslo | 05:10 | |
| *** dims has quit IRC | 05:12 | |
| *** arnaud____ has joined #openstack-oslo | 05:17 | |
| *** arnaud____ has quit IRC | 05:22 | |
| *** tobe has joined #openstack-oslo | 05:26 | |
| openstackgerrit | Steve Martinelli proposed openstack/oslo.policy: Move fileutils functions to oslo.policy https://review.openstack.org/197420 | 05:27 |
| openstackgerrit | Merged openstack/taskflow: Updated from global requirements https://review.openstack.org/194836 | 05:30 |
| *** rjaiswal has joined #openstack-oslo | 05:32 | |
| openstackgerrit | Merged openstack/futurist: Updated from global requirements https://review.openstack.org/196262 | 05:37 |
| *** jgrimm is now known as zz_jgrimm | 05:55 | |
| *** rpodolyaka1 has quit IRC | 05:57 | |
| *** tobe has quit IRC | 05:59 | |
| *** e0ne has joined #openstack-oslo | 05:59 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.i18n: Imported Translations from Transifex https://review.openstack.org/197427 | 06:01 |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Add support for conditional execution https://review.openstack.org/164922 | 06:04 |
| openstackgerrit | Joshua Harlow proposed openstack/taskflow: Add ability to reset an engine via a `reset` method https://review.openstack.org/186239 | 06:04 |
| *** e0ne has quit IRC | 06:11 | |
| *** browne1 has quit IRC | 06:21 | |
| *** stevemar has quit IRC | 06:28 | |
| *** stevemar has joined #openstack-oslo | 06:28 | |
| *** stevemar has quit IRC | 06:31 | |
| *** rpodolyaka1 has joined #openstack-oslo | 06:34 | |
| openstackgerrit | Joshua Harlow proposed openstack/futurist: Ensure we check callback '_is_periodic' on add https://review.openstack.org/196457 | 06:36 |
| *** yamahata has joined #openstack-oslo | 06:39 | |
| *** ViswaV has joined #openstack-oslo | 06:44 | |
| *** ViswaV_ has joined #openstack-oslo | 06:48 | |
| *** ViswaV has quit IRC | 06:48 | |
| *** tobe has joined #openstack-oslo | 06:53 | |
| *** sreshetnyak has quit IRC | 06:57 | |
| *** ndipanov has quit IRC | 06:57 | |
| *** sreshetnyak has joined #openstack-oslo | 06:57 | |
| openstackgerrit | Peng Wu proposed openstack/oslo.i18n: Support contextual and plural form of gettext functions https://review.openstack.org/134850 | 06:58 |
| *** kzaitsev_mb has joined #openstack-oslo | 07:19 | |
| *** subscope has quit IRC | 07:24 | |
| *** ihrachyshka has joined #openstack-oslo | 07:28 | |
| *** sputnik13 has joined #openstack-oslo | 07:30 | |
| *** zz_jgrimm has quit IRC | 07:31 | |
| *** zz_jgrimm has joined #openstack-oslo | 07:33 | |
| *** kzaitsev_mb has quit IRC | 07:33 | |
| *** haypo has joined #openstack-oslo | 07:34 | |
| *** kzaitsev_mb has joined #openstack-oslo | 07:34 | |
| *** subscope has joined #openstack-oslo | 07:37 | |
| *** kzaitsev_mb has left #openstack-oslo | 07:39 | |
| *** rpodolyaka1 has quit IRC | 07:49 | |
| *** shardy has joined #openstack-oslo | 07:50 | |
| *** e0ne has joined #openstack-oslo | 07:56 | |
| *** shardy has quit IRC | 08:00 | |
| *** shardy has joined #openstack-oslo | 08:00 | |
| openstackgerrit | Merged openstack/oslo.db: Updated from global requirements https://review.openstack.org/197263 | 08:17 |
| *** arnaud____ has joined #openstack-oslo | 08:23 | |
| *** ViswaV_ has quit IRC | 08:25 | |
| *** rpodolyaka1 has joined #openstack-oslo | 08:25 | |
| *** yamahata has quit IRC | 08:26 | |
| *** arnaud____ has quit IRC | 08:28 | |
| *** stevemar has joined #openstack-oslo | 08:30 | |
| *** stevemar has quit IRC | 08:34 | |
| *** exploreshaifali has quit IRC | 08:35 | |
| *** yassine__ has joined #openstack-oslo | 08:36 | |
| *** e0ne is now known as e0ne_ | 08:36 | |
| *** e0ne_ is now known as e0ne | 08:38 | |
| *** dguitarbite has quit IRC | 08:40 | |
| *** e0ne has quit IRC | 08:41 | |
| *** rjaiswal has quit IRC | 08:44 | |
| *** rpodolyaka1 has quit IRC | 08:49 | |
| *** pblaho has joined #openstack-oslo | 08:51 | |
| openstackgerrit | Kairat Kushaev proposed openstack/oslo.config: Add allowed_length restriction fot StrOpt https://review.openstack.org/197477 | 09:04 |
| *** viktors1 has joined #openstack-oslo | 09:05 | |
| *** viktors1 is now known as viktors|wfh | 09:06 | |
| *** exploreshaifali has joined #openstack-oslo | 09:06 | |
| *** rpodolyaka1 has joined #openstack-oslo | 09:07 | |
| *** dims has joined #openstack-oslo | 09:08 | |
| *** dims has quit IRC | 09:14 | |
| openstackgerrit | Merged openstack/oslo.middleware: Support PasteDeploy https://review.openstack.org/191238 | 09:19 |
| *** katyafervent is now known as katyafervent_awa | 09:19 | |
| *** katyafervent_awa is now known as katyafervent | 09:20 | |
| *** ozamiatin has joined #openstack-oslo | 09:23 | |
| openstackgerrit | Merged openstack/oslo.config: Add set_default() method to fixture https://review.openstack.org/197300 | 09:25 |
| *** ihrachyshka has quit IRC | 09:30 | |
| *** e0ne has joined #openstack-oslo | 09:35 | |
| openstackgerrit | Merged openstack/oslo.db: Add legacy get_sessionmaker() method https://review.openstack.org/196885 | 09:39 |
| *** sdake has quit IRC | 09:46 | |
| *** ozamiatin has quit IRC | 09:49 | |
| *** ozamiatin has joined #openstack-oslo | 09:52 | |
| *** dims has joined #openstack-oslo | 09:59 | |
| *** ihrachyshka has joined #openstack-oslo | 09:59 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.config: Updated from global requirements https://review.openstack.org/197262 | 10:01 |
| haypo | jd__: seen in nova: now = timeutils.parse_isotime(timeutils.strtime()) | 10:01 |
| jd__ | haypo: I know, I know… | 10:01 |
| haypo | haha | 10:02 |
| haypo | jd__: did you propose a patch to replace strtime with .isoformat()? i'm asking because i'm writing such patch :) | 10:02 |
| dims | viktors: rpodolyaka: can you please take a peek at https://review.openstack.org/#/c/195578/ (oslo.db change in support of a nova test) | 10:03 |
| haypo | jd__: oh. timeutils.parse_isotime(timeutils.strtime()) creates an aware datetime object using the UTC timezone | 10:04 |
| haypo | replacing it with timeutils.utcnow() is not enough | 10:04 |
| *** rpodolyaka1 has quit IRC | 10:05 | |
| haypo | jd__: we should start to use ".. versionchanged::" in doc, i don't remember which oslo.utils version introduced the with_timezone parameter to oslo_utils.timeutils.utcnow() | 10:06 |
| haypo | so, it was introduced in oslo.utils 1.6 | 10:08 |
| haypo | and now master requires oslo.utils >= 1.6, good :) | 10:11 |
| *** stevemar has joined #openstack-oslo | 10:19 | |
| *** stevemar has quit IRC | 10:22 | |
| openstackgerrit | Florian Wilhelm proposed openstack-dev/pbr: Fix retrieval of commit data and most recent tag. https://review.openstack.org/197508 | 10:34 |
| *** ndipanov has joined #openstack-oslo | 10:36 | |
| openstackgerrit | Florian Wilhelm proposed openstack-dev/pbr: Fix retrieval of commit data and most recent tag. https://review.openstack.org/194954 | 10:38 |
| *** exploreshaifali has quit IRC | 10:39 | |
| *** e0ne is now known as e0ne_ | 10:39 | |
| *** e0ne_ is now known as e0ne | 10:39 | |
| haypo | File "/home/haypo/prog/openstack/nova/.tox/py27/lib/python2.7/site-packages/oslo_utils/timeutils.py", line 108, in parse_strtime | 10:40 |
| haypo | return datetime.datetime.strptime(timestr, fmt) | 10:40 |
| haypo | ValueError: time data '2015-07-01T10:35:15' does not match format '%Y-%m-%dT%H:%M:%S.%f' | 10:40 |
| haypo | jd__: ^^ too bad | 10:40 |
| haypo | (i will investigate that later) | 10:40 |
| *** amrith is now known as _amrith_ | 10:43 | |
| *** rpodolyaka1 has joined #openstack-oslo | 10:56 | |
| rpodolyaka1 | dims: sorry, was afk. I'll tweak the unit test a bit and +2 | 11:04 |
| dims | rpodolyaka1: no worries | 11:05 |
| *** e0ne is now known as e0ne_ | 11:24 | |
| *** e0ne_ is now known as e0ne | 11:24 | |
| *** pradk has joined #openstack-oslo | 11:34 | |
| *** prad has quit IRC | 11:36 | |
| *** pradk has quit IRC | 11:40 | |
| openstackgerrit | Flavio Percoco proposed openstack/oslo.messaging: Remove qpid-config call https://review.openstack.org/194723 | 11:46 |
| *** tobe has quit IRC | 11:46 | |
| *** ozamiatin has quit IRC | 11:52 | |
| *** ozamiatin has joined #openstack-oslo | 11:52 | |
| *** ozamiatin has joined #openstack-oslo | 11:53 | |
| openstackgerrit | Vipin Balachandran proposed openstack/oslo.vmware: Update log level of session re-creation logs https://review.openstack.org/197536 | 11:56 |
| *** jaosorior has joined #openstack-oslo | 11:56 | |
| openstackgerrit | Roman Podoliaka proposed openstack/oslo.db: Allow projects that use test_models_sync to filter some changes https://review.openstack.org/195578 | 11:57 |
| openstackgerrit | Vipin Balachandran proposed openstack/oslo.vmware: Update log level of session related logs https://review.openstack.org/197536 | 12:00 |
| haypo | jd__: so in short, it's not possible to replace timeutils.strtime(ts) with ts.isoformat() without breaking backward compatibility (with applications using timeutils.parse_strtime()) :-( | 12:09 |
| *** marcusvrn has left #openstack-oslo | 12:13 | |
| jd__ | haypo: I've changed many app to use parse_isotime instead | 12:18 |
| jd__ | haypo: in many place we can probably do that as parse_isotime() is a superset of parse_strtime() | 12:18 |
| haypo | jd__: oh good to know | 12:18 |
| haypo | jd__: but is it possible to upgrade one service (ex: nova-compute) and then the others? | 12:19 |
| *** _amrith_ is now known as amrith | 12:19 | |
| haypo | i mean, the new nova-compute may create timestamp that old services may be unable to parse? | 12:19 |
| *** rpodolyaka1 has quit IRC | 12:19 | |
| *** rpodolyaka1 has joined #openstack-oslo | 12:20 | |
| *** shardy has quit IRC | 12:27 | |
| *** zigo has quit IRC | 12:28 | |
| *** zigo has joined #openstack-oslo | 12:29 | |
| *** shardy has joined #openstack-oslo | 12:31 | |
| *** ozamiatin has quit IRC | 12:31 | |
| *** ozamiatin has joined #openstack-oslo | 12:32 | |
| jd__ | haypo: yeah you need to upgrade the parse_strtime to parse_isotime first | 12:36 |
| jd__ | haypo: I did that for nova/context a while back | 12:36 |
| dims | jd__: haypo: one of you please bless this? https://review.openstack.org/#/c/181980/ | 12:40 |
| haypo | dims: oh, i was checking oslo.utils in OpenStack :) | 12:41 |
| *** e0ne is now known as e0ne_ | 12:41 | |
| dims | haypo: aha | 12:41 |
| haypo | dims: +2a | 12:42 |
| *** e0ne_ is now known as e0ne | 12:42 | |
| haypo | dims: see https://review.openstack.org/197546 for oslo.utils in nova | 12:42 |
| dims | haypo: ack will do in a little bit | 12:43 |
| haypo | jd__: oh wait, parse_isotime() is not a drop in replacement. it also adds a timezone, UTC by default | 12:44 |
| haypo | jd__: handling time is much complex than what i expected /o\ (again) | 12:44 |
| *** bknudson has joined #openstack-oslo | 12:45 | |
| jd__ | haypo: yeah but you can normalize_time() | 12:45 |
| haypo | jd__: parse_isodate(ts).replace(tz=None)? :-D | 12:46 |
| jd__ | haypo: NO | 12:49 |
| jd__ | haypo: timeutils.normalize_time :) | 12:49 |
| *** exploreshaifali has joined #openstack-oslo | 12:50 | |
| haypo | lol | 12:51 |
| openstackgerrit | Merged openstack/oslo.cache: swap the order of conf for a function https://review.openstack.org/197403 | 12:53 |
| *** e0ne is now known as e0ne_ | 12:53 | |
| *** rpodolyaka1 has quit IRC | 12:55 | |
| *** e0ne_ is now known as e0ne | 12:57 | |
| *** rpodolyaka1 has joined #openstack-oslo | 12:58 | |
| *** kairat_kushaev has joined #openstack-oslo | 12:59 | |
| *** jecarey has joined #openstack-oslo | 13:00 | |
| *** shardy has quit IRC | 13:01 | |
| *** shardy has joined #openstack-oslo | 13:01 | |
| kairat_kushaev | Hi guys, I noticed that oslo_cache released recently. I have a question. | 13:03 |
| *** kbyrne has quit IRC | 13:03 | |
| kairat_kushaev | What is the difference between Noop backend and null backend available in dogpile cache 0.5. | 13:03 |
| kairat_kushaev | *0.5.4 | 13:04 |
| *** ihrachyshka_ has joined #openstack-oslo | 13:07 | |
| *** ihrachyshka has quit IRC | 13:08 | |
| kairat_kushaev | Don't we need to get rid of NoOp backend in oslo.cache? | 13:08 |
| *** stpierre has joined #openstack-oslo | 13:08 | |
| *** stevemar has joined #openstack-oslo | 13:12 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/oslo.messaging: Use ServiceBase from oslo.service as a parent class https://review.openstack.org/197574 | 13:15 |
| *** kbyrne has joined #openstack-oslo | 13:22 | |
| openstackgerrit | Merged openstack/oslo.i18n: Add tox target to find missing requirements https://review.openstack.org/195845 | 13:22 |
| *** mriedem has joined #openstack-oslo | 13:27 | |
| *** stevemar has quit IRC | 13:27 | |
| *** browne has joined #openstack-oslo | 13:28 | |
| openstackgerrit | Merged openstack/oslo.log: Add tox target to find missing requirements https://review.openstack.org/195844 | 13:35 |
| haypo | i hate mox. probably since i know that mock exists :) | 13:35 |
| openstackgerrit | Merged openstack/oslo.i18n: Imported Translations from Transifex https://review.openstack.org/197427 | 13:36 |
| haypo | how can I write mock.ANY with mox? | 13:36 |
| haypo | timeutils.parse_isotime(self.mox.ANY) | 13:36 |
| haypo | ^^ i need to insert somewhere that parse_isotime was called | 13:37 |
| openstackgerrit | Merged openstack/oslo.serialization: Remove oslo namespace package https://review.openstack.org/181980 | 13:38 |
| openstackgerrit | Merged openstack/oslo.serialization: Add tox target to find missing requirements https://review.openstack.org/195838 | 13:38 |
| openstackgerrit | Merged openstack/oslo.utils: Add tox target to find missing requirements https://review.openstack.org/195837 | 13:42 |
| dims | viktors: rpodolyaka: can you check if you see a problem with cinder + latest oslo.db? i think i am seeing a unit test failure | 13:45 |
| dims | viktors: rpodolyaka: {7} cinder.tests.unit.test_migrations.TestSqliteMigrations.test_walk_versions [60.571445s] ... FAILED | 13:45 |
| *** dbite has quit IRC | 13:46 | |
| *** pradk has joined #openstack-oslo | 13:46 | |
| openstackgerrit | Merged openstack/oslo.config: Updated from global requirements https://review.openstack.org/197262 | 13:47 |
| rpodolyaka1 | dims: will do | 13:49 |
| *** ozamiatin has quit IRC | 13:56 | |
| *** zzzeek has joined #openstack-oslo | 13:56 | |
| *** dims has quit IRC | 13:59 | |
| rpodolyaka1 | dims: works for me locally… will try the whole test suite | 14:00 |
| *** dims has joined #openstack-oslo | 14:00 | |
| dims | rpodolyaka: thanks | 14:01 |
| *** mtanino has joined #openstack-oslo | 14:02 | |
| openstackgerrit | Brant Knudson proposed openstack/oslo.cache: Use "group" rather than "section" https://review.openstack.org/197597 | 14:02 |
| *** manishg has joined #openstack-oslo | 14:02 | |
| *** subscope has quit IRC | 14:03 | |
| *** exploreshaifali has quit IRC | 14:03 | |
| *** e0ne is now known as e0ne_ | 14:08 | |
| *** amrith is now known as _amrith_ | 14:11 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 14:12 | |
| *** ozamiatin has joined #openstack-oslo | 14:15 | |
| viktors|wfh | dims: works for me locally. Where did you seen this error? | 14:15 |
| haypo | jd__: my "giant" patch replacing parse_strtime() with parse_isotime() in nova: https://review.openstack.org/#/c/197601/ | 14:16 |
| openstackgerrit | Merged openstack/oslo.db: Allow projects that use test_models_sync to filter some changes https://review.openstack.org/195578 | 14:17 |
| dims | viktors|wfh: looks like it timed out for me - https://s3.amazonaws.com/archive.travis-ci.org/jobs/69113049/log.txt | 14:18 |
| dims | viktors|wfh: could be the environment i am running in. don't worry about it, i'll try tweaking a few things there | 14:19 |
| jd__ | haypo: congrats | 14:19 |
| haypo | jd__: it was more complex than my first command sed -i -e 's,timeutils\.parse_strtime,timeutils.parse_isotime,g' $(find nova -name "*.py") | 14:19 |
| haypo | jd__: please wait until the patch is merged to congrat me :) | 14:20 |
| viktors|wfh | dims: ok | 14:20 |
| *** manishg has quit IRC | 14:21 | |
| openstackgerrit | Merged openstack/automaton: When a state has no transitions show its own `on_exit` and `on_enter` https://review.openstack.org/191482 | 14:22 |
| openstackgerrit | Merged openstack/oslo.config: Add tox target to find missing requirements https://review.openstack.org/195849 | 14:22 |
| haypo | jd__: it's strange. i didn't get any error related to unaware/aware datetime objects yet | 14:23 |
| *** rpodolyaka1 has quit IRC | 14:23 | |
| openstackgerrit | Merged openstack/automaton: Add and use a callback name fetching utility function https://review.openstack.org/196495 | 14:24 |
| *** shardy has quit IRC | 14:24 | |
| *** shardy has joined #openstack-oslo | 14:24 | |
| *** _amrith_ is now known as amrith | 14:27 | |
| *** stevemar has joined #openstack-oslo | 14:28 | |
| *** stevemar has quit IRC | 14:32 | |
| *** zz_jgrimm is now known as jgrimm | 14:32 | |
| *** pblaho has quit IRC | 14:34 | |
| *** pblaho has joined #openstack-oslo | 14:34 | |
| *** yamahata has joined #openstack-oslo | 14:37 | |
| *** rpodolyaka1 has joined #openstack-oslo | 14:40 | |
| *** kgiusti has joined #openstack-oslo | 14:41 | |
| openstackgerrit | Merged openstack-dev/pbr: Updated from global requirements https://review.openstack.org/197249 | 14:41 |
| *** gordc_af1 has joined #openstack-oslo | 14:41 | |
| *** Nakato_ has joined #openstack-oslo | 14:41 | |
| *** nikhil_k1away has joined #openstack-oslo | 14:42 | |
| openstackgerrit | Merged openstack/oslo.i18n: clarify translation policy https://review.openstack.org/197339 | 14:42 |
| *** sputnik13 has quit IRC | 14:43 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo-incubator: prints --insecure if verify=False and debug=True https://review.openstack.org/144176 | 14:44 |
| *** mtreinish_ has joined #openstack-oslo | 14:45 | |
| *** EmilienM_ has joined #openstack-oslo | 14:45 | |
| *** katyafervent_ has joined #openstack-oslo | 14:46 | |
| *** mtreinish has quit IRC | 14:46 | |
| *** rushiagr_away has quit IRC | 14:46 | |
| *** gordc_afk has quit IRC | 14:46 | |
| *** EmilienM has quit IRC | 14:46 | |
| *** rpodolyaka has quit IRC | 14:46 | |
| *** xylan_kong has quit IRC | 14:46 | |
| *** tchaypo has quit IRC | 14:46 | |
| *** kun_huang has quit IRC | 14:46 | |
| *** flaper87 has quit IRC | 14:46 | |
| *** Nakato has quit IRC | 14:46 | |
| *** katyafervent has quit IRC | 14:46 | |
| *** fungi has quit IRC | 14:46 | |
| *** nikhil_k-away has quit IRC | 14:46 | |
| *** ativelkov has quit IRC | 14:46 | |
| *** EmilienM_ is now known as EmilienM | 14:46 | |
| *** mtreinish_ is now known as mtreinish | 14:46 | |
| *** nkrinner has quit IRC | 14:47 | |
| *** rpodolyaka has joined #openstack-oslo | 14:47 | |
| *** ativelkov has joined #openstack-oslo | 14:47 | |
| *** flaper87 has joined #openstack-oslo | 14:49 | |
| *** flaper87 has quit IRC | 14:49 | |
| *** flaper87 has joined #openstack-oslo | 14:49 | |
| *** shardy has quit IRC | 14:49 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo-incubator: Remove quota module https://review.openstack.org/184626 | 14:50 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo-incubator: Crypto.Random.atfork before fetching random bytes https://review.openstack.org/191581 | 14:51 |
| *** yassine__ has quit IRC | 14:51 | |
| *** fungi has joined #openstack-oslo | 14:54 | |
| *** xylan_kong has joined #openstack-oslo | 14:56 | |
| *** shardy has joined #openstack-oslo | 14:56 | |
| *** subscope has joined #openstack-oslo | 14:57 | |
| haypo | dims: for https://review.openstack.org/#/c/191581/ we should use os.urandom() instead (see my comment there) | 14:58 |
| rpodolyaka1 | dims: gave the cinder issue another try - works just fine. must be something wrong with the env | 14:58 |
| dims | haypo: see sean's comment before :) my first patch did have urandom :) | 14:58 |
| dims | rpodolyaka1: thanks will do | 14:58 |
| *** kun_huang has joined #openstack-oslo | 14:59 | |
| *** rpodolyaka1 has quit IRC | 14:59 | |
| dims | haypo: normal use yes, for keys, no was the gist of it | 15:00 |
| openstackgerrit | Merged openstack/oslo-incubator: Add tools to clean up namespace package use https://review.openstack.org/195263 | 15:00 |
| haypo | dims: see http://sockpuppet.org/blog/2014/02/25/safely-generate-random-numbers/ | 15:01 |
| *** rpodolyaka1 has joined #openstack-oslo | 15:01 | |
| dims | haypo: i know :) | 15:01 |
| dims | haypo: you need to convince sdague :) | 15:02 |
| haypo | dims: ah? so why did you replace os.urandom() with user-space CSPRNG with a known fork bug? | 15:02 |
| openstackgerrit | Sergey Nikitin proposed openstack/oslo.cache: Register backends in a cycle https://review.openstack.org/197619 | 15:03 |
| *** manishg has joined #openstack-oslo | 15:03 | |
| dims | haypo: not sure what you mean, code in oslo-incubator has "from Crypto import Random" | 15:03 |
| dims | and the patch is to make it work slightly better under forking | 15:04 |
| *** rpodolyaka1 has quit IRC | 15:04 | |
| *** rpodolyaka3 has joined #openstack-oslo | 15:04 | |
| haypo | dims: your patch may make things worse | 15:04 |
| dims | https://review.openstack.org/#/c/191581/5/openstack/common/crypto/utils.py,cm | 15:04 |
| haypo | dims: it's common that trying to fix a bug related to security introduce a new bug :) | 15:04 |
| haypo | PHP has at least two funny examples | 15:04 |
| dims | haypo: all i am doing is calling re-initialize again | 15:05 |
| dims | "Crypto_Random.atfork()" | 15:05 |
| dims | the diff above does not have any os.urandom in it | 15:06 |
| *** tchaypo has joined #openstack-oslo | 15:06 | |
| *** rpodolyaka3 has quit IRC | 15:07 | |
| *** e0ne_ is now known as e0ne | 15:07 | |
| haypo | dims: i'm not sure that it's correct to "re-initialize" a RNG | 15:07 |
| *** rpodolyaka1 has joined #openstack-oslo | 15:08 | |
| haypo | dims: my collection of bugs in RNG, https://bitbucket.org/haypo/hasard/src/tip/doc/real_world.rst | 15:08 |
| *** rpodolyaka1 has quit IRC | 15:08 | |
| haypo | (Aug 2003) rand() & mt_rand() seed RNG at each call (call srand()/mt_srand()). http://bugs.php.net/bug.php?id=25007 http://svn.php.net/viewvc?view=revision&revision=137294 | 15:08 |
| dims | haypo: sdague is out this week, so next week let's raise it when he is here. FWIW, the current code fails for heat and needs fixing somehow. but let's debate with sean as you and he have a difference of opinion and i am just the middle man | 15:10 |
| haypo | dims: "i am just the middle man" i know :) i just replied to sean's comment | 15:10 |
| dims | thanks! | 15:10 |
| dims | haypo: i know you are passionate so i don't take it personally, but please be gentle :) | 15:12 |
| openstackgerrit | Merged openstack/oslo-incubator: prints --insecure if verify=False and debug=True https://review.openstack.org/144176 | 15:14 |
| haypo | dims: oh, my comment was not kind? | 15:15 |
| *** rpodolyaka1 has joined #openstack-oslo | 15:15 | |
| *** amrith is now known as _amrith_ | 15:18 | |
| *** e0ne is now known as e0ne_ | 15:19 | |
| haypo | dhellmann: you can unblock https://review.openstack.org/#/c/181984/ all dependencies have been merged, great job! | 15:19 |
| haypo | dims: oh in fact, dhellmann already wrote a patch to drop the oslo namespace in oslo.utils. coolness | 15:20 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo.vmware: Add tox target to find missing requirements https://review.openstack.org/195824 | 15:20 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/oslo.vmware: Updated from global requirements https://review.openstack.org/197268 | 15:20 |
| *** e0ne_ is now known as e0ne | 15:20 | |
| *** rjaiswal has joined #openstack-oslo | 15:21 | |
| dims | y should try to get all the namespaces out soon | 15:21 |
| dims | back in a bit getting on a call | 15:21 |
| *** manishg has quit IRC | 15:27 | |
| openstackgerrit | Lianhao Lu proposed openstack-dev/pbr: Use io.open() to open metadata files https://review.openstack.org/197626 | 15:29 |
| *** rpodolyaka1 has quit IRC | 15:30 | |
| *** CaerbannogRabbit is now known as morganfainberg | 15:30 | |
| *** rpodolyaka1 has joined #openstack-oslo | 15:31 | |
| *** rpodolyaka1 has quit IRC | 15:35 | |
| *** tsekiyama has joined #openstack-oslo | 15:36 | |
| *** jaypipes has quit IRC | 15:37 | |
| *** manishg has joined #openstack-oslo | 15:38 | |
| *** pblaho has quit IRC | 15:39 | |
| *** pblaho has joined #openstack-oslo | 15:39 | |
| *** ashleighfarnham has joined #openstack-oslo | 15:42 | |
| *** pblaho has quit IRC | 15:50 | |
| *** pblaho has joined #openstack-oslo | 15:50 | |
| *** ashleighfarnham has quit IRC | 15:51 | |
| openstackgerrit | Merged openstack/futurist: Add periodics to doc features section https://review.openstack.org/195859 | 15:54 |
| *** pblaho has quit IRC | 15:56 | |
| *** pblaho has joined #openstack-oslo | 15:58 | |
| *** stpierre has quit IRC | 16:02 | |
| *** rpodolyaka1 has joined #openstack-oslo | 16:12 | |
| *** harlowja_at_home has joined #openstack-oslo | 16:20 | |
| *** sputnik13 has joined #openstack-oslo | 16:22 | |
| *** manishg has quit IRC | 16:23 | |
| *** manishg has joined #openstack-oslo | 16:24 | |
| *** sdake has joined #openstack-oslo | 16:24 | |
| *** rushiagr_away has joined #openstack-oslo | 16:26 | |
| *** stpierre has joined #openstack-oslo | 16:30 | |
| *** tsbot_ has joined #openstack-oslo | 16:31 | |
| *** vilobhmm has joined #openstack-oslo | 16:35 | |
| *** harlowja_at_home has quit IRC | 16:37 | |
| *** manishg has quit IRC | 16:40 | |
| dims | ativelkov: did you figure out how to fix the revert we did on o.vo? | 16:41 |
| *** manishg has joined #openstack-oslo | 16:41 | |
| *** ashleighfarnham has joined #openstack-oslo | 16:59 | |
| *** browne has quit IRC | 17:02 | |
| *** ashleighfarnham has quit IRC | 17:02 | |
| *** ViswaV has joined #openstack-oslo | 17:03 | |
| *** sigmavirus24 is now known as sigmavirus24_awa | 17:04 | |
| *** yamahata has quit IRC | 17:07 | |
| *** ashleighfarnham has joined #openstack-oslo | 17:08 | |
| *** ViswaV has quit IRC | 17:08 | |
| *** ViswaV has joined #openstack-oslo | 17:09 | |
| *** vilobhmm has quit IRC | 17:10 | |
| *** vilobhmm has joined #openstack-oslo | 17:10 | |
| ativelkov | dims: yep, at least it properly passed the unittests for nova and cinder. Will submit a patch today | 17:14 |
| dims | ativelkov: thanks! | 17:18 |
| *** sdake_ has joined #openstack-oslo | 17:18 | |
| *** e0ne is now known as e0ne_ | 17:19 | |
| openstackgerrit | Merged openstack/oslo.messaging: Remove qpid-config call https://review.openstack.org/194723 | 17:19 |
| *** e0ne_ is now known as e0ne | 17:20 | |
| *** sdake has quit IRC | 17:21 | |
| *** ihrachyshka_ has quit IRC | 17:22 | |
| *** jaypipes has joined #openstack-oslo | 17:22 | |
| *** rpodolyaka1 has quit IRC | 17:22 | |
| *** ozamiatin has quit IRC | 17:23 | |
| *** achanda has joined #openstack-oslo | 17:23 | |
| *** stpierre has quit IRC | 17:24 | |
| *** e0ne is now known as e0ne_ | 17:29 | |
| *** e0ne_ is now known as e0ne | 17:32 | |
| *** rjaiswal has quit IRC | 17:34 | |
| *** e0ne has quit IRC | 17:35 | |
| *** sdake_ is now known as sdake | 17:39 | |
| *** ndipanov has quit IRC | 17:41 | |
| *** sdake has quit IRC | 17:44 | |
| *** browne has joined #openstack-oslo | 17:45 | |
| *** rjaiswal has joined #openstack-oslo | 17:46 | |
| *** rjaiswal has quit IRC | 17:46 | |
| *** rjaiswal has joined #openstack-oslo | 17:46 | |
| *** sdake has joined #openstack-oslo | 17:46 | |
| *** exploreshaifali has joined #openstack-oslo | 17:46 | |
| *** vilobhmm has quit IRC | 17:47 | |
| *** sdake has quit IRC | 17:48 | |
| *** sdake has joined #openstack-oslo | 17:50 | |
| openstackgerrit | Merged openstack/oslo.context: Add tox target to find missing requirements https://review.openstack.org/195848 | 17:51 |
| *** shardy has quit IRC | 17:53 | |
| *** shardy has joined #openstack-oslo | 17:54 | |
| *** browne has quit IRC | 18:01 | |
| *** browne has joined #openstack-oslo | 18:02 | |
| *** Kennan has quit IRC | 18:04 | |
| *** Kennan has joined #openstack-oslo | 18:05 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 18:06 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.messaging: Updated from global requirements https://review.openstack.org/192382 | 18:12 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.vmware: Updated from global requirements https://review.openstack.org/197268 | 18:12 |
| openstackgerrit | Alexander Tivelkov proposed openstack/oslo.versionedobjects: Compound fields now coerce their elements' values https://review.openstack.org/197688 | 18:16 |
| ativelkov | dims here it is: https://review.openstack.org/#/c/197688/ | 18:16 |
| *** e0ne has joined #openstack-oslo | 18:23 | |
| *** e0ne is now known as e0ne_ | 18:23 | |
| *** stpierre has joined #openstack-oslo | 18:23 | |
| *** e0ne_ is now known as e0ne | 18:23 | |
| *** e0ne has quit IRC | 18:23 | |
| dims | ativelkov: i was half expecting a test that does a deepcopy...is that possible? | 18:24 |
| *** e0ne has joined #openstack-oslo | 18:24 | |
| *** kairat_kushaev has quit IRC | 18:29 | |
| *** kairat_kushaev has joined #openstack-oslo | 18:30 | |
| *** sreshetnyak has quit IRC | 18:33 | |
| *** e0ne is now known as e0ne_ | 18:34 | |
| *** sreshetnyak has joined #openstack-oslo | 18:34 | |
| *** jaosorior has quit IRC | 18:36 | |
| *** e0ne_ is now known as e0ne | 18:37 | |
| *** sdake has quit IRC | 18:38 | |
| *** sdake has joined #openstack-oslo | 18:38 | |
| *** rlrossit has joined #openstack-oslo | 18:39 | |
| *** sdake_ has joined #openstack-oslo | 18:41 | |
| *** Rockyg has joined #openstack-oslo | 18:41 | |
| openstackgerrit | Morgan Fainberg proposed openstack/oslo.db: Disable utf-8 sanity check by default https://review.openstack.org/197692 | 18:44 |
| *** sdake has quit IRC | 18:44 | |
| openstackgerrit | Morgan Fainberg proposed openstack/oslo.db: No longer support db schema downgrades https://review.openstack.org/197697 | 18:48 |
| morganfainberg | dhellmann: ^^ as promised yesterday | 18:48 |
| morganfainberg | dhellmann: i expect that second one to cause errors somewhere and might need extra deprecation/warning logic. | 18:48 |
| *** stpierre has left #openstack-oslo | 18:49 | |
| openstackgerrit | Brant Knudson proposed openstack/oslo.cache: Fix up option help https://review.openstack.org/197700 | 18:50 |
| openstackgerrit | Brant Knudson proposed openstack/oslo.cache: Usage docs https://review.openstack.org/197701 | 18:50 |
| openstackgerrit | Brant Knudson proposed openstack/oslo.cache: Remove unused _on_arguments https://review.openstack.org/197702 | 18:50 |
| dhellmann | morganfainberg: ack | 18:50 |
| openstackgerrit | Brant Knudson proposed openstack/oslo.cache: Document the backends https://review.openstack.org/197703 | 18:51 |
| *** sputnik13 has quit IRC | 18:51 | |
| *** sputnik13 has joined #openstack-oslo | 18:51 | |
| *** shardy has quit IRC | 18:55 | |
| *** shardy has joined #openstack-oslo | 18:55 | |
| *** sputnik13 has quit IRC | 18:56 | |
| morganfainberg | bknudson: lots of oslo.cache +2s for you | 18:56 |
| bknudson | morganfainberg: I think oslo.cache is just about ready | 18:57 |
| morganfainberg | bknudson: i would agree with that assessment | 18:58 |
| bknudson | morganfainberg: although it could use someone who's been more involved to check the docs are complete | 18:58 |
| bknudson | and correct | 18:58 |
| morganfainberg | i'd like to get a pymemcache backend in there | 18:58 |
| morganfainberg | as the default "memcache" backend for it instead | 18:58 |
| morganfainberg | but... | 18:58 |
| bknudson | morganfainberg: but what? | 18:58 |
| morganfainberg | or memcached-client | 18:58 |
| openstackgerrit | Merged openstack/oslo.cache: Register backends in a cycle https://review.openstack.org/197619 | 18:58 |
| harlowja | pymemcache please :-P | 18:59 |
| morganfainberg | there are big gaps in pymemcache | 18:59 |
| morganfainberg | harlowja: pymemcache is missing massive functionality | 18:59 |
| bknudson | store and get? | 18:59 |
| morganfainberg | harlowja: so what happens is you lose the ability to handle multiple memcache servers at all | 18:59 |
| harlowja | ya, u want hashing support | 18:59 |
| morganfainberg | harlowja: which python-memcache supports. | 18:59 |
| *** _amrith_ is now known as amrith | 18:59 | |
| morganfainberg | it's one of those "really? you didn't even try did you" | 18:59 |
| morganfainberg | which makes it a hard sell for compat at all with python-memcache | 19:00 |
| morganfainberg | harlowja: i like it way way better, but it's just... well it's a lot of code to add hashing on top of it. | 19:00 |
| morganfainberg | harlowja: it made me sad :( | 19:00 |
| harlowja | https://github.com/pinterest/pymemcache/pull/59 ? | 19:00 |
| morganfainberg | also the fallback code to another server bit looked suspect | 19:00 |
| * harlowja would just rather not continue to have 2 memcache clients being used | 19:01 | |
| *** viktors|wfh has quit IRC | 19:01 | |
| morganfainberg | harlowja: we are likely stuck with 2 of them for aloooooong time | 19:01 |
| harlowja | only as long as we want to be | 19:01 |
| morganfainberg | harlowja: because lots of things we use support it | 19:01 |
| morganfainberg | and default to it | 19:01 |
| morganfainberg | so we can override, but doesn't mean the baseline isn't the bad choice | 19:01 |
| morganfainberg | (also pymemcache rocks for the no-thread.local explicitness) | 19:02 |
| harlowja | ya i guess its a shitty situation all around then (2 clients nobody can decide on), lol | 19:03 |
| *** sputnik13 has joined #openstack-oslo | 19:03 | |
| harlowja | so u have both used depending on how people feel :-P | 19:04 |
| morganfainberg | harlowja: yeah :( | 19:04 |
| harlowja | but the sharding and hashing support for pymecache appear to be getting better | 19:04 |
| harlowja | https://github.com/pinterest/pymemcache/pull/59 (another one) | 19:04 |
| harlowja | oops, meant https://github.com/pinterest/pymemcache/pull/44 | 19:04 |
| morganfainberg | yeah i think that'll make pymemcache closer to the clear winner | 19:04 |
| morganfainberg | but until that is released.... | 19:04 |
| morganfainberg | pymemcache is lacking in a large way | 19:05 |
| harlowja | the oslo team might be biasd though :-P | 19:05 |
| harlowja | https://github.com/pinterest/pymemcache#credits ha | 19:05 |
| morganfainberg | oncd the sharding stuff is solid | 19:05 |
| harlowja | hmmm, how'd harlowja and jd__ get in there :-P | 19:05 |
| openstackgerrit | Merged openstack/oslo.cache: Use "group" rather than "section" https://review.openstack.org/197597 | 19:06 |
| morganfainberg | i think i can convince zzzeek that we should make pymemcache the default memcache provider for dogpile | 19:06 |
| morganfainberg | it'll eliminate at least one layer of thread.local ick | 19:06 |
| harlowja | ya | 19:06 |
| zzzeek | morganfainberg: there is no “default” memcache backend... | 19:06 |
| morganfainberg | zzzeek: there is. the one you call "memcache" that uses python-memcache | 19:07 |
| morganfainberg | that is the "default" when you read it... semanitcally | 19:07 |
| morganfainberg | not so much in implementation | 19:07 |
| zzzeek | morganfainberg: so why wouldnt one for “pymemcache” be called….”pymemcache” ? | 19:07 |
| morganfainberg | zzzeek: so we can drop python-memcache on the floor where it belongs :P | 19:07 |
| morganfainberg | seriously. | 19:07 |
| zzzeek | morganfainberg: just dont use it. changing defaults around == surprises for ppl who i then have to assist | 19:08 |
| morganfainberg | s/seriiously/in all seriousness... | 19:08 |
| morganfainberg | i would say we should rename the "memcache" one to python-memcache [and deprecate the old one] then | 19:08 |
| *** crc32 has joined #openstack-oslo | 19:08 | |
| morganfainberg | and then add pymemcache | 19:08 |
| harlowja | what u guys storing in memcache anyway :-P | 19:09 |
| harlowja | token stuff? | 19:09 |
| morganfainberg | pylibmc and uhmm the otherone i can't ever remember, are clearly named. | 19:09 |
| morganfainberg | but the default for python for a long time was python-memcache | 19:09 |
| jd__ | harlowja: we're everywhere | 19:09 |
| morganfainberg | which is my point of why that one reads as the default for dogpile | 19:09 |
| harlowja | jd__ lol | 19:09 |
| morganfainberg | it is named the same as the technology [is all] | 19:09 |
| morganfainberg | where as the package is sortof-kindof-not | 19:10 |
| morganfainberg | harlowja: we are memoizing things in keystone as well | 19:10 |
| *** crc32 has quit IRC | 19:10 | |
| morganfainberg | harlowja: some of the more expensive lookups | 19:10 |
| *** crc32 has joined #openstack-oslo | 19:10 | |
| zzzeek | morganfainberg: how about. once a google search for “python memcached” turns up pymemcache and not python-memcached, then we know python-memcached is still not what people expect | 19:10 |
| harlowja | k, u ok with memcached flapping | 19:10 |
| morganfainberg | harlowja: that is the more important thing that dogpile is buying us | 19:10 |
| zzzeek | morganfainberg: “memcached”, which is the name, has for years meant “python-memcached” and i dont see how it helps anyone to change it | 19:11 |
| morganfainberg | zzzeek: if you don't mind being not py3 compatible as well. | 19:11 |
| * harlowja always thinks of memcached flapping experiences with memcached | 19:11 | |
| zzzeek | morganfainberg: you can use dogpile + memcache server + py3k, just not that driver | 19:11 |
| zzzeek | morganfainberg: the name is not very important | 19:11 |
| zzzeek | morganfainberg: its the driver name | 19:11 |
| harlowja | server X gets network disconnect, clients switch to new hashed server Y, start saving things there, server X comes back up, old-data found (?) | 19:12 |
| morganfainberg | zzzeek: annnnnnnnnnnnnnnnnnnnnyway not worth the conversation at this point because pymemcache isn't usable in the same context today as python-memcache | 19:12 |
| morganfainberg | zzzeek: i honestly hope python-memcache shutters the project. | 19:12 |
| zzzeek | morganfainberg: my experience is, none of these dead projects *ever* shutter their project, nor do they ever respond to emails about it | 19:13 |
| morganfainberg | zzzeek: and lets the other better libs take over. | 19:13 |
| harlowja | * server X gets network disconnect, clients switch to new hashed server Y, start saving things there, server X comes back up, clients eventually switch back to using X, old-data found (?), all stuff on Y in who knows where land... | 19:13 |
| harlowja | good ole caches, fun fun, ha | 19:13 |
| morganfainberg | zzzeek: shutter = dead/unmaintained in this context | 19:13 |
| zzzeek | morganfainberg: we ahve: python-memcached, PIL, MySQL-Python, etc, all dead crap whose maintainers just refuse to answer any queries | 19:13 |
| morganfainberg | zzzeek: m2crypto | 19:13 |
| morganfainberg | don't forget that one | 19:13 |
| zzzeek | morganfainberg: i actually had to use that a few yuears ago it supports some things that nobody else does | 19:13 |
| *** boris-42 has joined #openstack-oslo | 19:13 | |
| morganfainberg | it does | 19:13 |
| morganfainberg | sadly it's way way unmaintained | 19:14 |
| morganfainberg | in fact you can't even built it on modern ubuntu | 19:14 |
| morganfainberg | or debians | 19:14 |
| zzzeek | morganfainberg: and the author is dead silent to any qiuestions, right? | 19:14 |
| zzzeek | morganfainberg: thats always the main part | 19:14 |
| morganfainberg | because openssl doesn't support the symbols | 19:14 |
| zzzeek | morganfainberg: that they dont respond, “yeah, sorry i dont have time to update it" | 19:14 |
| zzzeek | morganfainberg: they dont put up an announcement | 19:14 |
| zzzeek | morganfainberg: nothing | 19:14 |
| morganfainberg | zzzeek: at some point we [as maintainers] do need to be opinionated and drop support for dead projects | 19:14 |
| morganfainberg | but i'm more aggressive in that view than some OSS folks | 19:15 |
| zzzeek | morganfainberg: absoutely but we need to *SAY IT* publicly | 19:15 |
| zzzeek | morganfainberg: “I am no longer maintaining XYZ” | 19:15 |
| zzzeek | morganfainberg: the reason they dont is, they dont want anyone taking it over | 19:15 |
| openstackgerrit | Brant Knudson proposed openstack/oslo.cache: Usage docs https://review.openstack.org/197701 | 19:15 |
| morganfainberg | zzzeek: i meant we need ot say that about dependant libs too | 19:15 |
| harlowja | lifeless tchaypo u have a sec, wondering why https://review.openstack.org/#/c/197691/ failed, its almost like it didn't get selectively installed :-/ | 19:15 |
| morganfainberg | zzzeek: so maybe i can convince you once pymemcache is solid to deprecate and say python-memecche is a poor choice and will be removed as an option in the future? | 19:16 |
| morganfainberg | zzzeek: ;) | 19:16 |
| harlowja | lifeless tchaypo nm, seems differnt | 19:16 |
| morganfainberg | zzzeek: when we are confident in pymemcache's support of needed features :) | 19:16 |
| zzzeek | morganfainberg: i dont see why we arent adding a pymemcache backend right now at least | 19:16 |
| zzzeek | morganfainberg: dogpile ahs no “default” driver, theres’ no issue here. we also have support for some dead thing called “bmemcache” too | 19:17 |
| morganfainberg | zzzeek: because it's missing some base features that those linked pull requests would prevent a ton of extra code | 19:17 |
| zzzeek | morganfainberg: ill gladly put a doc note | 19:17 |
| morganfainberg | zzzeek: simply you get exactly 1 server ever | 19:17 |
| morganfainberg | no hashing, etc | 19:17 |
| morganfainberg | so i think i'll wait until those land (looks close) and then a lot less "but we don't support X, or Y with pymemcache" and/or a lot less code | 19:18 |
| morganfainberg | zzzeek: i also still owe you a respin on a pull request for locks in memcache | 19:18 |
| harlowja | locks in memcache :-/ | 19:18 |
| * morganfainberg is way behind on non-openstack things :( | 19:18 | |
| zzzeek | morganfainberg: i look at dogpile like, never :) | 19:18 |
| morganfainberg | zzzeek: i know. ;) thankfully it really does mostly "just work" | 19:19 |
| morganfainberg | zzzeek: which is both a good and bad thing | 19:19 |
| zzzeek | morganfainberg: yeah i tried to write it that way :) | 19:19 |
| morganfainberg | zzzeek: i have some general enhancements for UX i'd like to propose, but it'll take a large context swithc from keystone | 19:19 |
| morganfainberg | and i don't have the energy for that at the moment | 19:19 |
| zzzeek | morganfainberg: basically, mako + dogpile are the replacements for all the garbage in beaker (the pylons project) | 19:19 |
| morganfainberg | yeah | 19:20 |
| morganfainberg | if we could un-tie the memoization decorators [in a clean way] from the region object, the developer ux would improve a lot | 19:20 |
| morganfainberg | but i want something that is equally clean to read | 19:20 |
| morganfainberg | so i figure it's some real sit down and examine the options before making a case to change it | 19:21 |
| morganfainberg | which is why i havne't tossed something at the wall yet :) | 19:21 |
| *** sdake has joined #openstack-oslo | 19:21 | |
| morganfainberg | also jd__ stop being everywhere i see your name *everywhere* :P | 19:22 |
| morganfainberg | ;) | 19:22 |
| jd__ | :D | 19:22 |
| *** sdake has quit IRC | 19:23 | |
| *** sdake has joined #openstack-oslo | 19:23 | |
| *** rpodolyaka1 has joined #openstack-oslo | 19:25 | |
| *** sdake_ has quit IRC | 19:25 | |
| harlowja | lol | 19:27 |
| *** dansmith has quit IRC | 19:30 | |
| *** dansmith has joined #openstack-oslo | 19:30 | |
| openstackgerrit | Merged openstack/oslo.vmware: Fix exception to string in py34 https://review.openstack.org/196408 | 19:33 |
| harlowja | jd__ and i are 007 and 008 | 19:33 |
| harlowja | lol | 19:33 |
| jd__ | licensed to kill? | 19:34 |
| harlowja | apache license to kill | 19:35 |
| harlowja | lol | 19:35 |
| harlowja | *apache licensed to kill | 19:35 |
| *** Rockyg has quit IRC | 19:35 | |
| *** e0ne has quit IRC | 19:38 | |
| *** ozamiatin has joined #openstack-oslo | 19:38 | |
| *** nkrinner has joined #openstack-oslo | 19:40 | |
| harlowja | dims https://review.openstack.org/#/c/197668/ (added my 2 cents there, about not creating another state-machine-mini-lib) | 19:41 |
| harlowja | if u want to add anything (or others to) feel free :-P | 19:41 |
| * harlowja very much likes the idea, but not having to fix more code to use a common-lib later... | 19:45 | |
| harlowja | *very much likes the idea, but does not like having to fix more code to use a common-lib later... | 19:45 |
| ativelkov | dims: there is such a test at def test_compound_clone(self) - it does obj.obj_clone()) which is a deepcopy | 19:50 |
| *** Rockyg has joined #openstack-oslo | 19:52 | |
| *** yamahata has joined #openstack-oslo | 19:55 | |
| openstackgerrit | Yuriy Taraday proposed openstack/oslo.rootwrap: Remove test-requirements-py3.txt https://review.openstack.org/197718 | 20:00 |
| *** shardy_ has joined #openstack-oslo | 20:01 | |
| *** achanda has quit IRC | 20:03 | |
| openstackgerrit | Yuriy Taraday proposed openstack/oslo.rootwrap: Updated from global requirements https://review.openstack.org/197266 | 20:07 |
| *** achanda has joined #openstack-oslo | 20:07 | |
| *** achanda has quit IRC | 20:09 | |
| openstackgerrit | Joshua Harlow proposed openstack/automaton: Allow providing and using a 'nested_start_state_fetcher' https://review.openstack.org/197351 | 20:11 |
| dims | LOL apache licensed to kill | 20:18 |
| *** e0ne has joined #openstack-oslo | 20:18 | |
| dims | ativelkov: thanks | 20:18 |
| openstackgerrit | Merged openstack/oslo.policy: Move fileutils functions to oslo.policy https://review.openstack.org/197420 | 20:18 |
| dims | lifeless: https://review.openstack.org/#/c/192382/ wont merge requirements job fails, but nothing in the log. you may have a fix for this in progress already? | 20:19 |
| *** cburgess_ has quit IRC | 20:23 | |
| *** cburgess has joined #openstack-oslo | 20:23 | |
| *** rpodolyaka1 has quit IRC | 20:26 | |
| *** jgrimm is now known as zz_jgrimm | 20:29 | |
| *** openstackgerrit has quit IRC | 20:37 | |
| *** openstackgerrit has joined #openstack-oslo | 20:37 | |
| *** nkrinner has quit IRC | 20:44 | |
| lifeless | dims: thats running on an old image. fungi was prepping new ones | 20:47 |
| *** ihrachyshka has joined #openstack-oslo | 20:47 | |
| fungi | yep, the last missing image update is just wrapping up now | 20:48 |
| fungi | waiting for the nova snapshot call to complete | 20:48 |
| *** mriedem has quit IRC | 20:49 | |
| dims | thanks fungi lifeless | 20:50 |
| *** shardy_ has quit IRC | 21:01 | |
| *** otherwiseguy has quit IRC | 21:04 | |
| *** sdake_ has joined #openstack-oslo | 21:07 | |
| *** e0ne has quit IRC | 21:08 | |
| *** sdake has quit IRC | 21:09 | |
| *** exploreshaifali has quit IRC | 21:21 | |
| *** rlrossit has left #openstack-oslo | 21:21 | |
| *** kgiusti has quit IRC | 21:29 | |
| *** bknudson has quit IRC | 21:51 | |
| haypo | lifeless: hi. do you know the old issue of .testrepository/times.dbm database not readable by Python 3 if created by Python 2? | 22:00 |
| haypo | lifeless: is there a plan to fix it in testrepository? | 22:00 |
| haypo | lifeless: hemna (walter) hitted this issue. but also the opposite issue (python 2 unable to read DB created by python 3!) because of a missing dependency on his setup: https://review.openstack.org/#/c/197624/1/tox.ini | 22:01 |
| haypo | lifeless: because of these two reasons, he prefers to run tests using testtools instead of testr | 22:02 |
| mtreinish | haypo: I've been planning to add a subunit2sql repository type to testrepository which will remove the dependence on using dbm which is causing that issue | 22:08 |
| *** manishg_ has joined #openstack-oslo | 22:08 | |
| mtreinish | the only blocker for that right now is making things work with sqlite (a couple migrations don't run on sqlite) | 22:08 |
| mtreinish | well I guess it's not 100% a blocker because you could use it without sqlite, but then it would require a mysql or postgres db to run testr, which seems a bit heavyweight | 22:11 |
| *** manishg has quit IRC | 22:11 | |
| haypo | mtreinish: which kind of DB will be used *by default*? | 22:11 |
| haypo | i'm not sure that dbm is the best choice to share a DB between python 2 and python 3, since both versions look to use incompatible dbm formats | 22:12 |
| mtreinish | haypo: well I was under the impression when the subunit2sql repository was ready and tested well enough it would become the default | 22:12 |
| mtreinish | so then it would be a local sqlite db | 22:12 |
| haypo | sqlite is maybe a better choice. or something even simpler, than pickle, or plain text :) | 22:12 |
| haypo | i don't know the content of .testrepository/times.dbm | 22:12 |
| mtreinish | haypo: it's key value | 22:13 |
| mtreinish | test_id: run_time | 22:13 |
| haypo | mtreinish: i don't know subunit2sql. what is it? | 22:13 |
| haypo | mtreinish: we need a DB to store a few numbers? | 22:13 |
| mtreinish | haypo: it's a project I've been working on to store data from all the gate runs in a db and perform analysis on it | 22:14 |
| mtreinish | http://git.openstack.org/cgit/openstack-infra/subunit2sql | 22:14 |
| *** pradk has quit IRC | 22:14 | |
| mtreinish | http://docs.openstack.org/developer/subunit2sql/ | 22:14 |
| mtreinish | http://docs.openstack.org/infra/system-config/logstash.html#subunit2sql | 22:14 |
| haypo | hum, is "test_id" the testr run identifier like 1, 2, 3, ... or something like nova.tests.unit.test_api? | 22:15 |
| mtreinish | haypo: well it's only a few numbers right now, but if you use it for a bit more it can be a bit more powerful | 22:15 |
| mtreinish | haypo: the latter | 22:15 |
| mtreinish | it's the identifier for each test | 22:15 |
| haypo | "the latter" oh ok | 22:15 |
| haypo | mtreinish: i don't know to install MySQL or whatever to run a dummy unit test | 22:15 |
| haypo | oops | 22:15 |
| haypo | mtreinish: i don't want to install MySQL or whatever to run a dummy unit test | 22:16 |
| mtreinish | haypo: it would be sqlite | 22:16 |
| haypo | the default should be something more convenient like dbm or sqlite | 22:16 |
| mtreinish | mysql would be an optional config for testr | 22:16 |
| haypo | mtreinish: ah, sqlite can be used with subunit2sql? ok | 22:16 |
| *** sdake_ is now known as sdake | 22:17 | |
| mtreinish | haypo: well that's the plan, right now sqlite support is the blocker for using it in testrepository | 22:17 |
| mtreinish | the last time I tried to run with sqlite a couple of the alembic migrations didn't work on a sqlite db | 22:17 |
| mtreinish | and I haven't had time to revisit it again (that was quite some time ago) | 22:17 |
| *** dims_ has joined #openstack-oslo | 22:17 | |
| haypo | for my usage, testr is too complex. i learnt how to use testtools.run directly, it saved me hours of work :) | 22:18 |
| haypo | testr startup is very long and the default output is not very useful :-( | 22:18 |
| haypo | many projects started to write a layer on top of testr to have a better output | 22:18 |
| haypo | well, i have many complains but no motivation to enhance it :) | 22:19 |
| mtreinish | haypo: do you mean: http://git.openstack.org/cgit/openstack/os-testr ? | 22:19 |
| mtreinish | or the pretty_tox stuff? | 22:19 |
| mtreinish | I kinda did both of those... :) | 22:19 |
| haypo | mtreinish: these two things, yes | 22:19 |
| haypo | mtreinish: why not contributing to testr directly? | 22:20 |
| haypo | when i started to develop on testr, i was shocked that tox doesn't say anything by default and there is no "verbose mode" :-/ | 22:20 |
| *** dims has quit IRC | 22:21 | |
| haypo | later i found: testr --parallel --subunit|subunit2pyunit | 22:21 |
| mtreinish | haypo: that's the long term goal. I did os-testr as a quick hack to get away from having divergent local copy and pasted bash scripts in ever projects repo with a deceptive name | 22:21 |
| haypo | it's a little bit better, but it still have a serious issue: it only shows when a test completed, not when a test started | 22:21 |
| mtreinish | I fully plan to contribute some of the ui elements back into testr | 22:21 |
| haypo | so we have no information when a test hangs | 22:21 |
| haypo | which is painful since testr is distributed (--parallel) :6/ | 22:21 |
| haypo | :-/ | 22:21 |
| mtreinish | the output filter, subunit-trace, was started in tempest to solve that output problem | 22:21 |
| mtreinish | we probably could package it back into python-subunit pretty easily | 22:22 |
| mtreinish | although it does some things which are probably fairly specific to what openstack wants | 22:22 |
| haypo | mtreinish: it's very cool to hear that some people are motivated than me and actively fix my issues. thanks! | 22:22 |
| mtreinish | that's the real reason in the short term these things are a layer on top | 22:22 |
| haypo | mtreinish: i'm not sure that testr is widely used outside OpenStack | 22:23 |
| mtreinish | because they're not really general solutions because subunit and friends are meant to be consumed by other things too | 22:23 |
| lifeless | haypo: there's a bug open on it | 22:23 |
| mtreinish | haypo: that's what I've always thought, but lifeless has told me otherwise | 22:23 |
| lifeless | haypo: there isn't a compatible dbm impleemntation across all current supported pythons on all reasonable platforms | 22:23 |
| haypo | (well, i have no idea. but testr looks very complex compared to all other test runners like py.test, nose, unittest, etc.) | 22:23 |
| lifeless | haypo: its terrible. | 22:23 |
| lifeless | haypo: (the dbm thing) | 22:24 |
| lifeless | haypo: I wrote testr long before openstack | 22:24 |
| haypo | lifeless: mtreinish was telling me that testr will move to sqlite. good. sqlite is probably the exception of things that work well to exchange data between python 2 and python 3 :) | 22:24 |
| lifeless | haypo: I don't know how widely its used elsewhere, I don't particularly care :) | 22:24 |
| mtreinish | haypo: well long term I want to move that way | 22:25 |
| mtreinish | but it'll take time as I'm still basically the only contributor the subunit2sql stuff | 22:25 |
| haypo | i started to copy/paste runtests.py of asyncio. it's ~300 lines of code and no dependency: https://github.com/python/asyncio/blob/master/runtests.py | 22:26 |
| haypo | it's enough for my small projects. usually, running all tests takes less than 5 minutes, more likely 60 seconds | 22:26 |
| haypo | so parallel execution doesn't matter | 22:26 |
| haypo | since tests are written with unittest, any other test runner can be used ;) | 22:27 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.rootwrap: Updated from global requirements https://review.openstack.org/197266 | 22:27 |
| haypo | i like the raw unittest module, no canddiers like testtools or testscenarios which require a specific test runner and are harder to replay | 22:27 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/oslo.vmware: Updated from global requirements https://review.openstack.org/197268 | 22:27 |
| lifeless | haypo: so just use 'unit2' ? | 22:28 |
| lifeless | haypo: thats why I backport the whole testing stack ... | 22:28 |
| haypo | lifeless: what? stop using python 2, it's dead! :) | 22:29 |
| lifeless | haypo: unittest2 is not about python2 | 22:29 |
| haypo | lifeless: use unittest, the regular unittest module, of python 3 | 22:29 |
| lifeless | haypo: its a backport from 3.6 to everything else. | 22:29 |
| *** sdake is now known as sdake_ | 22:29 | |
| lifeless | haypo: so if you support 3.4, then unittest2 (and traceback2 and linecache2) are useful. | 22:29 |
| lifeless | haypo: unless you're only going to support unreleased pythons. | 22:30 |
| haypo | lifeless: i don't think that i'm using features more advanced than new assert methods of python 2.7 :) | 22:30 |
| lifeless | haypo: properly loader support for instance, added about a year back to the stdlib | 22:30 |
| lifeless | haypo: not present in 3.4,3.3,3.2,2.7 | 22:30 |
| *** amrith is now known as _amrith_ | 22:30 | |
| haypo | lifeless: what is a loader? | 22:30 |
| lifeless | haypo: TestLoader | 22:31 |
| lifeless | haypo: the underglue of discover | 22:31 |
| haypo | lifeless: i don't think that i'm using discover or any loader. as i wrote, i'm using https://github.com/python/asyncio/blob/master/runtests.py which search for test_*.py, load them and search for test cases | 22:33 |
| lifeless | sure | 22:33 |
| haypo | lifeless: i guess that loader+discover avoids this script | 22:33 |
| lifeless | right | 22:33 |
| lifeless | anyhow my point is | 22:34 |
| lifeless | unittest2 is not abou tpython2 | 22:34 |
| lifeless | its just backports | 22:34 |
| haypo | lifeless: cool :) but i need to support 3.3+ | 22:34 |
| lifeless | it supports jython pypy and every supported cpython | 22:34 |
| haypo | lifeless: and i don't want a ton of dependencies | 22:34 |
| lifeless | [+ 2.6 and 3.2 atm] | 22:34 |
| haypo | 3.2 is dying with the new debian stable which switchs from 3.2 to 3.4! | 22:34 |
| lifeless | haypo: unittest2 only depends on traceback2, which is the same thing but the traceback module | 22:34 |
| haypo | lifeless: so it adds two dependencies :) | 22:35 |
| haypo | lifeless: i'm running my tests on windows, freebsd, windows, solaris, openbsd | 22:35 |
| haypo | dependencies are annoying | 22:35 |
| haypo | slowly, i'm moving to tox which handles dependencies for me ;) | 22:36 |
| *** droyal has joined #openstack-oslo | 22:36 | |
| lifeless | haypo: sure via pip | 22:36 |
| lifeless | haypo: and pip installs all these things easily across the board | 22:36 |
| lifeless | haypo: anyhow, no skin off my nose :) | 22:37 |
| haypo | ensurepip helped me a lot | 22:37 |
| haypo | it really was a pain to bootstrap all this shit. setuptools, pip, etc. it was a real nightware. it's better nowadays | 22:38 |
| haypo | lifeless: hum, can you please take a look at http://logs.openstack.org/14/186114/34/check/gate-glance-requirements/539f11b/console.html ? gate-glance-requirements failed in https://review.openstack.org/#/c/186114/ ("Updated from global requirements") | 22:40 |
| haypo | i don't see an obvious error message in this output | 22:41 |
| haypo | maybe "Requirement sphinx present in multiple files" is the error | 22:41 |
| haypo | (there are a ton of such message) | 22:41 |
| mtreinish | haypo: I believe it is, I think that's probably because of the changes eliminating the need for separate *-py3.txt requirements file | 22:42 |
| mtreinish | so it's complaining that each requirement is present in multiple files | 22:42 |
| *** browne has quit IRC | 22:43 | |
| haypo | mtreinish: should i propose my own patch based on this one? | 22:43 |
| haypo | i guess that the bot is too stupid to fix such corner case | 22:44 |
| mtreinish | haypo: yeah I think you have todo it manually | 22:44 |
| mtreinish | I think lifeless sent something to the ML about this | 22:44 |
| * mtreinish checks | 22:44 | |
| haypo | mtreinish: i noticed that lifeless tries to flood my mailing box, right :-p | 22:46 |
| lifeless | haypo: that was an old image, fungi has fixed it now - do a recheck | 22:47 |
| mtreinish | I found, http://lists.openstack.org/pipermail/openstack-dev/2015-June/068193.html so I guess that'll come in the future | 22:48 |
| haypo | lifeless: ok, let's do a recheck | 22:49 |
| haypo | i feel stupid when i schedule a blind recheck | 22:49 |
| haypo | it's common that i don't read the logs, i just recheck before that i know that the CI is unable and that i may be more lucky in the next try... | 22:50 |
| haypo | i already thaught of writing a bot to write a "recheck" comment on my reviews if a CI failed :-p | 22:50 |
| mtreinish | haypo: please don't do that it'll just introduce more bugs in the long run... | 22:51 |
| haypo | mtreinish: introduce bugs? what do you mean? | 22:52 |
| lifeless | it will allow races through | 22:52 |
| lifeless | by persistence | 22:53 |
| mtreinish | haypo: the unrelated issues that often cause failures are just the exposure of a race bug. If you just hammer away at rechecking it'll just allow more races through | 22:53 |
| mtreinish | heh, what lifeless said :) | 22:53 |
| *** sdake_ has quit IRC | 22:54 | |
| haypo | i'm not planning to flood the infra with recheck. maybe only retry once if the CI failed | 22:54 |
| haypo | well, it's just a project, just to share my feedback on the CI :) | 22:54 |
| haypo | the theory is that everybody is responsible of the CI | 22:54 |
| haypo | the practice is that most people hate the CI and don't want to fix it :) | 22:55 |
| haypo | (well, that's my guess in fact, i have no statistics :) | 22:55 |
| mtreinish | haypo: yep that's pretty accurate, which is why we gave this talk at summit: https://www.youtube.com/watch?v=sa67J6yMYZ0 | 22:55 |
| mtreinish | to try and change that | 22:55 |
| tsekiyama | lifeless: gate-glance-requirements is still failing. I hope https://review.openstack.org/#/c/197753/ will fix the issue.. | 22:56 |
| haypo | mtreinish: oh, i missed this talk. i will watch the video | 22:57 |
| *** jecarey has quit IRC | 22:57 | |
| haypo | tsekiyama: ah? i just scheduled a recheck, there is no result yet. https://review.openstack.org/#/c/186114/ | 22:59 |
| haypo | tsekiyama: or do you mean that this check fails on any glance patch? | 22:59 |
| lifeless | tsekiyama: do you have a current glance bug? | 22:59 |
| lifeless | tsekiyama: the one you linked 404 | 22:59 |
| lifeless | s | 22:59 |
| tsekiyama | lifeless: sorry please remove "/" at URL tail | 23:00 |
| lifeless | tsekiyama: so yes, that needs fixing | 23:00 |
| lifeless | tsekiyama: let me prepare a patch for glance | 23:00 |
| tsekiyama | haypo: I mean that review. It is displayed "failed" at http://status.openstack.org/zuul/ | 23:00 |
| tsekiyama | lifeless: thanks! | 23:01 |
| haypo | mtreinish: i'm working on cpython. we have a lot of tests which are unstable | 23:01 |
| haypo | mtreinish: IMHO it would be better to skip the test on most buildbots rather than asking developers to fix the CI | 23:02 |
| haypo | mtreinish: almost everybody ignore buildbots. it's different than in openstack, tests are run after the merge. so after the merge, the author goes away and don't care :) | 23:03 |
| lifeless | haypo: so- just incase | 23:03 |
| lifeless | haypo: please don't add [test-]requirements-py3.txt to any more projects. | 23:03 |
| lifeless | haypo: you probably can't, but even if you can, please don't. markers are fully usable now within the openstack master branches | 23:03 |
| haypo | mtreinish: so i think that it would be more efficient to identify unstable tests, to first get stable buildbots | 23:03 |
| haypo | lifeless: i still think that requiring a specific version of pip, setuptools, pbr, etc. is crazy | 23:04 |
| haypo | lifeless: but you are probably more brave than me :) | 23:04 |
| lifeless | haypo: We want stable buildbots | 23:04 |
| mtreinish | haypo: so have you looked at elastic-recheck? | 23:04 |
| haypo | lifeless: i didn't add any new -py3 file recently | 23:04 |
| mtreinish | haypo: but we do skip tempest tests when a failure rate is too high, but it's not something we want to do | 23:04 |
| lifeless | haypo: our number 1 systematic stability problem is project releases | 23:05 |
| lifeless | haypo: e766b9e4 (Victor Stinner 2015-06-19 14:16:38 +0200 31) | 23:05 |
| mtreinish | because it means something is fundamentaly broken | 23:05 |
| lifeless | haypo: that might not count as recent for you :) | 23:05 |
| haypo | lifeless: ah? in which project? glancE? | 23:05 |
| lifeless | yeah | 23:05 |
| haypo | lifeless: ah. i'm producing too much code, i'm unable to remember what i did yesterday | 23:06 |
| haypo | lifeless: i did that before you started your huge project of using env markers | 23:06 |
| haypo | lifeless: it looks like it was a reasonable choice between i saw a lot of issues related to env markers recently :) | 23:07 |
| lifeless | what is glance doing, sadface. | 23:07 |
| lifeless | -e .... | 23:07 |
| haypo | mtreinish: "so have you looked at elastic-recheck?" for which issue? | 23:07 |
| lifeless | haypo: why is -e git+https://github.com/openstack/glance_store.git#egg=glance_store | 23:08 |
| haypo | mtreinish: i'm reading http://logs.openstack.org/72/197072/1/check/check-tempest-dsvm-full-ceph/948642f/console.html for https://review.openstack.org/#/c/197072/ and it's a timeout. i don't feel able to dig deeper | 23:08 |
| lifeless | haypo: there ? | 23:08 |
| haypo | lifeless: yeah, the -e is funny. i'm suprised that it was accepted | 23:08 |
| haypo | lifeless: it was more an experimental patch to unblock glance port to python 3 | 23:08 |
| lifeless | haypo: infra didn't check requirements-py3.txt at all | 23:08 |
| lifeless | haypo: until recently | 23:08 |
| mtreinish | haypo: you mentioned it would be more efficient to identify unstable tests | 23:08 |
| lifeless | haypo: so its basically you could put anything in there and it wouldn't complain | 23:08 |
| haypo | lifeless: it's supposed to be temporary. you know temporary as "it will stay until 2020" | 23:08 |
| lifeless | haypo: has glance-store done a python3 ready release ? | 23:09 |
| mtreinish | that's what elastic-recheck is there for to classify and track bugs that are popping up in the gate | 23:09 |
| haypo | mtreinish: ah. i was talking about CPython which have a very minimal testing infra compared to the amazing openstack infra | 23:09 |
| haypo | lifeless: glance-store is one of the slowest project to get a project | 23:10 |
| lifeless | haypo: is that no ? | 23:10 |
| haypo | lifeless: the 3 slowest are: swift, glance-store and stackforge/sqlalchemy-migrate | 23:10 |
| haypo | lifeless: i still have 3 patches under review for glance-store | 23:10 |
| lifeless | haypo: ah so glance py3 stuff isn't gating yet? | 23:11 |
| mtreinish | haypo: oh, ok | 23:11 |
| haypo | lifeless: i don't know if they released anything recently. i would prefer to have a full python 3 support before a release | 23:11 |
| lifeless | haypo: so here's the problem | 23:11 |
| mtreinish | haypo: I thought you meant generally :) | 23:11 |
| lifeless | haypo: I can't fix the requirements stuff to merge | 23:11 |
| lifeless | haypo: until we remove this -e line | 23:11 |
| lifeless | haypo: I mean, I'm going to try, but I'm fairly sure the automation is going to reject me | 23:11 |
| haypo | lifeless: i added a non-voting py34 check job to glance, and quickly made it voting | 23:12 |
| haypo | lifeless: i'm using https://wiki.openstack.org/wiki/Python3#OpenStack_applications to check the py34 gate of each application | 23:12 |
| lifeless | haypo: does that depend on the -e line ? | 23:12 |
| mtreinish | haypo: so that failure is actually interesting, it's being caused because that test took >500 sec | 23:12 |
| haypo | lifeless: sure. glance explodes using the releases | 23:12 |
| mtreinish | which is too long so we killed it at 500 sec | 23:12 |
| lifeless | haypo: ok, so we're probbly going to have to unwind that | 23:13 |
| mtreinish | haypo: you can see the timeout exception is being raised by the timeout fixture | 23:13 |
| lifeless | haypo: I'll try using the -e first. | 23:13 |
| mtreinish | haypo: http://logs.openstack.org/72/197072/1/check/check-tempest-dsvm-full-ceph/948642f/console.html#_2015-07-01_01_46_14_873 | 23:13 |
| haypo | mtreinish: 500 sec is probably a bug. probably a race condition. | 23:13 |
| mtreinish | haypo: well that or we got a bad/slow vm in whatever cloud the slave was spun up in | 23:13 |
| mtreinish | the joy of using public clouds | 23:14 |
| lifeless | tsekiyama: https://review.openstack.org/197766 | 23:15 |
| haypo | lifeless: i read again the -py3.txt. so yes, i expect a release of glance-store shortly, which will fix this mess | 23:15 |
| haypo | mtreinish: we have to fix the cloud! | 23:15 |
| haypo | mtreinish: recently i heard that the cloud is vulnerable to hardware issues. i'm surprised. the hardware is not virtualized? hum, wait... | 23:15 |
| haypo | ;) | 23:16 |
| haypo | lifeless: can't you use the git branch on python 3, and the stable release on python 2, for glance_store? | 23:16 |
| haypo | lifeless: using the funny env markers | 23:17 |
| *** sputnik13 has quit IRC | 23:18 | |
| *** sputnik13 has joined #openstack-oslo | 23:18 | |
| *** ihrachyshka has quit IRC | 23:19 | |
| *** sputnik13 has quit IRC | 23:19 | |
| *** sputnik13 has joined #openstack-oslo | 23:19 | |
| *** Nakato_ is now known as Nakato | 23:19 | |
| lifeless | haypo: we could, but thats not the issue. the issue is whether -e will be accepted at all. | 23:20 |
| mtreinish | haypo: well looking at the dstat for the run it doesn't look like it's spinning on io or anything: http://logs.openstack.org/72/197072/1/check/check-tempest-dsvm-full-ceph/948642f/logs/screen-dstat.txt.gz | 23:20 |
| mtreinish | which is often an indicator of something wrong with the node | 23:20 |
| haypo | lifeless: ah | 23:21 |
| haypo | mtreinish: i never opened the dstat file | 23:21 |
| haypo | mtreinish: good to know | 23:22 |
| *** sputnik13 has quit IRC | 23:29 | |
| *** sputnik13 has joined #openstack-oslo | 23:29 | |
| *** stevemar has joined #openstack-oslo | 23:32 | |
| *** stevemar has quit IRC | 23:35 | |
| *** manishg_ has quit IRC | 23:39 | |
| *** browne has joined #openstack-oslo | 23:39 | |
| *** manishg has joined #openstack-oslo | 23:51 | |
| *** manishg has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!