| *** __ministry is now known as Guest3215 | 01:18 | |
| 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 | 03:52 |
|---|---|---|
| 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 |
| jlejeune | do you think it could be a good idea to skip it in that condition ? | 07:39 |
| frickler | stephenfin: did you happen to see https://bugs.launchpad.net/openstacksdk/+bug/2074200 ? | 09:43 |
| opendevreview | Merged openstack/osc-lib master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/osc-lib/+/928147 | 10:22 |
| 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:23 |
| 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 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Remove dependency on netifaces https://review.opendev.org/c/openstack/openstacksdk/+/928997 | 10:50 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Switch remaining image functions to proxy https://review.opendev.org/c/openstack/openstacksdk/+/923422 | 11:29 |
| 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 |
| 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 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Switch remaining identity functions to proxy https://review.opendev.org/c/openstack/openstacksdk/+/923425 | 11:29 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Switch remaining compute functions to proxy https://review.opendev.org/c/openstack/openstacksdk/+/923426 | 11:29 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Add missing parameter docs https://review.opendev.org/c/openstack/openstacksdk/+/923427 | 11:29 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Re-add support for passing objects https://review.opendev.org/c/openstack/openstacksdk/+/929010 | 12:10 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Deprecate passing dicts https://review.opendev.org/c/openstack/openstacksdk/+/929011 | 12:10 |
| opendevreview | Merged openstack/openstacksdk master: Remove unexpected Resource.update overrides https://review.opendev.org/c/openstack/openstacksdk/+/927516 | 12:30 |
| opendevreview | Merged openstack/openstacksdk master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/openstacksdk/+/928144 | 12:30 |
| opendevreview | Merged openstack/openstacksdk master: Add docs for common keystoneauth settings https://review.opendev.org/c/openstack/openstacksdk/+/920042 | 12:30 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Add timeline for removal of deprecated features https://review.opendev.org/c/openstack/openstacksdk/+/929015 | 12:31 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Deprecate unnecessary options, aliases https://review.opendev.org/c/openstack/openstacksdk/+/929016 | 12:31 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: exceptions: Deprecate http_status, request_id params https://review.opendev.org/c/openstack/openstacksdk/+/929017 | 12:31 |
| 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:50 |
| 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:51 |
| 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:53 |
| stephenfin | gtema: friendly reminder about those py38 patches ;) | 12:56 |
| gtema | thanks, totally missed that under the load | 12:57 |
| 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:07 |
| 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 |
| 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:19 |
| opendevreview | Merged openstack/openstacksdk master: config: Minimal validation of config files https://review.opendev.org/c/openstack/openstacksdk/+/926536 | 13:21 |
| opendevreview | Merged openstack/openstacksdk master: Fix parameter name clashing in resource lock query https://review.opendev.org/c/openstack/openstacksdk/+/925605 | 13:21 |
| 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:26 |
| opendevreview | Stephen Finucane proposed openstack/osc-lib master: parseactions: Use ArgumentError, not ArgumentTypeError https://review.opendev.org/c/openstack/osc-lib/+/836204 | 13:28 |
| opendevreview | Pavlo Shchelokovskyy proposed openstack/openstacksdk master: Followup to I0e5ac1288c0d0423fa3a7a4e63173675b78aae79 https://review.opendev.org/c/openstack/openstacksdk/+/929090 | 13:40 |
| opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Removed the emit_duplicated_warning() funtion. https://review.opendev.org/c/openstack/python-openstackclient/+/927743 | 13:43 |
| opendevreview | Merged openstack/cliff stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/cliff/+/928140 | 13:53 |
| opendevreview | Merged openstack/cliff stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/cliff/+/928141 | 13:53 |
| opendevreview | Merged openstack/osc-lib master: Fix log level, remove unnecessary debug https://review.opendev.org/c/openstack/osc-lib/+/914971 | 14:26 |
| 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 | 14:41 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Add timeline for removal of deprecated features https://review.opendev.org/c/openstack/openstacksdk/+/929015 | 16:18 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Deprecate unnecessary options, aliases https://review.opendev.org/c/openstack/openstacksdk/+/929016 | 16:18 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: exceptions: Deprecate http_status, request_id params https://review.opendev.org/c/openstack/openstacksdk/+/929017 | 16:18 |
| opendevreview | Merged openstack/python-openstackclient master: Drop support for Python 3.8 https://review.opendev.org/c/openstack/python-openstackclient/+/924493 | 16:28 |
| opendevreview | Stephen Finucane proposed openstack/cliff master: Migrate to ruff, ruff-format https://review.opendev.org/c/openstack/cliff/+/929157 | 16:41 |
| opendevreview | Stephen Finucane proposed openstack/cliff master: ruff: Enable pyupgrade rules https://review.opendev.org/c/openstack/cliff/+/929158 | 16:41 |
| opendevreview | Stephen Finucane proposed openstack/cliff master: Remove unnecessary shebangs https://review.opendev.org/c/openstack/cliff/+/929159 | 16:41 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: pre-commit: Migrate pyupgrade to ruff-format https://review.opendev.org/c/openstack/openstacksdk/+/929160 | 16:51 |
| opendevreview | Stephen Finucane proposed openstack/osc-lib master: ruff: Enable pyupgrade rules https://review.opendev.org/c/openstack/osc-lib/+/929161 | 16:55 |
| 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 |
| opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: pre-commit: Migrate pyupgrade to ruff https://review.opendev.org/c/openstack/python-openstackclient/+/929164 | 17:08 |
| JayF | stephenfin: <3 I have an item on Ironic's PTG to purse similar | 17:09 |
| JayF | re: pre-commit/ruff | 17:09 |
| stephenfin | we were discussing it in #openstack-keystone yesterday. It's a kick-ass tool | 17:09 |
| stephenfin | we're migrating/have migrated all sdk deps to it (we were already using black in many places) | 17:10 |
| JayF | Only real issue last time I looked at it is that it wasn't packaged everywhere | 17:11 |
| JayF | but I think in the intervening months that's changed a lot | 17:11 |
| 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:12 |
| 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:13 |
| stephenfin | fair | 17:17 |
| opendevreview | Merged openstack/osc-lib master: parseactions: Use ArgumentError, not ArgumentTypeError https://review.opendev.org/c/openstack/osc-lib/+/836204 | 17:19 |
| opendevreview | Merged openstack/osc-lib master: pre-commit: Migrate from flake8 to ruff https://review.opendev.org/c/openstack/osc-lib/+/929052 | 17:21 |
| opendevreview | Merged openstack/osc-lib master: pre-commit: Migrate from black to ruff format https://review.opendev.org/c/openstack/osc-lib/+/929053 | 17:21 |
| 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:37 |
| opendevreview | Merged openstack/osc-lib master: Exclude tests directory from coverage calculation https://review.opendev.org/c/openstack/osc-lib/+/906664 | 18:46 |
| opendevreview | Merged openstack/osc-lib master: ruff: Enable pyupgrade rules https://review.opendev.org/c/openstack/osc-lib/+/929161 | 18: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 |
| opendevreview | Merged openstack/python-openstackclient master: Removed the emit_duplicated_warning() funtion. https://review.opendev.org/c/openstack/python-openstackclient/+/927743 | 19:19 |
| opendevreview | Merged openstack/openstacksdk master: pre-commit: Bump mypy https://review.opendev.org/c/openstack/openstacksdk/+/927517 | 22:25 |
| opendevreview | Merged openstack/openstacksdk master: Drop support for Python 3.8 https://review.opendev.org/c/openstack/openstacksdk/+/923662 | 22:25 |
| opendevreview | Merged openstack/openstacksdk master: mypy: Run under minimum Python version https://review.opendev.org/c/openstack/openstacksdk/+/923663 | 22:25 |
| opendevreview | Merged openstack/openstacksdk master: Remove redundant code https://review.opendev.org/c/openstack/openstacksdk/+/923664 | 22:25 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!