Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update New Relic exporter error conditions. #3322

Merged

Conversation

a-feld
Copy link
Member

@a-feld a-feld commented May 4, 2021

Description: This PR updates the New Relic exporter "error" conditions in a couple of key ways:

  • Cumulative Sum metrics are now exported as gauges
  • The HTTP status error message now contains the status code.

Testing: Yes

Fixes: #3315

@a-feld a-feld changed the title Update newrelic errors Update New Relic exporter error conditions. May 4, 2021
@a-feld a-feld marked this pull request as ready for review May 4, 2021 19:44
@a-feld a-feld requested a review from a team as a code owner May 4, 2021 19:44
@a-feld
Copy link
Member Author

a-feld commented May 4, 2021

Fixes: #3315

@tigrannajaryan
Copy link
Member

@alanwest @jack-berg @nrcventura please review.

@a-feld
Copy link
Member Author

a-feld commented May 4, 2021

@tigrannajaryan we're all set now 👍

@bogdandrutu bogdandrutu merged commit b2d3a11 into open-telemetry:main May 4, 2021
@a-feld a-feld deleted the update-newrelic-errors branch May 5, 2021 02:06
thepeterstone pushed a commit to thepeterstone/opentelemetry-collector-contrib that referenced this pull request May 10, 2021
* Update New Relic error text to include http response code.

* Export cumulative metrics as gauges.
alexperez52 referenced this pull request in open-o11y/opentelemetry-collector-contrib Aug 18, 2021
* Remove the proto dependency in goldendataset for traces

* revert changelog

* update changelog

* Update consumer/pdata/common.go

Co-authored-by: Anthony Mirabella <a9@aneurysm9.com>

* Fix common.go where there is a missing "}"

* Add functions to convert proto keyvalues to AttributeMap

* further remove proto dependency

* Add PR number to CHANGELOG

* update resource_generator_test and resource_to_oc_test

* Return nil instead of empty objects for some generator functions.

Co-authored-by: Anthony Mirabella <a9@aneurysm9.com>
Co-authored-by: Bogdan Drutu <bogdandrutu@gmail.com>
mstumpfx pushed a commit to mstumpfx/opentelemetry-collector-contrib that referenced this pull request Aug 31, 2021
* Update New Relic error text to include http response code.

* Export cumulative metrics as gauges.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

batchprocessor/batch_processor.go:209 Sender failed {"kind": "processor", "name": "batch",
4 participants