Skip to content

Add tctl support for static host users#45925

Closed
atburke wants to merge 3 commits intomasterfrom
atburke/static-user-tctl
Closed

Add tctl support for static host users#45925
atburke wants to merge 3 commits intomasterfrom
atburke/static-user-tctl

Conversation

@atburke
Copy link
Copy Markdown
Contributor

@atburke atburke commented Aug 27, 2024

This change adds support in tctl for the static host user resource.

It also adds an internal StaticHostUser type to use where possible instead of the proto type. This makes working with the resource less painful for the end user. The new format is identical to role labels.

Original:

# ...
node_labels:
  Values:
    foo:
      Values:
        - bar

New:

# ...
node_labels:
  foo: bar

The new serialization is a breaking change in the backend. However, no part of static host users has been backported, so no one will be affected.

Part of #42712.

Changelog: Added StaticHostUser resource

@atburke atburke requested a review from rosstimothy August 27, 2024 20:17
@github-actions github-actions Bot added size/md tctl tctl - Teleport admin tool labels Aug 27, 2024
@espadolini
Copy link
Copy Markdown
Contributor

If this is only needed for user interaction, do we need to change all of our APIs and the backend serialization format?

How can we make sure we won't forget to update the user representation after changing the API?

Should IaC tooling be using the proto-side representation or the backend-side representation?

@atburke
Copy link
Copy Markdown
Contributor Author

atburke commented Aug 29, 2024

Replacing this with another PR after #46038.

@atburke atburke closed this Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/md tctl tctl - Teleport admin tool

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants