knife-ionoscloud 6.1.0 → 6.1.1

data/lib/chef/knife/ionoscloud_s3key_delete.rb

@@ -41,7 +41,7 @@ class Chef
 
           begin
             confirm('Do you really want to delete this S3 key')
-          rescue SystemExit => exc
+          rescue SystemExit
             next
           end
 

data/lib/chef/knife/ionoscloud_server_create.rb

@@ -48,10 +48,10 @@ class Chef
       def initialize(args = [])
         super(args)
         @description =
-        "One of the unique features of the Ionoscloud platform when compared "\
-        "with the other providers is that they allow you to define your own settings "\
+        'One of the unique features of the Ionoscloud platform when compared '\
+        'with the other providers is that they allow you to define your own settings '\
         "for cores, memory, and disk size without being tied to a particular size or flavor.\n\n"\
-        "Note: _The memory parameter value must be a multiple of 256, e.g. 256, 512, 768, 1024, and so forth._"
+        'Note: _The memory parameter value must be a multiple of 256, e.g. 256, 512, 768, 1024, and so forth._'
         @directory = 'compute-engine'
         @required_options = [:datacenter_id, :cores, :ram, :ionoscloud_username, :ionoscloud_password]
       end

data/lib/chef/knife/ionoscloud_server_delete.rb

@@ -16,8 +16,8 @@ class Chef
         super(args)
         @description =
         "This will remove a server from a VDC.\n\n"\
-        "**NOTE**: This will not automatically remove the storage volume\\(s\\) "\
-        "attached to a server. A separate API call is required to perform that action."
+        '**NOTE**: This will not automatically remove the storage volume\\(s\\) '\
+        'attached to a server. A separate API call is required to perform that action.'
         @directory = 'compute-engine'
         @required_options = [:datacenter_id, :ionoscloud_username, :ionoscloud_password]
       end
@@ -43,7 +43,7 @@ class Chef
 
           begin
             confirm('Do you really want to delete this server')
-          rescue SystemExit => exc
+          rescue SystemExit
             next
           end
 

data/lib/chef/knife/ionoscloud_server_upgrade.rb

@@ -20,7 +20,7 @@ class Chef
       def initialize(args = [])
         super(args)
         @description =
-        "This will upgrade the version of the server, if needed. To verify if there is an upgrade available for a server, "\
+        'This will upgrade the version of the server, if needed. To verify if there is an upgrade available for a server, '\
         "call '/datacenters/{datacenterId}/servers?upgradeNeeded=true'."
         @directory = 'compute-engine'
         @required_options = [:datacenter_id, :server_id, :ionoscloud_username, :ionoscloud_password]

data/lib/chef/knife/ionoscloud_share_delete.rb

@@ -41,7 +41,7 @@ class Chef
 
           begin
             confirm('Do you really want to delete this Resource Share')
-          rescue SystemExit => exc
+          rescue SystemExit
             next
           end
 

data/lib/chef/knife/ionoscloud_snapshot_delete.rb

@@ -36,7 +36,7 @@ class Chef
 
           begin
             confirm('Do you really want to delete this Snapshot')
-          rescue SystemExit => exc
+          rescue SystemExit
             next
           end
 

data/lib/chef/knife/ionoscloud_snapshot_list.rb

@@ -25,7 +25,7 @@ class Chef
           ui.color('Name', :bold),
           ui.color('Description', :bold),
           ui.color('Location', :bold),
-          ui.color('Size', :bold)
+          ui.color('Size', :bold),
         ]
 
         snapshot_api = Ionoscloud::SnapshotsApi.new(api_client)

data/lib/chef/knife/ionoscloud_user_delete.rb

@@ -38,7 +38,7 @@ class Chef
 
           begin
             confirm('Do you really want to delete this User')
-          rescue SystemExit => exc
+          rescue SystemExit
             next
           end
 

data/lib/chef/knife/ionoscloud_volume_delete.rb

@@ -45,7 +45,7 @@ class Chef
 
           begin
             confirm('Do you really want to delete this volume')
-          rescue SystemExit => exc
+          rescue SystemExit
             next
           end
 

data/lib/chef/knife/ionoscloud_volume_detach.rb

@@ -20,10 +20,10 @@ class Chef
       def initialize(args = [])
         super(args)
         @description =
-        "This will detach the volume from the server. Depending on the volume "\
+        'This will detach the volume from the server. Depending on the volume '\
         "HotUnplug settings, this may result in the server being rebooted.\n\n"\
-        "This will NOT delete the volume from your virtual data center. You will "\
-        "need to make a separate request to delete a volume."
+        'This will NOT delete the volume from your virtual data center. You will '\
+        'need to make a separate request to delete a volume.'
         @directory = 'compute-engine'
         @required_options = [:datacenter_id, :server_id, :ionoscloud_username, :ionoscloud_password]
       end
@@ -59,7 +59,7 @@ class Chef
 
           begin
             confirm('Do you really want to detach this volume')
-          rescue SystemExit => exc
+          rescue SystemExit
             next
           end
 

data/lib/knife-ionoscloud/version.rb

@@ -1,6 +1,6 @@
 module Knife
   module Ionoscloud
-    VERSION = '6.1.0'
+    VERSION = '6.1.1'
     MAJOR, MINOR, TINY = VERSION.split('.')
   end
 end

data/sonar-project.properties

@@ -1,6 +1,9 @@
 sonar.projectKey=knife-plugin
 sonar.organization=ionos-cloud
 
+sonar.ruby.coverage.reportPaths=coverage/coverage.json
+sonar.cpd.exclusions=spec/chef/knife/*.rb
+
 # This is the name and version displayed in the SonarCloud UI.
 #sonar.projectName=knife-plugin
 #sonar.projectVersion=1.0

data/spec/chef/knife/ionoscloud_backupunit_create_spec.rb

@@ -14,7 +14,7 @@ describe Chef::Knife::IonoscloudBackupunitCreate do
   describe '#run' do
     it 'should call BackupUnitsApi.backupunits_post with the expected arguments and output based on what it receives' do
       backupunit = backupunit_mock
-      subject_config = {
+      {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
         name: backupunit.properties.name,
@@ -48,7 +48,6 @@ describe Chef::Knife::IonoscloudBackupunitCreate 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]}")

data/spec/chef/knife/ionoscloud_backupunit_delete_spec.rb

@@ -84,7 +84,6 @@ describe Chef::Knife::IonoscloudBackupunitDelete 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]}")

data/spec/chef/knife/ionoscloud_backupunit_get_spec.rb

@@ -48,7 +48,6 @@ describe Chef::Knife::IonoscloudBackupunitGet 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]}")

data/spec/chef/knife/ionoscloud_backupunit_list_spec.rb

@@ -55,7 +55,6 @@ describe Chef::Knife::IonoscloudBackupunitList 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]}")

data/spec/chef/knife/ionoscloud_backupunit_ssourl_spec.rb

@@ -74,9 +74,7 @@ describe Chef::Knife::IonoscloudBackupunitSsourl do
       allow(subject).to receive(:puts)
       allow(subject).to receive(:print)
 
-      arrays_without_one_element(required_options).each {
-        |test_case|
-
+      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]}")
@@ -87,7 +85,7 @@ describe Chef::Knife::IonoscloudBackupunitSsourl do
         end
 
         required_options.each { |value| subject.config[value] = nil }
-      }
+      end
     end
   end
 end

data/spec/chef/knife/ionoscloud_backupunit_update_spec.rb

@@ -61,7 +61,6 @@ describe Chef::Knife::IonoscloudBackupunitUpdate 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]}")

data/spec/chef/knife/ionoscloud_base_spec.rb

@@ -99,14 +99,13 @@ describe Chef::Knife::IonoscloudBaseTest do
       expect { subject.is_done? request_id }.to raise_error(SystemExit) do |error|
         expect(error.status).to eq(1)
       end
-
     end
   end
 
   describe '#get_request_id' do
     it 'should return nil when Headers has no Location key' do
       headers = {
-        'another_key' => '/requests/123'
+        'another_key' => '/requests/123',
       }
 
       expect(subject.get_request_id headers).to be_nil
@@ -114,7 +113,7 @@ describe Chef::Knife::IonoscloudBaseTest do
 
     it 'should return nil when Headers[\'Location\'] contains no uuid' do
       headers = {
-        'Location' => '/requests/123'
+        'Location' => '/requests/123',
       }
 
       expect(subject.get_request_id headers).to be_nil
@@ -123,7 +122,7 @@ describe Chef::Knife::IonoscloudBaseTest do
     it 'should extract the ID from the headers' do
       request_id = SecureRandom.uuid
       headers = {
-        'Location' => "/requests/#{request_id}"
+        'Location' => "/requests/#{request_id}",
       }
 
       expect(subject.get_request_id headers).to eql(request_id)

data/spec/chef/knife/ionoscloud_composite_server_create_spec.rb

@@ -97,7 +97,6 @@ describe Chef::Knife::IonoscloudCompositeServerCreate 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]}")

data/spec/chef/knife/ionoscloud_contract_list_spec.rb

@@ -69,7 +69,6 @@ describe Chef::Knife::IonoscloudContractList 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]}")

data/spec/chef/knife/ionoscloud_cube_server_create_spec.rb

@@ -60,7 +60,7 @@ describe Chef::Knife::IonoscloudCubeServerCreate do
       expect(subject).to receive(:puts).with("Ram: #{server.properties.ram}")
       expect(subject).to receive(:puts).with("Availability Zone: #{server.properties.availability_zone}")
       expect(subject).to receive(:puts).with("Boot Volume: #{server.entities.volumes.items.first.id}")
-      expect(subject).to receive(:puts).with("Boot CDROM: ")
+      expect(subject).to receive(:puts).with('Boot CDROM: ')
 
       expected_properties = server.properties.to_hash
       expected_properties.delete(:bootCdrom)
@@ -176,7 +176,7 @@ describe Chef::Knife::IonoscloudCubeServerCreate do
       expect(subject).to receive(:puts).with("Ram: #{server.properties.ram}")
       expect(subject).to receive(:puts).with("Availability Zone: #{server.properties.availability_zone}")
       expect(subject).to receive(:puts).with("Boot Volume: #{server.entities.volumes.items.first.id}")
-      expect(subject).to receive(:puts).with("Boot CDROM: ")
+      expect(subject).to receive(:puts).with('Boot CDROM: ')
 
       expected_properties = server.properties.to_hash
       expected_properties.delete(:bootCdrom)
@@ -232,7 +232,6 @@ describe Chef::Knife::IonoscloudCubeServerCreate 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]}")

data/spec/chef/knife/ionoscloud_datacenter_create_spec.rb

@@ -59,7 +59,6 @@ describe Chef::Knife::IonoscloudDatacenterCreate 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]}")

data/spec/chef/knife/ionoscloud_datacenter_delete_spec.rb

@@ -88,7 +88,6 @@ describe Chef::Knife::IonoscloudDatacenterDelete 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]}")

data/spec/chef/knife/ionoscloud_datacenter_get_spec.rb

@@ -52,7 +52,6 @@ describe Chef::Knife::IonoscloudDatacenterGet 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]}")

data/spec/chef/knife/ionoscloud_datacenter_list_spec.rb

@@ -61,7 +61,6 @@ describe Chef::Knife::IonoscloudDatacenterList 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]}")

data/spec/chef/knife/ionoscloud_datacenter_update_spec.rb

@@ -65,7 +65,6 @@ describe Chef::Knife::IonoscloudDatacenterUpdate 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]}")

data/spec/chef/knife/ionoscloud_dbaas_postgres_backup_get_spec.rb

@@ -50,7 +50,6 @@ describe Chef::Knife::IonoscloudDbaasPostgresBackupGet 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]}")

data/spec/chef/knife/ionoscloud_dbaas_postgres_backup_list_spec.rb

@@ -67,7 +67,7 @@ describe Chef::Knife::IonoscloudDbaasPostgresBackupList do
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
-        cluster_id: 'cluster_id'
+        cluster_id: 'cluster_id',
       }
 
       subject_config.each { |key, value| subject.config[key] = value }
@@ -94,7 +94,6 @@ describe Chef::Knife::IonoscloudDbaasPostgresBackupList 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]}")

data/spec/chef/knife/ionoscloud_dbaas_postgres_cluster_create_spec.rb

@@ -21,6 +21,7 @@ describe Chef::Knife::IonoscloudDbaasPostgresClusterCreate do
         display_name: cluster.properties.display_name,
         postgres_version: cluster.properties.postgres_version,
         location: cluster.properties.location,
+        backup_location: cluster.properties.backup_location,
         instances: cluster.properties.instances,
         ram: cluster.properties.ram,
         cores: cluster.properties.cores,
@@ -37,8 +38,8 @@ describe Chef::Knife::IonoscloudDbaasPostgresClusterCreate do
         day_of_the_week: cluster.properties.maintenance_window.day_of_the_week,
         synchronization_mode: cluster.properties.synchronization_mode,
         state: cluster.metadata.state,
-        username: "usr",
-        password: "pass123",
+        username: 'usr',
+        password: 'pass123',
       }
 
       subject_config.each { |key, value| subject.config[key] = value }
@@ -47,6 +48,7 @@ describe Chef::Knife::IonoscloudDbaasPostgresClusterCreate do
       expect(subject).to receive(:puts).with("Display Name: #{cluster.properties.display_name}")
       expect(subject).to receive(:puts).with("Postgres Version: #{cluster.properties.postgres_version}")
       expect(subject).to receive(:puts).with("Location: #{cluster.properties.location}")
+      expect(subject).to receive(:puts).with("Backup location: #{cluster.properties.backup_location}")
       expect(subject).to receive(:puts).with("Instances: #{cluster.properties.instances}")
       expect(subject).to receive(:puts).with("RAM Size: #{cluster.properties.ram}")
       expect(subject).to receive(:puts).with("Cores: #{cluster.properties.cores}")
@@ -82,7 +84,6 @@ describe Chef::Knife::IonoscloudDbaasPostgresClusterCreate 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]}")

data/spec/chef/knife/ionoscloud_dbaas_postgres_cluster_delete_spec.rb

@@ -27,6 +27,7 @@ describe Chef::Knife::IonoscloudDbaasPostgresClusterDelete do
       expect(subject).to receive(:puts).with("Display Name: #{cluster.properties.display_name}")
       expect(subject).to receive(:puts).with("Postgres Version: #{cluster.properties.postgres_version}")
       expect(subject).to receive(:puts).with("Location: #{cluster.properties.location}")
+      expect(subject).to receive(:puts).with("Backup location: #{cluster.properties.backup_location}")
       expect(subject).to receive(:puts).with("Instances: #{cluster.properties.instances}")
       expect(subject).to receive(:puts).with("RAM Size: #{cluster.properties.ram}")
       expect(subject).to receive(:puts).with("Cores: #{cluster.properties.cores}")
@@ -92,7 +93,6 @@ describe Chef::Knife::IonoscloudDbaasPostgresClusterDelete 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]}")

data/spec/chef/knife/ionoscloud_dbaas_postgres_cluster_get_spec.rb

@@ -27,6 +27,7 @@ describe Chef::Knife::IonoscloudDbaasPostgresClusterGet do
       expect(subject).to receive(:puts).with("Display Name: #{cluster.properties.display_name}")
       expect(subject).to receive(:puts).with("Postgres Version: #{cluster.properties.postgres_version}")
       expect(subject).to receive(:puts).with("Location: #{cluster.properties.location}")
+      expect(subject).to receive(:puts).with("Backup location: #{cluster.properties.backup_location}")
       expect(subject).to receive(:puts).with("Instances: #{cluster.properties.instances}")
       expect(subject).to receive(:puts).with("RAM Size: #{cluster.properties.ram}")
       expect(subject).to receive(:puts).with("Cores: #{cluster.properties.cores}")
@@ -56,7 +57,6 @@ describe Chef::Knife::IonoscloudDbaasPostgresClusterGet 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]}")

data/spec/chef/knife/ionoscloud_dbaas_postgres_cluster_list_spec.rb

@@ -26,6 +26,7 @@ describe Chef::Knife::IonoscloudDbaasPostgresClusterList do
         subject.ui.color('Display Name', :bold),
         subject.ui.color('Postgres Version', :bold),
         subject.ui.color('Location', :bold),
+        subject.ui.color('Backup location', :bold),
         subject.ui.color('Instances', :bold),
         subject.ui.color('Datacenter ID', :bold),
         subject.ui.color('Lan ID', :bold),
@@ -38,6 +39,7 @@ describe Chef::Knife::IonoscloudDbaasPostgresClusterList do
         cluster_list << cluster.properties.display_name
         cluster_list << cluster.properties.postgres_version
         cluster_list << cluster.properties.location
+        cluster_list << cluster.properties.backup_location
         cluster_list << cluster.properties.instances
         cluster_list << cluster.properties.connections.first.datacenter_id
         cluster_list << cluster.properties.connections.first.lan_id
@@ -45,7 +47,7 @@ describe Chef::Knife::IonoscloudDbaasPostgresClusterList do
         cluster_list << cluster.metadata.state
       end
 
-      expect(subject.ui).to receive(:list).with(cluster_list, :uneven_columns_across, 9)
+      expect(subject.ui).to receive(:list).with(cluster_list, :uneven_columns_across, 10)
 
       mock_dbaas_call_api(
         subject,
@@ -67,7 +69,6 @@ describe Chef::Knife::IonoscloudDbaasPostgresClusterList 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]}")

data/spec/chef/knife/ionoscloud_dbaas_postgres_cluster_restore_spec.rb

@@ -42,7 +42,6 @@ describe Chef::Knife::IonoscloudDbaasPostgresClusterRestore 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]}")

data/spec/chef/knife/ionoscloud_dbaas_postgres_cluster_update_spec.rb

@@ -28,6 +28,7 @@ describe Chef::Knife::IonoscloudDbaasPostgresClusterUpdate do
       expect(subject).to receive(:puts).with("Display Name: #{subject_config[:display_name]}")
       expect(subject).to receive(:puts).with("Postgres Version: #{cluster.properties.postgres_version}")
       expect(subject).to receive(:puts).with("Location: #{cluster.properties.location}")
+      expect(subject).to receive(:puts).with("Backup location: #{cluster.properties.backup_location}")
       expect(subject).to receive(:puts).with("Instances: #{cluster.properties.instances}")
       expect(subject).to receive(:puts).with("RAM Size: #{cluster.properties.ram}")
       expect(subject).to receive(:puts).with("Cores: #{cluster.properties.cores}")
@@ -72,7 +73,6 @@ describe Chef::Knife::IonoscloudDbaasPostgresClusterUpdate 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]}")

data/spec/chef/knife/ionoscloud_dbaas_postgres_logs_get_spec.rb

@@ -12,20 +12,24 @@ describe Chef::Knife::IonoscloudDbaasPostgresLogsGet do
   end
 
   describe '#run' do
-    it 'should call LogsApi.cluster_logs_get' do
+    it 'should call LogsApi.cluster_logs_get and use start and end even if since and until are also provided' do
       cluster_logs = cluster_logs_mock
       subject_config = {
         ionoscloud_username: 'email',
         ionoscloud_password: 'password',
         cluster_id: 'cluster_id',
         limit: 3,
+        start: '2022-03-22T01:01:01Z',
+        end: '2022-03-22T09:01:01Z',
+        since: '2h',
+        until: '20m',
+        direction: 'FORWARD',
         yes: true,
       }
 
       subject_config.each { |key, value| subject.config[key] = value }
 
-      cluster_logs.instances.each do
-        |instance|
+      cluster_logs.instances.each do |instance|
         expect(subject).to receive(:puts).with("Instance Name: #{instance.name}")
         expect(subject).to receive(:puts).with(instance.messages.map { |message| message.message })
       end
@@ -37,6 +41,12 @@ describe Chef::Knife::IonoscloudDbaasPostgresLogsGet do
             method: 'GET',
             path: "/clusters/#{subject_config[:cluster_id]}/logs",
             operation: :'LogsApi.cluster_logs_get',
+            options: {
+              limit: subject_config[:limit],
+              direction: subject_config[:direction],
+              start: subject_config[:start],
+              _end: subject_config[:end],
+            },
             return_type: 'ClusterLogs',
             result: cluster_logs,
           },
@@ -46,11 +56,72 @@ describe Chef::Knife::IonoscloudDbaasPostgresLogsGet do
       expect { subject.run }.not_to raise_error(Exception)
     end
 
+    it 'should call LogsApi.cluster_logs_get using since and until' do
+      cluster_logs = cluster_logs_mock
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        cluster_id: 'cluster_id',
+        limit: 3,
+        since: '2h',
+        until: '20m',
+        direction: 'FORWARD',
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      cluster_logs.instances.each do |instance|
+        expect(subject).to receive(:puts).with("Instance Name: #{instance.name}")
+        expect(subject).to receive(:puts).with(instance.messages.map { |message| message.message })
+      end
+
+      mock_dbaas_call_api(
+        subject,
+        [
+          {
+            method: 'GET',
+            path: "/clusters/#{subject_config[:cluster_id]}/logs",
+            operation: :'LogsApi.cluster_logs_get',
+            options: {
+              limit: subject_config[:limit],
+              direction: subject_config[:direction],
+              start: (Time.now - 2 * 60 * 60).iso8601,
+              _end: (Time.now - 20 * 60).iso8601,
+            },
+            return_type: 'ClusterLogs',
+            result: cluster_logs,
+          },
+        ],
+      )
+
+      expect { subject.run }.not_to raise_error(Exception)
+    end
+
+    it 'should call not LogsApi.cluster_logs_get when using something different from minutes and hours for delta' do
+      subject_config = {
+        ionoscloud_username: 'email',
+        ionoscloud_password: 'password',
+        cluster_id: 'cluster_id',
+        since: '2d',
+        yes: true,
+      }
+
+      subject_config.each { |key, value| subject.config[key] = value }
+
+      expect(subject.ui).to receive(:error).with('Time delta may only be specified in hours(h) or minutes(m)!')
+
+      expect(subject.api_client_dbaas).not_to receive(:call_api)
+
+      expect { subject.run }.to raise_error(SystemExit) do |error|
+        expect(error.status).to eq(1)
+      end
+    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]}")