Re: Draft Policy: Dependency Changes and CI

2017-01-21 Thread Sandro Knauß
Hey,

thanks for writing this down!
> this month, a heated exchange on kde-core-devel about a dep
> change breaking CI has moved us to draft a new policy to better
> handle situations like this in the future.

Hopefully.

The first two paraphrases are okay

For me the whole text sounds to bureaucratic and too many "you can do this 
after waiting that long" . Can't the whole process described like following:
* You should plan at least two weeks for a dep bump on CI.
* If a developer knows or think about bumping a dep (any dependency of this 
software that is not created by any other KDE group) she/he should get in 
contact with sysadmins beforehand as soon as possible to sort things out. You 
can reach sysadmins, by adding the "Sysadmins" group on Phabricator to 
reviewers review requests or file a separate sysadmins ticket on Phabricator.
* Sysadmins normally react within two weeks and can tell how long it will take 
to update the dep on CI. This can take some hours for very easy updates up to 
several weeks for deps, that affect more/nearly all packages f.ex. Qt.
* Under special circumstances (e.g. urgency prior to a freeze), exception 
requests should be raised in an established venue such as release-team or kde-
core-devel. In general, changes that fellow developers can't be convinced of 
are probably not a good idea.
* Keep in mind if you don't get in touch with sysadmins, than the CI will be 
RED. Also after multiple times not informing sysadmins beforehand and giving 
time to sort things out, sysadmins may decide to kick the project from CI 
completely.

Best Regards,

sandro



Re: CI Requirements - Lessons Not Learnt?

2017-01-21 Thread Kevin Kofler
Kevin Kofler wrote:
> The new xkbcommon requirement is also an issue for us in Fedora. The new
> xkbcommon 0.7.0 is only available in Rawhide and it is doubtful that it
> will ever be backported to Fedora 25 or 24. So, if we cannot get
> libxkbcommon updated, we will either be unable to provide your new Plasma
> releases for our stable releases, or we will be forced to add a patch to
> revert your dependency bump, as we already had to do for Qt in the past:
> http://pkgs.fedoraproject.org/cgit/rpms/qt5-qtbase.git/tree/qtbase-opensource-src-5.3.2-old_xkbcommon.patch?id=e26fd4ffda851bfe13d975547e218e16f72ce556
> (Yes, this reintroduced a bug. It was just not possible to fix that bug
> with the xkbcommon that was available in the Fedora 19 release. Fedora 20
> eventually got the xkbcommon update, so we were able to drop this patch
> there. We still had to patch Qt for the old xcb-xkb (which could not be
> updated due to a soname bump) though.)

For the record, xkbcommon was updated to 0.7.1 in Fedora 25 as a response to 
https://bugzilla.redhat.com/show_bug.cgi?id=1414493 , so this particular 
dependency is no longer an issue for this particular Fedora release.

Kevin Kofler