Skip to content

Draft: feat(tailscale): New app - #1223

Draft
brettpetch wants to merge 9 commits into
developfrom
tailscale
Draft

brettpetch wants to merge 9 commits into
developfrom
tailscale

Conversation

@brettpetch

@brettpetch brettpetch commented Apr 15, 2025

Copy link
Copy Markdown
Member

Description

The following PR commits the ability to use tailscale.

Fixes issues:

  • Un-tracked issue...

Proposed Changes:

Change Categories

  • New feature
  • Breaking change

Checklist

  • Branch was made off the develop branch and the PR is targetting the develop branch
  • Docs have been made OR are not necessary
    • PR link:
  • Changes to panel have been made OR are not necessary
    • PR link:
  • Code conforms to project structure (See more)
  • Prints to terminal are handled (See more)
  • I have commented my code, particularly in hard-to-understand areas
  • Testing was done
    • Tests created or no new tests necessary
    • Tests executed

Test scenarios

Architectures

amd64 armhf arm64 Unspecified
Jammy
Focal
Bookworm
Bullseye
Buster
Raspbian ⚫️ ⚫️ ⚫️

✅❎ Passed

🛠🛠 TODO

❌❌ Currently failing

@brettpetch
brettpetch marked this pull request as draft April 15, 2025 01:20
@brettpetch brettpetch self-assigned this May 7, 2025
Comment on lines +13 to +14
echo_query "What would you like your machine hostname to be on your tailnet?" "$(hostname)"
read -re ts_hostname

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be cool if we could also optionally get these via env vars, makes testing easier

echo_error "Tailscale is already installed. Please remove /install/.tailscale.lock to reinstall."
fi

curl -fsSL https://tailscale.com/install.sh | sh >> $log 2>&1

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would appreciate having these wrapped in functions

fi

touch /install/.tailscale.lock
echo_progress_done "Tailscale installed and configured successfully."

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might be nice to send users to the docs for how to invite their users if they have any

@github-actions github-actions Bot added the has conflicts This PR has conflicts against master label Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has conflicts This PR has conflicts against master

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants