If this Vote passes I intend to call for an Accelerated Vote for rc5. Planning for rc5 to solve the issue with SQLA releases ( https://github.com/apache/airflow/issues/8211)
Regards, Kaxil On Wed, Apr 8, 2020 at 11:44 AM Robin Edwards <r...@bidnamic.com> wrote: > Just loaded up our dev environment and done a few sanity checks and > also ran our test suite. > > Thank you everyone for their hard work very much looking forward to > rolling this out. > > Python 3.7 / Postgres / Celery > > +1 (non-binding) > > On Tue, 7 Apr 2020 at 23:19, Xinbin Huang <bin.huan...@gmail.com> wrote: > > > > Test with Python 3.6 + LocalExecutor + Postgres backend > > > > Test both the AWS Parameter Store backend and GCP Secret Manager backend > > can work properly. > > > > +1 (non-binding). > > > > Regards, > > Xinbin > > > > On Tue, Apr 7, 2020 at 10:59 AM Deng Xiaodong <xd.den...@gmail.com> > wrote: > > > > > Thanks Kaxil & Ash for the clarification! > > > > > > +1 (non-binding). > > > > > > > > > > > > XD > > > > > > > On 7 Apr 2020, at 7:46 PM, Kaxil Naik <kaxiln...@gmail.com> wrote: > > > > > > > > Thanks, XD for testing. Please check my answers to your question > below: > > > > > > > > - "Delete DAG" doesn't check whether the DAG file is still in DAG > Folder > > > > anymore. It simply deletes all records of that DAG in DB, and that > DAG > > > will > > > > re-appear in UI and DB if the DAG file is still in DAG Folder. I > found > > > it's > > > > also behaving like this in 1.10.9. Is this intentional? > > > > > > > > This behavior is since 1.10.6 and the change was done to allow > deleting > > > renamed DAGs i.e "If we rename a dag_id but do not rename the file, > then we > > > cannot delete the old dag." > > > > > > > > > > > > - "Trigger DAG" allows users to give an optional configuration JSON > now. > > > > How the format of this config JSON should look like? Is it mentioned > > > > somewhere in the doc for 1.10.10? > > > > > > > > The Web UI would throw an error if the string provided is not VALID > JSON > > > string. Example Screenshot: > > > > > > > > > > > > Ash has already replied to your other questions. > > > > > > > > Regards, > > > > Kaxil > > > > > > > > > > > > > > > > > > > > On Tue, Apr 7, 2020 at 5:34 PM Ash Berlin-Taylor <a...@apache.org > > > <mailto:a...@apache.org>> wrote: > > > > "Trigger DAG" UI now exposes via the UI what was previously only > > > > available via the `--conf` option to `airflow trigger_dag`. > > > > > > > > Scheduling is not strictly in UTC anymore, see > > > > > http://airflow.apache.org/docs/stable/timezone.html#time-zone-aware-dags > > > < > http://airflow.apache.org/docs/stable/timezone.html#time-zone-aware-dags> > > > > > > > > On Apr 7 2020, at 5:30 pm, Deng Xiaodong <xd.den...@gmail.com > <mailto: > > > xd.den...@gmail.com>> wrote: > > > > > > > > > Thanks, Kaxil, for running the release! > > > > > > > > > > I tested with Python 3.6 + LocalExecutor + Postgres+RBAC in Docker > > > > > environment ( > > > > > > > > > https://github.com/XD-DENG/containerized-airflow/tree/airflow-1-10-10/localexecutor_postgres > > > < > > > > https://github.com/XD-DENG/containerized-airflow/tree/airflow-1-10-10/localexecutor_postgres > > > >), > > > > > including > > > > > > > > > > - Run example DAGs & self-defined DAG > > > > > - Create users via both CLI and UI > > > > > - RBAC feature (deleting DAG as User role before & after removing > User > > > > > role's "delete" permission) > > > > > - The Timzone-selection feature in UI (really nice new feature!) > > > > > > > > > > All of them work very well to me. > > > > > > > > > > Before I give +1, I have three questions to confirm: > > > > > > > > > > - "Delete DAG" doesn't check whether the DAG file is still in DAG > > > Folder > > > > > anymore. It simply deletes all records of that DAG in DB, and that > DAG > > > will > > > > > re-appear in UI and DB if the DAG file is still in DAG Folder. I > found > > > it's > > > > > also behaving like this in 1.10.9. Is this intentional? > > > > > - Timezone-selection in UI: is it highlighted anywhere in the doc > that > > > the > > > > > scheduling is still ONLY based on UTC? > > > > > - "Trigger DAG" allows users to give an optional configuration JSON > > > now. > > > > > How the format of this config JSON should look like? Is it > mentioned > > > > > somewhere in the doc for 1.10.10? > > > > > > > > > > Thanks again! > > > > > > > > > > > > > > > XD > > > > > > > > > > On Mon, Apr 6, 2020 at 9:35 PM Kaxil Naik <kaxiln...@gmail.com > > > <mailto:kaxiln...@gmail.com>> wrote: > > > > > > > > > >> Hello Apache Airflow Community, > > > > >> > > > > >> This is a call for the vote to release Apache Airflow version > 1.10.10. > > > > >> > > > > >> The release candidate: > > > > >> https://dist.apache.org/repos/dist/dev/airflow/1.10.10rc4/ < > > > https://dist.apache.org/repos/dist/dev/airflow/1.10.10rc4/> > > > > >> > > > > >> *apache-airflow-1.10.10rc4-source.tar.gz* is a source release that > > > comes > > > > >> with INSTALL instructions. > > > > >> *apache-airflow-1.10.10rc4-bin.tar.gz* is the binary Python > "sdist" > > > > >> release. > > > > >> *apache_airflow-1.10.10rc4-py2.py3-none-any.whl* is the binary > Python > > > wheel > > > > >> release. > > > > >> > > > > >> Public keys are available at > https://www.apache.org/dist/airflow/KEYS > > > <https://www.apache.org/dist/airflow/KEYS> > > > > >> > > > > >> For convenience, RC is also uploaded on PyPI so you can install it > > > using > > > > >> *pip install apache-airflow==1.10.10rc4* > > > > >> > > > > >> Changes since 1.10.10rc2: > > > > >> - Fix 500 error in Security screens (#8165) > > > > >> - Fix Viewing Dag Code for Stateless Webserver (#8159) > > > > >> > > > > >> Full Change Log for the release: > > > > >> https://github.com/apache/airflow/blob/1.10.10rc4/CHANGELOG.txt < > > > https://github.com/apache/airflow/blob/1.10.10rc4/CHANGELOG.txt> > > > > >> > > > > >> The vote will be open for at least 72 hours (9 April 19:35 UTC) or > > > until > > > > >> the necessary number > > > > >> of votes are reached. > > > > >> > > > > >> 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)". > > > > >> > > > > >> Please note that the version number excludes the `rcX` string, so > > > > >> it's now > > > > >> simply 1.10.10. This will allow us to rename the artifact without > > > modifying > > > > >> the artifact checksums when we actually release it. > > > > >> > > > > >> Thanks, > > > > >> Kaxil Naik > > > > >> > > > > > > > > > > > >