Closed Sovietaced closed 3 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 61.07%. Comparing base (
25c3596
) to head (52f13a8
). Report is 141 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
After this PR is merged, we can enable the agent-service
plugin in the binary helm chart by default.
https://github.com/flyteorg/flyte/pull/5460
So I think we only need to fix the typo.
So I think we only need to fix the typo.
Fair enough, I will back out the helm chart change.
Why are the changes needed?
This fixes an issue found where the Flyte sandbox workflow causes propeller to panic on startup. The cause of this panic is that propeller is attempting to initialize the agent plugin when the agent service is down (by default). Requests to the agent service to retrieve the list of supported tasks fails with a timeout.
What changes were proposed in this pull request?
This change remove the agent service from the enabled plugin list considering that the flyte agent service is disabled by default.
How was this patch tested?
See generated Kubernetes manifests.
Setup process
Screenshots
Check all the applicable boxes