Skip to content

Disable telemetry by default#1168

Merged
hshen14 merged 2 commits intomainfrom
disable_telemetry_by_default
Jan 17, 2025
Merged

Disable telemetry by default#1168
hshen14 merged 2 commits intomainfrom
disable_telemetry_by_default

Conversation

@lvliang-intel
Copy link
Copy Markdown
Collaborator

Description

Disable telemetry by default, if users want to enable telemetry, please pass the env 'ENABLE_OPEA_TELEMETRY' with true.

Many our opea services(e.g. opea/llm-textgen, opea/embedding, opea/reranking, opea/retriever, etc) by default requires a OpenTelemetry trace collector to be in the system. Otherwise, the service will report the following error every time it process a user request.

Issues

n/a

Type of change

List the type of change like below. Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds new functionality)
  • Breaking change (fix or feature that would break existing design and interface)
  • Others (enhancement, documentation, validation, etc.)

Dependencies

None

Tests

Local test and CI test.

Signed-off-by: lvliang-intel <liang1.lv@intel.com>
@joshuayao joshuayao added this to the v1.2 milestone Jan 17, 2025
@joshuayao joshuayao self-requested a review January 17, 2025 11:54
@hshen14 hshen14 merged commit 854c286 into main Jan 17, 2025
@hshen14 hshen14 deleted the disable_telemetry_by_default branch January 17, 2025 11:56
smguggen pushed a commit to opea-aws-proserve/GenAIComps that referenced this pull request Jan 23, 2025
* Disable telemetry by default

Signed-off-by: lvliang-intel <liang1.lv@intel.com>
madison-evans pushed a commit to SAPD-Intel/GenAIComps that referenced this pull request May 12, 2025
* Disable telemetry by default

Signed-off-by: lvliang-intel <liang1.lv@intel.com>
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.

3 participants