[19/30] incubator-ariatosca git commit: ARIA-324 Refactor ctx proxy access
ARIA-324 Refactor ctx proxy access Our previous use of "." to delimit nested dict keys was wrong (keys could have a ".") and inflexible. The new implementation uses subsequent args to move into the dict. The same format can now be used to access object attributes. This commit also changes how to support setting values: we must now use "=" as the penultimate argument with the new value following. Calling functions now requires delimiting function arguments via "[" and "]" arguments. This makes it possible to continue accessing the return value of a function after the "]". Also fixed: callables will now "grab" the number of args they need instead of all remaining args, making it possible to do further inspection on the returned value from the callable. To allow for this, kwargs are now expected as the first arg rather than the last. Relatedly, this commit instruments all parameter fields from all models and fixes related bugs in the instrumentation implementation. Furthmore, this commit fixes a bad null check in the ctx client, and also allows it to retrieve Unicode data. Project: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/commit/a43bc6b1 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/tree/a43bc6b1 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/diff/a43bc6b1 Branch: refs/heads/new_wagon_setuptools Commit: a43bc6b1706398dd79eabb2f49b6dafcadc8dfde Parents: 51e4ed0 Author: Tal Liron Authored: Thu Jul 27 17:58:17 2017 -0500 Committer: Tal Liron Committed: Tue Aug 8 09:32:15 2017 -0500 -- .travis.yml | 2 +- aria/orchestrator/context/common.py | 19 +- aria/orchestrator/context/operation.py | 46 ++--- .../execution_plugin/ctx_proxy/client.py| 7 +- .../execution_plugin/ctx_proxy/server.py| 196 +-- aria/orchestrator/workflows/api/task.py | 13 +- aria/storage/collection_instrumentation.py | 88 - aria/storage/core.py| 6 +- examples/hello-world/scripts/configure.sh | 29 ++- examples/hello-world/scripts/start.sh | 38 ++-- examples/hello-world/scripts/stop.sh| 17 +- tests/end2end/testenv.py| 24 ++- .../execution_plugin/test_ctx_proxy_server.py | 150 -- .../orchestrator/execution_plugin/test_local.py | 94 - tests/requirements.txt | 6 +- tests/resources/scripts/test_ssh.sh | 46 ++--- tox.ini | 8 +- 17 files changed, 358 insertions(+), 431 deletions(-) -- http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/a43bc6b1/.travis.yml -- diff --git a/.travis.yml b/.travis.yml index edd12d4..c8b7645 100644 --- a/.travis.yml +++ b/.travis.yml @@ -15,7 +15,7 @@ sudo: false language: python python: - - "2.7" + - '2.7' env: - TOX_ENV=pylint_code http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/a43bc6b1/aria/orchestrator/context/common.py -- diff --git a/aria/orchestrator/context/common.py b/aria/orchestrator/context/common.py index f400142..3c5f618 100644 --- a/aria/orchestrator/context/common.py +++ b/aria/orchestrator/context/common.py @@ -38,10 +38,27 @@ class BaseContext(object): """ INSTRUMENTATION_FIELDS = ( +modeling.models.Service.inputs, +modeling.models.ServiceTemplate.inputs, +modeling.models.Policy.properties, +modeling.models.PolicyTemplate.properties, modeling.models.Node.attributes, modeling.models.Node.properties, modeling.models.NodeTemplate.attributes, -modeling.models.NodeTemplate.properties +modeling.models.NodeTemplate.properties, +modeling.models.Group.properties, +modeling.models.GroupTemplate.properties, +modeling.models.Capability.properties, +# TODO ARIA-279: modeling.models.Capability.attributes, +modeling.models.CapabilityTemplate.properties, +# TODO ARIA-279: modeling.models.CapabilityTemplate.attributes +modeling.models.Relationship.properties, +modeling.models.Artifact.properties, +modeling.models.ArtifactTemplate.properties, +modeling.models.Interface.inputs, +modeling.models.InterfaceTemplate.inputs, +modeling.models.Operation.inputs, +modeling.models.OperationTemplate.inputs ) class PrefixedLogger(object): http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/a43bc6b1/aria/orchestrator/context/operation.py
[1/8] incubator-ariatosca git commit: ARIA-324 Refactor ctx proxy access [Forced Update!]
Repository: incubator-ariatosca Updated Branches: refs/heads/ARIA-174-Refactor-instantiation-phase 679bc81c5 -> 2a4527c75 (forced update) ARIA-324 Refactor ctx proxy access Our previous use of "." to delimit nested dict keys was wrong (keys could have a ".") and inflexible. The new implementation uses subsequent args to move into the dict. The same format can now be used to access object attributes. This commit also changes how to support setting values: we must now use "=" as the penultimate argument with the new value following. Calling functions now requires delimiting function arguments via "[" and "]" arguments. This makes it possible to continue accessing the return value of a function after the "]". Also fixed: callables will now "grab" the number of args they need instead of all remaining args, making it possible to do further inspection on the returned value from the callable. To allow for this, kwargs are now expected as the first arg rather than the last. Relatedly, this commit instruments all parameter fields from all models and fixes related bugs in the instrumentation implementation. Furthmore, this commit fixes a bad null check in the ctx client, and also allows it to retrieve Unicode data. Project: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/commit/a43bc6b1 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/tree/a43bc6b1 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/diff/a43bc6b1 Branch: refs/heads/ARIA-174-Refactor-instantiation-phase Commit: a43bc6b1706398dd79eabb2f49b6dafcadc8dfde Parents: 51e4ed0 Author: Tal Liron Authored: Thu Jul 27 17:58:17 2017 -0500 Committer: Tal Liron Committed: Tue Aug 8 09:32:15 2017 -0500 -- .travis.yml | 2 +- aria/orchestrator/context/common.py | 19 +- aria/orchestrator/context/operation.py | 46 ++--- .../execution_plugin/ctx_proxy/client.py| 7 +- .../execution_plugin/ctx_proxy/server.py| 196 +-- aria/orchestrator/workflows/api/task.py | 13 +- aria/storage/collection_instrumentation.py | 88 - aria/storage/core.py| 6 +- examples/hello-world/scripts/configure.sh | 29 ++- examples/hello-world/scripts/start.sh | 38 ++-- examples/hello-world/scripts/stop.sh| 17 +- tests/end2end/testenv.py| 24 ++- .../execution_plugin/test_ctx_proxy_server.py | 150 -- .../orchestrator/execution_plugin/test_local.py | 94 - tests/requirements.txt | 6 +- tests/resources/scripts/test_ssh.sh | 46 ++--- tox.ini | 8 +- 17 files changed, 358 insertions(+), 431 deletions(-) -- http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/a43bc6b1/.travis.yml -- diff --git a/.travis.yml b/.travis.yml index edd12d4..c8b7645 100644 --- a/.travis.yml +++ b/.travis.yml @@ -15,7 +15,7 @@ sudo: false language: python python: - - "2.7" + - '2.7' env: - TOX_ENV=pylint_code http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/a43bc6b1/aria/orchestrator/context/common.py -- diff --git a/aria/orchestrator/context/common.py b/aria/orchestrator/context/common.py index f400142..3c5f618 100644 --- a/aria/orchestrator/context/common.py +++ b/aria/orchestrator/context/common.py @@ -38,10 +38,27 @@ class BaseContext(object): """ INSTRUMENTATION_FIELDS = ( +modeling.models.Service.inputs, +modeling.models.ServiceTemplate.inputs, +modeling.models.Policy.properties, +modeling.models.PolicyTemplate.properties, modeling.models.Node.attributes, modeling.models.Node.properties, modeling.models.NodeTemplate.attributes, -modeling.models.NodeTemplate.properties +modeling.models.NodeTemplate.properties, +modeling.models.Group.properties, +modeling.models.GroupTemplate.properties, +modeling.models.Capability.properties, +# TODO ARIA-279: modeling.models.Capability.attributes, +modeling.models.CapabilityTemplate.properties, +# TODO ARIA-279: modeling.models.CapabilityTemplate.attributes +modeling.models.Relationship.properties, +modeling.models.Artifact.properties, +modeling.models.ArtifactTemplate.properties, +modeling.models.Interface.inputs, +modeling.models.InterfaceTemplate.inputs, +modeling.models.Operation.inputs, +modeling.models.OperationTemplate.inputs ) class P
[2/4] incubator-ariatosca git commit: ARIA-324 Refactor ctx proxy access
ARIA-324 Refactor ctx proxy access Our previous use of "." to delimit nested dict keys was wrong (keys could have a ".") and inflexible. The new implementation uses subsequent args to move into the dict. The same format can now be used to access object attributes. This commit also changes how to support setting values: we must now use "=" as the penultimate argument with the new value following. Calling functions now requires delimiting function arguments via "[" and "]" arguments. This makes it possible to continue accessing the return value of a function after the "]". Also fixed: callables will now "grab" the number of args they need instead of all remaining args, making it possible to do further inspection on the returned value from the callable. To allow for this, kwargs are now expected as the first arg rather than the last. Relatedly, this commit instruments all parameter fields from all models and fixes related bugs in the instrumentation implementation. Furthmore, this commit fixes a bad null check in the ctx client, and also allows it to retrieve Unicode data. Project: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/commit/a43bc6b1 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/tree/a43bc6b1 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/diff/a43bc6b1 Branch: refs/heads/ARIA-321-clearwater Commit: a43bc6b1706398dd79eabb2f49b6dafcadc8dfde Parents: 51e4ed0 Author: Tal Liron Authored: Thu Jul 27 17:58:17 2017 -0500 Committer: Tal Liron Committed: Tue Aug 8 09:32:15 2017 -0500 -- .travis.yml | 2 +- aria/orchestrator/context/common.py | 19 +- aria/orchestrator/context/operation.py | 46 ++--- .../execution_plugin/ctx_proxy/client.py| 7 +- .../execution_plugin/ctx_proxy/server.py| 196 +-- aria/orchestrator/workflows/api/task.py | 13 +- aria/storage/collection_instrumentation.py | 88 - aria/storage/core.py| 6 +- examples/hello-world/scripts/configure.sh | 29 ++- examples/hello-world/scripts/start.sh | 38 ++-- examples/hello-world/scripts/stop.sh| 17 +- tests/end2end/testenv.py| 24 ++- .../execution_plugin/test_ctx_proxy_server.py | 150 -- .../orchestrator/execution_plugin/test_local.py | 94 - tests/requirements.txt | 6 +- tests/resources/scripts/test_ssh.sh | 46 ++--- tox.ini | 8 +- 17 files changed, 358 insertions(+), 431 deletions(-) -- http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/a43bc6b1/.travis.yml -- diff --git a/.travis.yml b/.travis.yml index edd12d4..c8b7645 100644 --- a/.travis.yml +++ b/.travis.yml @@ -15,7 +15,7 @@ sudo: false language: python python: - - "2.7" + - '2.7' env: - TOX_ENV=pylint_code http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/a43bc6b1/aria/orchestrator/context/common.py -- diff --git a/aria/orchestrator/context/common.py b/aria/orchestrator/context/common.py index f400142..3c5f618 100644 --- a/aria/orchestrator/context/common.py +++ b/aria/orchestrator/context/common.py @@ -38,10 +38,27 @@ class BaseContext(object): """ INSTRUMENTATION_FIELDS = ( +modeling.models.Service.inputs, +modeling.models.ServiceTemplate.inputs, +modeling.models.Policy.properties, +modeling.models.PolicyTemplate.properties, modeling.models.Node.attributes, modeling.models.Node.properties, modeling.models.NodeTemplate.attributes, -modeling.models.NodeTemplate.properties +modeling.models.NodeTemplate.properties, +modeling.models.Group.properties, +modeling.models.GroupTemplate.properties, +modeling.models.Capability.properties, +# TODO ARIA-279: modeling.models.Capability.attributes, +modeling.models.CapabilityTemplate.properties, +# TODO ARIA-279: modeling.models.CapabilityTemplate.attributes +modeling.models.Relationship.properties, +modeling.models.Artifact.properties, +modeling.models.ArtifactTemplate.properties, +modeling.models.Interface.inputs, +modeling.models.InterfaceTemplate.inputs, +modeling.models.Operation.inputs, +modeling.models.OperationTemplate.inputs ) class PrefixedLogger(object): http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/a43bc6b1/aria/orchestrator/context/operation.py -
incubator-ariatosca git commit: ARIA-324 Refactor ctx proxy access
Repository: incubator-ariatosca Updated Branches: refs/heads/master 51e4ed004 -> a43bc6b17 ARIA-324 Refactor ctx proxy access Our previous use of "." to delimit nested dict keys was wrong (keys could have a ".") and inflexible. The new implementation uses subsequent args to move into the dict. The same format can now be used to access object attributes. This commit also changes how to support setting values: we must now use "=" as the penultimate argument with the new value following. Calling functions now requires delimiting function arguments via "[" and "]" arguments. This makes it possible to continue accessing the return value of a function after the "]". Also fixed: callables will now "grab" the number of args they need instead of all remaining args, making it possible to do further inspection on the returned value from the callable. To allow for this, kwargs are now expected as the first arg rather than the last. Relatedly, this commit instruments all parameter fields from all models and fixes related bugs in the instrumentation implementation. Furthmore, this commit fixes a bad null check in the ctx client, and also allows it to retrieve Unicode data. Project: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/commit/a43bc6b1 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/tree/a43bc6b1 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/diff/a43bc6b1 Branch: refs/heads/master Commit: a43bc6b1706398dd79eabb2f49b6dafcadc8dfde Parents: 51e4ed0 Author: Tal Liron Authored: Thu Jul 27 17:58:17 2017 -0500 Committer: Tal Liron Committed: Tue Aug 8 09:32:15 2017 -0500 -- .travis.yml | 2 +- aria/orchestrator/context/common.py | 19 +- aria/orchestrator/context/operation.py | 46 ++--- .../execution_plugin/ctx_proxy/client.py| 7 +- .../execution_plugin/ctx_proxy/server.py| 196 +-- aria/orchestrator/workflows/api/task.py | 13 +- aria/storage/collection_instrumentation.py | 88 - aria/storage/core.py| 6 +- examples/hello-world/scripts/configure.sh | 29 ++- examples/hello-world/scripts/start.sh | 38 ++-- examples/hello-world/scripts/stop.sh| 17 +- tests/end2end/testenv.py| 24 ++- .../execution_plugin/test_ctx_proxy_server.py | 150 -- .../orchestrator/execution_plugin/test_local.py | 94 - tests/requirements.txt | 6 +- tests/resources/scripts/test_ssh.sh | 46 ++--- tox.ini | 8 +- 17 files changed, 358 insertions(+), 431 deletions(-) -- http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/a43bc6b1/.travis.yml -- diff --git a/.travis.yml b/.travis.yml index edd12d4..c8b7645 100644 --- a/.travis.yml +++ b/.travis.yml @@ -15,7 +15,7 @@ sudo: false language: python python: - - "2.7" + - '2.7' env: - TOX_ENV=pylint_code http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/a43bc6b1/aria/orchestrator/context/common.py -- diff --git a/aria/orchestrator/context/common.py b/aria/orchestrator/context/common.py index f400142..3c5f618 100644 --- a/aria/orchestrator/context/common.py +++ b/aria/orchestrator/context/common.py @@ -38,10 +38,27 @@ class BaseContext(object): """ INSTRUMENTATION_FIELDS = ( +modeling.models.Service.inputs, +modeling.models.ServiceTemplate.inputs, +modeling.models.Policy.properties, +modeling.models.PolicyTemplate.properties, modeling.models.Node.attributes, modeling.models.Node.properties, modeling.models.NodeTemplate.attributes, -modeling.models.NodeTemplate.properties +modeling.models.NodeTemplate.properties, +modeling.models.Group.properties, +modeling.models.GroupTemplate.properties, +modeling.models.Capability.properties, +# TODO ARIA-279: modeling.models.Capability.attributes, +modeling.models.CapabilityTemplate.properties, +# TODO ARIA-279: modeling.models.CapabilityTemplate.attributes +modeling.models.Relationship.properties, +modeling.models.Artifact.properties, +modeling.models.ArtifactTemplate.properties, +modeling.models.Interface.inputs, +modeling.models.InterfaceTemplate.inputs, +modeling.models.Operation.inputs, +modeling.models.OperationTemplate.inputs ) class PrefixedLogger(object): http://git-wip-us.apache.org/repos/asf/incubator-ariato
incubator-ariatosca git commit: ARIA-324 Refactor ctx proxy access [Forced Update!]
Repository: incubator-ariatosca Updated Branches: refs/heads/ARIA-324-refactor-ctx-access 8a3f8b735 -> a43bc6b17 (forced update) ARIA-324 Refactor ctx proxy access Our previous use of "." to delimit nested dict keys was wrong (keys could have a ".") and inflexible. The new implementation uses subsequent args to move into the dict. The same format can now be used to access object attributes. This commit also changes how to support setting values: we must now use "=" as the penultimate argument with the new value following. Calling functions now requires delimiting function arguments via "[" and "]" arguments. This makes it possible to continue accessing the return value of a function after the "]". Also fixed: callables will now "grab" the number of args they need instead of all remaining args, making it possible to do further inspection on the returned value from the callable. To allow for this, kwargs are now expected as the first arg rather than the last. Relatedly, this commit instruments all parameter fields from all models and fixes related bugs in the instrumentation implementation. Furthmore, this commit fixes a bad null check in the ctx client, and also allows it to retrieve Unicode data. Project: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/commit/a43bc6b1 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/tree/a43bc6b1 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/diff/a43bc6b1 Branch: refs/heads/ARIA-324-refactor-ctx-access Commit: a43bc6b1706398dd79eabb2f49b6dafcadc8dfde Parents: 51e4ed0 Author: Tal Liron Authored: Thu Jul 27 17:58:17 2017 -0500 Committer: Tal Liron Committed: Tue Aug 8 09:32:15 2017 -0500 -- .travis.yml | 2 +- aria/orchestrator/context/common.py | 19 +- aria/orchestrator/context/operation.py | 46 ++--- .../execution_plugin/ctx_proxy/client.py| 7 +- .../execution_plugin/ctx_proxy/server.py| 196 +-- aria/orchestrator/workflows/api/task.py | 13 +- aria/storage/collection_instrumentation.py | 88 - aria/storage/core.py| 6 +- examples/hello-world/scripts/configure.sh | 29 ++- examples/hello-world/scripts/start.sh | 38 ++-- examples/hello-world/scripts/stop.sh| 17 +- tests/end2end/testenv.py| 24 ++- .../execution_plugin/test_ctx_proxy_server.py | 150 -- .../orchestrator/execution_plugin/test_local.py | 94 - tests/requirements.txt | 6 +- tests/resources/scripts/test_ssh.sh | 46 ++--- tox.ini | 8 +- 17 files changed, 358 insertions(+), 431 deletions(-) -- http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/a43bc6b1/.travis.yml -- diff --git a/.travis.yml b/.travis.yml index edd12d4..c8b7645 100644 --- a/.travis.yml +++ b/.travis.yml @@ -15,7 +15,7 @@ sudo: false language: python python: - - "2.7" + - '2.7' env: - TOX_ENV=pylint_code http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/a43bc6b1/aria/orchestrator/context/common.py -- diff --git a/aria/orchestrator/context/common.py b/aria/orchestrator/context/common.py index f400142..3c5f618 100644 --- a/aria/orchestrator/context/common.py +++ b/aria/orchestrator/context/common.py @@ -38,10 +38,27 @@ class BaseContext(object): """ INSTRUMENTATION_FIELDS = ( +modeling.models.Service.inputs, +modeling.models.ServiceTemplate.inputs, +modeling.models.Policy.properties, +modeling.models.PolicyTemplate.properties, modeling.models.Node.attributes, modeling.models.Node.properties, modeling.models.NodeTemplate.attributes, -modeling.models.NodeTemplate.properties +modeling.models.NodeTemplate.properties, +modeling.models.Group.properties, +modeling.models.GroupTemplate.properties, +modeling.models.Capability.properties, +# TODO ARIA-279: modeling.models.Capability.attributes, +modeling.models.CapabilityTemplate.properties, +# TODO ARIA-279: modeling.models.CapabilityTemplate.attributes +modeling.models.Relationship.properties, +modeling.models.Artifact.properties, +modeling.models.ArtifactTemplate.properties, +modeling.models.Interface.inputs, +modeling.models.InterfaceTemplate.inputs, +modeling.models.Operation.inputs, +modeling.models.OperationTemplate.inputs ) class PrefixedLogger(objec
incubator-ariatosca git commit: ARIA-324 Refactor ctx proxy access [Forced Update!]
Repository: incubator-ariatosca Updated Branches: refs/heads/ARIA-324-refactor-ctx-access 1baaedf5c -> 8a3f8b735 (forced update) ARIA-324 Refactor ctx proxy access Our previous use of "." to delimit nested dict keys was wrong (keys could have a ".") and inflexible. The new implementation uses subsequent args to move into the dict. The same format can now be used to access object attributes. This commit also changes how to support setting values: we must now use "=" as the penultimate argument with the new value following. Calling functions now requires delimiting function arguments via "[" and "]" arguments. This makes it possible to continue accessing the return value of a function after the "]". Also fixed: callables will now "grab" the number of args they need instead of all remaining args, making it possible to do further inspection on the returned value from the callable. To allow for this, kwargs are now expected as the first arg rather than the last. Relatedly, this commit instruments all parameter fields from all models and fixes related bugs in the instrumentation implementation. Furthmore, this commit fixes a bad null check in the ctx client, and also allows it to retrieve Unicode data. Project: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/commit/8a3f8b73 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/tree/8a3f8b73 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/diff/8a3f8b73 Branch: refs/heads/ARIA-324-refactor-ctx-access Commit: 8a3f8b73560350b40d6c2ff47d05f589b0be Parents: 51e4ed0 Author: Tal Liron Authored: Thu Jul 27 17:58:17 2017 -0500 Committer: Tal Liron Committed: Tue Aug 8 08:52:16 2017 -0500 -- .travis.yml | 2 +- aria/orchestrator/context/common.py | 19 +- aria/orchestrator/context/operation.py | 46 ++--- .../execution_plugin/ctx_proxy/client.py| 7 +- .../execution_plugin/ctx_proxy/server.py| 196 +-- aria/orchestrator/workflows/api/task.py | 13 +- aria/storage/collection_instrumentation.py | 88 - aria/storage/core.py| 6 +- examples/hello-world/scripts/configure.sh | 29 ++- examples/hello-world/scripts/start.sh | 38 ++-- examples/hello-world/scripts/stop.sh| 17 +- tests/end2end/testenv.py| 24 ++- .../execution_plugin/test_ctx_proxy_server.py | 150 -- .../orchestrator/execution_plugin/test_local.py | 94 - tests/requirements.txt | 6 +- tests/resources/scripts/test_ssh.sh | 46 ++--- tox.ini | 8 +- 17 files changed, 358 insertions(+), 431 deletions(-) -- http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/8a3f8b73/.travis.yml -- diff --git a/.travis.yml b/.travis.yml index edd12d4..c8b7645 100644 --- a/.travis.yml +++ b/.travis.yml @@ -15,7 +15,7 @@ sudo: false language: python python: - - "2.7" + - '2.7' env: - TOX_ENV=pylint_code http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/8a3f8b73/aria/orchestrator/context/common.py -- diff --git a/aria/orchestrator/context/common.py b/aria/orchestrator/context/common.py index f400142..3c5f618 100644 --- a/aria/orchestrator/context/common.py +++ b/aria/orchestrator/context/common.py @@ -38,10 +38,27 @@ class BaseContext(object): """ INSTRUMENTATION_FIELDS = ( +modeling.models.Service.inputs, +modeling.models.ServiceTemplate.inputs, +modeling.models.Policy.properties, +modeling.models.PolicyTemplate.properties, modeling.models.Node.attributes, modeling.models.Node.properties, modeling.models.NodeTemplate.attributes, -modeling.models.NodeTemplate.properties +modeling.models.NodeTemplate.properties, +modeling.models.Group.properties, +modeling.models.GroupTemplate.properties, +modeling.models.Capability.properties, +# TODO ARIA-279: modeling.models.Capability.attributes, +modeling.models.CapabilityTemplate.properties, +# TODO ARIA-279: modeling.models.CapabilityTemplate.attributes +modeling.models.Relationship.properties, +modeling.models.Artifact.properties, +modeling.models.ArtifactTemplate.properties, +modeling.models.Interface.inputs, +modeling.models.InterfaceTemplate.inputs, +modeling.models.Operation.inputs, +modeling.models.OperationTemplate.inputs ) class PrefixedLogger(objec
[2/3] incubator-ariatosca git commit: ARIA-324 Refactor ctx proxy access
ARIA-324 Refactor ctx proxy access Our previous use of "." to delimit nested dict keys was wrong (keys could have a ".") and inflexible. The new implementation uses subsequent args to move into the dict. The same format can now be used to access object attributes. This commit also changes how to support setting values: we must now use "=" as the penultimate argument with the new value following. Also fixed: callables will now "grab" the number of args they need instead of all remaining args, making it possible to do further inspection on the returned value from the callable. To allow for this, kwargs are now expected as the first arg rather than the last. Relatedly, this commit instruments all parameter fields from all models and fixes related bugs in the instrumentation implementation. Furthmore, this commit fixes a bad null check in the ctx client, and also allows it to retrieve Unicode data. Project: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/commit/12516271 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/tree/12516271 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/diff/12516271 Branch: refs/heads/ARIA-324-refactor-ctx-access Commit: 12516271e9aa497ecd6e61d1087e08677066d8dd Parents: 51e4ed0 Author: Tal Liron Authored: Thu Jul 27 17:58:17 2017 -0500 Committer: Tal Liron Committed: Wed Aug 2 15:58:09 2017 -0500 -- aria/orchestrator/context/common.py | 19 ++- aria/orchestrator/context/operation.py | 46 +++-- .../execution_plugin/ctx_proxy/client.py| 7 +- .../execution_plugin/ctx_proxy/server.py| 169 ++- aria/orchestrator/workflows/api/task.py | 13 +- aria/storage/collection_instrumentation.py | 88 +- aria/storage/core.py| 6 +- examples/hello-world/scripts/configure.sh | 6 +- examples/hello-world/scripts/start.sh | 13 +- examples/hello-world/scripts/stop.sh| 4 +- tests/end2end/testenv.py| 24 ++- .../execution_plugin/test_ctx_proxy_server.py | 111 +++- .../orchestrator/execution_plugin/test_local.py | 34 ++-- tests/resources/scripts/test_ssh.sh | 30 ++-- 14 files changed, 274 insertions(+), 296 deletions(-) -- http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/12516271/aria/orchestrator/context/common.py -- diff --git a/aria/orchestrator/context/common.py b/aria/orchestrator/context/common.py index f400142..3c5f618 100644 --- a/aria/orchestrator/context/common.py +++ b/aria/orchestrator/context/common.py @@ -38,10 +38,27 @@ class BaseContext(object): """ INSTRUMENTATION_FIELDS = ( +modeling.models.Service.inputs, +modeling.models.ServiceTemplate.inputs, +modeling.models.Policy.properties, +modeling.models.PolicyTemplate.properties, modeling.models.Node.attributes, modeling.models.Node.properties, modeling.models.NodeTemplate.attributes, -modeling.models.NodeTemplate.properties +modeling.models.NodeTemplate.properties, +modeling.models.Group.properties, +modeling.models.GroupTemplate.properties, +modeling.models.Capability.properties, +# TODO ARIA-279: modeling.models.Capability.attributes, +modeling.models.CapabilityTemplate.properties, +# TODO ARIA-279: modeling.models.CapabilityTemplate.attributes +modeling.models.Relationship.properties, +modeling.models.Artifact.properties, +modeling.models.ArtifactTemplate.properties, +modeling.models.Interface.inputs, +modeling.models.InterfaceTemplate.inputs, +modeling.models.Operation.inputs, +modeling.models.OperationTemplate.inputs ) class PrefixedLogger(object): http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/12516271/aria/orchestrator/context/operation.py -- diff --git a/aria/orchestrator/context/operation.py b/aria/orchestrator/context/operation.py index 7d5f40c..8613ec3 100644 --- a/aria/orchestrator/context/operation.py +++ b/aria/orchestrator/context/operation.py @@ -48,8 +48,7 @@ class BaseOperationContext(common.BaseContext): @property def task(self): """ -The task in the model storage -:return: Task model +The task in the model storage. """ # SQLAlchemy prevents from accessing an object which was created on a different thread. # So we retrieve the object from the storage if the current thread isn't the same as the @@ -62,7 +61,7 @@ class BaseOperation
incubator-ariatosca git commit: ARIA-324 Refactor ctx proxy access [Forced Update!]
Repository: incubator-ariatosca Updated Branches: refs/heads/ARIA-324-refactor-ctx-access 8d61120f2 -> 3f5a2715f (forced update) ARIA-324 Refactor ctx proxy access Our previous use of "." to delimit nested dict keys was wrong (keys could have a ".") and inflexible. The new implementation uses subsequent args to move into the dict. The same format can now be used to access object attributes. This commit also changes how to support setting values: we must now use "=" as the penultimate argument with the new value following. Also fixed: callables will now "grab" the number of args they need instead of all remaining args, making it possible to do further inspection on the returned value from the callable. To allow for this, kwargs are now expected as the first arg rather than the last. Relatedly, this commit instruments all parameter fields from all models and fixes related bugs in the instrumentation implementation. Furthmore, this commit fixes a bad null check in the ctx client, and also allows it to retrieve Unicode data. Project: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/commit/3f5a2715 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/tree/3f5a2715 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/diff/3f5a2715 Branch: refs/heads/ARIA-324-refactor-ctx-access Commit: 3f5a2715f643b0d341884dd37b60fc935adbc897 Parents: c2b8e65 Author: Tal Liron Authored: Thu Jul 27 17:58:17 2017 -0500 Committer: Tal Liron Committed: Fri Jul 28 16:53:10 2017 -0500 -- aria/orchestrator/context/common.py | 19 ++- aria/orchestrator/context/operation.py | 46 +++-- .../execution_plugin/ctx_proxy/client.py| 7 +- .../execution_plugin/ctx_proxy/server.py| 169 ++- aria/orchestrator/workflows/api/task.py | 13 +- aria/storage/collection_instrumentation.py | 88 +- aria/storage/core.py| 6 +- examples/hello-world/scripts/configure.sh | 6 +- examples/hello-world/scripts/start.sh | 13 +- examples/hello-world/scripts/stop.sh| 4 +- tests/end2end/testenv.py| 24 ++- .../execution_plugin/test_ctx_proxy_server.py | 111 +++- .../orchestrator/execution_plugin/test_local.py | 34 ++-- tests/resources/scripts/test_ssh.sh | 30 ++-- 14 files changed, 274 insertions(+), 296 deletions(-) -- http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/3f5a2715/aria/orchestrator/context/common.py -- diff --git a/aria/orchestrator/context/common.py b/aria/orchestrator/context/common.py index f400142..3c5f618 100644 --- a/aria/orchestrator/context/common.py +++ b/aria/orchestrator/context/common.py @@ -38,10 +38,27 @@ class BaseContext(object): """ INSTRUMENTATION_FIELDS = ( +modeling.models.Service.inputs, +modeling.models.ServiceTemplate.inputs, +modeling.models.Policy.properties, +modeling.models.PolicyTemplate.properties, modeling.models.Node.attributes, modeling.models.Node.properties, modeling.models.NodeTemplate.attributes, -modeling.models.NodeTemplate.properties +modeling.models.NodeTemplate.properties, +modeling.models.Group.properties, +modeling.models.GroupTemplate.properties, +modeling.models.Capability.properties, +# TODO ARIA-279: modeling.models.Capability.attributes, +modeling.models.CapabilityTemplate.properties, +# TODO ARIA-279: modeling.models.CapabilityTemplate.attributes +modeling.models.Relationship.properties, +modeling.models.Artifact.properties, +modeling.models.ArtifactTemplate.properties, +modeling.models.Interface.inputs, +modeling.models.InterfaceTemplate.inputs, +modeling.models.Operation.inputs, +modeling.models.OperationTemplate.inputs ) class PrefixedLogger(object): http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/3f5a2715/aria/orchestrator/context/operation.py -- diff --git a/aria/orchestrator/context/operation.py b/aria/orchestrator/context/operation.py index 7d5f40c..8613ec3 100644 --- a/aria/orchestrator/context/operation.py +++ b/aria/orchestrator/context/operation.py @@ -48,8 +48,7 @@ class BaseOperationContext(common.BaseContext): @property def task(self): """ -The task in the model storage -:return: Task model +The task in the model storage. """ # SQLAlchemy prevents from accessing an object which was created on a different thread.
incubator-ariatosca git commit: ARIA-324 Refactor ctx proxy access [Forced Update!]
Repository: incubator-ariatosca Updated Branches: refs/heads/ARIA-324-refactor-ctx-access 463cae7c0 -> 8d61120f2 (forced update) ARIA-324 Refactor ctx proxy access Our previous use of "." to delimit nested dict keys was wrong (keys could have a ".") and inflexible. The new implementation uses subsequent args to move into the dict. The same format can now be used to access object attributes. This commit also changes how to support setting values: we must now use "=" as the penultimate argument with the new value following. Also fixed: callables will now "grab" the number of args they need instead of all remaining args, making it possible to do further inspection on the returned value from the callable. To allow for this, kwargs are now expected as the first arg rather than the last. Relatedly, this commit instruments all parameter fields from all models and fixes related bugs in the instrumentation implementation. Furthmore, this commit fixes a bad null check in the ctx client, and also allows it to retrieve Unicode data. Project: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/commit/8d61120f Tree: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/tree/8d61120f Diff: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/diff/8d61120f Branch: refs/heads/ARIA-324-refactor-ctx-access Commit: 8d61120f2f37e5b57bf8bc4d902e5cf24c62dcf6 Parents: c2b8e65 Author: Tal Liron Authored: Thu Jul 27 17:58:17 2017 -0500 Committer: Tal Liron Committed: Fri Jul 28 15:53:43 2017 -0500 -- aria/orchestrator/context/common.py | 19 ++- aria/orchestrator/context/operation.py | 46 +++--- .../execution_plugin/ctx_proxy/client.py| 7 +- .../execution_plugin/ctx_proxy/server.py| 153 +-- aria/orchestrator/workflows/api/task.py | 13 +- aria/storage/collection_instrumentation.py | 88 +-- aria/storage/core.py| 6 +- examples/hello-world/scripts/configure.sh | 6 +- examples/hello-world/scripts/start.sh | 13 +- examples/hello-world/scripts/stop.sh| 4 +- tests/end2end/testenv.py| 24 +-- .../execution_plugin/test_ctx_proxy_server.py | 111 -- .../orchestrator/execution_plugin/test_local.py | 34 ++--- tests/resources/scripts/test_ssh.sh | 30 ++-- 14 files changed, 258 insertions(+), 296 deletions(-) -- http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/8d61120f/aria/orchestrator/context/common.py -- diff --git a/aria/orchestrator/context/common.py b/aria/orchestrator/context/common.py index f400142..3c5f618 100644 --- a/aria/orchestrator/context/common.py +++ b/aria/orchestrator/context/common.py @@ -38,10 +38,27 @@ class BaseContext(object): """ INSTRUMENTATION_FIELDS = ( +modeling.models.Service.inputs, +modeling.models.ServiceTemplate.inputs, +modeling.models.Policy.properties, +modeling.models.PolicyTemplate.properties, modeling.models.Node.attributes, modeling.models.Node.properties, modeling.models.NodeTemplate.attributes, -modeling.models.NodeTemplate.properties +modeling.models.NodeTemplate.properties, +modeling.models.Group.properties, +modeling.models.GroupTemplate.properties, +modeling.models.Capability.properties, +# TODO ARIA-279: modeling.models.Capability.attributes, +modeling.models.CapabilityTemplate.properties, +# TODO ARIA-279: modeling.models.CapabilityTemplate.attributes +modeling.models.Relationship.properties, +modeling.models.Artifact.properties, +modeling.models.ArtifactTemplate.properties, +modeling.models.Interface.inputs, +modeling.models.InterfaceTemplate.inputs, +modeling.models.Operation.inputs, +modeling.models.OperationTemplate.inputs ) class PrefixedLogger(object): http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/8d61120f/aria/orchestrator/context/operation.py -- diff --git a/aria/orchestrator/context/operation.py b/aria/orchestrator/context/operation.py index 7d5f40c..8613ec3 100644 --- a/aria/orchestrator/context/operation.py +++ b/aria/orchestrator/context/operation.py @@ -48,8 +48,7 @@ class BaseOperationContext(common.BaseContext): @property def task(self): """ -The task in the model storage -:return: Task model +The task in the model storage. """ # SQLAlchemy prevents from accessing an object which was created on a different threa
incubator-ariatosca git commit: ARIA-324 Refactor ctx proxy access [Forced Update!]
Repository: incubator-ariatosca Updated Branches: refs/heads/ARIA-324-refactor-ctx-access ddb85e240 -> 463cae7c0 (forced update) ARIA-324 Refactor ctx proxy access Our previous use of "." to delimit nested dict keys was wrong (keys could have a ".") and inflexible. The new implementation uses subsequent args to move into the dict. The same format can now be used to access object attributes. This commit also changes how to support setting values: we must now use "=" as the penultimate argument with the new value following. Also fixed: callables will now "grab" the number of args they need instead of all remaining args, making it possible to do further inspection on the returned value from the callable. To allow for this, kwargs are now expected as the first arg rather than the last. Relatedly, this commit instruments all parameter fields from all models and fixes related bugs in the instrumentation implementation. Furthmore, this commit fixes a bad null check in the ctx client, and also allows it to retrieve Unicode data. Project: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/commit/463cae7c Tree: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/tree/463cae7c Diff: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/diff/463cae7c Branch: refs/heads/ARIA-324-refactor-ctx-access Commit: 463cae7c091d05f1d7213706028259773b884fa7 Parents: c2b8e65 Author: Tal Liron Authored: Thu Jul 27 17:58:17 2017 -0500 Committer: Tal Liron Committed: Fri Jul 28 13:06:01 2017 -0500 -- aria/orchestrator/context/common.py | 19 ++- aria/orchestrator/context/operation.py | 46 +++--- .../execution_plugin/ctx_proxy/client.py| 7 +- .../execution_plugin/ctx_proxy/server.py| 149 +-- aria/orchestrator/workflows/api/task.py | 13 +- aria/storage/collection_instrumentation.py | 88 +-- aria/storage/core.py| 6 +- .../execution_plugin/test_ctx_proxy_server.py | 111 -- .../orchestrator/execution_plugin/test_local.py | 34 ++--- tests/resources/scripts/test_ssh.sh | 30 ++-- 10 files changed, 228 insertions(+), 275 deletions(-) -- http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/463cae7c/aria/orchestrator/context/common.py -- diff --git a/aria/orchestrator/context/common.py b/aria/orchestrator/context/common.py index f400142..3c5f618 100644 --- a/aria/orchestrator/context/common.py +++ b/aria/orchestrator/context/common.py @@ -38,10 +38,27 @@ class BaseContext(object): """ INSTRUMENTATION_FIELDS = ( +modeling.models.Service.inputs, +modeling.models.ServiceTemplate.inputs, +modeling.models.Policy.properties, +modeling.models.PolicyTemplate.properties, modeling.models.Node.attributes, modeling.models.Node.properties, modeling.models.NodeTemplate.attributes, -modeling.models.NodeTemplate.properties +modeling.models.NodeTemplate.properties, +modeling.models.Group.properties, +modeling.models.GroupTemplate.properties, +modeling.models.Capability.properties, +# TODO ARIA-279: modeling.models.Capability.attributes, +modeling.models.CapabilityTemplate.properties, +# TODO ARIA-279: modeling.models.CapabilityTemplate.attributes +modeling.models.Relationship.properties, +modeling.models.Artifact.properties, +modeling.models.ArtifactTemplate.properties, +modeling.models.Interface.inputs, +modeling.models.InterfaceTemplate.inputs, +modeling.models.Operation.inputs, +modeling.models.OperationTemplate.inputs ) class PrefixedLogger(object): http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/463cae7c/aria/orchestrator/context/operation.py -- diff --git a/aria/orchestrator/context/operation.py b/aria/orchestrator/context/operation.py index 7d5f40c..8613ec3 100644 --- a/aria/orchestrator/context/operation.py +++ b/aria/orchestrator/context/operation.py @@ -48,8 +48,7 @@ class BaseOperationContext(common.BaseContext): @property def task(self): """ -The task in the model storage -:return: Task model +The task in the model storage. """ # SQLAlchemy prevents from accessing an object which was created on a different thread. # So we retrieve the object from the storage if the current thread isn't the same as the @@ -62,7 +61,7 @@ class BaseOperationContext(common.BaseContext): @property def plugin_workdir(self): """ -
[1/3] incubator-ariatosca git commit: ARIA-324 Refactor ctx proxy access [Forced Update!]
Repository: incubator-ariatosca Updated Branches: refs/heads/ARIA-321-clearwater 80bae440b -> a638acdf1 (forced update) ARIA-324 Refactor ctx proxy access Our previous use of "." to delimit nested dict keys was wrong (keys could have a ".") and inflexible. The new implementation uses subsequent args to move into the dict. The same format can now be used to access object attributes. This commit also changes how to support setting values: we must now use "=" as the penultimate argument with the new value following. Also fixed: callables will now "grab" the number of args they need instead of all remaining args, making it possible to do further inspection on the returned value from the callable. To allow for this, kwargs are now expected as the first arg rather than the last. Relatedly, this commit instruments all parameter fields from all models and fixes related bugs in the instrumentation implementation. Furthmore, this commit fixes a bad null check in the ctx client, and also allows it to retrieve Unicode data. Project: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/commit/ddb85e24 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/tree/ddb85e24 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/diff/ddb85e24 Branch: refs/heads/ARIA-321-clearwater Commit: ddb85e240de400ef81b140240f8bc2036cd24b80 Parents: c2b8e65 Author: Tal Liron Authored: Thu Jul 27 17:58:17 2017 -0500 Committer: Tal Liron Committed: Thu Jul 27 21:50:50 2017 -0500 -- aria/orchestrator/context/common.py | 19 ++- aria/orchestrator/context/operation.py | 46 +++--- .../execution_plugin/ctx_proxy/client.py| 7 +- .../execution_plugin/ctx_proxy/server.py| 144 +-- aria/orchestrator/workflows/api/task.py | 13 +- aria/storage/collection_instrumentation.py | 88 ++-- aria/storage/core.py| 6 +- .../execution_plugin/test_ctx_proxy_server.py | 96 ++--- .../orchestrator/execution_plugin/test_local.py | 34 ++--- 9 files changed, 193 insertions(+), 260 deletions(-) -- http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/ddb85e24/aria/orchestrator/context/common.py -- diff --git a/aria/orchestrator/context/common.py b/aria/orchestrator/context/common.py index f400142..3c5f618 100644 --- a/aria/orchestrator/context/common.py +++ b/aria/orchestrator/context/common.py @@ -38,10 +38,27 @@ class BaseContext(object): """ INSTRUMENTATION_FIELDS = ( +modeling.models.Service.inputs, +modeling.models.ServiceTemplate.inputs, +modeling.models.Policy.properties, +modeling.models.PolicyTemplate.properties, modeling.models.Node.attributes, modeling.models.Node.properties, modeling.models.NodeTemplate.attributes, -modeling.models.NodeTemplate.properties +modeling.models.NodeTemplate.properties, +modeling.models.Group.properties, +modeling.models.GroupTemplate.properties, +modeling.models.Capability.properties, +# TODO ARIA-279: modeling.models.Capability.attributes, +modeling.models.CapabilityTemplate.properties, +# TODO ARIA-279: modeling.models.CapabilityTemplate.attributes +modeling.models.Relationship.properties, +modeling.models.Artifact.properties, +modeling.models.ArtifactTemplate.properties, +modeling.models.Interface.inputs, +modeling.models.InterfaceTemplate.inputs, +modeling.models.Operation.inputs, +modeling.models.OperationTemplate.inputs ) class PrefixedLogger(object): http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/ddb85e24/aria/orchestrator/context/operation.py -- diff --git a/aria/orchestrator/context/operation.py b/aria/orchestrator/context/operation.py index 7d5f40c..8613ec3 100644 --- a/aria/orchestrator/context/operation.py +++ b/aria/orchestrator/context/operation.py @@ -48,8 +48,7 @@ class BaseOperationContext(common.BaseContext): @property def task(self): """ -The task in the model storage -:return: Task model +The task in the model storage. """ # SQLAlchemy prevents from accessing an object which was created on a different thread. # So we retrieve the object from the storage if the current thread isn't the same as the @@ -62,7 +61,7 @@ class BaseOperationContext(common.BaseContext): @property def plugin_workdir(self): """ -A work directory that is unique to the plugin and the deployment id +
incubator-ariatosca git commit: ARIA-324 Refactor ctx proxy access [Forced Update!]
Repository: incubator-ariatosca Updated Branches: refs/heads/ARIA-324-refactor-ctx-access c4ee51772 -> ddb85e240 (forced update) ARIA-324 Refactor ctx proxy access Our previous use of "." to delimit nested dict keys was wrong (keys could have a ".") and inflexible. The new implementation uses subsequent args to move into the dict. The same format can now be used to access object attributes. This commit also changes how to support setting values: we must now use "=" as the penultimate argument with the new value following. Also fixed: callables will now "grab" the number of args they need instead of all remaining args, making it possible to do further inspection on the returned value from the callable. To allow for this, kwargs are now expected as the first arg rather than the last. Relatedly, this commit instruments all parameter fields from all models and fixes related bugs in the instrumentation implementation. Furthmore, this commit fixes a bad null check in the ctx client, and also allows it to retrieve Unicode data. Project: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/commit/ddb85e24 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/tree/ddb85e24 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/diff/ddb85e24 Branch: refs/heads/ARIA-324-refactor-ctx-access Commit: ddb85e240de400ef81b140240f8bc2036cd24b80 Parents: c2b8e65 Author: Tal Liron Authored: Thu Jul 27 17:58:17 2017 -0500 Committer: Tal Liron Committed: Thu Jul 27 21:50:50 2017 -0500 -- aria/orchestrator/context/common.py | 19 ++- aria/orchestrator/context/operation.py | 46 +++--- .../execution_plugin/ctx_proxy/client.py| 7 +- .../execution_plugin/ctx_proxy/server.py| 144 +-- aria/orchestrator/workflows/api/task.py | 13 +- aria/storage/collection_instrumentation.py | 88 ++-- aria/storage/core.py| 6 +- .../execution_plugin/test_ctx_proxy_server.py | 96 ++--- .../orchestrator/execution_plugin/test_local.py | 34 ++--- 9 files changed, 193 insertions(+), 260 deletions(-) -- http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/ddb85e24/aria/orchestrator/context/common.py -- diff --git a/aria/orchestrator/context/common.py b/aria/orchestrator/context/common.py index f400142..3c5f618 100644 --- a/aria/orchestrator/context/common.py +++ b/aria/orchestrator/context/common.py @@ -38,10 +38,27 @@ class BaseContext(object): """ INSTRUMENTATION_FIELDS = ( +modeling.models.Service.inputs, +modeling.models.ServiceTemplate.inputs, +modeling.models.Policy.properties, +modeling.models.PolicyTemplate.properties, modeling.models.Node.attributes, modeling.models.Node.properties, modeling.models.NodeTemplate.attributes, -modeling.models.NodeTemplate.properties +modeling.models.NodeTemplate.properties, +modeling.models.Group.properties, +modeling.models.GroupTemplate.properties, +modeling.models.Capability.properties, +# TODO ARIA-279: modeling.models.Capability.attributes, +modeling.models.CapabilityTemplate.properties, +# TODO ARIA-279: modeling.models.CapabilityTemplate.attributes +modeling.models.Relationship.properties, +modeling.models.Artifact.properties, +modeling.models.ArtifactTemplate.properties, +modeling.models.Interface.inputs, +modeling.models.InterfaceTemplate.inputs, +modeling.models.Operation.inputs, +modeling.models.OperationTemplate.inputs ) class PrefixedLogger(object): http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/ddb85e24/aria/orchestrator/context/operation.py -- diff --git a/aria/orchestrator/context/operation.py b/aria/orchestrator/context/operation.py index 7d5f40c..8613ec3 100644 --- a/aria/orchestrator/context/operation.py +++ b/aria/orchestrator/context/operation.py @@ -48,8 +48,7 @@ class BaseOperationContext(common.BaseContext): @property def task(self): """ -The task in the model storage -:return: Task model +The task in the model storage. """ # SQLAlchemy prevents from accessing an object which was created on a different thread. # So we retrieve the object from the storage if the current thread isn't the same as the @@ -62,7 +61,7 @@ class BaseOperationContext(common.BaseContext): @property def plugin_workdir(self): """ -A work directory that is unique to the plugin and the depl
incubator-ariatosca git commit: ARIA-324 Refactor ctx proxy access [Forced Update!]
Repository: incubator-ariatosca Updated Branches: refs/heads/ARIA-324-refactor-ctx-access 956e98b20 -> c4ee51772 (forced update) ARIA-324 Refactor ctx proxy access Our previous use of "." to delimit nested dict keys was wrong (keys could have a ".") and inflexible. The new implementation uses subsequent args to move into the dict. The same format can now be used to access object attributes. This commit also changes how to support setting values: we must now use "=" as the penultimate argument with the new value following. Also fixed: callables will now "grab" the number of args they need instead of all remaining args, making it possible to do further inspection on the returned value from the callable. To allow for this, kwargs are now expected as the first arg rather than the last. Relatedly, this commit instruments all parameter fields from all models and fixes related bugs in the instrumentation implementation. Furthmore, this commit fixes a bad null check in the ctx client, and also allows it to retrieve Unicode data. Project: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/commit/c4ee5177 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/tree/c4ee5177 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/diff/c4ee5177 Branch: refs/heads/ARIA-324-refactor-ctx-access Commit: c4ee51772294b3355d135baae09769f4c188f62d Parents: c2b8e65 Author: Tal Liron Authored: Thu Jul 27 17:58:17 2017 -0500 Committer: Tal Liron Committed: Thu Jul 27 21:36:42 2017 -0500 -- aria/orchestrator/context/common.py | 19 ++- aria/orchestrator/context/operation.py | 46 +++--- .../execution_plugin/ctx_proxy/client.py| 7 +- .../execution_plugin/ctx_proxy/server.py| 145 +-- aria/orchestrator/workflows/api/task.py | 13 +- aria/storage/collection_instrumentation.py | 88 +-- aria/storage/core.py| 6 +- .../execution_plugin/test_ctx_proxy_server.py | 96 ++-- .../orchestrator/execution_plugin/test_local.py | 34 ++--- 9 files changed, 194 insertions(+), 260 deletions(-) -- http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/c4ee5177/aria/orchestrator/context/common.py -- diff --git a/aria/orchestrator/context/common.py b/aria/orchestrator/context/common.py index f400142..3c5f618 100644 --- a/aria/orchestrator/context/common.py +++ b/aria/orchestrator/context/common.py @@ -38,10 +38,27 @@ class BaseContext(object): """ INSTRUMENTATION_FIELDS = ( +modeling.models.Service.inputs, +modeling.models.ServiceTemplate.inputs, +modeling.models.Policy.properties, +modeling.models.PolicyTemplate.properties, modeling.models.Node.attributes, modeling.models.Node.properties, modeling.models.NodeTemplate.attributes, -modeling.models.NodeTemplate.properties +modeling.models.NodeTemplate.properties, +modeling.models.Group.properties, +modeling.models.GroupTemplate.properties, +modeling.models.Capability.properties, +# TODO ARIA-279: modeling.models.Capability.attributes, +modeling.models.CapabilityTemplate.properties, +# TODO ARIA-279: modeling.models.CapabilityTemplate.attributes +modeling.models.Relationship.properties, +modeling.models.Artifact.properties, +modeling.models.ArtifactTemplate.properties, +modeling.models.Interface.inputs, +modeling.models.InterfaceTemplate.inputs, +modeling.models.Operation.inputs, +modeling.models.OperationTemplate.inputs ) class PrefixedLogger(object): http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/c4ee5177/aria/orchestrator/context/operation.py -- diff --git a/aria/orchestrator/context/operation.py b/aria/orchestrator/context/operation.py index 7d5f40c..8613ec3 100644 --- a/aria/orchestrator/context/operation.py +++ b/aria/orchestrator/context/operation.py @@ -48,8 +48,7 @@ class BaseOperationContext(common.BaseContext): @property def task(self): """ -The task in the model storage -:return: Task model +The task in the model storage. """ # SQLAlchemy prevents from accessing an object which was created on a different thread. # So we retrieve the object from the storage if the current thread isn't the same as the @@ -62,7 +61,7 @@ class BaseOperationContext(common.BaseContext): @property def plugin_workdir(self): """ -A work directory that is unique to the plugin and the deploy
incubator-ariatosca git commit: ARIA-324 Refactor ctx proxy access [Forced Update!]
Repository: incubator-ariatosca Updated Branches: refs/heads/ARIA-324-refactor-ctx-access 638a4346d -> 956e98b20 (forced update) ARIA-324 Refactor ctx proxy access Our previous use of "." to delimit nested dict keys was wrong (keys could have a ".") and inflexible. The new implementation uses subsequent args to move into the dict. The same format can now be used to access object attributes. This commit also changes how to support setting values: we must now use "=" as the penultimate argument with the new value following. Also fixed: callables will now "grab" the number of args they need instead of all remaining args, making it possible to do further inspection on the returned value from the callable. To allow for this, kwargs are now expected as the first arg rather than the last. Relatedly, this commit instruments all parameter fields from all models and fixes related bugs in the instrumentation implementation. Furthmore, this commit fixes a bad null check in the ctx client, and also allows it to retrieve Unicode data. Project: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/commit/956e98b2 Tree: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/tree/956e98b2 Diff: http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/diff/956e98b2 Branch: refs/heads/ARIA-324-refactor-ctx-access Commit: 956e98b20874b99e80b12849b3c10dc869a4b4f6 Parents: c2b8e65 Author: Tal Liron Authored: Thu Jul 27 17:58:17 2017 -0500 Committer: Tal Liron Committed: Thu Jul 27 21:18:20 2017 -0500 -- aria/orchestrator/context/common.py | 19 ++- aria/orchestrator/context/operation.py | 46 +++--- .../execution_plugin/ctx_proxy/client.py| 7 +- .../execution_plugin/ctx_proxy/server.py| 141 +-- aria/orchestrator/workflows/api/task.py | 13 +- aria/storage/collection_instrumentation.py | 88 ++-- aria/storage/core.py| 6 +- .../execution_plugin/test_ctx_proxy_server.py | 96 ++--- .../orchestrator/execution_plugin/test_local.py | 34 ++--- 9 files changed, 192 insertions(+), 258 deletions(-) -- http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/956e98b2/aria/orchestrator/context/common.py -- diff --git a/aria/orchestrator/context/common.py b/aria/orchestrator/context/common.py index f400142..3c5f618 100644 --- a/aria/orchestrator/context/common.py +++ b/aria/orchestrator/context/common.py @@ -38,10 +38,27 @@ class BaseContext(object): """ INSTRUMENTATION_FIELDS = ( +modeling.models.Service.inputs, +modeling.models.ServiceTemplate.inputs, +modeling.models.Policy.properties, +modeling.models.PolicyTemplate.properties, modeling.models.Node.attributes, modeling.models.Node.properties, modeling.models.NodeTemplate.attributes, -modeling.models.NodeTemplate.properties +modeling.models.NodeTemplate.properties, +modeling.models.Group.properties, +modeling.models.GroupTemplate.properties, +modeling.models.Capability.properties, +# TODO ARIA-279: modeling.models.Capability.attributes, +modeling.models.CapabilityTemplate.properties, +# TODO ARIA-279: modeling.models.CapabilityTemplate.attributes +modeling.models.Relationship.properties, +modeling.models.Artifact.properties, +modeling.models.ArtifactTemplate.properties, +modeling.models.Interface.inputs, +modeling.models.InterfaceTemplate.inputs, +modeling.models.Operation.inputs, +modeling.models.OperationTemplate.inputs ) class PrefixedLogger(object): http://git-wip-us.apache.org/repos/asf/incubator-ariatosca/blob/956e98b2/aria/orchestrator/context/operation.py -- diff --git a/aria/orchestrator/context/operation.py b/aria/orchestrator/context/operation.py index 7d5f40c..8613ec3 100644 --- a/aria/orchestrator/context/operation.py +++ b/aria/orchestrator/context/operation.py @@ -48,8 +48,7 @@ class BaseOperationContext(common.BaseContext): @property def task(self): """ -The task in the model storage -:return: Task model +The task in the model storage. """ # SQLAlchemy prevents from accessing an object which was created on a different thread. # So we retrieve the object from the storage if the current thread isn't the same as the @@ -62,7 +61,7 @@ class BaseOperationContext(common.BaseContext): @property def plugin_workdir(self): """ -A work directory that is unique to the plugin and the depl