+1 (binding). Checked signatures, checksum, licences. Concur with Elad. We can have 1.8.1 with those fixes.
J. On Mon, Feb 6, 2023 at 12:07 PM Elad Kalif <[email protected]> wrote: > > +1 (binding) > Since the reports by Jules don't indicate regression I see no reason to block > this release. > Should release manager decide the bug fixes are important we can prepare a > followup 1.8.1 release > > On Mon, Feb 6, 2023 at 12:23 PM Ephraim Anierobi <[email protected]> > wrote: >> >> +1 (binding) since the issues identified by Jules are not regressions. >> >> I checked licenses and signatures and also ran some example dags. >> >> >> On 2023/02/02 07:38:43 [email protected] wrote: >> > -1 (non-binding) : a few bugs were identified while testing the release >> > and have mergeable PRs to fix them. Most notable is >> > #29295<https://github.com/apache/airflow/pull/29295> but there are also >> > #29296<https://github.com/apache/airflow/pull/29296> and >> > #29297<https://github.com/apache/airflow/pull/29297>. I also see another >> > bugfix ready to merge : >> > #29214<https://github.com/apache/airflow/pull/29214>. >> > >> > From: Jed Cunningham <[email protected]> >> > Date: Tuesday, 31 January 2023 at 21:16 >> > To: [email protected] <[email protected]> >> > Subject: [VOTE] Release Apache Airflow Helm Chart 1.8.0 based on 1.8.0rc1 >> > Hello Apache Airflow Community, >> > >> > This is a call for the vote to release Helm Chart version 1.8.0. >> > >> > The release candidate is available at: >> > https://dist.apache.org/repos/dist/dev/airflow/helm-chart/1.8.0rc1/<https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdist.apache.org%2Frepos%2Fdist%2Fdev%2Fairflow%2Fhelm-chart%2F1.8.0rc1%2F&data=05%7C01%7CJules.Triomphe%40swisscom.com%7C9417faac568d47b3c9d008db03c81424%7C364e5b87c1c7420d9beec35d19b557a1%7C0%7C0%7C638107930086096425%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=DJEDeuMb6EZcG%2FkWOKmdEsvXgNZPIEGUjj0%2F6ymEw1o%3D&reserved=0> >> > >> > airflow-chart-1.8.0-source.tar.gz - is the "main source release" that >> > comes with INSTALL instructions. >> > airflow-1.8.0.tgz - is the binary Helm Chart release. >> > >> > Public keys are available at: >> > https://www.apache.org/dist/airflow/KEYS<https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.apache.org%2Fdist%2Fairflow%2FKEYS&data=05%7C01%7CJules.Triomphe%40swisscom.com%7C9417faac568d47b3c9d008db03c81424%7C364e5b87c1c7420d9beec35d19b557a1%7C0%7C0%7C638107930086096425%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=G2y4rGDQX1BUyk4n86DZ5ksOrUzZLZzOTiM3HvZg0qc%3D&reserved=0> >> > >> > 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.8.0rc1/<https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdist.apache.org%2Frepos%2Fdist%2Fdev%2Fairflow%2Fhelm-chart%2F1.8.0rc1%2F&data=05%7C01%7CJules.Triomphe%40swisscom.com%7C9417faac568d47b3c9d008db03c81424%7C364e5b87c1c7420d9beec35d19b557a1%7C0%7C0%7C638107930086096425%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=DJEDeuMb6EZcG%2FkWOKmdEsvXgNZPIEGUjj0%2F6ymEw1o%3D&reserved=0> >> > helm repo update >> > helm install airflow apache-airflow-dev/airflow >> > >> > airflow-1.8.0.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.8.0.tgz >> > gpg: Signature made Tue Jan 31 12:53:25 2023 MST >> > gpg: using RSA key E1A1E984F55B8F280BD9CBA20BB7163892A2E48E >> > gpg: issuer >> > "[email protected]<mailto:[email protected]>" >> > gpg: Good signature from "Jed Cunningham >> > <[email protected]<mailto:[email protected]>>" [ultimate] >> > plugin: Chart SHA verified. >> > sha256:62d5465b95d1b82fad8b575c6919f29200eb7a8011df49063733143fc9f4fd5a >> > >> > The vote will be open for at least 72 hours (2023-02-03 20:20 UTC) or >> > until the necessary number of votes is reached. >> > >> > https://www.timeanddate.com/countdown/to?iso=20230203T2020&p0=136&font=cursive<https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.timeanddate.com%2Fcountdown%2Fto%3Fiso%3D20230203T2020%26p0%3D136%26font%3Dcursive&data=05%7C01%7CJules.Triomphe%40swisscom.com%7C9417faac568d47b3c9d008db03c81424%7C364e5b87c1c7420d9beec35d19b557a1%7C0%7C0%7C638107930086096425%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=2OZMzpxfO2tKZhihSExkPZl5YtHRRFBK4ngOPrdrCcg%3D&reserved=0> >> > >> > 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: >> > >> > tar -xvf airflow-chart-1.8.0-source.tar.gz >> > cd airflow-chart-1.8.0 >> > java -jar /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.8.0. 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/29273<https://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fairflow%2Fissues%2F29273&data=05%7C01%7CJules.Triomphe%40swisscom.com%7C9417faac568d47b3c9d008db03c81424%7C364e5b87c1c7420d9beec35d19b557a1%7C0%7C0%7C638107930086096425%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=BaJMKzwON4%2FjyEsz6GHEKRsf8qfSdOR3bUGpY6TkRFY%3D&reserved=0> >> > >> > Thanks, >> > Jed >> >
