Build failed in Jenkins: brooklyn-master-windows #27

2016-03-24 Thread Apache Jenkins Server
See -- [...truncated 28262 lines...] at org.apache.maven.surefire.testng.TestNGDirectoryTestSuite.execute(TestNGDirectoryTestSuite.java:108) at org.apache.maven.surefire.testng.Tes

Jenkins build is back to normal : brooklyn-master-build #88

2016-03-24 Thread Apache Jenkins Server
See

Build failed in Jenkins: brooklyn-server-master #19

2016-03-24 Thread Apache Jenkins Server
See Changes: [aled.sage] Tidy test imports [aled.sage] Upgrade winrm4j to 0.3.3 [aled.sage] deserializingClassRenames: include TestEntity etc [aled.sage] softwareProcess.restart: even on error, set expected=running

Build failed in Jenkins: brooklyn-library-master #11

2016-03-24 Thread Apache Jenkins Server
See Changes: [sam.corbett] Fix KafkaZooKeeper archive directory name format -- [...truncated 2046 lines...] jdk1.7.0_80/bin/jcontrol jdk1.7.0_80/bin/java jdk1.7.0_80/bin/servertool jdk1.7.0

[GitHub] brooklyn-server pull request: Winrm4j upgrade

2016-03-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/81 --- 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 feature

[GitHub] brooklyn-library pull request: Fix KafkaZooKeeper archive director...

2016-03-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-library/pull/20 --- 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 featur

[GitHub] brooklyn-library pull request: Fix KafkaZooKeeper archive director...

2016-03-24 Thread aledsage
Github user aledsage commented on the pull request: https://github.com/apache/brooklyn-library/pull/20#issuecomment-201085855 LGTM - merging. --- 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 t

[GitHub] brooklyn-library pull request: Fix: ElasticSearch advertise datast...

2016-03-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-library/pull/23 --- 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 featur

[GitHub] brooklyn-library pull request: Fix: ElasticSearch advertise datast...

2016-03-24 Thread aledsage
Github user aledsage commented on the pull request: https://github.com/apache/brooklyn-library/pull/23#issuecomment-201085233 Jenkins build failure looks like yet another unrelated strange environment thing? ``` [FATAL] Non-resolvable parent POM: Could not find artifact o

[GitHub] brooklyn-library pull request: Support disabling http based sensor...

2016-03-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-library/pull/24 --- 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 featur

[GitHub] brooklyn-server pull request: Winrm4j upgrade

2016-03-24 Thread aledsage
GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/81 Winrm4j upgrade Also see the commit "softwareProcess.restart: even on error, set expected=running", and the details in that commit description. You can merge this pull request into a Git r

[jira] [Created] (BROOKLYN-244) Blueprint Composer: text disappears after few seconds

2016-03-24 Thread Aled Sage (JIRA)
Aled Sage created BROOKLYN-244: -- Summary: Blueprint Composer: text disappears after few seconds Key: BROOKLYN-244 URL: https://issues.apache.org/jira/browse/BROOKLYN-244 Project: Brooklyn Issue

[jira] [Commented] (BROOKLYN-243) MySql stop+restart: timed out waiting for serviceUp (due to enrichers/feeds?)

