From ab0a9fd1750c0d3d62669d18dcda44afb594f723 Mon Sep 17 00:00:00 2001
From: Ben Nemec <bnemec@redhat.com>
Date: Thu, 28 Sep 2017 21:48:26 +0000
Subject: [PATCH] Remove source admonition

We haven't actually recommended installing anything from source for
quite a while and it probably doesn't work anymore anyway.  Let's
get rid of this admonition and clean things up a little.

Change-Id: I005bc042f4d1ff855a90710599d796a9645b71f1
---
 _custom/custom.css                            |  1 -
 _templates/layout.html                        |  6 ------
 doc/source/index.rst                          |  6 ------
 .../basic_deployment/basic_deployment_cli.rst | 12 ------------
 doc/source/install/developer/developer.rst    |  1 -
 .../install/developer/in_progress_review.rst  | 19 -------------------
 .../install/installation/installing.rst       | 12 ------------
 7 files changed, 57 deletions(-)
 delete mode 100644 doc/source/install/developer/in_progress_review.rst

diff --git a/_custom/custom.css b/_custom/custom.css
index 00b0701d..2a4d2e13 100644
--- a/_custom/custom.css
+++ b/_custom/custom.css
@@ -68,7 +68,6 @@
 .baremetal {background: #eef;}
 .virtual {background: #efe;}
 .ceph {background: #eff;}
-.source {background: #bbb;}
 .ltom {background: #dee;}
 .mton {background: #ded;}
 .ntoo {background: #edd;}
diff --git a/_templates/layout.html b/_templates/layout.html
index 4bcff040..47912a4c 100644
--- a/_templates/layout.html
+++ b/_templates/layout.html
@@ -50,12 +50,6 @@
         <li><input type="checkbox" id="ceph" checked="checked"><label for="ceph" title="Step that should only be run when deploying Ceph for use by the Overcloud.">Ceph</label></li>
       </ul>
 
-      <span class="title">Development options</span>
-      <ul>
-         <li><input type="checkbox" id="source" checked=""><label for="source"
-            title="Step that should only be run when choosing to use some components directly from their git source code repositories instead of packages.">Install from source</label></li>
-      </ul>
-
       <span class="title">Upgrade Version</span>
       <ul>
         <li><input type="checkbox" id="mton" checked="checked"><label for="mton" title="Step that should only be run for upgrading from Mitaka to Newton">Upgrading Mitaka to Newton</label></li>
diff --git a/doc/source/index.rst b/doc/source/index.rst
index 3f4d8914..625dcc60 100644
--- a/doc/source/index.rst
+++ b/doc/source/index.rst
@@ -67,12 +67,6 @@ Linux (RHEL). These steps are marked as follows:
 
    Step that should only be run when deploying Ceph for use by the Overcloud
 
-.. admonition:: Source
-   :class: source
-
-   Step that should only be run when choosing to use some components directly
-   from their git source code repositories instead of packages.
-
 .. admonition:: Stable Branch
    :class: stable
 
diff --git a/doc/source/install/basic_deployment/basic_deployment_cli.rst b/doc/source/install/basic_deployment/basic_deployment_cli.rst
index df56f174..5cddecdd 100644
--- a/doc/source/install/basic_deployment/basic_deployment_cli.rst
+++ b/doc/source/install/basic_deployment/basic_deployment_cli.rst
@@ -206,18 +206,6 @@ created on the undercloud, one should use a non-root user.
             # rhel-7-server-rhceph-{2,1.3}-tools-rpms
             export REG_ACTIVATION_KEY="[activation key]"
 
-  .. admonition:: Source
-     :class: source
-
-     Git checkouts of the puppet modules can be used instead of packages. Export the
-     following environment variable::
-
-       export DIB_INSTALLTYPE_puppet_modules=source
-
-     It is also possible to use this functionality to use an in-progress review
-     as part of the overcloud image build.  See
-     :doc:`../developer/in_progress_review` for details.
-
   ::
 
     openstack overcloud image build
diff --git a/doc/source/install/developer/developer.rst b/doc/source/install/developer/developer.rst
index 18baf4ed..49c92752 100644
--- a/doc/source/install/developer/developer.rst
+++ b/doc/source/install/developer/developer.rst
@@ -7,6 +7,5 @@ Documentation of developer-specific options in |project|.
 .. toctree::
 
    tripleo.sh
-   in_progress_review
    tht_walkthrough/tht_walkthrough
    release
diff --git a/doc/source/install/developer/in_progress_review.rst b/doc/source/install/developer/in_progress_review.rst
deleted file mode 100644
index 5fb547cc..00000000
--- a/doc/source/install/developer/in_progress_review.rst
+++ /dev/null
@@ -1,19 +0,0 @@
-Using an In-Progress Review
-===========================
-
-To use a git checkout for only a specific module, export the following variable::
-
-    export DIB_INSTALLTYPE_puppet_tripleo=source
-
-Replace ``puppet_tripleo`` with the name of the puppet module to be installed
-from source, replacing any -'s with _'s.
-
-To use a pending review for a module, set its installtype to source as
-described above, then also export the following variables::
-
-    export DIB_REPOLOCATION_puppet_tripleo=https://review.openstack.org/openstack/puppet-tripleo
-    export DIB_REPOREF_puppet_tripleo=refs/changes/30/223330/1
-
-This time replace the name of the module in the variable name and the review URL.
-The correct value for the ``reporef`` can be found in the ``Download`` section
-of the Gerrit UI.  Look for a string that matches the format of the example above.
diff --git a/doc/source/install/installation/installing.rst b/doc/source/install/installation/installing.rst
index 63bc3b7d..7c628c8e 100644
--- a/doc/source/install/installation/installing.rst
+++ b/doc/source/install/installation/installing.rst
@@ -91,18 +91,6 @@ Installing the Undercloud
 
 #. Run the command to install the undercloud:
 
-   .. admonition:: Source
-      :class: source
-
-      Git checkouts of the puppet modules can be used instead of packages. Export the
-      following environment variable::
-
-        export DIB_INSTALLTYPE_puppet_modules=source
-
-      It is also possible to use this functionality to use an in-progress review
-      as part of the undercloud install.  See
-      :doc:`../developer/in_progress_review` for details.
-
    .. admonition:: SSL
       :class: optional