Skip to content

Rename retry loop spans from retry_attempt to attempt_N - #28

Merged
manas-fortifyroot merged 1 commit into
fr-v0.52.6.xfrom
feature/retry-loop-attempt-span-naming
Jun 13, 2026
Merged

Rename retry loop spans from retry_attempt to attempt_N#28
manas-fortifyroot merged 1 commit into
fr-v0.52.6.xfrom
feature/retry-loop-attempt-span-naming

Conversation

@manas-fortifyroot

Copy link
Copy Markdown

Summary

  • Rename FortifyRoot retry-loop child spans from retry_attempt to numbered attempt_N names.
  • Add attempt numbering so attempt_1 represents the first provider call and attempt_N where N > 1 clearly represents retries.
  • Replace retry-attempt role semantics with attempt-oriented span attributes.
  • Update retry-loop tests across supported instrumentations.

Testing

  • Ran fr-openllmetry-py/scripts/run-tests.sh --all.
  • Result: passed.

@manas-fortifyroot
manas-fortifyroot merged commit 04a9a27 into fr-v0.52.6.x Jun 13, 2026
10 checks passed
@manas-fortifyroot
manas-fortifyroot deleted the feature/retry-loop-attempt-span-naming branch June 13, 2026 08:13
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.

1 participant