Skip to content
This repository was archived by the owner on Apr 28, 2020. It is now read-only.

Add subsystem health #362

Merged
merged 28 commits into from
Apr 17, 2019
Merged

Add subsystem health #362

merged 28 commits into from
Apr 17, 2019

Conversation

yaacov
Copy link
Contributor

@yaacov yaacov commented Apr 16, 2019

Add subsystem health

Requires kubevirt/web-ui#312

Ref: https://jira.coreos.com/browse/CNV-1151

Screenshots:
Peek 2019-04-17 15-17

@coveralls
Copy link

coveralls commented Apr 16, 2019

Pull Request Test Coverage Report for Build 1344

  • 76 of 83 (91.57%) changed or added relevant lines in 9 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 87.141%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/components/Dashboard/DashboardCard/DashboardCardTitleSeeAll.js 7 8 87.5%
src/components/Dashboard/Health/utils.js 12 14 85.71%
src/components/ClusterOverview/Health/Health.js 19 23 82.61%
Totals Coverage Status
Change from base Build 1307: 0.02%
Covered Lines: 3661
Relevant Lines: 4021

💛 - Coveralls

@coveralls
Copy link

Pull Request Test Coverage Report for Build 1259

  • 19 of 21 (90.48%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.2%) to 87.092%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/components/Dashboard/DashboardCard/DashboardCardTitleSeeAll.js 7 8 87.5%
src/components/SubsystemHealth/SubsystemHealth.js 8 9 88.89%
Totals Coverage Status
Change from base Build 1257: -0.2%
Covered Lines: 3432
Relevant Lines: 3767

💛 - Coveralls

@yaacov
Copy link
Contributor Author

yaacov commented Apr 16, 2019

Ref: kubevirt/web-ui#312

@yaacov yaacov force-pushed the subsystem-health branch from eb69abe to 783e0a2 Compare April 17, 2019 08:10
@yaacov yaacov changed the title [WIP] Add subsystem health Add subsystem health Apr 17, 2019
@yaacov
Copy link
Contributor Author

yaacov commented Apr 17, 2019

Removeing WIP,
@suomiy, @rawagner ( although you wrote most of it :-) ) please review.

To @rawagner 🎉 🍰 🍺 big thanks !

Copy link
Contributor

@rawagner rawagner left a comment

Choose a reason for hiding this comment

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

ClusterOverview/Health/HealthBody is still used in Compliance card - can you convert it to Dashboard/Health/HealthBody and them remove ClusterOverview/Health/HealthBody ?

title: PropTypes.string.isRequired,
};

export default DashboardCardTitleSeeAll;
Copy link
Contributor

Choose a reason for hiding this comment

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

can you remove this default export and export DashboardCardTitleSeeAll itself ? we usually do not use default exports

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure 👍

@@ -0,0 +1,46 @@
import React from 'react';
Copy link
Contributor

Choose a reason for hiding this comment

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

SubsystemHealth component should go under Dashboard folder

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍

@yaacov
Copy link
Contributor Author

yaacov commented Apr 17, 2019

moved subsystem into dashboard,

@rawagner please re review.

@rawagner rawagner merged commit 2fdf260 into kubevirt:master Apr 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants