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

[v10] Machine ID support for Logins trait (#15117) #15470

Merged
merged 4 commits into from
Aug 15, 2022

Conversation

strideynet
Copy link
Contributor

Backport #15117

* Support user traits in impersonated role certificates

* Add param to tctl bots add for specifying logins

* Update comment on desiredAccessInfoForRoleRequest to be mroe accurate

* Update docs for --logins support with Machine ID

* Rearrange bot tests and name them more in-line with standards

* Add test coverage for Server.createBot

* Test role impersonation preserves user traits

* pass traits directly into desiredAccessInfoForRoleRequests

* Drop underscores from test names

* Remove unnecessary initialization of traits

* Address PR comments

* Update docs/pages/machine-id/getting-started.mdx

Co-authored-by: Paul Gottschling <paul.gottschling@goteleport.com>

* Update docs/pages/machine-id/getting-started.mdx

Co-authored-by: Paul Gottschling <paul.gottschling@goteleport.com>

* Update docs/pages/machine-id/getting-started.mdx

Co-authored-by: Paul Gottschling <paul.gottschling@goteleport.com>

Co-authored-by: Paul Gottschling <paul.gottschling@goteleport.com>
@strideynet strideynet changed the title [v10]Machine ID support for Logins trait (#15117) [v10] Machine ID support for Logins trait (#15117) Aug 12, 2022
@github-actions github-actions bot added backport documentation tctl tctl - Teleport admin tool labels Aug 12, 2022
@strideynet strideynet enabled auto-merge (squash) August 12, 2022 16:06
@strideynet strideynet merged commit 6b89f8e into branch/v10 Aug 15, 2022
@strideynet strideynet deleted the strideynet/branch/v10/backport-15117 branch August 15, 2022 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport documentation tctl tctl - Teleport admin tool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants