| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-12-01 | |||
| 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: tests: Add test for multiple blocks devices https://review.opendev.org/c/openstack/python-openstackclient/+/862151 | |
| 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 | |
| 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: Move _normalize_coe_* into _coe class https://review.opendev.org/c/openstack/openstacksdk/+/865043 | |
| 11:17:23 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Drop _normalize class https://review.opendev.org/c/openstack/openstacksdk/+/865044 | |
| 11:17:24 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Drop munch dependency https://review.opendev.org/c/openstack/openstacksdk/+/865045 | |
| 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: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 console methods https://review.opendev.org/c/openstack/openstacksdk/+/866570 | |
| 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: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 | |
| 01:41:55 | opendevreview | Ariel Berkowicz proposed openstack/python-openstackclient master: Created a functional test case ofr hyper list and show just to be able to make sure it works with the openstackclient sdk. Changed and modified the hypervisor stats test just to be able to work with openstackclient sdk with fakes. https://review.opendev.org/c/openstack/python-openstackclient/+/866634 | |
| 07:55:52 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Updating the glanceclient reference doc https://review.opendev.org/c/openstack/python-openstackclient/+/864118 | |
| 08:56:26 | opendevreview | Merged openstack/ansible-collections-openstack master: Refactored dns_zone{,_info} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/865545 | |
| 10:02:38 | opendevreview | Merged openstack/ansible-collections-openstack master: Bump minimum required openstacksdk release to 0.103.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866114 | |
| 10:06:40 | opendevreview | Grzegorz Grasza proposed openstack/keystoneauth master: Allow passing of version header https://review.opendev.org/c/openstack/keystoneauth/+/865439 | |
| 10:41:16 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Added tests for router interface creation in a shared network https://review.opendev.org/c/openstack/ansible-collections-openstack/+/840661 | |
| 11:00:14 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Dropped obsolete module templates https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866683 | |
| 11:20:02 | opendevreview | Merged openstack/ansible-collections-openstack master: Allow setting flavor description https://review.opendev.org/c/openstack/ansible-collections-openstack/+/864564 | |
| 13:25:32 | frickler | stephenfin: I noticed there is still a bit unclearness in your action doc regarding set/unset and boolean variables. like if we have "project set --enable", shouldn't we also have "project unset --enable"? | |
| 13:26:23 | frickler | or, where I first stumbled about this now, for subnet pool we have "set --default" and "set --no-default", but no unset | |
| 13:27:59 | frickler | for "set --address-scope <address-scope>" I think the right path towards consistency would be to deprecate "set --no-address-scope" and create "unset --address-scope", but I'm not sure about the others | |
| 13:37:42 | opendevreview | PB proposed openstack/ansible-collections-openstack master: Bulk creation of OpenStack SG rules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863208 | |
| 14:49:20 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Added tests for router interface creation in a shared network https://review.opendev.org/c/openstack/ansible-collections-openstack/+/840661 | |
| 14:49:56 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Dropped obsolete module templates https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866683 | |
| 16:21:45 | opendevreview | Huda Irshad proposed openstack/openstacksdk master: modified instance_uuid to be resource.Body in server_actions.py https://review.opendev.org/c/openstack/openstacksdk/+/866383 | |
| 16:24:25 | opendevreview | Merged openstack/keystoneauth master: Fix linters and bindep on jammy https://review.opendev.org/c/openstack/keystoneauth/+/865807 | |
| 17:38:03 | 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 ignore_missing input, modified expections to sdk version in server_event.py and added in dependencies for commit message https://review.opendev.org/c/openstack/python-openstackclient/+/863718 | |
| 18:57:08 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Moved Octavia image upload and CirrOS image identification to Ansible https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866795 | |
| 19:16:52 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: [DNM] Refactored coe_cluster{,_template} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866796 | |
| 20:12:03 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Moved Octavia image upload and CirrOS image identification to Ansible https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866795 | |