Merge lp://staging/~qzhang/lava-dispatcher/reliable-result-transfer into lp://staging/lava-dispatcher

174 + def enum_master_ip(self, network):
175 + #store ip in myip of current path
176 + print "Determine master image IP"
177 + client = self.client
178 + try:
179 + if client.board.network_interface:
180 + eth_device = client.board.network_interface
181 + else:
182 + eth_device = "eth0"
183 + except:
184 + eth_device = "eth0"
185 + cmd = ("ifconfig %s | grep 'inet addr' | awk -F: '{print $2}' |"
186 + "awk '{print $1}' > myip" % eth_device)
187 + client.run_shell_command(cmd, response=MASTER_STR)
188 + for ip in IPNetwork(network):
189 + try:
190 + client.run_shell_command('cat myip',
191 + response=str(ip), timeout=2)
This is pretty complicated, and fails to deal with the fact that some boards may need to have some other interface, say usb0 for instance. I think there's two things we need to do around this:
1. Add a config item for the network interface we will use (using the new config files would be great for this!)
2. simplify this by just running ifconfig ont hat interface, and using expect to match the pattern

97 + client.proc.sendline('python -m SimpleHTTPServer 80')
98 + time.sleep(5)
Instead, why not just run 'python -m SimpleHTTPServer &> /dev/null &' and expect the shell prompt back? Then when it's over, we can just kill %1 to end the process.

   Not sure if this helps, but we were thinking about devices with one
or more interfaces when discussing the new config system here. We were
thinking of something along the lines of:

----

File: niftyboard01.conf

[machine]
stuff=things
more_stuff = other things
interfaces = eth0, eth1, usb0

[eth0]
mac_addr = 11:22:33:44:55:66
ip_addr = 192.168.1.1

[eth1]
mac_addr = 22:33:44:55:66:77
ip_addr = 192.168.1.5

etc.

----

   Internally, you can then easily represent the interfaces for each
machine as a list of dictionaries.

David

What about also adding a default_interface config item? This would point to the nic that could be used to communicate with the controller for gathering results, or whatever else is needed.

I don't think this has been tested, when I tried to run with it, there were some quickly noticeable problems:

51 + bundle = "/tmp/%s/%s", % (LAVA_RESULT_DIR, bundle_name)
No comma there

189 + try:
190 + client.run_shell_command('cat myip',
191 + response=str(ip), timeout=2)
Indent

> I don't think this has been tested, when I tried to run with it, there were
> some quickly noticeable problems:
>
> 51 + bundle = "/tmp/%s/%s", % (LAVA_RESULT_DIR, bundle_name)
> No comma there
The normal image code is tested but download() fails. On Android part code, it is modified after I tested normal image, but I have no available android board to test, so I modify it and push. I have fixed the bug when I find it unable to run on recent commits.

>
> 189 + try:
> 190 + client.run_shell_command('cat myip',
> 191 + response=str(ip), timeout=2)
> Indent

change to match IP determination way

22 +from netaddr import IPNetwork
no longer needed

Otherwise, looks good. I'll try to get this merged today. The downloading worked fine for me