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

Monitoring: standard usage warnings should be ignored in Sentry #3600

Closed
PascalRepond opened this issue Feb 15, 2024 · 0 comments · Fixed by #3644
Closed

Monitoring: standard usage warnings should be ignored in Sentry #3600

PascalRepond opened this issue Feb 15, 2024 · 0 comments · Fixed by #3644
Assignees
Labels
dev: monitoring Alert system like Sentry or Zabbix. enhancement Improvement of an existing feature

Comments

@PascalRepond
Copy link
Contributor

How it works

Some warnings that are not system errors but just standard usage problems (linked to wrong user input in editors for example) are displayed in Sentry but they are not useful.

Improvement suggestion

At least the following types of errors should not show up in Sentry or should be easily ignorable:

  • AcqReceiptLine err:La quantité reçue est supérieure à la quantité commandée …
  • AcqReceipt err:'df' is too short
  • Document err:'periodical' is not one of ['article', 'materialUnit', 'privateFile', 'privateSubfile']
  • Document err:'XXXX' is a required property
  • Holdings: list index out of range (prédiction)
  • Holding err:Les fréquences régulières doivent avoir la prochaine date prévue renseignée.
  • Item err:Barcode XXXXX is already taken
  • Item err:La catégorie de prêt temporaire ne peut pas être identique à la catégorie de prêt par défaut
  • LocalField err:'fields' is a required property
  • Patron err:'barcode' is a required property
  • Patron err:At least one email should be defined for an email communication channel. …
@PascalRepond PascalRepond added the enhancement Improvement of an existing feature label Feb 15, 2024
@PascalRepond PascalRepond added the dev: monitoring Alert system like Sentry or Zabbix. label Feb 15, 2024
@PascalRepond PascalRepond moved this from Inbox to Product Backlog in RERO ILS issues Feb 15, 2024
@PascalRepond PascalRepond moved this from Product Backlog to Sprint backlog in RERO ILS issues Feb 26, 2024
@rerowep rerowep self-assigned this Apr 3, 2024
rerowep added a commit to rerowep/rero-ils that referenced this issue Apr 3, 2024
* Standard usage warnings should be ignored in Sentry.
* Close: rero#3600

Co-Authored-by: Peter Weber <[email protected]>
@rerowep rerowep moved this from Sprint backlog to In Development in RERO ILS issues Apr 4, 2024
rerowep added a commit to rerowep/rero-ils that referenced this issue Apr 4, 2024
* Standard usage warnings should be ignored in Sentry.
* Close: rero#3600

Co-Authored-by: Peter Weber <[email protected]>
rerowep added a commit to rerowep/rero-ils that referenced this issue Apr 10, 2024
* Standard usage warnings should be ignored in Sentry.
* Close: rero#3600

Co-Authored-by: Peter Weber <[email protected]>
rerowep added a commit that referenced this issue Apr 10, 2024
* Standard usage warnings should be ignored in Sentry.
* Close: #3600

Co-Authored-by: Peter Weber <[email protected]>
@github-project-automation github-project-automation bot moved this from In Development to Done in RERO ILS issues Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev: monitoring Alert system like Sentry or Zabbix. enhancement Improvement of an existing feature
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants