Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(3944)

Issue 325540043: New feature; now you can specify the placements of hosts as a parameter to Mininet (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
6 years, 7 months ago by Michail.Xirouchakis
Modified:
6 years, 6 months ago
CC:
sdnik2200ht17_ict.kth.se, michail.xirouchakis_gmail.com
Visibility:
Public.

Description

New feature; now you can specify the placements of hosts as a parameter to Mininet BUG=

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -13 lines) Patch
M topo-stanford-campus.py View 3 chunks +17 lines, -13 lines 1 comment Download

Messages

Total messages: 4
Michail.Xirouchakis
Please review my code. I have added a new functionality in mininet topology script. Now ...
6 years, 7 months ago (2017-09-18 19:42:51 UTC) #1
Vasileios Papageorgiou
On 2017/09/18 19:42:51, Michail.Xirouchakis wrote: > Please review my code. I have added a new ...
6 years, 7 months ago (2017-09-20 13:45:17 UTC) #2
Vasileios Papageorgiou
Topology seems to work properly
6 years, 7 months ago (2017-09-20 13:51:55 UTC) #3
engr.sikandar.iqbal
6 years, 6 months ago (2017-09-23 18:00:58 UTC) #4
A test code review

https://codereview.appspot.com/325540043/diff/1/topo-stanford-campus.py
File topo-stanford-campus.py (right):

https://codereview.appspot.com/325540043/diff/1/topo-stanford-campus.py#newco...
topo-stanford-campus.py:22: # creates 4 hosts (h1,h2,h3,h4) and attaches them to
switches s303,s304,s313 and s314
Weldone Michail! it's working for all 4 hosts thanks
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b