Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-10-08
12:12:57 gtema ok
12:19:42 openstackgerrit yang yawei proposed openstack/openstacksdk master: add migration list for sdk https://review.opendev.org/756692
12:42:30 openstackgerrit yang yawei proposed openstack/openstacksdk master: add defaults quotas for sdk https://review.opendev.org/756698
15:53:08 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add support for 'keypairs list --user' parameter https://review.opendev.org/755073
15:53:09 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add support for 'keypairs list --project' parameter https://review.opendev.org/755074
15:54:54 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add support for 'keypairs list --project' parameter https://review.opendev.org/755074
16:10:07 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Switch console url show operations to SDK https://review.opendev.org/751262
16:43:42 dtantsur stephenfin: I'll check the microversions patch, but likely only tomorrow, sorry
#openstack-sdks - 2020-10-09
00:51:29 openstackgerrit yang yawei proposed openstack/openstacksdk master: add defaults quotas for sdk https://review.opendev.org/756698
04:52:20 openstackgerrit Merged openstack/python-openstackclient master: Fix: port attribute name propagate_uplink_status https://review.opendev.org/746716
09:30:17 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add support for 'keypairs list --project' parameter https://review.opendev.org/755074
09:30:18 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add 'User' column to 'keypair list' output https://review.opendev.org/757040
10:43:59 openstackgerrit Merged openstack/python-openstackclient master: Add 'openstack server create --use-config-drive' https://review.opendev.org/738182
11:29:59 openstackgerrit Jesper Schmitz Mouridsen proposed openstack/ansible-collections-openstack master: Refactor loadbalancer module * enable check_mode * enable allowed_cidrs on listener if octavia version is >= 2.12 * Only send flavor_id if it is not None https://review.opendev.org/757081
11:32:27 openstackgerrit Jesper Schmitz Mouridsen proposed openstack/ansible-collections-openstack master: Refactor loadbalancer module https://review.opendev.org/757081
12:10:47 openstackgerrit Pavlo Shchelokovskyy proposed openstack/openstacksdk master: Expand user path when loading SSL-related files https://review.opendev.org/757088
14:52:25 dtantsur stephenfin, gtema, I think the microversion patch is doing a slightly wrong thing when default is None. It's not enough to ensure the server supports a microversion, you actually need to pass it.
14:53:03 gtema are you sure? Because in all my tests requests are really going with either the default one or the highest
14:53:53 dtantsur gtema: I'm not sure how it would work, default=None means the lowest, not the highest
14:54:05 dtantsur and you remove passing microversion argument to the calls
14:54:27 gtema in the nova area whenever I am not specifically choosing mv - all requests are going with highest one
14:54:39 dtantsur Oo
14:54:55 dtantsur that's.. not how it was supposed to work..
14:56:15 dtantsur maybe there is more going on with nova
14:56:57 dtantsur gtema: did you test with OSC only or also with pure SDK?
14:57:02 gtema just checked one more time with dummy script - it is using highest supported by the server in compute area
14:57:06 gtema both
14:57:14 gtema but again - only nova part
14:57:22 gtema I have no ironic to verify
14:57:38 dtantsur I wish the ironic job was not broken, we would just know
14:58:54 gtema stephenfin also checked some other pathes of mine also relying to this behaviour and seems to work properly
14:59:22 gtema https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/resource.py#L1191
14:59:40 gtema at least for the basic stuff we should be sending max
14:59:51 dtantsur gtema: yeah, this works, but only for functions that use get_microversion_for_*
15:00:02 dtantsur the ironic ones need to be changed to do that then
15:00:21 gtema okay. That is true
15:00:28 dtantsur or keep using pick_microversion which now seems to do the right thing after your change
15:00:55 gtema :D
15:01:37 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Disable dstat in devstack jobs https://review.opendev.org/757123
15:01:39 dtantsur maybe this will be enough to get back the job ^^
15:03:03 gtema so I should revert changes in ironic hoping pick works really correctly. Right?
15:04:21 dtantsur gtema: yes please. and also see my comment re error message.
15:04:30 gtema yupp
15:10:47 gtema do you know if train branch currently running also on focal?
15:11:08 dtantsur shouldn't
15:11:15 dtantsur focal is for victoria+
15:11:32 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Respect default microversion in the microversion negotiation https://review.opendev.org/756286
15:11:57 gtema yeah, there are some weird problems on train sdk-devstack jobs
15:12:17 gtema they seem to be cut on timeout since very recently and I was not able to figure out why
15:28:30 frickler gtema: dtantsur: I was at first blaming neutron but maybe it is something else after all, see also https://bugs.launchpad.net/neutron/+bug/1898748
15:28:30 openstack Launchpad bug 1898748 in neutron "[stable/train] Creation of the QoS policy takes ages" [Critical,New]
15:29:22 gtema yeah, I was also investigating that multiple logs show it stops on different tests
15:29:55 gtema mostly every time in neutron qos area, but I also don't see real neutron connect
16:46:18 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Compute: Add tag support for server add volume https://review.opendev.org/580579
19:37:49 openstackgerrit Merged openstack/python-openstackclient master: Remove usage of six https://review.opendev.org/756201
#openstack-sdks - 2020-10-10
14:14:24 tosky dtantsur|afk: no way reverting that change, no
20:54:42 openstackgerrit Marta Laís de Macedo Dantas proposed openstack/openstacksdk master: [WIP] Volume Target support for Ironic on OpenStack SDK https://review.opendev.org/753092
20:54:42 openstackgerrit Marta Laís de Macedo Dantas proposed openstack/openstacksdk master: [WIP] Volume Target support for Ironic on OpenStack SDK https://review.opendev.org/753092
23:14:14 openstackgerrit Luigi Toscano proposed openstack/openstacksdk master: DNM test a devstack-tox-functional change https://review.opendev.org/757304
#openstack-sdks - 2020-10-12
09:21:31 openstackgerrit Wenping Song proposed openstack/python-openstackclient master: Add create tags for server https://review.opendev.org/756664
09:51:22 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Support waiting for bare metal power states https://review.opendev.org/757523
10:05:14 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Support waiting for bare metal power states https://review.opendev.org/757523
10:38:00 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Compute: Add tag support for server add port https://review.opendev.org/580977
10:56:18 tremble Wow, Zuul injects pep8 comments directly into the review at the right line
11:26:06 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Compute: Add 'keypair create --type' parameter https://review.opendev.org/577768
12:40:44 tremble dtantsur, Is there a way to specify a minimum SDK version for an Ansible OpenStackModule ?
12:41:18 dtantsur tremble: I think some of our modules do that
12:41:58 tremble There's an option for the normal version of openstack_cloud_from_module, but I don't see one for OpenStackModule
12:42:11 dtantsur I only know https://opendev.org/openstack/ansible-collections-openstack/src/branch/master/plugins/modules/federation_mapping_info.py#L22
12:42:16 dtantsur maybe sshnaidm knows more?
12:42:41 tremble https://opendev.org/openstack/ansible-collections-openstack/src/branch/master/plugins/modules/federation_mapping_info.py#L78
12:57:56 sshnaidm tremble, hi, what do you need it for? To limit features by sdk version?
12:58:30 tremble sshnaidm, The module as a whole needs an SDK version, rather than just a specific feature.
12:59:03 sshnaidm tremble, you mean just version number?
12:59:21 tremble The module as a whole needs an SDK version, rather than just a specific VERSION
12:59:25 sshnaidm what is the use case?
13:00:23 tremble SDK 0.50 has very poor support for Cinder volume types (no update and multiple missing attributes)
13:01:04 tremble I'm putting together the relevant changes, but trying to figure out exactly which pieces might work pre-0.51 is going to be very messy
13:01:47 tremble An alternate use case would be federation_mapping where there was *no* support for the object type at all.
13:02:50 tremble sshnaidm, That make more sense?
13:03:58 openstackgerrit Merged openstack/openstacksdk master: Add "description" to cinder volume types. https://review.opendev.org/756649
13:07:15 sshnaidm tremble, yeah, sorry, in the middle of meetings
13:07:50 tremble ok
13:08:51 tremble sshnaidm, I think I just need to add support for saying the whole module needs SDK version X, which I can do.
13:09:24 sshnaidm tremble, trying to recall if we did something like that..
13:10:03 tremble openstack.openstack_cloud_from_module supports it, OpenStackModule.openstack_cloud_from_module does not
13:10:37 tremble and favours specific parameters needing a version of the SDK (which is a nice feature)
13:11:37 sshnaidm tremble, yeah, we can mention every parameter with min_ver or max_ver
13:12:11 sshnaidm and check_versioned https://opendev.org/openstack/ansible-collections-openstack/src/branch/master/plugins/module_utils/openstack.py#L303-L305
13:13:00 openstackgerrit Athlan-Guyot sofer proposed openstack/ansible-collections-openstack master: WIP: Add application credentials module. https://review.opendev.org/757556
13:13:35 openstackgerrit Athlan-Guyot sofer proposed openstack/ansible-collections-openstack master: WIP: Add application credentials module. https://review.opendev.org/757556
13:15:23 sshnaidm tremble, yeah, for whole module we need to tweak openstack_cloud_from_module function
13:15:51 tremble Ok, I'll finish prepping my change and send it your way.
13:15:59 sshnaidm tremble, thanks
13:23:26 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: OpenStackModule: Support defining a minimum version of the SDK https://review.opendev.org/757561
13:23:27 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: volume_type https://review.opendev.org/757562
13:31:06 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: volume_type https://review.opendev.org/757562
13:32:54 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: volume_type https://review.opendev.org/757562
13:55:30 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.opendev.org/569386
14:02:54 tremble sshnaidm, https://review.opendev.org/#/c/757561/

Earlier   Later