Closed GijsvanDulmen closed 3 months ago
Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all
/test all
The following is the coverage report on the affected files.
Say /test pull-tekton-triggers-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/apis/triggers/v1beta1/trigger_template_validation.go | 97.1% | 96.6% | -0.5 |
/test tekton-triggers-unit-tests
/retest
The following is the coverage report on the affected files.
Say /test pull-tekton-triggers-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/adapter/adapter.go | 23.5% | 23.2% | -0.3 |
pkg/apis/triggers/v1beta1/trigger_template_validation.go | 97.1% | 96.6% | -0.5 |
looks like the builds tests are failing due to goimports
:
pkg/apis/triggers/v1alpha1/trigger_template_validation.go:21: File is not `goimports`-ed (goimports)
"fmt"
pkg/apis/triggers/v1beta1/trigger_template_validation.go:21: File is not `goimports`-ed (goimports)
"fmt"
/test pull-tekton-triggers-integration-tests
/retest
The following is the coverage report on the affected files.
Say /test pull-tekton-triggers-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/adapter/adapter.go | 23.5% | 23.2% | -0.3 |
pkg/apis/triggers/v1beta1/trigger_template_validation.go | 97.1% | 96.6% | -0.5 |
The following is the coverage report on the affected files.
Say /test pull-tekton-triggers-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/adapter/adapter.go | 23.5% | 23.2% | -0.3 |
pkg/apis/triggers/v1beta1/trigger_template_validation.go | 97.1% | 96.6% | -0.5 |
/retest-required
The following is the coverage report on the affected files.
Say /test pull-tekton-triggers-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/adapter/adapter.go | 23.5% | 23.2% | -0.3 |
pkg/apis/triggers/v1beta1/trigger_template_validation.go | 97.1% | 96.6% | -0.5 |
The following is the coverage report on the affected files.
Say /test pull-tekton-triggers-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/adapter/adapter.go | 23.5% | 23.2% | -0.3 |
pkg/apis/triggers/v1beta1/trigger_template_validation.go | 97.1% | 96.6% | -0.5 |
The following is the coverage report on the affected files.
Say /test pull-tekton-triggers-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/adapter/adapter.go | 23.5% | 23.2% | -0.3 |
pkg/apis/triggers/v1beta1/trigger_template_validation.go | 97.1% | 96.6% | -0.5 |
The following is the coverage report on the affected files.
Say /test pull-tekton-triggers-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/adapter/adapter.go | 23.5% | 23.2% | -0.3 |
pkg/apis/triggers/v1beta1/trigger_template_validation.go | 97.1% | 96.6% | -0.5 |
/kind feature
Removes the limit of only creating Tekton resources from within your TriggerTemplate. This will allow for other K8s resources to be created as well like for example K8s Jobs.
The following is the coverage report on the affected files.
Say /test pull-tekton-triggers-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/adapter/adapter.go | 23.5% | 23.2% | -0.3 |
pkg/apis/triggers/v1beta1/trigger_template_validation.go | 97.1% | 96.6% | -0.5 |
@dibyom Thanks for the review! Will process those asap :-)
The following is the coverage report on the affected files.
Say /test pull-tekton-triggers-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
cmd/triggerrun/cmd/root.go | 45.2% | 45.5% | 0.4 |
pkg/adapter/adapter.go | 23.5% | 23.2% | -0.3 |
pkg/apis/triggers/v1beta1/trigger_template_validation.go | 97.1% | 96.6% | -0.5 |
@dibyom It's ready for another review.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dibyom
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Also cc @khrm @savitaashture if you'd like to take a look
The following is the coverage report on the affected files.
Say /test pull-tekton-triggers-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
cmd/triggerrun/cmd/root.go | 45.2% | 45.5% | 0.4 |
pkg/adapter/adapter.go | 23.5% | 23.2% | -0.3 |
pkg/apis/triggers/v1beta1/trigger_template_validation.go | 97.1% | 96.6% | -0.5 |
@savitaashture @khrm any objections to merging this?
Would take a look tomorrow.
The following is the coverage report on the affected files.
Say /test pull-tekton-triggers-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
cmd/triggerrun/cmd/root.go | 45.2% | 45.5% | 0.4 |
pkg/adapter/adapter.go | 23.5% | 23.2% | -0.3 |
pkg/apis/triggers/v1beta1/trigger_template_validation.go | 97.1% | 96.6% | -0.5 |
Everything looks fine. Let's merge this.
Changes
Makes it possible to create other resources in the trigger template besides Tekton resources. Solves https://github.com/tektoncd/triggers/issues/1696
Submitter Checklist
As the author of this PR, please check off the items in this checklist:
/kind <type>
. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tepRelease Notes