opennebula 6.0.2 → 6.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c860bdde4c2e7184d75f0433181a5aa485d39296
-  data.tar.gz: 7b773535ac63930fb85252435814b4f4e81adcf0
+  metadata.gz: ae4c2d6ffef598e821e3b6bcc4a429f766caadbf
+  data.tar.gz: 2e65f273fb1f2d383d799fa4fd31e6d2fb66cb41
 SHA512:
-  metadata.gz: 0b23945ca645beba28c3e898a13ef254b4a78ecdc63dec03922c808914233d550a3503efef499a521d2aa6fbfaeff795789976105ccb93eb80614ef3686662e1
-  data.tar.gz: 3337709c1791f07680d40dcb8c4a012139e93af38546fc200dacb07fea81b3890405a80f48a596f72465c4bb08e28f02b6c729801e8c414ba55559d1b2f7a097
+  metadata.gz: ce66ed43b8db71bcb0f8572a7e3a48de18b3e7c4819006d93333709e892718cd83c724a723c81870538b404ed3d80bc33786b1b1cc0bbd8a882412992c56bd17
+  data.tar.gz: e3694d73c467089a0ec6720c2ac22034bfcb49f273d14b607915d9aa1ba23abf6c5045a436ec08f47d7f87e5c394981c0e95d8b1614833f87c3666f64b731291

data/lib/CommandManager.rb

@@ -34,17 +34,9 @@ require 'base64'
 # * Log messages will be sent to STDOUT
 # * The script will return 0 if it succeded or any other value
 #   if there was a failure
-# * In case of failure the cause of the error will be written to STDERR
-#   wrapped by start and end marks as follows:
-#
-#     ERROR MESSAGE --8<------
-#     error message for the failure
-#     ERROR MESSAGE ------>8--
-
+# * In case of failure the cause of the error will be written to STDERR.
 
 class GenericCommand
-    ERROR_OPEN  = "ERROR MESSAGE --8<------"
-    ERROR_CLOSE = "ERROR MESSAGE ------>8--"
 
     attr_reader :code, :stdout, :stderr, :command
 
@@ -93,8 +85,9 @@ class GenericCommand
             end
 
             log(error_message)
-            @stderr = ERROR_OPEN + "\n" + error_message + "\n" + ERROR_CLOSE
-            @code = 255
+
+            @stderr = error_message
+            @code   = 255
         end
 
         return @code
@@ -102,9 +95,9 @@ class GenericCommand
 
     # Parses error message from +stderr+ output
     def get_error_message
-        tmp=@stderr.scan(/^#{ERROR_OPEN}\n(.*?)#{ERROR_CLOSE}$/m)
-        return "-" if !tmp[0]
-        tmp[0].join(' ').strip
+        return '-' if @stderr.empty?
+
+        @stderr.tr("\n",' ').strip
     end
 
     def to_xml
@@ -176,8 +169,12 @@ private
                 end
             }
 
-            i.write stdin_data
-            i.close
+            begin
+                i.write stdin_data
+                i.close
+            rescue Errno::EPIPE
+                # the cmd doesn't read the input, ignore error
+            end
 
             # blocking wait for process termination
             t.value

data/lib/DriverExecHelper.rb

@@ -124,31 +124,21 @@ module DriverExecHelper
     # Sends a log message to ONE. The +message+ can be multiline, it will
     # be automatically splitted by lines.
     def log(number, message, all = true)
-        in_error_message=false
-        msg=message.strip
+        msg = message.strip
+
         msg.each_line do |line|
             all ? severity='I' : severity=nil
-            l=line.strip
-
-            if l=='ERROR MESSAGE --8<------'
-                in_error_message=true
-                next
-            elsif l=='ERROR MESSAGE ------>8--'
-                in_error_message=false
-                next
-            else
-                if in_error_message
+
+            if line.match(/^(ERROR|DEBUG|INFO):(.*)$/)
+                line=Regexp.last_match(2)
+
+                case Regexp.last_match(1)
+                when 'ERROR'
                     severity='E'
-                elsif line.match(/^(ERROR|DEBUG|INFO):(.*)$/)
-                    line=Regexp.last_match(2)
-                    case Regexp.last_match(1)
-                    when 'ERROR'
-                        severity='E'
-                    when 'DEBUG'
-                        severity='D'
-                    when 'INFO'
-                        severity='I'
-                    end
+                when 'DEBUG'
+                    severity='D'
+                when 'INFO'
+                    severity='I'
                 end
             end
 

data/lib/cloud/CloudClient.rb

@@ -51,7 +51,7 @@ end
 module CloudClient
 
     # OpenNebula version
-    VERSION = '6.0.2'
+    VERSION = '6.0.3'
 
     # #########################################################################
     # Default location for the authentication file

data/lib/file_helper.rb

@@ -81,6 +81,10 @@ module VCenterDriver
             file.match(%r{^https?://}) || file.match(%r{^s3?://})
         end
 
+        def self.from_s3?(file)
+            file.match(%r{^s3?://})
+        end
+
         def self.vmdk?(file)
             type = `file #{file}`
 

data/lib/host.rb

@@ -1145,14 +1145,14 @@ module VCenterDriver
             vswitchspec =
                 RbVmomi::VIM::HostVirtualSwitchSpec(
                     :bridge => hostbridge,
-                     :mtu => mtu,
-                      :numPorts => num_ports
+                    :mtu => mtu,
+                    :numPorts => num_ports
                 )
             begin
                 nws
                     .UpdateVirtualSwitch(
                         :vswitchName => name,
-                         :spec => vswitchspec
+                        :spec => vswitchspec
                     )
             rescue StandardError => e
                 raise "The standard switch with name #{name} \
@@ -1463,7 +1463,7 @@ module VCenterDriver
                         nws
                             .UpdatePortGroup(
                                 :pgName => nr[:name],
-                                 :portgrp => nr[:spec]
+                                :portgrp => nr[:spec]
                             )
                     rescue StandardError => e
                         raise "A rollback operation for standard \

data/lib/models/service.rb

@@ -140,12 +140,14 @@ module OpenNebula
         # Return true if the service can be undeployed
         # @return true if the service can be undeployed, false otherwise
         def can_undeploy?
+            # rubocop:disable Style/IfWithBooleanLiteralBranches
             if (transient_state? && state != Service::STATE['UNDEPLOYING']) ||
-               state == Service::STATE['DONE'] || failed_state?
+                state == Service::STATE['DONE'] || failed_state?
                 false
             else
                 true
             end
+            # rubocop:enable Style/IfWithBooleanLiteralBranches
         end
 
         # Return true if the service can be updated
@@ -166,6 +168,12 @@ module OpenNebula
             RECOVER_SCALE_STATES.include? STATE_STR[state]
         end
 
+        # Return true if the service is running
+        # @return true if the service is runnning, false otherwise
+        def running?
+            state_str == 'RUNNING'
+        end
+
         # Returns the running_status_vm option
         # @return [true, false] true if the running_status_vm option is enabled
         def report_ready?
@@ -323,6 +331,36 @@ module OpenNebula
             nil
         end
 
+        # Adds a role to the service
+        #
+        # @param template [Hash] Role information
+        #
+        # @return [OpenNebula::Role] New role
+        def add_role(template)
+            template['state'] ||= Role::STATE['PENDING']
+            role                = Role.new(template, self)
+
+            if @roles[role.name]
+                return OpenNebula::Error.new("Role #{role.name} already exists")
+            end
+
+            @roles[role.name] = role
+            @body['roles'] << template if @body && @body['roles']
+
+            role
+        end
+
+        # Removes a role from the service
+        #
+        # @param name [String] Role name to delete
+        def remove_role(name)
+            @roles.delete(name)
+
+            @body['roles'].delete_if do |role|
+                role['name'] == name
+            end
+        end
+
         # Retrieves the information of the Service and all its Nodes.
         #
         # @return [nil, OpenNebula::Error] nil in case of success, Error
@@ -514,8 +552,12 @@ module OpenNebula
             [true, nil]
         end
 
-        def deploy_networks
-            body = JSON.parse(self['TEMPLATE/BODY'])
+        def deploy_networks(deploy = true)
+            if deploy
+                body = JSON.parse(self['TEMPLATE/BODY'])
+            else
+                body = @body
+            end
 
             return if body['networks_values'].nil?
 
@@ -531,7 +573,7 @@ module OpenNebula
                 if OpenNebula.is_error?(rc)
                     return rc
                 end
-            end
+            end if deploy
 
             # Replace $attibute by the corresponding value
             resolve_attributes(body)

data/lib/opennebula.rb

@@ -77,5 +77,5 @@ require 'opennebula/flow'
 module OpenNebula
 
     # OpenNebula version
-    VERSION = '6.0.2'
+    VERSION = '6.0.3'
 end

data/lib/opennebula/flow/service_template.rb

@@ -451,6 +451,16 @@ module OpenNebula
             validate_values(template)
         end
 
+        def self.validate_role(template)
+            validator = Validator::Validator.new(
+                :default_values => true,
+                :delete_extra_properties => false,
+                :allow_extra_properties => true
+            )
+
+            validator.validate!(template, ROLE_SCHEMA)
+        end
+
         def instantiate(merge_template)
             rc = nil
 

data/lib/opennebula/marketplace.rb

@@ -29,7 +29,15 @@ module OpenNebula
             :update     => "market.update",
             :chown      => "market.chown",
             :chmod      => "market.chmod",
-            :rename     => "market.rename"
+            :rename     => "market.rename",
+            :enable     => "market.enable"
+        }
+
+        MARKETPLACE_STATES=%w{ENABLED DISABLED}
+
+        SHORT_MARKETPLACE_STATES={
+            "ENABLED"              => "on",
+            "DISABLED"             => "off"
         }
 
         # Creates a MarketPlace description with just its identifier
@@ -130,13 +138,33 @@ module OpenNebula
         # @return [nil, OpenNebula::Error] nil in case of success, Error
         #   otherwise
         def rename(name)
-            return call(MARKETPLACE_METHODS[:rename], @pe_id, name)
+            call(MARKETPLACE_METHODS[:rename], @pe_id, name)
+        end
+
+        # Enables this marketplace
+        def enable
+            call(MARKETPLACE_METHODS[:enable], @pe_id, true)
+        end
+
+        # Enables this marketplace
+        def disable
+            call(MARKETPLACE_METHODS[:enable], @pe_id, false)
         end
 
         # ---------------------------------------------------------------------
         # Helpers to get information
         # ---------------------------------------------------------------------
 
+        # Returns the state of the Zone (numeric value)
+        def state
+            self['STATE'].to_i
+        end
+
+        # Returns the state of the Zone (string value)
+        def state_str
+            MARKETPLACE_STATES[state]
+        end
+
         # Returns whether or not the marketplace app with id 'id' is part of
         # this marketplace
         def contains(id)

data/lib/opennebula/wait_ext.rb

@@ -14,107 +14,118 @@
 # limitations under the License.                                             #
 #--------------------------------------------------------------------------- #
 
-
 require 'opennebula/host'
 require 'opennebula/image'
 require 'opennebula/virtual_machine'
 
-module OpenNebula::WaitExtEvent
-    def wait_event(ctx, event, timeout)
-        subscriber = ctx.socket(ZMQ::SUB)
+module OpenNebula
 
-        # Create subscriber
-        key        = ''
-        content    = ''
+    # Module to wait OpenNebula objects events using ZMQ
+    module WaitExtEvent
 
-        subscriber.setsockopt(ZMQ::RCVTIMEO, timeout * 1000)
-        subscriber.setsockopt(ZMQ::SUBSCRIBE, event)
-        subscriber.connect(@client.one_zmq)
+        def wait_event(ctx, event, timeout)
+            subscriber = ctx.socket(ZMQ::SUB)
 
-        rc = subscriber.recv_string(key)
-        rc = subscriber.recv_string(content) if rc != -1
+            # Create subscriber
+            key        = ''
+            content    = ''
 
-        return if ZMQ::Util.errno == ZMQ::EAGAIN || rc == -1
+            subscriber.setsockopt(ZMQ::RCVTIMEO, timeout * 1000)
+            subscriber.setsockopt(ZMQ::SUBSCRIBE, event)
+            subscriber.connect(@client.one_zmq)
 
-        content
-    ensure
-        subscriber.setsockopt(ZMQ::UNSUBSCRIBE, event)
-        subscriber.close
-    end
+            rc = subscriber.recv_string(key)
+            rc = subscriber.recv_string(content) if rc != -1
 
-    def wait2(sstr1, sstr2, timeout = 60, cycles = -1)
-        wfun = OpenNebula::WaitExt::WAIT[self.class]
-
-        # Start with a timeout of 2 seconds, to wait until the first
-        # info.
-        #
-        # The timeout is increased later, to avoid multiple info calls.
-        c_timeout = 2
-        recvs     = 0
-        in_state  = false
-
-        # Subscribe with timeout seconds
-        #
-        # Subscribe string:
-        #
-        #   EVENT STATE element_name/state_str//self.ID
-        #
-        #   - element_name: is the element name to find in the message
-        #   - self.ID: returns element ID to find in the message
-        ctx = ZMQ::Context.new(1)
-
-        until in_state || (cycles != -1 && recvs >= cycles)
-            content = wait_event(ctx,
-                                 wfun[:event].call(self, sstr1, sstr2),
-                                 c_timeout)
-
-            if content && !content.empty?
-                in_state = wfun[:in_state_e].call(sstr1, sstr2, content)
-
-                break if in_state
-            end
+            return if ZMQ::Util.errno == ZMQ::EAGAIN || rc == -1
+
+            content
+        ensure
+            subscriber.setsockopt(ZMQ::UNSUBSCRIBE, event)
+            subscriber.close
+        end
+
+        def wait2(sstr1, sstr2, timeout = 60, cycles = -1)
+            wfun = OpenNebula::WaitExt::WAIT[self.class]
+
+            # Start with a timeout of 2 seconds, to wait until the first
+            # info.
+            #
+            # The timeout is increased later, to avoid multiple info calls.
+            c_timeout = 2
+            recvs     = 0
+            in_state  = false
+
+            # Subscribe with timeout seconds
+            #
+            # Subscribe string:
+            #
+            #   EVENT STATE element_name/state_str//self.ID
+            #
+            #   - element_name: is the element name to find in the message
+            #   - self.ID: returns element ID to find in the message
+            ctx = ZMQ::Context.new(1)
+
+            until in_state || (cycles != -1 && recvs >= cycles)
+                content = wait_event(ctx,
+                                     wfun[:event].call(self, sstr1, sstr2),
+                                     c_timeout)
+
+                if content && !content.empty?
+                    in_state = wfun[:in_state_e].call(sstr1, sstr2, content)
+
+                    break if in_state
+                end
 
-            c_timeout *= 10
-            c_timeout  = timeout if c_timeout > timeout
+                c_timeout *= 10
+                c_timeout  = timeout if c_timeout > timeout
 
-            rco = info
+                rco = info
 
-            return false if OpenNebula.is_error?(rco)
+                return false if OpenNebula.is_error?(rco)
 
-            in_state = wfun[:in_state].call(self, sstr1, sstr2)
+                in_state = wfun[:in_state].call(self, sstr1, sstr2)
 
-            recvs += 1
+                recvs += 1
+            end
+
+            in_state
         end
 
-        in_state
     end
 
 end
 
-module OpenNebula::WaitExtPolling
-    def wait2(sstr1, sstr2, timeout = 60, cycles = -1)
-        wfun = OpenNebula::WaitExt::WAIT[self.class]
+module OpenNebula
+
+    # Module to wait OpenNebula objects events using polling
+    module WaitExtPolling
+
+        def wait2(sstr1, sstr2, timeout = 60, cycles = -1)
+            wfun = OpenNebula::WaitExt::WAIT[self.class]
+
+            stime    = 5
+            recvs    = 0
+            cycles   = timeout / stime if cycles == -1
+            in_state = false
 
-        stime    = 5
-        recvs    = 0
-        cycles   = timeout / stime
-        in_state = false
+            loop do
+                rco = info
 
-        loop do
-            rco = info
+                return false if OpenNebula.is_error?(rco)
 
-            return false if OpenNebula.is_error?(rco)
+                in_state = wfun[:in_state].call(self, sstr1, sstr2)
 
-            in_state = wfun[:in_state].call(self, sstr1, sstr2)
+                recvs += 1
 
-            recvs += 1
+                break if in_state || recvs >= cycles
 
-            break if in_state || recvs >= cycles
+                sleep stime
+            end
 
-            sleep stime
+            in_state
         end
 
-        in_state
     end
 
 end
@@ -124,6 +135,7 @@ end
 #
 # rubocop:disable Style/ClassAndModuleChildren
 module OpenNebula::WaitExt
+
     # Wait classes and the name published in ZMQ/STATE
     WAIT = {
         OpenNebula::Host  => {
@@ -176,7 +188,7 @@ module OpenNebula::WaitExt
             },
 
             :in_state => lambda {|o, s1, s2|
-              obj_s1 = Integer(o['STATE'])
+                obj_s1 = Integer(o['STATE'])
                 inx_s1 = OpenNebula::VirtualMachine::VM_STATE.index(s1)
 
                 obj_s2 = Integer(o['LCM_STATE'])
@@ -203,13 +215,15 @@ module OpenNebula::WaitExt
         wait?(obj)
 
         class << obj
-          begin
-            require 'ffi-rzmq'
 
-            include OpenNebula::WaitExtEvent
-          rescue LoadError
-            include OpenNebula::WaitExtPolling
-          end
+            begin
+                require 'ffi-rzmq'
+
+                include OpenNebula::WaitExtEvent
+            rescue LoadError
+                include OpenNebula::WaitExtPolling
+            end
+
         end
 
         super

data/lib/scripts_common.rb

@@ -45,16 +45,7 @@ module OpenNebula
 
     # This function is used to pass error message to the mad
     def self.error_message(message)
-        STDERR.puts format_error_message(message)
-    end
-
-    #This function formats an error message for OpenNebula
-    def self.format_error_message(message)
-        error_str = "ERROR MESSAGE --8<------\n"
-        error_str << message
-        error_str << "\nERROR MESSAGE ------>8--"
-
-        return error_str
+        STDERR.puts message
     end
 
     def self.is_disk?(arg)

data/lib/vcenter_driver.rb

@@ -133,19 +133,10 @@ end
 # Helper functions                                                             #
 # ---------------------------------------------------------------------------- #
 
-def error_message(message)
-    error_str = "ERROR MESSAGE --8<------\n"
-    error_str << message
-    error_str << "\nERROR MESSAGE ------>8--"
-
-    error_str
-end
-
 def check_valid(parameter, label)
     return unless parameter.nil? || parameter.empty?
 
-    STDERR.puts error_message("The parameter '#{label}'\
-                               is required for this action.")
+    STDERR.puts "The parameter '#{label}' is required for this action."
     exit(-1)
 end
 

data/lib/virtual_machine.rb

@@ -2418,10 +2418,10 @@ end
             # Adding a new disk in newer vSphere versions
             # automatically cleans all system snapshots
             # https://github.com/OpenNebula/one/issues/5409
-            if snapshots? or one_snapshots?
-              error_message =  'Existing sytem snapshots, cannot change disks. '
-              error_message << 'Please remove all snapshots and try again.'
-              raise error_message
+            if snapshots? || one_snapshots?
+                error_msg =  'Existing system snapshots, cannot change disks. '
+                error_msg << 'Please remove all snapshots and try again.'
+                raise error_message
             end
 
             spec_hash     = {}
@@ -2605,10 +2605,11 @@ end
         def detach_disk(disk)
             return unless disk.exists?
 
-            if snapshots? or one_snapshots?
-              error_message =  'Existing sytem snapshots, cannot change disks. '
-              error_message << 'Please remove all snapshots and try again.'
-              raise error_message
+            if snapshots? || one_snapshots?
+                error_message =  'Existing system snapshots, cannot change '
+                error_message << 'disks . Please remove all snapshots and '
+                error_message << 'try again.'
+                raise error_message
             end
 
             spec_hash = {}
@@ -3070,7 +3071,8 @@ end
         # Create a snapshot for the VM
         def create_snapshot(snap_id, snap_name)
             memory_dumps = true
-            memory_dumps = CONFIG[:memory_dumps] unless CONFIG[:memory_dumps].nil?
+            memory_dumps = CONFIG[:memory_dumps] \
+                                     unless CONFIG[:memory_dumps].nil?
 
             snapshot_hash = {
                 :name        => snap_id,

data/lib/vm_monitor.rb

@@ -175,8 +175,6 @@ module VirtualMachineMonitor
         @monitor[:diskwrbytes] = previous_diskwrbytes +
                                  (write_kbpersec * 1024 * refresh_rate).to_i
     end
-    # rubocop:enable Naming/VariableName
-    # rubocop:enable Style/FormatStringToken
 
     #  Generates a OpenNebula IM Driver valid string with the monitor info
     def info

data/lib/vm_template.rb

@@ -550,6 +550,10 @@ module VCenterDriver
                     net.ipConfig.ipAddress.each do |ip_config|
                         ip = IPAddr.new(ip_config.ipAddress)
 
+                        if ip.ipv6? && get_ipv6_prefix(ip.to_s, 10) == 'fe80'
+                            next
+                        end
+
                         if force
                             ipv4 = ip.to_s if ip.ipv4?
                             ipv6 = ip.to_s if ip.ipv6?

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: opennebula
 version: !ruby/object:Gem::Version
-  version: 6.0.2
+  version: 6.0.3
 platform: ruby
 authors:
 - OpenNebula
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-05-24 00:00:00.000000000 Z
+date: 2021-07-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -126,7 +126,6 @@ files:
 - lib/vcenter_importer.rb
 - lib/vi_client.rb
 - lib/vi_helper.rb
-- lib/virtual_machine.rb
 - lib/vm_device.rb
 - lib/vm_disk.rb
 - lib/vm_nic.rb
@@ -135,6 +134,7 @@ files:
 - lib/vm_folder.rb
 - lib/vm_template.rb
 - lib/vmm_importer.rb
+- lib/virtual_machine.rb
 - lib/nsx_driver.rb
 - lib/distributed_firewall.rb
 - lib/logical_port.rb
@@ -164,8 +164,8 @@ files:
 - lib/DriverExecHelper.rb
 - lib/scripts_common.rb
 - lib/models.rb
-- lib/models/role.rb
 - lib/models/service.rb
+- lib/models/role.rb
 - lib/opennebula/acl.rb
 - lib/opennebula/acl_pool.rb
 - lib/opennebula/client.rb
@@ -208,7 +208,6 @@ files:
 - lib/opennebula/utils.rb
 - lib/opennebula/vdc.rb
 - lib/opennebula/vdc_pool.rb
-- lib/opennebula/virtual_machine.rb
 - lib/opennebula/virtual_machine_ext.rb
 - lib/opennebula/virtual_machine_pool.rb
 - lib/opennebula/virtual_network.rb
@@ -224,13 +223,14 @@ files:
 - lib/opennebula/xml_pool.rb
 - lib/opennebula/xml_utils.rb
 - lib/opennebula/zone_pool.rb
+- lib/opennebula/virtual_machine.rb
 - lib/opennebula/zone.rb
 - lib/opennebula/flow/grammar.rb
 - lib/opennebula/flow/service_pool.rb
+- lib/opennebula/flow/service_template.rb
 - lib/opennebula/flow/service_template_ext.rb
 - lib/opennebula/flow/service_template_pool.rb
 - lib/opennebula/flow/validator.rb
-- lib/opennebula/flow/service_template.rb
 - lib/opennebula/ldap_auth.rb
 - lib/opennebula/ldap_auth_spec.rb
 - lib/opennebula/server_cipher_auth.rb