ModelEngine-Group / ModelEngine-Group/nexent

Bug: connectivity check blocks adding models in tenant resource management

Open
#3,463 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
5.9k
Forks
731
Avg merge
19h 34m
Merged PRs (30d)
172

Description

Bug Report

Describe the bug

In Tenant Resource Management → Models → Add custom model, the connectivity pre-check always reports the model as unavailable for a brand-new model that has not been saved yet, which blocks adding the model.

Steps to reproduce
  1. Open Tenant Resource Management, select a tenant
  2. Go to Models → Add custom model
  3. Fill in base_url / api_key / model_name
  4. Click connectivity test
Expected behavior

Connectivity test should verify the raw model config (base_url + api_key + model_name) and return available when the endpoint is reachable, without requiring the model to already exist in the database.

Actual behavior

The check always reports unavailable / blocks save.

Root cause

ModelAddDialog.handleVerifyConnectivity routes the manage flow (tenantId present) through checkManageTenantModelConnectivityPOST /model/manage/healthcheck, which looks up an already-persisted model by display_name + tenant_id. For a new model that lookup fails, so the check always fails.

Environment
  • Affected path: tenant resource management (manage flow)
  • Self-service add model path is unaffected (already uses /model/temporary_healthcheck)

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at ModelAddDialog.handleVerifyConnectivity and trace the manage flow through checkManageTenantModelConnectivity to POST /model/manage/healthcheck. Compare it with the self-service /model/temporary_healthcheck path and inspect how the new model's raw base_url, api_key, and model_name are handled. Done means a new manage-flow model can pass connectivity verification without an existing database record.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api, backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
72/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.