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

Merge request pipeline #311693 failed

Pipeline: Notification

#311694

    Merge request pipeline failed for e02fde19

    5 environments impacted.

    Merged by Derek HudsonDerek Hudson 3 weeks ago (Feb 28, 2025 6:58pm UTC)

    Loading

    Pipeline #311719 failed

    Pipeline failed for 5d079eec on master

    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