-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Add option to allow for host users not to be deleted #26892
Conversation
f3fcffc
to
e86423d
Compare
908e4cc
to
19916ad
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.
First pass. Could you fix CI and UT build. Also wonder if you can share any reference for this change ?
19916ad
to
dde1459
Compare
Theres an RFD update here with some information #27074 |
816e411
to
5897ffb
Compare
8ef4c68
to
b620787
Compare
@Joerger @gabrielcorado @smallinsky Can you guys review this? |
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.
Sorry I had stashed uncommitted review.
Mostly LGT though I would love to see UT coverage for Marshal/Unmarshal function.
32f61ba
to
745ce36
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.
LGTM, just some nit-picks
This adds a new role option called create_host_users_mode which allows for it to be configured to not delete users when a session ends. The old `create_host_user` option will be deprecated with this. If the deprecated option is set, the new mode option will default to dropping users as is the current behavior.
bceb7ea
to
6f604b2
Compare
This adds a new role option called create_host_users_mode which allows for it to be configured to not delete users when a session ends. The old
create_host_user
option will be deprecated with this and if the create_host_user_mode option is set it'll always be preffered overcreate_host_user
.If the deprecated option is set, the new mode option will default to dropping users as is the current behavior.