Skip to content
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

[refarch] Update node size for GCP and AWS #13356

Merged
merged 1 commit into from
Sep 27, 2022
Merged

[refarch] Update node size for GCP and AWS #13356

merged 1 commit into from
Sep 27, 2022

Conversation

Furisto
Copy link
Member

@Furisto Furisto commented Sep 27, 2022

Description

Update the node size for GCP and AWS self hosted reference architectures as laid out in https://github.com/gitpod-io/website/pull/2790.

Related Issue(s)

Related to https://github.com/gitpod-io/website/pull/2790

How to test

Not sure, would appreciate some pointers.

Release Notes

Update node sizes for self hosted reference architectures

Werft options:

  • /werft with-local-preview
    If enabled this will build install/preview
  • /werft with-preview
  • /werft with-integration-tests=all
    Valid options are all, workspace, webapp, ide

@Furisto Furisto added the team: workspace Issue belongs to the Workspace team label Sep 27, 2022
@Furisto Furisto requested a review from a team September 27, 2022 09:42
@Furisto Furisto self-assigned this Sep 27, 2022
@github-actions github-actions bot added the team: delivery Issue belongs to the self-hosted team label Sep 27, 2022
@nandajavarma
Copy link
Contributor

Hey @Furisto ! Thanks for this fix! I believe the following suggestion is an indicator that we should name the terraform modules better. The terraform setup corresponding to the Single cluster reference architecture is present here. In cluster.tf, you can see that the machine size is specified. We will have to update this value.

The eks module in the cluster.tf file essentially use the same module that you edited, but the variables are overridden in single-cluster ref arch and in the future from other ref arch modules.

@nandajavarma
Copy link
Contributor

/hold Since this is a small change, there might be a tendency to get tit merged. Hence holding. See comment above

@Furisto
Copy link
Member Author

Furisto commented Sep 27, 2022

@nandajavarma Thanks. Added the override for gke and eks.

Copy link
Contributor

@nandajavarma nandajavarma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

/unhold

@roboquat roboquat merged commit 4d3378a into main Sep 27, 2022
@roboquat roboquat deleted the fo/sh-node-size branch September 27, 2022 11:27
@roboquat roboquat added the deployed: workspace Workspace team change is running in production label Oct 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployed: workspace Workspace team change is running in production release-note size/S team: delivery Issue belongs to the self-hosted team team: workspace Issue belongs to the Workspace team
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants