Skip to content

Commit

Permalink
Merge pull request #3016 from haiyanmeng/sync-source
Browse files Browse the repository at this point in the history
🌱 (:seedling:, other) Log the syncing source when WaitForSync fails
  • Loading branch information
k8s-ci-robot authored Nov 20, 2024
2 parents e56f2c9 + bc669b1 commit b88f351
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion pkg/internal/controller/controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -197,7 +197,7 @@ func (c *Controller[request]) Start(ctx context.Context) error {
}
didStartSyncingSource.Store(true)
if err := syncingSource.WaitForSync(sourceStartCtx); err != nil {
err := fmt.Errorf("failed to wait for %s caches to sync: %w", c.Name, err)
err := fmt.Errorf("failed to wait for %s caches to sync %v: %w", c.Name, syncingSource, err)
log.Error(err, "Could not wait for Cache to sync")
sourceStartErrChan <- err
}
Expand Down
2 changes: 1 addition & 1 deletion pkg/internal/controller/controller_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -171,7 +171,7 @@ var _ = Describe("controller", func() {

err = ctrl.Start(context.TODO())
Expect(err).To(HaveOccurred())
Expect(err.Error()).To(ContainSubstring("failed to wait for testcontroller caches to sync: timed out waiting for cache to be synced"))
Expect(err.Error()).To(ContainSubstring("failed to wait for testcontroller caches to sync kind source: *v1.Deployment: timed out waiting for cache to be synced"))
})

It("should not error when controller Start context is cancelled during Sources WaitForSync", func() {
Expand Down

0 comments on commit b88f351

Please sign in to comment.