Skip to content
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

Adds falco and falco-driver-loader 0.39.0 and falcoctl 0.10.0 rocks #14

Merged
merged 3 commits into from
Oct 6, 2024

Conversation

claudiubelu
Copy link
Contributor

Note that the Dockerfiles are identical to the previous versions. Thus, the rockfiles are also the same.

Falco 0.39.0 comes with falcoctl 0.10.0, which is why we're building that version as well.

Added the new versions into the sanity and integration tests.

@claudiubelu claudiubelu force-pushed the adds-falco-0.39.0-rocks branch from 66a9628 to 80171ad Compare October 3, 2024 11:57
The falco-driver-loader is just the falco image with a different
entrypoint.

Note that Pebble doesn't like it when a process finishes too quickly.
Which is why we're adding a sleep workaround for this rock image, as
it is expected for the image workload to eventually end.

Adds falco-driver-loader usage to the integration test. We also need to
set the "driver.kind=modern_ebpf" Helm chart option, in order for the
falco-driver-loader init container to properly run. By default, it will
try to autodetect the right driver, but it will fail when trying to use
the kmod driver (insmod will fail with "Operation not permitted"), but
the script exit code is still 0, meaning that the init container "successfully"
finishes, which results in the workload container to not start properly.
@claudiubelu claudiubelu changed the title Adds falco 0.39.0 and falcoctl 0.10.0 rocks Adds falco and falco-driver-loader 0.39.0 and falcoctl 0.10.0 rocks Oct 4, 2024
Note that the Dockerfiles are identical to the previous versions. Thus, the
rockfiles are also the same.

Falco 0.39.0 comes with falcoctl 0.10.0, which is why we're building
that version as well.

Added the new versions into the sanity and integration tests.
@claudiubelu claudiubelu force-pushed the adds-falco-0.39.0-rocks branch from 80171ad to e02d467 Compare October 4, 2024 07:49
@claudiubelu claudiubelu requested a review from a team as a code owner October 4, 2024 07:49
@bschimke95 bschimke95 merged commit 7dbffa5 into canonical:main Oct 6, 2024
2 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants