Skip to content

Fix nexus forwarding criteria#9182

Merged
xwduan merged 1 commit intomainfrom
will/fix_nexus_redirection
Jan 31, 2026
Merged

Fix nexus forwarding criteria#9182
xwduan merged 1 commit intomainfrom
will/fix_nexus_redirection

Conversation

@xwduan
Copy link
Copy Markdown
Contributor

@xwduan xwduan commented Jan 30, 2026

What changed?

Fix nexus forwarding criteria

Why?

We don't need this criteria to decide if we should forward. If ActiveCluster is not current cluster, then we can do forward.

How did you test it?

  • built
  • run locally and tested manually
  • covered by existing tests
  • added new unit test(s)
  • added new functional test(s)

Potential risks

no risk.

@xwduan xwduan marked this pull request as ready for review January 30, 2026 22:48
@xwduan xwduan requested review from a team as code owners January 30, 2026 22:48
Copy link
Copy Markdown
Member

@bergundy bergundy left a comment

Choose a reason for hiding this comment

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

I admit to not knowing enough about how global namespace are qualified but I assume @xwduan does.

@xwduan xwduan merged commit 3820c3f into main Jan 31, 2026
65 of 66 checks passed
@xwduan xwduan deleted the will/fix_nexus_redirection branch January 31, 2026 01:13
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.

2 participants