18:00:21 #startmeeting Ansible Community Meeting 18:00:22 Meeting started Wed Oct 12 18:00:21 2022 UTC. 18:00:22 This meeting is logged and archived in a public location. 18:00:22 The chair is gotmax. Information about MeetBot at https://fedoraproject.org/wiki/Zodbot#Meeting_Functions. 18:00:22 Useful Commands: #action #agreed #halp #info #idea #link #topic. 18:00:22 The meeting name has been set to 'ansible_community_meeting' 18:00:24 hi anwesha 18:00:26 o/ 18:00:29 #topic Intro 18:00:37 .hello gotmax23 18:00:38 gotmax: Something blew up, please try again 18:00:41 gotmax: An error has occurred and has been logged. Please contact this bot's administrator for more information. 18:01:01 .fire zodbot 18:01:01 adamw fires zodbot 18:01:19 https://meetbot.fedoraproject.org/ansible-community/2022-09-28/ansible_community_meeting.2022-09-28-18.00.log.html as an example of commands 18:01:28 andersson007__, hey 18:01:45 acozine andersson007_ baptistemm bcoca briantist cyberpear cybette dericcrago dmsimard felixfontein geerlingguy gundalow gwmngilfen ikhan_ jillr jtanner lmodemal misc nitzmahone resmo samccann tadeboro cidrblock thaumos zbr: Community meeting 18:01:46 evening 18:02:00 o/ 18:02:00 o/ 18:02:12 i think it should be "info" 18:02:14 hi cybette 18:02:23 hi Don Naro :) 18:02:45 gotmax: not worries, don't rush 18:03:09 I was going to wait another minute for people to come in 18:03:28 ah, ok 18:03:40 then we can chair all at once:) 18:03:57 is there a lag? I see the topic change but I don't see the messages that invoked it 18:03:58 Ah, I forgot to do that 18:04:13 #chair andersson007__ cybette_ DonNaro[m] gotmax[m] 18:04:13 Current chairs: DonNaro[m] andersson007__ cybette_ gotmax gotmax[m] 18:04:22 Okay 18:04:26 #topic Info 18:04:29 #info Agenda: https://github.com/ansible/community/issues/645 / Topics: https://github.com/ansible-community/community-topics 18:04:54 #info ansible-core 2.12.10, 2.13.5, and 2.14.0b3 were released this week 18:04:58 i have one but i'd wait a bit 18:05:16 #chair anwesha 18:05:16 Current chairs: DonNaro[m] andersson007__ anwesha cybette_ gotmax gotmax[m] 18:05:50 There's a couple issues tagged with meeting 18:06:05 I think we've already addressed the editor configs one? 18:06:08 #info ansible-core 2.14.0b3 (3rd beta) has been released https://groups.google.com/g/ansible-devel/c/s2EwCSbS_hM 18:06:10 #info ansible-core 2.13.5 and ansible-core 2.12.10 have been released: https://groups.google.com/g/ansible-devel/c/k6FX5XNH9ww 18:06:44 o/ 18:06:46 #info the question to Legal about removing/not including stuff from collection artifacts has been sent 18:06:52 #chair mariolenz[m] 18:06:52 Current chairs: DonNaro[m] andersson007__ anwesha cybette_ gotmax gotmax[m] mariolenz[m] 18:07:00 andersson007__++ 18:07:21 I guess we can move on to my ticket about release tagging :)? 18:07:33 #info Ansible Contributor Summit 2022.10 HackMD - get all the info here, where/when/how to join, what sessions and tracks are available and some details about them: https://hackmd.io/@ansible-community/contrib-summit-202210 18:07:34 #info Breakout rooms info and live-stream links will be shared shortly. As long as you're in the main conference room on the day (#summit:ansible.com on Matrix, #ansible-conf on Libera Chat) you'll see the signposts for them 18:08:31 Okay, let's move on 18:08:43 ok I'm done :) 18:08:45 #topic Collections that don't tag releases - https://github.com/ansible-community/community-topics/issues/148 18:08:52 wait 18:08:56 #undo 18:08:56 Removing item from minutes: 18:09:06 #info I'm happy to announce that Maxwell G (aka [gotmax23](https://github.com/gotmax23) on GitHub, gotmax on Matrix) has joined the [Steering Committee](https://docs.ansible.com/ansible/devel/community/steering/community_steering_committee.html) !!! 18:09:08 :) 18:09:16 thank you gotmax ! 18:09:23 🎉 18:09:31 for contributing to the community and the project! 18:09:49 congrats gotmax! 18:09:51 we are happy to have you on board! 18:09:59 Thanks :) 18:10:06 gotmax, Congratulations 18:11:02 👍 18:11:15 Any other announcements ;)? 18:11:28 i think we hadn't expanded since early spring 18:11:51 nice to see new people coming 18:12:09 Indeed :) 18:12:16 congrats gotmax 18:13:34 exciting nows :) and I think now we can move on to discussion topics 18:13:42 #topic Collections that don't tag releases - https://github.com/ansible-community/community-topics/issues/148 18:13:48 Second time's the charm 18:14:18 Basically, there's at least seven collections that don't git tag their releases 18:14:25 This is a violation of the collection guidelines 18:14:39 I'd like to clarify those guidelines and then file issues against those collections 18:15:01 @cybette:ansible.im cyb-clock chimes every 15 minutes during the community meeting 18:15:01 sounds good to me 18:15:28 I proposed new text: #topic Collections that don't tag releases - https://github.com/ansible-community/community-topics/issues/148 18:15:52 I guess I'll wait for some initial comments and then create a PR/start a vote 18:16:29 after the clarifications are made, as i see the process 1) open issues in the repos 2) if no response/actions within some time, we'll create removal issues, i think 18:17:11 Makes sense to me 18:17:45 I think there are 5: mellanox.onyx is unmaintained (we've voted on this) and cisco.nso also looks unmaintained. But yes, we need to decide what to do with those 5 collections. 18:17:46 should we prepare a justification? we, of course, can say just that it's a requirement 18:18:36 Eventually, I'd like to add collections' repository URLs to ansible-build-data and have an automated check to make sure the releases are tagged 18:19:08 any automation would be great 18:19:11 andersson007__: A justification for removing cisco.nso or a justification for the proposed wording change? 18:19:35 gotmax: a justification for tagging releases 18:19:45 in case the question appears 18:19:56 i.e. "why?" 18:20:01 Good point 18:20:03 Or to tag releases? I think it's already a requirement, MUST is quite clear. 18:21:41 andersson007__: Mainly, collections artifacts should be reproducible and verifiable. 18:21:55 This willll be more important if we start removing tests from collections 18:22:09 s/willll/will/ 18:22:11 :) i think we don't need any justification at least at the moment, generally 18:22:17 tagging is in the collection requirements. were these collections grandfathered in that's why they slipped through without tagging releases? 18:22:18 it's just a requirement 18:22:29 if someone raises the topic, we could discuss 18:22:30 there 18:22:40 andersson007__: Works for me 18:22:41 the topic about tagging to be specific 18:22:44 yep 18:22:59 gotmax: ok, forget my question then:) 18:23:07 "Any changes made during the build process MUST be clearly documented so the collection artifact can be reproduced" provides kind of touches on this 18:23:19 cybette_: I think some collections just don't follow the rules 18:23:36 I don't think is (as much of) a problem with community collections 18:24:33 i also saw situations where there were tags in collections but on Galaxy there was a mismatch 18:24:59 You mean the contents were different? 18:25:04 e.g. the last tag in repo was older by version than on Galaxy or vice versa 18:25:11 Ah, yes 18:25:57 I don't think that's a guidelines violation, but it's confusing and should not happen 18:26:17 maybe we should also advertise Zuul for auto publishing 18:26:30 in the requirements 18:26:43 We have a process for removing collections because they look unmaintained. But I don't think we one to remove collections that fail requirements after they've been included. Maybe we should do this. 18:26:49 * gotmax is not super familiar with how that process works 18:27:31 mariolenz[m]: I think they should be removed if they don't follow the guidelines *and* don't fix issues that are pointed out to them 18:27:51 gotmax: basically, if you do what's written in https://github.com/ansible/zuul-config/README.md, zuul, when seeing a tag, will publish the targall on Galaxy 18:28:09 I think not fixing issues could count as being unresponsive and then fall under the current policy 18:28:19 https://github.com/ansible/zuul-config/blob/master/README.rst sorry 18:28:58 i believe to follow the requirements is a requirement:) 18:29:13 But how does it have access to collections outside of the community namespace? 18:29:14 i.e. included collections must satisfy 18:29:31 I mean permissions to the namespace on Galaxy 18:29:38 gotmax: i don't know how it works under the hood 18:30:02 @cybette:ansible.im cyb-clock chimes every 15 minutes during the community meeting 18:30:06 we can ask in galaxy channel 18:30:11 As long as any collection can participate (which seems to be the case), I'm fine with recommending that 18:30:23 Although maybe it should be a separate discussion 18:30:32 sounds sensible 18:30:43 gotmax: I agree, but did we document this somewhere? 18:30:46 about having the separate discussion 18:31:03 mariolenz[m]: Document which thing :)? 18:32:09 Removing collections that fail requirements. 18:33:07 I guess we should clarify that 18:33:22 by adding something to https://github.com/ansible-collections/overview/blob/main/removal_from_ansible.rst 18:33:25 i think 18:33:34 Yeah, we could add a condition to https://github.com/ansible-collections/overview/blob/main/removal_from_ansible.rst#id26 18:34:12 I guess I'll #action the things I said I'd do :) 18:34:34 +1 18:34:43 #action gotmax[m] to submit a PR and start a vote on the proposed guidelines changing after a couple days if there's no major objections 18:35:13 #action gotmax[m] to look into automation to ensure Galaxy collection releases are tagged 18:35:32 i think it can be derived indirectly from "The collection for inclusion must follow the requirements" but it sounds appropriate to have it written somewhere directly 18:35:43 nice 18:35:48 gotmax: thanks! 18:36:49 #info We'll create a separate issue to discuss considering collections that don't fix guidelines violations unmaintained under https://github.com/ansible-collections/overview/blob/main/removal_from_ansible.rst#id26 18:36:49 how about adding clarification that collections can be removed if they don't follow requirements ? 18:36:50 https://github.com/ansible-collections/overview/blob/main/removal_from_ansible.rst mentions broken and unmaintained collections, but doesn't talk about collections failing requirements. 18:37:28 mariolenz[m]: it's a good point, yes, we agreed a bit above on adding this 18:37:38 #info We'll create a separate issue to discuss recommending collection use Zuul for Galaxy releases (https://github.com/ansible/zuul-config/) 18:37:48 mariolenz[m]: 18:37:56 mariolenz[m]: it should be added 18:38:06 if no one wants, i could 18:38:14 but if someone wants, welcome 18:38:40 to update https://github.com/ansible-collections/overview/blob/main/removal_from_ansible.rst#id26 + Create an issue to vote 18:38:57 or we could create an issue first, then update 18:39:24 I think that we should add we'll remove collections that don't implement all requirements. 18:39:38 +1 18:40:23 I think there needs to be a warning and we should have a set waiting period before starting the removal process 18:40:34 sure 18:41:04 i think it should reflect the processes already described in the removal doc 18:41:26 * gotmax nods 18:41:55 feels like it's a very productive meeting today:) 18:42:12 :) 18:43:16 so how will we proceed with this? 18:43:51 Well we need to create an issue against community-topics and/or a PR 18:44:06 * gotmax is not yet familiar with the exact processes 18:44:52 we could 1) create the topic + the PR or 2) only the topic, then the PR if there's support for the idea 18:45:06 @cybette:ansible.im cyb-clock chimes every 15 minutes during the community meeting 18:45:21 I don't mind either way. 18:45:26 me too 18:45:35 Who wants to do that :)? 18:45:40 i could 18:46:00 * andersson007__ setting up a reminder 18:46:42 #action andersson007__ to create an issue re. the unmaintained collection process changes 18:46:55 gotmax: thanks! was typing the same:) 18:47:03 Oops :) 18:47:26 gotmax: you'll save me from typing several words, thanks:) 18:47:33 several more words 18:48:11 We're nearing the end, so I'd like to move on if there's nothing else pressing about this topic 18:48:32 sure 18:48:38 not from me 18:49:01 #topic Inclusion requirement - editor configs, and the meaning of "unnecessary" - https://github.com/ansible-community/community-topics/issues/138 18:49:10 I think we just need to merge the PR here? 18:49:55 It's tagged as meeting, so I figured I'd touch on it 18:50:45 * andersson007__ looking 18:51:31 btw, there's a topic for today's meeting according to the ansible 7 roadmap (https://docs.ansible.com/ansible/devel/roadmap/COLLECTIONS_7.html) - "List any backwards incompatible collection releases that beta1 should try to accommodate." I'm not sure if there's anything to discuss but I thought I'd mention it. 18:55:29 It looks like andersson007__ merged the PR, so I'll move on to open floor 18:55:33 #topic Open Floor 18:55:46 gotmax: I've merged the PR and it closed the issue, I added a comment for briantist. Thanks for reminding about the topic! 18:56:08 I don't think it's worth discussing the flatmapping issue in depth without felixfontein 18:57:16 andersson007__: Sure! 18:57:20 yep 18:58:45 https://github.com/ansible/ansible/pull/75116 19:00:14 bcoca: Looks like you have a merge commit remnant https://github.com/ansible/ansible/pull/75116/files#diff-ffb7d7139b0bce523da8e29927b84ced4210df9a774b763ccfccb5c57ef0b932 19:00:28 Anyways, the time has come 19:00:32 #endmeeting