-
Notifications
You must be signed in to change notification settings - Fork 543
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
Implemented simple echo plugin for testing #4489
Conversation
Signed-off-by: Daniel Rammer <daniel@union.ai>
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #4489 +/- ##
==========================================
- Coverage 59.76% 59.62% -0.14%
==========================================
Files 636 638 +2
Lines 53870 53995 +125
==========================================
+ Hits 32193 32196 +3
- Misses 19146 19267 +121
- Partials 2531 2532 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Daniel Rammer <daniel@union.ai>
Signed-off-by: Daniel Rammer <daniel@union.ai>
Signed-off-by: Daniel Rammer <daniel@union.ai>
Signed-off-by: Daniel Rammer <daniel@union.ai>
Signed-off-by: Daniel Rammer <daniel@union.ai>
Signed-off-by: Daniel Rammer <daniel@union.ai>
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.
thoughts on naming this plugin package echo
instead of testing
?
@eapolinario so it is actually named |
Signed-off-by: Daniel Rammer <daniel@union.ai>
Tracking issue
NA
Docs link
NA
Describe your changes
Implements a simple echo plugin which copies the task inputs to outputs. This facilitates local testing at scale without creating a large number of k8s pods. Additionally, this can support future extensions to force retries, etc.
Check all the applicable boxes
Setup Process
Enable to plugin in flyte configuration. Note that this plugin is set to be the default, so ALL tasks without a specific plugin assigned will use it. Additionally, two default plugins may not be enabled, so the PodPlugin cannot be simultaneously used.
Screenshots
NA
Note to reviewers
NA
Related PRs
NA