Skip to content

Allow suppression of warnings for dependencies. #1126

@michaelvanstraten

Description

@michaelvanstraten

When using registry dependencies in registry_manifest.yaml, those dependencies are resolved and warnings from their specs are merged into the dependent registry’s warnings. This results in hard-to-parse output from tools such as weaver registry check because, especially when using https://github.com/opentelemetry/semantic-conventions, there are many deprecation warnings.

It would be helpful to add an additional property in registry_manifest.yaml to allow suppressing warnings on a per-dependency basis. Additionally, a global CLI or config-file flag to disable all third-party warnings would be useful.

Example

# registry_manifest.yaml
dependencies:
  - name: otel-semconv
    uri: https://github.com/opentelemetry/semantic-conventions.git
    ref: v1.26.0
    suppress_warnings: true

An alternative would be to display warnings only when the corresponding signal/attribute is actually used. However, this would likely be more difficult to implement. I’ve attached a screenshot of the warnings below.

These warnings are coming from https://github.com/opentelemetry/semantic-conventions, even though I am not using any of the referenced attributes.

Image

Tip: React with 👍 to help prioritize this issue. Please use comments to provide useful context, avoiding +1 or me too, to help us triage it. Learn more here.

Metadata

Metadata

Assignees

Labels

enhancementNew feature or requestresolution engineIssues relating to behavior of resolving raw schema files into a cohesive model

Type

No type

Projects

Status

No status

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions