Glue workflow

[bdoepf]

Community Note

• Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
• Please do not leave "+1" comments, they generate extra noise for pull request followers and do not help prioritize the request

Closes #10116

Right now the Glue Crawler can't be used in the workflow graph. Probably the trigger's action must be extended by the 'crawler_name' field.

Release note for CHANGELOG:

resource/aws_glue_workflow: Add glue workflow resource
resource/aws_glue_trigger: Add 'workflow_name' attribute, add 'crawler_name' attribute to 'action'

Output from acceptance testing:

$ make testacc TESTARGS='-run=TestAccAWSGlueWorkflow*'
=== RUN   TestAccAWSGlueWorkflow_Basic
=== PAUSE TestAccAWSGlueWorkflow_Basic
=== RUN   TestAccAWSGlueWorkflow_DefaultRunProperties
=== PAUSE TestAccAWSGlueWorkflow_DefaultRunProperties
=== RUN   TestAccAWSGlueWorkflow_Description
=== PAUSE TestAccAWSGlueWorkflow_Description
=== CONT  TestAccAWSGlueWorkflow_Basic
=== CONT  TestAccAWSGlueWorkflow_Description
=== CONT  TestAccAWSGlueWorkflow_DefaultRunProperties
--- PASS: TestAccAWSGlueWorkflow_DefaultRunProperties (29.23s)
--- PASS: TestAccAWSGlueWorkflow_Basic (29.39s)
--- PASS: TestAccAWSGlueWorkflow_Description (46.62s)
PASS
ok      github.com/terraform-providers/terraform-provider-aws/aws       47.858s

$ make testacc TESTARGS='-run=TestAccAWSGlueTrigger*'
=== RUN   TestAccAWSGlueTrigger_Basic
=== PAUSE TestAccAWSGlueTrigger_Basic
=== RUN   TestAccAWSGlueTrigger_Crawler
=== PAUSE TestAccAWSGlueTrigger_Crawler
=== RUN   TestAccAWSGlueTrigger_Description
=== PAUSE TestAccAWSGlueTrigger_Description
=== RUN   TestAccAWSGlueTrigger_Enabled
=== PAUSE TestAccAWSGlueTrigger_Enabled
=== RUN   TestAccAWSGlueTrigger_Predicate
=== PAUSE TestAccAWSGlueTrigger_Predicate
=== RUN   TestAccAWSGlueTrigger_Schedule
=== PAUSE TestAccAWSGlueTrigger_Schedule
=== RUN   TestAccAWSGlueTrigger_WorkflowName
=== PAUSE TestAccAWSGlueTrigger_WorkflowName
=== CONT  TestAccAWSGlueTrigger_Basic
=== CONT  TestAccAWSGlueTrigger_Predicate
=== CONT  TestAccAWSGlueTrigger_Crawler
=== CONT  TestAccAWSGlueTrigger_WorkflowName
=== CONT  TestAccAWSGlueTrigger_Description
=== CONT  TestAccAWSGlueTrigger_Enabled
=== CONT  TestAccAWSGlueTrigger_Schedule
--- PASS: TestAccAWSGlueTrigger_WorkflowName (48.18s)
--- PASS: TestAccAWSGlueTrigger_Basic (56.47s)
--- PASS: TestAccAWSGlueTrigger_Crawler (66.11s)
--- PASS: TestAccAWSGlueTrigger_Description (67.95s)
--- PASS: TestAccAWSGlueTrigger_Predicate (98.46s)
--- PASS: TestAccAWSGlueTrigger_Schedule (104.25s)
--- PASS: TestAccAWSGlueTrigger_Enabled (132.27s)
PASS
ok      github.com/terraform-providers/terraform-provider-aws/aws       133.497s

[bdoepf]

Updated the PR. Now also Glue Crawlers can be used in Glue Workflows.

[bflad]

This pull request overlaps #9762 and #10190 for the aws_glue_trigger updates -- I am going to try to merge each of these in order (addressing review feedback) to preserve the earlier contributions.

[bflad]

LGTM, thanks @bdoepf 🚀

Output from acceptance testing (after fixing up complex merge):

--- PASS: TestAccAWSGlueTrigger_WorkflowName (30.76s)
--- PASS: TestAccAWSGlueTrigger_Basic (50.13s)
--- PASS: TestAccAWSGlueTrigger_Description (54.13s)
--- PASS: TestAccAWSGlueTrigger_Predicate (65.53s)
--- PASS: TestAccAWSGlueTrigger_Schedule (79.04s)
--- PASS: TestAccAWSGlueTrigger_Crawler (85.59s)
--- PASS: TestAccAWSGlueTrigger_Enabled (117.36s)

--- PASS: TestAccAWSGlueWorkflow_DefaultRunProperties (13.18s)
--- PASS: TestAccAWSGlueWorkflow_Basic (13.70s)
--- PASS: TestAccAWSGlueWorkflow_Description (20.63s)

[ghost]

This has been released in version 2.39.0 of the Terraform AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template for triage. Thanks!

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!