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

Include Dashboard into Radius releases #47

Closed
1 task
Tracked by #18
willtsai opened this issue Jan 30, 2024 · 1 comment
Closed
1 task
Tracked by #18

Include Dashboard into Radius releases #47

willtsai opened this issue Jan 30, 2024 · 1 comment
Labels
triaged This item has been triaged by project maintainers and is in the backlog

Comments

@willtsai
Copy link
Contributor

willtsai commented Jan 30, 2024

Overview of feature request

As a Radius user, I want the Radius Dashboard to be packaged into future releases of Radius so that I can install and run the Dashboard with my Radius installs without having to build the dashboard from source.

Acceptance criteria

  1. Dashboard is included into the Radius release process, including: publish/release pipelines, Helm authoring
  2. Dashboard is packaged as a part of the Radius release and can be installed along with the Radius control plane
  3. User is able to access the Dashboard after they install Radius without having to build dashboard from source

Additional context

Related:

Would you like to support us?

  • Yes, I would like to support you

AB#11022

@AaronCrawfis
Copy link
Contributor

Couple thoughts:

  • Installation of the dashboard to your cluster should be configurable via the Helm chart, where you can include the dashboard as part of the installation, or omit it
  • When running rad init and installing the Radius control-plane via the rad CLI, it should be automatically installed without any prompt/confirmation to the user. That way any downstream deep-linking or commands can assume it's there.
  • When installing via the Helm chart directly (helm install), we should think through if it should be opt-in or opt-out. My personal vote it opt-in, where a default Helm installation does not install the dashboard, at least for now. It increases the size of the default Radius installation and adds on a security risk for another service that could pose as surface area for attack.

@willtsai willtsai added the triaged This item has been triaged by project maintainers and is in the backlog label Jan 31, 2024
vinayada1 pushed a commit to radius-project/radius that referenced this issue Feb 27, 2024
# Description

Dependent on: radius-project/dashboard#51

## Type of change

<!--

Please select **one** of the following options that describes your
change and delete the others. Clearly identifying the type of change you
are making will help us review your PR faster, and is used in authoring
release notes.

If you are making a bug fix or functionality change to Radius and do not
have an associated issue link please create one now.

-->

- This pull request adds or changes features of Radius and has an
approved issue (issue link required).

<!--

Please update the following to link the associated issue. This is
required for some kinds of changes (see above).

-->

Part of: radius-project/dashboard#47

Signed-off-by: willdavsmith <willdavsmith@gmail.com>
Co-authored-by: Young Bu Park <youngp@microsoft.com>
@willtsai willtsai reopened this Feb 28, 2024
willdavsmith added a commit to willdavsmith/radius that referenced this issue Mar 4, 2024
…#7160)

# Description

Dependent on: radius-project/dashboard#51

## Type of change

<!--

Please select **one** of the following options that describes your
change and delete the others. Clearly identifying the type of change you
are making will help us review your PR faster, and is used in authoring
release notes.

If you are making a bug fix or functionality change to Radius and do not
have an associated issue link please create one now.

-->

- This pull request adds or changes features of Radius and has an
approved issue (issue link required).

<!--

Please update the following to link the associated issue. This is
required for some kinds of changes (see above).

-->

Part of: radius-project/dashboard#47

Signed-off-by: willdavsmith <willdavsmith@gmail.com>
Co-authored-by: Young Bu Park <youngp@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triaged This item has been triaged by project maintainers and is in the backlog
Projects
None yet
Development

No branches or pull requests

2 participants