-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Restore ability to manually run 5k tests on PR #34546
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
# Fork of ci-kubernetes-e2e-gce-scale-performance | ||
- name: ci-kubernetes-e2e-gce-scale-performance-manual | ||
cluster: k8s-infra-prow-build | ||
always_run: false |
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.
Taken always_run
, max_concurrency
, branches
from other optional presubmits like pull-kubernetes-kubemark-e2e-gce-scale
base_ref: master | ||
path_alias: k8s.io/perf-tests | ||
annotations: | ||
testgrid-dashboards: presubmits-kubernetes-scalability |
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.
Replaced annotations
- master | ||
- main | ||
decorate: true | ||
path_alias: k8s.io/kubernetes |
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.
Removed k8s.io/kubernetes from extra_refs and moved there as path_alias
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mborsz, serathius The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@serathius: Updated the
In 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-sigs/prow repository. |
- --env=KUBE_DNS_MEMORY_LIMIT=300Mi | ||
- --extract=ci/fast/latest-fast | ||
- --gcp-nodes=5000 | ||
- --gcp-project-type=scalability-scale-project |
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.
There is only one of these. So running this can disrupt the periodics?
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.
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.
Possibly, however we don't want to increase capacity for 5k tests.
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.
Clarification Note: We use SIG K8s Infra managed projects now, but we still need GCP to approve quota on a project like this, and I'm also not sure if we want to either.
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.
I'm concerned about disrupting the release-informing periodics though.
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.
Was this different before removing the Job? The goal of manual scalability tests is to test high risk performance changes, like new etcd minor release. I expect we only have 1-2 features like this in release cycle.
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.
Was this different before removing the Job? The goal of manual scalability tests is to test high risk performance changes, like new etcd minor release. I expect we only have 1-2 features like this in release cycle.
- --gcp-project-type=scalability-presubmit-5k-project
I guess we had another pool for 5k scale presubmit?
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.
kubernetes/k8s.io#7929 for rolling-forward when we can
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.
/assign @wojtek-t @mborsz