Skip to content

Commit

Permalink
[ecr] Fix Outputs (#35)
Browse files Browse the repository at this point in the history
* Disable default ecr

* Disable default ecr

* Disable default ecr

* Added add ecr

* Address PR comments

* Address Pr comments
  • Loading branch information
goruha committed Sep 4, 2018
1 parent 00087a1 commit fbf02a8
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 8 deletions.
12 changes: 8 additions & 4 deletions aws/ecr/ecr-repo-app.tf → aws/ecr/kops_ecr_app.tf
Original file line number Diff line number Diff line change
@@ -1,8 +1,12 @@
variable "kops_ecr_app_repository_name" {
description = "App repository name"
}

module "kops_ecr_app" {
source = "git::https://github.com/cloudposse/terraform-aws-kops-ecr.git?ref=tags/0.1.0"
source = "git::https://github.com/cloudposse/terraform-aws-kops-ecr.git?ref=tags/0.1.3"
namespace = "${var.namespace}"
stage = "${var.stage}"
name = "${var.repository_name}"
name = "${var.kops_ecr_app_repository_name}"
cluster_name = "${var.region}.${var.zone_name}"

users = [
Expand All @@ -20,12 +24,12 @@ output "kops_ecr_app_registry_id" {
}

output "kops_ecr_app_registry_url" {
value = "${module.kops_ecr_app.repository_url}"
value = "${module.kops_ecr_app.registry_url}"
description = "Registry app URL"
}

output "kops_ecr_app_repository_name" {
value = "${module.kops_ecr_app.name}"
value = "${module.kops_ecr_app.repository_name}"
description = "Registry app name"
}

Expand Down
File renamed without changes.
4 changes: 0 additions & 4 deletions aws/ecr/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,6 @@ variable "zone_name" {
description = "DNS zone name"
}

variable "repository_name" {
description = "Repository name"
}

provider "aws" {
assume_role {
role_arn = "${var.aws_assume_role_arn}"
Expand Down

0 comments on commit fbf02a8

Please sign in to comment.