Fixes the newline in the NGINX DaemonSet,
which would remove more newlines than it should.

Changelog: fixed

Clemens Beck authored 1 year ago and Jason Plum committed 1 year ago

Adds `global.hosts.externalGeoIP` to set a static IP for the
Geo NGINX LoadBalancer, which previously used `global.hosts.externalIP`
like the default NGINX controller.

This allows to configure distinct static IPs for both NGINX subcharts.

Fixes #5006

Changelog: fixed

Mitchell Nielsen authored 1 year ago and Jason Plum committed 1 year ago

Updates the NGINX Ingress Controller image default value
to v1.3.1.

Reference to last change of this kind:
!2843

Related to
#4856

yuanxiongxiong authored 1 year ago and Jason Plum committed 1 year ago

Changelog: fixed

Signed-off-by: yxxhero <aiopsclub@163.com>

Troy Lindsay authored 2 years ago and Mitchell Nielsen committed 2 years ago

* https://kubernetes.io/blog/2023/02/06/k8s-gcr-io-freeze-announcement/
  The Kubernetes project runs a community-owned image registry called
  registry.k8s.io to host its container images. On the 3rd of April 2023, the
  old registry k8s.gcr.io will be frozen and no further images for Kubernetes
  and related subprojects will be pushed to the old registry.

  This registry registry.k8s.io replaced the old one and has been generally
  available for several months. We have published a blog post about its
  benefits to the community and the Kubernetes project. This post also
  announced that future versions of Kubernetes will not be available in the old
  registry. Now that time has come.

Changelog: changed

Graeme Gillies authored 2 years ago and DJ Mountney committed 2 years ago

Part of gitlab-org/charts/gitlab#2860

In order to allow easy use of fips or ubi8 images, we want to provide
a single helm configuration option that allows users to specify that
all images being used will have the helm value `global.image.tagSuffix`
applied to the end of the image tag.

A practical use case for this would be using `--set global.image.tagSuffix="-fips"`
to install GitLab via helm using our fips compliant images.

Changelog: added

Mike Hobbs authored 2 years ago and Mitchell Nielsen committed 2 years ago

This MR adds new functionality to the fork of the Nginx Chart. It adds the following configuration parameters so that users can optionally enable or disable GitLab Shell (SSH access) to the generated external and internal services used by the ingress.

```
nginx-ingress:
  controller:
    service:
      enableShell: false
      internal:
        enableShell:  true
```

Closes #3373

Changelog: added

Robin Elfrink authored 2 years ago and Mitchell Nielsen committed 2 years ago

This MR adds the option to specify `priorityClassName` for pods in the chart and all included subcharts.

See the [Kubernetes documentation on PriorityClasses](https://kubernetes.io/docs/concepts/scheduling-eviction/pod-priority-preemption/#priorityclass) for detailed information.

Setting `global.priorityClassName` in the main `values.yaml` will result in all pods installed by this chart to be configured with the named `PriorityClass`, with the exception of GitLab Runner and Prometheus; these subcharts do not inherit `global.priorityClassName`.

A test has been added that sets `global.priorityClassName` to `system-cluster-critical`, and checks if all resulting `Deployments` and `Jobs` have `priorityClassName: system-cluster-critical` in their pod specs.

As a default installation does not include `StatefulSets`, `DaemonSets` or `CronJobs` these are not tested.

Changelog: added

References version 1.2.1 of the NGINX Ingress Controller, mirrored in
gitlab-org/cloud-native/mirror/images!6.

Digest was collected via:

```
$ skopeo inspect docker://k8s.gcr.io/ingress-nginx/controller:v1.2.1 | jq -r '.Digest'
sha256:5516d103a9c2ecc4f026efbd4b40662ce22dc1f824fb129ed121460aaa5c47f8
```

Closes #3403

Changelog: changed

Hossein Pursultani authored 2 years ago and Jason Plum committed 2 years ago

Changelog: changed

Hossein Pursultani authored 2 years ago and Jason Plum committed 2 years ago

Changelog: changed

* Add support for memory metrics to all HPAs
* Allow specifying either average value or average utilization for cpu/memory metrics in all HPAs
* Add support for custom metrics in all HPAs
* Deprecate hpa.targetAverage{Utilization,Value} in favor of hpa.cpu.targetAverage{Utilization,Value}
* Deprecate hpa.behaviour in favor of hpa.behavior

Changelog: added

Removes NGINX's `isControllerValid` helper, as the check had not been
updated since its implementation in 2020. Additionally, as part of
#3383, we'll need
to refer to a tag that includes `ubi`, which would break the
`semverCompare` function.

Changelog: removed

Filipe Santos authored 2 years ago and DJ Mountney committed 2 years ago

Adds support for Pod Disruption Budget maxUnavailable option

Changelog: added

Mitchell Nielsen authored 3 years ago and Jason Plum committed 3 years ago

- Creates a new template that returns only the IngressClass value
- Uses that new template in the existing template that writes
  `ingressClassName: "value"`
- Uses the new template in the IngressClass template to ensure that
  'global.ingress.class' is considered when setting its name
- Updates the fork documentation for the change

Changelog: fixed

Mitchell Nielsen authored 3 years ago and Jason Plum committed 3 years ago

This reverts commit 785d893d.

Changelog: fixed

Switch `global.busybox.image.repository` and
`nginx-ingress.*.image.repository` to pull from the GitLab.com mirror
of those at https://gitlab.com/gitlab-org/cloud-native/mirror/images

Changelog: changed

Updates eks example to clearly show how to enable `ProxyProtocol` in a
eks scenario.

Fix some entries to match the correct documented property name.

- Changes keyword `tcpProxyProtocol` in global values to allow a future
  use of encoding.
- Removes aws specific annotation for proxyProtocol.

- Allows ProxyProtocol to be enabled without breaking SSH.

Mitchell Nielsen authored 4 years ago and Jason Plum committed 4 years ago

This aligns the template used when naming the TCP configmap with what is
defined in the Deployment. This was a miss from our NGINX chart upgrade.

The incorrect name was provided for the "ingress-nginx.name" partial
in a few templates (incorretly set to "nginx-ingress.name").

Mitchell Nielsen authored 4 years ago and Jason Plum committed 4 years ago

3.11.1 was the last version that supported Helm v1 schema, which is
  required at the moment as we haven't fully upgraded to Helm 3 yet.

Imports ingress-nginx from:
https://github.com/kubernetes/ingress-nginx