-
Notifications
You must be signed in to change notification settings - Fork 404
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 more logging when ns deletion fails to work #1206
Add more logging when ns deletion fails to work #1206
Conversation
797ac0f
to
1b99beb
Compare
8db9172
to
38fb097
Compare
@@ -41,11 +41,17 @@ func deleteNamespace(namespace string) { | |||
for { | |||
select { | |||
case <-timeout: | |||
cmd := exec.Command("kubectl", "get", "ns", namespace, "-o yaml") |
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.
It will not solve our problem. Printing namespace details tell us only that namespace has "kubernetes" finalizer - but still, we don't know which object from this namespace cannot be removed.
Could you analize "namespaced_resources_deleter.go" (https://github.com/kubernetes/kubernetes/blob/7f23a743e8c23ac6489340bbb34fa6f1d392db9d/pkg/controller/namespace/deletion/namespaced_resources_deleter.go), especially deleteAllContent method. They are using the dynamic client to detect what should be deleted. Maybe we can do the same and just print resources which are still not deleted after the timeout (6min)
0094b4f
to
3185738
Compare
- Added force deletion of all k8s resources - Added extensive logging while deletion of ns times out
3185738
to
c7d4408
Compare
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.
lgtm
* Added pre-master-kyma-gke-integration-xip-dubg job. * Added debug job tests in separate function due to conflicts with other presubmit jobs settings.
* Added pre-master-kyma-gke-integration-xip-dubg job. * Added missing base_ref value for extra_refs. * Moved debug job tests to separate function due to conflicts with other presubmit jobs settings. * Changed branch name. * Changed SkipReport test expected value. * Reverted CLEANUP_CLUSTER value. It will be changed after merge. * Disabled cluster cleanup. * Enabled xip.io DNS for kyma-gke-integration-xip-debug job. * Corrected typo in job template name.
Description
This pull request add some logging to test-kubeless, in order to identify a possible reason for the time out when namespaces are being deleted.
Related issue(s)