Page MenuHomePhabricator

Fix helm namespace lifecycle in tests
ClosedPublic

Authored by dgibson on Fri, Oct 16, 7:52 PM.

Details

Summary

I think the intention of this contextmanager is to leave the helm resources for the whole lifecycle, but currently its returning but not yielding so that doesn't happen. Make it a contextmanager so that it will wait until the end of the test to cleanup.

Test Plan

k8s integration tests now pass locally again

Diff Detail

Repository
R1 dagster
Branch
kind2
Lint
Lint OK
Unit
No Unit Test Coverage