Skip to content

Commit

Permalink
Fixes #9122: Use hostname for Nodes instead of name in compliance api…
Browse files Browse the repository at this point in the history
… names
  • Loading branch information
VinceMacBuche committed Sep 26, 2016
1 parent a3457a4 commit 5992348
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ class ComplianceAPIService(
byNode.map { case (nodeId, components) =>
ByRuleNodeCompliance(
nodeId
, nodeInfos.get(nodeId).map(_.name).getOrElse("Unknown node")
, nodeInfos.get(nodeId).map(_.hostname).getOrElse("Unknown node")
, components.toSeq.sortBy(_._2.componentName).flatMap(_._2.componentValues.values)
)
}.toSeq
Expand Down Expand Up @@ -198,7 +198,7 @@ class ComplianceAPIService(

(nodeId, ByNodeNodeCompliance(
nodeId
, nodeInfos.get(nodeId).map(_.name).getOrElse("Unknown node")
, nodeInfos.get(nodeId).map(_.hostname).getOrElse("Unknown node")
, ComplianceLevel(noAnswer = rulesForNode.size)
, (rulesForNode.map { rule =>
ByNodeRuleCompliance(
Expand All @@ -219,7 +219,7 @@ class ComplianceAPIService(
nodeId,
ByNodeNodeCompliance(
nodeId
, nodeInfos.get(nodeId).map(_.name).getOrElse("Unknown node")
, nodeInfos.get(nodeId).map(_.hostname).getOrElse("Unknown node")
, ComplianceLevel.sum(ruleNodeStatusReports.map(_.compliance))
, ruleNodeStatusReports.toSeq.map(r =>
ByNodeRuleCompliance(
Expand Down

0 comments on commit 5992348

Please sign in to comment.