| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-09-10 | |||
| 16:48:35 | opendevreview | Merged openstack/codegenerator master: Add getting started guide to README https://review.opendev.org/c/openstack/codegenerator/+/927704 | |
| 16:49:54 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Drop support for Python 3.8 https://review.opendev.org/c/openstack/keystoneauth/+/928833 | |
| 16:50:28 | stephenfin | gtema: Are we ready to merge the "drop python3.8" patches now that we've branched? https://review.opendev.org/q/topic:%22drop-py37-py38%22 | |
| 16:51:57 | gtema | yupp, shoot it | |
| 16:52:25 | gtema | oh but not in the cli | |
| 16:52:41 | stephenfin | huh? | |
| 16:52:48 | gtema | cli is not branched, not that I see a branch | |
| 16:52:57 | stephenfin | I think it's independent | |
| 16:53:20 | stephenfin | nope | |
| 16:54:24 | stephenfin | only thing I can see is that it's 'type: other' | |
| 16:55:13 | gtema | ehm, I am confused | |
| 16:56:41 | stephenfin | looks like this has happened before https://review.opendev.org/c/openstack/releases/+/913494 | |
| 16:56:45 | stephenfin | I think there's a tooling issue | |
| 17:03:15 | gtema | fwik it should have happened in https://review.opendev.org/c/openstack/releases/+/927944 where sdk and osc-lib were branched | |
| 17:04:09 | stephenfin | discussing on #openstack-release now. elodilles confirmed it's because of the type | |
| 17:04:43 | stephenfin | osc-lib and sdk are 'type: library' so we can't do the same there. client-library doesn't really make sense either since it's not a library | |
| 17:49:12 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Add "trusted" attribut to the "port" https://review.opendev.org/c/openstack/python-openstackclient/+/927727 | |
| 18:25:06 | opendevreview | Slawek Kaplonski proposed openstack/openstacksdk master: Add "trusted" attribute to "port" resource https://review.opendev.org/c/openstack/openstacksdk/+/927723 | |
| 19:15:13 | samcat116 | Has --os-profile been removed from the OpenStack CLI? Its not a recognized argument for anything on client version 6.3.0 | |
| 20:29:29 | opendevreview | Tobias Urdin proposed openstack/openstacksdk master: Change block storage volume summary parameter https://review.opendev.org/c/openstack/openstacksdk/+/928867 | |
| 20:30:09 | opendevreview | Tobias Urdin proposed openstack/openstacksdk master: Change block storage volume summary parameter https://review.opendev.org/c/openstack/openstacksdk/+/928867 | |
| 21:16:14 | opendevreview | Tobias Urdin proposed openstack/openstacksdk master: Change block storage volume summary parameter https://review.opendev.org/c/openstack/openstacksdk/+/928867 | |
| 21:26:28 | opendevreview | Tobias Urdin proposed openstack/openstacksdk master: Change block storage volume summary parameter https://review.opendev.org/c/openstack/openstacksdk/+/928867 | |
| 21:49:47 | opendevreview | Tobias Urdin proposed openstack/openstacksdk master: Change block storage volume summary parameter https://review.opendev.org/c/openstack/openstacksdk/+/928867 | |
| 21:52:00 | opendevreview | Tobias Urdin proposed openstack/openstacksdk master: Add backup summary API https://review.opendev.org/c/openstack/openstacksdk/+/928870 | |
| 21:52:48 | opendevreview | Tobias Urdin proposed openstack/openstacksdk master: Add backup summary API https://review.opendev.org/c/openstack/openstacksdk/+/928870 | |
| 22:17:43 | opendevreview | Tobias Urdin proposed openstack/python-openstackclient master: Add volume backup summary command https://review.opendev.org/c/openstack/python-openstackclient/+/928871 | |
| 22:51:26 | JayF | gtema: took a browse with adamcarthur5 through the generator, and looking at ironic v nova microversion implementations, it seems extremely unlikely that the same approach will work unless significant changes are made to Ironic. Those changes likely will need another catalyst to happen (e.g. if we need to migrate something in our api library stack anyway) and would be a hot topic for PTG | |
| 22:52:06 | JayF | gtema: if you wanna see an example, start here: https://opendev.org/openstack/ironic/src/branch/master/ironic/api/controllers/v1/versions.py look at history and you'll see many commits adding microversions to see how we handle them, e.g.: https://opendev.org/openstack/ironic/commit/8e34d622aff72d7dd286add31e3d7cd366629bc2 | |
| #openstack-sdks - 2024-09-11 | |||
| 09:56:45 | opendevreview | Pavlo Shchelokovskyy proposed openstack/openstacksdk master: Return payload as text only for text/plain secrets https://review.opendev.org/c/openstack/openstacksdk/+/928151 | |
| 09:59:44 | opendevreview | Pavlo Shchelokovskyy proposed openstack/openstacksdk master: Return payload as text only for text/plain secrets https://review.opendev.org/c/openstack/openstacksdk/+/928151 | |
| 11:12:31 | opendevreview | Pavlo Shchelokovskyy proposed openstack/openstacksdk master: Return payload as text only for text/plain secrets https://review.opendev.org/c/openstack/openstacksdk/+/928151 | |
| 13:56:24 | 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 | |
| 15:22:42 | opendevreview | Artem Goncharov proposed openstack/keystoneauth master: Apply ruff, ruff-format https://review.opendev.org/c/openstack/keystoneauth/+/928805 | |
| 15:31:00 | stephenfin | gtema: I think using uv is not a half bad idea | |
| 15:31:03 | stephenfin | It supports lower-constraint-style installation (i.e. use the lower bounds of your deps) and I did some experiments around recreating the lower-constraint job with it some time back | |
| 15:31:30 | stephenfin | However, it was significantly less mature than it is now and I ran into some blockers | |
| 15:31:42 | stephenfin | (which I've since forgotten about, but...) | |
| 15:31:44 | gtema | I will start experimenting adding it to codegenerator to see what overall is necessary under the OpenStack style | |
| 15:33:12 | stephenfin | assuming you're focused on the uv the packaging tool (like setuptools) rather than uv the installation tool (like pip), then I guess just some pyproject.toml changes | |
| 15:34:41 | gtema | hehe, I assume usual fun ;-) | |
| 16:07:18 | opendevreview | OpenStack Release Bot proposed openstack/python-openstackclient stable/2024.2: Update .gitreview for stable/2024.2 https://review.opendev.org/c/openstack/python-openstackclient/+/928938 | |
| 16:07:20 | opendevreview | OpenStack Release Bot proposed openstack/python-openstackclient stable/2024.2: Update TOX_CONSTRAINTS_FILE for stable/2024.2 https://review.opendev.org/c/openstack/python-openstackclient/+/928939 | |
| 16:07:22 | opendevreview | OpenStack Release Bot proposed openstack/python-openstackclient master: Update master for stable/2024.2 https://review.opendev.org/c/openstack/python-openstackclient/+/928941 | |
| 16:31:00 | opendevreview | Merged openstack/keystoneauth master: Update betamax extra deps https://review.opendev.org/c/openstack/keystoneauth/+/928801 | |
| 16:36:59 | opendevreview | Merged openstack/keystoneauth master: Add pre-commit https://review.opendev.org/c/openstack/keystoneauth/+/928802 | |
| 16:37:02 | opendevreview | Merged openstack/keystoneauth master: docs: Remove cruft from configuration files https://review.opendev.org/c/openstack/keystoneauth/+/928803 | |
| 16:37:04 | opendevreview | Merged openstack/keystoneauth master: trivial: Remove errant trailing commas https://review.opendev.org/c/openstack/keystoneauth/+/928804 | |
| 16:55:14 | opendevreview | Tobias Urdin proposed openstack/openstacksdk master: Change block storage volume summary parameter https://review.opendev.org/c/openstack/openstacksdk/+/928867 | |
| 16:56:48 | opendevreview | Tobias Urdin proposed openstack/openstacksdk master: Fix volume summary passing non-existent all_projects https://review.opendev.org/c/openstack/openstacksdk/+/928867 | |
| 17:25:45 | stephenfin | gtema: (For tomorrow) The 2024.2 branch should be available for OSC now, so we can probably move ahead with dropping Python 3.8 support | |
| 17:26:09 | gtema | Right | |
| 17:26:13 | gtema | Thanks | |
| 17:44:58 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Apply ruff, ruff-format https://review.opendev.org/c/openstack/keystoneauth/+/928805 | |
| 17:44:59 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Enable ruff, ruff-format https://review.opendev.org/c/openstack/keystoneauth/+/928806 | |
| 17:45:00 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Make abstract method actually abstract https://review.opendev.org/c/openstack/keystoneauth/+/928807 | |
| 17:45:01 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Synchronise overridden method signatures https://review.opendev.org/c/openstack/keystoneauth/+/928808 | |
| 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 | |