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

Delete gke-test gci-gke-test #6503

Merged
merged 1 commit into from
Jan 29, 2018
Merged

Delete gke-test gci-gke-test #6503

merged 1 commit into from
Jan 29, 2018

Conversation

fejta
Copy link
Contributor

@fejta fejta commented Jan 28, 2018

http://velodrome.k8s.io/dashboard/db/bigquery-metrics?orgId=1

These have been failing for 240 days straight, looks like they always time out

@kubernetes/sig-gcp-test-failures
ref #2528

@k8s-ci-robot k8s-ci-robot added area/provider/gcp Issues or PRs related to gcp provider approved Indicates a PR has been approved by an approver from all required OWNERS files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Jan 28, 2018
@krzyzacy
Copy link
Member

/assign @abgworrall

@fejta
Copy link
Contributor Author

fejta commented Jan 29, 2018

@krzyzacy
Copy link
Member

I think they are just timing out - all our gke OSS jobs are running in test environment so we should already have the coverage

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jan 29, 2018
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fejta, krzyzacy

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@abgworrall
Copy link
Contributor

Can you help me understand what these test jobs are supposed to do, in the greater scheme of things ? There are so many with such similar names that I get lost.
We need some GKE tests, no matter how poorly they've been attended to.

@k8s-ci-robot k8s-ci-robot merged commit 5c56e36 into kubernetes:master Jan 29, 2018
@k8s-ci-robot
Copy link
Contributor

@fejta: I updated Prow config for you!

In response to this:

http://velodrome.k8s.io/dashboard/db/bigquery-metrics?orgId=1

These have been failing for 240 days straight, looks like they always time out

@kubernetes/sig-gcp-test-failures
ref #2528

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.

@krzyzacy
Copy link
Member

@abgworrall it runs ALL test suites in gke test endpoint, which is essentially equivalent with gke + gke-serial + gke-slow

and gke-test | gci-gke-test also duplicates each other, I guess one was intended for cvm test but now is defaulted to cos anyway

@fejta fejta deleted the old branch January 29, 2018 23:17
@abgworrall
Copy link
Contributor

Ah, cool - thanks for that, @krzyzacy :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. area/provider/gcp Issues or PRs related to gcp provider cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants