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

Refactor/telemetry types #841

Closed

Conversation

Miles-Garnsey
Copy link
Member

What this PR does:

Refactors MCAC and Cassandra telemetry agent configurations out of the TelemetrySpec and into CassandraTelemetrySpec. This is necessary because TelemetrySpec is used for Reaper and Stargate and these components do not support those configurations.

Which issue(s) this PR fixes:
Fixes #840

Checklist

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

…assandra reconciliation stage (remove an sts restart).
@Miles-Garnsey Miles-Garnsey force-pushed the refactor/telemetry-types branch from fd68f97 to 0cc0ac2 Compare February 2, 2023 02:46
@codecov-commenter
Copy link

Codecov Report

Merging #841 (0cc0ac2) into main (779eced) will decrease coverage by 0.06%.
The diff coverage is 57.53%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #841      +/-   ##
==========================================
- Coverage   56.80%   56.75%   -0.06%     
==========================================
  Files          91       96       +5     
  Lines        9154     9504     +350     
==========================================
+ Hits         5200     5394     +194     
- Misses       3487     3629     +142     
- Partials      467      481      +14     
Impacted Files Coverage Δ
apis/k8ssandra/v1alpha1/k8ssandracluster_types.go 36.23% <ø> (ø)
apis/telemetry/v1alpha1/telemetry_methods.go 40.00% <0.00%> (-10.00%) ⬇️
pkg/telemetry/test_objects.go 0.00% <0.00%> (ø)
pkg/telemetry/validation.go 43.47% <ø> (ø)
pkg/utils/deployment.go 0.00% <0.00%> (ø)
controllers/stargate/stargate_controller.go 45.87% <28.57%> (-0.49%) ⬇️
pkg/telemetry/prometheus_resourcer.go 26.86% <33.33%> (+2.27%) ⬆️
controllers/stargate/vector.go 43.36% <43.36%> (ø)
pkg/stargate/vector.go 62.50% <62.50%> (ø)
pkg/telemetry/prom_cass_servicemonitor.go 85.18% <64.70%> (+7.40%) ⬆️
... and 13 more

@Miles-Garnsey Miles-Garnsey force-pushed the refactor/telemetry-types branch from 0cc0ac2 to 50c99af Compare February 2, 2023 03:09
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 2, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

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

No Coverage information No Coverage information
9.6% 9.6% Duplication

@Miles-Garnsey
Copy link
Member Author

I need to put this on hold pending (what I suspect will be) a bugfix in controller-tools. See my issue here for more details.

@adejanovski
Copy link
Contributor

@Miles-Garnsey, should we close this PR?

@Miles-Garnsey
Copy link
Member Author

Undoubtedly.

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.

Refactor TelemetrySpec so that it makes sense for Reaper, Stargate
3 participants