enable enhanced container insights flag in emf exporter (#131)

amazon-contributing · Oct 25, 2023 · 10cc00e · 10cc00e
1 parent dc59406
commit 10cc00e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/exporter/awsemfexporter/config.go b/exporter/awsemfexporter/config.go
@@ -150,8 +150,7 @@ func (config *Config) Validate() error {
 }
 
 func (config *Config) IsEnhancedContainerInsights() bool {
-	return false // temporarily disable, also need to rename _config to config
-	// return config.EnhancedContainerInsights && !config.DisableMetricExtraction
+	return config.EnhancedContainerInsights && !config.DisableMetricExtraction
 }
 
 func (config *Config) IsAppSignalsEnabled() bool {

diff --git a/exporter/awsemfexporter/config_test.go b/exporter/awsemfexporter/config_test.go
@@ -320,7 +320,7 @@ func TestIsEnhancedContainerInsights(t *testing.T) {
 	cfg := factory.CreateDefaultConfig().(*Config)
 	cfg.EnhancedContainerInsights = true
 	cfg.DisableMetricExtraction = false
-	assert.False(t, cfg.IsEnhancedContainerInsights())
+	assert.True(t, cfg.IsEnhancedContainerInsights())
 	cfg.EnhancedContainerInsights = false
 	assert.False(t, cfg.IsEnhancedContainerInsights())
 	cfg.EnhancedContainerInsights = true