| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-11-30 | |||
| 14:00:09 | opendevreview | Merged openstack/python-openstackclient stable/xena: compute: Require image when rebuilding a volume-backed server https://review.opendev.org/c/openstack/python-openstackclient/+/859970 | |
| 14:21:33 | opendevreview | Johannes Kulik proposed openstack/openstacksdk master: Revert "Revert "Add "security_group_ids" to Port's query parameters"" https://review.opendev.org/c/openstack/openstacksdk/+/866008 | |
| 14:24:58 | opendevreview | Johannes Kulik proposed openstack/openstacksdk master: Revert "Revert "Add "security_group_ids" to Port's query parameters"" https://review.opendev.org/c/openstack/openstacksdk/+/866008 | |
| 14:39:29 | opendevreview | PB proposed openstack/ansible-collections-openstack master: Bulk creation of OpenStack SG rules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863208 | |
| 16:01:22 | opendevreview | Merged openstack/ansible-collections-openstack master: Refactored neutron_rbac_polic{ies_info,y} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/864871 | |
| 16:33:48 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: image: Fail if we can't find an image https://review.opendev.org/c/openstack/python-openstackclient/+/866184 | |
| 16:33:48 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Revert "Don't look up project by id if given id" https://review.opendev.org/c/openstack/python-openstackclient/+/866183 | |
| 16:33:49 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: image: Rename import https://review.opendev.org/c/openstack/python-openstackclient/+/866185 | |
| 17:01:46 | opendevreview | Pavlo Shchelokovskyy proposed openstack/keystoneauth master: Allow federation to work with unversioned auth_url https://review.opendev.org/c/openstack/keystoneauth/+/866189 | |
| 17:15:46 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: image: Add support for additional image import methods https://review.opendev.org/c/openstack/python-openstackclient/+/866191 | |
| 17:27:58 | opendevreview | Pavlo Shchelokovskyy proposed openstack/keystoneauth master: Allow federation to work with unversioned auth_url https://review.opendev.org/c/openstack/keystoneauth/+/866189 | |
| 18:13:35 | opendevreview | Merged openstack/ansible-collections-openstack master: Refactored subnet_pool module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/864902 | |
| 20:30:49 | opendevreview | Merged openstack/ansible-collections-openstack master: Refactored catalog_service{,_info} and auth modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/865122 | |
| 23:31:35 | opendevreview | Ariel-Berkowicz proposed openstack/python-openstackclient master: compute: Migrate hypervisor stats commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/865779 | |
| #openstack-sdks - 2022-12-01 | |||
| 06:37:44 | opendevreview | PB proposed openstack/ansible-collections-openstack master: Bulk creation of OpenStack SG rules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863208 | |
| 06:50:36 | opendevreview | PB proposed openstack/ansible-collections-openstack master: Bulk creation of OpenStack SG rules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863208 | |
| 10:04:29 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: image: Fail if we can't find an image https://review.opendev.org/c/openstack/python-openstackclient/+/866184 | |
| 10:04:29 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: image: Rename import https://review.opendev.org/c/openstack/python-openstackclient/+/866185 | |
| 10:13:24 | opendevreview | PB proposed openstack/ansible-collections-openstack master: Bulk creation of OpenStack SG rules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863208 | |
| 11:36:03 | opendevreview | Tobias Urdin proposed openstack/python-openstackclient master: Add --boot-from-volume-type to specify volume type for BFV https://review.opendev.org/c/openstack/python-openstackclient/+/866325 | |
| 11:43:59 | opendevreview | Tobias Urdin proposed openstack/python-openstackclient master: Add --boot-from-volume-type to specify volume type for BFV https://review.opendev.org/c/openstack/python-openstackclient/+/866325 | |
| 12:14:35 | opendevreview | Pavlo Shchelokovskyy proposed openstack/keystoneauth master: Fix linters and bindep on jammy https://review.opendev.org/c/openstack/keystoneauth/+/865807 | |
| 13:51:49 | tweining | hi, I'm currently trying to replace neutronclient with openstacksdk in Octavia, but I'm struggling with creating the Connection instance. | |
| 13:53:45 | tweining | it seems there are multiple ways to do it, and I couldn't get any of them to work yet. Can you point me to what is the recommended way to create a connection in an openstack service? | |
| 13:58:59 | tweining | some ways are documented in https://docs.openstack.org/openstacksdk/latest/user/connection.html, but there is also a way using getonecloud() https://github.com/openstack/octavia-dashboard/blob/master/octavia_dashboard/api/rest/lbaasv2.py#L39 | |
| 15:24:17 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Allow users to manually specify bootable volumes https://review.opendev.org/c/openstack/python-openstackclient/+/862176 | |
| 15:24:17 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Add test for multiple blocks devices https://review.opendev.org/c/openstack/python-openstackclient/+/862151 | |
| 19:13:47 | opendevreview | Huda Irshad proposed openstack/openstacksdk master: made modifications to allow server event list options to pass through to server actions https://review.opendev.org/c/openstack/openstacksdk/+/866383 | |
| 20:28:26 | opendevreview | Huda Irshad proposed openstack/python-openstackclient master: modified server event commands to work with sdk https://review.opendev.org/c/openstack/python-openstackclient/+/863718 | |
| #openstack-sdks - 2022-12-02 | |||
| 06:02:53 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Updating the glanceclient reference doc https://review.opendev.org/c/openstack/python-openstackclient/+/864118 | |
| 11:42:29 | 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:50 | sfinucan | nova is using openstacksdk for ironic stuff for a while. I'm looking at what they do now | |
| 11:44:41 | tweining | sfinucan: thanks. BTW, I found out the way octavia-dashboard does it is actually outdated. | |
| 11:45:38 | stephenfin | tweining: https://github.com/openstack/nova/blob/d05f9ef0dcecd55f37f362afe4b32544d8d81636/nova/utils.py#L959-L985 | |
| 11:45:48 | gtema | https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/connection.py#L80 describes how to pass creds directly with connection.Connection | |
| 11:46:04 | gtema | but all same params can also go to openstack.connect directly | |
| 11:47:07 | tweining | yeah, those two ways, per CONF and per keyword args, are the two that seem right to me. | |
| 11:48:23 | 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:24 | stephenfin | time | |
| 11:49:15 | 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:37 | stephenfin | yes, and less complex to boot | |
| 11:50:17 | 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:54:12 | 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:55:28 | 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:41 | gtema | those are the ones that we definitely support and keep maintaining | |
| 11:57:44 | 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:53 | gtema | this may give you some potential direction | |
| 11:58:57 | tweining | gtema: cool, thanks. I will have a look | |
| 12:59:31 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Rework network functional tests https://review.opendev.org/c/openstack/openstacksdk/+/859114 | |
| 13:00:47 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Whitelist cloud functional tests in acceptance https://review.opendev.org/c/openstack/openstacksdk/+/860009 | |
| 13:02:57 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Implement acceptance test job https://review.opendev.org/c/openstack/openstacksdk/+/860399 | |
| 14:21:39 | opendevreview | Merged openstack/python-openstackclient master: Use the compute SDK in server list https://review.opendev.org/c/openstack/python-openstackclient/+/861546 | |
| 14:21:41 | 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:43 | 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:45 | 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:47 | 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:49 | 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:52 | 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:54 | opendevreview | Merged openstack/openstacksdk master: coe: Add support for clusters https://review.opendev.org/c/openstack/openstacksdk/+/865266 | |
| 16:17:12 | 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 | |
| 18:03:48 | opendevreview | Pavlo Shchelokovskyy proposed openstack/keystoneauth master: Enforce scope mutual exclusion for system https://review.opendev.org/c/openstack/keystoneauth/+/802683 | |
| 18:04:09 | opendevreview | Pavlo Shchelokovskyy proposed openstack/keystoneauth master: Allow federation to work with unversioned auth_url https://review.opendev.org/c/openstack/keystoneauth/+/866189 | |
| #openstack-sdks - 2022-12-03 | |||
| 00:56:03 | opendevreview | Huda Irshad proposed openstack/openstacksdk master: mend https://review.opendev.org/c/openstack/openstacksdk/+/866507 | |
| 02:18:01 | opendevreview | Huda Irshad proposed openstack/python-openstackclient master: modified server event commands to work with sdk https://review.opendev.org/c/openstack/python-openstackclient/+/863718 | |
| 02:19:00 | opendevreview | Huda Irshad proposed openstack/python-openstackclient master: modified server event commands to work with sdk https://review.opendev.org/c/openstack/python-openstackclient/+/863718 | |
| 20:15:25 | opendevreview | Huda Irshad proposed openstack/openstacksdk master: removed kwargs from inheritance input in class https://review.opendev.org/c/openstack/openstacksdk/+/866383 | |
| 22:36:48 | opendevreview | Daniel Wilson proposed openstack/python-openstackclient master: WIP: Use the SDK for server show https://review.opendev.org/c/openstack/python-openstackclient/+/864341 | |
| 23:05:39 | opendevreview | Huda Irshad proposed openstack/python-openstackclient master: adding in the kwargs arguement for server_actions and adding in dependencies for the test https://review.opendev.org/c/openstack/python-openstackclient/+/863718 | |
| 23:22:53 | opendevreview | Huda Irshad proposed openstack/openstacksdk master: modified _proxy.py to correct linting error https://review.opendev.org/c/openstack/openstacksdk/+/866383 | |
| #openstack-sdks - 2022-12-04 | |||
| 01:19:31 | opendevreview | Huda Irshad proposed openstack/openstacksdk master: improve linting error from previous patch on _proxy.py https://review.opendev.org/c/openstack/openstacksdk/+/866383 | |
| 21:25:07 | opendevreview | Daniel Wilson proposed openstack/python-openstackclient master: Use the SDK for server show https://review.opendev.org/c/openstack/python-openstackclient/+/864341 | |
| 22:05:03 | opendevreview | Huda Irshad proposed openstack/openstacksdk master: removed the additional indentation on _proxy.py https://review.opendev.org/c/openstack/openstacksdk/+/866383 | |
| 22:09:40 | opendevreview | Huda Irshad proposed openstack/openstacksdk master: corrected the indentation error to aline with the inputs of the previous line on _proxy.py https://review.opendev.org/c/openstack/openstacksdk/+/866383 | |
| #openstack-sdks - 2022-12-05 | |||
| 00:53:04 | opendevreview | Huda Irshad proposed openstack/python-openstackclient master: adding in the kwargs arguement for server_actions, adding in paginate false to allow for limit to be decided by user, and adding in dependencies for the test https://review.opendev.org/c/openstack/python-openstackclient/+/863718 | |
| 02:48:38 | opendevreview | Huda Irshad proposed openstack/python-openstackclient master: added in kwargs options, added in pagniate argument, modified test_server_event in unit test to add paginated input, and added in dependencies for commit message https://review.opendev.org/c/openstack/python-openstackclient/+/863718 | |
| 03:34:58 | opendevreview | Huda Irshad proposed openstack/python-openstackclient master: added in kwargs options, added in pagniate argument, modified test_server_event in unit test to add paginated input, and added in dependencies for commit message https://review.opendev.org/c/openstack/python-openstackclient/+/863718 | |
| 06:39:15 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Updating the glanceclient reference doc https://review.opendev.org/c/openstack/python-openstackclient/+/864118 | |
| 08:26:21 | opendevreview | Arnaud Morin proposed openstack/ansible-collections-openstack master: Add reboot and reboot_hard actions on server https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866550 | |
| 08:36:39 | opendevreview | Arnaud Morin proposed openstack/ansible-collections-openstack master: Add reboot and reboot_hard actions on server https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866550 | |
| 10:19:16 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: docs: Fix typos https://review.opendev.org/c/openstack/python-openstackclient/+/866561 | |
| 11:17:22 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Move normalize security group to sg class https://review.opendev.org/c/openstack/openstacksdk/+/865025 | |
| 11:17:23 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Drop _normalize class https://review.opendev.org/c/openstack/openstacksdk/+/865044 | |
| 11:17:23 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Move _normalize_coe_* into _coe class https://review.opendev.org/c/openstack/openstacksdk/+/865043 | |
| 11:17:24 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Move normalize_server to compute mixin https://review.opendev.org/c/openstack/openstacksdk/+/866567 | |
| 11:17:24 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Drop munch dependency https://review.opendev.org/c/openstack/openstacksdk/+/865045 | |
| 11:37:03 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: baremetal: Add Node.inject_nmi method https://review.opendev.org/c/openstack/openstacksdk/+/866569 | |
| 11:37:04 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: baremetal: Add Node boot_device methods https://review.opendev.org/c/openstack/openstacksdk/+/866571 | |
| 11:37:04 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: baremetal: Add Node console methods https://review.opendev.org/c/openstack/openstacksdk/+/866570 | |
| 11:57:21 | opendevreview | Merged openstack/openstacksdk master: Implement unified search_resources method https://review.opendev.org/c/openstack/openstacksdk/+/859845 | |
| 12:29:59 | opendevreview | Merged openstack/python-openstackclient master: docs: Describe common actions found in OSC https://review.opendev.org/c/openstack/python-openstackclient/+/865690 | |
| 12:30:01 | opendevreview | Merged openstack/python-openstackclient master: docs: Document how we manage API versions https://review.opendev.org/c/openstack/python-openstackclient/+/865691 | |
| 12:59:44 | opendevreview | Merged openstack/python-openstackclient master: tests: Add test for multiple blocks devices https://review.opendev.org/c/openstack/python-openstackclient/+/862151 | |
| 12:59:47 | opendevreview | Merged openstack/python-openstackclient master: compute: Allow users to manually specify bootable volumes https://review.opendev.org/c/openstack/python-openstackclient/+/862176 | |
| 12:59:53 | opendevreview | Merged openstack/python-openstackclient master: image: Add 'image import' command https://review.opendev.org/c/openstack/python-openstackclient/+/862386 | |
| 15:20:19 | opendevreview | Merged openstack/openstacksdk master: Stop normalizing floating ips https://review.opendev.org/c/openstack/openstacksdk/+/865024 | |
| 19:50:16 | opendevreview | Merged openstack/ansible-collections-openstack master: Refactored object{,_container} modules breaking backward compatibility https://review.opendev.org/c/openstack/ansible-collections-openstack/+/865463 | |
| 23:22:13 | opendevreview | Huda Irshad proposed openstack/python-openstackclient master: added in kwargs options, added in pagniate argument, modified test_server_event in unit test to add paginated input, and added in dependencies for commit message https://review.opendev.org/c/openstack/python-openstackclient/+/863718 | |
| 23:57:51 | opendevreview | Ariel Berkowicz proposed openstack/python-openstackclient master: We migrate these commands. We also add some new functional tests to validate behavior of the hypervisor commands. https://review.opendev.org/c/openstack/python-openstackclient/+/866631 | |
| #openstack-sdks - 2022-12-06 | |||
| 00:25:47 | opendevreview | Ariel Berkowicz proposed openstack/python-openstackclient master: Compared review #858366 with #763414 to create a stable build based off of master https://review.opendev.org/c/openstack/python-openstackclient/+/865779 | |
| 01:41:54 | opendevreview | Ariel Berkowicz proposed openstack/python-openstackclient master: compute: Migrate hypervisor stats commands to SDK We migrate these commands. We also add some new functional tests to validate behavior of the hypervisor commands. https://review.opendev.org/c/openstack/python-openstackclient/+/865779 | |