Page MenuHomeElementl

[crag] default_config
ClosedPublic

Authored by alangenfeld on Wed, Jun 9, 9:00 PM.

Details

Summary

implement default_config by

  • creating a preset (that dagit should autoload in an upcoming change)
  • setting the defaults in the config schema so that if nothing is passed at any callsite it resolves
Test Plan

added tests

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Wed, Jun 9, 9:24 PM
Harbormaster failed remote builds in B31917: Diff 39314!

Will this config be used when invoking the job from a schedule or sensor? I had been imagining that it would.

Will this config be used when invoking the job from a schedule or sensor? I had been imagining that it would.

Yep unless something else is passed in the schedule/sensor callsite

This revision is now accepted and ready to land.Thu, Jun 10, 6:48 PM
This revision was automatically updated to reflect the committed changes.