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 url decode for OTEL_RESOUCE_ATTRIBUTES #3046

Merged

Conversation

shalevr
Copy link
Member

@shalevr shalevr commented Nov 21, 2022

Description

#2847

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • tox -e sdk

Does This PR Require a Contrib Repo Change?

Answer the following question based on these examples of changes that would require a Contrib Repo Change:

  • The OTel specification has changed which prompted this PR to update the method interfaces of opentelemetry-api/ or opentelemetry-sdk/

  • The method interfaces of test/util have changed

  • Scripts in scripts/ that were copied over to the Contrib repo have changed

  • Configuration files that were copied over to the Contrib repo have changed (when consistency between repositories is applicable) such as in

    • pyproject.toml
    • isort.cfg
    • .flake8
  • When a new .github/CODEOWNER is added

  • Major changes to project information, such as in:

    • README.md
    • CONTRIBUTING.md
  • Yes. - Link to PR:

  • No.

Checklist:

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added

@shalevr shalevr requested a review from a team November 21, 2022 13:27
@shalevr shalevr force-pushed the decode-values-from-otel-resource-attributes branch from fa5a016 to eb5c744 Compare November 21, 2022 13:29
@shalevr shalevr marked this pull request as draft November 21, 2022 13:35
@shalevr shalevr force-pushed the decode-values-from-otel-resource-attributes branch 2 times, most recently from 579e526 to e54833a Compare November 21, 2022 13:41
@shalevr shalevr marked this pull request as ready for review November 21, 2022 13:50
@shalevr shalevr changed the title Add uri decode for OTEL_RESOUCE_ATTRIBUTES Add url decode for OTEL_RESOUCE_ATTRIBUTES Nov 21, 2022
@shalevr shalevr force-pushed the decode-values-from-otel-resource-attributes branch from e54833a to e9c793d Compare November 21, 2022 13:55
…/github.com/shalevr/opentelemetry-python into decode-values-from-otel-resource-attributes

* 'decode-values-from-otel-resource-attributes' of https://github.com/shalevr/opentelemetry-python:
  Avoid generator in _ViewInstrumentMatch.collect() (open-telemetry#3035)
@lzchen lzchen enabled auto-merge (squash) November 28, 2022 19:05
@lzchen lzchen merged commit 638988c into open-telemetry:main Nov 30, 2022
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.

4 participants