-
Notifications
You must be signed in to change notification settings - Fork 66
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
Windows: Install OpenSSL standalone #107
Windows: Install OpenSSL standalone #107
Conversation
Signed-off-by: Sunjay Bhatia <sunjayb@vmware.com>
cc @lizan when you get a chance, this will allow us to untag all the failing/flaky |
@sunjayBhatia why do we need openssl in those tests? |
There are a few scripts in these tests that generate test data on the fly, see this tree: https://github.com/envoyproxy/envoy/tree/master/test/extensions/transport_sockets/tls specifically invocations show up here: https://github.com/envoyproxy/envoy/blob/db1a35213aa18399ba14ef545fc4894545cf68a2/test/extensions/transport_sockets/tls/ssl_certs_test.h |
@lizan do you prefer to have all the certs checked in? (imo its sorta nice to have things generated on the fly, makes sure we don't have to think about expiry dates and it keeps the tree cleaner, but its not a strong preference) |
@sunjayBhatia yeah I think we had issue before with macOS so we checked in most of the certs. Or we can use the |
From what I’ve looked at |
@lizan would you prefer a change to check in the certs rather than this one? it has an added benefit of ensuring the tests are runnable on macos if they arent already |
@envoyproxy/build-tools-maintainers PTAL, the alternative to this change to the build image is to check in all generated certs in the cc @envoyproxy/dependency-shepherds |
/retest |
OK if bssl doesn't work I don't have strong preference. In general I'd like to avoid openssl though. |
See envoyproxy/envoy#13702 as an alternative to updating OpenSSL @lizan |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Definitely @lizan 's area so will defer to him
superceded by envoyproxy/envoy#13702 |
Solves msys2 openssl fork/DLL base address issues in RBE
See: envoyproxy/envoy#13276
Downloads OpenSSL installer from here: https://slproweb.com/products/Win32OpenSSL.html
Should be same installer used by chocolatey (https://chocolatey.org/packages/openssl#files) and mentioned on OpenSSL site: https://wiki.openssl.org/index.php/Binaries