Skip to content

[enterprise-4.12] OSDOCS-14862 NETOBSERV 1.9 Features, Bugs, Known Issues#95608

Merged
mburke5678 merged 1 commit intoopenshift:enterprise-4.12from
gwynnemonahan:manual-cp-4-12-no-19-OSDOCS-14862-rel-notes
Jul 7, 2025
Merged

[enterprise-4.12] OSDOCS-14862 NETOBSERV 1.9 Features, Bugs, Known Issues#95608
mburke5678 merged 1 commit intoopenshift:enterprise-4.12from
gwynnemonahan:manual-cp-4-12-no-19-OSDOCS-14862-rel-notes

Conversation

@gwynnemonahan
Copy link
Copy Markdown
Contributor

Cherry-pick: e298598

Original PR: #95552

Version(s):
4.12

QE review:
QE is not required for this PR.

Additional information:

@openshift-ci openshift-ci Bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jul 3, 2025
@ocpdocs-previewbot
Copy link
Copy Markdown

ocpdocs-previewbot commented Jul 3, 2025

@gwynnemonahan
Copy link
Copy Markdown
Contributor Author

/label merge-review-needed

@openshift-ci openshift-ci Bot added the merge-review-needed Signifies that the merge review team needs to review this PR label Jul 3, 2025
@mburke5678 mburke5678 added the merge-review-in-progress Signifies that the merge review team is reviewing this PR label Jul 7, 2025
Comment on lines +50 to +55
* Track IPsec using `--enable_ipsec`
* Value that determines the ratio of packets being sampled using `--sampling`
* Filter flows using a custom query using `--query`
* A comma separated list of interfaces to monitor using `--interfaces`
* A comma separated list of interfaces to exclude using `--exclude_interfaces`
* A comma separated list of metric names to generate using `--include_list`
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

List items should be parallel. Your list goes back and forth between verb and noun. Also a couple of grammar things:

Suggested change
* Track IPsec using `--enable_ipsec`
* Value that determines the ratio of packets being sampled using `--sampling`
* Filter flows using a custom query using `--query`
* A comma separated list of interfaces to monitor using `--interfaces`
* A comma separated list of interfaces to exclude using `--exclude_interfaces`
* A comma separated list of metric names to generate using `--include_list`
* Track IPsec using `--enable_ipsec`
* Configure the ratio of packets being sampled by using `--sampling`
* Filter flows using a custom query by using `--query`
* Specify interfaces to monitor by using `--interfaces`
* Specify interfaces to exclude by using `--exclude_interfaces`
* Specify metric names to generate by using `--include_list`

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Since this is a cherry-pick from a release from last week, this is being addressed with https://issues.redhat.com/browse/OSDOCS-15160

=== Notable technical changes
* The `NetworkEvents` feature in Network Observability 1.9 has been updated to work with the newer Linux kernel of {product-title} 4.19. This update breaks compatibility with older kernels. As a result, the `NetworkEvents` feature can only be used with {product-title} 4.19. If you are using this feature with Network Observability 1.8 and {product-title} 4.18, consider avoiding a Network Observability upgrade or upgrading Network Observability to 1.9 and {product-title} to 4.19.

* The `netobserv-reader` `clusterrole` has been renamed to `netobserv-loki-reader`.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This seems confusing to me. What about:

Suggested change
* The `netobserv-reader` `clusterrole` has been renamed to `netobserv-loki-reader`.
* The `netobserv-reader` cluster role has been renamed to `netobserv-loki-reader`.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Since this is a cherry-pick from a release from last week, this is being addressed with https://issues.redhat.com/browse/OSDOCS-15160

=== Technology Preview features
Some features in this release are currently in Technology Preview. These experimental features are not intended for production use. Note the following scope of support on the Red Hat Customer Portal for these features:

link:https://access.redhat.com/support/offerings/techpreview[Technology Preview Features Support Scope]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This link is included in the technology-preview.adoc snippet. It isn't needed here.

Suggested change
link:https://access.redhat.com/support/offerings/techpreview[Technology Preview Features Support Scope]

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@mburke5678
Copy link
Copy Markdown
Contributor

@gwynnemonahan I had a few suggestions that I hope you can look at before we merge.

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Jul 7, 2025

@gwynnemonahan: all tests passed!

Full PR test history. Your PR dashboard.

Details

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-sigs/prow repository. I understand the commands that are listed here.

@gwynnemonahan
Copy link
Copy Markdown
Contributor Author

gwynnemonahan commented Jul 7, 2025

Hey @mburke5678 ,

Comments are being addressed by https://issues.redhat.com/browse/OSDOCS-15160 which includes comments from the original PR. I'm leaving them unresolved so I can more quickly check them, and then resolve them when OSDOCS-15160 is done.

@mburke5678 mburke5678 merged commit 58599b7 into openshift:enterprise-4.12 Jul 7, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-review-in-progress Signifies that the merge review team is reviewing this PR merge-review-needed Signifies that the merge review team needs to review this PR size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants