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

Upgrade k8ssandra-client to fix cassandra.yaml cluster name setting issue when running 4.1+ #1114

Merged
merged 2 commits into from
Nov 14, 2023

Conversation

adejanovski
Copy link
Contributor

What this PR does:
Updates the version of k8ssandra-client to v0.2.2 which has a fix for the cluster name issue with Cassandra 4.1+ (the name "Test Cluster" was always used).

Since this was creating issues with Reaper (as reported in #1112), one of the Reaper e2e test was modified to run against Cassandra 4.1.2.
Another tiny bug was fixed here as well: Running CreateSingleReaper would run both CreateSingleReaper and CreateSingleReaperWStargateAndHTTP e2e tests since they begin with the same string. That makes CreateSingleReaperWStargateAndHTTP run twice since it's also ran separately.
Renaming CreateSingleReaper to CreateSingleReaperNoHttp fixes the issues.

Which issue(s) this PR fixes:
Fixes #1110
Fixes #1112

Checklist

  • Changes manually tested
  • Automated Tests added/updated
  • Documentation added/updated
  • CHANGELOG.md updated (not required for documentation PRs)
  • CLA Signed: DataStax CLA

@adejanovski adejanovski requested a review from a team as a code owner November 13, 2023 14:58
Copy link

No linked issues found. Please add the corresponding issues in the pull request description.
Use GitHub automation to close the issue when a PR is merged

@adejanovski adejanovski force-pushed the upgrade-k8ssandra-client branch from db4b8e1 to 8b89df8 Compare November 13, 2023 14:59
Copy link

codecov bot commented Nov 13, 2023

Codecov Report

Merging #1114 (9461592) into release/1.10 (6d4a095) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff                @@
##           release/1.10    #1114      +/-   ##
================================================
- Coverage         57.40%   57.39%   -0.01%     
================================================
  Files               100      100              
  Lines             10193    10193              
================================================
- Hits               5851     5850       -1     
- Misses             3832     3833       +1     
  Partials            510      510              

see 2 files with indirect coverage changes

@adejanovski adejanovski force-pushed the upgrade-k8ssandra-client branch from 8b89df8 to 9461592 Compare November 13, 2023 16:00
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@adejanovski adejanovski merged commit 194d0a4 into release/1.10 Nov 14, 2023
58 of 60 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants