feedback

davidgamero · davidgamero · commit 7c910a6151ce · 2024-11-19T09:57:19.000-05:00
diff --git a/pkg/providers/az-client.go b/pkg/providers/az-client.go
@@ -71,9 +71,10 @@ func (az *AzClient) ListResourceGroups(ctx context.Context, subscriptionID strin
 }
 
 func (az *AzClient) ListTenants(ctx context.Context) ([]armsubscription.TenantIDDescription, error) {
-	log.Debug("listing Azure subscriptions")
+	log.Debug("Starting to list Azure Tenants")
 
-	var tenants []armsubscription.TenantIDDescription
+	// Initialize the tenant slice to store the results.
+	tenants := make([]armsubscription.TenantIDDescription, 0)
 
 	if az.TenantClient == nil {
 		c, err := armsubscription.NewTenantsClient(az.Credential, nil)
@@ -98,7 +99,7 @@ func (az *AzClient) ListTenants(ctx context.Context) ([]armsubscription.TenantID
 		}
 	}
 
-	log.Debug("finished listing Azure tenants")
+	log.Debugf("Successfully listed %d Azure tenants", len(tenants))
 	return tenants, nil
 }
 

Original file line number	Diff line number	Diff line change
`@@ -71,9 +71,10 @@ func (az *AzClient) ListResourceGroups(ctx context.Context, subscriptionID strin`
`71`	`71`	`}`
`72`	`72`
`73`	`73`	`func (az *AzClient) ListTenants(ctx context.Context) ([]armsubscription.TenantIDDescription, error) {`
`74`		`- log.Debug("listing Azure subscriptions")`
	`74`	`+ log.Debug("Starting to list Azure Tenants")`
`75`	`75`
`76`		`- var tenants []armsubscription.TenantIDDescription`
	`76`	`+ // Initialize the tenant slice to store the results.`
	`77`	`+ tenants := make([]armsubscription.TenantIDDescription, 0)`
`77`	`78`
`78`	`79`	`if az.TenantClient == nil {`
`79`	`80`	`c, err := armsubscription.NewTenantsClient(az.Credential, nil)`
`@@ -98,7 +99,7 @@ func (az *AzClient) ListTenants(ctx context.Context) ([]armsubscription.TenantID`
`98`	`99`	`}`
`99`	`100`	`}`
`100`	`101`
`101`		`- log.Debug("finished listing Azure tenants")`
	`102`	`+ log.Debugf("Successfully listed %d Azure tenants", len(tenants))`
`102`	`103`	`return tenants, nil`
`103`	`104`	`}`
`104`	`105`