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

Added the ip_hostname option when creating a jail #702

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

bramton
Copy link

@bramton bramton commented May 31, 2024

The ip_hostname option allows the jail to resolve it's own IP address. This is very useful in cases where you run your own DNS resolver and ensures that IP addresses can be managed at a single location.

The create subcommand documentation has been modified to reflect this change. In addition, the SLAAC and DHCP options are also added.

@yaazkal yaazkal added the enhancement New feature or request label Jul 8, 2024
@yaazkal
Copy link
Collaborator

yaazkal commented Jul 14, 2024

Note that this is only going to work if there are good DNS configuration and also if the hostname has been given correctly to the jail, which is not Bastille responsibility; actually, the user should take care if using this on a jail serving DNS. Why include this as a creation option and not let the user establish it using the config command?

@bramton
Copy link
Author

bramton commented Jul 16, 2024

Agreed. The user has full responsibility for running a working DNS solution. I tried with the config command, which works, but is a bit more cumbersome. Here is my recipe:

bastille create bogus 14.0-RELEASE 1.2.3.4
bastille config bogus set ip_hostname true
bastille config bogus set ip4.addr

Goal of the last line is to remove the ip4 address from the config file, but the downside is that the last line boils down to ip4.addr; in the jail.conf file. A subsequent change of the ip4 address will add an additional line to the jail.conf file. E.g. after running bastille config bogus set ip4.addr 42.42.42.42, the jail.conf will look like this:

ip4.addr;
ip4.addr = 42.42.42.42;

The config file is still functional, but not very pretty. I am not sure if there is a way to prevent this, any thoughts?

@yaazkal
Copy link
Collaborator

yaazkal commented Jul 16, 2024

OK. I don't want to close this PR since maybe anyone argues for that option at the creation time. Labeling now as "help wanted" if anyone wants to join the conversation. Let's also see what @cedwards thinks about this option. What I don't want is a plethora of issues in the future with the "sorry, it was DNS" as an answer.

That said, you are pointing that the config command maybe wants to have an unset option, which could be nice improvement. Please open a new PR with that feature if you want.

About the subsequent change of the ip.addr4, that could be a bug then, but the new unset option for the config command should prevent it in the first place.

Thanks for the feedback.

@yaazkal yaazkal added the help wanted Extra attention is needed label Jul 16, 2024
@yaazkal yaazkal requested a review from cedwards July 22, 2024 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants