[MetricBeat]keep tags original value, DeDot is not unnecessary?

[kwinstonix]

Describe a specific use case for the enhancement or feature:

tags value may contain dot, eg: IPv4 address. I think It is safe to include dots in the tag value.

beats/x-pack/metricbeat/module/aws/cloudwatch/cloudwatch.go

Lines 617 to 624 in 0c15394

	tags := resourceTagMap[v]
	if len(tags) != 0 {
	// By default, replace dot "." using underscore "_" for tag keys and values
	for _, tag := range tags {
	events[identifier].RootFields.Put("aws.tags."+common.DeDot(*tag.Key), common.DeDot(*tag.Value))
	}
	continue
	}

[elasticmachine]

Pinging @elastic/integrations-platforms (Team:Platforms)

[kaiyan-sheng]

Hi @kwinstonix, thanks for opening this issue! You are right, we only need to dedot for tag key, which will be the json field name in Elasticsearch. I will push a PR to fix this soon. Thank you!