| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-02-05 | |||
| 20:22:38 | opendevreview | Merged openstack/openstacksdk master: reno: Update master for unmaintained/yoga https://review.opendev.org/c/openstack/openstacksdk/+/907828 | |
| 20:22:41 | opendevreview | Merged openstack/python-openstackclient master: reno: Update master for unmaintained/yoga https://review.opendev.org/c/openstack/python-openstackclient/+/907832 | |
| 21:29:39 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'service' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/906395 | |
| 21:36:47 | 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-06 | |||
| 12:43:10 | frickler | gtema: could you please check https://review.opendev.org/c/openstack/releases/+/903303 (ussuri-eol) | |
| 12:49:26 | gtema | frickler - sure | |
| 12:52:44 | frickler | gtema: thx. also fwiw matrix bridge was not showing you as member of this channel once again | |
| 12:53:14 | gtema | hmm, what can I do :-( | |
| 12:56:39 | frickler | use some working client instead of matrix ;-D | |
| 12:59:02 | gtema | matrix is much more stable then irc, its the IRC that looses messages :D | |
| 15:50:26 | opendevreview | OpenStack Release Bot proposed openstack/keystoneauth master: reno: Update master for unmaintained/yoga https://review.opendev.org/c/openstack/keystoneauth/+/908152 | |
| 17:57:46 | whoami-rajat | gtema, hey, if you get some time, can you take another look at https://review.opendev.org/c/openstack/openstacksdk/+/894579 ? | |
| 18:03:07 | gtema | whoami-rajat: ack | |
| 18:03:19 | whoami-rajat | thanks! | |
| 18:23:20 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'service' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/906395 | |
| 19:02:03 | opendevreview | Merged openstack/openstacksdk master: Add snapshot manage unmanage support https://review.opendev.org/c/openstack/openstacksdk/+/894579 | |
| 19:26:08 | 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-07 | |||
| 10:10:57 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add missing snapshot parameters https://review.opendev.org/c/openstack/openstacksdk/+/894823 | |
| 10:12:18 | whoami-rajat | stephenfin, hey, can you approve this patch again? it went into merge conflict when the parent patch merged https://review.opendev.org/c/openstack/openstacksdk/+/894823 | |
| 10:30:18 | frickler | whoami-rajat: done | |
| 10:30:31 | whoami-rajat | thanks frickler ! | |
| 15:15:51 | opendevreview | George Shuklin proposed openstack/ansible-collections-openstack master: fix: subnet module: allow cidr option with subnet_pool https://review.opendev.org/c/openstack/ansible-collections-openstack/+/907301 | |
| 15:59:43 | opendevreview | Merged openstack/keystoneauth master: reno: Update master for unmaintained/yoga https://review.opendev.org/c/openstack/keystoneauth/+/908152 | |
| 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 | carloss | Uggla stephenfin: thank you for the reviews... I'll take a look | |
| 13:22:21 | Uggla | stephenfin, thx you vety much ! I will ping Carloss. | |
| 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: Remove use of unnecessary fake argparse Namespace https://review.opendev.org/c/openstack/python-openstackclient/+/909037 | |
| 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: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 | |