Skip to content

Commit

Permalink
Remove nginx-tests patch now that upstream supports AWS-LC (#1898)
Browse files Browse the repository at this point in the history
### Description of changes: 
All of these changes have been applied to upstream nginx-tests in
nginx/nginx-tests@52bf350
and is no longer necessary. The CodeBuild limitation remains so
`nginx_patch_tests` is still called to delete the incompatible test.

This will get the integration tests back to a passing state. 

### Testing:
In this CI run.

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license and the ISC license.
  • Loading branch information
andrewhop authored Oct 2, 2024
1 parent ad93747 commit 751fe2a
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 124 deletions.
120 changes: 0 additions & 120 deletions tests/ci/integration/nginx_tests_patch/aws-lc-nginx-tests.patch

This file was deleted.

4 changes: 0 additions & 4 deletions tests/ci/integration/run_nginx_integration.sh
Original file line number Diff line number Diff line change
Expand Up @@ -64,10 +64,6 @@ function nginx_patch_build() {
# * SSL_Conf Command
# * Stateful session resumption (Session Caches)
function nginx_patch_tests() {
for patchfile in $(find -L "${NGINX_PATCH_TEST_FOLDER}" -type f -name '*.patch'); do
echo "Apply patch $patchfile..."
patch -p1 --quiet -i "$patchfile"
done
# http_listen.t tries to open port 8182, but this port isn't available within the
# docker container from CI configurations. This isn't related to ssl functionality, so
# we skip/remove it.
Expand Down

0 comments on commit 751fe2a

Please sign in to comment.