Page MenuHomeElementl

Update k8s docs with new images
ClosedPublic

Authored by johann on Feb 6 2021, 1:18 AM.

Details

Summary

Update diagrams and docker image links with the new provided images.

Sorry about the auto format, the actual diff is here https://www.diffchecker.com/mp0uUARV

Test Plan

local render

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

johann requested review of this revision.Feb 6 2021, 1:36 AM
dgibson added a subscriber: sashank.

i've also run into the auto-formatting thing, cc @sashank maybe some kind of lint rule here would be good to make sure it goes in using the standard vscode formatting for these?

CHANGES.md
7

maybe make it clear where they're provided - is "in the Dagster Helm chart" accurate? Maybe link to the docs page?

8

'if applicable' => 'when using the celery_k8s_executor' (assuming that's right?)

Is 'host processes' a term that users will understand? Maybe 'processes that don't need to load user code'?

10

missing word

This revision is now accepted and ready to land.Feb 8 2021, 3:48 PM

Good call, will make sure everyone uses the same formatting settlings with prettier

This revision was landed with ongoing or failed builds.Feb 9 2021, 5:07 PM
This revision was automatically updated to reflect the committed changes.