Fix args order in strings.Contains in es-rollover (#3324)

If we do `strings.Contains("resource_already_exists_exception", errorMap["type"].(string))`,
we're basically doing `"resource_already_exists_exception" == errorMap["type"].(string)`.
I would assume that we intended to test whether `errorMap["type"]` contains
`"resource_already_exists_exception"` as a substring, so the reverse `strings.Contains` order
is more applicable here.

Signed-off-by: Pavol Loffay <[email protected]>

Co-authored-by: Iskander (Alex) Sharipov <[email protected]>
Co-authored-by: Albert <[email protected]>

cmd/es-rollover/app/init/action.go

@@ -94,7 +94,7 @@ func createIndexIfNotExist(c client.IndexAPI, index string) error {
 			}
 			errorMap := jsonError["error"].(map[string]interface{})
 			// check for reason, ignore already exist error
-			if strings.Contains("resource_already_exists_exception", errorMap["type"].(string)) {
+			if strings.Contains(errorMap["type"].(string), "resource_already_exists_exception") {
 				return nil
 			}
 		}