Reduce the memory requests for workspaces

[vulkoingim]

Description

Reduce the memory requests for workspaces. This should allow for more workspaces to fit in preview envs.

Related Issue(s)

Fixes #10028

NONE

[mads-hartmann]

A bit of context that I realise might be missing. The Gitpod Installer (the source in /install/installer) is used to Install Gitpod in general. We use it in preview environment, but also in SaaS and Self-Hosted. So we have to be careful about making changes to the config, and in this case we just want to change the config of Gitpod in preview environments.

To change resource requests for workspaces in preview environments you'd want to make the change in the Werft job where we're generating (and modifying) the configuration file for the Gitpod Installer, rather in the Gitpod Installer itself.

For workspaces that change would be here

gitpod/.werft/jobs/build/installer/installer.ts

Lines 107 to 110 in e148325

	private configureWorkspaces(slice: string) {
	exec(`yq w -i ${this.options.installerConfigPath} workspace.runtime.containerdRuntimeDir ${CONTAINERD_RUNTIME_DIR}`, { slice: slice });
	exec(`yq w -i ${this.options.installerConfigPath} workspace.resources.requests.cpu "100m"`, { slice: slice });
	}

[vulkoingim]

/werft run

👍 started the job as gitpod-build-aa-ws-resources.6
(with .werft/ from main)

[vulkoingim]

A bit of context that I realise might be missing. The Gitpod Installer (the source in /install/installer) is used to Install Gitpod in general. We use it in preview environment, but also in SaaS and Self-Hosted. So we have to be careful about making changes to the config, and in this case we just want to change the config of Gitpod in preview environments.

To change resource requests for workspaces in preview environments you'd want to make the change in the Werft job where we're generating (and modifying) the configuration file for the Gitpod Installer, rather in the Gitpod Installer itself.

For workspaces that change would be here

gitpod/.werft/jobs/build/installer/installer.ts

Lines 107 to 110 in e148325

	private configureWorkspaces(slice: string) {
	exec(`yq w -i ${this.options.installerConfigPath} workspace.runtime.containerdRuntimeDir ${CONTAINERD_RUNTIME_DIR}`, { slice: slice });
	exec(`yq w -i ${this.options.installerConfigPath} workspace.resources.requests.cpu "100m"`, { slice: slice });
	}

Thanks for the context! This makes sense.

Eventually I tried to remove the resource requests from the the installer.ts but for some reason yq d -i... doesn't work there. I saw that it's more widely used in the post-process script, but opted not to do it there. Therefore I've reduced the memory to something relatively low (128m)