Import - nelze zastavit dávku

[ZdenkaSera]

Ve verzi 2.0.2. v KNAV ani na Inovatice nelze zastavit import:

[SykoraLukas]

Myslíš toto:

Podle mě jsi to zastavila.

[ZdenkaSera]

Napadlo mě, že tady může být problém v oprávnění - a je,, jako superadmin dávku zastavím. Dává to smysl,, superadmin může pozastavit i cizí dávky, běžný uživatel je ani nevidí. tak se jen ptám - uniklo mi, že to bylo takhle domluveno?

[ZdenkaSera]

Ještě doplňuji - tohle byl až další pokus, ten první s chybou byl jako admin.

[SykoraLukas]

OK, Prosím o omezení zastavení importu jen na roli superAdmin.

[ZdenkaSera]

@SykoraLukas @albertoh Prosím, počkejte s tou úpravou. Ta chyba by měla sice vypadat lidsky (třeba "Nemáte oprávnění ....)", ale jaký uživatel to oprávnění má mít bych raději konzultovala. Prosím, jak to vidíte u Vás? @luckajirku @katerinasirova @erikamirova, (u nás se domluvíme interně a napíšu).

[SykoraLukas]

Jen doplnim, ze to vychází ze zadání v issue #1202. Hned v první větě zadání psala paní Nezbedová, že to má jít jen pro SuperAdmina.

[ZdenkaSera]

Děkuji - v issue https://github.com/proarc/proarc/issues/1202 to nikdo nerozporoval, tedy není třeba znovu ověřovat a prosím jen upravit chybové hlášení, pokud se o zastavení pokusí uživatel bez oprávnění. vyzkoušela jsem ještě jako superadmin zastavit dávku spuštěnou userem - to je ok. Díky.

[erikamirova]

@ZdenkaSera Já bych to udělala tak, že superadmin má možnost pozastavit dávky všech userů. A user si může pozastavit jen svoji dávku.

[ZdenkaSera]

@erikamirova Za KNAV může být - pak tedy user + admin by mohli zastavit jen svou dávku, superadmin jakoukoliv. Shodneme se na tomhle @luckajirku @katerinasirova ?

[luckajirku]

@ZdenkaSera může být

[katerinasirova]

@ZdenkaSera souhlasíme

[ZdenkaSera]

@albertoh @SykoraLukas Výše domluveno, kromě opravy chybové hlášky tedy prosíme umožnit při oprávnění "user" a "admin" zastavit (pouze) vlastní importní dávku. Superadminovi zůstane oprávnění zastavit jakoukoliv. Díky!

[albertoh]

Client pripraven. @SykoraLukas dodelej, prosim, kontrolu v jadre

[SykoraLukas]

Nasazeno, k testu na Inovatice.

[ZdenkaSera]

Zkusila jsem na inovatice a buď to ještě není ono nebo dělám něco špatně:¨

• jako user ani jako admin jsem nedokázala svou dávku zastavit, hláška se žádná neobjevila
• jako superadmin můžu zastavit jakoukoliv dávku, jen musím na křížek pro zastavení kliknout 2x
• jako user nebo admin můžu znovu spustit svou dávku zastavenou superadminem - to je trochu sporné, ale pokud mám mít jako user možnost vlastní dávku zastavit, měla bych mít možnost ji znovu pustit, takže bych tohle nechala a nenutila někde ukládat, kdo dávku zatavil

[ZdenkaSera]

Tady sice není viditelný další vývoj, ale chování se změnilo a je skoro ok - user a admin můžou zastavit a znovu spustti své dávka, superadmin může zastavit cokoli. Když jsem ale jako superadmin chtěla znovu spustit zastavenou dávku původně načítanou jiným uživatelem, dostala jsem chybovou hlášku (viz níže). Prosím, nebude tohle zastavování resp. opětovné spuštění cizích dávek vyžadovat nějaké mapování/oprávnění pro superadmina k cizím importním složkám? Díky.

java.lang.IllegalStateException: The folder tracked by another batch import! folder: /data/proarc/.proarc/users/knavuser/import/Monografie/Methodika krasopisu found ID: 117 at cz.cas.lib.proarc.common.imports.ImportProcess.start(ImportProcess.java:250) at cz.cas.lib.proarc.common.imports.ImportProcess.run(ImportProcess.java:230) at cz.cas.lib.proarc.common.imports.ImportDispatcher$ExceptionHandlingTask.run(ImportDispatcher.java:155) at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) at java.util.concurrent.FutureTask.run(FutureTask.java:266) at cz.cas.lib.proarc.common.imports.ImportDispatcher$PriorityFuture.run(ImportDispatcher.java:270) at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) at java.lang.Thread.run(Thread.java:748)

[ZdenkaSera]

@pkudela Petře, prosím, je tohle dotaz na tebe? Díky.

[SykoraLukas]

Jak dávku spouštíš? Ve správě procesů nebo jako novou dávku?

[SykoraLukas]

Problém je v tom, že původní dávka se načítala, pak ji někdo zastavil. Poté si někdo stejnou složku nechal připravit znovu k importu a tedy smazal celý obsah té již načítané, ale pozastavené dávky. I tato dávka byla zastavena. Poté jsi asi chtěl pokračovat v načítání té, co jsi zastavila první. Ale chyba je, že tato dávka už patří pod jiný proces. Takže vyletí chyba.

Z mého pohledu je vše správně nastavené. Vyletí chyba, že dávku nelze načíst + stacktrace s popisem chyby a zanořením v programu.

[ZdenkaSera]

@SykoraLukas Díky, šlo mi o test situace, kdy superadmin zastaví cizí dávku a pak ji chce znovu spustit, tj. sahá na data v cizím importním adresáři. To je to, co mi nešlo (zastavit ano, znovu spustit ne), ale hlavně k tomu by ta funkce měla sloužit. Zkusím to u nás s touhle verzí. Zbytek je ok, tak to navrhnu k uzavření a případně pak znovu otevřu.

[ZdenkaSera]

Zavírám dle domluvy ze schůzky 15. 6. 2023.