-
Notifications
You must be signed in to change notification settings - Fork 590
merge queue: embarking devel (75cb245) and #5714 together #5768
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
init that file. in the future add more nvmeof errors to this file. Signed-off-by: gadi-didi <[email protected]>
make getNVMeoFMetadata() returns nvmeof error codes. Signed-off-by: gadi-didi <[email protected]>
Qos for nvmeof namespace is added. allow the user limit the namesapce capabilities. Signed-off-by: gadi-didi <[email protected]>
added because want to reuse the private function parseQosParams() Signed-off-by: gadi-didi <[email protected]>
if qos params are provided, call to qos grpc after ns was created. Signed-off-by: gadi-didi <[email protected]>
the purpose of that function is to modify the qos for namesapce on the fly. Signed-off-by: gadi-didi <[email protected]>
modify_volume capability is added to nvmeof driver in order to call ControllerModifyVolume(). Signed-off-by: gadi-didi <[email protected]>
Add EXPAND_VOLUME capability and stub implementation to allow csi-resizer to start and handle VolumeAttributesClass modifications. Signed-off-by: gadi-didi <[email protected]>
add ControllerModifyVolume to request ID extraction for proper log correlation. Signed-off-by: gadi-didi <[email protected]>
add unit tests with multiple cases. Signed-off-by: gadi-didi <[email protected]>
|
/test ci/centos/k8s-e2e-external-storage/1.33 |
|
/test ci/centos/k8s-e2e-external-storage/1.34 |
|
/test ci/centos/mini-e2e-helm/k8s-1.33 |
|
/test ci/centos/mini-e2e-helm/k8s-1.34 |
|
/test ci/centos/upgrade-tests-cephfs |
|
/test ci/centos/k8s-e2e-external-storage/1.32 |
|
/test ci/centos/mini-e2e/k8s-1.33 |
|
/test ci/centos/mini-e2e/k8s-1.34 |
|
/test ci/centos/upgrade-tests-rbd |
|
/test ci/centos/mini-e2e-helm/k8s-1.32 |
|
/test ci/centos/mini-e2e/k8s-1.32 |
🎉 This pull request has been checked successfully and will be merged soon. 🎉
Branch devel (75cb245) and #5714 are embarked together for merge.
This pull request has been created by Mergify to speculatively check the mergeability of #5714.
You don't need to do anything. Mergify will close this pull request automatically when it is complete.
Required conditions of queue
defaultfor merge:#approved-reviews-by >= 2[🛡 GitHub branch protection]#changes-requested-reviews-by = 0[🛡 GitHub branch protection]base=develstatus-success=codespellstatus-success=go-teststatus-success=golangci-lintstatus-success=lint-extrasstatus-success=mod-checkstatus-success=multi-arch-buildstatus-success=uncommitted-code-checkstatus-success=ci/centos/k8s-e2e-external-storage/1.32status-success=ci/centos/k8s-e2e-external-storage/1.33status-success=ci/centos/k8s-e2e-external-storage/1.34status-success=ci/centos/mini-e2e-helm/k8s-1.32status-success=ci/centos/mini-e2e-helm/k8s-1.33status-success=ci/centos/mini-e2e-helm/k8s-1.34status-success=ci/centos/mini-e2e/k8s-1.32status-success=ci/centos/mini-e2e/k8s-1.33status-success=ci/centos/mini-e2e/k8s-1.34status-success=ci/centos/upgrade-tests-cephfsstatus-success=ci/centos/upgrade-tests-rbdlabel=ci/skip/e2ebase~=^(release-.+)$label=ci/skip/e2estatus-success=ci/centos/k8s-e2e-external-storage/1.31status-success=ci/centos/mini-e2e-helm/k8s-1.31status-success=ci/centos/mini-e2e/k8s-1.31status-success=ci/centos/k8s-e2e-external-storage/1.32status-success=ci/centos/k8s-e2e-external-storage/1.33status-success=ci/centos/mini-e2e-helm/k8s-1.32status-success=ci/centos/mini-e2e-helm/k8s-1.33status-success=ci/centos/mini-e2e/k8s-1.32status-success=ci/centos/mini-e2e/k8s-1.33status-success=ci/centos/upgrade-tests-cephfsstatus-success=ci/centos/upgrade-tests-rbdstatus-success=codespellstatus-success=go-teststatus-success=golangci-lintstatus-success=lint-extrasstatus-success=mod-checkstatus-success=multi-arch-buildstatus-success=uncommitted-code-checkbase=release-v3.14label=ci/skip/e2estatus-success=ci/centos/k8s-e2e-external-storage/1.30status-success=ci/centos/k8s-e2e-external-storage/1.31status-success=ci/centos/mini-e2e-helm/k8s-1.30status-success=ci/centos/mini-e2e-helm/k8s-1.31status-success=ci/centos/mini-e2e/k8s-1.30status-success=ci/centos/mini-e2e/k8s-1.31status-success=ci/centos/k8s-e2e-external-storage/1.32status-success=ci/centos/mini-e2e-helm/k8s-1.32status-success=ci/centos/mini-e2e/k8s-1.32status-success=ci/centos/upgrade-tests-cephfsstatus-success=ci/centos/upgrade-tests-rbdstatus-success=codespellstatus-success=go-teststatus-success=golangci-lintstatus-success=lint-extrasstatus-success=mod-checkstatus-success=multi-arch-buildstatus-success=uncommitted-code-checkbase=ci/centosstatus-success=ci/centos/jjb-validatestatus-success=ci/centos/job-validationRequired conditions to stay in the queue:
#approved-reviews-by >= 2[🛡 GitHub branch protection]#changes-requested-reviews-by = 0[🛡 GitHub branch protection]