Skip to content

Commit 5634918

Browse files
authored
Revert the revert: bring back integration test foundation (#130)
*Issue #, if available:* *Description of changes:* **Background** The integration test framework was previously reverted due to Docker build failures. This PR restores the functionality with a fix for the underlying issue. The original integration tests used reqwest = "0.12" with default features, which includes native-tls that depends on system OpenSSL libraries on Linux for the time being. This causes build failures in environments without openssl development tooling installed: ``` Could not find directory of OpenSSL installation, and this `-sys` crate cannot proceed without this knowledge. If OpenSSL is installed and this crate had trouble finding it, you can set the `OPENSSL_DIR` environment variable for the compilation process. ``` Switched `reqwest` from `native-tls` to `rustls-tls`, which uses AWS-LC under the hood. ``` # Before reqwest = "0.12" # After reqwest = { version = "0.12", default-features = false, features = ["rustls-tls"] } ``` **Additional Changes** * Added CDLA-Permissive-2.0 for webpki-roots dependency * Added PR triggers to Docker workflow to catch build issues before merge Testing * Local builds (macOS, Linux) * Docker builds (linux/amd64, linux/arm64) * Integration tests functionality preserved * License compliance checks pass By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
1 parent db24913 commit 5634918

File tree

9 files changed

+873
-6
lines changed

9 files changed

+873
-6
lines changed

.github/workflows/docker.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,9 @@ name: "Docker Image CI"
22

33
on:
44
push:
5-
branches: [main]
5+
branches: ["main"]
6+
pull_request:
7+
branches: ["main"]
68

79
jobs:
810
build:

0 commit comments

Comments
 (0)