Skip to content

Update helm bindings to work with typed credentials #987

@matthiasbruns

Description

@matthiasbruns

Description

In this ticket, helm input credentials need to be converted to runtime.Typed.
Helm has a lot of possible credential fields in the current input implementation

The credentials are mostly used in the tls communication with the helm repository

This code path is used by input with a remote chart as well as by access.

TODOs

  • migrate from map[string]string to runtime.Typed credentials in the helm binding
  • provide helm credential spec for valid combinations of the credentials linked above
  • (optional) drop deprecated credential fields

AC

  • helm, both input and access, fully support runtime.Typed credentials
  • old credentials must still work, unless we dropped support for them

Done Criteria

  • introduce runtime.Typed to bindings/helm
  • refine spec and credential types for the helm input & access
  • Code has been reviewed by other team members
  • Analysis of existing tests (Unit and Integration)
  • Unit Tests created for new code or existing Unit Tests updated
  • Integration Test Suite updated (includes deletion of existing unnecessary Integration Test and/or creation of new ones if required)
  • Enduser Documentation updated (if applicable)
  • Internal technical Documentation created/updated (if applicable)
  • Successful demonstration in Review

Metadata

Metadata

Assignees

Labels

area/ipceiImportant Project of Common European Interestkind/tasksmall task, normally part of feature or epic

Type

No fields configured for Task.

Projects

Status
🍺 Done

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions