[bitnami/airflow] Fix baseUrl generation with ingress enabled (#33399)

This commit is contained in:
Gonzalo Gómez Gracia
2025-05-06 15:01:12 +02:00
committed by GitHub
parent e9e4ae793f
commit 461bc9face
6 changed files with 48 additions and 40 deletions

View File

@@ -1,8 +1,12 @@
# Changelog
## 23.0.3 (2025-05-05)
## 23.0.4 (2025-05-06)
* [bitnami/airflow] Fix svc name for Ingress backend ([#33316](https://github.com/bitnami/charts/pull/33316))
* [bitnami/airflow] Fix baseUrl generation with ingress enabled ([#33399](https://github.com/bitnami/charts/pull/33399))
## <small>23.0.3 (2025-05-05)</small>
* [bitnami/airflow] Fix svc name for Ingress backend (#33316) ([d814a50](https://github.com/bitnami/charts/commit/d814a50bfb2756a817cbd4f66abbba71d1b74b63)), closes [#33316](https://github.com/bitnami/charts/issues/33316)
## <small>23.0.2 (2025-05-01)</small>

View File

@@ -40,4 +40,4 @@ maintainers:
name: airflow
sources:
- https://github.com/bitnami/charts/tree/main/bitnami/airflow
version: 23.0.3
version: 23.0.4

View File

@@ -1221,40 +1221,40 @@ The Bitnami Airflow chart relies on the PostgreSQL chart persistence. This means
### Airflow database parameters
| Name | Description | Value |
| ------------------------------------------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | ----------------- |
| `postgresql.enabled` | Switch to enable or disable the PostgreSQL helm chart | `true` |
| `postgresql.auth.enablePostgresUser` | Assign a password to the "postgres" admin user. Otherwise, remote access will be blocked for this user | `true` |
| `postgresql.auth.username` | Name for a custom user to create | `bn_airflow` |
| `postgresql.auth.password` | Password for the custom user to create | `""` |
| `postgresql.auth.database` | Name for a custom database to create | `bitnami_airflow` |
| `postgresql.auth.existingSecret` | Name of existing secret to use for PostgreSQL credentials | `""` |
| `postgresql.architecture` | PostgreSQL architecture (`standalone` or `replication`) | `standalone` |
| `postgresql.primary.resourcesPreset` | Set container resources according to one common preset (allowed values: none, nano, small, medium, large, xlarge, 2xlarge). This is ignored if primary.resources is set (primary.resources is recommended for production). | `nano` |
| `postgresql.primary.resources` | Set container requests and limits for different resources like CPU or memory (essential for production workloads) | `{}` |
| `externalDatabase.host` | Database host (ignored if externalDatabase.sqlConnection is set) | `localhost` |
| `externalDatabase.port` | Database port number (ignored if externalDatabase.sqlConnection is set) | `5432` |
| `externalDatabase.user` | Non-root username for Airflow (ignored if externalDatabase.sqlConnection is set) | `bn_airflow` |
| `externalDatabase.password` | Password for the non-root username for Airflow (ignored if externalDatabase.sqlConnection or externalDatabase.existingSecret are set) | `""` |
| `externalDatabase.database` | Airflow database name (ignored if externalDatabase.sqlConnection is set) | `bitnami_airflow` |
| `externalDatabase.sqlConnection` | SQL connection string | `""` |
| `externalDatabase.existingSecret` | Name of an existing secret resource containing the database credentials | `""` |
| `externalDatabase.existingSecretPasswordKey` | Name of an existing secret key containing the database credentials (ignored if externalDatabase.existingSecretSqlConnectionKey is set) | `""` |
| `externalDatabase.existingSecretSqlConnectionKey` | Name of an existing secret key containing the SQL connection string | `""` |
| `redis.enabled` | Switch to enable or disable the Redis&reg; helm | `true` |
| `redis.auth.enabled` | Enable password authentication | `true` |
| `redis.auth.password` | Redis&reg; password | `""` |
| `redis.auth.existingSecret` | The name of an existing secret with Redis&reg; credentials | `""` |
| `redis.architecture` | Redis&reg; architecture. Allowed values: `standalone` or `replication` | `standalone` |
| `redis.master.service.ports.redis` | Redis&reg; port | `6379` |
| `redis.master.resourcesPreset` | Set container resources according to one common preset (allowed values: none, nano, small, medium, large, xlarge, 2xlarge). This is ignored if master.resources is set (master.resources is recommended for production). | `nano` |
| `redis.master.resources` | Set container requests and limits for different resources like CPU or memory (essential for production workloads) | `{}` |
| `externalRedis.host` | Redis&reg; host | `localhost` |
| `externalRedis.port` | Redis&reg; port number | `6379` |
| `externalRedis.username` | Redis&reg; username | `""` |
| `externalRedis.password` | Redis&reg; password | `""` |
| `externalRedis.existingSecret` | Name of an existing secret resource containing the Redis&trade credentials | `""` |
| `externalRedis.existingSecretPasswordKey` | Name of an existing secret key containing the Redis&trade credentials | `""` |
| Name | Description | Value |
| ------------------------------------------------- | --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | ----------------- |
| `postgresql.enabled` | Switch to enable or disable the PostgreSQL helm chart | `true` |
| `postgresql.auth.enablePostgresUser` | Assign a password to the "postgres" admin user. Otherwise, remote access will be blocked for this user | `true` |
| `postgresql.auth.username` | Name for a custom user to create | `bn_airflow` |
| `postgresql.auth.password` | Password for the custom user to create | `""` |
| `postgresql.auth.database` | Name for a custom database to create | `bitnami_airflow` |
| `postgresql.auth.existingSecret` | Name of existing secret to use for PostgreSQL credentials | `""` |
| `postgresql.architecture` | PostgreSQL architecture (`standalone` or `replication`) | `standalone` |
| `postgresql.primary.resourcesPreset` | Set container resources according to one common preset (allowed values: none, nano, micro, small, medium, large, xlarge, 2xlarge). This is ignored if primary.resources is set (primary.resources is recommended for production). | `nano` |
| `postgresql.primary.resources` | Set container requests and limits for different resources like CPU or memory (essential for production workloads) | `{}` |
| `externalDatabase.host` | Database host (ignored if externalDatabase.sqlConnection is set) | `localhost` |
| `externalDatabase.port` | Database port number (ignored if externalDatabase.sqlConnection is set) | `5432` |
| `externalDatabase.user` | Non-root username for Airflow (ignored if externalDatabase.sqlConnection is set) | `bn_airflow` |
| `externalDatabase.password` | Password for the non-root username for Airflow (ignored if externalDatabase.sqlConnection or externalDatabase.existingSecret are set) | `""` |
| `externalDatabase.database` | Airflow database name (ignored if externalDatabase.sqlConnection is set) | `bitnami_airflow` |
| `externalDatabase.sqlConnection` | SQL connection string | `""` |
| `externalDatabase.existingSecret` | Name of an existing secret resource containing the database credentials | `""` |
| `externalDatabase.existingSecretPasswordKey` | Name of an existing secret key containing the database credentials (ignored if externalDatabase.existingSecretSqlConnectionKey is set) | `""` |
| `externalDatabase.existingSecretSqlConnectionKey` | Name of an existing secret key containing the SQL connection string | `""` |
| `redis.enabled` | Switch to enable or disable the Redis&reg; helm | `true` |
| `redis.auth.enabled` | Enable password authentication | `true` |
| `redis.auth.password` | Redis&reg; password | `""` |
| `redis.auth.existingSecret` | The name of an existing secret with Redis&reg; credentials | `""` |
| `redis.architecture` | Redis&reg; architecture. Allowed values: `standalone` or `replication` | `standalone` |
| `redis.master.service.ports.redis` | Redis&reg; port | `6379` |
| `redis.master.resourcesPreset` | Set container resources according to one common preset (allowed values: none, nano, small, medium, large, xlarge, 2xlarge). This is ignored if master.resources is set (master.resources is recommended for production). | `nano` |
| `redis.master.resources` | Set container requests and limits for different resources like CPU or memory (essential for production workloads) | `{}` |
| `externalRedis.host` | Redis&reg; host | `localhost` |
| `externalRedis.port` | Redis&reg; port number | `6379` |
| `externalRedis.username` | Redis&reg; username | `""` |
| `externalRedis.password` | Redis&reg; password | `""` |
| `externalRedis.existingSecret` | Name of an existing secret resource containing the Redis&trade credentials | `""` |
| `externalRedis.existingSecretPasswordKey` | Name of an existing secret key containing the Redis&trade credentials | `""` |
Specify each parameter using the `--set key=value[,key=value]` argument to `helm install`. For example,

View File

@@ -397,7 +397,7 @@ If not using ClusterIP, or if a host or LoadBalancerIP is not defined, the value
{{- $host := default (include "airflow.serviceIP" .) .Values.web.baseUrl -}}
{{- $port := printf ":%d" (int .Values.service.ports.http) -}}
{{- $schema := ternary "https://" "http://" (or .Values.web.tls.enabled (and .Values.ingress.enabled .Values.ingress.tls)) -}}
{{- if regexMatch "^https?://" .Values.web.baseUrl -}}
{{- if and (regexMatch "^https?://" .Values.web.baseUrl) (not .Values.ingress.enabled) -}}
{{- $schema = "" -}}
{{- end -}}
{{- if or (regexMatch ":\\d+$" .Values.web.baseUrl) (eq $port ":80") (eq $port ":443") -}}
@@ -405,6 +405,7 @@ If not using ClusterIP, or if a host or LoadBalancerIP is not defined, the value
{{- end -}}
{{- if and .Values.ingress.enabled .Values.ingress.hostname -}}
{{- $host = .Values.ingress.hostname -}}
{{- $port = "" -}}
{{- end -}}
{{- printf "%s%s%s" $schema $host $port -}}
{{- end -}}

View File

@@ -3108,7 +3108,7 @@ postgresql:
primary:
## PostgreSQL Primary resource requests and limits
## ref: https://kubernetes.io/docs/concepts/configuration/manage-compute-resources-container/
## @param postgresql.primary.resourcesPreset Set container resources according to one common preset (allowed values: none, nano, small, medium, large, xlarge, 2xlarge). This is ignored if primary.resources is set (primary.resources is recommended for production).
## @param postgresql.primary.resourcesPreset Set container resources according to one common preset (allowed values: none, nano, micro, small, medium, large, xlarge, 2xlarge). This is ignored if primary.resources is set (primary.resources is recommended for production).
## More information: https://github.com/bitnami/charts/blob/main/bitnami/common/templates/_resources.tpl#L15
##
resourcesPreset: "nano"