scylla-operator icon indicating copy to clipboard operation
scylla-operator copied to clipboard

Increase storage and set sysctls in Scylla Helm chart

Open rzetelskik opened this issue 2 years ago • 2 comments

Description of your changes: Currently deploying a Scylla Helm chart with predefined values results in an error (see #906). This PR increases the storage capacity in values.yaml. To mitigate further errors, it also sets sysctls.

Which issue is resolved by this Pull Request: Resolves #906

rzetelskik avatar Aug 10 '22 14:08 rzetelskik

Shouldn't we also set sysctls in scylla cluster for e2e tests?

Choraden avatar Sep 08 '22 10:09 Choraden

in CI / e2e sysctls are tuned for the nodes, I think that's where we should direct this in general but the automatic node sysctl tuning didn't land yet.

tnozicka avatar Sep 08 '22 13:09 tnozicka

@rzetelskik: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-gke f6297c9a989559423bf2655d70da20026c9c7fc2 link true /test e2e-gke

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Closing as stale

rzetelskik avatar Aug 17 '23 14:08 rzetelskik