<div dir="ltr">Hello, <span class="" 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">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/">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/">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?<br></span><div><div><br></div>-- <br><div class="gmail_signature"><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></div></div>