Merge ~barryprice/charm-k8s-discourse/+git/charm-k8s-discourse:loop-precompile into charm-k8s-discourse:master

[Barry Price]

We can't tell from bundler's return code whether it's successfully completed our assets:precompile or not, let's just take the dumb approach for now and run it three times

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

[🤖 prod-jenkaas-is (prod-jenkaas-is)]

A CI job is currently in progress. A follow up comment will be added when it completes.

[🤖 prod-jenkaas-is (prod-jenkaas-is)]

PASSED: Continuous integration, rev:8bb9bff6aa976cfa1af4548e1d918348577621e3
https://jenkins.canonical.com/is/job/lp-charm-k8s-discourse-ci/42/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/is/job/lp-charm-test/160/
    None: https://jenkins.canonical.com/is/job/lp-update-mp/588454/

Click here to trigger a rebuild:
https://jenkins.canonical.com/is/job/lp-charm-k8s-discourse-ci/42//rebuild

review: Approve (continuous-integration)

[🤖 prod-jenkaas-is (prod-jenkaas-is)]

A CI job is currently in progress. A follow up comment will be added when it completes.

[🤖 prod-jenkaas-is (prod-jenkaas-is)]

PASSED: Continuous integration, rev:8ae33eaffa343eb42b77dd3395467c9d3b2e7415
https://jenkins.canonical.com/is/job/lp-charm-k8s-discourse-ci/43/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/is/job/lp-charm-test/161/
    None: https://jenkins.canonical.com/is/job/lp-update-mp/588639/

Click here to trigger a rebuild:
https://jenkins.canonical.com/is/job/lp-charm-k8s-discourse-ci/43//rebuild

review: Approve (continuous-integration)

[Tom Haddon (mthaddon)]

This looks good to land now. It's obviously fairly fragile and we'll want to figure out a way of checking the output rather than just doing it three times, but this will unblock the security update portion of things and we can circle back on making the change more robust.

review: Approve

[🤖 Canonical IS Merge Bot (canonical-is-mergebot)]

Change successfully merged at revision 54f81a9eb88a577581877f9e03b447fbee89e882

diff --git a/image/scripts/pod_setup b/image/scripts/pod_setup
index d716174..7cb1cb7 100755
--- a/image/scripts/pod_setup
+++ b/image/scripts/pod_setup
@@ -39,3 +39,10 @@ fi
 # It is safe to build assets in the background to improve startup time.
 export SKIP_NODE_UGLIFY="true"
 su -s /bin/bash -c "bin/bundle exec rake assets:precompile RAILS_ENV=$RAILS_ENV" ${CONTAINER_APP_USERNAME} 2>&1 |sed 's/^/asset-build: /'
+
+# LP#1952681: First run consistently fails to process discourse/tests/theme_qunit_helper JS
+su -s /bin/bash -c "bin/bundle exec rake assets:precompile RAILS_ENV=$RAILS_ENV" ${CONTAINER_APP_USERNAME} 2>&1 |sed 's/^/asset-build-1: /'
+# LP#1952681: Second run consistently fails to process application JS
+su -s /bin/bash -c "bin/bundle exec rake assets:precompile RAILS_ENV=$RAILS_ENV" ${CONTAINER_APP_USERNAME} 2>&1 |sed 's/^/asset-build-2: /'
+# LP#1952681: Third run consistently succeeds
+su -s /bin/bash -c "bin/bundle exec rake assets:precompile RAILS_ENV=$RAILS_ENV" ${CONTAINER_APP_USERNAME} 2>&1 |sed 's/^/asset-build-3: /'