| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-06-21 | |||
| 13:41:12 | noonedeadpunk | But I assume it's not easy thing to do. | |
| 13:41:13 | gtema | yeah, all question of effort and available resources | |
| 13:41:24 | gtema | well, it is relatively easy, just lot work | |
| 13:42:26 | gtema | it even already uses SDK inside: https://opendev.org/openstack/heat/src/branch/master/heat/engine/clients/os/openstacksdk.py | |
| 13:42:56 | gtema | https://opendev.org/openstack/heat/src/branch/master/heat/engine/clients/os/nova.py - here you have novaclient usage | |
| 13:43:14 | gtema | so technically it is not that complex to exchange one client with another | |
| 13:45:34 | noonedeadpunk | hm, from what I can see mine usecase looks not that bad https://opendev.org/openstack/heat/src/branch/master/heat/engine/clients/os/nova.py#L271-L281 | |
| 13:45:38 | noonedeadpunk | interesting | |
| 13:46:02 | gtema | yes, should be | |
| 13:46:47 | noonedeadpunk | but likely pagination just not present inside novaclient at all | |
| 13:46:56 | noonedeadpunk | likely it's easiest thing to fix then | |
| 13:47:04 | noonedeadpunk | thanks for your time gtema! | |
| 13:47:09 | gtema | welcome | |
| 13:47:24 | gtema | if I read code properly: https://opendev.org/openstack/python-novaclient/src/branch/master/novaclient/base.py#L411 | |
| 13:48:25 | gtema | and that ends in https://opendev.org/openstack/python-novaclient/src/branch/master/novaclient/v2/flavors.py#L148 | |
| 13:48:30 | gtema | and there is no word of pagination | |
| 13:49:36 | noonedeadpunk | I should rather check how it's done in sdk :) | |
| 13:50:39 | gtema | https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/resource.py#L2068 | |
| 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 | |