Fixed issue with 'moveHost' event handling in the topo view, to
handle multi-homed hosts.
Change-Id: If605388b00aef70ffc89639b60bb88b22f0469d6
diff --git a/web/gui/src/main/webapp/app/view/topo/topoForce.js b/web/gui/src/main/webapp/app/view/topo/topoForce.js
index b4073a0..a629dfc 100644
--- a/web/gui/src/main/webapp/app/view/topo/topoForce.js
+++ b/web/gui/src/main/webapp/app/view/topo/topoForce.js
@@ -174,26 +174,8 @@
lu[id] = d;
updateNodes();
- function mkLinkKey(devId, devPort) {
- return id + '/0-' + devId + '/' + devPort;
- }
-
// need to handle possible multiple links (multi-homed host)
- d.links = [];
- data.allCps.forEach(function (cp) {
- var linkData = {
- key: mkLinkKey(cp.device, cp.port),
- dst: cp.device,
- dstPort: cp.port,
- };
- d.links.push(linkData);
-
- var lnk = tms.createHostLink(id, cp.device, cp.port);
- if (lnk) {
- network.links.push(lnk);
- lu[linkData.key] = lnk;
- }
- });
+ createHostLinks(data.allCps, d);
if (d.links.length) {
updateLinks();
@@ -213,15 +195,30 @@
}
}
+ function createHostLinks(cps, model) {
+ model.links = [];
+ cps.forEach(function (cp) {
+ var linkData = {
+ key: model.id + '/0-' + cp.device + '/' + cp.port,
+ dst: cp.device,
+ dstPort: cp.port,
+ };
+ model.links.push(linkData);
+
+ var lnk = tms.createHostLink(model.id, cp.device, cp.port);
+ if (lnk) {
+ network.links.push(lnk);
+ lu[linkData.key] = lnk;
+ }
+ });
+ }
+
function moveHost(data) {
var id = data.id,
- d = lu[id],
- lnk;
+ d = lu[id];
if (d) {
- // first remove the old host link
- // FIXME: what if the host has multiple links??????
- removeLinkElement(d.linkData);
+ removeAllLinkElements(d.links);
// merge new data
angular.extend(d, data);
@@ -229,19 +226,8 @@
sendUpdateMeta(d);
}
- // now create a new host link
- // TODO: verify this is the APPROPRIATE host link
- lnk = tms.createHostLink(id, data.cp.device, data.cp.port);
- if (lnk) {
- network.links.push(lnk);
- lu[lnk.key] = lnk;
-
- d.links.push({
- key: id + '/0-' + cp.device + '/' + cp.port,
- dst: data.cp.device,
- dstPort: data.cp.port,
- });
- }
+ // now create new host link(s)
+ createHostLinks(data.allCps, d);
updateNodes();
updateLinks();
@@ -391,6 +377,12 @@
}
}
+ function removeAllLinkElements(links) {
+ links.forEach(function (lnk) {
+ removeLinkElement(lnk);
+ });
+ }
+
function removeLinkElement(d) {
var idx = fs.find(d.key, network.links, 'key'),
removed;
diff --git a/web/gui/src/main/webapp/tests/package-lock.json b/web/gui/src/main/webapp/tests/package-lock.json
index b5f7f3c..4eb860d 100644
--- a/web/gui/src/main/webapp/tests/package-lock.json
+++ b/web/gui/src/main/webapp/tests/package-lock.json
@@ -9,6 +9,16 @@
"integrity": "sha1-Vdvr7wrZFVOC2enT5JfBNyNFtEo=",
"dev": true
},
+ "debug": {
+ "version": "2.6.8",
+ "resolved": "https://registry.npmjs.org/debug/-/debug-2.6.8.tgz",
+ "integrity": "sha1-5zFTHKLt4n0YgiJCfaF4IdaP9Pw="
+ },
+ "is-typedarray": {
+ "version": "1.0.0",
+ "resolved": "https://registry.npmjs.org/is-typedarray/-/is-typedarray-1.0.0.tgz",
+ "integrity": "sha1-5HnICFjfDBsR3dppQPlgEfzaSpo="
+ },
"jasmine-core": {
"version": "2.6.4",
"resolved": "https://registry.npmjs.org/jasmine-core/-/jasmine-core-2.6.4.tgz",
@@ -3514,6 +3524,16 @@
}
}
},
+ "ms": {
+ "version": "2.0.0",
+ "resolved": "https://registry.npmjs.org/ms/-/ms-2.0.0.tgz",
+ "integrity": "sha1-VgiurfwAvmwpAd9fmGF4jeDVl8g="
+ },
+ "nan": {
+ "version": "2.7.0",
+ "resolved": "https://registry.npmjs.org/nan/-/nan-2.7.0.tgz",
+ "integrity": "sha1-2Vv3IeyHfgjbJ27T/G63j5CDrUY="
+ },
"phantomjs": {
"version": "2.1.7",
"resolved": "https://registry.npmjs.org/phantomjs/-/phantomjs-2.1.7.tgz",
@@ -4261,6 +4281,21 @@
}
}
}
+ },
+ "typedarray-to-buffer": {
+ "version": "3.1.2",
+ "resolved": "https://registry.npmjs.org/typedarray-to-buffer/-/typedarray-to-buffer-3.1.2.tgz",
+ "integrity": "sha1-EBezLZhP9VbroQD1AViauhrOLgQ="
+ },
+ "websocket": {
+ "version": "1.0.24",
+ "resolved": "https://registry.npmjs.org/websocket/-/websocket-1.0.24.tgz",
+ "integrity": "sha1-dJA+dfJUW2suHeFCW8HJBZF6GJA="
+ },
+ "yaeti": {
+ "version": "0.0.6",
+ "resolved": "https://registry.npmjs.org/yaeti/-/yaeti-0.0.6.tgz",
+ "integrity": "sha1-8m9ITXJoTPQr7ft2lwqhYI+/lXc="
}
}
}
diff --git a/web/gui/src/main/webapp/tests/package.json b/web/gui/src/main/webapp/tests/package.json
index 59b8102..c0ae7c3 100644
--- a/web/gui/src/main/webapp/tests/package.json
+++ b/web/gui/src/main/webapp/tests/package.json
@@ -26,6 +26,7 @@
"karma-mocha-reporter": "^1.1.3",
"karma-ng-html2js-preprocessor": "^0.2.0",
"karma-phantomjs-launcher": "^0.2.1",
- "phantomjs": "^2.1.3"
+ "phantomjs": "^2.1.3",
+ "websocket": "^1.0.24"
}
}