Skip to content

Conversation

@eranra
Copy link
Collaborator

@eranra eranra commented Mar 7, 2022

No description provided.

@eranra eranra self-assigned this Mar 7, 2022
@eranra eranra linked an issue Mar 7, 2022 that may be closed by this pull request
@eranra eranra force-pushed the add_remove_fields_65 branch from c732ec7 to 0f42064 Compare March 7, 2022 10:11
@eranra eranra requested a review from KalmanMeth March 7, 2022 13:02
@eranra eranra force-pushed the add_remove_fields_65 branch from 0f42064 to d7ae77b Compare March 7, 2022 14:47
@eranra eranra requested review from jotak and ronensc March 7, 2022 14:47
@eranra
Copy link
Collaborator Author

eranra commented Mar 7, 2022

@jotak this might be used in future configuration if we decide to filter out entries that we don't want to output ... for example we can create new flow entries (with some new tag) using connection tracking with the entire/commemorative flow info (instead of the multiple flow entries that we get today from OVS for a flow) and send only those to Loki. This will save storage and provide more value to users giving them real view of the flow infomration

@stleerh ^^^ FYI

@eranra eranra force-pushed the add_remove_fields_65 branch from d7ae77b to c221d28 Compare March 7, 2022 15:05
Copy link
Member

@jotak jotak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, just a typo :)

@eranra eranra force-pushed the add_remove_fields_65 branch from c221d28 to 13820d7 Compare March 8, 2022 09:41
@eranra eranra force-pushed the add_remove_fields_65 branch from 13820d7 to 3de57a3 Compare March 8, 2022 10:50
@eranra eranra requested review from KalmanMeth and ronensc March 8, 2022 10:52
@eranra eranra merged commit 8ba43eb into netobserv:main Mar 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Transform: Add remove fields

4 participants