Yaroslav Dynnikov authored 2 years ago and Yaroslav Dynnikov committed 2 years ago

Also make `Default` trait implementation for `struct Instance` public.
Because keeping it private makes no sence for a public struct with
public fields.

Yaroslav Dynnikov authored 2 years ago and Yaroslav Dynnikov committed 2 years ago

"You should consider adding a `Default` implementation for `Mailbox<T>`"

This reverts commit 267dc61b.

It produces false-positive errors in CI.

Yaroslav Dynnikov authored 2 years ago and Yaroslav Dynnikov committed 2 years ago

Cargo used to complain:

```
error: the linking modifiers `+bundle` and `+whole-archive` are not
compatible with each other when generating rlibs
```

Also remove unnecessary subdir

1. Encapsulate printing rustc flags in their own mod.
2. Arrange paths formatting.

1. Remove build dependency on cmake crate. It doesn't fulfill all our
   needs, so we run cmake directly via `std::process::command`. It also
   makes `build.rs` easier to understand.
2. Arrange paths variable names.

Provide a supplementary `Run` trait for the
`std::process::Command`.

We haven't used rustc < 1.61 for a long time.

It's obsolete as we are not going to implement raft_applier fiber
anymore.

The typo was introduced in dd4f93f8.

1. Rename "check-deployment" stage, make it more concise using
   `parallel:matrix` Gitlab CI feature.
2. Ignore `gitlab-ci.yml` changes in `test-docker` job
3. Remove unused credentials from `sign-rpm-packages` job

Make it more alike `find_farthest`:

```rust
sum_distance(all, voters, candidate);
find_farthest(all, voters, candidates);
```

1. Besides `raft_conf` local state introduce `promotable` set of
   instances that are suitable for promotion.

2. Remove an instance from `promotable` set when it's actually promoted.

Dmitry Travyan authored 2 years ago and Yaroslav Dynnikov committed 2 years ago

This commit provides:

1. Building and pushing a docker image for kubernetes.
2. Helm chart for quick and easy installation in any cluster that supports the Kubernetes API.
3. Building and pushing a docker image with diagnostic tools.
4. Short readme about helm chart features.

lazy-static crate is deprecated and is only used in 2 places, where it
can be easily replaced with something simpler.

Georgy Moshkin authored 2 years ago and Georgy Moshkin committed 2 years ago

It's a small library whose benefits don't overweight the drawbacks.
Any additional dependency crate makes the build slower. But itertools in
particular is harmful because it hides some expensive computations
behind simple tempting to use functions such as `sorted_by`.