| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-02-14 | |||
| 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 | |
| 12:33:40 | opendevreview | Goutham Pacha Ravi proposed openstack/openstacksdk master: Resource locks and access rules restrictions https://review.opendev.org/c/openstack/openstacksdk/+/889519 | |
| 13:09:12 | stephenfin | M0weng[m]: 'sudo dnf install pre-commit; cd openstacksdk; pre-commit install; # ...?; # profit' | |
| 13:09:47 | stephenfin | s/openstacksdk/python-openstackclient/ | |
| 13:12:35 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: identity: Migrate 'user' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/909030 | |
| 14:02:09 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Resource locks and access rules restrictions https://review.opendev.org/c/openstack/openstacksdk/+/889519 | |
| 15:30:45 | opendevreview | Stephen Finucane proposed openstack/cliff master: Blacken code base https://review.opendev.org/c/openstack/cliff/+/893670 | |
| 15:30:45 | opendevreview | Stephen Finucane proposed openstack/cliff master: Ignore blacken patch https://review.opendev.org/c/openstack/cliff/+/893671 | |
| 15:30:46 | opendevreview | Stephen Finucane proposed openstack/cliff master: tox: Use pre-commit for linter checks https://review.opendev.org/c/openstack/cliff/+/893672 | |
| 18:33:18 | opendevreview | Merged openstack/openstacksdk master: docs: Add docs on configuration of a service user https://review.opendev.org/c/openstack/openstacksdk/+/893505 | |
| #openstack-sdks - 2024-02-18 | |||
| 17:30:17 | opendevreview | Takashi Kajinami proposed openstack/os-service-types master: Bump hacking https://review.opendev.org/c/openstack/os-service-types/+/909334 | |
| 19:45:08 | opendevreview | Merged openstack/openstacksdk master: Fix list of server migrations https://review.opendev.org/c/openstack/openstacksdk/+/887847 | |
| #openstack-sdks - 2024-02-19 | |||
| 07:41:27 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add support for managing volumes https://review.opendev.org/c/openstack/python-openstackclient/+/895214 | |
| 15:58:53 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Resource locks and access rules restrictions https://review.opendev.org/c/openstack/openstacksdk/+/889519 | |
| #openstack-sdks - 2024-02-20 | |||
| 10:36:26 | opendevreview | Pranali Deore proposed openstack/openstacksdk master: Adding support for new add/get location apis https://review.opendev.org/c/openstack/openstacksdk/+/883271 | |
| 13:12:52 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adds CLI support for glance md-tags https://review.opendev.org/c/openstack/python-openstackclient/+/905866 | |
| 13:20:45 | opendevreview | Tatsuya Hayashino proposed openstack/keystoneauth master: Update requests of device flow for public client https://review.opendev.org/c/openstack/keystoneauth/+/909561 | |
| 16:07:21 | stephenfin | M0weng[m]: I'm guessing you still have to push the change? https://review.opendev.org/c/openstack/python-openstackclient/+/909030/8/openstackclient/tests/unit/identity/v3/test_user.py#29 | |
| 17:09:54 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adding client support for ``glance image-upload`` https://review.opendev.org/c/openstack/python-openstackclient/+/882938 | |
| 17:17:50 | opendevreview | Merged openstack/openstacksdk master: Resource locks and access rules restrictions https://review.opendev.org/c/openstack/openstacksdk/+/889519 | |
| 18:21:02 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: identity: Migrate 'user' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/909030 | |
| 18:47:55 | ozzzo_work | I'm reading this: https://docs.openstack.org/openstacksdk/latest/user/guides/object_store.html | |
| 18:48:54 | ozzzo_work | and trying to use an admin account to look at user projects. For other Openstack components I'd use something like: cloud.network.networks(project_id="' + projectid + '")' | |
| 18:49:59 | ozzzo_work | but when I try that with Swift: for cont in cloud.object_store.containers(project_id="' + projectid + '"): | |
| 18:50:12 | ozzzo_work | I get an error: Invalid query params: project_id | |
| 18:55:33 | ozzzo_work | Looking here: https://docs.openstack.org/openstacksdk/latest/user/proxies/object_store.html | |
| 18:56:08 | ozzzo_work | I don't see an obvious way to specify the project. Do I need to use the old python API for this? | |
| 19:34:07 | ozzzo_work | maybe I can use the "session" parameter? Can I use the admin account to create a session pointing to a user project? | |
| 20:04:35 | ozzzo_work | When I look at the Swift API, I see a way to specify the account so it seems like there must be something corresponding in the SDK, but it's not obvious how to do it: cmd = "git --version" returned_value = os.system(cmd) # returns the exit code in unix print('returned value:', returned_value) | |
| 20:04:54 | ozzzo_work | oops bad paste; URL is: https://docs.openstack.org/api-ref/object-store/#show-container-details-and-list-objects | |
| #openstack-sdks - 2024-02-21 | |||
| 07:56:32 | adam-metal3 | Hi all! I am new to using openstack sdk and in general I am new in most openstack components, after reading the nova and openstacksdk docs I still have a question. I hope someone could give some direction. If I understand correctly multiple volumes could be attached to a VM and the VM can be booted from volume. | |
| 07:56:32 | adam-metal3 | Could there be a scenario where multiple bootable volume is attached to a VM and if yes then how could I tell via the openstacksdk that which volume was used for booting in this scenario? (I have asked this in nove IRC also not sure which channel is the better to discuss this) | |
| 09:46:45 | gtema | stephenfin: pls no merges anymore - code freeze | |
| 10:53:14 | stephenfin | gtema: ack | |
| 11:00:13 | stephenfin | adam-metal3: Really a question for #openstack-nova. I don't think it's possible though: you want the boot_index field which doesn't appear to be returned via any API | |
| 11:01:49 | adam-metal3 | stephenfin: thanks for the info ! | |
| 13:01:03 | opendevreview | Lajos Katona proposed openstack/openstacksdk master: Add sort_key and sort_dir to Network and Subnet https://review.opendev.org/c/openstack/openstacksdk/+/909656 | |
| 13:31:59 | ozzzo | Can anyone help with my Swift SDK question? | |
| 14:42:03 | amarao | Hello. Can someone see my PR for ansible module for subnet? https://review.opendev.org/c/openstack/ansible-collections-openstack/+/907301 It's already two three weeks without any feedback... | |
| 16:27:18 | opendevreview | Lajos Katona proposed openstack/python-openstackclient master: Router flavor_id can be a name https://review.opendev.org/c/openstack/python-openstackclient/+/902654 | |
| 17:09:21 | stephenfin | gtema: https://review.opendev.org/c/openstack/nova-specs/+/909448 | |
| 17:10:01 | gtema | oh, that's sweet | |
| 17:10:29 | gtema | https://github.com/gtema/openstack-codegenerator/blob/main/codegenerator/openapi/nova_schemas.py is what I ended up with | |
| 17:11:05 | stephenfin | Yeah, I saw that. I have a test that's partly inspired by that script. Just need to push it up next time I've a day to work on this | |
| 17:11:05 | gtema | what I also do not have so far are the schemas for the long deprecated operations | |
| 17:11:24 | stephenfin | Yeah, my schemas for those are '{}' | |
| 17:11:53 | stephenfin | If the resources is returning HTTP 410 (Gone), it's really crufty and not worth worrying about ImO | |
| 17:12:23 | gtema | that was also my decision | |
| 17:13:03 | stephenfin | Did you notice that the routes.Mapper.resource creates a whole load of routes that don't actually exist when you try to iterate on them? | |
| 17:13:24 | gtema | I still haven't asked TC for getting the generator back from GH to opendev (requires a new project under SDK umbrella) | |