-
Notifications
You must be signed in to change notification settings - Fork 529
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat : helm dependency update support added to HelmService #3098
feat : helm dependency update support added to HelmService #3098
Conversation
Eclipse JKube CI ReportStarted new GH workflow run for #3098 (2024-05-29T09:39:45Z) ⚙️ JKube E2E Tests (9283609681)
|
Arrays.stream(new Helm(Paths.get(helmConfig.getOutputDir(), helmType.getOutputDir())) | ||
.dependency() | ||
.update() | ||
.verify() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should add this as an opt-in.
At this stage we probably want config options for:
- debug (common to the rest of Helm config:
debug
-jkube.helm.debug
) - verify (
dependencyVerify
-jkube.helm.dependencyVerify
) - skipRefresh (
dependencySkipRefresh
-jkube.helm.dependencySkipRefresh
)
Related to eclipse-jkube#2110 Add a new method in dependencyUpdate in HelmService that will use helm-java dependency for dependency update of helm chart Signed-off-by: Rohan Kumar <[email protected]>
9c41e02
to
293f792
Compare
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thx!
Description
Related to #2110
Add a new method in dependencyUpdate in HelmService that will use helm-java dependency for dependency update of helm chart
Type of change
test, version modification, documentation, etc.)
Checklist