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

[terraform/aks] Implement separate nodepool types, node pool max sizes, machine types #13143

Merged
merged 2 commits into from
Oct 12, 2022

Conversation

adrienthebo
Copy link
Contributor

@adrienthebo adrienthebo commented Sep 20, 2022

Description

This pull request breaks up the AKS workloads into separate services, regular workspace, and headless workspace node pools. It also plumbs through machine type and node pool max instance counts.

Related Issue(s)

See #12740

How to test

Create a self-hosted preview with werft:

werft run github -j .werft/aks-installer-tests.yaml -a deps=external -a preview=true

Then configure the git integration, run a workspace, and ensure that imagebuilds/regular workspaces run on separate nodes.

Release Notes

[terraform/aks] Separate workloads into different pools, expose terraform variables for max node pool counts

Documentation

Werft options:

  • /werft with-preview
  • /werft with-integration-tests=all
    Valid options are all, workspace, webapp, ide

@nandajavarma
Copy link
Contributor

I realized since these nodegroups are getting created in different resource groups, it shouldn't matter. Also, that is how it has always been.

/hold for test to pass https://werft.gitpod-dev.com/job/gitpod-custom-alt-terraform-headless-ws-aks.0

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.

Merge at your discretion! Already holded the PR

@stale
Copy link

stale bot commented Oct 12, 2022

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the meta: stale This issue/PR is stale and will be closed soon label Oct 12, 2022
@adrienthebo adrienthebo force-pushed the alt/terraform/headless-ws/aks branch from b932fd7 to 3afb150 Compare October 12, 2022 15:57
@stale stale bot removed the meta: stale This issue/PR is stale and will be closed soon label Oct 12, 2022
@roboquat roboquat merged commit 36beceb into main Oct 12, 2022
@roboquat roboquat deleted the alt/terraform/headless-ws/aks branch October 12, 2022 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: install Terraform installation scripts, helm charts, installer images release-note size/L team: delivery Issue belongs to the self-hosted team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants