Skip to content
Snippets Groups Projects

Fix Opensearch compatibility check

Merged Dmitry Gruzd requested to merge 378958-update-opensearch-compatibility-check into master
All threads resolved!

What does this MR do and why?

In this MR we are doing a couple of things:

  • Match SystemCheck::App::SearchCheck with the stated Opensearch versions supported (1.x or later)
  • Switch ee/spec/lib/system_check/app/search_check_spec.rb to using fast_spec_helper, which drastically improve performance

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

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

Related to #378958 (closed)

Edited by Dmitry Gruzd

Merge request reports

Merged results pipeline #679050141 passed

Pipeline: GitLab

#679052446

    Merged results pipeline passed for 940a99a5

    Test coverage 95.24% (23.50%) from 1 job
    Approved by

    Merged by Terri ChuTerri Chu 2 years ago (Oct 27, 2022 6:26pm UTC)

    Pipeline #679082764 passed

    Pipeline passed for 0bb832e4 on master

    Test coverage 71.78% (23.50%) from 1 job
    10 environments impacted.

    Activity

    Filter activity
    • Approvals
    • Assignees & reviewers
    • Comments (from bots)
    • Comments (from users)
    • Commits & branches
    • Edits
    • Labels
    • Lock status
    • Mentions
    • Merge request status
    • Tracking
  • Terri Chu approved this merge request

    approved this merge request

  • Terri Chu resolved all threads

    resolved all threads

  • Terri Chu enabled an automatic merge when the pipeline for 64d9e040 succeeds

    enabled an automatic merge when the pipeline for 64d9e040 succeeds

  • merged

  • Terri Chu mentioned in commit 0bb832e4

    mentioned in commit 0bb832e4

  • added workflowstaging label and removed workflowcanary label

  • Dmitry Gruzd mentioned in merge request !89215 (merged)

    mentioned in merge request !89215 (merged)

  • GitLab Release Tools Bot picked the changes into the branch 15-5-stable-ee-patch-3 with commit c7094017

    picked the changes into the branch 15-5-stable-ee-patch-3 with commit c7094017

  • Automatically picked into !103157 (merged), will merge into 15-5-stable-ee ready for 15.5.3-ee.

  • Terri Chu mentioned in commit c7094017

    mentioned in commit c7094017

  • mentioned in merge request !103157 (merged)

  • Please register or sign in to reply
    Loading