Merge pull request #63 from katmastt/main

Sort vm selections
athenarc · Oct 19, 2023 · a3a4a12 · a3a4a12
2 parents 3911bea + 79c3581
commit a3a4a12
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 12 deletions.
diff --git a/models/MachineComputeRequest.php b/models/MachineComputeRequest.php
@@ -89,7 +89,16 @@ public function init()
         $flavors=$response->data['flavors'];
         // print_r($flavors);
         // exit(0);
-        usort($flavors, "self::cmp");
+
+        //sort by cpu. If cpu is the same, sort by ram. If ram is the same sort by disk
+        usort($flavors, function ($a, $b) {
+            if ($a['vcpus'] !== $b['vcpus']) {
+                return $a['vcpus'] - $b['vcpus'];
+            } elseif($a['ram'] !== $b['ram']) {
+                return $a['ram'] - $b['ram'];
+            }
+            return $a['disk'] - $b['disk'];
+        });
 
         foreach ($flavors as $flavor)
         {
@@ -120,9 +129,6 @@ public function init()
             $this->flavourIdName[$id]=$name;
 
         }
-
-        asort($this->flavours);
-
     }
     /**
      * {@inheritdoc}
@@ -391,8 +397,4 @@ public function getDiff($other) {
 
         return $diff;
     }
-
-    public function cmp($a, $b){
-        return strcmp($a['vcpus'], $b['vcpus']);
-    }
 }
diff --git a/models/ServiceRequest.php b/models/ServiceRequest.php
@@ -162,6 +162,14 @@ public function init()
         if($flag)
         {
             $flavors=$response->data['flavors'];
+
+            //sort by cpu. If cpu is the same, sort by ram
+            usort($flavors, function ($a, $b) {
+                if ($a['vcpus'] == $b['vcpus']) {
+                    return $a['ram'] - $b['ram'];
+                }
+                return $a['vcpus'] - $b['vcpus'];
+            });
 
             foreach ($flavors as $flavor)
             {
@@ -204,7 +212,6 @@ public function init()
             }
             // $this->flavour=(!empty($this->vm_flavour)) ? $this->flavourIdName[$this->vm_flavour] : '';
 
-            asort($this->flavours);
         }
         // print_r($this->vm_flavour);
         // print_r($this->flavour);
@@ -575,5 +582,4 @@ public function getDiff($other) {
 
         return $diff;
     }
-
 }
diff --git a/views/project/new_machine_compute_request.php b/views/project/new_machine_compute_request.php
@@ -99,7 +99,7 @@
         </div>
         <div class="row">&nbsp;</div>
 
-        <?= $form->field($service,'flavour')->dropDownList(array_reverse($service->flavours))->label($flavour_label)?>
+        <?= $form->field($service,'flavour')->dropDownList($service->flavours)->label($flavour_label)?>
         <?= $form->field($service,'num_of_vms')->dropDownList($num_vms_dropdown)?>
 
         <div class="row">

diff --git a/views/project/new_service_request.php b/views/project/new_service_request.php
@@ -126,7 +126,7 @@
         </div>
         <div class="row">&nbsp;</div>
 
-        <?= $form->field($service,'flavour')->dropDownList(array_reverse($service->flavours))->label($flavour_label)?>
+        <?= $form->field($service,'flavour')->dropDownList($service->flavours)->label($flavour_label)?>
 
 
         <div class="row">