Skip to content

Add Minerva Direct API support#262

Merged
bcote-anl merged 1 commit into
mainfrom
feature/minerva-direct-api-status-mtls
Jun 30, 2026
Merged

Add Minerva Direct API support#262
bcote-anl merged 1 commit into
mainfrom
feature/minerva-direct-api-status-mtls

Conversation

@nscottnichols

Copy link
Copy Markdown
Contributor

This adds Minerva as a Direct API-backed cluster. Direct API requests can now use optional mTLS config, Minerva model availability is read from the login-node status endpoint, and Minerva endpoints reject requests when a model is not live.

The direct health monitor now checks live Minerva models through their configured mTLS routes.

Validation:

  • python -m compileall resource_server_async cron_jobs
  • Minerva /status tested through mTLS from the login-node prototype

@nscottnichols
nscottnichols requested a review from masalim2 June 29, 2026 22:05
@bcote-anl

Copy link
Copy Markdown
Collaborator

Hi @nscottnichols , thanks for the PR! Could you run "make format" and "make lint-fix" so that the test suite passes? We currently enforce a specific coding style. Thanks.

@nscottnichols
nscottnichols force-pushed the feature/minerva-direct-api-status-mtls branch from 20af1e1 to f6af499 Compare June 30, 2026 19:54
@masalim2

Copy link
Copy Markdown
Collaborator

For posterity, linking the Fixture Repo MR: https://git.cels.anl.gov/inference-service/gateway-api-fixtures/-/merge_requests/4

@masalim2 masalim2 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks great to me!

@bcote-anl
bcote-anl merged commit d1f6ef4 into main Jun 30, 2026
1 check passed
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