-
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
[Epic] Dogfood Pipeline #267
Comments
Fwiw, we’re planning to work on getting Prow to support running Pipelines some time in the not too distant future. Cc @rawlingsj |
Absolutely, we'd love to help in any way we can. Right now Jenkins X builds Jenkins X with Jenkins X, using prow and Knative Build. We're looking to switch to Pipeline as soon as we can so definitely would like to help / contribute to this. |
So cool @abayer + @rawlingsj sounds super exciting!!! 🎉 |
We were using `stable` and `stable` was suddenly updated to install `ko` from https://github.com/google/ko but unfortunately that version of `ko` didn't include google/go-containerregistry#380 which was the critical change from building binaries at `/ko-app` to building them at `/ko-app/my-pkg` <- which we built some of our functionality around. Now we'll try to pin the image so we aren't caught by surprise by changes again and can decide for ourselves when we want to consume them. (Hopefully old images don't get deleted too often.. 🙏 Or at least not before tektoncd#267 😛)
We were using `stable` and `stable` was suddenly updated to install `ko` from https://github.com/google/ko but unfortunately that version of `ko` didn't include google/go-containerregistry#380 which was the critical change from building binaries at `/ko-app` to building them at `/ko-app/my-pkg` <- which we built some of our functionality around. Now we'll try to pin the image so we aren't caught by surprise by changes again and can decide for ourselves when we want to consume them. (Hopefully old images don't get deleted too often.. 🙏 Or at least not before #267 😛)
@bobcatfish |
Those are good questions @vincent-pli !
Yes, we'd need to have tekton deployed in the CI cluster, at least initially it'd be the cluster we use to run Prow
I'm not 100% sure! I think at least initially we would pin to whatever the most recent release was, and maybe the process would be to upgrade our own cluster whenever we publish a new release (maybe even as a sanity check before announcing it). Or we could use the nightly build, or we could even use HEAD. So options:
|
I'm not sure option (3) will be possible right from the beginning, but as soon as we sorted out other issues I would start using HEAD! If we by chance we merge a breaking change, we'll break our CI as well and thus we'll stop merging changes until that's fixed. As long as the volume of PRs is similar to what we have today we should be able to afford that. Besides doing CD with a project from the CDF is pretty cool :) |
Do we have some kind of design document / diagram that explains what kind of setup we're aiming for? Do you guys @bobcatfish @vdemeester need any help in this area? I would be happy to help. |
There's no doc that I know of! @vdemeester has been doing most of the actual work but afaik so far we've been kind of just taking it one step at a time without a clear overall picture. If you want to start putting together an overall plan and weighing the options @afrittoli that would be super helpful 🙏❤️🙏 I also just sent you an invite to our zenhub project in case you havent seen it yet, that'll let you view the other issues linked to this "epic" (none of which deal with what you're proposing tho). |
Thank you! I've been ramping up on the different bits and pieces, I'll start working on some document soon. |
|
This epic is about using the Pipeline project to test the Pipeline project (so that we can be our own customers and feel their pain :D).
Expected behavior
The Pipeline project should be tested using the Pipeline project itself, e.g. all testing, building, releasing should be done via Pipelines + Tasks.
Current behavior
The Pipeline project is tested, built and released using Prow jobs which trigger scripts.
Additional info
Once we're ready to take this on we'll probably break this out into separate epics, for example:
This will probably become a milestone in the new year.
This is a zenhub epic.
The text was updated successfully, but these errors were encountered: