Re: Google Summer of Code - PaaS services addition proposal

Hi It would be wonderful :) Please, write me if you need some information. Best, JoseEl 13/4/2016 8:57 a. m., Andrea Turli escribió: > > Jose, > > I'm indeed happy to mentor you. > > I will try to add myself as a mentor and will update you shortly. > > Andrea > > On 12 April 2016 at 23:48,

[jira] [Created] (BROOKLYN-255) Add public and private PaaS support to Brooklyn

Andrea Turli created BROOKLYN-255: - Summary: Add public and private PaaS support to Brooklyn Key: BROOKLYN-255 URL: https://issues.apache.org/jira/browse/BROOKLYN-255 Project: Brooklyn Issue

[GitHub] brooklyn-server pull request: [OSGi] Load properties from config a...

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/107#discussion_r59513491 --- Diff: karaf/init/src/main/java/org/apache/brooklyn/launcher/osgi/OsgiLauncher.java --- @@ -23,36 +26,78 @@ import org.apache.brooklyn.core.i

[GitHub] brooklyn-server pull request: [OSGi] Load properties from config a...

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/107#discussion_r59513542 --- Diff: karaf/init/src/main/java/org/apache/brooklyn/launcher/osgi/OsgiLauncher.java --- @@ -23,36 +26,78 @@ import org.apache.brooklyn.core.i

[GitHub] brooklyn-library pull request: Populate catalog from bundle bom

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-library/pull/25#discussion_r59519309 --- Diff: software/webapp/src/main/resources/catalog.bom --- @@ -0,0 +1,234 @@ +# Licensed to the Apache Software Foundation (ASF) under one +#

[GitHub] brooklyn-dist pull request: Populate catalog from bundle bom

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-dist/pull/26#discussion_r59518971 --- Diff: archetypes/quickstart/src/brooklyn-sample/src/main/resources/catalog.bom --- @@ -0,0 +1,24 @@ +# Licensed to the Apache Software Foundation

[GitHub] brooklyn-dist pull request: Populate catalog from bundle bom

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-dist/pull/26#discussion_r59518879 --- Diff: archetypes/quickstart/src/brooklyn-sample/src/main/resources/catalog.bom --- @@ -0,0 +1,24 @@ +# Licensed to the Apache Software Foundation

[GitHub] brooklyn-library pull request: Populate catalog from bundle bom

Github user neykov commented on the pull request: https://github.com/apache/brooklyn-library/pull/25#issuecomment-209336944 Can you squash. --- 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 thi

[GitHub] brooklyn-dist pull request: Populate catalog from bundle bom

Github user neykov commented on the pull request: https://github.com/apache/brooklyn-dist/pull/26#issuecomment-209337233 Also could you squash the commits. --- 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 d

[GitHub] brooklyn-library pull request: Populate catalog from bundle bom

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-library/pull/25#discussion_r59521509 --- Diff: software/webapp/src/main/resources/catalog.bom --- @@ -0,0 +1,234 @@ +# Licensed to the Apache Software Foundation (ASF) under one +

[GitHub] brooklyn-dist pull request: Populate catalog from bundle bom

Github user geomacy commented on the pull request: https://github.com/apache/brooklyn-dist/pull/26#issuecomment-209362988 Version updated and commits squashed. --- 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 proje

[GitHub] brooklyn-library pull request: Populate catalog from bundle bom

Github user geomacy commented on the pull request: https://github.com/apache/brooklyn-library/pull/25#issuecomment-209363921 Changes squashed. --- 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

[GitHub] brooklyn-dist pull request: Populate catalog from bundle bom

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-dist/pull/26#discussion_r59531395 --- Diff: archetypes/quickstart/src/brooklyn-sample/src/main/resources/catalog.bom --- @@ -0,0 +1,23 @@ +# Licensed to the Apache Software Foundation

[GitHub] brooklyn-dist pull request: Populate catalog from bundle bom

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-dist/pull/26#discussion_r59532598 --- Diff: archetypes/quickstart/src/brooklyn-sample/src/main/resources/catalog.bom --- @@ -0,0 +1,23 @@ +# Licensed to the Apache Software Foundatio

[GitHub] brooklyn-docs pull request: Fixes handling of undefined values in ...

GitHub user sjcorbett opened a pull request: https://github.com/apache/brooklyn-docs/pull/45 Fixes handling of undefined values in templates https://brooklyn.apache.org/learnmore/catalog is currently broken for entities with config keys without default values (e.g. [BrooklynNode](h

