Implement new HTTP semantic convention opt-in for Falcon by FilipNikolovski · Pull Request #2790 · open-telemetry/opentelemetry-python-contrib

FilipNikolovski · 2024-08-09T15:22:39Z

Description

This change updates the falcon instrumentor to adhere to the new HTTP semantic conventions, according to the migration plan outlined here:

also fixes #1532

Type of change

Please delete options that are not relevant.

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Updated the tests to assert that the new span and duration attributes have been added.

emdneto

A big thanks for working on this one. Just some comments

instrumentation/opentelemetry-instrumentation-falcon/tests/test_falcon.py

...on/opentelemetry-instrumentation-falcon/src/opentelemetry/instrumentation/falcon/__init__.py

CHANGELOG.md

…ests

xrmx · 2024-12-03T10:02:25Z

@FilipNikolovski tox -e generate step is failing, adding _semconv_status = "migration" in package.py should be enough

…ki/opentelemetry-python-contrib into falcon_new_semconv

FilipNikolovski · 2024-12-03T10:21:13Z

@FilipNikolovski tox -e generate step is failing, adding _semconv_status = "migration" in package.py should be enough

Thanks for the feedback. I've updated the file, but that step is still failing 🤷. Not sure if I need to run tox -e generate locally or not?

xrmx · 2024-12-03T11:00:52Z

@FilipNikolovski tox -e generate step is failing, adding _semconv_status = "migration" in package.py should be enough

Thanks for the feedback. I've updated the file, but that step is still failing 🤷. Not sure if I need to run tox -e generate locally or not?

You need to run it locally and commit the updated files so it will match on CI

...on/opentelemetry-instrumentation-falcon/src/opentelemetry/instrumentation/falcon/__init__.py

…ndependent from tracing decisions; update tests

FilipNikolovski · 2024-12-10T11:48:49Z

@xrmx running tox -e generate throws an error for me:

Could not get hatch version from path /opentelemetry-python-contrib/instrumentation/opentelemetry-instrumentation-sklearn
╭─────────────────────────────── Traceback (most recent call last) ────────────────────────────────╮
│ /opentelemetry-python-contrib/.tox/generate/lib/python3.12/site-pack │
│ ages/hatch/cli/__init__.py:221 in main                                                           │
│                                                                                                  │
│   218                                                                                            │
│   219 def main():  # no cov                                                                      │
│   220 │   try:                                                                                   │
    main()
  File "/opentelemetry-python-contrib/scripts/generate_instrumentation_bootstrap.py", line 70, in main
    for pkg in get_instrumentation_packages():
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opentelemetry-python-contrib/scripts/otel_packaging.py", line 51, in get_instrumentation_packages
    raise exc
  File "/opentelemetry-python-contrib/scripts/otel_packaging.py", line 42, in get_instrumentation_packages
    version = subprocess.check_output(
              ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/.pyenv/versions/3.12.7/lib/python3.12/subprocess.py", line 466, in check_output
│ ❱ 221 │   │   hatch(prog_name='hatch', windows_expand_args=False)                                │
│   222 │   except Exception:  # noqa: BLE001                                                      │
│   223 │   │   import sys                                                                         │
│   224                                                                                            │
│                                                                                                  │
│ /opentelemetry-python-contrib/.tox/generate/lib/python3.12/site-pack │
│ ages/click/core.py:1157 in __call__                                                              │
│                                                                                                  │
│ /opentelemetry-python-contrib/.tox/generate/lib/python3.12/site-pack │
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/.pyenv/versions/3.12.7/lib/python3.12/subprocess.py", line 571, in run
│ ages/click/core.py:1078 in main                                                                  │
│                                                                                                  │
│ /opentelemetry-python-contrib/.tox/generate/lib/python3.12/site-pack │
│ ages/click/core.py:1688 in invoke                                                                │
│                                                                                                  │
│ /opentelemetry-python-contrib/.tox/generate/lib/python3.12/site-pack │
│ ages/click/core.py:1434 in invoke                                                                │
│                                                                                                  │
│ /opentelemetry-python-contrib/.tox/generate/lib/python3.12/site-pack │
│ ages/click/core.py:783 in invoke                                                                 │
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command 'hatch version' returned non-zero exit status 1.
│                                                                                                  │
│ /opentelemetry-python-contrib/.tox/generate/lib/python3.12/site-pack │
│ ages/click/decorators.py:45 in new_func                                                          │
│                                                                                                  │
│ /opentelemetry-python-contrib/.tox/generate/lib/python3.12/site-pack │
│ ages/hatch/cli/version/__init__.py:35 in version                                                 │
│                                                                                                  │
│   32 │   │   if not (                                                                            │
│   33 │   │   │   'version' in app.project.metadata.dynamic or app.project.metadata.hatch.meta    │
│   34 │   │   ) or dependencies_in_sync(app.project.metadata.build.requires_complex):             │
│ ❱ 35 │   │   │   source = app.project.metadata.hatch.version.source                              │
│   36 │   │   │                                                                                   │
│   37 │   │   │   version_data = source.get_version_data()                                        │
│   38 │   │   │   original_version = version_data['version']                                      │
│                                                                                                  │
│ /opentelemetry-python-contrib/.tox/generate/lib/python3.12/site-pack │
│ ages/hatchling/metadata/core.py:1425 in version                                                  │
│                                                                                                  │
│   1422 │   │   if self._version is None:                                                         │
│   1423 │   │   │   if 'version' not in self.config:                                              │
│   1424 │   │   │   │   message = 'Missing `tool.hatch.version` configuration'                    │
│ ❱ 1425 │   │   │   │   raise ValueError(message)                                                 │
│   1426 │   │   │                                                                                 │
│   1427 │   │   │   options = self.config['version']                                              │
│   1428 │   │   │   if not isinstance(options, dict):                                             │
╰──────────────────────────────────────────────────────────────────────────────────────────────────╯
ValueError: Missing `tool.hatch.version` configuration

…ki/opentelemetry-python-contrib into falcon_new_semconv

...on/opentelemetry-instrumentation-falcon/src/opentelemetry/instrumentation/falcon/__init__.py

CHANGELOG.md

…ry-python-contrib into falcon_new_semconv

…ki/opentelemetry-python-contrib into falcon_new_semconv

instrumentation/opentelemetry-instrumentation-falcon/tests/test_falcon.py

…try#2790)

Implement new HTTP semantic convention opt-in for Falcon

2d68dc5

FilipNikolovski requested a review from a team August 9, 2024 15:22

fix tests for falcon 1 & 2

228897d

lzchen mentioned this pull request Aug 9, 2024

Implement migration plan for selected instrumentations #2453

Open

emdneto reviewed Aug 9, 2024

View reviewed changes

instrumentation/opentelemetry-instrumentation-falcon/tests/test_falcon.py Outdated Show resolved Hide resolved

...on/opentelemetry-instrumentation-falcon/src/opentelemetry/instrumentation/falcon/__init__.py Outdated Show resolved Hide resolved

lzchen reviewed Aug 12, 2024

View reviewed changes

CHANGELOG.md Show resolved Hide resolved

emdneto mentioned this pull request Aug 14, 2024

2677 jinja2 semconv #2798

Closed

7 tasks

Filip Nikolovski added 4 commits December 2, 2024 11:12

resolve conflicts

2d70a72

add semconv status as migration for falcon in instrumentations readme

a9a30a8

set schema_url based on the opt-in mode

b3db908

add histograms based on opt-in mode; set metrics attributes; update t…

aa54865

…ests

FilipNikolovski requested a review from a team as a code owner December 2, 2024 14:37

Filip Nikolovski added 3 commits December 2, 2024 15:40

run ruff

2ed1461

disable too-many-public-methods lint rule for TestFalconInstrumentation

e8d3e6c

Merge branch 'main' into falcon_new_semconv

d1e81c0

FilipNikolovski requested review from emdneto and lzchen December 3, 2024 08:49

Filip Nikolovski added 2 commits December 3, 2024 11:14

add _semconv_status in package.py

1815495

Merge branch 'falcon_new_semconv' of https://github.com/FilipNikolovs…

0307008

…ki/opentelemetry-python-contrib into falcon_new_semconv

emdneto reviewed Dec 3, 2024

View reviewed changes

...on/opentelemetry-instrumentation-falcon/src/opentelemetry/instrumentation/falcon/__init__.py Outdated Show resolved Hide resolved

Merge branch 'main' into falcon_new_semconv

a4419a6

lzchen reviewed Dec 3, 2024

View reviewed changes

...on/opentelemetry-instrumentation-falcon/src/opentelemetry/instrumentation/falcon/__init__.py Outdated Show resolved Hide resolved

lzchen reviewed Dec 3, 2024

View reviewed changes

...on/opentelemetry-instrumentation-falcon/src/opentelemetry/instrumentation/falcon/__init__.py Show resolved Hide resolved

Filip Nikolovski added 3 commits December 10, 2024 10:04

cleanup imports

7754f0d

set request attributes in env, set status code in metric attributes i…

b941569

…ndependent from tracing decisions; update tests

Merge branch 'main' into falcon_new_semconv

e14eaf9

emdneto approved these changes Dec 16, 2024

View reviewed changes

Filip Nikolovski added 3 commits December 31, 2024 12:14

sort imports

2d6b160

Merge branch 'falcon_new_semconv' of https://github.com/FilipNikolovs…

ecd351f

…ki/opentelemetry-python-contrib into falcon_new_semconv

fix tests

3d5ec55

xrmx reviewed Dec 31, 2024

View reviewed changes

...on/opentelemetry-instrumentation-falcon/src/opentelemetry/instrumentation/falcon/__init__.py Outdated Show resolved Hide resolved

xrmx reviewed Dec 31, 2024

View reviewed changes

CHANGELOG.md Outdated Show resolved Hide resolved

FilipNikolovski and others added 5 commits December 31, 2024 16:49

Merge branch 'open-telemetry:main' into falcon_new_semconv

e51f79d

Merge branch 'main' of https://github.com/FilipNikolovski/opentelemet…

bfb8a53

…ry-python-contrib into falcon_new_semconv

update changelog

606958a

Merge branch 'falcon_new_semconv' of https://github.com/FilipNikolovs…

6ee7d35

…ki/opentelemetry-python-contrib into falcon_new_semconv

Merge branch 'main' into falcon_new_semconv

e12ace6

FilipNikolovski requested a review from xrmx January 2, 2025 08:35

emdneto self-requested a review January 2, 2025 16:00

emdneto reviewed Jan 2, 2025

View reviewed changes

instrumentation/opentelemetry-instrumentation-falcon/tests/test_falcon.py Show resolved Hide resolved

emdneto reviewed Jan 2, 2025

View reviewed changes

instrumentation/opentelemetry-instrumentation-falcon/tests/test_falcon.py Outdated Show resolved Hide resolved

update tests

8b281b2

FilipNikolovski requested a review from emdneto January 2, 2025 18:34

Filip Nikolovski added 4 commits January 2, 2025 19:42

disable lint check on too-many-nested-blocks in metrics test

c662eae

use add_response_attributes function from wsgi

7702a59

resolve conflicts

174e144

cleanup unused functions

241a849

emdneto reviewed Jan 10, 2025

View reviewed changes

instrumentation/opentelemetry-instrumentation-falcon/tests/test_falcon.py Outdated Show resolved Hide resolved

emdneto reviewed Jan 10, 2025

View reviewed changes

instrumentation/opentelemetry-instrumentation-falcon/tests/test_falcon.py Outdated Show resolved Hide resolved

emdneto reviewed Jan 10, 2025

View reviewed changes

instrumentation/opentelemetry-instrumentation-falcon/tests/test_falcon.py Outdated Show resolved Hide resolved

Filip Nikolovski and others added 3 commits January 13, 2025 11:41

Merge branch 'main' into falcon_new_semconv

89a2a00

cleanup

2a5fac7

Merge branch 'main' into falcon_new_semconv

374f26d

lzchen approved these changes Jan 13, 2025

View reviewed changes

lzchen merged commit b7e7d0c into open-telemetry:main Jan 13, 2025

xrmx pushed a commit to xrmx/opentelemetry-python-contrib that referenced this pull request Jan 24, 2025

Implement new HTTP semantic convention opt-in for Falcon (open-teleme…

2fe1240

…try#2790)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Implement new HTTP semantic convention opt-in for Falcon#2790

Implement new HTTP semantic convention opt-in for Falcon#2790
lzchen merged 36 commits intoopen-telemetry:mainfrom
FilipNikolovski:falcon_new_semconv

FilipNikolovski commented Aug 9, 2024 •

edited

Loading

Uh oh!

emdneto left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

xrmx commented Dec 3, 2024

Uh oh!

FilipNikolovski commented Dec 3, 2024

Uh oh!

xrmx commented Dec 3, 2024 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

FilipNikolovski commented Dec 10, 2024

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

FilipNikolovski commented Aug 9, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Type of change

How Has This Been Tested?

Uh oh!

emdneto left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

xrmx commented Dec 3, 2024

Uh oh!

FilipNikolovski commented Dec 3, 2024

Uh oh!

xrmx commented Dec 3, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

FilipNikolovski commented Dec 10, 2024

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

FilipNikolovski commented Aug 9, 2024 •

edited

Loading

xrmx commented Dec 3, 2024 •

edited

Loading