<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>Thanks for marking as WPI</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>"Armando M." <<a href="mailto:armamig@gmail.com">armamig@gmail.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, September 17, 2015 at 6:20 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] Effective Neutron<br>
</div>
<div><br>
</div>
<div>
<div>
<div dir="ltr">Hi fellow developers and reviewers,
<div><br>
</div>
<div>Some of you may have noticed that I put together patch [1] up for review.</div>
<div><br>
</div>
<div>The intention of this initiative is to capture/share 'pills of wisdom' when it comes to Neutron development and reviewing. In fact, there are a number of common patterns (or anti-patterns) that we keep stumbling on, and I came to the realization that if
 we all stopped for a second and captured those in writing for any newcomer (or veteran) to see, we would all benefit during code reviews and development, because we'd all know what to watch out for. The wishful thinking is that once this document reaches critical
 mass, we will all have learned how to avoid common mistakes and get our patches merged swiftly.</div>
<div><br>
</div>
<div>It is particularly aimed at the Neutron contributor and it is not meant to replace the wealth of information that is available on
<a href="http://doc.openstack.org">doc.openstack.org</a>, the wiki or the Internet. This is also not meant to be a cross-project effort, because let's face it...every project is different, and pills of wisdom in Neutron may as well be everyone's knowledge in
 Heat, etc.<br>
</div>
<div><br>
</div>
<div>I aim to add more material over the next couple of days, also with the help of some of you, so bear with me while the patch is in WIP.</div>
<div><br>
</div>
<div>Feedback welcome.</div>
<div><br>
</div>
<div>Many thanks,</div>
<div>Armando</div>
<div><br>
</div>
<div>[1] <a href="https://review.openstack.org/#/c/224419/">https://review.openstack.org/#/c/224419/</a></div>
</div>
</div>
</div>
</span>
</body>
</html>