-
Notifications
You must be signed in to change notification settings - Fork 376
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
🐛Syncer namespace cleanup refactor #2374
🐛Syncer namespace cleanup refactor #2374
Conversation
Skipping CI for Draft Pull Request. |
/test all |
169c9c8
to
0fbb67b
Compare
56ae1aa
to
9dfa2cc
Compare
/test e2e-sharded |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
29ca135
to
0c557cd
Compare
8e292c3
to
7c250c6
Compare
f1bb8d8
to
f4d8dcc
Compare
@@ -466,3 +463,11 @@ func getAllGVRs(synctarget *workloadv1alpha1.SyncTarget) map[schema.GroupVersion | |||
|
|||
return gvrs | |||
} | |||
|
|||
func IndexByNamespace(obj interface{}) ([]string, error) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we please use the upstream index.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks! done!
186ecc4
to
c368f96
Compare
eb92474
to
5a3d31c
Compare
/test e2e-sharded |
5a3d31c
to
b415bc4
Compare
/test e2e-sharded |
1 similar comment
/test e2e-sharded |
[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 |
Summary
Refactor how the syncer handles the deletion of downstream namespaces.
--downstream-namespace-clean-delay
flag and set it to 2s for tests.