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
Branch
k8s-launcher-pg-optional (branched from master)
Lint
Lint Passed
Unit
No Test Coverage

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.