| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-10-16 | |||
| 14:31:04 | opendevreview | SEUNG TAE KIM proposed openstack/python-openstackclient master: Add "image task list" command https://review.opendev.org/c/openstack/python-openstackclient/+/813554 | |
| 14:33:00 | opendevreview | SEUNG TAE KIM proposed openstack/python-openstackclient master: Add "image task list" command https://review.opendev.org/c/openstack/python-openstackclient/+/813554 | |
| 14:59:07 | opendevreview | SEUNG TAE KIM proposed openstack/openstacksdk master: Add "page_size" string in _query_mapping variable in task.py https://review.opendev.org/c/openstack/openstacksdk/+/814270 | |
| #openstack-sdks - 2021-10-17 | |||
| 01:10:12 | opendevreview | Jinseok Kim proposed openstack/python-openstackclient master: Add column "Volume type" and "Bootable" https://review.opendev.org/c/openstack/python-openstackclient/+/813655 | |
| 04:46:58 | opendevreview | SEUNG TAE KIM proposed openstack/python-openstackclient master: Add "image task list" command https://review.opendev.org/c/openstack/python-openstackclient/+/813554 | |
| 04:53:17 | opendevreview | SEUNG TAE KIM proposed openstack/python-openstackclient master: Add "image task list" command https://review.opendev.org/c/openstack/python-openstackclient/+/813554 | |
| 16:18:09 | opendevreview | Yoon Soo Lim proposed openstack/python-openstackclient master: Add detailed option to show a missing field for hypervisor https://review.opendev.org/c/openstack/python-openstackclient/+/814321 | |
| #openstack-sdks - 2021-10-18 | |||
| 06:53:41 | thomasb06 | gtema: Hello. Did you have a chance to give a try to my script? I should walk the full directory tree, but I tried it only on one file in same directory as the script... | |
| 06:54:46 | thomasb06 | Maybe I should add a working directory change somewhere | |
| 07:14:33 | gtema | thomasb06 - we do not want single huge change, but rather lot of small ones | |
| 07:23:34 | thomasb06 | gtema: what I can do is change all the files at the same time, but commit them one by one? | |
| 07:24:11 | gtema | yes, this can be done. Pretty much same as before - one service at a time | |
| 07:24:25 | thomasb06 | Great | |
| 08:22:49 | thomasb06 | gtema: my script finds only these 25 files: https://x0.at/vx5Y.py | |
| 08:22:57 | thomasb06 | is it ok for you? | |
| 08:23:27 | gtema | yupp, those are precisely as expected | |
| 08:23:39 | thomasb06 | cool, I start commiting | |
| 08:23:43 | gtema | thks | |
| 08:28:23 | thomasb06 | the command `git review` returns `error: failed to push some refs` | |
| 08:28:33 | thomasb06 | maybe I need to rebase? | |
| 08:29:08 | gtema | Maybe. There should be bit more details | |
| 08:32:53 | thomasb06 | The full error message is: https://x0.at/UUhP.txt | |
| 08:33:58 | gtema | are you confirming upload of multiple commits? if not this is clear why it fails | |
| 08:35:10 | thomasb06 | To the question `Do you really want to submit the above commits?`, I say `yes`. Should I say no? | |
| 08:35:48 | thomasb06 | If I say `no`, it's aborting | |
| 08:36:01 | gtema | hm, "yes" is exactly that, or "git review -y" | |
| 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 | |