| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-06-21 | |||
| 13:51:27 | gtema | this is basic find: it tries to use name as filter and otherwise fall back to listing all and grepping through, but list by default fetches all pages | |
| 13:54:12 | noonedeadpunk | yeah, so it goes to implementing pagination in novaclient... ugh... what a disgusting thing to do | |
| 13:54:26 | noonedeadpunk | or switch heat to sdk | |
| 13:55:28 | gtema | as was discussed multiple summits and endless chats - ideally we have single API client tool that handles things just properly | |
| 13:55:41 | gtema | we all would have less mess to take care of | |
| 13:56:46 | noonedeadpunk | Yeah, so switching heat sounds like more proper solution I can be occupied with for next several days for sure. | |
| 13:57:06 | gtema | I would rather say - weeks | |
| 13:58:16 | noonedeadpunk | yeah... | |
| 13:59:15 | noonedeadpunk | And I can't even return back to finishing https://review.opendev.org/c/openstack/python-openstackclient/+/828708 :( | |
| 13:59:40 | gtema | LOL, it is as usual | |
| 14:00:27 | noonedeadpunk | ok. at least now I see several ways of going forward. All of them needs time | |
| 14:00:40 | gtema | :) | |
| 14:00:53 | gtema | generally you should not have so many flavors ;-) | |
| 14:00:55 | noonedeadpunk | and time to land as that's smth that won't be backported | |
| 14:01:45 | noonedeadpunk | 1715 to be percise. Now I'm thinking that I indeed can write super simple thing to clean out unused ones.... | |
| 14:02:26 | gtema | glad I gave you another idea. You can also increase nova pagination limit | |
| 14:04:18 | noonedeadpunk | I'm not sure where is that. I belive it should be in api-paste.ini? | |
| 14:04:35 | noonedeadpunk | and tbh 1000 is fair amount... | |
| 14:05:01 | gtema | sure, but still it is an option | |
| 14:05:49 | noonedeadpunk | yup, thanks again ! | |
| 14:05:58 | gtema | wlcm | |
| 14:14:01 | opendevreview | Artem Goncharov proposed openstack/ansible-collections-openstack master: Switch project_info to new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/845782 | |
| 14:27:39 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Implement supplementary list filtering https://review.opendev.org/c/openstack/openstacksdk/+/845726 | |
| 14:28:47 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Respect filters in list_projects https://review.opendev.org/c/openstack/openstacksdk/+/845776 | |
| 14:55:57 | noonedeadpunk | sorry, another thing.... Shouldn't list(conn.compute.servers(details=True))[0].flavor be an object rather then a simple dict? | |
| 14:56:17 | noonedeadpunk | I bet I asked that though several years ago.... | |
| 14:56:31 | noonedeadpunk | likely has smth to do with api version... | |
| 14:57:03 | gtema | an object in SDK implements dict interface | |
| 14:57:17 | noonedeadpunk | as that is empty https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/compute/v2/server.py#L111 | |
| 14:58:07 | noonedeadpunk | so basically there's no way to get flavor id or flavor name out of server? | |
| 14:58:16 | gtema | and here we may land in the microversions mess. Normally it is filled, but depending on the supported microversion you have one thing or another | |
| 14:58:41 | gtema | https://docs.openstack.org/api-ref/compute/?expanded=show-server-details-detail#show-server-details | |
| 14:58:42 | noonedeadpunk | I guess I need to set higher microversion explicitly.... | |
| 14:59:02 | gtema | with SDK you do not do this at all, SDK negotiates max supported MV by both sides | |
| 14:59:17 | gtema | but if server is old - it will not help | |
| 14:59:54 | gtema | flavor should be filled, and was filled once I looked at it last time | |
| 15:00:10 | noonedeadpunk | ok, yes, now I recall that | |
| 15:00:23 | gtema | maybe you look whether server returns this (you can do "openstack.enable_logging(debug=True)" to see requests | |
| 15:00:33 | noonedeadpunk | ah, it was likely openstackclient that always use min version | |
| 15:00:42 | gtema | right | |
| 15:01:09 | noonedeadpunk | So yes, as of 2.47 there's no way to get flavor uuid | |
| 15:01:09 | gtema | and once "server show" is switched to use SDK it will use latest MV by default | |
| 15:03:23 | noonedeadpunk | not sdk/client problem, which is good | |
| 15:03:33 | gtema | LOL | |
| 16:01:43 | noonedeadpunk | fwiw, in use there's only 237 flavors out of 1700 :D | |
| #openstack-sdks - 2022-06-22 | |||
| 00:30:06 | opendevreview | Yusuke Niimi proposed openstack/keystoneauth master: OAuth2.0 Client Credentials Grant Flow Support https://review.opendev.org/c/openstack/keystoneauth/+/830734 | |
| 09:03:17 | opendevreview | mbu proposed openstack/python-openstackclient master: Wait for volume being available to set bootable or readonly https://review.opendev.org/c/openstack/python-openstackclient/+/845991 | |
| 10:13:34 | opendevreview | Artem Goncharov proposed openstack/ansible-collections-openstack master: Add SDK logging option for openstack ansible collections. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844559 | |
| 11:25:54 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Makes security group rule info compatible with new sdk version https://review.opendev.org/c/openstack/ansible-collections-openstack/+/846148 | |
| 12:30:01 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Makes security group rule info compatible with new sdk version https://review.opendev.org/c/openstack/ansible-collections-openstack/+/846148 | |
| 16:41:45 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Makes security group rule info compatible with new sdk version https://review.opendev.org/c/openstack/ansible-collections-openstack/+/846148 | |
| #openstack-sdks - 2022-06-23 | |||
| 09:34:34 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored endpoint module and explained region attribute https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847293 | |
| 10:07:15 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: [DNM] keypair_info test https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847299 | |
| 11:06:38 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: [DNM] keypair_info test https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847299 | |
| 13:12:10 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Update project_info module to new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/837276 | |
| #openstack-sdks - 2022-06-24 | |||
| 07:14:50 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Applied workaround in CI for issue #78017 in ansible-core https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847529 | |
| 09:20:24 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: (wip) Refactor network fakes to sdk properties PART 5 https://review.opendev.org/c/openstack/python-openstackclient/+/846167 | |
| 09:20:25 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: (wip) Refactor network fakes to sdk properties PART6 https://review.opendev.org/c/openstack/python-openstackclient/+/847553 | |
| 12:04:06 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Applied workaround in CI for issue #78017 in ansible-core https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847529 | |
| 12:36:37 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: block storage: Added support for the GroupSnapshot resource https://review.opendev.org/c/openstack/openstacksdk/+/781163 | |
| 12:36:37 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: block storage: Add support for the Group resource https://review.opendev.org/c/openstack/openstacksdk/+/782013 | |
| 12:36:44 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: test: Remove duplicated tests https://review.opendev.org/c/openstack/openstacksdk/+/847577 | |
| 12:36:45 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: block storage: Add support for group type specs https://review.opendev.org/c/openstack/openstacksdk/+/847579 | |
| 12:36:45 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Add pointers to docs on running tests https://review.opendev.org/c/openstack/openstacksdk/+/847578 | |
| 13:11:44 | frickler | how do I set --os-interface in clouds.yaml? neither "interface: internal" nor "os_interface: internal" seem to work | |
| 13:13:55 | frickler | hmm, when I tried to search for it, I found https://storyboard.openstack.org/#!/story/2007380 in my browser history. need to check which version I'm at | |
| 13:20:05 | frickler | yep, osc-lib 2.6.0 which has the fix is only in zed. not sure if we'd want to backport that fix or bump reqs for stable branches. | |
| 15:30:43 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Applied workaround in CI for issue #78017 in ansible-core https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847529 | |
| 15:32:15 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Applied workaround in CI for issue #78017 in ansible-core https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847529 | |
| 15:36:01 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Applied workaround in CI for issue #78017 in ansible-core https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847529 | |
| #openstack-sdks - 2022-06-25 | |||
| 08:51:36 | frickler | all osc subnet commands show "The option [tenant_id] has been deprecated. Please avoid using it." | |
| 08:57:00 | frickler | also https://storyboard.openstack.org/#!/story/2009991 makes me sad | |
| 11:10:15 | gtema | There is a change to disable that s logging. Moreover we can try ensure OSC does not send tenant_id or find reason why this warning is being triggered | |
| 11:10:29 | gtema | Will also have a look on storyboard issue | |
| #openstack-sdks - 2022-06-26 | |||
| 11:09:26 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Applied workaround in CI for issue #78017 in ansible-core https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847529 | |
| 14:20:01 | opendevreview | Merged openstack/ansible-collections-openstack master: Applied workaround in CI for issue #78017 in ansible-core https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847529 | |
| 14:31:20 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Applied workaround in CI for issue #78017 in ansible-core https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847655 | |
| 17:48:14 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Add SDK logging option for openstack ansible collections. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844559 | |
| 19:03:59 | opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Applied workaround in CI for issue #78017 in ansible-core https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847655 | |
| #openstack-sdks - 2022-06-27 | |||
| 05:43:22 | frickler | gtema: iiuc the tenant_id warning is triggered not because OSC is using it, but because the API response from neutron still contains that field. | |
| 06:10:06 | gtema | Ok, having long day on the train today, maybe will have enough capability to check. That mean to me out deprecation warning is not really raised properly. And it pains on few fronts so maybe we should simply disable it | |
| 06:10:40 | gtema | Older clouds have all rights to use fields not yet deprecated for them | |
| 06:17:04 | frickler | gtema: yes, although that even is with neutron master still. not sure if we could hard drop tenant_id from the API response if project_id is present. otherwise disabling that deprecation might be the right way to do, yes | |
| 06:19:31 | gtema | There's even change open for dropping warnings | |
| 07:02:13 | frickler | maybe make that specific to tenant_id. I'm not sure there is another attribute that is deprecated but still in widespread use. the other option would be to check whether the resource is coming from user input or from the API, but I guess that distinction would be difficult to make | |
| 07:03:02 | gtema | Well, we can add additional (suppress_warning) attribute ;-) | |
| 07:11:56 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Make publish_collection more universal https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847659 | |
| 07:15:39 | frickler | but then everyone will set that and in the end we could have just dropped the warnings right away | |
| 07:15:51 | frickler | gtema: just found that this one was wrong, too https://review.opendev.org/c/openstack/python-openstackclient/+/824746/2/openstackclient/network/v2/security_group.py#b38 | |
| 07:15:55 | gtema | Right | |
| 07:16:12 | frickler | makes tenant_id show up in the rules. I'll do a revert for that line | |
| 07:16:38 | gtema | Yeah, this is what I meant - something explicitly touches attr | |
| 07:21:45 | frickler | oh, wait, it's not only that single line that was wrong, need to revert all the column_map changes in that patch, too. /me sighs | |
| 08:02:21 | frickler | this is really a can of worms, why do floating IPs have a subnet attr? can't find anything about that in the api-ref. maybe it was a nova-network thing? | |
| 08:03:07 | frickler | and I'm also not convinced that copying the floating_ip_address as "name" attribute is a good thing | |
| 08:03:44 | gtema | Right, networking is a pretty dangerous thing | |
| 08:53:46 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Don't show tenant_id in network objects https://review.opendev.org/c/openstack/python-openstackclient/+/847751 | |
| 08:53:47 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: WIP: Test for tenant_id and project_id columns https://review.opendev.org/c/openstack/python-openstackclient/+/847752 | |
| 08:54:40 | opendevreview | ribaudr proposed openstack/python-openstackclient master: Microversion 2.91: Support specifying destination host to unshelve https://review.opendev.org/c/openstack/python-openstackclient/+/831902 | |
| 08:58:43 | gtema | frickler - https://review.opendev.org/c/openstack/openstacksdk/+/816790 was to disable deprecation warning | |