| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-02-07 | |||
| 16:53:02 | amarao | Hello. Can someone see my patch for subnet ansible module to allow cidr and subnet_pool parameters together? https://review.opendev.org/c/openstack/ansible-collections-openstack/+/907301/2 | |
| 18:07:38 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'service' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/906395 | |
| 19:51:19 | opendevreview | Merged openstack/python-openstackclient master: pre-commit: Bump linter versions https://review.opendev.org/c/openstack/python-openstackclient/+/906362 | |
| 22:43:39 | opendevreview | Merged openstack/openstacksdk master: Add missing snapshot parameters https://review.opendev.org/c/openstack/openstacksdk/+/894823 | |
| #openstack-sdks - 2024-02-08 | |||
| 07:55:15 | lajoskatona | Hi, a quick/dumb question, sorry for it | |
| 07:56:18 | lajoskatona | I work on changing neutronclient to SDK in Horizon, and I have this SDK connection initializayion: https://review.opendev.org/c/openstack/horizon/+/891205/24/openstack_dashboard/api/neutron.py#950 | |
| 07:56:51 | lajoskatona | and I have terrible SSL errors like this: 'WARNING horizon.exceptions Recoverable error: SSL exception connecting to https://100.109.0.221:9696/networking/v2.0/subnets: HTTPSConnectionPool(host='100.109.0.221', port=9696): Max retries exceeded with url: /networking/v2.0/subnets (Caused by SSLError(SSLError(524297, '[SSL] PEM lib (_ssl.c:3921)')))" | |
| 07:59:12 | lajoskatona | If I understand well the keystoneauth and requests doc I can pass the path to the cert file in cert param or in verify param but actually neither one is working, so I am lost at the moment | |
| 10:02:13 | opendevreview | Emilien LEFRANCOIS proposed openstack/openstacksdk master: Fix AttributeError in delete_limit method https://review.opendev.org/c/openstack/openstacksdk/+/908402 | |
| 10:33:48 | opendevreview | Pranali Deore proposed openstack/openstacksdk master: Adding support for new add/get location apis https://review.opendev.org/c/openstack/openstacksdk/+/883271 | |
| 14:32:26 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient stable/victoria: [stable-only] CI: Drop all tips jobs https://review.opendev.org/c/openstack/python-openstackclient/+/896719 | |
| 14:36:06 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient stable/wallaby: [stable-only] CI: Drop all tips jobs https://review.opendev.org/c/openstack/python-openstackclient/+/896718 | |
| 14:37:07 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient stable/xena: [stable-only] CI: Drop all tips jobs https://review.opendev.org/c/openstack/python-openstackclient/+/896717 | |
| 15:57:40 | opendevreview | Pavlo Shchelokovskyy proposed openstack/openstacksdk master: Fix list of server migrations https://review.opendev.org/c/openstack/openstacksdk/+/887847 | |
| 18:16:35 | opendevreview | Merged openstack/python-openstackclient stable/victoria: [stable-only] CI: Drop all tips jobs https://review.opendev.org/c/openstack/python-openstackclient/+/896719 | |
| 18:46:13 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'service' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/906395 | |
| #openstack-sdks - 2024-02-09 | |||
| 19:53:11 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Adding SDK support for glance md-tags https://review.opendev.org/c/openstack/openstacksdk/+/897276 | |
| #openstack-sdks - 2024-02-10 | |||
| 18:12:01 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Adding SDK support for ``glance md-namespace-objects-delete`` https://review.opendev.org/c/openstack/openstacksdk/+/901671 | |
| 18:50:09 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Adding SDK support for ``glance md-namespace-properties-delete`` https://review.opendev.org/c/openstack/openstacksdk/+/902612 | |
| 19:21:05 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adds CLI support for ``glance md-object-property-show`` https://review.opendev.org/c/openstack/python-openstackclient/+/904077 | |
| #openstack-sdks - 2024-02-11 | |||
| 07:10:07 | opendevreview | Radoslaw Smigielski proposed openstack/openstacksdk master: Incorrect protocol type in create_security_group_rule() https://review.opendev.org/c/openstack/openstacksdk/+/908734 | |
| #openstack-sdks - 2024-02-12 | |||
| 11:29:35 | opendevreview | Merged openstack/openstacksdk master: Incorrect protocol type in create_security_group_rule() https://review.opendev.org/c/openstack/openstacksdk/+/908734 | |
| 18:26:04 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adds CLI support for ``glance md-namespace-properties-delete`` https://review.opendev.org/c/openstack/python-openstackclient/+/905746 | |
| 18:39:31 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adds CLI support for ``glance md-namespace-properties-delete`` https://review.opendev.org/c/openstack/python-openstackclient/+/905746 | |
| 19:21:19 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adds CLI support for ``glance md-namespace-properties-delete`` https://review.opendev.org/c/openstack/python-openstackclient/+/905746 | |
| 20:02:52 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adds CLI support for ``glance md-namespace-objects-delete`` https://review.opendev.org/c/openstack/python-openstackclient/+/904165 | |
| #openstack-sdks - 2024-02-13 | |||
| 07:34:42 | opendevreview | Pranali Deore proposed openstack/python-openstackclient master: Adds commands, 1. ``image add location`` 2. ``image get locations`` https://review.opendev.org/c/openstack/python-openstackclient/+/908884 | |
| 07:44:34 | opendevreview | Pranali Deore proposed openstack/openstacksdk master: Adding support for new add/get location apis https://review.opendev.org/c/openstack/openstacksdk/+/883271 | |
| 12:17:45 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: trivial: Don't ignore missing resources https://review.opendev.org/c/openstack/python-openstackclient/+/908898 | |
| 13:29:55 | lajoskatona | Hi, I work on changing neutronclient to SDK in Horizon, and I have this SDK connection initialization: https://review.opendev.org/c/openstack/horizon/+/891205/24/openstack_dashboard/api/neutron.py#950 | |
| 13:30:11 | lajoskatona | and I have terrible SSL errors like this: 'WARNING horizon.exceptions Recoverable error: SSL exception connecting to https://100.109.0.221:9696/networking/v2.0/subnets: HTTPSConnectionPool(host='100.109.0.221', port=9696): Max retries exceeded with url: /networking/v2.0/subnets (Caused by SSLError(SSLError(524297, '[SSL] PEM lib (_ssl.c:3921)')))" | |
| 13:30:37 | lajoskatona | If I understand well the keystoneauth and requests doc I can pass the path to the cert file in cert param or in verify param but actually neither one is working, so I am lost at the moment | |
| 13:31:22 | lajoskatona | is there some examples how to pass path to cert file ? | |
| 14:54:17 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Adding support for image upload https://review.opendev.org/c/openstack/openstacksdk/+/881939 | |
| 16:34:06 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: trivial: Don't ignore missing resources https://review.opendev.org/c/openstack/python-openstackclient/+/908898 | |
| 18:22:32 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Fixed update_metadef_object https://review.opendev.org/c/openstack/openstacksdk/+/908916 | |
| 18:33:02 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adds CLI support for ``glance md-object-update`` https://review.opendev.org/c/openstack/python-openstackclient/+/904074 | |
| 19:10:21 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adds CLI support for ``glance md-namespace-properties-delete`` https://review.opendev.org/c/openstack/python-openstackclient/+/905746 | |
| 19:27:45 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adds CLI support for ``glance md-namespace-objects-delete`` https://review.opendev.org/c/openstack/python-openstackclient/+/904165 | |
| 19:46:22 | M0weng[m] | I have a question about SDK - I am doing the following:... (full message at <https://matrix.org/_matrix/media/v3/download/matrix.org/bQBFrVDXqFeLVChaLTFUjRSh>) | |
| #openstack-sdks - 2024-02-14 | |||
| 02:50:43 | tkajinam | M0weng[m], can you share list of the domains (openstack domain list ?) and, the value passed to parsed_args.domain ? | |
| 02:50:58 | tkajinam | and actual items dumped by these print methods, probably | |
| 10:16:18 | opendevreview | Merged openstack/python-openstackclient stable/wallaby: [stable-only] CI: Drop all tips jobs https://review.opendev.org/c/openstack/python-openstackclient/+/896718 | |
| 10:16:20 | opendevreview | Merged openstack/python-openstackclient stable/xena: [stable-only] CI: Drop all tips jobs https://review.opendev.org/c/openstack/python-openstackclient/+/896717 | |
| 10:19:47 | opendevreview | Merged openstack/python-openstackclient master: trivial: Don't ignore missing resources https://review.opendev.org/c/openstack/python-openstackclient/+/908898 | |
| 12:01:11 | opendevreview | Merged openstack/python-openstackclient master: Adds CLI support for ``glance md-object-update`` https://review.opendev.org/c/openstack/python-openstackclient/+/904074 | |
| 12:53:49 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: docs: Add docs on configuration of a service user https://review.opendev.org/c/openstack/openstacksdk/+/893505 | |
| 13:05:53 | Uggla | stephenfin, could you please have a look at https://review.opendev.org/c/openstack/openstacksdk/+/889519 I would like it to be merged ASAP because it is required on my serie. | |
| 13:06:11 | stephenfin | ack | |
| 13:21:30 | stephenfin | Uggla: Comments left. Hopefully carloss can respin so I can +2 | |
| 13:22:21 | Uggla | stephenfin, thx you vety much ! I will ping Carloss. | |
| 13:22:21 | carloss | Uggla stephenfin: thank you for the reviews... I'll take a look | |
| 13:51:46 | lajoskatona | stephenfin, gtema: Hi, perhaps you have some idea where to search for examples for how to start SDK connection / keystoneauth Session to connect to https API? See my question: https://meetings.opendev.org/irclogs/%23openstack-sdks/%23openstack-sdks.2024-02-13.log.html | |
| 13:52:43 | lajoskatona | stephenfin, gtema: The issue for me is in my Horizon patch to change from neutronclient to SDK for neutron calls, (see the client initialization: https://review.opendev.org/c/openstack/horizon/+/891205/24/openstack_dashboard/api/neutron.py#950 ) | |
| 13:53:09 | gtema | lajoskatona: looking | |
| 13:55:14 | lajoskatona | gtema: thanks, the issue is quite interesting as it comes with TLS enabled in devstack and if I use horizon with pagination... | |
| 13:55:27 | gtema | https://docs.openstack.org/openstacksdk/latest/user/config/configuration.html#ssl-settings describes how ssl cert can be placed into the clouds.yaml | |
| 13:56:04 | gtema | sdk correspondingly reads cacert/key/cert and sets them into the request session | |
| 13:57:20 | lajoskatona | gtema: hmmmm, I am not sure if horizon cares about clouds.yaml, but I check if setting clouds.yaml, solves the issue | |
| 13:57:41 | gtema | sure, I just mean this is the beginning of how sdk currently deals with that | |
| 13:58:00 | gtema | https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/config/cloud_region.py#L695 - here the stuff is being injected into the keystone_session | |
| 13:59:16 | gtema | so in your example you should be constructing "verify" and "cert" that you pass into the connection as attributes | |
| 14:00:49 | gtema | https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/tests/unit/config/test_cloud_config.py#L98 - this is how sdk tests this stuff in units | |
| 14:06:22 | lajoskatona | gtema: thanks, I check it, I tried the verify cert combo, I run through again it with your examples | |
| 14:06:32 | gtema | cool | |
| 14:09:13 | opendevreview | Lajos Katona proposed openstack/openstacksdk master: Add Tap Mirrors to SDK https://review.opendev.org/c/openstack/openstacksdk/+/886084 | |
| 17:28:54 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: WIP: migrate user commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/909030 | |
| 17:30:24 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: [DNM] WIP: migrate user commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/909030 | |
| 17:46:58 | M0weng[m] | <tkajinam> "0weng, can you share list of the..." <- gtema/stephenfin are helping me with this; it seems to be a problem with the way I'm scoping the fakes in my unit test, not something specific to SDK :) | |
| 17:47:38 | M0weng[m] | stephenfin: Here is the review - https://review.opendev.org/c/openstack/python-openstackclient/+/909030 | |
| 18:15:52 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Add identity v2, v3 FakeClientMixin https://review.opendev.org/c/openstack/python-openstackclient/+/909035 | |
| 18:15:53 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Use consistent shortcut to fake identity client https://review.opendev.org/c/openstack/python-openstackclient/+/909036 | |
| 18:15:53 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Remove use of unnecessary fake argparse Namespace https://review.opendev.org/c/openstack/python-openstackclient/+/909037 | |
| 18:27:15 | stephenfin | M0weng[m]: reviewed | |
| 19:22:14 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: [DNM] WIP: migrate user commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/909030 | |
| 19:25:49 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: [DNM] WIP: migrate user commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/909030 | |
| 21:23:34 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Resource locks and access rules restrictions https://review.opendev.org/c/openstack/openstacksdk/+/889519 | |
| 21:45:55 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Resource locks and access rules restrictions https://review.opendev.org/c/openstack/openstacksdk/+/889519 | |
| 22:34:25 | opendevreview | Brian Haley proposed openstack/python-openstackclient master: Do not sort subnet dns_nameservers field https://review.opendev.org/c/openstack/python-openstackclient/+/909058 | |
| #openstack-sdks - 2024-02-15 | |||
| 03:28:35 | opendevreview | Tatsuya Hayashino proposed openstack/keystoneauth master: Output verification_uri_complete as stderr https://review.opendev.org/c/openstack/keystoneauth/+/907775 | |
| 08:37:48 | opendevreview | Takashi Kajinami proposed openstack/keystoneauth master: Use logger instead of print https://review.opendev.org/c/openstack/keystoneauth/+/909073 | |
| 12:18:14 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Fix typo https://review.opendev.org/c/openstack/openstacksdk/+/909109 | |
| 14:21:40 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adds CLI support for ``glance md-object-property-show`` https://review.opendev.org/c/openstack/python-openstackclient/+/904077 | |
| 14:24:14 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tox: Correct functional test factors https://review.opendev.org/c/openstack/openstacksdk/+/909116 | |
| 14:24:30 | stephenfin | gtema: gouthamr: carloss: tox 4 regressions strike again :( ^ | |
| 14:24:58 | stephenfin | what are the chances our functional tests have broken in the interim... hopefully not (I didn't run locally) | |
| 14:25:43 | stephenfin | tbc, I still like tox and don't want to switch to nox but it's irritating that their test suite was so minimal. This is the 4th or 5th large regression I've encountered (I fixed the previous ones) | |
| 14:25:57 | gtema | crap, I am getting so damned frustrated on the Py situation lately | |
| 14:34:14 | carloss | gouthamr stephenfin: good catch and thanks for investigating and proposing the fix | |
| 14:34:29 | carloss | it was surprising to see what this regression caused | |
| 19:28:05 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: [DNM] WIP: migrate user commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/909030 | |
| 19:50:11 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'user' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/909030 | |
| 21:03:15 | opendevreview | Pavlo Shchelokovskyy proposed openstack/openstacksdk master: Fix list of server migrations https://review.opendev.org/c/openstack/openstacksdk/+/887847 | |
| 21:08:18 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'user' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/909030 | |
| 21:14:41 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: WIP: Add support for cinder manage https://review.opendev.org/c/openstack/python-openstackclient/+/895214 | |
| #openstack-sdks - 2024-02-16 | |||
| 00:37:34 | opendevreview | Goutham Pacha Ravi proposed openstack/openstacksdk master: Resource locks and access rules restrictions https://review.opendev.org/c/openstack/openstacksdk/+/889519 | |
| 01:39:46 | opendevreview | Tatsuya Hayashino proposed openstack/keystoneauth master: Output verification_uri_complete as stderr https://review.opendev.org/c/openstack/keystoneauth/+/907775 | |
| 01:53:45 | opendevreview | Tatsuya Hayashino proposed openstack/keystoneauth master: Output verification_uri_complete as stderr https://review.opendev.org/c/openstack/keystoneauth/+/907775 | |
| 05:54:10 | opendevreview | Tatsuya Hayashino proposed openstack/keystoneauth master: Output verification_uri_complete by logger https://review.opendev.org/c/openstack/keystoneauth/+/907775 | |
| 09:18:55 | opendevreview | Merged openstack/openstacksdk master: tox: Correct functional test factors https://review.opendev.org/c/openstack/openstacksdk/+/909116 | |