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

[mdatagen] Fix gauge.NumberDataPoints field to use mapstructure #6683

Merged

Conversation

dmitryax
Copy link
Member

The field was left not updated due to this PR #6460 not rebased before merge.

The field was left not updated due to this PR open-telemetry#6460 not rebased before merge
@bogdandrutu bogdandrutu merged commit 450d5f8 into open-telemetry:main Dec 10, 2021
@dmitryax dmitryax deleted the mdatagen-fix-gauge-mubber-type branch December 10, 2021 18:37
PaurushGarg referenced this pull request in open-o11y/opentelemetry-collector-contrib Dec 11, 2021
The field was left not updated due to this PR open-telemetry#6460 not rebased before merge
PaurushGarg referenced this pull request in open-o11y/opentelemetry-collector-contrib Dec 11, 2021
The field was left not updated due to this PR open-telemetry#6460 not rebased before merge
PaurushGarg referenced this pull request in open-o11y/opentelemetry-collector-contrib Dec 14, 2021
The field was left not updated due to this PR open-telemetry#6460 not rebased before merge
animetauren pushed a commit to animetauren/opentelemetry-collector-contrib that referenced this pull request Apr 4, 2023
* Feat: add httpsprovider component

Add the httpsprovider component that allows the collector to fetch
configuration from a web server using the https protocol.

Signed-off-by: Raphael Silva <[email protected]>

* Add httpsprovider to the list of config providers

Signed-off-by: Raphael Silva <[email protected]>

* Update README

Signed-off-by: Raphael Silva <[email protected]>

* Apply suggestions from code review

Co-authored-by: Pablo Baeyens <[email protected]>

* Improve unit tests

Improve unit tests to check for error while parsing test URL.

Co-authored-by: Pablo Baeyens <[email protected]>

* Fix comments

Co-authored-by: Anthony Mirabella <[email protected]>

* Fix comments

Co-authored-by: Anthony Mirabella <[email protected]>

* Use SchemeType instead of TransportType

Signed-off-by: Raphael Silva <[email protected]>

* Refactor to improve readability

Signed-off-by: Raphael Silva <[email protected]>

* Use consts from http to set http status code

Signed-off-by: Raphael Silva <[email protected]>

* Refactor code organization

Signed-off-by: Raphael Silva <[email protected]>

* Use struct with zero initialized properties

Signed-off-by: Raphael Silva <[email protected]>

Signed-off-by: Raphael Silva <[email protected]>
Co-authored-by: Pablo Baeyens <[email protected]>
Co-authored-by: Anthony Mirabella <[email protected]>
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.

2 participants