snuyanzin commented on code in PR #29770:
URL: https://github.com/apache/flink/pull/29770#discussion_r4460251536
##########
flink-end-to-end-tests/test-scripts/common_ssl.sh:
##########
@@ -79,11 +79,6 @@ function _set_conf_ssl_helper {
if [ "${provider}" = "OPENSSL" -a "${provider_lib}" = "dynamic" ]; then
cp $FLINK_DIR/opt/flink-shaded-netty-tcnative-dynamic-*.jar
$FLINK_DIR/lib/
- # if the CI prepared a newer OpenSSL than the system one (see
e2e-template.yml), prefer it;
- # local/developer runs without that variable set keep using the system
OpenSSL as before
- if [ -n "${FLINK_E2E_OPENSSL32_LIB:-}" ]; then
- export
LD_LIBRARY_PATH="${FLINK_E2E_OPENSSL32_LIB}${LD_LIBRARY_PATH:+:${LD_LIBRARY_PATH}}"
Review Comment:
no need for this since ubuntu 26 contains required open ssl
##########
tools/azure-pipelines/e2e-template.yml:
##########
@@ -147,21 +147,6 @@ jobs:
echo "Installing required software for netty-tcnative"
sudo apt-get install -y bc libapr1
-
- # flink-shaded-netty-tcnative-dynamic links against OpenSSL symbols
newer than what this
Review Comment:
no need for this since ubuntu 26 contains required open ssl
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
