Page MenuHomePhabricator

[Refine Refactoring] Refactor refinery code for compatibility with Airflow integration
Closed, ResolvedPublic8 Estimated Story Points

Details

Event Timeline

Ahoelzl set the point value for this task to 8.Feb 1 2024, 3:00 AM

Change 1003745 had a related patch set uploaded (by Joal; author: Joal):

[analytics/refinery/source@master] [WIP] Extract RefineSingleApp code from Refine

https://gerrit.wikimedia.org/r/1003745

I had a question about this, because I just found out that refinery has its own email sending code. (re: https://gerrit.wikimedia.org/r/c/analytics/refinery/source/+/1014004)

Until this point, I was under the impression that the emails we were getting from refinery were the standard stdout/stderr based emails emitted by systemd timers.

Would we still want this integrated email functionality within refinery, when it's running under airflow?

Would we still want this integrated email functionality within refinery, when it's running under airflow?

Nope, when running from Airflow, no email would get sent by the Refine process itself, but rather by Airflow :)

Change #1003745 merged by Aqu:

[analytics/refinery/source@master] Extract RefineSingleApp code from Refine

https://gerrit.wikimedia.org/r/1003745