| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-05-27 | |||
| 09:50:50 | jm1 | gtema: what did you mean "we might need to think how to deal with whichever filters are supported or not"? like dropping jmespath stuff or something? | |
| 09:51:13 | gtema | not really | |
| 09:51:38 | gtema | we pass already query params as **params | |
| 09:51:50 | gtema | and those already implement most of the queries | |
| 09:51:58 | gtema | mean most of the params | |
| 09:52:27 | gtema | so we might want jmespath not to overtake everything | |
| 09:52:44 | gtema | filter on server side is better then on client | |
| 09:53:09 | gtema | so we might also "try" to parse jmsepath and whatever is supported as server-side filter pass it this way | |
| 09:53:34 | gtema | but maybe for beginning (just to improve broken backward compatibility) we can skip this for now | |
| 09:53:58 | gtema | so whatever is passed in query params is applied on server, whatever through jmes - on client side | |
| 09:57:44 | jm1 | gtema: my first idea was to drop this "filters" argument and instead add a param like "use_unknown_params_as_filters=False". We know what params are query params, so we could use all non-query-params as postprocessing filters. But i dropped that idea because this would not allow us to implement name_or_id in Resource.list() and does not allow using jmespath exprs. like a dedicated "filters" param does. But maybe you like that idea more? | |
| 10:00:04 | gtema | strategically this idea is better. We want to give user possibility to apply filters even if server side doesn't support this | |
| 10:00:34 | gtema | but it should be also disabled by default, since currently we through exception when user passes some unsupported query params | |
| 10:06:01 | jm1 | gtema: ack, let me updated the proc | |
| 10:06:04 | jm1 | *poc | |
| 10:06:26 | gtema | sure, thanks | |
| 12:10:32 | opendevreview | Merged openstack/ansible-collections-openstack master: Warn users about us breaking backward compatibility https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843419 | |
| 12:43:09 | opendevreview | Jakob Meng proposed openstack/openstacksdk master: [DNM] Added generic resource filtering by name_or_id, jmespath and dicts https://review.opendev.org/c/openstack/openstacksdk/+/843587 | |
| 12:44:10 | jm1 | gtema: updated poc https://review.opendev.org/c/openstack/openstacksdk/+/843587 | |
| 12:44:38 | gtema | cool, will look deeper once I have some spare time | |
| 12:44:51 | jm1 | gtema: sure, np | |
| 12:48:29 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Warn users about us breaking backward compatibility https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843528 | |
| 12:54:55 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Warn users about us breaking backward compatibility https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843528 | |
| 13:29:50 | opendevreview | Merged openstack/cliff master: Update Python testing per Zed cycle testing runtime https://review.opendev.org/c/openstack/cliff/+/843312 | |
| 14:36:38 | opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Use bifrost's stable/yoga branch for jobs with OpenStack SDK 0.x.x https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843569 | |
| 14:37:58 | opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Warn users about us breaking backward compatibility https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843528 | |
| #openstack-sdks - 2022-05-30 | |||
| 07:40:18 | 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 | |
| 07:43:44 | 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:09:11 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Update recordset module to be compatible with OpenStack SDK 0.90.x/1.0.x https://review.opendev.org/c/openstack/ansible-collections-openstack/+/842575 | |
| 08:39:21 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update port info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/833083 | |
| 09:46:53 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update role_assignment to use proxy https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843830 | |
| 10:41:02 | 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 | |
| 10:53:03 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Update recordset module to be compatible with OpenStack SDK 0.99.x/1.0.x https://review.opendev.org/c/openstack/ansible-collections-openstack/+/842575 | |
| 12:52:00 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update role_assignment to use proxy https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843830 | |
| 13:06:46 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Warn when no statsd library available https://review.opendev.org/c/openstack/openstacksdk/+/843855 | |
| 16:29:09 | 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 | |
| 17:16:49 | 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 | |
| 17:31:57 | 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 | |
| 18:41:41 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update project module to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839640 | |
| 18:44:58 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update recordset module to be compatible with OpenStack SDK 0.99.x/1.0.x https://review.opendev.org/c/openstack/ansible-collections-openstack/+/842575 | |
| 19:08:43 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update recordset module to be compatible with OpenStack SDK 0.99.x/1.0.x https://review.opendev.org/c/openstack/ansible-collections-openstack/+/842575 | |
| 22:06:29 | opendevreview | Rafael Castillo proposed openstack/openstacksdk master: Allow Resource.to_dict to allow returning unknown values https://review.opendev.org/c/openstack/openstacksdk/+/843963 | |
| 22:06:29 | opendevreview | Rafael Castillo proposed openstack/openstacksdk master: Allow unknown attributes in project resources https://review.opendev.org/c/openstack/openstacksdk/+/843964 | |
| 22:37:00 | opendevreview | Rafael Castillo proposed openstack/openstacksdk master: Allow unknown attributes in project resources https://review.opendev.org/c/openstack/openstacksdk/+/843964 | |
| 23:57:23 | opendevreview | Ghanshyam proposed openstack/openstacksdk master: Use compatible nodeset in openstacksdk-functional-devstack https://review.opendev.org/c/openstack/openstacksdk/+/843968 | |
| #openstack-sdks - 2022-05-31 | |||
| 00:39:25 | opendevreview | Ghanshyam proposed openstack/openstacksdk stable/yoga: Use compatible nodeset in openstacksdk-functional-devstack https://review.opendev.org/c/openstack/openstacksdk/+/843974 | |
| 00:40:29 | opendevreview | Ghanshyam proposed openstack/openstacksdk stable/xena: Use compatible nodeset in openstacksdk-functional-devstack https://review.opendev.org/c/openstack/openstacksdk/+/843975 | |
| 00:41:30 | opendevreview | Ghanshyam proposed openstack/openstacksdk stable/wallaby: Use compatible nodeset in openstacksdk-functional-devstack https://review.opendev.org/c/openstack/openstacksdk/+/843976 | |
| 00:42:04 | opendevreview | Ghanshyam proposed openstack/openstacksdk stable/victoria: Use compatible nodeset in openstacksdk-functional-devstack https://review.opendev.org/c/openstack/openstacksdk/+/843977 | |
| 00:42:50 | opendevreview | Ghanshyam proposed openstack/openstacksdk stable/ussuri: Use compatible nodeset in openstacksdk-functional-devstack https://review.opendev.org/c/openstack/openstacksdk/+/843978 | |
| 01:24:48 | opendevreview | Ghanshyam proposed openstack/openstacksdk stable/ussuri: [stable-only] Pin openstacksdk in openstacksdk-functional-devstack https://review.opendev.org/c/openstack/openstacksdk/+/843978 | |
| 04:46:18 | frickler | network jobs on master are ooming https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_b7b/843968/1/check/openstacksdk-functional-devstack-networking/b7b0921/controller/logs/syslog.txt | |
| 07:44:02 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Update recordset module to be compatible with OpenStack SDK 0.99.x/1.0.x https://review.opendev.org/c/openstack/ansible-collections-openstack/+/842575 | |
| 08:15:29 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update role_assignment to use proxy https://review.opendev.org/c/openstack/ansible-collections-openstack/+/843830 | |
| 08:28:39 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Update project module to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839640 | |
| 08:35:29 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Update project module to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839640 | |
| 08:38:53 | opendevreview | Yusuke Niimi proposed openstack/keystoneauth master: OAuth2.0 Client Credentials Grant Flow Support https://review.opendev.org/c/openstack/keystoneauth/+/830734 | |
| 08:46:23 | 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:52:20 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: DNM: Test networking job w/o cinder https://review.opendev.org/c/openstack/openstacksdk/+/844010 | |
| 09:03:26 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Update host_aggregate to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/841678 | |
| 09:10:31 | 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 | |
| 09:19:05 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update port info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/833083 | |
| 09:36:25 | 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 | |
| 10:43:34 | opendevreview | Yusuke Niimi proposed openstack/keystoneauth master: OAuth2.0 Client Credentials Grant Flow Support https://review.opendev.org/c/openstack/keystoneauth/+/830734 | |
| 10:46:21 | opendevreview | Jakob Meng 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 | |
| 11:04:50 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: DNM: Test networking job w/o cinder https://review.opendev.org/c/openstack/openstacksdk/+/844010 | |
| 11:16:33 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: DNM: Test networking job w/o cinder https://review.opendev.org/c/openstack/openstacksdk/+/844010 | |
| 11:44:27 | opendevreview | Jakob Meng 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 | |
| 11:44:34 | 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:55:54 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update port info https://review.opendev.org/c/openstack/ansible-collections-openstack/+/833083 | |
| 12:09:15 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Adds recordset_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/842550 | |
| 12:15:07 | 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 | |
| 14:40:46 | opendevreview | Merged openstack/ansible-collections-openstack master: Update host_aggregate to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/841678 | |
| 14:40:48 | opendevreview | Merged openstack/ansible-collections-openstack master: Update recordset module to be compatible with OpenStack SDK 0.99.x/1.0.x https://review.opendev.org/c/openstack/ansible-collections-openstack/+/842575 | |
| 15:06:38 | opendevreview | Elod Illes proposed openstack/openstacksdk stable/wallaby: Drop lower-constraints job https://review.opendev.org/c/openstack/openstacksdk/+/844094 | |
| 15:08:23 | opendevreview | Elod Illes proposed openstack/openstacksdk stable/victoria: Drop lower-constraints job https://review.opendev.org/c/openstack/openstacksdk/+/844095 | |
| 15:14:03 | opendevreview | Elod Illes proposed openstack/openstacksdk stable/ussuri: Drop lower-constraints job https://review.opendev.org/c/openstack/openstacksdk/+/844096 | |
| 16:08:32 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: DNM: Test networking job w/o cinder https://review.opendev.org/c/openstack/openstacksdk/+/844010 | |
| 16:53:30 | 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 | |
| 17:04:24 | opendevreview | Ghanshyam proposed openstack/openstacksdk stable/ussuri: [stable-only] Pin openstacksdk in openstacksdk-functional-devstack https://review.opendev.org/c/openstack/openstacksdk/+/843978 | |
| 17:30:26 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: DNM: Test networking job with extra swap https://review.opendev.org/c/openstack/openstacksdk/+/844010 | |
| 17:31:46 | opendevreview | Pedro Henrique Pereira Martins proposed openstack/keystoneauth master: Add OTP to v3OIDCpassword plugin https://review.opendev.org/c/openstack/keystoneauth/+/697348 | |
| 17:32:53 | opendevreview | Pedro Henrique Pereira Martins proposed openstack/keystoneauth master: Add OTP to v3OIDCpassword plugin https://review.opendev.org/c/openstack/keystoneauth/+/697348 | |
| 20:05:20 | opendevreview | Merged openstack/openstacksdk stable/wallaby: Drop lower-constraints job https://review.opendev.org/c/openstack/openstacksdk/+/844094 | |
| 20:05:22 | opendevreview | Merged openstack/openstacksdk stable/victoria: Drop lower-constraints job https://review.opendev.org/c/openstack/openstacksdk/+/844095 | |
| 22:09:20 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update project module to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/839640 | |
| 23:34:27 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack stable/1.0.0: Update recordset module to be compatible with OpenStack SDK 0.99.x/1.0.x https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844089 | |
| 23:35:22 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack stable/1.0.0: Update host_aggregate to be compatible with new sdk https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844150 | |
| 23:49:59 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack stable/1.0.0: Backport improvements to recordset module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844089 | |
| #openstack-sdks - 2022-06-01 | |||
| 00:03:34 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack stable/1.0.0: Backport enhancements to host_aggregate module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844150 | |
| 00:25:52 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack stable/1.0.0: Backport enhancements to host_aggregate module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844150 | |
| 00:56:37 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack stable/1.0.0: Backport improvements to recordset module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844089 | |
| 00:57:58 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Correctly build params dict in recordset https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844170 | |
| 01:36:54 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack stable/1.0.0: Backport enhancements to host_aggregate module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844150 | |
| 02:03:15 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack stable/1.0.0: Backport improvements to recordset module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844089 | |
| 02:38:48 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack stable/1.0.0: Backport enhancements to host_aggregate module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844150 | |
| 02:52:54 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack stable/1.0.0: Correctly build params dict in recordset https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844177 | |
| 03:08:10 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Correctly build params dict in recordset https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844178 | |
| 05:25:52 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Correctly build params dict in recordset https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844178 | |
| 05:30:14 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack stable/1.0.0: Backport improvements to recordset module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/844089 | |