Skip to content

[v11] Fetch ClusterAlerts a single time during login#27112

Merged
rosstimothy merged 1 commit intobranch/v11from
bot/backport-26903-branch/v11
May 30, 2023
Merged

[v11] Fetch ClusterAlerts a single time during login#27112
rosstimothy merged 1 commit intobranch/v11from
bot/backport-26903-branch/v11

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

Backport #26903 to branch/v11

`onLogin` was inadvertently retrieving ClusterAlerts twice by
explicitly retrieving them and by calling `onStatus`, which also gets
the alerts so they are shown when `tsh status` is invoked. The portion
of `onStatus` which prints the profile has been separated into
`outputProfiles` so that `onLogin` may call that directly instead
of `onStatus`.
@rosstimothy rosstimothy force-pushed the bot/backport-26903-branch/v11 branch from 839298f to f0c5259 Compare May 30, 2023 17:37
@rosstimothy rosstimothy marked this pull request as ready for review May 30, 2023 17:49
@rosstimothy rosstimothy enabled auto-merge May 30, 2023 17:49
@github-actions github-actions Bot requested review from Tener and zmb3 May 30, 2023 17:50
@github-actions github-actions Bot added backport size/sm tsh tsh - Teleport's command line tool for logging into nodes running Teleport. labels May 30, 2023
@rosstimothy rosstimothy added this pull request to the merge queue May 30, 2023
Merged via the queue into branch/v11 with commit ff76d15 May 30, 2023
@rosstimothy rosstimothy deleted the bot/backport-26903-branch/v11 branch May 30, 2023 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport size/sm 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