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

Automated cherry pick of #2730: Fix panic bug in Antrea Policy #2758

Conversation

wenyingd
Copy link
Contributor

Cherry pick of #2730 on release-1.2.

#2730: Fix panic bug in Antrea Policy

For details on the cherry pick process, see the cherry pick requests page.

@wenyingd wenyingd added the kind/cherry-pick Categorizes issue or PR as related to the cherry-pick of a bug fix from the main branch to a release label Sep 13, 2021
@wenyingd
Copy link
Contributor Author

/test-all
/test-ipv6-all
/test-ipv6-only-all

Check if the clause pointer is nil before looping the matches.

Signed-off-by: wenyingd <wenyingd@vmware.com>
@wenyingd wenyingd force-pushed the automated-cherry-pick-of-#2730-upstream-release-1.2 branch from 32d7ec8 to 6c7b8c2 Compare September 13, 2021 02:53
@codecov-commenter
Copy link

codecov-commenter commented Sep 13, 2021

Codecov Report

Merging #2758 (6c7b8c2) into release-1.2 (6d7478f) will decrease coverage by 3.58%.
The diff coverage is 50.00%.

Impacted file tree graph

@@               Coverage Diff               @@
##           release-1.2    #2758      +/-   ##
===============================================
- Coverage        60.12%   56.53%   -3.59%     
===============================================
  Files              284      284              
  Lines            22385    24748    +2363     
===============================================
+ Hits             13458    13991     +533     
- Misses            7493     9276    +1783     
- Partials          1434     1481      +47     
Flag Coverage Δ
e2e-tests 13.06% <0.00%> (?)
kind-e2e-tests 47.15% <0.00%> (+0.02%) ⬆️
unit-tests 42.52% <70.00%> (+0.33%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pkg/agent/openflow/network_policy.go 71.77% <50.00%> (-4.30%) ⬇️
pkg/ovs/openflow/default.go 50.00% <0.00%> (-50.00%) ⬇️
pkg/util/runtime/runtime.go 50.00% <0.00%> (-50.00%) ⬇️
pkg/ovs/ovsconfig/default.go 50.00% <0.00%> (-50.00%) ⬇️
pkg/controller/types/networkpolicy.go 50.00% <0.00%> (-50.00%) ⬇️
pkg/agent/config/traffic_encap_mode.go 60.86% <0.00%> (-39.14%) ⬇️
pkg/util/sets/string.go 61.53% <0.00%> (-38.47%) ⬇️
pkg/agent/config/node_config.go 63.63% <0.00%> (-36.37%) ⬇️
pkg/agent/proxy/service.go 64.28% <0.00%> (-35.72%) ⬇️
pkg/apis/controlplane/v1beta2/sets.go 37.87% <0.00%> (-23.10%) ⬇️
... and 205 more

@wenyingd
Copy link
Contributor Author

/test-all
/test-ipv6-all
/test-ipv6-only-all

@wenyingd wenyingd requested a review from tnqn September 13, 2021 03:53
@wenyingd
Copy link
Contributor Author

/test-e2e
/test-ipv6-only-conformance
/test-windows-conformance
/test-windows-networkpolicy

@tnqn tnqn merged commit f51a21a into antrea-io:release-1.2 Sep 14, 2021
@wenyingd wenyingd deleted the automated-cherry-pick-of-#2730-upstream-release-1.2 branch October 8, 2021 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/cherry-pick Categorizes issue or PR as related to the cherry-pick of a bug fix from the main branch to a release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants