Merge pull request docker#11600 from thaJeztah/prod_develop

Separate "development" and "production" configurations

Author: usha-mandya

.github/workflows/build-published.yml

@@ -17,7 +17,7 @@ jobs:
         run: docker version && docker info
       - uses: actions/checkout@v2
       - name: build current docs
-        run: docker build --target=deploy-source --output=./_site .
+        run: docker build --build-arg JEKYLL_ENV=production --target=deploy-source --output=./_site .
       - name: upload files to S3 bucket
         run: aws s3 sync --acl public-read _site s3://docs.docker.com-us-east-1/ --delete
         env:

Dockerfile

@@ -10,6 +10,8 @@
 #
 # When the image is run, it starts Nginx and serves the docs at port 4000
 
+# Jekyll environment (development/production)
+ARG JEKYLL_ENV=development
 
 # Engine
 ARG ENGINE_BRANCH="19.03"
@@ -53,9 +55,16 @@ COPY --from=upstream-resources /usr/src/app/md_source/. ./
 # substitute the "{site.latest_engine_api_version}" in the title for the latest
 # API docs, based on the latest_engine_api_version parameter in _config.yml
 RUN ./_scripts/update-api-toc.sh
-RUN jekyll build -d ${TARGET} \
- && find ${TARGET} -type f -name '*.html' | while read i; do sed -i 's#\(<a[^>]* href="\)https://docs.docker.com/#\1/#g' "$i"; done \
- && sed -i 's#<loc>/#<loc>https://docs.docker.com/#' "${TARGET}/sitemap.xml"
+ARG JEKYLL_ENV
+RUN echo "Building docs for ${JEKYLL_ENV} environment"
+RUN set -eux; \
+ if [ "${JEKYLL_ENV}" = "production" ]; then \
+    jekyll build -d ${TARGET} --config _config.yml,_config_production.yml; \
+ else \
+    jekyll build -d ${TARGET}; \
+ fi; \
+ find ${TARGET} -type f -name '*.html' | while read i; do sed -i 's#\(<a[^>]* href="\)https://docs.docker.com/#\1/#g' "$i"; done; \
+ sed -i 's#<loc>/#<loc>https://docs.docker.com/#' "${TARGET}/sitemap.xml";
 
 
 # This stage only contains the generated files. It can be used to host the
@@ -79,4 +88,6 @@ COPY --from=current  /usr/share/nginx/html .
 
 # Configure NGINX
 COPY _deploy/nginx/default.conf /etc/nginx/conf.d/default.conf
-CMD echo -e "Docker docs are viewable at:\nhttp://0.0.0.0:4000"; exec nginx -g 'daemon off;'
+ARG JEKYLL_ENV
+ENV JEKYLL_ENV=${JEKYLL_ENV}
+CMD echo -e "Docker docs are viewable at:\nhttp://0.0.0.0:4000 (build target: ${JEKYLL_ENV})"; exec nginx -g 'daemon off;'

_config.yml

@@ -11,13 +11,7 @@ incremental: true
 permalink: pretty
 safe: false
 lsi: false
-exclude: ["_scripts", "tests", "apidocs/layouts", "Gemfile", "hooks", "index.html", "404.html"]
-
-# Google Analytics, etc.
-# TODO these should only be set when building for production, or passed as an environment variable when building
-google_analytics: GTM-WL2QLG5
-polldaddy_id: 8453675
-
+exclude: ["_samples", "_scripts", "404.html", "datacenter", "ee", "index.html"]
 
 # Component versions -- address like site.docker_ce_version
 # You can't have - characters in these for non-YAML reasons
@@ -121,5 +115,5 @@ defaults:
 #
 # We specify the directory for Jekyll so we can use @imports.
 sass:
-  sass_dir:          _scss
-  style:            :compressed
+  sass_dir: _scss
+  style: expanded

_config_authoring.yml

@@ -0,0 +1,11 @@
+
+# Include files that are excluded in "development" ("enterprise" stubs) for production
+exclude: ["_scripts", "404.html", "index.html"]
+
+# Google Analytics, etc.
+google_analytics: GTM-WL2QLG5
+polldaddy_id:     8453675
+
+# Assets
+sass:
+  style:    compressed

_config_production.yml

@@ -73,5 +73,5 @@ <h1>{{ page.title }}</h1>
         {% include footer.html %}
     </footer>
     <script>const pageURL = "{{ page.url }}";</script>
-    {%- include analytics/polldaddy.html -%}
+    {%- if jekyll.environment == 'production' -%}{%- include analytics/polldaddy.html -%}{%- endif -%}
 </body>

_includes/body.html

@@ -24,16 +24,20 @@
         {%- assign page_description = content | strip_html | strip | truncatewords: 30 -%}
     {%- endif -%}
 {%- endif -%}
+{%- assign domain_name = 'https://docs.docker.com' -%}
+{%- if jekyll.environment == 'development' -%}
+{%- assign domain_name = 'https://localhost:4000' -%}
+{%- endif -%}
 <head>
   <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
   {%- if page.sitemap == false or site.GH_ENV == "gh_pages" %}
   <meta name="robots" content="noindex"/>
   {%- endif %}
-  {%- if site.google_analytics != '' -%}{%- include analytics/google_analytics.html GOOGLE_ID=site.google_analytics -%}{%- endif -%}
+  {%- if jekyll.environment == 'production' and site.google_analytics != '' -%}{%- include analytics/google_analytics.html GOOGLE_ID=site.google_analytics -%}{%- endif -%}
   <title>{{ page.title | default: page_title }} | Docker Documentation</title>
   <meta name="description" content="{{ page.description | default: page_description | escape}}" />
   <meta name="keywords" content="{{ page.keywords | default: 'docker, docker open source, docker platform, distributed applications, microservices, containers, docker containers, docker software, docker virtualization' }}">
-  <link rel="canonical" href="https://docs.docker.com{{ page.url }}" />
+  <link rel="canonical" href="{{ domain_name }}{{ page.url }}" />
 
   <!-- favicon -->
   <link rel="icon" type="image/x-icon" href="/favicons/[email protected]" sizes="129x128">

_includes/head.html

@@ -1,7 +1,21 @@
 version: "3.7"
 services:
   docs:
+    # By default, docker-compose up --build builds docs for a development
+    # environment (no Google Analytics, omitting some enterprise redirects,
+    # etc.
+    #
+    # To test a "production" build, override the environment using:
+    #
+    # JEKYLL_ENV=production docker-compose up --build
     build:
+      args:
+        # FIXME: docker-compose should behave the same as the docker CLI here
+        # and if `JEKYLL_ENV` is not set in the current environment, ignore
+        # the --build-arg, and use the default that's defined in the Dockerfile.
+        # Setting a default here as a workaround.
+        # - JEKYLL_ENV
+        - JEKYLL_ENV=${JEKYLL_ENV:-development}
       context: .
     image: docs/docstage
     ports: