search

slovensko-digital / govbox-pro

Nová, krajšia a lepšia schránka.
https://sluzby.slovensko.digital/govbox-pro/
European Union Public License 1.2
2 stars 4 forks source link

Nerobiť retry pri SubmissionErrore #492

Closed luciajanikova closed 6 days ago

luciajanikova commented 1 week ago

Tu máme nastavenia pre temporary error, vtedy retry chceme: https://github.com/slovensko-digital/govbox-pro/blob/main/app/jobs/govbox/submit_message_draft_job.rb#L8 . Ale naopak, pri terminujúcej chybe, SubmissionErrore nechceme retries. Podanie je nevalidné z nejakého dôvodu a to sa už nezmení vo fáze odosielania.

crutch commented 6 days ago

Na toto sa pozriem

crutch commented 6 days ago

Staci pridat toto do submit_message_draft_job.rb a joby budu rovno failovat na prvom attempte, bez retry

retry_on SubmissionError, attempts: 1 do |_job, _error|
    # no-op
end
luciajanikova commented 6 days ago

Pozri PR @crutch https://github.com/slovensko-digital/govbox-pro/pull/507

crutch commented 6 days ago

Je to spravne. Neviem ci si tie submission errors chcete niekde trackovat v nejakom monitoringu alebo je ten good job dashboard vlastne monitoring (ma to alerting?)

Kazdopadne tie tri riadky tam plnia zadanie :) 👍