Skip to content

Sweep forgotten client operation IDs#1965

Merged
MarcusSorealheis merged 3 commits intoTraceMachina:mainfrom
MarcusSorealheis:bugfix-sweep-operation-ids
Oct 18, 2025
Merged

Sweep forgotten client operation IDs#1965
MarcusSorealheis merged 3 commits intoTraceMachina:mainfrom
MarcusSorealheis:bugfix-sweep-operation-ids

Conversation

@MarcusSorealheis
Copy link
Collaborator

@MarcusSorealheis MarcusSorealheis commented Oct 10, 2025

Description

Client operations don't have an active operation in Redis, but they have an active client operation that's returning EXECUTING status. This PR removes those that do not have a corresponding active internal operation ID.

Fixes # (issue)

Type of change

Please delete options that aren't relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to
    not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Please also list any relevant details for your test configuration

Checklist

  • Updated documentation if needed
  • Tests added/amended
  • bazel test //... passes locally
  • PR is contained in a single commit, using git amend see some docs

This change is Reviewable

@MarcusSorealheis MarcusSorealheis merged commit 9fcf5b1 into TraceMachina:main Oct 18, 2025
27 checks passed
MarcusSorealheis added a commit to MarcusSorealheis/nativelink that referenced this pull request Nov 3, 2025
* Sweep forgotten client operation IDs

* add helpful log
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.

2 participants