Am 2017-01-06 05:57, schrieb Ben Cooksley:
On Fri, Jan 6, 2017 at 12:38 AM, Martin Gräßlin <mgraess...@kde.org>
wrote:
Am 2017-01-05 11:20, schrieb Ben Cooksley:
On Thu, Jan 5, 2017 at 10:28 PM, Martin Gräßlin
<pri...@martin-graesslin.com> wrote:
Am 2017-01-05 09:44, schrieb Ben Cooksley:
Hi all,
It seems that my previous vocal complaints about system level /
serious impact dependency bumps on the CI system have gone
completely
unnoticed by (some) members of our Community.
This was demonstrated earlier this week when components of Plasma
bumped their version requirements for XKBCommon and Appstream-Qt -
without even a thought about notifying Sysadmin or checking which
version the CI had, until their builds broke.
Neither of these is easy to fix at this stage, as the system base
is
now too old to receive updates such as these. Base upgrades require
a
full rebuild of everything on the CI system, and usually involve
significant additional churn and is a process that must be done
roughly twice a year, depending on dependency bump demands.
Does anyone have any suggestions as to how we may avoid this in the
future?
I have a few questions here:
1) Where is this requirement to check with sysadmins codified? So
far I
was
only aware of dependency freeze.
It's been codified since the PIM Qt 5.6 / WebEngine debacle, where
Sysadmin had to rush delivery of Qt 5.6 to the CI system because the
whole of PIM broke when they started using QtWebEngine. That was
around March/April 2016, my mail archives can't seem to find the
exact
thread though.
I'm sorry Ben, but I fear "sending out a mail about an issue with PIM"
doesn't
qualify as codifying it. Given what we have it looks like this did not
reach
the
target audience. And neither will this thread.
Uhm, it was far more than a single email. It was quite the thread, and
if memory serves was on at least one of the mailing lists this thread
was on.
One of the key things that came out of that thread was to ask for
things to be bumped well in advance if a newer version is needed.
I'm disappointed that you think that email threads have insufficient
reach given it's one of our communities principal means of
communication.
Email threads don't work to codify such requirements. What we need is
something like an "announce new dependency to sysadmin freeze" prior to
the dependency freeze in the release schedule. That's what I mean with
codifying it. We need to have it in a way where devs actually check.
It needs to be part of the process. An old email thread cannot be part
of
the process.
This needs to change the process, the way KDE develops software. It
needs to
be
listed in the release schedule (is not, I checked), maybe reviews need
to be
acked by release managers, etc.
It seems to be part of the process for many others already, so not
sure what needs to change. The gpgme transition went quite well for
PIM, and other applications developers have reached out and asked
about version upgrades to various libraries which were in their area
of interest which we have sorted out easily enough.
See above. Part of release schedule.
2) How can we easily check what build.kde.org has? Looking at cmake
output
is not a sufficient way as it gives me wrong information
If CMake is outputting wrong information, then your CMakeLists.txt
can't make the appropriate decisions as to whether the available
version is suitable, so i'd say you've got bigger problems here that
need to be addressed first.
Cmake feature summary says: "required version >= 0.5" and that's for
all
found
depeendencies. Unfortunately no information at all in the feature
summary
about
the actual version.
Quoting the KWin CMake log...
15:08:41 -- Found Wayland_Client:
/usr/lib/x86_64-linux-gnu/libwayland-client.so (found version
"1.12.90")
15:08:41 -- Found Wayland_Cursor:
/usr/lib/x86_64-linux-gnu/libwayland-cursor.so (found version
"1.12.90")
15:08:42 -- Found Wayland_Egl:
/usr/lib/x86_64-linux-gnu/libwayland-egl.so (found version "11.0.2")
15:08:42 -- Found Wayland:
/usr/lib/x86_64-linux-gnu/libwayland-client.so;/usr/lib/x86_64-linux-gnu/libwayland-cursor.so;/usr/lib/x86_64-linux-gnu/libwayland-egl.so
(found suitable version "1.12.90", minimum required is "1.2") found
components: Cursor Egl
15:08:42 -- Could NOT find XKB: Found unsuitable version "0.5.0", but
required is at least "0.7.0" (found
/usr/lib/x86_64-linux-gnu/libxkbcommon.so)
15:08:42 -- Found Libinput: /usr/lib/x86_64-linux-gnu/libinput.so
(found suitable version "1.5.2", minimum required is "1.5")
15:08:42 -- Found UDev: /usr/include
15:08:42 -- Found Libdrm: /usr/lib/x86_64-linux-gnu/libdrm.so (found
suitable version "2.4.64", minimum required is "2.4.62")
15:08:42 -- Found gbm: /usr/lib/x86_64-linux-gnu/libgbm.so (found
version "11.0.2")
The summary may not be of any use, but the rest of the output has the
information you're after.
In any case, you can see the Docker log of the container being
generated at https://build.kde.org/job/create_ubuntu_slave-ange/
and where do I find this information if I would not have asked in a
mail?
This is very related to properly codifying and documenting such
requirements.
Pretty sure i've pointed it out to you (among others) on IRC more than
once.
It's true that there isn't written documentation for the CI system as
such, but it's a highly organic system subject to regular change - so
documentation such as that can easily get outdated.
Well yes, I think it needs proper documentation in the
"how to update a dependency" documentation.
You cannot tell people "don't introduce new dependencies", without
telling
them
how to check.
3) What should we do when build.kde.org does not have the
requirement?
You have to file a Sysadmin ticket, also tagging the project
'build.kde.org' at the same time.
And then? What's the process then? How long do we have to expect this
to go?
Would it allow to block a finished feature or an important bug fix?
Would we
be
forced to write ifdef hackery?
Sorry, but I'm not thrilled by this process.
What matters to me is getting out good software to our users. And for
that I
have
a hard requirement I have to hit: dependency freeze.
The process would depend on what is needed.
If it is something we simply don't have installed, or a suitable
version is available in backports or another usable repository then
it's fairly trivial and the turnaround would probably be a day or two.
That's something you can accelerate by providing the names of the
needed packages for our base system(s), along with details of PPAs,
etc which may be needed. If it's something like Qt on the other hand,
that is far more involved and could easily span to a few weeks.
Note that you should probably be providing notice to the community at
large who also build your software anyway, so that developers can get
their systems ready for the dependency bump. Otherwise you're breaking
their workflows as well. By all accounts you broke the builds of other
Plasma developers, which indicates to me that this dependency bump
wasn't announced very well at all.
No Plasma developer complained to me. No Plasma developer raised
concerns
in the phab request. The complaints were from the larger KDE community.
Anyway what I was after is: how much in advance of the dependency freeze
do
we have to notify sysadmins? We need to codify this in our release
schedule.
We devs also need planning.
That's part of what I'm after in my replies to this thread: I want a
well
defined process which devs can follow. A process on how to get new
dependencies
into their next version of the system. Currently our process specifies
only a
dependency freeze which seems not sufficient.
Cheers
Martin
