knife-ionoscloud 6.1.0 → 6.2.0

data/lib/chef/knife/ionoscloud_dbaas_postgres_cluster_delete.rb

@@ -12,7 +12,6 @@ class Chef
         @description =
         'Delete a PostgreSQL cluster.'
         @directory = 'dbaas-postgres'
-        @required_options = [:ionoscloud_username, :ionoscloud_password]
       end
 
       def run
@@ -36,11 +35,11 @@ class Chef
 
           begin
             confirm('Do you really want to delete this cluster')
-          rescue SystemExit => exc
+          rescue SystemExit
             next
           end
 
-          _, _, headers = clusters_api.clusters_delete_with_http_info(cluster_id)
+          clusters_api.clusters_delete_with_http_info(cluster_id)
           ui.warn("Deleted Cluster #{cluster.id}.")
         end
       end

data/lib/chef/knife/ionoscloud_dbaas_postgres_cluster_get.rb

@@ -15,11 +15,11 @@ class Chef
       def initialize(args = [])
         super(args)
         @description =
-        'You can retrieve a PostgreSQL cluster by using its ID. This value can be
-        found in the response body when a PostgreSQL cluster is created or when
-        you GET a list of PostgreSQL clusters.'
+        'You can retrieve a PostgreSQL cluster by using its ID. This value can be '\
+        'found in the response body when a PostgreSQL cluster is created or when '\
+        'you GET a list of PostgreSQL clusters.'
         @directory = 'dbaas-postgres'
-        @required_options = [:cluster_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:cluster_id]
       end
 
       def run

data/lib/chef/knife/ionoscloud_dbaas_postgres_cluster_list.rb

@@ -12,7 +12,6 @@ class Chef
         @description =
         'Retrieves a list of PostgreSQL clusters.'
         @directory = 'dbaas-postgres'
-        @required_options = [:ionoscloud_username, :ionoscloud_password]
       end
 
       def run
@@ -25,6 +24,7 @@ class Chef
           ui.color('Display Name', :bold),
           ui.color('Postgres Version', :bold),
           ui.color('Location', :bold),
+          ui.color('Backup location', :bold),
           ui.color('Instances', :bold),
           ui.color('Datacenter ID', :bold),
           ui.color('Lan ID', :bold),
@@ -39,6 +39,7 @@ class Chef
           dbaas_cluster_list << cluster.properties.display_name
           dbaas_cluster_list << cluster.properties.postgres_version
           dbaas_cluster_list << cluster.properties.location
+          dbaas_cluster_list << cluster.properties.backup_location
           dbaas_cluster_list << cluster.properties.instances
           dbaas_cluster_list << cluster.properties.connections.first.datacenter_id
           dbaas_cluster_list << cluster.properties.connections.first.lan_id
@@ -46,7 +47,7 @@ class Chef
           dbaas_cluster_list << cluster.metadata.state
         end
 
-        puts ui.list(dbaas_cluster_list, :uneven_columns_across, 9)
+        puts ui.list(dbaas_cluster_list, :uneven_columns_across, 10)
       end
     end
   end

data/lib/chef/knife/ionoscloud_dbaas_postgres_cluster_restore.rb

@@ -27,7 +27,7 @@ class Chef
         @description =
         'Triggers an in-place restore of the given PostgreSQL.'
         @directory = 'dbaas-postgres'
-        @required_options = [:cluster_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:cluster_id]
       end
 
       def run

data/lib/chef/knife/ionoscloud_dbaas_postgres_cluster_update.rb

@@ -59,7 +59,7 @@ class Chef
         @description =
         'Updates information about a Ionoscloud Dbaas Cluster.'
         @directory = 'dbaas-postgres'
-        @required_options = [:cluster_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:cluster_id]
         @updatable_fields = [:cores, :ram, :storage_size, :display_name, :time, :day_of_the_week, :postgres_version, :instances]
       end
 
@@ -78,7 +78,7 @@ class Chef
             ram: (config[:ram].nil? ? nil : Integer(config[:ram])),
             storage_size: (config[:storage_size].nil? ? nil : Integer(config[:storage_size])),
             display_name: config[:display_name],
-            maintenance_window: (config[:time] && config[:day_of_the_week]) ? IonoscloudDbaas::MaintenanceWindow.new(
+            maintenance_window: (config[:time] && config[:day_of_the_week]) ? IonoscloudDbaasPostgres::MaintenanceWindow.new(
               time: config[:time],
               day_of_the_week: config[:day_of_the_week],
             ) : nil,
@@ -88,9 +88,7 @@ class Chef
           cluster_request = IonoscloudDbaasPostgres::PatchClusterRequest.new()
           cluster_request.properties = cluster_properties
 
-          cluster, _, headers  = clusters_api.clusters_patch_with_http_info(config[:cluster_id], cluster_request)
-
-          dot = ui.color('.', :magenta)
+          clusters_api.clusters_patch_with_http_info(config[:cluster_id], cluster_request)
         else
           ui.warn("Nothing to update, please set one of the attributes #{@updatable_fields}.")
         end

data/lib/chef/knife/ionoscloud_dbaas_postgres_logs_get.rb

@@ -19,18 +19,33 @@ class Chef
 
       option :start,
               long: '--start START',
-              description: 'The start time for the query in RFC3339 format.'
+              description: 'The start time for the query in RFC3339 format. If both start and since are set, start will be used.'
+
+      option :since,
+              long: '--since SINCE',
+              description: 'The start time for the query using a time delta since the current moment: 2h - 2 hours ago, 20m - 20 minutes ago. '\
+              'Only hours and minutes ar supported, and not at the same time. If both start and since are set, start will be used.'
 
       option :end,
               long: '--end END',
-              description: 'The end time for the query in RFC3339 format.'
+              description: 'The end time for the query in RFC3339 format. If both end and until are set, end will be used.'
+
+      option :until,
+              long: '--until UNTIL',
+              description: 'The end time for the query using a time delta since the current moment: 2h - 2 hours ago, 20m - 20 minutes ago. '\
+              'Only hours and minutes ar supported, and not at the same time. If both end and until are set, end will be used.'
+
+      option :direction,
+              long: '--direction DIRECTION',
+              description: 'The direction in which to scan through the logs. '\
+                           'The logs are returned in order of the direction. One of ["BACKWARD", "FORWARD"]'
 
       def initialize(args = [])
         super(args)
         @description =
         'Retrieves PostgreSQL logs based on the given parameters.'
         @directory = 'dbaas-postgres'
-        @required_options = [:cluster_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:cluster_id]
       end
 
       def run
@@ -38,21 +53,37 @@ class Chef
         handle_extra_config
         validate_required_params(@required_options, config)
 
+        def delta_to_date(delta)
+          return nil if delta.nil?
+
+          unit = delta[-1]
+          unless ['h', 'm'].include? unit
+            ui.error('Time delta may only be specified in hours(h) or minutes(m)!')
+            exit(1)
+          end
+
+          minute_count = Integer(delta[0..-2])
+          minute_count *= 60 if unit == 'h'
+
+          (Time.now - minute_count * 60).iso8601
+        end
+
         logs = IonoscloudDbaasPostgres::LogsApi.new(api_client_dbaas).cluster_logs_get(
           config[:cluster_id],
           {
             limit: (config[:limit] != nil ? Integer(config[:limit]) : nil),
-            start: config[:start],
-            end: config[:end],
+            start: config[:start].nil? ? delta_to_date(config[:since]) : config[:start],
+            _end: config[:end].nil? ? delta_to_date(config[:until]) : config[:end],
+            direction: config[:direction],
           },
         )
 
-        logs.instances.each { |instance|
+        logs.instances.each do |instance|
           puts "Instance Name: #{instance.name}"
           print "\n"
           puts instance.messages.map { |message| message.message }
           puts "\n"
-        }
+        end
       end
     end
   end

data/lib/chef/knife/ionoscloud_dbaas_postgres_version_list.rb

@@ -15,10 +15,10 @@ class Chef
       def initialize(args = [])
         super(args)
         @description =
-        'If cluster_id is provided, retrieves a list of all PostgreSQL versions available for a cluster including the current version,
-        otherwise retrieves a list of all available PostgreSQL versions.'
+        'If cluster_id is provided, retrieves a list of all PostgreSQL versions '\
+        'available for a cluster including the current version, '\
+        'otherwise retrieves a list of all available PostgreSQL versions.'
         @directory = 'dbaas-postgres'
-        @required_options = [:ionoscloud_username, :ionoscloud_password]
       end
 
       def run

data/lib/chef/knife/ionoscloud_firewall_create.rb

@@ -79,7 +79,7 @@ class Chef
         @description =
         'Creates a new firewall rule on an existing NIC.'
         @directory = 'compute-engine'
-        @required_options = [:datacenter_id, :server_id, :nic_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:datacenter_id, :server_id, :nic_id]
       end
 
       def run
@@ -99,7 +99,7 @@ class Chef
           port_range_end: config[:port_range_end],
           icmp_type: config[:icmp_type],
           icmp_code: config[:icmp_code],
-          type: config[:type]
+          type: config[:type],
         }
 
         firewallrules_api = Ionoscloud::FirewallRulesApi.new(api_client)

data/lib/chef/knife/ionoscloud_firewall_delete.rb

@@ -27,7 +27,7 @@ class Chef
         @description =
         'Deletes a firewall rule from an existing NIC.'
         @directory = 'compute-engine'
-        @required_options = [:datacenter_id, :server_id, :nic_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:datacenter_id, :server_id, :nic_id]
       end
 
       def run
@@ -53,7 +53,7 @@ class Chef
 
           begin
             confirm('Do you really want to delete this firewall rule')
-          rescue SystemExit => exc
+          rescue SystemExit
             next
           end
 

data/lib/chef/knife/ionoscloud_firewall_get.rb

@@ -32,7 +32,7 @@ class Chef
         @description =
         'Retrieves information about a Ionoscloud Firewall Rule.'
         @directory = 'compute-engine'
-        @required_options = [:firewall_id, :datacenter_id, :server_id, :nic_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:firewall_id, :datacenter_id, :server_id, :nic_id]
       end
 
       def run

data/lib/chef/knife/ionoscloud_firewall_list.rb

@@ -27,7 +27,7 @@ class Chef
         @description =
         'Lists all available firewall rules assigned to a NIC.'
         @directory = 'compute-engine'
-        @required_options = [:datacenter_id, :server_id, :nic_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:datacenter_id, :server_id, :nic_id]
       end
 
       def run
@@ -52,7 +52,7 @@ class Chef
         firewallrules_api = Ionoscloud::FirewallRulesApi.new(api_client)
 
         firewallrules_api.datacenters_servers_nics_firewallrules_get(
-          config[:datacenter_id], config[:server_id], config[:nic_id], { depth: 1 }
+          config[:datacenter_id], config[:server_id], config[:nic_id], { depth: 1 },
         ).items.each do |firewall|
           firewall_list << firewall.id
           firewall_list << firewall.properties.name

data/lib/chef/knife/ionoscloud_firewall_update.rb

@@ -80,7 +80,7 @@ class Chef
         @description =
         'Updates information about a Ionoscloud Firewall Rule.'
         @directory = 'compute-engine'
-        @required_options = [:datacenter_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:datacenter_id]
         @updatable_fields = [
           :name, :protocol, :source_mac, :source_ip, :target_ip,
           :port_range_start, :port_range_end, :icmp_type, :icmp_code,

data/lib/chef/knife/ionoscloud_flowlog_create.rb

@@ -61,7 +61,7 @@ class Chef
         @description =
         'This will add a Flow Log to the network interface, NAT Gateway or Network Load Balancer.'
         @directory = 'compute-engine'
-        @required_options = [:datacenter_id, :type, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:datacenter_id, :type]
       end
 
       def run

data/lib/chef/knife/ionoscloud_flowlog_delete.rb

@@ -42,7 +42,7 @@ class Chef
         @description =
         'Removes the specified Flow Logs.'
         @directory = 'compute-engine'
-        @required_options = [:datacenter_id, :type, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:datacenter_id, :type]
       end
 
       def run
@@ -50,8 +50,6 @@ class Chef
         handle_extra_config
         validate_required_params(@required_options, config)
 
-        flowlogs_api = Ionoscloud::FlowLogsApi.new(api_client)
-
         case config[:type]
         when 'nic'
           validate_required_params([:server_id, :nic_id], config)
@@ -89,7 +87,7 @@ class Chef
 
           begin
             confirm('Do you really want to delete this Flow Log')
-          rescue SystemExit => exc
+          rescue SystemExit
             next
           end
 

data/lib/chef/knife/ionoscloud_flowlog_get.rb

@@ -47,7 +47,7 @@ class Chef
         @description =
         'Retrieves information about a Ionoscloud Flow Log.'
         @directory = 'compute-engine'
-        @required_options = [:flowlog_id, :datacenter_id, :type, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:flowlog_id, :datacenter_id, :type]
       end
 
       def run

data/lib/chef/knife/ionoscloud_flowlog_list.rb

@@ -42,7 +42,7 @@ class Chef
         @description =
         'Lists all available flow logs assigned to a NIC, NAT Gateway or Network Load Balancer.'
         @directory = 'compute-engine'
-        @required_options = [:datacenter_id, :type, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:datacenter_id, :type]
       end
 
       def run
@@ -64,27 +64,25 @@ class Chef
           validate_required_params([:server_id, :nic_id], config)
           flowlogs_api = Ionoscloud::FlowLogsApi.new(api_client)
           flowlogs = flowlogs_api.datacenters_servers_nics_flowlogs_get(
-            config[:datacenter_id], config[:server_id], config[:nic_id], { depth: 1 }
+            config[:datacenter_id], config[:server_id], config[:nic_id], { depth: 1 },
           )
         when 'natgateway'
           validate_required_params([:natgateway_id], config)
           flowlogs_api = Ionoscloud::NATGatewaysApi.new(api_client)
           flowlogs = flowlogs_api.datacenters_natgateways_flowlogs_get(
-            config[:datacenter_id], config[:natgateway_id], { depth: 1 }
+            config[:datacenter_id], config[:natgateway_id], { depth: 1 },
           )
         when 'networkloadbalancer'
           validate_required_params([:network_loadbalancer_id], config)
           flowlogs_api = Ionoscloud::NetworkLoadBalancersApi.new(api_client)
           flowlogs = flowlogs_api.datacenters_networkloadbalancers_flowlogs_get(
-            config[:datacenter_id], config[:network_loadbalancer_id], { depth: 1 }
+            config[:datacenter_id], config[:network_loadbalancer_id], { depth: 1 },
           )
         else
           ui.error "Flow log cannot belong to #{config[:type]}. Value must be one of ['nic', 'natgateway', 'networkloadbalancer']"
           exit(1)
         end
 
-        flowlogs_api = Ionoscloud::FlowLogsApi.new(api_client)
-
         flowlogs.items.each do |flowlog|
           flowlog_list << flowlog.id
           flowlog_list << flowlog.properties.name

data/lib/chef/knife/ionoscloud_flowlog_update.rb

@@ -66,7 +66,7 @@ class Chef
         @description =
         'Updates information about a Ionoscloud Flow Log.'
         @directory = 'compute-engine'
-        @required_options = [:flowlog_id, :datacenter_id, :type, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:flowlog_id, :datacenter_id, :type]
         @updatable_fields = [:name, :action, :direction, :bucket]
       end
 

data/lib/chef/knife/ionoscloud_group_create.rb

@@ -71,7 +71,7 @@ class Chef
         @description =
         'Use this operation to create a new group and set group privileges.'
         @directory = 'user'
-        @required_options = [:name, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:name]
       end
 
       def run

data/lib/chef/knife/ionoscloud_group_delete.rb

@@ -14,7 +14,6 @@ class Chef
         'assigned to the group are NOT deleted, but are no longer accessible '\
         'to the group members unless the member is a Contract Owner, Admin, or Resource Owner.'
         @directory = 'user'
-        @required_options = [:ionoscloud_username, :ionoscloud_password]
       end
 
       def run
@@ -38,7 +37,7 @@ class Chef
 
           begin
             confirm('Do you really want to delete this Group')
-          rescue SystemExit => exc
+          rescue SystemExit
             next
           end
 

data/lib/chef/knife/ionoscloud_group_get.rb

@@ -18,7 +18,7 @@ class Chef
         'Retrieves detailed information about a specific group. This will also '\
         'retrieve a list of users who are members of the group.'
         @directory = 'user'
-        @required_options = [:group_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:group_id]
       end
 
       def run

data/lib/chef/knife/ionoscloud_group_list.rb

@@ -17,7 +17,6 @@ class Chef
         @description =
         'This retrieves a full list of all groups.'
         @directory = 'user'
-        @required_options = [:ionoscloud_username, :ionoscloud_password]
       end
 
       def run

data/lib/chef/knife/ionoscloud_group_update.rb

@@ -76,7 +76,7 @@ class Chef
         @description =
         'Updates information about a Ionoscloud Group.'
         @directory = 'user'
-        @required_options = [:group_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:group_id]
         @updatable_fields = [
           :name, :create_data_center, :create_snapshot, :reserve_ip, :access_activity_log, :s3_privilege,
           :create_backup_unit, :create_k8s_cluster, :create_pcc, :create_internet_access,
@@ -96,7 +96,7 @@ class Chef
 
           existing_group = user_management_api.um_groups_find_by_id(config[:group_id], depth: 1)
 
-          group, _, headers = user_management_api.um_groups_put_with_http_info(
+          _, _, headers = user_management_api.um_groups_put_with_http_info(
             config[:group_id],
             Ionoscloud::Group.new(
               properties: Ionoscloud::GroupProperties.new(

data/lib/chef/knife/ionoscloud_group_user_add.rb

@@ -17,7 +17,7 @@ class Chef
         @description =
         'Use this operation to add an existing user to a group.'
         @directory = 'user'
-        @required_options = [:group_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:group_id]
       end
 
       def run

data/lib/chef/knife/ionoscloud_group_user_remove.rb

@@ -17,7 +17,7 @@ class Chef
         @description =
         'Use this operation to remove a user from a group.'
         @directory = 'user'
-        @required_options = [:group_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:group_id]
       end
 
       def run

data/lib/chef/knife/ionoscloud_image_list.rb

@@ -13,7 +13,6 @@ class Chef
         'A list of disk and ISO images are available from Ionoscloud for immediate use. '\
         'Make sure the image you use is in the same location as the virtual data center.'
         @directory = 'compute-engine'
-        @required_options = [:ionoscloud_username, :ionoscloud_password]
       end
 
       def run

data/lib/chef/knife/ionoscloud_ipblock_create.rb

@@ -30,7 +30,7 @@ class Chef
         'one or more static IP addresses. The IP block size \(number of IP addresses\) '\
         'and location are required to reserve an IP block.'
         @directory = 'compute-engine'
-        @required_options = [:size, :location, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:size, :location]
       end
 
       def run

data/lib/chef/knife/ionoscloud_ipblock_delete.rb

@@ -12,7 +12,6 @@ class Chef
         @description =
         'Releases a currently assigned IP block.'
         @directory = 'compute-engine'
-        @required_options = [:ionoscloud_username, :ionoscloud_password]
       end
 
       def run
@@ -34,7 +33,7 @@ class Chef
 
           begin
             confirm('Do you really want to delete this IP block')
-          rescue SystemExit => exc
+          rescue SystemExit
             next
           end
 

data/lib/chef/knife/ionoscloud_ipblock_get.rb

@@ -17,7 +17,7 @@ class Chef
         @description =
         'Retrieves information about an IP Block.'
         @directory = 'compute-engine'
-        @required_options = [:ipblock_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:ipblock_id]
       end
 
       def run

data/lib/chef/knife/ionoscloud_ipblock_list.rb

@@ -12,7 +12,6 @@ class Chef
         @description =
         'Lists all available IP blocks.'
         @directory = 'compute-engine'
-        @required_options = [:ionoscloud_username, :ionoscloud_password]
       end
 
       def run

data/lib/chef/knife/ionoscloud_ipblock_update.rb

@@ -22,7 +22,7 @@ class Chef
         @description =
         'Updates information about an IP Block.'
         @directory = 'compute-engine'
-        @required_options = [:ipblock_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:ipblock_id]
         @updatable_fields = [:name]
       end
 

data/lib/chef/knife/ionoscloud_ipfailover_add.rb

@@ -35,7 +35,7 @@ class Chef
         "* Use PATCH or PUT to enable ipFailover by providing the relevant ip and nicUuid values.\n"\
         "* Add the same reserved IP address to any other NICs that are a member of the same LAN. Those NICs will become IP Failover members.\n"
         @directory = 'compute-engine'
-        @required_options = [:datacenter_id, :lan_id, :ip, :nic_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:datacenter_id, :lan_id, :ip, :nic_id]
       end
 
       def run

data/lib/chef/knife/ionoscloud_ipfailover_remove.rb

@@ -32,7 +32,7 @@ class Chef
         @description =
         'Remove IP Failover from LAN'
         @directory = 'compute-engine'
-        @required_options = [:datacenter_id, :lan_id, :ip, :nic_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:datacenter_id, :lan_id, :ip, :nic_id]
       end
 
       def run

data/lib/chef/knife/ionoscloud_k8s_create.rb

@@ -42,7 +42,7 @@ class Chef
         @description =
         'Creates a new Managed Kubernetes cluster.'
         @directory = 'kubernetes'
-        @required_options = [:name, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:name]
       end
 
       def run

data/lib/chef/knife/ionoscloud_k8s_delete.rb

@@ -12,7 +12,6 @@ class Chef
         @description =
         'Deletes a Kubernetes cluster. The cluster cannot contain any node pools when deleting.'
         @directory = 'kubernetes'
-        @required_options = [:ionoscloud_username, :ionoscloud_password]
       end
 
       def run
@@ -34,7 +33,7 @@ class Chef
           unless ['ACTIVE', 'TERMINATED'].include? cluster.metadata.state
             ui.error(
               "K8s Cluster #{cluster_id} state must be one of ['ACTIVE', 'TERMINATED'], "\
-              "actual state is '#{cluster.metadata.state}'. Skipping."
+              "actual state is '#{cluster.metadata.state}'. Skipping.",
             )
             next
           end
@@ -49,7 +48,7 @@ class Chef
 
           begin
             confirm('Do you really want to delete this K8s Cluster')
-          rescue SystemExit => exc
+          rescue SystemExit
             next
           end
 

data/lib/chef/knife/ionoscloud_k8s_get.rb

@@ -17,7 +17,7 @@ class Chef
         @description =
         'Retrieves information about a K8s Cluster.'
         @directory = 'kubernetes'
-        @required_options = [:cluster_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:cluster_id]
       end
 
       def run

data/lib/chef/knife/ionoscloud_k8s_list.rb

@@ -12,7 +12,6 @@ class Chef
         @description =
         'Retrieve a list of Kubernetes clusters.'
         @directory = 'kubernetes'
-        @required_options = [:ionoscloud_username, :ionoscloud_password]
       end
 
       def run

data/lib/chef/knife/ionoscloud_k8s_update.rb

@@ -46,7 +46,7 @@ class Chef
         @description =
         'Updates information about a Ionoscloud K8s Cluster.'
         @directory = 'kubernetes'
-        @required_options = [:cluster_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:cluster_id]
         @updatable_fields = [
           :name, :version, :maintenance_day, :maintenance_time, :api_subnet_allow_list, :s3_buckets,
         ]
@@ -62,14 +62,10 @@ class Chef
         if @updatable_fields.map { |el| config[el] }.any?
           print "#{ui.color('Updating K8s Cluster...', :magenta)}"
 
-          if config[:api_subnet_allow_list] && config[:api_subnet_allow_list].instance_of?(String)
-            config[:api_subnet_allow_list] = config[:api_subnet_allow_list].split(',')
-          end
+          config[:api_subnet_allow_list] = config[:api_subnet_allow_list].split(',') if config[:api_subnet_allow_list] && config[:api_subnet_allow_list].instance_of?(String)
           config[:s3_buckets] = config[:s3_buckets].split(',') if config[:s3_buckets] && config[:s3_buckets].instance_of?(String)
 
-          if config.key?(:s3_buckets)
-            config[:s3_buckets] = config[:s3_buckets].map { |el| Ionoscloud::S3Bucket.new(name: el) }
-          end
+          config[:s3_buckets] = config[:s3_buckets].map { |el| Ionoscloud::S3Bucket.new(name: el) } if config.key?(:s3_buckets)
 
           cluster = kubernetes_api.k8s_find_by_cluster_id(config[:cluster_id])
 

data/lib/chef/knife/ionoscloud_kubeconfig_get.rb

@@ -17,7 +17,7 @@ class Chef
         @description =
         'Retrieve the kubeconfig file for a given Kubernetes cluster.'
         @directory = 'kubernetes'
-        @required_options = [:cluster_id, :ionoscloud_username, :ionoscloud_password]
+        @required_options = [:cluster_id]
       end
 
       def run