Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Status of testing Providers that were prepared on March 04, 2024 #37890

Closed
41 of 79 tasks
eladkal opened this issue Mar 4, 2024 · 16 comments
Closed
41 of 79 tasks

Status of testing Providers that were prepared on March 04, 2024 #37890

eladkal opened this issue Mar 4, 2024 · 16 comments
Labels
kind:meta High-level information important to the community testing status Status of testing releases

Comments

@eladkal
Copy link
Contributor

eladkal commented Mar 4, 2024

Body

I have a kind request for all the contributors to the latest provider packages release.
Could you please help us to test the RC versions of the providers?

The guidelines on how to test providers can be found in

Verify providers by contributors

Let us know in the comment, whether the issue is addressed.

Those are providers that require testing as there were some substantial changes introduced:

Provider amazon: 8.19.0rc1

Provider apache.beam: 5.6.2rc1

Provider apache.druid: 3.9.0rc1

Provider apache.hdfs: 4.3.3rc1

Provider apache.hive: 7.0.1rc1

Provider apache.livy: 3.7.3rc1

Provider apache.pinot: 4.3.1rc1

Provider celery: 3.6.1rc1

Provider cncf.kubernetes: 8.0.1rc1

Provider common.sql: 1.11.1rc1

Provider dbt.cloud: 3.7.0rc1

Provider docker: 3.9.2rc1

Provider exasol: 4.4.3rc1

Provider google: 10.16.0rc1

Provider hashicorp: 3.6.4rc1

Provider http: 4.10.0rc1

Provider microsoft.azure: 9.0.1rc1

Provider microsoft.psrp: 2.6.0rc2

Provider mysql: 5.5.4rc1

Provider openlineage: 1.6.0rc1

Provider opensearch: 1.1.2rc1

Provider postgres: 5.10.2rc1

Provider presto: 5.4.2rc1

Provider salesforce: 5.6.3rc1

Provider smtp: 1.6.1rc1

Provider telegram: 4.4.0rc2

Provider trino: 5.6.3rc1

Provider weaviate: 1.3.3rc1

All users involved in the PRs:
@vincbeck @VladaZakharova @pankajastro @dabla @m1racoli @RNHTTR @danielbe11 @Taragolis @pavansharma36 @kacpermuda @moiseenkov @dirrao @kevgeo @okirialbert @eejbyfeldt @mattiabertorello @MaksYermak @hter

Committer

  • I acknowledge that I am a maintainer/committer of the Apache Airflow project.
@eladkal eladkal added kind:meta High-level information important to the community testing status Status of testing releases labels Mar 4, 2024
airflow-oss-bot added a commit to astronomer/astronomer-providers that referenced this issue Mar 5, 2024
airflow-oss-bot added a commit to astronomer/astro-sdk that referenced this issue Mar 5, 2024
@moiseenkov
Copy link
Contributor

hi,
#37282 works as expected

@kacpermuda
Copy link
Contributor

All the PR's for the Provider openlineage: 1.6.0rc1 works as expected.

@Taragolis
Copy link
Contributor

I've check my changes, most of them pretty simple.

@LudvigLundberg
Copy link

I've check my changes, most of them pretty simple.

Can confirm that the changes in #37361 have solved the issue I was experiencing! Big thanks for fixing it!

@vchiapaikeo
Copy link
Contributor

#37711 looks good.

Huge thank you to everyone that helped resolve this mini fire last week (Jarek, Andrey, Elad 🙇 )

@arjunanan6
Copy link
Contributor

#34766 works as expected. :)

@m1racoli
Copy link
Contributor

m1racoli commented Mar 5, 2024

I can confirm #37486. It works as expected.

Although I had to manually require apache-airflow-providers-cncf-kubernetes>=8.0.0 for my DAG to parse. Before that I had version 7.14.0 installed and getting

Broken DAG: [/usr/local/airflow/dags/test_gke.py] Traceback (most recent call last):
  File "/usr/local/airflow/dags/test_gke.py", line 3, in <module>
    from airflow.providers.google.cloud.operators.kubernetes_engine import GKEStartPodOperator, OnFinishAction
  File "/usr/local/lib/python3.11/site-packages/airflow/providers/google/cloud/operators/kubernetes_engine.py", line 35, in <module>
    from airflow.providers.cncf.kubernetes.operators.job import KubernetesJobOperator
ModuleNotFoundError: No module named 'airflow.providers.cncf.kubernetes.operators.job'

I assume this is because of #36847. Are we missing a requirement for apache-airflow-providers-cncf-kubernetes>=8.0.0 in the google provider's dependencies?

@moiseenkov
Copy link
Contributor

moiseenkov commented Mar 5, 2024

I assume this is because of #36847. Are we missing a requirement for apache-airflow-providers-cncf-kubernetes>=8.0.0 in the google provider's dependencies?

@m1racoli , yes you're right. Thank you!
@eladkal , can we please exclude google from this release for now, merge the fix #37899 and release google afterwards with RC2?

@potiuk
Copy link
Member

potiuk commented Mar 6, 2024

All my changes are there :)

@pavansharma36
Copy link
Contributor

Verified #36794

@eladkal
Copy link
Contributor Author

eladkal commented Mar 6, 2024

@moiseenkov I don't think this qualify for RC2. It's not a regression and there is a simple fix (bump k8s provider if not using latest). Also the RC1 contains many features and fixes that not worth delaying.
We can always improve the mechanism and have better error message for next release.

@moiseenkov
Copy link
Contributor

@moiseenkov I don't think this qualify for RC2. It's not a regression and there is a simple fix (bump k8s provider if not using latest). Also the RC1 contains many features and fixes that not worth delaying.
We can always improve the mechanism and have better error message for next release.

I totally agree. We will adjust error messages in the google provider package for the next release and will follow this convention from now. Thank you!

@pankajastro
Copy link
Member

#37634 looks good. thanks!

@eladkal
Copy link
Contributor Author

eladkal commented Mar 8, 2024

Thank you everyone. Providers are released.

I invite everyone to help improve providers for the next release, a list of open issues can be found here.

@eladkal eladkal closed this as completed Mar 8, 2024
@dabla
Copy link
Contributor

dabla commented Mar 8, 2024

Maybe a bit late but I tested #37103 and I can confirm it works :)

@potiuk
Copy link
Member

potiuk commented Mar 8, 2024

Never too late for good news :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:meta High-level information important to the community testing status Status of testing releases
Projects
None yet
Development

No branches or pull requests