-
Notifications
You must be signed in to change notification settings - Fork 205
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Refactor TektonComponent with new refactoring changes #424
Comments
Issues go stale after 90d of inactivity. /lifecycle stale Send feedback to tektoncd/plumbing. |
/remove-lifecycle stale |
@sm43 if you think all the reconcilers are ready to accept |
Issues go stale after 90d of inactivity. /lifecycle stale Send feedback to tektoncd/plumbing. |
Stale issues rot after 30d of inactivity. /lifecycle rotten Send feedback to tektoncd/plumbing. |
Rotten issues close after 30d of inactivity. /close Send feedback to tektoncd/plumbing. |
@tekton-robot: Closing this issue. 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. |
With recent refactoring of Operator, we have disabled
TektonComponent
as we have changed the statusfor components and condtion used in status for them.
We can refactor the
TektonComponent
with the common things which are there in all components now and so thatthere will be uniformity between all components method.
Refactoring PRs #385, #408, #421
Update the status methods which are common in status
operator/pkg/apis/operator/v1alpha1/common.go
Line 49 in 0390095
Once dashboard and results start using TektonInstallerSet we can update them otherwise they will break as they still use TektonComponent.
The text was updated successfully, but these errors were encountered: