apache / apache/uniffle

[Improvement] [k8s] Make HadoopConfig configurable via ConfigMap

Open
#1,028 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
454
Forks
172
Avg merge
5d 17h
Merged PRs (30d)
5

Description

### Code of Conduct

- [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)

### Search before asking

- [X] I have searched in the [issues](https://github.com/apache/incubator-uniffle/issues?q=is%3Aissue) and found no similar issues.

### What would you like to be improved?

The current approach of directly copying the Hadoop configuration into the RSS server image is not elegant and can be improved.
https://github.com/apache/incubator-uniffle/blob/2e54ce212c957ece37beaa8dc3c06856c2aac583/deploy/kubernetes/docker/Dockerfile#L50

### How should we improve?

We can use ConfigMap to configure Hadoop configurations, similar to what is done in rss-configuration. I will submit a PR to complete this task.
https://github.com/apache/incubator-uniffle/blob/master/deploy/kubernetes/operator/examples/configuration.yaml

### Are you willing to submit PR?

- [X] Yes I am willing to submit a PR!

Contributor guide

Open the contributing guide

Research direction

Read deploy/kubernetes/docker/Dockerfile at line 50 and deploy/kubernetes/operator/examples/configuration.yaml first. Compare the current Hadoop configuration copy with the existing ConfigMap pattern, then verify that Kubernetes consumes the Hadoop configuration externally rather than from the RSS server image.

Written by the indexing model from the issue text.

Assessment

Tech stack
kubernetes
Domain
devops, infrastructure
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.