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.

Sign up for GitHub

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

Jump to bottom

Fix Typos and Source Paths in README #27

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

Fix Typos and Source Paths in README #27

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
6005b5d
Fix source URLs
jalessio Sep 7, 2017
e33395b
Fix typo: 'tenacy' => 'tenancy'
jalessio Sep 7, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,11 @@ The following code will yield a fully functioning VPC environment:

```js
module "vpc_base" {
source = "github.com/terraform-aws-vpc?ref=master//base"
source = "github.com/unifio/terraform-aws-vpc//base?ref=master"
}

module "az" {
source = "github.com/unifio/terraform-aws-vpc?ref=master//az"
source = "github.com/unifio/terraform-aws-vpc//az?ref=master"

vpc_id = "${module.vpc_base.vpc_id}"
```
Expand All @@ -37,7 +37,7 @@ Name | Type | Required | Description
`enable_dns` | string | Default: `true` | Specifies whether DNS resolution is supported for the VPC.
`enable_hostnames` | string | Default: `true` | Specifies whether the instances launched in the VPC get DNS hostnames.
`flow_log_traffic_type` | string | Default: `ALL` | The type of traffic to capture. Valid values: ACCEPT,REJECT,ALL.
`instance_tenacy` | string | Default: `default` | The allowed tenancy of instances launched into the VPC. Only other option at this time is `dedicated`, which will force any instance launched into the VPC to be dedicated, regardless of the tenancy option specified when the instance is launched.
`instance_tenancy` | string | Default: `default` | The allowed tenancy of instances launched into the VPC. Only other option at this time is `dedicated`, which will force any instance launched into the VPC to be dedicated, regardless of the tenancy option specified when the instance is launched.
`stack_item_label` | string | Default: `qckstrt` | Short form identifier for this stack. This value is used to create the "Name" tag for resources created by this stack item, and also serves as a unique key for re-use.
`stack_item_fullname` | string | Default: `VPC Quick Start` | Long form descriptive name for this stack item. This value is used to create the "application" tag for resources created by this stack item.
`vpc_cidr` | string | Default: `172.16.0.0/21` | The CIDR block you want the VPC to cover.
Expand All @@ -47,7 +47,7 @@ Name | Type | Required | Description

```js
module "vpc_base" {
source = "github.com/unifio/terraform-aws-vpc?ref=master//base"
source = "github.com/unifio/terraform-aws-vpc//base?ref=master"

enable_dns = "true"
enable_hostnames = "false"
Expand Down Expand Up @@ -175,11 +175,11 @@ Name | Type | Required | Description

```js
module "vpc_base" {
source = "github.com/terraform-aws-vpc?ref=master//base"
source = "github.com/terraform-aws-vpc//base?ref=master"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like we missed unifio here as well.

}

module "az" {
source = "github.com/unifio/terraform-aws-vpc?ref=master//az"
source = "github.com/unifio/terraform-aws-vpc//az?ref=master"

azs_provisioned = 2
enable_dmz_public_ips = "true"
Expand Down