Merge ~cjwatson/launchpad:revert-build-tarball-assets into launchpad:master

Proposed by Colin Watson
Status: Merged
Approved by: Colin Watson
Approved revision: c1e30bc3fcafc1c9a2c1fd57751099b74ae9017a
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~cjwatson/launchpad:revert-build-tarball-assets
Merge into: launchpad:master
Diff against target: 41 lines (+2/-10)
2 files modified
ols-vms.conf (+1/-1)
utilities/build-tarball (+1/-9)
Reviewer Review Type Date Requested Status
Guruprasad Approve
Review via email: mp+441878@code.launchpad.net

Commit message

Revert "Include built assets in deployment tarballs"

Description of the change

This reverts commits 58bc1d894a8d5f7e17b643d11d8c54185eb010f8 and 97990c559e9c4a7234c78b2aa238523c432a2abf. It turns out that it's impossible to include assets in deployment tarballs, because API documentation includes the API host name and so needs to be built separately for each environment. We'll need to take a different approach here.

To post a comment you must log in.
Revision history for this message
Guruprasad (lgp171188) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/ols-vms.conf b/ols-vms.conf
index 0ff80f3..bad8b0b 100644
--- a/ols-vms.conf
+++ b/ols-vms.conf
@@ -3,7 +3,7 @@ vm.architecture = amd64
3vm.release = focal3vm.release = focal
44
5apt.sources = ppa:launchpad/ppa5apt.sources = ppa:launchpad/ppa
6vm.packages = launchpad-frontend-dependencies6vm.packages = launchpad-dependencies
77
8[launchpad]8[launchpad]
9vm.class = lxd9vm.class = lxd
diff --git a/utilities/build-tarball b/utilities/build-tarball
index fcbdc19..22cba4a 100755
--- a/utilities/build-tarball
+++ b/utilities/build-tarball
@@ -16,7 +16,7 @@ output_dir="$1"
16# the Makefile and also has to call out to the Makefile, but this is16# the Makefile and also has to call out to the Makefile, but this is
17# difficult to disentangle until we refactor our build system to use17# difficult to disentangle until we refactor our build system to use
18# something higher-level than pip.18# something higher-level than pip.
19make build build_wheels_only19make build_wheels_only
2020
21# Ensure that we have an updated idea of this tree's version.21# Ensure that we have an updated idea of this tree's version.
22scripts/update-version-info.sh22scripts/update-version-info.sh
@@ -33,14 +33,6 @@ mkdir -p "$output_dir"
33 echo ./version-info.py33 echo ./version-info.py
34 echo ./version-info.txt34 echo ./version-info.txt
35 find ./wheels/ -name \*.whl -print35 find ./wheels/ -name \*.whl -print
36 # We include assets (API documentation, CSS, and JavaScript); only some
37 # deployments need these, but including them in the tarball anyway is
38 # relatively cheap, and they'd otherwise take some time to build during
39 # rollouts.
40 find ./lib/canonical/launchpad/apidoc/ -not -type d -print
41 find ./yarn/node_modules/yui/ -not -type d -print
42 find ./lib/canonical/launchpad/icing/ -not -type d -print
43 find ./build/js/ -not -type d -print
44) | sort >"$output_dir/.files"36) | sort >"$output_dir/.files"
4537
46# Create the tarball.38# Create the tarball.

Subscribers

People subscribed via source and target branches

to status/vote changes: