Skip to content

[v16] Add template for client tools auto-update download url#51478

Merged
vapopov merged 1 commit intobranch/v16from
vapopov/client-tools-update-uri-template-v16
Jan 28, 2025
Merged

[v16] Add template for client tools auto-update download url#51478
vapopov merged 1 commit intobranch/v16from
vapopov/client-tools-update-uri-template-v16

Conversation

@vapopov
Copy link
Copy Markdown
Contributor

@vapopov vapopov commented Jan 24, 2025

Backport #51210 to branch/v16

changelog: Added support for customizing the base URL for downloading Teleport packages used in client tools managed updates

* Add templates for client tools auto-update download url

* Change to base url setting by env

MakeURL moved to common function to be general for both, agent and client tools

* Add godoc for common function and constant for default package

* Use flags and version arguments instead of revision

* Move base url env to shared constant
@vapopov vapopov added backport no-changelog Indicates that a PR does not require a changelog entry labels Jan 24, 2025
Copy link
Copy Markdown
Member

@sclevine sclevine left a comment

Choose a reason for hiding this comment

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

Is there a reason this backport has no-changelog, but the v15 and v17 backports do not?

@vapopov vapopov removed the no-changelog Indicates that a PR does not require a changelog entry label Jan 25, 2025
@vapopov
Copy link
Copy Markdown
Contributor Author

vapopov commented Jan 25, 2025

@sclevine muscle memory 🙃 I've removed label, thanks for noticing

@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from timothyb89 January 27, 2025 21:16
@vapopov vapopov added this pull request to the merge queue Jan 27, 2025
Merged via the queue into branch/v16 with commit 783084c Jan 28, 2025
@vapopov vapopov deleted the vapopov/client-tools-update-uri-template-v16 branch January 28, 2025 00:09
@camscale camscale mentioned this pull request Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants