<div dir="ltr">Hi,<div><br></div><div>I disagree with Bogdan. We have the same case in OpenStack components.</div><div><br></div><div>1. As a compony we may have own patches on top of 'master'.</div><div>2. There is no way to use tags on upstream modules so it will be very hard to support it. If we need to deliver fix in previous release there won't be simple way to create tag, and cherry-pick the particular commit.</div><div><br></div><div>So I support Alex to continue the way we have right now.</div><div><br></div><div><br></div><div><br></div><div><br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr">--<br>
Best regards,<br>
Sergii Golovatiuk,<br>
Skype #golserge<br>
IRC #holser<br></div></div></div>
<br><div class="gmail_quote">On Thu, Jan 28, 2016 at 5:07 PM, Bogdan Dobrelya <span dir="ltr"><<a href="mailto:bdobrelia@mirantis.com" target="_blank">bdobrelia@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 22.01.2016 13:56, Bogdan Dobrelya wrote:<br>
</span><span class="">> On 22.01.2016 12:19, Matthew Mosesohn wrote:<br>
>> +1 for defaulting to upstream for CI. If we have a strong case where we<br>
>> need to make a patch in order to make unit tests pass, we could switch a<br>
>> module back to <a href="http://review.fuel-infra.org/puppet-modules/" rel="noreferrer" target="_blank">review.fuel-infra.org/puppet-modules/</a>..<br>
>> <<a href="http://review.fuel-infra.org/puppet-modules/." rel="noreferrer" target="_blank">http://review.fuel-infra.org/puppet-modules/.</a>.>.. with a FIXME and a LP<br>
>> bug ID so we can switch it back quickly once the upstream issue is<br>
>> resolved. As far as I know, we don't have to worry about that scenario.<br>
><br>
> Yes, exactly so. Switching upstream/downstream links of modules in the<br>
> Puppetfile back and forth can be done as often as we need it, with no<br>
> issues at all. With "free bonuses" though! Which is, firstly, it would<br>
> be easier to estimate upstream-to-downstream sync status by just looking<br>
> at the Puppetfile. Secondly, each time one's switching an upstream link<br>
> to a downstream, reviewers may treat this as a "tech dept growing<br>
> alarm!" and perhaps motivate patch authors to contribute changes<br>
> upstream instead (or *faster*) rather than just stashing them<br>
> accumulated downstream.<br>
<br>
</span>We have a disagreement for the patches [0], [1] related to this topic.<br>
My point is that we should use direct upstream hashtags/releases as<br>
early as possible. Nothing prevents us from switching to a downstream<br>
one in case of emergency.<br>
<br>
So donwstream maintaining efforts shall not be done from the very<br>
beginning, if possible to save infra/engineering resource for something<br>
more useful.<br>
<br>
[0] <a href="https://review.openstack.org/#/c/271217" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/271217</a><br>
[1] <a href="https://review.openstack.org/#/c/273036/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/273036/</a><br>
<div class="HOEnZb"><div class="h5"><br>
><br>
>><br>
>> -Matthew<br>
>><br>
>> On Fri, Jan 22, 2016 at 12:51 PM, Bogdan Dobrelya<br>
>> <<a href="mailto:bdobrelia@mirantis.com">bdobrelia@mirantis.com</a> <mailto:<a href="mailto:bdobrelia@mirantis.com">bdobrelia@mirantis.com</a>>> wrote:<br>
>><br>
>> Another point is to use upstream links for modules w/o downstream<br>
>> patches.<br>
>> I noticed we *always* put it to the deployment/Puppetfile [0] as<br>
>> "<a href="https://review.fuel-infra.org/puppet-modules/.." rel="noreferrer" target="_blank">https://review.fuel-infra.org/puppet-modules/..</a>.". Why should we?<br>
>> Let's just do the best to reuse upstream modules as is, eventually.<br>
>><br>
>> [0] <a href="https://github.com/openstack/fuel-library/blob/master/deployment/Puppetfile" rel="noreferrer" target="_blank">https://github.com/openstack/fuel-library/blob/master/deployment/Puppetfile</a><br>
>><br>
>> Regards,<br>
>> Bogdan Dobrelya.<br>
>> Irc #bogdando<br>
>><br>
>> 2016-01-21 11:09 GMT+01:00 Bartlomiej Piotrowski<br>
>> <<a href="mailto:bpiotrowski@mirantis.com">bpiotrowski@mirantis.com</a> <mailto:<a href="mailto:bpiotrowski@mirantis.com">bpiotrowski@mirantis.com</a>>>:<br>
>><br>
>> Let's drop 3.3 as well. 3.4 is oldschool enough for vintage lovers.<br>
>><br>
>> BP<br>
>><br>
>> On Thu, Jan 21, 2016 at 11:03 AM, Aleksandr Didenko<br>
>> <<a href="mailto:adidenko@mirantis.com">adidenko@mirantis.com</a> <mailto:<a href="mailto:adidenko@mirantis.com">adidenko@mirantis.com</a>>> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> > I also think 3.3 is the version that ships with 14.04.<br>
>><br>
>> 3.4.3 is shipped with Ubuntu-14.04. I think 3.4, 3.8 and 4<br>
>> should be enough.<br>
>><br>
>> Regards,<br>
>> Alex<br>
>><br>
>> On Wed, Jan 20, 2016 at 6:38 PM, Sergii Golovatiuk<br>
>> <<a href="mailto:sgolovatiuk@mirantis.com">sgolovatiuk@mirantis.com</a> <mailto:<a href="mailto:sgolovatiuk@mirantis.com">sgolovatiuk@mirantis.com</a>>><br>
>> wrote:<br>
>><br>
>> +1 for 3.3, 3.4, 3.8 and 4<br>
>><br>
>><br>
>> --<br>
>> Best regards,<br>
>> Sergii Golovatiuk,<br>
>> Skype #golserge<br>
>> IRC #holser<br>
>><br>
>> On Wed, Jan 20, 2016 at 6:12 PM, Alex Schultz<br>
>> <<a href="mailto:aschultz@mirantis.com">aschultz@mirantis.com</a> <mailto:<a href="mailto:aschultz@mirantis.com">aschultz@mirantis.com</a>>><br>
>> wrote:<br>
>><br>
>> On Wed, Jan 20, 2016 at 9:02 AM, Matthew Mosesohn<br>
>> <<a href="mailto:mmosesohn@mirantis.com">mmosesohn@mirantis.com</a><br>
>> <mailto:<a href="mailto:mmosesohn@mirantis.com">mmosesohn@mirantis.com</a>>> wrote:<br>
>> > Hi all,<br>
>> ><br>
>> > Unit tests on CI and gate bottleneck are really slowing down commit<br>
>> > progress. We recently had a meeting to discuss possible ways to improve<br>
>> > this, including symlinks, caching git repositories, etc, but one thing we<br>
>> > can do much faster is to simply disable 3.3-3.7 puppet jobs. We don't deploy<br>
>> > Fuel 9.0 (or 8.0) on earlier Puppet versions, so what value is there to the<br>
>> > checks? I propose we remove these tests, and hopefully we will see some<br>
>> > immediate relief.<br>
>> ><br>
>><br>
>> How about we reduce to 3.3, 3.4, 3.8 and 4? We<br>
>> would remove 3.6 and<br>
>> 3.7 which would reduce the number of jobs by a<br>
>> third The goal of<br>
>> keeping the others was to ensure that if/when we are<br>
>> able to install<br>
>> fuel-library without our version of puppet that a<br>
>> user could use<br>
>> whatever version their environment has. There were<br>
>> some changes<br>
>> between 3.3 and 3.4 (if I remember correctly) so we<br>
>> should keep<br>
>> checking that as it's also the oldest version<br>
>> supported by the<br>
>> upstream puppet openstack modules. I also think 3.3<br>
>> is the version<br>
>> that ships with 14.04. Additionally we used 3.4 in<br>
>> fuel 7 and below<br>
>> so we should keep those around.<br>
>><br>
>> -Alex<br>
>><br>
>> > Best Regards,<br>
>> > Matthew Mosesohn<br>
>> ><br>
>> ><br>
>> __________________________________________________________________________<br>
>> > OpenStack Development Mailing List (not for usage<br>
>> questions)<br>
>> > Unsubscribe:<br>
>> <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://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a>><br>
>> ><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>
>><br>
>> __________________________________________________________________________<br>
>> OpenStack Development Mailing List (not for usage<br>
>> questions)<br>
>> Unsubscribe:<br>
>> <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://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">http://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>
>><br>
>><br>
>> __________________________________________________________________________<br>
>> OpenStack Development Mailing List (not for usage questions)<br>
>> Unsubscribe:<br>
>> <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://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">http://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>
>><br>
>><br>
>> __________________________________________________________________________<br>
>> OpenStack Development Mailing List (not for usage questions)<br>
>> Unsubscribe:<br>
>> <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a> <<a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">http://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>
>><br>
>><br>
>> __________________________________________________________________________<br>
>> OpenStack Development Mailing List (not for usage questions)<br>
>> Unsubscribe:<br>
>> <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://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">http://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>
>><br>
>><br>
>> __________________________________________________________________________<br>
>> OpenStack Development Mailing List (not for usage questions)<br>
>> Unsubscribe:<br>
>> <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://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">http://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>
>><br>
>><br>
>><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>
><br>
><br>
<br>
<br>
--<br>
Best regards,<br>
Bogdan Dobrelya,<br>
Irc #bogdando<br>
<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>
</div></div></blockquote></div><br></div>