-
Notifications
You must be signed in to change notification settings - Fork 254
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: Make the hack/e2e
folder architecture agnostic
#1487
Comments
To clarify and continue the discussion, I would not expect For CI we need to have one specific configuration that works. For developers we need flexibility more than anything. In my opinion we should not expect developers to run scripts that install htpasswd or any other things system wide at all. Instead we should document the dependencies so it is easy to set up a suitable environment in whatever way they like. I would expect people to want to run the tests on mac, windows, all kinds of linux distros, etc. We cannot cover everything so the most important part is to make it flexible. |
Very much agreed with @lentzi90 . It would go a long way if you had some sort of verify script, that'd tell you (but not try install) if you're missing something. Then it is up to developer to install said dependencies. If its not too complex, we may provide support for some OS as mentioned, since we need to do it for CI as well. We need to be extra careful in naming the scripts so people don't end up destroying their OS by running CI scripts that install or remove software and configs at whim. We might also want to reconsider placing the CI scripts elsewhere than in |
/triage accepted |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with /lifecycle stale |
/remove-lifecycle stale |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with /lifecycle stale |
/remove-lifecycle stale |
/remove-lifecycle stale |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with /lifecycle stale |
/remove-lifecycle stale |
Currently the scripts under the
hack/e2e
folder would work locally only forlinux-amd64
architecture. As andarwin-arm64
user, I would expect the script works locally for me as well atleast. We can add other architectures as well but atleast these two can be supported minimally.The text was updated successfully, but these errors were encountered: