opendevreview | melanie witt proposed openstack/python-openstackclient stable/victoria: compute: Show flavor in 'server list' with API >= 2.47 https://review.opendev.org/c/openstack/python-openstackclient/+/820960 | 00:41 |
---|---|---|
*** diablo_rojo is now known as Guest7904 | 02:01 | |
*** artom__ is now known as artom | 15:08 | |
thomasb06 | gtema: Hello. Shared file system: https://review.opendev.org/c/openstack/openstacksdk/+/821083 | 15:09 |
*** efried1 is now known as efried | 15:19 | |
*** diablo_rojo__ is now known as diablo_rojo | 15:48 | |
opendevreview | Ujjval proposed openstack/python-openstackclient master: compute: Switch server group to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820070 | 16:29 |
stephenfin | frickler: Great spot on the SDK front. I've no idea how that got through the gate though. I'm going to push a fix for the unit test so we can merge that. Hopefully that's okay | 17:57 |
stephenfin | *weird nova API behaviour front | 17:58 |
opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Fix RemoveServerVolume https://review.opendev.org/c/openstack/python-openstackclient/+/820917 | 17:59 |
opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Switch server group to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820070 | 18:10 |
frickler | stephenfin: thx, lgtm, I was just waiting for feedback whether that's the way we want to go before looking at the unit test. do you also have an idea for the SDK side yet? | 18:17 |
stephenfin | wdym? | 18:17 |
frickler | well the real issue is that the function in sdk claims to use attachment_id as parameter, when in fact it needs a volume_id | 18:22 |
frickler | the=delete_volume_attachment() | 18:23 |
frickler | in addition, the help text also claims that the server argument might be optional, which it isn't | 18:23 |
stephenfin | Gotcha. As I mentioned to gtema a few weeks ago, that whole part of SDK is pretty badly broken and need a rewrite | 18:24 |
stephenfin | This is another thing to add to the list | 18:24 |
opendevreview | Merged openstack/python-openstackclient master: Fix RemoveServerVolume https://review.opendev.org/c/openstack/python-openstackclient/+/820917 | 20:15 |
opendevreview | melanie witt proposed openstack/python-openstackclient stable/ussuri: compute: Reorder building of columns for 'server list' https://review.opendev.org/c/openstack/python-openstackclient/+/821122 | 20:35 |
opendevreview | melanie witt proposed openstack/python-openstackclient stable/ussuri: compute: Show flavor in 'server list' with API >= 2.47 https://review.opendev.org/c/openstack/python-openstackclient/+/821123 | 20:35 |
opendevreview | Thobias Trevisan proposed openstack/python-openstackclient master: compute: Add 'Security Groups' for 'server list' https://review.opendev.org/c/openstack/python-openstackclient/+/819368 | 20:47 |
opendevreview | melanie witt proposed openstack/python-openstackclient stable/train: compute: Reorder building of columns for 'server list' https://review.opendev.org/c/openstack/python-openstackclient/+/821127 | 20:58 |
opendevreview | melanie witt proposed openstack/python-openstackclient stable/train: compute: Show flavor in 'server list' with API >= 2.47 https://review.opendev.org/c/openstack/python-openstackclient/+/821128 | 20:58 |
*** artom_ is now known as artom | 23:01 | |
opendevreview | melanie witt proposed openstack/python-openstackclient stable/train: compute: Show flavor in 'server list' with API >= 2.47 https://review.opendev.org/c/openstack/python-openstackclient/+/821128 | 23:27 |
*** diablo_rojo is now known as Guest8022 | 23:44 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!