-
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
Add examples for some resolvers #5451
Add examples for some resolvers #5451
Conversation
/assign @lbernick @jerop @vdemeester |
ab5d495
to
d27a8aa
Compare
would you mind also adding an example of how to replace a clustertask with the cluster resolver here? |
oops! I forgot about that, will do. |
Note that there's no bundles resolver example at all for Pipeline, because I can't find a publicly available bundle containing a Pipeline. Also, the cluster resolvers for both Pipeline and Task are in `no-ci`, since they require an existing `Pipeline`/`Task` in another namespace in the cluster, and I didn't feel comfortable adding a new namespace to those examples. The hub resolver for `Pipeline` are in `no-ci` because they're using https://github.com/tektoncd/catalog/blob/main/pipeline/buildpacks/0.2/buildpacks.yaml, which requires various tasks already be present. For the moment, the git resolver example for `Pipeline` is also in `no-ci`, but the new file ./test/resolvers_yaml/pipeline-in-git.yaml is intended to be used in an updated git resolver example once this is merged and can be referenced in the example. Signed-off-by: Andrew Bayer <andrew.bayer@gmail.com>
d27a8aa
to
9eb5206
Compare
/retest |
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.
thanks for adding these examples @abayer!
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jerop, vdemeester 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 |
Changes
Note that there's no bundles resolver example at all for Pipeline, because I can't find a publicly available bundle containing a Pipeline. Also, the cluster resolvers for both Pipeline and Task are in
no-ci
, since they require an existingPipeline
/Task
in another namespace in the cluster, and I didn't feel comfortable adding a new namespace to those examples. The hub resolver example forPipeline
is inno-ci
because it's using https://github.com/tektoncd/catalog/blob/main/pipeline/buildpacks/0.2/buildpacks.yaml, which requires various tasks already be present.For the moment, the git resolver example for
Pipeline
is also inno-ci
, but the new file ./test/resolvers_yaml/pipeline-in-git.yaml is intended to be used in an updated git resolver example once this is merged and can be referenced in the example.Also fixes a mistake I made in
docs/hub-resolver.md
./kind misc
Submitter Checklist
As the author of this PR, please check off the items in this checklist:
functionality, content, code)
/kind <type>
. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tepRelease Notes