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

Fix Apigee PAYG environment node config #1024

Conversation

g-greatdevaks
Copy link
Collaborator

Fixed the Node Configuration node_config for Apigee PAYG specific Environments.

Removed current_aggregate_node_count as it is a read-only field and cannot be taken as an input for the Apigee Environment Terraform resource.

Changes committed:

  • modified: blueprints/cloud-operations/apigee/README.md
  • modified: blueprints/cloud-operations/apigee/variables.tf
  • modified: modules/apigee/README.md
  • modified: modules/apigee/main.tf
  • modified: modules/apigee/variables.tf
  • modified: tests/blueprints/cloud_operations/apigee/fixture/variables.tf
  • modified: tests/modules/apigee/fixture/variables.tf

Fixed the Node Configuration for Apigee PAYG
specific Environments.

Removed current_aggregate_node_count as it
is a read-only field and cannot be taken
as an input for the Apigee Environment Terraform
resource.

Changes committed:
modified:   blueprints/cloud-operations/apigee/README.md
modified:   blueprints/cloud-operations/apigee/variables.tf
modified:   modules/apigee/README.md
modified:   modules/apigee/main.tf
modified:   modules/apigee/variables.tf
modified:   tests/blueprints/cloud_operations/apigee/fixture/variables.tf
modified:   tests/modules/apigee/fixture/variables.tf
@juliocc juliocc changed the title FIX APIGEE PAYG ENVIRONMENT NODE CONFIG Fix Apigee PAYG environment node config Nov 29, 2022
@juliocc juliocc requested review from apichick and danistrebel and removed request for apichick November 29, 2022 11:41
@juliocc
Copy link
Collaborator

juliocc commented Nov 29, 2022

LGTM, @danistrebel @apichick can you TAL?

@apichick apichick merged commit d9edb8c into GoogleCloudPlatform:master Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants