+1 (non-binding) I tested the Helm chart using git-sync to fetch dags from a private repo. It works as expected!
On Tue, May 18, 2021 at 8:19 AM Xiaodong Deng <[email protected]> wrote: > +1 (binding) > > Tested installing in local cluster. > > Thanks Kaxil for addressing the provenance file comment quickly👍 > > > XD > > On Mon, May 17, 2021 at 22: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 >>> >>
