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

add mcs serviceexports and serviceimports #11

Merged
merged 1 commit into from
Aug 20, 2021

Conversation

luolanzone
Copy link
Collaborator

I think we need serviceexports and serviceimports crds from mcs, otherwise, user have to install it manually via k8s mcs for antrea multi-cluster features.

Signed-off-by: Lan Luo luola@vmware.com

@abhiraut
Copy link
Owner

agreed manual installation is not cool. i wonder what others do? perhaps an operator or some other tooling which must be installing these CRDs?

@@ -0,0 +1,133 @@
# Copyright 2020 The Kubernetes Authors.
Copy link
Owner

Choose a reason for hiding this comment

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

i don't think we put copyright header in yaml files.. at least don't see that in rest of antrea build/yamls

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ok, let me remove it.

@luolanzone
Copy link
Collaborator Author

@abhiraut I think if there is an operator, it should be done by operator to install these CRDs, submariner install them in operator. do you think we should introduce an operator now or leave it in current folder for demo?

Signed-off-by: Lan Luo <luola@vmware.com>
@abhiraut
Copy link
Owner

i think we keep it as is now.. but would like to explore the operator option in future..

type: string
port:
description: The port that will be exposed by this service.
format: int32
type: integer
protocol:
default: TCP
description: The IP protocol for this port. Supports
"TCP", "UDP", and "SCTP". Default is TCP.
description: The IP protocol for this port. Supports "TCP", "UDP", and "SCTP". Default is TCP.
Copy link
Owner

Choose a reason for hiding this comment

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

some of these changes look unrelated; probably because we didn't run make manifest in the past?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yes, I think I forgot to run make manifest last time. new labels are added now. the line format are new but it's controlled by generator, no change introduced in configuration.

@abhiraut abhiraut merged commit 192ce51 into abhiraut:staging/multi-cluster Aug 20, 2021
luolanzone added a commit that referenced this pull request Sep 10, 2021
Signed-off-by: Lan Luo <luola@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.

2 participants