NETOBSERV-1233 conntrack reportMissing + DNS response code

[jpinsonneau]

This PR requires netobserv/flowlogs-pipeline#470 to be merged first

Added missing DnsFlagsResponseCode + ReportMissing option for mandatory fields

[jpinsonneau]

/hold

[github-actions]

New images:

• quay.io/netobserv/network-observability-operator:a3c3d4f
• quay.io/netobserv/network-observability-operator-bundle:v0.0.0-a3c3d4f
• quay.io/netobserv/network-observability-operator-catalog:v0.0.0-a3c3d4f

They will expire after two weeks.

To deploy this build:

# Direct deployment, from operator repo
IMAGE=quay.io/netobserv/network-observability-operator:a3c3d4f make deploy

# Or using operator-sdk
operator-sdk run bundle quay.io/netobserv/network-observability-operator-bundle:v0.0.0-a3c3d4f

Or as a Catalog Source:

apiVersion: operators.coreos.com/v1alpha1
kind: CatalogSource
metadata:
  name: netobserv-dev
  namespace: openshift-marketplace
spec:
  sourceType: grpc
  image: quay.io/netobserv/network-observability-operator-catalog:v0.0.0-a3c3d4f
  displayName: NetObserv development catalog
  publisher: Me
  updateStrategy:
    registryPoll:
      interval: 1m

[codecov]

Codecov Report

Merging #401 (9a0b936) into main (9d208dd) will increase coverage by 0.00%.
The diff coverage is 78.94%.

@@           Coverage Diff           @@
##             main     #401   +/-   ##
=======================================
  Coverage   55.49%   55.49%           
=======================================
  Files          45       45           
  Lines        5865     5874    +9     
=======================================
+ Hits         3255     3260    +5     
- Misses       2389     2393    +4     
  Partials      221      221           

Flag	Coverage Δ
unittests	55.49% <78.94%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed	Coverage Δ
controllers/flowlogspipeline/flp_common_objects.go	81.67% <78.94%> (-0.34%)	⬇️

[msherif1234]

@jpinsonneau why there are lots of other pkg updated in this PR ? I understand the flp update but not sure why u have to update more pkgs ?

[jpinsonneau]

@jpinsonneau why there are lots of other pkg updated in this PR ? I understand the flp update but not sure why u have to update more pkgs ?

That's the output of go mod tidy + go mod vendor.
I can get rid of these and only update flp if you prefer 😉

[msherif1234]

/lgtm

[Amoghrd]

/label qe-approved

[msherif1234]

/lgtm

[jpinsonneau]

/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jpinsonneau

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 [jpinsonneau]

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