Skip to content

Commit bc4ea9f

Browse files
committed
instrument cloud/scope
1 parent bb98f0e commit bc4ea9f

File tree

5 files changed

+53
-0
lines changed

5 files changed

+53
-0
lines changed

cloud/scope/clients.go

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ import (
3131
"k8s.io/client-go/pkg/version"
3232
"k8s.io/client-go/util/flowcontrol"
3333
infrav1 "sigs.k8s.io/cluster-api-provider-gcp/api/v1beta1"
34+
"sigs.k8s.io/cluster-api-provider-gcp/util/telemetry"
3435
"sigs.k8s.io/controller-runtime/pkg/client"
3536
)
3637

@@ -89,6 +90,12 @@ func defaultClientOptions(ctx context.Context, credentialsRef *infrav1.ObjectRef
8990
}
9091

9192
func newComputeService(ctx context.Context, credentialsRef *infrav1.ObjectReference, crClient client.Client) (*compute.Service, error) {
93+
94+
ctx, span := telemetry.Tracer().Start(
95+
ctx, "cloud.clients.newComputeService",
96+
)
97+
defer span.End()
98+
9299
opts, err := defaultClientOptions(ctx, credentialsRef, crClient)
93100
if err != nil {
94101
return nil, fmt.Errorf("getting default gcp client options: %w", err)
@@ -103,6 +110,12 @@ func newComputeService(ctx context.Context, credentialsRef *infrav1.ObjectRefere
103110
}
104111

105112
func newClusterManagerClient(ctx context.Context, credentialsRef *infrav1.ObjectReference, crClient client.Client) (*container.ClusterManagerClient, error) {
113+
114+
ctx, span := telemetry.Tracer().Start(
115+
ctx, "cloud.clients.newClusterManagerClient",
116+
)
117+
defer span.End()
118+
106119
opts, err := defaultClientOptions(ctx, credentialsRef, crClient)
107120
if err != nil {
108121
return nil, fmt.Errorf("getting default gcp client options: %w", err)
@@ -117,6 +130,12 @@ func newClusterManagerClient(ctx context.Context, credentialsRef *infrav1.Object
117130
}
118131

119132
func newIamCredentialsClient(ctx context.Context, credentialsRef *infrav1.ObjectReference, crClient client.Client) (*credentials.IamCredentialsClient, error) {
133+
134+
ctx, span := telemetry.Tracer().Start(
135+
ctx, "cloud.clients.newIamCredentialsClient",
136+
)
137+
defer span.End()
138+
120139
opts, err := defaultClientOptions(ctx, credentialsRef, crClient)
121140
if err != nil {
122141
return nil, fmt.Errorf("getting default gcp client options: %w", err)
@@ -131,6 +150,12 @@ func newIamCredentialsClient(ctx context.Context, credentialsRef *infrav1.Object
131150
}
132151

133152
func newInstanceGroupManagerClient(ctx context.Context, credentialsRef *infrav1.ObjectReference, crClient client.Client) (*computerest.InstanceGroupManagersClient, error) {
153+
154+
ctx, span := telemetry.Tracer().Start(
155+
ctx, "cloud.clients.newInstanceGroupManagerClient",
156+
)
157+
defer span.End()
158+
134159
opts, err := defaultClientOptions(ctx, credentialsRef, crClient)
135160
if err != nil {
136161
return nil, fmt.Errorf("getting default gcp client options: %w", err)

cloud/scope/cluster.go

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ import (
2727
"k8s.io/utils/pointer"
2828
infrav1 "sigs.k8s.io/cluster-api-provider-gcp/api/v1beta1"
2929
"sigs.k8s.io/cluster-api-provider-gcp/cloud"
30+
"sigs.k8s.io/cluster-api-provider-gcp/util/telemetry"
3031
clusterv1 "sigs.k8s.io/cluster-api/api/v1beta1"
3132
"sigs.k8s.io/cluster-api/util/patch"
3233
"sigs.k8s.io/controller-runtime/pkg/client"
@@ -43,6 +44,12 @@ type ClusterScopeParams struct {
4344
// NewClusterScope creates a new Scope from the supplied parameters.
4445
// This is meant to be called for each reconcile iteration.
4546
func NewClusterScope(ctx context.Context, params ClusterScopeParams) (*ClusterScope, error) {
47+
48+
ctx, span := telemetry.Tracer().Start(
49+
ctx, "cloud.clusterScope.NewClusterScope",
50+
)
51+
defer span.End()
52+
4653
if params.Cluster == nil {
4754
return nil, errors.New("failed to generate new scope from nil Cluster")
4855
}

cloud/scope/managedcluster.go

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ import (
2727
infrav1 "sigs.k8s.io/cluster-api-provider-gcp/api/v1beta1"
2828
"sigs.k8s.io/cluster-api-provider-gcp/cloud"
2929
infrav1exp "sigs.k8s.io/cluster-api-provider-gcp/exp/api/v1beta1"
30+
"sigs.k8s.io/cluster-api-provider-gcp/util/telemetry"
3031
clusterv1 "sigs.k8s.io/cluster-api/api/v1beta1"
3132
"sigs.k8s.io/cluster-api/util/patch"
3233
"sigs.k8s.io/controller-runtime/pkg/client"
@@ -44,6 +45,12 @@ type ManagedClusterScopeParams struct {
4445
// NewManagedClusterScope creates a new Scope from the supplied parameters.
4546
// This is meant to be called for each reconcile iteration.
4647
func NewManagedClusterScope(ctx context.Context, params ManagedClusterScopeParams) (*ManagedClusterScope, error) {
48+
49+
ctx, span := telemetry.Tracer().Start(
50+
ctx, "cloud.managedClusterScope.NewManagedClusterScope",
51+
)
52+
defer span.End()
53+
4754
if params.Cluster == nil {
4855
return nil, errors.New("failed to generate new scope from nil Cluster")
4956
}

cloud/scope/managedcontrolplane.go

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ import (
2121
"fmt"
2222

2323
"sigs.k8s.io/cluster-api-provider-gcp/util/location"
24+
"sigs.k8s.io/cluster-api-provider-gcp/util/telemetry"
2425

2526
"sigs.k8s.io/cluster-api/util/conditions"
2627

@@ -52,6 +53,12 @@ type ManagedControlPlaneScopeParams struct {
5253
// NewManagedControlPlaneScope creates a new Scope from the supplied parameters.
5354
// This is meant to be called for each reconcile iteration.
5455
func NewManagedControlPlaneScope(ctx context.Context, params ManagedControlPlaneScopeParams) (*ManagedControlPlaneScope, error) {
56+
57+
ctx, span := telemetry.Tracer().Start(
58+
ctx, "cloud.managedControlPlaneScope.NewManagedControlPlaneScope",
59+
)
60+
defer span.End()
61+
5562
if params.Cluster == nil {
5663
return nil, errors.New("failed to generate new scope from nil Cluster")
5764
}

cloud/scope/managedmachinepool.go

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ import (
2222

2323
"sigs.k8s.io/cluster-api-provider-gcp/cloud"
2424
"sigs.k8s.io/cluster-api-provider-gcp/util/location"
25+
"sigs.k8s.io/cluster-api-provider-gcp/util/telemetry"
2526

2627
"sigs.k8s.io/cluster-api/util/conditions"
2728

@@ -51,6 +52,12 @@ type ManagedMachinePoolScopeParams struct {
5152
// NewManagedMachinePoolScope creates a new Scope from the supplied parameters.
5253
// This is meant to be called for each reconcile iteration.
5354
func NewManagedMachinePoolScope(ctx context.Context, params ManagedMachinePoolScopeParams) (*ManagedMachinePoolScope, error) {
55+
56+
ctx, span := telemetry.Tracer().Start(
57+
ctx, "cloud.managedMachinePoolScope.NewManagedMachinePoolScope",
58+
)
59+
defer span.End()
60+
5461
if params.Cluster == nil {
5562
return nil, errors.New("failed to generate new scope from nil Cluster")
5663
}

0 commit comments

Comments
 (0)