update(docker): remove minimal image

[leogr]

Signed-off-by: Leonardo Grasso [email protected]

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 contributing rules or changes to rules, please make sure to also uncomment one of the following line:

/kind rule-update

/kind rule-create

Any specific area of the project related to this PR?

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

/area build

/area engine

/area examples

/area rules

/area integrations

/area tests

/area proposals

What this PR does / why we need it:

As per "Falco Artifacts Scope" proposal 👉 #1184

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

• There's no reference to it on the web site (except for the changelog part of two blog posts
• It is not used by integrations nor examples
• The helm chart does not use it

PS
Since the multi-stage build approach used by this minimal image seems still valid to me, I would propose - in another PR - to refactor the current docker/slim/Dockerfile in the same way but once we have decided the proper naming of these images.

/cc @fntlnz
/cc @kris-nova
/cc @leodido

Does this PR introduce a user-facing change?:

update(docker): remove `minimal` image

[leogr]

/hold
for #1184

[poiana]

LGTM label has been added.

Git tree hash: ae5cfbd55fc6daa33e2042eac223a10aab05d09f

[leogr]

/hold cancel

[leogr]

Rebased.

[poiana]

LGTM label has been added.

Git tree hash: b81a7c94ce6288a76ee79324b7da12a981b40da6

[leogr]

/honk
🦅

[poiana]

@leogr:

In response to this:

/honk
🦅

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.

[poiana]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fntlnz, leodido

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

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [fntlnz,leodido]

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

[JPLachance]

I just found this PR and it makes me a little bit sad.

This image was a cool experiment made @mfdii. The goal was to reduce the amount of packages shipped in the Falco container image to easily pass container images scanners like Clair or Snyk.

A quick scan with Snyk of the falcosecurity/falco:0.22.1-slim Docker image reveals there are 103 known vulnerabilities in the image. The "minimal" image fixed that.

[leogr]

Hey @JPLachance
this PR was just an intermediate step, the content of the old minimal will be re-published as falcosecurity/falco-no-driver as proposed here https://github.com/falcosecurity/falco/blob/master/proposals/20200506-artifacts-scope-part-2.md#images

I believe we can do this before the next release.
So the only breaking change will be just to change the image name.
WDYT?

[JPLachance]

Great then! I'll be happy to test te new image once we will have a way to load the probe :D

#897