New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SFN] Support for Glue Optimised Service Integration #10802
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Nevertheless, I would prefer having a green test run before merging. Can we use companion branches to run the full test suite, in particular the new tests for this code in localstack-ext?
Re-basing both PRs (it seems the localstack community PR might be outdated causing issues in the ext pipeline) and pushing them simultaneously should work.
Just commented some (tiny) nits.
...pfunctions/asl/component/state/state_execution/state_task/service/state_task_service_glue.py
Outdated
Show resolved
Hide resolved
...pfunctions/asl/component/state/state_execution/state_task/service/state_task_service_glue.py
Outdated
Show resolved
Hide resolved
...pfunctions/asl/component/state/state_execution/state_task/service/state_task_service_glue.py
Show resolved
Hide resolved
925e41c
to
29e9f4d
Compare
877d977
to
0694d46
Compare
|
||
|
||
@pytest.fixture | ||
def sfn_events_to_sqs_queue(events_to_sqs_queue): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc @maxhoheiser We now have SFN fixtures in localstack, which can be useful for integrations in EventBridge and EventBridge Pipes 🥳
Motivation
Currently the SFN v2 provider does not support Glue optimised integrations. This PR adds base support for Glue optimised integration.
Closes: #10073 (I'm author of this issue)
Changes
Testing
Glue is a Pro feature, tests are added in the PR in tandem with this one.