Skip to content

Rename the object-store plugin proto package to objectstore.v1 - #2432

Merged
Haven Xia (HavenXia) merged 1 commit into
agent-substrate:mainfrom
aramase:aramase/c/objectstore_proto_rename
Oct 9, 2026
Merged

Haven Xia (HavenXia) merged 1 commit into
agent-substrate:mainfrom
aramase:aramase/c/objectstore_proto_rename

Conversation

@aramase

Copy link
Copy Markdown
Collaborator

Renames the object-store plugin proto package from objectstoresnapshot.v1 to objectstore.v1. It is split out of #2385 as its own refactor, since that PR adds a non-snapshot service to the same API.

  • Moves pkg/proto/objectstoresnapshotpb/v1 to pkg/proto/objectstorepb/v1 and objectstoresnapshot.proto to objectstore.proto, regenerated with hack/update/codegen.sh.
  • Updates every in-tree caller, the protoredact pinned field names, and docs/architecture.md.
  • Services, messages and fields keep their names. The cmd/snapshot-plugin binary, image and manifests are unchanged.

Open #1907 will need a rebase. It adds no references to the old package, so only its existing conflicts with main apply.

Breaking change

gRPC method names change from /objectstoresnapshot.v1.<Service>/<Method> to /objectstore.v1.<Service>/<Method>. Out-of-tree plugins must regenerate from objectstore.proto and roll out with atelet and ate-api-server. Please add the breaking-change label.

  • Tests pass
  • Appropriate changes to documentation are included in the PR

@aramase
Anish Ramasekar (aramase) force-pushed the aramase/c/objectstore_proto_rename branch from cb7e9f6 to fa77acf Compare October 9, 2026 21:24
@aramase

Copy link
Copy Markdown
Collaborator Author

cc ILLUM1N0X

Haven Xia (@HavenXia) the CI is failing because of docker rate limit. Could you rerun the CI?

@aramase
Anish Ramasekar (aramase) marked this pull request as ready for review October 9, 2026 21:36
@HavenXia

Copy link
Copy Markdown
Member

cc ILLUM1N0X

Haven Xia (Haven Xia (@HavenXia)) the CI is failing because of docker rate limit. Could you rerun the CI?

rerunning, will merge when they passed

@HavenXia
Haven Xia (HavenXia) added this pull request to the merge queue Oct 9, 2026
Merged via the queue into agent-substrate:main with commit 85b128a Oct 9, 2026
15 of 18 checks passed
@aramase
Anish Ramasekar (aramase) deleted the aramase/c/objectstore_proto_rename branch October 9, 2026 23:07
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.

3 participants