Github user asfgit closed the pull request at:
https://github.com/apache/incubator-ariatosca/pull/12
---
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 fea
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86780724
--- Diff: tests/workflows/api/test_task_graph.py ---
@@ -298,8 +491,139 @@ def
test_remove_dependency_nonexistent_dependency(self, graph):
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86781298
--- Diff: tests/workflows/api/test_task_graph.py ---
@@ -298,8 +491,139 @@ def
test_remove_dependency_nonexistent_dependency(self, graph):
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86781784
--- Diff: tests/workflows/api/test_task_graph.py ---
@@ -298,8 +491,139 @@ def
test_remove_dependency_nonexistent_dependency(self, graph):
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86781183
--- Diff: tests/workflows/api/test_task_graph.py ---
@@ -298,8 +491,139 @@ def
test_remove_dependency_nonexistent_dependency(self, graph):
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86782005
--- Diff: tests/workflows/api/test_task_graph.py ---
@@ -298,8 +491,139 @@ def
test_remove_dependency_nonexistent_dependency(self, graph):
Github user dankilman commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86780395
--- Diff: requirements.txt ---
@@ -15,4 +15,5 @@ networkx==1.9
requests==2.7.0
retrying==1.3.3
blinker==1.4
+ordereddict==1.1
Github user dankilman commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86779517
--- Diff: aria/workflows/core/engine.py ---
@@ -22,12 +22,11 @@
import networkx
-from aria import events, logger
-from a
Github user dankilman commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86778287
--- Diff: aria/events/builtin_event_handler.py ---
@@ -36,37 +36,24 @@
@sent_task_signal.connect
def _task_sent(task, *args, **kw
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86746343
--- Diff: setup.py ---
@@ -41,6 +41,11 @@
except ImportError:
install_requires.append('importlib')
+try:
--- End diff --
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86746403
--- Diff: tests/mock/context.py ---
@@ -0,0 +1,37 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor li
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86742769
--- Diff: aria/workflows/builtin/heal.py ---
@@ -73,46 +72,52 @@ def heal_uninstall(context, graph,
failing_node_instances, targeted_node_instanc
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86745883
--- Diff: requirements.txt ---
@@ -11,8 +11,10 @@
# limitations under the License.
PyYAML==3.10
+billiard==3.3.0.20
network
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86747953
--- Diff: tests/workflows/builtin/test_execute_operation.py ---
@@ -0,0 +1,74 @@
+# Licensed to the Apache Software Foundation (ASF) under one o
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86747839
--- Diff: tests/workflows/builtin/test_execute_operation.py ---
@@ -0,0 +1,74 @@
+# Licensed to the Apache Software Foundation (ASF) under one o
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86747144
--- Diff: tests/workflows/api/test_task_graph.py ---
@@ -0,0 +1,714 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86744626
--- Diff: aria/workflows/core/__init__.py ---
@@ -12,3 +12,9 @@
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86747179
--- Diff: tests/workflows/api/test_task_graph.py ---
@@ -0,0 +1,714 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86746483
--- Diff: tests/mock/context.py ---
@@ -0,0 +1,37 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor li
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86738881
--- Diff: aria/workflows/api/task_graph.py ---
@@ -0,0 +1,293 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# co
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86739211
--- Diff: aria/workflows/api/task_graph.py ---
@@ -0,0 +1,293 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# co
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86737581
--- Diff: aria/workflows/api/task.py ---
@@ -0,0 +1,98 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contribut
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86744468
--- Diff: aria/workflows/builtin/workflows.py ---
@@ -211,17 +209,24 @@ def relationships_tasks(operation_name, context,
node_instance):
s
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86748052
--- Diff: tests/workflows/builtin/test_install.py ---
@@ -0,0 +1,63 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86745812
--- Diff: requirements.txt ---
@@ -11,8 +11,10 @@
# limitations under the License.
PyYAML==3.10
+billiard==3.3.0.20
--- End
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86746856
--- Diff: tests/workflows/api/test_task.py ---
@@ -0,0 +1,96 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# con
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86746418
--- Diff: tests/mock/context.py ---
@@ -0,0 +1,37 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor li
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86748629
--- Diff: aria/.pylintrc ---
@@ -62,7 +62,7 @@ confidence=
# --enable=similarities". If you want to run only the classes checker, but
have
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86747985
--- Diff: tests/workflows/builtin/test_heal.py ---
@@ -0,0 +1,112 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86748112
--- Diff: tests/workflows/builtin/test_uninstall.py ---
@@ -0,0 +1,63 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86742397
--- Diff: aria/workflows/builtin/heal.py ---
@@ -19,50 +19,49 @@
from aria import workflow
-from .uninstall import uninstall
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86746290
--- Diff: setup.py ---
@@ -41,6 +41,11 @@
except ImportError:
install_requires.append('importlib')
+try:
--- End diff --
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86688866
--- Diff: aria/storage/models.py ---
@@ -397,3 +369,37 @@ class Plugin(Model):
excluded_wheels = Field()
supported_py_versions = F
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86689253
--- Diff: aria/workflows/core/task.py ---
@@ -0,0 +1,182 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contrib
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86688440
--- Diff: aria/.pylintrc ---
@@ -62,7 +62,7 @@ confidence=
# --enable=similarities". If you want to run only the classes checker, but
have
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86688451
--- Diff: aria/context/__init__.py ---
@@ -0,0 +1,20 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86688390
--- Diff: tox.ini ---
@@ -30,7 +30,7 @@ commands=pytest tests --cov-report term-missing --cov aria
commands=pytest tests --cov-report term-miss
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86688997
--- Diff: aria/workflows/api/task.py ---
@@ -0,0 +1,98 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contribut
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86688960
--- Diff: aria/workflows/api/task.py ---
@@ -0,0 +1,98 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contribut
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86688898
--- Diff: aria/workflows/api/task.py ---
@@ -0,0 +1,98 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contribut
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86689008
--- Diff: aria/workflows/api/task.py ---
@@ -0,0 +1,98 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contribut
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86688989
--- Diff: aria/workflows/api/task.py ---
@@ -0,0 +1,98 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contribut
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86688764
--- Diff: aria/decorators.py ---
@@ -34,61 +35,55 @@ def workflow(
if func is None:
return partial(
workflow,
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86689206
--- Diff: aria/workflows/core/task.py ---
@@ -0,0 +1,182 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contrib
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86688646
--- Diff: aria/context/workflow.py ---
@@ -164,48 +142,48 @@ def get_blueprint_resource_data(self, path=None):
return self.resource.blu
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86689180
--- Diff: aria/workflows/core/task.py ---
@@ -0,0 +1,182 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contrib
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86688701
--- Diff: aria/context/workflow.py ---
@@ -164,48 +142,48 @@ def get_blueprint_resource_data(self, path=None):
return self.resource.blu
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86689144
--- Diff: aria/workflows/core/engine.py ---
@@ -27,7 +27,7 @@
from .. import exceptions
from . import translation
-from . import
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86688529
--- Diff: aria/context/workflow.py ---
@@ -164,48 +142,48 @@ def get_blueprint_resource_data(self, path=None):
return self.resource.blu
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86688608
--- Diff: aria/context/workflow.py ---
@@ -164,48 +142,48 @@ def get_blueprint_resource_data(self, path=None):
return self.resource.blu
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r86688410
--- Diff: tox.ini ---
@@ -30,7 +30,7 @@ commands=pytest tests --cov-report term-missing --cov aria
commands=pytest tests --cov-report term-miss
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r85103593
--- Diff: tests/workflows/builtin/test_heal.py ---
@@ -0,0 +1,82 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+#
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r85104111
--- Diff: tests/workflows/builtin/test_heal.py ---
@@ -0,0 +1,82 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+#
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r85103673
--- Diff: tests/workflows/builtin/test_heal.py ---
@@ -0,0 +1,82 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+#
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r85103221
--- Diff: tests/workflows/builtin/test_install.py ---
@@ -0,0 +1,36 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r85102949
--- Diff: tests/workflows/builtin/test_execute_operation.py ---
@@ -0,0 +1,45 @@
+# Licensed to the Apache Software Foundation (ASF) under one o
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r85102304
--- Diff: tests/workflows/builtin/__init__.py ---
@@ -0,0 +1,177 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+#
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r85097566
--- Diff: aria/workflows/builtin/heal.py ---
@@ -141,35 +136,32 @@ def heal_install(context, graph,
failing_node_instances, targeted_node_instances
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r85097190
--- Diff: aria/workflows/builtin/heal.py ---
@@ -34,28 +34,27 @@ def heal(context, graph, node_instance_id):
:param node_instance_id: the i
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r85097366
--- Diff: aria/workflows/builtin/heal.py ---
@@ -141,35 +136,32 @@ def heal_install(context, graph,
failing_node_instances, targeted_node_instances
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r85099118
--- Diff: aria/workflows/builtin/workflows.py ---
@@ -135,30 +136,27 @@ def uninstall_node_instance(graph, context,
node_instance):
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r85099050
--- Diff: aria/workflows/builtin/workflows.py ---
@@ -57,59 +57,60 @@ def install_node_instance(context, graph,
node_instance):
'a
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r85096928
--- Diff: aria/workflows/api/task_graph.py ---
@@ -55,6 +55,23 @@ def tasks(self):
for _, data in self._graph.nodes_iter(data=True):
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r84592356
--- Diff: tests/workflows/api/test_task_graph.py ---
@@ -0,0 +1,658 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r84592363
--- Diff: tests/workflows/api/test_task_graph.py ---
@@ -0,0 +1,658 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r84592295
--- Diff: aria/workflows/api/task_graph.py ---
@@ -0,0 +1,305 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# co
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r84592297
--- Diff: aria/workflows/api/task_graph.py ---
@@ -0,0 +1,305 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# co
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r84592497
--- Diff: aria/workflows/api/task_graph.py ---
@@ -0,0 +1,305 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# co
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r84592495
--- Diff: aria/workflows/api/task_graph.py ---
@@ -0,0 +1,305 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# co
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r84592044
--- Diff: aria/workflows/api/task_graph.py ---
@@ -0,0 +1,305 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# co
Github user mxmrlv commented on a diff in the pull request:
https://github.com/apache/incubator-ariatosca/pull/12#discussion_r84592475
--- Diff: tests/workflows/api/test_task_graph.py ---
@@ -0,0 +1,658 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
GitHub user ran-z opened a pull request:
https://github.com/apache/incubator-ariatosca/pull/12
ARIA-3 Create an API for creating workflows
An API for creating workflows. Users can build graphs of tasks and
set depenedencies in between tasks to execute them in a specific
order
72 matches
Mail list logo