Skip to content

Conversation

@smatting
Copy link
Contributor

@smatting smatting commented Nov 7, 2025

This chart was originally added in PR #2304 for integration tests. In PR #3446, the integration tests were updated to stop deploying this Helm chart.

https://wearezeta.atlassian.net/browse/WPB-21317

Checklist

  • Confirm that this isn't in-use by other deployments
  • manually remove from wire-builds
  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@smatting smatting requested review from a team as code owners November 7, 2025 10:26
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Nov 7, 2025
@smatting smatting requested a review from jschaul November 7, 2025 10:30
@smatting
Copy link
Contributor Author

smatting commented Nov 7, 2025

the integration tests pass, except for test Spar.STM.testCreateIdpsAndScimsV7 which is a unrelated flaky test

Copy link
Member

@jschaul jschaul left a comment

Choose a reason for hiding this comment

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

Maybe remove the local deploy/dockerephemeral/docker-compose.yaml cluster mode also if it's not used.

@smatting smatting merged commit f2e881c into develop Nov 11, 2025
9 checks passed
@smatting smatting deleted the WPB-21317-remove-redis-cluster-chart branch November 11, 2025 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants