| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-09-13 | |||
| 13:45:29 | opendevreview | Yoonho Hann proposed openstack/openstacksdk master: Add getting the status of the services in designate system https://review.opendev.org/c/openstack/openstacksdk/+/927673 | |
| 14:33:05 | gtema | frickler, you mean https://review.opendev.org/c/openstack/python-openstackclient/+/929236? | |
| 15:28:08 | frickler | gtema: yes, sean is verifying now with https://review.opendev.org/c/openstack/nova/+/929187 but I also tested locally | |
| 15:29:00 | frickler | gtema: then there's also the regression discovered by tkajinam, I didn't look into how to fix that yet and likely won't have time to in the near future | |
| 15:31:49 | tkajinam | so it seems migration to sdk made 'user create' require --domain... To "fix" it we may need to let osc/sdk to accept no domain id which sounds like a bit tricky | |
| 15:32:32 | tkajinam | if 'user create' without --domain was explicitly deprecated then we can document the change in a release note ? Though I've never seen any explicit message (by osc at least) | |
| 15:33:01 | tkajinam | gtema, this is the one https://bugs.launchpad.net/python-openstackclient/+bug/2080600 | |
| 15:46:23 | JayF | that is a very unfortunate behavior change | |
| 15:46:35 | JayF | there are huge numbers of openstack deployers who don't use domain, or even have a mental model of it | |
| 15:46:49 | JayF | I anticipate this being an extremely breaking change | |
| 16:05:11 | gtema | what relates to me I moved from the believe of sdk/cli being able to negotiate all corner cases (and OpenStack is full of those) saying: if the service is doing it this way - blame the service and not the sdk/cli team | |
| 16:12:32 | JayF | To be clear: my comment is not meant to point blame; just indicate from a high level this will likely be messy if released this way. | |
| 16:19:44 | gtema | I do get it JayF, it is just extremely frustrating that people want improvements and extensions on one side and immediately crying on any logical interface change. It is impossible to maintain cli/sdk reasonably. | |
| 16:21:05 | JayF | Yes, I entirely understand. When we wrote the recent Ironic security fix, we specifically ensured to make a path for people to be secured even if they didn't update IPA | |
| 17:08:09 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Add status filtering options to port list https://review.opendev.org/c/openstack/python-openstackclient/+/444810 | |
| 17:09:32 | opendevreview | Merged openstack/keystoneauth master: Drop support for Python 3.8 https://review.opendev.org/c/openstack/keystoneauth/+/928833 | |
| 17:21:02 | opendevreview | Merged openstack/cliff master: Drop support for Python 3.8, add Python 3.12 https://review.opendev.org/c/openstack/cliff/+/929224 | |
| 17:35:51 | opendevreview | Merged openstack/python-openstackclient master: evacuate: Fix password parameter name for SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929236 | |
| 17:35:53 | opendevreview | Merged openstack/python-openstackclient master: pre-commit: Migrate bandit to ruff https://review.opendev.org/c/openstack/python-openstackclient/+/929163 | |
| 17:35:55 | opendevreview | Merged openstack/python-openstackclient master: pre-commit: Migrate pyupgrade to ruff https://review.opendev.org/c/openstack/python-openstackclient/+/929164 | |
| 18:20:43 | opendevreview | Merged openstack/openstacksdk master: Allow to override _max_microversion for Volume._action function https://review.opendev.org/c/openstack/openstacksdk/+/924206 | |
| 18:58:52 | opendevreview | Merged openstack/python-openstackclient master: Handle NotFoundException when listing floating IPs https://review.opendev.org/c/openstack/python-openstackclient/+/694571 | |
| #openstack-sdks - 2024-09-14 | |||
| 07:38:37 | opendevreview | CHOIDONGHAE proposed openstack/openstacksdk master: Add designate quotas https://review.opendev.org/c/openstack/openstacksdk/+/927827 | |
| 08:08:48 | opendevreview | CHOIDONGHAE proposed openstack/openstacksdk master: Add designate quotas https://review.opendev.org/c/openstack/openstacksdk/+/927827 | |
| 09:22:02 | opendevreview | CHOIDONGHAE proposed openstack/openstacksdk master: Add designate quotas https://review.opendev.org/c/openstack/openstacksdk/+/927827 | |
| #openstack-sdks - 2024-09-16 | |||
| 07:37:14 | opendevreview | Elod Illes proposed openstack/python-openstackclient stable/2024.2: evacuate: Fix password parameter name for SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929402 | |
| 08:12:12 | opendevreview | Merged openstack/python-openstackclient stable/2024.2: evacuate: Fix password parameter name for SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929402 | |
| 09:09:31 | opendevreview | Merged openstack/openstacksdk master: compute: Add support for targeting host during migrate https://review.opendev.org/c/openstack/openstacksdk/+/920948 | |
| 09:11:09 | tobias-urdin | can we merge this bugfix https://review.opendev.org/c/openstack/openstacksdk/+/928867 | |
| 11:28:04 | stephenfin | tobias-urdin: done | |
| 11:31:04 | tobias-urdin | stephenfin: \o/ thanks | |
| 11:51:40 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: identity: Don't pass unset options when creating user https://review.opendev.org/c/openstack/python-openstackclient/+/929456 | |
| 11:52:17 | stephenfin | frickler: tkajinam: That's the fix for https://bugs.launchpad.net/python-openstackclient/+bug/2080600 | |
| 11:52:24 | stephenfin | gtema too, I guess ^ | |
| 11:53:12 | opendevreview | Merged openstack/openstacksdk master: Fix volume summary passing non-existent all_projects https://review.opendev.org/c/openstack/openstacksdk/+/928867 | |
| 12:25:02 | elodilles | gtema: hi, if there are no other things that needs to be in the new release of python-openstackclient, than can you please review this one? https://review.opendev.org/c/openstack/releases/+/929454 | |
| 14:53:27 | cardoe | Ironic could definitely use the runbooks feature to land. | |
| 14:53:47 | cardoe | https://review.opendev.org/c/openstack/openstacksdk/+/926414 | |
| 15:59:51 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: Fix internal error when floating ip netwokr is not found by name https://review.opendev.org/c/openstack/openstacksdk/+/929548 | |
| 16:07:46 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: Fix internal error when floating ip network is not found by name https://review.opendev.org/c/openstack/openstacksdk/+/929548 | |
| 16:15:08 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: identity: Don't pass unset options when creating user https://review.opendev.org/c/openstack/python-openstackclient/+/929456 | |
| 16:19:53 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Add missing ignore_missing call https://review.opendev.org/c/openstack/openstacksdk/+/929553 | |
| 16:19:53 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Consistently set 'ignore_missing' arguments https://review.opendev.org/c/openstack/openstacksdk/+/929554 | |
| 16:19:54 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Stop reimplementing ignore_missing=False https://review.opendev.org/c/openstack/openstacksdk/+/929555 | |
| 16:22:24 | stephenfin | tkajinam: Thanks for https://review.opendev.org/c/openstack/openstacksdk/+/929548. That's going to fail pep8. You need to run pre-commit/the pep8 tox target | |
| 17:08:04 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Add helpers to create, delete temp projects https://review.opendev.org/c/openstack/openstacksdk/+/929560 | |
| 17:08:05 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Migrate quota tests to use a new project https://review.opendev.org/c/openstack/openstacksdk/+/929561 | |
| 22:08:55 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support assigning inherited roles https://review.opendev.org/c/openstack/openstacksdk/+/929578 | |
| 22:09:19 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support assigning inherited roles https://review.opendev.org/c/openstack/openstacksdk/+/929578 | |
| 22:12:53 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support assigning inherited roles https://review.opendev.org/c/openstack/openstacksdk/+/929578 | |
| 22:16:32 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support `options` property for roles https://review.opendev.org/c/openstack/openstacksdk/+/921411 | |
| #openstack-sdks - 2024-09-17 | |||
| 00:52:55 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: Fix internal error when floating ip netwokr is not found by name https://review.opendev.org/c/openstack/openstacksdk/+/929548 | |
| 01:25:33 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: Fix internal error when floating ip network is not found by name https://review.opendev.org/c/openstack/openstacksdk/+/929548 | |
| 09:29:54 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Use all resource names for path param naming https://review.opendev.org/c/openstack/codegenerator/+/929614 | |
| 09:35:21 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Consistently set 'ignore_missing' arguments https://review.opendev.org/c/openstack/openstacksdk/+/929554 | |
| 09:35:34 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Stop reimplementing ignore_missing=False https://review.opendev.org/c/openstack/openstacksdk/+/929555 | |
| 10:41:22 | opendevreview | Merged openstack/openstacksdk master: Fix internal error when floating ip network is not found by name https://review.opendev.org/c/openstack/openstacksdk/+/929548 | |
| 10:44:17 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Add functional tests for block storage quotas https://review.opendev.org/c/openstack/openstacksdk/+/929619 | |
| 10:52:56 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Add functional test for access rules https://review.opendev.org/c/openstack/python-openstackclient/+/923446 | |
| 10:52:56 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: identity: Migrate 'access rule' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/921882 | |
| 11:11:02 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: WIP: Switch to use of 'Command.log' https://review.opendev.org/c/openstack/python-openstackclient/+/860478 | |
| 11:25:38 | stephenfin | frickler: https://review.opendev.org/c/openstack/python-openstackclient/+/929456 is ready for your re-review. We probably want to ship a 7.1.2 with that in it | |
| 11:25:43 | stephenfin | gtema too ^ | |
| 11:56:28 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Add status filtering options to port list https://review.opendev.org/c/openstack/python-openstackclient/+/444810 | |
| 12:01:25 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Use all resource names for path param naming https://review.opendev.org/c/openstack/codegenerator/+/929614 | |
| 12:28:04 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: identity: Don't pass unset options when creating user https://review.opendev.org/c/openstack/python-openstackclient/+/929456 | |
| 13:05:05 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Use all resource names for path param naming https://review.opendev.org/c/openstack/codegenerator/+/929614 | |
| 13:13:56 | opendevreview | Merged openstack/python-openstackclient master: tests: Add functional test for access rules https://review.opendev.org/c/openstack/python-openstackclient/+/923446 | |
| 13:15:06 | opendevreview | Merged openstack/python-openstackclient master: identity: Migrate 'access rule' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/921882 | |
| 13:15:28 | opendevreview | Josephine Seifert proposed openstack/python-openstackclient master: WIP: Add admin-editable metadata field to volume types https://review.opendev.org/c/openstack/python-openstackclient/+/928132 | |
| 13:26:47 | opendevreview | Merged openstack/python-openstackclient master: compute: Allow adding, removing multiple SGs https://review.opendev.org/c/openstack/python-openstackclient/+/894529 | |
| 14:13:21 | opendevreview | Yoonho Hann proposed openstack/openstacksdk master: Add getting the status of the services in designate system https://review.opendev.org/c/openstack/openstacksdk/+/927673 | |
| 14:49:10 | opendevreview | Merged openstack/openstacksdk master: Followup to I0e5ac1288c0d0423fa3a7a4e63173675b78aae79 https://review.opendev.org/c/openstack/openstacksdk/+/929090 | |
| 16:18:09 | opendevreview | Merged openstack/codegenerator master: Use all resource names for path param naming https://review.opendev.org/c/openstack/codegenerator/+/929614 | |
| 16:22:33 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Switch to ruff https://review.opendev.org/c/openstack/codegenerator/+/929683 | |
| 17:20:49 | opendevreview | Merged openstack/cliff master: Migrate to ruff, ruff-format https://review.opendev.org/c/openstack/cliff/+/929157 | |
| 17:20:50 | opendevreview | Merged openstack/cliff master: ruff: Enable pyupgrade rules https://review.opendev.org/c/openstack/cliff/+/929158 | |
| 18:10:39 | opendevreview | Merged openstack/codegenerator master: Switch to ruff https://review.opendev.org/c/openstack/codegenerator/+/929683 | |
| 18:12:55 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Extend user path param with `--current-user` https://review.opendev.org/c/openstack/codegenerator/+/929692 | |
| 18:19:56 | opendevreview | Merged openstack/cliff master: Remove unnecessary shebangs https://review.opendev.org/c/openstack/cliff/+/929159 | |
| 19:44:19 | opendevreview | Merged openstack/codegenerator master: Extend user path param with `--current-user` https://review.opendev.org/c/openstack/codegenerator/+/929692 | |
| 20:24:34 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support `options` property for roles https://review.opendev.org/c/openstack/openstacksdk/+/921411 | |
| 21:03:32 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support `options` property for roles https://review.opendev.org/c/openstack/openstacksdk/+/921411 | |
| 21:15:22 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support `options` property for roles https://review.opendev.org/c/openstack/openstacksdk/+/921411 | |
| 21:15:35 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support `options` property for roles https://review.opendev.org/c/openstack/openstacksdk/+/921411 | |
| 21:42:37 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support assigning inherited roles https://review.opendev.org/c/openstack/openstacksdk/+/929578 | |
| 21:42:50 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support assigning inherited roles https://review.opendev.org/c/openstack/openstacksdk/+/929578 | |
| 22:03:01 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support assigning inherited roles https://review.opendev.org/c/openstack/openstacksdk/+/929578 | |
| 22:07:16 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support assigning inherited roles https://review.opendev.org/c/openstack/openstacksdk/+/929578 | |
| 22:15:48 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support assigning inherited roles https://review.opendev.org/c/openstack/openstacksdk/+/929578 | |
| 23:00:19 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'role' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929712 | |
| 23:00:44 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'role' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929712 | |
| 23:01:53 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'role' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929712 | |
| #openstack-sdks - 2024-09-18 | |||
| 06:47:39 | opendevreview | Sylvain Bauza proposed openstack/python-openstackclient master: evacuate SDK actually uses admin_pass param https://review.opendev.org/c/openstack/python-openstackclient/+/929726 | |
| 08:56:57 | opendevreview | Merged openstack/python-openstackclient master: evacuate SDK actually uses admin_pass param https://review.opendev.org/c/openstack/python-openstackclient/+/929726 | |
| 08:57:22 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient stable/2024.2: evacuate SDK actually uses admin_pass param https://review.opendev.org/c/openstack/python-openstackclient/+/929740 | |
| 09:02:02 | opendevreview | Stephen Finucane proposed openstack/openstacksdk stable/2024.2: Fix volume summary passing non-existent all_projects https://review.opendev.org/c/openstack/openstacksdk/+/929741 | |
| 09:02:02 | opendevreview | Stephen Finucane proposed openstack/openstacksdk stable/2024.2: compute: Add support for targeting host during migrate https://review.opendev.org/c/openstack/openstacksdk/+/929742 | |
| 09:08:09 | stephenfin | gtema: we'll need https://review.opendev.org/c/openstack/python-openstackclient/+/929740 also, unfortunately | |
| 09:08:26 | stephenfin | the sooner I get mypy running across everything the better... | |