summaryrefslogtreecommitdiff
path: root/bin
diff options
context:
space:
mode:
authorSteve Kemp <steve@steve.org.uk>2015-02-05 11:34:33 +0000
committerSteve Kemp <steve@steve.org.uk>2015-02-05 11:34:33 +0000
commitcf350013807bf347af29f17a4c87861c194e3e10 (patch)
treee223ffbd0890bd76093d21384b45bf6426956d3d /bin
parentb349e6ab6305a054bc1bf6b568a1e9fb7425f404 (diff)
More simplification - don't pass state around.
Rather than passing our settings-object around, as well as specific settings that are read from it, just pass the object. The worker can read the settings directly if/when it needs to.
Diffstat (limited to 'bin')
-rwxr-xr-xbin/custodian-dequeue31
1 files changed, 9 insertions, 22 deletions
diff --git a/bin/custodian-dequeue b/bin/custodian-dequeue
index 7a99186..2abf247 100755
--- a/bin/custodian-dequeue
+++ b/bin/custodian-dequeue
@@ -8,7 +8,6 @@
# [ -m | --manual]
# [ -f | --fail ]
# [ -l | --logfile FILE]
-# [ -S | --server 1.2.3.4:123 ]
# [ -s | --single ]
# [ -v | --verbose ]
#
@@ -20,8 +19,6 @@
#
# -l, --logfile Specify the path to the logfile to run.
#
-# -S, --server Specify the host:port for the beanstalkd queue.
-#
# -s, --single Run a single test and exit. Don't poll the queue for more.
#
# -f, --fail Run tests but stop the first time we see a fail.
@@ -31,10 +28,11 @@
#
# ABOUT
#
-# This tool is designed to pull network/protocol-tests from a beanstalkd server and execute
-# them one by one.
+# This tool is designed to pull network/protocol-tests from the central queue
+# and execute them one by one.
#
-# The results of the testing will be sent to a notifier, where they can later be acted upon.
+# The results of the testing will be sent to a notifier, where they can later
+# be acted upon.
#
#
# AUTHOR
@@ -68,22 +66,15 @@ if __FILE__ == $0 then
$manual = false
#
- # The beanstalkd server address, alerting method, and logfile.
+ # The settings object contains a lot of configuration-data.
#
settings = Custodian::Settings.instance()
- $SERVER = settings.queue_server
- $ALERTER = settings.alerter
- $LOGFILE = settings.log_file
-
begin
opts = GetoptLong.new(
[ "--help", "-h", GetoptLong::NO_ARGUMENT ],
[ "--manual", "-m", GetoptLong::NO_ARGUMENT ],
[ "--fail", "-f", GetoptLong::NO_ARGUMENT ],
- [ "--logfile", "-l", GetoptLong::REQUIRED_ARGUMENT ],
- [ "--server", "-S", GetoptLong::REQUIRED_ARGUMENT ],
- [ "--alerter", "-a", GetoptLong::REQUIRED_ARGUMENT ],
[ "--single", "-s", GetoptLong::NO_ARGUMENT ],
[ "--verbose", "-v", GetoptLong::NO_ARGUMENT ]
)
@@ -91,12 +82,6 @@ if __FILE__ == $0 then
case opt
when "--verbose" then
ENV["VERBOSE"] = "1"
- when "--logfile" then
- $LOGFILE = arg
- when "--server" then
- $SERVER = arg
- when "--alerter" then
- $ALERTER = arg
when "--single" then
ENV["SINGLE"] = "1"
when "--fail" then
@@ -143,10 +128,12 @@ if __FILE__ == $0 then
exit 0
end
+
#
- # Create the object
+ # Create the worker, passing it the settings object so it can
+ # sort out its own logfile, etc.
#
- worker = Custodian::Worker.new( $SERVER, $ALERTER, $LOGFILE, settings )
+ worker = Custodian::Worker.new( settings )
#