+1 (binding) Checked signatures, licences, and checksums, and tested the chart locally with kind; the duplicated labels issue seems to be fixed.
On Fri, Mar 22, 2024 at 6:44 PM Jarek Potiuk <ja...@potiuk.com> wrote: > +1 binding. Verified reproducibility, signatures, licences, checksums. > Installed it locally and got Airflow working. > > Though - one comment. While doing it now, the prometheus exporter image > has not been pulled yet and the exporter is failing. The problem is that ` > quay.io` is going through an outage currently it seems. > > It's a RedHat service, so I presume it's a temporary blip, I am not sure > how reliable it is, but it seems to be going for well over 30 minutes for > me at least and does not show a sign of recovery, so maybe we > should consider switching to dockerhub for all our images in the future - > at least we will rely on a single service, not two of those. It seems quay > is far less reliable than dockerhub: > > Quay last 30 days:: > > [image: image.png] > > Last 24 hrs: > [image: image.png] > > Docker: > > [image: image.png] > > > J, > > > On Thu, Mar 21, 2024 at 7:24 PM Jed Cunningham <jedcunning...@apache.org> > wrote: > >> Hello Apache Airflow Community, >> >> This is a call for the vote to release Helm Chart version 1.13.1. >> >> The release candidate is available at: >> https://dist.apache.org/repos/dist/dev/airflow/helm-chart/1.13.1rc1/ >> >> airflow-chart-1.13.1-source.tar.gz - is the "main source release" that >> comes with INSTALL instructions. >> airflow-1.13.1.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.13.1rc1/ >> helm repo update >> helm install airflow apache-airflow-dev/airflow >> >> airflow-1.13.1.tgz.prov - is also uploaded for verifying Chart Integrity, >> though not strictly required for releasing the artifact based on ASF >> Guidelines. >> >> $ helm gpg verify airflow-1.13.1.tgz >> gpg: Signature made Thu Mar 21 12:13:02 2024 MDT >> gpg: using RSA key E1A1E984F55B8F280BD9CBA20BB7163892A2E48E >> gpg: issuer "jedcunning...@apache.org" >> gpg: Good signature from "Jed Cunningham <jedcunning...@apache.org>" >> [ultimate] >> plugin: Chart SHA verified. >> sha256:ed6b2dea0d8f99eb9bd9cd6bc418db95f88a7bb3b8d1afb7fdc266b1ea411a15 >> >> The vote will be open for at least 72 hours (2024-03-24 18:29 UTC) or >> until >> the necessary number of votes is reached. >> >> >> https://www.timeanddate.com/countdown/to?iso=20240324T1829&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)". >> >> Consider this my (binding) +1. >> >> For license checks, the .rat-excludes files is included, so you can run >> the >> following to verify licenses (just update your path to rat): >> >> tar -xvf airflow-chart-1.13.1-source.tar.gz >> cd airflow-chart-1.13.1 >> java -jar 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.13.1. This will allow us to rename the artifact without modifying >> the artifact checksums when we actually release it. >> >> The status of testing the Helm Chart by the community is kept here: >> https://github.com/apache/airflow/issues/38382 >> >> Thanks, >> Jed >> >