Update ECS sample DAG and Docs to new standards #21828
Conversation
boring-cyborg
bot
added
area:providers
kind:documentation
provider:amazon-aws
| """ | ||
| This is an example dag for EcsOperator. | ||
|
|
||
| The task "hello_world" runs `hello-world` task in `c` cluster. | ||
| It overrides the command in the `hello-world-container` container. | ||
| """ |
There was a problem hiding this comment.
Is there a reason to remove it?
Since we have dag.doc_md = __doc__ this explanatory block will appear in the UI
There was a problem hiding this comment.
It's the only sample DAG in the AWS provider package that had it, and I didn't see any added value in this instance.
Discussed it a little on Slack here: https://apache-airflow.slack.com/archives/CJ1LVREHX/p1644527546510599****
github-actions
bot
added
the
okay to merge
|
The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase to the latest main or amend the last commit of the PR, and push it with --force-with-lease. |
Part of a project to simplify and standardize AWS sample dags and docs in preparation for adding System Testing.
Related: #21523
Related: #21475
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.