Skip to content

fix custom driver registration after reboot #140

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

Merged
merged 1 commit into from
Nov 16, 2020

Conversation

LeonidVas
Copy link
Contributor

Previously, if a custom driver is registered after calling box.cfg() it causes a problem when the instance will be restarted. The cause of the problem is an attempt to recreate a tube of custom type (which is not registered yet).
This bug is now fixed.

Fixes #137

@ChangeLog: fix custom driver registration after reboot. Previously, if a custom driver is registered after calling box.cfg() it causes a problem when the instance will be restarted. The cause of the problem is an attempt to recreate a tube of custom type (which is not registered yet).

@LeonidVas LeonidVas requested review from Mons and Totktonada November 10, 2020 23:24
@LeonidVas LeonidVas force-pushed the lvasiliev/gh-137-custom-driver-registration branch from aaba414 to a7a221a Compare November 11, 2020 08:34
@LeonidVas LeonidVas closed this Nov 11, 2020
@LeonidVas LeonidVas reopened this Nov 11, 2020
@LeonidVas LeonidVas requested a review from ligurio November 11, 2020 11:02
Previously, if a custom driver is registered after calling box.cfg()
it causes a problem when the instance will be restarted. The cause
of the problem is an attempt to recreate a tube of custom type
(which is not registered yet).
This bug is now fixed.

Fixes #137
@LeonidVas LeonidVas force-pushed the lvasiliev/gh-137-custom-driver-registration branch from a7a221a to 65b1683 Compare November 12, 2020 15:12
@LeonidVas LeonidVas requested a review from ligurio November 12, 2020 18:13
Copy link
Member

@Totktonada Totktonada left a comment

Choose a reason for hiding this comment

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

I didn't looked at the code thoroughly, but the idea looks good for me.

Copy link
Member

@ligurio ligurio left a comment

Choose a reason for hiding this comment

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

Leonid, thanks for the patch!

LGTM

@LeonidVas LeonidVas merged commit 905d912 into master Nov 16, 2020
@LeonidVas LeonidVas deleted the lvasiliev/gh-137-custom-driver-registration branch November 16, 2020 08:36
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.

Fix custom driver registration after box.cfg()
3 participants