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
Disable placement timeout with no peers #7418
Conversation
Disable placement leadership timeouts when there are no peers. This reduces startup time of placement by a second. Also remove initial delay of actor health check probe. Continue to use existing placement config if there is more then one peer (initial cluster) configured i.e. ha mode. `-focus "actor|placement" integration test results before and after change: ``` integration.go:82: Total integration test execution time: 2m2.7s integration.go:82: Total integration test execution time: 2m5.9s integration.go:82: Total integration test execution time: 1m36.3s integration.go:82: Total integration test execution time: 1m35s `` Signed-off-by: joshvanl <me@joshvanl.dev>
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #7418 +/- ##
==========================================
- Coverage 62.40% 62.36% -0.04%
==========================================
Files 244 244
Lines 22134 22139 +5
==========================================
- Hits 13812 13807 -5
- Misses 7181 7195 +14
+ Partials 1141 1137 -4 ☔ View full report in Codecov by Sentry. |
/test-version-skew |
Dapr Version Skew e2e test (dapr-sidecar-master - 1.12.4)Commit ref: c94bdde ✅ Version Skew tests passed |
Dapr Version Skew integration test (dapr-sidecar-master - 1.12.4)Commit ref: c94bdde ✅ Version Skew tests passed |
Dapr Version Skew e2e test (control-plane-master - 1.12.4)Commit ref: c94bdde ✅ Version Skew tests passed |
Dapr Version Skew integration test (control-plane-master - 1.12.4)Commit ref: c94bdde ✅ Version Skew tests passed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work Josh
Disable placement leadership timeouts when there are no peers. This reduces startup time of placement by a second. Also remove initial delay of actor health check probe.
Continue to use existing placement config if there is more then one peer (initial cluster) configured i.e. ha mode.
`-focus "actor|placement" integration test results before and after change: