[GitHub] incubator-ariatosca issue #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread tliron
Github user tliron commented on the issue: https://github.com/apache/incubator-ariatosca/pull/188 Hey guys, I suggest going over the complex use cases in the [Clearwater

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread tliron
Github user tliron commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130247609 --- Diff: aria/orchestrator/execution_plugin/ctx_proxy/server.py --- @@ -150,43 +150,101 @@ def __exit__(self, *args, **kwargs):

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread tliron
Github user tliron commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130247616 --- Diff: aria/orchestrator/execution_plugin/ctx_proxy/server.py --- @@ -150,43 +150,101 @@ def __exit__(self, *args, **kwargs):

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread tliron
Github user tliron commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130247602 --- Diff: aria/orchestrator/execution_plugin/ctx_proxy/server.py --- @@ -150,43 +150,101 @@ def __exit__(self, *args, **kwargs):

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread tliron
Github user tliron commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130247513 --- Diff: aria/orchestrator/execution_plugin/ctx_proxy/server.py --- @@ -150,43 +150,101 @@ def __exit__(self, *args, **kwargs):

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread tliron
Github user tliron commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130247470 --- Diff: aria/orchestrator/execution_plugin/ctx_proxy/server.py --- @@ -150,43 +150,101 @@ def __exit__(self, *args, **kwargs):

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread tliron
Github user tliron commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130247255 --- Diff: aria/orchestrator/execution_plugin/ctx_proxy/server.py --- @@ -150,43 +150,101 @@ def __exit__(self, *args, **kwargs):

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread tliron
Github user tliron commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130247244 --- Diff: aria/storage/collection_instrumentation.py --- @@ -213,32 +235,32 @@ def __init__(self, mapi, *args, **kwargs): """

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread tliron
Github user tliron commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130247176 --- Diff: aria/orchestrator/workflows/api/task.py --- @@ -140,13 +140,18 @@ def __init__(self, self.arguments =

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread tliron
Github user tliron commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130247158 --- Diff: aria/orchestrator/workflows/api/task.py --- @@ -140,13 +140,18 @@ def __init__(self, self.arguments =

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread tliron
Github user tliron commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130247107 --- Diff: aria/orchestrator/context/common.py --- @@ -38,10 +38,27 @@ class BaseContext(object): """

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

2017-07-30 Thread mxmrlv
GitHub user mxmrlv opened a pull request: https://github.com/apache/incubator-ariatosca/pull/189 ARIA-174 Refactor instantiation phase You can merge this pull request into a Git repository by running: $ git pull https://github.com/apache/incubator-ariatosca

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

2017-07-30 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/incubator-ariatosca/pull/189 Can one of the admins verify this patch? --- 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

[3/3] incubator-ariatosca git commit: wip

2017-07-30 Thread mxmrlv
wip Project: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/commit/90107846 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/tree/90107846 Diff:

[1/3] incubator-ariatosca git commit: wip [Forced Update!]

2017-07-30 Thread mxmrlv
Repository: incubator-ariatosca Updated Branches: refs/heads/ARIA-174-Refactor-instantiation-phase 4b6e51b0a -> 901078467 (forced update) http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/90107846/aria/parser/consumption/modeling.py

incubator-ariatosca git commit: fixed issue with reqs and caps issue where target capability types are not properly satisfied

2017-07-30 Thread mxmrlv
Repository: incubator-ariatosca Updated Branches: refs/heads/ARIA-174-Refactor-instantiation-phase d6b268958 -> 4b6e51b0a fixed issue with reqs and caps issue where target capability types are not properly satisfied Project: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/repo

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130244198 --- Diff: tests/orchestrator/execution_plugin/test_ctx_proxy_server.py --- @@ -39,26 +39,26 @@ def test_dict_prop_access_get_key(self, server):

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130244114 --- Diff: aria/orchestrator/execution_plugin/ctx_proxy/server.py --- @@ -150,43 +150,101 @@ def __exit__(self, *args, **kwargs):

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread AviaE
Github user AviaE commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130244178 --- Diff: aria/orchestrator/execution_plugin/ctx_proxy/server.py --- @@ -150,43 +150,101 @@ def __exit__(self, *args, **kwargs):

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130240698 --- Diff: aria/orchestrator/execution_plugin/ctx_proxy/server.py --- @@ -150,43 +150,101 @@ def __exit__(self, *args, **kwargs):

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130237525 --- Diff: aria/orchestrator/workflows/api/task.py --- @@ -140,13 +140,18 @@ def __init__(self, self.arguments =

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130239769 --- Diff: aria/orchestrator/execution_plugin/ctx_proxy/server.py --- @@ -150,43 +150,101 @@ def __exit__(self, *args, **kwargs):

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130239725 --- Diff: aria/orchestrator/execution_plugin/ctx_proxy/server.py --- @@ -150,43 +150,101 @@ def __exit__(self, *args, **kwargs):

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130238262 --- Diff: aria/orchestrator/execution_plugin/ctx_proxy/server.py --- @@ -150,43 +150,101 @@ def __exit__(self, *args, **kwargs):

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130241087 --- Diff: aria/orchestrator/execution_plugin/ctx_proxy/server.py --- @@ -150,43 +150,101 @@ def __exit__(self, *args, **kwargs):

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130237066 --- Diff: aria/orchestrator/context/common.py --- @@ -38,10 +38,27 @@ class BaseContext(object): """

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130238356 --- Diff: aria/orchestrator/execution_plugin/ctx_proxy/server.py --- @@ -150,43 +150,101 @@ def __exit__(self, *args, **kwargs):

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130237667 --- Diff: aria/storage/collection_instrumentation.py --- @@ -213,32 +235,32 @@ def __init__(self, mapi, *args, **kwargs): """

[GitHub] incubator-ariatosca pull request #188: ARIA-324 Refactor ctx proxy access

2017-07-30 Thread mxmrlv
Github user mxmrlv commented on a diff in the pull request: https://github.com/apache/incubator-ariatosca/pull/188#discussion_r130239705 --- Diff: aria/orchestrator/execution_plugin/ctx_proxy/server.py --- @@ -150,43 +150,101 @@ def __exit__(self, *args, **kwargs):

[incubator-ariatosca] Git Push Summary

2017-07-30 Thread avia
Repository: incubator-ariatosca Updated Branches: refs/heads/ARIA-313-fix-handling-the-required-field-of-inputs [deleted] 51e4ed004

[GitHub] incubator-ariatosca pull request #187: ARIA-313 Fix handling the `required` ...

2017-07-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-ariatosca/pull/187 --- 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

incubator-ariatosca git commit: ARIA-313 Fix handling the `required` field of inputs [Forced Update!]

2017-07-30 Thread avia
Repository: incubator-ariatosca Updated Branches: refs/heads/ARIA-313-fix-handling-the-required-field-of-inputs ee55bf4ca -> 51e4ed004 (forced update) ARIA-313 Fix handling the `required` field of inputs The required field is handled in the following way: topology_template inputs: