Skip to content

Conversation

@sijeesh
Copy link
Collaborator

@sijeesh sijeesh commented Feb 18, 2025

Release/ga/2.5.2

Copy link
Collaborator

@datamattsson datamattsson left a comment

Choose a reason for hiding this comment

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

A few things:

  • We also need yaml/csi-driver/v2.5.2 and while at it, can generate the YAML files with helm template -n hpe-storage as all Namespaced resources are pegged to "default" currently.
  • We need a copy of GA releases "values.yaml" in helm/values/csi-driver/v2.5.2
  • Your operators/hpe-csi-operator/Makefile does not correspond to what's in master as the certified destination does not incorporate the name change as per my last PR.

Signed-off-by: sijeesh <sijeesh.kattumunda@hpe.com>
Signed-off-by: sijeesh <sijeesh.kattumunda@hpe.com>
@sijeesh sijeesh changed the base branch from v2.5.2 to master February 19, 2025 09:46
sijeesh and others added 4 commits February 19, 2025 03:11
Signed-off-by: sijeesh <sijeesh.kattumunda@hpe.com>
Signed-off-by: sijeesh <sijeesh.kattumunda@hpe.com>
Signed-off-by: sijeesh <sijeesh.kattumunda@hpe.com>
Signed-off-by: sijeesh <sijeesh.kattumunda@hpe.com>
Signed-off-by: sijeesh <sijeesh.kattumunda@hpe.com>
Signed-off-by: sijeesh <sijeesh.kattumunda@hpe.com>
Copy link
Collaborator

@dileepds dileepds left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@AnushaY1916 AnushaY1916 left a comment

Choose a reason for hiding this comment

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

LGTM

@datamattsson
Copy link
Collaborator

Passes e2e tests on K8s v1.32 on Nimble and OpenShift 4.18-rc10 on Primera.

Signed-off-by: sijeesh <sijeesh.kattumunda@hpe.com>
Signed-off-by: sijeesh <sijeesh.kattumunda@hpe.com>
@sijeesh sijeesh merged commit a829871 into hpe-storage:master Feb 20, 2025
1 check passed
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.

4 participants