Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(kds): fix retry on NACK and add backoff (backport of #9736) #9859

Closed
wants to merge 1 commit into from

Conversation

kumahq[bot]
Copy link
Contributor

@kumahq kumahq bot commented Apr 8, 2024

Automatic cherry-pick of #9736 for branch release-2.3

Generated by action

cherry-picked commit 4752f7b
⚠️ ⚠️ ⚠️ Conflicts happened when cherry-picking! ⚠️ ⚠️ ⚠️

On branch release-2.3
Your branch is up to date with 'origin/release-2.3'.

You are currently cherry-picking commit 4752f7b82.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	new file:   pkg/kds/v2/reconcile/reconcile_suite_test.go
	new file:   pkg/kds/v2/reconcile/reconciler_test.go
	modified:   pkg/kds/v2/server/components.go

Unmerged paths:
  (use "git add/rm <file>..." as appropriate to mark resolution)
	both modified:   pkg/events/interfaces.go
	both modified:   pkg/kds/v2/reconcile/interfaces.go
	both modified:   pkg/kds/v2/reconcile/reconciler.go
	deleted by us:   pkg/kds/v2/server/event_based_watchdog.go
	deleted by us:   pkg/kds/v2/server/event_based_watchdog_test.go
	both modified:   pkg/kds/v2/server/resource_retry_forcer.go
	both modified:   test/framework/envs/multizone/env.go

Signed-off-by: slonka <[email protected]>
Signed-off-by: Jakub Dyszkiewicz <[email protected]>
Co-authored-by: Jakub Dyszkiewicz <[email protected]>
@jakubdyszkiewicz
Copy link
Contributor

delta not default. Closing

@lahabana lahabana deleted the chore/backport-release-2.3-9736 branch September 20, 2024 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants