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

[v9] fix tctl auth server flag #16260

Merged
merged 3 commits into from Sep 8, 2022

Conversation

GavinFrazar
Copy link
Contributor

Backports #16130

GavinFrazar and others added 3 commits September 8, 2022 13:14
* Fix flaky TestHandlerConnectionUpgrade

* Fix tctl --auth-server with logged in profile

* Export tctl applyConfig so we can test it from tsh/tctl tests

* Add test for tctl --auth-server w/ profile

* Update tool/tctl/common/tctl.go

Co-authored-by: Alan Parra <alan.parra@goteleport.com>

* Update tool/tsh/tctl_test.go

Co-authored-by: Alan Parra <alan.parra@goteleport.com>

* Alan's suggestion to not use new()

* Add message to require.NotEmpty

Co-authored-by: STeve Huang <xin.huang@goteleport.com>
Co-authored-by: Alan Parra <alan.parra@goteleport.com>
@GavinFrazar GavinFrazar enabled auto-merge (squash) September 8, 2022 21:46
@GavinFrazar GavinFrazar changed the title Gavinfrazar/v9 fix tctl auth server flag [v9] fix tctl auth server flag Sep 8, 2022
@github-actions github-actions bot added backport tctl tctl - Teleport admin tool tsh tsh - Teleport's command line tool for logging into nodes running Teleport. labels Sep 8, 2022
@GavinFrazar GavinFrazar merged commit 746b239 into branch/v9 Sep 8, 2022
@GavinFrazar GavinFrazar deleted the gavinfrazar/v9_fix_tctl_auth_server_flag branch September 8, 2022 23:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 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.

None yet

3 participants