Skip to content

GenAI: server.port is usually 443 but we require setting it on every spanΒ #1828

@aabmass

Description

@aabmass

Area(s)

area:gen-ai

What's missing?

Original thread for context: open-telemetry/opentelemetry-python-contrib#3208 (comment)

Right now, the general conventions require a server.port if server.address is present

| [`server.port`](/docs/attributes-registry/server.md) | int | GenAI server port. [5] | `80`; `8080`; `443` | `Conditionally Required` If `server.address` is set. | ![Stable](https://img.shields.io/badge/-stable-lightgreen) |

In most cases it's 443 so this introduces some overhead. Individual GenAI systems like Azure or Vertex AI may have a different default.

Describe the solution you'd like

Azure has a specific convention for this which allows eliding server.port:

Conditionally Required If not default (443)

However that conflicts with the general GenAI span conventions which require it. We should consider

  1. Loosening the GenAI server.port convention by adding a note there that system-specific conventions can provide a default value
  2. Specify a default server.port of 443 for all systems and not add system-specific conventions. This would cover most standard HTTPS communication and be a bit simpler.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions