honeycombio / helm-charts

Helm repository and charts for Honeycomb
Apache License 2.0
30 stars 39 forks source link

Update ST chart to support 1.12.1 #103

Closed kentquirk closed 2 years ago

kentquirk commented 2 years ago

This updates the Secure Tenancy chart to support Release 1.12.1 of Secure Tenancy, including adding support for the new max_parallelism configuration value. A few edits were also made to documentation and readme.

The editor also brought the edited files up to markdown formatting standards, so there are also a bunch of whitespace changes; use github's "ignore whitespace" feature to trim this down to the minimum diffs if necessary.

Closes #102

puckpuck commented 2 years ago

Should the default have such a large increase in resource allocation?

kentquirk commented 2 years ago

@puckpuck It's all commented out -- but we know of at least one case when someone used these values to massively underprovision a deployment. So I thought it better to at least suggest "2 CPUs" and enough memory to matter.