[prev in list] [next in list] [prev in thread] [next in thread]
List: bacula-devel
Subject: Re: [Bacula-devel] Is JS_Warnings job status implemented?
From: Kern Sibbald <kern () sibbald ! com>
Date: 2016-10-01 18:57:24
Message-ID: 55e2c1fa-5da0-ea0a-dfc2-e95e95320630 () sibbald ! com
[Download RAW message or body]
[Attachment #2 (text/html)]
<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 10/01/2016 08:52 PM, Hankins,
Jonathan wrote:<br>
</div>
<blockquote
cite="mid:CAAiD57hasZJcxCSkPF2EQDH8nOQJ3r-Y73mvBQ_kWhKR9Ajyhw@mail.gmail.com"
type="cite">
<div dir="ltr" class="gmail_msg">I may poke around it a bit when I
have time. I'm not a C programmer, but the code is pretty clear
from what I've seen. If I can get something working, I'll send a
patch.</div>
</blockquote>
<br>
If you implement a patch, please submit it in a "feature request" in
the bugs database. If you do not, you might want to submit a
feature request without a patch because maybe in 10 years, I won't
have any more projects :-) (just kidding) maybe someday, I will
just feel like doing a few small projects :-)<br>
<br>
Best regards,<br>
Kern<br>
<br>
<blockquote
cite="mid:CAAiD57hasZJcxCSkPF2EQDH8nOQJ3r-Y73mvBQ_kWhKR9Ajyhw@mail.gmail.com"
type="cite">
<div dir="ltr" class="gmail_msg"><br>
</div>
<div dir="ltr" class="gmail_msg">Thanks! </div>
<div dir="ltr" class="gmail_msg"><br>
</div>
<div dir="ltr" class="gmail_msg">-Jonathan Hankins </div>
<div dir="ltr" class="gmail_msg"><br>
</div>
<div class="gmail_quote gmail_msg">
<div dir="ltr" class="gmail_msg">On Sat, Oct 1, 2016, 1:43 PM
Kern Sibbald <<a moz-do-not-send="true"
href="mailto:kern@sibbald.com" class="gmail_msg"
target="_blank">kern@sibbald.com</a>> wrote:<br
class="gmail_msg">
</div>
<blockquote class="gmail_quote gmail_msg" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex"
id="gmail_block_quote0">
<div bgcolor="#FFFFFF" text="#000000" class="gmail_msg">
<div
class="m_-242504673718768394m_-1134079105431921520moz-cite-prefix
gmail_msg">Jonathan,<br class="gmail_msg">
<br class="gmail_msg">
That might be a good "compromise". The only thing that
doesn't appeal to me is adding yet another directive. I
have been trying hard to avoid adding new directives since
Bacula now has so many directives, I am starting to forget
what does what :-(.<br class="gmail_msg">
<br class="gmail_msg">
If someone sent me a patch, I think I would apply it.
However, Bacula already warns on errors -- they generate
an error message. We should think about it a bit, but a
better choice for the directive name, might be
AllowWarningStatus ... or maybe you have a better idea
such as just "WarningStatus = yes/no". I might even be
inclined to make it default yes if all the regression
scripts run with it on (very unlikely I think).<br
class="gmail_msg">
<br class="gmail_msg">
Best regards,<br class="gmail_msg">
Kern</div>
</div>
<div bgcolor="#FFFFFF" text="#000000" class="gmail_msg">
<div
class="m_-242504673718768394m_-1134079105431921520moz-cite-prefix
gmail_msg"><br class="gmail_msg">
<br class="gmail_msg">
On 10/01/2016 06:32 PM, Hankins, Jonathan wrote:<br
class="gmail_msg">
</div>
</div>
<div bgcolor="#FFFFFF" text="#000000" class="gmail_msg">
<blockquote type="cite" class="gmail_msg">
<p dir="ltr" class="gmail_msg">Kern, what are your
thoughts about, in an effort to phase in W job status
without breaking people's stuff, a
per-fileset/job/client/whatever directive such as
WarnOnError, default = no, and some logic in the
JobStatus code to write W to the database/wherever it
needs to go, in cases where the user has explicitly
asked for it, and T otherwise? I can imagine use cases
where errors on one fileset don't matter but on others,
you'd like it reflected in the db.</p>
<p dir="ltr" class="gmail_msg">-Jonathan Hankins </p>
<br class="gmail_msg">
<div class="gmail_quote gmail_msg">
<div dir="ltr" class="gmail_msg">On Sat, Oct 1, 2016,
11:10 AM Hankins, Jonathan <<a
moz-do-not-send="true"
href="mailto:jhankins@homewood.k12.al.us"
class="gmail_msg" \
target="_blank">jhankins@homewood.k12.al.us</a>> wrote:<br class="gmail_msg">
</div>
<blockquote class="gmail_quote gmail_msg"
style="margin:0 0 0 .8ex;border-left:1px #ccc
solid;padding-left:1ex">
<p dir="ltr" class="gmail_msg">And I think that I
confused myself. I'll check, but I think I saw that
the director code for show dir does print ok with
warnings, and not just ok, on jobs with warnings
(by looking at JobErrors). I was confusing that
output with list jobs, which shows JobStatus, and
thus indicates T even on jobs with warnings. It may
be that it also shows a column for JobErrors...my
brain was just expecting to see a W. I've read
through the code enough to understand the issues and
expected behavior now. I will do a patch to add %E
to the documentation when I get a minute. That gives
me what I need as far as Nagios runscripts.</p>
<p dir="ltr" class="gmail_msg">I agree, suddenly
having W where there hasn't been one before in the
db might break people's Custom scripts and queries.
It looks like bacula has been adapted for it
everywhere that I looked though.</p>
<p dir="ltr" class="gmail_msg">Thanks,</p>
<p dir="ltr" class="gmail_msg">-Jonathan Hankins </p>
<br class="gmail_msg">
<div class="gmail_quote gmail_msg">
<div dir="ltr" class="gmail_msg">On Sat, Oct 1,
2016, 7:07 AM Kern Sibbald <<a
moz-do-not-send="true"
href="mailto:kern@sibbald.com" class="gmail_msg"
target="_blank">kern@sibbald.com</a>> wrote:<br
class="gmail_msg">
</div>
<blockquote class="gmail_quote gmail_msg"
style="margin:0 0 0 .8ex;border-left:1px #ccc
solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000"
class="gmail_msg">
<div
class="m_-242504673718768394m_-1134079105431921520m_-1584655890962146154m_3123998883425235098moz-cite-prefix
gmail_msg">I have planned a JobStatus == 'W'
for quite some time, but I am very concerned
that implementing it will break many existing
environments, scripts/programs that look at
the output. Consequently, it remains
suspended.<br class="gmail_msg">
<br class="gmail_msg">
Best regards,<br class="gmail_msg">
Kern</div>
</div>
<div bgcolor="#FFFFFF" text="#000000"
class="gmail_msg">
<div
class="m_-242504673718768394m_-1134079105431921520m_-1584655890962146154m_3123998883425235098moz-cite-prefix
gmail_msg"><br class="gmail_msg">
<br class="gmail_msg">
On 09/30/2016 09:19 PM, Hankins, Jonathan
wrote:<br class="gmail_msg">
</div>
</div>
<div bgcolor="#FFFFFF" text="#000000"
class="gmail_msg">
<blockquote type="cite" class="gmail_msg">
<div dir="ltr" class="gmail_msg">
<div class="gmail_msg">My jobs that complete
"OK -- with warnings" with JobErrors >
0 always have JobStatus 'T'.</div>
<div class="gmail_msg"><br class="gmail_msg">
</div>
I posted about this in 2012: <a
moz-do-not-send="true"
href="https://www.mail-archive.com/bacula-devel@lists.sourceforge.net/msg07612.html"
class="gmail_msg" \
target="_blank">https://www.mail-archive.com/bacula-devel@lists.sourceforge.net/msg07612.html</a>
<div class="gmail_msg"><br class="gmail_msg">
</div>
<div class="gmail_msg">No replies to that
post. It looks like there was a commit in
2013: <a moz-do-not-send="true"
href="http://www.bacula.org/git/cgit.cgi/bacula/commit/?id=36b7c10b7fc787ab7b075f0b2521f7b20605ae6e"
class="gmail_msg" target="_blank">http://www.bacula.org/git/cgit.cgi/bacula/commit/?id=36b7c10b7fc787ab7b075f0b2521f7b20605ae6e</a> \
that disabled this functionality, because there
was code that needed to be updated to say
(JS_Terminated || JS_Warning). Not sure if
this was ever done? It looks like the DB
gets 'T' stored as well, even though there
are changelog comments indicating that the
DB code has been updated to say "JobStatus
IN ('T', 'W') where necessary.</div>
<div class="gmail_msg"><br class="gmail_msg">
</div>
<div class="gmail_msg">The committed
code/comment above doesn't seem to be in
bacula anymore, and I am wondering if
JobStatus = 'W' is supposed to be
implemented yet?</div>
<div class="gmail_msg"><br class="gmail_msg">
</div>
<div class="gmail_msg">It would be nice to
be able to easily tell from the status
command that a job had warnings, instead
of having to look through the job logs.</div>
<div class="gmail_msg"><br class="gmail_msg">
</div>
<div class="gmail_msg">Is this something
that could be implemented if I was
able/willing to track down the necessary
changes and make a patch?</div>
<div class="gmail_msg"><br class="gmail_msg">
</div>
<div class="gmail_msg">Thanks,</div>
<div class="gmail_msg"><br class="gmail_msg">
</div>
<div class="gmail_msg">-Jonathan Hankins</div>
<div class="gmail_msg"><br class="gmail_msg">
</div>
<div class="gmail_msg"><br class="gmail_msg">
</div>
</div>
<br class="gmail_msg">
</blockquote>
</div>
<div bgcolor="#FFFFFF" text="#000000"
class="gmail_msg">
<blockquote type="cite" class="gmail_msg"><span
style="color:rgb(128,128,128);font-family:Arial;font-size:x-small;background-color:rgb(255,255,255)"
class="gmail_msg">This e-mail is intended only for the recipient and \
 may contain confidential or proprietary information. If you are n \
ot the intended recipient, the review, distribution, duplication or  \
retention of this message and its attachments is prohibited. Pleaseâ \
€„notify the sender of this error immediately by reply e-mail, and \
 permanently delete this message and its attachments in any form in which they may have been preserved.</span>
<br class="gmail_msg">
<fieldset
class="m_-242504673718768394m_-1134079105431921520m_-1584655890962146154m_3123998883425235098mimeAttachmentHeader
gmail_msg"></fieldset>
<br class="gmail_msg">
<pre \
class="gmail_msg">------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! <a moz-do-not-send="true" \
class="m_-242504673718768394m_-1134079105431921520m_-1584655890962146154m_3123998883425235098moz-txt-link-freetext \
gmail_msg" href="http://sdm.link/slashdot" \
target="_blank">http://sdm.link/slashdot</a></pre> <br class="gmail_msg">
<fieldset
class="m_-242504673718768394m_-1134079105431921520m_-1584655890962146154m_3123998883425235098mimeAttachmentHeader
gmail_msg"></fieldset>
<br class="gmail_msg">
<pre \
class="gmail_msg">_______________________________________________ Bacula-devel \
mailing list <a moz-do-not-send="true" \
class="m_-242504673718768394m_-1134079105431921520m_-1584655890962146154m_3123998883425235098moz-txt-link-abbreviated \
gmail_msg" href="mailto:Bacula-devel@lists.sourceforge.net" \
target="_blank">Bacula-devel@lists.sourceforge.net</a> <a moz-do-not-send="true" \
class="m_-242504673718768394m_-1134079105431921520m_-1584655890962146154m_3123998883425235098moz-txt-link-freetext \
gmail_msg" href="https://lists.sourceforge.net/lists/listinfo/bacula-devel" \
target="_blank">https://lists.sourceforge.net/lists/listinfo/bacula-devel</a> </pre>
</blockquote>
</div>
</blockquote>
</div>
</blockquote>
</div>
<br class="gmail_msg">
<span
style="color:rgb(128,128,128);font-family:Arial;font-size:x-small;background-color:rgb(255,255,255)"
class="gmail_msg">This e-mail is intended only for the recipient and \
 may contain confidential or proprietary information. If you are n \
ot the intended recipient, the review, distribution, duplication or  \
retention of this message and its attachments is prohibited. Pleaseâ \
€„notify the sender of this error immediately by reply e-mail, and \
 permanently delete this message and its attachments in any form in which they may have been preserved.</span>
</blockquote>
<p class="gmail_msg"><br class="gmail_msg">
</p>
</div>
</blockquote>
</div>
<br>
<span
style="color:rgb(128,128,128);font-family:Arial;font-size:x-small;background-color:rgb \
(255,255,255)">This e-mail is intended only for the recipient and ma \
y contain confidential or proprietary information. If you are not  \
the intended recipient, the review, distribution, duplication or reten \
tion of this message and its attachments is prohibited. Please not \
ify the sender of this error immediately by reply e-mail, and pe \
rmanently delete this message and its attachments in any form in which they may have been preserved.</span>
</blockquote>
<p><br>
</p>
</body>
</html>
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
Bacula-devel mailing list
Bacula-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bacula-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic