Skip to content

Convert discovery config to new cache mechanism#54568

Merged
rosstimothy merged 1 commit intomasterfrom
tross/cache_discovery_config
May 9, 2025
Merged

Convert discovery config to new cache mechanism#54568
rosstimothy merged 1 commit intomasterfrom
tross/cache_discovery_config

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

Moves discovery configs to the new cache collection scheme that was introduced in #52210. No additional functionality changes have been made here. This should be a purely mechanical translation to the new internal caching machinery.

@rosstimothy rosstimothy added the no-changelog Indicates that a PR does not require a changelog entry label May 6, 2025
@rosstimothy rosstimothy marked this pull request as ready for review May 6, 2025 20:47
@github-actions github-actions Bot requested a review from flyinghermit May 6, 2025 20:47
Comment thread api/types/discoveryconfig/discoveryconfig.go
@rosstimothy rosstimothy force-pushed the tross/cache_discovery_config branch 2 times, most recently from 27778fd to 73bfe65 Compare May 9, 2025 15:01
Comment thread lib/cache/cache_test.go Outdated
Comment thread api/types/discoveryconfig/discoveryconfig.go
@rosstimothy rosstimothy requested a review from espadolini May 9, 2025 17:28
Comment thread api/types/discoveryconfig/discoveryconfig.go
@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from flyinghermit May 9, 2025 18:15
Moves discovery configs to the new cache collection scheme that
was introduced in #52210. No additional functionality changes have
been made here. This should be a purely mechanical translation to
the new internal caching machinery.
@rosstimothy rosstimothy force-pushed the tross/cache_discovery_config branch from 7f75e1b to 4f88eef Compare May 9, 2025 18:21
@rosstimothy rosstimothy enabled auto-merge May 9, 2025 18:21
@rosstimothy rosstimothy added this pull request to the merge queue May 9, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch May 9, 2025
@rosstimothy rosstimothy added this pull request to the merge queue May 9, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 9, 2025
@rosstimothy rosstimothy added this pull request to the merge queue May 9, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 9, 2025
@rosstimothy rosstimothy added this pull request to the merge queue May 9, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 9, 2025
@rosstimothy rosstimothy added this pull request to the merge queue May 9, 2025
Merged via the queue into master with commit 5177048 May 9, 2025
41 checks passed
@rosstimothy rosstimothy deleted the tross/cache_discovery_config branch May 9, 2025 20:45
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/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants