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
Reset values flag #602
Reset values flag #602
Conversation
@yxxhero would you mind to take a look at this? We talked about the problem in "discussions" week ago :) Im also open for suggestions if it can be implemented better |
I will review as soon as I can. |
1b80041
to
2994fe1
Compare
@kkoralsky when has --reset-vaules and --reuse-values in same. what will happen? |
@kkoralsky we can follow the behavior of helm. |
2994fe1
to
17c03a9
Compare
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.
If both --reset-values and --reuse-values are given, we use reset "values control mode" - this is also how helm beahaves
@kkoralsky please fix DCO issue. |
@mumoshu I think it's ok. do you have some suggestions? |
@kkoralsky BTW. please update the docs. |
…true Signed-off-by: Karol Ossowski <k@koralsky.pl>
Signed-off-by: Karol Ossowski <k@koralsky.pl>
Signed-off-by: Karol Ossowski <k@koralsky.pl>
Signed-off-by: Karol Ossowski <k@koralsky.pl>
…ocessing Signed-off-by: Karol Ossowski <k@koralsky.pl>
17c03a9
to
2fdb7c2
Compare
Signed-off-by: Karol Ossowski <k@koralsky.pl>
} | ||
|
||
return flags | ||
} |
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.
please add unittest for the func. @kkoralsky
rest of is LGTM. |
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.
LGTM. Thanks a lot for your contribution @kkoralsky and review @yxxhero
implemented
--reset-values
flag overriding.helmDefaults.reuseValues=true
as discussed: #593