Feature/update demo deployment #26
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Mathieudewet
force-pushed
the
feature/update-demo-deployment
branch
8 times, most recently
from
4956a6b to
38ddcc1
Compare
dunglas
requested changes
Jul 10, 2018
.travis.yml
Outdated
| sudo: required | ||
|
|
||
| # TODO: Verify this upgrade still works. | ||
| php: |
.travis.yml
Outdated
| php: | ||
| - 7.2 | ||
|
|
||
| # Specify which branches to build using a safelist: |
There was a problem hiding this comment.
Travis must still be execute for all branches. The deployment for master only.
.travis.yml
Outdated
|
|
||
| # clone repositories to a depth of 1 commit | ||
| # Warning: Travis CI won’t build commits that are in the queue when pushing a new commit and depth set to 1. | ||
| git: |
.travis.yml
Outdated
| services: | ||
| - docker | ||
| - redis-server |
.travis.yml
Outdated
|
|
||
| cache: | ||
| yarn: true | ||
| directories: | ||
| - admin/node_modules | ||
| - client/node_modules | ||
| - api/vendor |
.travis.yml
Outdated
|
|
||
| script: | ||
| - docker-compose up -d | ||
| - linux-amd64/helm lint api/helm/api/ | ||
| - sleep 20 | ||
| - sh -c "cd admin && PUBLIC_URL="/api-demo-admin-test" yarn install && yarn build" | ||
| - sh -c "cd client && PUBLIC_URL="/api-demo-client-test" yarn install && yarn build" | ||
| - sh -c "cd admin && PUBLIC_URL="/api-demo-admin-test" yarn install --pure-lockfile && yarn build" |
There was a problem hiding this comment.
Should be run in the Docker image (while not mandatory, it will probably be faster outside of the container)
.travis.yml
Outdated
| - gcloud config set project api-platform-demo-209807 | ||
| - mkdir -p ${HOME}/.kube | ||
| - cp ./kubeConfig ${HOME}/.kube/config | ||
| - curl -O https://storage.googleapis.com/kubernetes-helm/helm-v2.7.2-linux-amd64.tar.gz |
.travis.yml
Outdated
| - sudo mv linux-amd64/helm /usr/local/bin/helm | ||
| - gcloud container clusters get-credentials "api-platform-demo" | ||
| - helm init | ||
| - APP_ENV=prod composer install --no-scripts --no-dev --no-progress --no-suggest --optimize-autoloader --classmap-authoritative --no-interaction --prefer-dist |
There was a problem hiding this comment.
Not necessary, must be done in the Docker image directly
.travis.yml
Outdated
| --set secret=${APP_SECRET} \ | ||
| --set postgresUser=${DATABASE_USER},postgresPassword="${DATABASE_PASSWORD}",postgresDatabase=${DATABASE_NAME} \ | ||
| --set postgresql.persistence.enabled=true \ | ||
| --set corsAllowOrigin='^https?://localhost(:[0-9]*)?$' |
There was a problem hiding this comment.
* to make the API available publicly
docker-compose.yml
Outdated
| @@ -16,7 +16,7 @@ services: | |||
| volumes: | |||
| - ./api:/srv/api:rw,cached | |||
| # If you develop on Linux, uncomment the following line to use a bind-mounted host directory instead | |||
| # - ./api/var:/srv/api/var:rw | |||
There was a problem hiding this comment.
Should be reverted (people will run this project file on Mac)
Mathieudewet
force-pushed
the
feature/update-demo-deployment
branch
3 times, most recently
from
a5e1cec to
325f0a7
Compare
|
From https://docs.travis-ci.com/user/encrypting-files/#Automated-Encryption
|
Mathieudewet
force-pushed
the
feature/update-demo-deployment
branch
17 times, most recently
from
2e32cb3 to
7edfb4d
Compare
Mathieudewet
force-pushed
the
feature/update-demo-deployment
branch
4 times, most recently
from
7c5dea4 to
538a9b2
Compare
Closed
Mathieudewet
force-pushed
the
feature/update-demo-deployment
branch
3 times, most recently
from
d5602d8 to
07da825
Compare
Mathieudewet
force-pushed
the
feature/update-demo-deployment
branch
from
07da825 to
bb2ef04
Compare
Mathieudewet
force-pushed
the
feature/update-demo-deployment
branch
from
bb2ef04 to
a01058a
Compare
dunglas
approved these changes
Sep 18, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WIP - Please do not merge it.