-
Notifications
You must be signed in to change notification settings - Fork 152
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
test upgrade with no change in values #1618
base: main
Are you sure you want to change the base?
Conversation
The chart fails to upgrade:
|
So upgrade fails due to this change introduced in latest which makes instrumentation a non-hook object. On upgrade helm sees the object exists but it is not part of tracked objects in its release, hence the error about missing annotations |
- name: Update dependencies | ||
run: | | ||
make dep-update | ||
- name: try to import instrumentation in helm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
required cmds to get around the instrumentation hook to non-hook change on upgrade
I think I have tried all combinations of possible ways an existing user with operator enabled could try to helm upgrade. The only option which seems to work where upgrade does not fail and the crds are kept is where they explicitly add the var |
Description:
Link to Splunk idea: <Link to Splunk idea, see https://ideas.splunk.com>
Testing:
Documentation: