Page MenuHomeElementl

Adds guards for default probes.
Needs ReviewPublic

Authored by bob on Jan 7 2021, 8:52 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

For k8s probes, only one of exec, tcpSocket, and httpGet can be defined. Currently, if a user wishes to override the default probes in their values.yaml using tcpSocket or httpGet, then there will be a conflict. This change seeks to prevent conflicts if the user defines any of exec, tcpSocket, or httpGet in their custom values.yaml.

Test Plan

buildkite
integration

Diff Detail

Repository
R1 dagster
Branch
chenbobby/check-helm-probes
Lint
No Lint Coverage
Unit
No Test Coverage