-
Notifications
You must be signed in to change notification settings - Fork 366
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
✨ Update kcp sync command to support global kubernetes APIExport #2164
Conversation
2a0978e
to
af83817
Compare
af83817
to
77a519e
Compare
77a519e
to
8d3c5f6
Compare
pkg/reconciler/workload/defaultplacement/defaultplacement_controller.go
Outdated
Show resolved
Hide resolved
pkg/reconciler/workload/defaultplacement/defaultplacement_controller.go
Outdated
Show resolved
Hide resolved
2b0501c
to
1aa9bf2
Compare
1. use global kubernetes APIExport to sync by default 2. user can add additional export or resource using --apiexport and --resources flag 3. use "--resources deployment.apps --apiexport" keep the previous behavior
1. create only when at least on synctarget support it 2. skip adding certain APIResourceSchema to local kubernetes APIExport when at leat one synctarget supports global kubernetes APIExport
1aa9bf2
to
0ef43d4
Compare
@davidfestal I reorganize the commit, PTAL again. Thanks |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: davidfestal The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: davidfestal The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: Jian Qiu jqiu@redhat.com
Summary
This is to introduce the export flag in sync command. After this fix, the fix includes
After this change, the sync command will be changed:
kubectl kcp sync
without resource flag will sync resources in kubernetes APIExport in root:compute workspacekubectl kcp sync --resources xxx
will sync both resources specified in the flag and kubernetes APIExport in root:compute workspacekubectl kcp sync --apiexports workspace1|export1
will sync resources in specified export.kubectl kcp sync --resources xxx --apiexports
will sync resource xxx but not those in kubernetes APIExport in root:compute workspace.Related issue(s)
Fixes #1960