[GitHub] brooklyn-dist pull request: Populate catalog from bundle bom

Github user geomacy commented on the pull request: https://github.com/apache/brooklyn-dist/pull/26#issuecomment-209382589 That's more like it. --- 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

Brooklyn repos read-only

Trying to merge PRs in Brooklyn's repos fails with an error "Command 'git config hooks.asfgit.protect' returned non-zero exit status 1". Filed a ticket with Infra - https://issues.apache.org/jira/browse/INFRA-11673 Svet.

Re: Brooklyn repos read-only

This will be related to a problem I had yesterday - https://issues.apache.org/jira/browse/INFRA-11663 I've updated your ticket with a link. Richard. On 13 April 2016 at 12:46, Svetoslav Neykov < svetoslav.ney...@cloudsoftcorp.com> wrote: > Trying to merge PRs in Brooklyn's repos fails with an

[GitHub] brooklyn-dist pull request: lower minimum Vagrant version to 1.8.0

Github user johnmccabe commented on the pull request: https://github.com/apache/brooklyn-dist/pull/30#issuecomment-209399633 @neykov is there anything else I need to do before we merge this? --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] brooklyn-dist pull request: lower minimum Vagrant version to 1.8.0

Github user neykov commented on the pull request: https://github.com/apache/brooklyn-dist/pull/30#issuecomment-209400171 `1.8.0` won't work unless all env keys & values are strings, see this comment https://github.com/apache/brooklyn-dist/pull/30#issuecomment-205778386 --- If your p

[GitHub] brooklyn-dist pull request: lower minimum Vagrant version to 1.8.0

Github user johnmccabe commented on the pull request: https://github.com/apache/brooklyn-dist/pull/30#issuecomment-209401424 Ah ok, missed that, I'll ping you when I submit a fix. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] brooklyn-server pull request: Add catalog.bom application white/bl...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/106#discussion_r59538390 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java --- @@ -74,6 +80,17 @@ public void unbind(ServiceRefere

[GitHub] brooklyn-dist pull request: lower minimum Vagrant version to 1.8.0

Github user johnmccabe commented on the pull request: https://github.com/apache/brooklyn-dist/pull/30#issuecomment-209404907 Feels like a bad move to hack around a fixed bug in `1.8.0` that has been addressed in `1.8.1` - tempted to close the PR and leave as-is here but add a comment

[GitHub] brooklyn-dist pull request: lower minimum Vagrant version to 1.8.0

Github user neykov commented on the pull request: https://github.com/apache/brooklyn-dist/pull/30#issuecomment-209406371 Works for me. --- 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 fea

[GitHub] brooklyn-server pull request: Add catalog.bom application white/bl...

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/106#discussion_r59538521 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java --- @@ -74,6 +80,17 @@ public void unbind(ServiceRefer

[GitHub] brooklyn-server pull request: Add catalog.bom application white/bl...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/106#discussion_r59538603 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java --- @@ -74,6 +80,17 @@ public void unbind(ServiceRefere

[GitHub] brooklyn-server pull request: Add catalog.bom application white/bl...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/106#discussion_r59539139 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java --- @@ -74,6 +80,17 @@ public void unbind(ServiceRefere

[GitHub] brooklyn-server pull request: Add catalog.bom application white/bl...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/106#discussion_r59539285 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java --- @@ -137,9 +153,19 @@ public void removedBundle(Bund

[GitHub] brooklyn-server pull request: Add catalog.bom application white/bl...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/106#discussion_r59539820 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java --- @@ -121,12 +138,11 @@ public void removedBundle(Bun

[GitHub] brooklyn-dist pull request: lower minimum Vagrant version to 1.8.0

Github user johnmccabe commented on the pull request: https://github.com/apache/brooklyn-dist/pull/30#issuecomment-209410323 Closing --- 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 featu

[GitHub] brooklyn-server pull request: Add catalog.bom application white/bl...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/106#discussion_r59540009 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java --- @@ -137,9 +153,19 @@ public void removedBundle(Bund

[GitHub] brooklyn-dist pull request: lower minimum Vagrant version to 1.8.0

Github user johnmccabe closed the pull request at: https://github.com/apache/brooklyn-dist/pull/30 --- 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 featu