2016-03-24 Thread Aled Sage (JIRA)
[ https://issues.apache.org/jira/browse/BROOKLYN-243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15211037#comment-15211037 ] Aled Sage commented on BROOKLYN-243: I saw the same thing with RabbitMQ. I deployed

[GitHub] brooklyn-server pull request: Skip to next bundle on predicate fai...

2016-03-24 Thread geomacy
Github user geomacy commented on the pull request: https://github.com/apache/brooklyn-server/pull/78#issuecomment-201020297 Trying a recycle --- 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 t

[GitHub] brooklyn-server pull request: Skip to next bundle on predicate fai...

2016-03-24 Thread geomacy
Github user geomacy commented on the pull request: https://github.com/apache/brooklyn-server/pull/78#issuecomment-201020355 Reopening --- 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 feat

[GitHub] brooklyn-server pull request: Skip to next bundle on predicate fai...

2016-03-24 Thread geomacy
GitHub user geomacy reopened a pull request: https://github.com/apache/brooklyn-server/pull/78 Skip to next bundle on predicate failure. Without this change, if the predicate fails, the code flow just picks the next predicate to check, when really we want to skip to checking the

[GitHub] brooklyn-server pull request: Skip to next bundle on predicate fai...

2016-03-24 Thread geomacy
Github user geomacy closed the pull request at: https://github.com/apache/brooklyn-server/pull/78 --- 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 featur

[GitHub] brooklyn-docs pull request: Details about using CredSSP in Windows

2016-03-24 Thread aledsage
Github user aledsage commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/36#discussion_r57380878 --- Diff: guide/yaml/winrm/index.md --- @@ -289,11 +289,24 @@ When a script is run over WinRM, the credentials under which the script are run solu

[GitHub] brooklyn-docs pull request: Details about using CredSSP in Windows

2016-03-24 Thread bostko
GitHub user bostko opened a pull request: https://github.com/apache/brooklyn-docs/pull/36 Details about using CredSSP in Windows @aledsage can you review. You can merge this pull request into a Git repository by running: $ git pull https://github.com/bostko/brooklyn-docs winrm_

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

2016-03-24 Thread Apache Jenkins Server
See -- [...truncated 49288 lines...] [INFO] Brooklyn Root .. SKIPPED [INFO] [INFO] BUILD FA

[GitHub] brooklyn-dist pull request: use JAVA_HOME if available when starti...

2016-03-24 Thread johnmccabe
Github user johnmccabe commented on a diff in the pull request: https://github.com/apache/brooklyn-dist/pull/25#discussion_r57371163 --- Diff: dist/src/main/dist/bin/brooklyn --- @@ -48,4 +48,10 @@ JAVA_OPTS="-Dbrooklyn.location.localhost.address=127.0.0.1 ${JAVA_OPTS}"

Re: CLI packaging problem

2016-03-24 Thread John McCabe
And a reminder that https://github.com/apache/brooklyn-client/pull/11 will need to be merged to ensure Jenkins builds the current code in the repo rather than the cached copy in godeps/_workspace On Thu, 24 Mar 2016 at 18:47 Geoff Macartney < geoff.macart...@cloudsoftcorp.com> wrote: > > The broo

Re: CLI packaging problem

2016-03-24 Thread Geoff Macartney
The brooklyn-client Apache build adds the CLI binaries in a zip file to the Apache Jenkins. E.g. See links at end of https://builds.apache.org/view/Brooklyn/job/brooklyn-client-master/9/console … Uploaded: https://repository.apache.org/content/repositories/snapshots/org/apache/brooklyn/brookl

Re: CLI packaging problem

2016-03-24 Thread Aled Sage
+1 We won't have the brooklyn-client RPM in time for a 0.9.0 release, but we should definitely work on that afterwards. --- We can add a download link for the CLI to http://brooklyn.apache.org/download/index.html, once there is an official apache release of the CLI. I believe the release m

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

2016-03-24 Thread geomacy
GitHub user geomacy opened a pull request: https://github.com/apache/brooklyn-dist/pull/26 Populate catalog from bundle bom The PR adds updates to the existing jars to add a catalog.bom file that includes the details of the various entities in each. See https://github.com

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

2016-03-24 Thread geomacy
GitHub user geomacy opened a pull request: https://github.com/apache/brooklyn-library/pull/25 Populate catalog from bundle bom The PR adds updates to the existing jars to add a catalog.bom file that includes the details of the various entities in each. See https://github.

Re: CLI packaging problem

2016-03-24 Thread Robert Moss
+1 On 24 March 2016 at 18:12, John McCabe wrote: > +1 Geoff > Having a CLI download alongside the brooklyn release as well makes sense, > like the web ui the user should be able to run the client from their local > machine to connect to a remote brooklyn instance (which is supported and > imho p

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

2016-03-24 Thread geomacy
GitHub user geomacy opened a pull request: https://github.com/apache/brooklyn-server/pull/80 Populate catalog from bundle bom This PR adds the ability for bundles to contain a 'catalog.bom' file specifying items that the bundle contains which should be added to the catalog.

Re: CLI packaging problem

2016-03-24 Thread John McCabe
+1 Geoff Having a CLI download alongside the brooklyn release as well makes sense, like the web ui the user should be able to run the client from their local machine to connect to a remote brooklyn instance (which is supported and imho preferable since it keeps you off the server), bundling it only

[GitHub] brooklyn-server pull request: Skip to next bundle on predicate fai...

2016-03-24 Thread geomacy
Github user geomacy commented on the pull request: https://github.com/apache/brooklyn-server/pull/78#issuecomment-200953384 Did this not complete? The build seems to have finished successfully. --- If your project is set up for it, you can reply to this email and have your reply appe

Re: CLI packaging problem

2016-03-24 Thread Geoff Macartney
I suggest adding the CLI artifacts to the release alongside the brooklyn zip. At the same time as the brooklyn zip is copied to the release location, get the CLI artifact out of the Maven repository, for good measure unpack it, and upload the individual builds of the CLI tool to the same locati

CLI packaging problem

2016-03-24 Thread Duncan Godwin
Hi All, I've found a problem with the way the CLI is distributed for Brooklyn. It's included in the release files which means when an initial user uses the vagrant getting started guide, the CLI is inside the vagrant box, not on the users machine. As there's no download link to the CLI anywhere it

[GitHub] brooklyn-docs pull request: Minor wording changes around RPM insta...

2016-03-24 Thread murdoaird
GitHub user murdoaird opened a pull request: https://github.com/apache/brooklyn-docs/pull/35 Minor wording changes around RPM install Minor wording changes around RPM install You can merge this pull request into a Git repository by running: $ git pull https://github.com/murdoa

[GitHub] brooklyn-library pull request: Move custom-enable-credssp.ps1 to b...

2016-03-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-library/pull/22 --- 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 featur

[GitHub] brooklyn-library pull request: Move custom-enable-credssp.ps1 to b...

2016-03-24 Thread aledsage
Github user aledsage commented on the pull request: https://github.com/apache/brooklyn-library/pull/22#issuecomment-200913178 Now that https://github.com/apache/brooklyn-server/pull/75 is merged, I'll merge this. --- If your project is set up for it, you can reply to this email and h

[GitHub] brooklyn-server pull request: Add Windows script for enabling Cred...

2016-03-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/75 --- 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 feature

[GitHub] brooklyn-server pull request: Add Windows script for enabling Cred...

2016-03-24 Thread aledsage
Github user aledsage commented on the pull request: https://github.com/apache/brooklyn-server/pull/75#issuecomment-200911605 LGTM - merging. --- 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 th

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57339862 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57338936 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,385

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57339405 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,385

[GitHub] brooklyn-server pull request: Skip to next bundle on predicate fai...

2016-03-24 Thread neykov
Github user neykov commented on the pull request: https://github.com/apache/brooklyn-server/pull/78#issuecomment-200886790 Kicked a new build due to the unrelated failure. ``` Results : Tests run: 0, Failures: 0, Errors: 0, Skipped: 0 [WARNING] Failed to notify

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread aledsage
Github user aledsage commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57331002 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,385

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread grkvlt
Github user grkvlt commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57332394 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,385 @

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread grkvlt
Github user grkvlt commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57334104 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,385 @

[GitHub] brooklyn-server pull request: [OSGi] Improve REST API behaviour in...

2016-03-24 Thread neykov
GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-server/pull/79 [OSGi] Improve REST API behaviour in Karaf * Use the correct json serializer * Make version files accessible * Fix collecting version info from bundles - array should be mutable

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread aledsage
Github user aledsage commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57329562 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread grkvlt
Github user grkvlt commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57329713 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398 @

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread grkvlt
Github user grkvlt commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57330074 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398 @

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread aledsage
Github user aledsage commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57329026 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398

[GitHub] brooklyn-docs pull request: Minor wording changes around option of...

2016-03-24 Thread aledsage
Github user aledsage commented on the pull request: https://github.com/apache/brooklyn-docs/pull/34#issuecomment-200869930 LGTM; merging. --- 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

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread aledsage
Github user aledsage commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57330042 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,385

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread grkvlt
Github user grkvlt commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57329759 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398 @

[GitHub] brooklyn-docs pull request: Minor wording changes around option of...

2016-03-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-docs/pull/34 --- 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 feature i

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57326426 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57328790 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398

[GitHub] brooklyn-docs pull request: Update running.md

2016-03-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-docs/pull/33 --- 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 feature i

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread grkvlt
Github user grkvlt commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57326425 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398 @

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread grkvlt
Github user grkvlt commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57325776 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398 @

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread grkvlt
Github user grkvlt commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57325973 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398 @

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread grkvlt
Github user grkvlt commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57325453 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398 @

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread grkvlt
Github user grkvlt commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57325507 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398 @

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread grkvlt
Github user grkvlt commented on the pull request: https://github.com/apache/brooklyn-server/pull/53#issuecomment-200849031 @ahgittin I found the latches had better semantics than semaphores; will look into your other comments... --- If your project is set up for it, you can reply to

[GitHub] brooklyn-docs pull request: Update running.md

2016-03-24 Thread aledsage
Github user aledsage commented on the pull request: https://github.com/apache/brooklyn-docs/pull/33#issuecomment-200843649 Thanks - merging. @murdoaird can build on this, to give more info to those unfamiliar with RPMs and the like. --- If your project is set up for it, you can reply

[GitHub] brooklyn-server pull request: Skip to next bundle on predicate fai...

2016-03-24 Thread geomacy
Github user geomacy commented on the pull request: https://github.com/apache/brooklyn-server/pull/78#issuecomment-200835068 Have rebased and pushed the change suggested. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] brooklyn-server pull request: Add Windows script for enabling Cred...

2016-03-24 Thread bostko
Github user bostko commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/75#discussion_r57310466 --- Diff: logging/logback-includes/src/main/resources/brooklyn/logback-logger-excludes.xml --- @@ -42,6 +42,8 @@ +

[GitHub] brooklyn-server pull request: Add Windows script for enabling Cred...

2016-03-24 Thread aledsage
Github user aledsage commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/75#discussion_r57308247 --- Diff: logging/logback-includes/src/main/resources/brooklyn/logback-logger-excludes.xml --- @@ -42,6 +42,8 @@ +

Build failed in Jenkins: brooklyn-master-windows #26

2016-03-24 Thread Apache Jenkins Server
See -- [...truncated 52907 lines...] Compressed 648.22 KB of artifacts by 14.8% relative to #25 [JENKINS] Archiving F:\hudson\hudson-slave\workspace\brooklyn-master-windows\brooklyn-ui\pom.xml to

[GitHub] brooklyn-server pull request: Don't skip tests jar generation duri...

2016-03-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/77 --- 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 feature

[GitHub] brooklyn-server pull request: Don't skip tests jar generation duri...

2016-03-24 Thread ahgittin
Github user ahgittin commented on the pull request: https://github.com/apache/brooklyn-server/pull/77#issuecomment-200803214 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

[GitHub] brooklyn-server pull request: Add a label to predicates check in f...

2016-03-24 Thread geomacy
Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/78#discussion_r57306640 --- Diff: core/src/main/java/org/apache/brooklyn/util/core/osgi/Osgis.java --- @@ -181,11 +181,11 @@ public BundleFinder requiringFromUrl(String url)

[GitHub] brooklyn-server pull request: Add a label to predicates check in f...

2016-03-24 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/78#discussion_r57306501 --- Diff: core/src/main/java/org/apache/brooklyn/util/core/osgi/Osgis.java --- @@ -181,11 +181,11 @@ public BundleFinder requiringFromUrl(String url)

[GitHub] brooklyn-docs pull request: Describe SoftLayerSameVlanLocationCust...

2016-03-24 Thread ahgittin
Github user ahgittin commented on the pull request: https://github.com/apache/brooklyn-docs/pull/31#issuecomment-200802335 great content -- quite a few minor tweaks then good to merge once the change in server is merged --- If your project is set up for it, you can reply to t

[GitHub] brooklyn-docs pull request: Describe SoftLayerSameVlanLocationCust...

2016-03-24 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/31#discussion_r57305871 --- Diff: guide/ops/locations/more-clouds.md --- @@ -172,4 +172,86 @@ For example, for dev/demo purposes an "everything" network could be created that

[GitHub] brooklyn-docs pull request: Describe SoftLayerSameVlanLocationCust...

2016-03-24 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/31#discussion_r57305606 --- Diff: guide/ops/locations/more-clouds.md --- @@ -172,4 +172,86 @@ For example, for dev/demo purposes an "everything" network could be created that

[GitHub] brooklyn-docs pull request: Describe SoftLayerSameVlanLocationCust...

2016-03-24 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/31#discussion_r57305376 --- Diff: guide/ops/locations/more-clouds.md --- @@ -172,4 +172,86 @@ For example, for dev/demo purposes an "everything" network could be created that

[GitHub] brooklyn-docs pull request: Minor wording changes around option of...

2016-03-24 Thread murdoaird
GitHub user murdoaird opened a pull request: https://github.com/apache/brooklyn-docs/pull/34 Minor wording changes around option of installing binaries in own env Calling it "local install" was confusing - people didn't realise you could install the binaries anywhere. You can merge

[GitHub] brooklyn-docs pull request: Describe SoftLayerSameVlanLocationCust...

2016-03-24 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/31#discussion_r57305130 --- Diff: guide/ops/locations/more-clouds.md --- @@ -172,4 +172,86 @@ For example, for dev/demo purposes an "everything" network could be created that

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread ahgittin
Github user ahgittin commented on the pull request: https://github.com/apache/brooklyn-server/pull/53#issuecomment-200798220 A few more comments I'm afraid @grkvlt . The latching logic still looks dubious and needs addressing, same for the non-transience of the latch. The other comm

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57304674 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57304446 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57304133 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57303936 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398

[GitHub] brooklyn-server pull request: SoftLayer VLAN customizer

2016-03-24 Thread ahgittin
Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/53#discussion_r57302479 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/softlayer/SoftLayerSameVlanLocationCustomizer.java --- @@ -0,0 +1,398

[GitHub] brooklyn-ui pull request: Show error message on top of the YAML ed...

2016-03-24 Thread tbouron
Github user tbouron commented on a diff in the pull request: https://github.com/apache/brooklyn-ui/pull/21#discussion_r57298808 --- Diff: src/main/webapp/assets/tpl/editor/page.html --- @@ -47,12 +47,12 @@ Blueprint Composer

[GitHub] brooklyn-ui pull request: Show error message on top of the YAML ed...

2016-03-24 Thread bostko
Github user bostko commented on a diff in the pull request: https://github.com/apache/brooklyn-ui/pull/21#discussion_r57298452 --- Diff: src/main/webapp/assets/tpl/editor/page.html --- @@ -47,12 +47,12 @@ Blueprint Composer

Re: Brooklyn configuration in Karaf

2016-03-24 Thread Alex Heneveld
+1 this plan sounds excellent Svet. I think the focus on catalog.bom files will make it a lot easier to package and share blueprints -- whether as bundles, zips, plain text files, or a git url. I really like that we can use the power of OSGi for isolation and dependency resolution but this i

Brooklyn configuration in Karaf

2016-03-24 Thread Svetoslav Neykov
Hi all, There's lots of effort going into Brooklyn to make it run in a Karaf container. As part of the changes we need to figure out how to support the existing CLI arguments and brooklyn.properties in the new environment. Here's a suggestion of how this might look, open to discussion. Command

Jenkins build is back to normal : brooklyn-server-master #16

2016-03-24 Thread Apache Jenkins Server
See

[GitHub] brooklyn-library pull request: Fix: ElasticSearch advertise datast...

2016-03-24 Thread bostko
Github user bostko commented on the pull request: https://github.com/apache/brooklyn-library/pull/23#issuecomment-200732675 Looks good! --- 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 fe