NETOBSERV-1466 NodeDirections / IfDirections

[jpinsonneau]

Description

• Rename FlowDirection to NodeDirection and improved descriptions
• Update fields / queries to plural
• Restore deduping running parallel queries for metrics
• Only show related fields when necessary in side panel
• Keep the ability to show columns separately

Dependencies

netobserv/network-observability-operator#578

Checklist

If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

• Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix (in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes).
• Does this PR require product documentation?
  • If so, make sure the JIRA epic is labelled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.
• Does this PR require a product release notes entry?
  • If so, fill in "Release Note Text" in the JIRA.
• Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.
  • If so, make sure it is described in the JIRA ticket.
• QE requirements (check 1 from the list):
  • Standard QE validation, with pre-merge tests unless stated otherwise.
  • Regression tests only (e.g. refactoring with no user-facing change).
  • No QE (e.g. trivial change with high reviewer's confidence, or per agreement with the QE team).

[codecov]

Codecov Report

Attention: Patch coverage is 40.74074% with 32 lines in your changes are missing coverage. Please review.

Project coverage is 57.38%. Comparing base (50ea5cc) to head (a3db9c2).

Files	Patch %	Lines
web/src/components/netflow-record/record-field.tsx	20.00%	8 Missing ⚠️
web/src/components/netflow-record/record-panel.tsx	20.00%	8 Missing ⚠️
pkg/loki/filter.go	22.22%	7 Missing ⚠️
web/src/utils/filter-options.ts	22.22%	7 Missing ⚠️
pkg/model/fields/fields.go	66.66%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #482      +/-   ##
==========================================
- Coverage   57.45%   57.38%   -0.07%     
==========================================
  Files         167      167              
  Lines        8710     8727      +17     
  Branches     1126     1128       +2     
==========================================
+ Hits         5004     5008       +4     
- Misses       3382     3397      +15     
+ Partials      324      322       -2     

Flag	Coverage Δ
uitests	58.17% <34.28%> (-0.12%)	⬇️
unittests	55.11% <52.63%> (+0.07%)	⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[openshift-ci]

New changes are detected. LGTM label has been removed.

[jpinsonneau]

rebased without changes

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by:

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

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

[openshift-ci]

New changes are detected. LGTM label has been removed.