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 a submariner validation test to the e2e Validation suite #1696

Open
Tracked by #1717
raghavendra-talur opened this issue Dec 5, 2024 · 2 comments
Open
Tracked by #1717
Labels
test Testing related issue

Comments

@raghavendra-talur
Copy link
Member

It could be useful to have a simple submariner validation test in the validation suite. Sometimes the submariner connection is broken and exiting early on detecting that would be better than running the tests and failing at a later point.

Note: validation suite is supposed to be a readonly suite and therefore the test shouldn't create any namespace or pods to verify the connection. We can only check for the details in the hub and the managed clusters to verify that submariner is setup correctly.

@raghavendra-talur
Copy link
Member Author

@nirs @ShyamsundarR @BenamarMk Do you think this is a valid issue?

@nirs nirs added the test Testing related issue label Dec 5, 2024
@nirs nirs changed the title Add a submariner validation test to the e2e Validation suite e2e: Add a submariner validation test to the e2e Validation suite Dec 5, 2024
@nirs
Copy link
Member

nirs commented Dec 5, 2024

It could be useful to have a simple submariner validation test in the validation suite. Sometimes the submariner connection is broken and exiting early on detecting that would be better than running the tests and failing at a later point.

It can be useful for ocp clusters, with drenv cluster we already test the cluster during creation, so we know that submariner is working, and we know that we can replicate pvcs with volsync via submariner.

But if we will have #1702 this check can be there. Lets wait with this until we make more progress on the tool.

Note: validation suite is supposed to be a readonly suite and therefore the test shouldn't create any namespace or pods to verify the connection. We can only check for the details in the hub and the managed clusters to verify that submariner is setup correctly.

There is no way to test without creating a service export and import and using them to access one cluster from the other - as we do in the submariner self test:
https://github.com/RamenDR/ramen/blob/main/test/addons/submariner/test

I think there is no issue to create a temporary namespace like e2e-validation-xyz and deleting it after the test. I thinks that subctl is doing stuff like this.

@nirs nirs mentioned this issue Dec 11, 2024
19 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Testing related issue
Projects
None yet
Development

No branches or pull requests

2 participants