Add the option '--if-not-exists' in the command 'ongr:es:index:create'

Command/IndexCreateCommand.php

@@ -38,7 +38,13 @@ protected function configure()
                 InputOption::VALUE_NONE,
                 'If the time suffix is used, its nice to create an alias to the configured index name.'
             )
-            ->addOption('no-mapping', 'nm', InputOption::VALUE_NONE, 'Do not include mapping');
+            ->addOption('no-mapping', 'nm', InputOption::VALUE_NONE, 'Do not include mapping')
+            ->addOption(
+                'if-not-exists',
+                null,
+                InputOption::VALUE_NONE,
+                'Don\'t trigger an error, when the index already exists'
+            );
     }
 
     /**
@@ -55,6 +61,18 @@ protected function execute(InputInterface $input, OutputInterface $output)
             $finder->setNextFreeIndex($manager);
         }
 
+        if ($input->getOption('if-not-exists') && $manager->indexExists()) {
+            $output->writeln(
+                sprintf(
+                    '<info>Index `<comment>%s</comment>` already exists in `<comment>%s</comment>` manager.</info>',
+                    $manager->getIndexName(),
+                    $input->getOption('manager')
+                )
+            );
+
+            return 0;
+        }
+
         $manager->createIndex($input->getOption('no-mapping'));
 
         $output->writeln(

Resources/doc/commands.md

@@ -14,6 +14,7 @@ Creates a new index in Elasticsearch (including with mapping if not skipped) for
 |    `--time`    |    `-t`    |        *not required*        | Creates an index with current timestamp appended to its name.                          |
 |    `--alias`   |    `-a`    |        *not required*        | Creates an alias with index name specified in the configuration.                       |
 | `--no-mapping` |    `-nm`   |        *not required*        | Skips the mapping configuration on index create action.                                |
+| `--if-not-exists` |    *no value*   |        *not required*        | Skips an index creation, when the index already exists.                                |
 
 If you want to use timestabale indexes it's very handy to use it together with `-a` option. `-t` adds a date as the suffix and `-a` adds an alias as defined index name in manager configuration. So the code will work fine without any configuration changes, you dont need to do any other actions.
 

Tests/Functional/Command/CreateIndexCommandTest.php

@@ -63,6 +63,37 @@ public function testExecute($managerName, $arguments, $options)
         $manager->dropIndex();
     }
 
+    /**
+     * Tests creating index in case of existing this index. Configuration from tests yaml.
+     *
+     * @param string $managerName
+     * @param array  $arguments
+     * @param array  $options
+     *
+     * @dataProvider getTestExecuteData
+     */
+    public function testExecuteWithIndexExistence($managerName, $arguments, $options)
+    {
+        $manager = $this->getManager($managerName);
+
+        if (!$manager->indexExists()) {
+            $manager->createIndex();
+        }
+
+        try {
+            $arguments['--if-not-exists'] = null;
+            $this->runIndexCreateCommand($managerName, $arguments, $options);
+        } catch (\Exception $ex) {
+            $message = $ex->getMessage();
+            $expectedClassName = 'Elasticsearch\\Common\\Exceptions\\BadRequest400Exception';
+            $isExpectedException = $ex instanceof $expectedClassName;
+            if ($isExpectedException) {
+                $this->assertNotContains('IndexAlreadyExistsException', $message);
+            }
+        }
+        $manager->dropIndex();
+    }
+
     /**
      * Tests if right exception is thrown when manager is read only.
      *