Skip to content

Tweaks to the helm deployment#2784

Merged
jayoshih merged 2 commits intolearningequality:hotfixesfrom
aronasorman:full-access-affinity
Jan 14, 2021
Merged

Tweaks to the helm deployment#2784
jayoshih merged 2 commits intolearningequality:hotfixesfrom
aronasorman:full-access-affinity

Conversation

@aronasorman
Copy link
Copy Markdown
Collaborator

@aronasorman aronasorman commented Jan 14, 2021 •
edited
Loading

  • Ensure we have 5 replicas. A temporary hack to make sure every new deployment doesn't scale down the number of replicas to 1. Later one we should convince helm instead to not scale down the number of replicas when set manually.
  • Add in NodeAffinity to workers that ensures they run on nodes that have full GCP access scope. Otherwise they can't access GCS.

Aron Asor added 2 commits January 14, 2021 12:30
set a node affinity for workers
1c8cf08 
This ensures we run the workers on a node that has full GCP access scope
set a default of 5 replicas for the frontend pods
0961c22 
Temporary while I figure out how to convince helm not to overwrite the number of replicas
@aronasorman aronasorman changed the base branch from develop to hotfixes January 14, 2021 20:34
@jayoshih jayoshih merged commit cfe9674 into learningequality:hotfixes Jan 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@jayoshih jayoshih jayoshih approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@aronasorman @jayoshih