ansible-role-promtail icon indicating copy to clipboard operation
ansible-role-promtail copied to clipboard

macOS support

Open mzihlmann opened this issue 1 year ago • 13 comments

Hey there,

we need to deploy promtail on a fleet of mac-minis to collect some logs from them. here is a proposal on how to support macOS.

mzihlmann avatar Jun 02 '23 08:06 mzihlmann

sorry for the noise up there, had to switch to my private email and add gpg signature

mzihlmann avatar Jun 02 '23 08:06 mzihlmann

ill fix the nits after lunch. thank you!

mzihlmann avatar Jun 02 '23 09:06 mzihlmann

@mzihlmann thank you for your contribution.

The linter is catching some minor nitpicks - but otherwise looks very good Can you take a look at the one test failure ?

One question - do you have an idea how we could include testing this role on a MacOS target ? I'm otherwise a little concerned if I/we can give a garuantee that the MacOS support of this role is stable. Happy to hear your thoughts

patrickjahns avatar Jun 02 '23 09:06 patrickjahns

'os' is undefined when ansible_os_family == 'RedHat']

mzihlmann avatar Jun 02 '23 09:06 mzihlmann

yes testing was also one of my concerns. i think the main blocker would be legal, not technical. We will use this version in production on our servers, but i fully understand if you don't want to get this merged because of testability.

mzihlmann avatar Jun 02 '23 10:06 mzihlmann

im quite impressed with your testing setup :+1:

mzihlmann avatar Jun 02 '23 10:06 mzihlmann

yes testing was also one of my concerns. i think the main blocker would be legal, not technical. We will use this version in production on our servers, but i fully understand if you don't want to get this merged because of testability.

I'd still be happy to merge this, but would not officially support it. Which means if updates break the functionality on Mac , I won't be able to take care of the fixes. If your happy with this 'disclaimer' and open to helping out on the mac support in the long run there is no point of maintaining forks :-) I'd also be open to moving this out of my personal account to plan independent Organisation if there are concerns

I'm only on mobile today - I'll check if we can test this on MacOS somehow with GitHub actions

patrickjahns avatar Jun 02 '23 11:06 patrickjahns

hmmm legally you're only required to run on Apple hardware. Github seems to support macOS runners. So i think it would be possible.

mzihlmann avatar Jun 02 '23 11:06 mzihlmann

of course we would help in maintaining it. disclaimer is fine for us. no need to change organization, we don't have such a policy.

mzihlmann avatar Jun 07 '23 13:06 mzihlmann

ping @patrickjahns

mzihlmann avatar Jun 23 '23 14:06 mzihlmann

@mzihlmann Sorry for the delay - I'll try to get a closer look again this week

patrickjahns avatar Jul 03 '23 15:07 patrickjahns

no worries, i was just cleaning up my jira and saw that this is still open.

mzihlmann avatar Jul 04 '23 21:07 mzihlmann

Just went through some open topics - unfortunately I didn't have time to get around this issue. But it is still on my radar

patrickjahns avatar Sep 06 '23 16:09 patrickjahns