e2e: update ceph-csi-operator API from v1alpha1 to v1#6218
e2e: update ceph-csi-operator API from v1alpha1 to v1#6218mergify[bot] merged 1 commit intoceph:develfrom
Conversation
|
/test ci/centos/mini-e2e-operator/k8s-1.35 |
|
/test ci/centos/k8s-e2e-external-storage/1.35 |
|
*k8s-e2e-external-storage uses the operator deployment, and that passes. mini-e2e-operator is failing due to something else logs, likely the whole operator e2e suite has been broken (again) since some time 😞 |
|
Taking a look at it. |
|
@Mergifyio update |
❌ Pull request can't be updated with latest base branch changesDetailsThis pull request seems to come from a fork, and Mergify needs the author's permission to update its branch. |
|
@Mergifyio rebase |
3b9b660 to
c00b401
Compare
|
Deprecation notice: This pull request comes from a fork and was rebased using |
✅ Branch has been successfully rebased |
|
/test ci/centos/mini-e2e-operator/k8s-1.35 |
|
/test ci/centos/mini-e2e-operator/k8s-1.35 |
|
@iPraveenParihar , ci/centos/mini-e2e-operator/k8s-1.35 is a test run where e2e is not skipped. |
|
@nixpanic, The failure occurs because the csi-operator does not explicitly set --setmetadata=false. func SetMetadataContainerArg(on bool) string {
return If(on, "--setmetadata=true", "")
}This is problematic while ceph-csi may initially have certain feature flags disabled by default, those defaults can change in future releases. If a feature becomes enabled by default later on, it could unintentionally affect users who have explicitly set the option to false in the Driver CR since the operator does not enforce the false state, the feature may end up being enabled anyway. ceph/ceph-csi-operator#451 should solve the problem. PTAL |
|
/test ci/centos/mini-e2e-operator/k8s-1.35 |
1 similar comment
|
/test ci/centos/mini-e2e-operator/k8s-1.35 |
|
@Mergifyio rebase |
Update the API version in the deploy-ceph-csi-operator.sh script to use csi.ceph.io/v1 instead of csi.ceph.io/v1alpha1 for both OperatorConfig and Driver resources. Signed-off-by: Niels de Vos <ndevos@ibm.com>
c00b401 to
86ed32d
Compare
✅ Branch has been successfully rebased |
|
Deprecation notice: This pull request comes from a fork and was queued with |
Merge Queue Status
This pull request spent 4 minutes 8 seconds in the queue, including 3 minutes 53 seconds running CI. Required conditions to merge
|
Update the API version in the deploy-ceph-csi-operator.sh script to use
csi.ceph.io/v1 instead of csi.ceph.io/v1alpha1 for both OperatorConfig
and Driver resources.
Depends-on: #5672 ceph/ceph-csi-operator#451