foreman_kubevirt 0.1.3 → 0.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 0e0ae0ca0c55da65290264cdf02df32185314908
-  data.tar.gz: 12230c05d0f34969260ad7f3efaefb0486e094b9
+SHA256:
+  metadata.gz: 4a1ecc0522a29b6588b3f51feb09fcab28786f871ddea07b4042149a0c5a7e74
+  data.tar.gz: c7047e7a253dc251fc2efe3c3032e9d1dd0f298d0d3346a82532b9dc131b0b41
 SHA512:
-  metadata.gz: baf978d956388d8fccd84cb3e48c110b6a874542736ef6987a27c45732f9e807573ab6f87088b5a32fb46fa304c16e0a0f694965664314d8380a940dd720e4c5
-  data.tar.gz: 46c7b84303561d9a5cc2db9f72d5896775604df20263ce590f3d29f415230903e75b6ab06211478f121c717522fbea7535d9ab984bc41e82c02e57f9ff5260a9
+  metadata.gz: 02f1b764a26dc6211936b39bb6f6210622c59590f62de0eec90f5aa0017400547f798b94238847153a859dd35dd0c812f5407a20c9fce8c85278d9beb49a0b80
+  data.tar.gz: 434056ac6b789c39ea7334cc83923d30e5cec21f66908d1ddccb48a6e702c2d60774315367e895329459f12d79b15f98971bcb18d08e43bcd1d4a41b0b7d5df8

data/README.md

@@ -55,9 +55,11 @@ To verify that the installation was successful, go to Foreman, top bar **Adminis
 
 ## Compatibility
 
-| Foreman Version | Plugin Version |
-| --------------- | --------------:|
-| >= 1.21.x       | ~> 0.1.x       |
+
+
+| Foreman Version | Plugin Version |  Kubevirt API Version |
+| --------------- | --------------:|  -------------------- |
+| >= 1.21.x       | ~> 0.1.x       |  v1alpha3             |
 
 ## Usage
 Go to **Infrastructure > Compute Resources** and click on **New Compute Resource**.

data/app/assets/javascripts/bundle.js

@@ -0,0 +1 @@
+window.tfm = { kubevirt: { require('./foreman_kubevirt/kubevirt.js') };

data/app/controllers/foreman_kubevirt/concerns/api/compute_resources_controller_extensions.rb

@@ -2,25 +2,17 @@ module ForemanKubevirt
   module Concerns
     module Api
       module ComputeResourcesControllerExtensions
-        module ApiPieExtensions
-          extend ::Apipie::DSL::Concern
-          update_api(:create, :update) do
-            param :compute_resource, Hash do
-              param :token, String, :desc => N_("Token for KubeVirt only")
-              param :hostname, String, :desc => N_("Host name for KubeVirt only")
-              param :namespace, String, :desc => N_("Namespace for KubeVirt only")
-              param :ca_crt, String, :desc => N_("CA crt for KubeVirt only")
-              param :api_port, String, :desc => N_("API port for KubeVirt only")
-            end
+        extend ::Apipie::DSL::Concern
+        update_api(:create, :update) do
+          param :compute_resource, Hash do
+            param :token, String, :desc => N_("Token for KubeVirt only")
+            param :hostname, String, :desc => N_("Host name for KubeVirt only")
+            param :namespace, String, :desc => N_("Namespace for KubeVirt only")
+            param :ca_crt, String, :desc => N_("CA crt for KubeVirt only")
+            param :api_port, String, :desc => N_("API port for KubeVirt only")
           end
         end
       end
-
-      extend ActiveSupport::Concern
-
-      included do
-        include ApiPieExtensions
-      end
     end
   end
 end

data/app/models/foreman_kubevirt/compute_attribute_extensions.rb

@@ -0,0 +1,8 @@
+module ForemanKubevirt
+  module ComputeAttributeExtensions
+    extend ActiveSupport::Concern
+    included do
+      validates :vm_attrs, :volume => true
+    end
+  end
+end

data/app/models/foreman_kubevirt/kubevirt.rb

@@ -6,7 +6,7 @@ module ForemanKubevirt
     alias_attribute :token, :password
     alias_attribute :namespace, :user
     validates :hostname, :api_port, :namespace, :token, :presence => true
-    validate :test_connection
+    after_validation :validate_connectivity unless Rails.env.test?
 
     def ca_cert
       attrs[:ca_cert]
@@ -52,10 +52,29 @@ module ForemanKubevirt
       ComputeResource.model_name
     end
 
-    def test_connection(_options = {})
+    def test_connection(options = {})
+      super
+      validate_connectivity(options)
+    end
+
+    def caching_enabled
+      false
+    end
+
+    def validate_connectivity(options = {})
+      return unless connection_properties_valid?
+      return false if errors.any?
       client&.valid? && client&.virt_supported?
     rescue StandardError => e
-      errors[:base] << e.message
+      if e.message =~ /401/
+        errors[:base] << _('The compute resource could not be authenticated')
+      else
+        errors[:base] << e.message
+      end
+    end
+
+    def connection_properties_valid?
+      errors[:hostname].empty? && errors[:token].empty? && errors[:namespace].empty? && errors[:api_port].empty?
     end
 
     def networks
@@ -85,11 +104,12 @@ module ForemanKubevirt
       storage_classes.map { |sc| OpenStruct.new(id: sc.name, description: "#{sc.name} (#{sc.provisioner})") }
     end
 
-    def new_volume(attr = {})
+    def new_volume(attrs = {})
       return unless new_volume_errors.empty?
-
-      vol = Fog::Kubevirt::Compute::Volume.new(attr)
-      vol.boot_order = 1 if attr[:bootable] == "on" || attr[:bootable] == "true"
+      capacity = attrs.delete(:capacity)
+      args = {capacity: capacity}.merge(attrs)
+      vol = Fog::Kubevirt::Compute::Volume.new(args)
+      vol.boot_order = 1 if args[:bootable] == "on" || args[:bootable] == "true"
       vol
     end
 
@@ -139,12 +159,12 @@ module ForemanKubevirt
       options = vm_instance_defaults.merge(args.to_hash.deep_symbolize_keys)
       logger.debug("creating VM with the following options: #{options.inspect}")
 
-      volumes = create_volumes_for_vm(options)
-      interfaces, networks = create_network_devices_for_vm(options, volumes)
       # Add clound init user data
       user_data = { "userData" => options[:user_data] } if options[:user_data].present?
 
       begin
+        volumes = create_volumes_for_vm(options)
+        interfaces, networks = create_network_devices_for_vm(options, volumes)
         client.vms.create(:vm_name     => options[:name],
                           :cpus        => options[:cpu_cores].to_i,
                           :memory_size => convert_memory(options[:memory] + "b", :mi).to_s,
@@ -154,8 +174,8 @@ module ForemanKubevirt
                           :networks    => networks,
                           :interfaces  => interfaces)
         client.servers.get(options[:name])
-      rescue Fog::Kubevirt::Errors::ClientError => e
-        delete_pvcs(volumes)
+      rescue Exception => e
+        delete_pvcs(volumes) if volumes
         raise e
       end
     end
@@ -271,11 +291,33 @@ module ForemanKubevirt
       convert_memory(memory, :b)
     end
 
+    def plain_kubevirt_protocol
+      "plain.kubevirt.io".freeze
+    end
+
+    def token_protocol(token)
+      "base64url.bearer.authorization.k8s.io.#{token}"
+    end
+
+    def console(uuid)
+      vm = find_vm_by_uuid(uuid)
+      vnc_details = client.vminstances.get_vnc_console_details(vm.name, namespace)
+      token = Base64.encode64(vnc_details[:token]).delete!("\n").delete("==")
+      {
+        :host => vnc_details[:host],
+        :port => vnc_details[:port],
+        :path => vnc_details[:path],
+        :token_protocol => token_protocol(token),
+        :plain_protocol => plain_kubevirt_protocol,
+        :type => 'vnc',
+        :encrypt => true
+      }
+    end
+
     protected
 
     def client
       return @client if @client
-
       @client ||= Fog::Kubevirt::Compute.new(
         :kubevirt_hostname   => hostname,
         :kubevirt_port       => api_port,
@@ -283,7 +325,8 @@ module ForemanKubevirt
         :kubevirt_token      => token,
         :kubevirt_log        => logger,
         :kubevirt_verify_ssl => ca_cert.present?,
-        :kubevirt_ca_cert    => ca_cert
+        :kubevirt_ca_cert    => ca_cert,
+        :kubevirt_version    => "v1alpha3"
       )
     rescue OpenSSL::X509::CertificateError
       raise_certification_failure_exception
@@ -332,19 +375,24 @@ module ForemanKubevirt
     end
 
     def validate_volume_capacity(volumes_attributes)
-      volumes_attributes.each { |_, v| raise ::Foreman::Exception.new N_('Capacity was not found') if v[:capacity].empty? }
+      volumes_attributes.each do |_, vol|
+        if vol[:capacity].to_s.empty? || /\A\d+G?\Z/.match(vol[:capacity].to_s).nil?
+          raise Foreman::Exception.new(N_("Volume size #{vol[:capacity]} is not valid"))
+        end
+      end
     end
 
     def validate_only_single_bootable_volume(volumes_attributes)
-      raise ::Foreman::Exception.new N_('Only one volume can be bootable') if volumes_attributes.select { |_, v| v[:bootable] == "true" }.count > 1
+      raise ::Foreman::Exception.new N_('Only one volume can be bootable') if volumes_attributes.count { |_, v| v[:bootable] == "true" } > 1
     end
 
     def create_new_pvc(pvc_name, capacity, storage_class)
+      capacity = capacity + "G" unless capacity.end_with? "G"
       client.pvcs.create(:name          => pvc_name,
                          :namespace     => namespace,
                          :storage_class => storage_class,
                          :access_modes  => ['ReadWriteOnce'],
-                         :requests      => { :storage => capacity + "G" })
+                         :requests      => { :storage => capacity })
     end
 
     def delete_pvcs(volumes)
@@ -421,8 +469,8 @@ module ForemanKubevirt
     def create_network_devices_for_vm(options, volumes)
       interfaces = []
       networks = []
-
       options[:interfaces_attributes].values.each do |iface|
+        raise ::Foreman::Exception.new N_('cni_provider or network are missing') unless (iface.key?(:cni_provider) && iface.key?(:network))
         if iface[:cni_provider] == 'pod'
           nic, net = create_pod_network_element
         else

data/app/validators/volume_validator.rb

@@ -0,0 +1,13 @@
+class VolumeValidator < ActiveModel::EachValidator
+  def validate_each(record, attribute, value)
+    if value[:volumes_attributes].present?
+      value[:volumes_attributes].each do |_, attrs|
+        if attrs.key?("capacity") && attrs.key?("storage_class")
+          if attrs["capacity"].to_s.empty? || /\A\d+G?\Z/.match(attrs["capacity"].to_s).nil?
+            record.errors.add(attribute, _("Volume size #{attrs['capacity']} is not valid"))
+          end
+        end
+      end
+    end
+  end
+end

data/app/views/compute_resources/form/_kubevirt.html.erb

@@ -1,11 +1,13 @@
-<%= text_f f, :hostname, :label => _("Hostname") %>
+<%= text_f f, :hostname, :label => _("Hostname"), :required => true %>
 <%= text_f f, :api_port, :label => _("API Port"), :required => true %>
 <%= text_f f, :namespace, :label => _("Namespace"), :required => true %>
-<%= password_f f, :password, :label => _("Token"), :required => true,
+<%= password_f f, :token, :label => _("Token"), :required => true,
               :keep_value => true, :unset => unset_password?,
               :help_block => _("e.g. eyJhbGciOiJSUzI1NiIsImtpZCI6IiJ9..."),
               :help_inline => documentation_button('/foreman_kubevirt/0.x/index.html#4.1SettingComputeResourceTokenandX509CAvalues',
-                                                   root_url: 'https://theforeman.org/plugins') %>
+              :root_url => 'https://theforeman.org/plugins'),
+              :id => "compute_resource_password"
+%>
 
 <%= textarea_f f, :ca_cert, :label => _("X509 Certification Authorities"),
   :placeholder => _("Optionally provide a CA, or a correctly ordered CA chain or a path to a file. If left blank - insecure.") %>

data/app/views/compute_resources_vms/form/kubevirt/_volume.html.erb

@@ -1,6 +1,10 @@
 <% if f.object.type.nil? || f.object.type == 'persistentVolumeClaim' %>
+  <% if @set %>
+    <% @set.errors[:vm_attrs].each do | error| %>
+      <%= alert :class => 'alert-danger', :header => '', :text => error %>
+    <% end %>
+  <% end %>
   <% storage_classes = compute_resource.storage_classes %>
-
   <%= select_f f, :storage_class,
                   compute_resource.storage_classes_for_select,
                   :id,
@@ -10,7 +14,6 @@
                   :label_size => "col-md-2",
                   :disabled => !new_vm,
                   :class => "col-md-2" %>
-
   <%= text_f f,:capacity, :label => _('Size (GB)'), :label_size => "col-md-2", :disabled => !new_vm, :class => "col-md-2 pvc-size" %>
 
   <% bootable = f.object.boot_order.nil? ? false : f.object.bootable %>

data/lib/foreman_kubevirt/engine.rb

@@ -62,6 +62,8 @@ module ForemanKubevirt
         require "fog/kubevirt/compute/models/networkattachmentdef"
         require File.expand_path("../../app/models/concerns/fog_extensions/kubevirt/network", __dir__)
         Fog::Kubevirt::Compute::Networkattachmentdef.send(:include, ::FogExtensions::Kubevirt::Network)
+        ComputeAttribute.send :include, ForemanKubevirt::ComputeAttributeExtensions
+
       rescue StandardError => e
         Rails.logger.warn "Foreman-Kubevirt: skipping engine hook (#{e})"
       end

data/lib/foreman_kubevirt/version.rb

@@ -1,3 +1,3 @@
 module ForemanKubevirt
-  VERSION = '0.1.3'.freeze
+  VERSION = '0.1.9'.freeze
 end

data/locale/action_names.rb

@@ -0,0 +1,5 @@
+# Autogenerated!
+_("Action with sub plans")
+_("Import facts")
+_("Import Puppet classes")
+_("Remote action:")

data/locale/ca/foreman_kubevirt.po

@@ -0,0 +1,170 @@
+# Catalan translations for foreman_kubevirt package.
+# Copyright (C) 2020 THE PACKAGE'S COPYRIGHT HOLDER
+# This file is distributed under the same license as the foreman_kubevirt package.
+# FIRST AUTHOR <EMAIL@ADDRESS>, 2020.
+#
+msgid ""
+msgstr ""
+"Project-Id-Version: foreman_kubevirt 1.0.0\n"
+"Report-Msgid-Bugs-To: \n"
+"PO-Revision-Date: 2020-05-26 13:16+0530\n"
+"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
+"Language-Team: Catalan\n"
+"Language: ca\n"
+"MIME-Version: 1.0\n"
+"Content-Type: text/plain; charset=UTF-8\n"
+"Content-Transfer-Encoding: 8bit\n"
+"Plural-Forms: nplurals=2; plural=n != 1;\n"
+"\n"
+
+msgid "%{cpu_cores} Cores and %{memory} memory"
+msgstr ""
+
+msgid "API Port"
+msgstr ""
+
+msgid "API port for KubeVirt only"
+msgstr ""
+
+msgid "Actions"
+msgstr ""
+
+msgid "Bootable"
+msgstr ""
+
+msgid "CA crt for KubeVirt only"
+msgstr ""
+
+msgid "CNI Provider"
+msgstr ""
+
+msgid "CPUs"
+msgstr ""
+
+msgid "Disk"
+msgstr ""
+
+msgid "Does this image support user data input (e.g. via cloud-init)?"
+msgstr ""
+
+msgid "Host name for KubeVirt only"
+msgstr ""
+
+msgid "Hostname"
+msgstr ""
+
+msgid "Image"
+msgstr ""
+
+msgid "It is not possible to set a bootable volume and image based provisioning."
+msgstr ""
+
+msgid "Memory"
+msgstr ""
+
+msgid "NIC"
+msgstr ""
+
+msgid "Name"
+msgstr ""
+
+msgid "Namespace"
+msgstr ""
+
+msgid "Namespace for KubeVirt only"
+msgstr ""
+
+msgid "Network"
+msgstr ""
+
+msgid "Only one volume can be bootable"
+msgstr ""
+
+msgid "Optionally provide a CA, or a correctly ordered CA chain or a path to a file. If left blank - insecure."
+msgstr ""
+
+msgid "Password to authenticate with - used for SSH finish step."
+msgstr ""
+
+msgid "Please select an image"
+msgstr ""
+
+msgid "Power"
+msgstr ""
+
+msgid "Power ON this machine"
+msgstr ""
+
+msgid "Properties"
+msgstr ""
+
+msgid "Provision and manage Kubevirt Virtual Machines from Foreman."
+msgstr ""
+
+msgid "Running on"
+msgstr ""
+
+msgid "Size (GB)"
+msgstr ""
+
+msgid "Start"
+msgstr ""
+
+msgid "Storage Class"
+msgstr ""
+
+msgid "The compute resource could not be authenticated"
+msgstr ""
+
+msgid "The name of the image in the registry."
+msgstr ""
+
+msgid ""
+"The remote system presented a public key signed by an unidentified certificate authority.\n"
+"           If you are sure the remote system is authentic, go to the compute resource edit page, press the 'Test Connection' button and submit"
+msgstr ""
+
+msgid "The user that is used to ssh into the instance, normally cloud-user, ec2-user, ubuntu, root etc"
+msgstr ""
+
+msgid "Token"
+msgstr ""
+
+msgid "Token for KubeVirt only"
+msgstr ""
+
+msgid "Type"
+msgstr ""
+
+msgid "UUID"
+msgstr ""
+
+msgid "VCPU(s)"
+msgstr ""
+
+msgid "VM should be created based on Persistent Volume Claim or Image"
+msgstr ""
+
+msgid "VM should be created based on an image"
+msgstr ""
+
+msgid "X509 Certification Authorities"
+msgstr ""
+
+msgid "cni_provider or network are missing"
+msgstr ""
+
+msgid "e.g. eyJhbGciOiJSUzI1NiIsImtpZCI6IiJ9..."
+msgstr ""
+
+msgid "genie"
+msgstr ""
+
+msgid "multus"
+msgstr ""
+
+msgid "no Storage Classes available on provider"
+msgstr ""
+
+msgid "pod"
+msgstr ""