| opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Updating the glanceclient reference doc https://review.opendev.org/c/openstack/python-openstackclient/+/864118 | 06:02 |
|---|---|---|
| sfinucan | tweining: In my own scripts, I normally just use `openstack.connect()`. However, that assumes a cloud.yaml file which you obviously won't have... | 11:42 |
| sfinucan | nova is using openstacksdk for ironic stuff for a while. I'm looking at what they do now | 11:42 |
| *** sfinucan is now known as stephenfin | 11:42 | |
| tweining | sfinucan: thanks. BTW, I found out the way octavia-dashboard does it is actually outdated. | 11:44 |
| stephenfin | tweining: https://github.com/openstack/nova/blob/d05f9ef0dcecd55f37f362afe4b32544d8d81636/nova/utils.py#L959-L985 | 11:45 |
| gtema | https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/connection.py#L80 describes how to pass creds directly with connection.Connection | 11:45 |
| gtema | but all same params can also go to openstack.connect directly | 11:46 |
| tweining | yeah, those two ways, per CONF and per keyword args, are the two that seem right to me. | 11:47 |
| stephenfin | yeah, both should achieve the same thing. Presumably doing it with CONF involves less "unpacking" (i.e. you won't need to manually build a dict of credentials) but that shouldn't be too onerous at the same | 11:48 |
| stephenfin | time | 11:48 |
| tweining | I also found an example in ironic, which seems simlar to the one from nova you found. https://opendev.org/openstack/ironic/src/branch/master/ironic/common/neutron.py#L78 | 11:49 |
| stephenfin | yes, and less complex to boot | 11:49 |
| stephenfin | tbh, this is an area that I'm not overly familiar with and gtema might be better placed to comment, but I suspect simpler but working is better :) (i.e. use ironic's example) | 11:50 |
| tweining | ok, thanks. I think my main task now is to ensure I get all the information openstacksdk needs for the connection from somewhere. | 11:54 |
| gtema | use any method of ones described in examples in https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/connection.py and you should be fine | 11:55 |
| gtema | those are the ones that we definitely support and keep maintaining | 11:55 |
| gtema | tweining - you can have a look at https://github.com/opentelekomcloud-infra/octavia-proxy/blob/main/octavia_proxy/common/context.py We were forced to build octavia simulation API and just reuse user token | 11:57 |
| gtema | this may give you some potential direction | 11:57 |
| tweining | gtema: cool, thanks. I will have a look | 11:58 |
| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Rework network functional tests https://review.opendev.org/c/openstack/openstacksdk/+/859114 | 12:59 |
| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Whitelist cloud functional tests in acceptance https://review.opendev.org/c/openstack/openstacksdk/+/860009 | 13:00 |
| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Implement acceptance test job https://review.opendev.org/c/openstack/openstacksdk/+/860399 | 13:02 |
| opendevreview | Merged openstack/python-openstackclient master: Use the compute SDK in server list https://review.opendev.org/c/openstack/python-openstackclient/+/861546 | 14:21 |
| opendevreview | Merged openstack/python-openstackclient master: tests: Convert image tests to use 'parse_output' https://review.opendev.org/c/openstack/python-openstackclient/+/863987 | 14:21 |
| opendevreview | Merged openstack/python-openstackclient master: tests: Convert volume tests to use 'parse_output' https://review.opendev.org/c/openstack/python-openstackclient/+/863988 | 14:21 |
| opendevreview | Merged openstack/python-openstackclient master: tests: Convert identity tests to use 'parse_output' https://review.opendev.org/c/openstack/python-openstackclient/+/863989 | 14:21 |
| opendevreview | Merged openstack/python-openstackclient master: tests: Convert compute tests to use 'parse_output' https://review.opendev.org/c/openstack/python-openstackclient/+/863990 | 14:21 |
| opendevreview | Merged openstack/python-openstackclient master: tests: Convert network tests to use 'parse_output' https://review.opendev.org/c/openstack/python-openstackclient/+/863991 | 14:21 |
| opendevreview | Merged openstack/python-openstackclient master: Revert "Don't look up project by id if given id" https://review.opendev.org/c/openstack/python-openstackclient/+/866183 | 14:21 |
| opendevreview | Merged openstack/openstacksdk master: coe: Add support for clusters https://review.opendev.org/c/openstack/openstacksdk/+/865266 | 14:21 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Fix template required_one_of arg https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866486 | 16:17 |
| opendevreview | Pavlo Shchelokovskyy proposed openstack/keystoneauth master: Enforce scope mutual exclusion for system https://review.opendev.org/c/openstack/keystoneauth/+/802683 | 18:03 |
| opendevreview | Pavlo Shchelokovskyy proposed openstack/keystoneauth master: Allow federation to work with unversioned auth_url https://review.opendev.org/c/openstack/keystoneauth/+/866189 | 18:04 |
| *** Daviey29 is now known as Daviey2 | 19:19 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!