From 21330a54cbe1b7014fcfe69105aca2d62954fb06 Mon Sep 17 00:00:00 2001
From: nixocio <nixocio@gmail.com>
Date: Thu, 27 Aug 2020 16:40:09 -0400
Subject: [PATCH] Update instance groups

* Simplify criteria to instance group to be considered unavailable
* Round values for used capacity

See: https://github.com/ansible/awx/issues/7467
---
 .../InstanceGroupDetails/InstanceGroupDetails.jsx   | 13 ++++---------
 .../InstanceGroupList/InstanceGroupListItem.jsx     | 11 ++---------
 .../InstanceGroup/Instances/InstanceListItem.jsx    |  2 +-
 3 files changed, 7 insertions(+), 19 deletions(-)

diff --git a/awx/ui_next/src/screens/InstanceGroup/InstanceGroupDetails/InstanceGroupDetails.jsx b/awx/ui_next/src/screens/InstanceGroup/InstanceGroupDetails/InstanceGroupDetails.jsx
index 0d6964559a..b7c10da67d 100644
--- a/awx/ui_next/src/screens/InstanceGroup/InstanceGroupDetails/InstanceGroupDetails.jsx
+++ b/awx/ui_next/src/screens/InstanceGroup/InstanceGroupDetails/InstanceGroupDetails.jsx
@@ -39,13 +39,6 @@ function InstanceGroupDetails({ instanceGroup, i18n }) {
 
   const { error, dismissError } = useDismissableError(deleteError);
 
-  const isAvailable = item => {
-    return (
-      (item.policy_instance_minimum || item.policy_instance_percentage) &&
-      item.capacity
-    );
-  };
-
   const verifyIsIsolated = item => {
     if (item.is_isolated) {
       return (
@@ -89,10 +82,12 @@ function InstanceGroupDetails({ instanceGroup, i18n }) {
           dataCy="instance-group-policy-instance-percentage"
           content={`${instanceGroup.policy_instance_percentage} %`}
         />
-        {isAvailable(instanceGroup) ? (
+        {instanceGroup.capacity ? (
           <DetailBadge
             label={i18n._(t`Used capacity`)}
-            content={`${100 - instanceGroup.percent_capacity_remaining} %`}
+            content={`${Math.round(
+              100 - instanceGroup.percent_capacity_remaining
+            )} %`}
             dataCy="instance-group-used-capacity"
           />
         ) : (
diff --git a/awx/ui_next/src/screens/InstanceGroup/InstanceGroupList/InstanceGroupListItem.jsx b/awx/ui_next/src/screens/InstanceGroup/InstanceGroupList/InstanceGroupListItem.jsx
index f531f8b182..9ea19a5dd2 100644
--- a/awx/ui_next/src/screens/InstanceGroup/InstanceGroupList/InstanceGroupListItem.jsx
+++ b/awx/ui_next/src/screens/InstanceGroup/InstanceGroupList/InstanceGroupListItem.jsx
@@ -60,23 +60,16 @@ function InstanceGroupListItem({
 }) {
   const labelId = `check-action-${instanceGroup.id}`;
 
-  const isAvailable = item => {
-    return (
-      (item.policy_instance_minimum || item.policy_instance_percentage) &&
-      item.capacity
-    );
-  };
-
   const isContainerGroup = item => {
     return item.is_containerized;
   };
 
   function usedCapacity(item) {
     if (!isContainerGroup(item)) {
-      if (isAvailable(item)) {
+      if (item.capacity) {
         return (
           <Progress
-            value={100 - item.percent_capacity_remaining}
+            value={Math.round(100 - item.percent_capacity_remaining)}
             measureLocation={ProgressMeasureLocation.top}
             size={ProgressSize.sm}
             title={i18n._(t`Used capacity`)}
diff --git a/awx/ui_next/src/screens/InstanceGroup/Instances/InstanceListItem.jsx b/awx/ui_next/src/screens/InstanceGroup/Instances/InstanceListItem.jsx
index 83f2fef753..a26012a738 100644
--- a/awx/ui_next/src/screens/InstanceGroup/Instances/InstanceListItem.jsx
+++ b/awx/ui_next/src/screens/InstanceGroup/Instances/InstanceListItem.jsx
@@ -53,7 +53,7 @@ function InstanceListItem({
     if (item.enabled) {
       return (
         <Progress
-          value={100 - item.percent_capacity_remaining}
+          value={Math.round(100 - item.percent_capacity_remaining)}
           measureLocation={ProgressMeasureLocation.top}
           size={ProgressSize.sm}
           title={i18n._(t`Used capacity`)}