| *** bnemec has quit IRC | 02:19 | |
| *** bnemec has joined #openstack-oslo | 02:28 | |
| *** threestrands has joined #openstack-oslo | 03:39 | |
| *** toabctl has joined #openstack-oslo | 03:39 | |
| *** toabctl has quit IRC | 04:03 | |
| *** toabctl has joined #openstack-oslo | 04:07 | |
| *** dave-mccowan has joined #openstack-oslo | 05:14 | |
| *** rpittau|afk is now known as rpittau | 07:17 | |
| *** iurygregory has quit IRC | 07:28 | |
| *** tosky has joined #openstack-oslo | 07:29 | |
| *** iurygregory has joined #openstack-oslo | 07:29 | |
| *** ralonsoh has joined #openstack-oslo | 08:02 | |
| *** threestrands has quit IRC | 08:26 | |
| *** Luzi has joined #openstack-oslo | 08:44 | |
| *** rcernin has quit IRC | 08:54 | |
| openstackgerrit | Merged openstack/cookiecutter master: Update hacking version of new repo https://review.opendev.org/716065 | 09:54 |
|---|---|---|
| *** rcernin has joined #openstack-oslo | 09:59 | |
| *** rcernin has quit IRC | 10:14 | |
| *** zzzeek has quit IRC | 10:44 | |
| *** zzzeek has joined #openstack-oslo | 10:45 | |
| *** rpittau is now known as rpittau|bbl | 11:05 | |
| openstackgerrit | Merged openstack/oslo.limit master: Update to hacking 3.0 https://review.opendev.org/715973 | 11:13 |
| openstackgerrit | Merged openstack/oslo.vmware master: Document disable flake8/hacking warnings https://review.opendev.org/715935 | 11:24 |
| openstackgerrit | Andreas Jaeger proposed openstack/whereto master: Update to hacking 3.0 https://review.opendev.org/716257 | 11:43 |
| *** ralonsoh has quit IRC | 11:59 | |
| *** ralonsoh has joined #openstack-oslo | 12:00 | |
| *** raildo has joined #openstack-oslo | 12:00 | |
| *** zzzeek has quit IRC | 12:05 | |
| *** zzzeek has joined #openstack-oslo | 12:06 | |
| *** kgiusti has joined #openstack-oslo | 12:10 | |
| *** rpittau|bbl is now known as rpittau | 12:15 | |
| *** zzzeek has quit IRC | 12:44 | |
| *** zzzeek has joined #openstack-oslo | 12:45 | |
| *** moguimar has joined #openstack-oslo | 12:56 | |
| *** zzzeek has quit IRC | 13:38 | |
| *** zzzeek has joined #openstack-oslo | 13:39 | |
| *** Luzi has quit IRC | 13:54 | |
| *** bnemec has quit IRC | 14:00 | |
| openstackgerrit | Merged openstack/whereto master: Update to hacking 3.0 https://review.opendev.org/716257 | 14:05 |
| *** bnemec has joined #openstack-oslo | 14:16 | |
| *** bnemec has quit IRC | 14:22 | |
| *** dave-mccowan has quit IRC | 14:23 | |
| *** zzzeek has quit IRC | 14:28 | |
| openstackgerrit | Elod Illes proposed openstack/tooz stable/train: [stable-only] Cap stestr for python 2 https://review.opendev.org/715940 | 14:31 |
| *** dave-mccowan has joined #openstack-oslo | 14:31 | |
| *** zzzeek has joined #openstack-oslo | 14:32 | |
| *** tkajinam has quit IRC | 14:45 | |
| hberaud | stephenfin, smcginnis: I suppose you don't +Wed because you reviewed at the time? Can you confirm? https://review.opendev.org/#/c/715940/3 | 14:47 |
| hberaud | s/at the time/at the same time/ | 14:48 |
| smcginnis | hberaud: Yeah, feel free to approve. | 14:49 |
| hberaud | smcginnis: ok thanks | 14:49 |
| *** dave-mccowan has quit IRC | 15:04 | |
| *** dave-mccowan has joined #openstack-oslo | 15:07 | |
| openstackgerrit | Elod Illes proposed openstack/tooz master: Drop requirements-check job https://review.opendev.org/716337 | 15:17 |
| *** rpittau is now known as rpittau|afk | 16:07 | |
| *** bnemec has joined #openstack-oslo | 16:11 | |
| openstackgerrit | Radosław Piliszek proposed openstack/oslo.cache master: [WIP] Adding functional tests for memcache_pool backend https://review.opendev.org/712479 | 17:12 |
| *** tobias-urdin has joined #openstack-oslo | 17:16 | |
| tobias-urdin | hberaud: if you have a sec, i'm stuck on this traceback http://paste.openstack.org/show/791412/ | 17:17 |
| tobias-urdin | caused by nova-compute, seems to be related to oslo.db | 17:17 |
| tobias-urdin | versions: oslo.db 5.0.2 oslo.concurrency 3.30.0 oslo.versionedobjects 1.36.1 | 17:18 |
| tobias-urdin | nova 20.1.1 | 17:18 |
| openstackgerrit | Radosław Piliszek proposed openstack/oslo.cache master: [WIP] Adding functional tests for memcache_pool backend https://review.opendev.org/712479 | 17:29 |
| hberaud | tobias-urdin: o/ all your version seems related to train can you confirm? | 17:40 |
| hberaud | tobias-urdin: all these versions looks like "older" versions https://github.com/openstack/releases/commit/332951cead5e6f1a649ee8b90d7452aed354f252 https://github.com/openstack/releases/commit/eeae2df5bfe570912fdd63d5a340b369a6bef8bb I don't seen recent changes on our side | 17:42 |
| hberaud | tobias-urdin: have you seen recent changes on your side that could explain that? Have you upgraded something or noticed recent changes, or when was the last time you seen a normal execution related to this? | 17:46 |
| tobias-urdin | hberaud: yeah, im testing upgrade yeah | 17:46 |
| tobias-urdin | it's when nova calls register_opts for database_opts | 17:46 |
| tobias-urdin | crashes on: sqlite_synchronous | 17:46 |
| tobias-urdin | for some reason | 17:47 |
| hberaud | tobias-urdin: what's the package versions before update? | 17:47 |
| hberaud | tobias-urdin: can you file a LP to track related things please? | 17:49 |
| tobias-urdin | oslo.config was 6.4.1 oslo.db 4.40.1 | 17:49 |
| hberaud | tobias-urdin: or maybe your already have an existing one | 17:50 |
| tobias-urdin | not yet, i'm assuming there is some version incompatibility that is not fixed in packaging, but i've upgraded all pkgs i can think off and the issue hasn't been resolved yet | 17:51 |
| hberaud | tobias-urdin: ack | 17:55 |
| hberaud | tobias-urdin: I need to eject soon, I'll take a further look tomorrow, let me know if you find something by waiting for tomorrow | 17:56 |
| hberaud | tobias-urdin: what's the version of oslo.config after upgrade? | 17:58 |
| tobias-urdin | hberaud: ack https://bugs.launchpad.net/oslo.config/+bug/1869929 | 18:01 |
| openstack | Launchpad bug 1869929 in oslo.config "RuntimeError: maximum recursion depth exceeded while calling a Python object" [Undecided,New] | 18:01 |
| hberaud | tobias-urdin: maybe you faced something like this => https://review.opendev.org/#/c/699522/ https://bugs.launchpad.net/nova/+bug/1856312 | 18:01 |
| openstack | Launchpad bug 1856312 in oslo.config "RuntimeError during calling log_opts_values" [Undecided,Fix released] - Assigned to Stephen Finucane (stephenfinucane) | 18:01 |
| openstackgerrit | Merged openstack/tooz stable/train: [stable-only] Cap stestr for python 2 https://review.opendev.org/715940 | 18:02 |
| hberaud | tobias-urdin: thanks for the LP | 18:02 |
| tobias-urdin | i'll try to dig more, but i'm kind of stuck right now :) | 18:04 |
| hberaud | tobias-urdin: thanks i ping you if I get something :) | 18:05 |
| openstackgerrit | Radosław Piliszek proposed openstack/oslo.cache master: [WIP] Adding functional tests for memcache_pool backend https://review.opendev.org/712479 | 18:22 |
| openstackgerrit | Radosław Piliszek proposed openstack/oslo.cache master: [WIP] Adding functional tests for memcache_pool backend https://review.opendev.org/712479 | 18:26 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.versionedobjects master: Use unittest.mock instead of third party mock https://review.opendev.org/716383 | 18:37 |
| smcginnis | bnemec: We may need/want to make a freeze exception for that ^ | 18:38 |
| smcginnis | Requirements update patches are failing for some of the oslo libs that were just released. | 18:38 |
| smcginnis | Some projects have already dropped third party mock support, and so has oslotest. | 18:38 |
| smcginnis | Some other oslo libs are using mock, but only have it in the test-requirements. | 18:38 |
| smcginnis | So now that other projects are not pulling that in indirectly, tests are failing due to the missing lib. | 18:39 |
| bnemec | Hmm, we probably should have waited to do oslotest until last. | 18:39 |
| bnemec | Hindsight. :-) | 18:40 |
| smcginnis | Looks like privsep uses mock, but doesn't even have it in any requirements files. | 18:40 |
| smcginnis | I'm going to try to make a quick pass and find where we have issues. | 18:40 |
| *** ralonsoh has quit IRC | 18:40 | |
| smcginnis | But yeah, another option would be to just revert that drop to oslotest and get a new release of that out. | 18:41 |
| bnemec | Yeah, if it turns out to be a big problem that might be the safest approach. | 18:41 |
| smcginnis | That would give us all of victoria to make sure there aren't any other side effects. | 18:41 |
| smcginnis | Quick codesearch, looks like we have 15 libs that still use "import mock" | 18:43 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.privsep master: Use unittest.mock instead of third party mock https://review.opendev.org/716384 | 18:43 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.db master: Use unittest.mock instead of third party mock https://review.opendev.org/716385 | 18:47 |
| openstackgerrit | Radosław Piliszek proposed openstack/oslo.cache master: [WIP] Adding functional tests for memcache_pool backend https://review.opendev.org/712479 | 18:50 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.vmware master: Use unittest.mock instead of third party mock https://review.opendev.org/716387 | 18:51 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.service master: Use unittest.mock instead of third party mock https://review.opendev.org/716388 | 18:54 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.i18n master: Use unittest.mock instead of third party mock https://review.opendev.org/716390 | 18:57 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.policy master: Use unittest.mock instead of third party mock https://review.opendev.org/716391 | 18:59 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.utils master: Use unittest.mock instead of third party mock https://review.opendev.org/716392 | 19:04 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.config master: Use unittest.mock instead of third party mock https://review.opendev.org/716393 | 19:06 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.log master: Use unittest.mock instead of third party mock https://review.opendev.org/716394 | 19:09 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.middleware master: Use unittest.mock instead of third party mock https://review.opendev.org/716395 | 19:11 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.reports master: Use unittest.mock instead of third party mock https://review.opendev.org/716397 | 19:12 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.cache master: Use unittest.mock instead of third party mock https://review.opendev.org/716399 | 19:14 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.concurrency master: Use unittest.mock instead of third party mock https://review.opendev.org/716400 | 19:15 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.rootwrap master: Use unittest.mock instead of third party mock https://review.opendev.org/716401 | 19:16 |
| smcginnis | bnemec: OK, I think I've hit them all now. | 19:16 |
| bnemec | smcginnis: Oof. Do we know if any other projects outside of Oslo are also going to be affected? | 19:17 |
| smcginnis | Looks like at least cinder and nova can't pick up the new oslo releases because of this. Basically anyone that doesn't pull in mock indirectly for it to work. | 19:19 |
| openstackgerrit | Radosław Piliszek proposed openstack/oslo.cache master: [WIP] Adding functional tests for memcache_pool backend https://review.opendev.org/712479 | 19:19 |
| bnemec | I'm wondering how many projects that aren't gating requirements are going to be broken if/when we get it into u-c. | 19:21 |
| smcginnis | Yeah, it's a good question. The requirements repo does hit most of the big ones, but definitely no all projects. | 19:24 |
| *** iurygregory has quit IRC | 19:27 | |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.db master: Use unittest.mock instead of third party mock https://review.opendev.org/716385 | 19:31 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.middleware master: Use unittest.mock instead of third party mock https://review.opendev.org/716395 | 19:32 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.privsep master: Use unittest.mock instead of third party mock https://review.opendev.org/716384 | 19:36 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.vmware master: Use unittest.mock instead of third party mock https://review.opendev.org/716387 | 19:37 |
| tobias-urdin | hberaud: updated LP with reproduce and more info | 19:37 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.service master: Use unittest.mock instead of third party mock https://review.opendev.org/716388 | 19:37 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.policy master: Use unittest.mock instead of third party mock https://review.opendev.org/716391 | 19:38 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.utils master: Use unittest.mock instead of third party mock https://review.opendev.org/716392 | 19:39 |
| openstackgerrit | Radosław Piliszek proposed openstack/oslo.cache master: [WIP] Adding functional tests for memcache_pool backend https://review.opendev.org/712479 | 19:39 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.config master: Use unittest.mock instead of third party mock https://review.opendev.org/716393 | 19:40 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.log master: Use unittest.mock instead of third party mock https://review.opendev.org/716394 | 19:41 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.reports master: Use unittest.mock instead of third party mock https://review.opendev.org/716397 | 19:43 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.concurrency master: Use unittest.mock instead of third party mock https://review.opendev.org/716400 | 19:44 |
| johnsom | We have a user that is reporting a problem with oslopolicy-policy-generator, it seems similar to https://review.opendev.org/#/c/714161/. He provided a paste: http://paste.openstack.org/show/791411/ | 19:44 |
| johnsom | I just stacked and didn't have the issue, but my devstack setup is older and has 2.4.1 oslo.policy | 19:44 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.rootwrap master: Use unittest.mock instead of third party mock https://review.opendev.org/716401 | 19:44 |
| johnsom | If this rings a bell for someone, let me know. I'm going to try to reproduce with an updated devstack. | 19:45 |
| smcginnis | johnsom: Looks like the right signature. | 19:46 |
| smcginnis | johnsom: That fix was included in 3.0.2, so that would be a good thing for them to check. | 19:47 |
| johnsom | Yeah, I pointed them at that. He claimed to have it, but I want to try to reproduce. | 19:47 |
| bnemec | I'm not sure that would be fixed by 3.0.2. This is an actual call to the policy generator, not the sphinx plugin. | 19:48 |
| bnemec | I think the problem here is that cli opts are being registered on import of common code. | 19:53 |
| johnsom | Yeah, I can reproduce this with a fresh venv, install octavia, then run the oslopolicy-policy-generator tool | 20:03 |
| johnsom | It has 3.0.2 in it | 20:03 |
| johnsom | Switching back to 2.4.1 it runs fine | 20:04 |
| johnsom | 3.0.0 it breaks | 20:05 |
| johnsom | This patch seems to have introduced the change: https://github.com/openstack/oslo.policy/commit/686aa238f921e8b6dff814d001690e15fa8ccea6 | 20:08 |
| openstackgerrit | melanie witt proposed openstack/oslo.db master: DNM: investigating bug 1844929 https://review.opendev.org/714802 | 20:18 |
| openstack | bug 1844929 in OpenStack Compute (nova) "grenade jobs failing due to "Timed out waiting for response from cell" in scheduler" [High,Confirmed] https://launchpad.net/bugs/1844929 | 20:18 |
| openstackgerrit | melanie witt proposed openstack/oslo.db stable/train: DNM: investigating bug 1844929 https://review.opendev.org/714833 | 20:19 |
| *** dawzon has joined #openstack-oslo | 20:24 | |
| bnemec | Yeah, it's because of the cli arg fix. I'm looking at whether there's a simple fix. | 20:24 |
| bnemec | Why do we have both oslopolicy-policy-generator and oslopolicy-sample-generator? | 20:29 |
| bnemec | We don't even document the former. :-/ | 20:29 |
| bnemec | Ah, I see. One is outputting the default policies in code, the other respects overrides in the policy file. | 20:33 |
| johnsom | Yeah, this is wayback machine time for me. It's been a while since I have had to look at the oslo policy stuff. | 20:36 |
| johnsom | Thanks for taking a look BTW | 20:38 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.cache master: Use unittest.mock instead of third party mock https://review.opendev.org/716399 | 20:55 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.i18n master: Use unittest.mock instead of third party mock https://review.opendev.org/716390 | 20:55 |
| openstackgerrit | Sean McGinnis proposed openstack/oslo.versionedobjects master: Use unittest.mock instead of third party mock https://review.opendev.org/716383 | 20:56 |
| bnemec | Ah, you fixed this bug in a different way by filtering the args. | 20:58 |
| bnemec | I have a change that fixes the test run. I'll push it and see if it breaks anything else in ci. | 20:59 |
| bnemec | Mmm, failing unit tests. | 21:01 |
| *** rcernin has joined #openstack-oslo | 21:02 | |
| *** iurygregory has joined #openstack-oslo | 21:21 | |
| bnemec | johnsom: I think https://review.opendev.org/716440 will fix the problem, but let's see what ci has to say. | 21:29 |
| johnsom | bnemec Ok, thanks, was just reading your commit message | 21:29 |
| openstackgerrit | Merged openstack/tooz master: Drop requirements-check job https://review.opendev.org/716337 | 21:41 |
| *** raildo has quit IRC | 22:00 | |
| *** hberaud has quit IRC | 22:34 | |
| *** hberaud has joined #openstack-oslo | 22:34 | |
| *** tkajinam has joined #openstack-oslo | 22:49 | |
| *** tosky has quit IRC | 22:54 | |
| *** ianychoi has quit IRC | 23:15 | |
| *** ianychoi has joined #openstack-oslo | 23:16 | |
| *** gibi has quit IRC | 23:57 | |
| *** gibi has joined #openstack-oslo | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!