These are chat archives for cloudsoft/brooklyn-tosca

24th
Nov 2015
Alex Heneveld
@ahgittin
Nov 24 2015 09:16
hi @/all — meetings yesterday, sorry i missed this
re building, currently it makes a dist. i’ve not tried osgi or dropin jars.
i’m sure the latter will work but it will take a bit of work to determine the list of all the non-brooklyn jars (ie including A4C deps).
osgi will be good test for karaf, although again you’ll need to determine the list of non-brooklyn deps; and they might not all be osgi (but you could probably tell maven to bundle those into an uber somehow)
as for namespace i suggest io.cloudsoft as we own that domain (at sonatype). it might move to apache at some point.
Andrea Turli
@andreaturli
Nov 24 2015 09:28
ok thanks @ahgittin so we need to rename <groupId> from org.apache.brooklyn to io.cloudsoft ?
Andrea Turli
@andreaturli
Nov 24 2015 09:43
sorry I meant from <groupId>org.apache.brooklyn.tosca</groupId> to <groupId>io.cloudsoft.brooklyn.tosca</groupId> ?
Sam Corbett
@sjcorbett
Nov 24 2015 10:34
Why io.cloudsoft over io.brooklyn?
Andrea Turli
@andreaturli
Nov 24 2015 10:35
hi @sjcorbett, I guess it is because we owned io.cloudsoft
Sam Corbett
@sjcorbett
Nov 24 2015 10:35
We own io.brooklyn too, do we not?
Andrea Turli
@andreaturli
Nov 24 2015 10:35
not sure
looks like both io.brooklyn and io.cloudsoft are available
so either ways are ok with me
Andrea Turli
@andreaturli
Nov 24 2015 11:17
@sjcorbett I created a PR to add Travis support cloudsoft/brooklyn-tosca#35
Sam Corbett
@sjcorbett
Nov 24 2015 11:19
Nice one
Sam Corbett
@sjcorbett
Nov 24 2015 11:39
@kiuby88 Andrea has configured the project to publish builds. Later today I will rename the packages in brooklyn-tosca-transformer from org.apache.brooklyn to io.cloudsoft. Will let you know once it's done.
Jose
@kiuby88
Nov 24 2015 11:40
Nice
Thanks a lot
Andrea Turli
@andreaturli
Nov 24 2015 11:42
@sjcorbett the upload failed because seems that my account is not authorized for that sonatype space https://travis-ci.org/cloudsoft/brooklyn-tosca/builds/92914523#L4999
I’ll try manually once
Sam Corbett
@sjcorbett
Nov 24 2015 11:42
I just noticed too.
Jose
@kiuby88
Nov 24 2015 11:43
:)
Andrea Turli
@andreaturli
Nov 24 2015 11:43
I guess I forgot the distribution repo
@sjcorbett can you merge this cloudsoft/brooklyn-tosca#36 ?
Sam Corbett
@sjcorbett
Nov 24 2015 11:48
Yes
Andrea Turli
@andreaturli
Nov 24 2015 11:49
thanks, let’s see if I fixed it
Alex Heneveld
@ahgittin
Nov 24 2015 11:56
personally i’m happy w either io.cloudsoft.tosca or io.brooklyn.tosca
ops
Alex Heneveld
@ahgittin
Nov 24 2015 11:56
only concern w the latter is if we fall afoul of apache branding
i know we own the domain but we transferred the trademark to them
Andrea Turli
@andreaturli
Nov 24 2015 11:56
because I think it is easier to move to org.apache.brooklyn.tosca later on
Alex Heneveld
@ahgittin
Nov 24 2015 11:57
+1 that seems safer for now
Andrea Turli
@andreaturli
Nov 24 2015 11:57
but happy to change it, f needed
Sam Corbett
@sjcorbett
Nov 24 2015 11:57
Thanks for clarifying
Maven plugin is under io.brooklyn. Should it be moved?
Alex Heneveld
@ahgittin
Nov 24 2015 11:58
and we might well transfer it either to io.brooklyn or even ASF
so io.cloudsoft.brooklyn.tosca:brooklyn-tosca
re maven plugin … fine to leave it where it is
Sam Corbett
@sjcorbett
Nov 24 2015 11:58
Ta
Alex Heneveld
@ahgittin
Nov 24 2015 11:59
“consistency is the hobgoblin of small minds” :)
Andrea Turli
@andreaturli
Nov 24 2015 16:01
hi @nakomis, how is it going with the ToscaMetadataProvider proposal?
I think I missed the result
Ghost
@ghost~54aa7954db8155e6700e5ead
Nov 24 2015 16:03
@andreaturli it’s been merged #33
Andrea Turli
@andreaturli
Nov 24 2015 16:07
thanks @robertgmoss!
Ghost
@ghost~54aa7954db8155e6700e5ead
Nov 24 2015 16:13
So the current implementation is very basic, but we have added it in such a way that you can provide your own implementations on the classpath and specify in the alien gui what these classes are.
Andrea Turli
@andreaturli
Nov 24 2015 16:14
I’m not using a4c, so if I add my impl to the classpath, what else I need to do, theoretically?
Sam Corbett
@sjcorbett
Nov 24 2015 16:20
andreaturli: the code you linked to is in the alien4cloud Brooklyn plugin. What are you doing with it if you aren't using Alien?
Andrea Turli
@andreaturli
Nov 24 2015 16:21
suppose I want to re-use the providers approach with SeaClouds, a downstream project that uses brookly-tosca-transformer
maybe it is not as easy as I think
Sam Corbett
@sjcorbett
Nov 24 2015 16:25
Re-reading @nakomis' original proposal we are only halfway
The nice section on listing providers in yaml is not possible at the moment.
Unless I'm mistaken.
Andrea Turli
@andreaturli
Nov 24 2015 16:26
ok that’s fine, so long term I can expect that to be possible
so maybe @kiuby88 you should help there, instead of re-doing the same in SeaClouds
wdyt?
Jose
@kiuby88
Nov 24 2015 16:27
it sounds good
Sam Corbett
@sjcorbett
Nov 24 2015 16:28
You want to use ToscaMetadataProvider but not Alien. Correct?
Andrea Turli
@andreaturli
Nov 24 2015 16:29
I think so
that would avoid SeaClouds to write and maintain a subsequent transformation
Sam Corbett
@sjcorbett
Nov 24 2015 16:30
Ok. I can do the first part, move it into a separate Maven module (brooklyn-tosca-common? @kiuby88 maybe you would like to pick up the second part, of making it possible to customise?
Keeping it separate from brooklyn-tosca-transformer.
Jose
@kiuby88
Nov 24 2015 16:33
Currently, SeaClouds defines its own tosca model (NodeTypes, Relationships, etc). Then, we are transforming from sea clouds-model (a template) to brooklyn-transformer-model (another template).
Really, both of them (models) are TOSCA models so, we are transform from seaclouds types to brooklyn types
Andrea Turli
@andreaturli
Nov 24 2015 16:35
I think @sjcorbett is right, what we need is a way to customise the providers
the SeaCloudsToscaMetdataProvider will live into SeaClouds project and will implement the ToscaMetadataProvider, I think
Sam Corbett
@sjcorbett
Nov 24 2015 16:37
Well, implement ToscaTypeProvider and instruct ToscaMetadataProvider to use your class.
It's the latter bit that you can't do at the moment.
Jose
@kiuby88
Nov 24 2015 16:38
I agree with @andreaturli . Really, it will not be a problem.
of course @sjcorbett . I glad to help
Andrea Turli
@andreaturli
Nov 24 2015 16:39
correct @sjcorbett, sorry for the confusion with names
Sam Corbett
@sjcorbett
Nov 24 2015 16:49
First step for you: cloudsoft/brooklyn-tosca#38
@andreaturli @kiuby88 Would you like to review?
Andrea Turli
@andreaturli
Nov 24 2015 16:53
thx @sjcorbett, so basically with brooklyn-tosca-common and brooklyn-tosca-transformer now @kiuby88 can implement the equivalent of the BrooklynOrchestratorFactory.init
or I’ve completely misunderstood the point?
Sam Corbett
@sjcorbett
Nov 24 2015 16:54
Yes
Without depending on a project meant for a4c integration.
Andrea Turli
@andreaturli
Nov 24 2015 16:55
so no real need to add the second step ^^^ (yaml configurability)
which maybe we can implement with java SPI later, if needed
Sam Corbett
@sjcorbett
Nov 24 2015 16:56
Probably a straightforward way to go.
Though indicating the ordering would be awkward.
Andrea Turli
@andreaturli
Nov 24 2015 16:57
yup, good point, ordering seems crucial here
Jose
@kiuby88
Nov 24 2015 17:10
@sjcorbett thanks a lot!!!!!
I will take a look to cloudsoft/brooklyn-tosca#38
Alex Heneveld
@ahgittin
Nov 24 2015 17:34
hey guys — the CSAR stuff i’m seeing in PR's
does this mean we support uploads of ZIPs ?
in the type-registry i’ve removed assumptions that the plan is yaml so that we will be able to support ZIPs as plans etc. it’s maybe worth converting them on-the-fly to osgi bundles so that class-loading works.
Sam Corbett
@sjcorbett
Nov 24 2015 17:40
@nakomis @robertgmoss ^^