<div dir="ltr">Fuelers<div><br></div><div>I am writing that we had a really sad incident - we noticed that after we merged upstream keystone module we lost modifications (<a href="https://review.openstack.org/#/q/Idfe4b54caa0d96a93e93bfff12d8b6216f83e2f1,n,z">Change-Id: Idfe4b54caa0d96a93e93bfff12d8b6216f83e2f1</a>) for memcached dogpile driver which are crucial for us. And here I can see 2 problems:</div><div><br></div><div>1) how can we ensure that we did not lose anything else?</div><div>2) how can we ensure that this will never happen again?</div><div><br></div><div>Sadly, it seems that the first question implies that we recheck all the upstream merge/adaptation commits by hand and check that we did not lose anything.</div><div><br></div><div>Regarding question number 2 we do already have established process for upstream code merge: <a href="http://docs.mirantis.com/fuel-dev/develop/module_structure.html#contributing-to-existing-fuel-library-modules">http://docs.mirantis.com/fuel-dev/develop/module_structure.html#contributing-to-existing-fuel-library-modules</a>. It seems that this process had  not been established when keystone code was reviewed. I see two ways here:<a href="/"></a></div><div><br></div><div>1) We should enforce code review workflow and specifically say that upstream merges can be accepted only after we have 2 '+2s' from core reviewers after they recheck that corresponding change does not introduce any regressions.</div><div>2) We should speed up development of some modular testing framework that will check that corresponding change affects only particular pieces. It seems much easier if we split deployment into stages (oh my, I am again talking about granular deployment feature) and each particular commit affects only one of the stages, so that we can see the difference and catch regressions eariler.</div><div><br></div><div><br></div><div><br></div><div><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Yours Faithfully,<br>Vladimir Kuklin,<br>Fuel Library Tech Lead,<br>Mirantis, Inc.<br>+7 (495) 640-49-04<br>+7 (926) 702-39-68<br>Skype kuklinvv<br>45bk3, Vorontsovskaya Str.<br>Moscow, Russia,<br><a href="http://www.mirantis.ru/" target="_blank">www.mirantis.com</a><br><a href="http://www.mirantis.ru/" target="_blank">www.mirantis.ru</a><br><a href="mailto:vkuklin@mirantis.com" target="_blank">vkuklin@mirantis.com</a></div></div>
</div></div>