[awscontainerinsightreceiver] Enable usage of user defined collection interval in awscontainerinsightreceiver

[TravisStark]

Description

Container Insights supports configuring a collection interval via the CWAgent json config that dictates how often the metrics should be collected from the various sources such as cadvisor.

The collection internal is not used for all the sources the metrics are retrieved from, including:

• DCGM monitor
• neuron monitor
• kueue metrics
• EFA

This change adjusts the containerinsights receiver to pass the CollectionInterval defined in the receiver config on to each scraper.
(metrics_collection_interval in CW Agent config)

opentelemetry-collector-contrib/receiver/awscontainerinsightreceiver/config.go

Line 19 in cb251fe

CollectionInterval time.Duration `mapstructure:"collection_interval"`

opentelemetry-collector-contrib/receiver/awscontainerinsightskueuereceiver/config.go

Line 13 in cb251fe

CollectionInterval time.Duration `mapstructure:"collection_interval"`

Note: For EFA, the original default collection interval value of 20s will be used if:

• collection interval is 60s
• collection interval is not explicitly defined in agent configuration,

Testing

Tested on an EKS Cluster running GPU and Neuron.

Changed CollectionInterval to 30.

{
    "agent":{
        "region":"us-east-1"
    },
    "logs":{
        "metrics_collected":{
            "application_signals":{
                "hosted_in":"multi-accelerator-cluster-1dot32"
            },
            "kubernetes":{
                "cluster_name":"multi-accelerator-cluster-1dot32",
                "enhanced_container_insights":true,
                "metrics_collection_interval": 30
            }
        }
    },
    "traces":{
        "traces_collected" :{
            "application_signals":{}
        }
    }
}    

Count of metrics increased from 1 per minute to 2 per minute.

Graph of DCGM Exporter metrics count with multiple collection intervals specified:
15s, 30s, 60s, and not-specified (defaults to 60s)

[github-actions]

This PR was marked stale due to lack of activity. It will be closed in 14 days.

[TravisStark]

Making a change in the Agent to set the default collection interval to -1, this will be used to infer whether the collection interval was supplied by the config or not

[github-actions]

This PR was marked stale due to lack of activity. It will be closed in 14 days.

[github-actions]

This PR was marked stale due to lack of activity. It will be closed in 14 days.

[github-actions]

Closed as inactive. Feel free to reopen if this PR is still being worked on.