CCC-2211: Add CUR 2.0 Template for CCM CloudFormation Setup Method#334
Conversation
This comment has been minimized.
This comment has been minimized.
…pdated, I don't think we really need this link in the readme
| COST_AND_USAGE_REPORT: | ||
| TIME_GRANULARITY: HOURLY | ||
| INCLUDE_RESOURCES: "TRUE" | ||
| INCLUDE_SPLIT_COST_ALLOCATION_DATA: "TRUE" |
There was a problem hiding this comment.
issue (correctness): Include the split cost allocation columns in the CUR 2.0 query
The table configuration enables INCLUDE_SPLIT_COST_ALLOCATION_DATA, but the SELECT list stops at tags and omits the split_line_item_* columns. The merged aws_cloud_cost_cur2 template selects these fields; without them, the exported Parquet files will not contain the split cost allocation data Datadog expects even though the export advertises it.
— Codex
There was a problem hiding this comment.
that's a good catch, it's not in the minimal set of columns we require so we probably want to not include the split cost allocation by default. It'd be exporting more data than we need
There was a problem hiding this comment.
Ok, just changed it to INCLUDE_SPLIT_COST_ALLOCATION_DATA: "FALSE"
| ## Templates | ||
|
|
||
| - **`main.yaml`** — legacy Cost and Usage Report (CUR), via `AWS::CUR::ReportDefinition`. | ||
| - **`cur2-main.yaml`** — CUR 2.0 / BCM Data Exports, via `AWS::BCMDataExports::Export`. |
There was a problem hiding this comment.
might be out of my depth here so i'll let aws ints confirm but 2 main.yaml files feels like a weird pattern. should we just update the template that David created previously?
There was a problem hiding this comment.
Yeah I can see why it's not ideal. If we updated David's template then the two CUR versions would have a different release.sh, and our existing one is already integrated into this repo's CI (so it doesn't need to be manually run).
There was a problem hiding this comment.
I'm honestly OK with having just one template here (updating mine or just using yours). Feel free to remove mine in this PR, since long term we'll probably be only using just one of these anyways.

Note: Please remember to review the contribution guidelines
if you have not yet done so.
What does this PR do?
Testing Guidelines