scylladb / scylla-ansible-roles

Ansible roles for deploying and managing Scylla, Scylla-Manager and Scylla-Monitoring
44 stars 38 forks source link

scylla-node: don't upgrade already installed scylla unless explicitly requested #288

Closed vladzcloudius closed 1 year ago

vladzcloudius commented 1 year ago

Guys, please, don't merge yet. I'll do it myself.

tarzanek commented 1 year ago

offtopic: let's write the proper (private build) version to metadata/inventory to error out, too

vladzcloudius commented 1 year ago

We brainstormed with Lubos and it was decided to rewrite this PR completely. The idea is to implement the following logic:

1) The flow in question will never upgrade/downgrade the already installed scylla version. 2) The Role by default will error out if the installed scylla version is different from the one specified in scylla_version. 3) The Role will always install the scylla version specified in scylla_version on the node that doesn't have any scylla installed. 4) There will be a flag that would allow disabling the check (2).