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

MachineDeployment stuck waiting for next resync if it can't find the infrastructure template #2737

Closed
ncdc opened this issue Mar 20, 2020 · 1 comment · Fixed by #2740
Closed
Labels
kind/bug Categorizes issue or PR as related to a bug.
Milestone

Comments

@ncdc
Copy link
Contributor

ncdc commented Mar 20, 2020

What steps did you take and what happened:

  1. Create a MachineDeployment
  2. Create the infrastructure template for it
  3. MachineDeployment has a ReconcileError event:
machinedeployment-controller  failed to retrieve AWSMachineTemplate external object "foo"/"test1-md-0": AWSMachineTemplate.infrastructure.cluster.x-k8s.io "test1-md-0" not found
  1. It takes another 10-20 minutes (resync period) for the MachineDeployment to be processed and for the MachineSet to be created

What did you expect to happen:

  1. It should requeue periodically if it gets a not-found error

Anything else you would like to add:
We get the error, log/event it, and return nil, so it doesn't requeue:

result, err := r.reconcile(ctx, cluster, deployment)
if err != nil {
logger.Error(err, "Failed to reconcile MachineDeployment")
r.recorder.Eventf(deployment, corev1.EventTypeWarning, "ReconcileError", "%v", err)
}
return result, nil

Environment:

  • Cluster-api version: v0.3.2

/kind bug

@k8s-ci-robot k8s-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Mar 20, 2020
@vincepri
Copy link
Member

/milestone v0.3.3

@k8s-ci-robot k8s-ci-robot added this to the v0.3.3 milestone Mar 20, 2020
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.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants