CLI commands for parser #25
CLI commands for parser #25
Conversation
c079c54
to
bd4cf91
Compare
@@ -295,3 +294,68 @@ def _load_workflow_handler(handler_path): | |||
except AttributeError: | |||
# todo: exception handler | |||
raise | |||
|
|||
from ..parser.consumption import (ConsumptionContext, ConsumerChain, Read, Validate, Model, Types, Inputs, Instance) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why are the imports here and not at the top?
self.logger.info('blueprint parsed successfully') | ||
deployment_plan = prepare_deployment_plan(plan=plan.copy(), inputs=inputs) | ||
return plan, deployment_plan | ||
#plan = parse_from_path(blueprint_path) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we implement this functionality using the new parser? if not, let's remove the commented lines
@@ -44,6 +46,7 @@ def __init__(self, *args, **kwargs): | |||
self.commands = { | |||
'init': InitCommand.with_logger(base_logger=self.logger), | |||
'execute': ExecuteCommand.with_logger(base_logger=self.logger), | |||
'parse': ParseCommand.with_logger(base_logger=self.logger), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why did you decide to leave out spec
?
==== | ||
|
||
See http://ariatosca.org/ | ||
[ARIA](http://ariatosca.org/) is a minimal TOSCA orchestrator, as well as a platform for building | ||
TOSCA-based products. Its features can be accessed via a well-documented Python API, as well as a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no language-agnostic RESTful API for now
pip install tox | ||
tox | ||
|
||
To build the documentation: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we currently have no make file (or docs), so probably best to remove this line
Also, please amend to commit message to include |
Please see this document regarding commit message guidelines: |
76acef4
to
4ca7579
Compare
4ca7579
to
3323819
Compare
Also includes some fixed to TOSCA extension.