Monday, 2020-10-12

*** larsks has quit IRC03:38
*** larsks has joined #openstack-sdks03:38
*** evrardjp has quit IRC04:33
*** evrardjp has joined #openstack-sdks04:33
*** ricolin_ has joined #openstack-sdks04:35
*** tremble has quit IRC05:35
*** dmsimard has quit IRC05:55
*** slaweq has joined #openstack-sdks05:55
*** dmsimard has joined #openstack-sdks05:59
*** slaweq has quit IRC06:08
*** gtema has joined #openstack-sdks06:10
*** gtema has quit IRC06:27
*** gtema has joined #openstack-sdks06:29
*** gtema has quit IRC06:36
*** tosky has joined #openstack-sdks07:03
*** slaweq has joined #openstack-sdks07:13
*** tremble has joined #openstack-sdks07:15
*** gtema has joined #openstack-sdks07:29
*** brinzhang has joined #openstack-sdks07:32
*** rpittau|afk is now known as rpittau07:35
*** slaweq_ has joined #openstack-sdks07:47
*** slaweq has quit IRC07:48
*** sshnaidm|off is now known as sshnaidm07:58
*** jpich has joined #openstack-sdks08:01
*** gtema has quit IRC08:14
*** brinzhang_ has joined #openstack-sdks08:33
*** brinzhang has quit IRC08:35
*** holser has joined #openstack-sdks08:46
*** dtantsur|afk is now known as dtantsur09:03
*** ricolin_ has quit IRC09:04
*** ricolin has joined #openstack-sdks09:05
*** openstackgerrit has joined #openstack-sdks09:21
openstackgerritWenping Song proposed openstack/python-openstackclient master: Add create tags for server  https://review.opendev.org/75666409:21
*** nikparasyr has joined #openstack-sdks09:22
*** ralonsoh has joined #openstack-sdks09:39
openstackgerritDmitry Tantsur proposed openstack/openstacksdk master: Support waiting for bare metal power states  https://review.opendev.org/75752309:51
*** iurygregory has quit IRC09:57
*** iurygregory has joined #openstack-sdks09:58
openstackgerritDmitry Tantsur proposed openstack/openstacksdk master: Support waiting for bare metal power states  https://review.opendev.org/75752310:05
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: Compute: Add tag support for server add port  https://review.opendev.org/58097710:38
*** Luzi has joined #openstack-sdks10:52
trembleWow, Zuul injects pep8 comments directly into the review at the right line10:56
*** holser has quit IRC11:10
*** gtema has joined #openstack-sdks11:13
*** gtema has quit IRC11:18
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: Compute: Add 'keypair create --type' parameter  https://review.opendev.org/57776811:26
*** holser has joined #openstack-sdks11:33
*** brinzhang_ has quit IRC11:37
*** brinzhang_ has joined #openstack-sdks11:37
*** gtema has joined #openstack-sdks12:29
*** gtema has quit IRC12:33
*** lbragstad has joined #openstack-sdks12:39
*** lbragstad has quit IRC12:40
trembledtantsur, Is there a way to specify a minimum SDK version for an Ansible OpenStackModule ?12:40
dtantsurtremble: I think some of our modules do that12:41
trembleThere's an option for the normal version of openstack_cloud_from_module, but I don't see one for OpenStackModule12:41
dtantsurI only know https://opendev.org/openstack/ansible-collections-openstack/src/branch/master/plugins/modules/federation_mapping_info.py#L2212:42
dtantsurmaybe sshnaidm knows more?12:42
tremblehttps://opendev.org/openstack/ansible-collections-openstack/src/branch/master/plugins/modules/federation_mapping_info.py#L7812:42
*** lbragstad has joined #openstack-sdks12:46
*** holser has quit IRC12:54
*** holser has joined #openstack-sdks12:56
sshnaidmtremble, hi, what do you need it for? To limit features by sdk version?12:57
tremblesshnaidm, The module as a whole needs an SDK version, rather than just a specific feature.12:58
sshnaidmtremble, you mean just version number?12:59
trembleThe module as a whole needs an SDK version, rather than just a specific VERSION12:59
sshnaidmwhat is the use case?12:59
trembleSDK 0.50 has very poor support for Cinder volume types (no update and multiple missing attributes)13:00
trembleI'm putting together the relevant changes, but trying to figure out exactly which pieces might work pre-0.51 is going to be very messy13:01
trembleAn alternate use case would be federation_mapping where there was *no* support for the object type at all.13:01
tremblesshnaidm, That make more sense?13:02
openstackgerritMerged openstack/openstacksdk master: Add "description" to cinder volume types.  https://review.opendev.org/75664913:03
sshnaidmtremble, yeah, sorry, in the middle of meetings13:07
trembleok13:07
tremblesshnaidm, I think I just need to add support for saying the whole module needs SDK version X, which I can do.13:08
sshnaidmtremble, trying to recall if we did something like that..13:09
trembleopenstack.openstack_cloud_from_module supports it, OpenStackModule.openstack_cloud_from_module does not13:10
trembleand favours specific parameters needing a version of the SDK (which is a nice feature)13:10
sshnaidmtremble, yeah, we can mention every parameter with min_ver or max_ver13:11
sshnaidmand check_versioned https://opendev.org/openstack/ansible-collections-openstack/src/branch/master/plugins/module_utils/openstack.py#L303-L30513:12
openstackgerritAthlan-Guyot sofer proposed openstack/ansible-collections-openstack master: WIP: Add application credentials module.  https://review.opendev.org/75755613:13
openstackgerritAthlan-Guyot sofer proposed openstack/ansible-collections-openstack master: WIP: Add application credentials module.  https://review.opendev.org/75755613:13
sshnaidmtremble, yeah, for whole module we need to tweak openstack_cloud_from_module function13:15
trembleOk, I'll finish prepping my change and send it your way.13:15
sshnaidmtremble, thanks13:15
openstackgerritMark Chappell proposed openstack/ansible-collections-openstack master: OpenStackModule: Support defining a minimum version of the SDK  https://review.opendev.org/75756113:23
openstackgerritMark Chappell proposed openstack/ansible-collections-openstack master: New Module: volume_type  https://review.opendev.org/75756213:23
*** Luzi has quit IRC13:26
*** iurygregory has quit IRC13:27
*** iurygregory has joined #openstack-sdks13:28
openstackgerritMark Chappell proposed openstack/ansible-collections-openstack master: New Module: volume_type  https://review.opendev.org/75756213:31
openstackgerritMark Chappell proposed openstack/ansible-collections-openstack master: New Module: volume_type  https://review.opendev.org/75756213:32
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: Compute: Add tags support for server  https://review.opendev.org/56938613:55
tremblesshnaidm, https://review.opendev.org/#/c/757561/14:02
*** slaweq_ is now known as slaweq14:09
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: trivial: Rework 'CreateServer' function  https://review.opendev.org/75758114:13
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: Compute: Add tag support for server add network  https://review.opendev.org/58157314:19
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: Compute: Add tag support for server add fixed ip  https://review.opendev.org/58132514:25
*** dave-mccowan has joined #openstack-sdks14:27
*** gtema has joined #openstack-sdks14:29
*** gtema has quit IRC14:34
sshnaidmtremble, a few nits14:35
*** tremble has quit IRC14:37
*** rpittau is now known as rpittau|afk16:01
openstackgerritMerged openstack/python-openstackclient master: Add server migration list CLI  https://review.opendev.org/67530416:01
*** ralonsoh has quit IRC16:05
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: Add an error message when server bind floating IP  https://review.opendev.org/66648316:09
*** jpich has quit IRC16:11
stephenfindtantsur: do you have any idea why we run tempest against OSC patches?16:19
stephenfinSeeing as Tempest maintains its own client and doesn't use OSC, that seems rather...useless16:20
*** tosky has quit IRC16:21
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: zuul: Stop testing against Tempest  https://review.opendev.org/75761916:26
openstackgerritStephen Finucane proposed openstack/python-openstackclient master: Remove oslo.utils  https://review.opendev.org/73959916:30
dtantsurstephenfin: no idea tbh16:33
*** dtantsur is now known as dtantsur|afk16:35
*** nikparasyr has left #openstack-sdks16:39
openstackgerritJesper Schmitz Mouridsen proposed openstack/ansible-collections-openstack master: Refactor loadbalancer module  https://review.opendev.org/75708117:14
*** cmurphy has left #openstack-sdks17:41
*** mordred has quit IRC19:16
*** mordred has joined #openstack-sdks19:24
*** tosky has joined #openstack-sdks20:26
*** hyang has joined #openstack-sdks21:00
*** slaweq has quit IRC21:13
*** slaweq has joined #openstack-sdks22:08
*** slaweq has quit IRC22:22
*** hyang has quit IRC23:01
*** hyang has joined #openstack-sdks23:05
*** tosky has quit IRC23:15
*** larsks has quit IRC23:59

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!