-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Adding automatic migration path for existing static host users #46804
Conversation
@@ -1009,6 +1009,9 @@ type HostUsersInfo struct { | |||
GID string | |||
// Shell is the default login shell for a host user | |||
Shell string | |||
// TakeOwnership determines whether or not an existing user should be | |||
// taken over by teleport | |||
TakeOwnership bool |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should eventually support this on the role spec as well and consider removing the implicit migration when teleport-keep
is present in a role's host_groups
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the moment does this only apply to static host users? If so, let's note that in the comment for this field to set expectations.
de90e7d
to
ed73366
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remember to follow up with a docs PR so that this feature is discoverable by users.
@@ -1009,6 +1009,9 @@ type HostUsersInfo struct { | |||
GID string | |||
// Shell is the default login shell for a host user | |||
Shell string | |||
// TakeOwnership determines whether or not an existing user should be | |||
// taken over by teleport | |||
TakeOwnership bool |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the moment does this only apply to static host users? If so, let's note that in the comment for this field to set expectations.
ed73366
to
8095db5
Compare
8095db5
to
b99fa8e
Compare
Closes #46803
Adding
take_ownership_if_user_exists
flag to static host user resource to allow for automatic migration/import of unmanaged users with colliding names. When set totrue
, host user creation will add theteleport-static
group to the existing user along with any groups defined on the SHU resource.changelog: Added a new flag to static host users spec that allows teleport to automatically take ownership across matching hosts of any users with the same name as the static host user.