Skip to content

Open erlang port range up to support greater than 2 nodes in a cluster#46

Closed
chriswiggins wants to merge 1 commit intoapache:masterfrom
chriswiggins:fix-erlang-portrange
Closed

Open erlang port range up to support greater than 2 nodes in a cluster#46
chriswiggins wants to merge 1 commit intoapache:masterfrom
chriswiggins:fix-erlang-portrange

Conversation

@chriswiggins
Copy link
Copy Markdown

Overview

Erlang portrange in 2.1.1 vm.args was restricted to a single port. This changes that so more than 2 nodes can be supported in a cluster

Testing recommendations

Run the docker container with the new vm.args

GitHub issue number

N/A

Related Pull Requests

N/A

Checklist

  • Code is written and works correctly;
  • Changes are covered by tests;
  • Documentation reflects the changes;

@wohali
Copy link
Copy Markdown
Member

wohali commented Nov 27, 2017

This is actually not necessary. See c726be0

As long as you're only running one node per container, you only need one port open.

@wohali wohali closed this Nov 27, 2017
@chriswiggins
Copy link
Copy Markdown
Author

chriswiggins commented Nov 27, 2017 via email

@wohali
Copy link
Copy Markdown
Member

wohali commented Nov 28, 2017

Oof. You're right, but that's not a recommended deployment scenario. You always want the Erlang distribution traffic to be on a private network, as much as possible. Why wouldn't you leverage the mechanism Docker already has in place for that?

@chriswiggins
Copy link
Copy Markdown
Author

@wohali I'm running the docker container on 3 separate VMs, which are on the same Layer-2 network segment, but different hosts. They can't share private docker networks (to my knowledge anyway but happy to be proven wrong!) Unless I'm deploying this in a weird fashion? My docker hosts are statically configured so there is no orchestration going on under the hood, just my fingers on the keyboard and the CLI!

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.

2 participants