<div dir="ltr">Yes, that is correct way.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Nov 6, 2019 at 11:22 AM Tony Lee (李文富) <<a href="mailto:Tony.Lee@quantatw.com">Tony.Lee@quantatw.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Rahul,<br>
<br>
For example, case "Retrieve IP Address Via IPMI And Verify Using Redfish" in /ipmi/test_ipmi_network.robot.<br>
The new way is as follows:<br>
<br>
IPMI network channel logically starts from 1. For example, once it figure out the channel count is 3,<br>
we can retrieve IP addresses through channels 1, 2, and 3, and verify them correspond to eth0, eth1, and eth2, respectively by redfish.<br>
Is it right?<br>
<br>
Thanks<br>
Best Regards,<br>
Tony<br>
<br>
From: Rahul Maheshwari <<a href="mailto:rahulmaheshwari01@gmail.com" target="_blank">rahulmaheshwari01@gmail.com</a>> <br>
Sent: Tuesday, November 5, 2019 6:51 PM<br>
To: Tony Lee (李文富) <<a href="mailto:Tony.Lee@quantatw.com" target="_blank">Tony.Lee@quantatw.com</a>><br>
Cc: <a href="mailto:openbmc@lists.ozlabs.org" target="_blank">openbmc@lists.ozlabs.org</a><br>
Subject: Re: Questions about openbmc-test-automation<br>
<br>
Thanks for pointing that out. Passing interface/channel number is an easier option but we should be avoiding passing environment variable unless there is no other way. I would say better way to overcome this is to figure out the channel count using "ip addr" command from BMC cli. <br>
Let us know if that sounds good to you.<br>
Also let us know if you would like to drop code change for the same? <br>
<br>
Thanks<br>
Rahul<br>
<br>
<br>
<br>
<br>
 <br>
<br>
On Tue, Nov 5, 2019 at 2:28 PM Tony Lee (李文富) <mailto:<a href="mailto:Tony.Lee@quantatw.com" target="_blank">Tony.Lee@quantatw.com</a>> wrote:<br>
Hi Rahul,<br>
 <br>
I'm wondering that why "REDFISH_NW_ETH0" is hardcode as "Managers/bmc/EthernetInterfaces/eth0/" in /data/variables.py<br>
<a href="https://github.com/openbmc/openbmc-test-automation/blob/master/data/variables.py#L155" rel="noreferrer" target="_blank">https://github.com/openbmc/openbmc-test-automation/blob/master/data/variables.py#L155</a><br>
Furthermore, the command "lan print" didn't specify the channel number in /ipmi/test_ipmi_network.robot.<br>
Therefore, the default will print information on the first found LAN channel. <br>
 <br>
Isn't it more reasonable to set the interface and the channel number as environment variables and give them default values?<br>
 <br>
Thanks<br>
Best Regards,<br>
Tony<br>
 <br>
</blockquote></div>