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

fails to update IP's on existing jail #2

Open
dgeo opened this issue Aug 15, 2018 · 7 comments
Open

fails to update IP's on existing jail #2

dgeo opened this issue Aug 15, 2018 · 7 comments
Assignees

Comments

@dgeo
Copy link
Contributor

dgeo commented Aug 15, 2018

Have to test others properties too…

@mike-pt
Copy link

mike-pt commented Nov 25, 2018

I was actually just noticing that properties are not updated at all... I have:
state: set in the playbook and even then it just seems to do nothing if jails are already created

I.E. I update vnet:on and dhcp:on and its not reflecting on the jail

I see:

TASK [criecm.iocage : create jail] **************************************************************************************
ok: [fbsdtest]

which sugests there was no need for changes

@mike-pt
Copy link

mike-pt commented Nov 25, 2018

oh ofc set jail is never run... we don't have a task for it

https://github.com/criecm/ansible-role-iocage/blob/master/tasks/jail.yml#L6

@dgeo
Copy link
Contributor Author

dgeo commented Nov 25, 2018

This should be set by iocage module (where state='create' also means 'present' here), feel free to PR there: https://github.com/criecm/ansible-iocage

@mike-pt
Copy link

mike-pt commented Nov 25, 2018

ah right... I was mostly testing this stuff but I'd love to contribute... just have time on weekends though...

Do you have any plans on moving to iocage-devel? I find that it works best for VNET jails although consider experimental looks very stable... I only prefer the fact that we can get json output in most things, which makes it easy to parse as python dicts

@dgeo dgeo added the bug label Nov 26, 2018
@dgeo dgeo self-assigned this Nov 26, 2018
@dgeo dgeo added the module label Nov 26, 2018
@dgeo
Copy link
Contributor Author

dgeo commented Nov 28, 2018

I'm not planning to use -devel for master now (maybe in a -devel branch ?): I need real stability here, and don't have much time for dev... Top much Ops ;)

But I'd be very interested in such a (re-)work the day iocage reach next major release !

I'm following dev there and it seems there are great things to do here (like import iocage lib directly for exemple)

Last time I played with vnet (some monthes ago) my machine had bad times (PF/VIMAGE old but serious bugs), I'll re-test with 12.0

@mike-pt
Copy link

mike-pt commented Nov 28, 2018

Its better now, Its even enabled on 12.0 (no need to rebuild kernel) but even in 11 I have stable servers with vnet and pf (even in jails) but I use jail.conf and netgraph for that... epair didn't like pf in jails... which is what iocage actually uses :/

Its a pity I don't have much time now too I wouldn't mind helping with a devel branch, but hopefully I can still submit some PRs once in while 🤞

@dgeo
Copy link
Contributor Author

dgeo commented Nov 28, 2018

You are welcome :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants