diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 9caa313537764ff0bac34d984b905ef7feddd677..d78d87cb18cf45cb6964e1c17153bd9a65a76759 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -183,7 +183,7 @@ spinach:ruby21: notify:slack: stage: notifications script: - - bundle exec rake ci:slack:error + - bundle exec rake ci:slack:error["#ci-test","Build failed for master/tags"] when: on_failure # only: # - master diff --git a/app/models/project.rb b/app/models/project.rb index 2313cad05d3949cd57e51c98e6fd106044b0ae78..043f08b9a13a36f4245f9a8aa0f6daeb934b4614 100644 --- a/app/models/project.rb +++ b/app/models/project.rb @@ -42,8 +42,7 @@ require 'carrierwave/orm/activerecord' require 'file_size_validator' - -lass Project < ActiveRecord::Base +class Project < ActiveRecord::Base include Gitlab::ConfigHelper include Gitlab::ShellAdapter include Gitlab::VisibilityLevel diff --git a/lib/tasks/ci/slack.rake b/lib/tasks/ci/slack.rake index c4edda466ac24eeb51cd1b4cb9050f9a115981af..0afc0e908d1400d743fdc95da00b2e9ac46d7865 100644 --- a/lib/tasks/ci/slack.rake +++ b/lib/tasks/ci/slack.rake @@ -1,9 +1,9 @@ namespace :ci do namespace :slack do desc "GitLab CI | Send slack notification on build failure" - task error: :environment do - error_text = 'Build failed for master/tags' - Kernel.system "curl -X POST --data-urlencode 'payload={\"channel\": \"#ci-test\", \"username\": \"gitlab-ci\", \"text\": \"#{error_text}\", \"icon_emoji\": \":gitlab:\"}' $CI_SLACK_WEBHOOK_URL" + task :error, [:channel, :error] do |t, args| + next unless !"#{ENV['CI_SLACK_WEBHOOK_URL']}".blank? && args.channel && args.error + Kernel.system "curl -X POST --data-urlencode 'payload={\"channel\": \"#{args.channel}\", \"username\": \"gitlab-ci\", \"text\": \"#{args.error}\", \"icon_emoji\": \":gitlab:\"}' $CI_SLACK_WEBHOOK_URL" end end end