-
Notifications
You must be signed in to change notification settings - Fork 183
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 e2e tests for triggers and dashboard #174
Conversation
72daebb
to
f7e3b1e
Compare
The following is the coverage report on the affected files.
|
The following is the coverage report on the affected files.
|
f7e3b1e
to
a0e1b95
Compare
The following is the coverage report on the affected files.
|
a0e1b95
to
76e8064
Compare
The following is the coverage report on the affected files.
|
76e8064
to
ecffba5
Compare
The following is the coverage report on the affected files.
|
The following is the coverage report on the affected files.
|
0f37be7
to
7551b80
Compare
The following is the coverage report on the affected files.
|
7551b80
to
3ace148
Compare
The following is the coverage report on the affected files.
|
[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 |
Fix `CheckDeployment` stage in `pkg/reconciler/common`: all reconcilers were flaky as events were no requeued on deployment unavailable. Add create, restore-deployment, and delete e2e tests for triggers Refactor pipelines e2e tests Signed-off-by: Nikhil Thomas <nikthoma@redhat.com>
Add create, restore-deployments, delete e2e tests Fix flakiness in TektonPipline e2e tests by explicity adding a delete TektonPipelineCR mechanism in triggers, and dashboard e2e tests. Signed-off-by: Nikhil Thomas <nikthoma@redhat.com>
/hold |
Remove obsolete make target (based on operator-sdk) Modify `test/e2e-common.sh` to make sure that we get `No resource found` (or resource deletion), instead of `CRD not defined error` Signed-off-by: Nikhil Thomas <nikthoma@redhat.com>
3ace148
to
5ec44f9
Compare
/hold cancel |
The following is the coverage report on the affected files.
|
/lgtm |
Fix
CheckDeployment
stage inpkg/reconciler/common
: all reconcilerswere flaky as events were no requeued on deployment unavailable.
Add create, restore-deployments and delete e2e tests for TektonTriggers and TektonDashboard.
comment
aggregationRule
resource: ClusterRole, name: tekton-dashboard-extensions` to dashboard e2e tests and reconciler run without providing cluster admin privileges to operator sa. (#131)Comment
Refactor TektonPipelines e2e tests
Signed-off-by: Nikhil Thomas nikthoma@redhat.com
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.
Release Notes