<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</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>Hi,</div>
<div>Changed the subject so that it may draw a little attention.</div>
<div>There were 2 patches approved that kind of break the API (in my humble opinion):</div>
<div><a href="https://review.openstack.org/#/c/154921">https://review.openstack.org/#/c/154921</a>/ and <a href="https://review.openstack.org/#/c/158420">https://review.openstack.org/#/c/158420</a>/ </div>
<div>In both of these two new fields were added to the base attributes – mtu and vlan_transparency</div>
<div>Reverts for them are:</div>
<div><a href="https://review.openstack.org/165801">https://review.openstack.org/165801</a> (mtu) and <a href="https://review.openstack.org/165776">https://review.openstack.org/165776</a> (vlan transparency).</div>
<div>In my opinion these should be added as separate extensions.</div>
<div>Thanks</div>
<div>Gary</div>
<div><br>
</div>
<span id="OLK_SRC_BODY_SECTION">
<div style="font-family:Calibri; font-size:11pt; text-align:left; color:black; BORDER-BOTTOM: medium none; BORDER-LEFT: medium none; PADDING-BOTTOM: 0in; PADDING-LEFT: 0in; PADDING-RIGHT: 0in; BORDER-TOP: #b5c4df 1pt solid; BORDER-RIGHT: medium none; PADDING-TOP: 3pt">
<span style="font-weight:bold">From: </span>Gary Kotton <<a href="mailto:gkotton@vmware.com">gkotton@vmware.com</a>><br>
<span style="font-weight:bold">Reply-To: </span>OpenStack List <<a href="mailto:openstack-dev@lists.openstack.org">openstack-dev@lists.openstack.org</a>><br>
<span style="font-weight:bold">Date: </span>Thursday, March 19, 2015 at 2:32 PM<br>
<span style="font-weight:bold">To: </span>OpenStack List <<a href="mailto:openstack-dev@lists.openstack.org">openstack-dev@lists.openstack.org</a>><br>
<span style="font-weight:bold">Subject: </span>Re: [openstack-dev] [Neutron] VLAN transparency support<br>
</div>
<div><br>
</div>
<div>
<div 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>Hi,</div>
<div>This patch has the same addition too - <a href="https://review.openstack.org/#/c/154921">https://review.openstack.org/#/c/154921</a>/. We should also revert that one.</div>
<div>Thanks</div>
<div>Gary</div>
<div><br>
</div>
<span id="OLK_SRC_BODY_SECTION">
<div style="font-family:Calibri; font-size:11pt; text-align:left; color:black; BORDER-BOTTOM: medium none; BORDER-LEFT: medium none; PADDING-BOTTOM: 0in; PADDING-LEFT: 0in; PADDING-RIGHT: 0in; BORDER-TOP: #b5c4df 1pt solid; BORDER-RIGHT: medium none; PADDING-TOP: 3pt">
<span style="font-weight:bold">From: </span>Gary Kotton <<a href="mailto:gkotton@vmware.com">gkotton@vmware.com</a>><br>
<span style="font-weight:bold">Reply-To: </span>OpenStack List <<a href="mailto:openstack-dev@lists.openstack.org">openstack-dev@lists.openstack.org</a>><br>
<span style="font-weight:bold">Date: </span>Thursday, March 19, 2015 at 1:14 PM<br>
<span style="font-weight:bold">To: </span>OpenStack List <<a href="mailto:openstack-dev@lists.openstack.org">openstack-dev@lists.openstack.org</a>><br>
<span style="font-weight:bold">Subject: </span>[openstack-dev] [Neutron] VLAN transparency support<br>
</div>
<div><br>
</div>
<div>
<div 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>Hi,</div>
<div>It appears that <a href="https://review.openstack.org/#/c/158420">https://review.openstack.org/#/c/158420</a>/ update the base attributes for the networks. Is there any reason why this was not added as a separate extension like all others.</div>
<div>I do not think that this is the correct way to go and we should do this as all other extensions have been maintained. I have posted a revert (<a href="https://review.openstack.org/#/c/165776">https://review.openstack.org/#/c/165776</a>/) – please feel
 free to knack if it is invalid.</div>
<div>Thanks</div>
<div>Gary</div>
</div>
</div>
</span></div>
</div>
</span>
</body>
</html>