Page MenuHomePhabricator

Fix build failure from six cleanup
ClosedPublic

Authored by nate on Wed, Jan 13, 6:19 PM.

Details

Summary

k8s API actually returns binary string, but our mock was returning regular string - this makes the mock consistent w/ the k8s API, and re-runs w/ integration tests also

Test Plan

integration

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 retitled this revision from Fix six build failure to Fix build failure from six cleanup.Wed, Jan 13, 6:33 PM
nate edited the summary of this revision. (Show Details)
nate published this revision for review.Wed, Jan 13, 6:37 PM
This revision is now accepted and ready to land.Wed, Jan 13, 6:43 PM
This revision was landed with ongoing or failed builds.Wed, Jan 13, 6:58 PM
This revision was automatically updated to reflect the committed changes.