Skip to content

Commit

Permalink
[dagster-aws] add ECS Task executor (#26098)
Browse files Browse the repository at this point in the history
## Summary & Motivation

This PR ads a new ECS Tasks Executor.

Currently a `ECSRunLauncher` is required in order to use the new `ecs_executor` and most of the ECS config comes from the run launcher. The executor can override a few important fields (such as resources and container overrides). 

Resolve #9671

## How I Tested These Changes

- manual testing in personal AWS account
- adding tests with mocks. Heavily using the existing run launcher testing infra since the executor currently requires it anyway

## Changelog

[dagster-aws] new `ecs_executor` which executes Dagster steps via AWS ECS tasks. Initially, we expect it to be used in conjunction with `ECSRunLauncher`.
  • Loading branch information
danielgafni authored Dec 17, 2024
1 parent 5499873 commit 82e2189
Show file tree
Hide file tree
Showing 11 changed files with 778 additions and 43 deletions.
Binary file modified docs/content/api/modules.json.gz
Binary file not shown.
Binary file modified docs/content/api/searchindex.json.gz
Binary file not shown.
Binary file modified docs/content/api/sections.json.gz
Binary file not shown.
Binary file modified docs/next/public/objects.inv
Binary file not shown.
3 changes: 3 additions & 0 deletions docs/sphinx/sections/api/apidocs/libraries/dagster-aws.rst
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,9 @@ ECS
.. autoconfigurable:: dagster_aws.ecs.EcsRunLauncher
:annotation: RunLauncher

.. autoconfigurable:: dagster_aws.ecs.ecs_executor
:annotation: ExecutorDefinition


Redshift
--------
Expand Down
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
from dagster_aws.ecs.executor import ecs_executor as ecs_executor
from dagster_aws.ecs.launcher import EcsRunLauncher as EcsRunLauncher
from dagster_aws.ecs.tasks import EcsEventualConsistencyTimeout as EcsEventualConsistencyTimeout
Loading

2 comments on commit 82e2189

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deploy preview for dagster-docs ready!

✅ Preview
https://dagster-docs-q9w0bghxn-elementl.vercel.app
https://master.dagster.dagster-docs.io

Built with commit 82e2189.
This pull request is being automatically deployed with vercel-action

@github-actions
Copy link

@github-actions github-actions bot commented on 82e2189 Dec 17, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deploy preview for dagster-docs-beta ready!

✅ Preview
https://dagster-docs-beta-2syysoxmr-elementl.vercel.app

Built with commit 82e2189.
This pull request is being automatically deployed with vercel-action

Please sign in to comment.