[RESULT][VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-05 Thread Brent Bovenzi
Hi all, The vote for AIP-79 and AIP 84 has passed. AIP-79 https://cwiki.apache.org/confluence/display/AIRFLOW/AIP-79%3A+Remove+Flask+AppBuilder+as+Core+dependency AIP-84 UI REST API https://cwiki.apache.org/confluence/display/AIRFLOW/AIP-84+UI+REST+API Discussion thread: https://lists.apache.or

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-05 Thread Pierre Jeambrun
+1 (binding On Mon 5 Aug 2024 at 08:17, Ephraim Anierobi wrote: > +1 (binding) > > On Mon, 5 Aug 2024 at 03:58, Vishnu Chilukoori > wrote: > > > +1 (non-binding) for both > > > > On Sun, Aug 4, 2024 at 7:07 PM Wei Lee wrote: > > > > > +1 (binding) for both APIs > > > > > > Best, > > > Wei > >

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-04 Thread Ephraim Anierobi
+1 (binding) On Mon, 5 Aug 2024 at 03:58, Vishnu Chilukoori wrote: > +1 (non-binding) for both > > On Sun, Aug 4, 2024 at 7:07 PM Wei Lee wrote: > > > +1 (binding) for both APIs > > > > Best, > > Wei > > > > > On Aug 5, 2024, at 4:00 AM, Shahar Epstein wrote: > > > > > > +1 (binding) for both

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-04 Thread Vishnu Chilukoori
+1 (non-binding) for both On Sun, Aug 4, 2024 at 7:07 PM Wei Lee wrote: > +1 (binding) for both APIs > > Best, > Wei > > > On Aug 5, 2024, at 4:00 AM, Shahar Epstein wrote: > > > > +1 (binding) for both > > > > On Thu, Aug 1, 2024 at 9:20 PM Brent Bovenzi > > > wrote: > > > >> Hi, > >> > >> Je

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-04 Thread Wei Lee
+1 (binding) for both APIs Best, Wei > On Aug 5, 2024, at 4:00 AM, Shahar Epstein wrote: > > +1 (binding) for both > > On Thu, Aug 1, 2024 at 9:20 PM Brent Bovenzi > wrote: > >> Hi, >> >> Jed and I prepared two AIPs to flesh out how to finish AIP-38 Modern Web >> Application for Airflow 3.0

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-04 Thread Shahar Epstein
+1 (binding) for both On Thu, Aug 1, 2024 at 9:20 PM Brent Bovenzi wrote: > Hi, > > Jed and I prepared two AIPs to flesh out how to finish AIP-38 Modern Web > Application for Airflow 3.0. We would like to open it up for a vote. > > AIP-79 > > https://cwiki.apache.org/confluence/display/AIRFLOW/A

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-02 Thread Bishundeo, Rajeshwar
+1 non-binding -- Rajesh On 2024-08-02, 1:51 PM, "Buğra Öztürk" mailto:ozturkbugr...@gmail.com>> wrote: CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you can confirm the sender and know the content is safe. AVERTISS

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-02 Thread Buğra Öztürk
+1 non-binding On Fri, Aug 2, 2024 at 6:20 PM Elad Kalif wrote: > +1 binding > > I am not sure I agree with previous statements regarding the LDAP issue > (specifically about blocker or not blocker for Airflow 3) but let's not > discuss this here as it's out of scope for the AIPs we are voting h

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-02 Thread Elad Kalif
+1 binding I am not sure I agree with previous statements regarding the LDAP issue (specifically about blocker or not blocker for Airflow 3) but let's not discuss this here as it's out of scope for the AIPs we are voting here. On Fri, Aug 2, 2024 at 7:15 PM Vincent Beck wrote: > Big +1 to this

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-02 Thread Vincent Beck
Big +1 to this one. I think this is the major blocker on that workstream: deciding which tool we want to leverage to build the default auth manager for Airflow 3. Once that decision taken I'll be happy to help to implement the auth manager based upon that tool. On 2024/08/02 16:06:00 Jarek Pot

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-02 Thread Jarek Potiuk
Yeah. KeyCloak was just a repeating theme in a number of questions/issues some users raised (How do I integrate with KeyCloak) and it's well recognized in enterprise. But this is about as much as I know about it :) So yeah, if we have anyone who can dig deeper in the options we have or ideally som

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-02 Thread Vikram Koka
Agreed Jarek on the parallel workstream for auth and also that should not be a blocker for 3.0. I don't know if the right answer is actually Keycloak. There was some research done by my colleagues within Astronomer using Casbin for the same, but I don't know the differences between those and other

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-02 Thread Jarek Potiuk
Yeah. And (a little tangential) - I really feel that we should have a separate parallel workstream `Implement "proper" Auth Manager` (for example authorizing users via Keycloak) - which should be creating a new provider. Note that this provider should NOT have a way to manage users and roles - it s

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-02 Thread Jed Cunningham
> Just to verify, users will still be able connect FAB to LDAP by installing > FAB provider explicitly? Yes. That and configuring the FAB auth manager as the auth manager, as it won't be the default most likely. Being able to maintain that is a primary goal of this AIP. > But I want to make sur

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-02 Thread rom sharon
Meyer, Dr. Tanja Rückert > > > > > > -----Original Message- > > > From: Michał Modras > > > Sent: Thursday, August 1, 2024 10:23 PM > > > To: dev@airflow.apache.org > > > Subject: Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-01 Thread Jarek Potiuk
al Message----- > > From: Michał Modras > > Sent: Thursday, August 1, 2024 10:23 PM > > To: dev@airflow.apache.org > > Subject: Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core > > Dependency & UI REST API > > > > +1 for both, n

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-01 Thread Kaxil Naik
Grosch, Dr. Markus Heyn, Dr. Frank Meyer, Dr. Tanja Rückert > > -Original Message- > From: Michał Modras > Sent: Thursday, August 1, 2024 10:23 PM > To: dev@airflow.apache.org > Subject: Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core > Dependency &

RE: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-01 Thread Scheffler Jens (XC-AS/EAE-ADA-T)
Heyn, Dr. Frank Meyer, Dr. Tanja Rückert -Original Message- From: Michał Modras Sent: Thursday, August 1, 2024 10:23 PM To: dev@airflow.apache.org Subject: Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API +1 for both, non-binding On Thu, Aug

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-01 Thread Michał Modras
+1 for both, non-binding On Thu, Aug 1, 2024 at 9:34 PM Igor Kholopov wrote: > +1 (non-binding, both) > > On Thu, Aug 1, 2024 at 9:04 PM Vikram Koka > wrote: > > > +1 binding on both AIP-79 and AIP-84. > > > > Vikram > > > > > > On Thu, Aug 1, 2024 at 11:35 AM Vincent Beck > wrote: > > > > > +

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-01 Thread Igor Kholopov
+1 (non-binding, both) On Thu, Aug 1, 2024 at 9:04 PM Vikram Koka wrote: > +1 binding on both AIP-79 and AIP-84. > > Vikram > > > On Thu, Aug 1, 2024 at 11:35 AM Vincent Beck wrote: > > > +1 binding for both > > > > On 2024/08/01 18:33:51 Jed Cunningham wrote: > > > +1 binding! > > > > > > > --

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-01 Thread Vikram Koka
+1 binding on both AIP-79 and AIP-84. Vikram On Thu, Aug 1, 2024 at 11:35 AM Vincent Beck wrote: > +1 binding for both > > On 2024/08/01 18:33:51 Jed Cunningham wrote: > > +1 binding! > > > > - > To unsubscribe, e-mail: dev-un

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-01 Thread Vincent Beck
+1 binding for both On 2024/08/01 18:33:51 Jed Cunningham wrote: > +1 binding! > - To unsubscribe, e-mail: dev-unsubscr...@airflow.apache.org For additional commands, e-mail: dev-h...@airflow.apache.org

Re: [VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-01 Thread Jed Cunningham
+1 binding!

[VOTE] AIP-79 & AIP-84 Remove Flask AppBuilder as a Core Dependency & UI REST API

2024-08-01 Thread Brent Bovenzi
Hi, Jed and I prepared two AIPs to flesh out how to finish AIP-38 Modern Web Application for Airflow 3.0. We would like to open it up for a vote. AIP-79 https://cwiki.apache.org/confluence/display/AIRFLOW/AIP-79%3A+Remove+Flask+AppBuilder+as+Core+dependency AIP-84 UI REST API https://cwiki.apach