This was a workaround for odd nodes packages that
apt was not able to parse. See
infrastructure/apertis-docker-images@e954d2be

This is no longer required since it was fixed in apt:
https://salsa.debian.org/apt-team/apt/-/merge_requests/167



Having this workaround leads to other apt failures
to resolve dependencies.

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

apertis-pkg-merge-upstream-to-downstreams: ensure merge request is created against the right project

In case of a forked project, like we are doing with the test of the rebase,
the GitLab API creates by default the MR against the original project
instead of the forked project.
To ensure the merge request is created against the forked project, we use
the push option merge_request.target_project to overwrite the default
behavior and to create the merge request against the forked project.

See:
https://docs.gitlab.com/ee/topics/git/commit.html#push-options-for-merge-requests



Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

apertis-pkg-merge-upstream-to-downstreams: Don't print useless information while canceling pipelines

Printing these values only creates noise in the log.

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

The version of scan-copyrights from Trixie tries to parse the copyright
data to provide cleaner information. At the same time, this version
is more strict on the data use to fill debian/fill.copyright.blanks.yml,
and in case of non ascii characters the tool exits with error.

Since ci-license-scan fill the file with data, make sure that it only
adds ascii characters to the copyright.

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

When ci-license-scan tries to fill copyright information from d/copyright
it includes \n if they are present in copyright information. If later,
there is copyright data, the \n is placed in the middle creating an invalid
copyright file.

To overcome this issue, strip \n from the copyright.

Signed-off-by: Walter Lozano <walter.lozano@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

debian/apertis/local-gitlab-ci.yml is deprecated in favor of
debian/apertis/local-pre-source-build-gitlab-ci.yml to match
debian/apertis/local-post-source-build-gitlab-ci.yml

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

The location of this file has changed with:
infrastructure/apertis-infrastructure@41fed1b1



Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

A new temporary branch (proposed-package/apertis/v2020) is created
to generate debian/apertis/copyright.

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Actually pipelines on downstreams fail at this step because these branches
are missing the debian/apertis/copyright file. Instead we create a merge request,
thus the missing file will be generated with the job scan-licenses.

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

apertis-pkg-merge-upstream-to-downstreams: cancel pipelines created through the creation of a branch

Due to GitLab limitation (https://gitlab.com/gitlab-org/gitlab/-/issues/26422

),
it's currently impossible to avoid to trigger a pipeline when creating a
branch with the API. Since these useless pipelines are wasting
GitLab and OBS ressources (and devs' time), as a workaround for now
we can cancel them.

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Otherwse, we won't get the full list of pipelines and python will emit
a warning.

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

This is only a workaround for missing features in GitLab to disable
some unneeded pipelines.

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

This message is also required when a maintainer has to summarize
the remaining Apertis changes. Thus, always add this message to
save their time.

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Sudarshan CP authored 7 months ago and Dylan Aïssi committed 6 months ago

Signed-off-by: Sudarshan C P <sudarshan.cp2@in.bosch.com>

This is deprecated and will be removed in python 3.13.

/usr/bin/apertis-pkg-merge-updates:38: DeprecationWarning: pathlib.Path.__enter__() is deprecated and scheduled for removal in Python 3.13; Path objects as a context manager is a no-op
/usr/bin/apertis-pkg-merge-updates:75: DeprecationWarning: pathlib.Path.__enter__() is deprecated and scheduled for removal in Python 3.13; Path objects as a context manager is a no-op

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

apertis-pkg-merge-upstream-to-downstreams: Add link to the bug report about missing skip_ci option in REST API

Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>