Uploaded image for project: 'Helm'
  1. Helm
  2. HELM-342

Implement basic authentication for ODC Helm backend

XMLWordPrintable

    • Icon: Story Story
    • Resolution: Done
    • Icon: Undefined Undefined
    • 4.12
    • None
    • Helm
    • AppSvc Sprint 219, AppSvc Sprint 220, AppSvc Sprint 221, AppSvc Sprint 222, AppSvc Sprint 223, AppSvc Sprint 224

      Owner: Architect:

      Story (Required)

      As an OCP user I will like to be able to install helm charts from repos added to ODC with basic authentication fields populated

      Background (Required)

      We need to support helm installs for Repos that have the basic authentication secret name and namespace.

      Glossary

      Out of scope

      Updating the ProjectHelmChartRepository CRD, already done in diff story
      Supporting the HelmChartRepository CR, this feature will be scoped first to project/namespace scope repos.

      In Scope

      <Defines what is included in this story>

      Approach(Required)

      If the new fields for basic auth are set in the repo CR then use those credentials when making API calls to helm to install/upgrade charts. We will error out if user logged in does not have access to the secret referenced by Repo CR. If basic auth fields are not present we assume is not an authenticated repo.

      Dependencies

      Nonet

      Edge Case

      NA

      Acceptance Criteria

      I can list, install and update charts on authenticated repos from ODC
      Needs Documentation both upstream and downstream
      Needs new unit test covering repo auth

      INVEST Checklist

      Dependencies identified
      Blockers noted and expected delivery timelines set
      Design is implementable
      Acceptance criteria agreed upon
      Story estimated

      Legend

      Unknown
      Verified
      Unsatisfied

            kmamgain@redhat.com Kartikey Mamgain
            dperaza@redhat.com David Peraza
            Igor Sutton (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: