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

Fixing E2E Tests #140

Merged
merged 3 commits into from
Dec 14, 2023
Merged

Fixing E2E Tests #140

merged 3 commits into from
Dec 14, 2023

Conversation

luthermonson
Copy link
Contributor

@luthermonson luthermonson commented Dec 6, 2023

Core changes to get these running again and we should pass linting again. To get these running you can boot an LKE cluster and edit the image in the stateful/daemon set for the csi-driver to be an image you build/push to your own image repo. More updates coming to automate that a bit more.

  • changed default interval to 1s to make tests run faster, still more to do but this helped cut off 2-3min
  • refactored pods/pvcs to have an random int in their name, made them easier to track in kube while watching tests and hopefully we can run them all in parallel some day soon
  • found dupe tests running two cleanups
  • hardened the funcs around delete/detach to return errors if 404s/notfound/etc
  • refactored raw block tests to be a part of the Pod + PVC test
  • refactored mkfs.ext4 func to be like the other ExecInPods
  • updated all tests to run in ubuntu for raw block to have mkfs.ext4 and consolidated into one GetPodObject func

e2e/test/csi_driver_test.go Outdated Show resolved Hide resolved
e2e/test/csi_driver_test.go Outdated Show resolved Hide resolved
e2e/test/csi_driver_test.go Outdated Show resolved Hide resolved
e2e/test/csi_driver_test.go Outdated Show resolved Hide resolved
e2e/test/framework/pod.go Outdated Show resolved Hide resolved
e2e/test/framework/pod.go Outdated Show resolved Hide resolved
e2e/test/e2e_suite_test.go Outdated Show resolved Hide resolved
e2e/test/framework/pod.go Outdated Show resolved Hide resolved
e2e/test/framework/pod.go Outdated Show resolved Hide resolved
@luthermonson luthermonson changed the title [WIP] Fixing E2E Tests Fixing E2E Tests Dec 11, 2023
@lgarber-akamai
Copy link
Contributor

@luthermonson Given the description mentions running this against an LKE cluster, do you plan on removing the dependency on terraform-linode-k8s?

@luthermonson
Copy link
Contributor Author

@luthermonson Given the description mentions running this against an LKE cluster, do you plan on removing the dependency on terraform-linode-k8s?

ya when we move to booting an LKE cluster in the next pr this will probably get pulled

@luthermonson luthermonson merged commit f28e59c into linode:main Dec 14, 2023
@luthermonson luthermonson deleted the fix-e2e branch December 14, 2023 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants