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

Add static host user service #44778

Merged
merged 1 commit into from
Jul 31, 2024
Merged

Add static host user service #44778

merged 1 commit into from
Jul 31, 2024

Conversation

atburke
Copy link
Contributor

@atburke atburke commented Jul 29, 2024

This change adds a local service for the static host user resource (RFD 175).

Part of #42712.

@atburke atburke requested a review from rosstimothy July 29, 2024 22:13
@github-actions github-actions bot requested review from smallinsky and tcsc July 29, 2024 22:13
Copy link

The PR changelog entry failed validation: Changelog entry not found in the PR body. Please add a "no-changelog" label to the PR, or changelog lines starting with changelog: followed by the changelog entries for the PR.

lib/services/statichostuser.go Outdated Show resolved Hide resolved
lib/services/statichostuser_test.go Show resolved Hide resolved
@atburke atburke added the no-changelog Indicates that a PR does not require a changelog entry label Jul 29, 2024
lib/services/local/statichostuser.go Outdated Show resolved Hide resolved
@atburke atburke force-pushed the atburke/static-user-proto branch from fa529da to a134b2d Compare July 30, 2024 16:15
Base automatically changed from atburke/static-user-proto to master July 30, 2024 16:49
Copy link
Contributor

@rosstimothy rosstimothy left a comment

Choose a reason for hiding this comment

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

Looks like something got messed up after your other PR with the proto change was merged as they are included here as well.

lib/services/local/statichostuser_test.go Outdated Show resolved Hide resolved
lib/services/local/statichostuser_test.go Outdated Show resolved Hide resolved
lib/services/local/statichostuser_test.go Outdated Show resolved Hide resolved
lib/services/local/statichostuser_test.go Outdated Show resolved Hide resolved
@atburke atburke force-pushed the atburke/static-user-service branch from 986b994 to 83688e5 Compare July 30, 2024 19:59
@atburke
Copy link
Contributor Author

atburke commented Jul 30, 2024

Looks like something got messed up after your other PR with the proto change was merged as they are included here as well.

I think I forgot to run make grpc one last time in that PR. The proto code should be fixed here now.

assert: require.Error,
},
{
name: "invalid node labels",
Copy link
Contributor

Choose a reason for hiding this comment

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

A label of *:* should be allowed right? Can we add a test case to cover it?

@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from tcsc July 31, 2024 02:10
This change adds a local service for the static host user resource.
@atburke atburke force-pushed the atburke/static-user-service branch from 05bea55 to 7e38e84 Compare July 31, 2024 16:42
@atburke atburke enabled auto-merge July 31, 2024 16:42
@atburke atburke added this pull request to the merge queue Jul 31, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 31, 2024
@atburke atburke added this pull request to the merge queue Jul 31, 2024
Merged via the queue into master with commit 3a9b760 Jul 31, 2024
39 checks passed
@atburke atburke deleted the atburke/static-user-service branch July 31, 2024 18:01
atburke added a commit that referenced this pull request Sep 10, 2024
This change adds a local service for the static host user resource.
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