Skip to content

Conversation

@amit-o
Copy link

@amit-o amit-o commented Aug 12, 2025

Description

Add a simple status to the standalone Redis CRD

Fixes #638

Type of change

  • New feature (non-breaking change which adds functionality)

Checklist

  • Tests have been added/modified and all tests pass.
  • Functionality/bugs have been confirmed to be unchanged or fixed.
  • I have performed a self-review of my own code.
  • Documentation has been updated or added where necessary.

Additional Context

amit-o added 2 commits August 13, 2025 17:39
Signed-off-by: Amit Oren <[email protected]>
Signed-off-by: Amit Oren <[email protected]>
@drivebyer drivebyer added the enhancement New feature or request label Aug 14, 2025
@amit-o amit-o changed the title Add redis standalone status feat: Add redis standalone status Aug 14, 2025
amit-o added 3 commits August 14, 2025 12:41
Signed-off-by: Amit Oren <[email protected]>
Signed-off-by: Amit Oren <[email protected]>
@codecov
Copy link

codecov bot commented Aug 15, 2025

Codecov Report

❌ Patch coverage is 4.00000% with 48 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (main@88d688a). Learn more about missing BASE report.

Files with missing lines Patch % Lines
internal/controller/redis/redis_controller.go 11.76% 13 Missing and 2 partials ⚠️
internal/k8sutils/statefulset.go 0.00% 15 Missing ⚠️
internal/k8sutils/status.go 0.00% 15 Missing ⚠️
internal/cmd/manager/cmd.go 0.00% 3 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1477   +/-   ##
=======================================
  Coverage        ?   31.61%           
=======================================
  Files           ?       78           
  Lines           ?     6999           
  Branches        ?        0           
=======================================
  Hits            ?     2213           
  Misses          ?     4608           
  Partials        ?      178           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@amit-o amit-o marked this pull request as draft August 18, 2025 09:22
Signed-off-by: Amit Oren <[email protected]>
@amit-o amit-o marked this pull request as ready for review September 3, 2025 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] : Add status sub resource

2 participants