| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-06-21 | |||
| 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 | gtema | and once "server show" is switched to use SDK it will use latest MV by default | |
| 15:01:09 | noonedeadpunk | So yes, as of 2.47 there's no way to get flavor uuid | |
| 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: Add support for the Group resource https://review.opendev.org/c/openstack/openstacksdk/+/782013 | |
| 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: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: tests: Add pointers to docs on running tests https://review.opendev.org/c/openstack/openstacksdk/+/847578 | |
| 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 | |
| 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 | |
| 10:25:37 | frickler | gtema: yes, I found that, that's what I was referring to earlier. don't disable all warnings, only if attr="tenant_id", then I think we should merge that | |
| 10:27:12 | gtema | I fear this is really more widespread than only tenant_id, but ok, will update the change | |
| 10:39:48 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Disable deprecation warning for tenant_id https://review.opendev.org/c/openstack/openstacksdk/+/816790 | |
| 10:42:33 | gtema | frickler: ^^ | |
| 11:12:54 | 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 | |
| 11:34:27 | 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 | |
| 12:45:35 | 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 | |
| 12:48:51 | opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Make publish_collection more universal https://review.opendev.org/c/openstack/ansible-collections-openstack/+/847659 | |
| 13:11:34 | frickler | it seems that https://review.opendev.org/c/openstack/python-openstackclient/+/828106 is responsible for much of the functional test instability we are seeing | |
| 13:12:10 | frickler | it sets the network quota to 1, leading to failures if any other test runs in parallel and tries to create a network | |
| 13:13:14 | frickler | test_quota_network_set_with_force() in particular. not sure what ralonsoh was thinking there. I'll make a patch to skip that test | |
| 13:25:23 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Skip test_quota_network_set_with_force func test https://review.opendev.org/c/openstack/python-openstackclient/+/847786 | |
| 13:29:23 | frickler | is there any chance we could revert to using launchpad instead of storyboard? trying to find out whether a story already exists for an issue seems impossible to me | |
| 13:33:40 | gtema | I feel also terribly about storyboard. Honestly I want to see the outcome of the general 3rd party projects discussion, cause using gitea issue tracker is at least mentioned there. If we get a chance to get this working I would rather go this way and skip fallback to launchpad now (it has also not really an awesome UX) | |
| 13:56:43 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves security_group_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/840660 | |
| 14:05:21 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves security_group_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/840660 | |
| 14:11:09 | opendevreview | Jan Hartkopf proposed openstack/python-openstackclient master: add support for microversion 2.91 https://review.opendev.org/c/openstack/python-openstackclient/+/847792 | |
| 14:28:53 | opendevreview | Merged openstack/osc-lib master: Do not output log on bash autocomplete generation https://review.opendev.org/c/openstack/osc-lib/+/841307 | |
| 14:44:36 | slaweq | gtema hi, I have a quick question to You about this neutron-vpnaas support in OSC/SDK - is there any story or something created already so if I would like to help with it, I will not e.g. duplicate work done by someone else already? | |
| 14:45:16 | gtema | we do not have a story, since nobody here is happy about using Storyboard | |
| 14:46:19 | slaweq | so there is no any tracker for that now? | |
| 14:46:26 | gtema | nope | |
| 14:46:35 | slaweq | ok | |