| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-12-08 | |||
| 18:46:21 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: image: Add missing image import options https://review.opendev.org/c/openstack/openstacksdk/+/864029 | |
| 18:46:22 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: image: Remove _base_proxy module https://review.opendev.org/c/openstack/openstacksdk/+/867041 | |
| 18:46:22 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: image: Reformat proxy modules https://review.opendev.org/c/openstack/openstacksdk/+/867040 | |
| 18:46:23 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: image: Modify signatures of various image methods https://review.opendev.org/c/openstack/openstacksdk/+/867043 | |
| 18:46:23 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: image: Remove unnecessary abstractions https://review.opendev.org/c/openstack/openstacksdk/+/867042 | |
| 18:46:24 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: image: Prevent passing conflicts args to stage_image https://review.opendev.org/c/openstack/openstacksdk/+/867045 | |
| 18:46:24 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: image: Remove unsupported parameters from v1 proxy https://review.opendev.org/c/openstack/openstacksdk/+/867044 | |
| #openstack-sdks - 2022-12-09 | |||
| 08:03:08 | eest | stephenfin: my compute/v2/server.py states _max_microversion = '2.73', and if i set that version in clouds.yaml i see '-H "OpenStack-API-Version: compute 2.73"' in the debug out put, but if i comment it out there is no such header being sent and the request fails, not sure i am doing something wrong | |
| 08:06:49 | eest | i guess it is also interesting that even if that max version is present in the code i can also make it do -H "OpenStack-API-Version: compute 2.79" requests by setting that higher version in the config, so it does not seem to cap it either | |
| 08:15:04 | eest | i did note that calling conn.config.get_api_version("compute") returns None unless i set a value in clouds.yaml | |
| 08:19:05 | eest | frickler: would be intersting to hear what your code returns regarding the config when you are not setting the value | |
| 08:29:17 | frickler | the get_api_version returns None for me too when I don't have the override in clouds.yaml. and the debug log for the server create does also show the 2.73 being requested | |
| 08:30:32 | eest | interesting | |
| 08:30:52 | frickler | but I omitted some of the parameters you use, no user_data, key_name and scheduler_hints. maybe the issue is hiding somewhere in those | |
| 08:31:12 | eest | right, yes it is a lot of stuff im sending | |
| 08:31:40 | eest | ill try commenting them out | |
| 08:32:43 | 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:35:34 | eest | same error with https://paste.opendev.org/show/bGVtHfX0SdE3UECFa68R/ | |
| 08:39:40 | eest | other things i have in my config is auth_url, application_credential_id, application_credential_secret, region_name, interface, identity_api_version and auth_type | |
| 08:54:23 | 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:44:50 | stephenfin | eest: That's very strange. It works as intended for me https://paste.opendev.org/show/b6wyrVyFUB4Xs4PL09La/ | |
| 10:45:11 | stephenfin | eest: Can you create a venv, install openstacksdk from master, and try again? | |
| 10:45:28 | stephenfin | Also, are you certain there's no compute_api_version defined in clouds.yaml? | |
| 10:46:08 | stephenfin | I also changed '_max_microversion' in openstack/compute/v2/server.py to 2.76 and SDK started using that version once I restarted the interpreter | |
| 11:48:54 | opendevreview | PB proposed openstack/ansible-collections-openstack master: Bulk creation of OpenStack SG rules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863208 | |
| 12:24:10 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Allow all external commands in tox https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867096 | |
| 12:25:54 | 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 | |
| 12:26:17 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Dropped obsolete module templates https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866683 | |
| 12:26:39 | 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:27: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 | |
| 13:09:35 | opendevreview | PB proposed openstack/ansible-collections-openstack master: Bulk creation of OpenStack SG rules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/863208 | |
| 13:56:22 | eest | stephenfin: yes since i am able to get None out of conn.config.get_api_version("compute") when commenting out the line i added after the debugging here it feels certain that there are no overlapping stuff (the config is tiny, just for one site so it would be hard for stuff to hide) | |
| 13:56:29 | eest | ill see if i can throw together a venv | |
| 14:10:08 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Allow all external commands in tox https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867096 | |
| 14:59:30 | eest | stephenfin: so i did this: python3 -m venv my-venv; . my-venv/bin/activate; pip3 install git+https://github.com/openstack/openstacksdk; pip3 list | grep openstack | |
| 14:59:33 | eest | openstacksdk 0.103.1.dev7 | |
| 14:59:54 | eest | still get "Additional properties are not allowed (u'volume_type' was unexpected)" | |
| 15:00:04 | eest | i am running this on macOS, not sure if that is relevant | |
| 15:05:34 | eest | i also see from the stack trace that i am inside my-venv/lib/python3.10/site-packages/openstack/exceptions.py so my script run should be picking up the venv-stuff as expected | |
| 15:19:57 | stephenfin | eest: just for sanity checking | |
| 15:19:58 | stephenfin | >>> import openstack | |
| 15:20:04 | stephenfin | >>> openstack.__path__ | |
| 15:20:09 | stephenfin | points to the venv? | |
| 15:20:30 | stephenfin | I ran the following against a local DevStack env using master of SDK. No issues | |
| 15:21:03 | stephenfin | https://paste.opendev.org/show/bfOAZEjUrUbXpTLwOCJ8/ | |
| 15:23:23 | stephenfin | https://paste.opendev.org/show/bA6QDIFI1ztr2IpLU2hW/ <-- output | |
| 15:24:54 | stephenfin | https://paste.opendev.org/show/bkrfwsJNzvo4tNsvGbVI/ <-- bog standard clouds.yaml generated by devstack | |
| 15:44:30 | opendevreview | Brian Haley proposed openstack/python-openstackclient master: Fix parameter handling in server add fixed ip cmd https://review.opendev.org/c/openstack/python-openstackclient/+/866811 | |
| 16:41:27 | opendevreview | Brian Haley proposed openstack/python-openstackclient master: Fix parameter handling in server add fixed ip cmd https://review.opendev.org/c/openstack/python-openstackclient/+/866811 | |
| 19:54:19 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Allow all external commands in tox https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867096 | |
| 20:10:06 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Allow all external commands in tox https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867096 | |
| 20:59:14 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Allow all external commands in tox https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867096 | |
| 21:01:36 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Allow all external commands in tox https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867096 | |
| #openstack-sdks - 2022-12-10 | |||
| 02:19:41 | opendevreview | Brian Haley proposed openstack/python-openstackclient master: Fix parameter handling in server add fixed ip cmd https://review.opendev.org/c/openstack/python-openstackclient/+/866811 | |
| 10:48:54 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Allow all external commands in tox https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867096 | |
| 12:21:39 | 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 | |
| 13:24:39 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Dropped obsolete module templates https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866683 | |
| 13:25:15 | 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 | |
| 13:25:39 | 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-11 | |||
| 16:50:35 | opendevreview | Merged openstack/ansible-collections-openstack master: Allow all external commands in tox https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867096 | |
| 21:02:42 | opendevreview | Ariel Berkowicz proposed openstack/python-openstackclient master: Migrate hypervisor stats commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/865779 | |
| 21:02:43 | opendevreview | Ariel Berkowicz proposed openstack/python-openstackclient master: Create a functional test case for hyper list and show https://review.opendev.org/c/openstack/python-openstackclient/+/866634 | |
| 23:05:21 | opendevreview | Daniel Wilson proposed openstack/python-openstackclient master: Use the SDK for server show https://review.opendev.org/c/openstack/python-openstackclient/+/864341 | |
| #openstack-sdks - 2022-12-12 | |||
| 09:39:58 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Use OpenDev's mirror for Ansible Galaxy in CI https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867202 | |
| 09:42: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 | |
| 10:22:50 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Allow all external commands in tox https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867204 | |
| 10:23:07 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Renamed image->image_name and flavor->flavor_name to avoid collisions https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867205 | |
| 12:15:06 | opendevreview | Merged 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:05:21 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Use OpenDev's mirror for Ansible Galaxy in CI https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867202 | |
| 14:20:04 | opendevreview | Merged openstack/ansible-collections-openstack master: Dropped obsolete module templates https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866683 | |
| 14:29:18 | 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 | |
| 15:01:48 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Allow all external commands in tox and fixed CI on older branches https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867204 | |
| 15:02:23 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Renamed image->image_name and flavor->flavor_name to avoid collisions https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867205 | |
| 17:28:01 | opendevreview | Brian Haley proposed openstack/python-openstackclient master: Fix parameter handling in server add fixed ip cmd https://review.opendev.org/c/openstack/python-openstackclient/+/866811 | |
| 18:04:25 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Fix really long help strings https://review.opendev.org/c/openstack/python-openstackclient/+/867272 | |
| 18:15:00 | opendevreview | Stephen Finucane proposed openstack/cliff master: Strip trailing periods when getting description https://review.opendev.org/c/openstack/cliff/+/867274 | |
| 19:05:07 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Allow external commands in tox and use non-master branches on older releases https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867204 | |
| 19:20:52 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Allow external commands in tox and use non-master branches on older releases https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867204 | |
| 23:58:52 | opendevreview | Ariel Berkowicz proposed openstack/python-openstackclient master: Create a functional test case for hyper list and show https://review.opendev.org/c/openstack/python-openstackclient/+/866634 | |
| 23:58:52 | opendevreview | Ariel Berkowicz proposed openstack/python-openstackclient master: Migrate hypervisor stats commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/865779 | |
| #openstack-sdks - 2022-12-13 | |||
| 00:58:13 | opendevreview | Harsh Mutha proposed openstack/python-openstackclient master: Fixing Review Comments for Host List and Host Show Commands# https://review.opendev.org/c/openstack/python-openstackclient/+/867317 | |
| 02:08:23 | 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 | |
| 08:08:06 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored coe_cluster{,_template} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866796 | |
| 08:09:30 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Allow external commands in tox and use non-master branches on older releases https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867204 | |
| 13:29:04 | jm1 | stephenfin, gtema: hello :) we have devstack ci jobs in ansible openstack collection which use stable/yoga branch of openstacksdk. with devstack's change to ubuntu-jammy, those jobs are broken and i cannot get them working https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867204 | |
| 13:29:40 | jm1 | stephenfin, gtema: openstacksdk's stable/yoga branch wants to install python-dev which is not available on ubuntu jammy | |
| 13:30:05 | jm1 | stephenfin, gtema: python-dev comes from https://opendev.org/openstack/openstacksdk/src/branch/stable/yoga/bindep.txt | |
| 13:31:48 | jm1 | stephenfin, gtema: with "nodeset: ubuntu-bionic" those jobs fail in "TASK [write-devstack-local-conf : Write a job-specific local_conf file]" with "hostvars['controller']\" is undefined" https://5b1f19b050590724e14e-a478df4e06fea242eac8452441f6d28b.ssl.cf5.rackcdn.com/867204/5/check/ansible-collections-openstack-functional-devstack-ansible-2.9/a2e58fb/job-output.txt | |
| 13:32:37 | jm1 | stephenfin, gtema: did you run any ci jobs for openstacksdk on stable/yoga recently? any idea how to solve this issue? | |
| 13:51:53 | jm1 | stephenfin, gtema: maybe i should ping folks in #openstack-qa ?! | |
| 15:02:57 | gmann | jm1: stable/yoga jobs need to keep running on ubuntu focal. you can pin the nodeset if they are running on jammy | |
| 15:04:59 | eest | stephenfin: i tried your code, just modifying it to fit my available flavors etc and it breaks the same way (i also printed openstack.__path__ as requested and it points to venv). something i notice when i compare our output is that your server creation directly results in a single POST at https://paste.opendev.org/show/bA6QDIFI1ztr2IpLU2hW/ (-X POST http://10.0.109.180/compute/v2.1/servers) while | |
| 15:05:05 | eest | mine first does a bunch of what looks like authorization requests, and then eventually POSTs towards https://example.com/v2.1/ |
|
| 15:05:21 | eest | sorry for the late callback, noticed you had dropped from the channel for a while | |
| 15:06:01 | jm1 | gmann: with both nodeset: bionic and nodeset: focal the stable/yoga devstack jobs fail in "TASK [write-devstack-local-conf : Write a job-specific local_conf file]" with "hostvars['controller']\" is undefined". e.g. nodeset: ubuntu-focal: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_c97/867204/2/check/ansible-collections-openstack-functional-devstack-ansible-2.9/c97001b/job-output.txt | |
| 15:07:40 | eest | i guess it is possible that either the usage of projects, or the different auth setup can be interfering with the api version stuff | |
| 15:08:04 | gmann | jm1: patch link? i can take a look today | |
| 15:09:42 | jm1 | gmann: tried several different approaches in https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867204/ | |
| 15:10:12 | jm1 | gmann: ^ interesting jobs are e.g. ansible-collections-openstack-functional-devstack-ansible-2.9 and ansible-collections-openstack-functional-devstack-ansible-2.12 | |
| 15:10:49 | gmann | ok | |