Conversation
|
Adding this check back (https://github.com/causify-ai/helpers/pull/1203/changes#r3113612752), as we need to ensure it does not cause issues when running on ECS Fargate through Airflow, since ECS Fargate does not support either DinD or sibling containers. See: https://docs.aws.amazon.com/AmazonECS/latest/developerguide/fargate-security-considerations.html
|
PomazkinG
left a comment
There was a problem hiding this comment.
Overall LG since it solves the problem at hand
Leaving a non-blocking comment
| """ | ||
| if not has_docker(): | ||
| return False | ||
| # This check is required to ensure it does not cause issues when running on ECS |
There was a problem hiding this comment.
I think here we want to return False if we are inside AirFlow.
This is a proxy for "if we are inside AirFlow"
if not has_dind_support() and not use_docker_sibling_containers():
We used to use is_inside_ecs_container() previously for that. However, I am not sure if that still would work. Or maybe we can create a new function that detects if we are running smth via AirFlow
Just leaving a TODO is fine
Task #1207
Pre-commit checks:
All checks passed ✅