Skip to content

Catch semver_dialects parse errors

Philip Cunningham requested to merge philipcunningham-nomethoderror-on-sidekiq-ingest-reports-worker-428251 into master

What does this MR do and why?

This merge request wraps calls to semver_dialects in a rescue block in order to mitigate a regression that was introduced by Check supported versions when querying compress... (!133083 - merged). It surfaces a limitation of the semver_dialects that suggests it is prone to raising exceptions rather than returning actionable parse failures.

Related Issue(s)

NoMethodError on Sidekiq/Sbom::IngestReportsWorker (#428251 - closed)

Related Sentry Events

https://new-sentry.gitlab.net/organizations/gitlab/issues/401718/?query=is%3Aunresolved+IngestReportsWorker&referrer=issue-stream&statsPeriod=14d&stream_index=0

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Philip Cunningham

Merge request reports