Skip to content

Improve baseline cost output format to show all clusters#214

Merged
homatthew merged 1 commit intoNetflix-Skunkworks:mainfrom
homatthew:mho/baseline-format
Jan 26, 2026
Merged

Improve baseline cost output format to show all clusters#214
homatthew merged 1 commit intoNetflix-Skunkworks:mainfrom
homatthew:mho/baseline-format

Conversation

@homatthew
Copy link
Contributor

Summary

The previous format only showed instance info from the first cluster, which was misleading for composite models like Key-Value that have multiple cluster types (Cassandra, EVCache, dgwkv).

  • Show all clusters with full details: cluster_type, deployment (zonal/regional), instance, count, annual_cost
  • Add attached_drives field per cluster (only when drives exist)
  • Remove misleading single instance_name/instance_count/deployment fields

This makes the baseline output self-documenting and easier to review when costs change.

Example: kv_with_cache (before)

{
  "instance_name": "c6id.2xlarge",
  "instance_count": 39,
  "deployment": "zonal",
  ...
}

Example: kv_with_cache (after)

{
  "clusters": [
    {"cluster_type": "evcache", "deployment": "zonal", "instance": "c6id.2xlarge", "count": 39, ...},
    {"cluster_type": "cassandra", "deployment": "zonal", "instance": "c5d.12xlarge", "count": 2, ...},
    {"cluster_type": "dgwkv", "deployment": "regional", "instance": "c7a.2xlarge", "count": 78,
     "attached_drives": ["gp2 : 20GB"], ...}
  ],
  ...
}

Test plan

  • tox -e pre-commit passes
  • tox -e capture-baseline regenerates with new format
  • Regression tests pass

🤖 Generated with Claude Code

The previous format only showed instance info from the first cluster,
which was misleading for composite models like Key-Value that have
multiple cluster types (Cassandra, EVCache, dgwkv).

Changes:
- Show all clusters with full details: cluster_type, deployment
  (zonal/regional), instance, count, annual_cost
- Add attached_drives field per cluster (only when drives exist)
- Remove misleading single instance_name/instance_count/deployment fields

This makes the baseline output self-documenting and easier to review
when costs change.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@homatthew homatthew merged commit 4165e73 into Netflix-Skunkworks:main Jan 26, 2026
4 checks passed
@homatthew homatthew deleted the mho/baseline-format branch February 18, 2026 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments