Allow to disable owner inheritance from dbt into airflow DAG owners

[CorsettiS]

Description

From slack
Hey everyone! :wave: I'm working on an Airflow DAG that uses DbtTaskGroup (Cosmos v1.10.1). I've noticed that the Airflow tasks are showing both the DAG owner (set in default_args) and the owner defined in my dbt model's _doc_.yml. Is there a way to configure Cosmos to not pull the owner from the dbt model? I'm aiming to keep the DAG owner clean. Any insights would be greatly

This PR introduces the flag disable_owner_inheritance which should make it possible

Related Issue(s)

Breaking Change?

No

Checklist

• I have made corresponding changes to the documentation (if required)
• I have added tests that prove my fix is effective or that my feature works

[netlify]

✅ Deploy Preview for sunny-pastelito-5ecb04 canceled.

Name	Link
🔨 Latest commit	6645725
🔍 Latest deploy log	https://app.netlify.com/projects/sunny-pastelito-5ecb04/deploys/683c13c7f92cb00008abc68c

[corsettigyg]

• For reference, I created the disable_owner_inheritance to avoid making it into a breaking change, although I particularly believe that the default behaviour should be enable_owner_inheritance and this one should be set to False by default. To avoid headaches, I kept things as they are though.

• Since the owners are being propagated in many places and now it is possible to disable/enable this behaviour, creating the tests is no pleasant task since it is a very repetitive process if we want to hit the coverage on the changed lines for all situations. I ended up using AI to automate most of the test creation since it would be very time consuming to do it manually. On a quick look the tests looked fine to me, but as a traditional data engineer I might still take a better look later once I have more time 😄

[tatiana]

This looks great, @CorsettiS , thanks a lot for contributing! Since it is a new feature, we'll release it as part of Cosmos 1.11.

I just enabled the integration tests and left a minor feedback inline.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.02%. Comparing base (003969e) to head (6645725).
Report is 4 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1787   +/-   ##
=======================================
  Coverage   98.02%   98.02%           
=======================================
  Files          85       85           
  Lines        5256     5263    +7     
=======================================
+ Hits         5152     5159    +7     
  Misses        104      104           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[pankajkoti]

LGTM. couple of minor suggestions in-line

[tatiana]

Hi @corsettigyg could we rename disable_owner_inheritance to enable_owner_inheritance to avoid double negation? I had commented this inline, but I think it got lost in one of the previous commits. We can set it to True to avoid being a breaking change, as you suggested - and we can review the default in Cosmos 2.x

[corsettigyg]

@tatiana done 😄 I thought about adding a deprecation warning to it but it would be annoying to have a constant warning showing up.

[linchun3]

Thanks for this PR! It should help to close #1728

Any idea on the timeline for the release of Cosmos 1.11?

[tatiana]

Thanks a lot, @corsettigyg ! @linchun3 We'll release this in 1.10.2 (although it can be perceived as a feature, it is also a sort of bug fix for some users). The estimate is to release it on 6 June (Friday)