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

Why domains is required for kubernetes module? #182

Open
RustamGimadiev opened this issue Jul 30, 2021 · 2 comments
Open

Why domains is required for kubernetes module? #182

RustamGimadiev opened this issue Jul 30, 2021 · 2 comments

Comments

@RustamGimadiev
Copy link
Contributor

Describe the bug
Unable use sak-kubernetes module without providing domains for docker proxy

╷
│ Error: Invalid index
│
│   on .terraform/modules/kubernetes/local.tf line 2, in locals:
│    2:   registry = "https://registry.${var.domains[0]}"
│     ├────────────────
│     │ var.domains is empty list of string
│
│ The given key does not identify an element in this collection value: the collection has no elements.

Use version
I use sak-kuberentes from master branch (provectus/sak-kubernetes@e6cf16d)

To Reproduce

module "kubernetes" {
  source = "github.com/provectus/sak-kubernetes?ref=e6cf16d2d0d86b5fc0d61ed431c5e8041471de1f"

  environment        = local.environment
  project            = local.project
  availability_zones = data.aws_availability_zones.current.zone_ids
  cluster_name       = "${local.environment}-${local.project}"
  vpc_id             = module.network.vpc_id
  subnets            = module.network.private_subnets
}
@akastav
Copy link
Contributor

akastav commented Jul 30, 2021

We need to use ingress because the registry required HTTPS connections. You can try to add insecure options for dockerconfig in kubernetes modules and use internal domain (http://registry..svc.cluster.local)

@RustamGimadiev
Copy link
Contributor Author

maybe it is better to leave such values as default?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants