fix: log context for future debugging

goller · goller · commit 395e72c52b12 · 2023-08-31T08:37:58.000-05:00
Signed-off-by: Chris Goller &lt;goller@gmail.com&gt;
diff --git a/cache/refs.go b/cache/refs.go
@@ -1103,6 +1103,13 @@ func (sr *immutableRef) prepareRemoteSnapshotsStargzMode(ctx context.Context, s
 								for k := range tmpLabels {
 									info.Labels[k] = ""
 								}
+							} else {
+								// We are logging here to track to try to debug when and why labels are nil.
+								// Log can be removed when not happening anymore.
+								bklog.G(ctx).
+									WithField("snapshotID", snapshotID).
+									WithField("name", info.Name).
+									Debug("snapshots exist but labels are nil")
 							}
 							if _, err := r.cm.Snapshotter.Update(ctx, info, tmpFields...); err != nil {
 								bklog.G(ctx).Warn(errors.Wrapf(err,

Original file line number	Diff line number	Diff line change
`@@ -1103,6 +1103,13 @@ func (sr *immutableRef) prepareRemoteSnapshotsStargzMode(ctx context.Context, s`
`1103`	`1103`	`for k := range tmpLabels {`
`1104`	`1104`	`info.Labels[k] = ""`
`1105`	`1105`	`}`
	`1106`	`+ } else {`
	`1107`	`+ // We are logging here to track to try to debug when and why labels are nil.`
	`1108`	`+ // Log can be removed when not happening anymore.`
	`1109`	`+ bklog.G(ctx).`
	`1110`	`+ WithField("snapshotID", snapshotID).`
	`1111`	`+ WithField("name", info.Name).`
	`1112`	`+ Debug("snapshots exist but labels are nil")`
`1106`	`1113`	`}`
`1107`	`1114`	`if _, err := r.cm.Snapshotter.Update(ctx, info, tmpFields...); err != nil {`
`1108`	`1115`	`bklog.G(ctx).Warn(errors.Wrapf(err,`