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.

Sign up for GitHub

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

Jump to bottom

update readme and rules for 3.12.0 release #4765

Merged
merged 3 commits into from
Aug 14, 2024
Merged

update readme and rules for 3.12.0 release #4765

merged 3 commits into from
Aug 14, 2024

Conversation

Madhu-1
Copy link
Collaborator

@Madhu-1 Madhu-1 commented Aug 14, 2024 •
edited by nixpanic
Loading

Updating readme and Rules for the 3.12.0 release

Depends-on: #4766

@Madhu-1 Madhu-1 force-pushed the kube.1.31-readme-update branch 2 times, most recently from ab0a694 to 3b14ddc Compare August 14, 2024 08:46
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 14, 2024

/test ci/centos/k8s-e2e-external-storage/1.31

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 14, 2024

/test ci/centos/mini-e2e-helm/k8s-1.31

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 14, 2024

/test ci/centos/mini-e2e/k8s-1.31

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 14, 2024

Running 1.31 tests on #4757

nixpanic
nixpanic previously approved these changes Aug 14, 2024
View reviewed changes
Rakshith-R
Rakshith-R reviewed Aug 14, 2024
View reviewed changes
Copy link
Contributor

@Rakshith-R Rakshith-R left a comment

Choose a reason for hiding this comment

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

Tiny nit,
k8s 1.25 test was added for v3.11 branch at one place

.mergify.yml Outdated
@@ -57,12 +57,15 @@ queue_rules:
- "status-success=ci/centos/k8s-e2e-external-storage/1.26"
- "status-success=ci/centos/k8s-e2e-external-storage/1.27"
- "status-success=ci/centos/k8s-e2e-external-storage/1.28"
- "status-success=ci/centos/k8s-e2e-external-storage/1.29"
- "status-success=ci/centos/mini-e2e-helm/k8s-1.25"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- "status-success=ci/centos/mini-e2e-helm/k8s-1.25"

@mergify mergify bot dismissed nixpanic’s stale review August 14, 2024 09:17

Pull request has been modified.

iPraveenParihar
iPraveenParihar approved these changes Aug 14, 2024
View reviewed changes
Copy link
Contributor

@iPraveenParihar iPraveenParihar left a comment

Choose a reason for hiding this comment

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

Thanks!

@Madhu-1 Madhu-1 added the DNM DO NOT MERGE label Aug 14, 2024
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 14, 2024

Addinng DNM to avoid merging , started CI job for 1.31

@nixpanic
Copy link
Member

Running 1.31 tests on #4757

This one failed: https://jenkins-ceph-csi.apps.ocp.cloud.ci.centos.org/blue/organizations/jenkins/k8s-e2e-external-storage-1.31/detail/k8s-e2e-external-storage-1.31/2/pipeline

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 14, 2024 

[🎩︎]mrajanna@li-2cfbef4c-22d9-11b2-a85c-a3e4a93c405f ceph-csi $]./scripts/minikube.sh up
minikube already installed with v1.31.0
😄  minikube v1.31.0 on Fedora 40
    ▪ KUBECONFIG=/home/mrajanna/workspace/src/github.com/k8s-cluster/kubeconfig
    ▪ MINIKUBE_VERSION=v1.31.0
❗  minikube skips various validations when --force is supplied; this may lead to unexpected behavior
❗  Specified Kubernetes version 1.31.0 is newer than the newest supported version: v1.27.3. Use `minikube config defaults kubernetes-version` for details.
✨  Using the kvm2 driver based on user configuration
👍  Starting control plane node minikube in cluster minikube
🔥  Creating kvm2 VM (CPUs=12, Memory=4096MB, Disk=32768MB) ...
🐳  Preparing Kubernetes v1.31.0 on Docker 24.0.4 ...
    ▪ kubelet.resolv-conf=/run/systemd/resolve/resolv.conf
    ▪ kubelet.read-only-port=10255
❌  Unable to load cached images: loading cached images: stat /home/mrajanna/.minikube/cache/images/amd64/registry.k8s.io/pause_test2: no such file or directory
    ▪ Generating certificates and keys ...
    ▪ Booting up control plane ...
    ▪ Configuring RBAC rules ...
🔗  Configuring bridge CNI (Container Networking Interface) ...
    ▪ Using image gcr.io/k8s-minikube/storage-provisioner:v5
🔎  Verifying Kubernetes components...
🌟  Enabled addons: storage-provisioner, default-storageclass
E0814 13:57:27.599459  171608 pod_ready.go:66] WaitExtra: waitPodCondition: error getting pod "coredns-6f6b679f8f-967t5" in "kube-system" namespace (skipping!): pods "coredns-6f6b679f8f-967t5" not found
🏄  Done! kubectl is now configured to use "minikube" cluster and "default" namespace by default
root        2117    2096 27 11:57 ?        00:00:05 kube-apiserver --advertise-address=192.168.39.84 --allow-privileged=true --authorization-mode=Node,RBAC --client-ca-file=/var/lib/minikube/certs/ca.crt --enable-admission-plugins=NamespaceLifecycle,LimitRanger,ServiceAccount,DefaultStorageClass,DefaultTolerationSeconds,NodeRestriction,MutatingAdmissionWebhook,ValidatingAdmissionWebhook,ResourceQuota --enable-bootstrap-token-auth=true --etcd-cafile=/var/lib/minikube/certs/etcd/ca.crt --etcd-certfile=/var/lib/minikube/certs/apiserver-etcd-client.crt --etcd-keyfile=/var/lib/minikube/certs/apiserver-etcd-client.key --etcd-servers=https://127.0.0.1:2379 --kubelet-client-certificate=/var/lib/minikube/certs/apiserver-kubelet-client.crt --kubelet-client-key=/var/lib/minikube/certs/apiserver-kubelet-client.key --kubelet-preferred-address-types=InternalIP,ExternalIP,Hostname --proxy-client-cert-file=/var/lib/minikube/certs/front-proxy-client.crt --proxy-client-key-file=/var/lib/minikube/certs/front-proxy-client.key --requestheader-allowed-names=front-proxy-client --requestheader-client-ca-file=/var/lib/minikube/certs/front-proxy-ca.crt --requestheader-extra-headers-prefix=X-Remote-Extra- --requestheader-group-headers=X-Remote-Group --requestheader-username-headers=X-Remote-User --secure-port=8443 --service-account-issuer=https://kubernetes.default.svc.cluster.local --service-account-key-file=/var/lib/minikube/certs/sa.pub --service-account-signing-key-file=/var/lib/minikube/certs/sa.key --service-cluster-ip-range=10.96.0.0/12 --tls-cert-file=/var/lib/minikube/certs/apiserver.crt --tls-private-key-file=/var/lib/minikube/certs/apiserver.key
root        3843       1  0 11:57 ?        00:00:00 /var/lib/minikube/binaries/v1.31.0/kubelet --bootstrap-kubeconfig=/etc/kubernetes/bootstrap-kubelet.conf --config=/var/lib/kubelet/config.yaml --container-runtime-endpoint=unix:///var/run/cri-dockerd.sock --hostname-override=minikube --kubeconfig=/etc/kubernetes/kubelet.conf --node-ip=192.168.39.84 --read-only-port=10255 --resolv-conf=/run/systemd/resolve/resolv.conf --v=4
docker      4012    4004  0 11:57 pts/0    00:00:00 bash -c ps -Af |grep kubelet
docker      4014    4012  0 11:57 pts/0    00:00:00 grep kubelet
* "The 'default-storageclass' addon is disabled
* "The 'storage-provisioner' addon is disabled
    > kubectl.sha256:  64 B / 64 B [-------------------------] 100.00% ? p/s 0s
    > kubectl:  53.77 MiB / 53.77 MiB [-------------] 100.00% 6.26 MiB p/s 8.8s
Kubernetes control plane is running at https://192.168.39.84:8443
CoreDNS is running at https://192.168.39.84:8443/api/v1/namespaces/kube-system/services/kube-dns:dns/proxy

To further debug and diagnose cluster problems, use 'kubectl cluster-info dump'.
[🎩︎]mrajanna@li-2cfbef4c-22d9-11b2-a85c-a3e4a93c405f ceph-csi $]kubectl version
Client Version: v1.30.3
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.31.0

it worked locally, lets try one more time 🗡️

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 14, 2024

https://jenkins-ceph-csi.apps.ocp.cloud.ci.centos.org/blue/organizations/jenkins/k8s-e2e-external-storage-1.31/detail/k8s-e2e-external-storage-1.31/3/pipeline/107/ 1.31 passed

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 14, 2024

Removing DNM as all the required tests are passed here #4766

@Madhu-1 Madhu-1 removed the DNM DO NOT MERGE label Aug 14, 2024
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 14, 2024

@Mergifyio rebase

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 14, 2024

@Mergifyio queue

@Madhu-1
doc: readme update for 3.12
7549865 
readme update for 3.12.0 release

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
@Madhu-1
doc: update the upgrade doc
199c379 
updating the upgrade doc to
point to 3.12.0 doc for upgrades.

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
@Madhu-1
ci: update rules for 3.12 release
b322725 
updating release for 3.12 release

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
@mergify Mergify
Copy link
Contributor

mergify bot commented Aug 14, 2024

rebase

✅ Branch has been successfully rebased

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 14, 2024

@nixpanic PTAL

@mergify Mergify
Copy link
Contributor

mergify bot commented Aug 14, 2024 •
edited
Loading

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at b0751cb

@mergify mergify bot added the ok-to-test Label to trigger E2E tests label Aug 14, 2024
@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.27

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.27

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.27

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.29

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.28

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.29

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.28

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.29

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.28

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.30

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.30

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.30

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Aug 14, 2024
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Aug 14, 2024

I think this needs manual merge

@nixpanic
Copy link
Member

I think this needs manual merge

It should not, Mergify can merge changes to its own configuration file.

@mergify mergify bot merged commit b0751cb into ceph:devel Aug 14, 2024
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iPraveenParihar iPraveenParihar iPraveenParihar approved these changes

@Rakshith-R Rakshith-R Rakshith-R approved these changes

@nixpanic nixpanic Awaiting requested review from nixpanic

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Madhu-1 @nixpanic @ceph-csi-bot @iPraveenParihar @Rakshith-R