fixing the type for new document

otc_metadata/services.py

@@ -209,22 +209,3 @@ class Services(object):
                 doc_struct[srv["service_category"]].append(srv_res)
 
         return dict(categories=doc_struct)
-
-    def get_service_with_docs_by_service_type(self, service_type):
-        """Retrieve service and service docs by service_type
-
-        :param str service_type: Filter by service_type
-        """
-        res = dict()
-        res['service'] = {}
-        docs = []
-        services = self._service_data
-        for doc in services['documents']:
-            if doc['service_type'] == service_type:
-                docs.append(doc)
-        res['documents'] = docs
-        for service in services['services']:
-            if service['service_type'] == service_type:
-                res['service'] = service
-                break
-        return res

otc_metadata/templates/tox.ini.j2

@@ -35,7 +35,7 @@ commands =
   {[testenv:json-{{ doc.type }}]commands}
 {%- endfor %}
 
-[testenv:pdf-docs]
+[testenv:docs-pdf]
 deps = -r{toxinidir}/doc/requirements.txt
 allowlist_externals =
   rm

tools-requirements.txt

@@ -2,5 +2,3 @@ GitPython
 ruamel.yaml
 requests
 jinja2
-dirsync
-cookiecutter

tools/sync_doc_repo.py

@@ -18,9 +18,9 @@ import argparse
 import logging
 import os
 import pathlib
+import shutil
 import subprocess
 import warnings
-from dirsync import sync
 
 from git import Repo
 from git import SymbolicReference
@@ -111,21 +111,13 @@ def process_repositories(args, service):
         new_branch.set_tracking_branch(remote_ref)
         new_branch.checkout()
 
-        source_path = pathlib.Path(copy_from, doc["rst_location"])
-        target_path = pathlib.Path(copy_to, doc["rst_location"])
-        sync(
-            source_path,
-            target_path,
-            'sync',
-            purge=True,
-            create=True,
-            content=True,
-            ignore=['conf.py']
+        shutil.copytree(
+            pathlib.Path(copy_from, doc["rst_location"]),
+            pathlib.Path(copy_to, doc["rst_location"]),
+            ignore=lambda a, b: ["conf.py"],
+            dirs_exist_ok=True,
         )
         repo_to.index.add([doc["rst_location"]])
-
-        for obj in repo_to.index.diff(None).iter_change_type('D'):
-            repo_to.index.remove([obj.b_path])
         if len(repo_to.index.diff("HEAD")) == 0:
             # Nothing to commit
             logging.debug("No changes.")

tools/update_zuul_project_configs.py

@@ -130,7 +130,6 @@ def process_repositories(args, service):
         zuul_templates = None
         zuul_jobs = dict()
         zuul_new_jobs = list()
-        zuul_vars = dict()
         zuul_config_updated = False
         for item in zuul_config:
             if "project" in item.keys():
@@ -138,11 +137,10 @@ def process_repositories(args, service):
                 zuul_templates = project.setdefault("templates", [])
                 if not zuul_templates:
                     zuul_templates = []
-                zuul_vars = project.setdefault("vars", {})
             elif "job" in item.keys():
                 job = item["job"]
                 zuul_jobs[job["name"]] = job
-        logging.debug(f"Existing jobs {zuul_jobs}")
+        print(f"Existing jobs {zuul_jobs}")
         if "helpcenter-base-jobs" not in zuul_templates:
             zuul_templates.append("helpcenter-base-jobs")
             zuul_config_updated = True
@@ -150,7 +148,6 @@ def process_repositories(args, service):
         job_suffix = (
             "-hc-int-jobs" if args.environment == "internal" else "-hc-jobs"
         )
-        sphinx_pdf_files = zuul_vars.setdefault('sphinx_pdf_files', [])
         for doc in data.docs_by_service_type(service["service_type"]):
             logging.debug(f"Analyzing document {doc}")
             if not doc.get("type"):
@@ -159,12 +156,6 @@ def process_repositories(args, service):
                 doc_type = "dev-guide"
             else:
                 doc_type = doc["type"]
-            # Collect all PDF files into sphinx_pdf_files var
-            pdf_name = doc.get('pdf_name')
-            if pdf_name and f"{pdf_name}.pdf" not in sphinx_pdf_files:
-                sphinx_pdf_files.append(f"{pdf_name}.pdf")
-                zuul_config_updated = True
-
             template_name = f"{doc_type}{job_suffix}"
             if doc_type in ["api-ref", "umn", "dev-guide"]:
                 if template_name not in zuul_templates:
@@ -199,7 +190,6 @@ def process_repositories(args, service):
                 if "project" in item.keys():
                     project = item["project"]
                     project["templates"] = zuul_templates
-                    project["vars"] = zuul_vars
                     # Ensure new jobs are in check
                     if len(zuul_new_jobs) > 0:
                         project.setdefault(