Browse files

I don't always test my code...

  • Loading branch information...
1 parent b377d67 commit fe5d8f80796f79045f0a704e664fe311bf93bc79 @roidrage roidrage committed Jul 27, 2012
Showing with 17 additions and 3 deletions.
  1. +17 −3 blog/_posts/
@@ -55,9 +55,10 @@ So far, we've pushed builds immediately onto our RabbitMQ queue as they come in.
This had the advantage that they'd be popped off in the order they arrived in
Travis, and we wouldn't have to worry about scheduling.
-But it also meant that our queues would be the single source of truth about what
-needs to be built next, which in turn introduces a single point of failure and
-pushed state into the message broker.
+But it also meant that our queues would have to be kept in sync with the
+database. Should we lose the jobs queued in RabbitMQ for some reason, we had to
+manually queue them. With the new logic, the database is the single source of
+truth and jobs are only queued if there's capacity available to build them.
Instead of pushing a job immediately on the queue, it's now only stored in the
database. A job is now scheduled based on the heartbeats we receive from a
@@ -72,10 +73,23 @@ components. We could use ZeroMQ or simple HTTP instead, because we're doing
simple inter-process communication instead of relying on the ordered nature of
AMQP, because exact order is not important anymore.
+Another upshot of this is that we don't need to do manual requeing anymore,
+the new logic will automatically pick up any build that's ready to be built.
We have several areas in Travis where order is still important currently, but we
already have ideas on how to tackle them. We'll keep you posted on the technical
+On a fun side note, the changes described here were deployed earlier today.
+Here's a graph of the average run time of handling worker updates went through
+the roof, middle finger style, until we added a database index:
+![Librato Metrics](
+Today's motto:
### The Bottom Line
Introducing a quality of service level to Travis CI doesn't only introduce a

0 comments on commit fe5d8f8

Please sign in to comment.