Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-10-09
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 openstack Launchpad bug 1898748 in neutron "[stable/train] Creation of the QoS policy takes ages" [Critical,New]
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: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/
14:13:27 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: trivial: Rework 'CreateServer' function https://review.opendev.org/757581
14:19:50 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Compute: Add tag support for server add network https://review.opendev.org/581573
14:25:20 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Compute: Add tag support for server add fixed ip https://review.opendev.org/581325
14:35:05 sshnaidm tremble, a few nits
16:01:53 openstackgerrit Merged openstack/python-openstackclient master: Add server migration list CLI https://review.opendev.org/675304
16:09:34 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add an error message when server bind floating IP https://review.opendev.org/666483
16:19:42 stephenfin dtantsur: do you have any idea why we run tempest against OSC patches?
16:20:16 stephenfin Seeing as Tempest maintains its own client and doesn't use OSC, that seems rather...useless
16:26:12 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: zuul: Stop testing against Tempest https://review.opendev.org/757619
16:30:10 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Remove oslo.utils https://review.opendev.org/739599
16:33:38 dtantsur stephenfin: no idea tbh
17:14:27 openstackgerrit Jesper Schmitz Mouridsen proposed openstack/ansible-collections-openstack master: Refactor loadbalancer module https://review.opendev.org/757081
#openstack-sdks - 2020-10-13
00:49:13 openstackgerrit Wenping Song proposed openstack/cliff master: Remove Babel from lower-constraints.txt https://review.opendev.org/757485
01:36:47 openstackgerrit Merged openstack/python-openstackclient master: Compute: Add tag support for server add port https://review.opendev.org/580977
01:40:18 openstackgerrit wu.shiming proposed openstack/openstacksdk master: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/757664
03:42:52 openstackgerrit Merged openstack/python-openstackclient master: Compute: Add tag support for server add network https://review.opendev.org/581573
04:59:43 openstackgerrit Merged openstack/python-openstackclient master: Add an error message when server bind floating IP https://review.opendev.org/666483
07:22:38 openstackgerrit Merged openstack/python-openstackclient master: Remove oslo.utils https://review.opendev.org/739599
09:10:12 stephenfin dtantsur: That makes two of us then. Does removing it make sense so? I've proposed just that here https://review.opendev.org/757619?
09:12:08 dtantsur +2
09:12:28 gtema +A, though I am on PTO
09:12:40 stephenfin \o/
09:12:49 stephenfin gtema: now go back to PTO'ing :P

Earlier   Later