Re: [ovirt-devel] [ACTION REQUIRED] new dependency -- ovirt-js-dependencies (replaces PatternFly)

On 14 February 2017 at 09:53, Martin Perina <mperina(a)redhat.com&gt; wrote: > > > On Mon, Feb 13, 2017 at 9:59 PM, Greg Sheremeta <gshereme(a)redhat.com&gt; wrote: >> >> On Mon, Feb 13, 2017 at 3:55 PM, Eyal Edri <eedri(a)redhat.com&gt; wrote: >>> >>> >>> >>> On Mon, Feb 13, 2017 at 10:34 PM, Martin Perina <mperina(a)redhat.com&gt; >>> wrote: >>>> >>>> Hi, >>>> >>>> why is this package not contained also in ovirt-master-snapshot >>>> repository [6]? Most of developers are using ovirt-master-snapshot, because >>>> this is the official repository for oVirt depelopers as mentioned in [7] and >>>> [8]. AFAIK there was not yet any official announcement that every developer >>>> should switch from ovirt-master-snapshot to ovirt-tested-master ... >>> >>> >>> I think we should make it official then for master, we've hit too many >>> issues in the past weeks due to this repository, that I don't want to see >>> new projects added to it. > > > You've run into issues, because > > migration to the "new" system is not well prepared. I'm still a bit angry > that you have forced me to migrate all ovirt-engine-extensions* projects > into standard CI (which took me more than 2 days) by breaking existing build > jobs which worked fine until recent changes. And I had to do fast that > otherwise I won't be able to provide new build for upstream 4.1.0 async and > 4.1.1 builds ... I think there is some miss-communication here. The old '-snapshot' repo is causing issues because of the way it is built. We want people to move away from it. However, we did not intend to force anyone to move. It is still there, and the nightly jobs still update it with all the packages that have been in it so far. http://jenkins.ovirt.org/job/ovirt_master_publish-rpms_nightly/ I really don't see how any of the recent changes forced you to change your project's own jobs, but it is a good thing if you did.

> So I don't have an issue to move from ovirt-master-snapshot repos to > ovirt-tested-master repos, but please do that properly: > > 1. Announce on mailing lists that every developer should switch at least a > week before the change We were planning, since all repos currently exist side-by-side we saw no rush to do that. > 2. Update all developer related documentation about this change Well, I'm not sure where such things are documented currently, but that is a reasonable request. > 2. Maintain both repos for a week and only after that turn off > ovirt-master-snapshots repos We did not turn it off yet. It is still there. The only thing that happened is that the new JS projects, that were never in that repo to begin with, chose to forgo publishing to it.

> Exported artifacts is not enough, please provide ovirt-release-tested-master > RPM which will include all necessary repositories same way as we currenlty > have in ovirt-release-master There is some subtle distinction here that needs to be well understood. Some repos are build to emulate an oVirt release, and all or most of the packages are collected in them. The 'tested' repo is an example of such a repo. For such repos it makes sense to create a '*-release-*.rpm'.

The 'exported-artifacts' repos are meant to allow "upstream" or "build-dependency" projects to have their own release stream that is independent of oVirt's release stream. For such repos it makes little sense to keep a '*-release-*.rpm'. All projects that have a 'build-articats job now have an 'exported-artifacts' repo and their packages are submitted to OST so eventually also end up in the 'tested' repo. It is left to the consuming projects to pick which repo to use depending on how tightly are they coupled with consumed packages. -- Barak Korren bkorren(a)redhat.com RHCE, RHCi, RHV-DevOps Team https://ifireball.wordpress.com/