Skip to content

Commit

Permalink
NETOBSERV-1190: use DNS latency instead of TS
Browse files Browse the repository at this point in the history
This PR depends on PR #331 so 331 need to be merged 1st

Signed-off-by: msherif1234 <mmahmoud@redhat.com>
  • Loading branch information
msherif1234 committed Jul 17, 2023
1 parent def94dc commit 01ae4ec
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 5 deletions.
3 changes: 3 additions & 0 deletions config/samples/flows_v1beta1_flowcollector.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@ spec:
sampling: 50
cacheActiveTimeout: 5s
cacheMaxFlows: 100000
privileged: false
enableTCPDrop: false
enableDNSTracking: false
interfaces: [ ]
excludeInterfaces: [ "lo" ]
logLevel: info
Expand Down
6 changes: 1 addition & 5 deletions controllers/flowlogspipeline/flp_common_objects.go
Original file line number Diff line number Diff line change
Expand Up @@ -400,11 +400,7 @@ func (b *builder) addTransformStages(stage *config.PipelineBuilderStage) (*corev
if helper.IsDNSTrackingEnabled(b.desired) {
outDNSTrackingFields := []api.OutputField{
{
Name: "DnsRequestTimeMs",
Operation: "min",
},
{
Name: "DnsResponseTimeMs",
Name: "DnsLatencyMs",
Operation: "max",
},
}
Expand Down

0 comments on commit 01ae4ec

Please sign in to comment.