Page MenuHomeElementl

K8sRunLauncher make postgres optional
ClosedPublic

Authored by johann on Apr 9 2021, 4:54 PM.

Details

Summary

The K8sRunLauncher currently assumes that the instance is using postgres, and needs a password to be piped in as an env var to the run worker. This diff makes postgres optional.

Test Plan

Integration

units

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

johann requested review of this revision.Apr 9 2021, 5:15 PM
This revision is now accepted and ready to land.Apr 9 2021, 5:18 PM
rexledesma added inline comments.
python_modules/libraries/dagster-k8s/dagster_k8s/launcher.py
57

Could you update the docstrings?

This revision was automatically updated to reflect the committed changes.