dapp 0.7.28 → 0.7.29

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 591dfeda702d8e91d15e1e3da4152556324a7d1a
-  data.tar.gz: e9d75837a10c303a8af9de51de347c098a33a66a
+  metadata.gz: 27f41edc93f675eb3cb343639e176e7ed7163643
+  data.tar.gz: dd34396d0d2f76e0bf83ec0a51e5744655be40e4
 SHA512:
-  metadata.gz: 857a9902f01ef9f65def9481cd538aa71902b9908f569faeeaf1256409bb567228689bf53782bc1c3c20669356abb2be92a8cd42de2091d149d2b05a58f2f512
-  data.tar.gz: af3996880805bff85b6bf26d0e42464c710e6d2729c06cdbf6e4e8683e760e042fa7e7a117a4a81ff8594bca96d7e61b81713a4831984483b829c3f86de72d84
+  metadata.gz: f743206bfe53fbc9a01adfd730a58d0533cf5c8aa1819b719fc6adaf6b7e551733bb1b52e1f6d9693fee02b5cd92bb3c9694a556ecbd28561e51ce98cb947df3
+  data.tar.gz: 358972edf1f1293639ea69a26a0d1ddb1338e425c2dd1431ce79ffaddefb9ffdca0e8292050993427f3daf7cb171d8293a8224aa756c5494399d2414e5c06172

data/config/en/net_status.yml

@@ -17,6 +17,7 @@ en:
       image_not_exist: "Image `%{name}` not exist!"
       built_id_not_defined: '`from.built_id` not defined!'
       from_image_not_found: 'Image `%{name}` not found!'
+      unsupported_docker_image_size_format: "Unsupported docker image size format `%{value}`"
     project:
       command_unexpected_dimgs_number: "Command can process only one dimg!\nAmbiguous dimg pattern: `%{dimgs_names}`!"
       no_such_dimg: "No such dimg: `%{dimgs_patterns}`!"

data/lib/dapp/build/stage/artifact_default.rb

@@ -31,32 +31,44 @@ module Dapp
 
         # rubocop:disable Metrics/ParameterLists, Metrics/AbcSize, Metrics/MethodLength
         def safe_cp(from, to, owner, group, include_paths = [], exclude_paths = [])
-          credentials = ''
-          credentials += "-o #{owner} " if owner
-          credentials += "-g #{group} " if group
-          excludes = find_command_excludes(from, exclude_paths).join(' ')
-
-          copy_files = proc do |from_, path_ = ''|
-            "if [[ -d #{File.join(from_, path_)} ]] || [[ -f #{File.join(from_, path_)} ]]; then " \
-            "#{dimg.project.find_bin} #{File.join(from_, path_)} #{excludes} -type f -exec " \
-            "#{dimg.project.bash_bin} -ec '#{dimg.project.install_bin} -D #{credentials} \"{}\" " \
-            "\"#{File.join(to, "$(echo \"{}\" | " \
-            "#{dimg.project.sed_bin} -e \"s/^#{from_.gsub('/', '\\/')}\\///g\")")}\"' \\; ;" \
-            'fi'
-          end
+          ''.tap do |cmd|
+            cmd << dimg.project.rsync_bin
+            cmd << ' --archive --links'
+            cmd << " --chown=#{owner}:#{group}" if owner or group
 
-          commands = []
-          commands << [dimg.project.install_bin, credentials, '-d', to].join(' ')
-          commands.concat(include_paths.empty? ? Array(copy_files.call(from)) : include_paths.map { |path| copy_files.call(from, path) })
-          commands << "#{dimg.project.find_bin} #{to} -type d -exec " \
-                      "#{dimg.project.bash_bin} -ec '#{dimg.project.install_bin} -d #{credentials} {}' \\;"
-          commands.join(' && ')
-        end
-        # rubocop:enable Metrics/ParameterLists, Metrics/AbcSize, Metrics/MethodLength
+            if include_paths.any?
+              # Если указали include_paths ­— это означает, что надо копировать
+              # только указанные пути. Поэтому exclude_paths в приоритете, т.к. в данном режиме
+              # exclude_paths может относится только к путям, указанным в include_paths.
+              # При этом случай, когда в include_paths указали более специальный путь, чем в exclude_paths,
+              # будет обрабатываться в пользу exclude, этот путь не скопируется.
+              exclude_paths.each do |p|
+                cmd << " --filter='-/ #{File.join(from, p)}'"
+              end
+
+              include_paths.each do |p|
+                # * На данный момент не знаем директорию или файл имел в виду пользователь,
+                #   поэтому подставляем фильтры для обоих возможных случаев.
+                # * Автоматом подставляем паттерн ** для включения файлов, содержащихся в
+                #   директории, которую пользователь указал в include_paths.
+                cmd << " --filter='+/ #{File.join(from, p)}'"
+                cmd << " --filter='+/ #{File.join(from, p, '**')}'"
+              end
 
-        def find_command_excludes(from, exclude_paths)
-          exclude_paths.map { |path| "-not \\( -path #{File.join(from, path)} -prune \\)" }
+              # Все что не подошло по include — исключается
+              cmd << " --filter='-/ #{File.join(from, '**')}'"
+            else
+              exclude_paths.each do |p|
+                cmd << " --filter='-/ #{File.join(from, p)}'"
+              end
+            end
+
+            # Слэш после from — это инструкция rsync'у для копирования
+            # содержимого директории from, а не самой директории.
+            cmd << " #{from}/ #{to}"
+          end
         end
+        # rubocop:enable Metrics/ParameterLists, Metrics/AbcSize, Metrics/MethodLength
       end # ArtifactDefault
     end # Stage
   end # Build

data/lib/dapp/image/docker.rb

@@ -90,17 +90,23 @@ module Dapp
         def cache_reset(name = '')
           cache.delete(name)
           Project.shellout!("docker images --format='{{.Repository}}:{{.Tag}};{{.ID}};{{.CreatedAt}};{{.Size}}' --no-trunc #{name}").stdout.lines.each do |l|
-            name, id, created_at, size_field = l.split(';')
+            name, id, created_at, size_field = l.split(';').map(&:strip)
             size = begin
-              number, unit = size_field.split
-              coef = case unit.to_s.downcase
-                     when 'b'  then return number.to_f
+              match = size_field.match(/^(\d+(\.\d+)?)\ ?(b|kb|mb|gb|tb)$/i)
+              raise Error::Build, code: :unsupported_docker_image_size_format, data: {value: size_field} unless match and match[1] and match[3]
+
+              number = match[1].to_f
+              unit = match[3].downcase
+
+              coef = case unit
+                     when 'b'  then 0
                      when 'kb' then 1
                      when 'mb' then 2
                      when 'gb' then 3
                      when 'tb' then 4
                      end
-              number.to_f * (1000 ** coef)
+
+              number * (1000**coef)
             end
             cache[name] = { id: id, created_at: created_at, size: size }
           end

data/lib/dapp/version.rb

@@ -1,5 +1,5 @@
 # Version
 module Dapp
-  VERSION = '0.7.28'.freeze
-  BUILD_CACHE_VERSION = 6
+  VERSION = '0.7.29'.freeze
+  BUILD_CACHE_VERSION = '6.1'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dapp
 version: !ruby/object:Gem::Version
-  version: 0.7.28
+  version: 0.7.29
 platform: ruby
 authors:
 - Dmitry Stolyarov
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-03-30 00:00:00.000000000 Z
+date: 2017-04-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mixlib-shellout