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

specification for cluster list view #82

Merged
merged 1 commit into from
Apr 28, 2017

Conversation

gnehapk
Copy link
Member

@gnehapk gnehapk commented Dec 19, 2016


* To add the response format for the API call
* To add the polling API call and its response format

Copy link
Member Author

Choose a reason for hiding this comment

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

@anivargi Please provide the required inputs for API calls.

@gnehapk
Copy link
Member Author

gnehapk commented Dec 19, 2016

Addresses #75.

* Cluster Name
* Status
* Storage Usage
* CPU Utilisation
Copy link
Contributor

Choose a reason for hiding this comment

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

typo? is it CPU utilization?

* For showing storage usage and cpu utilisation, use donut chart component.
* Implement client side filer and sorting functionality.
* Filter call will trigger after 1 sec the user enters the filter data.
* Send polling call after 10 mins to get the updated data.
Copy link
Contributor

Choose a reason for hiding this comment

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

10 mins?


==== Tendrl API impact:

/api/1.0/GetClusterList
Copy link
Contributor

Choose a reason for hiding this comment

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

This part lack information. state what is available currently(cluster wide list API available), what extra information reqd in the current API. Also specify the any extra APIs required(syatem wide listing?)
Which component implements sorting and filtering?
Any APis required to get the utlization data?


== References:

https://redhat.invisionapp.com/share/BR8JDCGSQ#/screens/185937524
Copy link
Contributor

Choose a reason for hiding this comment

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

Add a reference to topic issue as well here - #75


==== Notifications/Monitoring impact:

Monitoring data support is not present for cluster list.
Copy link
Contributor

Choose a reason for hiding this comment

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

what about cluster utilization?

@gnehapk
Copy link
Member Author

gnehapk commented Dec 20, 2016

@cloudbehl @anivargi I have updated the PR. Please review it.

* Cluster Name
* Status
* Storage Usage
* Utilisation
Copy link

@ltrilety ltrilety Dec 20, 2016

Choose a reason for hiding this comment

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

Is it CPU utilization?
Anyway is it a good information at all to have CPU or memory utilization for cluster? What does it say about cluster?

@gnehapk
Copy link
Member Author

gnehapk commented Dec 21, 2016

@anivargi For the cluster list view, below are the data that is not present in the json response of cluster -

  1. Status
  2. IOPS
  3. Host count
  4. Pool count
  5. Last Sync
  6. Provider

These are the data(with their mapped attribute) which are provided by API -

  1. name - maps.mon_status.cluster_name
  2. Storage(donut chart) - stats.storage
  3. Alert count - stats.alert_cnt

Please confirm, whether these mappings are correct or not?

@mkudlej
Copy link

mkudlej commented Jan 2, 2017

Please add usmqe/usmqe-tests#16 for testing reference.

@r0h4n r0h4n merged commit 0dc1c02 into Tendrl:master Apr 28, 2017
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.

None yet

6 participants