Re: [VOTE] Airflow 1.10.2rc3

2019-01-22 Thread Tao Feng
+1 given it is a time based release. On Tue, Jan 22, 2019 at 7:00 AM Ash Berlin-Taylor wrote: > +1 (binding) > > - Checksums match > - Signing good > - version.py doesn't contain "rc" > - UPDATING.md looks good > - Tested celery executor on redis. > - Pausing/unpausing dags on RBAC UI > - Impor

Re: [VOTE] Airflow 1.10.2rc3

2019-01-22 Thread Ash Berlin-Taylor
+1 (binding) - Checksums match - Signing good - version.py doesn't contain "rc" - UPDATING.md looks good - Tested celery executor on redis. - Pausing/unpausing dags on RBAC UI - Import errors from dags showing up in RBAC UI. I did notice something in the task instance logs that I want to look at,

Re: [VOTE] Airflow 1.10.2rc3

2019-01-22 Thread Bolke de Bruin
Can I suggest that we get in the habit of time based releases, now that we don’t have to vote twice? In this context I suggest that not having documentation is not blocking for release, but that we should do a quick .3 release Tao can you change your vote to +1? Verstuurd vanaf mijn iPad > Op

Re: [VOTE] Airflow 1.10.2rc3

2019-01-21 Thread Felix Uellendall
+1 (non-binding) I could not found any more blocker issues. I ran all example dags and some of our own dags via RBAC UI. - feluelle Am 19/01/2019 um 17:35 schrieb Kaxil Naik: Hey all, I have cut Airflow 1.10.2 RC3. This email is calling a vote on the release, which will last for 72 hours. C

Re: [VOTE] Airflow 1.10.2rc3

2019-01-21 Thread Tao Feng
My vote is changed to 0 for this RC. I have prepared a pr to fix the viewer issue(https://github.com/apache/airflow/pull/4569). @Deng Xiaodong could you test the pr and see if the issue still exists? On Mon, Jan 21, 2019 at 11:05 AM Kaxil Naik wrote: > @FengTao > > Do you have a "-1" vote or

Re: [VOTE] Airflow 1.10.2rc3

2019-01-21 Thread Kaxil Naik
@FengTao Do you have a "-1" vote or "+1" vote? On Mon, Jan 21, 2019, 18:21 Tao Feng Create a jira for the viewer bug( > https://issues.apache.org/jira/browse/AIRFLOW-3745). > > On Mon, Jan 21, 2019 at 9:12 AM Deng Xiaodong wrote: > >> Hi Kaxil, >> >> >> After double-checking, I believe the View

Re: [VOTE] Airflow 1.10.2rc3

2019-01-21 Thread Tao Feng
Create a jira for the viewer bug( https://issues.apache.org/jira/browse/AIRFLOW-3745). On Mon, Jan 21, 2019 at 9:12 AM Deng Xiaodong wrote: > Hi Kaxil, > > > After double-checking, I believe the Viewer access/permission issue is not > a bug actually. Seems it's because the DAG-level access contr

Re: [VOTE] Airflow 1.10.2rc3

2019-01-21 Thread Deng Xiaodong
Hi Kaxil, After double-checking, I believe the Viewer access/permission issue is not a bug actually. Seems it's because the DAG-level access control feature is cherry-picked into 1.10.2rc3. But my understanding is this feature is still under development? @feng-tao kindly advise. On the other

Re: [VOTE] Airflow 1.10.2rc3

2019-01-21 Thread patchusj
+1 (non-binding) but it should be noted you have to run airflow initdb to get the super role added; i keep forgetting to do that. Cheers, Josh PXYData On 2019/01/21 09:53:15, Naik Kaxil wrote: > Hi XD, > > I could not replicate that in non-rbac UI but could replicate it on RBAC one. > > Pl

Re: [VOTE] Airflow 1.10.2rc3

2019-01-21 Thread Joshua Patchus
Yea this worked for me. I forgot you have to airflow initdb to create the superuser field, may want to note that in the docs somewhere. Cheers, Josh PXYData On 2019/01/19 16:35:49, Kaxil Naik wrote: > Hey all,> > > I have cut Airflow 1.10.2 RC3. This email is calling a vote on the release,> > wh

Re: [VOTE] Airflow 1.10.2rc3

2019-01-21 Thread Naik Kaxil
Hi XD, I could not replicate that in non-rbac UI but could replicate it on RBAC one. Please check this screenshot https://imgur.com/a/TgiQItO for non-RBAC (Flask-Admin UI) All the below issues don't seem to be a blocker for this release, however I am open to views from all. (1) Errors not sh

Re: [VOTE] Airflow 1.10.2rc3

2019-01-21 Thread Naik Kaxil
Hi XD, I will test this and let you know. One more bug cropped up https://issues.apache.org/jira/browse/AIRFLOW-3737 . @fokko or someone with more K8s experience can have a look at it, please? Regards, Kaxil On 21/01/2019, 09:05, "Deng Xiaodong" wrote: Hi Kaxil, I found anothe

Re: [VOTE] Airflow 1.10.2rc3

2019-01-21 Thread Deng Xiaodong
Hi Kaxil, I found another potential bug which is applicable for both RBAC and non-RBAC UI: Let’s say we’re trying to import a module which doesn’t exist at all, or have any syntax error in the DAG, the DAG will not be parsed. In addition, In earlier version, there will be a warning message appear

Re: [VOTE] Airflow 1.10.2rc3

2019-01-21 Thread Kaxil Naik
Hi Feng Tao, As mentioned in my previous email to XD, as this is not a BLOCKER and the fix exist as mentioned by Seelman, can you change your *vote back to +1 *. And we can fix that in the upcoming release with other fixes which should be quick and aim to release that in a month as well? RBAC is

Re: [VOTE] Airflow 1.10.2rc3

2019-01-20 Thread Kaxil Naik
Hi XD, Even if this issue exist, it should!'t block the release. As mentioned by Seelman, you can apply that fix. Remember RBAC is still not a default UI for this release. We should aim to have regular releases to fix this kinds of issues. As this is not blocking and the fix exists, can you chang

Re: [VOTE] Airflow 1.10.2rc3

2019-01-20 Thread Tao Feng
Thanks XD and Stefan. I see the issue now. I agree that this should be a bug which should be fixed. Please remove my +1 vote for release if possible. On Sun, Jan 20, 2019 at 11:12 PM Deng Xiaodong wrote: > Hi Feng Tao, > > I tried again by deleting the DB and initdb again, the issue is still >

Re: [VOTE] Airflow 1.10.2rc3

2019-01-20 Thread Deng Xiaodong
Hi Feng Tao, I tried again by deleting the DB and initdb again, the issue is still there. Please note the issue is not “see all the roles”. The issue is after logging in as a Viewer role, I can’t access the pages including Tree View, Graph View, Task Duration, Gantt, Code View, etc. XD On Mon,

Re: [VOTE] Airflow 1.10.2rc3

2019-01-20 Thread Stefan Seelmann
I was able to reproduce, to fix I modified the "Viewer" role via the UI and added permission "can dag read on all_dags" manually. On 1/21/19 6:49 AM, Deng Xiaodong wrote: > Hi Kaxil, > > A potential bug found in 1.10.2rc3. > > > # Potential Bug: > > Viewer Role can't access pages to which it

Re: [VOTE] Airflow 1.10.2rc3

2019-01-20 Thread Tao Feng
+1 (binding) I install the rc3 locally and create a new role and could view it in UI without issues. On Sat, Jan 19, 2019 at 8:36 AM Kaxil Naik wrote: > Hey all, > > I have cut Airflow 1.10.2 RC3. This email is calling a vote on the release, > which will last for 72 hours. Consider this my (bin

Re: [VOTE] Airflow 1.10.2rc3

2019-01-20 Thread Tao Feng
Hi Xiaodong, I just tried with a viewer role which can't reproduce your issue. I could see all the roles without any issues. Have you reset your db with this rc? On Sun, Jan 20, 2019 at 9:50 PM Deng Xiaodong wrote: > Hi Kaxil, > > A potential bug found in 1.10.2rc3. > > > # Potential Bug: > > V

Re: [VOTE] Airflow 1.10.2rc3

2019-01-20 Thread Deng Xiaodong
Hi Kaxil, A potential bug found in 1.10.2rc3. # Potential Bug: Viewer Role can't access pages to which it has permissions # How to Reproduce: - Under RBAC UI, create a user with "Viewer" role. Then use this account to log in. - You will be able to access the main page. However, you will not b

Re: [VOTE] Airflow 1.10.2rc3

2019-01-20 Thread Kaxil Naik
You won't need to build assets. When we release (create tar file), the assets are compiled using node and are already included. Hence, you would just need to do `pip install apache-airflow==1.10.2rc3` Regards, Kaxil On Sun, Jan 20, 2019, 21:53 Stefan Seelmann +1 (non-binding) > > Deployed in our

Re: [VOTE] Airflow 1.10.2rc3

2019-01-20 Thread Kaxil Naik
@seelman, sorry my bad - I didn't read that you build it from source. :-) Yes, you would need to build it using nom if you build it from source. Regards, Kaxil On Sun, Jan 20, 2019, 22:22 Kaxil Naik You won't need to build assets. When we release (create tar file), the > assets are compiled usin

Re: [VOTE] Airflow 1.10.2rc3

2019-01-20 Thread Stefan Seelmann
+1 (non-binding) Deployed in our test env and run the test suite. The update from 1.10.0 to 1.10.2rc3 was a bit bumpy: * DAG level access was only added in 1.10.2, not already in 1.10 as described in `UPDATING.md`. We use functions from `www_rbac/security.py` to create custom roles, but the file

Re: [VOTE] Airflow 1.10.2rc3

2019-01-19 Thread Bolke de Bruin
+1, binding Checked: * KEYS * Signatures * Git Tag Bolke > On 19 Jan 2019, at 17:35, Kaxil Naik wrote: > > Hey all, > > I have cut Airflow 1.10.2 RC3. This email is calling a vote on the release, > which will last for 72 hours. Consider this my (binding) +1. > > Airflow 1.10.2 RC3 is availa

[VOTE] Airflow 1.10.2rc3

2019-01-19 Thread Kaxil Naik
Hey all, I have cut Airflow 1.10.2 RC3. This email is calling a vote on the release, which will last for 72 hours. Consider this my (binding) +1. Airflow 1.10.2 RC3 is available at: https://dist.apache.org/repos/dist/dev/airflow/1.10.2rc3/ *apache-airflow-1.10.2rc3-source.tar.gz* is a source rel