| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-04-21 | |||
| 19:11:59 | jm1 | frickler: the patch is a one liner https://review.opendev.org/c/openstack/openstacksdk/+/838941 | |
| 19:13:21 | jm1 | frickler: but i am not sure if gtema likes to keep backward compat. because he accepted the original patch 🤷 | |
| 19:13:31 | jm1 | frickler: maybe we ask gtema when he is back from PTO | |
| #openstack-sdks - 2022-04-22 | |||
| 09:05:37 | opendevreview | Jorhson Deng proposed openstack/openstacksdk master: WIP Add support for verifying glance image signatures https://review.opendev.org/c/openstack/openstacksdk/+/737135 | |
| 09:38:31 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Renamed image property protected to is_protected https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839033 | |
| 12:02:19 | opendevreview | Merged openstack/ansible-collections-openstack master: Renamed image property protected to is_protected https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839033 | |
| 12:05:20 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Follow up to bump of minimum required OpenStack SDK release to SDK 0.36.0 (Train) https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839053 | |
| 15:14:36 | opendevreview | Merged openstack/ansible-collections-openstack master: Follow up to bump of minimum required OpenStack SDK release to SDK 0.36.0 (Train) https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839053 | |
| 18:53:45 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Added support for specifying a maximum version of the OpenStack SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839093 | |
| #openstack-sdks - 2022-04-24 | |||
| 01:45:00 | opendevreview | Jorhson Deng proposed openstack/openstacksdk master: WIP Add support for verifying glance image signatures https://review.opendev.org/c/openstack/openstacksdk/+/737135 | |
| 01:48:44 | opendevreview | Jorhson Deng proposed openstack/openstacksdk master: WIP Add support for verifying glance image signatures https://review.opendev.org/c/openstack/openstacksdk/+/737135 | |
| 02:26:04 | opendevreview | Jorhson Deng proposed openstack/openstacksdk master: WIP Add support for verifying glance image signatures https://review.opendev.org/c/openstack/openstacksdk/+/737135 | |
| 06:44:11 | opendevreview | Jorhson Deng proposed openstack/openstacksdk master: WIP Add support for verifying glance image signatures https://review.opendev.org/c/openstack/openstacksdk/+/737135 | |
| 08:03:56 | opendevreview | Jorhson Deng proposed openstack/openstacksdk master: Add support for verifying glance image signatures https://review.opendev.org/c/openstack/openstacksdk/+/737135 | |
| 23:21:49 | opendevreview | James Palmer proposed openstack/openstacksdk master: WIP: Add support for Workers, Cleanup Services https://review.opendev.org/c/openstack/openstacksdk/+/798215 | |
| #openstack-sdks - 2022-04-25 | |||
| 08:56:13 | 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 | |
| 09:13:37 | 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 | |
| 11:17:29 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves image_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828158 | |
| 12:34:08 | 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 | |
| 12:42:23 | 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 | |
| 13:08:53 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update port info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/833083 | |
| 13:42:50 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update identity_role_info for latest openstacksdk release https://review.opendev.org/c/openstack/ansible-collections-openstack/+/837751 | |
| 13:47:30 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828742 | |
| 14:12:51 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Use proxy layer in identity_user module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828742 | |
| 15:27:23 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves keypair_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827274 | |
| 15:28:04 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves keypair_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827274 | |
| 15:46:27 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves keypair_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827274 | |
| 16:13:02 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves keypair_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827274 | |
| 19:29:16 | 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 | |
| #openstack-sdks - 2022-04-26 | |||
| 07:55:52 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves keypair_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827274 | |
| 07:57:53 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Added support for specifying a maximum version of the OpenStack SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839093 | |
| 08:21:55 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves keypair_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827274 | |
| 09:18:24 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Constrain filters in compute_service_info to SDK >= 0.53.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839322 | |
| 09:24:56 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Constrain filters in compute_service_info to SDK >= 0.53.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839322 | |
| 09:41:39 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Use Rocky release of Heat in Queens job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839327 | |
| 09:51:50 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Support description in sg-rule creation https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839328 | |
| 10:04:48 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Move dns zone info to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/833028 | |
| 10:18:32 | 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 | |
| 10:20:14 | 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 | |
| 11:39:49 | opendevreview | Merged openstack/ansible-collections-openstack master: Move dns zone info to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/833028 | |
| 11:49:17 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update identity_group_info to new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839365 | |
| 12:41:59 | 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 | |
| 12:43:12 | opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Support description in sg-rule creation https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839328 | |
| 12:46:57 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Follow up to bump of minimum required OpenStack SDK release to SDK 0.36.0 (Train) https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839371 | |
| 12:49:09 | 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 | |
| 12:53:18 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update port info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/833083 | |
| 12:59:05 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves image_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828158 | |
| 13:01:06 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update identity_group_info to new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839365 | |
| 13:05:30 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Temporarily run passing tests in our Zuul CI jobs only https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839374 | |
| 13:14:16 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Use Rocky release of Heat in Queens job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839335 | |
| 13:40:51 | opendevreview | Merged openstack/ansible-collections-openstack master: Use Rocky release of Heat in Queens job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839335 | |
| 15:11:06 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Change network modules to work with new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | |
| 15:36:47 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: WIP: Moves image_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828158 | |
| 16:24:03 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: WIP: Moves image_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828158 | |
| 17:28:57 | opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Follow up to bump of minimum required OpenStack SDK release to SDK 0.36.0 (Train) https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839371 | |
| 17:31:01 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Change network modules to work with new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | |
| 19:05:30 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Move dns zone info to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839428 | |
| 23:09:30 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update identity_domain_info to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839445 | |
| #openstack-sdks - 2022-04-27 | |||
| 06:45:31 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Added support for specifying a maximum version of the OpenStack SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839093 | |
| 07:32:00 | opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Move dns zone info to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839428 | |
| 08:12:11 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Use Rocky release of Heat in Queens job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839327 | |
| 08:18:39 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: WIP: Moves image_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828158 | |
| 08:36:22 | opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Use Rocky release of Heat in Queens job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839327 | |
| 09:33:06 | opendevreview | Merged openstack/ansible-collections-openstack master: Added support for specifying a maximum version of the OpenStack SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839093 | |
| 09:45:57 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Added support for specifying a maximum version of the OpenStack SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839349 | |
| 09:48:24 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Removed object tags from ci role server https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839505 | |
| 11:42:00 | jm1 | gtema: hey ho :) are sdk's cloud functions going to be removed or deprecated in future? | |
| 11:43:35 | gtema | no, but it is advised to use them only when they combine multiple API calls (like upload image or create server). For other cases it is recommended to use proxy directly (nobody wants to use additional wrapping level without real need) | |
| 11:44:41 | jm1 | gtema: we are using cloud functions such as search_projects. they provide "postprocessing" filters which resource proxy functions such as identity.projects() do not. So we are unsure whether we can or should stick to cloud layer functions. we will loose the filters functionality when moving to resource proxy functions | |
| 11:46:29 | gtema | are those some specific filters? If those "can" be addressed by API filters - I would prefer to fix proxy and stick to it. If there is something special what can not be done in proxy - then you can continue using cloud layer | |
| 11:49:12 | jm1 | atm users can choose filters freely, we just pass them to the cloud layer unchanged. https://opendev.org/openstack/ansible-collections-openstack/src/branch/master/plugins/modules/identity_user_info.py#L144 | |
| 11:50:13 | jm1 | gtema: we are not sure what is the best solution to this. either continue to use cloud layer functions or change/remove filters and thus break user code? | |
| 11:51:18 | gtema | I think you can stick to cloud layer here | |
| 12:00:13 | opendevreview | Merged openstack/ansible-collections-openstack master: Removed object tags from ci role server https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839505 | |
| 12:01:33 | jm1 | gtema++ ack, thank you :) | |
| 12:01:41 | gtema | wlcm | |
| 12:19:23 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Temporarily run passing tests in our Zuul CI jobs only https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839374 | |
| 12:27:51 | 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 | |
| 13:08:14 | jm1 | gtema: sdk seems to be inconsistent with postprocessing filters. some functions from cloud layer allow for applying filters to results, e.g. search_users https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/cloud/_identity.py#L162 | |
| 13:09:24 | jm1 | gtema: other cloud layer functions such as search_projects do not allow filtering to the same extent because they use filters as query parameters | |
| 13:10:01 | jm1 | gtema: for example, this fails with the new sdk list(conn.search_projects(filters={"id": "123"})) | |
| 13:10:41 | jm1 | gtema: but this wont list(conn.search_users(filters={"id": "bla2"})) | |
| 13:11:41 | jm1 | gtema: in the first case (where it fails), what is the point of the postprocessing filter if you cannot use anything except for the query parameters anyway? | |
| 13:13:36 | jm1 | gtema: is this "pass all filters as query params and thus only allow valid query params as filters" a design decision which might be applied to other cloud functions such as search_users() later? | |
| 13:14:17 | jm1 | gtema: or is it the opposite, i.e. search_projects() has to be fixed to only pass filters which are valid query params to proxy functions? | |
| 13:49:39 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Temporarily run passing tests in our Zuul CI jobs only https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839374 | |
| 13:57:04 | gtema | jm1: I would say cloud layer may need to be fixed to know what it can pass to API and filter rest. On the other side same might be applied in proxy directly. I wouldn't say there were very strict design decisions, except that proxy for now only supports what API will support (but we may rethink this) | |
| 14:12:01 | jm1 | gtema: afaik proxy layer is a slim python layer on top of the openstack rest api. hence its valid that it raises exceptions on unknown query parameters. QueryParameters and Resource classes have a allow_unknown_params flag which would allow users to force sdk to pass params to rest apis. In case of our cloud layer functions, we do not want to pass unknown params to openstack. instead we want to filter results in a postprocessing step by | |
| 14:12:02 | jm1 | parameters which are not supported as query params. | |
| 14:12:24 | jm1 | gtema: so imho cloud layer functions should only pass valid query params to proxy functions | |
| 14:13:05 | jm1 | gtema: since every resource already has a list of valid query params (_query_parameters), implementing a function to drop invalid params is kind of easy, isnt it? | |
| 14:13:26 | jm1 | gtema: then we could call this drop function from cloud layer functions such as search_projects | |
| 14:13:50 | jm1 | gtema: i am happy to help but i need some guidance on where to best add this kind of code | |
| 14:13:53 | jm1 | ..in the sdk | |
| 14:15:15 | gtema | well yes, it's not so complex, but we need then to implement it in every cloud layer function, and this is close to insanity again | |
| 14:15:43 | gtema | in proxy there is also possibility not to raise exception for unsupported query params | |
| 14:16:01 | jm1 | gtema: why insanity? we only have to add the drop function call to cloud functions which pass filters as query params | |
| 14:16:31 | gtema | so it would be possible to swap process that cloud layer passes everything possible, proxy uses only whatever supported and cloud filters again | |
| 14:16:56 | gtema | insanity because there are many cloud layer functions and we should have things working same everywhere | |
| 14:17:07 | gtema | means we need to invoke it from every list function | |