Re: [PR] [RFC] fix(providers/airbyte): make OAuth credentials optional for Airbyte OSS deployments [airflow]

["aoelvp94 (via GitHub)" <gi...@apache.org>]

aoelvp94 commented on PR #64986:
URL: https://github.com/apache/airflow/pull/64986#issuecomment-4231674129

   > Still WIP ?
   
   Now RFC — apologies for the delay, busy days. I hit this issue at work, which motivated me to push the fix. I've been running a temporary patch on my operator, but the right solution is applying it directly to the hook.
   
   I did smoke tests on my side and everything worked well — both against an Airbyte deployment on EKS and a local quickstart one.
   
   I also put together a self-contained playground for reviewing the fix (and potentially other PRs related to this provider). It stands up a local no-auth Airbyte via abctl, runs the patched hook through AirbyteTriggerSyncOperator end-to-end, and lands real rows in a target Postgres. Repo: https://github.com/aoelvp94/airflow-airbyte-oss-playground/.
   
   <img width="2988" height="1666" alt="image" src="https://github.com/user-attachments/assets/c69fb982-44cd-4370-b4b7-c07020f9ebfc" />
   
   <img width="2862" height="1194" alt="image" src="https://github.com/user-attachments/assets/c5337ff3-bce3-45aa-ad24-bf0145baea05" />
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org