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

add error checks to googlecloudexporter package. #10373

Merged
merged 7 commits into from
Jul 3, 2022

Conversation

DiptoChakrabarty
Copy link
Contributor

Description:

adding error checks to googlecloudexporter package.
Link to tracking Issue:
#9749
Testing:

Documentation:

@github-actions
Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Jun 28, 2022
@mx-psi mx-psi removed the Stale label Jun 28, 2022
@mx-psi
Copy link
Member

mx-psi commented Jun 28, 2022

@DiptoChakrabarty could you look at @dashpole's comment and fix this?

@mx-psi
Copy link
Member

mx-psi commented Jul 1, 2022

The changelog check is failing because the changelog process changed. I don't think this PR needs a changelog entry anyway, so could you just revert your changes to the CHANGELOG.md file?

@mx-psi mx-psi added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Jul 3, 2022
@mx-psi mx-psi merged commit 508b172 into open-telemetry:main Jul 3, 2022
atoulme pushed a commit to atoulme/opentelemetry-collector-contrib that referenced this pull request Jul 16, 2022
* error checks for googlecloud exporter package

* update changelog

* stop srv with defer

* remove changelog entry

Co-authored-by: Alex Boten <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants