knife-ionoscloud 5.1.1 → 6.0.0.beta.2

data/spec/chef/knife/ionoscloud_group_create_spec.rb

@@ -27,10 +27,15 @@ describe Chef::Knife::IonoscloudGroupCreate do
         create_k8s_cluster: group.properties.create_k8s_cluster,
         create_pcc: group.properties.create_pcc,
         create_internet_access: group.properties.create_internet_access,
+        create_flow_log: group.properties.create_flow_log,
+        access_and_manage_monitoring: group.properties.access_and_manage_monitoring,
+        access_and_manage_certificates: group.properties.access_and_manage_certificates,
       }
 
       subject_config.each { |key, value| subject.config[key] = value }
 
+      users = group.entities.users.items.map { |el| el.id }
+
       expect(subject).to receive(:puts).with("ID: #{group.id}")
       expect(subject).to receive(:puts).with("Name: #{group.properties.name}")
       expect(subject).to receive(:puts).with("Create Datacenter: #{group.properties.create_data_center.to_s}")
@@ -42,6 +47,10 @@ describe Chef::Knife::IonoscloudGroupCreate do
       expect(subject).to receive(:puts).with("Create K8s Clusters: #{group.properties.create_k8s_cluster.to_s}")
       expect(subject).to receive(:puts).with("Create PCC: #{group.properties.create_pcc.to_s}")
       expect(subject).to receive(:puts).with("Create Internet Acess: #{group.properties.create_internet_access.to_s}")
+      expect(subject).to receive(:puts).with("Create Flow Logs: #{group.properties.create_flow_log.to_s}")
+      expect(subject).to receive(:puts).with("Access and Manage Monitoring: #{group.properties.access_and_manage_monitoring.to_s}")
+      expect(subject).to receive(:puts).with("Access and Manage Certificates: #{group.properties.access_and_manage_certificates.to_s}")
+      expect(subject).to receive(:puts).with("Users: #{users.to_s}")
 
       mock_wait_for(subject)
       mock_call_api(

data/spec/chef/knife/ionoscloud_group_delete_spec.rb

@@ -33,6 +33,12 @@ describe Chef::Knife::IonoscloudGroupDelete do
       expect(subject).to receive(:puts).with("Access Activity Log: #{group.properties.access_activity_log.to_s}")
       expect(subject).to receive(:puts).with("S3 Privilege: #{group.properties.s3_privilege.to_s}")
       expect(subject).to receive(:puts).with("Create Backup Unit: #{group.properties.create_backup_unit.to_s}")
+      expect(subject).to receive(:puts).with("Create K8s Clusters: #{group.properties.create_k8s_cluster.to_s}")
+      expect(subject).to receive(:puts).with("Create PCC: #{group.properties.create_pcc.to_s}")
+      expect(subject).to receive(:puts).with("Create Internet Acess: #{group.properties.create_internet_access.to_s}")
+      expect(subject).to receive(:puts).with("Create Flow Logs: #{group.properties.create_flow_log.to_s}")
+      expect(subject).to receive(:puts).with("Access and Manage Monitoring: #{group.properties.access_and_manage_monitoring.to_s}")
+      expect(subject).to receive(:puts).with("Access and Manage Certificates: #{group.properties.access_and_manage_certificates.to_s}")
       expect(subject).to receive(:puts).with("Users: #{users.to_s}")
       expect(subject.ui).to receive(:warn).with("Deleted Group #{group.id}. Request ID: ")
 

data/spec/chef/knife/ionoscloud_group_get_spec.rb

@@ -34,6 +34,12 @@ describe Chef::Knife::IonoscloudGroupGet do
       expect(subject).to receive(:puts).with("Access Activity Log: #{group.properties.access_activity_log.to_s}")
       expect(subject).to receive(:puts).with("S3 Privilege: #{group.properties.s3_privilege.to_s}")
       expect(subject).to receive(:puts).with("Create Backup Unit: #{group.properties.create_backup_unit.to_s}")
+      expect(subject).to receive(:puts).with("Create K8s Clusters: #{group.properties.create_k8s_cluster.to_s}")
+      expect(subject).to receive(:puts).with("Create PCC: #{group.properties.create_pcc.to_s}")
+      expect(subject).to receive(:puts).with("Create Internet Acess: #{group.properties.create_internet_access.to_s}")
+      expect(subject).to receive(:puts).with("Create Flow Logs: #{group.properties.create_flow_log.to_s}")
+      expect(subject).to receive(:puts).with("Access and Manage Monitoring: #{group.properties.access_and_manage_monitoring.to_s}")
+      expect(subject).to receive(:puts).with("Access and Manage Certificates: #{group.properties.access_and_manage_certificates.to_s}")
       expect(subject).to receive(:puts).with("Users: #{users.to_s}")
 
       mock_call_api(

data/spec/chef/knife/ionoscloud_group_list_spec.rb

@@ -11,12 +11,18 @@ describe Chef::Knife::IonoscloudGroupList do
     @group_list = group_list = [
       subject.ui.color('ID', :bold),
       subject.ui.color('Name', :bold),
-      subject.ui.color('Create Datacenter', :bold),
-      subject.ui.color('Create Snapshot', :bold),
-      subject.ui.color('Reserve IP', :bold),
-      subject.ui.color('Access Activity Log', :bold),
-      subject.ui.color('S3 Privilege', :bold),
-      subject.ui.color('Create Backup Unit', :bold),
+      subject.ui.color('Datacenter', :bold),
+      subject.ui.color('Snapshot', :bold),
+      subject.ui.color('IP', :bold),
+      subject.ui.color('Activity', :bold),
+      subject.ui.color('S3', :bold),
+      subject.ui.color('Backup', :bold),
+      subject.ui.color('K8s', :bold),
+      subject.ui.color('PCC', :bold),
+      subject.ui.color('Internet', :bold),
+      subject.ui.color('FlowLogs', :bold),
+      subject.ui.color('Monitoring', :bold),
+      subject.ui.color('Certificates', :bold),
       @groups.items.first.id,
       @groups.items.first.properties.name,
       @groups.items.first.properties.create_data_center.to_s,
@@ -25,6 +31,12 @@ describe Chef::Knife::IonoscloudGroupList do
       @groups.items.first.properties.access_activity_log.to_s,
       @groups.items.first.properties.s3_privilege.to_s,
       @groups.items.first.properties.create_backup_unit.to_s,
+      @groups.items.first.properties.create_k8s_cluster.to_s,
+      @groups.items.first.properties.create_pcc.to_s,
+      @groups.items.first.properties.create_internet_access.to_s,
+      @groups.items.first.properties.create_flow_log.to_s,
+      @groups.items.first.properties.access_and_manage_monitoring.to_s,
+      @groups.items.first.properties.access_and_manage_certificates.to_s,
       @groups.items[1].id,
       @groups.items[1].properties.name,
       @groups.items[1].properties.create_data_center.to_s,
@@ -33,6 +45,12 @@ describe Chef::Knife::IonoscloudGroupList do
       @groups.items[1].properties.access_activity_log.to_s,
       @groups.items[1].properties.s3_privilege.to_s,
       @groups.items[1].properties.create_backup_unit.to_s,
+      @groups.items[1].properties.create_k8s_cluster.to_s,
+      @groups.items[1].properties.create_pcc.to_s,
+      @groups.items[1].properties.create_internet_access.to_s,
+      @groups.items[1].properties.create_flow_log.to_s,
+      @groups.items[1].properties.access_and_manage_monitoring.to_s,
+      @groups.items[1].properties.access_and_manage_certificates.to_s,
     ]
 
     allow(subject).to receive(:puts)
@@ -48,7 +66,7 @@ describe Chef::Knife::IonoscloudGroupList do
 
       subject_config.each { |key, value| subject.config[key] = value }
 
-      expect(subject.ui).to receive(:list).with(@group_list, :uneven_columns_across, 8)
+      expect(subject.ui).to receive(:list).with(@group_list, :uneven_columns_across, 14)
 
       mock_call_api(
         subject,
@@ -75,7 +93,7 @@ describe Chef::Knife::IonoscloudGroupList do
 
       subject_config.each { |key, value| subject.config[key] = value }
 
-      expect(subject.ui).to receive(:list).with(@group_list, :uneven_columns_across, 8)
+      expect(subject.ui).to receive(:list).with(@group_list, :uneven_columns_across, 14)
 
       expect(subject.api_client).not_to receive(:wait_for)
       mock_call_api(

data/spec/chef/knife/ionoscloud_group_update_spec.rb

@@ -0,0 +1,121 @@
+require 'spec_helper'
+require 'ionoscloud_group_update'
+
+Chef::Knife::IonoscloudGroupUpdate.load_deps
+
+describe Chef::Knife::IonoscloudGroupUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudGroupUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call UserManagementApi.um_groups_put' do
+      group = group_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        group_id: group.id,
+        name: group.properties.name + '_edited',
+        create_snapshot: !group.properties.create_snapshot,
+        reserve_ip: group.properties.reserve_ip,
+        access_activity_log: !group.properties.access_activity_log,
+        create_backup_unit: !group.properties.create_backup_unit,
+        create_pcc: !group.properties.create_pcc,
+        create_flow_log: !group.properties.create_flow_log,
+        access_and_manage_monitoring: !group.properties.access_and_manage_monitoring,
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      users = group.entities.users.items.map { |el| el.id }
+
+      expect(subject).to receive(:puts).with("ID: #{group.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("Create Datacenter: #{group.properties.create_data_center.to_s}")
+      expect(subject).to receive(:puts).with("Create Snapshot: #{subject_config[:create_snapshot]}")
+      expect(subject).to receive(:puts).with("Reserve IP: #{group.properties.reserve_ip.to_s}")
+      expect(subject).to receive(:puts).with("Access Activity Log: #{subject_config[:access_activity_log]}")
+      expect(subject).to receive(:puts).with("S3 Privilege: #{group.properties.s3_privilege.to_s}")
+      expect(subject).to receive(:puts).with("Create Backup Unit: #{subject_config[:create_backup_unit]}")
+      expect(subject).to receive(:puts).with("Create K8s Clusters: #{group.properties.create_k8s_cluster.to_s}")
+      expect(subject).to receive(:puts).with("Create PCC: #{subject_config[:create_pcc]}")
+      expect(subject).to receive(:puts).with("Create Internet Acess: #{group.properties.create_internet_access.to_s}")
+      expect(subject).to receive(:puts).with("Create Flow Logs: #{subject_config[:create_flow_log]}")
+      expect(subject).to receive(:puts).with("Access and Manage Monitoring: #{subject_config[:access_and_manage_monitoring]}")
+      expect(subject).to receive(:puts).with("Access and Manage Certificates: #{group.properties.access_and_manage_certificates.to_s}")
+      expect(subject).to receive(:puts).with("Users: #{users.to_s}")
+
+      expected_body = group.properties.to_hash.merge({
+        name: subject_config[:name],
+        createSnapshot: subject_config[:create_snapshot],
+        reserveIp: subject_config[:reserve_ip],
+        accessActivityLog: subject_config[:access_activity_log],
+        createBackupUnit: subject_config[:create_backup_unit],
+        createPcc: subject_config[:create_pcc],
+        createFlowLog: subject_config[:create_flow_log],
+        accessAndManageMonitoring: subject_config[:access_and_manage_monitoring],
+      })
+
+      group.properties.name = subject_config[:name]
+      group.properties.create_snapshot = subject_config[:create_snapshot]
+      group.properties.access_activity_log = subject_config[:access_activity_log]
+      group.properties.create_backup_unit = subject_config[:create_backup_unit]
+      group.properties.create_pcc = subject_config[:create_pcc]
+      group.properties.create_flow_log = subject_config[:create_flow_log]
+      group.properties.access_and_manage_monitoring = subject_config[:access_and_manage_monitoring]
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/um/groups/#{group.id}",
+            operation: :'UserManagementApi.um_groups_find_by_id',
+            return_type: 'Group',
+            result: group,
+          },
+          {
+            method: 'PUT',
+            path: "/um/groups/#{group.id}",
+            operation: :'UserManagementApi.um_groups_put',
+            return_type: 'Group',
+            body: { properties: expected_body },
+            result: group,
+          },
+          {
+            method: 'GET',
+            path: "/um/groups/#{group.id}",
+            operation: :'UserManagementApi.um_groups_find_by_id',
+            return_type: 'Group',
+            result: group,
+          },
+        ],
+      )
+
+      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/chef/knife/ionoscloud_group_user_add_spec.rb

@@ -35,6 +35,12 @@ describe Chef::Knife::IonoscloudGroupUserAdd do
       expect(subject).to receive(:puts).with("Access Activity Log: #{group.properties.access_activity_log.to_s}")
       expect(subject).to receive(:puts).with("S3 Privilege: #{group.properties.s3_privilege.to_s}")
       expect(subject).to receive(:puts).with("Create Backup Unit: #{group.properties.create_backup_unit.to_s}")
+      expect(subject).to receive(:puts).with("Create K8s Clusters: #{group.properties.create_k8s_cluster.to_s}")
+      expect(subject).to receive(:puts).with("Create PCC: #{group.properties.create_pcc.to_s}")
+      expect(subject).to receive(:puts).with("Create Internet Acess: #{group.properties.create_internet_access.to_s}")
+      expect(subject).to receive(:puts).with("Create Flow Logs: #{group.properties.create_flow_log.to_s}")
+      expect(subject).to receive(:puts).with("Access and Manage Monitoring: #{group.properties.access_and_manage_monitoring.to_s}")
+      expect(subject).to receive(:puts).with("Access and Manage Certificates: #{group.properties.access_and_manage_certificates.to_s}")
       expect(subject).to receive(:puts).with("Users: #{users.to_s}")
       expect(subject.ui).to receive(:info).with("Added User #{user.id} to the Group #{group.id}. Request ID: .")
 

data/spec/chef/knife/ionoscloud_group_user_remove_spec.rb

@@ -35,6 +35,12 @@ describe Chef::Knife::IonoscloudGroupUserRemove do
       expect(subject).to receive(:puts).with("Access Activity Log: #{group.properties.access_activity_log.to_s}")
       expect(subject).to receive(:puts).with("S3 Privilege: #{group.properties.s3_privilege.to_s}")
       expect(subject).to receive(:puts).with("Create Backup Unit: #{group.properties.create_backup_unit.to_s}")
+      expect(subject).to receive(:puts).with("Create K8s Clusters: #{group.properties.create_k8s_cluster.to_s}")
+      expect(subject).to receive(:puts).with("Create PCC: #{group.properties.create_pcc.to_s}")
+      expect(subject).to receive(:puts).with("Create Internet Acess: #{group.properties.create_internet_access.to_s}")
+      expect(subject).to receive(:puts).with("Create Flow Logs: #{group.properties.create_flow_log.to_s}")
+      expect(subject).to receive(:puts).with("Access and Manage Monitoring: #{group.properties.access_and_manage_monitoring.to_s}")
+      expect(subject).to receive(:puts).with("Access and Manage Certificates: #{group.properties.access_and_manage_certificates.to_s}")
       expect(subject).to receive(:puts).with("Users: #{users.to_s}")
       expect(subject.ui).to receive(:warn).with("Removed User #{user.id} from the Group #{group.id}. Request ID: .")
 

data/spec/chef/knife/ionoscloud_image_list_spec.rb

@@ -12,7 +12,7 @@ describe Chef::Knife::IonoscloudImageList do
   end
 
   describe '#run' do
-    it 'should call ImageApi.images_get' do
+    it 'should call ImagesApi.images_get' do
       images = images_mock
       subject_config = {
         ionoscloud_username: 'email',
@@ -44,7 +44,7 @@ describe Chef::Knife::IonoscloudImageList do
           {
             method: 'GET',
             path: '/images',
-            operation: :'ImageApi.images_get',
+            operation: :'ImagesApi.images_get',
             return_type: 'Images',
             result: images,
           },

data/spec/chef/knife/ionoscloud_ipblock_get_spec.rb

@@ -0,0 +1,66 @@
+require 'spec_helper'
+require 'ionoscloud_ipblock_get'
+
+Chef::Knife::IonoscloudIpblockGet.load_deps
+
+describe Chef::Knife::IonoscloudIpblockGet do
+  before :each do
+    subject { Chef::Knife::IonoscloudIpblockGet.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call IPBlocksApi.ipblocks_find_by_id' do
+      ipblock = ipblock_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        ipblock_id: ipblock.id,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      ip_consumers = (ipblock.properties.ip_consumers.nil? ? [] : ipblock.properties.ip_consumers.map { |el| el.to_hash })
+      expect(subject).to receive(:puts).with("ID: #{ipblock.id}")
+      expect(subject).to receive(:puts).with("Name: #{ipblock.properties.name}")
+      expect(subject).to receive(:puts).with("IP Addresses: #{ipblock.properties.ips.to_s}")
+      expect(subject).to receive(:puts).with("Location: #{ipblock.properties.location}")
+      expect(subject).to receive(:puts).with("IP Consumers: #{ip_consumers}")
+
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/ipblocks/#{ipblock.id}",
+            operation: :'IPBlocksApi.ipblocks_find_by_id',
+            return_type: 'IpBlock',
+            result: ipblock,
+          },
+        ],
+      )
+
+      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/chef/knife/ionoscloud_ipblock_update_spec.rb

@@ -0,0 +1,79 @@
+require 'spec_helper'
+require 'ionoscloud_ipblock_update'
+
+Chef::Knife::IonoscloudIpblockUpdate.load_deps
+
+describe Chef::Knife::IonoscloudIpblockUpdate do
+  before :each do
+    subject { Chef::Knife::IonoscloudIpblockUpdate.new }
+
+    allow(subject).to receive(:puts)
+    allow(subject).to receive(:print)
+  end
+
+  describe '#run' do
+    it 'should call DataCenterApi.datacenters_patch' do
+      ipblock = ipblock_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        ipblock_id: ipblock.id,
+        name: ipblock.properties.name + '_edited',
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      ip_consumers = (ipblock.properties.ip_consumers.nil? ? [] : ipblock.properties.ip_consumers.map { |el| el.to_hash })
+      expect(subject).to receive(:puts).with("ID: #{ipblock.id}")
+      expect(subject).to receive(:puts).with("Name: #{subject_config[:name]}")
+      expect(subject).to receive(:puts).with("IP Addresses: #{ipblock.properties.ips.to_s}")
+      expect(subject).to receive(:puts).with("Location: #{ipblock.properties.location}")
+      expect(subject).to receive(:puts).with("IP Consumers: #{ip_consumers}")
+
+      ipblock.properties.name = subject_config[:name]
+
+      mock_wait_for(subject)
+      mock_call_api(
+        subject,
+        [
+          {
+            method: 'PATCH',
+            path: "/ipblocks/#{ipblock.id}",
+            operation: :'IPBlocksApi.ipblocks_patch',
+            return_type: 'IpBlock',
+            body: { name: subject_config[:name] },
+            result: ipblock,
+          },
+          {
+            method: 'GET',
+            path: "/ipblocks/#{ipblock.id}",
+            operation: :'IPBlocksApi.ipblocks_find_by_id',
+            return_type: 'IpBlock',
+            result: ipblock,
+          },
+        ],
+      )
+
+      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/chef/knife/ionoscloud_ipfailover_add_spec.rb

@@ -20,7 +20,7 @@ describe Chef::Knife::IonoscloudIpfailoverAdd do
         datacenter_id: 'datacenter_id',
         lan_id: lan.id,
         nic_id: 'nic_id',
-        ip: '1.1.1.1',
+        ip: '127.1.1.1',
         yes: true,
       }
 
@@ -38,21 +38,21 @@ describe Chef::Knife::IonoscloudIpfailoverAdd do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{lan.id}",
-            operation: :'LanApi.datacenters_lans_find_by_id',
+            operation: :'LansApi.datacenters_lans_find_by_id',
             return_type: 'Lan',
             result: lan,
           },
           {
             method: 'PATCH',
             path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{lan.id}",
-            operation: :'LanApi.datacenters_lans_patch',
+            operation: :'LansApi.datacenters_lans_patch',
             body: Ionoscloud::LanProperties.new({ ip_failover: [({ ip: subject_config[:ip], nicUuid: subject_config[:nic_id] })] }).to_hash,
             return_type: 'Lan',
           },
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{lan.id}",
-            operation: :'LanApi.datacenters_lans_find_by_id',
+            operation: :'LansApi.datacenters_lans_find_by_id',
             return_type: 'Lan',
             result: lan,
           },

data/spec/chef/knife/ionoscloud_ipfailover_remove_spec.rb

@@ -13,7 +13,7 @@ describe Chef::Knife::IonoscloudIpfailoverRemove do
 
   describe '#run' do
     it 'should call Lan.datacenters_lans_patch when the ip failvoer rule exists' do
-      lan = lan_mock({ ip_failover: [ Ionoscloud::IPFailover.new(ip: '1.1.1.2', nic_uuid: 'nic_id2')] })
+      lan = lan_mock({ ip_failover: [ Ionoscloud::IPFailover.new(ip: '127.1.1.2', nic_uuid: 'nic_id2')] })
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
@@ -41,21 +41,21 @@ describe Chef::Knife::IonoscloudIpfailoverRemove do
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{lan.id}",
-            operation: :'LanApi.datacenters_lans_find_by_id',
+            operation: :'LansApi.datacenters_lans_find_by_id',
             return_type: 'Lan',
             result: lan,
           },
           {
             method: 'PATCH',
             path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{lan.id}",
-            operation: :'LanApi.datacenters_lans_patch',
+            operation: :'LansApi.datacenters_lans_patch',
             body: Ionoscloud::LanProperties.new({ ip_failover: [] }).to_hash,
             return_type: 'Lan',
           },
           {
             method: 'GET',
             path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{lan.id}",
-            operation: :'LanApi.datacenters_lans_find_by_id',
+            operation: :'LansApi.datacenters_lans_find_by_id',
             return_type: 'Lan',
             result: lan_copy,
           },
@@ -66,14 +66,14 @@ describe Chef::Knife::IonoscloudIpfailoverRemove do
     end
 
     it 'should not call Lan.datacenters_lans_patch when the ip failvoer rule does not exist' do
-      lan = lan_mock({ ip_failover: [ Ionoscloud::IPFailover.new(ip: '1.1.1.2', nic_uuid: 'nic_id2')] })
+      lan = lan_mock({ ip_failover: [ Ionoscloud::IPFailover.new(ip: '127.1.1.2', nic_uuid: 'nic_id2')] })
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
         datacenter_id: 'datacenter_id',
         lan_id: lan.id,
         nic_id: 'nic_id',
-        ip: '1.1.1.1',
+        ip: '127.1.1.1',
         yes: true,
       }
 
@@ -91,7 +91,7 @@ describe Chef::Knife::IonoscloudIpfailoverRemove do
           {
               method: 'GET',
               path: "/datacenters/#{subject_config[:datacenter_id]}/lans/#{lan.id}",
-              operation: :'LanApi.datacenters_lans_find_by_id',
+              operation: :'LansApi.datacenters_lans_find_by_id',
               return_type: 'Lan',
               result: lan,
           },

data/spec/chef/knife/ionoscloud_k8s_create_spec.rb

@@ -21,17 +21,24 @@ describe Chef::Knife::IonoscloudK8sCreate do
         version: cluster.properties.k8s_version,
         maintenance_day: cluster.properties.maintenance_window.day_of_the_week,
         maintenance_time: cluster.properties.maintenance_window.time,
