Don’t close the ES index before updating settings (#4858)

The only settings we auto-fix are dynamic settings, and we don’t need to close the index first before changing those. Closing/opening an index appears to cause quite a bit of stress on the clusters.
Automattic · Sep 12, 2023 · 41d9212 · 41d9212
1 parent 787f7c2
commit 41d9212
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/search/includes/classes/class-health.php b/search/includes/classes/class-health.php
@@ -1128,7 +1128,7 @@ public function heal_index_settings_for_indexable( \ElasticPress\Indexable $inde
 		$desired_settings_to_heal = self::limit_index_settings_to_keys( $desired_settings, self::INDEX_SETTINGS_HEALTH_AUTO_HEAL_KEYS );
 		$index_name               = $indexable->get_index_name();
 		if ( method_exists( '\Automattic\VIP\Search\Search', 'should_load_new_ep' ) && \Automattic\VIP\Search\Search::should_load_new_ep() ) {
-			$result = $this->elasticsearch->update_index_settings( $index_name, $desired_settings_to_heal, true );
+			$result = $this->elasticsearch->update_index_settings( $index_name, $desired_settings_to_heal, false );
 		} else {
 			$result = $indexable->update_index_settings( $desired_settings_to_heal );
 		}

diff --git a/tests/search/includes/classes/test-class-health.php b/tests/search/includes/classes/test-class-health.php
@@ -1182,7 +1182,7 @@ public function test_heal_index_settings_for_indexable( $desired_settings, $opti
 
 		$health->elasticsearch->expects( $this->once() )
 			->method( 'update_index_settings' )
-			->with( $index_name, $expected_updated_settings, true );
+			->with( $index_name, $expected_updated_settings, false );
 
 		$result = $health->heal_index_settings_for_indexable( $mocked_indexable, $options );