Skip to content

Commit

Permalink
Add aws:elasticbeanstalk:container:nodejs NodeVersion settings (#50)
Browse files Browse the repository at this point in the history
* Add aws:elasticbeanstalk:container:nodejs NodeVersion variable and settings

* Fix new line on EOF

* Regenerate documentation. Fix variable description

* Fix linting in variables.tf
  • Loading branch information
bober2000 authored and aknysh committed Oct 12, 2018
1 parent 784bc31 commit ebc0da4
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 0 deletions.
2 changes: 2 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
<!-- This file was automatically generated by the `build-harness`. Make all changes to `README.yaml` and run `make readme` to rebuild this file. -->


[![Cloud Posse](https://cloudposse.com/logo-300x69.svg)](https://cloudposse.com)

# terraform-aws-elastic-beanstalk-environment
Expand Down Expand Up @@ -76,6 +77,7 @@ Available targets:
| loadbalancer_type | Load Balancer type, e.g. 'application' or 'classic' | string | `classic` | no |
| name | Solution name, e.g. 'app' or 'jenkins' | string | `app` | no |
| namespace | Namespace, which could be your organization name, e.g. 'eg' or 'cp' | string | - | yes |
| nodejs_version | Elastic Beanstalk NodeJS version to deploy | string | `` | no |
| notification_endpoint | Notification endpoint | string | `` | no |
| notification_protocol | Notification protocol | string | `email` | no |
| notification_topic_arn | Notification topic arn | string | `` | no |
Expand Down
1 change: 1 addition & 0 deletions docs/terraform.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@
| loadbalancer_type | Load Balancer type, e.g. 'application' or 'classic' | string | `classic` | no |
| name | Solution name, e.g. 'app' or 'jenkins' | string | `app` | no |
| namespace | Namespace, which could be your organization name, e.g. 'eg' or 'cp' | string | - | yes |
| nodejs_version | Elastic Beanstalk NodeJS version to deploy | string | `` | no |
| notification_endpoint | Notification endpoint | string | `` | no |
| notification_protocol | Notification protocol | string | `email` | no |
| notification_topic_arn | Notification topic arn | string | `` | no |
Expand Down
5 changes: 5 additions & 0 deletions main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -663,6 +663,11 @@ resource "aws_elastic_beanstalk_environment" "default" {
name = "InstanceRefreshEnabled"
value = "${var.instance_refresh_enabled}"
}
setting {
namespace = "aws:elasticbeanstalk:container:nodejs"
name = "NodeVersion"
value = "${var.nodejs_version}"
}
###===================== Application ENV vars ======================###
setting {
namespace = "aws:elasticbeanstalk:application:environment"
Expand Down
5 changes: 5 additions & 0 deletions variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -272,3 +272,8 @@ variable "version_label" {
default = ""
description = "Elastic Beanstalk Application version to deploy"
}

variable "nodejs_version" {
default = ""
description = "Elastic Beanstalk NodeJS version to deploy"
}

0 comments on commit ebc0da4

Please sign in to comment.