Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-10-12
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
09:13:28 gtema yeah, heating pipe exploded, heater itself got also electronic problems, so much fun ;-(
09:13:46 gtema and it is cold outside
09:17:18 stephenfin gtema: oof, sorry to hear that. Been there, done that :(
09:38:40 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: compute: Add --password option for openstack server create https://review.opendev.org/490718
10:06:21 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/openstacksdk master: Disable dstat on ironic job https://review.opendev.org/757805
10:23:14 openstackgerrit Merged openstack/python-openstackclient master: zuul: Stop testing against Tempest https://review.opendev.org/757619
10:34:07 openstackgerrit Merged openstack/cliff master: Remove Babel from lower-constraints.txt https://review.opendev.org/757485
14:00:48 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add changelog for 1.2.0 https://review.opendev.org/757867
14:14:39 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add 'server migration abort' command https://review.opendev.org/716960
14:14:43 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add 'server migration force complete' command https://review.opendev.org/721594
14:14:47 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Cleanup of 'server migration list' command https://review.opendev.org/757873
14:14:47 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Add 'openstack server migration list --type' option https://review.opendev.org/757874
14:27:17 stephenfin smcginnis: Are you stable core on OSC? Fancy shoving this in, if so? https://review.opendev.org/#/c/755896/
14:28:34 stephenfin smcginnis: On that note, any idea what one need to do to get stable core on OSC? I imagine we're rather light on the ground there
14:36:20 openstackgerrit Stephen Finucane proposed openstack/python-openstackclient master: Validate 'server group create --policy' option https://review.opendev.org/757879
14:53:11 openstackgerrit Merged openstack/openstacksdk master: Add support for updating cinder (v3) volume types https://review.opendev.org/756650
14:53:12 openstackgerrit Merged openstack/python-openstackclient master: Compute: Add 'keypair create --type' parameter https://review.opendev.org/577768
14:54:31 smcginnis stephenfin: I'm at least stable core.
14:54:48 smcginnis stephenfin: Yep, doesn't look like I am on master.
14:57:21 smcginnis stephenfin: Only core I still see around for python-openstack-core is maybe amotoki. It does also include openstacksdk-core, which brings in quite a few more. But most of those names are also no longer around and should probably be cleaned up.
14:57:43 stephenfin might be something to bring up on openstack-discuss
14:57:52 stephenfin see if we can get some people from various projects involved
14:58:26 smcginnis Looks like that might have been the idea with openstacksdk-core based on some of the names there. Just really, really out of date at this point.
15:01:50 tkajinam smcginnis, could you also check this one as well ? https://review.opendev.org/#/c/755900/ This fixes the other issue caused by the same change in nova...
15:02:02 smcginnis Looking...
15:02:23 tkajinam would be nice if we can have both of them at the same time
15:04:18 tkajinam smcginnis, thanks
15:19:34 openstackgerrit Mark Chappell proposed openstack/openstacksdk master: Add support for volume type encyption objects https://review.opendev.org/756655
15:19:35 openstackgerrit Mark Chappell proposed openstack/openstacksdk master: Add support for updating Block Storage Volume type extra_spec attributes https://review.opendev.org/757891
15:32:33 openstackgerrit Merged openstack/ansible-collections-openstack master: Add changelog for 1.2.0 https://review.opendev.org/757867
18:07:12 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: OpenStackModule: Support defining a minimum version of the SDK https://review.opendev.org/757561
18:07:13 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: volume_type https://review.opendev.org/757562
23:26:24 openstackgerrit Merged openstack/python-openstackclient master: Always display direction for security group rules https://review.opendev.org/709017
23:26:26 openstackgerrit Merged openstack/python-openstackclient master: compute: Add --password option for openstack server create https://review.opendev.org/490718
#openstack-sdks - 2020-10-14
00:05:39 openstackgerrit Merged openstack/openstacksdk master: Disable dstat on ironic job https://review.opendev.org/757805
01:27:59 openstackgerrit Merged openstack/openstacksdk master: Expand user path when loading SSL-related files https://review.opendev.org/757088
03:45:55 openstackgerrit Merged openstack/python-openstackclient master: Add 'server migration abort' command https://review.opendev.org/716960

Earlier   Later