Skip to content

Commit

Permalink
Fix errors in integration tests
Browse files Browse the repository at this point in the history
  • Loading branch information
mugdha-adhav committed Mar 7, 2025
1 parent da2c075 commit a2fb0b1
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 10 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/metrics.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ jobs:
uses: helm/kind-action@v1.12.0
with:
cluster_name: kind-${{ github.run_id }}
kubectl_version: "v1.25.2"
kubectl_version: "v1.32.2"
config: ./hack/ci/containerd-cluster-conf.yaml
- name: Build image
run: ./hack/ci/build.sh
Expand Down
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,12 +29,12 @@ git remote set-head origin -a
```

### Migration of CSI driver
In release `v2.0.0`, we are updating the CSI driver name from `container-image.warm-metal.tech` to `container-image.csi.k8s.io`. This change may cause disruptions to your existing workloads if the driver name is not updated.
In release `v2.0.0`, we are updating the CSI driver name from `csi-image.warm-metal.tech` to `container-image.csi.k8s.io`. This change may cause disruptions to your existing workloads if the driver name is not updated.

**To ensure a smooth transition:**
1. **Install Both Versions**: To avoid any breaking changes, you can install both the old and new versions of the CSI driver simultaneously. Both versions are compatible and have been tested to work side-by-side, as verified in our integration tests.

1. **Update Your Workloads**: Migrate your workloads to use the new driver name `container-image.warm-metal.tech`. This process will involve updating your storage class definitions and any other configurations that reference the old driver name.
1. **Update Your Workloads**: Migrate your workloads to use the new driver name `container-image.csi.k8s.io`. This process will involve updating your storage class definitions and any other configurations that reference the old driver name.

1. **Remove the Old Driver**: Once all workloads have been successfully migrated and verified with the new driver, you can safely remove the older version of the driver from your cluster.

Expand Down
4 changes: 2 additions & 2 deletions test/integration/compatible-manifests/ephemeral-volume.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,13 +17,13 @@ spec:
volumes:
- name: rw
csi:
driver: container-image.csi.k8s.io
driver: csi-image.warm-metal.tech
volumeAttributes:
image: "docker.io/warmmetal/container-image-csi-driver-test:simple-fs"
pullAlways: "true"
- name: ro
csi:
driver: container-image.csi.k8s.io
driver: csi-image.warm-metal.tech
readOnly: true
volumeAttributes:
image: "docker.io/warmmetal/container-image-csi-driver-test:simple-fs"
Expand Down
10 changes: 5 additions & 5 deletions test/integration/compatible-manifests/pre-provisioned-pv.yaml
Original file line number Diff line number Diff line change
@@ -1,30 +1,30 @@
kind: StorageClass
apiVersion: storage.k8s.io/v1
metadata:
name: container-image.csi.k8s.io
provisioner: container-image.csi.k8s.io
name: csi-image.warm-metal.tech
provisioner: csi-image.warm-metal.tech
---
apiVersion: v1
kind: PersistentVolume
metadata:
name: compatible-pv-test-container-image-csi-driver-test-simple-fs
spec:
storageClassName: container-image.csi.k8s.io
storageClassName: csi-image.warm-metal.tech
capacity:
storage: 5Gi
accessModes:
- ReadOnlyMany
persistentVolumeReclaimPolicy: Retain
csi:
driver: container-image.csi.k8s.io
driver: csi-image.warm-metal.tech
volumeHandle: "docker.io/warmmetal/container-image-csi-driver-test:simple-fs"
---
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
name: compatible-pvc-test-container-image-csi-driver-test-simple-fs
spec:
storageClassName: container-image.csi.k8s.io
storageClassName: csi-image.warm-metal.tech
accessModes:
- ReadOnlyMany
resources:
Expand Down

0 comments on commit a2fb0b1

Please sign in to comment.