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

add paramater for max_instances for gcp_cloudfunctions_cloud_function … #290

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

nolandseigler
Copy link

Module: gcp_cloudfunctions_cloud_function

SUMMARY

Add the ability to set the max_instances parameter when creating a cloud function.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

gcp_cloudfunctions_cloud_function

ADDITIONAL INFORMATION

maxInstances: https://cloud.google.com/functions/docs/reference/rest/v1/projects.locations.functions#CloudFunction.FIELDS.max_instances:~:text=projects.-,maxInstances,See%20the%20Max%20Instances%20Guide%20for%20more%20details.

modular-magician and others added 16 commits August 6, 2020 07:23
Signed-off-by: Modular Magician <magic-modules@google.com>
…nsible-collections#273)

* add source_image to google_compute_image

* add source_snapshot to google_compute_image

* PR comment changes

Signed-off-by: Modular Magician <magic-modules@google.com>
Signed-off-by: Modular Magician <magic-modules@google.com>
* Add new field filter to pubsub.

Fixes: hashicorp/terraform-provider-google#6727

* Fixed filter name, it was improperly set.

* add filter key to pubsub subscription unit test

* spaces not tabs!

* hardcode filter value in test

* revert remove escaped quotes

Co-authored-by: Tim O'Connell <hi@timoconnell.co.uk>
Signed-off-by: Modular Magician <magic-modules@google.com>

Co-authored-by: Tim O'Connell <hi@timoconnell.co.uk>
Signed-off-by: Modular Magician <magic-modules@google.com>
Signed-off-by: Modular Magician <magic-modules@google.com>
…llections#278)

Add enableMessageOrdering to Pub/Sub Subscription

Signed-off-by: Modular Magician <magic-modules@google.com>
* use {product}.googleapis.com endpoints

* use actual correct urls

* fix zone data source test

* fix network peering tests

* possibly fix deleting default network

Signed-off-by: Modular Magician <magic-modules@google.com>
…le-collections#280)

* [provider-yaml] - Removed instances where input and output are both true

* modified to only supply output. Following pattern from bigquerydatatransfer

Co-authored-by: Scott Suarez <scottsuarez@google.com>
Signed-off-by: Modular Magician <magic-modules@google.com>

Co-authored-by: Scott Suarez <scottsuarez@google.com>
* retrypolicy attribute added

* test case updated

Signed-off-by: Modular Magician <magic-modules@google.com>
…nsible-collections#282)

Co-authored-by: Dana Hoffman <danahoffman@google.com>
Signed-off-by: Modular Magician <magic-modules@google.com>

Co-authored-by: Dana Hoffman <danahoffman@google.com>
…tions#283)

Co-authored-by: Cameron Thornton <camthornton@google.com>
Signed-off-by: Modular Magician <magic-modules@google.com>

Co-authored-by: Cameron Thornton <camthornton@google.com>
…s#286)

* added storage locations

* add storage locations to field

* tweak cmek logic

* fix the decoder logic and cleanup whitespaces

* remove duplicate entry

Signed-off-by: Modular Magician <magic-modules@google.com>
…e-collections#287)

Signed-off-by: Modular Magician <magic-modules@google.com>
…ol (#3973) (ansible-collections#289)

Co-authored-by: Edward Sun <sunedward@google.com>
Signed-off-by: Modular Magician <magic-modules@google.com>

Co-authored-by: Edward Sun <sunedward@google.com>
@nolandseigler nolandseigler changed the title add paramater for max_instancesfor gcp_cloudfunctions_cloud_function … add paramater for max_instances for gcp_cloudfunctions_cloud_function … Sep 13, 2020
@b-long
Copy link

b-long commented Oct 5, 2020

Hi friends,

I'd love to see this feature, max instances parameter exposed to the Ansible gcp_cloudfunctions_cloud_function module, land.

Does anyone have any feedback that might help this move forward?

cc/ @thedoubl3j @rambleraptor @chrisst

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants