-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Documentations for using tekton as agent in prowjob #1187
Comments
@yfei1 I would say it might be more a prow issue but I feel we could also document it here (or at least on |
Thanks for your prompt reply! Could you point out some resources that I could look into for using tekton with prow? |
In the context of #922 I got Prow + Pipelines up and running for my own repo (thanks to @JoelSpeed 's notes in kubernetes/test-infra#13874 (comment) and some help from @krzyzacy + @BenTheElder !!), so I'll take this on and document how to get it to work! |
A step toward tektoncd/pipeline#922 ! This commit adds configuration and supporting docs to get Prow to execute a simple Pipeline on every pull request to the plumbing repo. I am also working on docs in the Prow repo (for kubernetes/test-infra#13874 and tektoncd/pipeline#1187) to explain how one could put this all together from scratch. One important caveat is that we were previously using the Tekton installation in this cluster to do manual releases, and we were keeping our release configuration up to date with the latest version of Tekton Pipelines. Because of kubernetes/test-infra#13948 the newest version we can use with Prow is v0.3.1 so we need to keep our release configs in sync with this for now ( attn folks doing releases @vdemeester @a-roberts @mnuttall - the next thing I want to do is add release automation via Prow so hopefully this won't impact you for long!) Note I have already manually applied this configuration to the cluster 🙏
A step toward tektoncd/pipeline#922 ! This commit adds configuration and supporting docs to get Prow to execute a simple Pipeline on every pull request to the plumbing repo. I am also working on docs in the Prow repo (for kubernetes/test-infra#13874 and tektoncd/pipeline#1187) to explain how one could put this all together from scratch. One important caveat is that we were previously using the Tekton installation in this cluster to do manual releases, and we were keeping our release configuration up to date with the latest version of Tekton Pipelines. Because of kubernetes/test-infra#13948 the newest version we can use with Prow is v0.3.1 so we need to keep our release configs in sync with this for now ( attn folks doing releases @vdemeester @a-roberts @mnuttall - the next thing I want to do is add release automation via Prow so hopefully this won't impact you for long!) Note I have already manually applied this configuration to the cluster 🙏
I think at this point it would make sense for this issue to live in the Prow repo only so I'll close this one. Also anyone should feel free to pickup where I left off with kubernetes/test-infra#13974 🙏 |
Did any documentation on Prow + Tekton ever happen? I couldnt find anything |
This is an example prow config for a tekton job:
With an example tekton pipeline and task:
... using tekton v0.36.0, prow v20220705-20e880da9b, and prow pipeline. Need to use the legacy prowjob CRD (
|
@masseybradley if you attach the github ssh secret to the |
Expected Behavior
I'm not sure if this is a Prow or Tekton related issue. I was working on migrating a CI pipeline from Google Cloud Build to Prow. However, it seems like the native Kubernetes agent does not support running build/test sequentially in a pipelined way. I looked into api.ProwJob's source code and found there is a tekton controller defined in this file, yet it seems like we don't have documentations for using
tekton-pipeline
in Prow yet.Also, I believe using
tekton-pipeline
as agent in a prowjob may require having tekton being deployed somewhere. Could you possibly point out some resources/examples I could look into to setup tekton along with my prow deployment? Thank!Actual Behavior
The text was updated successfully, but these errors were encountered: