Remove stale comment

xrmx · xrmx · commit 813f40f30fb2 · 2024-11-18T17:09:40.000+01:00
Model is mandatory with azure client too.
diff --git a/instrumentation/elastic-opentelemetry-instrumentation-openai/src/opentelemetry/instrumentation/openai/helpers.py b/instrumentation/elastic-opentelemetry-instrumentation-openai/src/opentelemetry/instrumentation/openai/helpers.py
@@ -158,7 +158,6 @@ def _get_span_attributes_from_wrapper(instance, kwargs) -> Attributes:
         GEN_AI_SYSTEM: "openai",
     }
 
-    # on some azure clients the model was not mandatory
     if (request_model := kwargs.get("model")) is not None:
         span_attributes[GEN_AI_REQUEST_MODEL] = request_model
 

Original file line number	Diff line number	Diff line change
`@@ -158,7 +158,6 @@ def _get_span_attributes_from_wrapper(instance, kwargs) -> Attributes:`
`158`	`158`	`GEN_AI_SYSTEM: "openai",`
`159`	`159`	`}`
`160`	`160`
`161`		`- # on some azure clients the model was not mandatory`
`162`	`161`	`if (request_model := kwargs.get("model")) is not None:`
`163`	`162`	`span_attributes[GEN_AI_REQUEST_MODEL] = request_model`
`164`	`163`