-
Notifications
You must be signed in to change notification settings - Fork 669
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update pyflyte serve into pyflyte serve agent #4526
Update pyflyte serve into pyflyte serve agent #4526
Conversation
Signed-off-by: Chao-Heng Lee <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4526 +/- ##
=======================================
Coverage 59.02% 59.02%
=======================================
Files 643 643
Lines 55153 55153
=======================================
+ Hits 32552 32555 +3
+ Misses 20020 20018 -2
+ Partials 2581 2580 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
# Conflicts: # deployment/agent/flyte_agent_helm_generated.yaml
…e-pyflyte-serve-agent-command Signed-off-by: Future-Outlier <[email protected]>
Signed-off-by: Future-Outlier <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, tested it in comments.
Tracking issue
#3936
Why are the changes needed?
Because of updating the cli for agent in flyteorg/flytekit#2019 , we have to update the helm chart to fit the changes.
What changes were proposed in this pull request?
Updating
pyflyte serve
intopyflyte serve agent
.Check all the applicable boxes
Related PRs
flyteorg/flytekit#2019