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 a new all-in-one manifest for the Multi-cluster leader cluster #5389

Merged
merged 1 commit into from
Aug 22, 2023

Conversation

luolanzone
Copy link
Contributor

@luolanzone luolanzone commented Aug 15, 2023

For the multi-cluster leader cluster deployment, we now provide two separate
manifests for CRDs and namespaced Controller deployment respectively.
However this is not the most convenient way for users, so we add a new
all-in-one manifest to simplify the deployment.

@luolanzone luolanzone added the area/multi-cluster Issues or PRs related to multi cluster. label Aug 15, 2023
@luolanzone
Copy link
Contributor Author

/test-multicluster-e2e

@luolanzone luolanzone added the action/release-note Indicates a PR that should be included in release notes. label Aug 15, 2023
@luolanzone luolanzone force-pushed the merge-leader-manifest branch from ce768ce to f5b8de2 Compare August 15, 2023 02:45
@luolanzone
Copy link
Contributor Author

/test-multicluster-e2e

@luolanzone luolanzone requested a review from jianjuns August 17, 2023 07:43
@luolanzone
Copy link
Contributor Author

/test-all

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.

Thanks for working on this.

docs/multicluster/user-guide.md Outdated Show resolved Hide resolved
docs/multicluster/user-guide.md Outdated Show resolved Hide resolved
docs/multicluster/user-guide.md Outdated Show resolved Hide resolved
For the multi-cluster leader cluster deployment, we provide two separate
manifests for CRDs and namespaced Controller deployment. However this is
not a typical way for OSS users, so we add a new all-in-one manifest to
simplify the deployment.

Signed-off-by: Lan Luo <luola@vmware.com>
@luolanzone luolanzone force-pushed the merge-leader-manifest branch from f5b8de2 to c6396d8 Compare August 21, 2023 04:08
@luolanzone
Copy link
Contributor Author

/test-all
/test-multicluster-e2e

@jianjuns
Copy link
Contributor

/skip-all

@jianjuns
Copy link
Contributor

/test-multicluster-e2e

@jianjuns
Copy link
Contributor

/test-multicluster-e2e

@jianjuns
Copy link
Contributor

@luolanzone : could you check if multi-cluster e2e keeps failing?

@luolanzone
Copy link
Contributor Author

/test-multicluster-e2e

1 similar comment
@luolanzone
Copy link
Contributor Author

/test-multicluster-e2e

@luolanzone
Copy link
Contributor Author

@jianjuns there was a testbed issue, the MC test is passed now after rerun.

@jianjuns jianjuns merged commit 285f169 into antrea-io:main Aug 22, 2023
@luolanzone luolanzone deleted the merge-leader-manifest branch September 26, 2023 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action/release-note Indicates a PR that should be included in release notes. area/multi-cluster Issues or PRs related to multi cluster.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants