tosca message
[Date Prev]
| [Thread Prev]
| [Thread Next]
| [Date Next]
--
[Date Index]
| [Thread Index]
| [List Home]
Subject: Re: 2015-04-13 tosca_elk_DRAFT_CSAR: issues with requirements
- From: Sahdev P Zala <spzala@us.ibm.com>
- To: Chris Lauwers <lauwers@ubicity.com>
- Date: Tue, 14 Apr 2015 11:17:59 -0400
Hi Chris,
Good catches. I will make updates per
your suggestion for Logstash and any other node where it makes sense.
I have a question on node types grammar
in spec for providing arguments to the 'relationship', interfaces in Logstash
case. I will bring it up in today's YAML call.
Thanks!
Regards,
Sahdev Zala
____________________________
development,
IBM Cloud
Durham, NC
(919)486-2915 T/L: 526-2915
From:
Chris Lauwers <lauwers@ubicity.com>
To:
Sahdev P Zala/Durham/IBM@IBMUS
Cc:
"tosca@lists.oasis-open.org"
<tosca@lists.oasis-open.org>
Date:
04/13/2015 11:25 PM
Subject:
2015-04-13 tosca_elk_DRAFT_CSAR:
issues with requirements
As promised, here are a number
of comments on the latest CSAR file. Most of the issues with this file
are with requirements definitions and assignments and are similar in nature.
I’ll use the logstash.yaml file to illustrate a couple of them.
The first set of issues has
to do with requirement definitions in node types. logstash.yaml defines
the following node type:
node_types:
tosca.nodes.SoftwareComponent.Logstash:
derived_from: tosca.nodes.SoftwareComponent
requirements:
search_endpoint:
capability: tosca.capabilities.Endpoint
node: tosca.nodes.SoftwareComponent.Elasticsearch
relationship:
type: tosca.relationships.ConnectsTo
interfaces:
tosca.interfaces.relationship.Configure:
pre_configure_source:
implementation:
type: string
inputs:
elasticsearch_ip:
type: string
This node type has invalid
grammar for the search_endpoint requirement definition. According to the
requirements definition grammar (section A.6.2.2) the ‘relationship’
keyname is only allowed to take a relationship type name, not an extended
grammar that overrides interfaces.
The second set of issues
have to do with requirement assignments. The logstash node template in
the tosca_elk.yaml file contains the following:
logstash:
type: tosca.nodes.SoftwareComponent.Logstash
requirements:
- host:
logstash_server
- search_endpoint:
elasticsearch
interfaces:
tosca.interfaces.relationship.Configure:
pre_configure_source:
implementation: Python/logstash/configure_elasticsearch.py
input:
elasticsearch_ip: { get_attribute: [elasticsearch_server,
ip_address] }
There are 2 issues with this:
1. The
syntax for ordered requirements assignment lists is invalid based on section
A.7.2.2.2. The correct syntax is as follows:
logstash:
type: tosca.nodes.SoftwareComponent.Logstash
requirements:
- host:
logstash_server
- search_endpoint:
node: elasticsearch
interfaces:
tosca.interfaces.relationship.Configure:
pre_configure_source:
implementation: Python/logstash/configure_elasticsearch.py
input:
elasticsearch_ip: { get_attribute: [elasticsearch_server,
ip_address] }
2. More
importantly, a requirement assignment does not take ‘interfaces’ as a
top-level keyname. I imagine the intent here is to override interfaces
for the relationship in the requirement as follows:
logstash:
type: tosca.nodes.SoftwareComponent.Logstash
requirements:
- host:
logstash_server
- search_endpoint:
node: elasticsearch
relationship:
interfaces:
tosca.interfaces.relationship.Configure:
pre_configure_source:
implementation: Python/logstash/configure_elasticsearch.py
input:
elasticsearch_ip: { get_attribute:
[elasticsearch_server, ip_address] }
Unfortunately, the latest
version of the grammar seems to have removed support for that as well so
it is no longer possible to define interfaces directly in requirements.
The only option here is to introduce a relationship template with the appropriate
interfaces.
Chris
From: Sahdev P Zala [mailto:spzala@us.ibm.com]
Sent: Monday, April 13, 2015 4:42 PM
To: Chris Lauwers
Cc: tosca@lists.oasis-open.org
Subject: RE: [tosca] Groups - 2015-04-13 tosca_elk_DRAFT_CSAR.zip uploaded
Hi Chris, sounds great!
Thanks!!
Regards,
Sahdev Zala
From: Chris
Lauwers <lauwers@ubicity.com>
To: Sahdev
P Zala/Durham/IBM@IBMUS
Cc: "tosca@lists.oasis-open.org"
<tosca@lists.oasis-open.org>
Date: 04/13/2015
07:17 PM
Subject: RE:
[tosca] Groups - 2015-04-13 tosca_elk_DRAFT_CSAR.zip uploaded
Sent by: <tosca@lists.oasis-open.org>
Thanks Sahdev, I’ll start with an email, since a number of the issues
will likely need some discussion. Based on the discussion, I can upload
edited YAML files.
I’ll work on this later today and tomorrow.
Thanks,
Chris
From: Sahdev P Zala [mailto:spzala@us.ibm.com]
Sent: Monday, April 13, 2015 4:15 PM
To: Chris Lauwers
Cc: tosca@lists.oasis-open.org
Subject: RE: [tosca] Groups - 2015-04-13 tosca_elk_DRAFT_CSAR.zip uploaded
Hi Chris,
Thank you so much for the review. I am looking forward to have your comments.
I think email probably is a good way or may be you can upload them as a
doc at the same place we have CSAR? I will make changes per your comments
unless I have questions or need to have a discussion.
Thanks again!
Regards,
Sahdev Zala
From: Chris
Lauwers <lauwers@ubicity.com>
To: Sahdev
P Zala/Durham/IBM@IBMUS, "tosca@lists.oasis-open.org"
<tosca@lists.oasis-open.org>
Date: 04/13/2015
04:57 PM
Subject: RE:
[tosca] Groups - 2015-04-13 tosca_elk_DRAFT_CSAR.zip uploaded
Sent by: <tosca@lists.oasis-open.org>
Hi Sahdev,
Thanks for the update. I finally managed to run this test case through
my validator, and it turns out there are many issues with the YAML files
in this package. In particular, many of the “requirements” sections (in
Node Templates as well as in Node Types) do not conform to the spec.
What is the best way for me to submit my comments so we can get this package
updated?
Best regards,
Chris
From: tosca@lists.oasis-open.org
[mailto:tosca@lists.oasis-open.org]
On Behalf Of Sahdev Zala
Sent: Monday, April 13, 2015 7:10 AM
To: tosca@lists.oasis-open.org
Subject: [tosca] Groups - 2015-04-13 tosca_elk_DRAFT_CSAR.zip uploaded
Submitter's message
Document for InterOp. SC review.
Thanks to Matt and Christopher Kaufmann for their initial comments, update
the draft with some minor changes. Thanks!
-- Sahdev Zala
[Date Prev]
| [Thread Prev]
| [Thread Next]
| [Date Next]
--
[Date Index]
| [Thread Index]
| [List Home]