Fix service ids to include container UUIDs

chaseajen · Dec 17, 2015 · 41c76c6 · 41c76c6
1 parent 398b012
commit 41c76c6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/consulWriter.js b/src/consulWriter.js
@@ -10,7 +10,8 @@ function registerServices(consulHost, consulPort, services) {
     var consul = require('consul')({ host: consulHost, port: consulPort });
 
     // Enumerate the services
-    services.forEach(function (service) {// Register the service
+    services.forEach(function (service) {
+        // Register the service
         consul.agent.service.register({
             name: service.name,
             id: service.id,

diff --git a/src/tutumReader.js b/src/tutumReader.js
@@ -34,6 +34,7 @@ function processResponseBody(body, callback) {
         // Get both the private IP and the public DNS
         var privateIp = container.private_ip;
         var publicDns = container.public_dns;
+        var uuid = container.uuid;
 
         // Enumerate the ports
         var ports = container.container_ports;
@@ -48,7 +49,7 @@ function processResponseBody(body, callback) {
 
             // Create the service object
             var service = {
-                id: containerName + "-" + serviceName,
+                id: containerName + "-" + serviceName + "-" + uuid,
                 name: serviceName,
                 container: containerName,
                 published: published