Blame SOURCES/bz902407-pcmk-provide_a_meaningful_error_if_master_is_used_for_primitives_and_groups.patch

ed0026
commit 4cc5bf3a67cae9b17c49cd73d840b5388500ad13
ed0026
Author: Andrew Beekhof <andrew@beekhof.net>
ed0026
Date:   Tue Sep 17 14:06:41 2013 +1000
ed0026
ed0026
    Fix: crm_resource: Provide a meaningful error if --master is used for primitives and groups
ed0026
ed0026
diff --git a/tools/crm_resource.c b/tools/crm_resource.c
ed0026
index 69470d7..b63b9ae 100644
ed0026
--- a/tools/crm_resource.c
ed0026
+++ b/tools/crm_resource.c
ed0026
@@ -1942,6 +1942,17 @@ main(int argc, char **argv)
ed0026
         } else if (rsc->variant < pe_clone && g_list_length(rsc->running_on) > 1) {
ed0026
             CMD_ERR("Resource '%s' not moved: active on multiple nodes\n", rsc_id);
ed0026
             goto bail;
ed0026
+
ed0026
+        } else if (rsc->variant < pe_clone && scope_master) {
ed0026
+            resource_t *p = uber_parent(rsc);
ed0026
+            if(p->variant == pe_master) {
ed0026
+                CMD_ERR("Resource '%s' not moved: The --master option is not a valid for %s resources."
ed0026
+                        "  Did you mean '%s'?\n", rsc_id, get_resource_typename(rsc->variant), p->id);
ed0026
+            } else {
ed0026
+                CMD_ERR("Resource '%s' not moved: The --master option is not a valid for %s resources.\n",
ed0026
+                        rsc_id, get_resource_typename(rsc->variant));
ed0026
+            }
ed0026
+            goto bail;
ed0026
         }
ed0026
 
ed0026
         if(dest == NULL) {