Skip to content
This repository has been archived by the owner on Jun 28, 2023. It is now read-only.

Add antrea-interworking package #5578

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

wenqiq
Copy link

@wenqiq wenqiq commented Nov 30, 2022

Signed-off-by: Wenqi Qiu wenqiq@vmware.com

What this PR does / why we need it

Add Antrea-interworking Package

Which issue(s) this PR fixes

Fixes: #

Describe testing done for PR

Special notes for your reviewer

@github-actions
Copy link

Hi @wenqiq! And thank you for opening your first Pull Request. Someone will review it soon. Thank you for committing to making Tanzu Community Edition better.

config:
#@schema/desc "echo -n 'dummyAdmin' | base64"
nsxUser: ZHVtbXlBZG1pbg==
#@schema/desc " echo -n 'dummyPassword' | base64"
Copy link
Contributor

Choose a reason for hiding this comment

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

No unified style for filed name? . like nsxUser. ClusterName --> clusterName?

Copy link
Author

Choose a reason for hiding this comment

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

done

#@schema/desc " "
ClusterName: dummyClusterName
#@schema/desc " "
NSXIP1: dummyNSXIP1
Copy link
Contributor

Choose a reason for hiding this comment

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

Does these default value make sense if applied?

Copy link
Author

Choose a reason for hiding this comment

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

updated

- anyMatcher:
matchers:
- kindNamespaceNameMatcher: {kind: APIService, namespace: kube-system, name: v1alpha1.stats.antrea.io}
- kindNamespaceNameMatcher: {kind: APIService, namespace: kube-system, name: v1alpha1.stats.antrea.tanzu.vmware.com}
Copy link
Contributor

Choose a reason for hiding this comment

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

No apiservice changes?

Copy link
Author

Choose a reason for hiding this comment

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

I have checked and there is no APIService in interworking.
Do you have any comments about this?@edwardbadboy

Copy link
Contributor

Choose a reason for hiding this comment

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

Correct. No APIService in interworking.

@hangyan
Copy link
Contributor

hangyan commented Dec 6, 2022

Also, yaml and lint tests should pass

@github-actions
Copy link

This PR is being marked stale due to a period of inactivty. If this PR is still relevant, please comment or remove the stale label. Otherwise, this PR will close in 30 days.

@github-actions github-actions bot added the Stale Labeled by Stale-bot label Feb 19, 2023
@github-actions github-actions bot removed the Stale Labeled by Stale-bot label Mar 11, 2023
Signed-off-by: Wenqi Qiu <wenqiq@vmware.com>
Signed-off-by: Wenqi Qiu <wenqiq@vmware.com>
Signed-off-by: Wenqi Qiu <wenqiq@vmware.com>
Signed-off-by: Wenqi Qiu <wenqiq@vmware.com>
Signed-off-by: Wenqi Qiu <wenqiq@vmware.com>
Signed-off-by: Wenqi Qiu <wenqiq@vmware.com>
Signed-off-by: Wenqi Qiu <wenqiq@vmware.com>
Signed-off-by: Wenqi Qiu <wenqiq@vmware.com>
Signed-off-by: Wenqi Qiu <wenqiq@vmware.com>
Signed-off-by: Wenqi Qiu <wenqiq@vmware.com>
Signed-off-by: Wenqi Qiu <wenqiq@vmware.com>
…ootstrapFrom

Signed-off-by: Wenqi Qiu <wenqiq@vmware.com>
Signed-off-by: Wenqi Qiu <wenqiq@vmware.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants