OTLP Examples fixed to be consistent

[cijothomas]

Fix the example to use same Resource for all signals.
The http one is now matching the grpc one, though I plan to send a follow up later to maintain single example with feature flag to control transport, as opposed to copying a lot.

Also, the http one is not working for logs, metrics. (this was same before this PR also, so not a blocker for this PR.)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.6%. Comparing base (348ec9e) to head (eaec230).
Report is 27 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##            main   #1752     +/-   ##
=======================================
+ Coverage   71.0%   71.6%   +0.5%     
=======================================
  Files        135     136      +1     
  Lines      20751   20845     +94     
=======================================
+ Hits       14746   14931    +185     
+ Misses      6005    5914     -91     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[cijothomas]

Also, the http one is not working for logs, metrics. (this was same before this PR also, so not a blocker for this PR.)

#1706 might have broken the endpoint.

[TommyCpp]

Also, the http one is not working for logs, metrics. (this was same before this PR also, so not a blocker for this PR.)

#1706 might have broken the endpoint.

The metrics endpoint should work as after #1706 we won't change anything from user input in builders. For logs, we need to manually append the /v1/logs path to the base URL

[cijothomas]

Also, the http one is not working for logs, metrics. (this was same before this PR also, so not a blocker for this PR.)

#1706 might have broken the endpoint.

The metrics endpoint should work as after #1706 we won't change anything from user input in builders. For logs, we need to manually append the /v1/logs path to the base URL

Its still not working. :( Could you take a look ?

"Error: Other("[Other(reqwest::Error { kind: Status(400), url: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("localhost")), port: Some(4318), path: "/v1/logs", query: None, fragment: None } })]")"

[lalitb]

LGTM. Assuming the issue of OTLP HTTP Log Exporter would be handled separately.

[TommyCpp]

Also, the http one is not working for logs, metrics. (this was same before this PR also, so not a blocker for this PR.)

#1706 might have broken the endpoint.

The metrics endpoint should work as after #1706 we won't change anything from user input in builders. For logs, we need to manually append the /v1/logs path to the base URL

Its still not working. :( Could you take a look ?

"Error: Other("[Other(reqwest::Error { kind: Status(400), url: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("localhost")), port: Some(4318), path: "/v1/logs", query: None, fragment: None } })]")"

Yeah taking a look now

[lalitb]

"Error: Other("[Other(reqwest::Error { kind: Status(400), url: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("localhost")), port: Some(4318), path: "/v1/logs", query: None, fragment: None } })]")"

Bad Request, Could also be related to PR for sending Resource separately, and then combining before export. Let me check too :(

[TommyCpp]

Alright I think there are some issues with OTLP http/json exporter. After set the http binary protocol everything works fine
cijothomas#1