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

Setup playbook #599

Open
wants to merge 2,411 commits into
base: master
Choose a base branch
from
Open

Setup playbook #599

wants to merge 2,411 commits into from

Conversation

thejsj
Copy link
Member

@thejsj thejsj commented Jan 30, 2017

What this PR does

  • Add dynamic ports for all HTTP accessible services
  • Add new environment: bear-clone
  • Fix socket-server proxy setup
  • Make Rollbar, Intercom and Loggly keys optional
  • Create proxy for Drake
  • Create a single-host-proxy to proxy all requests in a single host
  • Fix setting of ENVs for Shiva, API, other services
  • Fix container_kill_start role to also handle names
  • Add new IP address

TODO

  • Fix {{ name }} issue for kill container start
  • Figure out what to do with certs

Tests

Test any modifications on one of our environments.

  • tested on environment by someone

Deployment (post-merge)

Ensure that all environments have the given changes.

  • deployed to gamma
  • deployed to delta

Myztiq and others added 30 commits August 25, 2016 16:01
Add deploy songs for big-poppa and cream
…er-headers

Increased header size to avoid 502 gateway errors
thejsj and others added 27 commits February 9, 2017 16:08
Added ability to deploy to marketing again
…-playbook

* 'master' of github.com:CodeNow/devops-scripts:
  Change region name
  Added marketing deploy keys
  Updated deploy bucket for marketing.
  Add bucket region for web
  remove epsilon (#605)
  change version (#604)
  Tags
  Remove '-'
  PR comments
  Added postgres connect strings
  Fix osx removal of deamon (#592)
  Container Kill Start
  Add >65000 port for deployment
  Change node version
  Added port options
  Added user content domain var
  PR comments
  Changed names to agreeable-egret
  Added egret and env vars
[bastion]
gamma-bastion

[hipache]

Choose a reason for hiding this comment

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

did you remove this?

@@ -0,0 +1,170 @@
[api_group:vars]
api_aws_access_key_id=AKIAIVCZL2ARDILEI6AA

Choose a reason for hiding this comment

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

we are going to burn in 👿 🔥

@@ -22,6 +22,7 @@ api_workers_rollbar_key=3edfe8fe4fd640ae9fdbbe08fcb9f121
api_s3_context_bucket=runnable.context.resources.production
api_intercom_app_id=wqzm3rju
api_intercom_api_key=46e75ada5d21f248787689b35fe80e11efe9303a
api_github_deploy_key_title=Runnable-io

Choose a reason for hiding this comment

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

should this be {{ domain }}

@@ -23,20 +23,29 @@ api_base_container_envs: >-
-e AWS_SECRET_ACCESS_KEY={{ api_aws_secret_access_key }}

Choose a reason for hiding this comment

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

Start next review from here

@anandkumarpatel
Copy link

@thejsj is this good to close since we dont need this anymore?

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.

8 participants