| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2020-06-04 | |||
| 10:50:00 | openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Redesign OpenstackModule class https://review.opendev.org/722283 | |
| 10:50:12 | frickler | nightmare_unreal: seems this block messes things up for json https://opendev.org/openstack/python-openstackclient/src/branch/master/openstackclient/compute/v2/server.py#L173-L179 | |
| 10:50:54 | nightmare_unreal | Thanks frickler :) . I will look into it | |
| 11:47:56 | nightmare_unreal | found the fix | |
| 12:11:00 | mordred | sshnaidm: oh, you can use collections if they've already been installed in 2.8? | |
| 12:11:01 | openstackgerrit | jayaditya gupta proposed openstack/python-openstackclient master: Fixes incorrect json output when command openstack server show -f json is executed. The security group json was nor correct. Story: 2007755 https://review.opendev.org/733573 | |
| 12:11:17 | mordred | interesting - I thought you had to be on 2.9 to even use them | |
| 12:11:18 | sshnaidm | mordred, yes | |
| 12:11:53 | sshnaidm | mordred, just for usage 2.8 is enough | |
| 12:19:04 | mordred | sshnaidm: cool, I think once we drop 2.7 in zuul then we can start rolling out some initial collections stuff | |
| 12:19:20 | mordred | like releasing the zuul ansible modules in a collection | |
| 12:19:25 | mordred | and installing them that way | |
| 12:19:43 | sshnaidm | mordred, nice, and what are zuul ansible modules? | |
| 12:20:58 | mordred | we have several zuul-specific modules, like zuul_return - that we're planning on moving to a collection for in-zuul use and releasing on galaxy for better ease of linting/testing ansible content for zuul in a non-zuul context | |
| 12:21:39 | mordred | and then there's a few other things where we have (or need) some generally useful modules that we want to release in a similar way, but that might just be useful to people who aren't using zuul as well | |
| 12:21:55 | mordred | but since we still support 2.7 - it doesn't make a ton of sense to do yet | |
| 12:22:11 | mordred | before this morning I was thinking we had to wait until 2.9 - so that's good news | |
| 12:44:28 | sshnaidm | mordred, btw, recently ansible-galaxy was patched to support installation from git, url, directory on host | |
| 12:46:05 | mordred | sshnaidm: yeah - I saw that, that's good! | |
| 12:47:58 | mordred | sshnaidm: although I think the install-from-git likely won't work as awesome for us given that we generate our galaxy.yml. we should figure out how to make a patch for galaxy to support our usecase for that. pabelanger said he wanted to do similar | |
| 12:48:09 | mordred | I think "install from directory" is gonna be awesome | |
| 12:49:06 | sshnaidm | well, in jobs we should install from directory, anyway zuul clones it with dependencies etc | |
| 12:49:34 | sshnaidm | for others would be nice to test unpublished collection installing it from repo directly | |
| 12:49:49 | sshnaidm | or from patch | |
| 12:50:34 | sshnaidm | well, need to see if it's possible w/o having galaxy.yml in repo.. | |
| 12:53:38 | mordred | sshnaidm: we could just publish every commit to galaxy as a pre-release | |
| 12:54:01 | mordred | :) | |
| 12:55:06 | mordred | sshnaidm: or, at the very least, we could do a master tarball to tarballs.openstack.org - so a consistent url that you could use to get the latest unreleased version | |
| 12:56:11 | mordred | similar to https://tarballs.opendev.org/openstack/openstacksdk/openstacksdk-master.tar.gz | |
| 13:25:24 | AJaeger | could I get some review love for docstheme updates, please? python-openstackclient: https://review.opendev.org/#/c/729033/ ; openstackclient: https://review.opendev.org/#/c/732056/ ; os-service-types: https://review.opendev.org/732058 ; osc-lib: https://review.opendev.org/732060 | |
| 13:27:06 | openstackgerrit | jayaditya gupta proposed openstack/python-openstackclient master: Output correct json for security groups in 'openstack server show' https://review.opendev.org/733573 | |
| 13:44:14 | gtema | mordred: isn't SDK team also core in os-service-types? | |
| 13:46:58 | mordred | gtema: sort of - it includes the wrong team | |
| 13:47:01 | mordred | let me go fix | |
| 13:49:19 | mordred | gtema: it was including python-openstacksdk-core not openstacksdk-core :) | |
| 13:49:26 | gtema | oki | |
| 13:49:50 | gtema | works now | |
| 13:51:02 | AJaeger | thanks, gtema for the reviews! | |
| 13:51:07 | gtema | wlcm | |
| 13:53:48 | openstackgerrit | Merged openstack/openstackclient master: Switch to newer openstackdocstheme version https://review.opendev.org/732056 | |
| 14:04:24 | openstackgerrit | Merged openstack/os-service-types master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/732058 | |
| 14:10:54 | vkmc | gtema, o/ | |
| 14:11:25 | vkmc | did you have the openstack client "common" commands discussion? | |
| 14:11:48 | vkmc | we might have some time tomorrow in our PTG if you don't have any other commitment, it would be nice to have you there and brainstorm on how we can help from the manila side | |
| 14:11:58 | gtema | vkmc, no, completely forgot that, sorry | |
| 14:12:14 | vkmc | gtema, no probs | |
| 14:12:30 | gtema | which time? | |
| 14:13:20 | gtema | currently 14:30-16 UTC window is ok for me | |
| 14:13:41 | gtema | 16 means actually till 17 | |
| 14:14:01 | vkmc | gtema, I need to check with our ptl gouthamr, but we have our ptg running from 13 to 17 | |
| 14:14:20 | gtema | exactly, I mean that from 14:30 I can join any time | |
| 14:14:38 | vkmc | maybe 14.30 could work | |
| 14:14:44 | vkmc | but let me confirm with him :) | |
| 14:14:55 | gtema | okay, then ping me if I mess up | |
| 14:17:01 | openstackgerrit | Merged openstack/ansible-collections-openstack master: Redesign OpenstackModule class https://review.opendev.org/722283 | |
| 14:17:39 | vkmc | gtema, sure | |
| 14:17:42 | vkmc | gtema, what timezone are you? | |
| 14:17:57 | gtema | I'm CEST (UTC+2) | |
| 14:18:10 | vkmc | ok, so earlier is better | |
| 14:18:55 | gtema | no necessarily, after 16 is less good, but 15 is absolutely fine | |
| 14:23:24 | openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add setup.py for install with pip https://review.opendev.org/730400 | |
| 14:23:49 | sshnaidm|mtg | noonedeadpunk, these should help with tripleo and tempest ^ | |
| 14:24:55 | noonedeadpunk | ok, cool would keep a track on that | |
| 14:38:26 | nightmare_unreal | frickler: I saw your review. yes volume_attached is affected and also properties field too. Yeah need to find a solution with which format is requested. | |
| 14:38:59 | nightmare_unreal | looking where the -f/--format is processed | |
| 15:03:49 | openstackgerrit | Merged openstack/osc-lib master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/732060 | |
| 15:57:50 | smcginnis | A/20 | |
| 16:03:03 | elmiko | API SIG office hour open | |
| 16:03:32 | elmiko | i know ptg is going, but i never sent an email about cancelling, or the new email about our plans. so... once more, with /feeling/ | |
| 16:31:06 | openstackgerrit | Andreas Jaeger proposed openstack/python-openstackclient master: Remove congress https://review.opendev.org/733656 | |
| 17:40:06 | AJaeger | python-openstackclient is missing the Victoria changes https://review.opendev.org/724062 and https://review.opendev.org/724063 - please review | |
| 19:10:28 | openstackgerrit | Merged openstack/python-openstackclient master: Update master for stable/ussuri https://review.opendev.org/724062 | |
| 19:26:09 | AJaeger | python-openstackclient needs one more change: https://review.opendev.org/733656 - removal of congress, please review to unblock requirements-check | |
| 19:49:38 | openstack | bug 1844708 in python-openstackclient (Ubuntu) "Unable to set flavor description with flavor name" [Undecided,Fix committed] https://launchpad.net/bugs/1844708 - Assigned to Gabriel Ramirez (gabriel1109) | |
| 19:49:38 | openstackgerrit | Gabriel Ramirez proposed openstack/python-openstackclient master: Allow openstack flavor set to update flavor description using name * Modified take_action() method for SetFlavor to use flavor id instead of flavor name when setting description * Closes-Bug: #1844708 / https://bugs.launchpad.net/ubuntu/+source/python- https://review.opendev.org/733688 | |
| 19:55:35 | openstackgerrit | Merged openstack/python-openstackclient master: Add Python3 victoria unit tests https://review.opendev.org/724063 | |
| 19:55:37 | openstackgerrit | Merged openstack/python-openstackclient master: Add py38 package metadata https://review.opendev.org/730227 | |
| 22:50:59 | openstackgerrit | Gabriel Ramirez proposed openstack/python-openstackclient master: Allow openstack flavor set to update flavor description using name https://review.opendev.org/733688 | |
| #openstack-sdks - 2020-06-05 | |||
| 04:21:48 | openstackgerrit | Merged openstack/keystoneauth master: Use unittest.mock instead of third party mock https://review.opendev.org/720925 | |
| 04:45:24 | openstackgerrit | Merged openstack/keystoneauth master: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/733139 | |
| 09:40:15 | openstackgerrit | Merged openstack/api-sig master: Switch to newer openstackdocstheme version https://review.opendev.org/733331 | |
| 13:42:21 | AJaeger | mordred, python-openstackclient needs one more change: https://review.opendev.org/733656 - removal of congress, please review to unblock requirements-check | |
| 13:45:39 | mordred | AJaeger: done | |
| 13:45:59 | AJaeger | thanks! | |
| 14:58:37 | vkmc | gtema, o/ | |
| 14:59:03 | gtema | I am here vkmc | |
| 14:59:06 | vkmc | probably you know by now, but we added the discussion on the common commands in 30 min from now | |
| 14:59:18 | vkmc | https://etherpad.opendev.org/p/victoria-ptg-manila | |
| 14:59:18 | vkmc | 15:30 utc | |
| 14:59:20 | vkmc | etherpad there | |
| 14:59:26 | gtema | yes, have seen. thanks for remind | |
| 14:59:30 | vkmc | cool | |
| 15:50:41 | vkmc | elmiko !!!!! | |
| 15:50:42 | vkmc | hey :D | |
| 16:16:39 | openstackgerrit | Merged openstack/python-openstackclient master: Remove congress https://review.opendev.org/733656 | |
| 18:32:12 | openstackgerrit | Merged openstack/python-openstackclient master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/729033 | |
| 19:32:09 | openstackgerrit | Merged openstack/python-openstackclient master: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/732393 | |
| #openstack-sdks - 2020-06-06 | |||
| 13:04:45 | openstackgerrit | Monty Taylor proposed openstack/cliff master: Add load_plugins option https://review.opendev.org/733961 | |
| 14:32:54 | openstackgerrit | Monty Taylor proposed openstack/cliff master: Import command group support from osc-lib https://review.opendev.org/733964 | |
| 14:32:54 | openstackgerrit | Monty Taylor proposed openstack/cliff master: Add load_plugins option https://review.opendev.org/733961 | |