diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index d8e4408..85792ef 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -29,7 +29,7 @@ We also recommend updating the automated tests *before* updating any code (see [ Development](https://en.wikipedia.org/wiki/Test-driven_development)). That means you add or update a test case, verify that it's failing with a clear error message, and *then* make the code changes to get that test to pass. This ensures the tests stay up to date and verify all the functionality in this Module, including whatever new -functionality you're adding in your contribution. Check out the [tests](/tests) folder for instructions on running the +functionality you're adding in your contribution. Check out the [tests](https://github.com/hashicorp/terraform-azurerm-vault/tree/master/tests) folder for instructions on running the automated tests. ## Update the code diff --git a/_docs/package-managers.md b/_docs/package-managers.md index 30ce499..6f46d0e 100644 --- a/_docs/package-managers.md +++ b/_docs/package-managers.md @@ -33,7 +33,7 @@ We need to write and package the scripts in this Module in a way that satisfies manual approval (e.g. a PR for each new version). - **Testable in dev mode**. We must be able to do local, iterative development on the example code in the - [examples](/examples) folder. That means there is a way to "package" these scripts so that, in dev mode, they are + [examples](https://github.com/hashicorp/terraform-azurerm-vault/tree/master/examples) folder. That means there is a way to "package" these scripts so that, in dev mode, they are downloaded from the local file system, rather than some package repo such as apt or yum. - **Mature**: We want to use a solution that is mature, battle-tested, and has an active community around it. diff --git a/modules/consul-cluster/README.md b/modules/consul-cluster/README.md index fd260d2..c35f938 100644 --- a/modules/consul-cluster/README.md +++ b/modules/consul-cluster/README.md @@ -122,7 +122,7 @@ Finally, you can try opening up the Consul UI in your browser at the URL `http:/ This module creates the following architecture: -![Consul architecture](/_docs/architecture.png) +![Consul architecture](https://github.com/hashicorp/terraform-azurerm-vault/tree/master/_docs/architecture.png) ## How do you roll out updates? diff --git a/modules/consul-cluster/outputs.tf b/modules/consul-cluster/outputs.tf index 44e50cf..0a88458 100644 --- a/modules/consul-cluster/outputs.tf +++ b/modules/consul-cluster/outputs.tf @@ -9,3 +9,11 @@ output "cluster_size" { output "load_balancer_ip_address" { value = "${azurerm_public_ip.consul_access.*.ip_address}" } + +output "load_balancer_id" { + value = "${azurerm_lb.consul_access.*.id}" +} + +output "backend_pool_id" { + value = "${azurerm_lb_backend_address_pool.consul_bepool.*.id}" +} diff --git a/modules/install-consul/README.md b/modules/install-consul/README.md index a8b1c0e..330143d 100644 --- a/modules/install-consul/README.md +++ b/modules/install-consul/README.md @@ -94,5 +94,5 @@ After the `install-consul` script finishes running, you may wish to do the follo We needed an easy way to install these scripts that satisfied a number of requirements, including working on a variety of operating systems and supported versioning. Our current solution is to use `git`, but this may change in the future. -See [Package Managers](/_docs/package-managers.md) for a full discussion of the requirements, trade-offs, and why we +See [Package Managers](https://github.com/hashicorp/terraform-azurerm-vault/tree/master/_docs/package-managers.md) for a full discussion of the requirements, trade-offs, and why we picked `git`. diff --git a/outputs.tf b/outputs.tf index dfb7c00..d1a7a78 100644 --- a/outputs.tf +++ b/outputs.tf @@ -12,4 +12,4 @@ output "load_balancer_ip_address_servers" { output "load_balancer_ip_address_clients" { value = "${module.consul_clients.load_balancer_ip_address}" -} \ No newline at end of file +}