Skip to content

Fix log level in resolution algorithm#995

Merged
jsuereth merged 2 commits intoopen-telemetry:mainfrom
lmolkova:fix-log-level
Nov 4, 2025
Merged

Fix log level in resolution algorithm#995
jsuereth merged 2 commits intoopen-telemetry:mainfrom
lmolkova:fix-log-level

Conversation

@lmolkova
Copy link
Member

@lmolkova lmolkova commented Nov 4, 2025

I was experimenting with dev version of weaver and noticed it writes confusing log message by default.

It looks like an error, but everything resolves fine.

Fixing log level - this is just debug info.

Resolved 200 extends in this iteration, found errors [
    UnresolvedExtendsRef {
        group_id: "span.http.client",
        extends_ref: "attributes.http.client",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/http/spans.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "span.http.server",
        extends_ref: "attributes.http.server",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/http/spans.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "metric.rpc.server.requests_per_rpc",
        extends_ref: "attributes.metrics.rpc.server",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/rpc/deprecated/metrics-deprecated.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "metric.rpc.server.responses_per_rpc",
        extends_ref: "attributes.metrics.rpc.server",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/rpc/deprecated/metrics-deprecated.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "metric.messaging.client.published.messages",
        extends_ref: "metric.messaging.attributes",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/messaging/deprecated/metrics-deprecated.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "metric.rpc.server.duration",
        extends_ref: "attributes.metrics.rpc.server",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/rpc/metrics.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "metric.rpc.server.request.size",
        extends_ref: "attributes.metrics.rpc.server",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/rpc/metrics.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "metric.rpc.server.response.size",
        extends_ref: "attributes.metrics.rpc.server",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/rpc/metrics.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "span.db.sql_server.client",
        extends_ref: "span.db.sql.client",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/database/spans.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "span.db.postgresql.client",
        extends_ref: "span.db.sql.client",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/database/spans.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "span.db.mysql.client",
        extends_ref: "span.db.sql.client",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/database/spans.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "span.db.mariadb.client",
        extends_ref: "span.db.sql.client",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/database/spans.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "span.rpc.client",
        extends_ref: "rpc_service",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/rpc/spans.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "span.rpc.server",
        extends_ref: "rpc_service.server",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/rpc/spans.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "span.rpc.connect_rpc.client",
        extends_ref: "rpc_service",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/rpc/spans.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "span.rpc.connect_rpc.server",
        extends_ref: "rpc_service.server",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/rpc/spans.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "span.rpc.grpc.client",
        extends_ref: "rpc_service",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/rpc/spans.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "span.rpc.grpc.server",
        extends_ref: "rpc_service.server",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/rpc/spans.yaml",
        },
    },
    UnresolvedExtendsRef {
        group_id: "span.rpc.jsonrpc.server",
        extends_ref: "rpc.server",
        provenance: Provenance {
            registry_id: "main",
            path: "https://github.com/open-telemetry/semantic-conventions.git[model]/rpc/spans.yaml",
        },
    },
]

@lmolkova lmolkova requested a review from a team as a code owner November 4, 2025 00:25
@lmolkova lmolkova moved this to Next Release in OTel Weaver Project Nov 4, 2025
@codecov
Copy link

codecov bot commented Nov 4, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.3%. Comparing base (7ee54dc) to head (0df6c8e).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files 
@@          Coverage Diff          @@
##            main    #995   +/-   ##
=====================================
  Coverage   78.3%   78.3%           
=====================================
  Files         77      77           
  Lines       6122    6122           
=====================================
+ Hits        4794    4797    +3     
+ Misses      1328    1325    -3

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lmolkova
Copy link
Member Author

lmolkova commented Nov 4, 2025

Clippy fixes: #996

@jsuereth jsuereth merged commit 286dc6a into open-telemetry:main Nov 4, 2025
23 checks passed
@github-project-automation github-project-automation bot moved this from Next Release to Done in OTel Weaver Project Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jsuereth jsuereth jsuereth approved these changes

Assignees

No one assigned

Labels

None yet

Projects

OTel Weaver Project
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lmolkova @jsuereth