dapp 0.22.16 → 0.22.17

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 236182f0450745e08a605cd74b57cee3c49216e2
-  data.tar.gz: c58e45331daec2b1a514094b5dcf5c961020e033
+  metadata.gz: 125ccf57682d13035150191173dd8b2f0ad8af3b
+  data.tar.gz: a1c74f6511df262e5a3423549c0909ff21110f23
 SHA512:
-  metadata.gz: 42186468517a2928235edc3c5de7f77f3b9a6c79cc67eb5d55a8124963af1027ea63e870d3f415aff2c9516c70a159de209fae32a88d73dc901eeeef67a47e36
-  data.tar.gz: 9dca98eba47c551cd7ab60b12c6dedcb2c72e2e5a77a4984af4b587069f1ca5d4ce9f144d72bc09853553392ed73fff0b8bc6f4e9ebbd5941df463dac4203ee3
+  metadata.gz: 60cb923e5e5cf44001c0dabdc04d56a903f7680219492f336900a2e36622da578c77836f77d3fc3c3a0855f5740a8ba82747d46d2c09296373c529118844a730
+  data.tar.gz: 47a57d0155fd13388240fdae1b42dc475d8f5ce3b30c051f118afb783b1ef805ad7aeb00a25a38848967c4a60b508ad87d6d508ab724503c52a9ba6b4dd2e2ef

data/bin/dapp

@@ -26,6 +26,10 @@ begin
       $stderr.puts "\033[1m\033[90mStacktrace dumped to #{filename}\033[0m"
     end
 
+    if Dapp::CLI.dapp_object
+      Dapp::CLI.dapp_object.sentry_exception(e)
+    end
+
     raise
   end
 rescue Dapp::Error::Shellout => e

data/lib/dapp.rb

@@ -25,6 +25,8 @@ require 'etc'
 require 'zlib'
 require 'slugify'
 require 'base64'
+require 'sentry-raven'
+require 'toml-rb'
 
 require 'dapp/version'
 require 'dapp/core_ext/hash'
@@ -35,6 +37,7 @@ require 'dapp/helper/sha256'
 require 'dapp/helper/net_status'
 require 'dapp/helper/tar'
 require 'dapp/helper/yaml'
+require 'dapp/helper/url'
 require 'dapp/prctl'
 require 'dapp/error/mod/user'
 require 'dapp/error/base'
@@ -55,6 +58,7 @@ require 'dapp/config/directive/base'
 require 'dapp/config/config'
 require 'dapp/dapp/lock'
 require 'dapp/dapp/ssh_agent'
+require 'dapp/dapp/sentry'
 require 'dapp/dapp/git_artifact'
 require 'dapp/dapp/dappfile'
 require 'dapp/dapp/chef'

data/lib/dapp/cli.rb

@@ -38,6 +38,10 @@ BANNER
            show_options: true,
            exit: 0
 
+    class << self
+      attr_accessor :dapp_object
+    end
+
     def initialize(*args)
       super(*args)
 

data/lib/dapp/cli/command/base.rb

@@ -58,6 +58,8 @@ module Dapp
 
         def run_dapp_command(run_method, options: {}, log_running_time: true, try_host_docker_login: false)
           dapp = ::Dapp::Dapp.new(options: options)
+          ::Dapp::CLI.dapp_object = dapp
+          dapp.sentry_message("Manual usage: `#{options[:dapp_command]}` command") unless ENV['CI']
 
           log_dapp_running_time(dapp, ignore: !log_running_time) do
             begin
@@ -74,6 +76,10 @@ module Dapp
           end
         end
 
+        def run_method
+          class_to_lowercase
+        end
+
         def log_dapp_running_time(dapp, ignore: false)
           return yield if ignore
 
@@ -95,7 +101,7 @@ module Dapp
             self.class.print_error_with_help_and_die! self, "cannot use alias options --run-dir, --build-dir, --deploy-dir at the same time"
           end
 
-          config.merge(build_dir: dirs.compact.first, **kwargs)
+          config.merge(build_dir: dirs.compact.first, dapp_command: run_method, **kwargs)
         end
       end
     end

data/lib/dapp/dapp.rb

@@ -17,10 +17,13 @@ module Dapp
     include Logging::Paint
 
     include SshAgent
+    include Sentry
+
     include Helper::Sha256
-    extend Helper::Trivia
+    extend  Helper::Trivia
     include Helper::Trivia
     include Helper::Tar
+    include Helper::Url
 
     include Deps::Toolchain
     include Deps::Gitartifact
@@ -41,6 +44,18 @@ module Dapp
       self.class.options
     end
 
+    def settings
+      @settings ||= begin
+        settings_path = File.join(self.class.home_dir, "settings.toml")
+
+        if File.exists? settings_path
+          TomlRB.load_file(settings_path)
+        else
+          {}
+        end
+      end
+    end
+
     def name
       @name ||= begin
         n = begin
@@ -86,8 +101,8 @@ module Dapp
       self.class.tmp_base_dir
     end
 
-    def build_path(*path)
-      @build_path ||= begin
+    def build_dir
+      @build_dir ||= begin
         if option_build_dir
           Pathname.new(option_build_dir)
         else
@@ -95,7 +110,10 @@ module Dapp
           Pathname.new(dir)
         end.expand_path.tap(&:mkpath)
       end
-      make_path(@build_path, *path)
+    end
+
+    def build_path(*path)
+      make_path(build_dir, *path)
     end
 
     def local_git_artifact_exclude_paths(&blk)

data/lib/dapp/dapp/option_tags.rb

@@ -11,6 +11,12 @@ module Dapp
 
       def tags_by_scheme
         @tags_by_scheme_name ||= begin
+          if simple_tags[:custom].any?
+            if settings.fetch("sentry", {}).fetch("detect-push-tag-usage", false)
+              sentry_message("--tag or --tag-slug usage detected", extra: {"slug_tags" => simple_tags})
+            end
+          end
+
           [simple_tags, branch_tags, commit_tags, build_tags, ci_tags].reduce({}) do |some_tags_by_scheme, tags_by_scheme|
             tags_by_scheme.in_depth_merge(some_tags_by_scheme)
           end.tap do |tags_by_scheme|

data/lib/dapp/dapp/sentry.rb

@@ -0,0 +1,112 @@
+module Dapp
+  class Dapp
+    module Sentry
+      def sentry_message(msg, **kwargs)
+        return if not ensure_sentry_configured
+        kwargs[:level] ||= "info"
+        Raven.capture_message(msg, _make_sentry_params(**kwargs))
+      end
+
+      def sentry_exception(exception, **kwargs)
+        return if not ensure_sentry_configured
+        (kwargs[:tags] ||= {})['error-code'] = begin
+          net_status = exception.net_status
+          [net_status[:context], net_status[:code]].compact.join('_')
+        end
+        Raven.capture_exception(exception, _make_sentry_params(**kwargs))
+      end
+
+      def ensure_sentry_configured
+        return false unless sentry_settings = settings["sentry"]
+
+        unless @sentry_settings_configured
+          Raven.configure do |config|
+            logger = ::Logger.new(STDOUT)
+            logger.level = ::Logger::WARN
+
+            config.logger = logger
+            config.dsn = sentry_settings["dsn"]
+          end
+
+          @sentry_settings_configured = true
+        end
+
+        return true
+      end
+
+      def _make_sentry_params(level: nil, tags: {}, extra: {}, user: {})
+        {
+          level: level,
+          tags:  _sentry_tags_context.merge(tags),
+          extra: _sentry_extra_context.merge(extra),
+          user:  _sentry_user_context.merge(user),
+        }
+      end
+
+      def _sentry_extra_context
+        @_sentry_extra_context ||= {
+          "pwd" => Dir.pwd,
+          "dapp-dir" => self.work_dir,
+          "options" => self.options,
+          "env-options" => {
+            "DAPP_FORCE_SAVE_CACHE" => ENV["DAPP_FORCE_SAVE_CACHE"],
+            "DAPP_BIN_DAPPFILE_YML" => ENV["DAPP_BIN_DAPPFILE_YML"],
+            "ANSIBLE_ARGS" => ENV["ANSIBLE_ARGS"],
+            "DAPP_CHEF_DEBUG" => ENV["DAPP_CHEF_DEBUG"],
+          },
+        }.tap {|extra|
+          extra["ci-env"] = {"CI" => ENV["CI"]}
+          ENV.select {|k, v| k.start_with?("CI_")}.each do |k, v|
+            extra["ci-env"][k] = v
+          end
+        }
+      end
+
+      def _sentry_tags_context
+        name = options[:name] || 
+        @_sentry_tags_context ||= {
+          "dapp-short-version" => ::Dapp::VERSION.split(".")[0..1].join("."),
+          "dapp-version" => ::Dapp::VERSION,
+          "dapp-build-cache-version" => ::Dapp::BUILD_CACHE_VERSION,
+          "dapp-command" => self.options[:dapp_command],
+        }.tap {|tags|
+          git_config_path = File.join(Dir.pwd, ".git/config")
+
+          tags["dapp-name"] = options[:name]
+
+          if File.exists? git_config_path
+            cfg = IniFile.load(File.join(Dir.pwd, ".git/config"))
+            remote_origin_cfg = cfg['remote "origin"']
+            remote_origin_url = remote_origin_cfg["url"]
+            if remote_origin_url
+              tags["dapp-name"] ||= begin
+                repo_name = remote_origin_url.split('/').last
+                repo_name = repo_name[/.*(?=\.git)/] if repo_name.end_with? '.git'
+                repo_name
+              end
+
+              tags["git-host"] = self.get_host_from_git_url(remote_origin_url)
+
+              git_name = self.git_url_to_name(remote_origin_url)
+
+              tags["git-group"] = git_name.partition("/")[0]
+              tags["git-name"] = git_name
+            end
+          end
+
+          tags["dapp-name"] ||= File.basename(Dir.pwd)
+
+          begin
+            ver = self.class.host_docker_minor_version
+            tags["docker-minor-version"] = ver.to_s
+          rescue ::Exception
+          end
+        }
+      end
+
+      def _sentry_user_context
+        @_sentry_user_context ||= {}
+      end
+    end # Sentry
+  end # Dapp
+end # Dapp

data/lib/dapp/dimg/cli/command/base.rb

@@ -9,10 +9,6 @@ module Dapp::Dimg::CLI
         self.class.parse_options(self, argv)
         run_dapp_command(run_method, options: cli_options(dimgs_patterns: cli_arguments))
       end
-
-      def run_method
-        class_to_lowercase
-      end
     end
   end
 end

data/lib/dapp/dimg/config/directive/git_artifact_remote.rb

@@ -3,14 +3,13 @@ module Dapp
     module Config
       module Directive
         class GitArtifactRemote < GitArtifactLocal
+          include ::Dapp::Helper::Url
+
           attr_reader :_url, :_name, :_branch, :_commit
 
           def initialize(url, **kwargs, &blk)
             @_url  = url
-
-            url_without_scheme = url.split("://", 2).last
-            url_without_creds = url_without_scheme.split(":", 2).last
-            @_name = url_without_creds.gsub(%r{.*?([^\/ ]+\/[^\/ ]+)\.git}, '\\1')
+            @_name = git_url_to_name(url)
 
             super(**kwargs, &blk)
           end

data/lib/dapp/helper/url.rb

@@ -0,0 +1,23 @@
+module Dapp
+  module Helper
+    module Url
+      def git_url_to_name(url)
+        url_without_scheme = url.split("://", 2).last
+        # This may be broken, because "@" should delimit creds, not a ":"
+        url_without_creds = url_without_scheme.split(":", 2).last
+        url_without_creds.gsub(%r{.*?([^\/ ]+\/[^\/ ]+)\.git}, '\\1')
+      end
+
+      def get_host_from_git_url(url)
+        url_without_scheme = url.split("://", 2).last
+        url_without_creds = url_without_scheme.split("@", 2).last
+
+        # Split out part after ":" in this kind of url: github.com:flant/dapp.git
+        url_part = url_without_creds.split(":", 2).first
+
+        # Split out part after first "/": github.com/flant/dapp.git
+        url_part.split("/", 2).first
+      end
+    end # Url
+  end # Helper
+end # Dapp

data/lib/dapp/kube/dapp/command/deploy.rb

@@ -71,7 +71,14 @@ module Dapp
               watch_hooks_thr = nil
               unless dry_run?
                 watch_hooks_thr = Thread.new do
-                  watch_hooks.each {|job| Kubernetes::Manager::Job.new(self, job.name).watch_till_done!}
+                  watch_hooks.each {|job|
+                    begin
+                      Kubernetes::Manager::Job.new(self, job.name).watch_till_done!
+                    rescue ::Exception => e
+                      sentry_exception(e, extra: {"job-spec" => job.spec})
+                      raise
+                    end
+                  }
                 end
               end
 

data/lib/dapp/version.rb

@@ -1,4 +1,4 @@
 module Dapp
-  VERSION = '0.22.16'.freeze
+  VERSION = '0.22.17'.freeze
   BUILD_CACHE_VERSION = 26.1
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dapp
 version: !ruby/object:Gem::Version
-  version: 0.22.16
+  version: 0.22.17
 platform: ruby
 authors:
 - Dmitry Stolyarov
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-02-22 00:00:00.000000000 Z
+date: 2018-03-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mixlib-shellout
@@ -186,6 +186,34 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 1.0.6
+- !ruby/object:Gem::Dependency
+  name: sentry-raven
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.7.2
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.7.2
+- !ruby/object:Gem::Dependency
+  name: toml-rb
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.1.1
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.1.1
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -448,6 +476,7 @@ files:
 - lib/dapp/dapp/logging/paint.rb
 - lib/dapp/dapp/logging/process.rb
 - lib/dapp/dapp/option_tags.rb
+- lib/dapp/dapp/sentry.rb
 - lib/dapp/dapp/shellout/base.rb
 - lib/dapp/dapp/shellout/streaming.rb
 - lib/dapp/dapp/slug.rb
@@ -647,6 +676,7 @@ files:
 - lib/dapp/helper/sha256.rb
 - lib/dapp/helper/tar.rb
 - lib/dapp/helper/trivia.rb
+- lib/dapp/helper/url.rb
 - lib/dapp/helper/yaml.rb
 - lib/dapp/kube.rb
 - lib/dapp/kube/cli/cli.rb