Comment 1 for bug 1667192

Revision history for this message
Victor Morales (electrocucaracha) wrote :

Sindhu,

The main problem is that those values are optional but they considered as required in the _pod_from_args function, can you apply this change?

diff --git a/tricircleclient/v1/pods_cli.py b/tricircleclient/v1/pods_cli.py
index 232247e..73cbef0 100644
--- a/tricircleclient/v1/pods_cli.py
+++ b/tricircleclient/v1/pods_cli.py
@@ -46,11 +46,20 @@ class CreatePod(command.ShowOne):

     @staticmethod
     def _pod_from_args(parsed_args):
- return {'pod': {'region_name': parsed_args.region_name,
- 'az_name': parsed_args.availability_zone,
- 'pod_az_name': parsed_args.pod_availability_zone,
- 'dc_name': parsed_args.data_center,
- }}
+ result = {
+ 'pod':
+ {
+ 'region_name': parsed_args.region_name,
+ 'az_name': parsed_args.availability_zone,
+ }
+ }
+ if parsed_args.pod_availability_zone:
+ result['pod_az_name'] = parsed_args.pod_availability_zone
+
+ if parsed_args.data_center:
+ result['dc_name'] = parsed_args.data_center
+
+ return result

     def get_parser(self, prog_name):
         parser = super(CreatePod, self).get_parser(prog_name)