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

Convert SetAuthPreferences usages to new Create/Update/Upsert API #38895

Merged
merged 1 commit into from
Mar 14, 2024

Conversation

rosstimothy
Copy link
Contributor

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

@rosstimothy rosstimothy added the no-changelog Indicates that a PR does not require a changelog entry label Mar 1, 2024
Base automatically changed from tross/cfg_service to master March 5, 2024 17:22
@rosstimothy rosstimothy force-pushed the tross/use_cfg_service branch 5 times, most recently from c5adf99 to 340ee4b Compare March 11, 2024 20:09
@rosstimothy rosstimothy marked this pull request as ready for review March 11, 2024 20:27
@github-actions github-actions bot added database-access Database access related issues and PRs size/md tctl tctl - Teleport admin tool tsh tsh - Teleport's command line tool for logging into nodes running Teleport. labels Mar 11, 2024
@rosstimothy rosstimothy force-pushed the tross/use_cfg_service branch 4 times, most recently from 49cd5b7 to d183db4 Compare March 14, 2024 20:34
Migrates callers off the deprecated API and onto the new API so that
AuthPreferences can be protected via optimistic locking when appropriate.
@rosstimothy
Copy link
Contributor Author

PTAL @fspmarshall @mdwn

@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from mdwn March 14, 2024 20:50
@zmb3
Copy link
Collaborator

zmb3 commented Mar 14, 2024

/excludeflake *

@rosstimothy rosstimothy added this pull request to the merge queue Mar 14, 2024
Merged via the queue into master with commit 20888bf Mar 14, 2024
37 checks passed
@rosstimothy rosstimothy deleted the tross/use_cfg_service branch March 14, 2024 22:22
rosstimothy added a commit that referenced this pull request Mar 25, 2024
…8895)

Migrates callers off the deprecated API and onto the new API so that
AuthPreferences can be protected via optimistic locking when appropriate.
rosstimothy added a commit that referenced this pull request Mar 25, 2024
…8895)

Migrates callers off the deprecated API and onto the new API so that
AuthPreferences can be protected via optimistic locking when appropriate.
rosstimothy added a commit that referenced this pull request Mar 25, 2024
…8895)

Migrates callers off the deprecated API and onto the new API so that
AuthPreferences can be protected via optimistic locking when appropriate.
rosstimothy added a commit that referenced this pull request Mar 27, 2024
…8895)

Migrates callers off the deprecated API and onto the new API so that
AuthPreferences can be protected via optimistic locking when appropriate.
github-merge-queue bot pushed a commit that referenced this pull request Mar 27, 2024
…ngConfig, SessionRecordingConfig (#39785)

* Register cluster configuration service (#38734)

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.

* Convert SetClusterNetworkingConfig usages to new Create/Update/Upsert API (#39219)

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

* Convert SetAuthPreferences usages to new Create/Update/Upsert API (#38895)

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

* Convert SetSessionRecordingConfig usages to new Create/Update/Upsert API (#39249)

Migrates callers off the deprecated API and onto the new API so
that SessionRecordingConfig can be protected via optimistic locking
when appropriate.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
database-access Database access related issues and PRs no-changelog Indicates that a PR does not require a changelog entry size/md 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

4 participants