Skip to content
Snippets Groups Projects
Commit 6595da35 authored by Dylan Aïssi's avatar Dylan Aïssi
Browse files

apertis-pkg-merge-upstream-to-downstreams: ensure merge request is created...

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: default avatarDylan Aïssi <dylan.aissi@collabora.com>
parent 82ea8537
No related branches found
No related tags found
No related merge requests found
......@@ -183,6 +183,24 @@ def ensure_downstream_branch(project_url, downstream, dry_run):
raise
def get_path_with_namespace(project_url):
url = urllib.parse.urlsplit(project_url)
project_id = urllib.parse.quote(removesuffix(url.path.strip("/"), ".git"), safe="")
url = url._replace(
path=f"/api/v4/projects/{project_id}",
)
project_metadata = url.geturl()
try:
res = urllib.request.urlopen(project_metadata)
json_res = json.load(res)
path_with_namespace = json_res["path_with_namespace"]
except urllib.error.HTTPError as e:
print("ERROR:", e.read().decode())
raise
return path_with_namespace
def push_merge_request(
project_url, proposed_branch, upstream, downstream, auto_merge="", dry_run=False
):
......@@ -213,8 +231,11 @@ def push_merge_request(
ensure_downstream_branch(project_url, downstream, dry_run)
project_path = get_path_with_namespace(project_url)
print(
f"Create merge request from '{proposed_branch}' to '{downstream}'", flush=True
f"Create merge request from '{proposed_branch}' to '{project_path}/{downstream}'",
flush=True,
)
git_push_custom(
"-o",
......@@ -224,6 +245,8 @@ def push_merge_request(
"-o",
f"merge_request.target={downstream}",
"-o",
f"merge_request.target_project={project_path}",
"-o",
f"merge_request.title={title}",
project_url,
f"{proposed_branch}:{proposed_branch}",
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment