Skip to content

docs: align EDB endpoint table and config params with controller code#470

Merged
kichasov merged 1 commit into
feat/operator-devfrom
docs/sync-with-code
May 21, 2026
Merged

docs: align EDB endpoint table and config params with controller code#470
kichasov merged 1 commit into
feat/operator-devfrom
docs/sync-with-code

Conversation

@kichasov
Copy link
Copy Markdown
Collaborator

Summary

Three small corrections in docs/howto/DBaaS Operator.md found during a code-vs-docs audit. No behavior changes.

# Fix Why
1 EDB success row: 200 OK / 201 Created (was: only 201 Created) aggregator_client.go:151 accepts both: resp.StatusCode() != http.StatusOK && resp.StatusCode() != http.StatusCreated
2 Add 410 / 422 to the EDB aggregator-rejection table client/types.go:191–199 IsSpecRejection covers 400, 403, 409, 410, 422 — all five route to InvalidConfiguration / AggregatorRejected. DD endpoint table already listed all five; EDB was inconsistent.
3 Add DBAAS_AGGREGATOR_URL env var to the Service Parameters table cmd/main.go:107 reads it with default http://dbaas-aggregator:8080; Helm chart does not set it. Documenting the override path for cross-cluster setups.

Test plan

  • Doc rendering verified (3 lines changed, no structural changes)
  • Each claim cross-checked against the cited code line

@kichasov kichasov requested a review from ArkuNC as a code owner May 21, 2026 13:38
@kichasov kichasov force-pushed the docs/sync-with-code branch from 38e6574 to ee4a3a4 Compare May 21, 2026 13:43
@kichasov kichasov merged commit 06143cc into feat/operator-dev May 21, 2026
12 checks passed
@kichasov kichasov deleted the docs/sync-with-code branch May 21, 2026 14:00
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.

1 participant