Skip to content

use codehost_name to resolve codehost when loading service via openapi - #4900

Merged
PetrusZ merged 2 commits into
koderover:mainfrom
Cynthia-0203:feat/openapi-load-service-by-codehost-name
Aug 18, 2026
Merged

use codehost_name to resolve codehost when loading service via openapi#4900
PetrusZ merged 2 commits into
koderover:mainfrom
Cynthia-0203:feat/openapi-load-service-by-codehost-name

Conversation

@Cynthia-0203

@Cynthia-0203 Cynthia-0203 commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

What this PR does / Why we need it:

The OpenAPI "create K8s YAML service from codebase" endpoint previously identified the code host via the codehostId path parameter. This PR changes it to accept codehost_name instead, and the backend resolves the code host by its alias (system-level) to obtain the codehost_id before loading the service.

What is changed and how it works?

  • request field CodehostID int → CodehostName string; Validate() now requires codehost_name to be non-empty.
  • route changed from /load/:codehostId to /load.

Does this PR introduce a user-facing change?

  • API change
  • database schema change
  • upgrade assistant change
  • change in non-functional attributes such as efficiency or availability
  • fix of a previous issue

This change is Reviewable

…openapi

Signed-off-by: Cynthia <chenchen@koderover.com>
… via openapi

Signed-off-by: Cynthia <chenchen@koderover.com>
@PetrusZ
PetrusZ merged commit 682a6d8 into koderover:main Aug 18, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants