Skip to content
This repository was archived by the owner on Jul 16, 2020. It is now read-only.

Deployment script: use git pull instead of extracting from zip #9

Open
jvanloov opened this issue Jun 15, 2015 · 2 comments
Open

Deployment script: use git pull instead of extracting from zip #9

jvanloov opened this issue Jun 15, 2015 · 2 comments

Comments

@jvanloov
Copy link
Contributor

'git pull' doesn't overwrite local changes, so it should be OK to use for deployment as well; it should leave the config files alone.

@biertie
Copy link
Contributor

biertie commented Jun 15, 2015

git pull doesn't do that since the latest patch by Sasja I think. So we can already close this one, am I right?

@jvanloov
Copy link
Contributor Author

No, the idea is to modify the deployment script (devops/deploy_to_{test,prod}.sh)

Currently, running this script will retrieve a ZIP archive from github, from which various .py, .html,… files are extracted and copied into the directory of the {test,prod} installation on 172.22.32.{6,4}. I did it like that because I wanted to be specific about not overwriting configuration files specific for test or prod.

However, I noticed that if you do ‘git pull’ to update your local code, local modifications are left alone. So the scripts can be simplified by simply running ‘git pull’ on the test or prod installations. (Of course it leaves local modifications alone, it is meant to only pull in remote changes, I should have realized that, but hey…)

Please leave the issue open until the scripts are updated...

J.

Joris Van Looveren
[email protected] mailto:[email protected]
+32 496 45 67 01

Op 15-jun.-2015, om 20:40 heeft biertie [email protected] het volgende geschreven:

git pull doesn't do that since the latest patch by Sasja I think. So we can already close this one, am I right?


Reply to this email directly or view it on GitHub #9 (comment).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants