Page MenuHomePhabricator

(helm) Consolidate dagster_home and clean up image pull policy
ClosedPublic

Authored by nate on Wed, May 13, 3:41 AM.

Details

Summary
  • Image pull policy should be per component, not global
  • Just use a single dagster_home - we weren't properly consuming the overrides anyway
Test Plan

bk

Diff Detail

Repository
R1 dagster
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

nate created this revision.Wed, May 13, 3:41 AM
nate edited the summary of this revision. (Show Details)Wed, May 13, 3:43 AM
Harbormaster returned this revision to the author for changes because remote builds failed.Wed, May 13, 3:55 AM
Harbormaster failed remote builds in B11250: Diff 13823!
nate requested review of this revision.Wed, May 13, 4:35 AM
max accepted this revision.Wed, May 13, 4:36 PM
This revision is now accepted and ready to land.Wed, May 13, 4:36 PM