Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add frontend-endpoint and hide-side-menu-button to webserver's pipeline configs #1692

Merged
merged 4 commits into from
Nov 8, 2023

Conversation

rapsealk
Copy link
Member

@rapsealk rapsealk commented Nov 6, 2023

This PR is a follow-up of #1576

This PR adds additional configs frontend-endpoint and hide-button to webserver's pipeline configs.

  1. pipeline.frontend-endpoint will be used for "FastTrack" shortcut button on WebUI. Currently, the button is using pipeline.endpoint, which references the pipeline service's API server. The value has to be the pipeline service's frontend app as the button is intended to be a shortcut for the pipeline service.

  2. hide-button will provide an option to hide the "FastTrack" button or not.

image

Checklist: (if applicable)

  • Milestone metadata specifying the target backport version
  • Mention to the original issue
  • Documentation
  • API server-client counterparts (e.g., manager API -> client SDK)
  • Test case(s) to demonstrate the difference of before/after

@rapsealk rapsealk added this to the 23.09 milestone Nov 6, 2023
@rapsealk rapsealk self-assigned this Nov 6, 2023
@github-actions github-actions bot added comp:webserver Related to Web Server component size:S 10~30 LoC labels Nov 6, 2023
Copy link
Contributor

@agatha197 agatha197 left a comment

Choose a reason for hiding this comment

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

In lablup/backend.ai-webui#1929, the fasttrack's endpoint is saved named endpoint not fasttrack-endpoint. If you want to rename this, it needs a new PR to handle that in webui.

Copy link
Contributor

@agatha197 agatha197 left a comment

Choose a reason for hiding this comment

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

LGTM!

@rapsealk rapsealk changed the title feat: Add frontend-endpoint and hide-button to webserver's pipeline configs feat: Add frontend-endpoint and hide-side-menu-button to webserver's pipeline configs Nov 8, 2023
@yomybaby
Copy link
Member

yomybaby commented Nov 8, 2023

After this PR merged, let's make a WebUI PR to use these options.

@kyujin-cho kyujin-cho added this pull request to the merge queue Nov 8, 2023
Merged via the queue into main with commit da97265 Nov 8, 2023
20 checks passed
@kyujin-cho kyujin-cho deleted the feature/additional-configs-for-pipeline branch November 8, 2023 07:01
kyujin-cho pushed a commit that referenced this pull request Nov 27, 2023
…r's pipeline configs (#1692)

Backported-from: main
Backported-to: 23.09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp:webserver Related to Web Server component size:S 10~30 LoC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants