<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class=""><br class=""><blockquote type="cite" class="">Thanks for the clarification. Do you think that there are cases where<br class="">the value for PHYSICAL_NETWORK and PUBLIC_PHYSICAL_NETWORK will be different? <br class=""></blockquote><div class="">No, I cannot find the cases. I think it’s traditional something.</div><br class=""><blockquote type="cite" class="">Perhaps we can change the default value for PUBLIC_PHYSICAL_NETWORK from<br class="">"public" to just pulling in what PHYSICAL_NETWORK is set to, if it is<br class="">set?</blockquote></div><div class="">Do you expect anything like the following?</div><div class=""><br class=""></div><div class="">PUBLIC_PHYSICAL_NETWORK=${PUBLIC_PHYSICAL_NETWORK:-$PHYSICAL_NETWORK}</div><div class="">PUBLIC_PHYSICAL_NETWORK=${PUBLIC_PHYSICAL_NETWORK:-public}</div><div class=""><br class=""></div><div class="">I guess it’s a simple solution even though it’s not essential.</div><div class="">The issue probably is caused by two points.</div><div class="">1) PHYSICAL_NETWORK and PUBLIC_PHYSICAL_NETWORK confuse users.</div><div class="">2) Devstack lacks the condition for avoiding the issue in <a href="https://github.com/openstack-dev/devstack/blob/master/lib/neutron_plugins/openvswitch_agent#L58" class="">https://github.com/openstack-dev/devstack/blob/master/lib/neutron_plugins/openvswitch_agent#L58</a></div><div class=""><br class=""></div><div class="">If I solve the issue, I would unify the two parameters.</div><div class="">I would leave the following for backward compatibility.</div><div class="">PUBLIC_PHYSICAL_NETWORK=${PUBLIC_PHYSICAL_NETWORK:-$PHYSICAL_NETWORK}</div><div class="">and</div><div class="">change PHYSICAL_NETWORK into PUBLIC_PHYSICAL_NETWORK in all the code.</div><br class=""><div apple-content-edited="true" class="">
<div style="color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; " class=""><div style="color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; " class=""><div style="color: rgb(0, 0, 0); font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; " class=""><div class="">---------------------------------------------<br class="">Hirofumi Ichihara<br class=""></div></div></div></div></div></body></html>