You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As described in falcosecurity/falco#2418, we missed incrementing FALCO_ENGINE_VERSION for the latest Falco release. #20 is the first PR taking advantage of one of the newly introduced fields and should bump the required_engine_version to its proper number, which would be 16. However, since we have falcoctl that is now able to follow new rules, extra care should be taken: a Falco 0.34.0 ingesting the new rules containing the one that we are introducing in PR #20, will fail to restart since it is stuck at version 15.
A possible way to avoid this is to properly tag rules in a way that the new ones are not downloaded. Since in the Helm charts we are following tag 0, I think a bump of the major is needed.
The text was updated successfully, but these errors were encountered:
However, if we agree to switch to 1.0.0 (and leave the 0.x untouched with this non-addressable issue), I will rename just rename the milestone.
cc @falcosecurity/rules-maintainers
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.
Describe the bug
As described in falcosecurity/falco#2418, we missed incrementing FALCO_ENGINE_VERSION for the latest Falco release. #20 is the first PR taking advantage of one of the newly introduced fields and should bump the
required_engine_version
to its proper number, which would be 16. However, since we havefalcoctl
that is now able to follow new rules, extra care should be taken: a Falco 0.34.0 ingesting the new rules containing the one that we are introducing in PR #20, will fail to restart since it is stuck at version 15.A possible way to avoid this is to properly tag rules in a way that the new ones are not downloaded. Since in the Helm charts we are following tag 0, I think a bump of the major is needed.
The text was updated successfully, but these errors were encountered: