
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">

</head>

<body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; color: rgb(0, 0, 0); font-size: 14px; font-family: Calibri, sans-serif;">

<div><br>

</div>

<div>

<div style="font-family: Consolas, monospace; font-size: 12px;">Hi All,</div>

<div style="font-family: Consolas, monospace; font-size: 12px;"><br>

</div>

<div style="font-family: Consolas, monospace; font-size: 12px;">It was brought to my attention that the recent changes with the vnic_type</div>

<div style="font-family: Consolas, monospace; font-size: 12px;">possibly should not include the colon in the property value.</div>

<div style="font-family: Consolas, monospace; font-size: 12px;">In the earlier versions of the review the colon was not in the property,</div>

<div style="font-family: Consolas, monospace; font-size: 12px;">and was appended later.</div>

</div>

<div style="font-family: Consolas, monospace; font-size: 12px;"><br>

</div>

<div><a href="https://review.openstack.org/#/c/129353/4..5/heat/engine/resources/neutron/port.py">https://review.openstack.org/#/c/129353/4..5/heat/engine/resources/neutron/port.py</a></div>

<div><br>

</div>

<div>

<div style="font-family: Consolas, monospace; font-size: 12px;">let me know if this should go back to version 4, and I will open a bug</div>

<div style="font-family: Consolas, monospace; font-size: 12px;">and fix it.</div>

<div style="font-family: Consolas, monospace; font-size: 12px;"><br>

</div>

<div style="font-family: Consolas, monospace; font-size: 12px;">Thanks - Rob</div>

</div>

</body>

</html>