Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(hadoop): Add hdfs-utils and topology-provider to classpath by default #569

Merged
merged 1 commit into from
Feb 21, 2024

Conversation

sbernauer
Copy link
Member

@sbernauer sbernauer commented Feb 19, 2024

Description

Previously you had to fiddle around using HADOOP_CLASSPATH.
I don't know if this is the right direction, feedback welcome!

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 self-assigned this Feb 20, 2024
@sbernauer sbernauer added this pull request to the merge queue Feb 21, 2024
Merged via the queue into main with commit d614b4a Feb 21, 2024
@sbernauer sbernauer deleted the fix/hadoop-utils-classpath branch February 21, 2024 07:14
sbernauer added a commit to stackabletech/hdfs-operator that referenced this pull request Feb 21, 2024
github-merge-queue bot pushed a commit to stackabletech/hdfs-operator that referenced this pull request Feb 28, 2024
* feat: Add OPA authorizer

* feat: First draft of CRD structure

* changelog

* let tests pass again

* re-enable chaos monkey

* re-add pull policy

* add some docs

* charts

* remove comment

* Move example into file

* Remove uneeded HADOOP_CLASSPATH, as stackabletech/docker-images#569 was merged

* Add section on username extraction

* Adopt to CRD decision

* Rip out GroupMapper

* changelog

* add needed file

* round of docs

* improve docs

* improve docs

* english linter

* add doc comments

* added group entry for check-hdfs

* Apply suggestions from code review

Co-authored-by: Andrew Kenworthy <andrew.kenworthy@stackable.de>

* Update docs/modules/hdfs/pages/usage-guide/security.adoc

Co-authored-by: Andrew Kenworthy <andrew.kenworthy@stackable.de>

* comment out

* Improve rego

* Set hadoop.user.group.static.mapping.overrides in case OPA is not enabled

* Use envsubst instead of kubectl apply inline

* Only envsubst $NAMESPACE

---------

Co-authored-by: Andrew Kenworthy <andrew.kenworthy@stackable.de>
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants