image-builder/boot time: remove unneeded systemd units and files#290
Conversation
|
/test |
|
wow, ouch - now, that is the sort of graph we like to see @devimc :-) |
jodh-intel
left a comment
There was a problem hiding this comment.
Thanks @devimc.
A few comments...
grahamwhaley
left a comment
There was a problem hiding this comment.
lgtm
as long as nobody can think of any downsides... anything going away here that we rely on etc.
I think so, because these files are part of the |
Remove systemd units and files that are not needed in Kata Containers. Removing this files we can improve the boot time. fixes kata-containers#289 Signed-off-by: Julio Montes <julio.montes@intel.com>
875e593 to
02b3b3b
Compare
|
/test |
|
Heh, looks like we don't run the metrics CI on the osbuilder repo - I suspect the metrics CI might start failing now with this (positive) merge - and we'll maybe need an update PR'd to https://github.com/kata-containers/tests/blob/master/cmd/checkmetrics/ci_slaves/checkmetrics-json-kata-metric2.toml once we have gathered enough new data to set the new value. |
Remove systemd units and files that are not needed in Kata Containers.
Removing this files we can improve the boot time.
fixes #289
before= with all systemd files and unitsafter= without unneeded systemd files and units (this patch)cc @grahamwhaley @jodh-intel @bergwolf
Signed-off-by: Julio Montes julio.montes@intel.com