- Notifications
You must be signed in to change notification settings - Fork 1.1k
Add support for maxInstances when creating a function #1998
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
Conversation
| Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here with What to do if you already signed the CLAIndividual signers
Corporate signers
ℹ️ Googlers: Go here for more info. |
| @googlebot I signed it! |
| CLAs look good, thanks! ℹ️ Googlers: Go here for more info. |
| @efiShtain thanks for sending this! Just a few things to know:
|
| @samtstern I'll see if I can fill in the rest of the details There is a matching pull request on the firebase-functions too Didn't know how to link them together, I also didn't change the package version as I didn't know what's the process regarding that too |
| @efiShtain oh thanks! I just saw your other PR: So that one will have to be first, I'll get someone to take a look. |
| @efiShtain a few scenarios you might want to test:
Oh and don't worry about the version number, our release script does that automatically. |
| @samtstern |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code here is LGTM, we'll just need to wait for this to go through API review. Thanks a ton for contributing!
| Hey @efiShtain, we just added this proposal to our 3/19 API review meeting. Thanks so much for working on this - I'll update here again after the meeting. |
| Wooo, this was approved! Thanks agin for working on this! |
Description
This is a feature added to the public api to allow passing maxInstances property when creating or updating a firebase function.
This change requires a matching change on firebase-functions (there is another pull request for that)
firebase/firebase-functions#624
Scenarios Tested
No special testing is needed, as the parameter added is optional.
If you need to test it, create a function using firebase-functions and pass maxInstances parameter to RuntimeOptions.
The function on google console will have the maxInstances parameter set correctly
It Cannot be viewed on the firebase console as far as I know
Sample Commands