-
Notifications
You must be signed in to change notification settings - Fork 65
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
Bump z2jh to v1.0 beta #439
Conversation
This is needed to be compatible with z2jh 1.0.0 with schema validation logic that will complain about unrecognized configuration options under singleuser or anything but `custom`.
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.
Thanks @yuvipanda this LGTM!
I saw that you have pushed the k8s-hub replacement image as well. Merge?
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.
My questions were answered, this LGTM now!
Thanks, @damianavila! |
singleuser.admin
tocustom.singleuserAdmin
to satisfy the helmJSON schema, which doesn't allow extra properties under
singleuser
requirements.yaml
file intoChart.yaml
files - with helm3 thesedo not need to be together
Splitting up code from #429 that
works on just upgrading code to z2jh 1.0-beta1
Fixes #414