| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-02-09 | |||
| 13:23:21 | opendevreview | Harald Jens책s proposed openstack/ansible-collections-openstack master: Add openstack.cloud.baremetal_port_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827937 | |
| 13:23:21 | opendevreview | Harald Jensås proposed openstack/ansible-collections-openstack master: Add openstack.cloud.baremetal_port_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827937 | |
| 13:58:15 | opendevreview | Harald Jens책s proposed openstack/ansible-collections-openstack master: Add openstack.cloud.baremetal_port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827816 | |
| 13:58:15 | opendevreview | Harald Jensås proposed openstack/ansible-collections-openstack master: Add openstack.cloud.baremetal_port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827816 | |
| 14:06:04 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | |
| 14:07:50 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | |
| 14:09:41 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Dropped deprecated tenant_id in floating_ip_info and assert remaining fields https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828384 | |
| 14:13:02 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Dropped deprecated tenant_id in floating_ip_info and assert remaining fields https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828384 | |
| 14:32:34 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | |
| 14:36:01 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | |
| 15:19:43 | stephenfin | gtema: If you've time today/later this week, I've a load of OSC patches looking for some love <3 https://review.opendev.org/q/owner:stephenfin%2540redhat.com+project:openstack/python-openstackclient+is:open | |
| 15:20:09 | gtema | okay | |
| 15:20:43 | stephenfin | ty, I went through yours earlier but there was only one I could meaningfully review so I just did that 0:) | |
| 15:21:46 | gtema | thks anyway. Trying also not to be completely useless while recovering from sickness (thus passive last days) | |
| 15:22:17 | stephenfin | oh, hope it wasn't anything too serious /o\ hopefully you're feeling better now too | |
| 15:24:20 | gtema | not serious, whole world is now slowly passing through it. But still pretty ugly | |
| 16:09:51 | opendevreview | Merged openstack/ansible-collections-openstack master: Add openstack.cloud.baremetal_port_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827937 | |
| 16:18:45 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | |
| 16:25:23 | opendevreview | Tom Weininger proposed openstack/openstacksdk master: Add docstring to wait_for_load_balancer() method https://review.opendev.org/c/openstack/openstacksdk/+/828112 | |
| 16:32:35 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Move compute flavor info to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828108 | |
| 16:33:34 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test flavor changes https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828290 | |
| 17:40:58 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Add QoS rule type filtering keys https://review.opendev.org/c/openstack/openstacksdk/+/827671 | |
| 18:53:18 | opendevreview | James Kirsch proposed openstack/openstacksdk master: Identity: Add support for system role assignment https://review.opendev.org/c/openstack/openstacksdk/+/828602 | |
| 19:05:31 | opendevreview | James Kirsch proposed openstack/openstacksdk master: Identity: Add support for system role assignment https://review.opendev.org/c/openstack/openstacksdk/+/826193 | |
| 20:10:48 | opendevreview | melanie witt proposed openstack/python-openstackclient master: Fix 'server event list|show' for deleted servers https://review.opendev.org/c/openstack/python-openstackclient/+/828608 | |
| 20:18:38 | opendevreview | melanie witt proposed openstack/python-openstackclient master: Fix 'server event list|show' for deleted servers https://review.opendev.org/c/openstack/python-openstackclient/+/828608 | |
| 20:42:34 | opendevreview | Merged openstack/python-openstackclient master: compute: Don't warn if disk overcommit params unset https://review.opendev.org/c/openstack/python-openstackclient/+/816448 | |
| 20:42:41 | opendevreview | Merged openstack/python-openstackclient master: docs: Document additional 'server migration' commands https://review.opendev.org/c/openstack/python-openstackclient/+/828363 | |
| 20:42:44 | opendevreview | Merged openstack/python-openstackclient master: image: Remove FakeImage test helper https://review.opendev.org/c/openstack/python-openstackclient/+/818224 | |
| 21:00:47 | opendevreview | Merged openstack/python-openstackclient master: tests: Update fake image client in tests https://review.opendev.org/c/openstack/python-openstackclient/+/818225 | |
| 21:00:50 | opendevreview | Merged openstack/python-openstackclient master: image: Sanity check the 'SetImage' command https://review.opendev.org/c/openstack/python-openstackclient/+/818226 | |
| 21:00:53 | opendevreview | Merged openstack/python-openstackclient stable/wallaby: compute: Fix filtering servers by tags https://review.opendev.org/c/openstack/python-openstackclient/+/813804 | |
| 21:03:01 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM testing image info changes https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828294 | |
| 21:11:26 | opendevreview | Merged openstack/openstacksdk master: Restore get_compute_limits backward compatibility https://review.opendev.org/c/openstack/openstacksdk/+/826920 | |
| 21:42:59 | opendevreview | James Kirsch proposed openstack/ansible-collections-openstack master: Add support for system role in role assignment https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826395 | |
| 21:55:12 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | |
| 22:04:24 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored and documented floating_ip module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828613 | |
| 22:18:54 | opendevreview | melanie witt proposed openstack/python-openstackclient master: Fix 'server event list|show' for deleted servers https://review.opendev.org/c/openstack/python-openstackclient/+/828608 | |
| 22:41:35 | opendevreview | Harald Jensås proposed openstack/ansible-collections-openstack master: Add openstack.cloud.baremetal_port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827816 | |
| 23:23:38 | opendevreview | Alex Hussey proposed openstack/ansible-collections-openstack master: Adds use_name variable https://review.opendev.org/c/openstack/ansible-collections-openstack/+/817150 | |
| 23:27:13 | opendevreview | melanie witt proposed openstack/python-openstackclient master: Fix 'server event list|show' for deleted servers https://review.opendev.org/c/openstack/python-openstackclient/+/828608 | |
| #openstack-sdks - 2022-02-10 | |||
| 06:43:52 | opendevreview | Merged openstack/ansible-collections-openstack master: Add openstack.cloud.baremetal_port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827816 | |
| 08:27:49 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored and documented floating_ip module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828613 | |
| 09:29:16 | opendevreview | Tom Weininger proposed openstack/openstacksdk master: Add docstring to wait_for_load_balancer() method https://review.opendev.org/c/openstack/openstacksdk/+/828112 | |
| 09:50:42 | tweining__ | Hi, are there any guidelines about what to do with methods that have a name_or_id arguments but no longer accept a name? Can I just update the documentation? | |
| 09:51:26 | tweining__ | I mean openstacksdk Proxy methods | |
| 10:02:54 | opendevreview | Alex Hussey proposed openstack/ansible-collections-openstack master: Adds use_name variable https://review.opendev.org/c/openstack/ansible-collections-openstack/+/817150 | |
| 10:03:41 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP fix collection for new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | |
| 10:06:25 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Remove project properties tests and support https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828682 | |
| 10:08:50 | opendevreview | Alex Hussey proposed openstack/ansible-collections-openstack master: Adds use_name variable https://review.opendev.org/c/openstack/ansible-collections-openstack/+/817150 | |
| 10:10:45 | opendevreview | Alex Hussey proposed openstack/ansible-collections-openstack master: Adds use_name variable https://review.opendev.org/c/openstack/ansible-collections-openstack/+/817150 | |
| 10:17:33 | frickler | tweining__: to me that would sound like bugs, can you share an example? | |
| 10:17:52 | tweining__ | frickler: https://storyboard.openstack.org/#!/story/2009808 | |
| 10:20:07 | opendevreview | Alex Hussey proposed openstack/ansible-collections-openstack master: Adds use_name variable https://review.opendev.org/c/openstack/ansible-collections-openstack/+/817150 | |
| 10:22:39 | opendevreview | Alex Hussey proposed openstack/ansible-collections-openstack master: Adds use_name variable https://review.opendev.org/c/openstack/ansible-collections-openstack/+/817150 | |
| 10:28:03 | frickler | tweining__: hmm, in that case it looks like a documentation error indeed, according to what the tests do, you would have to use find_load_balancer() if you have a name | |
| 10:28:43 | tweining__ | thanks, I'm just updating the docstrings. | |
| 10:31:50 | opendevreview | Alex Hussey proposed openstack/ansible-collections-openstack master: Adds use_name variable https://review.opendev.org/c/openstack/ansible-collections-openstack/+/817150 | |
| 10:43:58 | opendevreview | Tom Weininger proposed openstack/openstacksdk master: Correct documentation about Load Balancer API https://review.opendev.org/c/openstack/openstacksdk/+/828689 | |
| 10:44:52 | opendevreview | Alex Hussey proposed openstack/ansible-collections-openstack master: Adds use_name variable https://review.opendev.org/c/openstack/ansible-collections-openstack/+/817150 | |
| 11:40:54 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update compute service info module to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828698 | |
| 11:46:48 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Move compute flavor info to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828108 | |
| 13:12:02 | opendevreview | Dmitriy Rabotyagov proposed openstack/python-openstackclient master: Implement server migrate commands https://review.opendev.org/c/openstack/python-openstackclient/+/828708 | |
| 13:12:50 | opendevreview | Dmitriy Rabotyagov proposed openstack/python-openstackclient master: Implement server and host evacuate commands https://review.opendev.org/c/openstack/python-openstackclient/+/828708 | |
| 13:40:30 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test flavor changes https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828290 | |
| 13:51:37 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP fix collection for new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | |
| 14:00:30 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | |
| 14:56:31 | Adri2000 | hello, has anyone tried to resize (extend) an *attached* Cinder volume using the SDK? | |
| 14:57:33 | Adri2000 | according to https://docs.openstack.org/api-ref/block-storage/v3/?expanded=extend-a-volume-size-detail#extend-a-volume-size it requires API microversion 3.42 which doesn't seem to be specified: https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/block_storage/v3/volume.py#L112-L115 | |
| 14:59:05 | Adri2000 | I can file a bug for this, and would be happy to make an attempt at a patch, but am not familiar with the code. would it be easy from the code to check the status of the volume? e.g. if (volume is attached) then utils.require_microversion(session, "3.42") | |
| 16:44:04 | opendevreview | Merged openstack/ansible-collections-openstack master: Remove project properties tests and support https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828682 | |
| 16:48:22 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer identity_user https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828742 | |
| 16:52:24 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828742 | |
| 16:58:55 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828742 | |
| 17:02:20 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | |
| 17:50:41 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | |
| 18:28:16 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | |
| 18:33:26 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828742 | |
| 18:36:33 | headphoneJames | gtema: if you have a chance, please let me know if I'm missing any other parts for: https://review.opendev.org/c/openstack/openstacksdk/+/826193 | |
| 18:36:40 | headphoneJames | Thanks! | |
| 18:41:29 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Allow retrieval of migration by UUID https://review.opendev.org/c/openstack/python-openstackclient/+/828785 | |
| 20:50:34 | opendevreview | Florian Haas proposed openstack/openstacksdk master: Add get_*_by_id() methods for Magnum clusters and templates https://review.opendev.org/c/openstack/openstacksdk/+/828791 | |
| 20:54:48 | opendevreview | Florian Haas proposed openstack/openstacksdk master: Add get_*_by_id() methods for Magnum clusters and templates https://review.opendev.org/c/openstack/openstacksdk/+/828791 | |
| 21:41:47 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | |
| 22:43:13 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828742 | |
| #openstack-sdks - 2022-02-11 | |||
| 02:15:26 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | |
| 08:00:39 | gtema | ok headphoneJames, will have a look today | |
| 09:38:56 | opendevreview | Jakob Meng proposed openstack/osc-lib master: Respect 'interface' key from clouds.yaml https://review.opendev.org/c/openstack/osc-lib/+/784559 | |
| 09:42:28 | jm1 | hello :) who would i have to ask for a workflow+1 on this osc-lib patch? https://review.opendev.org/c/openstack/osc-lib/+/784559 | |
| 10:35:33 | opendevreview | Tom Weininger proposed openstack/openstacksdk master: Correct documentation about Load Balancer API https://review.opendev.org/c/openstack/openstacksdk/+/828689 | |
| 10:59:17 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Allow retrieval of migration by UUID https://review.opendev.org/c/openstack/python-openstackclient/+/828785 | |
| 10:59:18 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Move server migrations commands to their own file https://review.opendev.org/c/openstack/python-openstackclient/+/828855 | |
| 11:08:54 | frickler | jm1: I've put it into my review list, but won't get to it today. from a quick glance, I'm wondering whether it would help to have a test for that scenario that would avoid regressing again | |
| 12:46:47 | opendevreview | Florian Haas proposed openstack/openstacksdk master: Add get_*_by_id() methods for Magnum clusters and templates https://review.opendev.org/c/openstack/openstacksdk/+/828791 | |
| 13:17:50 | jm1 | frickler: thanks :) | |
| 13:18:49 | jm1 | frickler: regarding the tests.. not sure but i think the tests did not catch this flaw because it asserts 'public' which is exactly DEFAULT_INTERFACE https://opendev.org/openstack/osc-lib/src/branch/master/osc_lib/tests/test_shell.py#L356 | |
| 13:31:14 | opendevreview | Jakob Meng proposed openstack/osc-lib master: Fixed test for interface key https://review.opendev.org/c/openstack/osc-lib/+/828876 | |
| 13:41:06 | jm1 | frickler: submitted a patch which fixes the test. of course zuul now fails but this is to be expected as the other patch has not been merged yet. https://review.opendev.org/c/openstack/osc-lib/+/828876 | |
| 13:52:59 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828539 | |
| 13:57:34 | Adri2000 | about the in-use/attached volume resize issue I mentioned here yesterday, here is a bug report I just filed: https://storyboard.openstack.org/#!/story/2009850 | |