+1 non-binding Tested the helm chart with a sample dag using LocalExecutor, CeleryExecutor, and KubernetesExecutor
-Ephraim On Mon, 17 May 2021 at 21:12, Ash Berlin-Taylor <[email protected]> wrote: > +1 binding > > Checked sigs, spot checked license files, and deployed to a local kind > cluster with celery. > > -ash > > On Mon, May 17 2021 at 09:16:46 -0600, Jed Cunningham > <[email protected]> wrote: > > +1 (non-binding) > > Ran the binary chart through a number of scenarios (CeleryExecutor, > KubernetesExecutor, persistence, pgbouncer, gitsync) with a sample DAG and > everything looked good. > > On Sun, May 16, 2021 at 2:32 AM Jarek Potiuk <[email protected]> wrote: > >> Huge +1 from me. >> >> Thanks Kaxil for addressing ALL my concerns that quickly! >> >> * Verified licences, signatures, checksums >> * I was able to verify the provenance of the package with --verify >> flag using Kaxil's public key >> * installed airflow in several combinations after unpacking from sources >> * installed airflow in several combinations using the index.html "dev" >> source >> * airflow works out-of-the box in all those combinations >> >> I also see the changes coming from Jed which are fixing the documentation >> about the env vars which look really good. >> >> Fantastic job! >> >> J. >> >> >> >> On Sun, May 16, 2021 at 3:49 AM Kaxil Naik <[email protected]> wrote: >> >>> Hello Apache Airflow Community, >>> >>> This is a call for the vote to release the first stable version of >>> the Helm Chart version 1.0.0 that supports both Airflow 1.10.x and Airflow >>> 2.x. >>> >>> The release candidate is available at: >>> https://dist.apache.org/repos/dist/dev/airflow/helm-chart/1.0.0rc2/ >>> >>> *airflow-chart-1.0.0-source.tar.gz* - is the "main source release" that >>> comes with INSTALL instructions. >>> *airflow-1.0.0.tgz* - is the binary Helm Chart release. >>> >>> Public keys are available at: https://www.apache.org/dist/airflow/KEYS >>> >>> For convenience "index.yaml" has been uploaded (though excluded from >>> voting), so you can also run the below commands. >>> >>> helm repo add apache-airflow-dev >>> https://dist.apache.org/repos/dist/dev/airflow/helm-chart/1.0.0rc2/ >>> helm repo update >>> helm install airflow apache-airflow-dev/airflow >>> >>> *airflow-1.0.0.tgz.prov *- is also uploaded for verifying Chart >>> Integrity, though not strictly required for releasing the artifact based on >>> ASF Guidelines. >>> >>> $ helm verify airflow-1.0.0.tgz --keyring ~/.gnupg/secring.gpg >>> Signed by: Kaxil Naik <[email protected]> >>> Signed by: Kaxil Naik <[email protected]> >>> Using Key With Fingerprint: CDE15C6E4D3A8EC4ECF4BA4B6674E08AD7DE406F >>> Chart Hash Verified: >>> sha256:6cd3f13fc93d60424a771a1a8a4121c4439f7b6b48fab946436da0ab70d5a507 >>> >>> The vote will be open for at least 72 hours (2021-05-19 01:30 UTC) or >>> until the necessary number of votes are reached. >>> >>> >>> https://www.timeanddate.com/countdown/to?iso=20210519T0230&p0=136&font=cursive >>> >>> Please vote accordingly: >>> >>> [ ] +1 approve >>> [ ] +0 no opinion >>> [ ] -1 disapprove with the reason >>> >>> Only votes from PMC members are binding, but members of the community are >>> encouraged to test the release and vote with "(non-binding)". >>> >>> For license checks, the .rat-excludes files is included, so you can run >>> the following to verify licenses (just update $PATH_TO_RAT): >>> >>> tar -xvf airflow-chart-1.0.0-source.tar.gz >>> cd airflow-chart-1.0.0 >>> java -jar $PATH_TO_RAT/apache-rat-0.13/apache-rat-0.13.jar chart -E >>> .rat-excludes >>> >>> Please note that the version number excludes the `rcX` string, so it's >>> now >>> simply 1.0.0. This will allow us to rename the artifact without modifying >>> the artifact checksums when we actually release. >>> >>> Thanks, >>> Kaxil Naik >>> >> >> >> -- >> +48 660 796 129 >> >
