Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Commit

Permalink
Removed Invalid ECS Task Register Fields (#128)
Browse files Browse the repository at this point in the history
  • Loading branch information
anthony-desmier-axomic committed Oct 14, 2022
1 parent 11aedc4 commit 8197a92
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- Added `config` parameter to AwsClientParameters to support advanced configuration (e.g. accessing public S3 buckets) [#117](https://github.com/PrefectHQ/prefect-aws/pull/117)
- Add `@sync_compatible` to `S3Bucket` methods to allow calling them in sync contexts - [#119](https://github.com/PrefectHQ/prefect-aws/pull/119).
- Add `ECSTask.task_customizations` for customization of arbitary fields in the run task payload — [#120](https://github.com/PrefectHQ/prefect-aws/pull/120)
- Removed invalid ecs task register fields - [#126](https://github.com/PrefectHQ/prefect-aws/issues/126)

### Added

Expand Down
3 changes: 3 additions & 0 deletions prefect_aws/ecs.py
Original file line number Diff line number Diff line change
Expand Up @@ -913,6 +913,9 @@ def _register_task_definition(
task_definition_request.pop("taskDefinitionArn", None)
task_definition_request.pop("revision", None)
task_definition_request.pop("status", None)
task_definition_request.pop("requiresAttributes", None)
task_definition_request.pop("registeredAt", None)
task_definition_request.pop("registeredBy", None)

response = ecs_client.register_task_definition(**task_definition_request)
return response["taskDefinition"]["taskDefinitionArn"]
Expand Down

0 comments on commit 8197a92

Please sign in to comment.