Skip to content

docs(magento): describe the database backend where Magento was memory or Temporal only (#739, #1036) - #1041

Open
gplanchat wants to merge 3 commits into
mainfrom
docs/magento-public-promise
Open

gplanchat wants to merge 3 commits into
mainfrom
docs/magento-public-promise

Conversation

@gplanchat

@gplanchat gplanchat commented Oct 9, 2026 •

Copy link
Copy Markdown
Owner

What

The Magento passages that still described two backends now describe three: the database backend that resource/durable selects, Temporal when durable/temporal/dsn is set, and the in-process journal otherwise. Declaring both throws BackendSelectionException.

#842 is already satisfied on main (the re-audit comment on the ticket lists each box), so I changed nothing for it.

Checked

  • Facts read in RuntimeFactory::backend(), JournalConnectionResolver, MagentoWorkflowRunCatalog (picked_up_at fills waitingForWorkerSince) and HealthCommand.
  • hugo --minify builds. git grep "only two of the four" returns nothing in EN and FR.
  • Not run: the picker over HTTP (not changed), anything needing a Magento install.

Not in this pull request

  • The landing-page picker (ALLOWED.magento, WHYNOT.magento, FWNOTE.magento in index.html and index.fr.html): [Task] Magento on a database: change the public promise (picker, backends and configuration pages, UPGRADE) #739 leaves the key of the new SQL row to you. Which key?
  • Stale docblocks in RuntimeFactory ("Magento reaches no other persistent journal"), outside the documentation.
  • durable:health still prints the "No Temporal DSN" message on the database backend; the pages say that durable:health only checks that the journal database answers, and that the grid shows a run that waits for a worker.

Refs #739, #1036

This branch has not been deployed

No deployments
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