Add a pipeline to deploy and test a release of a Tekton project #73
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add a Task that gets a release of a Tekton project from the release bucket, and installs it in a target cluster.
tekton-robot
added
approved
|
/hold |
tekton-robot
added
the
do-not-merge/hold
|
Added "hold" to allow for a couple of reviews. This is not used anywhere yet but I'm building a pipeline to test releases bit by bit. |
Add a task to verify the release YAML from the bucket against that stored in the GitHub release assets. Add a pipeline that runs the verify and the deploy tasks, and that eventually will also run the E2E tests.
|
Related to #75 |
Add a step to the deploy task that verifies that all pods in the target namespace are running normally. This uses functions from plumbing/scripts.sh.
Similar to what is done today in CI logs, add a task to log the versions of tools in the test-runner. This is helpful for debugging failures.
tekton-robot
added
size/XL
afrittoli
force-pushed
the
dogfooding_folder
branch
2 times, most recently
from
b0087b1 to
bbc341b
Compare
The task uses functions from the library.sh to run the e2e tests. Update the pipeline and docs to run the tests.
afrittoli
force-pushed
the
dogfooding_folder
branch
from
bbc341b to
571c38e
Compare
afrittoli
changed the title
tekton-robot
removed
the
approved
afrittoli
force-pushed
the
dogfooding_folder
branch
3 times, most recently
from
ebc528f to
b9ff276
Compare
tekton-robot
added
size/XXL
afrittoli
force-pushed
the
dogfooding_folder
branch
2 times, most recently
from
5f261af to
0635667
Compare
vdemeester
reviewed
Oct 8, 2019
| @@ -0,0 +1,48 @@ | |||
| # How to follow log outputs | |||
There was a problem hiding this comment.
Shouldn't we just point to tkn instead ?
There was a problem hiding this comment.
this is copied straight from pipeline - we might do something different eventually - but I wanted to have a smooth migration, by using the existing scripts mostly wrapped into tasks
There was a problem hiding this comment.
ok 👼
(I was about to remove that piece of code from pipeline 😅)
There was a problem hiding this comment.
I've seen your PR about this, once it's done on pipelines I can port it to plumbing if that's ok.
afrittoli
force-pushed
the
dogfooding_folder
branch
from
0635667 to
ed1a1f3
Compare
afrittoli
force-pushed
the
dogfooding_folder
branch
from
ed1a1f3 to
797bfd7
Compare
|
/hold cancel |
tekton-robot
removed
the
do-not-merge/hold
|
I think this is in a reasonable state to go into the repo now, and then iterate from here. |
YAML tests are executed in pipeline, but they may also be useful for catalog and triggers for instance. Copying some common functions from pipeline to library.sh in plumbing, adding a Task to run the yaml tests and adding the YAML tests task to the release test pipeline.
afrittoli
force-pushed
the
dogfooding_folder
branch
from
797bfd7 to
7711941
Compare
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: 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 |
tekton-robot
added
the
approved
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add a Task that gets a release of a Tekton project from the release
bucket, and installs it in a target cluster.
Changes
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:
See the contribution guide
for more details.