-
Notifications
You must be signed in to change notification settings - Fork 308
ROB-3611: updated docs to match new cluster-sizes approach #2066
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
nimrik
wants to merge
4
commits into
master
Choose a base branch
from
ROB-3611-cluster-sizes-doc-update
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
4 commits
Select commit
Hold shift + click to select a range
ac2ecc0
updated docs to match new cluster-sizes approach
nimrik e3ba6cf
Merge branch 'master' into ROB-3611-cluster-sizes-doc-update
nimrik 42cca4e
resolved conflicts
nimrik 754b625
Merge branch 'master' into ROB-3611-cluster-sizes-doc-update
moshemorad File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
97 changes: 97 additions & 0 deletions
97
docs/setup-robusta/installation/_helm_install_no_prometheus.inc.rst
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,97 @@ | ||
| .. updated to .inc.rst because of "WARNING: duplicate label" | ||
|
|
||
| Install with Helm | ||
| ------------------------------ | ||
|
|
||
| Copy the below commands, replacing the ``<YOUR_CLUSTER_NAME>`` placeholder. | ||
|
|
||
| On some clusters this can take a while, so don't panic if it appears stuck: | ||
|
|
||
| .. tab-set:: | ||
|
|
||
| .. tab-item:: Normal Clusters | ||
| :name: install-standard | ||
|
|
||
| .. code-block:: bash | ||
| :name: cb-helm-install-only-robusta | ||
|
|
||
| helm repo add robusta https://robusta-charts.storage.googleapis.com && helm repo update | ||
| helm install robusta robusta/robusta -f ./generated_values.yaml --set clusterName=<YOUR_CLUSTER_NAME> | ||
|
|
||
| .. tab-item:: EKS | ||
| :name: install-eks | ||
|
|
||
| To use all Robusta features, ensure storage is enabled on your cluster. If necessary, refer to the EKS documentation and install the `EBS CSI add-on <https://docs.aws.amazon.com/eks/latest/userguide/managing-ebs-csi.html#adding-ebs-csi-eks-add-on>`_ | ||
|
|
||
| .. details:: How do I know if my cluster has storage enabled? | ||
|
|
||
| Try installing Robusta. If storage is not configured, you'll receive an error: | ||
|
|
||
| .. code-block:: | ||
|
|
||
| PreBind plugin "VolumeBinding": binding volumes: timed out waiting for the condition | ||
|
|
||
| Running ``kubectl get pvc -A`` will also show PersistentVolumeClaims in ``Pending`` state. | ||
|
|
||
| In this case, follow the instructions above and enable storage for your cluster. | ||
|
|
||
| .. code-block:: bash | ||
| :name: cb-helm-install-eks | ||
|
|
||
| helm repo add robusta https://robusta-charts.storage.googleapis.com && helm repo update | ||
| helm install robusta robusta/robusta -f ./generated_values.yaml --set clusterName=<YOUR_CLUSTER_NAME> | ||
|
|
||
| .. tab-item:: GKE Autopilot | ||
| :name: install-gke-autopilot | ||
|
|
||
| Due to Autopilot restrictions, some components are disabled. Don't worry, everything will still work. | ||
|
|
||
| .. code-block:: bash | ||
| :name: cb-helm-install-gke-autopilot | ||
|
|
||
| helm repo add robusta https://robusta-charts.storage.googleapis.com && helm repo update | ||
| helm install robusta robusta/robusta -f ./generated_values.yaml \ | ||
| --set clusterName=<YOUR_CLUSTER_NAME> \ | ||
| --set kube-prometheus-stack.coreDns.enabled=false \ | ||
| --set kube-prometheus-stack.kubeControllerManager.enabled=false \ | ||
| --set kube-prometheus-stack.kubeDns.enabled=false \ | ||
| --set kube-prometheus-stack.kubeEtcd.enabled=false \ | ||
| --set kube-prometheus-stack.kubeProxy.enabled=false \ | ||
| --set kube-prometheus-stack.kubeScheduler.enabled=false \ | ||
| --set kube-prometheus-stack.nodeExporter.enabled=false \ | ||
| --set kube-prometheus-stack.prometheusOperator.kubeletService.enabled=false | ||
|
|
||
| .. tab-item:: OpenShift | ||
| :name: install-openshift | ||
|
|
||
| First :ref:`modify the Helm values to enable OpenShift support<openshift-permissions>`. | ||
|
|
||
| Then install Robusta as usual with Helm: | ||
|
|
||
| .. code-block:: bash | ||
| :name: cb-helm-install-openshift | ||
|
|
||
| helm repo add robusta https://robusta-charts.storage.googleapis.com && helm repo update | ||
| helm install robusta robusta/robusta -f ./generated_values.yaml --set clusterName=<YOUR_CLUSTER_NAME> | ||
|
|
||
| .. tab-item:: Local/Test Cluster | ||
| :name: install-test-clusters | ||
|
|
||
| Test clusters tend to have fewer resources. To lower Robusta's resource requests, set ``clusterSize=small``. | ||
|
|
||
| .. code-block:: bash | ||
| :name: cb-helm-install-test-clusters | ||
|
|
||
| helm repo add robusta https://robusta-charts.storage.googleapis.com && helm repo update | ||
| helm install robusta robusta/robusta -f ./generated_values.yaml --set clusterName=<YOUR_CLUSTER_NAME> --set clusterSize=small | ||
|
|
||
| Verifying Installation | ||
| ------------------------------ | ||
|
|
||
| Confirm that Robusta pods are running with no errors in the logs: | ||
|
|
||
| .. code-block:: bash | ||
| :name: cb-get-pods-robusta-logs | ||
|
|
||
| kubectl get pods -A | grep robusta | ||
| robusta logs |
108 changes: 108 additions & 0 deletions
108
docs/setup-robusta/installation/_helm_install_with_prometheus.inc.rst
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,108 @@ | ||
| .. updated to .inc.rst because of "WARNING: duplicate label" | ||
|
|
||
| Install with Helm | ||
| ------------------------------ | ||
|
|
||
| Copy the below commands, replacing the ``<YOUR_CLUSTER_NAME>`` placeholder. | ||
|
|
||
| On some clusters this can take a while, so don't panic if it appears stuck: | ||
|
|
||
| .. tab-set:: | ||
|
|
||
| .. tab-item:: Normal Clusters | ||
| :name: install-standard | ||
|
|
||
| .. code-block:: bash | ||
| :name: cb-helm-install-only-robusta | ||
|
|
||
| helm repo add robusta https://robusta-charts.storage.googleapis.com && helm repo update | ||
| helm install robusta robusta/robusta -f ./generated_values.yaml --set clusterName=<YOUR_CLUSTER_NAME> | ||
|
|
||
| .. tab-item:: EKS | ||
| :name: install-eks | ||
|
|
||
| To use all Robusta features, ensure storage is enabled on your cluster. If necessary, refer to the EKS documentation and install the `EBS CSI add-on <https://docs.aws.amazon.com/eks/latest/userguide/managing-ebs-csi.html#adding-ebs-csi-eks-add-on>`_ | ||
|
|
||
| .. details:: How do I know if my cluster has storage enabled? | ||
|
|
||
| Try installing Robusta. If storage is not configured, you'll receive an error: | ||
|
|
||
| .. code-block:: | ||
|
|
||
| PreBind plugin "VolumeBinding": binding volumes: timed out waiting for the condition | ||
|
|
||
| Running ``kubectl get pvc -A`` will also show PersistentVolumeClaims in ``Pending`` state. | ||
|
|
||
| In this case, follow the instructions above and enable storage for your cluster. | ||
|
|
||
| .. code-block:: bash | ||
| :name: cb-helm-install-eks | ||
|
|
||
| helm repo add robusta https://robusta-charts.storage.googleapis.com && helm repo update | ||
| helm install robusta robusta/robusta -f ./generated_values.yaml --set clusterName=<YOUR_CLUSTER_NAME> | ||
|
|
||
| .. tab-item:: GKE Autopilot | ||
| :name: install-gke-autopilot | ||
|
|
||
| Due to Autopilot restrictions, some components are disabled. Don't worry, everything will still work. | ||
|
|
||
| .. code-block:: bash | ||
| :name: cb-helm-install-gke-autopilot | ||
|
|
||
| helm repo add robusta https://robusta-charts.storage.googleapis.com && helm repo update | ||
| helm install robusta robusta/robusta -f ./generated_values.yaml \ | ||
| --set clusterName=<YOUR_CLUSTER_NAME> \ | ||
| --set kube-prometheus-stack.coreDns.enabled=false \ | ||
| --set kube-prometheus-stack.kubeControllerManager.enabled=false \ | ||
| --set kube-prometheus-stack.kubeDns.enabled=false \ | ||
| --set kube-prometheus-stack.kubeEtcd.enabled=false \ | ||
| --set kube-prometheus-stack.kubeProxy.enabled=false \ | ||
| --set kube-prometheus-stack.kubeScheduler.enabled=false \ | ||
| --set kube-prometheus-stack.nodeExporter.enabled=false \ | ||
| --set kube-prometheus-stack.prometheusOperator.kubeletService.enabled=false | ||
|
|
||
| .. tab-item:: OpenShift | ||
| :name: install-openshift | ||
|
|
||
| First :ref:`modify the Helm values to enable OpenShift support<openshift-permissions>`. | ||
|
|
||
| Then install Robusta as usual with Helm: | ||
|
|
||
| .. code-block:: bash | ||
| :name: cb-helm-install-openshift | ||
|
|
||
| helm repo add robusta https://robusta-charts.storage.googleapis.com && helm repo update | ||
| helm install robusta robusta/robusta -f ./generated_values.yaml --set clusterName=<YOUR_CLUSTER_NAME> | ||
|
|
||
| .. tab-item:: Local/Test Cluster | ||
| :name: install-test-clusters | ||
|
|
||
| Test clusters tend to have fewer resources. To lower Robusta's resource requests, set ``clusterSize=small``. | ||
|
|
||
| .. code-block:: bash | ||
| :name: cb-helm-install-test-clusters | ||
|
|
||
| helm repo add robusta https://robusta-charts.storage.googleapis.com && helm repo update | ||
| helm install robusta robusta/robusta -f ./generated_values.yaml --set clusterName=<YOUR_CLUSTER_NAME> --set clusterSize=small \ | ||
| --set kube-prometheus-stack.prometheus.prometheusSpec.retentionSize=9GB \ | ||
| --set kube-prometheus-stack.prometheus.prometheusSpec.storageSpec.volumeClaimTemplate.spec.resources.requests.storage=10Gi \ | ||
| --set kube-prometheus-stack.prometheus.prometheusSpec.resources.requests.memory=512Mi | ||
|
|
||
| .. note:: | ||
| If you are using docker desktop you will need to disable prometheus-node-exporter mounting host root, by adding the following to the above command: | ||
|
|
||
| .. code-block:: bash | ||
| :name: disable host mount | ||
|
|
||
| --set kube-prometheus-stack.prometheus-node-exporter.hostRootFsMount.enabled=false | ||
|
|
||
| Verifying Installation | ||
| ------------------------------ | ||
|
|
||
| Confirm that Robusta pods are running with no errors in the logs: | ||
|
|
||
| .. code-block:: bash | ||
| :name: cb-get-pods-robusta-logs | ||
|
|
||
| kubectl get pods -A | grep robusta | ||
| robusta logs | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
🏁 Script executed:
Repository: robusta-dev/robusta
Length of output: 3404
🏁 Script executed:
Repository: robusta-dev/robusta
Length of output: 122
🏁 Script executed:
Repository: robusta-dev/robusta
Length of output: 3219
🏁 Script executed:
Repository: robusta-dev/robusta
Length of output: 6246
Fix documentation or chart:
--set clusterSize=smalldoes not reduce resource requests.The docs instruct
--set clusterSize=small, but the Helm chart never readsclusterSize. Templates check.Values.isSmallCluster(in runner.yaml and forwarder.yaml) to reduce memory/storage requests for test clusters. No mapping exists between the two parameters, so following the docs produces no effect—test clusters will use full resource requests and may fail silently.Either update docs to use
--set isSmallCluster=true, or add mapping logic in the chart (e.g., in_helpers.tplor values.yaml) to setisSmallClusterbased onclusterSize.🤖 Prompt for AI Agents