Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release 1.12.0 #5014

Merged
merged 2 commits into from
May 25, 2023
Merged

Release 1.12.0 #5014

merged 2 commits into from
May 25, 2023

Conversation

luolanzone
Copy link
Contributor

No description provided.

@luolanzone luolanzone requested review from tnqn, antoninbas and jianjuns and removed request for tnqn May 22, 2023 05:59
Copy link
Member

@tnqn tnqn left a comment

Choose a reason for hiding this comment

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

Note not all PRs are labelled properly. I have commented several changes I thought of but there are probably more. "--all" should be specified when generating the change log.

CHANGELOG/CHANGELOG-1.12.md Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
@luolanzone luolanzone force-pushed the release-1.12-note branch 2 times, most recently from 0a99d75 to ffba166 Compare May 23, 2023 08:01
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
- Add WireGuard tunnel mode for Antrea Multi-cluster to support encryption of the traffic between member clusters. ([#4737](https://github.com/antrea-io/antrea/pull/4737) [#4606](https://github.com/antrea-io/antrea/pull/4606), [@hjiajing])
- Add WireGuard encryption tunnel mode to Multi-cluster user guide. ([#4848](https://github.com/antrea-io/antrea/pull/4848), [@hjiajing])
- Add support for EndpointSlice API for Multi-cluster Services. When the EndpointSlice API is available for the cluster, EndpointSlice resources of the exported Service, rather than the Endpoints resource, will be watched. For the Pod IP type of Services, EndpointSlice resources will be exported as the Multi-cluster Service's endpoints. ([#4895](https://github.com/antrea-io/antrea/pull/4895), [@luolanzone])
- Add IP based group association query API and ClientSet for querying the Antrea Group/ClusterGroups. ([#4807](https://github.com/antrea-io/antrea/pull/4807), [@Dyanngg])
Copy link
Member

Choose a reason for hiding this comment

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

This is policy related, should be moved to similar changes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Moved

CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
- Add a new field 'logLabel' to ACNP and ANP CRDs to specify firewall log label for Audit Logging. ([#4748](https://github.com/antrea-io/antrea/pull/4748), [@qiyueyao])
- Add a new Controller configuration 'clientCAFile' to allow user to specify client CA. ([#4664](https://github.com/antrea-io/antrea/pull/4664), [@wenyingd])
- Add support for ExternalIP in AntreaProxy to allow a Service to be accessed from outside the cluster using an external IP address. ([#4866](https://github.com/antrea-io/antrea/pull/4866), [@hongliangl])
- Add OVS table name as label for 'ovs_flow_count' Prometheus metrics. ([#4893](https://github.com/antrea-io/antrea/pull/4893), [@cr7258])
Copy link
Member

Choose a reason for hiding this comment

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

"Changed" is more proper for this. (It just adds a new label in an existing metric)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Moved to Changed

CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
@tnqn
Copy link
Member

tnqn commented May 23, 2023

The following PRs merged today should be included:
#4876
#5017


## 1.12.0 - 2023-05-24

- The Multicast, TopologyAwareHints, NodeIPAM features are graduated from Alpha to Beta and therefore enabled by default.
Copy link
Member

Choose a reason for hiding this comment

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

We should mention the new multicast.enable configuration parameter.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added.

CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
Copy link
Contributor

@antoninbas antoninbas left a comment

Choose a reason for hiding this comment

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

small comments and suggestions, otherwise lgtm


## 1.12.0 - 2023-05-24

- The Multicast, TopologyAwareHints, NodeIPAM features are graduated from Alpha to Beta. The TopologyAwareHints, NodeIPAM features are enabled by default. Multicast can be enabled by a new configuration parameter `multicast.enable`.
Copy link
Contributor

Choose a reason for hiding this comment

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

can be enabled with a new Antrea Agent configuration parameter: multicast.enable.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

### Added

- Add two new fields `sourcePort` and `sourceEndPort` in Antrea-native policy API to match traffic initiated from specific ports. ([#4687](https://github.com/antrea-io/antrea/pull/4687), [@Dyanngg])
- Add a new field `logLabel` to ACNP and ANP CRDs to specify firewall log label for Audit Logging. ([#4748](https://github.com/antrea-io/antrea/pull/4748), [@qiyueyao])
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Add a new field `logLabel` to ACNP and ANP CRDs to specify firewall log label for Audit Logging. ([#4748](https://github.com/antrea-io/antrea/pull/4748), [@qiyueyao])
- Add a new field `logLabel` to Antrea-native policy CRDs; the user-provided label is added to audit logs. ([#4748](https://github.com/antrea-io/antrea/pull/4748), [@qiyueyao])

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated.

CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
CHANGELOG/CHANGELOG-1.12.md Outdated Show resolved Hide resolved
@luolanzone luolanzone force-pushed the release-1.12-note branch 2 times, most recently from c47be31 to d348849 Compare May 24, 2023 05:39
@luolanzone
Copy link
Contributor Author

/test-whole-conformance
/skip-conformance
/test-networkpolicy
/test-e2e
/test-flexible-ipam-e2e
/test-multicluster-e2e
/test-multicast-e2e
/test-windows-all
/test-ipv6-all
/test-ipv6-only-all
/test-vm-e2e

@wenyingd
Copy link
Contributor

Is this one missed in fixed? #4941

@tnqn
Copy link
Member

tnqn commented May 24, 2023

Is this one missed in fixed? #4941

#4941 is not required as it fixes issues introduced by #3889. But #3889 is not added even it has the release-note label. Did you remove some labelled items? @luolanzone

@luolanzone
Copy link
Contributor Author

@tnqn no, I didn't remove the label, but I probably thought it's not a user-faced improvement and remove it when I re-org the release log. Let me add it back.

@tnqn
Copy link
Member

tnqn commented May 24, 2023

The PR description didn't reflect the impact properly. The description for #3889 in my mind:

Unify AntreaProxy behavior across Linux and Windows. Windows agents now configure only a single route for all Service ClusterIPs and can restore routes after they are deleted by accident.

@tnqn
Copy link
Member

tnqn commented May 24, 2023

I remember there are a number of bugfix for Multicast.

@wenyingd
Copy link
Contributor

wenyingd commented May 24, 2023

I remember there are a number of bugfix for Multicast.

These two are resolving concurrency failures in multicast feature. #4903 #4870
This one is resolving a IPv4 packet unmarshaling issue when options exist in IP header #5015, but the related issue is observed after switching to packetIn2, so I don't think it is required in the release notes.

@luolanzone
Copy link
Contributor Author

Thanks for reminder, I am adding these PRs into release log, will push a new version soon.

Signed-off-by: Lan Luo <luola@vmware.com>
Signed-off-by: Lan Luo <luola@vmware.com>
@luolanzone luolanzone changed the base branch from main to release-1.12 May 24, 2023 07:05
@luolanzone
Copy link
Contributor Author

/test-whole-conformance
/skip-conformance
/test-networkpolicy
/test-e2e
/test-flexible-ipam-e2e
/test-multicluster-e2e
/test-multicast-e2e
/test-windows-all
/test-ipv6-all
/test-ipv6-only-all
/test-vm-e2e

@ceclinux
Copy link
Contributor

/test-multicast-e2e

1 similar comment
@ceclinux
Copy link
Contributor

/test-multicast-e2e

@luolanzone
Copy link
Contributor Author

/test-networkpolicy

@XinShuYang
Copy link
Contributor

/test-whole-conformance

@luolanzone
Copy link
Contributor Author

/test-flexible-ipam-e2e

@luolanzone
Copy link
Contributor Author

/test-latest-e2e

@luolanzone
Copy link
Contributor Author

/test-ipv6-only-e2e

@luolanzone
Copy link
Contributor Author

/test-windows-containerd-e2e

@luolanzone
Copy link
Contributor Author

There is a known issue for ipv6-only-networkpolicy conformance test and tracked by #4887, we may ignore the failure atm.

@luolanzone
Copy link
Contributor Author

/test-vm-e2e

@XinShuYang
Copy link
Contributor

/test-latest-e2e

@luolanzone
Copy link
Contributor Author

/test-latest-networkpolicy
/test-latest-conformance

Copy link
Member

@tnqn tnqn left a comment

Choose a reason for hiding this comment

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

LGTM

@XinShuYang
Copy link
Contributor

/test-latest-networkpolicy
/test-latest-conformance

@wenyingd
Copy link
Contributor

/test-vm-e2e

@XinShuYang
Copy link
Contributor

/test-networkpolicy
/test-conformance
/test-whole-conformance

@luolanzone
Copy link
Contributor Author

/test-ipv6-only-networkpolicy

@tnqn tnqn merged commit 8c08547 into antrea-io:release-1.12 May 25, 2023
@luolanzone luolanzone deleted the release-1.12-note branch June 28, 2023 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants