-
Notifications
You must be signed in to change notification settings - Fork 14.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
BugFix: DAG trigger via UI error in RBAC UI #8411
Conversation
@@ -1062,6 +1062,7 @@ def trigger(self, session=None): | |||
conf=conf | |||
) | |||
|
|||
dag = dagbag.get_dag(dag_id) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lol what? Do we have no tests of this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This only errors if the Webserver does not have access to Dag Files (like was mentioned in the issue when the DAGs dir was not mounted to Webserver).
We might have tests to see when the config is enabled this still works but we might not have tests to check the Webserver doesn't have access to Dag file.
Have kept the PR in draft so that I can add some tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ashb I have added the tests to cover this case.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will add these tests to Master too once this PR is merged
resolves apache#8247 (cherry picked from commit cf43394)
resolves #8247
Make sure to mark the boxes below before creating PR: [x]
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.
Read the Pull Request Guidelines for more information.