kubecost / kubectl-cost

CLI for determining the cost of Kubernetes workloads
Apache License 2.0
872 stars 55 forks source link

Remove content type header for /prediction/speccost API calls #161

Closed michaelmdresser closed 9 months ago

michaelmdresser commented 10 months ago

What does this PR change?

Remove content type header for /prediction/speccost calls

Backend no longer requires it, makes us flexible for yaml and json

Does this PR rely on any other PRs?

How does this PR impact users? (This is the kind of thing that goes in release notes!)

Added JSON manifest support to kubectl cost predict

How was this PR tested?

Have you made an update to documentation?

chipzoller commented 9 months ago

Tested the server-side change in https://github.com/kubecost/kubecost-cost-model/pull/1916 with kubectl-cost and predictions with an input YAML file are unaffected.