Skip to content
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

SAR versioning is behind #250

Open
jeremiahlukus opened this issue May 25, 2024 · 3 comments
Open

SAR versioning is behind #250

jeremiahlukus opened this issue May 25, 2024 · 3 comments
Assignees

Comments

@jeremiahlukus
Copy link

{
  "name": "Branching",
  "output": {
    "lambdaARN": "arn:aws:lambda:us-east-1:blah:function:staging-controller",
    "powerValues": [
      128,
      256,
      512,
      1024,
      2048,
      3008
    ],
    "num": 5,
    "payload": {},
    "parallelInvocation": false,
    "strategy": "cost",
    "error": {
      "Error": "SyntaxError",
      "Cause": "{\"errorType\":\"SyntaxError\",\"errorMessage\":\"Unexpected token 'W', \\\"WARN: hand\\\"... is not valid JSON\",\"trace\":[\"SyntaxError: Unexpected token 'W', \\\"WARN: hand\\\"... is not valid JSON\",\"    at JSON.parse (<anonymous>)\",\"    at /var/task/utils.js:551:54\",\"    at Array.map (<anonymous>)\",\"    at module.exports.extractDurationFromJSON (/var/task/utils.js:551:35)\",\"    at module.exports.extractDuration (/var/task/utils.js:528:22)\",\"    at /var/task/utils.js:472:22\",\"    at Array.map (<anonymous>)\",\"    at module.exports.parseLogAndExtractDurations (/var/task/utils.js:470:17)\",\"    at computeStatistics (/var/task/executor.js:180:29)\",\"    at module.exports.handler (/var/task/executor.js:71:12)\"]}"
    }
  },
  "outputDetails": {
    "truncated": false
  }
}

Created using the aws GUI

https://serverlessrepo.aws.amazon.com/applications/arn:aws:serverlessrepo:us-east-1:451282441545:applications~aws-lambda-power-tuning

@jeremiahlukus
Copy link
Author

last release was 4.3.4 (2024-02-26): maybe doing a new release will fix the issue?

@jeremiahlukus
Copy link
Author

After using sam to deploy i no longer get this error. I think just doing another release to AWS would fix.

@alexcasalboni
Copy link
Owner

Hi @jeremiahlukus thanks for flagging this!

We are aware that SAR versioning is a bit behind - we're waiting to get a major release out for #206 (something like 5.0.0). Apologies for the trouble, but glad you managed to use SAM to deploy the latest in the meantime.

I'm keeping this issue open for now, even though the bug has been resolved already. Will share an update and close once the new SAR version is out.

@alexcasalboni alexcasalboni changed the title Error on execute SAR versioning is behind May 27, 2024
@alexcasalboni alexcasalboni self-assigned this May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants