Skip to content

Replace UpdateTask with CreateTask and enforce immutability - #398

Merged
rakyll merged 4 commits into
mainfrom
immutable-task2
Sep 26, 2026
Merged

rakyll merged 4 commits into
mainfrom
immutable-task2

Conversation

@rakyll

@rakyll rakyll commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Replaced UpdateTask RPC with CreateTask. Tasks are now immutable once created; modifying an existing task returns a FailedPrecondition error.

  • Renamed UpdateTask/UpdateTaskRequest to CreateTask/CreateTaskRequest.
  • Checked existence in Server.CreateTask to prevent mutation.
  • Updated CLI apply logic, tests, and documentation.
  • We will revisit updates for version upgrades but we will introduce new semantics for it.

Replaced UpdateTask RPC with CreateTask. Tasks are now
immutable once created; modifying an existing task returns a
FailedPrecondition error.

- Renamed UpdateTask/UpdateTaskRequest to CreateTask/CreateTaskRequest.
- Checked existence in Server.CreateTask to prevent mutation.
- Updated CLI apply logic, tests, and documentation.
@rakyll
rakyll marked this pull request as ready for review September 25, 2026 01:23
Comment thread internal/server/server.go
Comment thread cmd/ax/main.go Outdated
Comment thread cmd/ax/main.go Outdated
Comment thread pkg/apis/v1alpha1/ax.proto
Comment thread internal/server/server.go Outdated
@rakyll
rakyll merged commit c7de928 into main Sep 26, 2026
9 checks passed
@rakyll
rakyll deleted the immutable-task2 branch September 26, 2026 03:59
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