Skip to content
This repository was archived by the owner on Apr 14, 2023. It is now read-only.

Conversation

@gravesm
Copy link

@gravesm gravesm commented Oct 21, 2019

This adds a custom Airflow plugin that includes an operator and sensor
that can be used together to run completely isolated workflows. There's
an example workflow included in the root of this repo to show how it
works.

How can a reviewer see these changes?

There's an example task definition in Fargate called airflow-stage-example-task. If you want to add a workflow that uses this task, all it does is start up an alpine container, print the date and then exit.

Reviewer Checklist

  • The commit message is clear and follows our guidelines
  • There are tests covering any new functionality
  • The documentation has been updated if necessary
  • The changes, if applicable, have been verified

This adds a custom Airflow plugin that includes an operator and sensor
that can be used together to run completely isolated workflows. There's
an example workflow included in the root of this repo to show how it
works.
json.loads(base64.b64decode(os.getenv('ECS_NETWORK_CONFIG')))

to retrieve the network configuration. The ``task_definition`` will need to be manually configured after the task has been created in Terraform. There is an example workflow (``example_ecs.py``) in the root of this repo.
Copy link
Member

Choose a reason for hiding this comment

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

I don't see any associated work in terraform. Did this not get pushed? It isn't blocking on this work, but I was just trying to better understand the bigger picture.

Copy link
Author

Choose a reason for hiding this comment

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

Oh, yeah, I haven't pushed it yet. I'll do that shortly.

@gravesm gravesm merged commit 91bc258 into master Oct 22, 2019
@gravesm gravesm deleted the plugin branch October 22, 2019 14:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants