Release 2.2 #6766

luolanzone · 2024-10-24T05:33:21Z

No description provided.

CHANGELOG/CHANGELOG-2.2.md

hongliangl · 2024-10-28T07:04:19Z

CHANGELOG/CHANGELOG-2.2.md

+  - Refer to [this document](https://github.com/antrea-io/antrea/blob/release-2.2/docs/packetcapture-guide.md) for more information about this feature.
+- Add a few new antctl sub-commands for `BGPPolicy` feature to improve usability:
+  -  `antctl get bgppolicy` to get effective BGP policy applied on the Node. ([#6646](https://github.com/antrea-io/antrea/pull/6646), [@Atish-iaf])
+  -  `antctl get bgppeers` to print current status of all BGP peers of effective BGP policy applied on the local Node. ([#6689](https://github.com/antrea-io/antrea/pull/6689) [#6755](https://github.com/antrea-io/antrea/pull/6755), [@Atish-iaf])


Suggested change

- `antctl get bgppeers` to print current status of all BGP peers of effective BGP policy applied on the local Node. ([#6689](https://github.com/antrea-io/antrea/pull/6689) [#6755](https://github.com/antrea-io/antrea/pull/6755), [@Atish-iaf])

- `antctl get bgppeers` to print current status of all BGP peers of effective BGP policy applied on the local Node. ([#6689](https://github.com/antrea-io/antrea/pull/6689) [#6755](https://github.com/antrea-io/antrea/pull/6755), [@Atish-iaf])

"the current status of all BGP peers of the effective BGPPolicy on the local Node". Or maybe we can remove "on the local Node" to make it shorter.

Updated, but keep on the local Node for all BGP command PRs to keep consistent.

CHANGELOG/CHANGELOG-2.2.md

jianjuns · 2024-10-28T19:38:21Z

CHANGELOG/CHANGELOG-2.2.md

+  - Refer to [this document](https://github.com/antrea-io/antrea/blob/release-2.2/docs/packetcapture-guide.md) for more information about this feature.
+- Add a few new antctl sub-commands for `BGPPolicy` feature to improve usability:
+  -  `antctl get bgppolicy` to get effective BGP policy applied on the Node. ([#6646](https://github.com/antrea-io/antrea/pull/6646), [@Atish-iaf])
+  -  `antctl get bgppeers` to print current status of all BGP peers of effective BGP policy applied on the local Node. ([#6689](https://github.com/antrea-io/antrea/pull/6689) [#6755](https://github.com/antrea-io/antrea/pull/6755), [@Atish-iaf])


"the current status of all BGP peers of the effective BGPPolicy on the local Node". Or maybe we can remove "on the local Node" to make it shorter.

CHANGELOG/CHANGELOG-2.2.md

jianjuns · 2024-10-28T19:54:35Z

CHANGELOG/CHANGELOG-2.2.md

+- Fix the template ID not existing error in IPFIX exporter for `FlowAggregator`. ([#6630](https://github.com/antrea-io/antrea/pull/6630), [@antoninbas])
+- Fix an agent crash issue if the host interface is already attached to OVS bridge for `SecondaryNetwork`. ([#6666](https://github.com/antrea-io/antrea/pull/6666), [@xliuxu])
+- Delay the initialization of ARP / NDP responders to fix the `ServiceExternalIP` feature when `SecondaryNetwork` is enabled. ([#6700](https://github.com/antrea-io/antrea/pull/6700), [@xliuxu])
+- Fix a VM agent crash issue which is caused by a nil pointer exception of the config `SNATFullyRandomPorts`. ([#6748](https://github.com/antrea-io/antrea/pull/6748), [@wenyingd])


config -> config option

CHANGELOG/CHANGELOG-2.2.md

jianjuns · 2024-10-28T19:56:32Z

CHANGELOG/CHANGELOG-2.2.md

+- Run `IPPool` webhook handler when `SecondaryNetwork` is enabled. ([#6691](https://github.com/antrea-io/antrea/pull/6691), [@luolanzone])
+- Fix slice init length issue for `NetworkPolicy` controller. ([#6715](https://github.com/antrea-io/antrea/pull/6715), [@cuishuang])
+- Improve memory copying logic to avoid potential fault memory issue on Windows. ([#6664](https://github.com/antrea-io/antrea/pull/6664) [#6673](https://github.com/antrea-io/antrea/pull/6673), [@XinShuYang] [@tnqn])
+- Resolve a Windows agent crash issue which is caused by a nil pointer exception of the config `egressSNATRandomFully`. ([#6668](https://github.com/antrea-io/antrea/pull/6668), [@XinShuYang])


config option

Could we move this one to be together with the SNATFullyRandomPorts nil pointer issue?

Technically because the feature itself was introduced in this release, we don't need to mention the 2 corresponding fixes in the release notes, it just creates noise

Support --random-fully for iptables SNAT / MASQUERADE rules.

Removed two corresponding fixes in the release log.

CHANGELOG/CHANGELOG-2.2.md

antoninbas · 2024-10-29T17:33:32Z

@luolanzone this PR should have been opened against the release-2.2 branch, not main

luolanzone · 2024-10-31T06:15:05Z

@luolanzone this PR should have been opened against the release-2.2 branch, not main

Hi @antoninbas the release-2.2 branch is not ready yet. I think @tnqn can help to create it once the PacketCapture PRs are merged.

luolanzone · 2024-10-31T06:47:09Z

Hi @jianjuns @antoninbas @hongliangl , thanks for the review, I have addressed all comments, PTAL again.

antoninbas

a few more minor comments from me

CHANGELOG/CHANGELOG-2.2.md

antoninbas · 2024-10-31T21:54:14Z

CHANGELOG/CHANGELOG-2.2.md

+- More robust system Tier creation / update for Antrea-native policies. ([#6696](https://github.com/antrea-io/antrea/pull/6696), [@antoninbas])
+- Handle `ExternalIPPool` range changes in Egress controller. ([#6685](https://github.com/antrea-io/antrea/pull/6685), [@antoninbas])
+- Close connection to IPFIX collector explicitly on Stop for `FlowAggregator`. ([#6635](https://github.com/antrea-io/antrea/pull/6635), [@antoninbas])
+- Support `--random-fully` for iptables SNAT / MASQUERADE rules. ([#6602](https://github.com/antrea-io/antrea/pull/6602), [@antoninbas])


This is a new feature, with a new configuration switch, so should be in Added

antoninbas · 2024-10-31T21:54:44Z

CHANGELOG/CHANGELOG-2.2.md

+- Close connection to IPFIX collector explicitly on Stop for `FlowAggregator`. ([#6635](https://github.com/antrea-io/antrea/pull/6635), [@antoninbas])
+- Support `--random-fully` for iptables SNAT / MASQUERADE rules. ([#6602](https://github.com/antrea-io/antrea/pull/6602), [@antoninbas])
+- Unify the checker image and make it configurable when running `antctl check cluster`. ([#6579](https://github.com/antrea-io/antrea/pull/6579), [@tnqn])
+- Update the `Finalizer` of `ResourceExport` to be domain-qualified strings. ([#6742](https://github.com/antrea-io/antrea/pull/6742), [@Dyanngg])


to be a domain-qualified string

I think there is only one Finalizer in that case?

Yes, updated.

CHANGELOG/CHANGELOG-2.2.md

antoninbas · 2024-10-31T21:58:35Z

CHANGELOG/CHANGELOG-2.2.md

+- Document a workaround for using `EgressSeparateSubnet` feature on OpenShift. ([#6622](https://github.com/antrea-io/antrea/pull/6622), [@luolanzone])
+- Clean up stale resources when `antctl check cluster` failed. ([#6597](https://github.com/antrea-io/antrea/pull/6597), [@luolanzone])
+- Fix hint annotation implementation in `AntreaProxy`. ([#6607](https://github.com/antrea-io/antrea/pull/6607), [@hongliangl])
+- Add initialization of `creationTimestamp` when creating instances of `NodeLatencyStats` to fix a null `creationTimestamp` issue. ([#6574](https://github.com/antrea-io/antrea/pull/6574), [@hkiiita])


remove "to fix a ..."

I refined this sentence to the following:

Initialize `creationTimestamp` when creating instances of `NodeLatencyStats` to prevent a null `creationTimestamp` issue

Signed-off-by: Lan Luo <[email protected]>

luolanzone requested review from antoninbas, tnqn, rajnkamr, jianjuns, xliuxu and hongliangl October 24, 2024 05:34

rajnkamr reviewed Oct 24, 2024

View reviewed changes

CHANGELOG/CHANGELOG-2.2.md Outdated Show resolved Hide resolved

antoninbas reviewed Oct 24, 2024

View reviewed changes

luolanzone force-pushed the release-2.2-log branch 2 times, most recently from d4ce81c to e808f8f Compare October 25, 2024 03:12

Atish-iaf reviewed Oct 25, 2024

View reviewed changes

CHANGELOG/CHANGELOG-2.2.md Outdated Show resolved Hide resolved

luolanzone force-pushed the release-2.2-log branch from e808f8f to 3b946c9 Compare October 28, 2024 07:36

hongliangl reviewed Oct 28, 2024

View reviewed changes

antrea-io deleted a comment from antoninbas Oct 28, 2024

jianjuns reviewed Oct 28, 2024

View reviewed changes

CHANGELOG/CHANGELOG-2.2.md Show resolved Hide resolved

hongliangl reviewed Oct 29, 2024

View reviewed changes

CHANGELOG/CHANGELOG-2.2.md Outdated Show resolved Hide resolved

CHANGELOG/CHANGELOG-2.2.md Outdated Show resolved Hide resolved

luolanzone force-pushed the release-2.2-log branch from 3b946c9 to 5a514e2 Compare October 31, 2024 06:40

antoninbas reviewed Oct 31, 2024

View reviewed changes

luolanzone force-pushed the release-2.2-log branch from 5a514e2 to c956e0a Compare November 1, 2024 01:45

luolanzone added 2 commits November 1, 2024 09:50

Update CHANGELOG for v2.2.0 release

75ae6be

Signed-off-by: Lan Luo <[email protected]>

Set VERSION to v2.2.0

1e27276

Signed-off-by: Lan Luo <[email protected]>

luolanzone force-pushed the release-2.2-log branch from c956e0a to 1e27276 Compare November 1, 2024 01:51

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Release 2.2 #6766

Release 2.2 #6766

luolanzone commented Oct 24, 2024

hongliangl Oct 28, 2024

jianjuns Oct 28, 2024

luolanzone Oct 31, 2024 •

edited

Loading

jianjuns Oct 28, 2024

jianjuns Oct 28, 2024

jianjuns Oct 28, 2024

antoninbas Oct 28, 2024

luolanzone Oct 31, 2024

antoninbas commented Oct 29, 2024

luolanzone commented Oct 31, 2024

luolanzone commented Oct 31, 2024

antoninbas left a comment

antoninbas Oct 31, 2024

luolanzone Nov 1, 2024

antoninbas Oct 31, 2024

luolanzone Nov 1, 2024

antoninbas Oct 31, 2024

luolanzone Nov 1, 2024

	- `antctl get bgppeers` to print current status of all BGP peers of effective BGP policy applied on the local Node. ([#6689](https://github.com/antrea-io/antrea/pull/6689) [#6755](https://github.com/antrea-io/antrea/pull/6755), [@Atish-iaf])
	- `antctl get bgppeers` to print current status of all BGP peers of effective BGP policy applied on the local Node. ([#6689](https://github.com/antrea-io/antrea/pull/6689) [#6755](https://github.com/antrea-io/antrea/pull/6755), [@Atish-iaf])

Release 2.2 #6766

Are you sure you want to change the base?

Release 2.2 #6766

Conversation

luolanzone commented Oct 24, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

luolanzone Oct 31, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

antoninbas commented Oct 29, 2024

luolanzone commented Oct 31, 2024

luolanzone commented Oct 31, 2024

antoninbas left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

luolanzone Oct 31, 2024 •

edited

Loading