Skip to content

add system.forceNamespaceCacheRefreshOnRead#10565

Draft
stephanos wants to merge 1 commit into
mainfrom
stephanos/remove-namespace-registry-refs
Draft

add system.forceNamespaceCacheRefreshOnRead#10565
stephanos wants to merge 1 commit into
mainfrom
stephanos/remove-namespace-registry-refs

Conversation

@stephanos
Copy link
Copy Markdown
Contributor

@stephanos stephanos commented Jun 5, 2026

What changed

XDC namespace setup now waits on namespace state through DescribeNamespace calls instead of reaching into each onebox host's namespace registries.

Why?

Removes test-specific onebox code.

@stephanos stephanos force-pushed the stephanos/remove-namespace-registry-refs branch 3 times, most recently from 1c14415 to 944e8d9 Compare June 5, 2026 23:50
@stephanos stephanos changed the title Remove XDC namespace registry access add system.forceNamespaceCacheRefreshOnRead Jun 5, 2026
@stephanos stephanos force-pushed the stephanos/remove-namespace-registry-refs branch 2 times, most recently from 485a0cc to d84ce0b Compare June 6, 2026 00:10
Comment thread tests/testcore/onebox.go
fxApps []*fx.App

// This is used to wait for namespace registries to have noticed a change in some xdc tests.
namespaceRegistries []namespace.Registry
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Removing this is the goal of the PR

Comment thread tests/xdc/base.go
s.waitForClusterSynced()
}

func (s *xdcBaseSuite) describeNamespace(
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

once the frontend has the namespace from persistence, we know that all services do due to the "force refresh"

@stephanos stephanos force-pushed the stephanos/remove-namespace-registry-refs branch 2 times, most recently from 3607062 to 8a633c7 Compare June 6, 2026 00:15
@stephanos stephanos force-pushed the stephanos/remove-namespace-registry-refs branch from 8a633c7 to bbc8283 Compare June 6, 2026 00:19
@stephanos stephanos mentioned this pull request Jun 6, 2026
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant