Skip to content
Snippets Groups Projects

Testing with allowing some indexer failures in the Notification integration tests.

Merged Derek Hudson requested to merge aws-allow-some-indexer-failures into master

Type of change

  • Bug Fix
  • Feature

Please provide link to gitlab issue or ADR(Architecture Decision Record)

Does this introduce a change in the core logic?

  • [NO]

Does this introduce a change in the cloud provider implementation, if so which cloud?

  • AWS
  • Azure
  • Google Cloud
  • IBM

Does this introduce a breaking change?

  • [NO]

What is the current behavior?

Currently, if an indexer service initially fails to index a record, that message will be pushed to the notification queue. This is accepted behavior, but will fail the tests.

What is the new/expected behavior?

Adding in a way to allow a handful of failed indexing messages in the notification integration tests should address this issue.

Have you added/updated Unit Tests and Integration Tests?

Any other useful information

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading