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

feat(hadoop): Add patches to fix missing operationType for some operations in authorizer #555

Merged
merged 9 commits into from
Feb 15, 2024

Conversation

sbernauer
Copy link
Member

@sbernauer sbernauer commented Feb 9, 2024

Description

Part of stackabletech/hdfs-operator#400
https://issues.apache.org/jira/browse/HDFS-17378

Definition of Done Checklist

  • Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
  • Please make sure all these things are done and tick the boxes

Tasks

Preview Give feedback

@sbernauer sbernauer changed the title feat(hadoop): Add patches to fix authorizer properly checking create operations feat(hadoop): Add patches to fix missing operationType for some operations in authorizer Feb 14, 2024
@sbernauer sbernauer changed the title feat(hadoop): Add patches to fix missing operationType for some operations in authorizer feat(hadoop): Add patches to fix missing operationType for some operations in authorizer Feb 14, 2024
@sbernauer sbernauer self-assigned this Feb 14, 2024
NickLarsenNZ
NickLarsenNZ previously approved these changes Feb 15, 2024
Copy link
Member

@NickLarsenNZ NickLarsenNZ left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@NickLarsenNZ NickLarsenNZ left a comment

Choose a reason for hiding this comment

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

LGTM

@sbernauer
Copy link
Member Author

I'm going to merge this, as this is required for the authorizer and makes life easier.
We can always change the patches to what was merged upstream (if it will ever be ;))
So far I developed the authorizer quite far and the patch worked flawlessly

@sbernauer sbernauer added this pull request to the merge queue Feb 15, 2024
Merged via the queue into main with commit 661a38b Feb 15, 2024
13 checks passed
@sbernauer sbernauer deleted the fix/hdfs-authorizer branch February 15, 2024 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants