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

NETOBSERV-184: decouple components into a pipeline #28

Closed
wants to merge 0 commits into from
Closed

NETOBSERV-184: decouple components into a pipeline #28

wants to merge 0 commits into from

Conversation

mariomac
Copy link
Contributor

@mariomac mariomac commented Feb 8, 2022

As a previous step for NETOBSERV-184 (Split and Integrate flowlogs2metrics in
kube-enricher), this PR provides an asynchronous pipeline and wraps the current
components into the following roles:

  • Ingester: a wrapped version of Goflow2
  • Transformer: the Pod & Service Enricher
  • Submitter: the Loki processor

This is only a previous step of the above task to avoid having too large PRs.
The following PRs will try to simplify and decouple even more some existing
components (e.g. Goflow-Kube), remove duplicities. After that, we will
make the Pipeline object to evolve to accomodate the new functionalities
from flowlogs2metrics

@mariomac mariomac added the no-qe This PR doesn't necessitate QE approval label Feb 8, 2022
@openshift-ci
Copy link

openshift-ci bot commented Feb 8, 2022

@mariomac: PR needs rebase.

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.

@mariomac mariomac closed this Feb 8, 2022
@openshift-ci
Copy link

openshift-ci bot commented Feb 8, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:

The full list of commands accepted by this bot can be found 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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs-rebase no-qe This PR doesn't necessitate QE approval
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant