Minor url cache logging & counting improvements (#4801)

<!-- PR description-->

1. Move cache miss logs to debug, as it can be chatty sometimes. We will leverage the end of operation `URLCacheMiss` counter to get this information instead.
2. Counter renaming. 
---

#### Does this PR need a docs update or release note?

- [ ] ✅ Yes, it's included
- [ ] 🕐 Yes, but in a later PR
- [x] ⛔ No

#### Type of change

<!--- Please check the type of change your PR introduces: --->
- [ ] 🌻 Feature
- [ ] 🐛 Bugfix
- [ ] 🗺️ Documentation
- [ ] 🤖 Supportability/Tests
- [ ] 💻 CI/Deployment
- [x] 🧹 Tech Debt/Cleanup

#### Issue(s)

<!-- Can reference multiple issues. Use one of the following "magic words" - "closes, fixes" to auto-close the Github issue. -->
* #<issue>

#### Test Plan

<!-- How will this be tested prior to merging.-->
- [x] 💪 Manual
- [ ] ⚡ Unit test
- [ ] 💚 E2E

src/internal/m365/collection/drive/collection.go

@@ -379,11 +379,10 @@ func downloadContent(
 		return content, nil
 	}
 
-	// Consider cache errors(including deleted items) as cache misses. This is
+	// Consider cache errors(including deleted items) as cache misses.
-	// to preserve existing behavior. Fallback to refetching the item using the
+	// Fallback to refetching the item using the graph API.
-	// API.
+	logger.CtxErr(ctx, err).Debug("url cache miss: refetching from API")
-	logger.CtxErr(ctx, err).Info("url cache miss: refetching from API")
+	counter.Inc(count.URLCacheMiss)
-	counter.Inc(count.ItemDownloadURLRefetch)
 
 	di, err := iaag.GetItem(ctx, driveID, ptr.Val(item.GetId()))
 	if err != nil {
@@ -423,7 +422,7 @@ func readItemContents(
 
 	rc, err := downloadFile(ctx, iaag, props.downloadURL)
 	if graph.IsErrUnauthorizedOrBadToken(err) {
-		logger.CtxErr(ctx, err).Info("stale item in cache")
+		logger.CtxErr(ctx, err).Debug("stale item in cache")
 	}
 
 	if err != nil {

src/internal/m365/collection/drive/url_cache.go

@@ -35,7 +35,7 @@ type itemProps struct {
 
 var _ getItemPropertyer = &urlCache{}
 
-// urlCache caches download URLs for drive items
+// urlCache caches download URLs for drive file items
 type urlCache struct {
 	driveID         string
 	prevDelta       string
@@ -207,7 +207,7 @@ func (uc *urlCache) readCache(
 
 	props, ok := uc.idToProps[itemID]
 	if !ok {
-		uc.counter.Inc(count.URLCacheMiss)
+		uc.counter.Inc(count.URLCacheItemNotFound)
 		return itemProps{}, clues.NewWC(ctx, "item not found in cache")
 	}
 

src/pkg/count/keys.go

@@ -38,7 +38,6 @@ const (
 	DriveTombstones               Key = "drive-tombstones"
 	Files                         Key = "files"
 	Folders                       Key = "folders"
-	ItemDownloadURLRefetch        Key = "item-download-url-refetch"
 	ItemsAdded                    Key = "items-added"
 	ItemsRemoved                  Key = "items-removed"
 	LazyDeletedInFlight           Key = "lazy-deleted-in-flight"
@@ -68,6 +67,7 @@ const (
 	TotalContainersSkipped        Key = "total-containers-skipped"
 	URLCacheMiss                  Key = "url-cache-miss"
 	URLCacheRefresh               Key = "url-cache-refresh"
+	URLCacheItemNotFound          Key = "url-cache-item-not-found"
 )
 
 // Total___Processed counts are used to track raw processing numbers