[dpdk-ci] OBS pull request in pw-ci
aconole at redhat.com
Mon Jul 6 19:31:44 CEST 2020
Juraj Linkeš <juraj.linkes at pantheon.tech> writes:
> Hi folks,
> I've submitted a PR with OBS changes to pw-ci: https://github.com/orgcandman/pw-ci/pull/12
Cool! I'll take a look. I was on PTO, so didn't get a chance yet.
> I have a couple of points/questions:
> · The scripts are using https://build.opensuse.org/package/show/home:bluca:dpdk/dpdk to fork the debian and
> redhat spec and control files.
Okay. A quick look shows that it's configurable, though. Should be
okay. I refer to:
> · We should create a community OBS project (that the community can manage) where the forked packages will be
> created and where the build will run (OBS_TARGET_PROJECT in 3rd-party/dpdk/jenkins-rc)
> o The project should be configured according to what we want to build - distros/architectures. Can be done via OBS GUI.
> · Only after all patches have been submitted to a series branch does the DPDK Jenkins job upload and configure an OBS
> package. This means that we'll have a build just for the whole series, not for each patch. Not sure if this is a problem.
The flip side is the amount of time to build. We'd prefer to build just
one at a time.
> · One thing that doesn't make sense to me is that the DPDK Jenkins job builds dpdk after each patch - is this needed,
> since Travis is going to do the same thing anyway?
Travis doesn't. The ovsrobot travis build is configured to cancel a
build on a branch if a new one comes. That way we only build the
latest. Builds on travis take 40+ min to complete, so if a 20-patch
series came in, we wouldn't be done building it for more than half a
day. Then if a v2+ is posted in rapid succession, we lost all this
time. So for now, we only build the absolute last patch in the series
Also, the build server we run the robot on doesn't do the builds anymore
because it also got overloaded - so meson and ninja are not the actual
meson/ninja but just dummy commands. :-/ Maybe I should re-enable and
see if things improved.
> · I've added some more comments to the PR itself.
More information about the ci