| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-06-21 | |||
| 13:39:32 | gtema | I would say it is "conincedence" that you can reproduce it in OSC | |
| 13:39:42 | noonedeadpunk | then hopefully osc can pick that up and use... | |
| 13:39:48 | gtema | coincidence | |
| 13:40:26 | gtema | from osc pov we can fix that by switching to sdk. But in any way that problem would require novaclient fix | |
| 13:40:37 | gtema | or heat switching to SDK as well | |
| 13:40:51 | noonedeadpunk | I was thinking about heat switching to sdk tbh :D | |
| 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 | |