Batch of upgrades and cleanups

OperationCode / operationcode_infra

Platform infrastructure for the Operation Code site.

MIT License

15 stars 14 forks source link

Batch of upgrades and cleanups #173

Closed irvingpop closed 3 years ago

irvingpop commented 3 years ago

This PR is purely to sync our codebase with reality. These updates are already tested and deployed out to our cluster.

Upgrade our EKS cluster to 1.20 and switch to a Managed Node Group
- This lets us take advantage of higher pod:node ratios due to networking advances
Add S3-based logging to our ALBs so we can better catch errors when they alarm
Upgrade ArgoCD to the latest, 2.0.5
Clean up Vertical Pod Autoscalers (VPAs) since we don't use them anymore

Signed-off-by: Irving Popovetsky irving@honeycomb.io

aaron-junot commented 3 years ago

@irvingpop looks like it failed CI because it couldn't install a dnsimple plugin. Are we concerned about that or should we merge in spite of failures?

irvingpop commented 3 years ago

hey @aaron-suarez I noticed that as well. We're at the point now where we aren't really using any of the Terraform<->DNSimple stuff anyway.

My preference would be to start cleaning it up in a future PR, but I'm open to the idea of at least removing those CI checks in this PR so it's green. what do you think?

aaron-junot commented 3 years ago

I'm fine merging it now and fixing it in a new PR. No need to drag it out if we're not using the functionality anyway. Ready for me to merge?

irvingpop commented 3 years ago

let's do it!