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

Requeue instead of logging reconciler errors when owners not set on CloudStackMachine yet #299

Open
hrak opened this issue Jul 26, 2023 · 5 comments
Labels
kind/bug Categorizes issue or PR as related to a bug. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Milestone

Comments

@hrak
Copy link
Contributor

hrak commented Jul 26, 2023

/kind bug

What steps did you take and what happened:

During reconciliation of a cluster, in several places GetParent gets called (which calls GetOwnerOfKind). When the CloudStackMachines do not have an owner yet, this results in a whole bunch of reconciler errors being logged like so:

E0726 12:18:22.767591 1 controller.go:329] "msg"="Reconciler error" "error"="couldn't find owner of kind Machine in namespace default" "CloudStackMachine"={"name":"hrak-cluster-control-plane-nrr5m","namespace":"default"} "controller"="cloudstackmachine" "controllerGroup"="infrastructure.cluster.x-k8s.io" "controllerKind"="CloudStackMachine" "name"="hrak-cluster-control-plane-nrr5m" "namespace"="default" "reconcileID"="04d91e20-21a8-4b8b-b04e-c57b4c01067f"

Maybe this can be adapted so that it re queues this step in the same way that RequeueIfCloudStackClusterNotReady does, but then by checking the ownerrefs, to reduce the amount of 'bad news' in the logs.

What did you expect to happen:

No errors being logged

Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]

Environment:

  • Cluster-api-provider-cloudstack version: HEAD
  • Kubernetes version: (use kubectl version): 1.27.3
  • OS (e.g. from /etc/os-release):
@k8s-ci-robot k8s-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Jul 26, 2023
@rohityadavcloud
Copy link
Member

@hrak is that still an issue?

@rohityadavcloud rohityadavcloud added this to the v0.4.9 milestone Sep 20, 2023
@hrak
Copy link
Contributor Author

hrak commented Sep 26, 2023

yes!

@g-gaston
Copy link
Contributor

g-gaston commented Nov 1, 2023

This is a completely valid issue. However, since it's not a show stopper bug (it seems to me is more of a nice day to day improvement), moving it to the v0.5 milestone

/triage accept
/milestone v0.5.0

@k8s-ci-robot
Copy link
Contributor

@g-gaston: The label(s) triage/accept cannot be applied, because the repository doesn't have them.

In response to this:

This is a completely valid issue. However, since it's not a show stopper bug (it seems to me is more of a nice day to day improvement), moving it to the v0.5 milestone

/triage accept
/milestone v0.5.0

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.

@g-gaston
Copy link
Contributor

g-gaston commented Nov 1, 2023

/triage accepted

@k8s-ci-robot k8s-ci-robot added the triage/accepted Indicates an issue or PR is ready to be actively worked on. label Nov 1, 2023
@g-gaston g-gaston modified the milestones: v0.4.9, v0.5.0 Nov 1, 2023
@vishesh92 vishesh92 modified the milestones: v0.5.0, v0.6 May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
None yet
Development

No branches or pull requests

5 participants