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

Kubernetes Upgrade v1.25 -> v1.29 #299

Merged
merged 4 commits into from
Jul 9, 2024

Conversation

ronardcaktus
Copy link
Member

This PR serves as documentation/reference for the Kubernetes upgrade of the trafficstops-stack-cluster.

❯ kubectl version
Client Version: v1.29.4
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.29.4-eks-036c24b

Ingress Nginx

❯ helm -n ingress-nginx list
NAME            NAMESPACE       REVISION        UPDATED                                 STATUS          CHART                   APP VERSION
ingress-nginx   ingress-nginx   7               2024-05-22 15:05:24.926974667 +0000 UTC deployed        ingress-nginx-4.9.1     1.9.6 

Cert Manager

❯ helm -n cert-manager list
NAME            NAMESPACE       REVISION        UPDATED                                 STATUS          CHART                   APP VERSION
cert-manager    cert-manager    6               2024-05-22 15:06:38.568046174 +0000 UTC deployed        cert-manager-v1.14.3    v1.14.3    

Descheduler Version

❯ kubectl -n kube-system get cronjob -o wide
NAME          SCHEDULE      SUSPEND   ACTIVE   LAST SCHEDULE   AGE    CONTAINERS    IMAGES                                            SELECTOR
descheduler   */2 * * * *   False     1        2s              452d   descheduler   registry.k8s.io/descheduler/descheduler:v0.29.0   <none>

Update New Relic Bundle(nri-bundle)

❯  helm -n newrelic list
NAME            NAMESPACE       REVISION        UPDATED                                 STATUS          CHART                   APP VERSION
newrelic-bundle newrelic        4               2024-05-22 18:04:06.061540342 +0000 UTC deployed        nri-bundle-5.0.68

Cluster and pods version

Screenshot 2024-05-22 at 2 30 51 PM

Notes:

  • The highest Ansible version I could upgrade to was 6.7.0 since the project still runs Python 3.8
  • I found some log references to Papertrail that I believe can be deleted.

Closes:

@copelco
Copy link
Member

copelco commented May 22, 2024

Task linked: CU-8687f7bkt trafficstops-stack-cluster

@@ -106,11 +106,11 @@ k8s_aws_load_balancer_type: nlb
# caktus.k8s-hosting-services: Logging and monitoring configuration
# ----------------------------------------------------------------------------

k8s_papertrail_logspout_destination: "syslog+tls://logs2.papertrailapp.com:20851"
k8s_papertrail_logspout_memory_limit: 128Mi
# k8s_papertrail_logspout_destination: "syslog+tls://logs2.papertrailapp.com:20851"
Copy link
Member Author

Choose a reason for hiding this comment

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

I’m pretty certain this isn’t needed anymore since we moved away from Papertrail, but I did not want to delete without asking first.

Copy link
Collaborator

Choose a reason for hiding this comment

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

@ronardcaktus I would like to defer to either @copelco or @tobiasmcnulty on removing this from the codebase.

@ronardcaktus ronardcaktus requested a review from copelco May 22, 2024 18:39
Copy link
Collaborator

@jeanetteobr jeanetteobr left a comment

Choose a reason for hiding this comment

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

@ronardcaktus This looks good to me. I'm getting the same output from the helm and kubeclt commands as you have listed in PR description! 🎉

@ronardcaktus ronardcaktus force-pushed the CU-8687f7bkt-ts-129-k8s-upgrade branch from 4fb6db6 to e010d3e Compare July 9, 2024 15:38
@ronardcaktus ronardcaktus merged commit a637ce2 into develop Jul 9, 2024
1 check failed
@ronardcaktus ronardcaktus deleted the CU-8687f7bkt-ts-129-k8s-upgrade branch July 9, 2024 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants