| opendevreview | Anvi Joshi proposed openstack/openstacksdk master: Support manage/unmanage shares with manila https://review.opendev.org/c/openstack/openstacksdk/+/888940 | 04:26 |
|---|---|---|
| fungi | gtema: and now https://review.opendev.org/890366 and https://review.opendev.org/890374 | 04:43 |
| frickler | gtema: do we maybe want to unify sdk and osc gerrit groups? or is there a reason to keep them separate? that would make my comment about adding designate cores to osc obsolete | 05:37 |
| gtema | fricker: who knows what future brings us. Maybe we will have people that can be made cores in SDK, but not in osc. But I also don't really object of unifying them. This is just historically this way as complete independent projects | 05:41 |
| gtema | Feel free to also propose merge | 05:41 |
| frickler | hmm, no, I don't want to push something you're not convinced about, let's keep the current setup then for now | 06:49 |
| johnsom | I added Designate today as I had core previously under the Monty/Dean (for Octavia primarily though) plan of yesteryear. If we want to get more granular that is ok too. | 07:41 |
| opendevreview | Omer Schwartz proposed openstack/openstacksdk master: Add support to Octavia (load_balancer) amphora get stats https://review.opendev.org/c/openstack/openstacksdk/+/889296 | 08:23 |
| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: fix block storage resource registry https://review.opendev.org/c/openstack/openstacksdk/+/890398 | 08:36 |
| opendevreview | Merged openstack/openstacksdk master: cloud: Remove '_is_client_version' https://review.opendev.org/c/openstack/openstacksdk/+/884496 | 10:00 |
| opendevreview | Merged openstack/openstacksdk master: cloud: Remove '_{service}_client' properties https://review.opendev.org/c/openstack/openstacksdk/+/884497 | 10:00 |
| opendevreview | Merged openstack/openstacksdk master: cloud: Remove '_object_store_client' https://review.opendev.org/c/openstack/openstacksdk/+/884499 | 10:00 |
| fungi | a downside to these acl addition changes is they're going to need to be strictly sequenced or require a lot of rebases to deal with the merge conflicts between one another | 10:48 |
| fungi | besides the fact that it needs a project-config core reviewer to review and approve, while doing it via group management instead of acls would be under the full control of the sdks team | 10:49 |
| frickler | once this is set up, I don't expect a lot of changes | 11:11 |
| opendevreview | Christian Rohmann proposed openstack/openstacksdk master: Cleanup logic to either dry-run, bulk_delete or do single object deletes https://review.opendev.org/c/openstack/openstacksdk/+/890410 | 12:20 |
| opendevreview | Christian Rohmann proposed openstack/openstacksdk master: Cleanup logic to either dry-run, bulk_delete or do single object deletes https://review.opendev.org/c/openstack/openstacksdk/+/890410 | 14:20 |
| frickler | johnsom: no, let's go on with this, you just need to rebase the patch for designate now | 14:37 |
| johnsom | Yep, just starting my day, so give me a minute. | 14:38 |
| frickler | johnsom: completely unrelated you may be able to help oschwart_ with the CI for amphora stats? https://review.opendev.org/c/openstack/openstacksdk/+/889296 | 14:38 |
| johnsom | frickler Yeah, I can take a look | 14:39 |
| opendevreview | Christian Rohmann proposed openstack/python-openstackclient master: Add is_incremental to ListVolumeBackup https://review.opendev.org/c/openstack/python-openstackclient/+/890426 | 14:40 |
| johnsom | Rebase posted | 14:44 |
| frickler | stephenfin: gtema: I've done some testing on the traits issue, but didn't make real progress. do we want to skip the test for now? | 15:13 |
| frickler | seems this is affecting multiple projects, at least devstack and neutron | 15:13 |
| stephenfin | yeah, I think that makes sense now | 15:14 |
| gtema | agreed | 15:15 |
| frickler | if it helps, I can reproduce the issue with only creating traits in the test, but not when using OSC. I've also done tracing of the uwsgi socket, but that looks identical for passing and failing attempts afaict | 15:15 |
| frickler | so very likely some apache proxy issue, but even with debug log level no clue so far either | 15:16 |
| frickler | my gut feeling is that it might somehow be related to the create call not returning a body, but that's just guessing | 15:17 |
| gtema | that means to me the issue is most likely happening also for everybody deploying it productively | 15:18 |
| frickler | gtema: well I still want to test in a kolla env where haproxy is used as frontend instead apache | 15:18 |
| frickler | it would also be interesting to see how far stable branches are affected | 15:19 |
| gtema | I recently faced the issue that gzip lib since recently became more strict and is rejecting certain web response if it doesn't end the stream properly. I don't mean it is related, but just that some weird things are happening all over the stack | 15:19 |
| frickler | stephenfin: can you do a patch to skip the test, then, or shall I? | 15:19 |
| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: skip placement test https://review.opendev.org/c/openstack/openstacksdk/+/890431 | 15:23 |
| gtema | frickler - here you go | 15:23 |
| *** oschwart_ is now known as oschwart | 15:27 | |
| oschwart | frickler johnsom: thanks for any help, meanwhile I am trying to understand why I am getting that error in https://review.opendev.org/c/openstack/openstacksdk/+/889296 | 15:28 |
| frickler | seems we just made an sdk release with that failing test in place? guess that's bad timing | 15:49 |
| frickler | looks like the failure happens at least back in xena, too https://zuul.opendev.org/t/openstack/build/c016418972cb44bc86def85484b6aa81 but I didn't manage to see any failure on my production zed kolla cloud so far | 16:01 |
| frickler | btw. my repro was on bookworm, so it is also not ubuntu specific | 16:01 |
| frickler | also maybe I should continue this in #-nova actually | 16:02 |
| gtema | frickler - thanks for trying to debug the issue | 16:11 |
| opendevreview | Lajos Katona proposed openstack/openstacksdk master: Add SFC to SDK https://review.opendev.org/c/openstack/openstacksdk/+/887387 | 16:32 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Stop randomly sorting error lines https://review.opendev.org/c/openstack/openstacksdk/+/890441 | 16:47 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Skip intermittently failing placement traits test https://review.opendev.org/c/openstack/openstacksdk/+/890444 | 17:13 |
| stephenfin | frickler: ^ | 17:13 |
| stephenfin | Whoops, I see gtema beat me to it | 17:13 |
| stephenfin | though mine passes pep8 and has a bug linked so I think it's preferable? | 17:14 |
| gtema | No problem | 17:18 |
| opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Resource locks and access rules restrictions https://review.opendev.org/c/openstack/openstacksdk/+/889519 | 19:46 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!