*** mriedem has joined #openstack-oslo | 00:02 | |
*** praneshp has quit IRC | 00:24 | |
*** yamahata has joined #openstack-oslo | 00:41 | |
*** shakamunyi has quit IRC | 01:16 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Add a tree type https://review.openstack.org/96325 | 01:29 |
---|---|---|
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Add a tree type https://review.openstack.org/96325 | 01:39 |
*** yamahata has quit IRC | 01:43 | |
*** yamahata has joined #openstack-oslo | 01:43 | |
*** arnaud has quit IRC | 02:16 | |
*** arnaud has joined #openstack-oslo | 02:16 | |
*** arnaud has quit IRC | 02:28 | |
*** dims has quit IRC | 02:29 | |
*** zhiyan_ is now known as zhiyan | 02:34 | |
*** mriedem has left #openstack-oslo | 03:21 | |
*** mriedem has quit IRC | 03:21 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Allow indent text to be passed in https://review.openstack.org/96343 | 03:32 |
*** wpf has joined #openstack-oslo | 03:38 | |
*** zhiyan is now known as zhiyan_ | 03:47 | |
*** shakamunyi has joined #openstack-oslo | 03:48 | |
*** shakamunyi has quit IRC | 04:05 | |
*** rpodolyaka1 has joined #openstack-oslo | 04:06 | |
*** zhiyan_ is now known as zhiyan | 04:10 | |
*** harlowja_ is now known as harlowja_away | 04:38 | |
*** zhiyan is now known as zhiyan_ | 04:52 | |
*** arnaud has joined #openstack-oslo | 04:53 | |
*** xiej has joined #openstack-oslo | 04:56 | |
*** xiej has quit IRC | 04:58 | |
*** xiej has joined #openstack-oslo | 05:00 | |
*** xiej has quit IRC | 05:02 | |
*** shakamunyi has joined #openstack-oslo | 05:03 | |
*** czzl has joined #openstack-oslo | 05:06 | |
*** xiej has joined #openstack-oslo | 05:11 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Allow indent text to be passed in https://review.openstack.org/96343 | 05:12 |
*** shakamunyi has quit IRC | 05:14 | |
*** czzl has quit IRC | 05:17 | |
*** czzl has joined #openstack-oslo | 05:17 | |
*** zhiyan_ is now known as zhiyan | 05:29 | |
*** rpodolyaka1 has quit IRC | 05:29 | |
*** czzl has left #openstack-oslo | 05:30 | |
*** shakamunyi has joined #openstack-oslo | 05:40 | |
*** wpf has quit IRC | 05:48 | |
*** wpf has joined #openstack-oslo | 05:49 | |
*** shakamunyi has quit IRC | 05:50 | |
*** rpodolyaka1 has joined #openstack-oslo | 05:56 | |
*** rpodolyaka1 has quit IRC | 06:02 | |
*** arnaud has quit IRC | 06:07 | |
*** morganfainberg is now known as stevemar2 | 06:09 | |
*** stevemar2 is now known as morganfainberg | 06:10 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Add a runtime module https://review.openstack.org/94461 | 06:16 |
*** praneshp has joined #openstack-oslo | 06:21 | |
*** HenryG has quit IRC | 06:27 | |
*** praneshp_ has joined #openstack-oslo | 06:29 | |
*** praneshp has quit IRC | 06:32 | |
*** praneshp_ is now known as praneshp | 06:32 | |
*** HenryG has joined #openstack-oslo | 06:33 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Fix docstring list format https://review.openstack.org/96368 | 06:36 |
*** shakamunyi has joined #openstack-oslo | 06:47 | |
*** wpf has quit IRC | 06:52 | |
*** shakamunyi has quit IRC | 06:59 | |
*** flaper87|afk is now known as flaper87 | 07:06 | |
rpodolyaka | morning all | 07:18 |
rpodolyaka | any oslo cores around? :) | 07:18 |
*** shakamunyi has joined #openstack-oslo | 07:25 | |
*** bogdando has quit IRC | 07:26 | |
*** bogdando has joined #openstack-oslo | 07:26 | |
*** shakamunyi has quit IRC | 07:36 | |
*** mrda is now known as mrda-away | 07:39 | |
*** morganfainberg is now known as morganfainberg_Z | 07:54 | |
*** shakamunyi has joined #openstack-oslo | 08:03 | |
*** ihrachyshka has joined #openstack-oslo | 08:04 | |
*** morganfainberg_Z is now known as morganfainberg | 08:12 | |
openstackgerrit | yangxurong proposed a change to openstack/oslo-incubator: Python 3: replace "im_self" by "__self__" https://review.openstack.org/76725 | 08:12 |
*** shakamunyi has quit IRC | 08:14 | |
*** shakamunyi has joined #openstack-oslo | 08:40 | |
*** shakamunyi has quit IRC | 08:52 | |
*** ihrachyshka has quit IRC | 09:01 | |
*** AAzza has joined #openstack-oslo | 09:18 | |
*** shakamunyi has joined #openstack-oslo | 09:19 | |
*** zhiyan is now known as zhiyan_ | 09:30 | |
*** shakamunyi has quit IRC | 09:31 | |
*** praneshp has quit IRC | 09:42 | |
viktors|afk | jd__: Hi! Can you please add your +2 to oslo.db patch https://review.openstack.org/#/c/95702/ (Add import_exceptions to tox.ini) ? | 09:55 |
*** viktors|afk is now known as viktors | 09:55 | |
*** shakamunyi has joined #openstack-oslo | 09:58 | |
*** shakamunyi has quit IRC | 10:05 | |
*** shakamunyi has joined #openstack-oslo | 10:05 | |
*** shakamunyi has quit IRC | 10:10 | |
*** shakamunyi has joined #openstack-oslo | 10:10 | |
*** shakamunyi has quit IRC | 10:20 | |
*** markmc has joined #openstack-oslo | 10:29 | |
*** HenryG has quit IRC | 10:41 | |
*** shakamunyi has joined #openstack-oslo | 10:47 | |
*** HenryG has joined #openstack-oslo | 10:56 | |
*** shakamunyi has quit IRC | 11:00 | |
*** ihrachyshka has joined #openstack-oslo | 11:13 | |
*** dims has joined #openstack-oslo | 11:16 | |
*** shakamunyi has joined #openstack-oslo | 11:27 | |
*** viktors has quit IRC | 11:34 | |
openstackgerrit | Ihar Hrachyshka proposed a change to openstack/oslo-incubator: file_open: fixed docstring to refer to open() instead of file() https://review.openstack.org/96424 | 11:35 |
*** viktors has joined #openstack-oslo | 11:36 | |
viktors | jd__: thanks! :) | 11:37 |
*** shakamunyi has quit IRC | 11:41 | |
openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: Opportunistic migration tests https://review.openstack.org/93424 | 11:48 |
*** lbragstad has quit IRC | 11:52 | |
openstackgerrit | Nataliia Uvarova proposed a change to openstack/oslo-incubator: Add kwargs to jsonutils.load(s) functions https://review.openstack.org/96431 | 12:00 |
*** dims has quit IRC | 12:00 | |
*** shakamunyi has joined #openstack-oslo | 12:07 | |
*** shakamunyi has quit IRC | 12:20 | |
ihrachyshka | dhellmann_: hi. now that jsonutils work to enable simplejson is done in master, can we proceed with backporting it to stable branches? | 12:25 |
*** yamahata has quit IRC | 12:51 | |
*** jecarey has quit IRC | 12:53 | |
*** lbragstad has joined #openstack-oslo | 13:02 | |
*** mriedem has joined #openstack-oslo | 13:10 | |
*** dims has joined #openstack-oslo | 13:10 | |
*** zhiyan_ is now known as zhiyan | 13:11 | |
*** shakamunyi has joined #openstack-oslo | 13:16 | |
*** shakayumi has joined #openstack-oslo | 13:18 | |
*** shakamunyi has quit IRC | 13:21 | |
*** HenryG has quit IRC | 13:30 | |
*** bknudson has joined #openstack-oslo | 13:30 | |
*** HenryG has joined #openstack-oslo | 13:31 | |
openstackgerrit | Andrey Kurilin proposed a change to openstack/oslo.db: Add eventlet.tpool.Proxy for DB API calls https://review.openstack.org/96467 | 13:32 |
*** shakayumi has quit IRC | 13:34 | |
openstackgerrit | Andrey Kurilin proposed a change to openstack/oslo.db: Add eventlet.tpool.Proxy for DB API calls https://review.openstack.org/96467 | 13:37 |
openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: Log a cause of db backend connection failure https://review.openstack.org/93425 | 13:40 |
*** yamahata has joined #openstack-oslo | 13:44 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo-incubator: Updated from global requirements https://review.openstack.org/96236 | 13:46 |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/oslo.db: Updated from global requirements https://review.openstack.org/92947 | 13:46 |
*** shakayumi has joined #openstack-oslo | 13:48 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/taskflow: Updated from global requirements https://review.openstack.org/93254 | 13:48 |
openstackgerrit | Roman Podoliaka proposed a change to openstack/oslo-specs: Add add-tpool-proxy-wrapper spec https://review.openstack.org/96468 | 13:50 |
*** ihrachyshka_ has joined #openstack-oslo | 14:01 | |
*** ihrachyshka has quit IRC | 14:04 | |
*** yfujioka has joined #openstack-oslo | 14:19 | |
*** ihrachyshka_ is now known as ihrachyshka | 14:19 | |
openstackgerrit | Roman Podoliaka proposed a change to openstack/oslo-specs: Add add-tpool-proxy-wrapper spec https://review.openstack.org/96468 | 14:24 |
mriedem | dims: looking at this https://review.openstack.org/#/c/59319/1/openstack/common/periodic_task.py | 14:27 |
mriedem | it's old | 14:27 |
mriedem | but isn't with_parents wrong there? | 14:27 |
mriedem | i think it meant parens as in parenthesis | 14:27 |
openstackgerrit | Nataliia Uvarova proposed a change to openstack/oslo-incubator: Add kwargs to jsonutils.load(s) functions https://review.openstack.org/96431 | 14:28 |
*** dims has quit IRC | 14:33 | |
*** ihrachyshka has quit IRC | 14:37 | |
*** flaper87 is now known as flaper87|afk | 14:38 | |
*** mriedem has quit IRC | 14:39 | |
*** yfujioka has quit IRC | 14:39 | |
*** mriedem has joined #openstack-oslo | 14:41 | |
openstackgerrit | Ivan Melnikov proposed a change to openstack/taskflow: Update sphinx pin from global requirements https://review.openstack.org/96481 | 14:45 |
openstackgerrit | Roman Podoliaka proposed a change to openstack/oslo-specs: Add add-tpool-proxy-wrapper spec https://review.openstack.org/96468 | 14:47 |
*** zhiyan is now known as zhiyan_ | 14:52 | |
*** jecarey has joined #openstack-oslo | 14:52 | |
*** praneshp has joined #openstack-oslo | 14:53 | |
*** praneshp_ has joined #openstack-oslo | 15:00 | |
*** praneshp has quit IRC | 15:03 | |
*** praneshp_ is now known as praneshp | 15:03 | |
openstackgerrit | Roman Podoliaka proposed a change to openstack/oslo-specs: Add add-tpool-proxy-wrapper spec https://review.openstack.org/96468 | 15:13 |
openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo.db: Fix the test using in-file SQLite database https://review.openstack.org/93423 | 15:21 |
openstackgerrit | Christian Berendt proposed a change to openstack/cliff: fixed typos found by RETF rules https://review.openstack.org/96501 | 15:41 |
*** zhiyan_ is now known as zhiyan | 15:47 | |
*** zhiyan is now known as zhiyan_ | 15:57 | |
*** ihrachyshka has joined #openstack-oslo | 15:57 | |
*** shakayumi is now known as shakamunyi | 15:59 | |
*** zhiyan_ is now known as zhiyan | 16:12 | |
*** zhiyan is now known as zhiyan_ | 16:21 | |
*** praneshp has quit IRC | 16:25 | |
*** amotoki has quit IRC | 16:48 | |
bnemec | mriedem: I'm inclined to agree re: the parens/parents thing. | 17:02 |
*** harlowja_away is now known as harlowja_ | 17:02 | |
*** arnaud has joined #openstack-oslo | 17:05 | |
mriedem | bnemec: ok, if i can remember i'll push a patch to fix at some point | 17:06 |
mriedem | lunch time | 17:06 |
openstackgerrit | Yuriy Taraday proposed a change to openstack/oslo.rootwrap: Add an option to run rootwrap as a daemon https://review.openstack.org/81798 | 17:10 |
*** zhiyan_ is now known as zhiyan | 17:12 | |
openstackgerrit | Ivan Melnikov proposed a change to openstack/taskflow: docs: Use linkcode instead of viewcode extension https://review.openstack.org/96526 | 17:19 |
*** markmc has quit IRC | 17:21 | |
*** zhiyan is now known as zhiyan_ | 17:22 | |
*** AAzza has quit IRC | 17:32 | |
*** AAzza has joined #openstack-oslo | 17:33 | |
*** arnaud has quit IRC | 17:35 | |
*** ihrachyshka has quit IRC | 17:40 | |
*** rpodolyaka1 has joined #openstack-oslo | 17:43 | |
*** praneshp has joined #openstack-oslo | 17:44 | |
*** rpodolyaka1 has quit IRC | 17:55 | |
*** openstackgerrit has quit IRC | 17:57 | |
*** yamahata has quit IRC | 17:59 | |
*** rpodolyaka1 has joined #openstack-oslo | 18:05 | |
*** arnaud has joined #openstack-oslo | 18:13 | |
*** zhiyan_ is now known as zhiyan | 18:13 | |
mriedem | shiiit | 18:15 |
mriedem | bnemec: i need some python heroing | 18:15 |
bnemec | mriedem: Then you're probably looking for Doug. :-) | 18:15 |
mriedem | maybe | 18:16 |
mriedem | bnemec: was syncing this to cinder and neutron https://review.openstack.org/#/c/70484/ | 18:16 |
mriedem | apparently they have some classes that extend PeriodicTask | 18:16 |
mriedem | *PeriodicTasks | 18:17 |
mriedem | but don't call __init__ on the super | 18:17 |
mriedem | fixed one for neutron lbaas, that was easy | 18:17 |
mriedem | but cinder has a class with multiple inheritance and is already calling super on the other parent class's init | 18:17 |
bnemec | mriedem: I think Nova had a similar situation when I first tried syncing that change. Do you have a link to the file? | 18:18 |
bnemec | mriedem: https://review.openstack.org/#/c/71131/1/nova/db/base.py | 18:22 |
*** zhiyan is now known as zhiyan_ | 18:23 | |
bnemec | IIRC, Python needs super calls from all of the classes involved in the hierarchy in order to properly handle multiple inheritance. | 18:23 |
bnemec | So even if one class is doing a super call correctly, it may still not work right if another class isn't. | 18:25 |
bnemec | That's what was happening in Nova anyway. | 18:25 |
mriedem | bnemec: http://git.openstack.org/cgit/openstack/cinder/tree/cinder/manager.py#n76 | 18:28 |
*** mrodden has joined #openstack-oslo | 18:28 | |
*** rpodolyaka1 has quit IRC | 18:28 | |
mriedem | bnemec: ah i see http://git.openstack.org/cgit/openstack/cinder/tree/cinder/db/base.py#n37 | 18:29 |
bnemec | mriedem: Yep, same problem. | 18:29 |
mriedem | bnemec: i'm wondering why they are even extending PeriodicTasks | 18:29 |
*** arnaud has quit IRC | 18:29 | |
mriedem | rather than just using the decorator | 18:29 |
bnemec | mriedem: I think you have to extend PeriodicTasks to use the decorator. There's some funky metaclass stuff going on there. | 18:30 |
mriedem | ah yeah nova's base manager class does the same class Manager(base.Base, periodic_task.PeriodicTasks): | 18:30 |
mriedem | mrodden: https://review.openstack.org/#/c/71131/ | 18:30 |
mriedem | fyi | 18:30 |
*** rpodolyaka1 has joined #openstack-oslo | 18:32 | |
*** morganfainberg is now known as morganfainberg_Z | 18:33 | |
mriedem | bnemec: cool, thanks for the help | 18:34 |
bnemec | mriedem: np | 18:35 |
mrodden | oh ic | 18:35 |
mrodden | missing super call | 18:35 |
mriedem | so manager calls db.base which then calls periodictasks init? | 18:35 |
mriedem | wacky | 18:35 |
mrodden | would continue the MRO to get the __init__ from periodic | 18:36 |
mrodden | its Python Fun(tm) | 18:36 |
mrodden | :) | 18:36 |
*** ihrachyshka has joined #openstack-oslo | 18:36 | |
*** morganfainberg_Z is now known as morganfainberg | 18:37 | |
*** ihrachyshka has quit IRC | 18:43 | |
*** rpodolyaka1 has quit IRC | 18:47 | |
*** praneshp has quit IRC | 18:51 | |
*** praneshp has joined #openstack-oslo | 18:55 | |
*** rpodolyaka1 has joined #openstack-oslo | 19:07 | |
*** openstackgerrit has joined #openstack-oslo | 19:07 | |
*** AAzza has quit IRC | 19:11 | |
*** openstackgerrit has quit IRC | 19:13 | |
*** zhiyan_ is now known as zhiyan | 19:14 | |
*** arnaud has joined #openstack-oslo | 19:17 | |
*** dims has joined #openstack-oslo | 19:21 | |
*** zhiyan is now known as zhiyan_ | 19:24 | |
*** arnaud has quit IRC | 19:25 | |
*** AAzza has joined #openstack-oslo | 19:26 | |
*** rpodolyaka1 has quit IRC | 19:28 | |
dims | mriedem, ack. that does seem to be parenthesis now that i go over the history | 19:33 |
*** openstackgerrit has joined #openstack-oslo | 19:35 | |
*** flaper87|afk is now known as flaper87 | 19:44 | |
mriedem | dims: i'll fix it at some point here | 19:44 |
viktors | dhellmann_: hi! If you'll have a time, please re-review rpodolyaka's patch https://review.openstack.org/#/c/94552/ (Fix usage of oslo.config). This patch is one of the oslo.db release blockers. Thanks! | 19:46 |
*** notmyname has left #openstack-oslo | 19:48 | |
*** arnaud has joined #openstack-oslo | 19:52 | |
*** markmc has joined #openstack-oslo | 19:55 | |
*** markmc has quit IRC | 19:55 | |
*** vito_ has joined #openstack-oslo | 19:57 | |
*** vito_ is now known as Guest32518 | 19:57 | |
*** Guest32518 has quit IRC | 19:57 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Add a runtime module https://review.openstack.org/94461 | 19:59 |
*** rmcall has joined #openstack-oslo | 20:00 | |
openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Finish factoring apart the graph_action module https://review.openstack.org/94461 | 20:05 |
*** zhiyan_ is now known as zhiyan | 20:15 | |
-openstackstatus- NOTICE: Gerrit is going offline to correct an issue with a recent project rename. ETA 20:45 UTC. | 20:21 | |
*** ChanServ changes topic to "Gerrit is going offline to correct an issue with a recent project rename. ETA 20:45 UTC." | 20:21 | |
*** openstackgerrit has quit IRC | 20:25 | |
*** zhiyan is now known as zhiyan_ | 20:26 | |
*** viktors has left #openstack-oslo | 20:28 | |
-openstackstatus- NOTICE: Gerrit is back online | 20:40 | |
*** ChanServ changes topic to "#openstack-oslo" | 20:40 | |
*** jecarey has quit IRC | 20:44 | |
*** shakamunyi has quit IRC | 20:51 | |
*** shakamunyi has joined #openstack-oslo | 20:52 | |
dims | mriedem, ack. +1 | 20:52 |
*** jecarey has joined #openstack-oslo | 20:54 | |
*** flaper87 is now known as flaper87|afk | 21:03 | |
*** shakamunyi has quit IRC | 21:14 | |
*** zhiyan_ is now known as zhiyan | 21:18 | |
*** arnaud has quit IRC | 21:18 | |
*** zhiyan is now known as zhiyan_ | 21:27 | |
*** AAzza has quit IRC | 21:28 | |
*** openstackgerrit has joined #openstack-oslo | 21:34 | |
*** mriedem has left #openstack-oslo | 21:46 | |
*** openstackgerrit has quit IRC | 21:47 | |
*** mrda-away is now known as mrda | 21:48 | |
*** ujjain has quit IRC | 21:50 | |
*** ujjain has joined #openstack-oslo | 21:55 | |
*** dims has quit IRC | 21:58 | |
*** shakamunyi has joined #openstack-oslo | 22:11 | |
*** lbragstad has quit IRC | 22:12 | |
*** mrodden has quit IRC | 22:13 | |
*** zhiyan_ is now known as zhiyan | 22:18 | |
*** shakayumi has joined #openstack-oslo | 22:22 | |
*** shakamunyi has quit IRC | 22:22 | |
*** dims has joined #openstack-oslo | 22:24 | |
*** zhiyan is now known as zhiyan_ | 22:27 | |
*** yamahata has joined #openstack-oslo | 22:28 | |
*** bknudson has quit IRC | 22:31 | |
*** arnaud has joined #openstack-oslo | 22:42 | |
*** zhiyan_ is now known as zhiyan | 23:19 | |
*** yamahata has quit IRC | 23:20 | |
*** zhiyan is now known as zhiyan_ | 23:28 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!