-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
empty the pool, or delete the pool when endpoints are deleted #11940
Conversation
[test] |
@ramr |
@rajatchopra can you make the commit message include: That is what the bot looks for that update BZ. It's a rather hostile format for humans, so I prefer to say something like: Thanks |
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
de024b2
to
c515555
Compare
…1395545) empty the pool, or delete the pool when endpoints are deleted
c515555
to
b1d8e0b
Compare
[test] flake #11016 |
Evaluated for origin test up to b1d8e0b |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/11503/) (Base Commit: 3712aec) |
LGTM |
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
[merge] @rajatchopra we'll need to cherry pick over to 1.4. |
Evaluated for origin merge up to b1d8e0b |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/11527/) (Base Commit: 266bc63) (Image: devenv-rhel7_5378) |
(commit a631fd3) Fixes bug 1395545 (https://bugzilla.redhat.com/show_bug.cgi?id=1395545)
bz: https://bugzilla.redhat.com/show_bug.cgi?id=1395545
The pool was never getting deleted on a 'delete' event.