Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add script to migrate existing build results to Pulp #3509

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
119 changes: 119 additions & 0 deletions backend/run/copr-change-storage
Original file line number Diff line number Diff line change
@@ -0,0 +1,119 @@
#! /usr/bin/python3
Fixed Show fixed Hide fixed

"""
Migrate existing build results for a given project and all of its CoprDirs
from one storage (Copr backend) to another (Pulp).
"""

import os
import sys
import argparse
import logging
from copr_common.log import setup_script_logger
from copr_backend.helpers import BackendConfigReader
from copr_backend.storage import PulpStorage


STORAGES = ["backend", "pulp"]

log = logging.getLogger(__name__)
setup_script_logger(log, "/var/log/copr-backend/change-storage.log")


def get_arg_parser():
Fixed Show fixed Hide fixed
"""
CLI argument parser
"""
parser = argparse.ArgumentParser()
parser.add_argument(
"--src",
required=True,
choices=STORAGES,
help="The source storage",
)
parser.add_argument(
"--dst",
required=True,
choices=STORAGES,
help="The destination storage",
)
parser.add_argument(
"--project",
required=True,
help="Full name of the project that is to be migrated",
)
parser.add_argument(
"--delete",
action="store_true",
default=False,
help="After migrating the data, remove it from the old storage",
)
return parser


def main():
Fixed Show fixed Hide fixed
"""
The main function
"""
parser = get_arg_parser()
args = parser.parse_args()

if args.src == args.dst:
print("The source and destination storage is the same, nothing to do.")
return

if args.src == "pulp":
print("Migration from pulp to somewhere else is not supported")
sys.exit(1)

if args.delete:
print("Data removal is not supported yet")
sys.exit(1)

config_file = "/etc/copr/copr-be.conf"
config = BackendConfigReader(config_file).read()
owner, project = args.project.split("/")
ownerdir = os.path.join(config.destdir, owner)

for subproject in os.listdir(ownerdir):
if not (subproject == project or subproject.startswith(project + ":")):
continue

coprdir = os.path.join(ownerdir, subproject)
for chroot in os.listdir(coprdir):
if chroot == "srpm-builds":
continue

chrootdir = os.path.join(coprdir, chroot)
if not os.path.isdir(chrootdir):
continue

appstream = None
devel = None
storage = PulpStorage(
owner, subproject, appstream, devel, config, log)

for builddir in os.listdir(chrootdir):
resultdir = os.path.join(chrootdir, builddir)
if not os.path.isdir(resultdir):
continue

full_name = "{0}/{1}".format(owner, subproject)
result = storage.init_project(full_name, chroot)
if not result:
log.error("Failed to initialize project: %s", resultdir)
break

# We cannot check return code here
storage.upload_build_results(chroot, resultdir, None)

result = storage.publish_repository(chroot)
if not result:
log.error("Failed to publish a repository: %s", resultdir)
break

log.info("OK: %s", resultdir)


if __name__ == "__main__":
main()
11 changes: 8 additions & 3 deletions frontend/coprs_frontend/commands/change_storage.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@
configure the storage type for a given project and while doing so, it makes sure
DNF repositories for the project are created.

Existing builds are not migrated from one storage to another. This may be an
useful feature but it is not implemented yet.
To migrate existing build results for a given project and all of its CoprDirs,
run also `copr-change-storage` script on backend.
"""

import sys
Expand Down Expand Up @@ -44,4 +44,9 @@ def change_storage(fullname, storage):
db.session.commit()
print("Configured storage for {0} to {1}".format(copr.full_name, storage))
print("Submitted action to create repositories: {0}".format(action.id))
print("Existing builds not migrated (not implemented yet).")
print("To migrate existing build results for this project and all of its "
"CoprDirs, run also this command on backend:")

cmd = "sudo -u copr copr-change-storage --src backend --dst pulp "
cmd += "--project {0}".format(fullname)
print(" {0}".format(cmd))