Skip to content

Commit

Permalink
update comments
Browse files Browse the repository at this point in the history
  • Loading branch information
dmitryax committed Apr 12, 2024
1 parent e36db38 commit 105494b
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions exporter/loadbalancingexporter/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -192,9 +192,9 @@ exporters:
# except the endpoint
timeout: 3s
resolver:
awsCloudMap:
aws_cloud_map:
namespace: aws-namespace
serviceName: aws-otel-col-service-name
service_name: aws-otel-col-service-name
interval: 30s

service:
Expand Down
2 changes: 1 addition & 1 deletion exporter/loadbalancingexporter/loadbalancer.go
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ func newLoadBalancer(params exporter.CreateSettings, cfg component.Config, facto
}

if oCfg.Resolver.AWSCloudMap != nil {
awsCloudMapLogger := params.Logger.With(zap.String("resolver", "awsCloudMap"))
awsCloudMapLogger := params.Logger.With(zap.String("resolver", "aws_cloud_map"))
var err error
res, err = newCloudMapResolver(awsCloudMapLogger, &oCfg.Resolver.AWSCloudMap.NamespaceName, &oCfg.Resolver.AWSCloudMap.ServiceName, oCfg.Resolver.AWSCloudMap.Port, &oCfg.Resolver.AWSCloudMap.HealthStatus, oCfg.Resolver.AWSCloudMap.Interval, oCfg.Resolver.AWSCloudMap.Timeout)
if err != nil {
Expand Down
6 changes: 3 additions & 3 deletions exporter/loadbalancingexporter/resolver_aws_cloudmap.go
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ const (

var (
errNoNamespace = errors.New("no Cloud Map namespace specified to resolve the backends")
errNoServiceName = errors.New("no Cloud Map serviceName specified to resolve the backends")
errNoServiceName = errors.New("no Cloud Map service_name specified to resolve the backends")

awsResolverMutator = tag.Upsert(tag.MustNewKey("resolver"), "aws")

Expand Down Expand Up @@ -115,10 +115,10 @@ func (r *cloudMapResolver) start(ctx context.Context) error {
go r.periodicallyResolve()

r.logger.Info("AWS CloudMap resolver started",
zap.Stringp("serviceName", r.serviceName),
zap.Stringp("service_name", r.serviceName),
zap.Stringp("namespaceName", r.namespaceName),
zap.Uint16p("port", r.port),
zap.String("healthStatus", string(*r.healthStatus)),
zap.String("health_status", string(*r.healthStatus)),
zap.Duration("interval", r.resInterval), zap.Duration("timeout", r.resTimeout))
return nil
}
Expand Down

0 comments on commit 105494b

Please sign in to comment.