Complete CRUD: GET-single and DELETE endpoints #3

2026-07-19T22:18:08+10:00

unkinben commented

2026-07-19 22:18:08 +10:00

Why

The Terraform provider needs full read/update/delete lifecycle per resource, but the name-keyed resources only exposed list + upsert. This completes their CRUD.

Changes

Add GET /{name} and DELETE /{name} for zones, address-groups, portgroups, fabrics, devices, and bindings, plus GET /rules/{id}.
Add the matching store Get/Delete methods; deletes bump the config generation and return 404 when nothing matched.
Factor shared respondOne/respondDelete helpers.

## Why The Terraform provider needs full read/update/delete lifecycle per resource, but the name-keyed resources only exposed list + upsert. This completes their CRUD. ## Changes - Add GET /{name} and DELETE /{name} for zones, address-groups, portgroups, fabrics, devices, and bindings, plus GET /rules/{id}. - Add the matching store Get/Delete methods; deletes bump the config generation and return 404 when nothing matched. - Factor shared respondOne/respondDelete helpers.

unkinben added 1 commit 2026-07-19 22:18:08 +10:00

Complete CRUD: add GET-single and DELETE endpoints

ci/woodpecker/pr/pre-commit Pipeline was successful

Details

ci/woodpecker/pr/build Pipeline was successful

Details

ci/woodpecker/pr/test Pipeline was successful

Details

82c7d9c5f7

Add GET /{name} and DELETE /{name} for zones, address-groups, portgroups,
fabrics, devices, and bindings, plus GET /rules/{id}, with the matching store
Get/Delete methods (deletes bump the generation and 404 on no-match). This gives
the resources full read/update/delete lifecycle so the Terraform provider can
manage them.

benvin merged commit de8ce4b4ec into main

2026-07-20 07:39:22 +10:00

benvin deleted branch benvin/complete-crud

2026-07-20 07:39:22 +10:00

unkinben referenced this pull request

2026-07-20 23:52:04 +10:00

Store device FIB for observability (no rule limiting) #5

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: unkin/tomswallapi#3