Looks good. I've got a couple of comment (see inline) but more importantly:
Now that boot_type is a property on the node, we don't really need the should_use_fastpath_installer/should_use_traditional_installer encapsulations anymore.
Actually, if you exclude the tests, should_use_fastpath_installer is used only once and should_use_traditional_installer is not even used anymore (!).
Looks good. I've got a couple of comment (see inline) but more importantly:
Now that boot_type is a property on the node, we don't really need the should_ use_fastpath_ installer/ should_ use_traditional _installer encapsulations anymore.
Actually, if you exclude the tests, should_ use_fastpath_ installer is used only once and should_ use_traditional _installer is not even used anymore (!).