47 replies to this topic

[scriptmelvin †]

Op de mount zelf zou dat kunnen, het probleem hier is een bestaande mount, waarbij de server is uitgezet zonder een deugdelijke umount te doen.

Dan zou je nog "timeo,retrans" kunnen proberen.

[luisteraar]

heb op een forum ergens gelezen
dat bij automount de mount unmounted
wordt als de share weg is.
maar we weten op forums is veel onzin te vinden.

Dat enigma gaat hangen bij het schrijven
naar een niet werkende mount houd
toch in dat een exception geweest is.
Dat punt moet te vinden zijn en
daarop een try except zetten.

Edited by luisteraar, 1 November 2022 - 20:48.

[littlesat]

Maar hier crashed python heel duidelijk volgens de log file en dat is altijd op te vangen met een try/except????

Edited by littlesat, 1 November 2022 - 20:57.

[luisteraar]

nu krijg ik een rare gedachte en
hoop dat ik ongelijk heb.
stel dat de nfs mount automatisch
verdwenen is en de image wordt naar
mount point geschreven dan word
in systeem geschreven die vol
raakt dat wordt ook hangen bij
te weinig ruimte.
Zal een kromme ingeving zijn.

[scriptmelvin †]

Als het goed is verdwijnt het mount point mee en zal je een error krijgen als je ernaar schrijft.

[luisteraar]

Maar hier crashed python heel duidelijk volgens de log file en dat is altijd op te vangen met een try/except????

daarvoor is try except om
één vangnet te hebben.
Altijd durf ik niet te zeggen.

[WanWizard]

Je haalt wat dingen door elkaar.

 

systemd heeft een mechanisme voor disk automount, dat werkt op dezelfde manier als autofs, maar meer geintegreerd. En daar kun je configureren dat net als bij autofs de mount na een idle timeout wordt verwijderd.'

 

Of dat ook werkt als je de server om zeep helpt waag ik te betwijfelen, maar dat kan ik testen als ik weer thuis ben, want daar gebruik ik het.

[luisteraar]

nfs enz is eigenlijk niet belangrijk het is een crash van enigma.
In enigma komt een misgreep naar
iets buiten enigma.
Als we weten welke functie die misgreep
doet kan je die in een try zetten en bij
mislukken actie onder nemen bv mount vergeten.

Edited by luisteraar, 1 November 2022 - 22:06.

[WanWizard]

Voor deze ene specifieke case wel ja.

 

Maar waarschijnlijk ook alleen maar omdat nfs al een timeout heeft gehad (en die kan best lang duren). Ik denk dat als je de nfs server uit zet en direct de flash actie van de TS start, de box net zo vast staat als het bij de PVR functie zo is.

 

En dan los je er niet zo bijzonder veel mee op, want je pakt de root cause er niet mee aan...

[littlesat]

Klopt het is altijd beter de oorzaak op te lossen… dus vandaar niet meteen try except hier doen omdat als je de kern oplost het meteen ook op meerdere plekke. Oplost en zomaar over try except maskeert ook een boel (zoals we met de py3 migratie geleerd hebben).
Oa de lijst die daar geparsed wordt daar zouden eigenlijk al de bronnen die niet werken niet in voor moeten komen.

Edited by littlesat, 2 November 2022 - 07:42.

[luisteraar]

Try except is een nood oplossing om een crash te voorkomen waarbij

de oorzaak niet aan het programma ligt.

Oorzaak buiten programma kunnen zijn open close read write seek in out en die lijst

kan ik veel groter maken.

 

In boerentaal wil het zeggen als program merkt dat netwerk harde schijf usb stick enz niet goed

werkt moet een andere actie ondernomen worden ipv crashen.

[WanWizard]

Ook weer zo.

[scriptmelvin †]

Veel kwaad kan het ook niet, een try/except in een deel dat zoekt naar een locatie om de backup neer te zetten. Ik doe een gooi: Fix OSError: host is down #3558

[luisteraar]

Mijn python kennis ontbreekt in C zou dat zo niet werken denk ik.

In de try mag maar 1 functie.

Afwachten op resultaten.

[WanWizard]

Zou je 'm niet een niveau hoger zetten, in checkIfDevice() ?

def checkIfDevice(path, diskstats):
    try:
        st_dev = os.stat(path).st_dev
    except OSError:
        return false

    return (os.major(st_dev), os.minor(st_dev)) in diskstats

of wellicht nog beter:

def checkIfDevice(path, diskstats):
    try:
        st_dev = os.stat(path).st_dev
    except OSError:
        pass

    return (os.major(st_dev), os.minor(st_dev)) in diskstats if st_dev else false

Edited by WanWizard, 2 November 2022 - 14:12.

[scriptmelvin †]

Nee, want dan returnt ie False en komt ie in mounts terecht. Het lijkt me dat een mount/device die een OSError gooit tijdens stat helemaal niet in de backuplocatiekandidatenlijst terecht moet komen.

[luisteraar]

st_dev = os.stat(path).st_dev is geen functie dat gaat altijd goed.

Je moet het punt pakken waar de functie een extern device aanspreekt.

Edited by luisteraar, 2 November 2022 - 14:17.

[WanWizard]

Nee, het is os.stat() die de fout gaat, want path is een stale mount.

 

Nee, want dan returnt ie False en komt ie in mounts terecht. Het lijkt me dat een mount/device die een OSError gooit tijdens stat helemaal niet in de backuplocatiekandidatenlijst terecht moet komen.

 

Je hebt helemaal gelijk.

[scriptmelvin †]

En voor 8.3: Fix OSError: host is down #3559

[WanWizard]

Merged.