fix: fix logging

Author: comtalyst

pkg/auth/cred.go

@@ -89,10 +89,10 @@ func (w *expireEarlyTokenCredential) GetToken(ctx context.Context, options polic
 	// IMDS may have the MI token already, and have an expiration of less than 2h when we receive the token. We don't want to set that value beyond the ExpiresOn time and potentially miss a refresh
 	// So we just return earlier here. See discussion here: https://github.com/Azure/karpenter-provider-azure/pull/391/files#r1648633051
 	if token.ExpiresOn.Before(twoHoursFromNow) {
-		log.FromContext(ctx).Info("XPMT: ExpiresOn %s is before two hours from now %s, not adjusting", token.ExpiresOn, twoHoursFromNow)
+		log.FromContext(ctx).Info(fmt.Sprintf("XPMT: ExpiresOn %s is before two hours from now %s, not adjusting", token.ExpiresOn, twoHoursFromNow))
 		return token, nil
 	}
-	log.FromContext(ctx).Info("XPMT: adjusting ExpiresOn from %s to %s", token.ExpiresOn, twoHoursFromNow)
+	log.FromContext(ctx).Info(fmt.Sprintf("XPMT: adjusting ExpiresOn from %s to %s", token.ExpiresOn, twoHoursFromNow))
 	// If the token expires in more than 2 hours, this means we are taking in a new token with a fresh 24h expiration time or one already in the cache that hasn't been modified by us, so we want to set that to two hours so
 	// we can refresh it early to avoid the polling bugs mentioned in the above issue
 	token.ExpiresOn = twoHoursFromNow

pkg/providers/imagefamily/nodebootstrapping_client.go

@@ -61,7 +61,7 @@ func (t *tokenCache) getToken(ctx context.Context, credential azcore.TokenCreden
 		return t.token, nil
 	}
 	if t.token.Token != "" {
-		log.FromContext(ctx).Info("XPMT: getToken(): cached token is expired: refreshAfter=%s; ExpiresOn=%s, RefreshOn=%s", t.refreshAfter, t.token.ExpiresOn, t.token.RefreshOn)
+		log.FromContext(ctx).Info(fmt.Sprintf("XPMT: getToken(): cached token is expired: refreshAfter=%s; ExpiresOn=%s, RefreshOn=%s", t.refreshAfter, t.token.ExpiresOn, t.token.RefreshOn))
 	} else {
 		log.FromContext(ctx).Info("XPMT: getToken(): no cached token present, fetching a new one")
 	}
@@ -74,7 +74,7 @@ func (t *tokenCache) getToken(ctx context.Context, credential azcore.TokenCreden
 		return azcore.AccessToken{}, fmt.Errorf("failed to get token: %w", err)
 	}
 
-	log.FromContext(ctx).Info("XPMT: getToken(): fetched new token: ExpiresOn=%s, RefreshOn=%s", tokenObj.ExpiresOn, tokenObj.RefreshOn)
+	log.FromContext(ctx).Info(fmt.Sprintf("XPMT: getToken(): fetched new token: ExpiresOn=%s, RefreshOn=%s", tokenObj.ExpiresOn, tokenObj.RefreshOn))
 
 	// Store the token with its expiration
 	t.token = tokenObj
@@ -83,7 +83,7 @@ func (t *tokenCache) getToken(ctx context.Context, credential azcore.TokenCreden
 	if tokenObj.RefreshOn.IsZero() {
 		defaultRefreshBuffer := 5 * time.Minute
 		t.refreshAfter = tokenObj.ExpiresOn.Add(-defaultRefreshBuffer)
-		log.FromContext(ctx).Info("XPMT: getToken(): no RefreshOn provided, using default buffer: refreshAfter=%s", t.refreshAfter)
+		log.FromContext(ctx).Info(fmt.Sprintf("XPMT: getToken(): no RefreshOn provided, using default buffer: refreshAfter=%s", t.refreshAfter))
 	} else {
 		// Use the RefreshOn time if provided, otherwise define our own buffer
 		t.refreshAfter = tokenObj.RefreshOn
@@ -155,7 +155,7 @@ func (c *NodeBootstrappingClient) Get(
 		return "", "", err
 	}
 	xpmtEndTime := time.Now()
-	log.FromContext(ctx).Info("XPMT: NodeBootstrappingClient.Get(): request completed, duration=%s", xpmtEndTime.Sub(xpmtStartTime))
+	log.FromContext(ctx).Info(fmt.Sprintf("XPMT: NodeBootstrappingClient.Get(): request completed, duration=%s", xpmtEndTime.Sub(xpmtStartTime)))
 
 	if resp.Payload == nil {
 		return "", "", fmt.Errorf("no payload in response")