search

DataDog / datadog-ci

Use Datadog from your CI.
https://datadoghq.com
Apache License 2.0
129 stars 55 forks source link

Handle another case when sfn->Lambda context injection is already set… #1470

Closed lym953 closed 1 month ago

lym953 commented 1 month ago

… up (case 4.2)

What and why?

If context injection is already set up using

"Payload.$": "$$['Execution', 'State', 'StateMachine']",

we currently treats it as custom Payload and print a warning message saying context injection will be skipped, which is wrong. This PR fixes it to print a message saying context injection is already set up.

How?

A brief description of implementation details of this PR.

Review checklist

datadog-datadog-prod-us1[bot] commented 1 month ago

Datadog Report

Branch report: yiming.luo/fix-step-func-16 Commit report: 2cc8717 Test service: datadog-ci-tests

:white_check_mark: 0 Failed, 436 Passed, 0 Skipped, 1m 30.39s Total duration (2m 5.08s time saved)