feat(get-images-from-files): add support for helm values #29

Merged
t.behrendt merged 4 commits from feat-get-images-from-files-add-helm-values-support into main 2026-02-05 17:29:01 +01:00
Owner

Adding support for images inside of helm value files as per helm convention.
Limitation: Default images the chart may propose are not exposed and therefore not supported, only what's specifically mentioned in the values file.

Adding support for images inside of helm value files as per helm convention. Limitation: Default images the chart may propose are not exposed and therefore not supported, only what's specifically mentioned in the values file.
t.behrendt added 2 commits 2026-02-05 15:49:35 +01:00
docs: better list supported formats
CI / Test Setup Trivy amd64 (pull_request) Successful in 9s
CI / Test Setup DB (pull_request) Successful in 15s
CI / Test Get Images From Files (pull_request) Failing after 22s
CI / Test Setup Trivy arm64 (pull_request) Successful in 25s
8ed8e7304c
t.behrendt requested review from branch-buddy 2026-02-05 15:49:37 +01:00
branch-buddy approved these changes 2026-02-05 15:52:18 +01:00
Dismissed
branch-buddy left a comment
Collaborator

The changes align with the PR title and description, adding Helmfile values support. The diff includes necessary test cases, updates to CI workflow, and README documentation. The awk script correctly handles repository/tag parsing with indentation logic. No obvious issues detected.

The changes align with the PR title and description, adding Helmfile values support. The diff includes necessary test cases, updates to CI workflow, and README documentation. The awk script correctly handles repository/tag parsing with indentation logic. No obvious issues detected.
Author
Owner

The changes align with the PR title and description, adding Helmfile values support. The diff includes necessary test cases, updates to CI workflow, and README documentation. The awk script correctly handles repository/tag parsing with indentation logic. No obvious issues detected.

Well, not quite bro.

> The changes align with the PR title and description, adding Helmfile values support. The diff includes necessary test cases, updates to CI workflow, and README documentation. The awk script correctly handles repository/tag parsing with indentation logic. No obvious issues detected. Well, not quite bro.
t.behrendt added 1 commit 2026-02-05 17:15:17 +01:00
refactor: remove handling of sole repository, always require both repo and tag
CI / Test Setup Trivy amd64 (pull_request) Successful in 9s
CI / Test Setup DB (pull_request) Successful in 15s
CI / Test Get Images From Files (pull_request) Successful in 21s
CI / Test Setup Trivy arm64 (pull_request) Successful in 24s
19cf9c7dd6
t.behrendt dismissed branch-buddy's review 2026-02-05 17:15:17 +01:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

t.behrendt requested review from branch-buddy 2026-02-05 17:16:07 +01:00
t.behrendt added 1 commit 2026-02-05 17:16:44 +01:00
test: add invalid reference to helmfile
CI / Test Setup Trivy amd64 (pull_request) Successful in 9s
CI / Test Setup DB (pull_request) Successful in 16s
CI / Test Get Images From Files (pull_request) Successful in 20s
CI / Test Setup Trivy arm64 (pull_request) Successful in 24s
8f03d59b51
branch-buddy approved these changes 2026-02-05 17:18:45 +01:00
branch-buddy left a comment
Collaborator

The PR aligns with the title and description by adding Helmfile values support. The diff includes necessary test cases, documentation updates, and code changes to extract images from Helm values. No obvious issues detected.

The PR aligns with the title and description by adding Helmfile values support. The diff includes necessary test cases, documentation updates, and code changes to extract images from Helm values. No obvious issues detected.
t.behrendt merged commit 9411465688 into main 2026-02-05 17:29:01 +01:00
t.behrendt deleted branch feat-get-images-from-files-add-helm-values-support 2026-02-05 17:29:01 +01:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: t.behrendt/sec-actions#29