fix: Update helm-docs (airbytehq#16531)

* fix: revert extraEnv. Move updated extraEnv to env_vars

* fix: Add node pool selector feature for gke deployments

* fix: hotfix. Fix webapp values and imagepullsecrets

* fix: remove nodeSelector duplicate

* fix: revert imagePullSecrets changes

* fix nit

* fix: fix bootloader imagepullsecrets

* fix: fix image pull secrets again...

* feat: add acceptance tests for helm

* fix: add step to install unzip

* fix incorrect command spelling(helm)

* fix: update slack notification sending for Airbyte CI workflow

* fix: Rename Helm E2E test report name

* fix: Update helm charts

* fix: fix temporal DB Port bug

* fix nits

* fix: do a small testing for sake of science

* fix: Update test chart.yaml with note block

* fix: Revert changes for AC tests

* fix: Include airbytehq#16166

* fix: add metrics to the ci workflow

* fix: revert changes for temporal service. move them to server service;

* fix: revert fullstory deletion

* fix: change way of defining image tag and make it to be set in global values. Add values.yaml.test for integration testing and local deployment

* fix: udpate docs, update values.yaml

* fix: fix helpers.tpl in all subcharts

* Update chart docs

* fix: update helm-docs

* fix: revert changes to ingress

Co-authored-by: Davin Chia <davinchia@gmail.com>

charts/airbyte-temporal/README.md

@@ -17,6 +17,7 @@ Helm chart to deploy airbyte-temporal
 | affinity | object | `{}` |  |
 | containerSecurityContext | object | `{}` |  |
 | enabled | bool | `true` |  |
+| env_vars | object | `{}` |  |
 | extraContainers | list | `[]` |  |
 | extraEnv | list | `[]` |  |
 | extraInitContainers | list | `[]` |  |
@@ -51,6 +52,7 @@ Helm chart to deploy airbyte-temporal
 | replicaCount | int | `1` |  |
 | resources.limits | object | `{}` |  |
 | resources.requests | object | `{}` |  |
+| secrets | object | `{}` |  |
 | service.port | int | `7233` |  |
 | service.type | string | `"ClusterIP"` |  |
 | tolerations | list | `[]` |  |

charts/airbyte/README.md

@@ -212,7 +212,7 @@ Helm chart to deploy airbyte
 | webapp.image.repository | string | `"airbyte/webapp"` |  |
 | webapp.ingress.annotations | object | `{}` |  |
 | webapp.ingress.className | string | `""` |  |
-| webapp.ingress.enabled | bool | `false` |  |
+| webapp.ingress.enabled | bool | `true` |  |
 | webapp.ingress.hosts | list | `[]` |  |
 | webapp.ingress.tls | list | `[]` |  |
 | webapp.livenessProbe.enabled | bool | `true` |  |
@@ -238,6 +238,8 @@ Helm chart to deploy airbyte
 | webapp.service.type | string | `"ClusterIP"` |  |
 | webapp.tolerations | list | `[]` |  |
 | worker.affinity | object | `{}` |  |
+| worker.containerOrchestrator.enabled | bool | `true` |  |
+| worker.containerOrchestrator.image | string | `""` |  |
 | worker.containerSecurityContext | object | `{}` |  |
 | worker.enabled | bool | `true` |  |
 | worker.extraEnv | list | `[]` |  |

charts/airbyte/values.yaml

@@ -254,7 +254,7 @@ webapp:
   ##  webapp.ingress.hosts The list of hostnames to be covered with this ingress record.
   ##  webapp.ingress.tls [array] Custom ingress TLS configuration
   ingress:
-    enabled: true
+    enabled: false
     className: ""
     annotations: {}
     # kubernetes.io/ingress.class: nginx