Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

Upgrade C* to 3.11.5, add integration tests #29

Merged
merged 17 commits into from
Dec 11, 2019
Merged

Conversation

mpereira
Copy link
Contributor

  • 2dc8c4d: Make params.yaml a template (it should have been one already)
  • 821ce5f: Upgrade C* to 3.11.5
  • 51521cb: Add integration test for upgrade
  • 2ffc70c: Create function to return a C* cluster nodes
  • ec47331: Assert the C* has the expected number of UN nodes all around

This supersedes #17 and #22.

@mpereira mpereira requested review from viivek46 and zmalik and removed request for viivek46 December 10, 2019 19:49
@mpereira
Copy link
Contributor Author

mpereira commented Dec 10, 2019

To do: add "Unreleased" entry to the CHANGELOG.

Edit: done.

Copy link
Contributor

@zmalik zmalik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!
leaving a comment around empty vs nil for parameters value

operator/templates/cassandra-yaml.yaml Show resolved Hide resolved
@mpereira mpereira merged commit 1dbc984 into master Dec 11, 2019
@mpereira mpereira deleted the upgrade-3.11.5 branch December 11, 2019 16:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants