Merge pull request #609 from JustAnotherArchivist/bot-check-pipeline-option

Verify that there is a suitable running pipeline when --pipeline/-p is used

Author: JustAnotherArchivist

bot/brain.rb

@@ -2,6 +2,7 @@
 
 require File.expand_path('../../lib/job', __FILE__)
 require File.expand_path('../../lib/pipeline_info', __FILE__)
+require File.expand_path('../../lib/pipeline_collection', __FILE__)
 require File.expand_path('../summary', __FILE__)
 require File.expand_path('../post_registration_hook', __FILE__)
 require File.expand_path('../add_ignore_sets', __FILE__)
@@ -90,6 +91,14 @@ def request_archive(m, target, params, depth=:inf, url_file=false)
       return
     end
 
+    # Is there any suitable running --pipeline?
+    if h[:pipeline]
+      if !PipelineCollection.new(redis).any? { |pipeline| pipeline['nickname'].include?(h[:pipeline]) && pipeline['status'] == 'Running' }
+        reply m, "Sorry, there is no running pipeline that matches --pipeline #{h[:pipeline]}."
+        return
+      end
+    end
+
     # Is the URI in our list of recognized schemes?
     if !schemes.include?(uri.scheme)
       reply m, "Sorry, I can only handle #{schemes.join(', ')}."