[opentelemetry-cpp] Update to 1.13.0

[waywardmonkeys]

The patch to add a dependency of opentelemetry-proto on gRPC should not be needed after the upstream changes in:

open-telemetry/opentelemetry-cpp#2268

Additionally, the "zpages" feature is removed as it is no longer present upstream after having been deprecated in a previous release.

Fixes #35992.

• Changes comply with the maintainer guide
• SHA512s are updated for each updated download
• The "supports" clause reflects platforms that may be fixed by this new version
• Any fixed CI baseline entries are removed from that file.
• Any patches that are no longer applied are deleted from the port's directory.
• The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
• Only one version is added to each modified port's versions file.

Might be best to have this reviewed by someone that uses this more or who knows more about the upstream.

[jimwang118]

I used the following command locally: .\vcpkg.exe install opentelemetry-cpp[*] when testing the feature installation, the following error occurred.

F:\opentelemetry-cpp\buildtrees\opentelemetry-cpp\src\v1.13.0-18976a2470.clean\exporters\otlp\include\opentelemetry/exporters/otlp/otlp_grpc_exporter.h(61): error C2589: '(': illegal token on right side of '::'
F:\opentelemetry-cpp\buildtrees\opentelemetry-cpp\src\v1.13.0-18976a2470.clean\exporters\otlp\include\opentelemetry/exporters/otlp/otlp_grpc_exporter.h(61): error C2059: syntax error: ')'
F:\opentelemetry-cpp\buildtrees\opentelemetry-cpp\src\v1.13.0-18976a2470.clean\exporters\otlp\include\opentelemetry/exporters/otlp/otlp_grpc_exporter.h(70): warning C4003: not enough arguments for function-like macro invocation 'max'
F:\opentelemetry-cpp\buildtrees\opentelemetry-cpp\src\v1.13.0-18976a2470.clean\exporters\otlp\include\opentelemetry/exporters/otlp/otlp_grpc_exporter.h(70): error C2589: '(': illegal token on right side of '::'
F:\opentelemetry-cpp\buildtrees\opentelemetry-cpp\src\v1.13.0-18976a2470.clean\exporters\otlp\include\opentelemetry/exporters/otlp/otlp_grpc_exporter.h(70): error C2059: syntax error: ')'
F:\opentelemetry-cpp\buildtrees\opentelemetry-cpp\src\v1.13.0-18976a2470.clean\exporters\otlp\include\opentelemetry/exporters/otlp/otlp_grpc_exporter.h(61): error C2062: type 'unknown-type' unexpected
F:\opentelemetry-cpp\buildtrees\opentelemetry-cpp\src\v1.13.0-18976a2470.clean\exporters\otlp\include\opentelemetry/exporters/otlp/otlp_grpc_exporter.h(61): error C2059: syntax error: ')'
F:\opentelemetry-cpp\buildtrees\opentelemetry-cpp\src\v1.13.0-18976a2470.clean\exporters\otlp\include\opentelemetry/exporters/otlp/otlp_grpc_exporter.h(70): error C2059: syntax error: 'default argument'
F:\opentelemetry-cpp\buildtrees\opentelemetry-cpp\src\v1.13.0-18976a2470.clean\exporters\otlp\include\opentelemetry/exporters/otlp/otlp_grpc_exporter.h(70): error C2059: syntax error: ')'
F:\opentelemetry-cpp\buildtrees\opentelemetry-cpp\src\v1.13.0-18976a2470.clean\exporters\otlp\include\opentelemetry/exporters/otlp/otlp_grpc_exporter.h(89): error C2059: syntax error: 'member initializer'
F:\opentelemetry-cpp\buildtrees\opentelemetry-cpp\src\v1.13.0-18976a2470.clean\exporters\otlp\include\opentelemetry/exporters/otlp/otlp_grpc_exporter.h(92): error C2059: syntax error: 

install-x64-windows-dbg-out.log

[jimwang118]

Note: I will be converting your PR to draft status. The above suggested changes are only recommendations. If you are willing to adopt them, please click "ready for review" after making the modifications. If you do not wish to make any changes, please click "ready for review" directly. That way, I can be aware that you've responded since you can't modify the tags.

[waywardmonkeys]

@jimwang118 This error was because of a change in opentelemetry-cpp 1.13 that stopped defining NOMINMAX on Windows and the fixes to Open Telemetry included in 1.13 were insufficient. There was a PR submitted and I've included that patch here which should fix the problem that you identified.

[waywardmonkeys]

Updated again to include open-telemetry/opentelemetry-cpp#2449 to fix one more NOMINMAX problem.

[jimwang118]

All features passed with following triplets:

x86-windows
x64-windows

[BillyONeal]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[BillyONeal]

The android build lab appears to be broken and I only made whitespace changes so I'll merge this if all other platforms come back OK