-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[AWS][Billing] Support Configurable Group By Types #38755
base: main
Are you sure you want to change the base?
[AWS][Billing] Support Configurable Group By Types #38755
Conversation
This pull request does not have a backport label.
To fixup this pull request, you need to add the backport labels for the needed
|
This pull request is now in conflicts. Could you fix it? 🙏
|
This pull request is now in conflicts. Could you fix it? 🙏
|
Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane) |
Enhancement
Proposed commit message
WHAT: Implements the ability to configure the AWS Billing modules "Group By" values. Previously these were hardcoded to
DIMENSION
andTAG
. But AWS supports defining any combo of these values.WHY: Expands the use-cases that the AWS Billing module can support.
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Author's Checklist
DIMENSION
&TAG
setup.group_by_dimension_keys
&group_by_tag_keys
have been marked as deprecated, and are only referenced if the new config valuesgroup_by_primary_keys
&group_by_secondary_keys
are unset.supportedDimensionKeys
as they were last touched ~4 years ago and were out-of-date)How to test this PR locally
main
and run it with a config. This run will be used as the "baseline" for validating this PR.Base Config used for Testing
Base Config used for Testing (no changes to data should be observed)
New Config with two dimension keys
New Config with two dimension keys (more keys)
New Config with two tag keys
New Config using new config values, with two dimensions
Related issues
Use cases
Screenshots
Logs