-1 (binding): I tested sources (new check)
/licences/checksums/signatures: those look good.

However when I tested it with
https://raw.githubusercontent.com/apache/airflow-client-python/main/dev/test_python_client.py,
it turns out that the (rather basic) get_tasks method fails with:

Exception when calling DagAPI->get_tasks: Invalid type for variable
'execution_timeout'. Required value type is TimeDelta and passed type was
NoneType at ['received_data']['tasks'][0]['execution_timeout']. -

Despite execution_timeout marked as "nullable" in the API specification.

The same test works when I just reinstall  python_client 2.6.1, so I am
quite sure 2.6.2rc1 (I can reliably reproduce success/failure on just
switching between the two versions).

No time to investigate it (I am host stage tomorrow at Berlin Buzzwords and
will be busy whole day) but I created an issue:
https://github.com/apache/airflow-client-python/issues/85

J.




On Sun, Jun 18, 2023 at 10:30 PM Pierre Jeambrun <pierrejb...@gmail.com>
wrote:

> Hey fellow Airflowers,
>
> I have cut the release candidate for the Airflow Python Client 2.6.2rc1.
> The client consists of APIs corresponding to REST APIs available in
> *Apache Airflow 2.6.2*. This email is calling for a vote on
> the release, which will last for 72 hours. Consider this my (binding) +1.
>
> Airflow Client 2.6.2rc1 is available at:
> https://dist.apache.org/repos/dist/dev/airflow/clients/python/2.6.2rc1/
>
> Or also available at PyPI:
> https://pypi.org/project/apache-airflow-client/2.6.2rc1/
>
> *apache-airflow-client-2.6.2rc1-source.tar.gz* is a source release that
> comes with
> INSTALL instructions.
> *apache-airflow-client-2.6.2rc1-bin.tar.gz* is the binary Python "sdist"
> release.
>
> Public keys are available at:
> https://dist.apache.org/repos/dist/release/airflow/KEYS
>
> Only votes from PMC members are binding, but the release manager should
> encourage members of the community to test the release and vote with
> "(non-binding)".
>
> *Changelog:*
>
> *Major changes:*
>
>  - Add TriggererStatus to OpenAPI spec ([#31579](
> https://github.com/apache/airflow/pull/31579))
>
> *Major Fixes*:
>
>  - Fixing broken links in openapi/v1.yaml ([#31619](
> https://github.com/apache/airflow/pull/31619))
>
> *NEW API supported*:
>
>  - NA
>
> Cheers,
> Pierre Jeambrun
>

Reply via email to