Skip to content

[bitnami/airflow] Minor performance improvements#33276

Merged
juan131 merged 5 commits intomainfrom
airflow-improvements
May 1, 2025
Merged

[bitnami/airflow] Minor performance improvements#33276
juan131 merged 5 commits intomainfrom
airflow-improvements

Conversation

@gongomgra
Copy link
Contributor

@gongomgra gongomgra commented Apr 30, 2025

Description of the change

Includes some minor improvements to the chart:

  • The setupDBJob.resourcesPreset default value is now small instead of micro, as the database migration process requires a bit more of CPU.
  • The sync-perm command now includes the --include-dags flag to update permissions for both Airflow roles and DAGs.
  • Restore httpGet based readinessProbe for the airflow-web pod to ensure the service is running before marking the pod as "ready". The airflow api-service process takes some time to be healthy, and the current status leads to flaky results in some k8s environments.

Checklist

  • Chart version bumped in Chart.yaml according to semver. This is not necessary when the changes only affect README.md files.
  • Variables are documented in the values.yaml and added to the README.md using readme-generator-for-helm
  • Title of the pull request follows this pattern [bitnami/<name_of_the_chart>] Descriptive title
  • All commits signed off and in agreement of Developer Certificate of Origin (DCO)

Signed-off-by: Gonzalo Gomez Gracia <gonzalo.gomez@broadcom.com>
@gongomgra gongomgra requested a review from juan131 April 30, 2025 21:40
@gongomgra gongomgra self-assigned this Apr 30, 2025
@gongomgra gongomgra added verify Execute verification workflow for these changes and removed airflow labels Apr 30, 2025
gongomgra and others added 3 commits April 30, 2025 23:45
Signed-off-by: Gonzalo Gomez Gracia <gonzalo.gomez@broadcom.com>
Signed-off-by: Bitnami Bot <bitnami.bot@broadcom.com>
Signed-off-by: Bitnami Bot <bitnami.bot@broadcom.com>
juan131
juan131 previously approved these changes May 1, 2025
Copy link
Contributor

@juan131 juan131 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Signed-off-by: Bitnami Bot <bitnami.bot@broadcom.com>
@juan131 juan131 merged commit be6729e into main May 1, 2025
10 checks passed
@juan131 juan131 deleted the airflow-improvements branch May 1, 2025 06:20
@github-actions github-actions bot added the solved label May 1, 2025
jpinxten pushed a commit to jpinxten/bitnami-charts that referenced this pull request May 5, 2025
Signed-off-by: jpinxten <jeroen.pinxten@icometrix.com>
rstribrn pushed a commit to rstribrn/bitnami_charts that referenced this pull request May 9, 2025
QustodioPablo pushed a commit to QustodioPablo/charts that referenced this pull request May 16, 2025
Signed-off-by: Pablo Fernandez <pablo.fernandez@qustodio.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

solved verify Execute verification workflow for these changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants