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

Jails starting RC twice, leading to corruption #291

Closed
gizahNL opened this issue May 5, 2018 · 6 comments
Assignees

Comments

@gizahNL
Copy link

@gizahNL gizahNL commented May 5, 2018

Jails started using cbsd jstart run RC twice, starting all daemons twice. This is the case for at least vnet configured jails.

@gizahNL

This comment has been minimized.

Copy link
Author

@gizahNL gizahNL commented May 5, 2018

commenting out line 627 /usr/sbin/jexec ${jname} ${exec_start} from sudoexec/jstart fixes the issue for me.

@olevole olevole self-assigned this May 5, 2018
@olevole

This comment has been minimized.

Copy link
Collaborator

@olevole olevole commented May 5, 2018

Hi, you observe this behavior only for vnet jails? I need time to test this

@gizahNL

This comment has been minimized.

Copy link
Author

@gizahNL gizahNL commented May 5, 2018

I have not tested this outside of vnet jails. I can take a look tomorrow.

@olevole

This comment has been minimized.

Copy link
Collaborator

@olevole olevole commented May 5, 2018

Yes, this is vnet-only issue (already tested). Thanks, I will fix this error ASAP and prepare 11.1.20 bugfix release.

olevole added a commit that referenced this issue May 5, 2018
We move exec_start sequence into jstart script for vnet due to
need to split atomic jail start for VNET and ZFS-attached jail mode.
For some wrong reason we execute this sequence by twice
which in some cases can lead to corruption.
Reported, debug and fixes by Gijs Peskens aka gizahNL. Tested by olevole.
Issue #291
@olevole

This comment has been minimized.

Copy link
Collaborator

@olevole olevole commented May 5, 2018

It seems everything is in order now. I'm preparing a new release. Thanks!

@olevole olevole closed this May 5, 2018
@olevole

This comment has been minimized.

Copy link
Collaborator

@olevole olevole commented May 6, 2018

Commited into ports tree, v11.20, Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.