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: add hdfs hive, hbase and hadoop addon #1375

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

shanshanying
Copy link
Contributor

@shanshanying shanshanying commented Dec 26, 2024

This PR includes addons for HDFS, Spark, and Hive, contributed by @kissycn.

@shanshanying shanshanying marked this pull request as draft December 26, 2024 02:57
@shanshanying shanshanying force-pushed the feat/add-hdfs-hive-spark branch 3 times, most recently from aef8ae7 to 5cddd97 Compare February 8, 2025 07:36
@codecov-commenter
Copy link

codecov-commenter commented Feb 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 0.00%. Comparing base (24c704b) to head (2ddbc69).

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #1375   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files         69      69           
  Lines       6900    6900           
=====================================
  Misses      6900    6900           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@shanshanying shanshanying marked this pull request as ready for review February 10, 2025 10:11
@shanshanying shanshanying requested review from lynnleelhl and a team as code owners February 10, 2025 10:11
@shanshanying shanshanying force-pushed the feat/add-hdfs-hive-spark branch from a9a855f to 2ddbc69 Compare February 10, 2025 10:13
@shanshanying shanshanying changed the title feat: add hdfs hive and spark addon feat: add hdfs hive, hbase and hadoop addon Feb 10, 2025

type: application

version: 0.9.1
Copy link
Contributor

Choose a reason for hiding this comment

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

0.9.1 -> 1.0.0-alpha.0


type: application

version: 0.9.0
Copy link
Contributor

Choose a reason for hiding this comment

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

ditto

// GNU Affero General Public License for more details.
//
// You should have received a copy of the GNU Affero General Public License
// along with this program. If not, see <http://www.gnu.org/licenses/>.
Copy link
Contributor

Choose a reason for hiding this comment

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

remove the comments

description: {{ .Chart.Description }}
serviceKind: hdfs-datanode
runtime:
hostNetwork: true
Copy link
Contributor

Choose a reason for hiding this comment

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

Is dynamic port allocation not necessary for this component?

runAsGroup: 0
runAsNonRoot: true
runAsUser: 10000
updateStrategy: BestEffortParallel
Copy link
Contributor

Choose a reason for hiding this comment

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

updateStrategy is incorrectly indented?

securityContext:
fsGroupChangePolicy: Always
fsGroup: 1000
updateStrategy: BestEffortParallel
Copy link
Contributor

@Y-Rookie Y-Rookie Feb 13, 2025

Choose a reason for hiding this comment

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

ditto,check them all

apiVersion: apps.kubeblocks.io/v1
kind: ComponentDefinition
metadata:
name: hbase-hmaster-2.5
Copy link
Contributor

Choose a reason for hiding this comment

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

add multi-version supported?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants