Skip to content

Commit

Permalink
working on getting tests to pass
Browse files Browse the repository at this point in the history
  • Loading branch information
frodopwns committed Oct 28, 2019
1 parent a9935b7 commit 12aeb5a
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
4 changes: 3 additions & 1 deletion controllers/async_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,9 @@ func (r *AsyncReconciler) Reconcile(req ctrl.Request, local runtime.Object) (ctr
log.Error(ensureErr, "ensure err")
}

final := multierror.Append(ensureErr, r.Update(ctx, local))
log.Info("local", "object", local)

final := multierror.Append(ensureErr, r.Status().Update(ctx, local))
err := final.ErrorOrNil()
if err != nil {
r.Recorder.Event(local, "Warning", "FailedReconcile", fmt.Sprintf("Failed to reconcile resource: %s", err.Error()))
Expand Down
4 changes: 2 additions & 2 deletions pkg/resourcemanager/resourcegroups/reconcile.go
Original file line number Diff line number Diff line change
Expand Up @@ -46,11 +46,11 @@ func (g *AzureResourceGroupManager) Ensure(ctx context.Context, obj runtime.Obje
// return true, nil
// }
// }

instance.Status.Provisioned = false
return false, fmt.Errorf("ResourceGroup create error %v", err)

}

instance.Status.Provisioned = true
return true, nil
}

Expand Down

0 comments on commit 12aeb5a

Please # to comment.