Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add capistrano for deployment and setup dev/qa stages #30

Merged
merged 1 commit into from
Apr 27, 2016
Merged

Conversation

jgrevich
Copy link
Member

@jgrevich jgrevich commented Apr 20, 2016

This adds capistrano3 support so we can deploy to remote servers. I setup dev and qa envs at https://upaya-dev.18f.gov and https://upaya-qa.18f.gov. They should be fully functional aside from SMS support; I need to setup a twilio subaccount and dev account to get that working.

To deploy to a remote server use cap <env name> deploy. We may need to whitelist your IP if you're not using the GSA VPN or in a GSA building. Similarly we'll need your ssh pub key (I have everyone from save-ferris). I created a base image with all of the OS deps and the app fully installed in case we need to spin up more envs. This should be converted to a cloud formation template if we're going to use this more often.

@monfresh
Copy link
Contributor

I think this looks good. I'll try a deployment from the office tomorrow. In the meantime, can you please fix the issues reported by Code Climate? Thanks!

@jgrevich
Copy link
Member Author

👍

We may want to remove https://github.com/18F/identity-idp/pull/30/files#diff-21497849d8f00507c9c8dcaf6288b136R1 although I kind of like it until we're closer to a prod env.

@@ -31,6 +31,7 @@ gem 'sass-rails', '~> 5.0'
gem 'secure_headers', '~> 3.0.0'
gem 'sidekiq'
gem 'simple_form', github: 'amoose/simple_form', branch: 'feature/aria-invalid'
gem 'sinatra', require: false
Copy link
Contributor

Choose a reason for hiding this comment

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

Was there a particular reason you moved sinatra to the main group? As far as I know, it's only needed for tests (to run the fake IdPaaS server).

Copy link
Contributor

Choose a reason for hiding this comment

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

I think this was inadvertent on your part. I just put it back in the test group and the deployment worked fine.

Copy link
Member Author

Choose a reason for hiding this comment

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

I added that so we can view the Sidekiq admin UI on the remote servers. We can remove that when we're closer to a production ready app or add it at a later point/separate PR.

@monfresh
Copy link
Contributor

I just tried deploying and everything worked fine, after making the changes I noted: moving sinatra back to the test group, and adding the ubuntu user in config/deploy/dev.rb.

This should be good to go after making those changes and fixing the Code Climate issues.

@jgrevich
Copy link
Member Author

CC fixes en route.

@jgrevich
Copy link
Member Author

I'll also add a snippet to the readme with deploy instructions.

@jgrevich jgrevich force-pushed the cap-deploy branch 3 times, most recently from 8694b38 to edb5088 Compare April 27, 2016 03:12
@jgrevich
Copy link
Member Author

FYI, phony_rails removes the #improbable_phone method in 0.14 and greater. I restored the 0.13.1 release for now.

@monfresh
Copy link
Contributor

🚀

@monfresh monfresh merged commit e58c25c into master Apr 27, 2016
@monfresh monfresh deleted the cap-deploy branch April 27, 2016 17:26
@monfresh
Copy link
Contributor

I looked into phony_rails. It's not that the method was removed, it's that they introduced a bug: joost/phony_rails#140

@jgrevich jgrevich restored the cap-deploy branch April 28, 2016 22:14
@jgrevich jgrevich deleted the cap-deploy branch April 28, 2016 22:15
stevegsa added a commit that referenced this pull request Jul 15, 2022
zachmargolis pushed a commit that referenced this pull request Sep 20, 2024
Better default certificate, install notes explaining the situation.
zachmargolis pushed a commit that referenced this pull request Sep 20, 2024
…adata

Remove single logout service from metadata
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants