nutanix/calm-dsl

Keep Calm and DSL On!

calm-dsl's Issues

Support for external subnets from remote PC

Useful for calm-saas setups having no host pc accounts. Some commands like calm create provider_spec may fail for projects having remote pc registered

Support for macros in AHV NIC

Need support for Calm macros in the AHV NIC property.

nics = [AhvVmNic.DirectNic.ingress("@@{macro}@@")]

`calm watch app` command jumps shouldn't dump output to new window

Presently if we use calm watch app <app_name>. It jumps to new window and displays the action runlogs there.
It may cause a problem when we run this command in non-interactive manner. For ex: (create a task in blueprint in which docker run will trigger this command)
, It will fail by error: fd=_sys.__stdout__.fileno()) _curses.error: setupterm: could not find terminal

Error can be resolved if we run this command in interactive command. Ex: (for above case, run docker in interactive (-it flag) way for "calm watch app" command).

Expected Behaviour: The error should not come. Command should be independent of how user is executing it.

Attached screenshot for problem :

Show more details in the describe app command

Currently, calm describe app <name> shows the application summary. It should also show service/deployment level details as exposed in the UI.

The workaround right now is to use calm describe app <name> --out json to get the whole json and parse it manually.

Add CONTRIBUTING file

Lay down the coding standards and process to be followed.

Dynamic variables not evaluated during launching with calm launch

This is related to issue #66 too.

When launching an application from Calm DSL with calm launch, if there are dynamic variables with predefined values or eScript/HTTP task, they don't get evaluated returning the possible values. The possible values are empty.

Additionally, when there is variable dependency during launch, for example the first variable gives you a list of predefined values, and whatever value you use will be evaluated in a second variable to present different values depending on the previous input, that will need to be cover in the CLI too.

Passing config file alternative path

Right now the path for the config file is $HOME/.calm/config.ini. It would be great to allow setting the config file path to a different location.

Decompiled script from dynamic variable has no identifying name

Steps to reproduce:

1. Create a dynamic variable with type EScript.
2. Decompile the blueprint.
3. Script is decompiled with the name: _Task_Task.py

Calm-dsl is unable to decompile blueprint, error validating schema

Seeing the error below when trying to decompile (download) a blueprint.

calm.dsl.tools.validator.validation_error: '' is not one of ['PROVISION_EXISTING_MACHINE']

Failed validating instance schema at ['type']:
"''\n"

By 'enum' validator in validating schema at ['properties']['type']:
|
default: PROVISION_EXISTING_MACHINE
enum: [PROVISION_EXISTING_MACHINE]
type: string

odd unicode-like characters are left at the terminal cursor after action run screen

Related command:

calm run action <action_name> --app <app_name> --watch

when action run gets finished, odd unicode-like characters are left at the terminal cursor.

Ability to support inheritance at class level attributes

Ex:
**
class Service1(Service):

     ENV = Variable.WithOptions.Predefined.string(
            ["DEV", "PROD"], default="DEV", is_mandatory=True, runtime=True
     )

class Service2(Service1):

pass
**

In this case, attributes defined in Service1 (i.e. variables and actions) should also become the attributes of Service2 class.
This should happen for all valid CALM-DSL entities. It will help user to keep repetitive code at single place, and it will be more robust to errors

Support for passing encrypted secret values in blueprint creation

Right now we strip out secrets before creating blueprint.
Support to pass encrypted secret value and passphrase during bp create command.

Use case:
Will be helpful for decompiling secret while blueprint decompilation

Catch user import errors during compile

Improve error handling.

+ calm create bp --file blueprint.py --name Calm_Era_0bd47dfb8d3c342860c951664edf8203d17b63be
Traceback (most recent call last):
  File "/root/calm-dsl/venv/bin/calm", line 11, in <module>
    load_entry_point('calm.dsl', 'console_scripts', 'calm')()
  File "/root/calm-dsl/venv/lib/python3.6/site-packages/click/core.py", line 764, in __call__
    return self.main(*args, **kwargs)
  File "/root/calm-dsl/venv/lib/python3.6/site-packages/click/core.py", line 717, in main
    rv = self.invoke(ctx)
  File "/root/calm-dsl/venv/lib/python3.6/site-packages/click/core.py", line 1137, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/root/calm-dsl/venv/lib/python3.6/site-packages/click/core.py", line 1137, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/root/calm-dsl/venv/lib/python3.6/site-packages/click/core.py", line 956, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/root/calm-dsl/venv/lib/python3.6/site-packages/click/core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "/root/calm-dsl/calm/dsl/cli/bp_commands.py", line 214, in create_blueprint_command
    client, bp_file, name=name, description=description, force_create=force
  File "/root/calm-dsl/calm/dsl/cli/bp_commands.py", line 168, in create_blueprint_from_dsl
    bp_payload = compile_blueprint(bp_file)
  File "/root/calm-dsl/calm/dsl/cli/bps.py", line 231, in compile_blueprint
    bp_payload = UserBlueprintPayload.get_dict()
  File "/root/calm-dsl/calm/dsl/builtins/models/entity.py", line 399, in get_dict
    return json.loads(cls.json_dumps())
  File "/root/calm-dsl/calm/dsl/builtins/models/entity.py", line 359, in json_dumps
    separators=(",", ": ") if pprint else (",", ":"),
  File "/usr/lib64/python3.6/json/__init__.py", line 238, in dumps
    **kw).encode(obj)
  File "/usr/lib64/python3.6/json/encoder.py", line 199, in encode
    chunks = self.iterencode(o, _one_shot=True)
  File "/usr/lib64/python3.6/json/encoder.py", line 257, in iterencode
    return _iterencode(o, 0)
  File "/root/calm-dsl/calm/dsl/builtins/models/entity.py", line 412, in default
    return cls.compile()
  File "/root/calm-dsl/calm/dsl/builtins/models/substrate.py", line 32, in compile
    cdict = super().compile()
  File "/root/calm-dsl/calm/dsl/builtins/models/entity.py", line 308, in compile
    attrs = cls.get_all_attrs()
  File "/root/calm-dsl/calm/dsl/builtins/models/entity.py", line 304, in get_all_attrs
    return ncls.get_user_attrs()
  File "/root/calm-dsl/calm/dsl/builtins/models/entity.py", line 232, in get_user_attrs
    user_attrs[name] = getattr(cls, name, value)
  File "/root/calm-dsl/calm/dsl/builtins/models/action.py", line 191, in __get__
    node_visitor.visit(node)
  File "/usr/lib64/python3.6/ast.py", line 253, in visit
    return visitor(node)
  File "/usr/lib64/python3.6/ast.py", line 261, in generic_visit
    self.visit(item)
  File "/usr/lib64/python3.6/ast.py", line 253, in visit
    return visitor(node)
  File "/usr/lib64/python3.6/ast.py", line 261, in generic_visit
    self.visit(item)
  File "/usr/lib64/python3.6/ast.py", line 253, in visit
    return visitor(node)
  File "/usr/lib64/python3.6/ast.py", line 263, in generic_visit
    self.visit(value)
  File "/usr/lib64/python3.6/ast.py", line 253, in visit
    return visitor(node)
  File "/root/calm-dsl/calm/dsl/builtins/models/action.py", line 67, in visit_Call
    task = eval(compile(ast.Expression(node), "", "eval"), self._globals)
  File "", line 78, in <module>
NameError: name 'json' is not defined

Inconsistency with Variable attributes

When creating variables and the IDE autocomplete the possible attributes, they are inconsistent across variable types like String, Int, Time and so on.

Int, Time and I guess others (didn't check) don't have the attribute name.

String

Integer

Time

ValueError: I/O operation on closed file while running tests

When we invoke command make test, it runs all the test successfully.
But produces logging error at the end.

Results (157.68s):
     202 passed
      68 deselected
--- Logging error ---
Traceback (most recent call last):
  File "/usr/local/Cellar/python/3.7.5/Frameworks/Python.framework/Versions/3.7/lib/python3.7/logging/__init__.py", line 1028, in emit
    stream.write(msg + self.terminator)
  File "/Users/abhijeet.kaurav/calm-dsl/venv/lib/python3.7/site-packages/colorama/ansitowin32.py", line 41, in write
    self.__convertor.write(text)
  File "/Users/abhijeet.kaurav/calm-dsl/venv/lib/python3.7/site-packages/colorama/ansitowin32.py", line 162, in write
    self.write_and_convert(text)
  File "/Users/abhijeet.kaurav/calm-dsl/venv/lib/python3.7/site-packages/colorama/ansitowin32.py", line 187, in write_and_convert
    self.write_plain_text(text, cursor, start)
  File "/Users/abhijeet.kaurav/calm-dsl/venv/lib/python3.7/site-packages/colorama/ansitowin32.py", line 195, in write_plain_text
    self.wrapped.write(text[start:end])
  File "/Users/abhijeet.kaurav/calm-dsl/venv/lib/python3.7/site-packages/_pytest/capture.py", line 427, in write
    self.buffer.write(obj)
ValueError: I/O operation on closed file
Call stack:
  File "/Users/abhijeet.kaurav/calm-dsl/calm/dsl/db/handler.py", line 62, in close
    LOG.debug("Closing connection to local DB")
  File "/Users/abhijeet.kaurav/calm-dsl/calm/dsl/tools/logger.py", line 193, in debug
    return logger.debug(self.__add_caller_info(msg), *args, **kwargs)
Message: ':62] Closing connection to local DB'
Arguments: ()

Findings: This is producible because atexit handler is registered with db.close() method. And inside this method we are logging debug message Closing connection to local DB.

Mandatory variable is not marked as runtime by default

When using Calm UI, a variable marked as mandatory automatically enables the runtime icon. In Calm DSL the runtime=True must be implicit when is_madatory=True

Support for JSON output for marketplace blueprints/items

calm describe marketplace_bp/marketplace_item <bp_name> --out=json should dump out the json for the blueprint

Add option to specify "run action" variables

When running something like this:

calm run action <action_name> --app <app_name>

It seems there's no option to specify action variable values. For example, the ScaleOut actions allow specification of @@{COUNT}@@ value in the UI. This should also be an option in the DSL.

Maybe an additional @click.option for --payload?

This assumes that passing a JSON payload is the correct way to specify action variable values in the first place.

Runbook tasks for system actions which are not overrided in bp file

For ex:

class Service1(Service):
    """Service1 Service description"""
    pass

class Default(Profile):
    """Default Profile description"""

    deployments = [sample_deployment]
    @action
    def test_action():
        """test_action Action description"""

        Service1.__create__(name="task1")

In above example, we have not override system action("create") for service "Service1". We are creating runbook task i.e. Calling service action from profile actions.

If we compile the above bp, compilation will fail with error AttributeError: type object 'Service1' has no attribute '__create__', untill we override these actions in bp file.

Add a force create option to allow blueprint with the same name.

Flag:
calm create bp --force ...

Behavior:
Allow bp create with the same name. I will delete any existing bp with same name first.

Support for subnets across multiple clusters for AHV helpers

So for now in ahv subnets we are using:
AhvVmNic(subnet="vlan.0"),

Apart from them we should also accept parameter cluster name for identifying subnet having same name across multiple PE.
i.e.
AhvVmNic(subnet="vlan.0", cluster_name="name"),

Include watch option with Launch App action

It would be great to have the watch option with calm launch

calm launch bp -w/--watch BLUEPRINT_NAME

Support to pass runtime value in non-interactive at bp launch

Right now for launching a blueprint we have two options:

1. Launch a blueprint using -i/--ignore_runtime_variables to ignore runtime variables
2. Pass all the runtime variables at cli in interactive mode.

Need to have a way to supply those values in non-interactive mode (As approach 2 is not good for large no. of variables/entities).

Can't decompile blueprints if downloadable image description contains a carriage return

Case:

Can't decompile blueprints if images under Configuration > Downloadable Image Configuration have a carriage return in the image description. Doesn't seem to matter where the carriage return is. Example:

https://cloud.centos.org/centos/7/images/

Command:

calm decompile bp blueprint1

Error:

[2020-07-14 02:30:11] calm.dsl.cli.bps [INFO] blueprint1 found 
[2020-07-14 02:30:12] calm.dsl.cli.bps [INFO] Decompiling blueprint blueprint1
[2020-07-14 02:30:12] calm.dsl.decompile.decompile_render [INFO] Creating blueprint directory
[2020-07-14 02:30:12] calm.dsl.decompile.decompile_render [INFO] Rendering blueprint file template
[2020-07-14 02:30:12] calm.dsl.decompile.decompile_render [INFO] Formatting blueprint file using black
Traceback (most recent call last):
  File "/home/chris/Data/solutions/nutanix/calm-dsl/venv/bin/calm", line 11, in <module>
    load_entry_point('calm.dsl', 'console_scripts', 'calm')()
  File "/home/chris/Data/solutions/nutanix/calm-dsl/venv/lib/python3.8/site-packages/click/core.py", line 764, in __call__
    return self.main(*args, **kwargs)
  File "/home/chris/Data/solutions/nutanix/calm-dsl/venv/lib/python3.8/site-packages/click/core.py", line 717, in main
    rv = self.invoke(ctx)
  File "/home/chris/Data/solutions/nutanix/calm-dsl/venv/lib/python3.8/site-packages/click/core.py", line 1137, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/chris/Data/solutions/nutanix/calm-dsl/venv/lib/python3.8/site-packages/click/core.py", line 1137, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/chris/Data/solutions/nutanix/calm-dsl/venv/lib/python3.8/site-packages/click/core.py", line 956, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/chris/Data/solutions/nutanix/calm-dsl/venv/lib/python3.8/site-packages/click/core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "/home/chris/Data/solutions/nutanix/calm-dsl/calm/dsl/cli/bp_commands.py", line 114, in _decompile_bp
    decompile_bp(name, with_secrets)
  File "/home/chris/Data/solutions/nutanix/calm-dsl/calm/dsl/cli/bps.py", line 263, in decompile_bp
    create_bp_dir(bp_cls=bp_cls, with_secrets=with_secrets)
  File "/home/chris/Data/solutions/nutanix/calm-dsl/calm/dsl/decompile/decompile_render.py", line 28, in create_bp_dir
    bp_data = format_str(bp_data, mode=FileMode())
  File "/home/chris/Data/solutions/nutanix/calm-dsl/venv/lib/python3.8/site-packages/black.py", line 725, in format_str
    src_node = lib2to3_parse(src_contents.lstrip(), mode.target_versions)
  File "/home/chris/Data/solutions/nutanix/calm-dsl/venv/lib/python3.8/site-packages/black.py", line 836, in lib2to3_parse
    raise exc from None
black.InvalidInput: Cannot parse: 19:64: CENTOS_IMAGE = vm_disk_package(name='CENTOS_IMAGE', description='

Image descriptions that failed:

• Carriage return only
• Text followed by carriage return
• Text, carriage return then more text i.e. no blank line between lines of text
• Text, carriage return, blank line of text, carriage return, more text

Fix:

• Remove all carriage returns from the image description

Validations for blueprint file.

We need more validations on blueprint classes related to calm domain.
Need to have these validations on client side

For ex:

1. All the downloadable images that are used in any of disk images should be included in Blueprint.packages (Blueprint Class attribute).
2. Check multiple profiles are using same set of services object. Basically we cannot each different services for different profiles.
3. Go through Blueprint -> Profiles -> flow and check each required class are defined in blueprint class attributes i.e Blueprint.services/packages/substrates/profiles.

Show appropriate error messages when blueprint does not goes to ACTIVE state

calm create bp --file lamp-v4/lamp-v4.py --name jg-dsl-LAMP
[2020-04-15 14:06:11] calm.dsl.cli.bps [INFO] Syncing cache
[2020-04-15 14:06:45] calm.dsl.cli.bp_commands [INFO] Blueprint state: DRAFT
Traceback (most recent call last):
File "/root/.local/bin/calm", line 8, in
sys.exit(main())
File "/root/.local/lib/python3.7/site-packages/click/core.py", line 764, in call
return self.main(*args, **kwargs)
File "/root/.local/lib/python3.7/site-packages/click/core.py", line 717, in main
rv = self.invoke(ctx)
File "/root/.local/lib/python3.7/site-packages/click/core.py", line 1137, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/root/.local/lib/python3.7/site-packages/click/core.py", line 1137, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/root/.local/lib/python3.7/site-packages/click/core.py", line 956, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/root/.local/lib/python3.7/site-packages/click/core.py", line 555, in invoke
return callback(*args, **kwargs)
File "/root/.local/lib/python3.7/site-packages/calm/dsl/cli/bp_commands.py", line 201, in create_blueprint_command
assert bp_state == "ACTIVE"
AssertionError

Broken pytest-sugar causes tests to fail

When running make test or make test_all, an exception is generated by an incompatibility between the installed version of pytest-sugar and pytest 5.4.0+.

Fix when running these versions together is to downgrade pytest to version 5.3.5:

(venv) pip3 uninstall pytest
(venv) pip3 install pytest==5.3.5

This requires changes in dev-requirements.txt. Confirmed after discussion with @abhijeetkaurav1st who is running pytest 5.3.4.

Related issue: Teemu/pytest-sugar#187

Readiness probe is not working with latest DSL master

Old DSL implementation has readiness probe with default credential defined as below
readiness_probe["credential"] = ref(default_credential)
Same is failing with below error on latest master

  File "lamp-v4.py", line 129, in MySQLSubstrate
    readiness_probe["credential"] = ref(default_credential)
TypeError: 'ReadinessProbeType' object does not support item assignment

I had to change it to readiness_probe.credential = ref(default_credential) to make it work

Ability to set description attribute on all CalmVariable types

Would be useful if we could set the description attribute similarly to that of other classes / methods - via python docstring...

Support for `basic_with_cred` type credentials in http tasks

Right now, DSL only supports cred of type basic. It doesn't support by which we can give reference to existing credential in http tasks.

Launching bp with secret runtime variable shows variable in plain text

DSL prompts for values of secret runtime variables. User input is shown in plain text.

Screenshot of issue:

Example command used:

calm launch bp <bp_name> -a <app_name>

Include in AHV provider_spec missing settings (GPU, Serial and Boot)

When using calm create provider_spec -t AHV_VM it is missing the options for GPU, Serial and Boot.

Default value of secret variable shows blank while launching bp

Problem Statement:
Hi Team, We observed that when we try to launch bp with secret variables it is getting blank.

Steps to reproduce:

1. Try to launch a bp which has secret vars.

Findings/Troubleshooting Done:
Attaching a screenshot.

Include description attribute for variables

Variables in Calm UI can have a description additional to the label. In DSL it's only the label available.

Pretty print error responses

[2020-04-16 16:13:00 calm.dsl.api.connection [ERROR] Error Response: {'error': '{\n  "api_version": "3.1", \n  "code": 422, \n  "message_list": [\n    {\n      "details": {}, \n      "message": "Launch with \'HelloProfile\' Profile require configured accounts to be verified,\\n                    Please verify Account(s) from Settings page: TestProvider.", \n      "reason": "INVALID ENTITY"\n    }\n  ], \n  "state": "ERROR"\n}\n', 'code': 422}

Fix terminal emulation issues with running actions with --watch

Related command:

calm run action <action_name> --app <app_name> --watch

Dynamic terminal output displays some lines on the same lines as previous. Also, when process has finished, odd unicode-like characters are left at the terminal cursor.

Screenshot of broken lines attached.

Environment when reproducing issue:

• Kubuntu 20.04
• Konsole 19.12.3
• $TERM value xterm-256color

Add more validation when checking for valid provider spec with "calm validate"

When running commands like this:

calm validate provider_spec --file ./specs/MYSQL_AWS_provider_spec.yaml

The DSL will throw an exception such as the following:

AWS_provider_spec.yaml 
[2020-06-10 04:00:36] calm.dsl.cli.main [INFO] File ./specs/MYSQL_AWS_provider_spec.yaml is invalid AHV_VM spec
Traceback (most recent call last):
  File "/home/chris/Data/solutions/nutanix/calm-dsl/calm/dsl/cli/main.py", line 94, in validate_provider_spec
    Provider.validate_spec(spec)
  File "/home/chris/Data/solutions/nutanix/calm-dsl/calm/dsl/providers/base.py", line 68, in validate_spec
    Validator.validate(spec)
  File "/home/chris/Data/solutions/nutanix/calm-dsl/venv/lib/python3.8/site-packages/jsonschema/validators.py", line 353, in validate
    raise error
calm.dsl.tools.validator.validation_error: Additional properties are not allowed : 'associate_public_ip_address'

Seems as if the DSL assumes AHV spec unless explicitly told that it is not AHV spec.

This error can be avoided by altering command to be as follows (note addition of --type parameter):

calm validate provider_spec --file ./specs/MYSQL_AWS_provider_spec.yaml --type AWS_VM

DSL should be able to take the provided spec and check more than just presence of "type: PROVISION_AWS_VM" or use of "--type" parameter, then validate accordingly.

Comment from @abhijeetkaurav1st as follows:

First check if type attribute is defined, else go for type in spec

This approach makes sense.

Add a command line switch to log all API calls

Use-Case:

When scripting, I rely almost entirely on the ability to see what API calls are made by the CLI and I literally copy these URLs into the my scripts.

Notes:
Use [API] identifier in logs

Creating mutliple ahv vm classes mess up the boot config

Problem Statement:
Hi Team, in ahv Bootable disk check box getting disabled when we create a bp using decompiled bp. Discussed with Abhijeet Kaurav on the same.

Steps to reproduce:

1. Decompile a bp which has multiple services from calm pc.
2. Try to create a bp in Calm PC again with this decompiled bp
3. Check the bootable check box of any service.

Findings/Troubleshooting Done:
Disk index is getting changed. Attaching the screenshot.

Expectations:
Bootable check box should be enabled as it is.

Fail earlier if application name exists during launch

When launching an application from CLI it is not evaluating first if the application name already exists. It is asking for any variable and then it fails. If you have several variables to complete and at the latest step the launch fails because the app name exists, it is then a bit disappointing.

Launching a blueprint asks for variables of a day 2 action

When you launch a blueprint from the CLI, it is asking for variables that are part of day 2 actions and not in the profile.

app_profile.Default.action.Patch_Image.runbook.Default_Patch_Image_runbook.variable -> ControlVmIP.value []:
app_profile.Default.action.Patch_Image.runbook.Default_Patch_Image_runbook.variable -> IMG_URI.value []:

This must not be asked during initial launch. Only when running the action.

Command to create blueprint file based on user input for n service, m profiles etc.

Would be nice to have command to create blueprint file based on no. of service, profiles provided as options.
Ex:
calm init bp --ns 2 -np 3.
It should create bp having these entities accordingly.

Options should be a little more informative to give some more flexibility over the blueprint.

Variables created under pre-create tasks should not be declared on the service too

calm compile/create commands should blow a error if above behaviour is not followed inside the blueprint.

Update `calm create provider_spec` command to have support for latest changes from calm 2.9.0

Field operation type is introduced to merge disks and virtual disks into disks options only.
In Calm-dsl, disks and virtual disks are still exposed separately

Pagination support for various cli commands

Api response can be configured to support for pagination. For now default length for api response used is 20. So following steps may happen:

1. Increase default length in requested payload for apis
2. Add pagination if enquired entity still doesn't fit in default requested length(using offset).
3. Search for entity by name if api supports
   Affected commands: calm create provider_spec, calm describe project etc.

Improve the python file output by decompile method to not to show defaults for various entities

Right now, the output python file via decompile command shows the default value for various entities too.
For ex:

var9 = CalmVariable.Simple.Secret.date(
"",
label="var9_label",
regex="^((0[1-9]|[12]\d|3[01])/(0[1-9]|1[0-2])/[12]\d{3})$",
validate_regex=True,
is_mandatory=False,
is_hidden=False,
runtime=False,
)

It shows the default value for various attributes like is_mandatory, is_hidden, runtime etc. If we can just omit default values from python file, it would remove redundant code.

Support for reading env variables for server configs

Two scenario we need to take care of:

1. If no config file is present we should read from env variables only.
2. If some value are present in env and some in config file, we should give priority to env variables and then config file data

Get Blueprint Filter doesn't work

Executing calm get bps -f jg doesn't work

[2020-04-15 13:32:10] calm.dsl.api.connection [ERROR] Error Response: {'error': '{\n "api_version": "3.1", \n "code": 500, \n "message_list": [\n {\n "details": {}, \n "message": "list index out of range", \n "reason": "INTERNAL_ERROR"\n }\n ], \n "state": "ERROR"\n}\n', 'code': 500}

Unavailable Prism Central connection results in unexpected behaviour

When a user workstation to Prism Central connection is unavailable, the Calm DSL does not throw an error.

Expected behaviour is for the DSL to throw an exception or warning when the configured Prism Central connection is unavailable.

A good example is:

calm create provider_spec

When Prism Central connection isn't available, this should fail gracefully. Observed behaviour is the DSL does "nothing" - no error or exception, just permanent CLI cursor.

Allow blueprint name for `calm init bp`

calm init bp doesn not allow to pass a name. I'd like to use something like calm init bp --name myblueprint so I can use this command for the creation of all my new blueprints to keep consistency with the structure.