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.

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

Make critical jobs Guaranteed Pod QOS: pull-kubernetes-files-remake #18592

Closed
spiffxp opened this issue Aug 1, 2020 · 5 comments · Fixed by #18676
Closed

Make critical jobs Guaranteed Pod QOS: pull-kubernetes-files-remake #18592

spiffxp opened this issue Aug 1, 2020 · 5 comments · Fixed by #18676
Assignees
Labels
area/jobs area/release-eng Issues or PRs related to the Release Engineering subproject kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. sig/release Categorizes an issue or PR as relevant to SIG Release. sig/testing Categorizes an issue or PR as relevant to SIG Testing.

Comments

@spiffxp
Copy link
Member

spiffxp commented Aug 1, 2020

What should be cleaned up or changed:

This is part of #18530

The following jobs should be Guaranteed Pod QOS, meaning they should have CPU and memory resource limits, and matching resource requests:

  • pull-kubernetes-files-remake

These jobs run on (google.com only) k8s-prow-build, so @spiffxp has provided the following guess:

  • suggest 1500m cpu, 1 Gi mem

General steps to follow:

/sig testing
/sig release
/area jobs
/area release-eng

@spiffxp spiffxp added the kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. label Aug 1, 2020
@k8s-ci-robot k8s-ci-robot added sig/testing Categorizes an issue or PR as relevant to SIG Testing. sig/release Categorizes an issue or PR as relevant to SIG Release. area/jobs area/release-eng Issues or PRs related to the Release Engineering subproject labels Aug 1, 2020
@spiffxp spiffxp added the help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. label Aug 1, 2020
@BenTheElder
Copy link
Member

NOTE: this is now an optional presubmit. We're testing this in the periodic verify job

@RobertKielty
Copy link
Member

/remove-help
/assign

@k8s-ci-robot k8s-ci-robot removed the help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. label Aug 5, 2020
@spiffxp
Copy link
Member Author

spiffxp commented Aug 8, 2020

I'm going to call this done once #18752 merges, because then this job won't be merge-blocking anymore

@spiffxp
Copy link
Member Author

spiffxp commented Aug 8, 2020

/close

@k8s-ci-robot
Copy link
Contributor

@spiffxp: Closing this issue.

In response to this:

/close

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/jobs area/release-eng Issues or PRs related to the Release Engineering subproject kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. sig/release Categorizes an issue or PR as relevant to SIG Release. sig/testing Categorizes an issue or PR as relevant to SIG Testing.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants