[Deepsea-users] DeepSea 0.7.6
Joao Eduardo Luis
joao at suse.de
Sun Apr 23 07:50:55 MDT 2017
On 04/22/2017 02:43 PM, Eric Jackson wrote:
> Hello everyone,
> DeepSea 0.7.6 has been released. The notable feature is the rolling
> upgrade. With a running Ceph cluster, an admin can gracefully upgrade the OS,
> Salt and Ceph. See my previous email about specifics. The CHANGELOG is listed
> - Rolling upgrade
I must say I'm a bit surprised seeing this today, just a single day
after you gave a "heads up about a large PR to implement the rolling
Has this behemoth of a PR, with "81 commits and 55 files [...] Several
of these [... being ...] new" been properly peer-reviewed and discussed?
I ask this because upon checking the PR on github, as well as the merge
commit, I see no discussion or reviews/Reviewed-by.
It feels strange seeing such a large PR, with roughly 1.4k added lines,
to be announced one day and merged the very next; especially without
seeing any sort of involvement from anyone else (beside the authors).
Additionally, I'm inclined to presume there were no other set of eyes on
the PR due to commits such as
which fixes a 'typo', and could have been squashed with the previous commit
(which, in this case, would have reduced the number of commits in the
And I have this feeling that many more like them are out there,
considering some of the commit messages.
More information about the Deepsea-users