| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-05-18 | |||
| 12:05:54 | gtema | okay | |
| 12:06:04 | stephenfin | (it all adds up) | |
| 12:06:18 | gtema | sure, it always does | |
| 12:10:11 | gtema | import of cmd2 inly due to exception is really nice catch. | |
| 14:06:48 | opendevreview | Jan Horstmann proposed openstack/ansible-collections-openstack master: Return details in baremetal_node_info when iterating over all machines https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839776 | |
| 14:37:30 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update catalog service for the new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839352 | |
| 16:24:56 | frickler | deliverables/zed/openstacksdk.yaml: validate_version_numbers: could not validate version '1.0.0.0rc1': Version 1.0.0.0rc1 looks like a pre-release and the release model does not allow for it | |
| 16:25:03 | frickler | :( | |
| 16:25:18 | gtema | ugh, sad | |
| 16:25:57 | gtema | do you know where release model is managed at all? I was searching for how to go to independent model and couldn't find | |
| 16:26:15 | gtema | maybe I have overseen something though | |
| 16:30:47 | gtema | hm, this may be really just release-model inside individual yaml file | |
| 16:48:12 | frickler | gtema: doesn't seem to work, either. I've asked in #openstack-release for help now | |
| 16:48:56 | frickler | but good to see 1.0.0 coming closer finally \o/ | |
| 16:51:44 | gtema | Ok, maybe need to move it to _independent folder completely | |
| 16:51:53 | gtema | Cool, let's see what they say | |
| 19:11:31 | opendevreview | Merged openstack/python-openstackclient master: Add 'Host Status' to 'server list --long' with >= v2.16 https://review.opendev.org/c/openstack/python-openstackclient/+/818306 | |
| #openstack-sdks - 2022-05-19 | |||
| 08:28:40 | frickler | gtema: could you join #openstack-release? we're discussing whether there is really a need for an rc1 release | |
| 08:30:53 | gtema | yupp, but I am not able to be very active now - am in the meeting | |
| 14:40:52 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Adds recordset_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/842550 | |
| 14:58:56 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update project_info module to new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/837276 | |
| 16:40:05 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update recordset module to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/842575 | |
| 17:41:21 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Adds recordset_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/842550 | |
| 17:56:20 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Adds recordset_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/842550 | |
| 18:26:00 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Adds recordset_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/842550 | |
| #openstack-sdks - 2022-05-20 | |||
| 09:32:10 | opendevreview | Lajos Katona proposed openstack/python-openstackclient master: Add support for CRUD operations for QoS minimum packet rate rule https://review.opendev.org/c/openstack/python-openstackclient/+/810559 | |
| 13:15:21 | frickler | \o/ Merged openstack/releases master: Release openstacksdk for Zed-1 milestone https://review.opendev.org/c/openstack/releases/+/841844 | |
| 13:17:53 | frickler | its a bit sad that this section is labeled "Unreleased Versions" when they actually are released and active on pypi https://docs.openstack.org/releasenotes/openstacksdk/unreleased.html | |
| 13:46:05 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: Change title for "unreleased" renos https://review.opendev.org/c/openstack/openstacksdk/+/842714 | |
| 13:47:29 | frickler | gtema: ^^ elod pointed out that we can simply change this. I hope this will help avoid some confusion | |
| 13:47:59 | frickler | although I now notice that the prelude was meant for 1.0.0 really. I guess we can just update and repeat it there | |
| 14:05:53 | gtema | thks frickler, approved. I was myself pretty confused with it | |
| 14:40:52 | opendevreview | Merged openstack/openstacksdk master: Change title for "unreleased" renos https://review.opendev.org/c/openstack/openstacksdk/+/842714 | |
| 20:39:58 | mnaser | frickler: welp, ansible-collection-openstack is broken :) | |
| 20:46:49 | mnaser | https://review.opendev.org/c/openstack/openstacksdk/+/820926 | |
| 20:46:51 | mnaser | i think this is teh issue | |
| 20:49:39 | mnaser | https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839033 | |
| 20:49:58 | mnaser | i think we'll need a release of the ansible collection? | |
| 21:26:51 | frickler | iiuc you need to cap openstacksdk<0.99 until a-c-o has fixed all issues | |
| #openstack-sdks - 2022-05-22 | |||
| 06:48:57 | eandersson | Starting with openstacksdk==0.99 flavod id is now represented by the flavor name and not the flavor id. This breaks Senlin. Is that intended? | |
| 06:49:26 | eandersson | *flavor | |
| 06:54:50 | eandersson | Actually looks like the openstacksdk returns more data now and one of the fields is flavor id (but represenated by the flavor name instead) | |
| 06:58:47 | eandersson | So basically compute.get_server now returns flavor id with 0.99 while in 0.61 it did not return a flavor id. | |
| 06:59:42 | eandersson | *include | |
| 07:18:36 | eandersson | It seems odd that flavor_find would return an id, but the flavor field in server_get returns a "name" instead of "id" in the if field. | |
| 07:21:40 | eandersson | https://review.opendev.org/c/openstack/senlin/+/842851 | |
| 07:35:21 | frickler | eandersson: yes that is intended, let me find the fix we did in nodepool | |
| 07:36:38 | frickler | https://review.opendev.org/c/zuul/nodepool/+/834152 has some extended discussion on this very issue | |
| 07:38:56 | frickler | the issue is that original_name can point to a flavor that has since been deleted. so you cannot rely on finding a match in the current list of flavors | |
| 13:27:33 | opendevreview | Takashi Kajinami proposed openstack/keystoneauth master: Bump tox minversion to 3.18.0 https://review.opendev.org/c/openstack/keystoneauth/+/842870 | |
| 14:18:19 | opendevreview | Takashi Kajinami proposed openstack/keystoneauth master: Fix outdated python job template https://review.opendev.org/c/openstack/keystoneauth/+/842885 | |
| 14:18:35 | opendevreview | Takashi Kajinami proposed openstack/keystoneauth stable/yoga: stably-only: Fix wrong python job template https://review.opendev.org/c/openstack/keystoneauth/+/842886 | |
| 14:21:59 | opendevreview | Takashi Kajinami proposed openstack/keystoneauth master: Bump tox minversion to 3.18.0 https://review.opendev.org/c/openstack/keystoneauth/+/842870 | |
| 18:37:02 | eandersson | frickler: Looks like the same fix that I went with as well. We already had it in place, but kept an alternative for older versions. I still don’t get why the api returns an name as an id though | |
| 18:49:24 | frickler | the nova API does not return an id. the old sdk returned the API response 1:1 as a dict, the new sdk converts it to a Flavor object, which has an id field even if that is null | |
| 18:50:25 | frickler | to be clear, this is for the flavor field in the nova server API, not the flavor API | |
| 20:33:16 | eandersson | Yea - but if the id field was null that would be fine | |
| 20:41:29 | frickler | ah, right, now I remember, gtema made a weird (IMHO) patch that fills original_name into the id. I would agree to revert that. | |
| 20:43:09 | eandersson | Yea - that is what breaks it for us. | |
| 20:43:18 | frickler | https://review.opendev.org/c/openstack/openstacksdk/+/826924/4/openstack/compute/v2/flavor.py#69 | |
| 20:43:48 | eandersson | Our code checks if id is there, if not it gets it using the flavor call | |
| 20:44:14 | eandersson | but because id is not none it assumes that it does not need to get any additional data | |
| 20:44:22 | frickler | how do you handle deleted flavors? | |
| 20:44:45 | eandersson | I doubt this code is smart enough to handle that | |
| 20:45:16 | frickler | what do you need the flavor for? | |
| 20:46:03 | eandersson | It's used to handle scale in so creating vms | |
| 20:46:25 | eandersson | the problem is that I don't want to change the current behavior without digging in a lot further into how this works | |
| 20:46:55 | eandersson | But I don't really have the time and this is blocking the CI | |
| 20:48:22 | eandersson | This is all stored in the database, so it would probably break something to switch away from id | |
| 20:48:43 | eandersson | https://paste.openstack.org/show/bcAcLpv3GNnvdZWxvWtY/ | |
| 20:52:55 | eandersson | I guess I could change it to use name, throw in a release note and hope for the best :p | |
| 20:53:04 | eandersson | Pretty sure I am the last maintainer of this project | |
| 20:53:25 | frickler | hmm, I guess then I can only suggest to use a different attribute like "vcpus" as a proxy to decide whether the id is a real id or a name that needs converting, cf. https://review.opendev.org/c/zuul/nodepool/+/834152/2/nodepool/driver/openstack/provider.py#142 | |
| 20:58:21 | eandersson | Gonna try to go the simplest route. I remember going through this last time as well and I feel like it's just a ticking timebomb. | |
| 20:58:30 | eandersson | I just wish I understood these code paths better. | |
| 21:01:37 | eandersson | Any other project just testing against trunk openstacksdk? | |
| 21:01:52 | eandersson | Feel like Senlin should just have a job that does that to give an earlier headsup | |
| 21:04:10 | frickler | well osc obviously has one, and I think ansible-collection-openstack | |
| 21:04:34 | frickler | usually also we try not to break as much, but in this case it was necessary | |
| 21:20:05 | eandersson | Sounds good. Thanks for the help. | |
| #openstack-sdks - 2022-05-23 | |||
| 02:02:25 | eandersson | Got a simple patch up to fix the Senlin issue. | |
| 07:49:55 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update routers_info for the new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/838489 | |
| 07:51:34 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update project_info module to new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/837276 | |
| 08:06:12 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update port info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/833083 | |
| 08:21:21 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update catalog service for the new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839352 | |
| 08:25:45 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update endpoint to new proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/840640 | |
| 08:26:39 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update endpoint to new proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/840640 | |
| 09:30:24 | opendevreview | Merged openstack/microversion-parse master: Drop python3.6/3.7 support in testing runtime https://review.opendev.org/c/openstack/microversion-parse/+/840621 | |
| 12:05:00 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Updated pip constraints for release candidates of OpenStackSDK's first major release https://review.opendev.org/c/openstack/ansible-collections-openstack/+/842966 | |
| 12:07:49 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Updated pip constraints for release candidates of OpenStackSDK's first major release https://review.opendev.org/c/openstack/ansible-collections-openstack/+/842966 | |
| 12:22:19 | opendevreview | Dmitry Tantsur proposed openstack/openstacksdk master: Do not log to stdout by default https://review.opendev.org/c/openstack/openstacksdk/+/842967 | |
| 12:55:56 | dtantsur | even with this ^^^ the python's logging machinery logs everything in stdout, breaking any code that tries to parse a tool output | |
| 12:56:02 | dtantsur | I don't know what to do | |
| 13:18:16 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update recordset module to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/842575 | |
| 13:21:40 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Adds recordset_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/842550 | |
| 16:44:13 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update catalog service for the new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839352 | |
| #openstack-sdks - 2022-05-24 | |||
| 06:50:51 | opendevreview | Jiri Podivin proposed openstack/ansible-collections-openstack master: DNM testing override behavior https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843106 | |
| 06:52:12 | opendevreview | Jiri Podivin proposed openstack/ansible-collections-openstack master: DNM testing override behavior https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843106 | |
| 07:29:20 | opendevreview | Jiri Podivin proposed openstack/ansible-collections-openstack master: DNM testing override behavior https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843106 | |
| 07:34:39 | opendevreview | Jiri Podivin proposed openstack/ansible-collections-openstack master: DNM testing override behavior https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843110 | |