hybrid_platforms_conductor 32.17.0 → 33.0.2

data/spec/hybrid_platforms_conductor_test/api/deployer/provisioners/proxmox/reserve_proxmox_container/expired_containers_spec.rb

@@ -2,21 +2,23 @@ require 'hybrid_platforms_conductor/hpc_plugins/provisioner/proxmox'
 
 describe HybridPlatformsConductor::HpcPlugins::Provisioner::Proxmox do
 
-  context 'checking the reserve_proxmox_container sync tool' do
+  context 'when checking the reserve_proxmox_container sync tool' do
 
-    context 'checking expiration strategy for containers' do
+    context 'when checking expiration strategy for containers' do
 
       it 'does not expire a VM when there are enough free resources on a PVE node' do
         with_sync_node do
-          mock_proxmox(mocked_pve_nodes: {
-            'pve_node_name' => {
-              memory_total: 4 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                # Make sure it is expired
-                1000 => { ip: '192.168.0.100', maxmem: 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+          mock_proxmox(
+            mocked_pve_nodes: {
+              'pve_node_name' => {
+                memory_total: 4 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  # Make sure it is expired
+                  1000 => { ip: '192.168.0.100', maxmem: 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+                }
               }
             }
-          })
+          )
           expect(call_reserve_proxmox_container(2, 1024, 1)).to eq(
             pve_node: 'pve_node_name',
             vm_id: 1001,
@@ -28,15 +30,17 @@ describe HybridPlatformsConductor::HpcPlugins::Provisioner::Proxmox do
       it 'expires a VM when there are not enough free resources on a PVE node' do
         with_sync_node do
           creation_date = (Time.now - 31 * 24 * 60 * 60).utc
-          mock_proxmox(mocked_pve_nodes: {
-            'pve_node_name' => {
-              memory_total: 4 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                # Make sure it is expired
-                1000 => { ip: '192.168.0.100', maxmem: 4 * 1024 * 1024 * 1024, creation_date: creation_date }
+          mock_proxmox(
+            mocked_pve_nodes: {
+              'pve_node_name' => {
+                memory_total: 4 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  # Make sure it is expired
+                  1000 => { ip: '192.168.0.100', maxmem: 4 * 1024 * 1024 * 1024, creation_date: creation_date }
+                }
               }
             }
-          })
+          )
           expect(call_reserve_proxmox_container(2, 1024, 1)).to eq(
             pve_node: 'pve_node_name',
             vm_id: 1000,
@@ -61,26 +65,28 @@ describe HybridPlatformsConductor::HpcPlugins::Provisioner::Proxmox do
               }
             ]
           ]
-          expect(Time.parse(@proxmox_actions[2][2]['description'].match(/^===== HPC info =====\nnode: test_node\nenvironment: test_env\ncreation_date: (.+)\n/)[1])).to be > creation_date
+          expect(Time.parse(proxmox_actions[2][2]['description'].match(/^===== HPC info =====\nnode: test_node\nenvironment: test_env\ncreation_date: (.+)\n/)[1])).to be > creation_date
         end
       end
 
       it 'expires a VM without stopping it when there are not enough free resources on a PVE node and the VM is not running' do
         with_sync_node do
-          mock_proxmox(mocked_pve_nodes: {
-            'pve_node_name' => {
-              memory_total: 4 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1000 => {
-                  ip: '192.168.0.100',
-                  maxmem: 4 * 1024 * 1024 * 1024,
-                  status: 'stopped',
-                  # Make sure it is expired
-                  creation_date: (Time.now - 31 * 24 * 60 * 60).utc
+          mock_proxmox(
+            mocked_pve_nodes: {
+              'pve_node_name' => {
+                memory_total: 4 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1000 => {
+                    ip: '192.168.0.100',
+                    maxmem: 4 * 1024 * 1024 * 1024,
+                    status: 'stopped',
+                    # Make sure it is expired
+                    creation_date: (Time.now - 31 * 24 * 60 * 60).utc
+                  }
                 }
               }
             }
-          })
+          )
           expect(call_reserve_proxmox_container(2, 1024, 1)).to eq(
             pve_node: 'pve_node_name',
             vm_id: 1000,
@@ -105,16 +111,18 @@ describe HybridPlatformsConductor::HpcPlugins::Provisioner::Proxmox do
 
       it 'reuses the IP address and VM IDs of an expired VM when there are not enough free resources on a PVE node' do
         with_sync_node do
-          mock_proxmox(mocked_pve_nodes: {
-            'pve_node_name' => {
-              memory_total: 8 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1000 => { ip: '192.168.0.100', maxmem: 2 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
-                1001 => { ip: '192.168.0.101', maxmem: 4 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc },
-                1002 => { ip: '192.168.0.102', maxmem: 2 * 1024 * 1024 * 1024, creation_date: Time.now.utc }
+          mock_proxmox(
+            mocked_pve_nodes: {
+              'pve_node_name' => {
+                memory_total: 8 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1000 => { ip: '192.168.0.100', maxmem: 2 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
+                  1001 => { ip: '192.168.0.101', maxmem: 4 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc },
+                  1002 => { ip: '192.168.0.102', maxmem: 2 * 1024 * 1024 * 1024, creation_date: Time.now.utc }
+                }
               }
             }
-          })
+          )
           expect(call_reserve_proxmox_container(2, 1024, 1)).to eq(
             pve_node: 'pve_node_name',
             vm_id: 1001,
@@ -140,42 +148,49 @@ describe HybridPlatformsConductor::HpcPlugins::Provisioner::Proxmox do
 
       it 'does not try to expire expired VMs when the freed resources would not be enough anyway for the new container' do
         with_sync_node do
-          mock_proxmox(mocked_pve_nodes: {
-            'pve_node_name' => {
-              memory_total: 6 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1000 => { ip: '192.168.0.100', maxmem: 2 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
-                1001 => { ip: '192.168.0.101', maxmem: 2 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc },
-                1002 => { ip: '192.168.0.102', maxmem: 2 * 1024 * 1024 * 1024, creation_date: Time.now.utc }
+          mock_proxmox(
+            mocked_pve_nodes: {
+              'pve_node_name' => {
+                memory_total: 6 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1000 => { ip: '192.168.0.100', maxmem: 2 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
+                  1001 => { ip: '192.168.0.101', maxmem: 2 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc },
+                  1002 => { ip: '192.168.0.102', maxmem: 2 * 1024 * 1024 * 1024, creation_date: Time.now.utc }
+                }
               }
             }
-          })
-          expect(call_reserve_proxmox_container(2, 1024, 1,
-            config: {
-              vm_ips_list: %w[
-                192.168.0.100
-                192.168.0.101
-                192.168.0.102
-                192.168.0.103
-              ]
-            }
-          )).to eq(error: 'not_enough_resources')
+          )
+          expect(
+            call_reserve_proxmox_container(
+              2, 1024, 1,
+              config: {
+                vm_ips_list: %w[
+                  192.168.0.100
+                  192.168.0.101
+                  192.168.0.102
+                  192.168.0.103
+                ]
+              }
+            )
+          ).to eq(error: 'not_enough_resources')
           expect_proxmox_actions_to_be []
         end
       end
 
       it 'expires all expired containers of a PVE node if needed' do
         with_sync_node do
-          mock_proxmox(mocked_pve_nodes: {
-            'pve_node_name' => {
-              memory_total: 8 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1000 => { ip: '192.168.0.100', maxmem: 2 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
-                1001 => { ip: '192.168.0.101', maxmem: 4 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc },
-                1002 => { ip: '192.168.0.102', maxmem: 2 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+          mock_proxmox(
+            mocked_pve_nodes: {
+              'pve_node_name' => {
+                memory_total: 8 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1000 => { ip: '192.168.0.100', maxmem: 2 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
+                  1001 => { ip: '192.168.0.101', maxmem: 4 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc },
+                  1002 => { ip: '192.168.0.102', maxmem: 2 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+                }
               }
             }
-          })
+          )
           expect(call_reserve_proxmox_container(2, 1024, 1)).to eq(
             pve_node: 'pve_node_name',
             vm_id: 1001,
@@ -203,16 +218,18 @@ describe HybridPlatformsConductor::HpcPlugins::Provisioner::Proxmox do
 
       it 'does not expire containers that don\'t belong to the VM IDs range' do
         with_sync_node do
-          mock_proxmox(mocked_pve_nodes: {
-            'pve_node_name' => {
-              memory_total: 8 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1000 => { ip: '192.168.0.100', maxmem: 2 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
-                1001 => { ip: '192.168.0.101', maxmem: 4 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc },
-                2002 => { ip: '192.168.0.102', maxmem: 2 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+          mock_proxmox(
+            mocked_pve_nodes: {
+              'pve_node_name' => {
+                memory_total: 8 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1000 => { ip: '192.168.0.100', maxmem: 2 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
+                  1001 => { ip: '192.168.0.101', maxmem: 4 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc },
+                  2002 => { ip: '192.168.0.102', maxmem: 2 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+                }
               }
             }
-          })
+          )
           expect(call_reserve_proxmox_container(2, 1024, 1)).to eq(
             pve_node: 'pve_node_name',
             vm_id: 1001,
@@ -238,22 +255,24 @@ describe HybridPlatformsConductor::HpcPlugins::Provisioner::Proxmox do
 
       it 'selects a PVE node that still has free resources without expiring VMs even if it would have freed more resources' do
         with_sync_node do
-          mock_proxmox(mocked_pve_nodes: {
-            # Expiring VMs from this node would free a lot of resources
-            'pve_node_1' => {
-              memory_total: 16 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1000 => { ip: '192.168.0.100', maxmem: 14 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
-              }
-            },
-            # But this node has still a bit of resources left without expiring VMs
-            'pve_node_2' => {
-              memory_total: 16 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1001 => { ip: '192.168.0.101', maxmem: 10 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+          mock_proxmox(
+            mocked_pve_nodes: {
+              # Expiring VMs from this node would free a lot of resources
+              'pve_node_1' => {
+                memory_total: 16 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1000 => { ip: '192.168.0.100', maxmem: 14 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+                }
+              },
+              # But this node has still a bit of resources left without expiring VMs
+              'pve_node_2' => {
+                memory_total: 16 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1001 => { ip: '192.168.0.101', maxmem: 10 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+                }
               }
             }
-          })
+          )
           expect(call_reserve_proxmox_container(2, 1024, 1, config: { pve_nodes: nil })).to eq(
             pve_node: 'pve_node_2',
             vm_id: 1002,
@@ -277,34 +296,39 @@ describe HybridPlatformsConductor::HpcPlugins::Provisioner::Proxmox do
 
       it 'selects the PVE node that would have the more resources free after expiration when no other PVE node has free resources' do
         with_sync_node do
-          mock_proxmox(mocked_pve_nodes: {
-            'pve_node_1' => {
-              memory_total: 16 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1000 => { ip: '192.168.0.100', maxmem: 8 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
-                1001 => { ip: '192.168.0.101', maxmem: 6 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
-              }
-            },
-            'pve_node_2' => {
-              memory_total: 16 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1002 => { ip: '192.168.0.102', maxmem: 10 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
-                1003 => { ip: '192.168.0.103', maxmem: 4 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+          mock_proxmox(
+            mocked_pve_nodes: {
+              'pve_node_1' => {
+                memory_total: 16 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1000 => { ip: '192.168.0.100', maxmem: 8 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
+                  1001 => { ip: '192.168.0.101', maxmem: 6 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+                }
+              },
+              'pve_node_2' => {
+                memory_total: 16 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1002 => { ip: '192.168.0.102', maxmem: 10 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
+                  1003 => { ip: '192.168.0.103', maxmem: 4 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+                }
               }
             }
-          })
-          expect(call_reserve_proxmox_container(2, 1024, 1,
-            config: {
-              pve_nodes: nil,
-              vm_ips_list: %w[
-                192.168.0.100
-                192.168.0.101
-                192.168.0.102
-                192.168.0.103
-                192.168.0.104
-              ]
-            }
-          )).to eq(
+          )
+          expect(
+            call_reserve_proxmox_container(
+              2, 1024, 1,
+              config: {
+                pve_nodes: nil,
+                vm_ips_list: %w[
+                  192.168.0.100
+                  192.168.0.101
+                  192.168.0.102
+                  192.168.0.103
+                  192.168.0.104
+                ]
+              }
+            )
+          ).to eq(
             pve_node: 'pve_node_1',
             vm_id: 1001,
             vm_ip: '192.168.0.101'
@@ -329,25 +353,30 @@ describe HybridPlatformsConductor::HpcPlugins::Provisioner::Proxmox do
 
       it 'expires VMs even if free resources are available when IPs are all used' do
         with_sync_node do
-          mock_proxmox(mocked_pve_nodes: {
-            'pve_node_name' => {
-              memory_total: 16 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1000 => { ip: '192.168.0.100', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
-                1001 => { ip: '192.168.0.101', maxmem: 1 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc },
-                1002 => { ip: '192.168.0.102', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc }
+          mock_proxmox(
+            mocked_pve_nodes: {
+              'pve_node_name' => {
+                memory_total: 16 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1000 => { ip: '192.168.0.100', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
+                  1001 => { ip: '192.168.0.101', maxmem: 1 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc },
+                  1002 => { ip: '192.168.0.102', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc }
+                }
               }
             }
-          })
-          expect(call_reserve_proxmox_container(2, 1024, 1,
-            config: {
-              vm_ips_list: %w[
-                192.168.0.100
-                192.168.0.101
-                192.168.0.102
-              ]
-            }
-          )).to eq(
+          )
+          expect(
+            call_reserve_proxmox_container(
+              2, 1024, 1,
+              config: {
+                vm_ips_list: %w[
+                  192.168.0.100
+                  192.168.0.101
+                  192.168.0.102
+                ]
+              }
+            )
+          ).to eq(
             pve_node: 'pve_node_name',
             vm_id: 1001,
             vm_ip: '192.168.0.101'
@@ -372,33 +401,38 @@ describe HybridPlatformsConductor::HpcPlugins::Provisioner::Proxmox do
 
       it 'expires VMs from non-selected PVE nodes even if free resources are available when IPs are all used' do
         with_sync_node do
-          mock_proxmox(mocked_pve_nodes: {
-            # Make sure this node should be selected
-            'pve_node_1' => {
-              memory_total: 16 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1000 => { ip: '192.168.0.100', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
-                1002 => { ip: '192.168.0.102', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc }
-              }
-            },
-            # But this node is the only one having expired VMs
-            'pve_node_2' => {
-              memory_total: 2 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1001 => { ip: '192.168.0.101', maxmem: 1 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+          mock_proxmox(
+            mocked_pve_nodes: {
+              # Make sure this node should be selected
+              'pve_node_1' => {
+                memory_total: 16 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1000 => { ip: '192.168.0.100', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
+                  1002 => { ip: '192.168.0.102', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc }
+                }
+              },
+              # But this node is the only one having expired VMs
+              'pve_node_2' => {
+                memory_total: 2 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1001 => { ip: '192.168.0.101', maxmem: 1 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+                }
               }
             }
-          })
-          expect(call_reserve_proxmox_container(2, 1024, 1,
-            config: {
-              pve_nodes: nil,
-              vm_ips_list: %w[
-                192.168.0.100
-                192.168.0.101
-                192.168.0.102
-              ]
-            }
-          )).to eq(
+          )
+          expect(
+            call_reserve_proxmox_container(
+              2, 1024, 1,
+              config: {
+                pve_nodes: nil,
+                vm_ips_list: %w[
+                  192.168.0.100
+                  192.168.0.101
+                  192.168.0.102
+                ]
+              }
+            )
+          ).to eq(
             pve_node: 'pve_node_1',
             vm_id: 1001,
             vm_ip: '192.168.0.101'
@@ -423,27 +457,32 @@ describe HybridPlatformsConductor::HpcPlugins::Provisioner::Proxmox do
 
       it 'expires VMs even if free resources are available when VM IDs are all used' do
         with_sync_node do
-          mock_proxmox(mocked_pve_nodes: {
-            'pve_node_name' => {
-              memory_total: 16 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1000 => { ip: '192.168.0.100', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
-                1001 => { ip: '192.168.0.101', maxmem: 1 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc },
-                1002 => { ip: '192.168.0.102', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc }
+          mock_proxmox(
+            mocked_pve_nodes: {
+              'pve_node_name' => {
+                memory_total: 16 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1000 => { ip: '192.168.0.100', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
+                  1001 => { ip: '192.168.0.101', maxmem: 1 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc },
+                  1002 => { ip: '192.168.0.102', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc }
+                }
               }
             }
-          })
-          expect(call_reserve_proxmox_container(2, 1024, 1,
-            config: {
-              vm_ips_list: %w[
-                192.168.0.100
-                192.168.0.101
-                192.168.0.102
-                192.168.0.103
-              ],
-              vm_ids_range: [1000, 1002]
-            }
-          )).to eq(
+          )
+          expect(
+            call_reserve_proxmox_container(
+              2, 1024, 1,
+              config: {
+                vm_ips_list: %w[
+                  192.168.0.100
+                  192.168.0.101
+                  192.168.0.102
+                  192.168.0.103
+                ],
+                vm_ids_range: [1000, 1002]
+              }
+            )
+          ).to eq(
             pve_node: 'pve_node_name',
             vm_id: 1001,
             vm_ip: '192.168.0.101'
@@ -468,35 +507,40 @@ describe HybridPlatformsConductor::HpcPlugins::Provisioner::Proxmox do
 
       it 'expires VMs from non-selected PVE nodes even if free resources are available when VM IDs are all used' do
         with_sync_node do
-          mock_proxmox(mocked_pve_nodes: {
-            # Make sure this node should be selected
-            'pve_node_1' => {
-              memory_total: 16 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1000 => { ip: '192.168.0.100', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
-                1002 => { ip: '192.168.0.102', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc }
-              }
-            },
-            # But this node is the only one having expired VMs
-            'pve_node_2' => {
-              memory_total: 2 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1001 => { ip: '192.168.0.101', maxmem: 1 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+          mock_proxmox(
+            mocked_pve_nodes: {
+              # Make sure this node should be selected
+              'pve_node_1' => {
+                memory_total: 16 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1000 => { ip: '192.168.0.100', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
+                  1002 => { ip: '192.168.0.102', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc }
+                }
+              },
+              # But this node is the only one having expired VMs
+              'pve_node_2' => {
+                memory_total: 2 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1001 => { ip: '192.168.0.101', maxmem: 1 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+                }
               }
             }
-          })
-          expect(call_reserve_proxmox_container(2, 1024, 1,
-            config: {
-              pve_nodes: nil,
-              vm_ips_list: %w[
-                192.168.0.100
-                192.168.0.101
-                192.168.0.102
-                192.168.0.103
-              ],
-              vm_ids_range: [1000, 1002]
-            }
-          )).to eq(
+          )
+          expect(
+            call_reserve_proxmox_container(
+              2, 1024, 1,
+              config: {
+                pve_nodes: nil,
+                vm_ips_list: %w[
+                  192.168.0.100
+                  192.168.0.101
+                  192.168.0.102
+                  192.168.0.103
+                ],
+                vm_ids_range: [1000, 1002]
+              }
+            )
+          ).to eq(
             pve_node: 'pve_node_1',
             vm_id: 1001,
             vm_ip: '192.168.0.101'
@@ -521,32 +565,37 @@ describe HybridPlatformsConductor::HpcPlugins::Provisioner::Proxmox do
 
       it 'expires VMs even if free resources are available when the maximum number of VMs has been reached' do
         with_sync_node do
-          mock_proxmox(mocked_pve_nodes: {
-            'pve_node_name' => {
-              memory_total: 16 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1000 => { ip: '192.168.0.100', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
-                1001 => { ip: '192.168.0.101', maxmem: 1 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc },
-                1002 => { ip: '192.168.0.102', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc }
+          mock_proxmox(
+            mocked_pve_nodes: {
+              'pve_node_name' => {
+                memory_total: 16 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1000 => { ip: '192.168.0.100', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
+                  1001 => { ip: '192.168.0.101', maxmem: 1 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc },
+                  1002 => { ip: '192.168.0.102', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc }
+                }
               }
             }
-          })
-          expect(call_reserve_proxmox_container(2, 1024, 1,
-            config: {
-              vm_ips_list: %w[
-                192.168.0.100
-                192.168.0.101
-                192.168.0.102
-                192.168.0.103
-              ],
-              limits: {
-                nbr_vms_max: 3,
-                cpu_loads_thresholds: [10, 10, 10],
-                ram_percent_used_max: 0.75,
-                disk_percent_used_max: 0.75
+          )
+          expect(
+            call_reserve_proxmox_container(
+              2, 1024, 1,
+              config: {
+                vm_ips_list: %w[
+                  192.168.0.100
+                  192.168.0.101
+                  192.168.0.102
+                  192.168.0.103
+                ],
+                limits: {
+                  nbr_vms_max: 3,
+                  cpu_loads_thresholds: [10, 10, 10],
+                  ram_percent_used_max: 0.75,
+                  disk_percent_used_max: 0.75
+                }
               }
-            }
-          )).to eq(
+            )
+          ).to eq(
             pve_node: 'pve_node_name',
             vm_id: 1001,
             vm_ip: '192.168.0.101'
@@ -571,40 +620,45 @@ describe HybridPlatformsConductor::HpcPlugins::Provisioner::Proxmox do
 
       it 'expires VMs from non-selected PVE nodes even if free resources are available when the maximum number of VMs has been reached' do
         with_sync_node do
-          mock_proxmox(mocked_pve_nodes: {
-            # Make sure this node should be selected
-            'pve_node_1' => {
-              memory_total: 16 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1000 => { ip: '192.168.0.100', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
-                1002 => { ip: '192.168.0.102', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc }
-              }
-            },
-            # But this node is the only one having expired VMs
-            'pve_node_2' => {
-              memory_total: 2 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                1001 => { ip: '192.168.0.101', maxmem: 1 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+          mock_proxmox(
+            mocked_pve_nodes: {
+              # Make sure this node should be selected
+              'pve_node_1' => {
+                memory_total: 16 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1000 => { ip: '192.168.0.100', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc },
+                  1002 => { ip: '192.168.0.102', maxmem: 1 * 1024 * 1024 * 1024, creation_date: Time.now.utc }
+                }
+              },
+              # But this node is the only one having expired VMs
+              'pve_node_2' => {
+                memory_total: 2 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  1001 => { ip: '192.168.0.101', maxmem: 1 * 1024 * 1024 * 1024, creation_date: (Time.now - 31 * 24 * 60 * 60).utc }
+                }
               }
             }
-          })
-          expect(call_reserve_proxmox_container(2, 1024, 1,
-            config: {
-              pve_nodes: nil,
-              vm_ips_list: %w[
-                192.168.0.100
-                192.168.0.101
-                192.168.0.102
-                192.168.0.103
-              ],
-              limits: {
-                nbr_vms_max: 3,
-                cpu_loads_thresholds: [10, 10, 10],
-                ram_percent_used_max: 0.75,
-                disk_percent_used_max: 0.75
+          )
+          expect(
+            call_reserve_proxmox_container(
+              2, 1024, 1,
+              config: {
+                pve_nodes: nil,
+                vm_ips_list: %w[
+                  192.168.0.100
+                  192.168.0.101
+                  192.168.0.102
+                  192.168.0.103
+                ],
+                limits: {
+                  nbr_vms_max: 3,
+                  cpu_loads_thresholds: [10, 10, 10],
+                  ram_percent_used_max: 0.75,
+                  disk_percent_used_max: 0.75
+                }
               }
-            }
-          )).to eq(
+            )
+          ).to eq(
             pve_node: 'pve_node_1',
             vm_id: 1001,
             vm_ip: '192.168.0.101'
@@ -629,21 +683,23 @@ describe HybridPlatformsConductor::HpcPlugins::Provisioner::Proxmox do
 
       it 'does not expire a VM that is stopped for long time but still used for debug purposes' do
         with_sync_node do
-          mock_proxmox(mocked_pve_nodes: {
-            'pve_node_name' => {
-              memory_total: 4 * 1024 * 1024 * 1024,
-              lxc_containers: {
-                # Make sure it is not expired
-                1000 => {
-                  ip: '192.168.0.100',
-                  maxmem: 4 * 1024 * 1024 * 1024,
-                  creation_date: Time.now.utc,
-                  status: 'stopped',
-                  debug: true
+          mock_proxmox(
+            mocked_pve_nodes: {
+              'pve_node_name' => {
+                memory_total: 4 * 1024 * 1024 * 1024,
+                lxc_containers: {
+                  # Make sure it is not expired
+                  1000 => {
+                    ip: '192.168.0.100',
+                    maxmem: 4 * 1024 * 1024 * 1024,
+                    creation_date: Time.now.utc,
+                    status: 'stopped',
+                    debug: true
+                  }
                 }
               }
             }
-          })
+          )
           expect(call_reserve_proxmox_container(2, 1024, 1)).to eq(error: 'not_enough_resources')
           expect_proxmox_actions_to_be []
         end
@@ -691,7 +747,8 @@ describe HybridPlatformsConductor::HpcPlugins::Provisioner::Proxmox do
 
       it 'does not expire a VM that is stopped for some time even when it is not used for debug purposes' do
         with_sync_node do
-          mock_proxmox(mocked_pve_nodes: [{
+          mock_proxmox(
+            mocked_pve_nodes: [{
               # 2 seconds separate each run.
               # Make sure the third and later runs mock the container as running instead of stopped
               'pve_node_name' => {
@@ -738,7 +795,8 @@ describe HybridPlatformsConductor::HpcPlugins::Provisioner::Proxmox do
         with_sync_node do
           creation_date_1 = (Time.now - 20).utc
           creation_date_2 = (Time.now - 10).utc
-          mock_proxmox(mocked_pve_nodes: [{
+          mock_proxmox(
+            mocked_pve_nodes: [{
               # 2 seconds separate each run.
               # Make sure the third and later runs mock the container as another one, still stopped
               'pve_node_name' => {