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

[ExternalNode] Add ExternalNode CRD #3639

Merged

Conversation

mengdie-song
Copy link
Contributor

This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song songm@vmware.com

@mengdie-song
Copy link
Contributor Author

/test-all

@codecov-commenter
Copy link

codecov-commenter commented Apr 14, 2022

Codecov Report

Merging #3639 (00e425e) into feature/externalnode (4a3b76c) will decrease coverage by 8.97%.
The diff coverage is 100.00%.

Impacted file tree graph

@@                   Coverage Diff                    @@
##           feature/externalnode    #3639      +/-   ##
========================================================
- Coverage                 64.67%   55.70%   -8.98%     
========================================================
  Files                       278      392     +114     
  Lines                     39363    54809   +15446     
========================================================
+ Hits                      25459    30529    +5070     
- Misses                    11935    21911    +9976     
- Partials                   1969     2369     +400     
Flag Coverage Δ
integration-tests 38.36% <ø> (?)
kind-e2e-tests 49.07% <100.00%> (-3.36%) ⬇️
unit-tests 43.82% <ø> (-0.04%) ⬇️

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

Impacted Files Coverage Δ
pkg/apis/crd/v1alpha1/register.go 92.30% <100.00%> (+0.64%) ⬆️
...g/agent/apiserver/handlers/featuregates/handler.go 4.54% <0.00%> (-77.28%) ⬇️
pkg/controller/egress/store/egressgroup.go 1.72% <0.00%> (-54.32%) ⬇️
...kg/apiserver/registry/system/supportbundle/rest.go 22.17% <0.00%> (-50.44%) ⬇️
pkg/support/dump.go 7.90% <0.00%> (-49.16%) ⬇️
pkg/support/dump_others.go 0.00% <0.00%> (-44.74%) ⬇️
...gregator/apiserver/handlers/flowrecords/handler.go 1.44% <0.00%> (-37.69%) ⬇️
...egator/apiserver/handlers/recordmetrics/handler.go 4.00% <0.00%> (-36.00%) ⬇️
...g/agent/apiserver/handlers/addressgroup/handler.go 5.00% <0.00%> (-35.00%) ⬇️
...agent/apiserver/handlers/appliedtogroup/handler.go 5.00% <0.00%> (-35.00%) ⬇️
... and 147 more

Copy link
Contributor

@wenyingd wenyingd left a comment

Choose a reason for hiding this comment

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

LGTM overall

@@ -1,4 +1,4 @@
// Copyright 2021 Antrea Authors
Copy link
Contributor

@wenyingd wenyingd Apr 15, 2022

Choose a reason for hiding this comment

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

No need to change year?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

These files are generated automatically by make codegen so I think the change is expected.

// +genclient
// +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object

type ExternalNode struct {
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you add some comments here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks Jianjun, I have added a description, could you help take another look?

Copy link
Contributor

Choose a reason for hiding this comment

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

The current version works for me. Or you can add "bare-metal server" too.

@mengdie-song
Copy link
Contributor Author

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

@jianjuns
Copy link
Contributor

After #3578, we use Helm to generate the YAMLs. You need to re-generate the YAMLs.

This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
@mengdie-song
Copy link
Contributor Author

/test-all

@jianjuns jianjuns merged commit 78264c6 into antrea-io:feature/externalnode Apr 20, 2022
wenyingd pushed a commit that referenced this pull request Apr 26, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
wenyingd pushed a commit to wenyingd/antrea that referenced this pull request Apr 29, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
wenyingd pushed a commit to wenyingd/antrea that referenced this pull request May 7, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
wenyingd pushed a commit to wenyingd/antrea that referenced this pull request May 20, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
wenyingd pushed a commit to wenyingd/antrea that referenced this pull request May 20, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
wenyingd pushed a commit to wenyingd/antrea that referenced this pull request May 26, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
wenyingd pushed a commit to wenyingd/antrea that referenced this pull request Jun 1, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
wenyingd pushed a commit to wenyingd/antrea that referenced this pull request Jun 1, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
wenyingd pushed a commit to wenyingd/antrea that referenced this pull request Jun 9, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
wenyingd pushed a commit to wenyingd/antrea that referenced this pull request Jun 27, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
wenyingd pushed a commit to wenyingd/antrea that referenced this pull request Jun 28, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
wenyingd pushed a commit to wenyingd/antrea that referenced this pull request Jul 12, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
wenyingd pushed a commit to wenyingd/antrea that referenced this pull request Jul 27, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
mengdie-song added a commit to mengdie-song/antrea that referenced this pull request Aug 3, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
mengdie-song added a commit to mengdie-song/antrea that referenced this pull request Aug 3, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
mengdie-song added a commit to mengdie-song/antrea that referenced this pull request Aug 8, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
mengdie-song added a commit to mengdie-song/antrea that referenced this pull request Aug 8, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
mengdie-song added a commit to mengdie-song/antrea that referenced this pull request Aug 8, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
wenyingd pushed a commit to wenyingd/antrea that referenced this pull request Aug 10, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
wenyingd pushed a commit to wenyingd/antrea that referenced this pull request Aug 11, 2022
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
heanlan pushed a commit to heanlan/antrea that referenced this pull request Mar 29, 2023
This change adds API definition for ExternalNode CRD
and generates the corresponding client and yaml files.

Signed-off-by: Mengdie Song <songm@vmware.com>
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.

None yet

4 participants