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
Copy link
Copy Markdown
Contributor

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

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 release
String values = Helm.get("my-release")
    .values()
    .withKubeConfig(kubeConfigPath)
    .call();

// Get all values (including chart defaults)
String allValues = Helm.get("my-release")
    .values()
    .allValues()
    .withKubeConfig(kubeConfigPath)
    .call();

// Get values for a specific revision
String revisionValues = Helm.get("my-release")
    .values()
    .withRevision(2)
    .withKubeConfig(kubeConfigPath)
    .call();

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

@manusa manusa left a comment

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:main Feb 7, 2026
3 checks passed
@manusa manusa mentioned this pull request Feb 7, 2026
Merged
@manusa manusa 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 free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@manusa manusa manusa 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