Alexander Kukushkin

If debug logs are enabled it prints (occasionally) `Retrieved list of machines: %s`.

It was always like that. To be honest, I don't know what would be better, to keep them or remove.

Adding a new parameter would be too much, I think it should be fine to not remove any slots if cluster is paused.

@Dsolik the next one after it is implemented and committed to the master branch.

> Oh dear, does it fail on the trailing white space or something else? Yes: https://github.com/zalando/patroni/runs/7339310338?check_suite_focus=true#step:5:9 ``` running flake8 tests/test_api.py:289:28: W291 trailing whitespace tests/test_api.py:290:83: W291 trailing whitespace ```

> warning on crypto yes, it is unrelated. Please also update docs: https://github.com/zalando/patroni/blob/master/docs/rest_api.rst#health-check-endpoints

@jocutajar now you need to wait a little bit. Even if it is merged to the master branch the next release will not happen immediately.

👍

> maximum_lag_on_replica Patroni on the master publishes its WAL position to the DCS (`/optime/leader`) once per `loop_wait`, it means that the value could be quite outdated. In most cases you...

In general, I agree with @ants, I don't really like an idea of turning Patroni into a monitoring agent which also perform some reactive feedback actions. It is basically your...