<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 29, 2018 at 6:41 AM, Jehan-Guillaume de Rorthais <span dir="ltr"><<a href="mailto:jgdr@dalibo.com" target="_blank">jgdr@dalibo.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, 28 Mar 2018 12:40:25 -0500<br>
Ken Gaillot <<a href="mailto:kgaillot@redhat.com">kgaillot@redhat.com</a>> wrote:<br>
<br>
> Hi all,<br>
><br>
> Andrew Beekhof brought up a potential change to help with reading<br>
> Pacemaker logs.<br>
><br>
> Currently, pacemaker daemon names are not intuitive, making it<br>
> difficult to search the system log or understand what each one does.<br>
><br>
> The idea is to rename the daemons, with a common prefix, and a name<br>
> that better reflects the purpose.<br>
><br>
> I think it's a great idea, but we have to consider two drawbacks:<br>
><br>
> * I'm about to release 2.0.0-rc2, and it's late in the cycle for a<br>
> major change. But if we don't do it now, it'll probably sit on the back<br>
> burner for a few years, as it wouldn't make sense to introduce such a<br>
> change shortly after a major bump.<br>
><br>
> * We can change *only* the names used in the logs, which will be<br>
> simple, but give us inconsistencies with what shows up in "ps", etc. Or<br>
> we can try to change everything -- process names, library names, API<br>
> function/structure names -- but that will impact other projects such as<br>
> sbd, crmsh, etc., potentially causing compatibility headaches.<br>
><br>
> What are your thoughts? Change or not?<br>
<br>
</span>change<br>
<br>
> Now or later?<br>
<br>
I'm not sure how much work it involves during rc time... But I would pick now<br>
if possible.<br>
<br>
> Log tags, or everything?<br>
<br>
Everything.<br>
<br>
I'm from the PostgreSQL galaxy. In this galaxy, parameter<br>
"update_process_title" controls if PostgreSQL should set human readable process<br>
title and is "on" by default. In ps, it gives:<br>
<br>
  ioguix@firost:~$ ps f -o cmd -C postmaster<br>
  CMD<br>
  postmaster -D /home/ioguix/var/lib/pgsql/96<br>
   \_ postgres: checkpointer process<br>
   \_ postgres: writer process<br>
   \_ postgres: wal writer process<br>
   \_ postgres: autovacuum launcher process<br>
   \_ postgres: stats collector process<br>
