| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-04-21 | |||
| 13:57:47 | ralonsoh | I think we can release this version, there is a long diff between now and last version | |
| 14:02:47 | artom | ralonsoh, I think you mean gtema? | |
| 14:02:56 | artom | I'm the other Artom, with an O :) | |
| 14:03:01 | artom | From Nova :) | |
| 14:03:01 | ralonsoh | artom, sorry, right! | |
| 14:06:34 | gtema | I am on PTO this week, sorry. | |
| 14:06:43 | ralonsoh | no problem, thanks! | |
| 14:07:00 | ralonsoh | enjoy | |
| 16:02:31 | opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Drop username from return values of identity_user_info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/837484 | |
| 16:19:24 | jm1 | frickler: the patch you mentioned is the reason for our issues. actually it is not create_image() which fails but update_image_properties(). previously, one could call update_image_properties(..., protected=False) but since this patch is has to be is_protected instead of protected. Question is, do we want to accept both parameters in update_image_properties()? Then i would submit a patch. If we are fine with breaking backward compatibility ( | |
| 16:19:25 | jm1 | which sdk 1.0.0 does anyway), then i wont ^^ | |
| 16:28:02 | frickler | jm1: if the patch is easy, then go ahead, otherwise best wait for gtema's opinion on this | |
| 16:48:29 | mgagne | May I get this review attention? It's updating Internap auth URL in vendors config. The current URL will be removed in the following days. https://review.opendev.org/c/openstack/openstacksdk/+/838734 | |
| 19:10:45 | opendevreview | Jakob Meng proposed openstack/openstacksdk master: Fix creation of protected image for old user code https://review.opendev.org/c/openstack/openstacksdk/+/838941 | |
| 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 | |