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

Set Status of Physical Servers and Physical Switches in Topology View #4288

Merged
merged 1 commit into from
Jul 13, 2018
Merged

Set Status of Physical Servers and Physical Switches in Topology View #4288

merged 1 commit into from
Jul 13, 2018

Conversation

caiocmpaes
Copy link
Contributor

What this PR does:

  • Set the status of the PhysicalServers and PhysicalSwitches nodes in the topology view to be relative to the entity's health_state.

physical_topology
As the picture shows, the nodes get borders in one of this colors:

  • Gray: Unknown Health State status;
  • Green: Valid Health State status;
  • Yellow: Warning Health State status;
  • Red: Critical Health State status;

The health_state status of the PhysicalServers and PhysicalSwitches is going to be shown as the border color of the topology node.
@miq-bot
Copy link
Member

miq-bot commented Jul 12, 2018

Checked commit https://github.com/caiocmpaes/manageiq-ui-classic/commit/803be80d6c0446c5e3a514bb078280bb02851af1 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🍪

@caiocmpaes
Copy link
Contributor Author

@miq-bot assign @skateman

@skateman Should I make a refactoring on my code at the physical_infra_topology_service.rb in order to avoid the issues of Codeclimate? Or I can let the code as It is?

@skateman
Copy link
Member

@caiocmpaes I assume you will add other stuff to this topology, so let's do it after all the things are in.

@caiocmpaes
Copy link
Contributor Author

Ok. This PR is only intended to add this border to the topology nodes. This is my only goal. When #4065 get merged, I will add PhysicalChassis nodes and relationships to the topology. Probably, this should be in a new PR, right?

@skateman
Copy link
Member

Yup, so after all is in, we can refactor 😉

@mzazrivec
Copy link
Contributor

@skateman Are the changes here OK with you?

Copy link
Member

@skateman skateman left a comment

Choose a reason for hiding this comment

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

The Seal of Approval

@mzazrivec mzazrivec added this to the Sprint 90 Ending Jul 16, 2018 milestone Jul 13, 2018
@mzazrivec mzazrivec merged commit fdb5c1e into ManageIQ:master Jul 13, 2018
@caiocmpaes caiocmpaes deleted the set_status_of_physical_topology_nodes branch July 13, 2018 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants