| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-10-20 | |||
| 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 | |
| 15:33:11 | stephenfin | yeah, there's a lot of those in the nova API, sadly | |
| 15:33:37 | stephenfin | i.e. 'GET /servers/{id}/detail' instead of e.g. 'GET /servers/{id}?detail=1' | |
| 15:34:45 | gtema | https://review.opendev.org/c/openstack/python-openstackclient/+/763414 is making me doubt in myself | |
| 15:35:01 | gtema | check passes, gate fails (single test I mean) | |
| 15:36:36 | stephenfin | not entirely related but why is HypervisorTests.test_hypervisor_list calling 'hypervisor show'? :) | |
| 15:37:11 | stephenfin | that's a strange failure | |
| 15:37:15 | gtema | nice question | |
| 15:39:39 | stephenfin | so the test fails at 2021-10-22 15:04:02.759154 after 5.132781s | |
| 15:39:45 | gtema | ah, cause I first try to get list of existing ids | |
| 15:42:11 | stephenfin | there's the request hitting the API at 15:04:02.564172 https://zuul.opendev.org/t/openstack/build/1f6c15565b2840bfb6e4f09ec7714ea0/log/controller/logs/screen-n-api.txt#1979 | |
| 15:43:07 | gtema | apparently OSC receives empty response back | |
| 15:46:20 | stephenfin | the compute service is still up | |
| 15:46:30 | stephenfin | https://zuul.opendev.org/t/openstack/build/1f6c15565b2840bfb6e4f09ec7714ea0/log/controller/logs/screen-n-cpu.txt#7018 | |
| 15:47:58 | gtema | I'm not able to understand why check and gate behaves differently - and this 2 times | |
| 15:48:07 | gtema | maybe I should rebase | |
| 15:48:22 | stephenfin | I'd suggest trying it. That's weird | |
| 15:48:36 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Switch hypervisor operations to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/763414 | |
| 15:49:14 | gtema | well, and if check will fail now I would be even more confused ;-) | |
| #openstack-sdks - 2021-10-23 | |||
| 12:39:30 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server backup to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | |
| 12:45:03 | 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-25 | |||
| 08:18:01 | thomasb06 | gtema: Hello. The lines are two long, would you any guidelines in this case: https://review.opendev.org/c/openstack/openstacksdk/+/814395 | |
| 08:18:14 | thomasb06 | sometimes the class name is huge | |
| 10:59:31 | gtema | thomasb06 - pushed update to this change | |
| 11:32:25 | 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 | |
| 15:02:37 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server backup to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | |
| 16:00:45 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server backup to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | |
| 16:45:10 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server backup to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | |
| 18:55:00 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server backup to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | |
| 20:41:49 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server backup to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | |
| 22:34:38 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server backup to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | |
| 23:40:28 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server backup to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/814332 | |