<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 14, 2017 at 9:54 AM, Barak Korren <span dir="ltr"><<a href="mailto:bkorren@redhat.com" target="_blank">bkorren@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 14 February 2017 at 09:53, Martin Perina <<a href="mailto:mperina@redhat.com">mperina@redhat.com</a>> wrote:<br>
><br>
><br>
> On Mon, Feb 13, 2017 at 9:59 PM, Greg Sheremeta <<a href="mailto:gshereme@redhat.com">gshereme@redhat.com</a>> wrote:<br>
>><br>
>> On Mon, Feb 13, 2017 at 3:55 PM, Eyal Edri <<a href="mailto:eedri@redhat.com">eedri@redhat.com</a>> wrote:<br>
>>><br>
>>><br>
>>><br>
>>> On Mon, Feb 13, 2017 at 10:34 PM, Martin Perina <<a href="mailto:mperina@redhat.com">mperina@redhat.com</a>><br>
>>> wrote:<br>
>>>><br>
>>>> Hi,<br>
>>>><br>
>>>> why is this package not contained also in ovirt-master-snapshot<br>
>>>> repository [6]? Most of developers are using ovirt-master-snapshot, because<br>
>>>> this is the official repository for oVirt depelopers as mentioned in [7] and<br>
>>>> [8]. AFAIK there was not yet any official announcement that every developer<br>
>>>> should switch from ovirt-master-snapshot to ovirt-tested-master ...<br>
>>><br>
>>><br>
>>> I think we should make it official then for master, we've hit too many<br>
>>> issues in the past weeks due to this repository, that I don't want to see<br>
>>> new projects added to it.<br>
><br>
><br>
> You've run into issues, because<br>
><br>
> migration to the "new" system is not well prepared. I'm still a bit angry<br>
> that you have forced me to migrate all ovirt-engine-extensions* projects<br>
> into standard CI (which took me more than 2 days) by breaking existing build<br>
> jobs which worked fine until recent changes. And I had to do fast that<br>
> otherwise I won't be able to provide new build for upstream 4.1.0 async and<br>
> 4.1.1 builds ...<br>
<br>
I think there is some miss-communication here. The old '-snapshot'<br>
repo is causing issues because of the way it is built. We want people<br>
to move away from it. However, we did not intend to force anyone to<br>
move. It is still there, and the nightly jobs still update it with all<br>
the packages that have been in it so far.<br>
<br>
<a href="http://jenkins.ovirt.org/job/ovirt_master_publish-rpms_nightly/" rel="noreferrer" target="_blank">http://jenkins.ovirt.org/job/<wbr>ovirt_master_publish-rpms_<wbr>nightly/</a><br>
<br>
I really don't see how any of the recent changes forced you to change<br>
your project's own jobs, but it is a good thing if you did.<br></blockquote><div><br><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">Well, build jobs just started to fail and I discussed with either you or Gil and you just told me that why it's failing and the only solution is to move std CI.<br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">But let's leave this behind us, </div> <div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">ovirt-engine-extensions* move to std-CI and build jobs are working now correctly ...<br><br></div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> So I don't have an issue to move from ovirt-master-snapshot repos to<br>
> ovirt-tested-master repos, but please do that properly:<br>
><br>
> 1. Announce on mailing lists that every developer should switch at least a<br>
> week before the change<br>
<br>
We were planning, since all repos currently exist side-by-side we saw<br>
no rush to do that.<br>
<br>
> 2. Update all developer related documentation about this change<br>
<br>
Well, I'm not sure where such things are documented currently, but<br>
that is a reasonable request.<br>
<br>
> 2. Maintain both repos for a week and only after that turn off<br>
> ovirt-master-snapshots repos<br>
<br>
We did not turn it off yet. It is still there. The only thing that<br>
happened is that the new JS projects, that were never in that repo to<br>
begin with, chose to forgo publishing to it.<br></blockquote><div><br><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">Sure, that's why I asked to put this ovirt-js-dependencies package into ovirt-master-snapshot repo, which all developers are using ...<br></div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> Exported artifacts is not enough, please provide ovirt-release-tested-master<br>
> RPM which will include all necessary repositories same way as we currenlty<br>
> have in ovirt-release-master<br>
<br>
There is some subtle distinction here that needs to be well understood.<br>
Some repos are build to emulate an oVirt release, and all or most of<br>
the packages are collected in them. The 'tested' repo is an example of<br>
such a repo. For such repos it makes sense to create a<br>
'*-release-*.rpm'.<br></blockquote><div><br><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">So ovirt-release-master RPM will add repositories with needed packages to develop engine/VDSM. For engine development it means you have following packages:<br><br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">ovirt-master-snapshot:<br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default"> otopi<br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default"> ovirt-host-deploy<br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default"> ovirt-setup-lib<br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default"> ovirt-vmconsole<br><br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">ovirt-master-snapshot-static<br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default"> ovirt-engine-wildfly<br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default"> ovirt-engine-wildfly-overlay<br><br>ovirt-master-patternfly1-noarch-epel<br> patternfly1<br><br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">There are other packages, but those are needed only when developing relevant parts of engine (for example "unboundid-ldapsdk" needed for ovirt-engine-extension-aaa-ldap).<br><br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">So in my opinion if you want to switch to ovirt-latest-tested repo, we need to have available also those dependencies, that's why I think it's required to have ovirt-release-latest-tested RPM which will install all repos to develop and even install/run master oVirt from RPM.<br><br></div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
The 'exported-artifacts' repos are meant to allow "upstream" or<br>
"build-dependency" projects to have their own release stream that is<br>
independent of oVirt's release stream. For such repos it makes little<br>
sense to keep a '*-release-*.rpm'.<br>
<br>
All projects that have a 'build-articats job now have an<br>
'exported-artifacts' repo and their packages are submitted to OST so<br>
eventually also end up in the 'tested' repo. It is left to the<br>
consuming projects to pick which repo to use depending on how tightly<br>
are they coupled with consumed packages.<br>
<span class="gmail-HOEnZb"><font color="#888888"><br>
<br>
--<br>
Barak Korren<br>
<a href="mailto:bkorren@redhat.com">bkorren@redhat.com</a><br>
RHCE, RHCi, RHV-DevOps Team<br>
<a href="https://ifireball.wordpress.com/" rel="noreferrer" target="_blank">https://ifireball.wordpress.<wbr>com/</a><br>
</font></span></blockquote></div><br></div></div>