Skip to content

chore: initial implementation of Helm Chart overrides at deploy time#2131

Merged
Racer159 merged 6 commits intomainfrom
2127-helm-chart-overrides
Nov 8, 2023
Merged

chore: initial implementation of Helm Chart overrides at deploy time#2131
Racer159 merged 6 commits intomainfrom
2127-helm-chart-overrides

Conversation

@Racer159
Copy link
Copy Markdown
Contributor

@Racer159 Racer159 commented Nov 7, 2023

Description

This is an initial implementation for Helm Chart overrides on deploy as library code.

Related Issue

Fixes #2127

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Other (security config, docs update, etc)

Checklist before merging

@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 7, 2023

Deploy Preview for zarf-docs canceled.

Name Link
🔨 Latest commit ecb3728
🔍 Latest deploy log https://app.netlify.com/sites/zarf-docs/deploys/654bccd274d0d70008d64290

Comment thread src/internal/packager/helm/utils.go
Copy link
Copy Markdown
Contributor

@Noxsios Noxsios left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

after sync-up, lgmt

@Racer159 Racer159 merged commit f145e3f into main Nov 8, 2023
@Racer159 Racer159 deleted the 2127-helm-chart-overrides branch November 8, 2023 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pass arbitrary helm values files at package deploy time

2 participants