-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
clientupdate: add linux tarball updates #9144
Conversation
files["tailscaled"]++ | ||
if err := writeFile(tr, tailscaled+".new", 0755); err != nil { | ||
return fmt.Errorf("failed extracting the new tailscaled binary from %q: %w", path, err) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we have a default:
to check for weird junk? If we did that, we'd need a case to account for tailscaled.service, even if just to say "Yes, we got it."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That would cause updates to fail if we add a new file to the tarball in a new release.
Without default
we have some forward-compatibility :)
a67d7cb
to
c343542
Compare
18f7b85
to
4636b4c
Compare
4636b4c
to
89c5f8c
Compare
As a fallback to package managers, allow updating tailscale that was self-installed in some way. There are some tricky bits around updating the systemd unit (should we stick to local binary paths or to the ones in tailscaled.service?), so leaving that out for now. Updates #6995 Signed-off-by: Andrew Lytvynov <awly@tailscale.com>
89c5f8c
to
1f6e075
Compare
As a fallback to package managers, allow updating tailscale that was self-installed in some way. There are some tricky bits around updating the systemd unit (should we stick to local binary paths or to the ones in tailscaled.service?), so leaving that out for now. Updates tailscale#6995 Signed-off-by: Andrew Lytvynov <awly@tailscale.com> Signed-off-by: Alex Paguis <alex@windscribe.com>
As a fallback to package managers, allow updating tailscale that was self-installed in some way. There are some tricky bits around updating the systemd unit (should we stick to local binary paths or to the ones in tailscaled.service?), so leaving that out for now.
Updates #6995