-
Notifications
You must be signed in to change notification settings - Fork 228
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
Import raft #568
Import raft #568
Conversation
2961582
to
d369229
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #568 +/- ##
===========================================
+ Coverage 61.30% 81.17% +19.87%
===========================================
Files 34 187 +153
Lines 6836 26683 +19847
Branches 2031 4865 +2834
===========================================
+ Hits 4191 21661 +17470
- Misses 1349 3478 +2129
- Partials 1296 1544 +248 ☔ View full report in Codecov by Sentry. |
bd2e01c
to
c097b03
Compare
I've got the clang-tidy check mostly working for patches. For some reason the identifier case check doesn't work for type names (https://discourse.llvm.org/t/clang-tidying-c-code-structcase-not-working/76863), but otherwise it seems to run as expected. |
9e052c8
to
d3dca96
Compare
Signed-off-by: Cole Miller <cole.miller@canonical.com>
Signed-off-by: Cole Miller <cole.miller@canonical.com>
Signed-off-by: Cole Miller <cole.miller@canonical.com>
Signed-off-by: Cole Miller <cole.miller@canonical.com>
Signed-off-by: Cole Miller <cole.miller@canonical.com>
Signed-off-by: Cole Miller <cole.miller@canonical.com>
11cbc6a
to
f99d254
Compare
I have been unable to reproduce that test failure locally and it doesn't seem to be recurring in CI after a few re-runs. I propose we merge this, see if it comes back, and fix the test in a follow-up if it turns flaky again. |
Hey @cole-miller now that this PR is merged, any plan to cut a new dqlite release soon? So distributions can start update their packages to the new layout. Thanks. |
canonical/dqlite#568 Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
canonical/dqlite#568 Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
canonical/dqlite#568 Signed-off-by: Simon Deziel <simon.deziel@canonical.com>
canonical/dqlite#568 Signed-off-by: Simon Deziel <simon.deziel@canonical.com> (cherry picked from commit 3816410)
canonical/dqlite#568 Signed-off-by: Simon Deziel <simon.deziel@canonical.com> (cherry picked from commit 3816410)
canonical/dqlite#568 Signed-off-by: Simon Deziel <simon.deziel@canonical.com> (cherry picked from commit 3816410)
canonical/dqlite#568 Signed-off-by: Simon Deziel <simon.deziel@canonical.com> (cherry picked from commit 3816410)
canonical/dqlite#568 Signed-off-by: Simon Deziel <simon.deziel@canonical.com> (cherry picked from commit 3816410)
canonical/dqlite#568 Signed-off-by: Simon Deziel <simon.deziel@canonical.com> (cherry picked from commit 3816410)
canonical/dqlite#568 Signed-off-by: Simon Deziel <simon.deziel@canonical.com> (cherry picked from commit 3816410)
canonical/dqlite#568 Signed-off-by: Simon Deziel <simon.deziel@canonical.com> (cherry picked from commit 3816410)
canonical/dqlite#568 Signed-off-by: Simon Deziel <simon.deziel@canonical.com> (cherry picked from commit 3816410)
canonical/dqlite#568 Signed-off-by: Simon Deziel <simon.deziel@canonical.com> (cherry picked from commit 3816410)
canonical/dqlite#568 Signed-off-by: Simon Deziel <simon.deziel@canonical.com> (cherry picked from commit 3816410)
canonical/dqlite#568 Signed-off-by: Simon Deziel <simon.deziel@canonical.com> (cherry picked from commit 3816410)
canonical/dqlite#568 Signed-off-by: Simon Deziel <simon.deziel@canonical.com> (cherry picked from commit 3816410)
canonical/dqlite#568 Signed-off-by: Simon Deziel <simon.deziel@canonical.com> (cherry picked from commit 3816410)
Still WIP but this is already enough to get the dqlite tests passing (on my machine, both with bundled raft and external raft). Major points:
Still todo: