Skip to content

test: Add airflow 2.10.5 #625

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 5 commits into from
May 9, 2025
Merged

Conversation

NickLarsenNZ
Copy link
Member

@NickLarsenNZ NickLarsenNZ commented May 8, 2025

Part of stackabletech/docker-images#1074

Definition of Done Checklist

  • Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
  • Please make sure all these things are done and tick the boxes

Author

  • Integration tests passed (for non trivial changes)

Reviewer

  • Code contains useful comments
  • Code contains useful logging statements
  • (Integration-)Test cases added
  • Documentation added or updated. Follows the style guide.
  • Changelog updated
  • Cargo.toml only contains references to git tags (not specific commits or branches)

Acceptance

  • Feature Tracker has been updated
  • Proper release label has been added
  • Roadmap has been updated

@NickLarsenNZ
Copy link
Member Author

Tests pass

        --- PASS: kuttl/harness/ldap_airflow-latest-2.10.5_ldap-authentication-no-tls_openshift-false_executor-kubernetes (304.14s)
        --- PASS: kuttl/harness/ldap_airflow-latest-2.10.5_ldap-authentication-no-tls_openshift-false_executor-celery (192.61s)
        --- PASS: kuttl/harness/ldap_airflow-latest-2.10.5_ldap-authentication-insecure-tls_openshift-false_executor-kubernetes (151.50s)
        --- PASS: kuttl/harness/ldap_airflow-latest-2.10.5_ldap-authentication-insecure-tls_openshift-false_executor-celery (183.58s)
        --- PASS: kuttl/harness/resources_airflow-latest-2.10.5_openshift-false (138.30s)
        --- PASS: kuttl/harness/opa_airflow-2.10.5_opa-latest-1.4.2_openshift-false (125.27s)
        --- PASS: kuttl/harness/orphaned-resources_airflow-latest-2.10.5_openshift-false (151.51s)
        --- PASS: kuttl/harness/mount-dags-configmap_airflow-latest-2.10.5_openshift-false_executor-kubernetes (154.26s)
        --- PASS: kuttl/harness/mount-dags-configmap_airflow-latest-2.10.5_openshift-false_executor-celery (169.48s)
        --- PASS: kuttl/harness/smoke_airflow-2.10.5_openshift-false_executor-celery (216.36s)
        --- PASS: kuttl/harness/cluster-operation_airflow-latest-2.10.5_openshift-false (257.01s)
        --- PASS: kuttl/harness/mount-dags-gitsync_airflow-latest-2.10.5_openshift-false_executor-kubernetes (234.85s)
        --- PASS: kuttl/harness/mount-dags-gitsync_airflow-latest-2.10.5_openshift-false_executor-celery (232.36s)
        --- PASS: kuttl/harness/external-access_airflow-2.10.5_openshift-false (206.11s)
        --- PASS: kuttl/harness/overrides_airflow-latest-2.10.5_openshift-false (158.00s)
        --- PASS: kuttl/harness/logging_airflow-2.10.5_openshift-false_executor-kubernetes (283.15s)
        --- PASS: kuttl/harness/oidc_airflow-2.10.5_openshift-false (141.56s)
        --- PASS: kuttl/harness/ldap_airflow-latest-2.10.5_ldap-authentication-server-verification-tls_openshift-false_executor-kubernetes (150.10s)
        --- PASS: kuttl/harness/ldap_airflow-latest-2.10.5_ldap-authentication-server-verification-tls_openshift-false_executor-celery (192.91s)
        --- PASS: kuttl/harness/smoke_airflow-2.10.5_openshift-false_executor-kubernetes (187.23s)
        --- PASS: kuttl/harness/logging_airflow-2.10.5_openshift-false_executor-celery (288.98s)

@NickLarsenNZ NickLarsenNZ moved this from Development: In Progress to Development: Waiting for Review in Stackable Engineering May 8, 2025
@NickLarsenNZ NickLarsenNZ enabled auto-merge May 8, 2025 19:59
@lfrancke
Copy link
Member

lfrancke commented May 8, 2025

Could you also update this page please: https://github.com/stackabletech/airflow-operator/blob/chore/bump-airflow-pre-25.7.0/docs/modules/airflow/pages/required-external-components.adoc

PostgreSQL: Remove 11, add 16
MySQL: Remove 5.7, add "Innovation"
MSSQL has been removed in 2.9
Is it even possible to use SQLite with our operator? If not I'd just remove it too..

We really need a checklist item to update this page for each of our tools.... or link to the upstream page.

@sbernauer sbernauer disabled auto-merge May 9, 2025 05:43
@sbernauer sbernauer moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering May 9, 2025
@NickLarsenNZ
Copy link
Member Author

NickLarsenNZ commented May 9, 2025

Could you also update this page please: chore/bump-airflow-pre-25.7.0/docs/modules/airflow/pages/required-external-components.adoc

PostgreSQL: Remove 11, add 16 MySQL: Remove 5.7, add "Innovation" MSSQL has been removed in 2.9 Is it even possible to use SQLite with our operator? If not I'd just remove it too..

I have updated the airflow prereqs in 9e70671

We really need a checklist item to update this page for each of our tools.... or link to the upstream page.

I have updated the templates for the products that have operators with a required-external-components.adoc page:
stackabletech/docker-images#1109

I have also manually updated the existing issues to include the task.

@NickLarsenNZ NickLarsenNZ requested review from lfrancke and sbernauer May 9, 2025 06:55
@NickLarsenNZ NickLarsenNZ moved this from Development: In Review to Development: Waiting for Review in Stackable Engineering May 9, 2025
@NickLarsenNZ NickLarsenNZ enabled auto-merge May 9, 2025 06:55
Copy link
Member

@lfrancke lfrancke left a comment

Choose a reason for hiding this comment

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

Thank you very much for the additional changes. I didn't test this but it looks good to me!

@lfrancke lfrancke moved this from Development: Waiting for Review to Development: Done in Stackable Engineering May 9, 2025
@NickLarsenNZ NickLarsenNZ added this pull request to the merge queue May 9, 2025
Merged via the queue into main with commit fce0f72 May 9, 2025
17 checks passed
@NickLarsenNZ NickLarsenNZ deleted the chore/bump-airflow-pre-25.7.0 branch May 9, 2025 07:13
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
Status: Development: Done
Development

Successfully merging this pull request may close these issues.

2 participants