Skip to content

Add static host user cache#45573

Merged
atburke merged 1 commit intomasterfrom
atburke/static-user-cache
Aug 19, 2024
Merged

Add static host user cache#45573
atburke merged 1 commit intomasterfrom
atburke/static-user-cache

Conversation

@atburke
Copy link
Copy Markdown
Contributor

@atburke atburke commented Aug 16, 2024

This change adds cache support for static host users.

Part of #42712.

@atburke atburke added no-changelog Indicates that a PR does not require a changelog entry backport/branch/v16 labels Aug 16, 2024
@atburke atburke requested a review from rosstimothy August 16, 2024 22:22
Comment thread lib/auth/statichostuser/service.go
Comment thread lib/auth/statichostuser/service.go
Comment thread lib/cache/collections.go Outdated
Comment thread lib/services/local/statichostuser.go Outdated
Comment thread lib/cache/collections.go Outdated
collections.byKind[resourceKind] = collections.kubeWaitingContainers
case types.KindStaticHostUser:
if c.StaticHostUsers == nil {
return nil, trace.BadParameter("mising parameter StaticHostUsers")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
return nil, trace.BadParameter("mising parameter StaticHostUsers")
return nil, trace.BadParameter("missing parameter StaticHostUsers")

@rosstimothy
Copy link
Copy Markdown
Contributor

It looks like TestNodeCAFiltering needs to be updated to populate the static host users service.

@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from fspmarshall August 19, 2024 17:26
This change adds cache support for static host users.
@atburke atburke force-pushed the atburke/static-user-cache branch from bfc288c to 2510cb7 Compare August 19, 2024 17:49
@atburke atburke enabled auto-merge August 19, 2024 17:49
@atburke atburke added this pull request to the merge queue Aug 19, 2024
Merged via the queue into master with commit ca7e49b Aug 19, 2024
@atburke atburke deleted the atburke/static-user-cache branch August 19, 2024 19:08
@public-teleport-github-review-bot
Copy link
Copy Markdown

@atburke See the table below for backport results.

Branch Result
branch/v16 Failed

atburke added a commit that referenced this pull request Sep 10, 2024
This change adds cache support for static host users.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Indicates that a PR does not require a changelog entry size/md

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants