https://pfspipe.ipmu.jp/jira This file is an XML representation of an issue en-us 8.3.4 803005 13-09-2019 https://pfspipe.ipmu.jp/jira/browse/INSTRM-241 Instrument control development <p>At some init script, invoke of daemon will exit when /etc/dnsmasq.d/README is missing.<br/> Add blank file...</p> <p>&gt; # /etc/dnsmasq.d/README is a non-conffile installed by the dnsmasq package.<br/> &gt; # Should the dnsmasq package be removed, the following test ensures that<br/> &gt; # the daemon is no longer started, even if the dnsmasq-base package is<br/> &gt; # still in place.<br/> &gt; test -e /etc/dnsmasq.d/README || exit 0</p> INSTRM-241

Add blank README

Task Normal Won't Fix Won't Fix shimono shimono Tue, 24 Oct 2017 16:10:23 +0000 Wed, 6 Dec 2017 20:55:17 +0000 Wed, 6 Dec 2017 20:55:17 +0000 ics_ansible ics_dnsmasq 0 3 <p>I'm not sure I understand: are you saying that (some) dnsmasq packages already use README in that way and so we always need to provide one? Or are you saying that you want to add that logic to avoid some later problem?</p> <p>Above lines are copied from /etc/init.d/dnsmasq file in my environment, which is a bit old.<br/> so, it was fine in new production environment. but I think LAM is in the same version (<a href="https://pfspipe.ipmu.jp/jira/secure/ViewProfile.jspa?name=arnaud.lefur" class="user-hover" rel="arnaud.lefur">arnaud.lefur</a> no?)?</p> <p>&gt; $ apt show dnsmasq<br/> &gt; Package: dnsmasq<br/> &gt; Version: 2.72-3+deb8u2</p> <div class="preformatted panel" style="border-width: 1px;"><div class="preformattedContent panelContent"> <pre>Package: dnsmasq Version: 2.72-3+deb8u2 </pre> </div></div> <p>Yes, that's the same version.</p> <p>I think I should made some mistake. </p> <ul> <li>README was originally installed by the package itself, and init script depends on that file</li> <li>ansible role downloads git repository to /etc/dnsmasq.d, and to avoid fatal error, role deletes README before execution of git</li> </ul> <p>So, it seems, I shall change ansible role rather than adding blank README to our dnsmasq repository.</p> <ul> <li>clearing the target directory of configuration (/etc/dnsmasq.d/) is required for git clone</li> <li>backup/restore README file is not good on idempotency point of view</li> <li>current active version does not depend on README file in init script, and no reason to install old version for new installation</li> </ul> <p>therefore, I'd propose to</p> <ul> <li>mark this as wontfix</li> <li>just 'touch README' if required in existing installation, but not to have README in repository</li> </ul> <p>For updating configuration files of dnsmasq, it would be easier just to run git pull (and reload service) but not to re-run ansible role.</p> <p>mark as in review for approval to move this into WONTFIX.</p> <p>Since current versions do not depend on the README, I agree that we can ignore it.</p> Development Rank 0|ii03fz: Reviewers cloomis