Skip to content

Conversation

@hellolittlej
Copy link
Collaborator

@hellolittlej hellolittlej commented Dec 9, 2025

Context

  • Enable access control for the job cluster
  • Add new field job principal to allow user specify the principal when create the new job cluster, the principal will be used to enforce the job cluster management.

Explain context and other details for this pull request.

Backward compatibility

READ

  • the jobPrincipal field will be omitted from the get request if it's absent from the data store

WRITE

  • client can send the payload without jobPrincipal field

TEST

local test with internal repo

Checklist

  • ./gradlew build compiles code correctly
  • Added new tests where applicable
  • ./gradlew test passes all tests
  • Extended README or added javadocs where applicable

@github-actions
Copy link

github-actions bot commented Dec 9, 2025

Test Results

153 files  +1  153 suites  +1   9m 29s ⏱️ +31s
664 tests +2  652 ✅ +1  11 💤 ±0  1 ❌ +1 
665 runs  +3  653 ✅ +2  11 💤 ±0  1 ❌ +1 

For more details on these failures, see this check.

Results for commit d0cb7f6. ± Comparison against base commit bff0aeb.

♻️ This comment has been updated with latest results.

@hellolittlej hellolittlej requested a deployment to Integrate Pull Request December 16, 2025 21:30 — with GitHub Actions Waiting
@hellolittlej hellolittlej merged commit 32f543f into master Dec 16, 2025
5 of 7 checks passed
@hellolittlej hellolittlej deleted the add-job-principal branch December 16, 2025 21:46
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