Skip to content

Navi proxy + clustering#477

Merged
anandkumarpatel merged 17 commits intomasterfrom
navi-lb
May 27, 2016
Merged

Navi proxy + clustering#477
anandkumarpatel merged 17 commits intomasterfrom
navi-lb

Conversation

@anandkumarpatel
Copy link
Copy Markdown

@anandkumarpatel anandkumarpatel commented May 25, 2016

  • removed https and clustering from navi configs
  • unified getting ports for proxy services
  • added default nginx config file

Dependencies

  • navi pr

Reviewers

Tests

Test any modifications on one of our environments.

  • deploy api
  • deploy eru
  • deploy navi

Deployment (post-merge)

Ensure that all environments have the given changes.

  • deployed to epsilon
  • deployed to gamma
  • deployed to delta

@anandkumarpatel anandkumarpatel changed the title Navi lb Navi proxy + clustering May 25, 2016
Comment thread ansible/group_vars/alpha-navi-proxy.yml Outdated
name: nginx

docker_image: "{{ name }}"
docker_image_version: "1.10"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

1.10?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

nope, ansible turns that into 1.1, need the quotes

-v /etc/nginx/sites-available/:/etc/nginx/sites-enabled/:ro
-v /etc/nginx/ssl/dhparam.pem:/etc/nginx/ssl/dhparam.pem:ro
-v /etc/ssl/certs/{{ user_content_domain }}:/etc/ssl/certs/{{ user_content_domain }}:ro
-v /var/log/nginx:/var/log/nginx
Copy link
Copy Markdown
Contributor

@bkendall bkendall May 26, 2016

Choose a reason for hiding this comment

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

I think @und1sk0 was coming up with a plan with @rsandor to log nginx logs - may want to follow up with them later (though, I don't know if nginx logging is super important here [read: do it later])

Anandkumar Patel added 2 commits May 25, 2016 18:44
container_image: swarm
container_tag: 1.2.0
container_image: soft-swarm
container_tag: latest
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I assume you didn't mean to check this in?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

yep you right

@anandkumarpatel anandkumarpatel merged commit f71f555 into master May 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants