Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add phase to wait for Cassandra to be ready before scale down #2625

Merged
merged 6 commits into from
Jan 25, 2024

Conversation

PrasadG193
Copy link
Contributor

Change Overview

In the integration tests, we have noticed an issue with the Cassandra BP where the Cassandra DB connection was failing after restore.
This PR updates the Blueprint to fix the Cassandra connection issue by introducing a new phase that waits for connections to be successful before executing scale down phase.

Pull request type

Please check the type of change your PR introduces:

  • 🚧 Work in Progress
  • 🌈 Refactoring (no functional changes, no api changes)
  • 🐹 Trivial/Minor
  • 🐛 Bugfix
  • 🌻 Feature
  • 🗺️ Documentation
  • 🤖 Test

Test Plan

  • 💪 Manual
  • ⚡ Unit test
  • 💚 E2E

Manually varified backup and restore of Cassandra app

Signed-off-by: Prasad Ghangal <prasad.ganghal@veeam.com>
Signed-off-by: Prasad Ghangal <prasad.ganghal@veeam.com>
Signed-off-by: Prasad Ghangal <prasad.ganghal@veeam.com>
Signed-off-by: Prasad Ghangal <prasad.ganghal@veeam.com>
Signed-off-by: Prasad Ghangal <prasad.ganghal@veeam.com>
@infraq infraq added this to In Progress in Kanister Jan 25, 2024
Kanister automation moved this from In Progress to Reviewer approved Jan 25, 2024
@mergify mergify bot merged commit 6293ba1 into master Jan 25, 2024
12 of 14 checks passed
Kanister automation moved this from Reviewer approved to Done Jan 25, 2024
@mergify mergify bot deleted the cassandra-bp-waitforready branch January 25, 2024 08:20
plar pushed a commit to plar/kanister that referenced this pull request Feb 1, 2024
…erio#2625)

* Add wait for ready logic in cassandra BP

Signed-off-by: Prasad Ghangal <prasad.ganghal@veeam.com>

* Set replication factor to 1

Signed-off-by: Prasad Ghangal <prasad.ganghal@veeam.com>

* Add wait for cassandra conn ready logic

Signed-off-by: Prasad Ghangal <prasad.ganghal@veeam.com>

* Update helm value replicaCount

Signed-off-by: Prasad Ghangal <prasad.ganghal@veeam.com>

* Add nodetool scrub step in waitforready

Signed-off-by: Prasad Ghangal <prasad.ganghal@veeam.com>

---------

Signed-off-by: Prasad Ghangal <prasad.ganghal@veeam.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants