Skip to content

Introduce playbook to update kubernetes cluster's version#37

Open
kishen-v wants to merge 1 commit intokubernetes-sigs:mainfrom
kishen-v:update-k8s-patch
Open

Introduce playbook to update kubernetes cluster's version#37
kishen-v wants to merge 1 commit intokubernetes-sigs:mainfrom
kishen-v:update-k8s-patch

Conversation

@kishen-v
Copy link
Copy Markdown
Contributor

@kishen-v kishen-v commented Jul 4, 2025

This PR contains changes to update the build cluster from one patch/minor version to another through playbooks.
The steps have been documented under /docs in the k8s-ansible directory.

@k8s-ci-robot k8s-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 4, 2025
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jul 4, 2025
@k8s-ci-robot k8s-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jul 4, 2025
@kishen-v kishen-v force-pushed the update-k8s-patch branch from c852294 to 835f4e2 Compare July 6, 2025 07:16
@kishen-v kishen-v changed the title [WIP]Introduce playbook to update kubernetes clusters Introduce playbook to update kubernetes clusters Jul 6, 2025
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 6, 2025
@kishen-v kishen-v changed the title Introduce playbook to update kubernetes clusters Introduce playbook to update kubernetes cluster's version Jul 6, 2025
@kishen-v
Copy link
Copy Markdown
Contributor Author

kishen-v commented Jul 9, 2025

/test pull-provider-ibmcloud-test-infra-kubernetes

@kishen-v
Copy link
Copy Markdown
Contributor Author

kishen-v commented Jul 28, 2025

/cc @Rajalakshmi-Girish

Rajalakshmi-Girish
Rajalakshmi-Girish requested changes Sep 2, 2025
View reviewed changes
Comment thread kubetest2-tf/data/k8s-ansible/roles/download-k8s/tasks/main.yml
Comment thread kubetest2-tf/data/k8s-ansible/roles/install-k8s/templates/override.conf.j2
@kishen-v
Copy link
Copy Markdown
Contributor Author

kishen-v commented Sep 30, 2025

/cc @Rajalakshmi-Girish

@kishen-v
Copy link
Copy Markdown
Contributor Author

kishen-v commented Oct 7, 2025

Thanks @Rajalakshmi-Girish for sharing the comments, I've made changes as pointed..

Rajalakshmi-Girish
Rajalakshmi-Girish requested changes Oct 7, 2025
View reviewed changes
Comment thread kubetest2-tf/data/k8s-ansible/docs/update-os.md Outdated
Rajalakshmi-Girish
Rajalakshmi-Girish requested changes Oct 7, 2025
View reviewed changes
Comment thread kubetest2-tf/data/k8s-ansible/docs/update-k8s.md Outdated
Comment thread kubetest2-tf/data/k8s-ansible/docs/update-os.md
@kishen-v kishen-v force-pushed the update-k8s-patch branch 2 times, most recently from 52d57bb to 2dec8eb Compare October 8, 2025 07:14
Rajalakshmi-Girish
Rajalakshmi-Girish requested changes Oct 10, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@Rajalakshmi-Girish Rajalakshmi-Girish left a comment

Choose a reason for hiding this comment

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

/lgtm except the minor comment on bastion node related data in update-os.md

Comment thread kubetest2-tf/data/k8s-ansible/docs/update-os.md Outdated
@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 10, 2025
@Rajalakshmi-Girish
Copy link
Copy Markdown
Contributor

Rajalakshmi-Girish commented Oct 16, 2025

/lgtm
/hold for @mkumatag final walk through.

@k8s-ci-robot k8s-ci-robot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 16, 2025
@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 16, 2025
@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jan 13, 2026
@k8s-ci-robot k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 9, 2026
@k8s-ci-robot k8s-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 9, 2026
@kishen-v
Copy link
Copy Markdown
Contributor Author

kishen-v commented Feb 9, 2026

Retested once again on the internal build cluster -

Bump from 1.34.2 to 1.34.3
kishenv@kishen-mbp-m1 ~ % kubectl get nodes -A -o wide -w
	    NAME              STATUS   ROLES           AGE    VERSION   INTERNAL-IP      EXTERNAL-IP   OS-IMAGE                      KERNEL-VERSION            CONTAINER-RUNTIME
            compute-0         Ready    <none>          151d   v1.34.2   192.168.25.219   <none>        CentOS Stream 10 (Coughlan)   6.12.0-126.el10.ppc64le   containerd://1.7.13
            control-plane-0   Ready    control-plane   151d   v1.34.2   192.168.25.112   <none>        CentOS Stream 10 (Coughlan)   6.12.0-126.el10.ppc64le   containerd://1.7.13
            control-plane-1   Ready    control-plane   151d   v1.34.2   192.168.25.247   <none>        CentOS Stream 10 (Coughlan)   6.12.0-126.el10.ppc64le   containerd://1.7.13
            control-plane-2   Ready    control-plane   151d   v1.34.2   192.168.25.91    <none>        CentOS Stream 10 (Coughlan)   6.12.0-126.el10.ppc64le   containerd://1.7.13


