ivooskamp/backupchecks
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

v20260106-16-reset #50

Merged
ivooskamp merged 3 commits from v20260106-16-reset into main 2026-01-13 11:19:39 +01:00
Conversation 0 Commits 3 Files Changed 11 +9 -38
3 changed files with 9 additions and 38 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit e23e194e40 - Show all commits

2
.last-branch
View File

@ -1 +1 @@
v20260106-15-jobrun-popup-objects-sort
v20260106-16-reset

11
containers/backupchecks/src/backend/app/parsers/veeam.py
View File

@ -932,17 +932,8 @@ def try_parse_veeam(msg: MailMessage) -> Tuple[bool, Dict, List[Dict]]:
# Keep detailed overall message for non-success states, and always keep
# the "Processing <object>" marker when present (used for overrides/rules).
# Veeam Backup for Microsoft 365 can include a meaningful overall warning/info
# even when the run is reported as Success (e.g. missing application
# permissions/roles). Store it so it becomes visible in details and can be
# used for overrides.
is_m365 = (backup_type or "") == "Veeam Backup for Microsoft 365"
if overall_message:
if (
status_word != "Success"
or overall_message.lower().startswith("processing ")
or is_m365
):
if status_word != "Success" or overall_message.lower().startswith("processing "):
result["overall_message"] = overall_message
return True, result, objects

34
containers/backupchecks/src/templates/main/job_detail.html
View File

@ -274,7 +274,7 @@
"</body></html>"
);
}
function renderObjects(objects) {
function renderObjects(objects) {
var container = document.getElementById("run_msg_objects_container");
if (!container) return;
@ -283,36 +283,16 @@
return;
}
// Sort: objects with an error_message first (alphabetically by name), then the rest (also by name).
var sorted = (objects || []).slice().sort(function (a, b) {
a = a || {};
b = b || {};
var aHasErr = !!(a.error_message && a.error_message.toString().trim());
var bHasErr = !!(b.error_message && b.error_message.toString().trim());
if (aHasErr !== bHasErr) return aHasErr ? -1 : 1;
var an = (a.name || "").toString().toLowerCase();
var bn = (b.name || "").toString().toLowerCase();
if (an < bn) return -1;
if (an > bn) return 1;
return 0;
});
var html = "<div class=\"table-responsive\"><table class=\"table table-sm table-bordered mb-0\">";
html += "<thead><tr><th>Object</th><th>Type</th><th>Status</th><th>Error</th></tr></thead><tbody>";
for (var i = 0; i < sorted.length; i++) {
var o = sorted[i] || {};
for (var i = 0; i < objects.length; i++) {
var o = objects[i] || {};
html += "<tr>";
html += "<td>" + escapeHtml(o.name || "") + "</td>";
html += "<td>" + escapeHtml(o.type || "") + "</td>";
html += "<td>" + (o.name || "") + "</td>";
html += "<td>" + (o.type || "") + "</td>";
var d = statusDotClass(o.status);
html += "<td class=\"status-text " + statusClass(o.status) + "\">" +
(d ? ("<span class=\"status-dot " + d + " me-2\" aria-hidden=\"true\"></span>") : "") +
escapeHtml(o.status || "") +
"</td>";
html += "<td>" + escapeHtml(o.error_message || "") + "</td>";
html += "<td class=\"status-text " + statusClass(o.status) + "\">" + (d ? ('<span class=\\\"status-dot ' + d + ' me-2\\\" aria-hidden=\\\"true\\\"></span>') : '') + escapeHtml(o.status || "") + "</td>";
html += "<td>" + (o.error_message || "") + "</td>";
html += "</tr>";
}
html += "</tbody></table></div>";