Meshery: [UI] MeshSync: Include list of existing namespaces in /management namespace text field

Created on 1 Dec 2020  路  9Comments  路  Source: layer5io/meshery

Current Behavior
The individual service mesh management pages have a namespace field with "default" prepopulated.

Desired Behavior
The namespace field should not only show "default" as an option, but should also include the list of namespaces that are currently present in the user's Kubernetes environment. Pre-populating this text field will be a UX improvement and it will also help with cleanliness of input.

In general, the namespace field should be behave similarly as to how the "Mesh Adapter URL" field functions on the Settings page - http://localhost:9081/settings#service-mesh

Implementation
Using the MeshSync endpoint at /api/mesh/scan (e.g. http://localhost:9081/api/mesh/scan), populate the Namespace field (shown in the screenshot below) with the namespaces returned in this API.

Screen Shot 2020-12-01 at 3 05 51 PM

  • List "default" in light gray text as the default value for this text field.
  • Enhance text field to include a dropdown selector as well as text from the user.

Acceptance Tests

  1. User can type in a string of a new namespace.
  2. User can select the name of any existing namespace.
  3. If no namespace is returned in the MeshSync API, show "default" in the text field.
  4. Namespaces should be alphabetically ordered.
  5. If one or more namespaces are returned in the MeshSync API, show the first namespace as the select item.
componenui frameworreact good first issue help wanted kinenhancement

All 9 comments

I would like to take up this issue.

Very good!

Hi @leecalcote,

Can I also work on this issue?

Nice to hear of your collaboration here, @Keenal and @MaximeCoignus

Is this Issue from Meshery Websites or the Documentation?

Is this Issue from Meshery Websites or the Documentation?

It's a Meshery UI issue. Not related to Documentation.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@Keenal @MaximeCoignus I'll remove your assignment to the issue, assuming that you've moved on. Just signal if that is not the case.

@leecalcote We might want to "graphqlise" this?
// cc @kumarabd

Was this page helpful?
0 / 5 - 0 ratings

Related issues

chandrashritii picture chandrashritii  路  3Comments

leecalcote picture leecalcote  路  4Comments

leecalcote picture leecalcote  路  3Comments

leecalcote picture leecalcote  路  4Comments

leecalcote picture leecalcote  路  4Comments