| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2023-02-10 | |||
| 14:21:20 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Add share snapshot instance resource https://review.opendev.org/c/openstack/openstacksdk/+/873364 | |
| 18:46:31 | opendevreview | Ashley Rodriguez proposed openstack/openstacksdk master: Add share instances to shared file systems https://review.opendev.org/c/openstack/openstacksdk/+/873109 | |
| 18:49:07 | opendevreview | Ashley Rodriguez proposed openstack/openstacksdk master: Add share network resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/873363 | |
| #openstack-sdks - 2023-02-13 | |||
| 10:03:06 | opendevreview | Marvin Vogt proposed openstack/openstacksdk master: Remove usage of deprecated `sre_constants` module https://review.opendev.org/c/openstack/openstacksdk/+/873521 | |
| 10:07:29 | opendevreview | Marvin Vogt proposed openstack/openstacksdk master: Remove usage of deprecated `sre_constants` module https://review.opendev.org/c/openstack/openstacksdk/+/873521 | |
| 16:02:37 | whoami-rajat | stephenfin, hey, around ? | |
| 16:03:34 | whoami-rajat | stephenfin, wanted to mention that I've proposed patches for all missing cinder commands, would be good if we can get it in this cycle's OSC release (this week) https://review.opendev.org/q/topic:add-missing-cinderclient-commands | |
| 16:04:02 | stephenfin | yup, back today. I'll take a look (y) | |
| 20:12:38 | opendevreview | Ashley Rodriguez proposed openstack/openstacksdk master: Add share snapshot instance resource https://review.opendev.org/c/openstack/openstacksdk/+/798464 | |
| 20:15:23 | opendevreview | Ashley Rodriguez proposed openstack/openstacksdk master: Add share network subnet resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/783582 | |
| #openstack-sdks - 2023-02-14 | |||
| 02:35:58 | gmann | gtema: just in case you missed, next cycle PTL nomination are closing on Feb 15 | |
| 12:42:12 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Add support for glance cache https://review.opendev.org/c/openstack/openstacksdk/+/869710 | |
| 14:22:05 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM: check CI run https://review.opendev.org/c/openstack/ansible-collections-openstack/+/873713 | |
| 15:46:05 | stephenfin | whoami-rajat: Done. You'll need to ask gtema if you want a second +2/+W on the ones I've approved | |
| 15:46:27 | stephenfin | I think he also accepts 🍺 as part payment | |
| 15:46:32 | stephenfin | :) | |
| 16:42:35 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Remove duplication from 'consistency group create' opts https://review.opendev.org/c/openstack/python-openstackclient/+/873730 | |
| 19:36:57 | whoami-rajat | stephenfin, sure, thanks for all your reviews :) | |
| 19:53:58 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Convert recordset module to use StateMachine https://review.opendev.org/c/openstack/ansible-collections-openstack/+/873747 | |
| 20:55:20 | opendevreview | Merged openstack/openstacksdk master: Add support for glance cache https://review.opendev.org/c/openstack/openstacksdk/+/869710 | |
| #openstack-sdks - 2023-02-15 | |||
| 01:11:28 | opendevreview | Reynaldo proposed openstack/openstacksdk master: Add share snapshot instance resource https://review.opendev.org/c/openstack/openstacksdk/+/798464 | |
| 08:19:14 | frickler | gtema: could you add https://review.opendev.org/c/openstack/python-openstackclient/+/864497 to your review list please? | |
| 08:19:28 | gtema | ack | |
| 08:20:32 | gtema | frickler - we were talking with you about naming: "force" is a bad idea from my pov | |
| 08:24:13 | frickler | gtema: so what would you prefer instead? "--no-confirm"? | |
| 08:26:16 | gtema | yes, maybe, or --assume-yes or --auto-approve (like in TF) | |
| 08:27:20 | frickler | the other option would be to make the whole thing non-interactive and just have a "--dry-run" option for the timid | |
| 08:32:54 | gtema | hmm, this is too dangerous in my eyes | |
| 08:34:58 | frickler | o.k., so I'll amend to use --auto-approve | |
| 08:35:13 | gtema | cool, thanks | |
| 08:43:15 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Add auto-approve option to project cleanup https://review.opendev.org/c/openstack/python-openstackclient/+/864497 | |
| 09:10:57 | frickler | seems I totally missed 1.0.0, was there a good party? | |
| 10:27:10 | gtema | frickler - a party was sadly in mind only | |
| 10:56:19 | lajoskatona | gtema, frickler: just a note, sdk 1.0.0 is still not in use, as u-c bump is not merged: https://review.opendev.org/c/openstack/requirements/+/872282 | |
| 11:00:42 | frickler | lajoskatona: yes, though we actually have 1.0.1 now. but requirements CI is broken as well as kuryr like you mentioned | |
| 11:01:48 | frickler | you could try to depend your neutronclient patch on the u-c bump, though | |
| 11:03:59 | lajoskatona | frickler: yeah, but anyway we can't merge (and release) it without having correct u-c.txt | |
| 16:46:35 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Include "security_groups" to "Port" query parameters https://review.opendev.org/c/openstack/openstacksdk/+/873946 | |
| 16:51:22 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Include "security_groups" to "Port" query parameters https://review.opendev.org/c/openstack/openstacksdk/+/873946 | |
| 19:27:12 | opendevreview | Elod Illes proposed openstack/cliff master: DNM: dummy change to test gate health https://review.opendev.org/c/openstack/cliff/+/873979 | |
| 19:30:04 | opendevreview | Elod Illes proposed openstack/osc-lib master: DNM: dummy change to test gate health https://review.opendev.org/c/openstack/osc-lib/+/873986 | |
| #openstack-sdks - 2023-02-16 | |||
| 00:58:59 | opendevreview | Goutham Pacha Ravi proposed openstack/openstacksdk master: Remove "feature/r1" override from manila job https://review.opendev.org/c/openstack/openstacksdk/+/874046 | |
| 06:23:50 | opendevreview | Reynaldo proposed openstack/openstacksdk master: Add share snapshot instance resource https://review.opendev.org/c/openstack/openstacksdk/+/798464 | |
| 09:01:52 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add volume revert command https://review.opendev.org/c/openstack/python-openstackclient/+/869654 | |
| 09:01:53 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add block storage cleanup command https://review.opendev.org/c/openstack/python-openstackclient/+/872390 | |
| 09:01:53 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add block storage log level {list, set} commands https://review.opendev.org/c/openstack/python-openstackclient/+/869811 | |
| 09:01:54 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add options to create volume group from source https://review.opendev.org/c/openstack/python-openstackclient/+/872798 | |
| 09:01:54 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add block storage manageable list commands https://review.opendev.org/c/openstack/python-openstackclient/+/872692 | |
| 09:12:20 | whoami-rajat | stephenfin, gtema updated all the patches ^ | |
| 10:27:12 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: tox 4 compatibility https://review.opendev.org/c/openstack/osc-lib/+/874075 | |
| 11:00:45 | frickler | gtema: stephenfin: would one of you have time to look into https://zuul.opendev.org/t/openstack/build/284499a2edc041f78d6b94dbee1e1e07 ? this is blocking getting sdk 1.0.1 into u-c (not the only blocker, but kuryr at least has a fix proposed) | |
| 11:01:54 | gtema | frickler, ack, looking | |
| 11:02:50 | gtema | but I am not sure whether this is OSC issue or octaviaclient | |
| 11:30:41 | gtema | frickler - I think I found a direction. python-openstackclient is not installing python-octaviaclient through requirements.txt. It is done in doc/requirements.txt | |
| 11:31:31 | gtema | but I am not sure this is respected in the requirements job, cause according to the log python-octaviaclient is not visible in the packages installed into the tox venv | |
| 11:33:20 | gtema | at least it does not look like it is there and if I remove it locally in the docs venv of OSC I get the same error | |
| 12:05:58 | frickler | gtema: ah, so I'll check the job definition in the reqs repo, thx for looking | |
| 12:09:02 | gtema | frickler, since this was running before I assume this might be something new in toxv4 as well. Logs show octaviaclient is installed (into some env), but during the build it is just not there where expected | |
| 12:46:22 | frickler | yes, this is weird. the job is only failing for the sdk bump, not for other updates, so there must be some connection. let me try with a dep on osc | |
| 12:57:09 | frickler | I can reproduce the error locally if I manually update openstacksdk in the venv or point to the updated u-c file. but I have no idea why this is happening. also not related to tox4, can repro with tox3, too | |
| 12:58:51 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: DNM: Test sdk update https://review.opendev.org/c/openstack/python-openstackclient/+/874095 | |
| 13:01:22 | gtema | Weird | |
| 13:02:51 | gtema | What we had figured out for Octaviaclient is that in SDK 1.0 we dropped munch from dependencies | |
| 13:02:55 | gtema | I added it i to requirements there and it was merged | |
| 13:03:50 | frickler | ah, maybe the bump for octaviaclient needs to be merged first, then | |
| 13:04:23 | gtema | Maybe. But that is weird that in this case it causes such a hidden failure | |
| 13:05:13 | gtema | Can you try where you reporoduced the issue to manually install munch and check whether it works then or not | |
| 13:05:14 | gtema | It may be the pip issue and not tox | |
| 13:07:48 | frickler | I just had the same idea, with much installed into the docs venv, the issue goes away | |
| 13:08:33 | frickler | so we could add that to doc/reqs.txt in osc. or try to find out why the transitive requirement doesn't seem to work | |
| 13:09:03 | frickler | I'll wait for test results in my patch above and then try that | |
| 13:09:27 | frickler | s/much/munch/ above | |
| 15:23:43 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Add munch to doc requirements https://review.opendev.org/c/openstack/python-openstackclient/+/874095 | |
| 17:10:33 | opendevreview | Merged openstack/osc-lib master: tox 4 compatibility https://review.opendev.org/c/openstack/osc-lib/+/874075 | |
| 19:12:39 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: WIP: Follow-up for I87482a5dd43c519dfdcf981635aa879914a70a5c https://review.opendev.org/c/openstack/python-openstackclient/+/874151 | |
| #openstack-sdks - 2023-02-17 | |||
| 01:17:07 | opendevreview | Merged openstack/python-openstackclient master: Add volume revert command https://review.opendev.org/c/openstack/python-openstackclient/+/869654 | |
| 01:17:10 | opendevreview | Merged openstack/python-openstackclient master: Add block storage log level {list, set} commands https://review.opendev.org/c/openstack/python-openstackclient/+/869811 | |
| 12:19:06 | opendevreview | Merged openstack/os-api-ref master: Remove deprecated sphinx-testing dependency https://review.opendev.org/c/openstack/os-api-ref/+/860080 | |
| 12:22:00 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add block storage manageable list commands https://review.opendev.org/c/openstack/python-openstackclient/+/872692 | |
| 12:22:00 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add block storage cleanup command https://review.opendev.org/c/openstack/python-openstackclient/+/872390 | |
| 12:22:01 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: WIP: Follow-up for I87482a5dd43c519dfdcf981635aa879914a70a5c https://review.opendev.org/c/openstack/python-openstackclient/+/874151 | |
| 12:22:01 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add options to create volume group from source https://review.opendev.org/c/openstack/python-openstackclient/+/872798 | |
| 12:31:45 | opendevreview | Merged openstack/openstacksdk master: Remove "feature/r1" override from manila job https://review.opendev.org/c/openstack/openstacksdk/+/874046 | |
| 13:15:24 | opendevreview | Merged openstack/ansible-collections-openstack master: Fix issue with multiple records in recordset https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872392 | |
| 13:51:04 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Deprecate positional args for 'volume group create' https://review.opendev.org/c/openstack/python-openstackclient/+/874151 | |
| 21:04:06 | opendevreview | Alexander Fadeev proposed openstack/os-api-ref master: Fix the 'No such file or directory' error https://review.opendev.org/c/openstack/os-api-ref/+/874332 | |
| #openstack-sdks - 2023-02-18 | |||
| 22:00:27 | opendevreview | Samuel Loegering proposed openstack/openstacksdk master: Add share network resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/873363 | |
| #openstack-sdks - 2023-02-20 | |||
| 03:09:00 | opendevreview | Takashi Kajinami proposed openstack/os-client-config master: Remove ceilometerclient https://review.opendev.org/c/openstack/os-client-config/+/874366 | |
| 05:25:26 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Adding support for glance cache-queue Command https://review.opendev.org/c/openstack/openstacksdk/+/874372 | |
| 13:00:35 | whoami-rajat | stephenfin, gtema hey, I see OSC hasn't been released yet, I would really like to get my cinder commands into the 2023.1 cycle if possible :) https://review.opendev.org/q/topic:add-missing-cinderclient-commands | |
| 14:00:00 | opendevreview | Yadnesh Kulkarni proposed openstack/python-openstackclient master: [DNM] Test CI https://review.opendev.org/c/openstack/python-openstackclient/+/874422 | |
| 15:58:24 | opendevreview | Gaël THEROND proposed openstack/ansible-collections-openstack master: Add module to filter available volume services. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/874431 | |
| 15:58:24 | opendevreview | Gaël THEROND proposed openstack/ansible-collections-openstack master: Add module to filter available volume services. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/874431 | |
| 16:00:44 | opendevreview | Gaël THEROND proposed openstack/ansible-collections-openstack master: Add module to filter available volume services. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868480 | |
| 16:08:43 | opendevreview | Gaël THEROND proposed openstack/ansible-collections-openstack master: Add module to filter available volume services. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868480 | |
| 16:30:39 | opendevreview | Gaël THEROND proposed openstack/ansible-collections-openstack master: Add module to filter available volume services. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868480 | |
| 16:51:20 | opendevreview | Gaël THEROND proposed openstack/ansible-collections-openstack master: Add module to filter available volume services. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868480 | |
| 17:07:00 | whoami-rajat | gtema, thank you for the reviews :) | |
| 17:07:33 | gtema | welcome, once merges I will go on and submit release request | |
| 18:35:29 | opendevreview | Merged openstack/python-openstackclient master: Add block storage cleanup command https://review.opendev.org/c/openstack/python-openstackclient/+/872390 | |