apache / apache/gravitino

[Improvement] Add priorityClassName support to Gravitino Helm charts

Open Beginner friendly
#10,865 1 comment 0 reactions 0 assignees View on GitHub
improvement
Dominant language
Java
Stars
3.2k
Forks
935
Avg merge
1d 16h
Merged PRs (30d)
298

Description

### What would you like to be improved?

The Gravitino, Iceberg REST, and Lance REST Helm charts do not expose `priorityClassName` on the pod spec. On shared clusters where batch jobs compete with production services, Kubernetes uses PriorityClass to decide which pods to evict. Without this, Gravitino pods have no priority tier and can be evicted by lower-priority workloads during resource pressure, causing unexpected downtime for production catalog services.

### How should we improve?

Add `priorityClassName: ""` to all three charts' `values.yaml` and conditionally render it in the pod spec in each `deployment.yaml`:

Applies to:
dev/charts/gravitino
dev/charts/gravitino-iceberg-rest-server
dev/charts/gravitino-lance-rest-server

Contributor guide

Open the contributing guide

Research direction

Start with values.yaml and deployment.yaml in dev/charts/gravitino, dev/charts/gravitino-iceberg-rest-server, and dev/charts/gravitino-lance-rest-server. Compare the pod-spec sections and existing optional fields, then render the charts to verify that priorityClassName is omitted when empty and present when configured. Done means all three charts expose and conditionally render the value.

Written by the indexing model from the issue text.

Assessment

Tech stack
helm, kubernetes
Domain
devops, infrastructure
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
78/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.