| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-07-30 | |||
| 13:05:41 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Remove TestClustering functional tests https://review.opendev.org/c/openstack/openstacksdk/+/923223 | |
| 13:05:42 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: WIP: mypy: Enable stricter type checking https://review.opendev.org/c/openstack/openstacksdk/+/925199 | |
| 13:05:42 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Enable warn_return_any https://review.opendev.org/c/openstack/openstacksdk/+/923226 | |
| 13:41:29 | ganso | stephenfin: hi! just following up on our previous discussion, would it make sense to merge https://review.opendev.org/c/openstack/openstacksdk/+/922965 as-is while the large rework you're working on is still far from merging? | |
| 13:44:44 | stephenfin | sure | |
| 14:57:02 | opendevreview | Merged openstack/openstacksdk master: zuul: Use more stable dib job https://review.opendev.org/c/openstack/openstacksdk/+/923671 | |
| 15:39:22 | opendevreview | Merged openstack/openstacksdk master: block_storage: Add support for project_id in Limits https://review.opendev.org/c/openstack/openstacksdk/+/922965 | |
| 16:34:16 | gibi | gtema: thanks for the hints about instantiating a sdk connection based on a user context. This seems to work how I expect https://review.opendev.org/c/openstack/nova/+/925277/1/nova/utils.py#1003 If you have any comments about it then feel free to leave it on the review | |
| 16:50:50 | gtema | gibi: perfect | |
| #openstack-sdks - 2024-07-31 | |||
| 00:34:39 | opendevreview | Callum Dickinson proposed openstack/openstacksdk master: Switch to netifaces-plus https://review.opendev.org/c/openstack/openstacksdk/+/925307 | |
| 00:44:43 | opendevreview | Callum Dickinson proposed openstack/openstacksdk master: Switch to netifaces-plus https://review.opendev.org/c/openstack/openstacksdk/+/925307 | |
| 06:44:17 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Prepare git patch and archive for rust package https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 08:24:05 | opendevreview | Merged openstack/codegenerator master: Prepare git patch and archive for rust package https://review.opendev.org/c/openstack/codegenerator/+/925062 | |
| 09:08:58 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Move propose job from post to promote pipeline https://review.opendev.org/c/openstack/codegenerator/+/925325 | |
| 11:29:06 | opendevreview | Merged openstack/codegenerator master: Move propose job from post to promote pipeline https://review.opendev.org/c/openstack/codegenerator/+/925325 | |
| 11:34:02 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix undefined var in promote job https://review.opendev.org/c/openstack/codegenerator/+/925383 | |
| 12:39:28 | opendevreview | Riccardo Pittau proposed openstack/openstacksdk master: Move dib test to CentOS Stream 9 https://review.opendev.org/c/openstack/openstacksdk/+/922390 | |
| 12:39:54 | rpittau | stephenfin: hi! I've rebased this ^ | |
| 13:36:19 | opendevreview | Merged openstack/codegenerator master: Fix undefined var in promote job https://review.opendev.org/c/openstack/codegenerator/+/925383 | |
| 13:40:08 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Accept github newhostkey in promote job. https://review.opendev.org/c/openstack/codegenerator/+/925398 | |
| 14:07:59 | ganso | stephenfin: hi! thanks for merging the patch yesterday! Quick question: Does it make sense to backport that patch to have it included in the latest stable release so the customer can have access to it now rather than have to wait for the next release? I don't know what the SDK release cadence and versioning is | |
| 14:33:21 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Accept github newhostkey in promote job. https://review.opendev.org/c/openstack/codegenerator/+/925398 | |
| 16:24:37 | opendevreview | Merged openstack/codegenerator master: Accept github newhostkey in promote job. https://review.opendev.org/c/openstack/codegenerator/+/925398 | |
| 16:27:45 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Set git push parameters in promote job https://review.opendev.org/c/openstack/codegenerator/+/925409 | |
| 17:45:09 | stephenfin | ganso: It's probably a little too invasive to backport, but a new SDK release should be out in the next fortnight or so. Just waiting on some final typing changes to land | |
| 17:55:11 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix security_group.security_group_rules schema https://review.opendev.org/c/openstack/codegenerator/+/925419 | |
| 18:31:45 | opendevreview | Merged openstack/codegenerator master: Set git push parameters in promote job https://review.opendev.org/c/openstack/codegenerator/+/925409 | |
| 18:46:58 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix security_group_rules schema https://review.opendev.org/c/openstack/codegenerator/+/925419 | |
| 19:32:23 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Stringify response_code decorator https://review.opendev.org/c/openstack/codegenerator/+/925428 | |
| 20:26:51 | opendevreview | Merged openstack/codegenerator master: Fix security_group_rules schema https://review.opendev.org/c/openstack/codegenerator/+/925419 | |
| 21:18:18 | opendevreview | Merged openstack/codegenerator master: Stringify response_code decorator https://review.opendev.org/c/openstack/codegenerator/+/925428 | |
| 21:42:14 | opendevreview | Merged openstack/openstacksdk master: cloud: Fix annotations for proxy method https://review.opendev.org/c/openstack/openstacksdk/+/924853 | |
| #openstack-sdks - 2024-08-01 | |||
| 07:24:00 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Fix generation of openapi with py310 https://review.opendev.org/c/openstack/codegenerator/+/925443 | |
| 07:38:25 | opendevreview | Joel Capitao proposed openstack/python-openstackclient master: Bump requests minimal version https://review.opendev.org/c/openstack/python-openstackclient/+/925445 | |
| 08:35:10 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add crate-ci/typos for typo correction https://review.opendev.org/c/openstack/codegenerator/+/925448 | |
| 09:11:24 | opendevreview | Merged openstack/codegenerator master: Fix generation of openapi with py310 https://review.opendev.org/c/openstack/codegenerator/+/925443 | |
| 09:15:21 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add crate-ci/typos for typo correction https://review.opendev.org/c/openstack/codegenerator/+/925448 | |
| 09:22:29 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add crate-ci/typos for typo correction https://review.opendev.org/c/openstack/codegenerator/+/925448 | |
| 09:31:09 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add crate-ci/typos for typo correction https://review.opendev.org/c/openstack/codegenerator/+/925448 | |
| 09:38:16 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add crate-ci/typos for typo correction https://review.opendev.org/c/openstack/codegenerator/+/925448 | |
| 11:40:09 | opendevreview | Merged openstack/codegenerator master: Add crate-ci/typos for typo correction https://review.opendev.org/c/openstack/codegenerator/+/925448 | |
| 13:15:13 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Use non-keyset pagination for object-storage in rust https://review.opendev.org/c/openstack/codegenerator/+/925479 | |
| 13:21:55 | opendevreview | Merged openstack/python-openstackclient master: Respect --skip-resource when deleting. https://review.opendev.org/c/openstack/python-openstackclient/+/924659 | |
| 13:52:28 | opendevreview | Pranali Deore proposed openstack/openstacksdk master: Adding support for new add/get location apis https://review.opendev.org/c/openstack/openstacksdk/+/883271 | |
| 15:03:38 | opendevreview | Merged openstack/openstacksdk master: mypy: Enable checks for openstack.tests.fixtures https://review.opendev.org/c/openstack/openstacksdk/+/923225 | |
| 15:03:40 | opendevreview | Merged openstack/openstacksdk master: cloud: Make _OpenStackCloudMixin subclass ServicesMixin https://review.opendev.org/c/openstack/openstacksdk/+/923214 | |
| 15:06:31 | opendevreview | Merged openstack/codegenerator master: Use non-keyset pagination for object-storage in rust https://review.opendev.org/c/openstack/codegenerator/+/925479 | |
| #openstack-sdks - 2024-08-02 | |||
| 08:19:23 | stephenfin | gtema: The rest of those typing changes are good to go hint hint :) https://review.opendev.org/c/openstack/openstacksdk/+/923215/ Only if you have time | |
| 08:20:11 | gtema | yeah, those failing runs are moving attention somewhere else | |
| 08:21:50 | gtema | btw stephenfin: I finally managed to have a pipeline from codegenerator down to the github repo with rust. With that every time we change smth on codegenerator (ideally this should be also integrated to the repos themselves) new generated code is being proposed to the destination sdk/cli | |
| 08:22:02 | gtema | s/repos/services/ | |
| 08:24:03 | stephenfin | class. nicely done :) | |
| 08:24:34 | stephenfin | tbc, its only the code generator for now. There's not anything watching for e.g. new schemas being added to Manila yet, I assume? | |
| 10:37:30 | opendevreview | Merged openstack/openstacksdk master: cloud: Make service mixins subclass _OpenStackCloudMixin https://review.opendev.org/c/openstack/openstacksdk/+/923215 | |
| 10:37:31 | opendevreview | Merged openstack/openstacksdk master: cloud: Reorganize _OpenStackCloudMixin, Connection (1/2) https://review.opendev.org/c/openstack/openstacksdk/+/923216 | |
| 10:37:33 | opendevreview | Merged openstack/openstacksdk master: cloud: Reorganize _OpenStackCloudMixin, Connection (2/2) https://review.opendev.org/c/openstack/openstacksdk/+/923217 | |
| 10:37:35 | opendevreview | Merged openstack/openstacksdk master: cloud: Misc type fixes https://review.opendev.org/c/openstack/openstacksdk/+/923218 | |
| 12:25:19 | opendevreview | Merged openstack/openstacksdk master: cloud: Combine networking-related classes https://review.opendev.org/c/openstack/openstacksdk/+/923220 | |
| 15:35:35 | opendevreview | Balazs Gibizer proposed openstack/openstacksdk master: Fix parameter name clashing in resource lock query https://review.opendev.org/c/openstack/openstacksdk/+/925605 | |
| 18:41:54 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'credential' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/923674 | |
| 20:19:56 | opendevreview | Merged openstack/openstacksdk master: mypy: Enable checks for openstack.cloud https://review.opendev.org/c/openstack/openstacksdk/+/923219 | |
| 20:39:17 | opendevreview | Merged openstack/openstacksdk master: tests: Remove TestClustering functional tests https://review.opendev.org/c/openstack/openstacksdk/+/923223 | |
| 20:39:19 | opendevreview | Merged openstack/openstacksdk master: mypy: Enable checks for openstack.tests.functional https://review.opendev.org/c/openstack/openstacksdk/+/923224 | |
| #openstack-sdks - 2024-08-03 | |||
| 21:35:26 | opendevreview | Manuel Osorio proposed openstack/openstacksdk master: Add share transfer to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/925653 | |
| 22:04:22 | opendevreview | Manuel Osorio proposed openstack/openstacksdk master: Add share transfer to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/925653 | |
| 22:24:55 | opendevreview | Manuel Osorio proposed openstack/openstacksdk master: Add share transfer to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/925653 | |
| 23:30:17 | opendevreview | Manuel Osorio proposed openstack/openstacksdk master: Add share transfer to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/925653 | |
| #openstack-sdks - 2024-08-05 | |||
| 10:11:37 | opendevreview | Balazs Gibizer proposed openstack/openstacksdk master: Support server unshelve to specific host https://review.opendev.org/c/openstack/openstacksdk/+/925684 | |
| 15:18:04 | opendevreview | Valentin Chassignol proposed openstack/python-openstackclient master: Add retrieve password with optional decryption https://review.opendev.org/c/openstack/python-openstackclient/+/923621 | |
| #openstack-sdks - 2024-08-06 | |||
| 08:14:29 | opendevreview | Valentin Chassignol proposed openstack/python-openstackclient master: Add retrieve password with optional decryption https://review.opendev.org/c/openstack/python-openstackclient/+/923621 | |
| 08:15:44 | opendevreview | Valentin Chassignol proposed openstack/python-openstackclient master: Add retrieve password with optional decryption https://review.opendev.org/c/openstack/python-openstackclient/+/923621 | |
| 08:18:59 | opendevreview | Valentin Chassignol proposed openstack/python-openstackclient master: Add retrieve password with private key decryption https://review.opendev.org/c/openstack/python-openstackclient/+/923621 | |
| 09:39:55 | opendevreview | Valentin Chassignol proposed openstack/python-openstackclient master: Add retrieve password with private key decryption https://review.opendev.org/c/openstack/python-openstackclient/+/923621 | |
| 14:22:46 | frickler | gtema: likely this can be fixed in devstack, but I'm not sure if it should be considered a regression in OSC https://zuul.opendev.org/t/openstack/build/ce0ea76a7c394b35aaeec5ee02d9914d ("-c id" not working for some commands, "-c ID" needed) | |
| 14:23:50 | frickler | I'm also not sure how this does not show up in OSC testing, likely we do not use the patch under test everywhere in devstack? | |
| 14:24:37 | gtema | ohm, respecting we tagged 7.0.0 I think we should rather fix in devstack. it should actually stop doing this madness as such | |
| 14:25:00 | gtema | wrt testing - we have tips jobs and I am also wondering it didn't pop up earlier | |
| 14:27:14 | frickler | o.k., I'll push a fix in devstack after the current meeting ends | |
| 14:27:33 | gtema | thks a lot | |
| #openstack-sdks - 2024-08-07 | |||
| 06:29:12 | frickler | gtema: fyi I haven't pushed patches yet because the issue doesn't appear in devstack itself, but in multiple plugins. in addition I now think that it is not acceptable from a stable user perspective to make common patterns stop working without a deprecation period and without even a releasenote mentioning the change | |
| 06:29:37 | frickler | so I would prefer if we could treat this as a regression and fix OSC behaviour for now | |
| 06:46:02 | frickler | oh, wait, the situation is even worse, as you cannot seem to have one command that works both on 7.0.0 and older releases https://paste.opendev.org/show/bSZX8DQj2ddKCeRaQTHL/ | |
| 06:52:46 | frickler | https://bugs.launchpad.net/python-openstackclient/+bug/2076212 | |
| 10:08:28 | frickler | another failure that seems related to 7.0.0 https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_6e8/925827/1/check/kolla-ansible-ubuntu/6e87c17/primary/logs/ansible/init-runonce | |
| 10:21:48 | frickler | "quota set --instances" is failing if there is no volume service. that's another severe regression IMO. cc stephenfin | |
| 10:52:17 | mnasiadka | frickler: thanks for picking that up, I was just thinking of posting it here :) | |
| 10:56:08 | frickler | https://bugs.launchpad.net/python-openstackclient/+bug/2076229 | |
| 11:04:51 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Add Python 3.12 classifier https://review.opendev.org/c/openstack/openstacksdk/+/924136 | |
| 11:04:52 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Run under minimum Python version https://review.opendev.org/c/openstack/openstacksdk/+/923663 | |
| 11:04:52 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Drop support for Python 3.7, 3.8 https://review.opendev.org/c/openstack/openstacksdk/+/923662 | |
| 11:04:53 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Remove redundant code https://review.opendev.org/c/openstack/openstacksdk/+/923664 | |
| 11:20:15 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: identity: Use previous naming for 'service show' fields https://review.opendev.org/c/openstack/python-openstackclient/+/925860 | |
| 11:21:19 | stephenfin | frickler: One fix there ^ Can you give me a traceback for the quota error? | |
| 11:21:41 | stephenfin | (I don't want to have to deploy devstack without cinder to validate it) | |
| 11:23:39 | frickler | stephenfin: it is at the end of the init-runonce log I linked above | |
| 11:25:00 | frickler | stephenfin: does the column name change only affect the service resource or will there be similar issues for others? | |
| 11:27:03 | stephenfin | frickler: I couldn't say, to be honest. I don't think I changed names in any of the compute changes but someone would need to check to be sure | |
| 11:54:52 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: quota: Allow 'quota set' to function without volume service https://review.opendev.org/c/openstack/python-openstackclient/+/925861 | |
| 11:56:00 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: identity: Use previous naming for 'service show' fields https://review.opendev.org/c/openstack/python-openstackclient/+/925860 | |
| 11:56:07 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: quota: Allow 'quota set' to function without volume service https://review.opendev.org/c/openstack/python-openstackclient/+/925861 | |