Skip to content

e2e: update ceph-csi-operator API from v1alpha1 to v1#6218

Merged
mergify[bot] merged 1 commit intoceph:develfrom
nixpanic:rebase/ceph-csi-operator/v1
Apr 14, 2026
Merged

e2e: update ceph-csi-operator API from v1alpha1 to v1#6218
mergify[bot] merged 1 commit intoceph:develfrom
nixpanic:rebase/ceph-csi-operator/v1

Conversation

@nixpanic
Copy link
Copy Markdown
Member

@nixpanic nixpanic commented Apr 2, 2026

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

@nixpanic nixpanic added the ci/skip/multi-arch-build skip building on multiple architectures label Apr 2, 2026
@mergify mergify Bot added the component/testing Additional test cases or CI work label Apr 2, 2026
@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 2, 2026

/test ci/centos/mini-e2e-operator/k8s-1.35

@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 2, 2026

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

@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 2, 2026

*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 😞

@nixpanic nixpanic added the ci/skip/e2e skip running e2e CI jobs label Apr 2, 2026
@nixpanic nixpanic requested review from a team April 2, 2026 12:45
@iPraveenParihar
Copy link
Copy Markdown
Contributor

Taking a look at it.

@iPraveenParihar
Copy link
Copy Markdown
Contributor

*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 😞

#6221 should fix this

@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 8, 2026

@Mergifyio update

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 8, 2026

update

❌ Pull request can't be updated with latest base branch changes

Details

This pull request seems to come from a fork, and Mergify needs the author's permission to update its branch.
The author needs to enable "Allow edits from maintainers" on this pull request, or update the branch manually.

@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 8, 2026

@Mergifyio rebase

@ceph-csi-bot ceph-csi-bot force-pushed the rebase/ceph-csi-operator/v1 branch from 3b9b660 to c00b401 Compare April 8, 2026 11:53
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 8, 2026

Deprecation notice: This pull request comes from a fork and was rebased using bot_account impersonation. This capability will be removed on July 1, 2026. After this date, the rebase action will no longer be able to rebase fork pull requests with this configuration. Please switch to the update action/command to ensure compatibility going forward.

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 8, 2026

rebase

✅ Branch has been successfully rebased

@iPraveenParihar
Copy link
Copy Markdown
Contributor

/test ci/centos/mini-e2e-operator/k8s-1.35

@nixpanic nixpanic removed the ci/skip/e2e skip running e2e CI jobs label Apr 8, 2026
@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 8, 2026

/test ci/centos/mini-e2e-operator/k8s-1.35

@nixpanic nixpanic added the ci/skip/e2e skip running e2e CI jobs label Apr 8, 2026
@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 8, 2026

@iPraveenParihar , ci/centos/mini-e2e-operator/k8s-1.35 is a test run where e2e is not skipped.

@iPraveenParihar
Copy link
Copy Markdown
Contributor

iPraveenParihar commented Apr 9, 2026

@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

@iPraveenParihar
Copy link
Copy Markdown
Contributor

/test ci/centos/mini-e2e-operator/k8s-1.35

1 similar comment
@iPraveenParihar
Copy link
Copy Markdown
Contributor

/test ci/centos/mini-e2e-operator/k8s-1.35

@iPraveenParihar
Copy link
Copy Markdown
Contributor

@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>
@ceph-csi-bot ceph-csi-bot force-pushed the rebase/ceph-csi-operator/v1 branch from c00b401 to 86ed32d Compare April 14, 2026 10:57
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 14, 2026

rebase

✅ Branch has been successfully rebased

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 14, 2026

Deprecation notice: This pull request comes from a fork and was queued with update_method=rebase and update_bot_account impersonation. This capability will be removed on July 1, 2026. After this date, the merge queue will no longer be able to rebase fork pull requests with this configuration. To avoid disruption, switch to update_method=merge in your queue rule.

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 14, 2026

Merge Queue Status

  • Entered queue2026-04-14 10:58 UTC · Rule: default
  • Checks passed · in-place
  • Merged2026-04-14 11:02 UTC · at 86ed32d7d0c271530b2767d47884bab5b2f4619e

This pull request spent 4 minutes 8 seconds in the queue, including 3 minutes 53 seconds running CI.

Required conditions to merge

@mergify mergify Bot added the queued label Apr 14, 2026
@mergify mergify Bot merged commit aa4201f into ceph:devel Apr 14, 2026
19 checks passed
@mergify mergify Bot removed the queued label Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/skip/e2e skip running e2e CI jobs ci/skip/multi-arch-build skip building on multiple architectures component/testing Additional test cases or CI work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants