Skip to content
This repository has been archived by the owner on Jun 29, 2022. It is now read-only.

aws ci: Use c5a.2xlarge for controllers #1510

Merged
merged 1 commit into from
Jun 22, 2021

Conversation

surajssd
Copy link
Member

AWS CI keeps failing on cert-rotation test because apiserver takes a
really long time to come back up.

It is possible that the failure rate on AWS is higher because the
machine type on AWS is tiny which is unable to sustain the load
generated due to control plane component restarts.

This commit upgrades the control plane machine to fairly larger one.

@surajssd surajssd requested review from iaguis and invidian June 21, 2021 14:01
invidian
invidian previously approved these changes Jun 21, 2021
Copy link
Member

@invidian invidian left a comment

Choose a reason for hiding this comment

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

So change from 2 vCPU/4GB RAM to 4 vCPU/16GB RAM, sounds reasonable.

In my experience 8GB of RAM can sometimes struggle with handling 2 API servers.

invidian
invidian previously approved these changes Jun 22, 2021
@surajssd
Copy link
Member Author

Moved the machine from the default to t3.xlarge but that did not help. Neither did moving to t3.2xlarge. I think there is something specific to AWS other than the machine type is causing issues. I will give one last try and then venture on to look for other solutions to fix this problem.

AWS CI keeps failing on cert-rotation test because apiserver takes a
really long time to come back up.

It is possible that the failure rate on AWS is higher because the
machine type on AWS is tiny which is unable to sustain the load
generated due to control plane component restarts.

This commit upgrades the control plane machine to fairly larger one.

Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
@surajssd surajssd force-pushed the surajssd/upgrade-controller-machine-type branch from 001c2b6 to 948ac6e Compare June 22, 2021 10:36
@surajssd surajssd changed the title aws ci: Use t3.xlarge for controllers aws ci: Use c5a.2xlarge for controllers Jun 22, 2021
@surajssd surajssd requested a review from invidian June 22, 2021 14:40
@surajssd surajssd merged commit d8fe3ac into master Jun 22, 2021
@surajssd surajssd deleted the surajssd/upgrade-controller-machine-type branch June 22, 2021 14:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants