| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Volume module: - Add 'is_multiattach' parameter. When it is true, volume will be shareble https://review.opendev.org/c/openstack/ansible-collections-openstack/+/865477 | 08:08 |
|---|---|---|
| opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add volume revert snapshot command https://review.opendev.org/c/openstack/python-openstackclient/+/869654 | 08:08 |
| opendevreview | Jakob Meng 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:16 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Updated docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868923 | 10:02 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Updated docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868923 | 10:07 |
| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Drop _normalize class https://review.opendev.org/c/openstack/openstacksdk/+/865044 | 10:50 |
| opendevreview | Merged openstack/ansible-collections-openstack master: Add reboot and reboot_hard actions on server https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866550 | 12:11 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Updated docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868923 | 12:16 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Added is_multiattach parameter to volume module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/865477 | 12:34 |
| eest | i dont know if people recall my problem where openstack-sdk did not allow me to create a server with some specific settings unless i configured compute_api_version: "2.79" in clouds.yaml. I have now discovered that while it fixes my call to create a server, it breaks my call to create a server_group: Additional properties are not allowed (u'policies' was unexpected). It seems that even if i have set | 12:34 |
| eest | my expected API version the code will instead inspect its "session" to see what microversion is supported and since it is 2.1 it tries using the older "policies" field instead of "policy" that it should be using | 12:34 |
| gtema | do you hardcode microversion now? It should not be required, since sdk tries to choose max supported by user and cloud | 12:35 |
| eest | yeah, i set "compute_api_version" in clouds.yaml, it is a bit of a mystery why the automatic microversion stuff does not work for me... i have a hard time figuring out where in the code this is actually discovered, maybe i could find out why | 12:36 |
| eest | not sure if it is the openstack server side stuff that is not reporting the actually supported microversion, or if it is somehow lost in openstacksdk for some reason (i am using application credential auth which incurs extra API lookups for instance) | 12:38 |
| gtema | pls try to figure out which param exactly is causing you issues without specifying MV. Hardcoding it this way is something very tricky and not without consequences | 12:38 |
| eest | btw, i am not setting the "default" microversion, i believe that is another setting | 12:38 |
| eest | right | 12:38 |
| gtema | you should normally not set any version at all | 12:39 |
| opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Add support for glance cache list https://review.opendev.org/c/openstack/openstacksdk/+/869710 | 12:40 |
| opendevreview | Merged openstack/ansible-collections-openstack master: Updated docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868923 | 12:45 |
| eest | i have went so far as to checking compute/v2/server_group.py where it does "if utils.supports_microversion(session, '2.64'):" to figure out what policy thing to set, and inside supprts_microversion() it calls endpoint_data = adapter.get_endpoint_data(). printing endpoint_data i see: | 12:47 |
| eest | EndpointData{api_version=(2, 1), catalog_url=https://url:port/v2.1/project-id, endpoint_id=someid, interface=public, major_version=None, max_microversion=None, min_microversion=None, next_min_version=None, not_before=None, raw_endpoint={'region_id': 'regionname', 'url': 'https://url:port/v2.1/project-id', 'region': 'regionname', 'interface': 'public', 'id': 'someid'}, region_name=regionname, | 12:47 |
| eest | service_id=someid, service_name=nova, service_type=compute, service_url=None, url=https://url:port/v2.1/projectid} | 12:47 |
| eest | where might the code try to figure out what should be supported? i obviously am able to do calls with a higher version than 2.1 (needed for my compute creation) yet it seems determined that my "session" is 2.1 | 12:48 |
| opendevreview | Artem Goncharov proposed openstack/os-client-config master: Update tox to support v4 https://review.opendev.org/c/openstack/os-client-config/+/869712 | 12:52 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Updated docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/869717 | 13:21 |
| opendevreview | Stephen Finucane proposed openstack/os-client-config master: Add support for tox 4 https://review.opendev.org/c/openstack/os-client-config/+/869723 | 13:58 |
| stephenfin | gtema: Need that to unblock openstacksdk gates since the os-client-config-tox-py38-tips job is failing | 14:07 |
| gtema | I know stephenfin | 14:07 |
| stephenfin | Okay, cool :) | 14:08 |
| gtema | I hope your change will pass, since my just failed | 14:08 |
| gtema | but I gave your +2 already | 14:08 |
| stephenfin | oh, I missed yours. Sorry /o\ | 14:08 |
| gtema | np | 14:08 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Fixed docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/869726 | 14:20 |
| opendevreview | Merged openstack/ansible-collections-openstack master: Added is_multiattach parameter to volume module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/865477 | 14:55 |
| opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Updated docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/869717 | 14:55 |
| opendevreview | Merged openstack/ansible-collections-openstack master: Fixed docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/869726 | 14:55 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack stable/1.0.0: Fixed docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/869737 | 15:13 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored loadbalancer module breaking backward compatibility https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867558 | 15:15 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored lb_{health_monitor,listener,member,pool} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/867560 | 15:16 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored coe_cluster{,_template} modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/866796 | 15:16 |
| opendevreview | Merged openstack/ansible-collections-openstack stable/1.0.0: Fixed docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/869737 | 15:32 |
| eest | hmm, might actually be closer to a solution, from the debug output i can actually see "No version document at..." errors which appears to be thrown by the keystone auth stuff | 15:38 |
| eest | guessing this is the reason the client is not able to detect microversions, to be continued | 15:38 |
| gtema | yes, this may be really a reason | 15:40 |
| opendevreview | Merged openstack/os-client-config master: Add support for tox 4 https://review.opendev.org/c/openstack/os-client-config/+/869723 | 16:50 |
| opendevreview | Tom Weininger proposed openstack/openstacksdk master: Add typing information and documentation https://review.opendev.org/c/openstack/openstacksdk/+/867968 | 17:52 |
| opendevreview | Lajos Katona proposed openstack/openstacksdk master: Add BGP Speakers and Peers to SDK https://review.opendev.org/c/openstack/openstacksdk/+/869485 | 19:08 |
| opendevreview | Merged openstack/openstacksdk master: Move _normalize_coe_* into _coe class https://review.opendev.org/c/openstack/openstacksdk/+/865043 | 21:39 |
| opendevreview | Merged openstack/openstacksdk master: Allow passing more arguments to create_port https://review.opendev.org/c/openstack/openstacksdk/+/868106 | 21:53 |
| -opendevstatus- NOTICE: One of our CI job log storage providers appears to be having trouble with log uploads and retrievals. We are in the process of removing that provider from the pool. | 22:45 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!