Skip to content

Commit

Permalink
Merge pull request #5 from sparkfabrik/fix/remove-helm-provider
Browse files Browse the repository at this point in the history
remove helm provider config
  • Loading branch information
Monska85 authored Dec 5, 2023
2 parents 5fbd9e4 + 3ae0285 commit f3ac465
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 28 deletions.
8 changes: 8 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,14 @@ to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

---

## [0.2.1] - 2023-12-05

[Compare with previous version](https://github.com/sparkfabrik/terraform-helm-ingress-nginx/compare/0.2.0...0.2.1)

### Changed

- Removed `helm` provider config and variables.

## [0.2.0] - 2023-12-05

[Compare with previous version](https://github.com/sparkfabrik/terraform-helm-ingress-nginx/compare/0.1.0...0.2.0)
Expand Down
3 changes: 0 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -25,13 +25,10 @@ This is Terraform module to install and configure the Nginx Ingress Controller.
|------|-------------|------|---------|:--------:|
| <a name="input_additional_values"></a> [additional\_values](#input\_additional\_values) | Additional values to pass to the helm chart | `list(string)` | `[]` | no |
| <a name="input_chart_version"></a> [chart\_version](#input\_chart\_version) | Chart version of the ingress controller | `string` | n/a | yes |
| <a name="input_cluster_ca_certificate"></a> [cluster\_ca\_certificate](#input\_cluster\_ca\_certificate) | PEM-encoded root certificates bundle for TLS authentication | `string` | n/a | yes |
| <a name="input_create_namespace"></a> [create\_namespace](#input\_create\_namespace) | Create namespace for the ingress controller. If false, the namespace must be created before using this module | `bool` | `true` | no |
| <a name="input_helm_release_name"></a> [helm\_release\_name](#input\_helm\_release\_name) | Name of the helm release | `string` | `"ingress-nginx"` | no |
| <a name="input_host"></a> [host](#input\_host) | The hostname (in form of URI) of Kubernetes master | `string` | n/a | yes |
| <a name="input_namespace"></a> [namespace](#input\_namespace) | Namespace of the ingress controller | `string` | n/a | yes |
| <a name="input_namespace_additional_labels"></a> [namespace\_additional\_labels](#input\_namespace\_additional\_labels) | Additional labels for the namespace of the ingress controller | `map(string)` | `{}` | no |
| <a name="input_token"></a> [token](#input\_token) | Token to authenticate a service account | `string` | n/a | yes |

## Outputs

Expand Down
3 changes: 0 additions & 3 deletions examples/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,4 @@ module "ingress_nginx" {
source = "../"
chart_version = "4.8.3"
namespace = "ingress-nginx"
host = "https://test"
cluster_ca_certificate = base64decode("certificate")
token = "token"
}
7 changes: 0 additions & 7 deletions providers.tf
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,3 @@ terraform {
}
}
}
provider "helm" {
kubernetes {
host = var.host
cluster_ca_certificate = var.cluster_ca_certificate
token = var.token
}
}
15 changes: 0 additions & 15 deletions variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -20,21 +20,6 @@ variable "namespace_additional_labels" {
default = {}
}

variable "host" {
description = "The hostname (in form of URI) of Kubernetes master"
type = string
}

variable "cluster_ca_certificate" {
description = "PEM-encoded root certificates bundle for TLS authentication"
type = string
}

variable "token" {
description = "Token to authenticate a service account"
type = string
}

variable "additional_values" {
description = "Additional values to pass to the helm chart"
type = list(string)
Expand Down

0 comments on commit f3ac465

Please sign in to comment.