[GitHub] brooklyn-server pull request: Add catalog.bom application white/bl...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/106#discussion_r59540999 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java --- @@ -155,9 +181,42 @@ public void removedBundle(Bund

[GitHub] brooklyn-server pull request: Add catalog.bom application white/bl...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/106#discussion_r59541706 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java --- @@ -155,9 +181,42 @@ public void removedBundle(Bund

[GitHub] brooklyn-server pull request: Add catalog.bom application white/bl...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/106#discussion_r59542324 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java --- @@ -168,11 +227,18 @@ private String addLibraryDeta

[GitHub] brooklyn-server pull request: Deprecate BashCommands.ifExecutableE...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/109#discussion_r59542719 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/ssh/BashCommands.java --- @@ -235,17 +235,33 @@ public static String onlyIfExecutableMis

[GitHub] brooklyn-server pull request: Deprecate BashCommands.ifExecutableE...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/109#discussion_r59542746 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/ssh/BashCommands.java --- @@ -235,17 +235,33 @@ public static String onlyIfExecutableMis

[GitHub] brooklyn-server pull request: Deprecate BashCommands.ifExecutableE...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/109#discussion_r59542960 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/ssh/BashCommands.java --- @@ -235,17 +235,33 @@ public static String onlyIfExecutableMis

[GitHub] brooklyn-server pull request: Deprecate BashCommands.ifExecutableE...

Github user neykov commented on the pull request: https://github.com/apache/brooklyn-server/pull/109#issuecomment-209426674 That's a nice improvement, clears up the intended usage. A few minor changes needed. --- If your project is set up for it, you can reply to this email and have

[GitHub] brooklyn-server pull request: Wait windows up time parameter.

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/110#discussion_r59544043 --- Diff: software/winrm/src/main/java/org/apache/brooklyn/location/winrm/WinRmMachineLocation.java --- @@ -80,6 +81,9 @@ public static fina

[GitHub] brooklyn-server pull request: Wait windows up time parameter.

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/110#discussion_r59544168 --- Diff: software/winrm/src/main/java/org/apache/brooklyn/location/winrm/WinRmMachineLocation.java --- @@ -80,6 +81,9 @@ public static fina

[GitHub] brooklyn-server pull request: Wait windows up time parameter.

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/110#discussion_r59544306 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/JcloudsLocation.java --- @@ -2742,14 +2743,41 @@ protected LoginCreden

[GitHub] brooklyn-server pull request: Wait windows up time parameter.

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/110#discussion_r59544904 --- Diff: software/winrm/src/main/java/org/apache/brooklyn/location/winrm/WinRmMachineLocation.java --- @@ -80,6 +81,9 @@ public static fina

[GitHub] brooklyn-server pull request: Wait windows up time parameter.

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/110#discussion_r59545659 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/JcloudsLocation.java --- @@ -2742,14 +2743,41 @@ protected LoginCreden

[GitHub] brooklyn-server pull request: Wait windows up time parameter.

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/110#discussion_r59546879 --- Diff: software/winrm/src/main/java/org/apache/brooklyn/location/winrm/WinRmMachineLocation.java --- @@ -80,6 +81,9 @@ public static fina

[GitHub] brooklyn-server pull request: Wait windows up time parameter.

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/110#discussion_r59547127 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/JcloudsLocation.java --- @@ -2742,14 +2743,41 @@ protected LoginCreden

[GitHub] brooklyn-server pull request: Wait windows up time parameter.

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/110#discussion_r59547577 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/JcloudsLocation.java --- @@ -2742,14 +2743,41 @@ protected LoginCreden

[GitHub] brooklyn-server pull request: Wait windows up time parameter.

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/110#discussion_r59547487 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/JcloudsLocation.java --- @@ -2742,14 +2743,41 @@ protected LoginCreden

[GitHub] brooklyn-server pull request: Wait windows up time parameter.

Github user bostko commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/110#discussion_r59548283 --- Diff: software/winrm/src/main/java/org/apache/brooklyn/location/winrm/WinRmMachineLocation.java --- @@ -80,6 +81,9 @@ public static fina

[GitHub] brooklyn-server pull request: Wait windows up time parameter.

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/110#discussion_r59548168 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/JcloudsLocation.java --- @@ -2742,14 +2743,41 @@ protected LoginCreden

[GitHub] brooklyn-server pull request: Wait windows up time parameter.

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/110#discussion_r59548931 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/JcloudsLocation.java --- @@ -2742,14 +2743,41 @@ protected LoginCreden

[GitHub] brooklyn-server pull request: Wait windows up time parameter.

Github user neykov commented on the pull request: https://github.com/apache/brooklyn-server/pull/110#issuecomment-209445859 Finished review. The PR was updated while commenting, so part of my comments are now marked as outdated. Worth going through each one the confirm that its addres

[GitHub] brooklyn-server pull request: pull displayName from flags in Locat...

Github user neykov commented on the pull request: https://github.com/apache/brooklyn-server/pull/111#issuecomment-209446947 LGTM --- 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 e

[jira] [Commented] (BROOKLYN-250) displayName in location brooklyn.config not being used

[ https://issues.apache.org/jira/browse/BROOKLYN-250?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15239259#comment-15239259 ] ASF GitHub Bot commented on BROOKLYN-250: - Github user neykov commented on the p

[GitHub] brooklyn-server pull request: Configurable removal strategies

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/112#discussion_r59550901 --- Diff: core/src/main/java/org/apache/brooklyn/entity/group/DynamicClusterImpl.java --- @@ -151,12 +151,14 @@ public ZoneFailureDetector apply(final

[GitHub] brooklyn-server pull request: Configurable removal strategies

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/112#discussion_r59552437 --- Diff: core/src/main/java/org/apache/brooklyn/entity/group/FirstFromRemovalStrategy.java --- @@ -0,0 +1,53 @@ +/* + * Licensed to the Apac

[GitHub] brooklyn-server pull request: Configurable removal strategies

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/112#discussion_r59552571 --- Diff: core/src/main/java/org/apache/brooklyn/entity/group/SensorMatchingRemovalStrategy.java --- @@ -0,0 +1,50 @@ +/* + * Licensed to the

[GitHub] brooklyn-server pull request: Configurable removal strategies

Github user neykov commented on the pull request: https://github.com/apache/brooklyn-server/pull/112#issuecomment-209458893 LGTM, could make it more YAML friendly. --- 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 p

[GitHub] brooklyn-server pull request: Fix NPE in forcePersistNow

Github user neykov commented on the pull request: https://github.com/apache/brooklyn-server/pull/113#issuecomment-209461976 Looks like there's a similar problem with `forcePersistNow` for the `full` case - should we fix the method in a more general way? --- If your project is set up

[GitHub] brooklyn-server pull request: Wait windows up time parameter.

Github user bostko commented on the pull request: https://github.com/apache/brooklyn-server/pull/110#issuecomment-209467839 Thank you @neykov most of the comments were addressed. I am verifying the changes now. It takes some time to wait all of this :) --- If your project is set u

[GitHub] brooklyn-server pull request: Add catalog.bom application white/bl...

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/106#discussion_r59562130 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java --- @@ -74,6 +80,17 @@ public void unbind(ServiceRefer

[GitHub] brooklyn-server pull request: Add catalog.bom application white/bl...

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/106#discussion_r59563955 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java --- @@ -155,9 +181,42 @@ public void removedBundle(Bun

[GitHub] brooklyn-server pull request: Add catalog.bom application white/bl...

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/106#discussion_r59564287 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java --- @@ -155,9 +181,42 @@ public void removedBundle(Bun

[GitHub] brooklyn-server pull request: Add catalog.bom application white/bl...

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/106#discussion_r59564515 --- Diff: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java --- @@ -168,11 +227,18 @@ private String addLibraryDet

[GitHub] brooklyn-server pull request: Feature/runtime files directory

GitHub user sjcorbett opened a pull request: https://github.com/apache/brooklyn-server/pull/114 Feature/runtime files directory So blueprint writers can replace ```yaml brooklyn.config: runtimeTemplates: /path/to/source1: dest1 /path/to/source2:

Re: Google Summer of Code - PaaS services addition proposal

Hi all Firstly, I would like to thank all of you, specially to Andrea and Aled, the effort from your side. I would like to remind the deadline description which was sent by Aled: * 2016-04-12: Proposals to ASF projects must be reviewed roughly and have a potential mentor so that we know how ma

Build failed in Jenkins: brooklyn-master-build #132

See -- [...truncated 47410 lines...] [INFO] Brooklyn Jclouds Location Targets .. SUCCESS [ 30.902 s] [INFO] Brooklyn Software Base . SUCCESS [01:36 min] [I