update nginx base image, handle jaeger propagation format

[timmysilv]

Rel #6936

Updating the nginx base image to create a new one in the staging env.

[k8s-ci-robot]

Hi @timmysilv. Thanks for your PR.

I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[timmysilv]

the downgrade of Alpine to 3.10 was needed to avoid a segfault in the new jaeger-cpp-client, but modsecurity seems to not be backwards compatible. I formerly thought the Jaeger issue was compatibility, but I now think something's actually wrong with some of the new yaml parsing. I'm going to look deeper into the function in the Jaeger client then come back here.

[timmysilv]

^ shows that e2e passes for #6936 to be approved. Manuel's comment suggests that this means the other PR can be merged, then a new image will be auto-released. I'll update the Makefile in this PR to use that resulting image. The jaeger code changes are to support the new Jaeger client configuration features available with the new base image.

[ElvinEfendi]

/ok-to-test
/hold

[ElvinEfendi]

The PR is doing more than just updating the base image, is that intentional?

[timmysilv]

yes, I alluded to that in the above comment. The new base image updates the jaeger client which handles the new propagation parameter. It can be separated into a different PR if you prefer, but the jaegerTmpl variable should probably be updated with this image update anyway.

[ElvinEfendi]

/lgtm

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ElvinEfendi, timmysilv

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [ElvinEfendi]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[ElvinEfendi]

/hold cancel