-
Notifications
You must be signed in to change notification settings - Fork 192
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 Tekton Chains to the operator #630
Add Tekton Chains to the operator #630
Conversation
6ef0723
to
40b2f80
Compare
The following is the coverage report on the affected files.
|
40b2f80
to
1886fa3
Compare
The following is the coverage report on the affected files.
|
1886fa3
to
cdbe445
Compare
The following is the coverage report on the affected files.
|
/test pull-tekton-operator-integration-tests |
The following is the coverage report on the affected files.
|
/retest |
1ee7170
to
a54860a
Compare
The following is the coverage report on the affected files.
|
a54860a
to
0c9ede5
Compare
The following is the coverage report on the affected files.
|
0c9ede5
to
377c2eb
Compare
The following is the coverage report on the affected files.
|
377c2eb
to
56dafbe
Compare
The following is the coverage report on the affected files.
|
The following is the coverage report on the affected files.
|
821b91b
to
d8d563b
Compare
The following is the coverage report on the affected files.
|
d8d563b
to
4ce2041
Compare
/retest |
/test pull-tekton-operator-integration-tests |
/retest |
/test pull-tekton-operator-integration-tests |
1 similar comment
/test pull-tekton-operator-integration-tests |
/retest |
/test pull-tekton-operator-integration-tests |
2 similar comments
/test pull-tekton-operator-integration-tests |
/test pull-tekton-operator-integration-tests |
/retest |
2023eca
to
d1a9054
Compare
This will update the tektonchains reconciler logic for having better upgrades and making sure the existence of only one installerset by having type in labels
This will fix the order of running e2e tests as config test should be run before component tests in common pkg we need to run the tekton config first because other tests remove the tektonconfig object
This will add a fix that if targetNamespace of pipeline and chains are same so remove the namespace object from chains manifest as the namespace will be already created and we dont need to delete that if chains is deleted
d1a9054
to
c6fd995
Compare
The following is the coverage report on the affected files.
|
CI 🎉 |
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Add chains to the operator
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