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

[Barak Korren]

On 14 February 2017 at 09:53, Martin Perina <mperina at redhat.com> wrote:
>
>
> On Mon, Feb 13, 2017 at 9:59 PM, Greg Sheremeta <gshereme at redhat.com> wrote:
>>
>> On Mon, Feb 13, 2017 at 3:55 PM, Eyal Edri <eedri at redhat.com> wrote:
>>>
>>>
>>>
>>> On Mon, Feb 13, 2017 at 10:34 PM, Martin Perina <mperina at redhat.com>
>>> 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 at redhat.com
RHCE, RHCi, RHV-DevOps Team
https://ifireball.wordpress.com/