hussein-awala commented on PR #30558:
URL: https://github.com/apache/airflow/pull/30558#issuecomment-1668523646

   @potiuk I tested this PR with 2.4.0 (the current min version) and with main, 
I can create connections with and without the `extra__azure__` prefix without 
any issue. What kind of tests should I do before merging it?
   And for this commet:
   
   > Hmm. I think the problem was that without the "ensure prefixes" 
providers_manager woudl crash during building the UI fir connections or at 
least wreak some havoc. I think we would need to build such provder and install 
it in airflow 2.3 to be sure.
   
   There is no problems with the UI:
   
![image](https://github.com/apache/airflow/assets/21311487/f151c9e8-a2b0-4aab-ba62-59931724bd60)
   
![image](https://github.com/apache/airflow/assets/21311487/1dc80db3-a0ef-4c19-8d77-97947ae51dfb)
   
![image](https://github.com/apache/airflow/assets/21311487/f90bab1f-a3b3-46ed-84ca-fa50393e52f4)
   (6.2.3 because I built it with breeze and installed it from source in a 
clean conda env)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to