Add ADR to precompute API cache for resilience#677
Draft
edavey wants to merge 1 commit into
Draft
Conversation
edavey
requested review from
ChrisBAshton,
brucebolt,
graycodes,
jfharden,
leenagupte,
mike3985,
minglis,
patrickjfl and
yndajas
brucebolt
reviewed
May 21, 2026
| ### 3. Rely on Sidekiq retry for resilience (initially) | ||
|
|
||
| If the render API is unavailable or times out, `DownstreamLiveJob` in Publishing API will fail and retry via | ||
| Sidekiq's built-in exponential backoff (25 retries over ~21 days). This provides resilience without application-level retry logic. |
Member
There was a problem hiding this comment.
This proposal seems to depend heavily on processing of documents in Publishing API being asynchronous. However we are moving away from this with GraphQL, where links are expanded and content blocks are embedded at the point of a user requesting the page.
Have you taken this into consideration, as it'll mean we'll need to additionally make a request to Content Block Manager when rendering content to the public?
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.
DRAFT: Please let us know what you think
ADR to precompute API cache for resilience
ADR 13: Add an API for Content Block Manager establishes that Content Block Manager (CBM) will expose a
rendering API (
POST /api/blocks/render) which Publishing API and other applications will call instead of renderingcontent blocks locally via the
content_block_toolsgem.This will introduce a network dependency where none exists today.
See ADR for details.