<div dir="ltr">Hi Ivan<div><br></div><div>Thanks for bringing this up. Frankly, I think that we hurried a little bit by making our integration with upstream puppet manifests too continuous. I would suppose that we used a little bit different technique.</div><div><br></div><div>First of all, we need to have a set of stable Fuel commits against which the changes to upstream manifests should be tested. Could you please elaborate on whether we are doing this already?</div><div><br></div><div>Secondly, we need to have FUEL CI working with a set of stable commits of puppet openstack manifests which passed those upstream tests as we should not have too much moving parts or we will get into situation similar to requirements.txt updates when we have pypi updated with new library, e.g. oslo-serialization.</div><div><br></div><div>In this case, we will be able to do proper testing against frozen code-base for each piece thus avoiding such issues while retaining fair amount of integration with upstream puppet manifests for OpenStack.</div><div><br></div><div>So what do you think?</div><div><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 26, 2016 at 1:28 PM, Ivan Berezovskiy <span dir="ltr"><<a href="mailto:iberezovskiy@mirantis.com" target="_blank">iberezovskiy@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hello, <span style="font-size:12.8px">Fuelers!</span><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Yesterday we've faced an issue which came from puppet-neutron</span></div><div><span style="font-size:12.8px">module: <a href="https://bugs.launchpad.net/fuel/+bug/1549934" target="_blank">LP #1549934</a>. Fix was prepared very fast:</span></div><div><span style="font-size:12.8px"><a href="https://review.openstack.org/#/c/284882/" target="_blank">https://review.openstack.org/#/c/284882/</a> (thanks Sergey for this).</span></div><div><span style="font-size:12.8px">So, If CI is red on your patch </span><span style="font-size:12.8px">please re-base it on top of master.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Anyway, this issue affected a lot of patches and blocked some developers,</span></div><div><span style="font-size:12.8px">because BVT and neutron_smoke tests was also broken. We need to find</span></div><div><span style="font-size:12.8px">a way </span><span style="font-size:12.8px">how to minimize risks and affection of such changes on fuel-library.</span></div><div><span style="font-size:12.8px">We have jobs which monitors upstream patches:</span></div><div><span style="font-size:12.8px"><a href="https://ci.fuel-infra.org/view/puppet-openstack/" target="_blank">https://ci.fuel-infra.org/view/puppet-openstack/</a></span></div><div><span style="font-size:12.8px">Let's start to monitor those jobs on daily basis. We should have at least 1</span></div><div><span style="font-size:12.8px">(ideally 2 or more) engineers which are responsible for analysis of those</span></div><div><span style="font-size:12.8px">CI failures. If patch to puppet module is incorrect - we should review it</span></div><div><span style="font-size:12.8px">with explanation what is actually wrong. If patch is correct, but breaks</span></div><div><span style="font-size:12.8px">current Fuel CI, it means that problem is in our side and we should prepare</span></div><div><span style="font-size:12.8px">fuel-library adapt patch to fix the issue. Ideally, we should have an ability</span></div><div><span style="font-size:12.8px">to test </span><span style="font-size:12.8px">this fuel-library patch together with upstream one e.g. using 'Depends on'</span></div><div><span style="font-size:12.8px">in commit message.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Thoughts?<span class="HOEnZb"><font color="#888888"><br></font></span></span><span class="HOEnZb"><font color="#888888"><div><div><br></div>-- <br><div><div dir="ltr"><div><div dir="ltr"><font face="verdana, sans-serif">Thanks, Ivan Berezovskiy</font><div><font face="verdana, sans-serif">MOS Puppet Team Lead</font></div><div><font face="verdana, sans-serif">at <a href="https://www.mirantis.com/" target="_blank">Mirantis</a></font></div><div><font face="verdana, sans-serif"><br></font></div><div><font face="verdana, sans-serif">slack: iberezovskiy</font></div><div><font face="verdana, sans-serif">skype: bouhforever</font></div><div><font face="verdana, sans-serif">phone: + 7-960-343-42-46</font></div><div><font face="verdana, sans-serif"><br></font></div></div></div></div></div>
</div></font></span></div></div>
<br>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><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>35bk3, 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>
</div>