Skip to content

Conversation

@Amplifiyer
Copy link
Contributor

Problem

After the recent change of #2438, health checks are failing as lambda execution is failing with error

"TypeError [ERR_INVALID_ARG_TYPE]: The \"callback\" argument must be of type function. Received undefined",
        "    at setInterval (node:timers:212:3)",
        "    at file:///var/task/index.mjs:1:1769",
        "    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)"

I tried changing it to use IIFE as well and then instead I got the following error from lambda

  "TypeError [ERR_INVALID_ARG_TYPE]: The \"callback\" argument must be of type function. Received an instance of Promise",
        "    at setInterval (node:timers:212:3)",
        "    at file:///var/task/index.mjs:1:1769",
        "    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)"

Issue number, if available:

Changes

Reverting the workaround and turning off eslint to unblock the pipeline

Corresponding docs PR, if applicable:

Validation

Checklist

  • If this PR includes a functional change to the runtime behavior of the code, I have added or updated automated test coverage for this change.
  • If this PR requires a change to the Project Architecture README, I have included that update in this PR.
  • If this PR requires a docs update, I have linked to that docs PR above.
  • If this PR modifies E2E tests, makes changes to resource provisioning, or makes SDK calls, I have run the PR checks with the run-e2e label set.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@Amplifiyer Amplifiyer added the run-e2e Label that will include e2e tests in PR checks workflow label Jan 23, 2025
@Amplifiyer Amplifiyer requested a review from a team as a code owner January 23, 2025 01:12
@changeset-bot
Copy link

changeset-bot bot commented Jan 23, 2025

🦋 Changeset detected

Latest commit: 21756b7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@aws-amplify/backend-function Patch
@aws-amplify/backend Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

sobolk
sobolk previously approved these changes Jan 23, 2025
@rtpascual rtpascual merged commit 501ec56 into main Jan 23, 2025
95 checks passed
@rtpascual rtpascual deleted the ssm_refresh_regression branch January 23, 2025 02:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-e2e Label that will include e2e tests in PR checks workflow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants