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

Fix indexer deletion in upgrade scenario #1153

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

tmaltaverne
Copy link
Contributor

This PR addresses issue #1151.
There are 2 changes:

  • Make sure that the fix of PR cspl-2253: indexer deletion in upgrade scenario #1041 is applied in the cluster-manager usecase
  • Fix the check on the image version. The format of the image field being like splunk/splunk-operator:8.2.0, it is needed to get the version that is after the colon. The previous check was only checking if the image was starting with 8 or 9, which is always false.

@Philbow
Copy link

Philbow commented Jul 17, 2023

Discovered the same problem when triggering a helm upgrade from 2.2.1 to 2.3.0. Unfortunately the operator is not logging out the deletion of the STS.

Any plans to fix the pull request in order to merge it for the upcoming release? This is currently blocking the seamless upgrade for our Splunk customers.

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.

None yet

4 participants