<div dir="ltr"><div>Hi,</div><div>I've realized we have a double opt-in atm.</div><div>1. enable the service</div><div>2. change the config file</div><div><br></div><div>After install:</div><div>systemctl status dpdk</div><div>● dpdk.service - DPDK runtime environment</div><div>   Loaded: loaded (/lib/systemd/system/dpdk.service; disabled; vendor preset: enabled)</div><div>   Active: inactive (dead)</div><div><br></div><div>ubuntu@node-horsea:~$ systemctl is-enabled dpdk</div><div>disabled</div><div><br></div><div>Not super-bad, as it is meant to be an opt-in.</div><div>But since we made the config in a way to be (mostly) a no-op by default I wonder if we should change that.</div><div><br></div><div>Currently the opt-in is enabling the service + changing the conf.</div><div>This could be only changing the conf.</div><div><br></div><div>I think we would need a dh_systemd_enable to achive the right thing, but I wanted to poll for opinions first.</div><div><br></div><div>One more detail - we currently "fix up" vm.max_map_count in any case that has hugepages. If wanted for a default on we could guard that with a check to the conf file.</div><div><br></div><div>In other places I've seen an more explicit</div><div> SERVICE_ENABLED=FALSE</div><div>in the config to make it very clear to opt-in - maybe such a thing would be ok as well.</div><div><br></div><div>Yet if we would do that we would have some work for upgrades.</div><div>Maybe the value not being set means enabled (as it did so far) but if the default from the conffile is set (FALSE) it skips the service tasks.</div><div><br></div><div>Too much options/styles to do so, so before a  patch thoughts?</div><div><br></div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><span style="color:rgb(136,136,136);font-size:12.8px">Christian Ehrhardt</span><div style="color:rgb(136,136,136);font-size:12.8px">Software Engineer, Ubuntu Server</div><div style="color:rgb(136,136,136);font-size:12.8px">Canonical Ltd</div></div></div></div></div>
</div>