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

Fix for Bug : 1328085 #214

Open
wants to merge 16 commits into
base: feature-dev
Choose a base branch
from

Conversation

manojgn
Copy link
Contributor

@manojgn manojgn commented Jul 23, 2014

Following changes are implemented.

  1. Sort based on node color in "Host Name" column in node summary pages
  2. Move "version","CPU","Memory" columns to the end
  3. "Host name" column contents should be visible clearly with out wrapping. - Increased the column width to accomodate bigger hostnames.
  4. For "Interfaces", have total and down count in seperate lines
    12 Total
    5 Down
  5. Display color for all nodes in "Host Name" column.

knagakiran and others added 16 commits July 28, 2014 23:08
Updated menu.xml to make it compatible with the format to support multpl...
Made the updateView as a local function such that it doesn't get added
Following changes are implemented.
1. Sort based on node color in "Host Name" column in node summary pages
2. Move "version","CPU","Memory" columns to the end
3. "Host name" column contents should be visible clearly with out wrapping. - Increased the column width to accomodate bigger hostnames.
4. For "Interfaces", have total and down count in seperate lines
    12 Total
    5 Down
5. Display color for all nodes in "Host Name" column.

Change-Id: I258308cc2946366d26219d514e4b6d3fe1ae4252
    - Changed the UI instance parse function as per the latest stats changes

Change-Id: Idc934a0e4ff4bcef50ff4264052a1836abd4f0ac
Stats changes in Instance UVE,bug is #1352227
Replaced the use of process_state_list to NodeStatus:process_info

Change-Id: Ia6fdeaf059952dee6aa8b55a7c588c211da69951
…ty object in control node routes

Change-Id: Ifcf99ee6a5fb81ccfc2e816c4d8c4b0528951136
Fix for Bug 1351211: Update process_state_list to process_info
Replaced the following :
- contrail-vrouter to contrail-vrouter-agent
- contrail-opserver to contrail-analytics-api
- contrail-qe to contrail-query-engine

Change-Id: Idc27271793628d63b5be2a39ae2947ff4be47b8d
Fix for Bug:1354280 - UI changes to incorporate process name changes
… in infra monitor.

Had to reorganized the code to reuse the code to build the label value pairs.

Change-Id: I09ae7f3f33a743c88461ff3f41c717ee23a5880f
Fix for Bug: 1328085- Added the detail expansion to all summary pages in infra monitor.
    2)Implemented the dynamic axis params,while initializing the charts we need the pass the parameters that required to display in and X and Y axis eg:

    xAxisParams:[{lbl:'CPU (%)',key:'cpu',type:'float'},{lbl:'Instances',key:'instCnt'}]}};
    Parameters are
    "lbl" -  value displayed in the drop down
    "key" -  key is used the look in the chart data
    "type" - type of the value,if it is not present it will considered as the Integer

Conflicts:
	webroot/monitor/infra/common/ui/js/monitor_infra_utils.js
	webroot/monitor/infra/dashboard/ui/js/monitor_infra_dashboard.js

Change-Id: Iccf580e83627429022a908c1bf8310613f718173
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

3 participants