-
Notifications
You must be signed in to change notification settings - Fork 659
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
[BUG] Agents cannot pass custom info back to flyte #5603
Comments
The To address this, Flytekit and Flyte need to be enhanced to include the Would you like more detailed steps on how to implement these changes? Sources: This is an AI-generated response and your feedback is appreciated! Please leave a 👍 if this is helpful and 👎 if it is not. Mention @RunLLM to ask follow-up questions. See runllm.com for more! |
FYI some PRs which implement a fix to this issue. #5604 (this should land first) |
Closing since both PRs have been merged. Thanks! 🎉 |
Proposed fix: #5604 flyteorg/flytekit#2426
Describe the bug
Agents may use the Resource object to pass some data back to flyte, but the
CustomInfo
field is not exposed in this object.The solution is to enhance flytekit to add this field to the Resource object, and enhance flyte (flyteplugins webapi agent) to read this field from the agent response: flyteorg/flytekit#2426 https://github.com/flyteorg/flyte/pull/5604/files
Expected behavior
Agents should be able to pass custom info back to Flyte. See
custom info
under https://docs.flyte.org/en/latest/protos/docs/event/event.html#taskexecutioneventAdditional context to reproduce
No response
Screenshots
No response
Are you sure this issue hasn't been raised already?
Have you read the Code of Conduct?
The text was updated successfully, but these errors were encountered: