UNCLASSIFIED - NO CUI

Skip to content
Snippets Groups Projects

update provider input under helmRepositories instead of flux

Merged Christopher O'Connell requested to merge update-helmrepo-providers into master

General MR

Summary

Original MR added this provider configuration under the flux values but added it to the helmRepositories section in the schema. This MR fixes that and adds it under helmRepositories in the values

Relevant logs/screenshots

Feedback from user -> #2375 (comment 2660771)

Validated with helm template . AND values as follows, also works without provider ->

helmRepositories:
  - name: "ecr"
    repository: "oci://123456.dkr.ecr.us-gov-west-1.amazonaws.com"
    provider: aws
    type: "oci"

Linked Issue

Closes #2375 (closed)

Upgrade Notices

N/A

Edited by Christopher O'Connell

Merge request reports

Merge request pipeline #3844380 passed with warnings

Merge request pipeline passed with warnings for f086b848

Approved by

Merged by Andrew ShoellAndrew Shoell 3 months ago (Jan 7, 2025 4:57pm UTC)

Merge details

  • Changes merged into master with 2fa22061 (commits were squashed).
  • Deleted the source branch.

Pipeline #3852878 passed

Pipeline passed for 2fa22061 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading