Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-10-05
11:37:37 openstackgerrit Merged openstack/openstacksdk master: Add 'project_id' to Snapshot query parameters https://review.opendev.org/755445
#openstack-sdks - 2020-10-06
00:30:55 openstackgerrit Merged openstack/python-openstackclient master: Update master for stable/victoria https://review.opendev.org/755889
01:36:03 openstackgerrit Merged openstack/python-openstackclient master: Add Python3 wallaby unit tests https://review.opendev.org/755890
03:05:39 openstackgerrit Sam Morrison proposed openstack/python-openstackclient master: Restore behavior of image create with same name. https://review.opendev.org/756187
03:41:38 openstackgerrit Wenping Song proposed openstack/cliff master: Remove six https://review.opendev.org/756188
09:23:23 openstackgerrit Wenping Song proposed openstack/osc-lib master: Bump py37 to py38 in tox.ini https://review.opendev.org/756233
09:26:49 openstackgerrit Wenping Song proposed openstack/openstacksdk master: Add list tags for server https://review.opendev.org/756028
09:54:26 openstackgerrit zhoulinhui proposed openstack/os-api-ref master: Fix hacking min version to 3.0.1 https://review.opendev.org/756237
09:57:07 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Remove oslo.utils https://review.opendev.org/739599
10:00:32 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Remove oslo.utils https://review.opendev.org/739599
10:16:58 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Fix: port attribute name propagate_uplink_status https://review.opendev.org/746716
10:20:20 openstackgerrit Merged openstack/os-api-ref master: Fix hacking min version to 3.0.1 https://review.opendev.org/756237
12:57:42 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Respect default microversion in the server.create_console switch https://review.opendev.org/756286
12:59:09 openstackgerrit Artem Goncharov proposed openstack/python-openstackclient master: Switch console url show operations to SDK https://review.opendev.org/751262
13:04:17 openstackgerrit Artem Goncharov proposed openstack/python-openstackclient master: Switch openstack console log show operation to use OpenStackSDK https://review.opendev.org/750738
14:48:46 gtema dtantsur, are you here? Have few minutes?
14:52:30 dtantsur gtema: hi, sure
14:53:51 gtema I would like to hear your opinion on https://review.opendev.org/#/c/756286 please
14:54:31 gtema I tend to agree with stephenfin opinion, but need yours as well
14:54:58 stephenfin gtema: ah, are you Artem?
14:55:03 stephenfin I never put the two together /o\
14:55:06 gtema yupp
14:55:32 stephenfin gtk :)
14:57:20 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:49 dtantsur tremble: I'm surprised it doesn't happen automagically..
14:58:19 gtema it happens when the default EP on the block_storage points to v2
14:58:27 gtema and there is endpoint volumev3
14:58:43 gtema but version discovery doesn't work - you end on v2
14:58:51 dtantsur gtema, stephenfin, so, it's a horrible mess, and largely I'm to blame for it.
14:59:05 dtantsur there are currently two approaches to microversions in the SDK and neither work 100% correct
14:59:25 dtantsur there is also pick_microversion that explicitly raises a version higher than the default
14:59:30 dtantsur which you could argue is also incorrect
14:59:49 gtema tremble: for me I use volume_service_type: volumev3 in clouds. But know you want it other way around
15:00:18 gtema dtantsur - that's why I ping you ;-)
15:00:41 dtantsur gtema: remind me please, the default microversion comes from clouds.yaml?
15:00:57 gtema no explicitly, but can be set also there
15:00:58 dtantsur so it's None is most cases except for an explicit OSC argument?
15:01:04 gtema in this case it is argument in OSC
15:01:17 dtantsur yeah, okay, OSC and clouds.yaml, but not implicitly?
15:01:20 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:27 gtema yes, it is none by default, but if you do --os-compute-api-version 2.5 - it is 2.5
15:01:48 dtantsur commented
15:02:18 gtema thanks dtantsur, I will change the default
15:02:30 dtantsur gtema: and please fix pick_microversion
15:02:36 dtantsur I hope ironic won't blow up afterwards :)
15:02:46 gtema would we want an old functionality ever? to simply check whether server side supports the mv?
15:02:59 dtantsur gtema: I don't think so..
15:03:25 dtantsur I mean, it's not useless in theory, but in this context we should respect the default
15:04:09 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:52 gtema but in the later case we can not simply check whether server supports given MV
15:05:24 dtantsur *shrug* I'd drop it until somebody needs it
15:05:32 gtema okay. Thks
15:12:38 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:13:21 gtema frickler: ack
15:15:49 gtema dtantsur: I think pick_mv is working correctly, at least wrt "get a new mv if it is higher then default"
15:16:39 gtema if somebody is using it - blame himself and take care of consequences
15:18:15 dtantsur gtema: if we change the behavior, we need to change it in two places
15:18:29 dtantsur otherwise our services will work inconsistently wrt defaults
15:18:37 dtantsur (i.e. compute will respect it, baremetal won't)
15:19:13 gtema in this case I would rather move other services to use "supports"
15:19:34 gtema otherwise the function is doing not what it is describes to be doing
15:20:48 gtema there's only openstack/baremetal/v1/node.py using pick (and one mistakenly openstack/cloud/_compute.py)
15:27:13 dtantsur gtema: you can use pick as a handier wrapper around supports that also handles errors, I guess
15:27:55 gtema not sure. Currently looking to the cloud/_compute.py
15:28:16 gtema it simply wants a MV not even checking it is supported or not
15:28:27 gtema at least verification must be done
15:29:10 gtema so pick is used to enforce using mv other then selected (default)
15:29:30 gtema guess in ironic as well
15:31:42 dtantsur so, the question is: do we want to allow ignoring the default?
15:31:49 dtantsur we used to, with your patch we'll no longer
15:31:59 dtantsur whatever the answer is, we need it to be consistent across services
15:32:11 dtantsur if we think nova shouldn't do it, ironic shouldn't either
15:32:17 gtema I would say in cloud/_copmute I would change it to check whether feature is supported or not
15:32:28 gtema if not supported - ignore setting tags
15:32:50 gtema (I guess this is mostly the ansible usage)
15:35:35 gtema with my patch we will actually start respecting properly default, while currently we don't
15:36:10 gtema I will try to switch all the usages to supports raising error if it feature is not supported
15:36:24 gtema s/it/this/
15:40:37 dtantsur ++
16:21:27 openstackgerrit Merged openstack/cliff master: Bump py37 to py38 in tox.ini https://review.opendev.org/756210
20:07:00 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Respect default microversion in the microversion negotiation https://review.opendev.org/756286
#openstack-sdks - 2020-10-07
02:15:42 openstackgerrit Wenping Song proposed openstack/python-openstackclient master: Remove usage of six https://review.opendev.org/756201
06:26:15 openstackgerrit Arthur Dayne proposed openstack/openstacksdk master: Add 'attachment_update' to openstacksdk https://review.opendev.org/756436
06:31:07 openstackgerrit Xuan Yandong proposed openstack/cliff master: Remove Six https://review.opendev.org/754683
06:35:43 openstackgerrit Qiu Fossen proposed openstack/openstacksdk master: add migrate volume completion https://review.opendev.org/756437
06:37:03 openstackgerrit Arthur Dayne proposed openstack/openstacksdk master: Add 'attachment_update' to openstacksdk https://review.opendev.org/756436
06:38:27 openstackgerrit Xuan Yandong proposed openstack/cliff master: Remove six https://review.opendev.org/754683
06:39:57 openstackgerrit Arthur Dayne proposed openstack/openstacksdk master: Add 'attachment_update' to openstacksdk https://review.opendev.org/756436
07:25:29 openstackgerrit Wenping Song proposed openstack/osc-lib master: Check item type for str https://review.opendev.org/756445
07:46:56 openstackgerrit xuyuanhao proposed openstack/openstacksdk master: add attachment interface for sdk https://review.opendev.org/756447
13:33:35 openstackgerrit Rafael Weingärtner proposed openstack/openstacksdk master: Deprecate 'remote_ip_prefix' parameter in metering label rules https://review.opendev.org/746578
13:33:35 openstackgerrit Rafael Weing채rtner proposed openstack/openstacksdk master: Deprecate 'remote_ip_prefix' parameter in metering label rules https://review.opendev.org/746578
13:34:47 openstackgerrit Rafael Weingärtner proposed openstack/openstacksdk master: Add source_ip_prefix and destination_ip_prefix to metering label rules https://review.opendev.org/746586
13:34:47 openstackgerrit Rafael Weing채rtner proposed openstack/openstacksdk master: Add source_ip_prefix and destination_ip_prefix to metering label rules https://review.opendev.org/746586
15:54:19 openstackgerrit Rafael Weingärtner proposed openstack/openstacksdk master: Deprecate 'remote_ip_prefix' parameter in metering label rules https://review.opendev.org/746578
15:54:19 openstackgerrit Rafael Weing채rtner proposed openstack/openstacksdk master: Deprecate 'remote_ip_prefix' parameter in metering label rules https://review.opendev.org/746578
15:57:36 openstackgerrit Rafael Weingärtner proposed openstack/openstacksdk master: Add source_ip_prefix and destination_ip_prefix to metering label rules https://review.opendev.org/746586
15:57:36 openstackgerrit Rafael Weing채rtner proposed openstack/openstacksdk master: Add source_ip_prefix and destination_ip_prefix to metering label rules https://review.opendev.org/746586
17:00:27 stephenfin dtantsur|afk: Could you cast an eye over https://review.opendev.org/#/c/756286/ again at some point this week, please. I really don't have enough context on that yet
#openstack-sdks - 2020-10-08
02:45:08 openstackgerrit Jorhson Deng proposed openstack/openstacksdk master: Add 'unreserve_volume' to openstacksdk https://review.opendev.org/756622

Earlier   Later