| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-10-18 | |||
| 08:36:27 | gtema | maybe try adding "-v"? | |
| 08:43:23 | thomasb06 | after a `git reset HEAD~3` it worked, but I haven't a clue what I did: https://review.opendev.org/c/openstack/openstacksdk/+/814369 | |
| 08:43:33 | frickler | thomasb06: "! [remote rejected] HEAD -> refs/for/feature/r1 (no new changes)" indicates that you are trying so submit an empty commit | |
| 08:43:55 | gtema | yeah, I also noticed that and was suspecting it | |
| 08:46:05 | thomasb06 | frickler: indeed, my script generates a new _proxy_new.py, but in this case, the diff was null... | |
| 08:46:10 | thomasb06 | thanks though | |
| 08:46:26 | thomasb06 | message/v2: https://review.opendev.org/c/openstack/openstacksdk/+/814370 | |
| 08:48:14 | thomasb06 | accelerator/v2: https://review.opendev.org/c/openstack/openstacksdk/+/814372 | |
| 08:48:56 | thomasb06 | going for my meal, I'll do the rest this afternoon. Meanwhile, let's see what Zuul says... | |
| 08:49:22 | gtema | ok | |
| 10:16:28 | thomasb06 | baremetal/v1: https://review.opendev.org/c/openstack/openstacksdk/+/814395 | |
| 10:18:39 | thomasb06 | identity/v2: https://review.opendev.org/c/openstack/openstacksdk/+/814396 | |
| 10:19:52 | thomasb06 | identity/v3: https://review.opendev.org/c/openstack/openstacksdk/+/814398 | |
| 10:21:11 | thomasb06 | key_manager/v1: https://review.opendev.org/c/openstack/openstacksdk/+/814399 | |
| 10:24:08 | thomasb06 | Arg, `:rtype:` is also an entry... | |
| 16:03:14 | gtema | stephenfin, diablo_rojo_phone joining the meeting? | |
| 16:04:06 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server backup to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | |
| #openstack-sdks - 2021-10-19 | |||
| 06:39:17 | opendevreview | Frode Nordahl proposed openstack/python-openstackclient master: Add support for 'smart-nic' vnic-type https://review.opendev.org/c/openstack/python-openstackclient/+/814518 | |
| 14:16:53 | thomasb06 | gtema: hello. The build fails in the second commit, but it seems it comes from the first one: https://review.opendev.org/c/openstack/openstacksdk/+/814370 | |
| 17:15:59 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server backup to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | |
| 18:33:55 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server backup to sdk. Add temporary helper methods in fakes.py for server commands. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | |
| #openstack-sdks - 2021-10-20 | |||
| 02:02:41 | opendevreview | WooJongSeong proposed openstack/python-openstackclient master: remove not used codes and fix the format https://review.opendev.org/c/openstack/python-openstackclient/+/814686 | |
| 02:02:48 | opendevreview | WooJongSeong proposed openstack/python-openstackclient master: fix the type used in server.py https://review.opendev.org/c/openstack/python-openstackclient/+/814687 | |
| 02:02:55 | opendevreview | WooJongSeong proposed openstack/python-openstackclient master: add new test codes for new command https://review.opendev.org/c/openstack/python-openstackclient/+/814688 | |
| 05:30:58 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: Keep creating keystone admin endpoint for heat https://review.opendev.org/c/openstack/openstacksdk/+/777343 | |
| 06:39:51 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: Add Neutron Local IP CRUD https://review.opendev.org/c/openstack/openstacksdk/+/804988 | |
| 06:40:01 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 08:13:19 | opendevreview | WooJongSeong proposed openstack/python-openstackclient master: fix the code format https://review.opendev.org/c/openstack/python-openstackclient/+/814723 | |
| 10:10:54 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 10:11:08 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: Add Neutron Local IP CRUD https://review.opendev.org/c/openstack/openstacksdk/+/804988 | |
| 12:55:04 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: Add Neutron Local IP CRUD https://review.opendev.org/c/openstack/openstacksdk/+/804988 | |
| 13:48:27 | opendevreview | Jiří Stránský proposed openstack/ansible-collections-openstack master: Add documentation links to README.md https://review.opendev.org/c/openstack/ansible-collections-openstack/+/814782 | |
| 14:24:07 | opendevreview | WooJongSeong proposed openstack/python-openstackclient master: impl OSC's cmd hypervisor servers set https://review.opendev.org/c/openstack/python-openstackclient/+/814790 | |
| 14:24:14 | opendevreview | WooJongSeong proposed openstack/python-openstackclient master: update docs for new command https://review.opendev.org/c/openstack/python-openstackclient/+/814791 | |
| 14:28:27 | opendevreview | WooJongSeong proposed openstack/python-openstackclient master: impl OSC's cmd hypervisor servers set https://review.opendev.org/c/openstack/python-openstackclient/+/814792 | |
| 17:14:20 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Remove "tenant_id" members from any Network resource https://review.opendev.org/c/openstack/openstacksdk/+/814813 | |
| 20:05:10 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: Add Neutron Local IP CRUD https://review.opendev.org/c/openstack/openstacksdk/+/804988 | |
| #openstack-sdks - 2021-10-21 | |||
| 09:17:40 | opendevreview | Stephen Finucane proposed openstack/os-api-ref master: Fix exception logging https://review.opendev.org/c/openstack/os-api-ref/+/799216 | |
| 09:17:41 | opendevreview | Stephen Finucane proposed openstack/os-api-ref master: Drop support for Sphinx < 4.x https://review.opendev.org/c/openstack/os-api-ref/+/814873 | |
| 09:17:42 | opendevreview | Stephen Finucane proposed openstack/os-api-ref master: Remove six https://review.opendev.org/c/openstack/os-api-ref/+/814874 | |
| 09:17:43 | opendevreview | Stephen Finucane proposed openstack/os-api-ref master: tox: Don't use distutils entrypoint for coverage https://review.opendev.org/c/openstack/os-api-ref/+/814875 | |
| 09:43:59 | opendevreview | Merged openstack/os-api-ref master: Drop support for Sphinx < 4.x https://review.opendev.org/c/openstack/os-api-ref/+/814873 | |
| 09:50:16 | opendevreview | Merged openstack/os-api-ref master: Remove six https://review.opendev.org/c/openstack/os-api-ref/+/814874 | |
| 09:50:18 | opendevreview | Merged openstack/os-api-ref master: tox: Don't use distutils entrypoint for coverage https://review.opendev.org/c/openstack/os-api-ref/+/814875 | |
| 09:55:50 | opendevreview | Merged openstack/os-api-ref master: Fix exception logging https://review.opendev.org/c/openstack/os-api-ref/+/799216 | |
| 10:23:24 | opendevreview | Merged openstack/python-openstackclient master: Update the Nova CLI docoder document https://review.opendev.org/c/openstack/python-openstackclient/+/812421 | |
| 13:43:21 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server backup to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | |
| 13:43:32 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server backup to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | |
| 14:01:48 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server backup to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | |
| 16:16:38 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tox: Ignore virtualenvs for pep8 environment https://review.opendev.org/c/openstack/python-openstackclient/+/815031 | |
| 16:16:38 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Remove 'get_osc_show_columns_for_sdk_resource' duplicates https://review.opendev.org/c/openstack/python-openstackclient/+/815032 | |
| 16:16:39 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Remove remnants of 'six' https://review.opendev.org/c/openstack/python-openstackclient/+/815033 | |
| 16:17:18 | stephenfin | gtema: Some dead simple patches there ^ | |
| 16:17:26 | stephenfin | if you've got 5 | |
| 16:17:37 | gtema | ok, looking | |
| 16:28:24 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Add 'volume cluster *' commands https://review.opendev.org/c/openstack/python-openstackclient/+/795829 | |
| 16:28:25 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: WIP: volume: Add 'volume filter list' command https://review.opendev.org/c/openstack/python-openstackclient/+/794645 | |
| #openstack-sdks - 2021-10-22 | |||
| 11:04:48 | opendevreview | Merged openstack/python-openstackclient master: tox: Ignore virtualenvs for pep8 environment https://review.opendev.org/c/openstack/python-openstackclient/+/815031 | |
| 11:04:51 | opendevreview | Merged openstack/python-openstackclient master: Remove 'get_osc_show_columns_for_sdk_resource' duplicates https://review.opendev.org/c/openstack/python-openstackclient/+/815032 | |
| 11:04:53 | opendevreview | Merged openstack/python-openstackclient master: Remove remnants of 'six' https://review.opendev.org/c/openstack/python-openstackclient/+/815033 | |
| 13:48:49 | stephenfin | gtema: Bike shedding time! cinder allows you to configure the fields that you can filter various resources on (volumes, attachments, backups, snapshots, etc.). cinderclient lets you list these using a list-filter command. I need to add this to OSC. What do I call it? | |
| 13:50:20 | stephenfin | gtema: I was going to add a separate 'filter list' command for each resource (i.e. 'volume snapshot filter list', 'volume backup filter list') but the underlying API returns filters for all resources, so this seems wasteful: we'd be dumping info for all resources except the one we care about | |
| 13:50:39 | stephenfin | example cinderclient output https://paste.opendev.org/show/810167/ | |
| 13:52:59 | stephenfin | I'm thinking 'volume service filter list' or 'volume filter list' but both have issues. The latter is simpler, but I'm concerned people might think the filters are only for volumes. The former avoids this but people could read it as a filter for the volume service (i.e. the entity shown by 'volume service list'). 'filter list' is too generic | |
| 13:53:15 | stephenfin | Any opinions? I'll just choose one if you don't :-D | |
| 14:04:50 | gtema | Uhm, no opinion in this now | |
| 14:05:36 | frickler | stephenfin: what about "volume resource filter list", just like you kind of said in the first sentence? | |
| 14:05:58 | stephenfin | frickler: oh, that's a good idea. I didn't think of that | |
| 14:06:07 | stephenfin | that'll do nicely. Thanks, frickler | |
| 14:06:26 | frickler | :) | |
| 14:07:56 | gtema | maybe also `block storage filter list` | |
| 14:08:24 | gtema | not to mix with volume as a resource | |
| 14:10:15 | stephenfin | OSC seems to use volume rather than block storage everywhere (though object store instead of simply object), so I'm not so sure about that one. I think 'volume resource' is as clear as e.g. 'volume service' is | |
| 14:16:17 | gtema | that is exactly what I mean. atm we have volume rather for the resource, than the service. If we want to have command related to the service (if I understood correctly what you mean) I would rather try using different name | |
| 14:18:10 | stephenfin | I agree that we shouldn't overload the term. I'm saying that we're already overloading it though and don't use "block storage" as a term anywhere except some docs. It doesn't seem wise to change direction unless we want to rename the existing commands? | |
| 14:19:06 | gtema | I wouldn't change existing commands, cause they refer to the volume as a resource. Where we refer to volume as a service - we can think | |
| 14:20:21 | gtema | object store is not having a prefix at all and that is already creating confusion (not only in OSC, but also in ansible) | |
| 14:21:02 | stephenfin | hmm, fair. Let me see what it looks like and how it fits with existing commands | |
| 14:22:41 | stephenfin | gtema: How about 'volume service resource filter list'? It's a heck of a mouthful but it works? | |
| 14:23:00 | stephenfin | If not, I'd be tempted to rename 'volume service list' to 'block storage service list' | |
| 14:23:11 | stephenfin | with a deprecated alias, of course | |
| 14:23:22 | gtema | maybe it would be really better in the major release introduce block_storage and object_storage | |
| 14:23:55 | gtema | `block storage service list` sounds much better to me | |
| 14:24:38 | stephenfin | so also 'block storage service cluster' for clusters of block storage services? | |
| 14:25:17 | gtema | service cluster? not `block storage cluster`? | |
| 14:25:40 | gtema | which cluster you mean? | |
| 14:27:09 | stephenfin | They're clusters of services https://docs.openstack.org/cinder/latest/contributor/api_microversion_history.html#id7 | |
| 14:27:33 | gtema | oh great, this is not on https://docs.openstack.org/api-ref/block-storage/v3 | |
| 14:27:39 | stephenfin | I think it's a HA thing, but I'm not really sure /o\ I'll have to ask Gorka | |
| 14:27:46 | stephenfin | Yeah, they're missing a few things from the docs :( | |
| 14:28:42 | gtema | very sad - really lot | |
| 14:29:08 | gtema | anyway, in this case I would rather go for `block storage cluster` | |
| 14:29:27 | gtema | I do not think using `service` inside in the command is useful | |
| 14:30:28 | gtema | missing stuff in docs mean we might miss lot in SDK | |
| 14:55:27 | gtema | stephenfin, if you are still around - is nova /limits still accept tenant_id as query_param? https://docs.openstack.org/api-ref/compute/?expanded=show-rate-and-absolute-limits-detail#show-rate-and-absolute-limits | |
| 15:27:45 | stephenfin | gtema: yes, it's still tenant_id | |
| 15:28:02 | gtema | ok, thks | |
| 15:28:13 | stephenfin | there's a blueprint to change it to project_id (ditto for all other remaining references to tenant) but that hasn't landed yet | |
| 15:28:34 | gtema | good, adding it this way | |
| 15:31:10 | gtema | uhm, and quota-set is also non-standard (whatever standard actually is) - no support for ?usage=, but with /quota-sets/XXX/details | |