Merge ~cjwatson/launchpad:charm-recipe-basic-browser into launchpad:master

[Colin Watson]

Add a basic CharmRecipe:+index view

[Cristian Gonzalez (cristiangsp)]

Looks good!

review: Approve

diff --git a/lib/lp/app/browser/configure.zcml b/lib/lp/app/browser/configure.zcml
index be9d898..b08d380 100644
--- a/lib/lp/app/browser/configure.zcml
+++ b/lib/lp/app/browser/configure.zcml
@@ -1,4 +1,4 @@
-<!-- Copyright 2009-2020 Canonical Ltd.  This software is licensed under the
+<!-- Copyright 2009-2021 Canonical Ltd.  This software is licensed under the
      GNU Affero General Public License version 3 (see the file LICENSE).
 -->
 
@@ -582,6 +582,12 @@
       factory="lp.app.browser.tales.BuildImageDisplayAPI"
       name="image"
       />
+  <adapter
+      for="lp.charms.interfaces.charmrecipebuild.ICharmRecipeBuild"
+      provides="zope.traversing.interfaces.IPathAdapter"
+      factory="lp.app.browser.tales.BuildImageDisplayAPI"
+      name="image"
+      />
 
   <adapter
       for="lp.soyuz.interfaces.archive.IArchive"
diff --git a/lib/lp/charms/browser/charmrecipe.py b/lib/lp/charms/browser/charmrecipe.py
index a111fca..e247bf0 100644
--- a/lib/lp/charms/browser/charmrecipe.py
+++ b/lib/lp/charms/browser/charmrecipe.py
@@ -9,18 +9,27 @@ __metaclass__ = type
 __all__ = [
     "CharmRecipeNavigation",
     "CharmRecipeURL",
+    "CharmRecipeView",
     ]
 
 from zope.component import getUtility
 from zope.interface import implementer
+from zope.security.interfaces import Unauthorized
 
 from lp.charms.interfaces.charmrecipe import ICharmRecipe
 from lp.charms.interfaces.charmrecipebuild import ICharmRecipeBuildSet
 from lp.registry.interfaces.personproduct import IPersonProductFactory
+from lp.services.propertycache import cachedproperty
+from lp.services.utils import seconds_since_epoch
 from lp.services.webapp import (
+    LaunchpadView,
     Navigation,
     stepthrough,
     )
+from lp.services.webapp.breadcrumb import (
+    Breadcrumb,
+    NameBreadcrumb,
+    )
 from lp.services.webapp.interfaces import ICanonicalUrlData
 from lp.soyuz.browser.build import get_build_by_id_str
 
@@ -28,7 +37,7 @@ from lp.soyuz.browser.build import get_build_by_id_str
 @implementer(ICanonicalUrlData)
 class CharmRecipeURL:
     """Charm recipe URL creation rules."""
-    rootsite = 'mainsite'
+    rootsite = "mainsite"
 
     def __init__(self, recipe):
         self.recipe = recipe
@@ -61,3 +70,89 @@ class CharmRecipeNavigation(Navigation):
         if build is None or build.recipe != self.context:
             return None
         return build
+
+
+class CharmRecipeBreadcrumb(NameBreadcrumb):
+
+    @property
+    def inside(self):
+        # XXX cjwatson 2021-06-04: This should probably link to an
+        # appropriate listing view, but we don't have one of those yet.
+        return Breadcrumb(
+            self.context.project, text=self.context.project.display_name,
+            inside=self.context.project)
+
+
+class CharmRecipeView(LaunchpadView):
+    """Default view of a charm recipe."""
+
+    @cachedproperty
+    def builds_and_requests(self):
+        return builds_and_requests_for_recipe(self.context)
+
+    @property
+    def build_frequency(self):
+        if self.context.auto_build:
+            return "Built automatically"
+        else:
+            return "Built on request"
+
+    @property
+    def sorted_auto_build_channels_items(self):
+        if self.context.auto_build_channels is None:
+            return []
+        return sorted(self.context.auto_build_channels.items())
+
+    @property
+    def store_channels(self):
+        return ", ".join(self.context.store_channels)
+
+    @property
+    def user_can_see_source(self):
+        try:
+            return self.context.source.visibleByUser(self.user)
+        except Unauthorized:
+            return False
+
+
+def builds_and_requests_for_recipe(recipe):
+    """A list of interesting builds and build requests.
+
+    All pending builds and pending build requests are shown, as well as up
+    to 10 recent builds and recent failed build requests.  Pending items are
+    ordered by the date they were created; recent items are ordered by the
+    date they finished (if available) or the date they started (if the date
+    they finished is not set due to an error).  This allows started but
+    unfinished builds to show up in the view but be discarded as more recent
+    builds become available.
+
+    Builds that the user does not have permission to see are excluded (by
+    the model code).
+    """
+    # We need to interleave items of different types, so SQL can't do all
+    # the sorting for us.
+    def make_sort_key(*date_attrs):
+        def _sort_key(item):
+            for date_attr in date_attrs:
+                if getattr(item, date_attr, None) is not None:
+                    return -seconds_since_epoch(getattr(item, date_attr))
+            return 0
+
+        return _sort_key
+
+    items = sorted(
+        list(recipe.pending_builds) + list(recipe.pending_build_requests),
+        key=make_sort_key("date_created", "date_requested"))
+    if len(items) < 10:
+        # We need to interleave two unbounded result sets, but we only need
+        # enough items from them to make the total count up to 10.  It's
+        # simplest to just fetch the upper bound from each set and do our
+        # own sorting.
+        recent_items = sorted(
+            list(recipe.completed_builds[:10 - len(items)]) +
+            list(recipe.failed_build_requests[:10 - len(items)]),
+            key=make_sort_key(
+                "date_finished", "date_started",
+                "date_created", "date_requested"))
+        items.extend(recent_items[:10 - len(items)])
+    return items
diff --git a/lib/lp/charms/browser/configure.zcml b/lib/lp/charms/browser/configure.zcml
index 1f5a8af..92d78bf 100644
--- a/lib/lp/charms/browser/configure.zcml
+++ b/lib/lp/charms/browser/configure.zcml
@@ -11,9 +11,23 @@
         <browser:url
             for="lp.charms.interfaces.charmrecipe.ICharmRecipe"
             urldata="lp.charms.browser.charmrecipe.CharmRecipeURL" />
+        <browser:defaultView
+            for="lp.charms.interfaces.charmrecipe.ICharmRecipe"
+            name="+index" />
+        <browser:page
+            for="lp.charms.interfaces.charmrecipe.ICharmRecipe"
+            class="lp.charms.browser.charmrecipe.CharmRecipeView"
+            permission="launchpad.View"
+            name="+index"
+            template="../templates/charmrecipe-index.pt" />
         <browser:navigation
             module="lp.charms.browser.charmrecipe"
             classes="CharmRecipeNavigation" />
+        <adapter
+            provides="lp.services.webapp.interfaces.IBreadcrumb"
+            for="lp.charms.interfaces.charmrecipe.ICharmRecipe"
+            factory="lp.charms.browser.charmrecipe.CharmRecipeBreadcrumb"
+            permission="zope.Public" />
         <browser:url
             for="lp.charms.interfaces.charmrecipe.ICharmRecipeBuildRequest"
             path_expression="string:+build-request/${id}"
diff --git a/lib/lp/charms/browser/tests/test_charmrecipe.py b/lib/lp/charms/browser/tests/test_charmrecipe.py
index 89f2945..fbad88e 100644
--- a/lib/lp/charms/browser/tests/test_charmrecipe.py
+++ b/lib/lp/charms/browser/tests/test_charmrecipe.py
@@ -7,11 +7,48 @@ from __future__ import absolute_import, print_function, unicode_literals
 
 __metaclass__ = type
 
+from datetime import (
+    datetime,
+    timedelta,
+    )
+import re
+
+from fixtures import FakeLogger
+import pytz
+import soupmatchers
+from testtools.matchers import (
+    Equals,
+    MatchesListwise,
+    MatchesStructure,
+    )
+import transaction
+from zope.component import getUtility
+from zope.security.proxy import removeSecurityProxy
+
+from lp.app.interfaces.launchpad import ILaunchpadCelebrities
+from lp.buildmaster.enums import BuildStatus
+from lp.buildmaster.interfaces.processor import IProcessorSet
+from lp.charms.browser.charmrecipe import CharmRecipeView
 from lp.charms.interfaces.charmrecipe import CHARM_RECIPE_ALLOW_CREATE
 from lp.services.features.testing import FeatureFixture
+from lp.services.job.interfaces.job import JobStatus
+from lp.services.propertycache import get_property_cache
 from lp.services.webapp import canonical_url
-from lp.testing import TestCaseWithFactory
-from lp.testing.layers import DatabaseFunctionalLayer
+from lp.testing import (
+    BrowserTestCase,
+    person_logged_in,
+    TestCaseWithFactory,
+    time_counter,
+    )
+from lp.testing.layers import (
+    DatabaseFunctionalLayer,
+    LaunchpadFunctionalLayer,
+    )
+from lp.testing.publication import test_traverse
+from lp.testing.views import (
+    create_initialized_view,
+    create_view,
+    )
 
 
 class TestCharmRecipeNavigation(TestCaseWithFactory):
@@ -30,3 +67,229 @@ class TestCharmRecipeNavigation(TestCaseWithFactory):
         self.assertEqual(
             "http://launchpad.test/~person/project/+charm/charm",
             canonical_url(recipe))
+
+    def test_charm_recipe(self):
+        recipe = self.factory.makeCharmRecipe()
+        obj, _, _ = test_traverse(
+            "http://launchpad.test/~%s/%s/+charm/%s" % (
+                recipe.owner.name, recipe.project.name, recipe.name))
+        self.assertEqual(recipe, obj)
+
+
+class BaseTestCharmRecipeView(BrowserTestCase):
+
+    layer = LaunchpadFunctionalLayer
+
+    def setUp(self):
+        super(BaseTestCharmRecipeView, self).setUp()
+        self.useFixture(FeatureFixture({CHARM_RECIPE_ALLOW_CREATE: "on"}))
+        self.useFixture(FakeLogger())
+        self.person = self.factory.makePerson(
+            name="test-person", displayname="Test Person")
+
+
+class TestCharmRecipeView(BaseTestCharmRecipeView):
+
+    def setUp(self):
+        super(TestCharmRecipeView, self).setUp()
+        self.project = self.factory.makeProduct(
+            name="test-project", displayname="Test Project")
+        self.ubuntu = getUtility(ILaunchpadCelebrities).ubuntu
+        self.distroseries = self.factory.makeDistroSeries(
+            distribution=self.ubuntu)
+        processor = getUtility(IProcessorSet).getByName("386")
+        self.distroarchseries = self.factory.makeDistroArchSeries(
+            distroseries=self.distroseries, architecturetag="i386",
+            processor=processor)
+        self.factory.makeBuilder(virtualized=True)
+
+    def makeCharmRecipe(self, **kwargs):
+        if "project" not in kwargs:
+            kwargs["project"] = self.project
+        if "git_ref" not in kwargs:
+            kwargs["git_ref"] = self.factory.makeGitRefs()[0]
+        return self.factory.makeCharmRecipe(
+            registrant=self.person, owner=self.person, name="charm-name",
+            **kwargs)
+
+    def makeBuild(self, recipe=None, date_created=None, **kwargs):
+        if recipe is None:
+            recipe = self.makeCharmRecipe()
+        if date_created is None:
+            date_created = datetime.now(pytz.UTC) - timedelta(hours=1)
+        build = self.factory.makeCharmRecipeBuild(
+            requester=self.person, recipe=recipe,
+            distro_arch_series=self.distroarchseries,
+            date_created=date_created, **kwargs)
+        job = removeSecurityProxy(
+            removeSecurityProxy(build.build_request)._job)
+        job.job._status = JobStatus.COMPLETED
+        return build
+
+    def test_breadcrumb(self):
+        recipe = self.makeCharmRecipe()
+        view = create_view(recipe, "+index")
+        # To test the breadcrumbs we need a correct traversal stack.
+        view.request.traversed_objects = [
+            recipe.owner, recipe.project, recipe, view]
+        view.initialize()
+        breadcrumbs_tag = soupmatchers.Tag(
+            "breadcrumbs", "ol", attrs={"class": "breadcrumbs"})
+        self.assertThat(
+            view(),
+            soupmatchers.HTMLContains(
+                soupmatchers.Within(
+                    breadcrumbs_tag,
+                    soupmatchers.Tag(
+                        "project breadcrumb", "a",
+                        text="Test Project",
+                        attrs={"href": re.compile(r"/test-project$")})),
+                soupmatchers.Within(
+                    breadcrumbs_tag,
+                    soupmatchers.Tag(
+                        "charm breadcrumb", "li",
+                        text=re.compile(r"\scharm-name\s")))))
+
+    def test_index_git(self):
+        [ref] = self.factory.makeGitRefs(
+            owner=self.person, target=self.project, name="charm-repository",
+            paths=["refs/heads/master"])
+        recipe = self.makeCharmRecipe(git_ref=ref)
+        build = self.makeBuild(
+            recipe=recipe, status=BuildStatus.FULLYBUILT,
+            duration=timedelta(minutes=30))
+        self.assertTextMatchesExpressionIgnoreWhitespace(r"""\
+            Test Project
+            charm-name
+            .*
+            Charm recipe information
+            Owner: Test Person
+            Project: Test Project
+            Source: ~test-person/test-project/\+git/charm-repository:master
+            Build schedule: \(\?\)
+            Built on request
+            Builds of this charm recipe are not automatically uploaded to
+            the store.
+            Latest builds
+            Status When complete Architecture
+            Successfully built 30 minutes ago i386
+            """, self.getMainText(build.recipe))
+
+    def test_index_success_with_buildlog(self):
+        # The build log is shown if it is there.
+        build = self.makeBuild(
+            status=BuildStatus.FULLYBUILT, duration=timedelta(minutes=30))
+        build.setLog(self.factory.makeLibraryFileAlias())
+        self.assertTextMatchesExpressionIgnoreWhitespace(r"""\
+            Latest builds
+            Status When complete Architecture
+            Successfully built 30 minutes ago buildlog \(.*\) i386
+            """, self.getMainText(build.recipe))
+
+    def test_index_no_builds(self):
+        # A message is shown when there are no builds.
+        recipe = self.makeCharmRecipe()
+        self.assertIn(
+            "This charm recipe has not been built yet.",
+            self.getMainText(recipe))
+
+    def test_index_pending_build(self):
+        # A pending build is listed as such.
+        build = self.makeBuild()
+        build.queueBuild()
+        self.assertTextMatchesExpressionIgnoreWhitespace(r"""\
+            Latest builds
+            Status When complete Architecture
+            Needs building in .* \(estimated\) i386
+            """, self.getMainText(build.recipe))
+
+    def test_index_pending_build_request(self):
+        # A pending build request is listed as such.
+        recipe = self.makeCharmRecipe()
+        with person_logged_in(recipe.owner):
+            recipe.requestBuilds(recipe.owner)
+        self.assertTextMatchesExpressionIgnoreWhitespace("""\
+            Latest builds
+            Status When complete Architecture
+            Pending build request
+            """, self.getMainText(recipe))
+
+    def test_index_failed_build_request(self):
+        # A failed build request is listed as such, with its error message.
+        recipe = self.makeCharmRecipe()
+        with person_logged_in(recipe.owner):
+            request = recipe.requestBuilds(recipe.owner)
+        job = removeSecurityProxy(removeSecurityProxy(request)._job)
+        job.job._status = JobStatus.FAILED
+        job.job.date_finished = datetime.now(pytz.UTC) - timedelta(hours=1)
+        job.error_message = "Boom"
+        self.assertTextMatchesExpressionIgnoreWhitespace(r"""\
+            Latest builds
+            Status When complete Architecture
+            Failed build request 1 hour ago \(Boom\)
+            """, self.getMainText(recipe))
+
+    def setStatus(self, build, status):
+        build.updateStatus(
+            BuildStatus.BUILDING, date_started=build.date_created)
+        build.updateStatus(
+            status, date_finished=build.date_started + timedelta(minutes=30))
+
+    def test_builds_and_requests(self):
+        # CharmRecipeView.builds_and_requests produces reasonable results,
+        # interleaving build requests with builds.
+        recipe = self.makeCharmRecipe()
+        # Create oldest builds first so that they sort properly by id.
+        date_gen = time_counter(
+            datetime(2000, 1, 1, tzinfo=pytz.UTC), timedelta(days=1))
+        builds = [
+            self.makeBuild(recipe=recipe, date_created=next(date_gen))
+            for i in range(3)]
+        self.setStatus(builds[2], BuildStatus.FULLYBUILT)
+        with person_logged_in(recipe.owner):
+            request = recipe.requestBuilds(recipe.owner)
+        job = removeSecurityProxy(removeSecurityProxy(request)._job)
+        job.job.date_created = next(date_gen)
+        view = CharmRecipeView(recipe, None)
+        # The pending build request is interleaved in date order with
+        # pending builds, and these are followed by completed builds.
+        self.assertThat(view.builds_and_requests, MatchesListwise([
+            MatchesStructure.byEquality(id=request.id),
+            Equals(builds[1]),
+            Equals(builds[0]),
+            Equals(builds[2]),
+            ]))
+        transaction.commit()
+        builds.append(self.makeBuild(recipe=recipe))
+        del get_property_cache(view).builds_and_requests
+        self.assertThat(view.builds_and_requests, MatchesListwise([
+            Equals(builds[3]),
+            MatchesStructure.byEquality(id=request.id),
+            Equals(builds[1]),
+            Equals(builds[0]),
+            Equals(builds[2]),
+            ]))
+        # If we pretend that the job failed, it is still listed, but after
+        # any pending builds.
+        job.job._status = JobStatus.FAILED
+        job.job.date_finished = job.date_created + timedelta(minutes=30)
+        del get_property_cache(view).builds_and_requests
+        self.assertThat(view.builds_and_requests, MatchesListwise([
+            Equals(builds[3]),
+            Equals(builds[1]),
+            Equals(builds[0]),
+            MatchesStructure.byEquality(id=request.id),
+            Equals(builds[2]),
+            ]))
+
+    def test_store_channels_empty(self):
+        recipe = self.factory.makeCharmRecipe()
+        view = create_initialized_view(recipe, "+index")
+        self.assertEqual("", view.store_channels)
+
+    def test_store_channels_display(self):
+        recipe = self.factory.makeCharmRecipe(
+            store_channels=["track/stable/fix-123", "track/edge/fix-123"])
+        view = create_initialized_view(recipe, "+index")
+        self.assertEqual(
+            "track/stable/fix-123, track/edge/fix-123", view.store_channels)
diff --git a/lib/lp/charms/configure.zcml b/lib/lp/charms/configure.zcml
index 68acb5e..9cda4c5 100644
--- a/lib/lp/charms/configure.zcml
+++ b/lib/lp/charms/configure.zcml
@@ -12,6 +12,8 @@
 
     <include package=".browser" />
 
+    <lp:help-folder folder="help" name="+help-charms" />
+
     <!-- CharmRecipe -->
     <class class="lp.charms.model.charmrecipe.CharmRecipe">
         <require
diff --git a/lib/lp/charms/help/charm-recipe-build-frequency.html b/lib/lp/charms/help/charm-recipe-build-frequency.html
new file mode 100644
index 0000000..b15cf50
--- /dev/null
+++ b/lib/lp/charms/help/charm-recipe-build-frequency.html
@@ -0,0 +1,38 @@
+<html>
+  <head>
+    <title>Charm recipe build schedule</title>
+    <link rel="stylesheet" type="text/css"
+          href="/+icing/yui/cssreset/reset.css" />
+    <link rel="stylesheet" type="text/css"
+          href="/+icing/yui/cssfonts/fonts.css" />
+    <link rel="stylesheet" type="text/css"
+          href="/+icing/yui/cssbase/base.css" />
+    <style type="text/css">
+      dt { font-weight: bold }
+      dd p { margin-bottom: 0.5em }
+    </style>
+  </head>
+  <body>
+    <h1>Charm recipe build schedule</h1>
+
+    <p>There are two options for when charm recipes get built:</p>
+    <dl>
+      <dt>Built automatically</dt>
+      <dd>
+        <p>A build will be scheduled automatically once a change to the
+          top-level source branch for the charm recipe is detected.</p>
+        <p>If there has been a build of the charm recipe within the previous
+          hour, the build will not be scheduled until an hour since the last
+          build.</p>
+        <p>If you really want the build to happen before the one-hour period
+          is up, you can use the "Request builds" action.</p>
+      </dd>
+      <dt>Built on request</dt>
+      <dd>
+        <p>Builds of the charm recipe have to be manually requested using
+          the "Request builds" action.</p>
+      </dd>
+    </dl>
+
+  </body>
+</html>
diff --git a/lib/lp/charms/interfaces/charmrecipe.py b/lib/lp/charms/interfaces/charmrecipe.py
index 82a2e6a..6b70d1b 100644
--- a/lib/lp/charms/interfaces/charmrecipe.py
+++ b/lib/lp/charms/interfaces/charmrecipe.py
@@ -41,7 +41,10 @@ from lazr.restful.fields import (
     ReferenceChoice,
     )
 from six.moves import http_client
-from zope.interface import Interface
+from zope.interface import (
+    Attribute,
+    Interface,
+    )
 from zope.schema import (
     Bool,
     Choice,
@@ -260,6 +263,9 @@ class ICharmRecipeView(Interface):
         vocabulary="ValidPersonOrTeam",
         description=_("The person who registered this charm recipe."))
 
+    source = Attribute(
+        "The source branch for this charm recipe (VCS-agnostic).")
+
     private = Bool(
         title=_("Private"), required=False, readonly=False,
         description=_("Whether this charm recipe is private."))
@@ -330,6 +336,40 @@ class ICharmRecipeView(Interface):
         :return: `ICharmRecipeBuildRequest`.
         """
 
+    pending_build_requests = CollectionField(
+        title=_("Pending build requests for this charm recipe."),
+        value_type=Reference(ICharmRecipeBuildRequest),
+        required=True, readonly=True)
+
+    failed_build_requests = CollectionField(
+        title=_("Failed build requests for this charm recipe."),
+        value_type=Reference(ICharmRecipeBuildRequest),
+        required=True, readonly=True)
+
+    builds = CollectionField(
+        title=_("All builds of this charm recipe."),
+        description=_(
+            "All builds of this charm recipe, sorted in descending order "
+            "of finishing (or starting if not completed successfully)."),
+        # Really ICharmRecipeBuild.
+        value_type=Reference(schema=Interface), readonly=True)
+
+    completed_builds = CollectionField(
+        title=_("Completed builds of this charm recipe."),
+        description=_(
+            "Completed builds of this charm recipe, sorted in descending "
+            "order of finishing."),
+        # Really ICharmRecipeBuild.
+        value_type=Reference(schema=Interface), readonly=True)
+
+    pending_builds = CollectionField(
+        title=_("Pending builds of this charm recipe."),
+        description=_(
+            "Pending builds of this charm recipe, sorted in descending "
+            "order of creation."),
+        # Really ICharmRecipeBuild.
+        value_type=Reference(schema=Interface), readonly=True)
+
 
 class ICharmRecipeEdit(Interface):
     """`ICharmRecipe` methods that require launchpad.Edit permission."""
diff --git a/lib/lp/charms/model/charmrecipe.py b/lib/lp/charms/model/charmrecipe.py
index 28458ca..da24bf6 100644
--- a/lib/lp/charms/model/charmrecipe.py
+++ b/lib/lp/charms/model/charmrecipe.py
@@ -21,8 +21,10 @@ from storm.databases.postgres import JSON
 from storm.locals import (
     Bool,
     DateTime,
+    Desc,
     Int,
     Join,
+    Not,
     Or,
     Reference,
     Store,
@@ -41,6 +43,8 @@ from lp.app.enums import (
     )
 from lp.app.interfaces.launchpad import ILaunchpadCelebrities
 from lp.buildmaster.enums import BuildStatus
+from lp.buildmaster.interfaces.buildqueue import IBuildQueueSet
+from lp.buildmaster.model.builder import Builder
 from lp.charms.adapters.buildarch import determine_instances_to_build
 from lp.charms.interfaces.charmrecipe import (
     CannotFetchCharmcraftYaml,
@@ -94,6 +98,10 @@ from lp.services.database.interfaces import (
     IStore,
     )
 from lp.services.database.stormbase import StormBase
+from lp.services.database.stormexpr import (
+    Greatest,
+    NullsLast,
+    )
 from lp.services.features import getFeatureFlag
 from lp.services.job.interfaces.job import JobStatus
 from lp.services.librarian.model import LibraryFileAlias
@@ -325,6 +333,11 @@ class CharmRecipe(StormBase):
             self.git_path = None
 
     @property
+    def source(self):
+        """See `ICharmRecipe`."""
+        return self.git_ref
+
+    @property
     def store_channels(self):
         """See `ICharmRecipe`."""
         return self._store_channels or []
@@ -527,6 +540,83 @@ class CharmRecipe(StormBase):
         """See `ICharmRecipe`."""
         return CharmRecipeBuildRequest(self, job_id)
 
+    @property
+    def pending_build_requests(self):
+        """See `ICharmRecipe`."""
+        job_source = getUtility(ICharmRecipeRequestBuildsJobSource)
+        # The returned jobs are ordered by descending ID.
+        jobs = job_source.findByRecipe(
+            self, statuses=(JobStatus.WAITING, JobStatus.RUNNING))
+        return DecoratedResultSet(
+            jobs, result_decorator=CharmRecipeBuildRequest.fromJob)
+
+    @property
+    def failed_build_requests(self):
+        """See `ICharmRecipe`."""
+        job_source = getUtility(ICharmRecipeRequestBuildsJobSource)
+        # The returned jobs are ordered by descending ID.
+        jobs = job_source.findByRecipe(self, statuses=(JobStatus.FAILED,))
+        return DecoratedResultSet(
+            jobs, result_decorator=CharmRecipeBuildRequest.fromJob)
+
+    def _getBuilds(self, filter_term, order_by):
+        """The actual query to get the builds."""
+        query_args = [
+            CharmRecipeBuild.recipe == self,
+            ]
+        if filter_term is not None:
+            query_args.append(filter_term)
+        result = Store.of(self).find(CharmRecipeBuild, *query_args)
+        result.order_by(order_by)
+
+        def eager_load(rows):
+            getUtility(ICharmRecipeBuildSet).preloadBuildsData(rows)
+            getUtility(IBuildQueueSet).preloadForBuildFarmJobs(rows)
+            load_related(Builder, rows, ["builder_id"])
+
+        return DecoratedResultSet(result, pre_iter_hook=eager_load)
+
+    @property
+    def builds(self):
+        """See `ICharmRecipe`."""
+        order_by = (
+            NullsLast(Desc(Greatest(
+                CharmRecipeBuild.date_started,
+                CharmRecipeBuild.date_finished))),
+            Desc(CharmRecipeBuild.date_created),
+            Desc(CharmRecipeBuild.id))
+        return self._getBuilds(None, order_by)
+
+    @property
+    def _pending_states(self):
+        """All the build states we consider pending (non-final)."""
+        return [
+            BuildStatus.NEEDSBUILD,
+            BuildStatus.BUILDING,
+            BuildStatus.UPLOADING,
+            BuildStatus.CANCELLING,
+            ]
+
+    @property
+    def completed_builds(self):
+        """See `ICharmRecipe`."""
+        filter_term = Not(CharmRecipeBuild.status.is_in(self._pending_states))
+        order_by = (
+            NullsLast(Desc(Greatest(
+                CharmRecipeBuild.date_started,
+                CharmRecipeBuild.date_finished))),
+            Desc(CharmRecipeBuild.id))
+        return self._getBuilds(filter_term, order_by)
+
+    @property
+    def pending_builds(self):
+        """See `ICharmRecipe`."""
+        filter_term = (CharmRecipeBuild.status.is_in(self._pending_states))
+        # We want to order by date_created but this is the same as ordering
+        # by id (since id increases monotonically) and is less expensive.
+        order_by = Desc(CharmRecipeBuild.id)
+        return self._getBuilds(filter_term, order_by)
+
     def destroySelf(self):
         """See `ICharmRecipe`."""
         IStore(CharmRecipe).remove(self)
diff --git a/lib/lp/charms/templates/charmrecipe-index.pt b/lib/lp/charms/templates/charmrecipe-index.pt
new file mode 100644
index 0000000..90a849a
--- /dev/null
+++ b/lib/lp/charms/templates/charmrecipe-index.pt
@@ -0,0 +1,166 @@
+<html
+  xmlns="http://www.w3.org/1999/xhtml"
+  xmlns:tal="http://xml.zope.org/namespaces/tal"
+  xmlns:metal="http://xml.zope.org/namespaces/metal"
+  xmlns:i18n="http://xml.zope.org/namespaces/i18n"
+  metal:use-macro="view/macro:page/main_side"
+  i18n:domain="launchpad"
+>
+
+<body>
+  <metal:registering fill-slot="registering">
+    Created by
+      <tal:registrant replace="structure context/registrant/fmt:link"/>
+    on
+      <tal:created-on replace="structure context/date_created/fmt:date"/>
+    and last modified on
+      <tal:last-modified replace="structure context/date_last_modified/fmt:date"/>
+  </metal:registering>
+
+  <metal:side fill-slot="side">
+    <div tal:replace="structure context/@@+global-actions"/>
+  </metal:side>
+
+  <metal:heading fill-slot="heading">
+    <h1 tal:content="context/name"/>
+  </metal:heading>
+
+  <div metal:fill-slot="main">
+    <h2>Charm recipe information</h2>
+    <div class="two-column-list">
+      <dl id="owner">
+        <dt>Owner:</dt>
+        <dd tal:content="structure context/owner/fmt:link"/>
+      </dl>
+      <dl id="project" tal:define="project context/project">
+        <dt>Project:</dt>
+        <dd>
+          <a tal:attributes="href context/project/fmt:url"
+             tal:content="context/project/display_name"/>
+        </dd>
+      </dl>
+      <dl id="source"
+          tal:define="source context/source" tal:condition="source">
+        <dt>Source:</dt>
+        <dd tal:condition="view/user_can_see_source">
+          <a tal:replace="structure source/fmt:link"/>
+        </dd>
+        <dd tal:condition="not: view/user_can_see_source">
+          <span class="sprite private">&lt;redacted&gt;</span>
+        </dd>
+      </dl>
+
+      <dl id="auto_build">
+        <dt>Build schedule:
+          <a href="/+help-charms/charm-recipe-build-frequency.html"
+             target="help" class="sprite maybe action-icon">(?)</a>
+        </dt>
+        <dd>
+          <span tal:replace="view/build_frequency"/>
+        </dd>
+      </dl>
+      <dl id="auto_build_channels" tal:condition="context/auto_build_channels">
+        <dt>
+          Source snap channels for automatic builds:
+        </dt>
+        <dd>
+          <table class="listing compressed">
+            <tbody>
+              <tr tal:repeat="pair view/sorted_auto_build_channels_items">
+                <td tal:repeat="value pair" tal:content="value"/>
+              </tr>
+            </tbody>
+          </table>
+        </dd>
+      </dl>
+    </div>
+
+    <div id="store_upload" class="two-column-list"
+         tal:condition="context/store_upload">
+      <dl id="store_name">
+        <dt>Registered store package name:</dt>
+        <dd>
+          <span tal:content="context/store_name"/>
+        </dd>
+      </dl>
+      <dl id="store_channels" tal:condition="view/store_channels">
+        <dt>Store channels:</dt>
+        <dd>
+          <span tal:content="view/store_channels"/>
+        </dd>
+      </dl>
+      <p id="store_channels" tal:condition="not: view/store_channels">
+        This charm recipe will not be released to any channels on the store.
+      </p>
+    </div>
+    <p id="store_upload" tal:condition="not: context/store_upload">
+      Builds of this charm recipe are not automatically uploaded to the store.
+    </p>
+
+    <h2>Latest builds</h2>
+    <table id="latest-builds-listing" class="listing"
+           style="margin-bottom: 1em;">
+      <thead>
+        <tr>
+          <th>Status</th>
+          <th>When complete</th>
+          <th>Architecture</th>
+        </tr>
+      </thead>
+      <tbody>
+        <tal:charm-recipe-builds-and-requests repeat="item view/builds_and_requests">
+          <tal:charm-recipe-build-request condition="item/date_requested|nothing">
+            <tr tal:define="request item"
+                tal:attributes="id string:request-${request/id}">
+              <td tal:attributes="class string:request_status ${request/status/name}">
+                <span tal:replace="structure request/image:icon"/>
+                <tal:title replace="request/status/title"/> build request
+              </td>
+              <td>
+                <tal:date condition="request/date_finished"
+                          replace="structure request/date_finished/fmt:displaydatetitle"/>
+                <tal:error-message condition="request/error_message">
+                  (<span tal:replace="request/error_message"/>)
+                </tal:error-message>
+              </td>
+              <td/>
+            </tr>
+          </tal:charm-recipe-build-request>
+          <tal:charm-recipe-build condition="not: item/date_requested|nothing">
+            <tr tal:define="build item"
+                tal:attributes="id string:build-${build/id}">
+              <td tal:attributes="class string:build_status ${build/status/name}">
+                <span tal:replace="structure build/image:icon"/>
+                <a tal:content="build/status/title"
+                   tal:attributes="href build/fmt:url"/>
+              </td>
+              <td class="datebuilt">
+                <tal:date replace="structure build/date/fmt:displaydatetitle"/>
+                <tal:estimate condition="build/estimate">
+                  (estimated)
+                </tal:estimate>
+
+                <tal:build-log define="file build/log" tal:condition="file">
+                  <a class="sprite download"
+                     tal:attributes="href build/log_url">buildlog</a>
+                  (<span tal:replace="file/content/filesize/fmt:bytes"/>)
+                </tal:build-log>
+              </td>
+              <td>
+                <a class="sprite distribution"
+                   tal:define="archseries build/distro_arch_series"
+                   tal:attributes="href archseries/fmt:url"
+                   tal:content="archseries/architecturetag"/>
+              </td>
+            </tr>
+          </tal:charm-recipe-build>
+        </tal:charm-recipe-builds-and-requests>
+      </tbody>
+    </table>
+    <p tal:condition="not: view/builds_and_requests">
+      This charm recipe has not been built yet.
+    </p>
+  </div>
+
+</body>
+</html>