Skip to content
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

Controller is watching all CRDs not just Sources #1

Open
zroubalik opened this issue Jul 16, 2020 · 7 comments
Open

Controller is watching all CRDs not just Sources #1

zroubalik opened this issue Jul 16, 2020 · 7 comments
Labels
bug Something isn't working triage/accepted Issues which should be fixed (post-triage)

Comments

@zroubalik
Copy link
Contributor

The controller is watching all CRDs, not just those with duck.knative.dev/source label. Though they are ignored by controller, it would be nice to fix this, to avoid the flood of messages in the log.

I expected this part of code would do the trick, but it doesn't:
https://github.com/zroubalik/autoscaler-keda/blob/85186c0c257126f6aa71fac9b26f9b252210e523/pkg/reconciler/crd/controller.go#L64-L66

Log:

I0716 16:09:16.991938       1 leaderelection.go:242] attempting to acquire leader lease  autoscaler-keda/controller.github.com-zroubalik-autoscaler-keda-pkg-reconciler-crd.reconciler.00-of-01...
{"level":"info","ts":"2020-07-16T16:09:17.001Z","logger":"controller","caller":"leaderelection/context.go:146","msg":"\"controller-76fb8d6756-5f4vm_efc63512-fff9-478e-b322-f49b34e0e0d3\" has started leading \"controller.github.com-zroubalik-autoscaler-keda-pkg-reconciler-crd.reconciler.00-of-01\""}
I0716 16:09:17.001699       1 leaderelection.go:252] successfully acquired lease autoscaler-keda/controller.github.com-zroubalik-autoscaler-keda-pkg-reconciler-crd.reconciler.00-of-01
{"level":"info","ts":"2020-07-16T16:09:17.003Z","logger":"controller.github.com-zroubalik-autoscaler-keda-pkg-reconciler-crd.Reconciler","caller":"crd/reconciler.go:98","msg":"GVKKKconfig.openshift.io/v1, Kind=Scheduler","knative.dev/traceid":"0cf0bbb3-b9c5-4149-84e3-c4e14d58bdc2","knative.dev/key":"schedulers.config.openshift.io"}
{"level":"info","ts":"2020-07-16T16:09:17.003Z","logger":"controller.github.com-zroubalik-autoscaler-keda-pkg-reconciler-crd.Reconciler","caller":"customresourcedefinition/reconciler.go:291","msg":"Returned an event","knative.dev/traceid":"0cf0bbb3-b9c5-4149-84e3-c4e14d58bdc2","knative.dev/key":"schedulers.config.openshift.io","targetMethod":"ReconcileKind","event":"CustomResourceDefinition reconciliation ignored, KEDA autoscaling for this CRD is not supported: \"/schedulers.config.openshift.io\""}
{"level":"info","ts":"2020-07-16T16:09:17.003Z","logger":"controller.github.com-zroubalik-autoscaler-keda-pkg-reconciler-crd.Reconciler","caller":"controller/controller.go:474","msg":"Reconcile succeeded. Time taken: 219.24µs","knative.dev/traceid":"0cf0bbb3-b9c5-4149-84e3-c4e14d58bdc2","knative.dev/key":"schedulers.config.openshift.io"}
{"level":"info","ts":"2020-07-16T16:09:17.003Z","logger":"controller.github.com-zroubalik-autoscaler-keda-pkg-reconciler-crd.Reconciler","caller":"crd/reconciler.go:98","msg":"GVKKKconfig.openshift.io/v1, Kind=FeatureGate","knative.dev/traceid":"acfe0c7d-8bdc-4b00-bfeb-d8052fc76a7a","knative.dev/key":"featuregates.config.openshift.io"}
{"level":"info","ts":"2020-07-16T16:09:17.003Z","logger":"controller.github.com-zroubalik-autoscaler-keda-pkg-reconciler-crd.Reconciler","caller":"customresourcedefinition/reconciler.go:291","msg":"Returned an event","knative.dev/traceid":"acfe0c7d-8bdc-4b00-bfeb-d8052fc76a7a","knative.dev/key":"featuregates.config.openshift.io","targetMethod":"ReconcileKind","event":"CustomResourceDefinition reconciliation ignored, KEDA autoscaling for this CRD is not supported: \"/featuregates.config.openshift.io\""}
{"level":"info","ts":"2020-07-16T16:09:17.003Z","logger":"controller.github.com-zroubalik-autoscaler-keda-pkg-reconciler-crd.Reconciler","caller":"controller/controller.go:474","msg":"Reconcile succeeded. Time taken: 155.731µs","knative.dev/traceid":"acfe0c7d-8bdc-4b00-bfeb-d8052fc76a7a","knative.dev/key":"featuregates.config.openshift.io"}
{"level":"info","ts":"2020-07-16T16:09:17.003Z","logger":"controller.github.com-zroubalik-autoscaler-keda-pkg-reconciler-crd.Reconciler","caller":"crd/reconciler.go:98","msg":"GVKKKconfig.openshift.io/v1, Kind=Proxy","knative.dev/traceid":"b3477a47-c9b7-4e61-9823-9176337331f4","knative.dev/key":"proxies.config.openshift.io"}
{"level":"info","ts":"2020-07-16T16:09:17.004Z","logger":"controller.github.com-zroubalik-autoscaler-keda-pkg-reconciler-crd.Reconciler","caller":"customresourcedefinition/reconciler.go:291","msg":"Returned an event","knative.dev/traceid":"b3477a47-c9b7-4e61-9823-9176337331f4","knative.dev/key":"proxies.config.openshift.io","targetMethod":"ReconcileKind","event":"CustomResourceDefinition reconciliation ignored, KEDA autoscaling for this CRD is not supported: \"/proxies.config.openshift.io\""}
@zroubalik zroubalik added the bug Something isn't working label Jul 23, 2020
@github-actions
Copy link

