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

chore: fix null addon.spec.helm.installValues value #8010

Merged
merged 2 commits into from
Aug 22, 2024

Conversation

ldming
Copy link
Collaborator

@ldming ldming commented Aug 21, 2024

fix https://kubeblocks.slack.com/archives/C050K0LPR70/p1724227335152709

Addon.extensions.kubeblocks.io "mysql" is invalid: [spec.helm.installOptions: Invalid value: "null": spec.helm.installOptions in body must be of type object: "null", <nil>: Invalid value: "null": some validation rules were not checked because the object was invalid; correct the existing errors to complete validation]

@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines. label Aug 21, 2024
@apecloud-bot apecloud-bot added the approved PR Approved Test label Aug 21, 2024
@apecloud-bot apecloud-bot removed the approved PR Approved Test label Aug 21, 2024
@github-actions github-actions bot added size/M Denotes a PR that changes 30-99 lines. and removed size/XS Denotes a PR that changes 0-9 lines. labels Aug 21, 2024
@ldming
Copy link
Collaborator Author

ldming commented Aug 21, 2024

/approve

@apecloud-bot apecloud-bot added the approved PR Approved Test label Aug 21, 2024
@ldming ldming merged commit 46bed94 into main Aug 22, 2024
30 checks passed
@ldming ldming deleted the support/fix-null-addon-field branch August 22, 2024 01:09
@github-actions github-actions bot added this to the Release 0.9.2 milestone Aug 22, 2024
@ldming
Copy link
Collaborator Author

ldming commented Aug 22, 2024

/cherry-pick release-0.9

Copy link

🤖 says: ‼️ cherry pick action failed.
See: https://github.com/apecloud/kubeblocks/actions/runs/10500025783

ldming added a commit that referenced this pull request Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved PR Approved Test size/M Denotes a PR that changes 30-99 lines.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants