| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-05-05 | |||
| 12:54:12 | frickler | gtema: noonedeadpunk: actually https://review.opendev.org/c/openstack/openstacksdk/+/783849 is causing the above regression, seems it was a mistake to ignore the failing osc job, sadly logs have already expired | |
| 12:54:37 | frickler | also not sure if you can find a quick fix or want to revert first | |
| 12:54:37 | frickler | also not sure if you can find a quick fix or want to revert first | |
| 12:57:06 | gtema | not that I understand why it is causing this | |
| 12:57:06 | gtema | not that I understand why it is causing this | |
| 12:57:18 | noonedeadpunk | this one https://review.opendev.org/c/openstack/openstacksdk/+/783849/2/openstack/image/v2/image.py | |
| 12:57:18 | noonedeadpunk | this one https://review.opendev.org/c/openstack/openstacksdk/+/783849/2/openstack/image/v2/image.py | |
| 12:57:55 | noonedeadpunk | so yeah, might be worth at least partially reverting it... | |
| 12:57:55 | noonedeadpunk | so yeah, might be worth at least partially reverting it... | |
| 12:58:38 | noonedeadpunk | or at least really quick fix | |
| 12:58:38 | noonedeadpunk | or at least really quick fix | |
| 12:59:26 | gtema | someone show me logs where it fails? | |
| 12:59:26 | gtema | someone show me logs where it fails? | |
| 12:59:36 | gtema | I was not really looking into the discussion recently | |
| 12:59:36 | gtema | I was not really looking into the discussion recently | |
| 12:59:57 | noonedeadpunk | It tries to ` tempest.lib.exceptions.CommandFailed: Command 'openstack --os-cloud=devstack-admin image list --tag my_tag --long -f json' returned non-zero exit status 1.` | |
| 12:59:57 | noonedeadpunk | It tries to ` tempest.lib.exceptions.CommandFailed: Command 'openstack --os-cloud=devstack-admin image list --tag my_tag --long -f json' returned non-zero exit status 1.` | |
| 13:01:48 | gtema | I need to see API call | |
| 13:01:48 | gtema | I need to see API call | |
| 13:02:52 | noonedeadpunk | well with patch I renamed option I gues :((( | |
| 13:02:52 | noonedeadpunk | well with patch I renamed option I gues :((( | |
| 13:04:30 | noonedeadpunk | I jsut kind of assumed, that it can't be used since it's not really working properly | |
| 13:04:30 | noonedeadpunk | I jsut kind of assumed, that it can't be used since it's not really working properly | |
| 13:05:05 | openstackgerrit | Dmitriy Rabotyagov proposed openstack/openstacksdk master: Revert tags option back to tag https://review.opendev.org/c/openstack/openstacksdk/+/789827 | |
| 13:05:37 | noonedeadpunk | not super sure about that though ^ | |
| 13:05:37 | noonedeadpunk | not super sure about that though ^ | |
| 13:06:26 | noonedeadpunk | I don't think it even comes to API call | |
| 13:06:26 | noonedeadpunk | I don't think it even comes to API call | |
| 13:06:47 | gtema | noho, the fix should be in https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/image/v2/image.py#L613 | |
| 13:06:47 | gtema | noho, the fix should be in https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/image/v2/image.py#L613 | |
| 13:06:48 | noonedeadpunk | it fails just on running openstack as unknown option is passed | |
| 13:06:48 | noonedeadpunk | it fails just on running openstack as unknown option is passed | |
| 13:07:08 | gtema | compare with https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/image/v2/image.py#L323 | |
| 13:07:08 | gtema | compare with https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/image/v2/image.py#L323 | |
| 13:07:42 | gtema | this should be also constructing array of tags | |
| 13:07:42 | gtema | this should be also constructing array of tags | |
| 13:07:44 | frickler | hmm, for list-image, the filter is called "tag", but for show-image, a "tags" field is returned https://docs.openstack.org/api-ref/image/v2/index.html?expanded=list-images-detail,show-image-detail#list-images | |
| 13:07:44 | frickler | hmm, for list-image, the filter is called "tag", but for show-image, a "tags" field is returned https://docs.openstack.org/api-ref/image/v2/index.html?expanded=list-images-detail,show-image-detail#list-images | |
| 13:08:15 | noonedeadpunk | oh, well, I think you're right gtema | |
| 13:08:15 | noonedeadpunk | oh, well, I think you're right gtema | |
| 13:08:45 | noonedeadpunk | frickler: well, you can use --tag several times, right? | |
| 13:08:46 | noonedeadpunk | frickler: well, you can use --tag several times, right? | |
| 13:08:51 | gtema | noonedeadpunk - will you prepare change or should I? | |
| 13:08:51 | gtema | noonedeadpunk - will you prepare change or should I? | |
| 13:09:10 | gtema | yes - you should be able to have same interface of repeating multiple --tag | |
| 13:09:10 | gtema | yes - you should be able to have same interface of repeating multiple --tag | |
| 13:09:23 | noonedeadpunk | doesn't matter for me:) | |
| 13:09:23 | noonedeadpunk | doesn't matter for me:) | |
| 13:09:38 | noonedeadpunk | probably to merge things faster (in terms of self vote) I can push it | |
| 13:09:38 | noonedeadpunk | probably to merge things faster (in terms of self vote) I can push it | |
| 13:09:40 | gtema | if you have time - please, I'm pretty overloaded right now | |
| 13:09:40 | gtema | if you have time - please, I'm pretty overloaded right now | |
| 13:27:24 | noonedeadpunk | sure, testing it :) | |
| 13:27:24 | noonedeadpunk | sure, testing it :) | |
| 13:27:35 | gtema | thks | |
| 13:40:16 | noonedeadpunk | gtema: nah, I was right with https://review.opendev.org/c/openstack/openstacksdk/+/789827 | |
| 13:40:16 | noonedeadpunk | gtema: nah, I was right with https://review.opendev.org/c/openstack/openstacksdk/+/789827 | |
| 13:40:43 | gtema | why? | |
| 13:40:43 | gtema | why? | |
| 13:40:51 | noonedeadpunk | as with your suggestion sdk sends request /v2/images?tags=openstack | |
| 13:40:51 | noonedeadpunk | as with your suggestion sdk sends request /v2/images?tags=openstack | |
| 13:41:08 | noonedeadpunk | and glance list api has only tag | |
| 13:41:09 | noonedeadpunk | and glance list api has only tag | |
| 13:41:29 | noonedeadpunk | while create is tags | |
| 13:41:30 | noonedeadpunk | while create is tags | |
| 13:41:48 | noonedeadpunk | https://docs.openstack.org/api-ref/image/v2/index.html?expanded=list-images-detail,create-image-detail#id10 | |
| 13:41:48 | noonedeadpunk | https://docs.openstack.org/api-ref/image/v2/index.html?expanded=list-images-detail,create-image-detail#id10 | |
| 13:42:28 | gtema | oh thats not cool | |
| 13:42:28 | gtema | oh thats not cool | |
| 13:42:41 | noonedeadpunk | so there's kind of inconsistency in glance itself I really skipped | |
| 13:42:41 | noonedeadpunk | so there's kind of inconsistency in glance itself I really skipped | |
| 13:43:56 | gtema | and what if we pass array of tags as "tag" into SDK. What does it send to glance? I roughly remenber some other case | |
| 13:43:56 | gtema | and what if we pass array of tags as "tag" into SDK. What does it send to glance? I roughly remenber some other case | |
| 13:44:19 | gtema | where we need to send same attirbute multiple times, but I can't recall where it was | |
| 13:44:20 | gtema | where we need to send same attirbute multiple times, but I can't recall where it was | |
| 13:44:25 | noonedeadpunk | it's sending only last mentioned one | |
| 13:44:25 | noonedeadpunk | it's sending only last mentioned one | |
| 13:44:50 | noonedeadpunk | ie `openstack image list --tag openstack --tag test ` ends up in `/v2/images?tag=test` | |
| 13:44:50 | noonedeadpunk | ie `openstack image list --tag openstack --tag test ` ends up in `/v2/images?tag=test` | |
| 13:45:07 | noonedeadpunk | but maybe we can make it list... not sure | |
| 13:45:07 | noonedeadpunk | but maybe we can make it list... not sure | |
| 13:45:25 | gtema | there is way, I'm searching | |
| 13:45:25 | gtema | there is way, I'm searching | |
| 13:45:56 | noonedeadpunk | Yeah, I think we can just change to append here | |
| 13:45:56 | noonedeadpunk | Yeah, I think we can just change to append here | |
| 13:46:14 | noonedeadpunk | https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/image/v2/image.py#L612-L616 | |
| 13:46:14 | noonedeadpunk | https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/image/v2/image.py#L612-L616 | |
| 13:47:02 | noonedeadpunk | will produce /v2/images?tag=openstack&tag=test | |
| 13:47:02 | noonedeadpunk | will produce /v2/images?tag=openstack&tag=test | |
| 13:47:08 | gtema | that's what I meant, but exactly here I'm not sure how SDK passes this to glance | |
| 13:47:08 | gtema | that's what I meant, but exactly here I'm not sure how SDK passes this to glance | |
| 13:49:51 | noonedeadpunk | doh. indeed | |
| 13:49:51 | noonedeadpunk | doh. indeed | |
| 13:51:01 | noonedeadpunk | reverting like 789827 breaks image update... | |
| 13:51:01 | noonedeadpunk | reverting like 789827 breaks image update... | |
| 13:53:45 | openstackgerrit | Dmitriy Rabotyagov proposed openstack/openstacksdk master: Revert tags option back to tag https://review.opendev.org/c/openstack/openstacksdk/+/789827 | |
| 13:59:34 | openstackgerrit | Dmitriy Rabotyagov proposed openstack/python-openstackclient master: Allow to filter multiple tags for image list https://review.opendev.org/c/openstack/python-openstackclient/+/789838 | |
| 14:23:33 | nightmare_unreal | https://www.irccloud.com/pastebin/1JKwl1hg/ | |
| 14:23:33 | nightmare_unreal | https://www.irccloud.com/pastebin/1JKwl1hg/ | |
| 14:28:37 | openstackgerrit | Dmitriy Rabotyagov proposed openstack/openstacksdk master: Revert tags option back to tag https://review.opendev.org/c/openstack/openstacksdk/+/789827 | |