<div dir="ltr"><div><div><div><div></div><div>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. <div>Let us know if that sounds good to you.</div><div>Also let us know if you would like to drop code change for the same? <div><div><div><br></div><div>Thanks</div><div>Rahul</div></div></div></div><div><br></div><div><br><div><br></div><div><br></div><div> </div></div></div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Nov 5, 2019 at 2:28 PM 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">





<div lang="ZH-TW">
<div class="gmail-m_3386610193825965617WordSection1">
<p class="MsoNormal"><span lang="EN-US">Hi Rahul,<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">I'm wondering that why "REDFISH_NW_ETH0" is hardcode as "Managers/bmc/EthernetInterfaces/eth0/" in /data/variables.py<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><a href="https://github.com/openbmc/openbmc-test-automation/blob/master/data/variables.py#L155" target="_blank">https://github.com/openbmc/openbmc-test-automation/blob/master/data/variables.py#L155</a><u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">Furthermore, the command "lan print" didn't specify the channel number in /ipmi/test_ipmi_network.robot.<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">Therefore, the default will print information on the first found LAN channel.
<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">Isn't it more reasonable to set the interface and the channel number as environment variables and give them default values?<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">Thanks<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">Best Regards,<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US">Tony<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-US"><u></u> <u></u></span></p>
</div>
</div>

</blockquote></div>