Enable EMF by default for the K8s CloudWatch Agent Operator #173
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of the issue
Previously without the operator a user would have needed to expose the EMF ports on the node which could be conflicting, but now that the operator has a Service it's ok to enable the EMF port by default because it won't conflict with other agent installations.
The benefit is that now a user can install Container Insights, and use EMF straightaway out-of-the-box without additional configuration changes.
For this to be useful, this PR needs to be released first: aws/amazon-cloudwatch-agent-operator#182
Description of changes
The change is only for the operator to enable EMF by default, which is useful because it exposes a Service. The other yaml configurations without Service are not as useful because the user would need to specify an IP, or create a Service manually anyway.
License
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Tests
I've tested this manually locally, the pods are Running and emitting EMF metrics:
Also this is the JSON parsed from the new configmap:
Also tested by installing another CWA operator in another namespace, and it's running without having port conflicts:
Requirements
Before committing the code, please verify the following:
Yes it's changing existing sample configurations, however it does not impact existing customer behavior.