-
Notifications
You must be signed in to change notification settings - Fork 78
Bug 2071941: Replace collect-profile jobs that haven't completed #276
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
Conversation
|
@perdasilva: This pull request references Bugzilla bug 2071941, which is invalid:
Comment DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Signed-off-by: Per da Silva <perdasilva@gmail.com>
|
/cherry-pick release-4.10 |
|
@perdasilva: once the present PR merges, I will cherry-pick it on top of relase-4.10 in a new PR and assign it to you. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
/cherry-pick release-4.10 |
|
@perdasilva: once the present PR merges, I will cherry-pick it on top of release-4.10 in a new PR and assign it to you. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: perdasilva The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
/hold wait until next week - 4.9 branch is picky atm |
|
/test e2e-gcp |
|
@perdasilva: This pull request references Bugzilla bug 2071941, which is invalid:
Comment DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
@perdasilva: This pull request references Bugzilla bug 2071941, which is invalid:
Comment DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
@perdasilva: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
… environments (openshift#276) * Updated .gitignore to ignore vscode files. * Added validation for node affinity boundaries in the CSV. Upstream-repository: api Upstream-commit: 0ab3debaba2afcffcf16827798c876c7f35a9477
… environments (openshift#276) * Updated .gitignore to ignore vscode files. * Added validation for node affinity boundaries in the CSV. Upstream-repository: api Upstream-commit: 0ab3debaba2afcffcf16827798c876c7f35a9477
… environments (openshift#276) * Updated .gitignore to ignore vscode files. * Added validation for node affinity boundaries in the CSV. Upstream-repository: api Upstream-commit: 0ab3debaba2afcffcf16827798c876c7f35a9477
… environments (openshift#276) * Updated .gitignore to ignore vscode files. * Added validation for node affinity boundaries in the CSV.
… environments (openshift#276) * Updated .gitignore to ignore vscode files. * Added validation for node affinity boundaries in the CSV. Upstream-repository: api Upstream-commit: 0ab3debaba2afcffcf16827798c876c7f35a9477
Signed-off-by: grokspawn <jordan@nimblewidget.com>
Signed-off-by: grokspawn <jordan@nimblewidget.com>
Problem: The collect-profiles job should only take a few seconds to
run. There are instances, such as when the pod cannot be scheduled,
where the job will not complete in a reasonable amount of time. If
enough jobs are scheduled but unable to run, the number of
scheduled jobs can exceed pod quota limits.
Solution: Given that the collect-profiles job should only take a few
seconds to run and that the job is scheduled to run every 15 minutes,
set the collect-profiles cronJob's spec.concurrencyPolicy to
"Replace" so that only one active collect-profiles pod exists at any
time.