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

sidecar + gossip can listen to a custom ip:port #499

Merged
merged 2 commits into from
May 9, 2016

Conversation

bookshelfdave
Copy link
Contributor

hab-sup start behavior is as follows:

  • without --listen-peer
    • gossip starts on util::sys::ip() : DEFAULT_GOSSIP_LISTEN_PORT
  • with --listen-peer IP without port
    • gossip starts on DEFAULT_GOSSIP_LISTEN_PORT
  • with --listen-peer IP:PORT
    • gossip starts on IP:PORT

Misc:

  • remove rpassword dep, it was used with GPG

@thesentinels
Copy link
Contributor

By analyzing the blame information on this pull request, we identified @adamhjk, @reset, @fnichol, @juliandunn and @jtimberman to be potential reviewers

`hab-sup start` behavior is as follows:

- without `--listen-peer`
    - gossip starts on `util::sys::ip()` : `DEFAULT_GOSSIP_LISTEN_PORT`
- with `--listen-peer` IP without port
    - gossip starts on `IP:GOSSIP_DEFAULT_PORT`
- with `--listen-peer` IP:PORT
    - gossip starts on `IP:PORT`

- remove rpassword dep, it was used with GPG

Signed-off-by: Dave Parfitt <[email protected]>
Signed-off-by: Dave Parfitt <[email protected]>
@adamhjk
Copy link
Contributor

adamhjk commented May 9, 2016

This is pure noodling, but is v6 support just a couple of functions away? I haven't even looked. My guess is that it is.

@bookshelfdave
Copy link
Contributor Author

bookshelfdave commented May 9, 2016

@adamhjk I think so, it looks like utp supports IPv6 and we have std::net::Ipv6Addr available to us.

@adamhjk
Copy link
Contributor

adamhjk commented May 9, 2016

I think we take this as it is, and we should think about expanding all our bindings/references everywhere to work with v6 as well as v4. Agree?

@adamhjk
Copy link
Contributor

adamhjk commented May 9, 2016

(we can do that later - someone will request it eventually :)

@bookshelfdave
Copy link
Contributor Author

+1, I'd love to see IPv6

On May 9, 2016, at 5:30 PM, Adam Jacob [email protected] wrote:

I think we take this as it is, and we should think about expanding all our bindings/references everywhere to work with v6 as well as v4. Agree?


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

@adamhjk
Copy link
Contributor

adamhjk commented May 9, 2016

gif-keyboard-11765323913371214806

@thesentinels r+

@thesentinels
Copy link
Contributor

📌 Commit 60d83c7 has been approved by adamhjk

@thesentinels
Copy link
Contributor

⌛ Testing commit 60d83c7 with merge cca10fe...

thesentinels pushed a commit that referenced this pull request May 9, 2016
`hab-sup start` behavior is as follows:

- without `--listen-peer`
    - gossip starts on `util::sys::ip()` : `DEFAULT_GOSSIP_LISTEN_PORT`
- with `--listen-peer` IP without port
    - gossip starts on `IP:GOSSIP_DEFAULT_PORT`
- with `--listen-peer` IP:PORT
    - gossip starts on `IP:PORT`

- remove rpassword dep, it was used with GPG

Signed-off-by: Dave Parfitt <[email protected]>

Pull request: #499
Approved by: adamhjk
thesentinels pushed a commit that referenced this pull request May 9, 2016
Signed-off-by: Dave Parfitt <[email protected]>

Pull request: #499
Approved by: adamhjk
@thesentinels
Copy link
Contributor

☀️ Test successful - travis

@thesentinels thesentinels merged commit 60d83c7 into master May 9, 2016
jtimberman pushed a commit that referenced this pull request Jun 12, 2016
`hab-sup start` behavior is as follows:

- without `--listen-peer`
    - gossip starts on `util::sys::ip()` : `DEFAULT_GOSSIP_LISTEN_PORT`
- with `--listen-peer` IP without port
    - gossip starts on `IP:GOSSIP_DEFAULT_PORT`
- with `--listen-peer` IP:PORT
    - gossip starts on `IP:PORT`

- remove rpassword dep, it was used with GPG

Signed-off-by: Dave Parfitt <[email protected]>

Pull request: #499
Approved by: adamhjk
jtimberman pushed a commit that referenced this pull request Jun 12, 2016
Signed-off-by: Dave Parfitt <[email protected]>

Pull request: #499
Approved by: adamhjk
@bookshelfdave bookshelfdave deleted the dp_config_ports branch July 12, 2016 13:57
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.

3 participants