alifaz31 commented on issue #64195:
URL: https://github.com/apache/airflow/issues/64195#issuecomment-4122746397

   **Tested `alibaba: 3.3.6rc1`** — fix(providers/alibaba): pass relative path 
to oss_write in OSSRemoteLogIO.upload (#63246)
   
   I am the original issue reporter (#63242).
   
   **Environment:** Airflow 3.1.6 · KubernetesExecutor + CeleryExecutor · 
Python 3.12 · Alibaba Cloud OSS
   
   **Code-level check (inside pod):**
   ```python
   import inspect
   from airflow.providers.alibaba.cloud.log.oss_task_handler import 
OSSRemoteLogIO
   src = inspect.getsource(OSSRemoteLogIO.upload)
   print("✅ PASS: buggy line gone" if "oss_write(log, remote_loc)" not in src 
else "❌ FAIL: buggy line still present")
   print("✅ PASS: fix present" if "oss_write(log, relative_path)" in src else 
"❌ FAIL: fix not found")
   ```
   
   **Result:** Both checks passed. Triggered a DAG via Airflow UI — logs 
visible in UI and written to correct OSS path:
   ```
   # ❌ Before: 
s-airflow/logs/oss://bucket/s-airflow/logs/dag_id=.../attempt=1.log
   # ✅ After:  s-airflow/logs/dag_id=.../run_id=.../task_id=.../attempt=1.log
   ```
   
   Fix confirmed end-to-end. ✅ LGTM for release.
   


-- 
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