kishenv@kishen-mbp-m1 ~ % kubectl get nodes -A -o wide -w
            NAME              STATUS   ROLES           AGE    VERSION   INTERNAL-IP      EXTERNAL-IP   OS-IMAGE                      KERNEL-VERSION            CONTAINER-RUNTIME
	    compute-0         Ready    <none>          151d   v1.34.3   192.168.25.219   <none>        CentOS Stream 10 (Coughlan)   6.12.0-126.el10.ppc64le   containerd://1.7.13
            control-plane-0   Ready    control-plane   151d   v1.34.3   192.168.25.112   <none>        CentOS Stream 10 (Coughlan)   6.12.0-126.el10.ppc64le   containerd://1.7.13
            control-plane-1   Ready    control-plane   151d   v1.34.3   192.168.25.247   <none>        CentOS Stream 10 (Coughlan)   6.12.0-126.el10.ppc64le   containerd://1.7.13
            control-plane-2   Ready    control-plane   151d   v1.34.3   192.168.25.91    <none>        CentOS Stream 10 (Coughlan)   6.12.0-126.el10.ppc64le   containerd://1.7.13
______________________________________________________________________________________________

All pods reporting in as healthy
kishenv@kishen-mbp-m1 ~ % kubectl get pods -n kube-system
NAME                                       READY   STATUS    RESTARTS        AGE
calico-kube-controllers-6fd9cc49d6-84w5t   1/1     Running   0               15m
calico-node-cpvsz                          1/1     Running   0               151d
calico-node-g6cq8                          1/1     Running   0               151d
calico-node-jwln5                          1/1     Running   0               151d
calico-node-k9wfm                          1/1     Running   1 (73d ago)     151d
coredns-66bc5c9577-l6ks7                   1/1     Running   0               15m
coredns-66bc5c9577-r2x54                   1/1     Running   0               10m
etcd-control-plane-0                       1/1     Running   1 (18m ago)     55m
etcd-control-plane-1                       1/1     Running   1 (9m20s ago)   29m
etcd-control-plane-2                       1/1     Running   1 (14m ago)     35m
kube-apiserver-control-plane-0             1/1     Running   0               18m
kube-apiserver-control-plane-1             1/1     Running   0               9m
kube-apiserver-control-plane-2             1/1     Running   0               14m
kube-controller-manager-control-plane-0    1/1     Running   0               17m
kube-controller-manager-control-plane-1    1/1     Running   0               8m19s
kube-controller-manager-control-plane-2    1/1     Running   0               13m
kube-proxy-kt2xx                           1/1     Running   0               8m3s
kube-proxy-sjxsf                           1/1     Running   0               7m57s
kube-proxy-sm59t                           1/1     Running   0               8m2s
kube-proxy-xvlgk                           1/1     Running   0               8m1s
kube-scheduler-control-plane-0             1/1     Running   0               17m
kube-scheduler-control-plane-1             1/1     Running   0               8m5s
kube-scheduler-control-plane-2             1/1     Running   0               13m
______________________________________________________________________________________________
kishenv@kishen-mbp-m1 ~ % kubectl version
Client Version: v1.31.2
Kustomize Version: v5.4.2
Server Version: v1.34.3                   <--------------
WARNING: version difference between client (1.31) and server (1.34) exceeds the supported minor version skew of +/-1

@kishen-v
Copy link
Copy Markdown
Contributor Author

kishen-v commented Feb 10, 2026

/test pull-provider-ibmcloud-test-infra-kubetest2-secretmanager-build

@kishen-v kishen-v mentioned this pull request Feb 10, 2026
Open
kishen-v
kishen-v commented Feb 11, 2026
View reviewed changes
Comment thread kubetest2-tf/data/k8s-ansible/roles/download-k8s/tasks/main.yml
@kishen-v kishen-v force-pushed the update-k8s-patch branch 2 times, most recently from a0c62c6 to 127b6e1 Compare March 12, 2026 06:42
@kishen-v
Copy link
Copy Markdown
Contributor Author

kishen-v commented Mar 12, 2026

/test pull-provider-ibmcloud-test-infra-kubernetes

Rajalakshmi-Girish
Rajalakshmi-Girish reviewed Mar 12, 2026
View reviewed changes
Comment thread kubetest2-tf/data/k8s-ansible/roles/install-k8s/templates/kubelet.service.j2
Comment thread kubetest2-tf/data/k8s-ansible/roles/runtime/files/crio/crio.service
Comment thread kubetest2-tf/data/k8s-ansible/roles/runtime/tasks/main.yaml
Comment thread kubetest2-tf/data/k8s-ansible/roles/update-k8s-patch/tasks/main.yml
Rajalakshmi-Girish
Rajalakshmi-Girish approved these changes Apr 1, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@Rajalakshmi-Girish Rajalakshmi-Girish left a comment

Choose a reason for hiding this comment

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

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 1, 2026
@k8s-ci-robot
Copy link
Copy Markdown
Contributor

k8s-ci-robot commented Apr 1, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kishen-v, Rajalakshmi-Girish

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mkumatag
Copy link
Copy Markdown
Member

mkumatag commented Apr 8, 2026

@Rajalakshmi-Girish @kishen-v I will defer this PR for you folks, I'm not getting enough time for reviewing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mkumatag mkumatag mkumatag left review comments

@Rajalakshmi-Girish Rajalakshmi-Girish Rajalakshmi-Girish approved these changes

@Prajyot-Parab Prajyot-Parab Awaiting requested review from Prajyot-Parab

Assignees

@Rajalakshmi-Girish Rajalakshmi-Girish

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@kishen-v @Rajalakshmi-Girish @k8s-ci-robot @mkumatag