| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-12-03 | |||
| 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 | |
| 20:12:38 | 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 | |
| 22:22:21 | 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 | |
| 22:49:24 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Fix parameter handling in server add fixed ip cmd https://review.opendev.org/c/openstack/python-openstackclient/+/866811 | |
| #openstack-sdks - 2022-12-07 | |||
| 02:46:57 | opendevreview | Harsh Mutha proposed openstack/python-openstackclient master: Update Host List and Host Show command to use OpenStackSDK. https://review.opendev.org/c/openstack/python-openstackclient/+/866818 | |
| 03:22:17 | opendevreview | Ariel Berkowicz proposed openstack/python-openstackclient master: compute: Migrate hypervisor stats commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/865779 | |
| 03:22:18 | 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 | |
| 08:28:28 | 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 | |
| 08:43:02 | 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 | |
| 08:59:30 | 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 | |
| 10:17:05 | 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 | |
| 10:36:36 | 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 | |
| 11:18:22 | 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 | |
| 11:23:50 | 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 | |
| 12:37:20 | opendevreview | TaeJun Jang proposed openstack/python-openstackclient master: Add image metadef namespace command https://review.opendev.org/c/openstack/python-openstackclient/+/863525 | |
| 13:16:55 | 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 | |
| 13:25:07 | 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 | |
| 17:19:09 | opendevreview | PB proposed openstack/ansible-collections-openstack master: Bulk creation of OpenStack SG rules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863208 | |
| 17:24:53 | opendevreview | PB proposed openstack/ansible-collections-openstack master: Bulk creation of OpenStack SG rules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863208 | |
| 17:35:41 | opendevreview | PB proposed openstack/ansible-collections-openstack master: Bulk creation of OpenStack SG rules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863208 | |
| 19:52:31 | 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:55:17 | 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 | |
| 20:25:15 | 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 | |
| #openstack-sdks - 2022-12-08 | |||
| 11:55:15 | opendevreview | Grzegorz Grasza proposed openstack/keystoneauth master: Remove six https://review.opendev.org/c/openstack/keystoneauth/+/816671 | |
| 12:50:25 | eest | i am trying to create a server in openstack using block storage, i have gotten it to work by calling conn.compute.create_server() with a block_device_mapping_v2 list with the same content as is constructed in https://github.com/openstack/openstacksdk/blob/17fde652ada983d6a04dc98b90913f8066187dc3/openstack/cloud/_compute.py#L999-L1006 now to my issue: is there a way to point out what specific "volume | |
| 12:50:31 | eest | type" you want? if i run "openstack volume type list" i see there are more than one volume type and currently i get one of them, but i am not sure how i would pick a specific one | |
| 12:52:53 | eest | if i just try suppling an "volume_type" field i get a "Additional properties are not allowed (u'volume_type' was unexpected)" error | |
| 13:13:55 | frickler | eest: seems like support for this is still missing. stephenfin ^^ do you have a list of nova gaps? | |
| 13:18:22 | stephenfin | eest: '--volume' is a helper option that allows you to attach a volume using the most common configuration. If you want to configure additional things like volume type, you will need to use '--block-device'. See https://review.opendev.org/c/openstack/python-openstackclient/+/866325 for more context | |
| 13:19:45 | stephenfin | eest: You'll note there that we're currently unable to use '--block-device' for the boot volume though. There's a fix proposed but it hasn't been included in a release yet. If you need a boot volume, you'll have to use the legacy '--block-device-mapping' option | |
| 13:21:28 | eest | hmm, just to be clear i am not trying to create the server using the openstack CLI, i only used it to show how i proved there were multiple volume types to chose from | |
| 13:22:12 | stephenfin | oh, sorry, you're using SDK | |
| 13:22:35 | eest | i am right now calling the compute.create_server() function from code, and passing it block_device_mapping_v2 basically filled in the same way the code i linked on github does | |
| 13:23:07 | stephenfin | eest: https://docs.openstack.org/api-ref/compute/?expanded=create-server-detail#id11 | |
| 13:23:19 | stephenfin | Look for "block_device_mapping_v2 (Optional)" | |
| 13:24:22 | stephenfin | You'll see under that all the possible keys and values for each (JSON) object in the block_device_mapping_v2 (JSON) list field | |
| 13:24:40 | eest | found it, so it seems there is no way to pass in the "type" then? | |
| 13:24:53 | stephenfin | block_device_mapping_v2.volume_type (Optional) | |
| 13:25:05 | stephenfin | The device volume_type. This can be used to specify the type of volume which the compute service will create and attach to the server. If not specified, the block storage service will provide a default volume type. See the block storage volume types API for more details. | |
| 13:25:28 | frickler | stephenfin: but it seems the sdk rejects that option? | |
| 13:26:42 | frickler | eest: can you share the code you are invoking so we can doublecheck? use paste.opendev.org | |
| 13:26:50 | eest | yes, hold on | |
| 13:27:06 | stephenfin | I'd be surprised if it did. That seems to be mapped correctly in openstack/compute/v2/server.py | |
| 13:28:51 | eest | https://paste.opendev.org/show/bztGgATTQcwKdam5UwVl/ | |
| 13:29:44 | eest | results in "Additional properties are not allowed (u'volume_type' was unexpected)" | |
| 13:30:07 | eest | it is possible i should not supply it in the dict? | |
| 13:31:25 | stephenfin | that's a nova error, not openstacksdk. You're using the wrong microversion. You need 2.67 or greater | |
| 13:31:50 | eest | hmm, is that something supplied to create_server() or something else? | |
| 13:33:55 | stephenfin | yes, you should be able to pass microversion to create_server | |