Skip to content

fix(instrumentation): always process Git Operation spans#42119

Merged
jamietanna merged 6 commits intomainfrom
fix/git-instrument
Mar 25, 2026
Merged

fix(instrumentation): always process Git Operation spans#42119
jamietanna merged 6 commits intomainfrom
fix/git-instrument

Conversation

@jamietanna
Copy link
Copy Markdown
Contributor

Changes

As part of #39596, we added the ability
to report the Git Operations Statistics at the end of the Renovate
run, by introducing a SpanProcessor.

However, this didn't correctly take into account the fact that this
would only be enabled if Renovate was configured to log/export traces,
which is not the case in most deployments.

As noted in #42118, this leads to a blank log line output, which isn't
very useful.

To correct this, we nede to always process these spans, even if tracing
is not enabled.

Context

Please select one of the below:

AI assistance disclosure

Did you use AI tools to create any part of this pull request?

Please select one option and, if yes, briefly describe how AI was used (e.g., code, tests, docs) and which tool(s) you used.

  • No — I did not use AI for this contribution.
  • Yes — minimal assistance (e.g., IDE autocomplete, small code completions, grammar fixes).
  • Yes — substantive assistance (AI generated non‑trivial portions of code, tests, or documentation).
  • Yes — other (please describe): Claude Sonnet 4.6

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

The public repository:

Loading
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"Git operations statistics" does not fire on Mend-hosted platforms

3 participants