<br>
Some process might even add some useful information about their status, eg.:<br>
current replication status and location (process wal receiver/sender) or last<br>
WAL archived (process archiver).<br>
<br>
In source code, it boils down to this function:<br>
<br>
<a href="https://git.postgresql.org/gitweb/?p=postgresql.git;a=blob;f=src/backend/utils/misc/ps_status.c;h=5742de0802a54e38a2c2e3cfa8e8f445b6822883;hb=65c6b53991e1c56f6a0700ae26928962ddf2b9fe#l321" rel="noreferrer" target="_blank">https://git.postgresql.org/<wbr>gitweb/?p=postgresql.git;a=<wbr>blob;f=src/backend/utils/misc/<wbr>ps_status.c;h=<wbr>5742de0802a54e38a2c2e3cfa8e8f4<wbr>45b6822883;hb=<wbr>65c6b53991e1c56f6a0700ae269289<wbr>62ddf2b9fe#l321</a></blockquote><div><br></div><div>sbd also has similar code</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<span class=""><br>
> And the fun part, what would we change them to ...<br>
><br>
> Beekhof suggested renaming "pengine" to "cluster-planner", as an<br>
> example.<br>
><br>
> I think a prefix indicating pacemaker specifically would be better than<br>
> "cluster-" for grepping and intuitiveness.<br>
><br>
> For intuitiveness, long names are better ("pacemaker-FUNCTION"). On the<br>
> other hand, there's an argument for keeping names to 15 characters,<br>
> which is the default "ps" column width, and a reasonable limit for log<br>
> line tags. Maybe "pm-" or "pcmk-"? This prefix could also be used for<br>
> library names.<br>
<br>
</span>"pcmk-*" sounds better to me. "cluster" has so many different definiion in<br>
people mind...<br>
<span class=""><br>
> Looking at other projects with server processes, most use the<br>
> traditional "d" ending (for example, "rsyslogd"). A few add "-daemon"<br>
> ("rtkit-daemon"), and others don't bother with any suffix ("gdm").<br>
><br>
> Here are the current names, with some example replacements:<br>
><br>
>  pacemakerd: PREFIX-launchd, PREFIX-launcher<br>
<br>
</span>pacemakerd, alone, sounds perfectly fine to me.<br>
<br>
>  attrd: PREFIX-attrd, PREFIX-attributes<br>
<br>
PREFIX-attributes<br></blockquote><div><br></div><div><span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:small;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">PREFIX-keystored ?</span><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
>  cib: PREFIX-configd, PREFIX-state<br>
<br>
Tricky...It deals with both config and state...<br></blockquote><div><br></div><div><span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:small;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">PREFIX</span>-datastore ?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
By the way, why in the first place the CIB messages must be in the log file?<br></blockquote><div><br></div><div>Because its the only record of what changed and when.  </div><div>Which is almost never important, except for those times when the information is critical.</div><div>Which describes almost all of pacemaker's logging unfortunately.</div><div><br></div><div>On a related topic, I think if we made file logging mandatory then we could move a lot more logs (including most of the cib logs) out of syslog.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Filling logs with XML diffs resulting from other actions already logged earlier<br>
sounds like duplicated informations.<br></blockquote><div><br></div><div>We generally only log configuration changes - which aren't logged in any other form.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I suppose most of the CIB logging messages could be set in debug level or<br>
replaced by a simple "cib updated with new {setup|status}"?<br>
<br>
>  crmd: PREFIX-controld, PREFIX-clusterd, PREFIX-controller<br>
<br>
PREFIX-controld<br>
<br>
>  lrmd: PREFIX-locald, PREFIX-resourced, PREFIX-runner<br>
<br>
PREFIX-executord? PREFIX-execd ?<br></blockquote><div><br></div><div>PREFIX-executiond </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
>  pengine: PREFIX-policyd, PREFIX-scheduler<br>
<br>
PREFIX-schedulerd<br>
<br>
> stonithd: PREFIX-fenced, PREFIX-stonithd, PREFIX-executioner<br>
<br>
I always disliked the STONITH acronym. For the same reasons, I<br>
dislike "executioner" as well. Moreover, some people might think it actually<br>
"executes" some process in the sense of "running".<br></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
I would definitely vote for PREFIX-fenced<br></blockquote><div><br></div><div>+1</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
>  pacemaker_remoted: PREFIX-remoted, PREFIX-remote<br>
<br>
PREFIX-remoted<br>
<br>
<br>
My 2¢,<br>
<div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
Users mailing list: <a href="mailto:Users@clusterlabs.org">Users@clusterlabs.org</a><br>
<a href="https://lists.clusterlabs.org/mailman/listinfo/users" rel="noreferrer" target="_blank">https://lists.clusterlabs.org/<wbr>mailman/listinfo/users</a><br>
<br>
Project Home: <a href="http://www.clusterlabs.org" rel="noreferrer" target="_blank">http://www.clusterlabs.org</a><br>
Getting started: <a href="http://www.clusterlabs.org/doc/Cluster_from_Scratch.pdf" rel="noreferrer" target="_blank">http://www.clusterlabs.org/<wbr>doc/Cluster_from_Scratch.pdf</a><br>
Bugs: <a href="http://bugs.clusterlabs.org" rel="noreferrer" target="_blank">http://bugs.clusterlabs.org</a><br>
</div></div></blockquote></div><br></div></div>