| *** larainema has joined #openstack-oslo | 04:27 | |
| *** iurygregory has joined #openstack-oslo | 06:49 | |
| openstackgerrit | Arthur Dayne proposed openstack/oslo.policy master: Remove the conversion according to the comment of jdennis https://review.opendev.org/717191 | 06:52 |
|---|---|---|
| openstackgerrit | Arthur Dayne proposed openstack/oslo.policy master: Remove the conversion according to the comment of jdennis https://review.opendev.org/717191 | 06:56 |
| *** bnemec has quit IRC | 07:20 | |
| *** tosky has joined #openstack-oslo | 07:44 | |
| *** rpittau|afk is now known as rpittau | 07:56 | |
| *** ralonsoh has joined #openstack-oslo | 07:57 | |
| *** tkajinam has quit IRC | 08:08 | |
| *** tkajinam has joined #openstack-oslo | 08:54 | |
| openstackgerrit | Arthur Dayne proposed openstack/oslo.policy master: Remove the conversion according to the comment of jdennis https://review.opendev.org/717191 | 10:09 |
| *** tkajinam has quit IRC | 10:11 | |
| *** rpittau is now known as rpittau|bbl | 10:29 | |
| *** rcernin has quit IRC | 10:29 | |
| openstackgerrit | Hervé Beraud proposed openstack/oslo.cache master: [WIP] Adding functional tests https://review.opendev.org/712479 | 11:07 |
| *** rpittau|bbl is now known as rpitau | 12:08 | |
| *** rpitau is now known as rpittau | 12:09 | |
| *** moguimar has quit IRC | 12:45 | |
| *** zigo has quit IRC | 12:48 | |
| *** zigo has joined #openstack-oslo | 12:51 | |
| *** raildo has joined #openstack-oslo | 12:58 | |
| *** bnemec has joined #openstack-oslo | 14:13 | |
| *** bnemec is now known as beekneemech | 14:14 | |
| beekneemech | stephenfin: hberaud: I think we have a requirements issue on https://review.opendev.org/#/c/717191 | 14:19 |
| hberaud | beekneemech: mock seems missing with py38... | 14:20 |
| hberaud | beekneemech: I don't seen patch related to drop mock usage on oslo.policy | 14:20 |
| beekneemech | hberaud: That's not my concern. I left a comment explaining the problem. | 14:20 |
| hberaud | beekneemech: oh ok sorry, I'll take a look | 14:20 |
| beekneemech | There is one: https://review.opendev.org/#/c/716391/ | 14:21 |
| beekneemech | But that reminds me we need to revert the oslotest change for this cycle. | 14:21 |
| openstackgerrit | Ben Nemec proposed openstack/oslotest master: Revert "Switch to unittest.mock from mock" https://review.opendev.org/717310 | 14:24 |
| openstackgerrit | Ben Nemec proposed openstack/oslotest master: Revert "Revert "Switch to unittest.mock from mock"" https://review.opendev.org/717311 | 14:24 |
| beekneemech | smcginnis: I just realized we never wrapped up the oslotest discussion^ | 14:25 |
| smcginnis | beekneemech: Right, I kept meaning to follow up on that but being diverted. | 14:30 |
| beekneemech | Likewise. :-) | 14:30 |
| smcginnis | I wonder though... since this is really kind of stopgap for ussuri, maybe instead of a full revert on all of those changes, what if we do a really small change of just adding mock back into requirements.txt? | 14:31 |
| beekneemech | I would be fine with that too. | 14:42 |
| smcginnis | Eh, since you have the revert of the revert already lined up, maybe safest and best to just do the full revert. | 14:48 |
| beekneemech | It was the lowest effort option. :-) | 14:52 |
| beekneemech | Also, there shouldn't be a lot of changes going in between now and Victoria so the re-revert should stay clean. | 14:54 |
| smcginnis | Yeah, good point! | 14:57 |
| *** yoctozepto has quit IRC | 15:17 | |
| *** yoctozepto8 has joined #openstack-oslo | 15:18 | |
| *** rpittau is now known as rpittau|afk | 15:43 | |
| *** cmurphy is now known as cmorpheus | 16:13 | |
| *** hberaud has quit IRC | 16:40 | |
| yoctozepto8 | hah, and I fixed kolla proactively to avoid py38 fallout with mock, oh well | 16:40 |
| *** hberaud has joined #openstack-oslo | 16:40 | |
| yoctozepto8 | hberaud: hi, what about that memcached - looks like not retaining entries? | 16:41 |
| hberaud | yoctozepto8: yep I seen, I don't know why for now | 16:42 |
| *** yoctozepto8 is now known as yoctozepto | 16:43 | |
| yoctozepto | hberaud: I took a quick look but nothing obvious | 16:44 |
| hberaud | yoctozepto: the main difference with the previous version is that I call make_region directly | 16:44 |
| hberaud | previous version = patch set 21 (your changes) | 16:44 |
| hberaud | yoctozepto: I'll try a mixin version between PS21 and PS22 | 16:45 |
| hberaud | yoctozepto: etcd3gw functional tests are similar to the PS22 for memcached functional tests | 16:46 |
| hberaud | yoctozepto: so I don't see why this is an issue with memcache | 16:46 |
| yoctozepto | hberaud: me neither; maybe etcd3gw's depend on something non-obvious which is not in memcached | 16:50 |
| yoctozepto | your guess is as good as mine and probably better | 16:50 |
| hberaud | we will see soon... | 16:51 |
| yoctozepto | hberaud: ideally we would have both tested by the same code, only flipping tests on/off for specific features, but none look here any specific | 16:57 |
| hberaud | yoctozepto: yep I agree I've some thoughts about that to just switch the backend ands run the same tests on them | 16:59 |
| hberaud | yoctozepto: normally we should expect that 1 test pass every backend | 16:59 |
| hberaud | I mean 1 testing module | 17:00 |
| yoctozepto | hberaud: yeah, I mean we can have separate jobs to have it clear but no need for separate modules to test backends if they are much the same (due to being used for the same purpose functionally-wise) | 17:09 |
| hberaud | yoctozepto: I agree | 17:09 |
| hberaud | yoctozepto: +1 for the separate jobs | 17:10 |
| *** jlvillal is now known as jlvillal_laptop | 17:10 | |
| *** jlvillal_laptop is now known as jlvill-travel | 17:10 | |
| *** jlvill-travel is now known as jlvacation | 17:11 | |
| *** jlvacation is now known as jlv-sick | 17:11 | |
| *** jlv-sick is now known as jlviva-viva | 17:11 | |
| *** jlviva-viva is now known as jlvillal | 17:11 | |
| tobias-urdin | hberaud: see my latest comment on https://bugs.launchpad.net/oslo.config/+bug/1869929 | 17:22 |
| openstack | Launchpad bug 1869929 in oslo.config "RuntimeError: maximum recursion depth exceeded while calling a Python object" [Undecided,Invalid] | 17:22 |
| tobias-urdin | i updated the reproducer as well to be more realistic, i don't think this is a bug and I cannot reproduce it on the node where i saw it anymore | 17:22 |
| * hberaud take a look thanks tobias-urdin | 17:22 | |
| hberaud | tobias-urdin: thanks for the heads up | 17:29 |
| hberaud | it's a weird thing... maybe SELinux blocked some backends or services | 17:29 |
| hberaud | tobias-urdin: it could explain why oslo.db failed to retrieve things | 17:30 |
| hberaud | anyway it's good to have successfully fix that | 17:31 |
| *** sean-k-mooney has quit IRC | 17:56 | |
| *** ralonsoh has quit IRC | 18:38 | |
| *** hemna has quit IRC | 18:59 | |
| *** hemna_ has joined #openstack-oslo | 19:08 | |
| *** hemna_ has quit IRC | 19:08 | |
| *** hemna_ has joined #openstack-oslo | 19:10 | |
| *** hemna_ has quit IRC | 19:11 | |
| beekneemech | kgiusti: Could you approve https://review.opendev.org/#/c/717310/ ? We need to get that in ASAP for non-client library freeze. | 19:29 |
| kgiusti | beekneemech: lemme take a look | 19:30 |
| kgiusti | kgiusti: simple revert - done! | 19:32 |
| smcginnis | Thanks! | 19:32 |
| kgiusti | :D | 19:33 |
| *** hemna has joined #openstack-oslo | 19:34 | |
| *** hemna has quit IRC | 19:37 | |
| openstackgerrit | Merged openstack/oslotest master: Revert "Switch to unittest.mock from mock" https://review.opendev.org/717310 | 19:41 |
| smcginnis | beekneemech: If you want to propose a release just for that, we can get that through asap. | 19:42 |
| smcginnis | Think it's worth a Friday release. | 19:42 |
| beekneemech | Thanks, kgiusti! | 19:43 |
| beekneemech | smcginnis: Yep, that was my plan. | 19:43 |
| smcginnis | +1 | 19:43 |
| beekneemech | Done: https://review.opendev.org/717376 | 19:44 |
| *** hberaud has quit IRC | 20:15 | |
| *** hberaud has joined #openstack-oslo | 20:18 | |
| *** raildo has quit IRC | 21:01 | |
| tobias-urdin | hberaud: yeah, it's weird because i can't find any good reasons in the audit log, maybe it's been cleaned | 21:37 |
| tobias-urdin | or maybe (haven't tested) it solves itself after the service has started the first time, but no idea why that would be | 21:37 |
| *** larainema has quit IRC | 22:25 | |
| *** tosky has quit IRC | 23:16 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!