From 51efd38e7513a711ba159264777a32ca267b4ce7 Mon Sep 17 00:00:00 2001 From: Richard Larocque Date: Fri, 5 Dec 2014 11:07:45 -0800 Subject: [PATCH] Print SyncCloud errors This small change would have saved me a lot of time while investigating issue #2746. --- pkg/cloudprovider/controller/minioncontroller.go | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/pkg/cloudprovider/controller/minioncontroller.go b/pkg/cloudprovider/controller/minioncontroller.go index fb3c36b62b..a293d66c6a 100644 --- a/pkg/cloudprovider/controller/minioncontroller.go +++ b/pkg/cloudprovider/controller/minioncontroller.go @@ -54,7 +54,11 @@ func NewMinionController( // Run starts syncing instances from cloudprovider periodically, or create initial minion list. func (s *MinionController) Run(period time.Duration) { if s.cloud != nil && len(s.matchRE) > 0 { - go util.Forever(func() { s.SyncCloud() }, period) + go util.Forever(func() { + if err := s.SyncCloud(); err != nil { + glog.Errorf("Error syncing cloud: %v", err) + } + }, period) } else { go s.SyncStatic(period) }