OCPBUGS-34493: baremetal: Don't always enable provisioning-interface.service

zaneb · zaneb · commit 82690cc88e58 · 2024-05-29T14:49:38.000+12:00
In the baremetal bootstrap it is not sufficient to make ironic.service conditionally Require provisioning-interface.service, as 104eae1 attempted to do, when the latter is already enabled because it is WantedBy multi-user.target. As long as it is enabled, systemd will try to start it. Do not install provisioning-interface.service, so that it will only be enabled when ironic.service requires it (i.e. when the provisioning network is not disabled).
diff --git a/data/data/bootstrap/baremetal/systemd/units/provisioning-interface.service.template b/data/data/bootstrap/baremetal/systemd/units/provisioning-interface.service.template
@@ -16,4 +16,3 @@ Restart=on-failure
 RestartSec=10
 
 [Install]
-WantedBy=multi-user.target

Original file line number	Diff line number	Diff line change
`@@ -16,4 +16,3 @@ Restart=on-failure`
`16`	`16`	`RestartSec=10`
`17`	`17`
`18`	`18`	`[Install]`
`19`		`-WantedBy=multi-user.target`