Skip to content

Update ACL during job reset #38741

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.

Sign up for GitHub

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

Jump to bottom
Merged
pankajkoti merged 7 commits into from
Apr 10, 2024
Merged

Update ACL during job reset #38741

pankajkoti merged 7 commits into from
Apr 10, 2024

Conversation

@SubhamSinghal
Copy link
Contributor

@SubhamSinghal SubhamSinghal commented Apr 4, 2024 •
edited by eladkal
Loading

closes: #38721

@SubhamSinghal
Copy link
Contributor Author

SubhamSinghal commented Apr 4, 2024

@Lee-W Please review changes.

@Lee-W Lee-W self-requested a review April 5, 2024 15:59
@subham611
Copy link
Contributor

subham611 commented Apr 7, 2024

@Lee-W Wait for CI images step is failing. How should I fix this, or simply rerun would work?

@Lee-W
Copy link
Member

Lee-W commented Apr 7, 2024

@Lee-W Wait for CI images step is failing. How should I fix this, or simply rerun would work?

Let me trigger rerun and see how it works

@subham611
Copy link
Contributor

subham611 commented Apr 8, 2024

@Lee-W How to fix Wait for CI images error. Need help here.

@potiuk potiuk force-pushed the bugfix/create-job-does-not-update-acl branch from 349e8a1 to 7c238cf Compare April 8, 2024 14:01
@potiuk
Copy link
Member

potiuk commented Apr 8, 2024

Rebased it -> your PR was 62 commits behind -> next time you can do it yourself too (see our contributing docs and search for rebase) https://github.com/apache/airflow/blob/main/contributing-docs/10_working_with_git.rst

@Lee-W
Copy link
Member

Lee-W commented Apr 8, 2024

Thanks @potiuk for help out!

@SubhamSinghal There's one static check failure. We might need your help to fix it

Lee-W
Lee-W approved these changes Apr 8, 2024
View reviewed changes
Copy link
Member

@Lee-W Lee-W left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall LGTM.

@Lee-W Lee-W requested a review from pankajkoti April 8, 2024 14:36
pankajkoti
pankajkoti approved these changes Apr 9, 2024
View reviewed changes
Copy link
Member

@pankajkoti pankajkoti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. One minor suggestion inline to optimise the number of conditional checks.

@pankajkoti pankajkoti merged commit 6f21f7d into apache:main Apr 10, 2024
@eladkal eladkal mentioned this pull request Apr 10, 2024
Closed
@SubhamSinghal SubhamSinghal deleted the bugfix/create-job-does-not-update-acl branch April 11, 2024 08:07
@SubhamSinghal SubhamSinghal restored the bugfix/create-job-does-not-update-acl branch April 11, 2024 08:10
This was referenced Apr 13, 2024
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Taragolis Taragolis Taragolis left review comments

@Lee-W Lee-W Lee-W approved these changes

@pankajkoti pankajkoti pankajkoti approved these changes

+1 more reviewer

@subham611 subham611 subham611 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

area:providers provider:databricks

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

CreateJobOperator does not udpdate ACL

6 participants

@SubhamSinghal @subham611 @Lee-W @potiuk @Taragolis @pankajkoti