| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2025-02-11 | |||
| 17:27:25 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Enable disallow_incomplete_defs https://review.opendev.org/c/openstack/openstacksdk/+/941268 | |
| 17:27:25 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Enable stricter mode https://review.opendev.org/c/openstack/openstacksdk/+/941267 | |
| 17:27:26 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Annotate Resource.wait_for_* utils https://review.opendev.org/c/openstack/openstacksdk/+/941269 | |
| 17:27:27 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: resource: Rework fields https://review.opendev.org/c/openstack/openstacksdk/+/941271 | |
| 17:27:27 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: proxy: Add 'wait_for_*' helpers to all proxy APIs https://review.opendev.org/c/openstack/openstacksdk/+/941270 | |
| 17:27:29 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: fields: Be more explicit in our conversion https://review.opendev.org/c/openstack/openstacksdk/+/941273 | |
| 17:27:29 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: fields: Tweak _convert_type https://review.opendev.org/c/openstack/openstacksdk/+/941272 | |
| 17:27:31 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Annotate openstack.fields (1/2) https://review.opendev.org/c/openstack/openstacksdk/+/941275 | |
| 17:27:31 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Annotate openstack.format https://review.opendev.org/c/openstack/openstacksdk/+/941274 | |
| 17:27:33 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Don't rely on kwargs for _translate_response https://review.opendev.org/c/openstack/openstacksdk/+/941277 | |
| 17:27:33 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: typing: Annotate openstack.fields (2/2) https://review.opendev.org/c/openstack/openstacksdk/+/941276 | |
| 17:27:35 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: pre-commit: Prepare for bump https://review.opendev.org/c/openstack/openstacksdk/+/941279 | |
| 17:27:35 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: network: Set If-Match header properly https://review.opendev.org/c/openstack/openstacksdk/+/941278 | |
| 17:27:37 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: pre-commit: Bump versions https://review.opendev.org/c/openstack/openstacksdk/+/941280 | |
| 17:36:15 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: identity: Migrate 'endpoint' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/934817 | |
| 18:10:58 | opendevreview | Merged openstack/python-openstackclient master: Identity: Migrate 'role' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929712 | |
| 18:22:20 | stephenfin | M0weng[m]: Sounds like you're going to need to implement your own logic in OSC for that so :( | |
| 21:07:43 | M0weng[m] | <stephenfin> "0weng: Sounds like you're..." <- Yeah, I did a try-except where it tries with name/ID first and then tries to list all services with that type; is CommandError appropriate for cases where there are multiple services with that type? | |
| 21:17:54 | M0weng[m] | Oh I forgot, --debug will tell me what exception is being used; looks like it is CommandError :) | |
| 21:34:12 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: identity: Migrate 'endpoint' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/934817 | |
| 21:36:49 | M0weng[m] | Ah wait, I just realized there is a find_service_sdk() function in common already! I'll use that then :) | |
| 21:53:36 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: identity: Migrate 'endpoint' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/934817 | |
| 22:03:08 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: identity: Migrate 'endpoint' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/934817 | |
| #openstack-sdks - 2025-02-12 | |||
| 00:22:24 | opendevreview | Merged openstack/python-openstackclient master: Identity: Deprecate `--region None` in limits/registered limits https://review.opendev.org/c/openstack/python-openstackclient/+/940321 | |
| 03:27:01 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 04:30:53 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 06:07:03 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 06:38:51 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 07:19:06 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 08:19:56 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 08:51:51 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 09:53:15 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 10:07:45 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 11:13:09 | opendevreview | Grzegorz Grasza proposed openstack/keystoneauth master: [WiP] External OAuth2.0 plugin https://review.opendev.org/c/openstack/keystoneauth/+/941082 | |
| 12:34:11 | tkajinam | o/ can we merge this tiny cleanup ? https://review.opendev.org/c/openstack/os-service-types/+/907597 | |
| 12:44:00 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Hardcode barbican secret_store schemas https://review.opendev.org/c/openstack/codegenerator/+/941374 | |
| 12:44:00 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add barbican secret schemas https://review.opendev.org/c/openstack/codegenerator/+/941234 | |
| 13:36:21 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Hardcode barbican container schemas https://review.opendev.org/c/openstack/codegenerator/+/941406 | |
| 15:12:13 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Temporary ignore "is_vlan_qinq" column in the output https://review.opendev.org/c/openstack/python-openstackclient/+/940111 | |
| 15:43:01 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Temporary ignore "is_vlan_qinq" column in the output https://review.opendev.org/c/openstack/python-openstackclient/+/940111 | |
| 15:43:48 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Add "qinq-vlan" and "no-qinq-vlan" params to the "network create" cmd https://review.opendev.org/c/openstack/python-openstackclient/+/939706 | |
| #openstack-sdks - 2025-02-13 | |||
| 03:25:18 | opendevreview | Yanks proposed openstack/openstacksdk master: pass stream value https://review.opendev.org/c/openstack/openstacksdk/+/927778 | |
| 03:25:19 | opendevreview | Yanks proposed openstack/openstacksdk master: fix: lint https://review.opendev.org/c/openstack/openstacksdk/+/941078 | |
| 09:10:53 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Add "qinq-vlan" and "no-qinq-vlan" params to the "network create" cmd https://review.opendev.org/c/openstack/python-openstackclient/+/939706 | |
| 09:13:45 | slaweq | gtema stephenfin hi, please check https://review.opendev.org/c/openstack/python-openstackclient/+/940111 (temporary just to make sdk patch pass), then https://review.opendev.org/c/openstack/openstacksdk/+/939703 and finally https://review.opendev.org/c/openstack/python-openstackclient/+/939706 when you will have time | |
| 09:13:57 | slaweq | but that last one will probably need new SDK release first | |
| 09:14:10 | slaweq | thx in advance | |
| 09:14:59 | gtema | ack slaweq | |
| 09:36:42 | opendevreview | Niklas Schwarz proposed openstack/openstacksdk master: Add listing of groups for a user https://review.opendev.org/c/openstack/openstacksdk/+/941540 | |
| 11:08:37 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Hardcode barbican order schema https://review.opendev.org/c/openstack/codegenerator/+/941542 | |
| 11:08:38 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Hardcode barbican quota schemas https://review.opendev.org/c/openstack/codegenerator/+/941543 | |
| 11:11:35 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: pre-commit: Bump versions https://review.opendev.org/c/openstack/openstacksdk/+/941280 | |
| 11:11:35 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: pre-commit: Prepare for bump https://review.opendev.org/c/openstack/openstacksdk/+/941279 | |
| 11:11:36 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: network: Set If-Match header properly https://review.opendev.org/c/openstack/openstacksdk/+/941278 | |
| 12:15:08 | opendevreview | Niklas Schwarz proposed openstack/openstacksdk master: Add listing of groups for a user https://review.opendev.org/c/openstack/openstacksdk/+/941540 | |
| 12:57:36 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Deprecate use of AuthConstructor https://review.opendev.org/c/openstack/keystoneauth/+/935770 | |
| 12:57:37 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Removing additional unused kwargs (1/2) https://review.opendev.org/c/openstack/keystoneauth/+/935771 | |
| 12:57:38 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Removing additional unused kwargs (2/2) https://review.opendev.org/c/openstack/keystoneauth/+/941242 | |
| 12:57:39 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: pre-commit: Bump versions https://review.opendev.org/c/openstack/keystoneauth/+/941253 | |
| 12:57:41 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: [WiP] External OAuth2.0 plugin https://review.opendev.org/c/openstack/keystoneauth/+/941082 | |
| 13:01:06 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: [Neutron] "uplink-status-propagation" enabled by default https://review.opendev.org/c/openstack/python-openstackclient/+/941552 | |
| 14:24:30 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: [Neutron] Support ``uplink-status-propagation-updatable`` extension https://review.opendev.org/c/openstack/python-openstackclient/+/941555 | |
| 16:15:54 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 16:32:01 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 16:53:05 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| #openstack-sdks - 2025-02-14 | |||
| 04:14:27 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 05:18:07 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 05:30:38 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 05:54:20 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 06:14:16 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 06:33:28 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 06:49:30 | joelm21 | Long shot, but is there a document that covers the openstacksdk's functions, in relation to their relevant policy? i.e. `identity.get_user` has a corresponding policy, but `identity.find_user` doesn't, and I'm just trying to work out what privileges I need to grant my user to get this working. | |
| 07:08:42 | opendevreview | Yanks proposed openstack/openstacksdk master: pass stream value https://review.opendev.org/c/openstack/openstacksdk/+/927778 | |
| 07:26:23 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 07:41:49 | opendevreview | Yaguang Tang proposed openstack/keystoneauth master: fix: correct oidc request error handling logic https://review.opendev.org/c/openstack/keystoneauth/+/941203 | |
| 09:35:27 | Uggla | stephenfin, gtema, could you please have a look at 880056: Add shares API to compute | https://review.opendev.org/c/openstack/openstacksdk/+/880056 and 881540: Add share commands to compute | https://review.opendev.org/c/openstack/python-openstackclient/+/881540 | |
| 11:09:32 | stephenfin | gtema: Thoughts on merging OSC into SDK early in E? | |
| 11:09:49 | stephenfin | ...and making OSC a meta package | |
| 11:10:47 | stephenfin | or at least the compute, image, network and object storage parts of it since those are pure SDK now | |
| 11:10:55 | gtema | generally - ok. What I am concerned with is that at some stage everything is being splitted, later again re-merged. It is not about sdk/cli in particular, just an observation. | |
| 11:11:16 | gtema | are we now talking about merging into single repo while still having different pypi packages? | |
| 11:12:20 | stephenfin | At least in the short term, to avoid breaking workflows | |
| 11:12:27 | stephenfin | I think we can do one of two things | |
| 11:13:41 | stephenfin | Either (a) we do a wholesale import of everything from OSC into SDK but put it behind an extra in setup.cfg, and then have OSC literally have a requirements.txt e.g. 'openstacksdk[cli]' | |
| 11:14:13 | stephenfin | (a.k.a. similar to what the openstackclient meta-package is currently doing https://github.com/openstack/openstackclient/blob/master/requirements.txt) | |
| 11:15:13 | stephenfin | Or (b) we only import the actual service commands (i.e. 'openstackclient.compute', 'openstackclient.network') and add a 'openstack.cli.base' entry to our 'setup.cfg' | |
| 11:15:49 | stephenfin | I think (b) is conceptually nicer since that is using the same machinery that e.g. python-manilaclient is using to provide their OSC commands | |
| 11:16:49 | stephenfin | On your meta point of things being split out and remerged, I think that's a consequence of (a) tooling and (b) latterly the decreasing size of the community | |
| 11:17:31 | stephenfin | There was all this focus on various "lib" packages to share code between projects, mainly because Python doesn't historically make it easy to export multiple packages from a single repo | |
| 11:17:38 | gtema | (b) makes sense but then we still have 2 different repos and perhaps a mess questioning yourself: where the heck is x.y.z now coming from | |
| 11:18:37 | stephenfin | Plus Gerrit doesn't have anything like the k8s OWNER file thing: someone either has commit rights to the entire repo or they don't. You can't grant it for part of the repo. At least not natively. | |
| 11:18:39 | gtema | i am now often confused where is the actual implementation of the identity stuff that is somehow split between multiple different places | |
| 11:19:14 | stephenfin | The source of the CLI implementations is always shown if you do '--help' | |
| 11:19:22 | gtema | fyi: OWNER is not a k8s specific stuff, it is suported by git hosters | |
| 11:19:50 | gtema | I mean the code itself which is chaining pieces from different places and different repos | |
| 11:19:56 | gtema | not finding the entry point | |
| 11:20:19 | stephenfin | Gotcha. If we do (b) then I'm only proposing moving the services that are 100% SDK right now | |
| 11:20:32 | stephenfin | so there should be no novaclient, no swiftclient etc. | |
| 11:20:57 | gtema | ok, lets discuss more details during ptg | |
| 11:21:46 | stephenfin | OSC will take the shell utility stuff from osc-lib and cliff, and it will take the actual commands from either OSC itself (identity, block storage), SDK (compute, image, network and object storage), or one of the python-*client packages (shared filesystems, clustering etc.) | |