| *** yamahata has quit IRC | 00:24 | |
| openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Allow examining parsing exceptions. https://review.openstack.org/80856 | 01:04 |
|---|---|---|
| openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Permit pre-release versions with git metadata https://review.openstack.org/80857 | 01:04 |
| openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Teach pbr about post versioned dev versions. https://review.openstack.org/80449 | 01:04 |
| openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Add a converter to version_tuples. https://review.openstack.org/80457 | 01:04 |
| *** shakamunyi has joined #openstack-oslo | 01:37 | |
| *** bknudson has quit IRC | 01:38 | |
| *** yamahata has joined #openstack-oslo | 01:44 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Adjust the exception hierachy https://review.openstack.org/80372 | 02:03 |
| *** gcb has joined #openstack-oslo | 02:11 | |
| *** yamahata has quit IRC | 02:27 | |
| *** zhiyan_ is now known as zhiyan | 02:30 | |
| *** zhiyan is now known as zhiyan_ | 02:35 | |
| openstackgerrit | gordon chung proposed a change to openstack/pycadf: add sphinx base files https://review.openstack.org/80880 | 02:35 |
| openstackgerrit | gordon chung proposed a change to openstack/pycadf: pycadf documentation https://review.openstack.org/65139 | 02:54 |
| *** banix has joined #openstack-oslo | 02:59 | |
| openstackgerrit | gordon chung proposed a change to openstack/pycadf: pycadf documentation https://review.openstack.org/65139 | 03:11 |
| openstackgerrit | gordon chung proposed a change to openstack/pycadf: pycadf documentation https://review.openstack.org/65139 | 03:32 |
| openstackgerrit | gordon chung proposed a change to openstack/pycadf: pycadf documentation https://review.openstack.org/65139 | 03:42 |
| *** banix has quit IRC | 03:42 | |
| openstackgerrit | gordon chung proposed a change to openstack/pycadf: pycadf documentation https://review.openstack.org/65139 | 03:57 |
| openstackgerrit | gordon chung proposed a change to openstack/pycadf: pycadf documentation https://review.openstack.org/65139 | 04:15 |
| *** yamahata has joined #openstack-oslo | 04:25 | |
| openstackgerrit | gordon chung proposed a change to openstack/pycadf: pycadf documentation https://review.openstack.org/65139 | 04:28 |
| *** zhiyan_ is now known as zhiyan | 05:02 | |
| *** sergmelikyan has joined #openstack-oslo | 05:04 | |
| *** morganfainberg_Z is now known as morganfainberg | 05:34 | |
| *** rpodolyaka has joined #openstack-oslo | 06:02 | |
| *** rpodolyaka has quit IRC | 06:16 | |
| *** rpodolyaka has joined #openstack-oslo | 06:16 | |
| *** rpodolyaka has quit IRC | 06:19 | |
| *** rpodolyaka has joined #openstack-oslo | 06:38 | |
| *** rpodolyaka has quit IRC | 06:56 | |
| *** rpodolyaka has joined #openstack-oslo | 06:58 | |
| *** rpodolyaka has quit IRC | 07:00 | |
| *** rpodolyaka has joined #openstack-oslo | 07:03 | |
| *** rpodolyaka has quit IRC | 07:07 | |
| *** flaper87|afk is now known as flaper87 | 07:30 | |
| openstackgerrit | Zhongyue Luo proposed a change to openstack/oslo-incubator: Clean up test_imageutils https://review.openstack.org/78093 | 07:37 |
| openstackgerrit | Zhongyue Luo proposed a change to openstack/oslo-incubator: Remove redundant test cases in test_imageutils https://review.openstack.org/77030 | 07:37 |
| openstackgerrit | Zhongyue Luo proposed a change to openstack/oslo-incubator: Adds test condition in test_periodic https://review.openstack.org/72071 | 07:40 |
| openstackgerrit | Xiang BZ Zhou proposed a change to openstack/oslo.vmware: WIP: Wrapper for WaitForUpdatesEx in Vmware Vsphere SDK https://review.openstack.org/78836 | 07:44 |
| *** rpodolyaka1 has joined #openstack-oslo | 08:03 | |
| *** rpodolyaka1 has quit IRC | 08:07 | |
| *** openstack has quit IRC | 08:21 | |
| *** openstack has joined #openstack-oslo | 08:30 | |
| *** openstackstatus has joined #openstack-oslo | 08:30 | |
| *** gcb has joined #openstack-oslo | 08:31 | |
| openstackgerrit | Ivan Melnikov proposed a change to openstack/taskflow: docs: Add inheritance diagram to exceptions documentation https://review.openstack.org/80925 | 08:35 |
| *** morganfainberg is now known as morganfainberg_Z | 08:45 | |
| openstackgerrit | Oleg Bondarev proposed a change to openstack/oslo-incubator: save_and_reraise_exception: make logging respect the reraise parameter https://review.openstack.org/80222 | 08:49 |
| *** rpodolyaka1 has joined #openstack-oslo | 09:03 | |
| *** rpodolyaka1 has quit IRC | 09:08 | |
| *** ttx has joined #openstack-oslo | 09:25 | |
| openstackgerrit | Anastasia Karpinska proposed a change to openstack/taskflow: Move flow states to developers documentation https://review.openstack.org/80350 | 09:44 |
| openstackgerrit | Anastasia Karpinska proposed a change to openstack/taskflow: Flow smart revert with retry controller https://review.openstack.org/71621 | 09:44 |
| openstackgerrit | wingwj proposed a change to openstack/oslo-incubator: Add _retry_on_deadlock support for postgresql https://review.openstack.org/74917 | 09:47 |
| ihrachys1 | dhellmann: so how about (re)adding fake executor to oslo.messaging? what's your position on this? this can be beneficial for test runs. | 09:53 |
| *** dshulyak has joined #openstack-oslo | 09:57 | |
| *** ihrachys1 is now known as ihrachys | 10:01 | |
| openstackgerrit | Ivan Melnikov proposed a change to openstack/taskflow: Iteration over links in flow interface https://review.openstack.org/80942 | 10:30 |
| *** yamahata has quit IRC | 10:52 | |
| *** rpodolyaka1 has joined #openstack-oslo | 11:01 | |
| *** rpodolyaka1 has quit IRC | 11:05 | |
| *** openstackgerrit has quit IRC | 11:10 | |
| *** openstackgerrit has joined #openstack-oslo | 11:11 | |
| openstackgerrit | Ivan Melnikov proposed a change to openstack/taskflow: Iteration over links in flow interface https://review.openstack.org/80942 | 11:34 |
| *** yamahata has joined #openstack-oslo | 12:07 | |
| *** dims has quit IRC | 12:18 | |
| *** dims has joined #openstack-oslo | 12:18 | |
| openstackgerrit | Ivan Melnikov proposed a change to openstack/taskflow: Test storage with memory and sqlite backends https://review.openstack.org/80553 | 12:26 |
| *** banix has joined #openstack-oslo | 12:55 | |
| *** mriedem has joined #openstack-oslo | 12:57 | |
| *** banix has quit IRC | 12:59 | |
| openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo-incubator: WIP: Opportunistic migration tests https://review.openstack.org/75107 | 13:06 |
| *** lbragstad has quit IRC | 13:07 | |
| flaper87 | dhellmann: https://review.openstack.org/#/c/76458/ found it! There was some weirdness going on with the eventlet.Event signaling. Backporting https://review.openstack.org/#/c/52415/ fixes the issue in stable/havana | 13:07 |
| flaper87 | dhellmann: (btw, I'm talking about the issue I was having with that backport and the `test_service` test) :D | 13:07 |
| openstackgerrit | gordon chung proposed a change to openstack/oslo.messaging: graduate notifier middleware https://review.openstack.org/75933 | 13:09 |
| flaper87 | erm, the gate hates me | 13:10 |
| *** banix has joined #openstack-oslo | 13:31 | |
| *** lbragstad has joined #openstack-oslo | 13:38 | |
| openstackgerrit | sahid proposed a change to openstack/oslo-incubator: Add utility decorator to skip test if not sqllite https://review.openstack.org/80972 | 13:45 |
| openstackgerrit | Ivan Melnikov proposed a change to openstack/taskflow: Add tox environment for pypy https://review.openstack.org/74017 | 13:48 |
| openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo-incubator: WIP: Opportunistic migration tests https://review.openstack.org/75107 | 13:57 |
| *** bknudson has joined #openstack-oslo | 13:58 | |
| openstackgerrit | sahid proposed a change to openstack/oslo-incubator: Add utility decorator to skip test if not sqllite https://review.openstack.org/80972 | 14:00 |
| *** markmcclain has joined #openstack-oslo | 14:09 | |
| *** jraim has quit IRC | 14:11 | |
| *** jraim has joined #openstack-oslo | 14:11 | |
| *** flaper87 has quit IRC | 14:26 | |
| *** flaper87|afk has joined #openstack-oslo | 14:31 | |
| *** flaper87|afk is now known as flaper87 | 14:32 | |
| *** flaper87 has quit IRC | 14:32 | |
| *** flaper87 has joined #openstack-oslo | 14:32 | |
| ihrachys | dhellmann: ping re: fake executor in oslo.messaging | 14:32 |
| openstackgerrit | Davanum Srinivas (dims) proposed a change to openstack/oslosphinx: Prevent left sidebar text overflow https://review.openstack.org/79077 | 14:44 |
| dhellmann | ihrachys: I'm not sure why a fake executor is needed if we have a fake driver? | 14:44 |
| ihrachys | dhellmann: because fake driver does not influence RPCServer.start() behaviour | 14:45 |
| openstackgerrit | wingwj proposed a change to openstack/oslo-incubator: Add _retry_on_deadlock support for postgresql https://review.openstack.org/74917 | 14:45 |
| ihrachys | as I understand, this is what executors are for | 14:45 |
| openstackgerrit | Ilya Pekelny proposed a change to openstack/oslo-incubator: WIP: Opportunistic migration tests https://review.openstack.org/75107 | 15:05 |
| openstackgerrit | Ben Nemec proposed a change to openstack/oslo-incubator: save_and_reraise_exception: make logging respect the reraise parameter https://review.openstack.org/80222 | 15:08 |
| ihrachys | dhellmann: am I wrong? | 15:10 |
| dhellmann | ihrachys: I'm having some IRC trouble, so I may have missed an earlier statement. Why do you think we need a fake executor? | 15:11 |
| ihrachys | dhellmann: because fake driver does not influence RPCServer.start() behaviour | 15:11 |
| ihrachys | as I understand, this is what executors are for | 15:11 |
| dhellmann | ihrachys: right, but with a fake executor you wouldn't have found the bug you found last week, so it seems like using a real executor is better, no? | 15:12 |
| *** obondarev has joined #openstack-oslo | 15:15 | |
| openstackgerrit | Ben Nemec proposed a change to openstack/oslo-incubator: save_and_reraise_exception: make logging respect the reraise parameter https://review.openstack.org/80222 | 15:17 |
| ihrachys | dhellmann: well, this (maybe) makes unit tests run 2 to 3 times longer [assumption, not fact] | 15:19 |
| ihrachys | running unit tests for more than one hour is not acceptable | 15:20 |
| ihrachys | [but - maybe! - after we fix our issue, performance will return to normal] | 15:20 |
| dhellmann | ihrachys: have you tried the blocking executor? | 15:20 |
| ihrachys | no, I didn't. ok, I'll try it. | 15:21 |
| *** markmcclain has quit IRC | 15:21 | |
| dhellmann | ihrachys: I'd like to understand why performance changes so much, and why we didn't automatically create a fake executor to begin with, before adding one | 15:22 |
| ihrachys | I see your point. Yes, we should first fix our issues and check how it works before requesting fake executor. | 15:24 |
| *** shakamunyi has quit IRC | 15:31 | |
| *** flaper87 is now known as flaper87|afk | 15:34 | |
| ihrachys | dhellmann: we're not going to get oslo.messaging port for neutron in icehouse. can we make policy of not taking any 'featurey' patches to oslo-rpc in incubator a bit less strict? like in case of: https://review.openstack.org/#/c/67449/ | 15:41 |
| dhellmann | ihrachys: has that change landed in oslo.messaging? | 15:42 |
| ihrachys | dhellmann: yes, and I think the behaviour should be consistent between projects no matter whether they were lucky enough to get to oslo.messaging this cycle | 15:42 |
| ihrachys | dhellmann: it's 3fdd52e0d48dab3b5cf6623d58a4ac53530b20e8 in oslo.messaging | 15:44 |
| dhellmann | ihrachys: why is that change cycling through the index values of the list instead of the members of the list? | 15:45 |
| ihrachys | dhellmann: mostly because this is a backport from oslo.messaging; yes, that could be enhanced a bit in a way that you suggest. | 15:47 |
| dhellmann | ihrachys: ok, nevermind, I didn't make it back around to the oslo.messaging review before someone else approved it | 15:47 |
| dhellmann | ihrachys: I don't see the unit test in the original commit, was that added separately? | 15:48 |
| ihrachys | [plus there's a tiny drawback in cycling thru the brokers, not indices - cycle() makes a copy of all elements of the iterable] | 15:48 |
| ihrachys | at the moment when original commit was done, there were no tests for qpid whatsoever | 15:48 |
| dhellmann | ihrachys: ok | 15:49 |
| ihrachys | [yes, I know I should have started one back then; but I was completely new to the project, so didn't realize that] | 15:49 |
| ihrachys | :) | 15:49 |
| dhellmann | no, that's ok, I'm just trying to understand the history since the patches are different | 15:49 |
| ihrachys | the history was that I've started from oslo-incubator first, implemented a test et al. Then I was pointed to oslo.messaging, so the real direction for backport was incubator -> oslo.messaging, not the usual one | 15:50 |
| dhellmann | ihrachys: does that test exist in oslo.messaging, then? | 15:50 |
| ihrachys | no, that's why this part didn't reach there (I just skipped test part because I haven't found a place where to put the test) | 15:51 |
| ihrachys | but I think I've seen some patches recently to add qpid test in oslo.messaging | 15:51 |
| ihrachys | not sure they reached master | 15:51 |
| dhellmann | ihrachys: there is now a test_qpid.py file, so that's a good place | 15:52 |
| ihrachys | ok, putting this in my todo list | 15:52 |
| openstackgerrit | Ben Nemec proposed a change to openstack/oslo-incubator: Improve update.py docstring https://review.openstack.org/81004 | 15:53 |
| ihrachys | dhellmann: great; then there is another one backport for rabbitmq at: https://review.openstack.org/#/c/67449/, with test | 15:54 |
| ihrachys | ehm... sorry, probably worng link. I'll recheck | 15:56 |
| *** markmcclain has joined #openstack-oslo | 15:57 | |
| ihrachys | dhellmann: yeah, this one is correct: https://review.openstack.org/#/c/67450/ | 15:57 |
| dhellmann | ihrachys: the same restrictions would apply | 16:00 |
| ihrachys | dhellmann: surely, but we already have the test in oslo.messaging for this | 16:00 |
| dhellmann | ihrachys: ok, I didn't check | 16:00 |
| openstackgerrit | Carl Baldwin proposed a change to openstack/oslo-incubator: Return the thread created in consume_in_thread https://review.openstack.org/81009 | 16:23 |
| *** markmcclain has quit IRC | 17:04 | |
| *** harlowja_away is now known as harlowja | 17:09 | |
| *** morganfainberg_Z is now known as morganfainberg | 17:14 | |
| *** markmcclain has joined #openstack-oslo | 17:17 | |
| *** zhiyan is now known as zhiyan_ | 17:32 | |
| openstackgerrit | Solly Ross proposed a change to openstack/oslo-incubator: Report: Add basic processes section https://review.openstack.org/78793 | 17:56 |
| openstackgerrit | Solly Ross proposed a change to openstack/oslo-incubator: Report: Propagate view_type through collections https://review.openstack.org/78792 | 17:56 |
| openstackgerrit | Solly Ross proposed a change to openstack/oslo-incubator: Report: Support Sequences and Mappings as Data https://review.openstack.org/78791 | 17:56 |
| *** markmcclain has quit IRC | 17:57 | |
| *** markmcclain has joined #openstack-oslo | 17:59 | |
| *** jroll has joined #openstack-oslo | 18:15 | |
| jroll | hey :) | 18:15 |
| dhellmann | hi | 18:15 |
| dhellmann | so you want just the log module? | 18:15 |
| *** arnaud__ has joined #openstack-oslo | 18:15 | |
| jroll | yes | 18:15 |
| * dhellmann checks the dependency list | 18:16 | |
| dhellmann | https://wiki.openstack.org/wiki/Oslo/Dependencies | 18:16 |
| dhellmann | it looks like you will need a few other modules that it depends on | 18:16 |
| jroll | sure, that's fine | 18:16 |
| dhellmann | but if you add just log to your openstack-common.conf, update.py should bring those in for you | 18:16 |
| jroll | is there documentation on openstack-common.conf? | 18:17 |
| jroll | this is for ironic-python-agent, which is not yet officially openstack | 18:17 |
| jroll | we're working on getting rid of non-OS dependencies | 18:18 |
| jroll | but this is the first openstack.common thing we've brought in | 18:18 |
| dhellmann | https://wiki.openstack.org/wiki/Oslo#Incubation | 18:18 |
| dhellmann | that's not great, but it's a start | 18:18 |
| jroll | ok, thanks | 18:19 |
| dhellmann | jroll: are you already using oslo.config/ | 18:19 |
| dhellmann | ? | 18:19 |
| jroll | we are not | 18:20 |
| dhellmann | ok, the logging setup code wants that, too | 18:20 |
| jroll | that's fine | 18:20 |
| dhellmann | that *is* a lib, so you just add that to your requirements.txt | 18:20 |
| jroll | we're trying to keep dependencies as small as possible, but it's fine | 18:20 |
| jroll | right | 18:20 |
| jroll | :) | 18:21 |
| jroll | thanks again | 18:21 |
| dhellmann | jroll: any time, and come back if you run into trouble and I'll try to help you out | 18:21 |
| *** directxman12 has joined #openstack-oslo | 18:27 | |
| jroll | will do | 18:27 |
| *** wingwj has quit IRC | 18:41 | |
| *** wingwj has joined #openstack-oslo | 18:44 | |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Use a queue instead of recursion https://review.openstack.org/80736 | 18:55 |
| openstackgerrit | Joshua Harlow proposed a change to openstack/taskflow: Use a queue instead of recursion https://review.openstack.org/80736 | 19:00 |
| *** rpodolyaka1 has joined #openstack-oslo | 19:02 | |
| openstackgerrit | Solly Ross proposed a change to openstack/oslo-incubator: Report: Add basic processes section https://review.openstack.org/78793 | 19:59 |
| openstackgerrit | gordon chung proposed a change to openstack/pycadf: add OpenStack Foundation as author https://review.openstack.org/81066 | 20:04 |
| *** directxman12 has quit IRC | 20:09 | |
| *** directxman12 has joined #openstack-oslo | 20:10 | |
| *** arnaud__ has quit IRC | 20:44 | |
| openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Teach pbr VersionInfo about debian versions. https://review.openstack.org/81074 | 20:44 |
| openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Allow examining parsing exceptions. https://review.openstack.org/80856 | 20:44 |
| openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Permit pre-release versions with git metadata https://review.openstack.org/80857 | 20:44 |
| openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Teach pbr about post versioned dev versions. https://review.openstack.org/80449 | 20:44 |
| openstackgerrit | lifeless proposed a change to openstack-dev/pbr: Add a converter to version_tuples. https://review.openstack.org/80457 | 20:44 |
| *** harlowja is now known as harlowja_away | 20:44 | |
| *** arnaud__ has joined #openstack-oslo | 20:47 | |
| *** rpodolyaka1 has quit IRC | 20:52 | |
| *** rpodolyaka1 has joined #openstack-oslo | 20:53 | |
| *** rpodolyaka1 has quit IRC | 20:57 | |
| openstackgerrit | Terry Howe proposed a change to openstack/cliff: Add value only output formattter https://review.openstack.org/80614 | 21:00 |
| openstackgerrit | Terry Howe proposed a change to openstack/cliff: Add value only output formattter https://review.openstack.org/80614 | 21:07 |
| *** directxman12 has quit IRC | 21:16 | |
| *** rpodolyaka1 has joined #openstack-oslo | 21:23 | |
| *** rpodolyaka1 has quit IRC | 21:27 | |
| *** beekneemech is now known as bnemec | 21:36 | |
| bnemec | Umm, https://review.openstack.org/#/c/81066/ | 21:38 |
| bnemec | Not sure self-approving is a precedent we want to set. :-/ | 21:38 |
| *** directxman12 has joined #openstack-oslo | 21:42 | |
| *** mattymo has quit IRC | 21:51 | |
| *** mattymo has joined #openstack-oslo | 21:51 | |
| *** lbragstad has quit IRC | 22:05 | |
| *** banix has quit IRC | 22:09 | |
| *** harlowja_away is now known as harlowja | 22:17 | |
| *** yamahata has quit IRC | 22:22 | |
| *** rpodolyaka1 has joined #openstack-oslo | 22:24 | |
| *** yamahata has joined #openstack-oslo | 22:25 | |
| *** rpodolyaka1 has quit IRC | 22:28 | |
| *** mriedem has left #openstack-oslo | 22:32 | |
| *** mriedem has quit IRC | 22:32 | |
| *** openstackgerrit has quit IRC | 22:39 | |
| *** openstackgerrit has joined #openstack-oslo | 22:40 | |
| *** markmcclain has quit IRC | 22:51 | |
| openstackgerrit | Stanislav Kudriashev proposed a change to openstack/taskflow: [WBE] Collect information from workers https://review.openstack.org/79713 | 23:12 |
| openstackgerrit | Stanislav Kudriashev proposed a change to openstack/taskflow: [WBE] Collect information from workers https://review.openstack.org/79713 | 23:14 |
| *** directxman12 has quit IRC | 23:14 | |
| *** dhellmann has quit IRC | 23:18 | |
| *** yamahata has quit IRC | 23:18 | |
| *** adrian_otto has joined #openstack-oslo | 23:20 | |
| *** tjones has joined #openstack-oslo | 23:21 | |
| *** dhellmann has joined #openstack-oslo | 23:23 | |
| *** dims has quit IRC | 23:24 | |
| *** rpodolyaka1 has joined #openstack-oslo | 23:24 | |
| *** rpodolyaka1 has quit IRC | 23:29 | |
| *** tjones has quit IRC | 23:31 | |
| *** bknudson has quit IRC | 23:39 | |
| *** dims has joined #openstack-oslo | 23:39 | |
| *** tjones has joined #openstack-oslo | 23:46 | |
| *** adrian_otto has quit IRC | 23:50 | |
| *** tjones has quit IRC | 23:50 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!