Skip to content
This repository was archived by the owner on Jun 5, 2025. It is now read-only.

feat: initial work on endpoints for creating/updating workspace config - #1107

Merged
lxmcgvrn merged 23 commits into
mainfrom
feat/configure-workspace-endpoint
Mar 5, 2025
Merged

feat: initial work on endpoints for creating/updating workspace config#1107
lxmcgvrn merged 23 commits into
mainfrom
feat/configure-workspace-endpoint

Conversation

@lxmcgvrn

@lxmcgvrnlxmcgvrn commented Feb 19, 2025•
edited
Loading

Copy link
Copy Markdown
Contributor

Related to #1067

  • Modifies the existing POST /workspaces
    • it exclusively handles workspace creation — the ability to rename via this endpoint is removed
    • it can handle custom instructions & muxes configuration as part of the same request
  • Adds a PUT /workspaces/:workspace_name
    • this is how you rename a workspace now
    • this can also handle updating custom instructions & muxes configuration

Has a companion PR for the dashboard to handle the breaking changes: stacklok/codegate-ui#349

ToDo:

  • GET /workspaces/:workspace_name or maybe (/workspaces/:workspace_name/config, I'm not sure yet) to get the configuration for a workspace

lxmcgvrn
lxmcgvrn commented Feb 19, 2025
View reviewed changes
Comment threadsrc/codegate/api/v1.py Outdated
@lxmcgvrnlxmcgvrn changed the title feat: initial work on endpoints for creating/updatingfeat: initial work on endpoints for creating/updating workspace configFeb 19, 2025
lxmcgvrn
lxmcgvrn commented Feb 19, 2025
View reviewed changes
Comment threadtests/api/test_v1_workspaces.py
@lxmcgvrnlxmcgvrn mentioned this pull request Feb 21, 2025
Merged
@lxmcgvrn
lxmcgvrn marked this pull request as ready for review February 21, 2025 16:38
@lxmcgvrn
lxmcgvrn marked this pull request as draft February 21, 2025 16:38
@lxmcgvrn
lxmcgvrn marked this pull request as ready for review March 4, 2025 09:26
JAORMX
JAORMX reviewed Mar 4, 2025
View reviewed changes
Comment threadsrc/codegate/muxing/models.py Outdated
JAORMX
JAORMX reviewed Mar 4, 2025
View reviewed changes
Comment threadsrc/codegate/api/v1.py
Comment threadsrc/codegate/workspaces/crud.py
Comment threadsrc/codegate/workspaces/crud.py Outdated
Comment threadsrc/codegate/workspaces/crud.py Outdated
Comment threadsrc/codegate/db/connection.py
@lxmcgvrn
lxmcgvrn merged commit f310e46 into mainMar 5, 2025
@lxmcgvrn
lxmcgvrn deleted the feat/configure-workspace-endpoint branch March 5, 2025 11:15
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer
@JAORMXJAORMXJAORMX approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lxmcgvrn@JAORMX