Introduce the worker API - #1000
Merged
Julian Gutierrez Oschmann (juli4n) merged 6 commits intoAug 19, 2026
Merged
Conversation
sfunkenhauser
force-pushed
the
worker_api
branch
from
August 17, 2026 18:50
88c16cb to
313b4d2
Compare
sfunkenhauser
marked this pull request as ready for review
August 17, 2026 19:35
Make workers a gobal resource, identified by a unique name. Worker APIs will be implemented in a follow up change.
sfunkenhauser
force-pushed
the
worker_api
branch
from
August 18, 2026 15:28
313b4d2 to
6cea471
Compare
sfunkenhauser
commented
Aug 18, 2026
| // See the License for the specific language governing permissions and | ||
| // limitations under the License. | ||
|
|
||
| package controlapi |
Collaborator
Author
There was a problem hiding this comment.
worker.go has been created since I started this PR. I'll move all these API stubs to worker.go.
sfunkenhauser
force-pushed
the
worker_api
branch
from
August 19, 2026 14:58
f06d43d to
5accfaf
Compare
sfunkenhauser
force-pushed
the
worker_api
branch
from
August 19, 2026 15:09
5accfaf to
6bb753d
Compare
Julian Gutierrez Oschmann (juli4n)
previously approved these changes
Aug 19, 2026
Julian Gutierrez Oschmann (juli4n)
approved these changes
Aug 19, 2026
Julian Gutierrez Oschmann (juli4n)
merged commit Aug 19, 2026
43fa6cc
into
agent-substrate:main
9 checks passed
shrutiyam-glitch
added a commit
to shrutiyam-glitch/substrate
that referenced
this pull request
Aug 19, 2026
Main's worker rework (agent-substrate#1000 and successors) keys workers by a single resource name (Metadata.Name), moves the assignment under WorkerStatus, and references workers in MintCert by ObjectRef. Adapt the branch: - DeleteWorker keeps the feed append (writeAndAppendChange) on the new name-keyed signature; DebugClearAll truncates the current table set plus worker_changes/worker_changes_trim. - authorizeActor keeps the store read-through on denial, now reading through by worker name; denyMint logs the worker resource name. - Feed tests seed Metadata-named workers and use the name-keyed Get/Update/Delete; the read-through test uses the actorFixture/ WorkerStatus.Assignment shapes. All store, actoridentity, workercache and controlapi suites pass.
2 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Make workers a gobal resource, identified by a unique name. Worker APIs will be implemented in a follow up change.
Part of #730