Merge pull request #1614 from mnaser/fix/add-user-agent

k8s-ci-robot · web-flow · commit 3c742d2cd136 · 2023-08-02T13:56:20.000-07:00
🌱 chore: use capo cluster agent for API requests
diff --git a/pkg/scope/provider.go b/pkg/scope/provider.go
@@ -37,6 +37,7 @@ import (
 
 	infrav1 "sigs.k8s.io/cluster-api-provider-openstack/api/v1alpha7"
 	"sigs.k8s.io/cluster-api-provider-openstack/pkg/clients"
+	"sigs.k8s.io/cluster-api-provider-openstack/version"
 )
 
 const (
@@ -152,6 +153,10 @@ func NewProviderClient(cloud clientconfig.Cloud, caCert []byte, logger logr.Logg
 		return nil, nil, "", fmt.Errorf("create providerClient err: %v", err)
 	}
 
+	ua := gophercloud.UserAgent{}
+	ua.Prepend(fmt.Sprintf("cluster-api-provider-openstack/%s", version.Get().String()))
+	provider.UserAgent = ua
+
 	config := &tls.Config{
 		MinVersion: tls.VersionTLS12,
 	}

Original file line number	Diff line number	Diff line change
`@@ -37,6 +37,7 @@ import (`
`37`	`37`
`38`	`38`	`infrav1 "sigs.k8s.io/cluster-api-provider-openstack/api/v1alpha7"`
`39`	`39`	`"sigs.k8s.io/cluster-api-provider-openstack/pkg/clients"`
	`40`	`+ "sigs.k8s.io/cluster-api-provider-openstack/version"`
`40`	`41`	`)`
`41`	`42`
`42`	`43`	`const (`
`@@ -152,6 +153,10 @@ func NewProviderClient(cloud clientconfig.Cloud, caCert []byte, logger logr.Logg`
`152`	`153`	`return nil, nil, "", fmt.Errorf("create providerClient err: %v", err)`
`153`	`154`	`}`
`154`	`155`
	`156`	`+ ua := gophercloud.UserAgent{}`
	`157`	`+ ua.Prepend(fmt.Sprintf("cluster-api-provider-openstack/%s", version.Get().String()))`
	`158`	`+ provider.UserAgent = ua`
	`159`	`+`
`155`	`160`	`config := &tls.Config{`
`156`	`161`	`MinVersion: tls.VersionTLS12,`
`157`	`162`	`}`