| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-09-11 | |||
| 17:45:03 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Add variable for auth plugin opts https://review.opendev.org/c/openstack/keystoneauth/+/928809 | |
| 17:45:05 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Resolve remaining initial issues https://review.opendev.org/c/openstack/keystoneauth/+/928810 | |
| 17:45:07 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Enable mypy https://review.opendev.org/c/openstack/keystoneauth/+/928811 | |
| 17:45:09 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: discover: Expand use of kwargs https://review.opendev.org/c/openstack/keystoneauth/+/928812 | |
| 17:45:11 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: discover: Rework normalization https://review.opendev.org/c/openstack/keystoneauth/+/928813 | |
| 17:45:13 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Annotate keystoneauth1.discover https://review.opendev.org/c/openstack/keystoneauth/+/928814 | |
| 20:22:15 | opendevreview | Merged openstack/python-openstackclient master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/python-openstackclient/+/928941 | |
| 20:22:17 | opendevreview | Merged openstack/python-openstackclient stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/python-openstackclient/+/928938 | |
| 20:22:18 | opendevreview | Merged openstack/python-openstackclient stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/python-openstackclient/+/928939 | |
| 21:16:34 | opendevreview | Merged openstack/openstacksdk master: pre-commit: Bump versions https://review.opendev.org/c/openstack/openstacksdk/+/927511 | |
| 21:16:35 | opendevreview | Merged openstack/openstacksdk master: pre-commit: Migrate from flake8 to ruff https://review.opendev.org/c/openstack/openstacksdk/+/927512 | |
| 21:16:37 | opendevreview | Merged openstack/openstacksdk master: trivial: Remove unnecessary trailing comma https://review.opendev.org/c/openstack/openstacksdk/+/927513 | |
| 21:16:39 | opendevreview | Merged openstack/openstacksdk master: pre-commit: Migrate from black to ruff format https://review.opendev.org/c/openstack/openstacksdk/+/927514 | |
| 21:16:41 | opendevreview | Merged openstack/openstacksdk master: Fix override of ShareAccessRule.delete https://review.opendev.org/c/openstack/openstacksdk/+/927515 | |
| 21:16:43 | opendevreview | Merged openstack/python-openstackclient master: pre-commit: Bump versions https://review.opendev.org/c/openstack/python-openstackclient/+/927489 | |
| 21:16:44 | opendevreview | Merged openstack/python-openstackclient master: pre-commit: Migrate from flake8 to ruff https://review.opendev.org/c/openstack/python-openstackclient/+/927490 | |
| 21:16:46 | opendevreview | Merged openstack/python-openstackclient master: trivial: Remove unnecessary trailing comma https://review.opendev.org/c/openstack/python-openstackclient/+/927491 | |
| 21:16:48 | opendevreview | Merged openstack/python-openstackclient master: pre-commit: Migrate from black to ruff format https://review.opendev.org/c/openstack/python-openstackclient/+/927492 | |
| #openstack-sdks - 2024-09-12 | |||
| 03:52:38 | opendevreview | HAESONG LEE proposed openstack/openstacksdk master: title : Add getting info about the pools in a designate system https://review.opendev.org/c/openstack/openstacksdk/+/927683 | |
| 07:39:00 | jlejeune | hello, I have a small question regarding the unit test: test__load_yaml_json_file_without_perm in openstacksdk which fails if it's run by a user with admin right | |
| 07:39:28 | jlejeune | do you think it could be a good idea to skip it in that condition ? | |
| 09:43:11 | frickler | stephenfin: did you happen to see https://bugs.launchpad.net/openstacksdk/+bug/2074200 ? | |
| 10:22:00 | opendevreview | Merged openstack/osc-lib master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/osc-lib/+/928147 | |
| 10:23:11 | opendevreview | HAESONG LEE proposed openstack/openstacksdk master: title : Add getting info about the pools in a designate system https://review.opendev.org/c/openstack/openstacksdk/+/927683 | |
| 10:33:41 | opendevreview | HAESONG LEE proposed openstack/openstacksdk master: title : Add getting info about the pools in a designate system https://review.opendev.org/c/openstack/openstacksdk/+/927683 | |
| 10:50:04 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Remove dependency on netifaces https://review.opendev.org/c/openstack/openstacksdk/+/928997 | |
| 11:29:53 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Switch remaining image functions to proxy https://review.opendev.org/c/openstack/openstacksdk/+/923422 | |
| 11:29:54 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Switch remaining block storage functions to proxy https://review.opendev.org/c/openstack/openstacksdk/+/923423 | |
| 11:29:54 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Replace use of cloud methods in identity cloud layer https://review.opendev.org/c/openstack/openstacksdk/+/923424 | |
| 11:29:55 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Switch remaining identity functions to proxy https://review.opendev.org/c/openstack/openstacksdk/+/923425 | |
| 11:29:55 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Switch remaining compute functions to proxy https://review.opendev.org/c/openstack/openstacksdk/+/923426 | |
| 11:29:56 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Add missing parameter docs https://review.opendev.org/c/openstack/openstacksdk/+/923427 | |
| 12:10:52 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Re-add support for passing objects https://review.opendev.org/c/openstack/openstacksdk/+/929010 | |
| 12:10:53 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Deprecate passing dicts https://review.opendev.org/c/openstack/openstacksdk/+/929011 | |
| 12:30:03 | opendevreview | Merged openstack/openstacksdk master: Remove unexpected Resource.update overrides https://review.opendev.org/c/openstack/openstacksdk/+/927516 | |
| 12:30:06 | opendevreview | Merged openstack/openstacksdk master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/openstacksdk/+/928144 | |
| 12:30:09 | opendevreview | Merged openstack/openstacksdk master: Add docs for common keystoneauth settings https://review.opendev.org/c/openstack/openstacksdk/+/920042 | |
| 12:31:18 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Add timeline for removal of deprecated features https://review.opendev.org/c/openstack/openstacksdk/+/929015 | |
| 12:31:18 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Deprecate unnecessary options, aliases https://review.opendev.org/c/openstack/openstacksdk/+/929016 | |
| 12:31:19 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: exceptions: Deprecate http_status, request_id params https://review.opendev.org/c/openstack/openstacksdk/+/929017 | |
| 12:50:12 | stephenfin | frickler: Yes, I'm aware. There's a fix here but it's waiting on another +2 https://review.opendev.org/c/openstack/python-openstackclient/+/926193 | |
| 12:51:36 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Fix --host in server list for new openstacksdk https://review.opendev.org/c/openstack/python-openstackclient/+/926193 | |
| 12:53:05 | frickler | stephenfin: ah, nice, I was just about to ask for adding the bug reference, thx. I'll give it a quick test locally and then approve | |
| 12:56:08 | stephenfin | gtema: friendly reminder about those py38 patches ;) | |
| 12:57:13 | gtema | thanks, totally missed that under the load | |
| 13:07:10 | 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 | |
| 13:19:16 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: pre-commit: Migrate from flake8 to ruff https://review.opendev.org/c/openstack/osc-lib/+/929052 | |
| 13:19:17 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: pre-commit: Migrate from black to ruff format https://review.opendev.org/c/openstack/osc-lib/+/929053 | |
| 13:21:44 | opendevreview | Merged openstack/openstacksdk master: config: Minimal validation of config files https://review.opendev.org/c/openstack/openstacksdk/+/926536 | |
| 13:21:46 | opendevreview | Merged openstack/openstacksdk master: Fix parameter name clashing in resource lock query https://review.opendev.org/c/openstack/openstacksdk/+/925605 | |
| 13:26:42 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Do not create a class in runtime on a potentially hot path https://review.opendev.org/c/openstack/openstacksdk/+/920541 | |
| 13:28:44 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: parseactions: Use ArgumentError, not ArgumentTypeError https://review.opendev.org/c/openstack/osc-lib/+/836204 | |
| 13:40:15 | opendevreview | Pavlo Shchelokovskyy proposed openstack/openstacksdk master: Followup to I0e5ac1288c0d0423fa3a7a4e63173675b78aae79 https://review.opendev.org/c/openstack/openstacksdk/+/929090 | |
| 13:43:18 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Removed the emit_duplicated_warning() funtion. https://review.opendev.org/c/openstack/python-openstackclient/+/927743 | |
| 13:53:05 | opendevreview | Merged openstack/cliff stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/cliff/+/928140 | |
| 13:53:06 | opendevreview | Merged openstack/cliff stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/cliff/+/928141 | |
| 14:26:43 | opendevreview | Merged openstack/osc-lib master: Fix log level, remove unnecessary debug https://review.opendev.org/c/openstack/osc-lib/+/914971 | |
| 14:41:10 | 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:18:21 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Add timeline for removal of deprecated features https://review.opendev.org/c/openstack/openstacksdk/+/929015 | |
| 16:18:22 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Deprecate unnecessary options, aliases https://review.opendev.org/c/openstack/openstacksdk/+/929016 | |
| 16:18:22 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: exceptions: Deprecate http_status, request_id params https://review.opendev.org/c/openstack/openstacksdk/+/929017 | |
| 16:28:24 | opendevreview | Merged openstack/python-openstackclient master: Drop support for Python 3.8 https://review.opendev.org/c/openstack/python-openstackclient/+/924493 | |
| 16:41:00 | opendevreview | Stephen Finucane proposed openstack/cliff master: Migrate to ruff, ruff-format https://review.opendev.org/c/openstack/cliff/+/929157 | |
| 16:41:00 | opendevreview | Stephen Finucane proposed openstack/cliff master: ruff: Enable pyupgrade rules https://review.opendev.org/c/openstack/cliff/+/929158 | |
| 16:41:01 | opendevreview | Stephen Finucane proposed openstack/cliff master: Remove unnecessary shebangs https://review.opendev.org/c/openstack/cliff/+/929159 | |
| 16:51:17 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: pre-commit: Migrate pyupgrade to ruff-format https://review.opendev.org/c/openstack/openstacksdk/+/929160 | |
| 16:55:20 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: ruff: Enable pyupgrade rules https://review.opendev.org/c/openstack/osc-lib/+/929161 | |
| 17:08:31 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: pre-commit: Migrate bandit to ruff https://review.opendev.org/c/openstack/python-openstackclient/+/929163 | |
| 17:08:32 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: pre-commit: Migrate pyupgrade to ruff https://review.opendev.org/c/openstack/python-openstackclient/+/929164 | |
| 17:09:18 | JayF | stephenfin: <3 I have an item on Ironic's PTG to purse similar | |
| 17:09:24 | JayF | re: pre-commit/ruff | |
| 17:09:47 | stephenfin | we were discussing it in #openstack-keystone yesterday. It's a kick-ass tool | |
| 17:10:23 | stephenfin | we're migrating/have migrated all sdk deps to it (we were already using black in many places) | |
| 17:11:35 | JayF | Only real issue last time I looked at it is that it wasn't packaged everywhere | |
| 17:11:42 | JayF | but I think in the intervening months that's changed a lot | |
| 17:12:33 | stephenfin | We run it via pre-commit (including when called via tox) and don't include it in our requirements.txt, so as long as the user can access git to pull the pre-built binaries, they're good | |
| 17:13:58 | JayF | I think I'd want distro packages on most common development distros (latest ubuntu; fedora) so that it wouldn't be an issue for users behind a firewall or some kind of security policy. | |
| 17:17:28 | stephenfin | fair | |
| 17:19:19 | opendevreview | Merged openstack/osc-lib master: parseactions: Use ArgumentError, not ArgumentTypeError https://review.opendev.org/c/openstack/osc-lib/+/836204 | |
| 17:21:41 | opendevreview | Merged openstack/osc-lib master: pre-commit: Migrate from flake8 to ruff https://review.opendev.org/c/openstack/osc-lib/+/929052 | |
| 17:21:42 | opendevreview | Merged openstack/osc-lib master: pre-commit: Migrate from black to ruff format https://review.opendev.org/c/openstack/osc-lib/+/929053 | |
| 18:37:15 | opendevreview | Merged openstack/python-openstackclient master: compute: Fix --host in server list for new openstacksdk https://review.opendev.org/c/openstack/python-openstackclient/+/926193 | |
| 18:46:39 | opendevreview | Merged openstack/osc-lib master: Exclude tests directory from coverage calculation https://review.opendev.org/c/openstack/osc-lib/+/906664 | |
| 18:46:40 | opendevreview | Merged openstack/osc-lib master: ruff: Enable pyupgrade rules https://review.opendev.org/c/openstack/osc-lib/+/929161 | |
| 19:19:46 | opendevreview | Merged openstack/openstacksdk master: Do not create a class in runtime on a potentially hot path https://review.opendev.org/c/openstack/openstacksdk/+/920541 | |
| 19:19:48 | opendevreview | Merged openstack/python-openstackclient master: Removed the emit_duplicated_warning() funtion. https://review.opendev.org/c/openstack/python-openstackclient/+/927743 | |
| 22:25:33 | opendevreview | Merged openstack/openstacksdk master: pre-commit: Bump mypy https://review.opendev.org/c/openstack/openstacksdk/+/927517 | |
| 22:25:36 | opendevreview | Merged openstack/openstacksdk master: Drop support for Python 3.8 https://review.opendev.org/c/openstack/openstacksdk/+/923662 | |
| 22:25:38 | opendevreview | Merged openstack/openstacksdk master: mypy: Run under minimum Python version https://review.opendev.org/c/openstack/openstacksdk/+/923663 | |
| 22:25:40 | opendevreview | Merged openstack/openstacksdk master: Remove redundant code https://review.opendev.org/c/openstack/openstacksdk/+/923664 | |
| #openstack-sdks - 2024-09-13 | |||
| 05:08:47 | 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 | |
| 05:36:39 | 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 | |
| 06:00:57 | tkajinam | "BadRequestException: 400: Client Error for url: http://127.0.0.1/identity/v3/users, Invalid input for field 'domain_id': None is not of type 'string'" | |
| 06:01:24 | tkajinam | This is now seen in response to 'openstack user create' without any domain options. Probably is something related to recent schema validation work ? | |
| 06:01:33 | tkajinam | (probably I should have asked this in keystone | |
| 06:01:52 | frickler | tkajinam: is this with osc? we made a new release yesterday | |
| 06:02:05 | frickler | and I was just about to mention another regression https://zuul.opendev.org/t/openstack/build/54d612aa4622478d87cd8c42eacc9daa | |
| 06:02:26 | frickler | ah, you said "user create" | |
| 06:02:39 | tkajinam | https://zuul.opendev.org/t/openstack/build/4165b7790b7e4fa19a1837b9e08e2d76/log/job-output.txt#10797-10806 | |
| 06:03:30 | tkajinam | Ugh. it turned out we don't capture keystone log in storlets CI... I'll look into it | |