Skip to content

feat: add support for "helm get values" command - #345

Merged
manusa merged 1 commit into
manusa:mainfrom
afalhambra-hivemq:feat/get-values
Feb 7, 2026
Merged

feat: add support for "helm get values" command#345
manusa merged 1 commit into
manusa:mainfrom
afalhambra-hivemq:feat/get-values

Conversation

@afalhambra-hivemq

@afalhambra-hivemqafalhambra-hivemq commented Feb 6, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

Closes#322

Summary

  • Adds support for the helm get values command
  • Implements the full stack: Go native layer, JNA bridge, and Java command API

Test plan

  • Go tests pass (go test ./... in native/)
  • Java compilation passes (mvn clean install -Dquickly)

Usage

// Get user-supplied values for a releaseStringvalues = Helm.get("my-release")
.values()
.withKubeConfig(kubeConfigPath)
.call();
// Get all values (including chart defaults)StringallValues = Helm.get("my-release")
.values()
.allValues()
.withKubeConfig(kubeConfigPath)
.call();
// Get values for a specific revisionStringrevisionValues = Helm.get("my-release")
.values()
.withRevision(2)
.withKubeConfig(kubeConfigPath)
.call();

@afalhambra-hivemqafalhambra-hivemq mentioned this pull request Feb 6, 2026
Closed
manusa
manusa approved these changes Feb 7, 2026
View reviewed changes

@manusamanusa left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM, thx!

@manusa
manusa merged commit 3fcf96a into manusa:mainFeb 7, 2026
3 checks passed
@manusamanusa mentioned this pull request Feb 7, 2026
Merged
@manusamanusa mentioned this pull request May 21, 2026
Merged
2 tasks
manusa added a commit that referenced this pull request May 21, 2026
@manusa@claude
chore(docs): refresh native codebase audit for 2026-05
c0e000c
Re-verified findings against current HEAD. Updates since 2025-11-14:
status/history/get-values shipped (PRs #345/#352/#353), NewCfg panic
fixed, newRegistryClient extraction done. Still open: install/upgrade
defer cancel() leak, three TODOs (helm.go, search.go, repotest.go),
and rollback/pull commands (#101, #351).
Restructured to match docs/research conventions: dated header with
related issue/PR links, TL;DR, status-of-prior-findings table.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Marc Nuri <marc@marcnuri.com>
manusa added a commit that referenced this pull request May 21, 2026
@manusa@claude
chore(docs): refresh native codebase audit for 2026-05
0ac5cf5
Re-verified findings against current HEAD. Updates since 2025-11-14:
status/history/get-values shipped (PRs #345/#352/#353), NewCfg panic
fixed, newRegistryClient extraction done. Still open: install/upgrade
defer cancel() leak, three TODOs (helm.go, search.go, repotest.go),
and rollback/pull commands (#101, #351).
Restructured to match docs/research conventions: dated header with
related issue/PR links, TL;DR, status-of-prior-findings table.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Marc Nuri <marc@marcnuri.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@manusamanusamanusa approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

"helm get values" not supported

2 participants

@afalhambra-hivemq@manusa