Code review comment for ~raharper/cloud-init:fix/centos-wait-network-manager

Revision history for this message
Scott Moser (smoser) wrote :

| After= won't pull NetworkManager.service into the boot graph (that
| would be Wants=), so this should only introduce a dependency on
| NetworkManager.service when it would already run during boot.
|
| I can't think of a case where this would cause problems.

Then why not go all-in?
The default case should have it.

Just move it up to after the 'systemd-networkd-wait-online.service'

« Back to merge proposal