Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

[stable/falco] Add Falco NATS output integration #6600

Merged
merged 3 commits into from
Jul 11, 2018
Merged

[stable/falco] Add Falco NATS output integration #6600

merged 3 commits into from
Jul 11, 2018

Conversation

nestorsalceda
Copy link
Collaborator

What this PR does / why we need it:

This PR add a NATS output to Sysdig Falco Helm Chart. It configures a sidecar container which receives alerts from Falco and sends to a NATS server.

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Special notes for your reviewer:

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Jul 11, 2018
@k8s-ci-robot k8s-ci-robot requested review from bencer and foxish July 11, 2018 14:46
@nestorsalceda nestorsalceda changed the title Falco nats output [stable/falco] Add Falco NATS output integration Jul 11, 2018
@nestorsalceda
Copy link
Collaborator Author

/assign @bencer

@bencer
Copy link
Collaborator

bencer commented Jul 11, 2018

/lgtm
/approve

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jul 11, 2018
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bencer, nestorsalceda

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:

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

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 11, 2018
@k8s-ci-robot k8s-ci-robot merged commit 0dbd351 into helm:master Jul 11, 2018
Jnig pushed a commit to Jnig/charts that referenced this pull request Nov 13, 2018
* Update value of bufferedOutputs in configmap documentation

* Add NATS output integration for Sysdig Falco

* Add a change log

Signed-off-by: Jakob Niggel <[email protected]>
gsemet pushed a commit to gsemet/charts that referenced this pull request Nov 13, 2018
* Update value of bufferedOutputs in configmap documentation

* Add NATS output integration for Sysdig Falco

* Add a change log
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants