Skip to content
This repository has been archived by the owner on Dec 9, 2024. It is now read-only.

Support concurrent activations #151

Closed
herzog31 opened this issue Jan 4, 2019 · 2 comments
Closed

Support concurrent activations #151

herzog31 opened this issue Jan 4, 2019 · 2 comments

Comments

@herzog31
Copy link
Contributor

herzog31 commented Jan 4, 2019

Recently concurrent activations were merged into OpenWhisk. For details, please check

As far as I understand, the concurrency is expected as part of the limits parameter and shout be treated similar to the memory and timeout parameters. See

limits: { timeout: params.Timeout * 1000, memory: params.MemorySize },

The scope of this issue is to support the usage of the concurrency parameter.

@jthomas
Copy link
Contributor

jthomas commented Jan 7, 2019

Thanks for letting us know @herzog31 - we should add this. This is a relatively simple fix - as the concurrency is controlled with the concurrency flag looking at the CLI PR.

Would you like to have a go at submitting the PR?

If you follow the code for setting the other limits, if should be a super-simple PR to open.

@herzog31
Copy link
Contributor Author

herzog31 commented Jan 7, 2019

@jthomas Yeah, I'll open a PR this week :)

@herzog31 herzog31 changed the title Support concurrent actions Support concurrent activations Jan 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants