renamed some hosts and switches, added ips for all hosts and added commands for starting tomcat.

renamed some hosts and switches, added ips for all hosts and added commands for starting tomcat.

[robert.erneborg, 2015-10-22 13:01:28]

On 2015/10/22 12:57:38, robert.erneborg wrote:

It looks like everything is new but it isn't. I only changed some names, added
ips for all hosts and added commands for launching all tomcat instances.

[teowey, 2015-10-22 13:15:11]

https://codereview.appspot.com/274020044/diff/1/topology1.py
File topology1.py (right):

https://codereview.appspot.com/274020044/diff/1/topology1.py#newcode90
topology1.py:90: #pre_commands(net)
Is it required to use this pre_commands(net) line when running the get() and
cmd() below?

[robert.erneborg, 2015-10-22 18:30:38]

https://codereview.appspot.com/274020044/diff/1/topology1.py
File topology1.py (right):

https://codereview.appspot.com/274020044/diff/1/topology1.py#newcode90
topology1.py:90: #pre_commands(net)
On 2015/10/22 13:15:11, teowey wrote:
> Is it required to use this pre_commands(net) line when running the get() and
> cmd() below? 

no idea, I didn't write this... but is is out commented so I guess not ;)

[marcus.falgert, 2015-10-22 21:00:41]

On 2015/10/22 13:15:11, teowey wrote:
> https://codereview.appspot.com/274020044/diff/1/topology1.py
> File topology1.py (right):
> 
> https://codereview.appspot.com/274020044/diff/1/topology1.py#newcode90
> topology1.py:90: #pre_commands(net)
> Is it required to use this pre_commands(net) line when running the get() and
> cmd() below?

not required, but we should probably put some of the code in fuctions, such as
the initialization phase (starting servers etc).

[teowey, 2015-10-23 08:41:15]

On 2015/10/22 21:00:41, marcus.falgert wrote:
> On 2015/10/22 13:15:11, teowey wrote:
> > https://codereview.appspot.com/274020044/diff/1/topology1.py
> > File topology1.py (right):
> > 
> > https://codereview.appspot.com/274020044/diff/1/topology1.py#newcode90
> > topology1.py:90: #pre_commands(net)
> > Is it required to use this pre_commands(net) line when running the get() and
> > cmd() below?
> 
> not required, but we should probably put some of the code in fuctions, such as
> the initialization phase (starting servers etc).

I agree. Just remember to commit in the "mininet-topology" branch of
network-emulator repo :)