| openstackgerrit | Merged openstack/python-openstackclient master: Update master for stable/victoria https://review.opendev.org/755889 | 00:30 |
|---|---|---|
| *** ricolin_ has joined #openstack-sdks | 00:55 | |
| *** ricolin_ has quit IRC | 01:01 | |
| *** ricolin_ has joined #openstack-sdks | 01:01 | |
| *** enriquetaso has joined #openstack-sdks | 01:04 | |
| openstackgerrit | Merged openstack/python-openstackclient master: Add Python3 wallaby unit tests https://review.opendev.org/755890 | 01:36 |
| *** enriquetaso has quit IRC | 02:04 | |
| openstackgerrit | Sam Morrison proposed openstack/python-openstackclient master: Restore behavior of image create with same name. https://review.opendev.org/756187 | 03:05 |
| openstackgerrit | Wenping Song proposed openstack/cliff master: Remove six https://review.opendev.org/756188 | 03:41 |
| *** brinzhang0 has joined #openstack-sdks | 03:42 | |
| *** brinzhang_ has quit IRC | 03:45 | |
| *** yolanda has quit IRC | 04:07 | |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #openstack-sdks | 04:33 | |
| *** nikparasyr has joined #openstack-sdks | 05:45 | |
| *** gouthamr has quit IRC | 06:05 | |
| *** fyx has quit IRC | 06:05 | |
| *** mgoddard has quit IRC | 06:05 | |
| *** openstackgerrit has quit IRC | 06:05 | |
| *** dansmith has quit IRC | 06:05 | |
| *** dansmith has joined #openstack-sdks | 06:05 | |
| *** gouthamr has joined #openstack-sdks | 06:05 | |
| *** mgoddard has joined #openstack-sdks | 06:05 | |
| *** fyx has joined #openstack-sdks | 06:05 | |
| *** adriant has quit IRC | 06:07 | |
| *** adriant has joined #openstack-sdks | 06:07 | |
| *** slaweq has joined #openstack-sdks | 06:10 | |
| *** ralonsoh has joined #openstack-sdks | 06:36 | |
| *** brinzhang_ has joined #openstack-sdks | 06:56 | |
| *** brinzhang0 has quit IRC | 07:00 | |
| *** slaweq has quit IRC | 07:03 | |
| *** ralonsoh has quit IRC | 07:21 | |
| *** ralonsoh has joined #openstack-sdks | 07:23 | |
| *** tosky has joined #openstack-sdks | 07:44 | |
| *** rpittau|afk is now known as rpittau | 07:45 | |
| *** mnasiadka has joined #openstack-sdks | 07:48 | |
| *** jpena|off is now known as jpena | 07:56 | |
| *** brinzhang0 has joined #openstack-sdks | 08:04 | |
| *** jpich has joined #openstack-sdks | 08:05 | |
| *** brinzhang_ has quit IRC | 08:07 | |
| *** dtantsur|afk is now known as dtantsur | 08:41 | |
| *** jpich has quit IRC | 08:41 | |
| *** jpich has joined #openstack-sdks | 08:42 | |
| *** slaweq has joined #openstack-sdks | 08:47 | |
| *** openstackgerrit has joined #openstack-sdks | 09:23 | |
| openstackgerrit | Wenping Song proposed openstack/osc-lib master: Bump py37 to py38 in tox.ini https://review.opendev.org/756233 | 09:23 |
| openstackgerrit | Wenping Song proposed openstack/openstacksdk master: Add list tags for server https://review.opendev.org/756028 | 09:26 |
| *** jpich has quit IRC | 09:42 | |
| *** jpich has joined #openstack-sdks | 09:43 | |
| openstackgerrit | zhoulinhui proposed openstack/os-api-ref master: Fix hacking min version to 3.0.1 https://review.opendev.org/756237 | 09:54 |
| openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: Remove oslo.utils https://review.opendev.org/739599 | 09:57 |
| openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: Remove oslo.utils https://review.opendev.org/739599 | 10:00 |
| *** yolanda has joined #openstack-sdks | 10:13 | |
| openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: Fix: port attribute name propagate_uplink_status https://review.opendev.org/746716 | 10:16 |
| openstackgerrit | Merged openstack/os-api-ref master: Fix hacking min version to 3.0.1 https://review.opendev.org/756237 | 10:20 |
| *** tkajinam has quit IRC | 10:33 | |
| *** artom has quit IRC | 10:33 | |
| *** lbragstad has quit IRC | 10:33 | |
| *** tkajinam has joined #openstack-sdks | 10:34 | |
| *** artom has joined #openstack-sdks | 10:34 | |
| *** lbragstad has joined #openstack-sdks | 10:34 | |
| *** mgariepy has quit IRC | 11:08 | |
| *** ricolin_ has quit IRC | 11:10 | |
| *** ricolin has joined #openstack-sdks | 11:11 | |
| *** brinzhang0 has quit IRC | 11:12 | |
| *** jpena is now known as jpena|lunch | 11:24 | |
| *** mgariepy has joined #openstack-sdks | 12:22 | |
| *** enriquetaso has joined #openstack-sdks | 12:22 | |
| *** jpena|lunch is now known as jpena | 12:35 | |
| *** Luzi has joined #openstack-sdks | 12:36 | |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Respect default microversion in the server.create_console switch https://review.opendev.org/756286 | 12:57 |
| *** gtema has joined #openstack-sdks | 12:58 | |
| openstackgerrit | Artem Goncharov proposed openstack/python-openstackclient master: Switch console url show operations to SDK https://review.opendev.org/751262 | 12:59 |
| openstackgerrit | Artem Goncharov proposed openstack/python-openstackclient master: Switch openstack console log show operation to use OpenStackSDK https://review.opendev.org/750738 | 13:04 |
| *** Luzi has quit IRC | 13:43 | |
| *** mgariepy has quit IRC | 14:32 | |
| gtema | dtantsur, are you here? Have few minutes? | 14:48 |
| dtantsur | gtema: hi, sure | 14:52 |
| gtema | I would like to hear your opinion on https://review.opendev.org/#/c/756286 please | 14:53 |
| gtema | I tend to agree with stephenfin opinion, but need yours as well | 14:54 |
| stephenfin | gtema: ah, are you Artem? | 14:54 |
| stephenfin | I never put the two together /o\ | 14:55 |
| * dtantsur thinks | 14:55 | |
| gtema | yupp | 14:55 |
| stephenfin | gtk :) | 14:55 |
| *** tremble has joined #openstack-sdks | 14:55 | |
| *** mgariepy has joined #openstack-sdks | 14:56 | |
| tremble | Python openstacksdk question: Without updating clouds.yaml how do I request a specific version of a services proxy - cloud.block_storage currently gets me the v2 proxy and I need the v3 proxy for a feature. | 14:57 |
| dtantsur | tremble: I'm surprised it doesn't happen automagically.. | 14:57 |
| gtema | it happens when the default EP on the block_storage points to v2 | 14:58 |
| gtema | and there is endpoint volumev3 | 14:58 |
| gtema | but version discovery doesn't work - you end on v2 | 14:58 |
| dtantsur | gtema, stephenfin, so, it's a horrible mess, and largely I'm to blame for it. | 14:58 |
| dtantsur | there are currently two approaches to microversions in the SDK and neither work 100% correct | 14:59 |
| dtantsur | there is also pick_microversion that explicitly raises a version higher than the default | 14:59 |
| dtantsur | which you could argue is also incorrect | 14:59 |
| gtema | tremble: for me I use volume_service_type: volumev3 in clouds. But know you want it other way around | 14:59 |
| gtema | dtantsur - that's why I ping you ;-) | 15:00 |
| dtantsur | gtema: remind me please, the default microversion comes from clouds.yaml? | 15:00 |
| gtema | no explicitly, but can be set also there | 15:00 |
| dtantsur | so it's None is most cases except for an explicit OSC argument? | 15:00 |
| gtema | in this case it is argument in OSC | 15:01 |
| dtantsur | yeah, okay, OSC and clouds.yaml, but not implicitly? | 15:01 |
| tremble | gtema, Yeah I want this for an Ansible module, and it seems kinda dumb to tell people to update their clouds.yaml if they want access to a feature | 15:01 |
| gtema | yes, it is none by default, but if you do --os-compute-api-version 2.5 - it is 2.5 | 15:01 |
| dtantsur | commented | 15:01 |
| gtema | thanks dtantsur, I will change the default | 15:02 |
| dtantsur | gtema: and please fix pick_microversion | 15:02 |
| dtantsur | I hope ironic won't blow up afterwards :) | 15:02 |
| gtema | would we want an old functionality ever? to simply check whether server side supports the mv? | 15:02 |
| dtantsur | gtema: I don't think so.. | 15:02 |
| dtantsur | I mean, it's not useless in theory, but in this context we should respect the default | 15:03 |
| gtema | I actually was meaning, whether respect_default_mv should stay as option but be True by default or whether drop it completely | 15:04 |
| gtema | but in the later case we can not simply check whether server supports given MV | 15:04 |
| dtantsur | *shrug* I'd drop it until somebody needs it | 15:05 |
| gtema | okay. Thks | 15:05 |
| frickler | gtema: the devstack openstacksdk job is still broken on stable/train it seems, see https://zuul.opendev.org/t/openstack/build/82a83f08f9064f18bcb9e12e4b0b9897 , I've asked neutron folk for help now | 15:12 |
| gtema | frickler: ack | 15:13 |
| gtema | dtantsur: I think pick_mv is working correctly, at least wrt "get a new mv if it is higher then default" | 15:15 |
| *** hyang has joined #openstack-sdks | 15:16 | |
| gtema | if somebody is using it - blame himself and take care of consequences | 15:16 |
| dtantsur | gtema: if we change the behavior, we need to change it in two places | 15:18 |
| dtantsur | otherwise our services will work inconsistently wrt defaults | 15:18 |
| dtantsur | (i.e. compute will respect it, baremetal won't) | 15:18 |
| gtema | in this case I would rather move other services to use "supports" | 15:19 |
| gtema | otherwise the function is doing not what it is describes to be doing | 15:19 |
| gtema | there's only openstack/baremetal/v1/node.py using pick (and one mistakenly openstack/cloud/_compute.py) | 15:20 |
| dtantsur | gtema: you can use pick as a handier wrapper around supports that also handles errors, I guess | 15:27 |
| gtema | not sure. Currently looking to the cloud/_compute.py | 15:27 |
| gtema | it simply wants a MV not even checking it is supported or not | 15:28 |
| gtema | at least verification must be done | 15:28 |
| gtema | so pick is used to enforce using mv other then selected (default) | 15:29 |
| gtema | guess in ironic as well | 15:29 |
| dtantsur | so, the question is: do we want to allow ignoring the default? | 15:31 |
| dtantsur | we used to, with your patch we'll no longer | 15:31 |
| dtantsur | whatever the answer is, we need it to be consistent across services | 15:31 |
| dtantsur | if we think nova shouldn't do it, ironic shouldn't either | 15:32 |
| gtema | I would say in cloud/_copmute I would change it to check whether feature is supported or not | 15:32 |
| gtema | if not supported - ignore setting tags | 15:32 |
| gtema | (I guess this is mostly the ansible usage) | 15:32 |
| gtema | with my patch we will actually start respecting properly default, while currently we don't | 15:35 |
| gtema | I will try to switch all the usages to supports raising error if it feature is not supported | 15:36 |
| gtema | s/it/this/ | 15:36 |
| dtantsur | ++ | 15:40 |
| *** rpittau is now known as rpittau|afk | 15:48 | |
| *** tremble has quit IRC | 15:55 | |
| *** hyang has quit IRC | 16:08 | |
| *** dtantsur is now known as dtantsur|afk | 16:18 | |
| openstackgerrit | Merged openstack/cliff master: Bump py37 to py38 in tox.ini https://review.opendev.org/756210 | 16:21 |
| *** jpich has quit IRC | 16:23 | |
| *** gtema has quit IRC | 16:37 | |
| *** jpena is now known as jpena|off | 17:16 | |
| *** tremble has joined #openstack-sdks | 17:31 | |
| *** hyang has joined #openstack-sdks | 17:35 | |
| *** hyang has quit IRC | 17:44 | |
| *** ralonsoh has quit IRC | 17:52 | |
| *** larsks has quit IRC | 18:00 | |
| *** nikparasyr has left #openstack-sdks | 19:01 | |
| *** gtema has joined #openstack-sdks | 19:06 | |
| *** yolanda has quit IRC | 19:20 | |
| *** yolanda has joined #openstack-sdks | 19:20 | |
| *** tosky has quit IRC | 19:52 | |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Respect default microversion in the microversion negotiation https://review.opendev.org/756286 | 20:07 |
| *** gtema has quit IRC | 20:07 | |
| *** larsks has joined #openstack-sdks | 20:53 | |
| *** slaweq has quit IRC | 21:07 | |
| *** slaweq has joined #openstack-sdks | 21:12 | |
| *** slaweq has quit IRC | 21:44 | |
| *** gtema has joined #openstack-sdks | 22:08 | |
| *** gtema has quit IRC | 22:12 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!