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

Protected socket on manager nodes #1786

Closed
wants to merge 7 commits into from

Conversation

ndegory
Copy link
Contributor

@ndegory ndegory commented Dec 13, 2017

PR cut in smaller pieces: #1791 #1792 #1793 and #1794


  • Fix AMP-102 (enable TLS on manager nodes), see details in https://techweb.axway.com/confluence/display/AMP/Cluster+with+secured+Docker+API
  • Fix AMP-132 (AMI updated to Docker CE 17.09.1)
  • Internal PKI for Docker daemon certificates
  • all manager nodes expose the engine API with TLS enabled
  • all nodes get a unique client certificate give access to the manager nodes API
  • fixes in pkg/docker package to allow connection with TLS options
  • move Prometheus to non manager nodes
  • move Amplifier to non manager nodes
  • ampagent can deploy on remote manager nodes
  • ampagent pass TLS options that can be used in core service stacks
  • templating for core stack files

Not in this PR:

How to check

  • deploy a cluster on AWS
  • check that the dashboard are able to display data about the cluster
  • list the service, amplifier should be on a worker node
  • signup to amp
  • deploy a stack with a service requiring access to the Docker API on a manager
    • mount the docker certificates: - /etc/docker/tls/client:/root/.docker
    • add an env var with the url to the manager LB: - DOCKER_HOST: ${InternalDockerHost} # this value can be read in amp cluster status)

@ndegory ndegory changed the title WIP: Protected socket on manager nodes Protected socket on manager nodes Dec 13, 2017
@ndegory ndegory force-pushed the cfssl branch 2 times, most recently from 6981784 to e4f9fb8 Compare December 13, 2017 20:52
@ndegory
Copy link
Contributor Author

ndegory commented Dec 19, 2017

PR deemed too complex for a single review, closed and replaced by smaller PRs: #1791 #1792 #1793 and #1794

@ndegory ndegory closed this Dec 19, 2017
@ndegory ndegory deleted the cfssl branch December 19, 2017 17:17
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.

1 participant