| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-12-04 | |||
| 16:51:41 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch remove fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820420 | |
| 16:54:47 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch remove fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820420 | |
| 21:32:01 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch remove fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820420 | |
| 21:39:29 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch remove fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820420 | |
| 21:45:39 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch remove fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820420 | |
| #openstack-sdks - 2021-12-05 | |||
| 15:40:00 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Switch remove fixed IP to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820420 | |
| 22:10:51 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch server dump create to using sdk https://review.opendev.org/c/openstack/python-openstackclient/+/816296 | |
| 22:42:23 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch list server volume to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/816716 | |
| #openstack-sdks - 2021-12-06 | |||
| 02:00:43 | opendevreview | Ritvik Vinodkumar proposed openstack/python-openstackclient master: Fixing issue with functional test https://review.opendev.org/c/openstack/python-openstackclient/+/820466 | |
| 08:30:12 | slaweq | gtema hi, can You check https://review.opendev.org/c/openstack/python-openstackclient/+/819024 once again when You will have some time? thx in advance | |
| 08:45:32 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test CI https://review.opendev.org/c/openstack/ansible-collections-openstack/+/820480 | |
| 10:27:59 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Improve logging for executed commands https://review.opendev.org/c/openstack/python-openstackclient/+/820512 | |
| 13:45:44 | sshnaidm | when is "files" in config-drive deprecated? https://github.com/openstack/openstacksdk/blob/215403731db17e14506d2d83b17afd6df6d257d4/openstack/cloud/_compute.py#L687 | |
| 13:45:57 | sshnaidm | is there replacement? | |
| 14:35:23 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Improve logging for executed commands https://review.opendev.org/c/openstack/python-openstackclient/+/820512 | |
| 14:43:44 | opendevreview | Thobias Trevisan proposed openstack/python-openstackclient master: compute: Add 'Security Groups' for 'server list' https://review.opendev.org/c/openstack/python-openstackclient/+/819368 | |
| 17:34:21 | opendevreview | Diwei Zhu proposed openstack/python-openstackclient master: Switch openstack server show to using sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/817871 | |
| #openstack-sdks - 2021-12-07 | |||
| 10:22:29 | frickler | gtema: could you have another look at https://review.opendev.org/c/openstack/python-openstackclient/+/736973 pls? I had messed up pep8. slaweq this may also be interesting for you | |
| 10:23:08 | gtema | sure, but first need to find reason for one functest failure we currently have | |
| 10:24:25 | frickler | gtema: I haven't seen that, do you have a pointer? | |
| 10:24:52 | gtema | https://review.opendev.org/c/openstack/python-openstackclient/+/820512 | |
| 10:26:27 | frickler | volumes once again ... sigh ... will take a look | |
| 10:26:38 | gtema | thks a lot | |
| 13:15:05 | frickler | gtema: so I can reproduce locally and if I revert https://review.opendev.org/c/openstack/python-openstackclient/+/817989/8 the issue is gone. maybe try to revert that for now? | |
| 13:15:58 | gtema | we thought yesterday on marking the test as expected to fail | |
| 13:16:23 | gtema | https://review.opendev.org/c/openstack/python-openstackclient/+/819473/1 added the functest itself (was not tested before at all) | |
| 13:18:34 | frickler | gtema: my current suspicion is that the test is good, but 817989 broken | |
| 13:18:45 | frickler | or is might be uncovering some issue in nova | |
| 13:19:58 | gtema | yeah, everything is possible | |
| 13:29:25 | frickler | so 817989 is broken for sure. nova returns a 404 for the delete_volume_attachment(), but the code ignores that instead of raising an error | |
| 13:31:08 | frickler | gtema: stephenfin: ^^ | |
| 13:34:24 | frickler | e.g. 15:08:07.785042 in https://210be1f5cbeabbdaba80-eeccdc6968a6b16fa4ca2e3ee7c1080d.ssl.cf2.rackcdn.com/820512/2/check/osc-functional-devstack-tips/4af9a0d/controller/logs/screen-n-api.txt | |
| 13:41:38 | frickler | delete_volume_attachment() expects the ID of the attachment, while the OSC patch hands it the volume ID | |
| 13:42:36 | frickler | hmm, no, ignore that | |
| 13:58:29 | frickler | the sdk code is broken it seems, it uses the attachment_id for the _delete call instead of the volume id | |
| 13:59:36 | gtema | ouch, bad | |
| 14:02:40 | frickler | this may be somehow related to https://review.opendev.org/c/openstack/openstacksdk/+/817997/1/openstack/compute/v2/volume_attachment.py , but I'm kinda confused about it still. I do remember trying to make sense of the nova api-ref there | |
| 14:03:36 | frickler | I also don't understand how it could pass CI if that were true | |
| 14:05:34 | gtema | and so the loop closes :-) | |
| 14:05:49 | gtema | we started failing with the raising of openstacksdk | |
| 14:14:28 | frickler | oh, wow, this is even weirder. with old microversion, nova returns the volume_id as attachment_id. with 2.89, it returns the real attachment_id. | |
| 14:18:08 | frickler | so that's actually the non-voting failure for 817997 https://zuul.opendev.org/t/openstack/build/1f7a3ab5e8b942c2b7b58b9ec2e27367 | |
| 14:18:53 | frickler | and it has been failing since then https://zuul.opendev.org/t/openstack/builds?job_name=osc-functional-devstack-tips&project=openstack/openstacksdk | |
| 14:19:53 | frickler | gtema: stephenfin: ^^ I have no idea how to actually fix that, so maybe that would be the patch to revert for now | |
| 14:28:35 | gtema | uhm, reverting sdk patch would require release to unblock SDK. Perhaps we should rather really mark functest as failing while we are searching for proper fix | |
| 14:52:24 | frickler | gtema: do you really mean @unittest.expectedFailure or just skipping the test? I haven't seen the former used in openstack yet, though it might indeed be useful | |
| 14:53:15 | gtema | expectedFailure perhaps | |
| 14:53:38 | frickler | o.k., I'll propose a patch | |
| 14:54:23 | gtema | great, thks | |
| 14:58:40 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Mark test_server_add_remove_volume as expected failure https://review.opendev.org/c/openstack/python-openstackclient/+/820895 | |
| 15:06:24 | frickler | meh, that is lacking an import | |
| 15:11:33 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Mark test_server_add_remove_volume as expected failure https://review.opendev.org/c/openstack/python-openstackclient/+/820895 | |
| 17:36:46 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Fix RemoveServerVolume https://review.opendev.org/c/openstack/python-openstackclient/+/820917 | |
| 17:38:29 | frickler | gtema: stephenfin: ^^ this would actually seem to be the fix for OSC matching the current SDK behavior. not sure if you'd want to use that and just amend the descriptions of the sdk functions accordingly, or whether you have an idea to make this volume attachment thing work both with and without 2.89 | |
| 17:39:21 | gtema | ideally should work with and without. Will have a look tomorrow | |
| 17:39:26 | gtema | thanks a lot | |
| 17:49:39 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Add network update quota "limit_check" parameter https://review.opendev.org/c/openstack/python-openstackclient/+/806016 | |
| 17:56:07 | opendevreview | Jan Hartkopf proposed openstack/openstacksdk master: fix creation of protected image https://review.opendev.org/c/openstack/openstacksdk/+/820926 | |
| 23:06:20 | opendevreview | melanie witt proposed openstack/python-openstackclient stable/victoria: compute: Reorder building of columns for 'server list' https://review.opendev.org/c/openstack/python-openstackclient/+/820959 | |
| 23:06:20 | 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 | |
| #openstack-sdks - 2021-12-08 | |||
| 00:41:25 | 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 | |
| 15:09:26 | thomasb06 | gtema: Hello. Shared file system: https://review.opendev.org/c/openstack/openstacksdk/+/821083 | |
| 16:29:06 | opendevreview | Ujjval proposed openstack/python-openstackclient master: compute: Switch server group to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820070 | |
| 17:57:54 | 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:58:09 | stephenfin | *weird nova API behaviour front | |
| 17:59:50 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Fix RemoveServerVolume https://review.opendev.org/c/openstack/python-openstackclient/+/820917 | |
| 18:10:31 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Switch server group to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/820070 | |
| 18:17:11 | 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:35 | stephenfin | wdym? | |
| 18:22:35 | 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:23:10 | frickler | the=delete_volume_attachment() | |
| 18:23:38 | frickler | in addition, the help text also claims that the server argument might be optional, which it isn't | |
| 18:24:45 | 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:50 | stephenfin | This is another thing to add to the list | |
| 20:15:16 | opendevreview | Merged openstack/python-openstackclient master: Fix RemoveServerVolume https://review.opendev.org/c/openstack/python-openstackclient/+/820917 | |
| 20:35:20 | 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:21 | 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:47:40 | 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:58:16 | 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:17 | 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:18 | 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 | |
| #openstack-sdks - 2021-12-09 | |||
| 12:42:03 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 13:49:28 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Allow setting gateway when creating a router https://review.opendev.org/c/openstack/python-openstackclient/+/736973 | |
| 15:37:20 | opendevreview | Merged openstack/python-openstackclient master: Allow setting gateway when creating a router https://review.opendev.org/c/openstack/python-openstackclient/+/736973 | |
| 17:23:47 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Improve logging for executed commands https://review.opendev.org/c/openstack/python-openstackclient/+/820512 | |
| 18:09:28 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute: Add support for triggering crash dumps https://review.opendev.org/c/openstack/openstacksdk/+/821270 | |
| 18:41:53 | opendevreview | Suneeth Ravi proposed openstack/python-openstackclient master: Switch server lock to sdk. https://review.opendev.org/c/openstack/python-openstackclient/+/819517 | |
| #openstack-sdks - 2021-12-10 | |||
| 04:46:38 | opendevreview | melanie witt proposed openstack/python-openstackclient master: Add 'Host Status' to 'server list --long' with >= v2.16 https://review.opendev.org/c/openstack/python-openstackclient/+/818306 | |
| 07:26:06 | thomasb06 | gtema: Hello. An old commit goes wrong, would you have an idea why: https://review.opendev.org/c/openstack/openstacksdk/+/814395 ? | |
| 08:25:51 | thomasb06 | Network: https://review.opendev.org/c/openstack/openstacksdk/+/821373 | |
| 09:22:57 | frickler | thomasb06: adding a blank line in front of the ":rtype::" lines fixes 814395 for me | |
| 09:34:34 | opendevreview | LEDUC Florian proposed openstack/openstacksdk master: Add get_actions for server resource https://review.opendev.org/c/openstack/openstacksdk/+/675113 | |
| 09:36:02 | thomasb06 | frickler: thanks for the hint, maybe it's fixed now | |
| 19:52:00 | fungi | mordred: (and anyone else who's a debian developer), responses to the ml thread at http://lists.spi-inc.org/pipermail/spi-general/2021-December/004172.html with your opinions would be most helpful | |
| #openstack-sdks - 2021-12-13 | |||
| 07:37:57 | isabek | stephenfin: Good morning! Can you please take a look to this patch [1] when you have a time? Fixed all comments. Thanks in advance! 1) https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 08:03:39 | opendevreview | LEDUC Florian proposed openstack/openstacksdk master: Add get_actions for server resource https://review.opendev.org/c/openstack/openstacksdk/+/675113 | |
| 08:16:56 | slaweq | gtema hi, if You will have some time, please check https://review.opendev.org/c/openstack/python-openstackclient/+/819024 | |
| 08:17:15 | gtema | Ok | |
| 08:17:22 | slaweq | thx | |
| 08:26:41 | opendevreview | LEDUC Florian proposed openstack/openstacksdk master: Add get_actions for server resource https://review.opendev.org/c/openstack/openstacksdk/+/675113 | |