15:01:46 <gundalow> #startmeeting Ansible Core public meeting
15:01:46 <zodbot> Meeting started Thu Feb 14 15:01:46 2019 UTC.
15:01:46 <zodbot> This meeting is logged and archived in a public location.
15:01:46 <zodbot> The chair is gundalow. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:01:46 <zodbot> Useful Commands: #action #agreed #halp #info #idea #link #topic.
15:01:46 <zodbot> The meeting name has been set to 'ansible_core_public_meeting'
15:02:35 * madonius waves
15:04:16 <maxamillion> .hello2
15:04:17 <zodbot> maxamillion: maxamillion 'Adam Miller' <maxamillion@gmail.com>
15:04:42 <gundalow> #chair alongchamps madonius maxamillion
15:04:42 <zodbot> Current chairs: alongchamps gundalow madonius maxamillion
15:05:40 <gundalow> #info Agenda https://github.com/ansible/community/labels/core
15:06:01 <gundalow> #topic Feature/alphanumeric password in passwordstore #38121
15:06:07 <gundalow> #info https://github.com/ansible/ansible/pull/38121
15:06:19 <gundalow> madonius: Hi, all yours
15:06:38 <madonius> I am not quite sure what part I am to play
15:06:53 <madonius> as I see it the only topic is my PR?
15:07:09 <gundalow> madonius: start with you :)
15:07:14 <madonius> ok
15:07:45 <madonius> I am a heavy user of ansible and there was a feature missing from the password store module
15:07:59 <gundalow> Thanks for adding it
15:08:05 <madonius> this was added by me in PR 38121
15:08:20 <madonius> and it's been there a while ^^'
15:08:49 <madonius> so, I'd like you to merge it. Especially since somebody else expressed interest
15:09:12 <maxamillion> looking
15:09:30 <gundalow> Looks sensible
15:09:56 <maxamillion> yeah, I'm on board
15:09:57 <maxamillion> +1
15:10:48 <maxamillion> gundalow: you want to `rebuild_merge` or you want me to?
15:11:00 * gundalow will, thanks
15:11:07 <maxamillion> +1
15:12:19 <gundalow> done
15:12:26 <gundalow> madonius: Thanks for raising it
15:13:04 <madonius> thanks for the work :)
15:13:13 <gundalow> No DW today
15:13:16 <gundalow> #topic Open Floor
15:14:27 <gundalow> Anyone got anything else?
15:14:48 <alongchamps> I had a failure on one of my tests that seems to be external (error was that docker couldn't find container id)
15:14:55 <alongchamps> not sure where to bring it up
15:14:58 <gundalow> Got a link?
15:15:01 <gundalow> Is it still failing?
15:15:12 <alongchamps> I didn't run it again yet, https://app.shippable.com/github/ansible/ansible/runs/107432/summary/console
15:15:25 <alongchamps> or more directly: https://app.shippable.com/github/ansible/ansible/runs/107432/80/console
15:15:46 <alongchamps> I wasn't sure if there was a command I could put on PR 52125 to run again, or if it needed another commit to trigger
15:16:25 <alongchamps> there's also a link on this page that 404's: https://github.com/ansible/community/wiki/Testing, links to https://github.com/ansible/community/tree/master/group-testing
15:16:53 <gundalow> ah, all the `group-*` have been removed, I'll fix that
15:17:07 <alongchamps> +1
15:17:41 <maxamillion> yeah, that test seems like it might be a shippable problem ... or we goofed something
15:18:39 <alongchamps> usually that test just gets skipped with no tests for my pulls
15:19:27 <dag> gundalow: new agenda item, I would like to revert this: https://github.com/ansible/ansible/pull/51657
15:21:24 <gundalow> alongchamps: wiki updated to remove broken link, thanks
15:21:31 <alongchamps> :)
15:22:04 <dag> gundalow: originally the Wiki pages had a link back to the group-entries (as those were considered more official)
15:22:19 <gundalow> dag: nod, I need to check out the wiki repo and find and remove them all
15:22:24 <gundalow> #chair dag
15:22:24 <zodbot> Current chairs: alongchamps dag gundalow madonius maxamillion
15:22:41 <gundalow> I need to drop maxamillion you OK to continue, just one other item from dag ?
15:24:06 <dag> may not be core-related though :-)
15:24:20 <maxamillion> gundalow: yeah, that's good
15:24:27 <dag> but since there was a meeting, and I think we need to revert it asap...
15:24:36 <gundalow> dag: or just fix the comments
15:24:45 <gundalow> #topic GitHub Issue templates
15:24:48 <gundalow> right, dropping now
15:25:11 <maxamillion> dag: so what's up? ... I'm not familiar with the changes that occurred with the issue template ... what needs changing or do you just want to revert all together?
15:26:41 <sivel> I honestly cannot see many if anyone actually filling out that portion of the template "Dependency Problems"
15:27:19 <maxamillion> yeah, same
15:27:42 <maxamillion> I'd wager some people don't even know exactly what 'dependency' means in that context
15:28:49 <maxamillion> I'm +1 to revert
15:29:53 <madonius> I am undecisive, therefore I would abstain from either down or upvoting it, if that is fine
15:31:10 <maxamillion> sivel: ?
15:31:30 <maxamillion> actually, do we even have quorum? (what is quorum numbers these days?)
15:31:32 <sivel> +1 for me too
15:31:54 * maxamillion misses working on core stuff :/
15:32:02 <sivel> maxamillion: if has never been defined, although I have repeatidly tried to convince people that a vote requires 5 people
15:32:14 <jtanner> if people are changing templates, i need to be informed.
15:32:30 <sivel> jtanner: someone added a new `DEPENDENCY PROBLEMS` section
15:32:37 <sivel> I think we don't believe it is useful, or will be sued
15:32:38 <maxamillion> jtanner: yo, we might change some ish
15:32:48 <maxamillion> :)
15:32:49 <sivel> we're talking about reverting that change
15:32:52 <jtanner> the bot died when  51657 was merged
15:33:05 <jtanner> if it's reverted, it will die again
15:33:16 <sivel> Ok, so not just revert, but remove
15:33:33 <jtanner> i have to go fix a line of code, so i just need to know when it's happening
15:33:35 <sivel> since the PR effectively moved the source of truth for the template
15:35:58 <madonius> afk, brb
15:38:15 <madonius> b
15:40:02 <maxamillion> jtanner: how should we proceed? the general consensus is in agreement with dag in that we need to remove that change, either by revert or otherwise
15:41:10 <jtanner> either revert the PR and let me know that i need to fix the bot, or make a commit to remove that paragraph in the new template location
15:41:34 <jtanner> changing the file location is what creates pain for the bot
15:44:48 <maxamillion> sivel: dag: any preference?
15:46:44 <maxamillion> alright, I'll call it ... we'll just revert
15:46:55 <maxamillion> jtanner: I'm going to revert the PR, do the needful on the bot
15:47:09 <sivel> maxamillion: can we just remove that section via a new PR instead of reverting?
15:47:17 <sivel> then we don't need any bot changes afaik
15:47:40 <jtanner> i may make a new section in BOTMETA so that i'm not having to deal with this anymore
15:47:42 <sivel> that would be easiest
15:47:49 <maxamillion> sivel: part of the initial complaint was that the old file was removed and that caused issues for certain users
15:51:54 <jtanner> i think the mistake is believing the templates are synonymous with troubleshooting guides. they need to remain simple and not arduous for issue reporters
15:52:26 <jtanner> so i see that as the reason why they're always in flux
15:56:51 <maxamillion> alright
15:57:10 <maxamillion> so what do we want to do here?
15:58:39 <madonius> considered all things, I would side with jtanner and 'revert'
15:59:19 <madonius> if actual reverting is an issue then a new PR would be a good way
16:00:20 <maxamillion> I thought jtanner was suggesting we don't revert
16:00:48 <jtanner> just make a new commit to remove the new stanza
16:00:57 <jtanner> or have someone do it
16:02:48 <madonius> maxamillion read the PR wrong
16:03:05 <maxamillion> alright
16:03:07 <madonius> i would be pro decluttering the templates
16:03:26 <maxamillion> I'll file a new PR to remove the `DEPENDENCY PROBLEMS` section
16:05:44 <jtanner> thnx
16:07:32 <maxamillion> alright, any other open floor business or can we light the fuse?
16:07:54 <jtanner> we're over time anyway
16:09:21 <madonius> I would agree
16:09:29 <madonius> let's call it a day
16:12:01 <maxamillion> #endmeeting