plugin: pass repos as arrays

Currently we were passing the repo information as a comma
separated string, which is fragile, since urls can contain
commas. Just transfer them as arrays of strings.

plugins/builder/osbuild.py

@@ -267,9 +267,8 @@ class OSBuildImage(BaseTaskHandler):
         return [Repository(repourl + "/$arch")]
 
     def make_repos_for_user(self, repos):
-        urls = repos.split(',')
+        self.logger.debug("user repo override: %s", str(repos))
-        self.logger.debug("user repo override: %s", urls)
+        return [Repository(r) for r in repos]
-        return [Repository(u.strip()) for u in urls]
 
     # pylint: disable=arguments-differ
     def handler(self, name, version, distro, image_types, target, arches, opts):

plugins/cli/osbuild.py

@@ -76,7 +76,7 @@ def handle_osbuild_image(options, session, argv):
         opts["release"] = args.release
 
     if args.repo:
-        opts["repo"] = ",".join(args.repo)
+        opts["repo"] = args.repo
 
     # Do some early checks to be able to give quick feedback
     check_target(session, target)

plugins/hub/osbuild.py

@@ -52,8 +52,11 @@ OSBUILD_IMAGE_SCHEMA = {
             "additionalProperties": False,
             "properties": {
                 "repo": {
-                    "type": "string",
+                    "type": "array",
-                    "description": "Repositories"
+                    "description": "Repositories",
+                    "items": {
+                        "type": "string"
+                    }
                 },
                 "release": {
                     "type": "string",

test/unit/test_builder.py

@@ -338,7 +338,7 @@ class TestBuilderPlugin(PluginTest):
                 ["image_type"],
                 "fedora-candidate",
                 ["x86_64"],
-                {"repo": ",".join(repos)}]
+                {"repo": repos}]
 
         url = self.plugin.DEFAULT_COMPOSER_URL
         composer = MockComposer(url, architectures=["x86_64"])