Skip to content

Commit

Permalink
feat: 🔧 Remove unused code
Browse files Browse the repository at this point in the history
  • Loading branch information
afranzi committed Mar 4, 2024
1 parent 0462720 commit f498352
Show file tree
Hide file tree
Showing 13 changed files with 29 additions and 69 deletions.
6 changes: 4 additions & 2 deletions docs/helms/airflow.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
!!! tip "Community Chart"
In our project, we use the [Airflow Community Helm chart](https://artifacthub.io/packages/helm/airflow-helm/airflow)
to deploy Airflow on our Minikube cluster.

This chart is our favorite one because it's flexible and easy to use, making it perfect for getting Airflow up
running in a Kubernetes environment like ours.

Expand All @@ -17,6 +17,9 @@ For the actual deployment into our Kubernetes cluster, we're using :simple-argo:
It's a tool that helps us deploy applications automatically, following the best practices of GitOps. This means we can
manage our Airflow setup with code, making changes easily and keeping everything up to date without hassle.

Using ArgoCD not only makes our lives easier by automating deployment tasks but also keeps our project tidy and
well-organized. It's a smart way to handle deployments, giving us more time to focus on making our data platform better.

![Airflow Argo deployment](../images/airflow-argocd.png)

The most important properties when defining our Airflow values are:
Expand Down Expand Up @@ -114,4 +117,3 @@ The most important properties when defining our Airflow values are:
| airflow.workers.replicas | int | `1` | |
| airflow.workers.resources.requests.cpu | string | `"256m"` | |
| airflow.workers.resources.requests.memory | string | `"1Gi"` | |

2 changes: 1 addition & 1 deletion docs/roadmap.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,4 +20,4 @@
- [ ] DBT [Elementary](https://www.elementary-data.com/) Report Website
* Showcase how to host and provide a static website inside K8s.

- [ ] K8s Certificates: Solve `Not Secure` issue with nginx without due fake certificates.
- [ ] K8s Certificates: Solve `Not Secure` issue with nginx without due fake certificates.
2 changes: 2 additions & 0 deletions docs/terraform/main.md
Original file line number Diff line number Diff line change
Expand Up @@ -94,8 +94,10 @@ No providers.
| Name | Source | Version |
|------|--------|---------|
| <a name="module_application"></a> [application](#module\_application) | ../modules/k8s/argocd/application | n/a |
| <a name="module_application_db"></a> [application\_db](#module\_application\_db) | ../modules/k8s/argocd/application | n/a |
| <a name="module_argocd"></a> [argocd](#module\_argocd) | ../modules/k8s/argocd/server | n/a |
| <a name="module_cluster"></a> [cluster](#module\_cluster) | ../modules/k8s/minicluster | n/a |
| <a name="module_namespace"></a> [namespace](#module\_namespace) | ../modules/k8s/namespace | n/a |
| <a name="module_project"></a> [project](#module\_project) | ../modules/k8s/argocd/project | n/a |

## Resources
Expand Down
8 changes: 4 additions & 4 deletions docs/terraform/modules/k8s/argocd/application.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,14 +5,12 @@
|------|---------|
| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 1.7.4 |
| <a name="requirement_argocd"></a> [argocd](#requirement\_argocd) | 6.0.3 |
| <a name="requirement_kubernetes"></a> [kubernetes](#requirement\_kubernetes) | ~> 2.26.0 |

## Providers

| Name | Version |
|------|---------|
| <a name="provider_argocd"></a> [argocd](#provider\_argocd) | 6.0.3 |
| <a name="provider_kubernetes"></a> [kubernetes](#provider\_kubernetes) | ~> 2.26.0 |

## Modules

Expand All @@ -23,22 +21,24 @@ No modules.
| Name | Type |
|------|------|
| [argocd_application.apps](https://registry.terraform.io/providers/oboukili/argocd/6.0.3/docs/resources/application) | resource |
| [kubernetes_namespace.namespace](https://registry.terraform.io/providers/hashicorp/kubernetes/latest/docs/resources/namespace) | resource |

## Inputs

| Name | Description | Type | Default | Required |
|------|-------------|------|---------|:--------:|
| <a name="input_argocd_namespace"></a> [argocd\_namespace](#input\_argocd\_namespace) | Namespace where the ArgoCD server has been deployed | `string` | n/a | yes |
| <a name="input_chart"></a> [chart](#input\_chart) | Repository path with helm | `string` | `null` | no |
| <a name="input_cluster_name"></a> [cluster\_name](#input\_cluster\_name) | Cluster url | `string` | `"https://kubernetes.default.svc"` | no |
| <a name="input_history_limit"></a> [history\_limit](#input\_history\_limit) | History limit | `number` | `3` | no |
| <a name="input_name"></a> [name](#input\_name) | Application name | `string` | n/a | yes |
| <a name="input_namespace"></a> [namespace](#input\_namespace) | K8s namespace to deploy application | `string` | n/a | yes |
| <a name="input_path"></a> [path](#input\_path) | Repository path with helm | `string` | `"helm"` | no |
| <a name="input_parameters"></a> [parameters](#input\_parameters) | Helm parameters which are passed to the helm template command upon manifest generation | `map(string)` | `{}` | no |
| <a name="input_path"></a> [path](#input\_path) | Repository path with helm | `string` | `""` | no |
| <a name="input_project_name"></a> [project\_name](#input\_project\_name) | ArgoCD Project name | `string` | n/a | yes |
| <a name="input_repo_url"></a> [repo\_url](#input\_repo\_url) | Repository URL | `string` | n/a | yes |
| <a name="input_target_revision"></a> [target\_revision](#input\_target\_revision) | Target revision to retrieve from Git | `string` | `"HEAD"` | no |
| <a name="input_value_files"></a> [value\_files](#input\_value\_files) | Helm value files | `list(string)` | <pre>[<br> "values.yaml"<br>]</pre> | no |
| <a name="input_values"></a> [values](#input\_values) | Helm values which are passed to the helm template command upon manifest generation | `any` | `null` | no |

## Outputs

Expand Down
6 changes: 4 additions & 2 deletions helms/airflow/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
!!! tip "Community Chart"
In our project, we use the [Airflow Community Helm chart](https://artifacthub.io/packages/helm/airflow-helm/airflow)
to deploy Airflow on our Minikube cluster.

This chart is our favorite one because it's flexible and easy to use, making it perfect for getting Airflow up
running in a Kubernetes environment like ours.

Expand All @@ -17,6 +17,9 @@ For the actual deployment into our Kubernetes cluster, we're using :simple-argo:
It's a tool that helps us deploy applications automatically, following the best practices of GitOps. This means we can
manage our Airflow setup with code, making changes easily and keeping everything up to date without hassle.

Using ArgoCD not only makes our lives easier by automating deployment tasks but also keeps our project tidy and
well-organized. It's a smart way to handle deployments, giving us more time to focus on making our data platform better.

![Airflow Argo deployment](../images/airflow-argocd.png)

The most important properties when defining our Airflow values are:
Expand Down Expand Up @@ -114,4 +117,3 @@ The most important properties when defining our Airflow values are:
| airflow.workers.replicas | int | `1` | |
| airflow.workers.resources.requests.cpu | string | `"256m"` | |
| airflow.workers.resources.requests.memory | string | `"1Gi"` | |

7 changes: 3 additions & 4 deletions helms/airflow/README.md.gotmpl
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,9 @@

!!! tip "Community Chart"
In our project, we use the [Airflow Community Helm chart](https://artifacthub.io/packages/helm/airflow-helm/airflow)
to deploy Airflow on our Minikube cluster.
This chart is our favorite one because it's flexible and easy to use, making it perfect for getting Airflow up
to deploy Airflow on our Minikube cluster.

This chart is our favorite one because it's flexible and easy to use, making it perfect for getting Airflow up
running in a Kubernetes environment like ours.

---
Expand Down Expand Up @@ -52,4 +52,3 @@ The most important properties when defining our Airflow values are:
{{ template "chart.requirementsSection" . }}

{{ template "chart.valuesSection" . }}

2 changes: 2 additions & 0 deletions terraform/main/readme.md
Original file line number Diff line number Diff line change
Expand Up @@ -94,8 +94,10 @@ No providers.
| Name | Source | Version |
|------|--------|---------|
| <a name="module_application"></a> [application](#module\_application) | ../modules/k8s/argocd/application | n/a |
| <a name="module_application_db"></a> [application\_db](#module\_application\_db) | ../modules/k8s/argocd/application | n/a |
| <a name="module_argocd"></a> [argocd](#module\_argocd) | ../modules/k8s/argocd/server | n/a |
| <a name="module_cluster"></a> [cluster](#module\_cluster) | ../modules/k8s/minicluster | n/a |
| <a name="module_namespace"></a> [namespace](#module\_namespace) | ../modules/k8s/namespace | n/a |
| <a name="module_project"></a> [project](#module\_project) | ../modules/k8s/argocd/project | n/a |

## Resources
Expand Down
7 changes: 4 additions & 3 deletions terraform/modules/k8s/argocd/application/00-variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,7 @@ variable "parameters" {
}

variable "values" {
type = any
default = null
}
type = any
default = null
description = "Helm values which are passed to the helm template command upon manifest generation"
}
5 changes: 0 additions & 5 deletions terraform/modules/k8s/argocd/application/providers.tf
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,5 @@ terraform {
source = "oboukili/argocd"
version = "6.0.3"
}
# https://registry.terraform.io/providers/hashicorp/kubernetes
kubernetes = {
version = "~> 2.26.0"
source = "hashicorp/kubernetes"
}
}
}
8 changes: 4 additions & 4 deletions terraform/modules/k8s/argocd/application/readme.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,14 +5,12 @@
|------|---------|
| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 1.7.4 |
| <a name="requirement_argocd"></a> [argocd](#requirement\_argocd) | 6.0.3 |
| <a name="requirement_kubernetes"></a> [kubernetes](#requirement\_kubernetes) | ~> 2.26.0 |

## Providers

| Name | Version |
|------|---------|
| <a name="provider_argocd"></a> [argocd](#provider\_argocd) | 6.0.3 |
| <a name="provider_kubernetes"></a> [kubernetes](#provider\_kubernetes) | ~> 2.26.0 |

## Modules

Expand All @@ -23,22 +21,24 @@ No modules.
| Name | Type |
|------|------|
| [argocd_application.apps](https://registry.terraform.io/providers/oboukili/argocd/6.0.3/docs/resources/application) | resource |
| [kubernetes_namespace.namespace](https://registry.terraform.io/providers/hashicorp/kubernetes/latest/docs/resources/namespace) | resource |

## Inputs

| Name | Description | Type | Default | Required |
|------|-------------|------|---------|:--------:|
| <a name="input_argocd_namespace"></a> [argocd\_namespace](#input\_argocd\_namespace) | Namespace where the ArgoCD server has been deployed | `string` | n/a | yes |
| <a name="input_chart"></a> [chart](#input\_chart) | Repository path with helm | `string` | `null` | no |
| <a name="input_cluster_name"></a> [cluster\_name](#input\_cluster\_name) | Cluster url | `string` | `"https://kubernetes.default.svc"` | no |
| <a name="input_history_limit"></a> [history\_limit](#input\_history\_limit) | History limit | `number` | `3` | no |
| <a name="input_name"></a> [name](#input\_name) | Application name | `string` | n/a | yes |
| <a name="input_namespace"></a> [namespace](#input\_namespace) | K8s namespace to deploy application | `string` | n/a | yes |
| <a name="input_path"></a> [path](#input\_path) | Repository path with helm | `string` | `"helm"` | no |
| <a name="input_parameters"></a> [parameters](#input\_parameters) | Helm parameters which are passed to the helm template command upon manifest generation | `map(string)` | `{}` | no |
| <a name="input_path"></a> [path](#input\_path) | Repository path with helm | `string` | `""` | no |
| <a name="input_project_name"></a> [project\_name](#input\_project\_name) | ArgoCD Project name | `string` | n/a | yes |
| <a name="input_repo_url"></a> [repo\_url](#input\_repo\_url) | Repository URL | `string` | n/a | yes |
| <a name="input_target_revision"></a> [target\_revision](#input\_target\_revision) | Target revision to retrieve from Git | `string` | `"HEAD"` | no |
| <a name="input_value_files"></a> [value\_files](#input\_value\_files) | Helm value files | `list(string)` | <pre>[<br> "values.yaml"<br>]</pre> | no |
| <a name="input_values"></a> [values](#input\_values) | Helm values which are passed to the helm template command upon manifest generation | `any` | `null` | no |

## Outputs

Expand Down
2 changes: 1 addition & 1 deletion terraform/modules/k8s/namespace/99-outputs.tf
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
output "name" {
description = "Namespace name"
value = kubernetes_namespace.namespace.metadata[0].name
}
}
28 changes: 0 additions & 28 deletions terraform/modules/k8s/postgres/db/00-variables.tf

This file was deleted.

15 changes: 0 additions & 15 deletions terraform/modules/k8s/postgres/db/providers.tf

This file was deleted.

0 comments on commit f498352

Please sign in to comment.