fixes for empty chars for pvc prefix

badri-pathak commented 2 months ago

Pull request checklist

Pull request type

This PR is going to fix the issue Custom Volume Prefix fails for wrong values such as empty string

Please check the type of change your PR introduces: - [x] Bugfix - [ ] Feature Enhancement - [ ] Test Automation - [ ] Code Refactoring (no functional changes, no api changes) - [ ] Build related changes - [ ] Community Operator listing - [ ] Other (please describe): ## What is the current behavior?

The current functionality allows empty three char strings also. e.g. "_","a_b","b"

What is the new behavior?

The new fixes will not allow any empty chars in the prefix value. Whenever any wrong or string containing empty chars will pass, it throws an error and default(pvc) will be set as pvc prefix

How risky is this change?

[ ] Small, isolated change
[x] Medium, requires regression testing
[ ] Large, requires functional and regression testing

badri-pathak commented 2 months ago

Image: quay.io/badri_pathak/ibm-spectrum-scale-csi-operator:volNamePrefix_v4

badri-pathak commented 2 months ago

quay.io/badri_pathak/ibm-spectrum-scale-csi-operator:volNamePrefix_v5

saurabhwani5 commented 2 months ago

thanks @badri-pathak , Image with fixes for upper case of pv name change : quay.io/badri_pathak/ibm-spectrum-scale-csi-operator:volNamePrefix_v6

IBM / ibm-spectrum-scale-csi