| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-09-20 | |||
| 06:47:55 | tobias-urdin | stephenfin: the patch to openstacksdk linked in this patch looks to have broken backward compat but it looks like it were suppose not to? or am i missing something https://review.opendev.org/c/openstack/ansible-collections-openstack/+/926525/5 | |
| 08:07:54 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Add method to create network resource tags https://review.opendev.org/c/openstack/openstacksdk/+/927779 | |
| 08:19:55 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Add the trunk subports informatrion to the port list command https://review.opendev.org/c/openstack/python-openstackclient/+/926611 | |
| 08:20:13 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Add the trunk subports information to the port list command https://review.opendev.org/c/openstack/python-openstackclient/+/926611 | |
| 08:36:45 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add support for default volume types https://review.opendev.org/c/openstack/openstacksdk/+/920464 | |
| 08:51:46 | opendevreview | Merged openstack/openstacksdk master: tests: Add helpers to create, delete temp projects https://review.opendev.org/c/openstack/openstacksdk/+/929560 | |
| 09:55:54 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add support for default volume types https://review.opendev.org/c/openstack/openstacksdk/+/920464 | |
| 10:13:24 | opendevreview | Stephen Finucane proposed openstack/openstacksdk stable/2024.2: Fix internal error when floating ip network is not found by name https://review.opendev.org/c/openstack/openstacksdk/+/930014 | |
| 10:18:27 | stephenfin | tobias-urdin: That previous behaviour should be deprecated but still functional. Looking | |
| 10:23:35 | stephenfin | tobias-urdin: Looks okay to me. Have I missed something? https://paste.openstack.org/show/btrOktqwqbGqc9LVZhgJ/ | |
| 10:24:16 | stephenfin | You get a warning if you pass a QuotaSet argument as the first argument - as expected - but it still appears to do the right thing? | |
| 10:29:41 | stephenfin | tobias-urdin: Yeah, see my comment on the bug. The bug pre-dates that patch. | |
| 10:30:02 | stephenfin | s/on the bug/on the ansible change/ | |
| 11:09:12 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute, volume: Tweak 'update_quota_set' compat shim https://review.opendev.org/c/openstack/openstacksdk/+/930015 | |
| 11:09:39 | stephenfin | tobias-urdin: I don't think this is the cause of the bug (owing to the dates), but ^ | |
| 12:50:31 | tobias-urdin | stephenfin: sorry for late reply, thanks for having a look! | |
| 16:11:27 | opendevreview | Merged openstack/keystoneauth master: Apply ruff, ruff-format https://review.opendev.org/c/openstack/keystoneauth/+/928805 | |
| 16:21:32 | opendevreview | Merged openstack/keystoneauth master: Enable ruff, ruff-format https://review.opendev.org/c/openstack/keystoneauth/+/928806 | |
| 17:02:13 | opendevreview | Merged openstack/openstacksdk master: tests: Migrate quota tests to use a new project https://review.opendev.org/c/openstack/openstacksdk/+/929561 | |
| 20:54:08 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support assigning inherited roles https://review.opendev.org/c/openstack/openstacksdk/+/929578 | |
| #openstack-sdks - 2024-09-21 | |||
| 17:53:32 | opendevreview | Chanho Jung proposed openstack/openstacksdk master: Add support for TLD operations https://review.opendev.org/c/openstack/openstacksdk/+/927656 | |
| 18:06:52 | opendevreview | Chanho Jung proposed openstack/openstacksdk master: Add support for TLD operations https://review.opendev.org/c/openstack/openstacksdk/+/927656 | |
| #openstack-sdks - 2024-09-22 | |||
| 03:06:23 | opendevreview | Chanho Jung proposed openstack/openstacksdk master: Add support for TLD operations https://review.opendev.org/c/openstack/openstacksdk/+/927656 | |
| 03:08:56 | opendevreview | Chanho Jung proposed openstack/openstacksdk master: Add support for TLD operations https://review.opendev.org/c/openstack/openstacksdk/+/927656 | |
| 03:13:06 | opendevreview | Chanho Jung proposed openstack/openstacksdk master: Add support for TLD operations https://review.opendev.org/c/openstack/openstacksdk/+/927656 | |
| 03:22:58 | opendevreview | Chanho Jung proposed openstack/openstacksdk master: Add support for TLD operations https://review.opendev.org/c/openstack/openstacksdk/+/927656 | |
| 03:24:35 | opendevreview | Chanho Jung proposed openstack/openstacksdk master: Add support for TLD operations https://review.opendev.org/c/openstack/openstacksdk/+/927656 | |
| 07:16:12 | opendevreview | Jun Woo Park proposed openstack/openstacksdk master: Add getting info about the Limit in a DNS SDK. https://review.opendev.org/c/openstack/openstacksdk/+/927854 | |
| 07:27:10 | opendevreview | Jun Woo Park proposed openstack/openstacksdk master: Add getting info about the Limit in a DNS SDK. https://review.opendev.org/c/openstack/openstacksdk/+/927854 | |
| #openstack-sdks - 2024-09-23 | |||
| 07:43:12 | opendevreview | Merged openstack/openstacksdk master: tests: Add functional tests for block storage quotas https://review.opendev.org/c/openstack/openstacksdk/+/929619 | |
| 16:04:23 | opendevreview | Pierre Riteau proposed openstack/ansible-collections-openstack master: Fix typo in parameter description https://review.opendev.org/c/openstack/ansible-collections-openstack/+/930221 | |
| 17:23:36 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'service provider' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929961 | |
| 20:13:26 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'role' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929712 | |
| 20:46:22 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'role' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929712 | |
| 23:18:55 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support assigning inherited roles https://review.opendev.org/c/openstack/openstacksdk/+/929578 | |
| 23:21:32 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support assigning inherited roles https://review.opendev.org/c/openstack/openstacksdk/+/929578 | |
| 23:36:14 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support assigning inherited roles https://review.opendev.org/c/openstack/openstacksdk/+/929578 | |
| 23:37:35 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support assigning inherited roles https://review.opendev.org/c/openstack/openstacksdk/+/929578 | |
| #openstack-sdks - 2024-09-24 | |||
| 04:28:41 | whoami-rajat | hey stephenfin , can you take another look at https://review.opendev.org/c/openstack/openstacksdk/+/920464 ? | |
| 06:49:03 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Adopt `uv` instead of pip https://review.opendev.org/c/openstack/codegenerator/+/929809 | |
| 07:34:22 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Adopt `uv` instead of pip https://review.opendev.org/c/openstack/codegenerator/+/929809 | |
| 07:34:22 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Update pre-commit tools versions https://review.opendev.org/c/openstack/codegenerator/+/930263 | |
| 08:38:15 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Update pre-commit tools versions https://review.opendev.org/c/openstack/codegenerator/+/930263 | |
| 11:44:21 | opendevreview | Merged openstack/codegenerator master: Update pre-commit tools versions https://review.opendev.org/c/openstack/codegenerator/+/930263 | |
| 14:15:43 | opendevreview | CHOIDONGHAE proposed openstack/openstacksdk master: Add designate quotas https://review.opendev.org/c/openstack/openstacksdk/+/927827 | |
| 14:25:52 | opendevreview | CHOIDONGHAE proposed openstack/openstacksdk master: Add designate quotas https://review.opendev.org/c/openstack/openstacksdk/+/927827 | |
| 14:28:59 | opendevreview | CHOIDONGHAE proposed openstack/openstacksdk master: Add designate quotas https://review.opendev.org/c/openstack/openstacksdk/+/927827 | |
| 14:44:01 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: pre-commit: Bump versions https://review.opendev.org/c/openstack/openstacksdk/+/930344 | |
| 15:28:54 | opendevreview | Merged openstack/openstacksdk master: cloud: Re-add support for passing objects https://review.opendev.org/c/openstack/openstacksdk/+/929010 | |
| 15:30:53 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support assigning inherited roles https://review.opendev.org/c/openstack/openstacksdk/+/929578 | |
| 15:31:19 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'service provider' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929961 | |
| 15:56:00 | opendevreview | KyeongNa Park proposed openstack/openstacksdk master: Add create, delete, find, list and UPDATE example connect code https://review.opendev.org/c/openstack/openstacksdk/+/928129 | |
| 16:06:22 | opendevreview | CHOIDONGHAE proposed openstack/openstacksdk master: Add designate quotas https://review.opendev.org/c/openstack/openstacksdk/+/927827 | |
| 16:42:57 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'credential' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/923674 | |
| 17:10:53 | stephenfin | gtema: btw, this is the cause of that cinder failure you spotted earlier https://review.opendev.org/c/openstack/python-openstackclient/+/929456 We'll need to backport that | |
| 17:12:45 | gtema | Cool | |
| 17:15:49 | frickler | which cinder failure? that should only affect 2024.2? but I admit that I lost track of this a bit, had hoped it would have been in 7.1.2 already. but yes, needs backport to 2024.2 for sure | |
| 17:18:09 | stephenfin | frickler: search for "failed validation" here https://60b694a5d64202b1898d-7363a31a9bfd26aa77ca998af3343ff8.ssl.cf2.rackcdn.com/929961/3/check/osc-functional-devstack/f9a7c0b/job-output.txt | |
| 17:18:26 | stephenfin | apparently we're not using that user because jobs would be failing otherwise | |
| 17:28:34 | opendevreview | KyeongNa Park proposed openstack/openstacksdk master: Add create, delete, find, list and UPDATE example connect code https://review.opendev.org/c/openstack/openstacksdk/+/928129 | |
| 17:47:02 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: cloud: Consistently set 'ignore_missing' arguments https://review.opendev.org/c/openstack/openstacksdk/+/929554 | |
| 17:52:23 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: cloud: Stop reimplementing ignore_missing=False https://review.opendev.org/c/openstack/openstacksdk/+/929555 | |
| 18:03:18 | opendevreview | Merged openstack/openstacksdk master: cloud: Deprecate passing dicts https://review.opendev.org/c/openstack/openstacksdk/+/929011 | |
| 18:03:20 | opendevreview | Merged openstack/openstacksdk master: Identity: Support `options` property for roles https://review.opendev.org/c/openstack/openstacksdk/+/921411 | |
| 18:03:22 | opendevreview | Merged openstack/openstacksdk stable/2024.2: Fix volume summary passing non-existent all_projects https://review.opendev.org/c/openstack/openstacksdk/+/929741 | |
| 18:03:23 | opendevreview | Merged openstack/openstacksdk stable/2024.2: compute: Add support for targeting host during migrate https://review.opendev.org/c/openstack/openstacksdk/+/929742 | |
| 18:03:34 | opendevreview | Merged openstack/openstacksdk master: Add timeline for removal of deprecated features https://review.opendev.org/c/openstack/openstacksdk/+/929015 | |
| 18:03:37 | opendevreview | Merged openstack/openstacksdk master: stack event loops: handle get None result https://review.opendev.org/c/openstack/openstacksdk/+/927587 | |
| 18:06:19 | frickler | stephenfin: interesting, why does devstack create a cinder user if it isn't needed? also strange that cinder doesn't complain on "cinder_internal_tenant_user_id =" | |
| 20:23:58 | opendevreview | Oria Weng proposed openstack/openstacksdk master: Identity: Support assigning inherited roles https://review.opendev.org/c/openstack/openstacksdk/+/929578 | |
| 21:03:25 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'role' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929712 | |
| 21:05:33 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'role' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929712 | |
| 23:03:13 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'role' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929712 | |
| 23:40:17 | 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-25 | |||
| 04:56:16 | frickler | looks like this is largely untested https://codesearch.openstack.org/?q=CINDER_UPLOAD_INTERNAL_TENANT&i=nope&literal=nope&files=&excludeFiles=&repos= . also if one were to test it, it should use a dedicated project and not the shared "service" one IMO | |
| 08:22:27 | opendevreview | Merged openstack/python-openstackclient master: identity: Don't pass unset options when creating user https://review.opendev.org/c/openstack/python-openstackclient/+/929456 | |
| 08:24:13 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient stable/2024.2: identity: Don't pass unset options when creating user https://review.opendev.org/c/openstack/python-openstackclient/+/930412 | |
| 13:47:41 | opendevreview | Merged openstack/openstacksdk stable/2024.2: Fix internal error when floating ip network is not found by name https://review.opendev.org/c/openstack/openstacksdk/+/930014 | |
| 14:21:46 | opendevreview | 이현규 proposed openstack/openstacksdk master: ADD DNS Zone Blacklist CRUD https://review.opendev.org/c/openstack/openstacksdk/+/927824 | |
| 14:22:53 | opendevreview | 이현규 proposed openstack/openstacksdk master: ADD DNS Zone Blacklist CRUD https://review.opendev.org/c/openstack/openstacksdk/+/927824 | |
| 16:22:52 | opendevreview | KyeongNa Park proposed openstack/openstacksdk master: Add create, delete, find, list and UPDATE example connect code https://review.opendev.org/c/openstack/openstacksdk/+/928129 | |
| #openstack-sdks - 2024-09-26 | |||
| 01:37:38 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'role' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929712 | |
| 11:48:36 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: fix network.agent property types https://review.opendev.org/c/openstack/codegenerator/+/930561 | |
| 12:02:29 | opendevreview | Rajesh Tailor proposed openstack/openstacksdk master: Add support for showing finish_time https://review.opendev.org/c/openstack/openstacksdk/+/930562 | |
| 17:16:38 | opendevreview | Merged openstack/openstacksdk master: cloud: Consistently set 'ignore_missing' arguments https://review.opendev.org/c/openstack/openstacksdk/+/929554 | |
| 17:42:53 | opendevreview | Merged openstack/codegenerator master: fix network.agent property types https://review.opendev.org/c/openstack/codegenerator/+/930561 | |
| #openstack-sdks - 2024-09-27 | |||
| 11:15:46 | tkajinam | stephenfin gtema , I'm hesitant to bother you again but I'm afraid we found another regression of osc https://bugs.launchpad.net/python-openstackclient/+bug/2083021 | |
| 11:16:41 | stephenfin | ugh, yeah, I'm thinking we'll have to go over all those identity changes for this class of bugs. A very annoying API choice :( | |
| 11:16:47 | stephenfin | tkajinam: I'll take a look today | |
| 11:20:43 | tkajinam | stephenfin, thanks ! | |
| 15:26:53 | opendevreview | Takashi Kajinami proposed openstack/requestsexceptions master: Convert CRLF to LF https://review.opendev.org/c/openstack/requestsexceptions/+/930733 | |
| 15:27:58 | opendevreview | Takashi Kajinami proposed openstack/requestsexceptions master: Convert CRLF to LF https://review.opendev.org/c/openstack/requestsexceptions/+/930733 | |
| 16:17:09 | opendevreview | Merged openstack/keystoneauth master: typing: Make abstract method actually abstract https://review.opendev.org/c/openstack/keystoneauth/+/928807 | |
| 16:17:35 | opendevreview | Merged openstack/keystoneauth master: typing: Synchronise overridden method signatures https://review.opendev.org/c/openstack/keystoneauth/+/928808 | |
| 16:21:31 | opendevreview | Merged openstack/keystoneauth master: typing: Add variable for auth plugin opts https://review.opendev.org/c/openstack/keystoneauth/+/928809 | |
| 16:21:32 | opendevreview | Merged openstack/keystoneauth master: typing: Resolve remaining initial issues https://review.opendev.org/c/openstack/keystoneauth/+/928810 | |
| 16:34:49 | opendevreview | Merged openstack/keystoneauth master: Enable mypy https://review.opendev.org/c/openstack/keystoneauth/+/928811 | |
| 16:34:50 | opendevreview | Merged openstack/keystoneauth master: discover: Expand use of kwargs https://review.opendev.org/c/openstack/keystoneauth/+/928812 | |
| 16:34:51 | opendevreview | Merged openstack/keystoneauth master: discover: Rework normalization https://review.opendev.org/c/openstack/keystoneauth/+/928813 | |
| 17:22:32 | JayF | https://review.opendev.org/c/openstack/openstacksdk/+/926414 is the last change we need to have landed to get ironic runbook support into all the SDKs (it was added this release), your reviews would be appreciated | |