Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nerobiť retry pri SubmissionErrore #492

Closed
luciajanikova opened this issue Nov 15, 2024 · 4 comments
Closed

Nerobiť retry pri SubmissionErrore #492

luciajanikova opened this issue Nov 15, 2024 · 4 comments
Assignees

Comments

@luciajanikova
Copy link
Member

luciajanikova commented Nov 15, 2024

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.

@luciajanikova luciajanikova changed the title Nerobit retry pri SubmissionErrore Nerobiť retry pri SubmissionErrore Nov 15, 2024
@crutch
Copy link
Contributor

crutch commented Nov 16, 2024

Na toto sa pozriem

@crutch
Copy link
Contributor

crutch commented Nov 16, 2024

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
Copy link
Member Author

Pozri PR @crutch #507

@crutch
Copy link
Contributor

crutch commented Nov 16, 2024

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 :) 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants