| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-02-08 | |||
| 13:04: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:07:13 | 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:35:18 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: DNM testing image info changes https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828294 | |
| 13:44:38 | 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 | |
| 13:51:26 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: DNM testing image info changes https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828294 | |
| 13:51:37 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: DNM testing image info changes https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828294 | |
| 14:56:32 | 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 | |
| 15:28:10 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: docs: Document additional 'server migration' commands https://review.opendev.org/c/openstack/python-openstackclient/+/828363 | |
| 17:30:42 | stephenfin | gtema: diablo_rojo__: I'm stuck in a meeting til :00, so I won't be able to join today :-( Ping me if you need anything | |
| 17:31:10 | gtema | oki stephenfin | |
| 17:41:21 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Drop deprecated fields in floating_ip_info and assert remaining fields https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828384 | |
| #openstack-sdks - 2022-02-09 | |||
| 09:37:10 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Drop deprecated field in floating_ip_info and assert remaining fields https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828384 | |
| 09:48:59 | opendevreview | Merged openstack/ansible-collections-openstack master: Add openstack.cloud.baremetal_node_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827586 | |
| 10:51:13 | isabek | stephenfin: Hi! Can you please approve patch [1] ,it has 2 +2 reviews? 1) https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 10:51:21 | isabek | Thanks in advance! | |
| 10:51:41 | stephenfin | sure, done | |
| 10:54:44 | isabek | Thank you very much! | |
| 10:57:34 | 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 | |
| 12:50:15 | opendevreview | Merged openstack/python-openstackclient master: Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 12:50:19 | opendevreview | Merged openstack/python-openstackclient master: Add missing command mapping in nova https://review.opendev.org/c/openstack/python-openstackclient/+/818684 | |
| 12:54:10 | 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:20:36 | 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:20:36 | 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:22:12 | 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:22:12 | 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: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 | |