Skip to content

set timeout in seconds in container examples #145

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

Merged
merged 1 commit into from
Mar 3, 2023
Merged

set timeout in seconds in container examples #145

merged 1 commit into from
Mar 3, 2023

Conversation

alelapeyre
Copy link
Contributor

Hello!

I got the following error when using the container example with the timeout value set:

→ sls deploy                                                                                                                                                                                         
...
Environment: linux, node 19.7.0, framework 3.16.0, plugin 6.2.2, SDK 4.3.2
Docs:        docs.serverless.com
Support:     forum.serverless.com
Bugs:        github.com/serverless/serverless/issue

Error:
Error: proto: syntax error (line 1:183): unexpected token 20000
    at manageError (<path>/node_modules/serverless-scaleway-functions/shared/api/utils.js:23:11)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Promise.all (index 0)

Searching a bit, I found that the timeout value should be provided in seconds. So here is a small PR to update examples containing the timeout field with a valid value.

@thomas-tacquet thomas-tacquet self-requested a review March 3, 2023 10:51
@thomas-tacquet
Copy link
Contributor

Hello 👋

Cool nice find thank your for your contribution we really appreciate it👌

@thomas-tacquet thomas-tacquet merged commit f435212 into scaleway:master Mar 3, 2023
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

Successfully merging this pull request may close these issues.

2 participants