+        s3_buckets: cluster.properties.s3_buckets.map { |el| el.name }.join(','),
+        api_subnet_allow_list: cluster.properties.api_subnet_allow_list.join(','),
       }
 
       subject_config.each { |key, value| subject.config[key] = value }
 
       maintenance_window = "#{cluster.properties.maintenance_window.day_of_the_week}, #{cluster.properties.maintenance_window.time}"
+      s3_buckets = (cluster.properties.s3_buckets.nil? ? [] : cluster.properties.s3_buckets.map { |el| el.name })
 
       expect(subject).to receive(:puts).with("ID: #{cluster.id}")
       expect(subject).to receive(:puts).with("Name: #{cluster.properties.name}")
       expect(subject).to receive(:puts).with("k8s Version: #{cluster.properties.k8s_version}")
       expect(subject).to receive(:puts).with("Maintenance Window: #{maintenance_window}")
       expect(subject).to receive(:puts).with("State: #{cluster.metadata.state}")
+      expect(subject).to receive(:puts).with("Api Subnet Allow List: #{cluster.properties.api_subnet_allow_list}")
+      expect(subject).to receive(:puts).with("S3 Buckets: #{s3_buckets}")
+      expect(subject).to receive(:puts).with("Available Upgrade Versions: #{cluster.properties.available_upgrade_versions}")
+      expect(subject).to receive(:puts).with("Viable NodePool Versions: #{cluster.properties.viable_node_pool_versions}")
 
       expected_body = cluster.properties.to_hash
       expected_body.delete(:viableNodePoolVersions)

