<@U03CLARPEJ0> wants to add a new field for the ag...
# flyte-connectorsd
damp-lion-88352
11/13/2024, 1:31 PM@brief-window-55364 wants to add a new field for the agent's error message. He suggests creating a new error IDL specifically for the agent and thinks we should avoid using
ContainerErrordamp-lion-88352
11/13/2024, 4:19 PMto @brief-window-55364 Kevin and I think this is good
damp-lion-88352
11/13/2024, 4:19 PMdamp-lion-88352
11/13/2024, 4:19 PMjsut need this
b
brief-window-55364
11/13/2024, 4:20 PMCool stuff! Leave it with me then tomorrow at the latest I'll get it ready for review.
d
damp-lion-88352
11/13/2024, 4:20 PMwe need to ask haytham's advice
damp-lion-88352
11/13/2024, 4:20 PMbut we are good to this
damp-lion-88352
11/13/2024, 4:21 PMcan you write a comprehensive PR descrpiotion?
b
brief-window-55364
11/13/2024, 4:23 PMOf course
d
damp-lion-88352
11/13/2024, 4:32 PMthank you, will ping Haytham to decide we should use this IDL or not.
damp-lion-88352
11/13/2024, 4:32 PMthank you so much
b
brief-window-55364
11/13/2024, 4:33 PMNo worries. Thank you for taking the time 😄
brief-window-55364
11/14/2024, 8:43 AMd
damp-lion-88352
11/14/2024, 1:25 PMlet me ping others when they wake up
damp-lion-88352
11/14/2024, 1:25 PMjust finished my review
damp-lion-88352
11/14/2024, 1:25 PMthis is great definitely
damp-lion-88352
11/14/2024, 4:00 PM[scheduled message]
@high-park-82026
Do you agree with this IDL change for flyte agent?
Kevin and I agreed with this, but we want to double-check
https://github.com/flyteorg/flyte/pull/5916
g
glamorous-carpet-83516
11/22/2024, 9:04 PMjust merged it, thank you @brief-window-55364