Skip to content

Convert remote cluster resources to new cache mechanism#54502

Merged
rosstimothy merged 1 commit intomasterfrom
tross/cache_remote_clusters
May 6, 2025
Merged

Convert remote cluster resources to new cache mechanism#54502
rosstimothy merged 1 commit intomasterfrom
tross/cache_remote_clusters

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

Moves tunnel connections and remote clusters 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 3, 2025
@rosstimothy rosstimothy force-pushed the tross/cache_remote_clusters branch from 97273eb to 33d149a Compare May 3, 2025 21:12
@rosstimothy rosstimothy marked this pull request as ready for review May 3, 2025 21:21
@rosstimothy rosstimothy requested a review from fspmarshall May 3, 2025 21:21
@github-actions github-actions Bot requested review from Joerger and espadolini May 3, 2025 21:22
Copy link
Copy Markdown
Contributor

@espadolini espadolini left a comment

Choose a reason for hiding this comment

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

Wrong base branch?

@rosstimothy
Copy link
Copy Markdown
Contributor Author

Wrong base branch?

It could be based on master, but I tried to chain the last few migration PRs to reduce merge conflicts.

@rosstimothy rosstimothy force-pushed the tross/cache_locks branch 3 times, most recently from 9ceb699 to 46a2f72 Compare May 6, 2025 14:01
@rosstimothy rosstimothy force-pushed the tross/cache_remote_clusters branch from 33d149a to f040b94 Compare May 6, 2025 14:06
Base automatically changed from tross/cache_locks to master May 6, 2025 14:42
Moves tunnel connections and remote clusters 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_remote_clusters branch from f040b94 to 7f673f8 Compare May 6, 2025 14:46
@rosstimothy rosstimothy enabled auto-merge May 6, 2025 14:47
@rosstimothy rosstimothy added this pull request to the merge queue May 6, 2025
Merged via the queue into master with commit e92f360 May 6, 2025
40 checks passed
@rosstimothy rosstimothy deleted the tross/cache_remote_clusters branch May 6, 2025 15:27
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