Remove references to ContainerStats pipeline #1183
Conversation
| @@ -1,5 +1,8 @@ | |||
| # ContainerStats Pipeline | |||
|
|
|||
| > [!CAUTION] | |||
| > The ContainerStats pipeline is deprecated. This page should no longer be consulted. Kubecost APIs no longer use this pipeline's data. | |||
There was a problem hiding this comment.
"deprecated" ≠ "removed". Is this deprecated (currently still works but is signaled for later removal) or removed (currently does not work)?
There was a problem hiding this comment.
Right, sorry! I keep conflating those terms.
In this case ContainerStats "works", but its resulting output is no longer being used by the Aggregator. I think this qualifies as being deprecated, but it may just be safer to say that this feature is being removed.
There was a problem hiding this comment.
I should have keyed in on this earlier (sorry), but if we remove the page then we can't also show this cautionary note. For IBM's docs (where this will be translated), we can't not have a page be referenced in the site map but also still served. So can we just strike mentions elsewhere as you've indicated here?
There was a problem hiding this comment.
Good call! I've addressed your feedback in a commit. I'm now using the term "Removed". And I'm keeping this doc in the SUMMARY.md file.
Related Issue
containerStatsEnabledfrom values files cost-analyzer-helm-chart#3683Proposed Changes
Remove references to ContainerStats, as its data is no longer required in the product.