From 82a5e2f7c23ac7636729f76628e744f78a4860c3 Mon Sep 17 00:00:00 2001 From: Mayank Pande Date: Mon, 5 Aug 2024 16:44:35 +0530 Subject: [PATCH] fix: fix for nil pointer --- flags/flags.go | 1 + tracing/tracing.go | 5 ++++- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/flags/flags.go b/flags/flags.go index 70c1c3d4..d0c0fc6d 100644 --- a/flags/flags.go +++ b/flags/flags.go @@ -24,6 +24,7 @@ var ( Provider = kingpin.Flag("provider", "`provider` label for `node_cloud_info` metric").Envar("PROVIDER").String() Region = kingpin.Flag("region", "`region` label for `node_cloud_info` metric").Envar("REGION").String() AvailabilityZone = kingpin.Flag("availability-zone", "`availability_zone` label for `node_cloud_info` metric").Envar("AVAILABILITY_ZONE").String() + AccountId = kingpin.Flag("account-id", "`account_id` label for `node_cloud_info` metric").Envar("ACCOUNT_ID").String() InstanceType = kingpin.Flag("instance-type", "`instance_type` label for `node_cloud_info` metric").Envar("INSTANCE_TYPE").String() InstanceLifeCycle = kingpin.Flag("instance-life-cycle", "`instance_life_cycle` label for `node_cloud_info` metric").Envar("INSTANCE_LIFE_CYCLE").String() LogPerSecond = kingpin.Flag("log-per-second", "The number of logs per second").Default("10.0").Envar("LOG_PER_SECOND").Float64() diff --git a/tracing/tracing.go b/tracing/tracing.go index dda0a525..d1874c7e 100644 --- a/tracing/tracing.go +++ b/tracing/tracing.go @@ -61,13 +61,16 @@ func Init(machineId, hostname, version string) { // if md is nil region := "" availabilityZone := "" + accountId := "" if md == nil { region = *flags.Region availabilityZone = *flags.AvailabilityZone + accountId = *flags.AccountId klog.Infoln("no cloud metadata available, using defaults") } else { region = md.Region availabilityZone = md.AvailabilityZone + accountId = md.AccountId } tracer = func(containerId string) trace.Tracer { provider := sdktrace.NewTracerProvider( @@ -78,7 +81,7 @@ func Init(machineId, hostname, version string) { semconv.HostID(machineId), semconv.ServiceName(common.ContainerIdToOtelServiceName(containerId)), semconv.ContainerID(containerId), - semconv.CloudAccountID(md.AccountId), + semconv.CloudAccountID(accountId), semconv.CloudRegion(region), semconv.CloudAvailabilityZone(availabilityZone), )),