knife-ionoscloud 6.0.0.alpha.1 → 6.1.0

data/spec/chef/knife/ionoscloud_volume_update_spec.rb

@@ -0,0 +1,116 @@
+require 'spec_helper'
+require 'ionoscloud_volume_update'
+
+Chef::Knife::IonoscloudVolumeUpdate.load_deps
+
+describe Chef::Knife::IonoscloudVolumeUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudVolumeUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call VolumesApi.datacenters_volumes_patch' do
+      volume = volume_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        datacenter_id: 'datacenter_id',
+        volume_id: volume.id,
+        name: volume.properties.name + '_edited',
+        size: volume.properties.size + 10,
+        bus: 'IDE',
+        cpu_hot_plug: (!volume.properties.cpu_hot_plug).to_s,
+        ram_hot_plug: (!volume.properties.ram_hot_plug).to_s,
+        nic_hot_plug: (!volume.properties.nic_hot_plug).to_s,
+        disc_virtio_hot_plug: (!volume.properties.disc_virtio_hot_plug).to_s,
+        disc_virtio_hot_unplug: (!volume.properties.disc_virtio_hot_unplug).to_s,
+
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject).to receive(:puts).with("ID: #{volume.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("Size: #{subject_config[:size]}")
+      expect(subject).to receive(:puts).with("Bus: #{subject_config[:bus]}")
+      expect(subject).to receive(:puts).with("Image: #{volume.properties.image}")
+      expect(subject).to receive(:puts).with("Type: #{volume.properties.type}")
+      expect(subject).to receive(:puts).with("Licence Type: #{volume.properties.licence_type}")
+      expect(subject).to receive(:puts).with("Backupunit ID: #{volume.properties.backupunit_id}")
+      expect(subject).to receive(:puts).with("User Data: #{volume.properties.user_data}")
+      expect(subject).to receive(:puts).with("Zone: #{volume.properties.availability_zone}")
+      expect(subject).to receive(:puts).with("CPU Hot Plug: #{subject_config[:cpu_hot_plug]}")
+      expect(subject).to receive(:puts).with("RAM Hot Plug: #{subject_config[:ram_hot_plug]}")
+      expect(subject).to receive(:puts).with("NIC Hot Plug: #{subject_config[:nic_hot_plug]}")
+      expect(subject).to receive(:puts).with("NIC Hot Unplug: #{volume.properties.nic_hot_unplug}")
+      expect(subject).to receive(:puts).with("Disc Virtio Hot Plug: #{subject_config[:disc_virtio_hot_plug]}")
+      expect(subject).to receive(:puts).with("Disc Virtio Hot Unplug: #{subject_config[:disc_virtio_hot_unplug]}")
+      expect(subject).to receive(:puts).with("Device number: #{volume.properties.device_number}")
+      expect(subject).to receive(:puts).with("PCI Slot: #{volume.properties.pci_slot}")
+
+      volume.properties.name = subject_config[:name]
+      volume.properties.size = subject_config[:size]
+      volume.properties.bus = subject_config[:bus]
+      volume.properties.cpu_hot_plug = subject_config[:cpu_hot_plug].to_s.downcase == 'true'
+      volume.properties.ram_hot_plug = subject_config[:ram_hot_plug].to_s.downcase == 'true'
+      volume.properties.nic_hot_plug = subject_config[:nic_hot_plug].to_s.downcase == 'true'
+      volume.properties.disc_virtio_hot_plug = subject_config[:disc_virtio_hot_plug].to_s.downcase == 'true'
+      volume.properties.disc_virtio_hot_unplug = subject_config[:disc_virtio_hot_unplug].to_s.downcase == 'true'
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'PATCH',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/volumes/#{subject_config[:volume_id]}",
+            operation: :'VolumesApi.datacenters_volumes_patch',
+            return_type: 'Volume',
+            body: {
+              name: subject_config[:name],
+              size: subject_config[:size],
+              bus: subject_config[:bus],
+              cpuHotPlug: subject_config[:cpu_hot_plug].to_s.downcase == 'true',
+              ramHotPlug: subject_config[:ram_hot_plug].to_s.downcase == 'true',
+              nicHotPlug: subject_config[:nic_hot_plug].to_s.downcase == 'true',
+              discVirtioHotPlug: subject_config[:disc_virtio_hot_plug].to_s.downcase == 'true',
+              discVirtioHotUnplug: subject_config[:disc_virtio_hot_unplug].to_s.downcase == 'true',
+            },
+            result: volume,
+          },
+          {
+            method: 'GET',
+            path: "/datacenters/#{subject_config[:datacenter_id]}/volumes/#{subject_config[:volume_id]}",
+            operation: :'VolumesApi.datacenters_volumes_find_by_id',
+            return_type: 'Volume',
+            result: volume,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should not make any call if any required option is missing' do
+      required_options = subject.instance_variable_get(:@required_options)
+
+      arrays_without_one_element(required_options).each do |test_case|
+
+        test_case[:array].each { |value| subject.config[value] = 'test' }
+
+        expect(subject).to receive(:puts).with("Missing required parameters #{test_case[:removed]}")
+        expect(subject.api_client).not_to receive(:call_api)
+
+        expect { subject.run }.to raise_error(SystemExit) do |error|
+          expect(error.status).to eq(1)
+        end
+
+        required_options.each { |value| subject.config[value] = nil }
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -7,7 +7,6 @@ require 'simplecov'
 RSpec.configure do |config|
   config.pattern = 'spec/chef/knife/*_spec.rb'
 end
-
 SimpleCov.start do
   add_group 'Commands', 'lib/chef/knife/'
   add_group 'Spec files', 'spec/chef/knife/'
@@ -60,7 +59,7 @@ def ipblock_mock(opts = {})
       name: opts[:name] || 'Test IpBlock',
       size: opts[:size] || 4,
       location: opts[:location] || 'de/fra',
-      ips: opts[:ips] || ['87.106.113.181', '87.106.113.176', '87.106.113.177', '87.106.113.178'],
+      ips: opts[:ips] || ['127.106.113.181', '127.106.113.176', '127.106.113.177', '127.106.113.178'],
     ),
   )
 end
@@ -236,7 +235,7 @@ def nic_mock(opts = {})
     id: opts[:id] || SecureRandom.uuid,
     properties: Ionoscloud::NicProperties.new(
       name: opts[:name] || 'nic_name',
-      ips: opts[:ips] || ['1.1.1.1', '1.1.1.2'],
+      ips: opts[:ips] || ['127.1.1.1', '127.1.1.2'],
       dhcp: opts[:dhcp] || true,
       firewall_active: opts[:firewall_active] || true,
       mac: opts[:mac] || '00:0a:95:9d:68:16',
@@ -266,8 +265,8 @@ def firewall_mock(opts = {})
       name: opts[:name] || 'firewall_name',
       protocol: opts[:protocol] || 'UDP',
       source_mac: opts[:source_mac] || '01:23:45:67:89:00',
-      source_ip: opts[:source_ip] || '10.9.20.11',
-      target_ip: opts[:target_ip] || '10.9.20.11',
+      source_ip: opts[:source_ip] || '127.9.20.11',
+      target_ip: opts[:target_ip] || '127.9.20.11',
       port_range_start: opts[:port_range_start] || 22,
       port_range_end: opts[:port_range_end] || 22,
       icmp_type: opts[:icmp_type] || 4,
@@ -290,7 +289,7 @@ def load_balancer_mock(opts = {})
     id: opts[:id] || SecureRandom.uuid,
     properties: Ionoscloud::LoadbalancerProperties.new(
       name: opts[:name] || 'load_balancer_name',
-      ip: opts[:ip] || '1.1.1.1',
+      ip: opts[:ip] || '127.1.1.1',
       dhcp: opts[:dhcp] || true,
     ),
     entities: Ionoscloud::LoadbalancerEntities.new(
@@ -371,6 +370,16 @@ def k8s_cluster_mock(opts = {})
       name: opts[:name] || 'k8s_cluster_name',
       k8s_version: opts[:k8s_version] || '1.15.4,',
       maintenance_window: opts[:maintenance_window] || maintenance_window_mock,
+      api_subnet_allow_list: opts[:api_subnet_allow_list] || [
+        "1.2.3.4/32",
+        "2002::1234:abcd:ffff:c0a8:101/64",
+        "1.2.3.4",
+        "2002::1234:abcd:ffff:c0a8:101"
+      ],
+      s3_buckets: opts[:s3_buckets] || [
+        Ionoscloud::S3Bucket.new(name: 'test_name1'),
+        Ionoscloud::S3Bucket.new(name: 'test_name2'),
+      ],
       available_upgrade_versions: opts[:available_upgrade_versions] || ['1.16.4', '1.17.7'],
       viable_node_pool_versions: opts[:viable_node_pool_versions] || ['1.17.7', '1.18.2']
     ),
@@ -398,8 +407,8 @@ def nodepool_lan_mock(opts = {})
     dhcp: opts[:dhcp] || false,
     routes: opts[:routes] || [
       Ionoscloud::KubernetesNodePoolLanRoutes.new(
-        network: opts[:network] || '1.2.3.4/24',
-        gateway_ip: opts[:gateway_ip] || '10.1.5.16',
+        network: opts[:network] || '127.2.3.4/24',
+        gateway_ip: opts[:gateway_ip] || '127.1.5.16',
       ),
     ],
   )
