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 deployment to cass-operator v1.20.0 #1307

Merged
merged 3 commits into from
May 6, 2024

Conversation

burmanm
Copy link
Contributor

@burmanm burmanm commented May 6, 2024

What this PR does:
Update to cass-operator v1.20.0

Which issue(s) this PR fixes:
Fixes #

Checklist

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

@burmanm burmanm requested a review from a team as a code owner May 6, 2024 12:47
Copy link

github-actions bot commented May 6, 2024

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

- github.com/k8ssandra/cass-operator/config/deployments/default?ref=v1.19.1

components:
- ../../components/cass-operator-image-config
Copy link
Contributor

Choose a reason for hiding this comment

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

Question: Why was this removed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because it has no purpose?

@@ -3,7 +3,7 @@ kind: ImageConfig
metadata:
name: image-config
images:
system-logger: "k8ssandra/system-logger:v1.19.1"
system-logger: "k8ssandra/system-logger:v1.20.0"
config-builder: "datastax/cass-config-builder:1.0.7"
k8ssandra-client: "k8ssandra/k8ssandra-client:v0.2.2"
Copy link
Contributor

Choose a reason for hiding this comment

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

Todo: shouldn't k8ssandra-client be upgraded to v0.3.0?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As this file isn't used, not really. That 0.3.0 comes when cass-operator is used directly.

@burmanm burmanm force-pushed the cass_operator_v1200 branch from f92d958 to 27db189 Compare May 6, 2024 14:24
@burmanm burmanm force-pushed the cass_operator_v1200 branch from a38de1a to 3fd180c Compare May 6, 2024 16:12
Copy link

sonarqubecloud bot commented May 6, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
8.5% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@adejanovski adejanovski merged commit 7486e18 into k8ssandra:main May 6, 2024
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
Development

Successfully merging this pull request may close these issues.

2 participants