Skip to content
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

e2e: add e2e testcase of new image features #2696

Merged
merged 1 commit into from
Dec 9, 2021
Merged

Conversation

Madhu-1
Copy link
Collaborator

@Madhu-1 Madhu-1 commented Dec 7, 2021

adding e2e test case to validate the workflow of pvc creation and attaching to pod works for new image features like fast-diff,obj-map,exclusive-lock, and layering.

fixes: #2695

Signed-off-by: Madhu Rajanna madhupr007@gmail.com

@Madhu-1 Madhu-1 requested review from pkalever and a team December 7, 2021 08:09
@mergify mergify bot added the component/testing Additional test cases or CI work label Dec 7, 2021
ExtraVersion: 0,
Distribution: "",
Backport: false,
}, // standard 5.3+ versions
Copy link

Choose a reason for hiding this comment

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

Do we know backport Centos kernel version?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

no idea about this one.

Copy link
Member

Choose a reason for hiding this comment

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

This runs inside minikube for the CI in this repo, the CentOS kernel is not very relevant (unless someone runs it on an other environment).

@Madhu-1 Madhu-1 force-pushed the fix-2695 branch 2 times, most recently from 24b1565 to 0a92d14 Compare December 7, 2021 10:04
@Madhu-1 Madhu-1 requested a review from pkalever December 7, 2021 10:04
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Dec 7, 2021

/retest ci/centos/k8s-e2e-external-storage/1.21

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Dec 7, 2021

/retest ci/centos/k8s-e2e-external-storage/1.22

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Dec 7, 2021

X Exiting due to GUEST_PROVISION: Failed to start host: creating host: create: Error creating machine: Error in driver during machine creation: IP not available after waiting: machine minikube didn't return IP after 1 minute

E2E failure

@Madhu-1 Madhu-1 added the ci/retry/e2e Label to retry e2e retesting on approved PR's label Dec 7, 2021
@Madhu-1 Madhu-1 requested a review from a team December 8, 2021 13:25
ExtraVersion: 0,
Distribution: "",
Backport: false,
}, // standard 5.3+ versions
Copy link
Member

Choose a reason for hiding this comment

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

This runs inside minikube for the CI in this repo, the CentOS kernel is not very relevant (unless someone runs it on an other environment).

Copy link

@pkalever pkalever left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks @Madhu-1

adding e2e testcase to validate the workflow
of pvc creation and attaching to pod works for
new image features like fast-diff,obj-map,exclusive-lock
and layering.

fixes: ceph#2695

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Dec 9, 2021

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

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Dec 9, 2021

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

@Madhu-1
Copy link
Collaborator Author

Madhu-1 commented Dec 9, 2021

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

@github-actions
Copy link

github-actions bot commented Dec 9, 2021

/retest ci/centos/k8s-e2e-external-storage/1.23

@github-actions
Copy link

github-actions bot commented Dec 9, 2021

@Madhu-1 "ci/centos/k8s-e2e-external-storage/1.23" test failed. Logs are available at location for debugging

@Madhu-1 Madhu-1 removed the ci/retry/e2e Label to retry e2e retesting on approved PR's label Dec 9, 2021
@mergify mergify bot merged commit 4ebfeaa into ceph:devel Dec 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/testing Additional test cases or CI work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add e2e for new image features
3 participants