@@ -422,10 +431,10 @@ def k8s_nodepool_mock(opts = {})
       maintenance_window: opts[:maintenance_window] || maintenance_window_mock,
       auto_scaling: opts[:auto_scaling] || auto_scaling_mock,
       lans: opts[:lans] || [nodepool_lan_mock(id: 12), nodepool_lan_mock(id: 15)],
-      labels: opts[:labels] || nil,
-      annotations: opts[:annotations] || nil,
-      public_ips: opts[:public_ips] || ['81.173.1.2', '82.231.2.5', '92.221.2.4'],
+      public_ips: opts[:public_ips] || ['127.173.1.2', '127.231.2.5', '127.221.2.4'],
       available_upgrade_versions: opts[:available_upgrade_versions] || ['1.16.4', '1.17.7'],
+      labels: opts[:labels] || { "test_labels": "test_labels" },
+      annotations: opts[:annotations] || { "test_annotations": "test_annotations" },
     ),
     metadata: Ionoscloud::KubernetesNodeMetadata.new(
       state: 'READY',
@@ -446,7 +455,7 @@ def k8s_node_mock(opts = {})
     id: opts[:id] || SecureRandom.uuid,
     properties: Ionoscloud::KubernetesNodeProperties.new(
       name: opts[:name] || 'k8s_node_name',
-      public_ip: opts[:public_ip] || '1.1.1.1',
+      public_ip: opts[:public_ip] || '127.1.1.1',
       k8s_version: opts[:k8s_version] || '1.17.7',
     ),
     metadata: Ionoscloud::KubernetesNodeMetadata.new(
@@ -492,6 +501,43 @@ def datacenters_mock(opts = {})
   )
 end
 
+def cluster_mock(opts = {})
+  IonoscloudDbaasPostgres::ClusterResponse.new(
+    id: opts[:id] || SecureRandom.uuid,
+    properties: IonoscloudDbaasPostgres::ClusterProperties.new(
+      display_name: opts[:display_name] || 'dbaas_Cluster',
+      postgres_version: opts[:postgres_version] || '10',
+      location: opts[:location] || 'us/las',
+      instances: opts[:instances] || 2,
+      ram: opts[:ram] || 8192,
+      cores: opts[:cores] || 4,
+      storage_size: opts[:storage_size] || 4096,
+      storage_type: opts[:storage_type] || 'HDD',
+      connections: opts[:connections] || [IonoscloudDbaasPostgres::Connection.new(
+        datacenter_id: opts[:datacenter_id] || SecureRandom.uuid,
+        lan_id: opts[:lan_id] || '1',
+        cidr: opts[:cidr] || '192.168.1.100/24',
+      ),],
+      maintenance_window: opts[:maintenance_window] || IonoscloudDbaasPostgres::MaintenanceWindow.new(
+        time: opts[:time] || "00:29:15", 
+        day_of_the_week: opts[:day_of_the_week] || "Monday", 
+      ),
+      synchronization_mode: opts[:synchronization_mode] || 'ASYNCHRONOUS',
+    ),
+    metadata: IonoscloudDbaasPostgres::Metadata.new(
+      state: 'Busy',
+    ),
+  )
+end
+
+def clusters_mock(opts = {})
+  IonoscloudDbaasPostgres::ClusterList.new(
+    id: SecureRandom.uuid,
+    type: 'collection',
+    items: [cluster_mock, cluster_mock],
+  )
+end
+
 def pcc_mock(opts = {})
   Ionoscloud::PrivateCrossConnect.new(
     id: opts[:id] || SecureRandom.uuid,
@@ -651,6 +697,11 @@ def user_mock(opts = {})
       administrator: opts[:administrator] || false,
       force_sec_auth: opts[:force_sec_auth] || false,
     ),
+    entities: Ionoscloud::UsersEntities.new(
+      groups: Ionoscloud::Groups.new(
+        items: opts[:groups] || []
+      )
+    )
   )
 end
 
@@ -727,9 +778,9 @@ def natgateway_rule_mock(opts = {})
       name: opts[:name] || 'test',
       type: opts[:type] || 'test',
       protocol: opts[:protocol] || 'test',
-      public_ip: opts[:public_ip] || '1.1.1.1',
-      source_subnet: opts[:source_subnet] || '1.1.1.1/24',
-      target_subnet: opts[:target_subnet] || '1.1.1.1/24',
+      public_ip: opts[:public_ip] || '127.1.1.1',
+      source_subnet: opts[:source_subnet] || '127.1.1.1/24',
+      target_subnet: opts[:target_subnet] || '127.1.1.1/24',
       target_port_range: Ionoscloud::TargetPortRange.new(
         start: opts[:target_port_range_start] || 10,
         _end: opts[:target_port_range_end] || 20,
@@ -749,7 +800,7 @@ end
 def natgateway_lan_mock(opts = {})
   Ionoscloud::NatGatewayLanProperties.new(
     id: opts[:lan_id] || 1,
-    gateway_ips: opts[:gateway_ips] || ['10.8.152.227/24', '10.8.152.227/24'],
+    gateway_ips: opts[:gateway_ips] || ['127.8.152.227/24', '127.8.152.227/24'],
   )
 end
 
@@ -758,11 +809,16 @@ def natgateway_mock(opts = {})
     id: opts[:id] || SecureRandom.uuid,
     properties: Ionoscloud::NatGatewayProperties.new(
       name: opts[:name] || 'test',
-      public_ips: opts[:ips] || ['1.1.1.1'],
+      public_ips: opts[:ips] || ['127.1.1.1'],
       lans: opts[:lans] || [natgateway_lan_mock],
     ),
     entities: Ionoscloud::NatGatewayEntities.new(
       rules: natgateway_rules_mock || opts[:rules],
+      flowlogs: opts[:flowlogs] || Ionoscloud::FlowLogs.new(
+        id: 'flowlogs',
+        type: 'collection',
+        items: [],
+      ),
     ),
   )
 end
@@ -800,10 +856,10 @@ def network_loadbalancer_mock(opts = {})
     id: opts[:id] || SecureRandom.uuid,
     properties: Ionoscloud::NetworkLoadBalancerProperties.new({
       name: opts[:name] || 'network_loadbalancer_name',
-      ips: opts[:ips] || ['123.123.123.123'],
+      ips: opts[:ips] || ['127.123.123.123'],
       listener_lan: opts[:listener_lan] || 1,
       target_lan: opts[:target_lan] || 2,
-      lb_private_ips: opts[:lb_private_ips] || ['12.12.12.12'],
+      lb_private_ips: opts[:lb_private_ips] || ['127.12.12.12'],
     }),
     entities: Ionoscloud::NetworkLoadBalancerEntities.new({
       forwardingrules: opts[:rules] || network_loadbalancer_rules_mock,
@@ -817,7 +873,7 @@ def network_loadbalancer_mock(opts = {})
 end
 
 def network_loadbalancers_mock(opts = {})
-  Ionoscloud::Templates.new(
+  Ionoscloud::NetworkLoadBalancers.new(
     id: 'network_loadbalancers',
     type: 'collection',
     items: [network_loadbalancer_mock, network_loadbalancer_mock],
@@ -831,11 +887,10 @@ def network_loadbalancer_rule_mock(opts = {})
       name: opts[:name] || 'network_loadbalancer_rule_name',
       algorithm: opts[:algorithm] || 'ROUND_ROBIN',
       protocol: opts[:protocol] || 'TCP',
-      listener_ip: opts[:listener_ip] || '123.123.123.123',
+      listener_ip: opts[:listener_ip] || '127.123.123.123',
       listener_port: opts[:listener_port] || 123,
       health_check: Ionoscloud::NetworkLoadBalancerForwardingRuleHealthCheck.new(
         client_timeout: opts[:client_timeout] || 100,
-        check_timeout: opts[:check_timeout] || 200,
         connect_timeout: opts[:connect_timeout] || 300,
         target_timeout: opts[:target_timeout] || 400,
         retries: opts[:retries] || 3,
@@ -855,7 +910,7 @@ end
 
 def network_loadbalancer_rule_target_mock(opts = {})
   Ionoscloud::NetworkLoadBalancerForwardingRuleTarget.new(
-    ip: opts[:ip] || '123.123.123.123',
+    ip: opts[:ip] || '127.123.123.123',
     port: opts[:port] || 3,
     weight: opts[:weight] || 10,
     health_check: Ionoscloud::NetworkLoadBalancerForwardingRuleTargetHealthCheck.new(
@@ -866,6 +921,52 @@ def network_loadbalancer_rule_target_mock(opts = {})
   )
 end
 
+def postgres_version_data_mock(opts = {})
+  IonoscloudDbaasPostgres::PostgresVersionListData.new(name: opts[:name] || 12)
+end
+
+def postgres_version_list_mock(opts = {})
+  IonoscloudDbaasPostgres::PostgresVersionList.new(data: [postgres_version_data_mock(name: 11), postgres_version_data_mock(name: 12)])
+end
+
+def cluster_logs_message(opts = {})
+  IonoscloudDbaasPostgres::ClusterLogsMessages.new(
+    time: Time.now,
+    message: SecureRandom.uuid.to_s,
+  )
+end
+
+def cluster_logs_instance(opts = {})
+  IonoscloudDbaasPostgres::ClusterLogsInstances.new(
+    name: 'test_' + SecureRandom.uuid.to_s,
+    messages: [cluster_logs_message, cluster_logs_message],
+  )
+end
+
+def cluster_logs_mock(opts = {})
+  IonoscloudDbaasPostgres::ClusterLogs.new(instances: [cluster_logs_instance, cluster_logs_instance])
+end
+
+def cluster_backup_mock(opts = {})
+  IonoscloudDbaasPostgres::BackupResponse.new(
+    id: opts[:id] || SecureRandom.uuid,
+    properties: IonoscloudDbaasPostgres::ClusterBackup.new(
+      id: SecureRandom.uuid.to_s,
+      cluster_id: SecureRandom.uuid.to_s,
+      is_active: true,
+      version: opts[:version] || '10',
+      earliest_recovery_target_time: Time.now
+    ),
+    metadata: IonoscloudDbaasPostgres::Metadata.new(
+      created_date: Time.now,
+    ),
+  )
+end
+
+def cluster_backups_mock(opts = {})
+  IonoscloudDbaasPostgres::ClusterBackupList.new(items: [cluster_backup_mock, cluster_backup_mock])
+end
+
 def arrays_without_one_element(arr)
   result = [{ array: arr[1..], removed: [arr[0]] }]
   (1..arr.length - 1).each { |i| result.append({ array: arr[0..i - 1] + arr[i + 1..], removed: [arr[i]] }) }
@@ -876,6 +977,30 @@ def mock_wait_for(subject)
   expect(subject.api_client).to receive(:wait_for).once { true }
 end
 
+def mock_dbaas_call_api(subject, rules)
+  rules.each do |rule|
+    expect(subject.api_client_dbaas).to receive(:call_api).once do |method, path, opts|
+      result = nil
+      received_body = opts[:body].nil? ? opts[:body] : JSON.parse(opts[:body], symbolize_names: true)
+
+      expect(method.to_s).to eq(rule[:method])
+      expect(path).to eq(rule[:path])
+      expect(opts[:operation]).to eq(rule[:operation])
+      expect(opts[:form_params]).to eq(rule[:form_params] || {})
+      expect(opts[:return_type]).to eq(rule[:return_type] || nil)
+      expect(received_body).to eq(rule[:body] || nil)
+      expect(opts.slice(*(rule[:options] || {}).keys)).to eql((rule[:options] || {}))
+
+      if rule[:exception]
+        raise rule[:exception]
+      end
+
+      rule[:result]
+    end
+  end
+  expect(subject.api_client_dbaas).not_to receive(:call_api)
+end
+
 def mock_call_api(subject, rules)
   rules.each do |rule|
     expect(subject.api_client).to receive(:call_api).once do |method, path, opts|