Skip to content

Retry site flow controller start after acquiring leader lease#2474

Merged
AryanP123 merged 1 commit into
skupperproject:mainfrom
AryanP123:retry-site-flow-controller
Jun 11, 2026
Merged

Retry site flow controller start after acquiring leader lease#2474
AryanP123 merged 1 commit into
skupperproject:mainfrom
AryanP123:retry-site-flow-controller

Conversation

@AryanP123

Copy link
Copy Markdown
Contributor

Fixes #2466

@c-kruse c-kruse left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me! Don't have a great way to validate how it behaves it in an error mode: i suspect this was identified in the first place during some transient error.

@fgiorgetti fgiorgetti left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I couldn't reproduce the original problem, but the retry mechanism seems to be working as expected.

Comment thread internal/kube/adaptor/collector.go Outdated
@AryanP123 AryanP123 force-pushed the retry-site-flow-controller branch from bbdabcd to 3213d9c Compare June 11, 2026 13:42
@AryanP123 AryanP123 merged commit 3822580 into skupperproject:main Jun 11, 2026
2 checks passed
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.

kube-adaptor does not retry site flow controller startup after acquiring leader lease

4 participants