Delete comments <a href="http://www.aptis.info/portfolio/delete-all-comments/" target="_blank" title="Help"> <span class="aui-icon aui-icon-small aui-iconfont-help aptis-delete-all-comments-title-icon">Help</span> </a>

Mark it as future work here as well for clarity

Shubham Mehta | Mar 28, 2023

Slightly different text - It makes user management component of Airflow pluggab...

Shubham Mehta | Mar 28, 2023

+ Native user management support in cloud providers means that roles can be map...

Shubham Mehta | Mar 28, 2023

I think as part of this AIP we can deprecate the REST API for user management. ...

Jarek Potiuk | Apr 24, 2023

I am good with that. That would actually makes things simpler. Just a note abou...

Vincent BECK | May 02, 2023

Same here. The CLI should be disabled when other than back-compat FAB user mana...

Jarek Potiuk | Apr 24, 2023

Agree with same comment as the previous one

Vincent BECK | May 02, 2023

No need for that (same reason as above).

Jarek Potiuk | Apr 24, 2023

I agree with you but only for the Rest API/CLI section of the table. We still c...

Vincent BECK | May 02, 2023

Again - when other than back-compat FAB user manager is used, Airflow should no...

Jarek Potiuk | Apr 24, 2023

I agree with you but similarly to my comment on Rest API and CLI, we should dis...

Vincent BECK | May 02, 2023

Hello vin100.beck Great work to write this AIP, it was very clear to me Two c...

Philippe Lanoe | Jun 08, 2023

Thank planoe ! Funny, I first named it "resource-type" then changed my mind ...

Vincent BECK | Jun 08, 2023

Ok then let's rename it to resource-type Well naming "resource-details" beca...

Philippe Lanoe | Jun 08, 2023

Done I am not sure I understand what you mean by "having a wrapper method fo...

Vincent BECK | Jun 08, 2023

planoe - this PR (https://github.com/apache/airflow/pull/27892 https://github.c...

Shubham Mehta | Jun 08, 2023

Thanks shubh.mehta vin100.becklet me summarize our private conversation: As ...

Philippe Lanoe | Jun 09, 2023

I think we should not mixed up both topics: AIP-44's purpose is to moderate D...

Vincent BECK | Jun 09, 2023

I agree on the abstract wrapper, we can easily create an interface/facade betwe...

Vincent BECK | Jun 09, 2023

I am also not sure we should consider auth managers as untrusted. The code will...

Vincent BECK | Jun 09, 2023

Why doesn't AIP-56 ensure that DB should not be accessed from untrusted compon...

Jarek Potiuk | Jun 09, 2023

should we call it auth_manager for consistency?

Shubham Mehta | Jun 08, 2023

Good catch!

Vincent BECK | Jun 08, 2023

vin100.beck I did not mean that untrusted component should not access the DB, I...

Philippe Lanoe | Jun 12, 2023

I do not agree it makes no sense. Those two efforts are on-going, and they are ...

Jarek Potiuk | Jun 12, 2023

First of all thanks a lot for your detailed explanation, it was really useful. ...

Philippe Lanoe | Jun 12, 2023

> Since we delegate the DAG Authoring access to upstream systems (NFS, Git etc....

Jarek Potiuk | Jun 12, 2023

Apologies for the miss on the CLI. AIP-56 goal is about doing the same for UI...

Philippe Lanoe | Jun 12, 2023

And just to correct one other misconception: "AIP-56 is about user access mana...

Jarek Potiuk | Jun 12, 2023

Sorry for this statement, it was incorrect. It is clear now

Philippe Lanoe | Jun 12, 2023

Delete comments Help