Skip to content

Commit

Permalink
[multicast] fix the issue multicast does not work if antreaPolicy is …
Browse files Browse the repository at this point in the history
…disabled

Currently, packetIn reason only supports 0 and 1. There is no more available packetIn reasons
as traceflow and antreaPolicy use both. Multicast reuses packetIn reason with antreaPolicy, and
uses a different register bit to indicate the real reason. There is no queue for packetIn reason
'PacketInReasonNP' if antreaPloicy is disabled. And multicast will not work then.

This patch fixes #3808

Signed-off-by: Bin Liu <biliu@vmware.com>
  • Loading branch information
liu4480 committed May 18, 2022
1 parent 3a51abe commit a2553c5
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion cmd/antrea-agent/agent.go
Expand Up @@ -638,7 +638,8 @@ func run(o *Options) error {
if features.DefaultFeatureGate.Enabled(features.Traceflow) {
packetInReasons = append(packetInReasons, uint8(openflow.PacketInReasonTF))
}
if features.DefaultFeatureGate.Enabled(features.AntreaPolicy) {
if features.DefaultFeatureGate.Enabled(features.AntreaPolicy) ||
features.DefaultFeatureGate.Enabled(features.Multicast) {
packetInReasons = append(packetInReasons, uint8(openflow.PacketInReasonNP))
}
if len(packetInReasons) > 0 {
Expand Down

0 comments on commit a2553c5

Please sign in to comment.