[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-ariatosca/pull/189 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the fe

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132168198 --- Diff: aria/orchestrator/topology/topology.py --- @@ -51,17 +51,17 @@ class Topology(issue.ReporterMixin): def __init__(self, *ar

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132167947 --- Diff: aria/orchestrator/topology/template_handler.py --- @@ -47,7 +47,7 @@ def coerce(self, **kwargs): self._model.wo

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132167856 --- Diff: aria/core.py --- @@ -118,8 +118,8 @@ def delete_service(self, service_id, force=False): def _parse_service_template(service_temp

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132168007 --- Diff: aria/orchestrator/topology/template_handler.py --- @@ -431,15 +430,14 @@ def dump(self, out_stream): out_stream.write('Subst

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132167801 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -550,7 +550,7 @@ def dump(self, out_stream): out_stream.write('Substit

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132123698 --- Diff: aria/modeling/utils.py --- @@ -175,7 +175,7 @@ def init(*args, **kwargs): return cls -def get_class_from_relatio

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132123290 --- Diff: aria/modeling/service_template.py --- @@ -530,6 +530,43 @@ def _next_name(self): return '{name}_{index}'.format(name=s

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132127992 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -527,78 +539,82 @@ def satisfy_requirements(self): for node

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132127314 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -527,78 +539,82 @@ def satisfy_requirements(self): for node

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r131950842 --- Diff: aria/core.py --- @@ -117,8 +118,8 @@ def delete_service(self, service_id, force=False): def _parse_service_template(service_temp

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132127179 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -527,78 +539,82 @@ def satisfy_requirements(self): for node

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132122306 --- Diff: aria/core.py --- @@ -74,18 +74,19 @@ def create_service(self, service_template_id, inputs, service_name=None): # setting no

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132122736 --- Diff: aria/core.py --- @@ -117,8 +118,8 @@ def delete_service(self, service_id, force=False): def _parse_service_template(service_temp

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132136296 --- Diff: aria/orchestrator/topology/template_handler.py --- @@ -584,13 +588,20 @@ def instantiate(self, instance_cls): return o

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132137574 --- Diff: aria/orchestrator/topology/topology.py --- @@ -50,28 +49,33 @@ class Topology(issue.Reporter): models.Type: models.Type

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132127229 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -527,78 +539,82 @@ def satisfy_requirements(self): for node

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132125166 --- Diff: aria/orchestrator/topology/common.py --- @@ -14,39 +14,53 @@ # limitations under the License. -class _Handler(object

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132137184 --- Diff: aria/orchestrator/topology/topology.py --- @@ -50,28 +49,33 @@ class Topology(issue.Reporter): models.Type: models.Type

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132137742 --- Diff: aria/orchestrator/topology/topology.py --- @@ -50,28 +49,33 @@ class Topology(issue.Reporter): models.Type: models.Type

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132129612 --- Diff: aria/orchestrator/topology/template_handler.py --- @@ -61,9 +64,9 @@ def instantiate(self, instance_cls, inputs=None): )

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132137988 --- Diff: aria/orchestrator/topology/topology.py --- @@ -50,28 +49,33 @@ class Topology(issue.Reporter): models.Type: models.Type

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-09 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r132140607 --- Diff: aria/utils/console.py --- @@ -31,51 +31,51 @@ class TopologyStylizer(object): -def __init__(self, indentation=2)

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-03 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r131134367 --- Diff: aria/orchestrator/topology/template_handler.py --- @@ -0,0 +1,597 @@ +# Licensed to the Apache Software Foundation (ASF) under one o

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-03 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r131133863 --- Diff: aria/utils/__init__.py --- @@ -16,3 +16,50 @@ """ General-purpose utilities package. """ + +from . import ( +

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-03 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r13111 --- Diff: aria/orchestrator/execution_plugin/instantiation.py --- @@ -110,20 +105,17 @@ def _configure_remote(operation): # Make su

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-03 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r131132912 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF) under one o

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-03 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r131132996 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF) under one o

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-03 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r131132744 --- Diff: aria/orchestrator/topology/__init__.py --- @@ -0,0 +1,16 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-03 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r131126610 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF) under one o

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-03 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r131084313 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-03 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r131089434 --- Diff: aria/orchestrator/topology/template_handler.py --- @@ -0,0 +1,597 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-03 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r131086718 --- Diff: aria/orchestrator/topology/template_handler.py --- @@ -0,0 +1,597 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-03 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r131083555 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-03 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r131095491 --- Diff: aria/orchestrator/topology/utils.py --- @@ -0,0 +1,67 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +#

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-03 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r131091773 --- Diff: aria/orchestrator/topology/template_handler.py --- @@ -0,0 +1,597 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-03 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r131085804 --- Diff: aria/orchestrator/topology/template_handler.py --- @@ -0,0 +1,597 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-03 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r131087352 --- Diff: aria/orchestrator/topology/template_handler.py --- @@ -0,0 +1,597 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130870769 --- Diff: aria/utils/console.py --- @@ -19,19 +19,68 @@ import os import sys +from StringIO import StringIO from con

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130873555 --- Diff: extensions/aria_extension_tosca/simple_v1_0/modeling/__init__.py --- @@ -497,8 +497,8 @@ def get(name, default=None): d

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130902625 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130902831 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130894358 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130844628 --- Diff: aria/parser/consumption/modeling.py --- @@ -105,9 +106,26 @@ def consume(self): self.context.validation.report('Instanti

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130842710 --- Diff: aria/parser/consumption/modeling.py --- @@ -74,7 +75,7 @@ def dump(self): raw = self.context.modeling.template_as_raw

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130844470 --- Diff: aria/parser/consumption/modeling.py --- @@ -105,9 +106,26 @@ def consume(self): self.context.validation.report('Instanti

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130876740 --- Diff: tests/parser/service_templates.py --- @@ -27,6 +27,7 @@ def consume_literal(literal, consumer_class_name='instance', cache=True, no_issu

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130829336 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130893648 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130826035 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130858804 --- Diff: aria/parser/consumption/modeling.py --- @@ -74,7 +75,7 @@ def dump(self): raw = self.context.modeling.template_as_raw

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130823646 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130871342 --- Diff: aria/utils/console.py --- @@ -19,19 +19,68 @@ import os import sys +from StringIO import StringIO from con

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130827261 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130820499 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130906157 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130824971 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130830529 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130900925 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130828395 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130875236 --- Diff: tests/instantiation/test_configuration.py --- @@ -165,8 +165,11 @@ def test_remote(service): def test_reserved_arguments

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130828782 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130876338 --- Diff: tests/instantiation/test_configuration.py --- @@ -165,8 +165,11 @@ def test_remote(service): def test_reserved_arguments

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130868717 --- Diff: aria/utils/console.py --- @@ -19,19 +19,68 @@ import os import sys +from StringIO import StringIO from con

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130868151 --- Diff: aria/utils/__init__.py --- @@ -16,3 +16,50 @@ """ General-purpose utilities package. """ + +from . import ( +

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130879245 --- Diff: tests/parser/test_reqs_caps.py --- @@ -0,0 +1,29 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +# cont

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130826916 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-02 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130904352 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130594593 --- Diff: aria/orchestrator/topology/common.py --- @@ -0,0 +1,52 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130606443 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130597368 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130600048 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130593730 --- Diff: aria/orchestrator/topology/common.py --- @@ -0,0 +1,52 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130593845 --- Diff: aria/orchestrator/topology/common.py --- @@ -0,0 +1,52 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130594613 --- Diff: aria/orchestrator/topology/common.py --- @@ -0,0 +1,52 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130593980 --- Diff: aria/orchestrator/topology/common.py --- @@ -0,0 +1,52 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130593564 --- Diff: aria/orchestrator/topology/__init__.py --- @@ -0,0 +1,16 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130583765 --- Diff: aria/cli/commands/service_templates.py --- @@ -57,7 +57,7 @@ def service_templates(): def show(service_template_name, model_storage,

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130604096 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130607151 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130593153 --- Diff: aria/orchestrator/execution_plugin/instantiation.py --- @@ -110,20 +105,17 @@ def _configure_remote(operation): # Make sur

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130595525 --- Diff: aria/orchestrator/topology/common.py --- @@ -0,0 +1,52 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130606598 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130600640 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130607886 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130602178 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130601638 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130599473 --- Diff: aria/orchestrator/topology/instance_handler.py --- @@ -0,0 +1,644 @@ +# Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130592682 --- Diff: aria/modeling/utils.py --- @@ -244,3 +173,9 @@ def init(*args, **kwargs): cls.__doc__ = cls.__bases__[-1].__doc__

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130586353 --- Diff: aria/core.py --- @@ -122,6 +118,8 @@ def delete_service(self, service_id, force=False): def _parse_service_template(service_temp

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130595794 --- Diff: aria/orchestrator/topology/common.py --- @@ -0,0 +1,52 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more +

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-08-01 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130585942 --- Diff: aria/core.py --- @@ -67,31 +68,25 @@ def delete_service_template(self, service_template_id): self.resource_storage.service_

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-07-31 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130331901 --- Diff: extensions/aria_extension_tosca/simple_v1_0/modeling/__init__.py --- @@ -497,8 +497,8 @@ def get(name, default=None):

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-07-31 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130331316 --- Diff: aria/orchestrator/topology/topology.py --- @@ -0,0 +1,217 @@ +# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-07-31 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130330669 --- Diff: aria/cli/commands/services.py --- @@ -20,6 +20,7 @@ import os from StringIO import StringIO +from aria.orchestrator.

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-07-31 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130331174 --- Diff: aria/orchestrator/execution_plugin/instantiation.py --- @@ -110,20 +106,24 @@ def _configure_remote(operation): # Make su

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-07-31 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130331038 --- Diff: aria/core.py --- @@ -67,31 +68,26 @@ def delete_service_template(self, service_template_id): self.resource_storage.service

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-07-31 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130330792 --- Diff: aria/modeling/functions.py --- @@ -103,10 +102,12 @@ def evaluate(value, container_holder, report_issues=False): # pylint: disable=to

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-07-31 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130330770 --- Diff: aria/core.py --- @@ -122,6 +118,8 @@ def delete_service(self, service_id, force=False): def _parse_service_template(service_tem

[GitHub] incubator-ariatosca pull request #189: ARIA-174 Refactor instantiation phase

2017-07-31 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/189#discussion_r130330616 --- Diff: aria/cli/commands/service_templates.py --- @@ -28,7 +28,7 @@ from ...storage import exceptions as storage_exceptions from ...p

  1   2   >