github-actions bot commented Nov 4, 2020

This issue is stale because it has been open for 90 days with no
activity. It will automatically close after 30 more days of
inactivity. Reopen the issue with /reopen. Mark the issue as
fresh by adding the comment /remove-lifecycle stale.

@github-actions github-actions bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Nov 4, 2020
@zroubalik
Copy link
Contributor Author

/remove-lifecycle stale

@knative-prow-robot knative-prow-robot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Nov 4, 2020
@github-actions
Copy link

github-actions bot commented Feb 3, 2021

This issue is stale because it has been open for 90 days with no
activity. It will automatically close after 30 more days of
inactivity. Reopen the issue with /reopen. Mark the issue as
fresh by adding the comment /remove-lifecycle stale.

@github-actions github-actions bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Feb 3, 2021
@zroubalik
Copy link
Contributor Author

/remove-lifecycle stale

@knative-prow-robot knative-prow-robot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Feb 3, 2021
@github-actions
Copy link

github-actions bot commented May 5, 2021

This issue is stale because it has been open for 90 days with no
activity. It will automatically close after 30 more days of
inactivity. Reopen the issue with /reopen. Mark the issue as
fresh by adding the comment /remove-lifecycle stale.

@github-actions github-actions bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label May 5, 2021
@github-actions github-actions bot closed this as completed Jun 4, 2021
@zroubalik zroubalik reopened this Jun 14, 2021
@zroubalik zroubalik reopened this Jul 19, 2021
@zroubalik zroubalik removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Aug 20, 2021
@zroubalik zroubalik reopened this Aug 20, 2021
@github-actions
Copy link

This issue is stale because it has been open for 90 days with no
activity. It will automatically close after 30 more days of
inactivity. Reopen the issue with /reopen. Mark the issue as
fresh by adding the comment /remove-lifecycle stale.

@github-actions github-actions bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Nov 19, 2021
@zroubalik zroubalik reopened this Dec 20, 2021
@zroubalik zroubalik reopened this Jan 21, 2022
@zroubalik zroubalik reopened this Feb 28, 2022
@pierDipi
Copy link
Member

/remove-lifecycle stale
/triage accepted

@pierDipi pierDipi reopened this Mar 31, 2022
@knative-prow knative-prow bot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Mar 31, 2022
@knative-prow knative-prow bot added the triage/accepted Issues which should be fixed (post-triage) label Mar 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working triage/accepted Issues which should be fixed (post-triage)
Projects
None yet
Development

No branches or pull requests

3 participants