data/spec/chef/knife/ionoscloud_k8s_delete_spec.rb

@@ -13,7 +13,7 @@ describe Chef::Knife::IonoscloudK8sDelete do
 
   describe '#run' do
     it 'should call KubernetesApi.k8s_delete when the ID is valid' do
-      k8s_cluster = k8s_cluster_mock({ entities: Ionoscloud::KubernetesClusterEntities.new(
+      cluster = k8s_cluster_mock({ entities: Ionoscloud::KubernetesClusterEntities.new(
         nodepools: k8s_nodepools_mock({ items: [] }),
       ) })
       subject_config = {
@@ -23,16 +23,21 @@ describe Chef::Knife::IonoscloudK8sDelete do
       }
 
       subject_config.each { |key, value| subject.config[key] = value }
-      subject.name_args = [k8s_cluster.id]
+      subject.name_args = [cluster.id]
 
-      maintenance_window = "#{k8s_cluster.properties.maintenance_window.day_of_the_week}, #{k8s_cluster.properties.maintenance_window.time}"
+      maintenance_window = "#{cluster.properties.maintenance_window.day_of_the_week}, #{cluster.properties.maintenance_window.time}"
+      s3_buckets = (cluster.properties.s3_buckets.nil? ? [] : cluster.properties.s3_buckets.map { |el| el.name })
 
-      expect(subject).to receive(:puts).with("ID: #{k8s_cluster.id}")
-      expect(subject).to receive(:puts).with("Name: #{k8s_cluster.properties.name}")
-      expect(subject).to receive(:puts).with("Version: #{k8s_cluster.properties.k8s_version}")
+      expect(subject).to receive(:puts).with("ID: #{cluster.id}")
+      expect(subject).to receive(:puts).with("Name: #{cluster.properties.name}")
+      expect(subject).to receive(:puts).with("k8s Version: #{cluster.properties.k8s_version}")
       expect(subject).to receive(:puts).with("Maintenance Window: #{maintenance_window}")
-      expect(subject).to receive(:puts).with("State: #{k8s_cluster.metadata.state}")
-      expect(subject.ui).to receive(:warn).with("Deleted K8s Cluster #{k8s_cluster.id}. Request ID: ")
+      expect(subject).to receive(:puts).with("State: #{cluster.metadata.state}")
+      expect(subject).to receive(:puts).with("Api Subnet Allow List: #{cluster.properties.api_subnet_allow_list}")
+      expect(subject).to receive(:puts).with("S3 Buckets: #{s3_buckets}")
+      expect(subject).to receive(:puts).with("Available Upgrade Versions: #{cluster.properties.available_upgrade_versions}")
+      expect(subject).to receive(:puts).with("Viable NodePool Versions: #{cluster.properties.viable_node_pool_versions}")
+      expect(subject.ui).to receive(:warn).with("Deleted K8s Cluster #{cluster.id}. Request ID: ")
 
       expect(subject).not_to receive(:wait_for)
       expect(subject).to receive(:get_request_id).once
@@ -41,14 +46,14 @@ describe Chef::Knife::IonoscloudK8sDelete do
         [
           {
             method: 'GET',
-            path: "/k8s/#{k8s_cluster.id}",
+            path: "/k8s/#{cluster.id}",
             operation: :'KubernetesApi.k8s_find_by_cluster_id',
             return_type: 'KubernetesCluster',
-            result: k8s_cluster,
+            result: cluster,
           },
           {
             method: 'DELETE',
-            path: "/k8s/#{k8s_cluster.id}",
+            path: "/k8s/#{cluster.id}",
             operation: :'KubernetesApi.k8s_delete',
           },
         ],