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

The requirements for the Tailscale service are not described #471 #484

Conversation

phillxnet
Copy link
Member

@phillxnet phillxnet commented Jun 21, 2024

Provide a new light-weight upstream instructions wrapper How-to, linked from our existing Web-UI Tailscale service section. And, in-turn, the How-to links back to the service section for configuration and enablement.

Fixes #471

Checklist

  • With the proposed changes no Sphinx errors or warnings are generated.
  • I have added my name to the AUTHORS file, if required (descending alphabetical order).

…#471

Provide a new light-weight upstream instructions wrapper How-to,
linked from our existing Web-UI Tailscale `service` section. And,
in-turn, the How-to links back to the service section for
configuration and enablement.
@phillxnet
Copy link
Member Author

phillxnet commented Jun 21, 2024

@FroggyFlox Give this changes a tad your original docs contribution, and that you are more familiar with our Tailscale integration, given you coded it in the first place, could you take a quick look at this before we merge. I think it basically covers our first pass at this doc enhancement, and gets folks off-the-ground on this new exciting feature. And hopefully keeps thinks simple regarding our stance re NOT pre-installing this cloud-based capability.

Plus it serves existing installs re the How-to link from our Tailscale services section and we can, once published, consider liking folks directly the 'Install Tailscale' How-to provided contextually upon no-install state such as is proposed in:

rockstor/rockstor-core#2845 Which should probably be re-named now given our new stance re NOT pre-installing and not offering an automated install (at least yet).

We can always re-visit the added How-to re an OBS alternative repo section etc: but upstream is often more the go-to on such things anyway. So we should at least start with the vanilla, and likely most supported option anyway.

I'm not keen on any install-via-curl advise so have mentioned only the zypper approach.

Copy link
Member

@FroggyFlox FroggyFlox left a comment

Choose a reason for hiding this comment

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

@phillxnet
Thank you so much for taking care of yet another issue.
This looks great!

howtos/tailscale_install.rst Outdated Show resolved Hide resolved
@phillxnet phillxnet merged commit 534b938 into rockstor:master Jun 22, 2024
3 checks passed
@phillxnet phillxnet deleted the 471-The-requirements-for-the-Tailscale-service-are-not-described branch June 22, 2024 11:02
@phillxnet
Copy link
Member Author

PR product PRODUCTION published.

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

Successfully merging this pull request may close these issues.

The requirements for the Tailscale service are not described
2 participants