Skip to content

Rename github.com/kubernetes/test-infra/ciongke -> k8s.io/test-infra/prow. #835

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

Merged
merged 2 commits into from
Oct 14, 2016

Conversation

spxtr
Copy link
Contributor

@spxtr spxtr commented Oct 14, 2016

I don't think it's go-gettable yet, but still worth making the change to keep it consistent with our other go projects.

It most likely won't pass on Jenkins because the docker run command is baked into the job config and not pulled into a script like it should be. I'd like to merge anyways, it will pass once the change goes in.


This change is Reviewable

@spxtr spxtr added the area/prow Issues or PRs related to prow label Oct 14, 2016
@k8s-ci-robot
Copy link
Contributor

Jenkins go test failed for commit ec6e5f7. Full PR test history.

The magic incantation to run this job again is @k8s-bot go test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@ixdy
Copy link
Member

ixdy commented Oct 14, 2016

Maybe update https://github.com/kubernetes/k8s.io/blob/master/k8s.io/configmap-www-golang.yaml to add k8s.io/test-infra first?

@spxtr
Copy link
Contributor Author

spxtr commented Oct 14, 2016

Nobody is go-getting this right now afaik, so the order doesn't matter much, but I'll make the PR now.

@spxtr spxtr merged commit f5bc706 into kubernetes:master Oct 14, 2016
@spxtr spxtr deleted the rename branch October 14, 2016 21:12
foxish pushed a commit to foxish/test-infra that referenced this pull request Jan 21, 2017
…icks

State why we are enforcing the release note process on a cherrypick
ostromart pushed a commit to ostromart/test-infra that referenced this pull request Jul 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/prow Issues or PRs related to prow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants