|
 |
21d5fd |
From 32d2aa2cea22bb38fc3eeded1f6682daad097908 Mon Sep 17 00:00:00 2001
|
|
|
21d5fd |
From: Gauvain Pocentek <gauvainpocentek@gmail.com>
|
|
|
21d5fd |
Date: Tue, 13 Aug 2019 20:11:21 +0200
|
|
|
21d5fd |
Subject: [PATCH] Fix argument parsing for region_name
|
|
|
21d5fd |
|
|
|
21d5fd |
The region_name attribute for fence_compute and fence_evacuate is not a boolean
|
|
|
21d5fd |
and should expect a string argument.
|
|
|
21d5fd |
---
|
|
|
21d5fd |
agents/compute/fence_compute.py | 2 +-
|
|
|
21d5fd |
agents/evacuate/fence_evacuate.py | 2 +-
|
|
|
21d5fd |
tests/data/metadata/fence_compute.xml | 2 +-
|
|
|
21d5fd |
tests/data/metadata/fence_evacuate.xml | 2 +-
|
|
|
21d5fd |
4 files changed, 4 insertions(+), 4 deletions(-)
|
|
|
21d5fd |
|
|
|
21d5fd |
diff --git a/agents/compute/fence_compute.py b/agents/compute/fence_compute.py
|
|
|
21d5fd |
index 0e6c8044..a94bdc46 100644
|
|
|
21d5fd |
--- a/agents/compute/fence_compute.py
|
|
|
21d5fd |
+++ b/agents/compute/fence_compute.py
|
|
|
21d5fd |
@@ -381,7 +381,7 @@ def define_new_opts():
|
|
|
21d5fd |
"order": 1,
|
|
|
21d5fd |
}
|
|
|
21d5fd |
all_opt["region_name"] = {
|
|
|
21d5fd |
- "getopt" : "",
|
|
|
21d5fd |
+ "getopt" : ":",
|
|
|
21d5fd |
"longopt" : "region-name",
|
|
|
21d5fd |
"help" : "--region-name=[region] Region Name",
|
|
|
21d5fd |
"required" : "0",
|
|
|
21d5fd |
diff --git a/agents/evacuate/fence_evacuate.py b/agents/evacuate/fence_evacuate.py
|
|
|
21d5fd |
index 2b9af273..76e78b87 100644
|
|
|
21d5fd |
--- a/agents/evacuate/fence_evacuate.py
|
|
|
21d5fd |
+++ b/agents/evacuate/fence_evacuate.py
|
|
|
21d5fd |
@@ -318,7 +318,7 @@ def define_new_opts():
|
|
|
21d5fd |
"order": 1,
|
|
|
21d5fd |
}
|
|
|
21d5fd |
all_opt["region_name"] = {
|
|
|
21d5fd |
- "getopt" : "",
|
|
|
21d5fd |
+ "getopt" : ":",
|
|
|
21d5fd |
"longopt" : "region-name",
|
|
|
21d5fd |
"help" : "--region-name=[region] Region Name",
|
|
|
21d5fd |
"required" : "0",
|
|
|
21d5fd |
diff --git a/tests/data/metadata/fence_compute.xml b/tests/data/metadata/fence_compute.xml
|
|
|
21d5fd |
index 1dcbfc54..abc276ca 100644
|
|
|
21d5fd |
--- a/tests/data/metadata/fence_compute.xml
|
|
|
21d5fd |
+++ b/tests/data/metadata/fence_compute.xml
|
|
|
21d5fd |
@@ -55,7 +55,7 @@
|
|
|
21d5fd |
</parameter>
|
|
|
21d5fd |
<parameter name="region_name" unique="0" required="0">
|
|
|
21d5fd |
<getopt mixed="--region-name=[region]" />
|
|
|
21d5fd |
- <content type="boolean" default="" />
|
|
|
21d5fd |
+ <content type="string" default="" />
|
|
|
21d5fd |
<shortdesc lang="en">Region Name</shortdesc>
|
|
|
21d5fd |
</parameter>
|
|
|
21d5fd |
<parameter name="tenant_name" unique="0" required="0">
|
|
|
21d5fd |
diff --git a/tests/data/metadata/fence_evacuate.xml b/tests/data/metadata/fence_evacuate.xml
|
|
|
21d5fd |
index 4f1f6a58..7decb05c 100644
|
|
|
21d5fd |
--- a/tests/data/metadata/fence_evacuate.xml
|
|
|
21d5fd |
+++ b/tests/data/metadata/fence_evacuate.xml
|
|
|
21d5fd |
@@ -55,7 +55,7 @@
|
|
|
21d5fd |
</parameter>
|
|
|
21d5fd |
<parameter name="region_name" unique="0" required="0">
|
|
|
21d5fd |
<getopt mixed="--region-name=[region]" />
|
|
|
21d5fd |
- <content type="boolean" default="" />
|
|
|
21d5fd |
+ <content type="string" default="" />
|
|
|
21d5fd |
<shortdesc lang="en">Region Name</shortdesc>
|
|
|
21d5fd |
</parameter>
|
|
|
21d5fd |
<parameter name="tenant_name" unique="0" required="0">
|