Skip to content

[v17] Fix auto-update re-exec arguments modified by aliases#51154

Merged
vapopov merged 6 commits intobranch/v17from
bot/backport-50228-branch/v17
Jan 17, 2025
Merged

[v17] Fix auto-update re-exec arguments modified by aliases#51154
vapopov merged 6 commits intobranch/v17from
bot/backport-50228-branch/v17

Conversation

@vapopov
Copy link
Copy Markdown
Contributor

@vapopov vapopov commented Jan 17, 2025

Backport #50228 to branch/v17

changelog: Fixed client tools autoupdates executed by aliases (causes recursive alias error)

Added a full-cycle integration test to verify client tools
auto-updates within a test cluster by modifying AutoUpdateConfig
and AutoUpdateVersion resources. The test executes the login
command using alias configurations to ensure no recursive
re-execution occurs.

The updater binary used in integration tests has been replaced
with the `Run` logic of tctl and tsh.
In case of double re-execution case we should stop second one to prevent loop re-execution
Drop localDir set during compilation
@github-actions github-actions Bot added backport size/md tctl tctl - Teleport admin tool tsh tsh - Teleport's command line tool for logging into nodes running Teleport. labels Jan 17, 2025
Copy link
Copy Markdown
Contributor

@Tener Tener left a comment

Choose a reason for hiding this comment

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

nit: the commits are unverified for some reason? Probably a configuration issue somewhere.

@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from sclevine January 17, 2025 16:03
@vapopov
Copy link
Copy Markdown
Contributor Author

vapopov commented Jan 17, 2025

nit: the commits are unverified for some reason? Probably a configuration issue somewhere.

@Tener this is configuration of my github account, to show commits not signed by me 🙂 they were created by backport bot

@vapopov vapopov added this pull request to the merge queue Jan 17, 2025
Merged via the queue into branch/v17 with commit b01e6d0 Jan 17, 2025
@vapopov vapopov deleted the bot/backport-50228-branch/v17 branch January 17, 2025 16:28
@camscale camscale mentioned this pull request Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport size/md tctl tctl - Teleport admin tool tsh tsh - Teleport's command line tool for logging into nodes running Teleport.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants