Skip to content

Commit

Permalink
ci: Store production built assets to /tmp/production-build.
Browse files Browse the repository at this point in the history
Previously, we copied them to /tmp and from there we specified those
assets we copied in circleci config in presist_to_workspace step.
Copying it to a directory allows us to get rid of list in circleci
config and GitHub Actions's upload artifact (their version of
presist to workspace) doesn't allow us to specify indivivual files
so only is this cleaner but required.
  • Loading branch information
priyank-p authored and timabbott committed Jul 24, 2020
1 parent c082fe3 commit 75ab267
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 12 deletions.
10 changes: 2 additions & 8 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -294,16 +294,10 @@ jobs:
# or relative path from working_directory.
# This is a directory on the container which is
# taken to be the root directory of the workspace.
root: /tmp
root: /tmp/production-build
# Must be relative path from root
paths:
- zulip-server-test.tar.gz
- success-http-headers.template.txt
- production-install
- production-verify
- production-upgrade-pg
- production
- production-extract-tarball
- "*"
- *notify_failure_status

"bionic-production-install":
Expand Down
9 changes: 5 additions & 4 deletions tools/ci/production-build
Original file line number Diff line number Diff line change
Expand Up @@ -28,14 +28,15 @@ if ! ./tools/build-release-tarball test; then
exit 1
fi

# This list matches up against the persist_to_workspace step in
# .circleci/config.yml
mv /tmp/tmp.*/zulip-server-test.tar.gz /tmp/
# Move all the required artifacts to /tmp/production-build
# that will be later sent down to downstream install jobs.
mkdir /tmp/production-build
mv /tmp/tmp.*/zulip-server-test.tar.gz /tmp/production-build
cp -a \
tools/ci/success-http-headers.template.txt \
tools/ci/production-install \
tools/ci/production-verify \
tools/ci/production-upgrade-pg \
tools/ci/production-extract-tarball \
\
/tmp/
/tmp/production-build

0 comments on commit 75ab267

Please sign in to comment.