Closed awels closed 1 month ago
/lgtm
So this is going to have issues with the latest nginx version and https imports:
$ k logs -n cdi cdi-file-host-8449b596c9-4pq6g | grep dumped
selecting docker as container runtime
Defaulted container "http" out of: http, s3, init (init)
2024/08/21 16:28:14 [alert] 1#1: worker process 2 exited on signal 11 (core dumped)
2024/08/21 16:28:14 [alert] 1#1: worker process 3 exited on signal 11 (core dumped)
2024/08/21 16:28:15 [alert] 1#1: worker process 4 exited on signal 11 (core dumped)
2024/08/21 16:28:15 [alert] 1#1: worker process 5 exited on signal 11 (core dumped)
2024/08/21 16:28:28 [alert] 1#1: worker process 6 exited on signal 11 (core dumped)
2024/08/21 16:28:28 [alert] 1#1: worker process 7 exited on signal 11 (core dumped)
2024/08/21 16:28:59 [alert] 1#1: worker process 8 exited on signal 11 (core dumped)
2024/08/21 16:28:59 [alert] 1#1: worker process 9 exited on signal 11 (core dumped)
Good news is that I have a commit freezing nginx to a "good" version, if you can give me rights to your fork I'll push it.
/test pull-cdi-unit-test
/test pull-cdi-unit-test
/lgtm
@awels: you cannot LGTM your own PR.
/approve /lgtm
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: akalenyu
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/test pull-containerized-data-importer-e2e-istio
What this PR does / why we need it: Ensure the dependencies are up to date so we can easily merge with s390x PRs in the future.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged): Fixes #Special notes for your reviewer:
Release note: