Skip to content

Add editor.ui.k8s.appscode.com API group#653

Closed
tamalsaha wants to merge 1 commit into
masterfrom
editor-api
Closed

Add editor.ui.k8s.appscode.com API group#653
tamalsaha wants to merge 1 commit into
masterfrom
editor-api

Conversation

@tamalsaha

Copy link
Copy Markdown
Contributor

What

Adds a new read-only aggregated API group editor.ui.k8s.appscode.com/v1alpha1 with two create-only action kinds:

  • EditorRender — renders an editor model, manifest or resources from a set of options, without touching any existing installation. Equivalent of the b3 /clusters/{owner}/{cluster}/helm/options/{model,manifest,resources} endpoints.
  • EditorTemplate — loads the editor model, manifest or resources for an existing installation identified by its metadata. Equivalent of the b3 /clusters/{owner}/{cluster}/helm/editor/{model,manifest,resources} endpoints.

Each kind carries a Request/Response and selects the artifact via an output discriminator (model | manifest | resources).

Why

These types back the read-only editor endpoints served in-cluster by kube-ui-server (which authorizes the resources touched as the extended-API caller, via impersonation) and consumed by b3. Apply/delete remain out of scope (read-only only).

Generated

make gen output: deepcopy, openapi, and clientset for the new group. Unrelated groups are untouched.

Part of a 3-repo change: resource-metadata (types) → ui-server (registry/handlers) → b3 (consumption).

Add read-only aggregated API types EditorRender and EditorTemplate under a
new editor.ui.k8s.appscode.com/v1alpha1 group:

- EditorRender renders an editor model, manifest or resources from a set of
  options, without touching any existing installation.
- EditorTemplate loads the editor model, manifest or resources for an
  existing installation identified by its metadata.

These back the read-only editor endpoints served by kube-ui-server (which
authorizes the resources touched as the API caller) and consumed by b3.

Signed-off-by: Tamal Saha <tamal@appscode.com>
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.

1 participant