diff --git a/config_specs/repository_v2.yaml b/config_specs/repository_v2.yaml index 343946e..1b1d18e 100644 --- a/config_specs/repository_v2.yaml +++ b/config_specs/repository_v2.yaml @@ -66,9 +66,7 @@ data: # # (mandatory field) architectures: - - armv7hl - aarch64 - - i686 - ppc64le - s390x - x86_64 diff --git a/eln_repo_split_prototype.py b/eln_repo_split_prototype.py index 647ff04..1fec77b 100755 --- a/eln_repo_split_prototype.py +++ b/eln_repo_split_prototype.py @@ -87,7 +87,6 @@ def load_settings(): # "aarch64" # ], # "x86_64": [ -# "i686", # "x86_64" # ] # }, @@ -1092,4 +1091,4 @@ def main(): if __name__ == "__main__": - main() \ No newline at end of file + main() diff --git a/feedback_pipeline.py b/feedback_pipeline.py index 25bd9b3..be44c4f 100755 --- a/feedback_pipeline.py +++ b/feedback_pipeline.py @@ -171,7 +171,7 @@ def load_settings(): settings["max_subprocesses"] = 10 - settings["allowed_arches"] = ["armv7hl", "aarch64", "ppc64le", "s390x", "x86_64"] + settings["allowed_arches"] = ["aarch64", "ppc64le", "s390x", "x86_64"] settings["weird_packages_that_can_not_be_installed"] = ["glibc32"] @@ -458,44 +458,7 @@ def _load_config_workload(document_id, document, settings): config["package_placeholders"]["pkgs"] = {} config["package_placeholders"]["srpms"] = {} if "package_placeholders" in document["data"]: - - # So yeah, this is kind of awful but also brilliant. - # The old syntax of package placeholders was a dict, - # but the new one is a list. - # So I can be backwards compatible! - # - # The old format - if isinstance(document["data"]["package_placeholders"], dict): - for pkg_name, pkg_data in document["data"]["package_placeholders"].items(): - pkg_description = pkg_data.get("description", "Description not provided.") - pkg_requires = pkg_data.get("requires", []) - pkg_buildrequires = pkg_data.get("buildrequires", []) - limit_arches = pkg_data.get("limit_arches", None) - srpm = pkg_data.get("srpm", pkg_name) - - config["package_placeholders"]["pkgs"][pkg_name] = {} - config["package_placeholders"]["pkgs"][pkg_name]["name"] = pkg_name - config["package_placeholders"]["pkgs"][pkg_name]["description"] = pkg_description - config["package_placeholders"]["pkgs"][pkg_name]["requires"] = pkg_requires - config["package_placeholders"]["pkgs"][pkg_name]["limit_arches"] = limit_arches - config["package_placeholders"]["pkgs"][pkg_name]["srpm"] = srpm - - # Because the old format isn't great, it needs a srpm - # to be defined for every rpm, including the build requires. - # That can cause conflicts. - # So the best thing (I think) is to just take the first one and ignore - # the others. This is better than nothing. And people should move - # to the new format anyway. - if srpm not in config["package_placeholders"]["srpms"]: - config["package_placeholders"]["srpms"][srpm] = {} - config["package_placeholders"]["srpms"][srpm]["name"] = srpm - config["package_placeholders"]["srpms"][srpm]["buildrequires"] = pkg_buildrequires - config["package_placeholders"]["srpms"][srpm]["limit_arches"] = limit_arches - - - # - # The new format - elif isinstance(document["data"]["package_placeholders"], list): + if isinstance(document["data"]["package_placeholders"], list): for srpm in document["data"]["package_placeholders"]: srpm_name = srpm["srpm_name"] if not srpm_name: diff --git a/test_config_files.py b/test_config_files.py index f9aebf6..57e8270 100755 --- a/test_config_files.py +++ b/test_config_files.py @@ -6,7 +6,7 @@ def create_mock_settings(): settings = {} settings["configs"] = "input/configs" settings["strict"] = True - settings["allowed_arches"] = ["armv7hl","aarch64","ppc64le","s390x","x86_64"] + settings["allowed_arches"] = ["aarch64","ppc64le","s390x","x86_64"] return settings