| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2020-06-03 | |||
| 17:43:34 | openstackgerrit | Merged openstack/python-openstackclient stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/724061 | |
| 17:57:33 | openstackgerrit | Merged openstack/keystoneauth master: Make header Case Insensitive https://review.opendev.org/731847 | |
| 18:13:00 | openstackgerrit | Merged openstack/keystoneauth master: Add in-tree upper-constraints for py35 job https://review.opendev.org/730687 | |
| 19:01:24 | openstackgerrit | Andreas Jaeger proposed openstack/api-sig master: Switch to newer openstackdocstheme version https://review.opendev.org/733331 | |
| 19:02:16 | openstackgerrit | Monty Taylor proposed openstack/ansible-collections-openstack master: Redesign OpenstackModule class https://review.opendev.org/722283 | |
| 22:49:40 | openstackgerrit | Steve Baker proposed openstack/keystoneauth master: Implement HTTP Basic client support in keystoneauth1 https://review.opendev.org/727562 | |
| #openstack-sdks - 2020-06-04 | |||
| 03:28:17 | openstackgerrit | Pete Zaitcev proposed openstack/python-openstackclient master: Allow to list containers of other accounts https://review.opendev.org/731395 | |
| 08:15:24 | dtantsur | mordred: iurygregory wants to look into switching bifrost to the collections once they're ready (are they?) | |
| 08:16:01 | dtantsur | but currently we cap ansible at < 2.9 | |
| 08:17:05 | iurygregory | o/ | |
| 08:17:22 | dtantsur | sshnaidm: hey, could you refresh my knowledge, which version of ansible supports collections? 2.9 and on? 2.10? | |
| 08:20:24 | dtantsur | iurygregory: https://galaxy.ansible.com/openstack/cloud is 1.0.1, we can start switching bifrost, I guess? But we need to figure out the supported ansible versions. | |
| 08:20:39 | dtantsur | "Installing collections with ansible-galaxy is only supported in ansible 2.9+" | |
| 08:21:08 | dtantsur | at step 0 we need to move the cap to >=2.9,<2.10 and fix any fall-out | |
| 08:21:29 | iurygregory | dtantsur, makes sense to me | |
| 08:21:34 | dtantsur | wanna do this? | |
| 08:21:38 | iurygregory | I will push a patch to change the version | |
| 08:21:42 | iurygregory | and see how it goes | |
| 08:21:45 | dtantsur | great, thanks! | |
| 08:22:05 | iurygregory | fyi stable/train is fixed =) | |
| 08:22:09 | dtantsur | \o/ | |
| 08:22:31 | dtantsur | iurygregory: I'm afraid we need to do the same on older branches as well | |
| 08:22:48 | iurygregory | dtantsur, we only need to backport i think =) | |
| 08:23:17 | dtantsur | yep | |
| 08:23:33 | iurygregory | if the backport doesn't work it's probably the version but 2.38.0 works fine in train | |
| 08:56:33 | sshnaidm | dtantsur, from 2.8 | |
| 08:56:54 | dtantsur | oh! so, only ansible-galaxy doesn't work in 2.8? | |
| 09:04:23 | sshnaidm | dtantsur, well, you can use collections from 2.8 in playbooks, but galaxy from 2.8 doesn't have any collection bits | |
| 09:05:56 | sshnaidm | better to use 2.9 everywhere | |
| 09:09:57 | dtantsur | okay, that's what we're trying for bifrost | |
| 10:37:14 | nightmare_unreal | https://www.irccloud.com/pastebin/xe8aHdGV/json_ouput | |
| 10:38:21 | nightmare_unreal | the json is not right | |
| 10:38:33 | nightmare_unreal | where should I look into to fix this | |
| 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 | |