https://prefect.io logo
Title
a

Adam Roderick

03/01/2022, 1:31 PM
After upgrading to v1, I am seeing an error when my ECS Agent attempts to start a new task. Any idea why?
botocore.errorfactory.ClientException: An error occurred (ClientException) when calling the RegisterTaskDefinition operation: Invalid setting for container 'flow'. At least one of 'memory' or 'memoryReservation' must be specified.
But memory is specified in the run config
a

Anna Geller

03/01/2022, 1:35 PM
would you be willing to open a Github issue for that? (otherwise I can do that) https://github.com/PrefectHQ/prefect/issues/new/choose This is something we would need to investigate in more detail
a

Adam Roderick

03/01/2022, 2:08 PM
a

Anna Geller

03/01/2022, 2:09 PM
amazing, thanks a lot! 👏
I'm adding this to my todo list later this week to check 😄
a

Adam Roderick

03/01/2022, 2:10 PM
Thanks
I added to the ticket one thought: We are setting the memory and we have a task definition--do those conflict?
flow.run_config = ECSRun(
        cpu="1 vcpu",  
        memory="8 GB",
        task_definition=task_definition,
a

Anna Geller

03/01/2022, 2:12 PM
they shouldn't, we have some functions that merge those kwargs in the end, but thanks for mentioning that and I'll definitely investigate - always better to provide more details than not enough 👍
k

Kevin Kho

03/01/2022, 2:56 PM
Could you show me the task definition you have? Just remove sensitive stuff I’ll try to reproduce
🙏 1
a

Adam Roderick

03/08/2022, 3:55 PM
Closing this out--it had to do with where I was defining memory. It needed to be in the task definition
👍 2
k

Kevin Kho

03/08/2022, 3:57 PM
Thanks for circling back!
a

Anna Geller

03/08/2022, 7:02 PM
@Adam Roderick that's weird, your memory override set on ECSRun should override those set on a task definition 🤔 anyway, adding your comment and closing the Github issue then
a

Adam Roderick

03/08/2022, 7:52 PM
Thanks @Anna Geller. There are several changes in flight so most likely it was caused by something else. What you explained makes sense
👍 1