Bug 26767 – Check in univention-check-printers nicht mehr funktionsfähig

Bug 26767 - Check in univention-check-printers nicht mehr funktionsfähig


Summary:	Check in univention-check-printers nicht mehr funktionsfähig

Status:	CLOSED FIXED

Product:	UCS
Classification:	Unclassified
Component:	Printserver
Version:	UCS 3.0
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	UCS 3.1
Assigned To:	Felix Botner
QA Contact:	Lukas Walter

URL:
Keywords:	interim-3

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2012-04-11 15:25 CEST by Tim Petersen
Modified:	2012-12-12 21:08 CET (History)
CC List:	2 users (show)

See Also:
What kind of report is it?:	---
What type of bug is this?:	---
Who will be affected by this bug?:	---
How will those affected feel about the bug?:	---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Tim Petersen

2012-04-11 15:25:29 CEST

In /usr/sbin/univention-check-printers aus gleichnamigem Paket wird folgende Prüfung verwendet:
"
# search disabled printers
for printer in $(lpstat -a 2>/dev/null | sed 's/ .*//'); do
        if ! /usr/sbin/lpc status $printer | grep 'printing is enabled' &>/dev/null ; then
                disabled="$disabled $printer"
        fi
done
"

Das "Problem" ist, dass seit geraumer Zeit die Ausgabe von lpc status $printer nicht mehr immer auf englisch erfolgt (mit deutscher locale steht dort dann "Drucken aktiviert").

Das führt dazu, dass der Check immer True ist.

Das konnte ich initial auf einem UCS 2.4-4 beobachten und dann für 3.0 verifizieren.

Comment 1 Jan Christoph Ebersbach

2012-04-11 17:31:11 CEST

Hilft es vor den Aufruf /usr/sbin/lpc status $printer | grep 'printing is enabled' ein LANG=C zu schreiben?

Comment 2 Tim Petersen

2012-09-13 09:22:03 CEST

export LC_ALL=C am Anfang des Skripts ist ein machbarer Workaround.

Comment 3 Felix Botner

2012-10-24 12:46:41 CEST

OK, die LP Tools werden nun mit LC_ALL=C aufgerufen.

Comment 4 Lukas Walter

2012-11-09 15:00:17 CET

Änderung sieht gut aus, das Skript funktioniert damit wieder.

Allerdings ist im Zuge des Tests ein anderer Fehler aufgefallen:
"/usr/sbin/univention-check-printers: Zeile 53: mail: Kommando nicht gefunden"

Da es sich dabei um ein von der Änderung unabhängiges Problem handelt erstelle ich dafür einen neuen Bug (Bug #29123)


Changelogeintrag vorhanden,
Verified.

Comment 5 Stefan Gohmann

2012-12-12 21:08:42 CET

UCS 3.1-0 has been released: 
 http://forum.univention.de/viewtopic.php?f=54&t=2125

If this error occurs again, please use "Clone This Bug".