Skip to content

feat: add new artifact state shared volume between initContainer and sidecar#936

Merged
poiana merged 3 commits intofalcosecurity:masterfrom
c2ndev:feat/add-artifact-state-volume
Jan 21, 2026
Merged

feat: add new artifact state shared volume between initContainer and sidecar#936
poiana merged 3 commits intofalcosecurity:masterfrom
c2ndev:feat/add-artifact-state-volume

Conversation

@c2ndev
Copy link
Copy Markdown
Contributor

@c2ndev c2ndev commented Jan 7, 2026

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If this PR will release a new chart version please make sure to also uncomment the following line:

/kind chart-release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area falco-chart

/area falcosidekick-chart

/area falco-talon-chart

/area event-generator-chart

/area k8s-metacollector-chart

What this PR does / why we need it:

This PR adds a shared artifact state volume between the falcoctl-artifact-install init container and the falcoctl-artifact-follow sidecar to support the new artifact state management feature introduced in falcoctl (falcosecurity/falcoctl#870).

Without this shared volume, the init container and sidecar cannot share state information about downloaded artifacts, causing the sidecar to re-download artifacts that were already installed by the init container. This leads to unnecessary network traffic and slower startup times.

The implementation follows the same pattern as the existing shared volumes (plugins-install-dir and rulesfiles-install-dir).

Which issue(s) this PR fixes:

Fixes #935

Special notes for your reviewer:

Checklist

  • Chart Version bumped
  • Variables are documented in the README.md
  • CHANGELOG.md updated

@poiana
Copy link
Copy Markdown
Contributor

poiana commented Jan 7, 2026

Welcome @c2ndev! It looks like this is your first PR to falcosecurity/charts 🎉

@poiana poiana requested review from alacuku and bencer January 7, 2026 14:54
@poiana poiana added the size/S label Jan 7, 2026
@c2ndev c2ndev changed the title wip: feat: add new artifact state shared volume between initContainer and sidecar feat: add new artifact state shared volume between initContainer and sidecar Jan 7, 2026
@leogr
Copy link
Copy Markdown
Member

leogr commented Jan 8, 2026

/hold for falcosecurity/falcoctl#870

…sidecar

Signed-off-by: cannarelladev <cannarella.dev@gmail.com>
Signed-off-by: cannarelladev <cannarella.dev@gmail.com>
Signed-off-by: cannarelladev <cannarella.dev@gmail.com>
@c2ndev c2ndev force-pushed the feat/add-artifact-state-volume branch from c2d6205 to ef926c1 Compare January 21, 2026 11:43
@leogr
Copy link
Copy Markdown
Member

leogr commented Jan 21, 2026

/hold cancel

@github-project-automation github-project-automation bot moved this from Todo to In progress in Falco Roadmap Jan 21, 2026
@poiana poiana added the lgtm label Jan 21, 2026
@poiana
Copy link
Copy Markdown
Contributor

poiana commented Jan 21, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: c2ndev, leogr

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@poiana
Copy link
Copy Markdown
Contributor

poiana commented Jan 21, 2026

LGTM label has been added.

DetailsGit tree hash: d5dfc234ed77488dcdabb2c9f84a6581505c5fe4

@poiana poiana merged commit ddd2492 into falcosecurity:master Jan 21, 2026
6 checks passed
@github-project-automation github-project-automation bot moved this from In progress to Done in Falco Roadmap Jan 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved area/falco-chart dco-signoff: yes kind/chart-release Add this label when the chart version has been bumped kind/feature New feature or request lgtm size/S

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Add shared artifact state volume between falcoctl init container and sidecar

3 participants