Skip to content

Conversation

phlogistonjohn
Copy link
Collaborator

Add a new --address option that takes an IP address values (v4 or v6) to specify exactly what address to listen on.

Internally, a slice of IP objects is used but we currently only support binding to one of them. Checking for an empty slice is easy and there's a chance we will want multiple listeners in the future but to keep things somewhat simple the CLI and listener only allows for one.

The goal of this feature is to eventually support colocation of smb services on a single node by having one set of smb related services bind only to a given address and a different set of smb services bind only to a different address on the same host.

Add a new --address option that takes an IP address values (v4 or v6) to
specify exactly what address to listen on.

Internally, a slice of IP objects is used but we currently only support
binding to one of them. Checking for an empty slice is easy and there's
a chance we will want multiple listeners in the future but to keep
things somewhat simple the CLI and listener only allows for one.

The goal of this feature is to eventually support colocation of smb
services on a single node by having one set of smb related services
bind only to a given address and a different set of smb services
bind only to a different address on the same host.

Signed-off-by: John Mulligan <[email protected]>
Copy link
Contributor

@anoopcs9 anoopcs9 left a comment

Choose a reason for hiding this comment

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

lgtm.

@phlogistonjohn
Copy link
Collaborator Author

@synarete ping

Copy link
Collaborator

@synarete synarete left a comment

Choose a reason for hiding this comment

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

LGTM

@synarete synarete merged commit a05f44e into samba-in-kubernetes:main Jun 23, 2025
14 of 15 checks passed
@phlogistonjohn phlogistonjohn deleted the jjm-bind-addrs branch June 24, 2025 20: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.

3 participants