Skip to content

feat(settings): per-product Coverage / Include-Exclude / Enabled for SP cards #136

Description

@cristim

Background

PR #123 added four per-plan-type SP cards with independent term/payment defaults. Coverage, include/exclude filters, and enabled toggles still come from the global config to match the RI cards' behaviour.

If users want divergent coverage between Compute and SageMaker SP (e.g., 90% coverage on Compute SP but 50% on SageMaker SP because production utilisation differs), plumb the existing ServiceConfig.Coverage / ServiceConfig.IncludeTypes / ServiceConfig.Enabled fields to the four new rows and expose UI controls per card.

Acceptance criteria

  • Each of the four per-plan-type SP cards exposes its own Coverage slider, Include/Exclude pickers, and Enabled toggle.
  • Save path writes per-card values to the matching (aws, savings-plans-<type>) ServiceConfig row.
  • Backend orchestrator already reads per-row values; no changes needed there.

References

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions