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

Allow multiple IP addresses per jail #68

Open
dlangille opened this issue Mar 6, 2015 · 5 comments
Open

Allow multiple IP addresses per jail #68

dlangille opened this issue Mar 6, 2015 · 5 comments

Comments

@dlangille
Copy link
Contributor

When creating a jail, we should be able to have multiple IP addresses.

@tomster
Copy link
Contributor

tomster commented Mar 9, 2015

@fschulze : FTR atm ploy_ezjail actually breaks, when jails with multiple IPs are present (i.e. if they were added manually).

it barfs at the output of ezjail-admin list:

STA JID  IP              Hostname                       Root Directory
--- ---- --------------- ------------------------------ ------------------------
ZR  2    10.0.1.20       var           /usr/jails/bar
DS  N/A  10.0.1.21       foo                            /usr/jails/foo
    N/A  10.0.1.22

like so:

# ploy -c etc/fitze_vbox.conf start webserver
Traceback (most recent call last):
  File "/Users/tomster/Development/tools/bin/ploy", line 42, in <module>
    sys.exit(ploy.ploy())
  File "/Users/tomster/Development/bsdploy/src/ploy/ploy/__init__.py", line 557, in ploy
    return ctrl(argv)
  File "/Users/tomster/Development/bsdploy/src/ploy/ploy/__init__.py", line 549, in __call__
    args.func(sub_argv, args.func.__doc__)
  File "/Users/tomster/Development/bsdploy/src/ploy/ploy/__init__.py", line 284, in cmd_start
    result = instance.start(overrides)
  File "/Users/tomster/Development/bsdploy/src/ploy_ezjail/ploy_ezjail/__init__.py", line 120, in start
    jails = self.master.ezjail_admin('list')
  File "/Users/tomster/Development/bsdploy/src/ploy_ezjail/ploy_ezjail/__init__.py", line 451, in ezjail_admin
    jails[entry.pop('name')] = entry
KeyError: 'name'

@fschulze
Copy link
Member

fschulze commented Mar 9, 2015

Thanks for the output dump, that will allow me to write a test and fix this.

So what about the syntax in ploy.conf?
I think we should just have a comma separated list for the ip option and use the first IP for ssh connections etc.

@Infiniverse
Copy link
Contributor

Is this still an issue?

@fschulze
Copy link
Member

Yes, I got no feedback on my last question and thus haven't implemented it yet.

@Infiniverse
Copy link
Contributor

Happy to give some feedback myself on this issue, but I need to resolve how to bind to different interfaces first (other than lo1). (Raised an issue about this in the bsdploy-ezjail repo).

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

No branches or pull requests

4 participants