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

[v15] Enforce optimistic locking for AuthPreferences, ClusterNetworkingConfig, SessionRecordingConfig #39785

Merged
merged 4 commits into from Mar 27, 2024

Conversation

rosstimothy
Copy link
Contributor

@rosstimothy rosstimothy force-pushed the tross/backport-cluster-config-api/v15 branch 2 times, most recently from 02b18bc to 6fb0795 Compare March 25, 2024 17:34
@rosstimothy rosstimothy marked this pull request as ready for review March 25, 2024 18:02
@rosstimothy
Copy link
Contributor Author

Will hold off merging this until 15.2

@github-actions github-actions bot added application-access backport database-access Database access related issues and PRs kubernetes-access size/lg tctl tctl - Teleport admin tool tsh tsh - Teleport's command line tool for logging into nodes running Teleport. labels Mar 25, 2024
Updates the auth gRPC server to include the new clusterconfigv1
service. The api client was also updated to allow interacting
with the new service(cluster audit config not included yet). Nothing
has yet been converted to use the new service, that will come in a
follow up.

The RPC service proto was also updated to fix a typo that was
reverted in #38736.
… API (#39219)

Migrates callers off the deprecated API and onto the new API so that
ClusterNetworkingConfig can be protected via optimistic locking when
appropriate.
…8895)

Migrates callers off the deprecated API and onto the new API so that
AuthPreferences can be protected via optimistic locking when appropriate.
…API (#39249)

Migrates callers off the deprecated API and onto the new API so
that SessionRecordingConfig can be protected via optimistic locking
when appropriate.
@rosstimothy rosstimothy force-pushed the tross/backport-cluster-config-api/v15 branch from 6fb0795 to 14e870a Compare March 27, 2024 15:17
@rosstimothy
Copy link
Contributor Author

PTAL @r0mant @gabrielcorado

Also @r0mant could I please get a flaky test bypass?

@r0mant
Copy link
Collaborator

r0mant commented Mar 27, 2024

/excludeflake *

@rosstimothy rosstimothy added this pull request to the merge queue Mar 27, 2024
Merged via the queue into branch/v15 with commit ee79696 Mar 27, 2024
35 checks passed
@rosstimothy rosstimothy deleted the tross/backport-cluster-config-api/v15 branch March 27, 2024 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
application-access backport database-access Database access related issues and PRs kubernetes-access size/lg tctl tctl - Teleport admin tool tsh tsh - Teleport's command line tool for logging into nodes running Teleport.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants