Fix helm login

[tpoliaw]

And pin the version of helm being used to prevent future changes
breaking workflows.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.09%. Comparing base (f42c3df) to head (82ba0b2).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #242   +/-   ##
=======================================
  Coverage   76.09%   76.09%           
=======================================
  Files          13       13           
  Lines        1945     1945           
=======================================
  Hits         1480     1480           
  Misses        465      465           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[DominicOram]

Probably shouldn't actually approve until the install passes

[tpoliaw]

Bit awkward that you can't pin to a major version. Is it better to pin to a specific patch version or deal with any changes that happen with the latest version?

[DominicOram]

Bit awkward that you can't pin to a major version. Is it better to pin to a specific patch version or deal with any changes that happen with the latest version?

My opinion tends to be to never pin and just fix things when they break. Otherwise, in my experience, you end up pinned to 1.X still when 8.X comes out, which has changes that you really need, and the job of moving them is horrible. However, that only works when you notice that the new version is failing, which we didn't here or on the config server so approving for now