Skip to content

e2e: use the NVMe-oF gateway that is deployed by Rook#6236

Open
nixpanic wants to merge 2 commits into
ceph:develfrom
nixpanic:issue/5618
Open

e2e: use the NVMe-oF gateway that is deployed by Rook#6236
nixpanic wants to merge 2 commits into
ceph:develfrom
nixpanic:issue/5618

Conversation

@nixpanic
Copy link
Copy Markdown
Member

@nixpanic nixpanic commented Apr 14, 2026

The IP-address of the gateway is configured in the StorageClass. This is
required as the short-hostname can not be resolved in DNS. The gateway and
Ceph-CSI get deployed in different namespaces, which means the domain names
differ.

The same issue happens in sripts/k8s-storage/create-storageclasses.sh.

Depends-on: #6214

@nixpanic nixpanic added the ci/skip/multi-arch-build skip building on multiple architectures label Apr 14, 2026
@nixpanic nixpanic linked an issue Apr 14, 2026 that may be closed by this pull request
nvmeof: remove gateway deployment in e2e suite #5618
Open
@mergify mergify Bot added component/nvme-of Issues and PRs related to NVMe-oF. component/testing Additional test cases or CI work labels Apr 14, 2026
@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 14, 2026

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

@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 14, 2026

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

@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 14, 2026

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

@nixpanic nixpanic marked this pull request as draft April 14, 2026 15:30
@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 14, 2026

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

@nixpanic nixpanic marked this pull request as ready for review April 14, 2026 15:59
@nixpanic nixpanic requested review from a team and gadididi April 14, 2026 15:59
gadididi
gadididi approved these changes Apr 15, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@gadididi gadididi left a comment

Choose a reason for hiding this comment

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

1 nit.
You don’t have to change it if you don’t want to
thanks!

Comment thread e2e/nvmeof-deploy.go
@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 15, 2026

@Mergifyio rebase

nixpanic added 2 commits April 15, 2026 13:11
@nixpanic @ceph-csi-bot
ci: deploy the NVMe-oF gateway with Rook
140e672 
Signed-off-by: Niels de Vos <ndevos@ibm.com>
@nixpanic @ceph-csi-bot
e2e: use the NVMe-oF gateway that is deployed by Rook
52b8713 
The IP-address of the gateway is configured in the StorageClass. This is
required as the short-hostname can not be resolved in DNS. The gateway
and Ceph-CSI get deployed in different namespaces, which means the
domain names differ.

The same issue happens in sripts/k8s-storage/create-storageclasses.sh.

Signed-off-by: Niels de Vos <ndevos@ibm.com>
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 15, 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 15, 2026

rebase

✅ Branch has been successfully rebased

@nixpanic nixpanic added the ok-to-test Label to trigger E2E tests label Apr 15, 2026
@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Apr 15, 2026

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Apr 15, 2026

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

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Apr 15, 2026

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

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Apr 15, 2026

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Apr 15, 2026

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

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Apr 15, 2026

/test ci/centos/mini-e2e-helm/k8s-1.33

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Apr 15, 2026

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

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Apr 15, 2026

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

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Apr 15, 2026

/test ci/centos/mini-e2e/k8s-1.33

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Apr 15, 2026

/test ci/centos/mini-e2e-helm/k8s-1.34

@ceph-csi-bot
Copy link
Copy Markdown
Collaborator

ceph-csi-bot commented Apr 15, 2026

/test ci/centos/mini-e2e/k8s-1.34

@ceph-csi-bot ceph-csi-bot added ci/in-progress/e2e This label acts like a guard and prevents Mergify from adding the `ok-to-test` label again. and removed ok-to-test Label to trigger E2E tests labels Apr 15, 2026
@mergify mergify Bot removed the ci/in-progress/e2e This label acts like a guard and prevents Mergify from adding the `ok-to-test` label again. label Apr 15, 2026
@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 15, 2026

/retest ci/centos/mini-e2e-helm/k8s-1.33

@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 15, 2026

/retest ci/centos/mini-e2e-helm/k8s-1.33

Failed to get kubernetes patch release logs.

@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 15, 2026

CI jobs fail with something that seems unrelated?

[FAIL] RBD Test RBD CSI [It] create a PVC and bind it to an app with encrypted RBD volume with VaultKMS (block)

and

  I0415 14:12:37.125856 77252 pvc.go:85] PVC rbd-pvc Event: ProvisioningFailed - Volume provisioning failed with infeasible error. Retries will be delayed. rpc error: code = InvalidArgument desc = invalid encryption kms configuration: failed connecting to Vault: failed to get the authentication token: unable to log in with Kubernetes auth: Error making API request.

  URL: PUT http://vault.cephcsi-e2e-46ef0914.svc.cluster.local:8200/v1/auth/kubernetes/login
  Code: 400. Errors:

  * missing client token

Running a test in an other PR with #6214 (comment)

@nixpanic nixpanic mentioned this pull request Apr 15, 2026
Open
@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 16, 2026

started an other test in #6229 (comment) to verify the RBD with VaultKMS test.

@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 16, 2026

/test ci/centos/mini-e2e/k8s-1.34/rbd

@nixpanic
Copy link
Copy Markdown
Member Author

nixpanic commented Apr 16, 2026 •
edited
Loading

/test ci/centos/mini-e2e/k8s-1.34/rbd

And running the same test on this PR: logs

On #6229: logs

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 22, 2026 •
edited
Loading

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🔴 ⛓️ Depends-On Requirements

Waiting for

This rule is failing.

Requirement based on the presence of Depends-On in the body of the pull request

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 22, 2026

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.

@github-actions github-actions Bot added stale and removed stale labels May 22, 2026
@gadididi gadididi added the keepalive This label can be used to disable stale bot activiity in the repo label May 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Rakshith-R Rakshith-R Rakshith-R approved these changes

@gadididi gadididi gadididi approved these changes

Assignees

No one assigned

Labels

ci/skip/multi-arch-build skip building on multiple architectures component/nvme-of Issues and PRs related to NVMe-oF. component/testing Additional test cases or CI work keepalive This label can be used to disable stale bot activiity in the repo

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

nvmeof: remove gateway deployment in e2e suite

4 participants

@nixpanic @ceph-csi-bot @Rakshith-R @gadididi