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.

Sign up for GitHub

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

Jump to bottom

[k8s.io] Garbage collector [Feature:GarbageCollector] should not delete dependents that have both valid owner and owner that's waiting for dependents to be deleted #16878

Closed
bparees opened this issue Oct 15, 2017 · 4 comments
Closed

[k8s.io] Garbage collector [Feature:GarbageCollector] should not delete dependents that have both valid owner and owner that's waiting for dependents to be deleted #16878

bparees opened this issue Oct 15, 2017 · 4 comments
Assignees
@tnozicka
Labels
component/kubernetes kind/test-flake Categorizes issue or PR as related to test flakes. priority/P1

Comments

@bparees
Copy link
Contributor

bparees commented Oct 15, 2017 

/tmp/openshift/build-rpm-release/tito/rpmbuild-originJsGQEK/BUILD/origin-3.7.0/_output/local/go/src/github.com/openshift/origin/vendor/k8s.io/kubernetes/test/e2e/garbage_collector.go:681
Oct 14 18:31:56.475: failed to delete rc simpletest-rc-to-be-deleted, err: timed out waiting for the condition
/tmp/openshift/build-rpm-release/tito/rpmbuild-originJsGQEK/BUILD/origin-3.7.0/_output/local/go/src/github.com/openshift/origin/vendor/k8s.io/kubernetes/test/e2e/garbage_collector.go:661

https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/16857/test_pull_request_origin_extended_conformance_gce/9718/
@bparees bparees added component/kubernetes kind/test-flake Categorizes issue or PR as related to test flakes. priority/P1 labels Oct 15, 2017
@bparees
Copy link
Contributor Author

bparees commented Oct 15, 2017

@mfojtik i've landed :)

@mfojtik mfojtik assigned tnozicka and unassigned mfojtik Oct 16, 2017
@smarterclayton
Copy link
Contributor

Fixing

@smarterclayton
Copy link
Contributor

In #16978

@smarterclayton smarterclayton mentioned this issue Oct 20, 2017
Merged
openshift-merge-robot added a commit that referenced this issue Oct 20, 2017
@openshift-merge-robot
Merge pull request #16978 from smarterclayton/garbage
f0d322f 
Automatic merge from submit-queue (batch tested with PRs 16912, 16931, 16939, 16967, 16978).

UPSTREAM: 54308: The garbage collector creates too many pods

Fixes #16878 #16838

Backport kubernetes/kubernetes#54308

We created too many pods
@php-coder
Copy link
Contributor

It has to be fixed but I see it right now: https://openshift-gce-devel.appspot.com/build/origin-ci-test/pr-logs/pull/17296/test_pull_request_origin_extended_conformance_gce/11330/

@php-coder php-coder mentioned this issue Nov 15, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tnozicka tnozicka

Labels
component/kubernetes kind/test-flake Categorizes issue or PR as related to test flakes. priority/P1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@mfojtik @php-coder @smarterclayton @bparees @tnozicka