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

Informational Rules Loaded When priority: notice #1884

Closed
mike-stewart opened this issue Feb 1, 2022 · 1 comment · Fixed by #1885
Closed

Informational Rules Loaded When priority: notice #1884

mike-stewart opened this issue Feb 1, 2022 · 1 comment · Fixed by #1885
Labels

Comments

@mike-stewart
Copy link
Contributor

mike-stewart commented Feb 1, 2022

Describe the bug

On falco 0.31.0 I am seeing Informational rules being loaded when the priority in the config is set to only load Notice and above.

How to reproduce it

I'm running falco.yaml with:

priority: notice

with no overrides for info-level rules. However, informational rules are still being loaded, and are alerting at the Notice level.

Expected behaviour

It is expected that when priority: notice, only Notice, Warning, Error, Critical, Alert, and Emergency rules will be loaded and fired.

Screenshots
Screen Shot 2022-02-01 at 5 57 39 PM

Note that this rule is supposed to be INFO level:

falco/rules/falco_rules.yaml

Lines 1886 to 1894 in f86423d

- rule: Launch Privileged Container
desc: Detect the initial process started in a privileged container. Exceptions are made for known trusted images.
condition: >
container_started and container
and container.privileged=true
and not falco_privileged_containers
and not user_privileged_containers
output: Privileged container started (user=%user.name user_loginuid=%user.loginuid command=%proc.cmdline %container.info image=%container.image.repository:%container.image.tag)
priority: INFO

Environment

  • Falco version: 0.31.0
  • System info:
  • Cloud provider or hardware configuration:
  • OS:
  • Kernel:
  • Installation method: Kubernetes

Additional context

I'm wondering if this code may be the problem:

priorities = {
Emergency=0, Alert=1, Critical=2, Error=3, Warning=4, Notice=5, Informational=5, Debug=7,
emergency=0, alert=1, critical=2, error=3, warning=4, notice=5, informational=5, debug=7,
EMERGENCY=0, ALERT=1, CRITICAL=2, ERROR=3, WARNING=4, NOTICE=5, INFORMATIONAL=5, DEBUG=7,
INFO=5, info=5
}

Note that Notice and Informational are both set to level 5.

I can open a PR from master...mike-stewart:patch-2 if that would be helpful.

mike-stewart added a commit to mike-stewart/falco that referenced this issue Feb 1, 2022
@jasondellaluce
Copy link
Contributor

Great catch @mike-stewart ! Thank you for noticing, would you mind opening a PR?

mike-stewart added a commit to mike-stewart/falco that referenced this issue Feb 2, 2022
Signed-off-by: Mike Stewart <mike.stewart@introhive.com>
poiana pushed a commit that referenced this issue Feb 4, 2022
Signed-off-by: Mike Stewart <mike.stewart@introhive.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants