Skip to content

[WIP][V2]Sandbox console UI#7011

Draft
popojk wants to merge 6 commits intov2from
sandbox_console_ui
Draft

[WIP][V2]Sandbox console UI#7011
popojk wants to merge 6 commits intov2from
sandbox_console_ui

Conversation

@popojk
Copy link
Contributor

@popojk popojk commented Mar 11, 2026

Tracking issue

Closes #6927

Why are the changes needed?

We should add UI to sandbox for easier debugging

What changes were proposed in this pull request?

Add a console deployment in sandbox

How was this patch tested?

Run make sandbox-build then make sandbox-run. A console pod will appeared.

Then access to the UI in browser via http://localhost:30080/v2/domain/development/project/flytesnacks/runs

Labels

Please add one or more of the following labels to categorize your PR:

  • added: For new features.

This is important to improve the readability of release notes.

Check all the applicable boxes

  • I updated the documentation accordingly.
  • All new and existing tests passed.
  • All commits are signed-off.

Related PRs

Docs link

popojk and others added 5 commits March 11, 2026 14:28
Signed-off-by: Alex Wu <c.alexwu@gmail.com>
Signed-off-by: Alex Wu <c.alexwu@gmail.com>
Signed-off-by: Alex Wu <c.alexwu@gmail.com>
Signed-off-by: Alex Wu <c.alexwu@gmail.com>
Signed-off-by: Alex Wu <c.alexwu@gmail.com>
@github-actions github-actions bot mentioned this pull request Mar 11, 2026
3 tasks
$(foreach arch,amd64 arm64,$(call FLYTE_BINARY_BUILD,$(arch)))

CONSOLE_IMAGE := 643379628101.dkr.ecr.us-east-1.amazonaws.com/union-cp/unionconsole
CONSOLE_TAG := 2026.2.3
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a work around for now, we should pull image from oss registry in the future

Signed-off-by: Alex Wu <c.alexwu@gmail.com>
@popojk popojk changed the title Sandbox console UI [WIP][V2]Sandbox console UI Mar 11, 2026
@@ -0,0 +1,38 @@
{{- if .Values.sandbox.console.enabled }}
apiVersion: apps/v1
kind: Deployment
Copy link
Member

Choose a reason for hiding this comment

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

We also need to add this to charts/flyte-binary

.claude/

# Console build output
console/dist/
Copy link
Member

Choose a reason for hiding this comment

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

remove? No need it now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants