Added MWAA Module #32
Conversation
|
@kukushking we should touch base on this. I don't see a requirements.txt reference and I am thinking that this will need a matching requirements? |
|
Thanks @dgraeber yes, this will need matching requirements file - @mahesh-balumuri is on top of this, working on a DAG and will add the requirements for it. |
|
Can you please add the example DAG module in README.MD table and update CHANGELOG.MD |
| @@ -0,0 +1,44 @@ | |||
| deploy: | |||
There was a problem hiding this comment.
This is a topic of advanced Seed-Farmer usage: to make this module generic we can leverage publishGenericEnvVariables. This will need a change of the naming conventions of the module parameters but will allow to use the module with other projects/deployments.
|
@mahesh-balumuri I just ran an E2E test and the DAG has succeeded. Great work! |
| @@ -0,0 +1,22 @@ | |||
| --constraint https://raw.githubusercontent.com/awslabs/aiops-modules/main/data/mwaa/requirements/constraint-requirements.txt | |||
There was a problem hiding this comment.
I'm not familiar with how constraints work but are we unable to reference the local file? I'm just worried that if we make a change to this requirements file and the constraints file in a dev branch, it will still be pulling the constraints file from main.
Signed-off-by: Anton Kukushkin <[email protected]>
Signed-off-by: Anton Kukushkin <[email protected]>
No description provided.