Skip to content

Commit 4f4679f

Browse files
committed
scripts/generate_instrumentation_bootstrap: don't force genai instrumentations versions
Since genai instrumentations are released on their own we cannot have a fixed required version on this side. While at it add vertexai to the list of excluded packages since: - it has not been released yet - the name is already claimed by openllmetry
1 parent 52bbcd6 commit 4f4679f

File tree

2 files changed

+22
-8
lines changed

2 files changed

+22
-8
lines changed

scripts/generate_instrumentation_bootstrap.py

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,15 +60,25 @@
6060
# See https://github.com/open-telemetry/opentelemetry-python-contrib/issues/2787
6161
packages_to_exclude = [
6262
"opentelemetry-instrumentation-aws-lambda",
63+
"opentelemetry-instrumentation-vertexai", # not released yet
64+
]
65+
66+
# We should not put any version limit for instrumentations that are released indipendently
67+
unversioned_packages = [
68+
"opentelemetry-instrumentation-openai-v2",
69+
"opentelemetry-instrumentation-vertexai",
6370
]
6471

6572

6673
def main():
6774
# pylint: disable=no-member
6875
default_instrumentations = ast.List(elts=[])
6976
libraries = ast.List(elts=[])
70-
for pkg in get_instrumentation_packages():
71-
if pkg.get("name") in packages_to_exclude:
77+
for pkg in get_instrumentation_packages(
78+
unversioned_packages=unversioned_packages
79+
):
80+
pkg_name = pkg.get("name")
81+
if pkg_name in packages_to_exclude:
7282
continue
7383
if not pkg["instruments"]:
7484
default_instrumentations.elts.append(ast.Str(pkg["requirement"]))

scripts/otel_packaging.py

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,8 @@
2424
genai_instrumentations_path = os.path.join(root_path, "instrumentation-genai")
2525

2626

27-
def get_instrumentation_packages():
27+
def get_instrumentation_packages(unversioned_packages=None):
28+
unversioned_packages = unversioned_packages or []
2829
pkg_paths = []
2930
for pkg in os.listdir(instrumentations_path):
3031
pkg_path = os.path.join(instrumentations_path, pkg)
@@ -62,12 +63,15 @@ def get_instrumentation_packages():
6263
"instruments"
6364
],
6465
}
65-
instrumentation["requirement"] = "==".join(
66-
(
67-
instrumentation["name"],
68-
instrumentation["version"],
66+
if instrumentation["name"] in unversioned_packages:
67+
instrumentation["requirement"] = instrumentation["name"]
68+
else:
69+
instrumentation["requirement"] = "==".join(
70+
(
71+
instrumentation["name"],
72+
instrumentation["version"],
73+
)
6974
)
70-
)
7175
yield instrumentation
7276

7377

0 commit comments

Comments
 (0)