Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Tailscale as a sidecar if enabled #162

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

tancou
Copy link

@tancou tancou commented May 3, 2024

Description of the change

Uptime Kuma natively supports pinging with the Tailscale VPN on the non-Dockerized version. I find it regrettable that we can't bridge with a Tailscale container in sidecar mode.
Therefore, I have added support, which can be enabled or not, with or without root, from the official Tailscale documentation in the chart.
The default values do not change the behavior of the chart.

Benefits

With a Tailscale device in sidecar mode, it is possible to access all the devices on the network directly via their IP, enabling all the monitoring supports that Uptime Kuma offers. Depending on the defined ACLs, it is also possible to access Uptime Kuma from the Tailscale network.

Possible drawbacks

To have direct access to the Tailscale network from Uptime Kuma without using the proxy option, the Tailscale in sidecar mode must run as root, which reduces security. See the documentation kubernetes#userspace-sidecar.

Root mode can be disabled via values.

Applicable issues

  • fixes #

Additional information

Checklist

  • Chart version bumped in Chart.yaml according to semver.
  • Variables are documented in the README.md

@tancou tancou requested a review from dirsigler as a code owner May 3, 2024 18:04
@dirsigler dirsigler added the enhancement New feature or request label May 17, 2024
@tancou
Copy link
Author

tancou commented Jul 30, 2024

I have just rebased my branch, bump the helm chart version to 2.20.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants