| tkajinam | M0weng[m], can you share list of the domains (openstack domain list ?) and, the value passed to parsed_args.domain ? | 02:50 |
|---|---|---|
| tkajinam | and actual items dumped by these print methods, probably | 02:50 |
| 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 |
| opendevreview | Merged openstack/python-openstackclient stable/xena: [stable-only] CI: Drop all tips jobs https://review.opendev.org/c/openstack/python-openstackclient/+/896717 | 10:16 |
| opendevreview | Merged openstack/python-openstackclient master: trivial: Don't ignore missing resources https://review.opendev.org/c/openstack/python-openstackclient/+/908898 | 10:19 |
| opendevreview | Merged openstack/python-openstackclient master: Adds CLI support for ``glance md-object-update`` https://review.opendev.org/c/openstack/python-openstackclient/+/904074 | 12:01 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: docs: Add docs on configuration of a service user https://review.opendev.org/c/openstack/openstacksdk/+/893505 | 12: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:05 |
| stephenfin | ack | 13:06 |
| stephenfin | Uggla: Comments left. Hopefully carloss can respin so I can +2 | 13:21 |
| Uggla | stephenfin, thx you vety much ! I will ping Carloss. | 13:22 |
| carloss | Uggla stephenfin: thank you for the reviews... I'll take a look | 13:22 |
| 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:51 |
| 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:52 |
| gtema | lajoskatona: looking | 13:53 |
| 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 |
| 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:55 |
| gtema | sdk correspondingly reads cacert/key/cert and sets them into the request session | 13:56 |
| 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 |
| gtema | sure, I just mean this is the beginning of how sdk currently deals with that | 13:57 |
| 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:58 |
| gtema | so in your example you should be constructing "verify" and "cert" that you pass into the connection as attributes | 13:59 |
| 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:00 |
| lajoskatona | gtema: thanks, I check it, I tried the verify cert combo, I run through again it with your examples | 14:06 |
| gtema | cool | 14:06 |
| opendevreview | Lajos Katona proposed openstack/openstacksdk master: Add Tap Mirrors to SDK https://review.opendev.org/c/openstack/openstacksdk/+/886084 | 14:09 |
| opendevreview | Oria Weng proposed openstack/python-openstackclient master: WIP: migrate user commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/909030 | 17:28 |
| 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:30 |
| 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:46 |
| M0weng[m] | stephenfin: Here is the review - https://review.opendev.org/c/openstack/python-openstackclient/+/909030 | 17:47 |
| 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 |
| 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 |
| 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 |
| stephenfin | M0weng[m]: reviewed | 18:27 |
| *** jph4 is now known as jph | 18:32 | |
| 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:22 |
| 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 |
| opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Resource locks and access rules restrictions https://review.opendev.org/c/openstack/openstacksdk/+/889519 | 21:23 |
| opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Resource locks and access rules restrictions https://review.opendev.org/c/openstack/openstacksdk/+/889519 | 21:45 |
| opendevreview | Brian Haley proposed openstack/python-openstackclient master: Do not sort subnet dns_nameservers field https://review.opendev.org/c/openstack/python-openstackclient/+/909058 | 22:34 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!