Skip to content

Code deploy#48

Merged
hbradio merged 30 commits into
developmentfrom
feature/codedeploy
Mar 3, 2017
Merged

Code deploy#48
hbradio merged 30 commits into
developmentfrom
feature/codedeploy

Conversation

@hbradio

@hbradio hbradio commented Mar 2, 2017

Copy link
Copy Markdown
Collaborator

Makes travis automatically deploy development commits to beta.rovercode.com and master commits to rovercode.com.

@hbradio hbradio added this to the Release 0.3 milestone Mar 2, 2017
@hbradio hbradio requested a review from cabarnes March 2, 2017 18:30
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling 711d841 on feature/codedeploy into 7d6f1da on development.

Comment thread .travis.yml
- sudo apt-get install -qq graphviz-dev python-setuptools python3-dev python-virtualenv python-pip
- sudo apt-get install -qq firefox automake libtool libreadline6 libreadline6-dev libreadline-dev
- sudo apt-get install -qq libsqlite3-dev libxml2 libxml2-dev libssl-dev libbz2-dev wget curl llvm
- sudo apt-get update -qq

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The other .yml files still have this indentation. Why did you change these?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I used the travis client tool to generate the deploy stuff, and it must have changed those indents. I left if because I figured it knew best.

Comment thread deploy-scripts/remove_prev_install.sh Outdated
@@ -0,0 +1,5 @@
#!/bin/bash
pushd /home/ubuntu/rovercode-web > /dev/null
docker-compose stop

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you run docker-compose down it will remove the container as well

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You might want to also want to delete the image because each docker-compose build will create a new one.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good ideas.

Comment thread deploy-scripts/start_app.sh Outdated
@@ -0,0 +1,4 @@
#!/bin/bash
cd /home/ubuntu/rovercode-web

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You may want to use && to combine all of these. Each is dependent on the previous step succeeding.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea.

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling 4a1aaa0 on feature/codedeploy into 7d6f1da on development.

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling eca3e52 on feature/codedeploy into 7d6f1da on development.

@hbradio

hbradio commented Mar 3, 2017

Copy link
Copy Markdown
Collaborator Author

Ok, changes done!

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling d601e3a on feature/codedeploy into 7d6f1da on development.

@hbradio hbradio merged commit a9ee60b into development Mar 3, 2017
@hbradio hbradio deleted the feature/codedeploy branch March 3, 2017 04:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants