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

Improve ci/kind/kind-setup.sh usage #2937

Merged
merged 1 commit into from
Nov 9, 2021

Conversation

antoninbas
Copy link
Contributor

  • Default "--antrea-cni" to false: most of the time, it is actually how
    we use it, and it is how users need to use the script.
  • Check-in build/yamls/antrea-kind.yml to avoid user confusion.
  • Improve Antrea with Kind documentation.

Fixes #2934

Signed-off-by: Antonin Bas abas@vmware.com

@antoninbas
Copy link
Contributor Author

@jayunit100 PTAL

ci/kind/kind-setup.sh Outdated Show resolved Hide resolved
@antoninbas antoninbas force-pushed the improve-kind-setup.sh-usage branch 3 times, most recently from 6e9ac0c to 98d6b73 Compare October 26, 2021 21:19
@codecov-commenter
Copy link

codecov-commenter commented Oct 26, 2021

Codecov Report

Merging #2937 (ee9762b) into main (ac6413f) will increase coverage by 20.86%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #2937       +/-   ##
===========================================
+ Coverage   40.19%   61.06%   +20.86%     
===========================================
  Files         166      289      +123     
  Lines       20713    24551     +3838     
===========================================
+ Hits         8326    14992     +6666     
+ Misses      11574     7935     -3639     
- Partials      813     1624      +811     
Flag Coverage Δ
kind-e2e-tests 48.23% <ø> (?)
unit-tests 40.15% <ø> (-0.04%) ⬇️

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

Impacted Files Coverage Δ
pkg/apiserver/certificate/certificate.go 69.86% <0.00%> (-6.85%) ⬇️
pkg/agent/cniserver/ipam/antrea_ipam_controller.go 61.53% <0.00%> (-3.85%) ⬇️
pkg/apiserver/handlers/webhook/convert_crd.go 2.56% <0.00%> (ø)
pkg/apiserver/handlers/webhook/mutation_labels.go 24.71% <0.00%> (ø)
...formers/externalversions/ops/v1alpha1/traceflow.go 64.28% <0.00%> (ø)
...ters/clusterinformation/v1beta1/antreaagentinfo.go 0.00% <0.00%> (ø)
...erinformation/v1beta1/clusterinformation_client.go 50.00% <0.00%> (ø)
pkg/agent/nodeportlocal/rules/iptable_rule.go 54.09% <0.00%> (ø)
pkg/apiserver/handlers/loglevel/handler.go 38.46% <0.00%> (ø)
pkg/apis/crd/v1alpha2/webhook.go 0.00% <0.00%> (ø)
... and 233 more

@antoninbas
Copy link
Contributor Author

@jianjuns could I get a review on this?

Copy link
Contributor

@jianjuns jianjuns left a comment

Choose a reason for hiding this comment

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

A typo.

docs/kind.md Outdated Show resolved Hide resolved
* Default "--antrea-cni" to false: most of the time, it is actually how
  we use it, and it is how users need to use the script.
* Check-in build/yamls/antrea-kind.yml to avoid user confusion.
* Improve Antrea with Kind documentation.

Fixes antrea-io#2934

Signed-off-by: Antonin Bas <abas@vmware.com>
@antoninbas
Copy link
Contributor Author

/skip-all

@antoninbas antoninbas merged commit 201e71b into antrea-io:main Nov 9, 2021
@antoninbas antoninbas deleted the improve-kind-setup.sh-usage branch November 9, 2021 05:21
qiyueyao added a commit to Dyanngg/antrea that referenced this pull request Nov 9, 2021
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.

We should pin ci/kind/
4 participants