open-telemetry / open-telemetry/opentelemetry-python-contrib

[celery] warning emitted regarding mixed attribute types for 'celery.timelimit'

Open
#1,685 8 comments 0 reactions 1 assignee View on GitHub

@Akochavi is already working on this.

Since Feb 22, 2023.

bug
Dominant language
Python
Stars
1.1k
Forks
1.1k
Avg merge
4d 15h
Merged PRs (30d)
16

Description

Describe your environment

python-version: 3.10.8
celery: 5.3.0.b2
opentelemetry-api: 1.16.0
opentelemetry-instrumentation: 0.37b0
opentelemetry-instrumentation-celery: 0.37b0

Configured celery task soft time limit value: 30 (int)

Steps to reproduce

celery -Acelery_app worker -lINFO 

Instrument celery via opentelemetry.instrumentation.celery.CeleryInstrumentor during worker_process_init signal.
Warning log is emitted in relation to task ending/completion

What is the expected behavior?

No warning in regards to celery attributes

What is the actual behavior?

Mixed types warning is emitted

opentelemetry.attributes.Attribute 'celery.timelimit' mixes types str and int in attribute value sequence

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.