Skip to content
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

Can Delete on Task Instances permission is required to clear task #34092

Closed
1 of 2 tasks
Raul824 opened this issue Sep 5, 2023 · 1 comment · Fixed by #34123
Closed
1 of 2 tasks

Can Delete on Task Instances permission is required to clear task #34092

Raul824 opened this issue Sep 5, 2023 · 1 comment · Fixed by #34123
Labels
area:core kind:bug This is a clearly a bug needs-triage label for new issues that we didn't triage yet

Comments

@Raul824
Copy link

Raul824 commented Sep 5, 2023

Apache Airflow version

Other Airflow 2 version (please specify below)

What happened

I was changing the roles to remove all delete access. But as soon as I remove permission "Can Delete on Task Instances" clear task takes to a blank page.

Shouldn't clear task access work without Delete access?

What you think should happen instead

Can Delete Task Instance should not affect clear task access.

How to reproduce

Copy the Op role and remove all Can Delete and assign that role to a user.
That user will not be able to clear task instance now.
add "Can Delete on Task Instances" permission to the role and user will now have access to clear task instances page.

Operating System

Redhat 8

Versions of Apache Airflow Providers

2.6.1

Deployment

Docker-Compose

Deployment details

airflow 2.6.1 running on Docker with postgres 13 and pgbouncer.

Anything else

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@Raul824 Raul824 added area:core kind:bug This is a clearly a bug needs-triage label for new issues that we didn't triage yet labels Sep 5, 2023
@hussein-awala
Copy link
Member

I created #34123 to discuss this issue, because I am not sure if it's a bug or if there was some reason to use delete instead of edit for TI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:core kind:bug This is a clearly a bug needs-triage label for new issues that we didn't triage yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants