Earlier  
Posted Nick Remark
#openstack-sdks - 2020-06-03
14:52:09 nightmare_unreal cool Thanks
14:52:13 diablo_rojo mordred, ah nevermind the requirement then. Just would be nice for numbers for the foundation.
14:52:16 diablo_rojo nightmare_unreal, oh cool.
14:52:20 nightmare_unreal so on meetpad .
14:52:22 diablo_rojo All good then :)
14:52:34 mordred diablo_rojo: yes - registering is good :)
14:53:11 nightmare_unreal :)
14:54:25 diablo_rojo mordred, indeed it is. As much as I hate measurements and numbers because they are usually inaccurate or require a lot of context.. I also depend on them.
15:03:22 gtema anyone joining the meeting, which has started?
15:03:38 mordred gtema: yes!
15:05:19 diablo_rojo gtema, unfortunately have to stay in TC things. Will try to catch up later!
15:56:39 dtantsur mordred: if you have a minute at some point, could you have a quick look at https://review.opendev.org/#/c/733111/ to see if I'm doing something stupid, esp with regards to backward compatibility?
15:58:21 mordred dtantsur: ooh fun
16:00:31 dtantsur a lot of fun :)
16:00:39 dtantsur I especially loved the network_interface issue
16:26:06 mordred dtantsur|afk: oh yeah - that's really nice!
16:26:44 mordred dtantsur|afk: I don't suppose we're at a point yet where bifrost can just use the ansible collection (I suppose bifrost probably supports an ansible < 2.8 yes?)
16:29:11 mordred dtantsur|afk: I thnik that patch looks great
16:57:28 openstackgerrit Merged openstack/python-openstackclient stable/ussuri: Fix ussuri jobs https://review.opendev.org/731122
16:57:29 openstackgerrit Merged openstack/python-openstackclient stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/724060
17:28:24 openstackgerrit Merged openstack/ansible-collections-openstack master: Fix ansible-tests for devel branch https://review.opendev.org/732160
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

Earlier   Later