HelmUpgrade settings order matters

[AntonPetrov83]

Usage Information

9.0.4

Description

Hi!

Calling HelmUpgrade with SetChart() then SetRelease() produces invalid arguments order for the helm. The valid one is SetRelease() then SetChart() (matching helm command line usage) what is not obvious at all.

Reproduction Steps

Call HelmUpgrade with SetChart() then SetRelease() to see the error.

Expected Behavior

Order of settings must not be important.

Actual Behavior

Build fails.

Regression?

Regression when updated from 8.0.x to 9.0.4:

Known Workarounds

Change methods order.

Could you help with a pull-request?

No

[HSKC]

Same thing with HelmRepoAdd. SetName() and SetUrl() must be in this exact order.