403 warning problem

[MiroCillik]

If user doesn't have access to resource (the file in Google Drive), only warning is logged in the events log, but the job ends with status: success.
Even if there was warning for all the files.

This should be solved with results.json and proper display of status: warning in the UI
keboola/docker-bundle#236

[tomasfejfar]

Podle mě to nedává uživatelsky smysl. Dokud nebude status warning, mělo by to končit errorem. Padající orchestrace si může uživatel ošetřit přímo v orchestraci, kde si dá k tomu writeru "Continue on failure".
-- https://keboola.zendesk.com/agent/tickets/9239

[MiroCillik]

Asi to moze skoncit errorom, akurat to nemoze skoncit na prvom chybnom subore, ale prejst vsetky.

[Matejkys]

@MiroCillik Když to porovnáš například s DB extractorem, tak ten taky skončí na první tabulce, kde dodje k chybě. A může se stát analogická věc, že ten user přijde o práva číst z jedné tabulky z těch všech.
Když mají nadefinovaných x sheetů v jednom configu, asi je chtějí tahat pohromadě. Podle mě by to mělo failnout okamžitě a uživatel by si to měl vyřešit. Buď získat přístup k souboru, nebo ho dočasně disablovat v komponentě.

[MiroCillik]

Ja by som to velmi rad tak spravil, problem je, ze napr na tej Traveloke, kde tych suborov maju vela, ked im to takto zmenim, tak budu pindat

…

On Wed, 3 Oct 2018 12:57 Matejkys, ***@***.***> wrote: @MiroCillik <https://github.com/MiroCillik> Když to porovnáš například s DB extractorem, tak ten taky skončí na první tabulce, kde dodje k chybě. A může se stát analogická věc, že ten user přijde o práva číst z jedné tabulky z těch všech. Když mají nadefinovaných x sheetů v jednom configu, asi je chtějí tahat pohromadě. Podle mě by to mělo failnout okamžitě a uživatel by si to měl vyřešit. Buď získat přístup k souboru, nebo ho dočasně disablovat v komponentě. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#21 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABa0j_FcAt4xaNqf319vT8zkjIUNSRPPks5uhJgEgaJpZM4XFlrw> .

[Matejkys]

To beru spíš jako fail toho, kdo jim pomáhal stavět řešení, páč by měly mít nějaký univerzální účet pro správu těch souborů. Teď je to kvůli jejich setupu nastavené obecně špatně pro ostatní klienty.

[MiroCillik]

To mas samozrejme pravdu. Spravim to teda tak ako som pisal, job skonci errorom, ale prejde to vsetky subory.

[padak]

@MiroCillik nemůžem bez varování měnit chování jen proto že jste si s Matejkysem řekli že je to lepší. Souhlasím s tou změnou, ale musí to bejt pořádně odkomunikovany. Ideálně zmigrovany klientem na novou verzi a stávající deprecated.

[MiroCillik]

@padak vsak to vravim, ze im to nemozeme len tak zmenit "pod rukami". Este sa o tom pobavim s Martinom.

[padak]

@MiroCillik oh, já to v autě četl na mobili a "Spravim to teda tak ako som pisal..." jsem četl jako "Spravim to teda tak ako si pisal", tak jsem se lekl :) Sorry za paniku!