Skip to content

Conversation

slovern
Copy link
Contributor

@slovern slovern commented Oct 6, 2025

[TELCODOCS-2273](https://issues.redhat.com//browse/TELCODOCS-2273) - Release Note - Ununmbered BGP Peering goes from Tech Preview to GA

Version(s):4.20

Issue: https://issues.redhat.com/browse/TELCODOCS-2273

Link to docs preview: Unnumbered BGP peering

QE review:

  • QE has approved this change.

Additional information:

@openshift-ci openshift-ci bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 6, 2025
@ocpdocs-previewbot
Copy link

ocpdocs-previewbot commented Oct 6, 2025

🤖 Wed Oct 08 18:51:00 - Prow CI generated the docs preview:

https://100042--ocpdocs-pr.netlify.app/openshift-enterprise/latest/release_notes/ocp-4-20-release-notes.html

@karampok
Copy link

karampok commented Oct 7, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 7, 2025
@gkopels
Copy link

gkopels commented Oct 8, 2025

ltgm

==== Unnumbered BGP peering

With this release, {product-title} includes unnumbered BGP peering.
This was previosuly available as a Technology Preview feature.
Copy link
Contributor

Choose a reason for hiding this comment

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

typo: previously


* Simplified permissions: You can now simplify permissions and reduce operational overhead by using namespaced SR-IOV networks.

For more information, see xref:../networking/hardware_networks/configuring-namespaced-sriov-resources.html#configuring-namespaced-sriov-resources[Configuring namespaced SR-IOV resources].
Copy link
Contributor

Choose a reason for hiding this comment

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

Not yours but pretty sure it should be .adoc rather than .html in xref

Copy link
Contributor

Choose a reason for hiding this comment

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

That is my bad can you fix it when in here sorry @slovern should be extension .adoc

Copy link
Contributor

@kquinn1204 kquinn1204 Oct 8, 2025

Choose a reason for hiding this comment

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

I made the change in #100184 so can ignore

With this release, {product-title} includes unnumbered BGP peering.
This was previosuly available as a Technology Preview feature.
You can use the `spec.interface` field of the BGP peer custom resource to configure unnumbered BGP peering.

Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need to add xref: For more information, see xref:../networking/ingress_load_balancing/metallb/metallb-configure-bgp-peers.adoc#nw-metallb-example-unnumbered-bgp-peering_configure-metallb-bgp-peers[Example: Specify BGP peers for unnumbered BGP peering]

@slovern slovern force-pushed the RN-420-TELCODOCS-2273 branch from a6fc46d to 4138ca1 Compare October 8, 2025 18:27
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Oct 8, 2025
Copy link

openshift-ci bot commented Oct 8, 2025

New changes are detected. LGTM label has been removed.

@openshift-ci openshift-ci bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Oct 8, 2025
With this release, {product-title} includes unnumbered BGP peering.
This was previously available as a Technology Preview feature.
You can use the `spec.interface` field of the BGP peer custom resource to configure unnumbered BGP peering.
For more information, see xref:../networking/ingress_load_balancing/metallb/metallb/metallb-frr-k8s.html#nw-metallb-frrconfiguration-crd-interface[The interface field].
Copy link
Collaborator

Choose a reason for hiding this comment

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

🤖 [error] OpenShiftAsciiDoc.XrefContainsHTML: Use .adoc instead of .html in xrefs

Copy link

openshift-ci bot commented Oct 8, 2025

@slovern: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/validate-portal 4138ca1 link true /test validate-portal

Full PR test history. Your PR dashboard.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants