← Back to team overview

launchpad-reviewers team mailing list archive

[Merge] ~cjwatson/launchpad:lpci-rename into launchpad:master

 

Colin Watson has proposed merging ~cjwatson/launchpad:lpci-rename into launchpad:master.

Commit message:
Follow rename of lpcraft to lpci

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/441348

This has no externally-visible effect; it just updates internal names.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad:lpci-rename into launchpad:master.
diff --git a/lib/lp/code/configure.zcml b/lib/lp/code/configure.zcml
index 48bde56..eaf4020 100644
--- a/lib/lp/code/configure.zcml
+++ b/lib/lp/code/configure.zcml
@@ -1280,9 +1280,9 @@
     for="lp.code.interfaces.sourcepackagerecipe.ISourcePackageRecipe zope.lifecycleevent.interfaces.IObjectModifiedEvent"
     handler="lp.code.model.sourcepackagerecipe.recipe_modified"/>
 
-  <!-- LPCraftConfiguration -->
-  <class class="lp.code.model.lpcraft.LPCraftConfiguration">
-    <allow interface="lp.code.interfaces.lpcraft.ILPCraftConfiguration" />
+  <!-- LPCIConfiguration -->
+  <class class="lp.code.model.lpci.LPCIConfiguration">
+    <allow interface="lp.code.interfaces.lpci.ILPCIConfiguration" />
   </class>
 
   <!-- CIBuild -->
diff --git a/lib/lp/code/interfaces/lpcraft.py b/lib/lp/code/interfaces/lpci.py
similarity index 73%
rename from lib/lp/code/interfaces/lpcraft.py
rename to lib/lp/code/interfaces/lpci.py
index 9ac4425..c8519ca 100644
--- a/lib/lp/code/interfaces/lpcraft.py
+++ b/lib/lp/code/interfaces/lpci.py
@@ -1,22 +1,22 @@
 # Copyright 2022 Canonical Ltd.  This software is licensed under the
 # GNU Affero General Public License version 3 (see the file LICENSE).
 
-"""Interface to lpcraft's configuration file."""
+"""Interface to lpci's configuration file."""
 
 __all__ = [
-    "ILPCraftConfiguration",
-    "LPCraftConfigurationError",
+    "ILPCIConfiguration",
+    "LPCIConfigurationError",
 ]
 
 from zope.interface import Interface
 from zope.schema import Dict, List, TextLine
 
 
-class LPCraftConfigurationError(Exception):
-    """Parsing lpcraft's configuration file failed."""
+class LPCIConfigurationError(Exception):
+    """Parsing lpci's configuration file failed."""
 
 
-class ILPCraftConfiguration(Interface):
+class ILPCIConfiguration(Interface):
     """An object representation of a `.launchpad.yaml` file."""
 
     pipeline = List(
diff --git a/lib/lp/code/model/cibuild.py b/lib/lp/code/model/cibuild.py
index 23cb47a..ad5bbfb 100644
--- a/lib/lp/code/model/cibuild.py
+++ b/lib/lp/code/model/cibuild.py
@@ -50,7 +50,7 @@ from lp.code.interfaces.githosting import IGitHostingClient
 from lp.code.interfaces.gitrepository import IGitRepository
 from lp.code.interfaces.revisionstatus import IRevisionStatusReportSet
 from lp.code.model.gitref import GitRef
-from lp.code.model.lpcraft import load_configuration
+from lp.code.model.lpci import load_configuration
 from lp.code.model.revisionstatus import (
     RevisionStatusArtifact,
     RevisionStatusReport,
@@ -726,8 +726,8 @@ class CIBuildSet(SpecificBuildFarmJobSourceMixin):
             commit_sha1s,
             # XXX cjwatson 2022-01-19: We should also fetch
             # debian/.launchpad.yaml (or perhaps make the path a property of
-            # the repository) once lpcraft and launchpad-buildd support
-            # using alternative paths for builds.
+            # the repository) once lpci and launchpad-buildd support using
+            # alternative paths for builds.
             filter_paths=[".launchpad.yaml"],
             logger=logger,
         )
diff --git a/lib/lp/code/model/lpcraft.py b/lib/lp/code/model/lpci.py
similarity index 78%
rename from lib/lp/code/model/lpcraft.py
rename to lib/lp/code/model/lpci.py
index 8a196c2..1aa04e2 100644
--- a/lib/lp/code/model/lpcraft.py
+++ b/lib/lp/code/model/lpci.py
@@ -1,16 +1,16 @@
 # Copyright 2022 Canonical Ltd.  This software is licensed under the
 # GNU Affero General Public License version 3 (see the file LICENSE).
 
-"""This module provides a parser for lpcraft's configuration file.
+"""This module provides a parser for lpci's configuration file.
 
 As currently Launchpad is only compatible with Python 3.5, it was not possible
-to reuse lpcraft's parser.
+to reuse lpci's parser.
 
-The implementation was copied from https://launchpad.net/lpcraft ->
-`lpcraft/config.py`.
+The implementation was copied from https://launchpad.net/lpci ->
+`lpci/config.py`.
 
-XXX jugmac00 2022-01-07: use lpcraft for parsing the configuration file once
-we are on Python 3.8
+XXX jugmac00 2022-01-07: use lpci for parsing the configuration file once we
+are on Python 3.8
 """
 
 __all__ = ["load_configuration"]
@@ -18,10 +18,7 @@ __all__ = ["load_configuration"]
 import yaml
 from zope.interface import implementer
 
-from lp.code.interfaces.lpcraft import (
-    ILPCraftConfiguration,
-    LPCraftConfigurationError,
-)
+from lp.code.interfaces.lpci import ILPCIConfiguration, LPCIConfigurationError
 
 
 def _expand_job_values(values):
@@ -54,10 +51,10 @@ def load_configuration(configuration_file):
     # load yaml
     content = yaml.safe_load(configuration_file)
     if content is None:
-        raise LPCraftConfigurationError("Empty configuration file")
+        raise LPCIConfigurationError("Empty configuration file")
     for required_key in "pipeline", "jobs":
         if required_key not in content:
-            raise LPCraftConfigurationError(
+            raise LPCIConfigurationError(
                 "Configuration file does not declare '{}'".format(required_key)
             )
     # normalize each element of `pipeline` into a list
@@ -75,18 +72,18 @@ def load_configuration(configuration_file):
         for i, job_values in enumerate(expanded_job_values):
             for required_key in "series", "architectures":
                 if required_key not in job_values:
-                    raise LPCraftConfigurationError(
+                    raise LPCIConfigurationError(
                         "Job {}:{} does not declare '{}'".format(
                             job_name, i, required_key
                         )
                     )
     # create "data class"
-    return LPCraftConfiguration(expanded_values)
+    return LPCIConfiguration(expanded_values)
 
 
-@implementer(ILPCraftConfiguration)
-class LPCraftConfiguration:
-    """See `ILPCraftConfiguration`."""
+@implementer(ILPCIConfiguration)
+class LPCIConfiguration:
+    """See `ILPCIConfiguration`."""
 
     def __init__(self, d):
         self.__dict__.update(d)
diff --git a/lib/lp/code/model/tests/test_cibuild.py b/lib/lp/code/model/tests/test_cibuild.py
index 722ecb2..ec452d6 100644
--- a/lib/lp/code/model/tests/test_cibuild.py
+++ b/lib/lp/code/model/tests/test_cibuild.py
@@ -48,7 +48,7 @@ from lp.code.model.cibuild import (
     determine_DASes_to_build,
     get_all_commits_for_paths,
 )
-from lp.code.model.lpcraft import load_configuration
+from lp.code.model.lpci import load_configuration
 from lp.code.tests.helpers import GitHostingFixture
 from lp.registry.interfaces.series import SeriesStatus
 from lp.registry.interfaces.sourcepackage import SourcePackageType
diff --git a/lib/lp/code/model/tests/test_lpcraft.py b/lib/lp/code/model/tests/test_lpci.py
similarity index 88%
rename from lib/lp/code/model/tests/test_lpcraft.py
rename to lib/lp/code/model/tests/test_lpci.py
index 5045aa3..1cddc56 100644
--- a/lib/lp/code/model/tests/test_lpcraft.py
+++ b/lib/lp/code/model/tests/test_lpci.py
@@ -3,11 +3,8 @@
 
 from textwrap import dedent
 
-from lp.code.interfaces.lpcraft import (
-    ILPCraftConfiguration,
-    LPCraftConfigurationError,
-)
-from lp.code.model.lpcraft import load_configuration
+from lp.code.interfaces.lpci import ILPCIConfiguration, LPCIConfigurationError
+from lp.code.model.lpci import load_configuration
 from lp.testing import TestCase
 
 
@@ -26,11 +23,11 @@ class TestLoadConfiguration(TestCase):
 
         configuration = load_configuration(c)
 
-        self.assertProvides(configuration, ILPCraftConfiguration)
+        self.assertProvides(configuration, ILPCIConfiguration)
 
     def test_load_configuration_empty(self):
         self.assertRaisesWithContent(
-            LPCraftConfigurationError,
+            LPCIConfigurationError,
             "Empty configuration file",
             load_configuration,
             "",
@@ -45,7 +42,7 @@ class TestLoadConfiguration(TestCase):
         )
 
         self.assertRaisesWithContent(
-            LPCraftConfigurationError,
+            LPCIConfigurationError,
             "Configuration file does not declare 'pipeline'",
             load_configuration,
             c,
@@ -60,7 +57,7 @@ class TestLoadConfiguration(TestCase):
         )
 
         self.assertRaisesWithContent(
-            LPCraftConfigurationError,
+            LPCIConfigurationError,
             "Configuration file does not declare 'jobs'",
             load_configuration,
             c,
@@ -77,7 +74,7 @@ class TestLoadConfiguration(TestCase):
         )
 
         self.assertRaisesWithContent(
-            LPCraftConfigurationError,
+            LPCIConfigurationError,
             "Job test:0 does not declare 'series'",
             load_configuration,
             c,
@@ -95,7 +92,7 @@ class TestLoadConfiguration(TestCase):
         )
 
         self.assertRaisesWithContent(
-            LPCraftConfigurationError,
+            LPCIConfigurationError,
             "Job test:0 does not declare 'architectures'",
             load_configuration,
             c,