date:20160406

See 

--
[...truncated 37423 lines...]
[INFO] Brooklyn Common Utilities .. SUCCESS [ 25.826 s]
[INFO] Brooklyn API ... SUCCESS [  3.790 s]
[INFO] CAMP Server Parent Project . SUCCESS [  0.480 s]
[INFO] CAMP Base .. SUCCESS [  4.647 s]
[INFO] Brooklyn Test Support .. SUCCESS [  2.605 s]
[INFO] Brooklyn REST Swagger Apidoc Utilities . SUCCESS [  3.095 s]
[INFO] Brooklyn Logback Configuration . SUCCESS [  0.522 s]
[INFO] CAMP Server  SUCCESS [  8.792 s]
[INFO] CAMP Server Jersey . SUCCESS [  8.505 s]
[INFO] Brooklyn Felix Runtime . SUCCESS [  4.836 s]
[INFO] Brooklyn Groovy Utilities .. SUCCESS [  1.247 s]
[INFO] Brooklyn Core .. SUCCESS [04:22 min]
[INFO] Brooklyn Policies .. SUCCESS [01:25 min]
[INFO] Brooklyn WinRM Software Entities ... SUCCESS [  9.316 s]
[INFO] Brooklyn Secure JMXMP Agent  SUCCESS [ 10.488 s]
[INFO] Brooklyn JMX RMI Agent . SUCCESS [  0.937 s]
[INFO] Brooklyn Jclouds Location Targets .. SUCCESS [ 24.313 s]
[INFO] Brooklyn Software Base . SUCCESS [01:21 min]
[INFO] Brooklyn CAMP REST API . SUCCESS [05:18 min]
[INFO] Brooklyn Hazelcast Storage . SUCCESS [  5.353 s]
[INFO] Brooklyn Launcher Common ... SUCCESS [  5.038 s]
[INFO] Brooklyn REST API .. SUCCESS [ 10.494 s]
[INFO] Brooklyn REST Resources  FAILURE [ 56.751 s]
[INFO] Brooklyn REST Server ... SKIPPED
[INFO] Brooklyn Launcher .. SKIPPED
[INFO] Brooklyn Command Line Interface  SKIPPED
[INFO] Brooklyn REST Client ... SKIPPED
[INFO] Brooklyn REST Server with Jersey dependencies .. SKIPPED
[INFO] Brooklyn Test Framework  SKIPPED
[INFO] Brooklyn OSGi init . SKIPPED
[INFO] Brooklyn Karaf . SKIPPED
[INFO] Jetty config fragment .. SKIPPED
[INFO] Brooklyn Karaf Features  SKIPPED
[INFO] Brooklyn Karaf Distro .. SKIPPED
[INFO] Brooklyn Karaf Shell Commands .. SKIPPED
[INFO] Brooklyn Karaf pax-exam itest .. SKIPPED
[INFO] Brooklyn Library Root .. SKIPPED
[INFO] Brooklyn CM SaltStack .. SKIPPED
[INFO] Brooklyn CM Ansible  SKIPPED
[INFO] Brooklyn CM Integration Root ... SKIPPED
[INFO] Brooklyn Network Software Entities . SKIPPED
[INFO] Brooklyn OSGi Software Entities  SKIPPED
[INFO] Brooklyn Database Software Entities  SKIPPED
[INFO] Brooklyn Web App Software Entities . SKIPPED
[INFO] Brooklyn Messaging Software Entities ... SKIPPED
[INFO] Brooklyn NoSQL Data Store Software Entities  SKIPPED
[INFO] Brooklyn Monitoring Software Entities .. SKIPPED
[INFO] Brooklyn QA  SKIPPED
[INFO] Brooklyn Examples Aggregator Project ... SKIPPED
[INFO] Brooklyn Examples Aggregator Project - Webapps . SKIPPED
[INFO] hello-world-webapp Maven Webapp  SKIPPED
[INFO] hello-world-sql-webapp Maven Webapp  SKIPPED
[INFO] Brooklyn Simple Web Cluster Example  SKIPPED
[INFO] Brooklyn Global Web Fabric Example . SKIPPED
[INFO] Brooklyn Simple Messaging Publish-Subscribe Example  SKIPPED
[INFO] Brooklyn NoSQL Cluster Examples  SKIPPED
[INFO] Brooklyn Downstream Project Parent . SKIPPED
[INFO] Brooklyn Dist Root . SKIPPED
[INFO] Brooklyn All Things  SKIPPED
[INFO] Brooklyn Distribution .. SKIPPED
[INFO] Brooklyn Vagrant Getting Started Environment ... SKIPPED
[INFO] Brooklyn Quick-Start Project Archetype . SKIPPED
[INFO] Brooklyn Root .. SKIPPED
[INFO] 
[INFO] BUILD FAILURE
[INFO] 
[INFO] Total time: 16:51 min
[INFO] Finished at: 2016-04-07T02:49:44+01:00
[INFO] Final Memory: 117M/463M
[INFO]

[GitHub] brooklyn-server pull request: Fixed command for installing python ...

2016-04-06 Thread jongyoul

GitHub user jongyoul opened a pull request:

https://github.com/apache/brooklyn-server/pull/105

Fixed command for installing python via apt-get and yum by using sudo

When I use a "Template 2: Bash Web Server" with non-root user and 
password-less sudo, I've got a error because I don't have any python installed. 
We should use sudo command for installing python.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/jongyoul/brooklyn-server python-install

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/brooklyn-server/pull/105.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #105


commit a0341a4c5d9cb2111d875caa8f1d7a1af9a058e4
Author: Jongyoul Lee 
Date:   2016-04-05T03:20:41Z

Fixed command for installing python via apt-get and yum by using sudo




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

See 

--
[...truncated 28941 lines...]
at 
org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:203)
at 
org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:155)
at 
org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:103)


Results :

Failed tests: 
  
SoftwareProcessPersisterInMemorySizeIntegrationTest>BrooklynMementoPersisterInMemorySizeIntegrationTest.testPersistenceVolumeFastNoTrigger:67->BrooklynMementoPersisterInMemorySizeIntegrationTest.doTestPersistenceVolume:83
 should have written fewer than 80 files, wrote 80 expected [true] but found 
[false]

Tests run: 155, Failures: 1, Errors: 0, Skipped: 0

[JENKINS] Recording test results
[INFO] 
[INFO] Reactor Summary:
[INFO] 
[INFO] Brooklyn REST JavaScript Web GUI ... SUCCESS [01:22 min]
[INFO] Brooklyn Server Root ... SUCCESS [  0.381 s]
[INFO] Brooklyn Parent Project  SUCCESS [ 12.306 s]
[INFO] Brooklyn Test Support Utilities  SUCCESS [ 11.055 s]
[INFO] Brooklyn Logback Includable Configuration .. SUCCESS [  1.059 s]
[INFO] Brooklyn Common Utilities .. SUCCESS [ 40.407 s]
[INFO] Brooklyn API ... SUCCESS [  4.878 s]
[INFO] CAMP Server Parent Project . SUCCESS [  0.920 s]
[INFO] CAMP Base .. SUCCESS [  6.557 s]
[INFO] Brooklyn Test Support .. SUCCESS [  3.447 s]
[INFO] Brooklyn REST Swagger Apidoc Utilities . SUCCESS [  4.211 s]
[INFO] Brooklyn Logback Configuration . SUCCESS [  0.730 s]
[INFO] CAMP Server  SUCCESS [ 12.430 s]
[INFO] CAMP Server Jersey . SUCCESS [ 12.248 s]
[INFO] Brooklyn Felix Runtime . SUCCESS [  4.911 s]
[INFO] Brooklyn Groovy Utilities .. SUCCESS [  1.493 s]
[INFO] Brooklyn Core .. SUCCESS [08:51 min]
[INFO] Brooklyn Policies .. SUCCESS [02:32 min]
[INFO] Brooklyn WinRM Software Entities ... SUCCESS [ 15.307 s]
[INFO] Brooklyn Secure JMXMP Agent  SUCCESS [ 19.363 s]
[INFO] Brooklyn JMX RMI Agent . SUCCESS [  1.555 s]
[INFO] Brooklyn Jclouds Location Targets .. SUCCESS [ 44.356 s]
[INFO] Brooklyn Software Base . FAILURE [02:18 min]
[INFO] Brooklyn CAMP REST API . SKIPPED
[INFO] Brooklyn Hazelcast Storage . SKIPPED
[INFO] Brooklyn Launcher Common ... SKIPPED
[INFO] Brooklyn REST API .. SKIPPED
[INFO] Brooklyn REST Resources  SKIPPED
[INFO] Brooklyn REST Server ... SKIPPED
[INFO] Brooklyn Launcher .. SKIPPED
[INFO] Brooklyn Command Line Interface  SKIPPED
[INFO] Brooklyn REST Client ... SKIPPED
[INFO] Brooklyn REST Server with Jersey dependencies .. SKIPPED
[INFO] Brooklyn Test Framework  SKIPPED
[INFO] Brooklyn OSGi init . SKIPPED
[INFO] Brooklyn Karaf . SKIPPED
[INFO] Jetty config fragment .. SKIPPED
[INFO] Brooklyn Karaf Features  SKIPPED
[INFO] Brooklyn Karaf Distro .. SKIPPED
[INFO] Brooklyn Karaf Shell Commands .. SKIPPED
[INFO] Brooklyn Karaf pax-exam itest .. SKIPPED
[INFO] Brooklyn Library Root .. SKIPPED
[INFO] Brooklyn CM SaltStack .. SKIPPED
[INFO] Brooklyn CM Ansible  SKIPPED
[INFO] Brooklyn CM Integration Root ... SKIPPED
[INFO] Brooklyn Network Software Entities . SKIPPED
[INFO] Brooklyn OSGi Software Entities  SKIPPED
[INFO] Brooklyn Database Software Entities  SKIPPED
[INFO] Brooklyn Web App Software Entities . SKIPPED
[INFO] Brooklyn Messaging Software Entities ... SKIPPED
[INFO] Brooklyn NoSQL Data Store Software Entities  SKIPPED
[INFO] Brooklyn Monitoring Software Entities .. SKIPPED
[INFO] Brooklyn QA  SKIPPED
[INFO] Brooklyn Examples Aggregator Project ... SKIPPED
[INFO] Brooklyn Examples Aggregator Project - Webapps . SKIPPED
[INFO]

[VOTE] Release Apache Brooklyn 0.9.0 [rc3]

2016-04-06 Thread Richard Downer

This is to call for a vote for the release of Apache Brooklyn 0.9.0 [rc3].

This release comprises of a source code distribution, and a
corresponding binary distribution, RPM packages, Vagrant environment
package, and Maven artifacts.

The source and binary distributions, including signatures, digests,
etc. can be found at:
https://dist.apache.org/repos/dist/dev/brooklyn/apache-brooklyn-0.9.0-rc3

The artifact SHA-256 checksums are as follows:
65e3438c81c76d47f98e637554e4b80c8c869ff482c18ca9b2d838778268a56b
*apache-brooklyn-0.9.0-rc3-1.noarch.rpm
1a1235c10962371f52240f7fcb1c89f1d066cbb55715c493f4fdffc0a6927da0
*apache-brooklyn-0.9.0-rc3-bin.tar.gz
f95a566d5bb086906d8a4657029757b08a90d2483ef00a7eae0ddd55832b0f4c
*apache-brooklyn-0.9.0-rc3-bin.zip
bbceefda7444a0235527d9830230acb6a3d9f9c6280f36bccc59486a3ee80db8
*apache-brooklyn-0.9.0-rc3-client-cli-linux.tar.gz
7a1cabc5d96be11cbd538a202c1712f9bd053be18f37dc533c543004b247a802
*apache-brooklyn-0.9.0-rc3-client-cli-linux.zip
51ecff55d2c7308efc7f215cc5ab4bdfd8db8d2588fde3346b3c5ad53cf9d396
*apache-brooklyn-0.9.0-rc3-client-cli-macosx.tar.gz
cce8cf9819cb57b1c0bbbdbef76da43fb8142ed2f0a6b9cb613850e2188081d6
*apache-brooklyn-0.9.0-rc3-client-cli-macosx.zip
6d5f1ca2d9550048c80cdf951f00c780c3766d09960f15a47585c37e44c16926
*apache-brooklyn-0.9.0-rc3-client-cli-windows.tar.gz
d0d021f17efe389558b5d12141694179f3245acd9d016bd9de03917fc6b9fa4c
*apache-brooklyn-0.9.0-rc3-client-cli-windows.zip
b993253273280027c45adb4d6899493e8df0301080128d6c8bc10148da3c1a74
*apache-brooklyn-0.9.0-rc3-src.tar.gz
f9e4396ab123b650fe02675e0e753f819c4a5bcbf8ed48fd5fa925a6ba93e149
*apache-brooklyn-0.9.0-rc3-src.zip
052412c38707152b01117ccf504aa75d742495743b134f2d328dd4725f19ee4e
*apache-brooklyn-0.9.0-rc3-vagrant.tar.gz
4a9cc996b827342c11176257bd1663ecb572913f2d3f3dc1f2d43b54be62aa52
*apache-brooklyn-0.9.0-rc3-vagrant.zip

The Nexus staging repositories for the Maven artifacts are located at:
https://repository.apache.org/content/repositories/orgapachebrooklyn-1018
https://repository.apache.org/content/repositories/orgapachebrooklyn-1019

All release artifacts are signed with the following key:
https://people.apache.org/keys/committer/richard.asc

KEYS file available here:
https://dist.apache.org/repos/dist/release/brooklyn/KEYS

The artifacts were built from these Git commit IDs:
brooklyn: acc8ff1930d243d2a5fae1ad2f1a1ef17ca4a19c
brooklyn-client: bc8593a933fcb76327ae4a511643e39d25a87ba2
brooklyn-dist: f08bddf201ab02a4f7066bfa3bcacf01bb430b99
brooklyn-docs: 12430d193e1891b87a677d6b45a3b17861c83518
brooklyn-library: 2565e6eb2868468ec2528df74fe85efdb887b6d2
brooklyn-server: 718aea7d27ce21835860d2de6a31d0e9474bee85
brooklyn-ui: 34d196678a8ae37804c698413d4ef19b86719a8f
All of the above have been tagged as "apache-brooklyn-0.9.0-rc3".


Please download the artifacts, test, and vote on releasing this
package as Apache Brooklyn 0.9.0.

The vote will be open for at least 72 hours.
[ ] +1 Release this package as Apache Brooklyn 0.9.0 (please describe
the tests you have performed)
[ ] +0 no opinion
[ ] -1 Do not release this package (please describe why not)

Thanks

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Richard Downer

Thanks Aled. I'm kicking off rc3 just now.

Richard.


On 6 April 2016 at 20:43, Aled Sage  wrote:
> I've reviewed and tested John's brooklyn-dist PR #31: looks good, so merged
> into master and 0.9.0.
>
> ---
> As discussed offline, in the vote email we'll want to include checksums of
> the individual brooklyn-client artifacts. This is because we want to include
> them as individually downloadable artifacts, so that someone visiting [1]
> can download the specific cli binary for their platform (rather than having
> to download the zip of all of them, unzip it, and then get the right
> artifact).
>
> Richard: over to you to make sure we comply with Apache for this, and to
> produce the next release candidate.
>
> Aled
>
> [1] http://brooklyn.apache.org/download/index.html
>
>
>
> On 06/04/2016 18:31, Aled Sage wrote:
>>
>> Thanks Sam,
>>
>> That is now cherry-picked into 0.9.0. Agree it is low-risk and worth
>> including.
>>
>> ---
>> Can someone please review and test John's
>> https://github.com/apache/brooklyn-dist/pull/31? It looks low risk and
>> simple.
>>
>> Aled
>>
>>
>> On 06/04/2016 17:29, Sam Corbett wrote:
>>>
>>> If I've not missed the cutoff I'd like
>>> https://github.com/apache/brooklyn-server/pull/99 to be included.
>>>
>>> On 6 April 2016 at 17:19, Richard Downer  wrote:
>>>
 On 6 April 2016 at 17:17, Aled Sage  wrote:
>
> I believe that Richard is discussing which Geoff about which

 brooklyn-client
>
> platform artifacts are required. Once that is agreed and sorted, then I
> think we are good to go.

 I've checked Geoff's PR and it's nice and simple: Mac 64bit, Windows
 and Linux 32bit. The 32bit executables execute on 64bit machines so
 there's no benefit to 64bit artifacts. To the best of my knowledge
 there has never been a 32-bit Intel-based Mac, and even if there was,
 it's obsolete.

 So it's ready to go IMO.

>>
>

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Aled Sage

I've reviewed and tested John's brooklyn-dist PR #31: looks good, so 
merged into master and 0.9.0.


---
As discussed offline, in the vote email we'll want to include checksums 
of the individual brooklyn-client artifacts. This is because we want to 
include them as individually downloadable artifacts, so that someone 
visiting [1] can download the specific cli binary for their platform 
(rather than having to download the zip of all of them, unzip it, and 
then get the right artifact).


Richard: over to you to make sure we comply with Apache for this, and to 
produce the next release candidate.


Aled

[1] http://brooklyn.apache.org/download/index.html


On 06/04/2016 18:31, Aled Sage wrote:

Thanks Sam,

That is now cherry-picked into 0.9.0. Agree it is low-risk and worth 
including.


---
Can someone please review and test John's 
https://github.com/apache/brooklyn-dist/pull/31? It looks low risk and 
simple.


Aled


On 06/04/2016 17:29, Sam Corbett wrote:

If I've not missed the cutoff I'd like
https://github.com/apache/brooklyn-server/pull/99 to be included.

On 6 April 2016 at 17:19, Richard Downer  wrote:


On 6 April 2016 at 17:17, Aled Sage  wrote:

I believe that Richard is discussing which Geoff about which

brooklyn-client
platform artifacts are required. Once that is agreed and sorted, 
then I

think we are good to go.

I've checked Geoff's PR and it's nice and simple: Mac 64bit, Windows
and Linux 32bit. The 32bit executables execute on 64bit machines so
there's no benefit to 64bit artifacts. To the best of my knowledge
there has never been a 32-bit Intel-based Mac, and even if there was,
it's obsolete.

So it's ready to go IMO.

[GitHub] brooklyn-dist pull request: use host only networking for brooklyn ...

2016-04-06 Thread asfgit

Github user asfgit closed the pull request at:

https://github.com/apache/brooklyn-dist/pull/31


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] brooklyn-dist pull request: use host only networking for brooklyn ...

Github user johnmccabe commented on a diff in the pull request:

https://github.com/apache/brooklyn-dist/pull/31#discussion_r58755945
  
--- Diff: vagrant/src/main/vagrant/servers.yaml ---
@@ -41,7 +41,9 @@ servers:
 box: ubuntu/wily64
 ram: 2048
 cpus: 4
-ip: 10.10.10.100
--- End diff --

As the doc update is to the website that shouldn't block release.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] brooklyn-dist pull request: use host only networking for brooklyn ...

Github user johnmccabe commented on a diff in the pull request:

https://github.com/apache/brooklyn-dist/pull/31#discussion_r58755772
  
--- Diff: vagrant/src/main/vagrant/servers.yaml ---
@@ -41,7 +41,9 @@ servers:
 box: ubuntu/wily64
 ram: 2048
 cpus: 4
-ip: 10.10.10.100
--- End diff --

@aledsage I've committed a change that enables `autocorrect`, vagrant will 
attempt to remap the port if it can - it doesn't work for me (I'll need to have 
a deeper look at why Vagrant refuses to map the port for me), instead throwing 
a warning that you're trying to bind to a port that's already in use - see the 
commit for more information on what happens.

I'm inclined to leave it as-is, and just fix the doc reference to the 
webconsole url 
[here](http://brooklyn.apache.org/v/0.9.0-SNAPSHOT/start/running.html#control-apache-brooklyn)


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] brooklyn-dist pull request: use host only networking for brooklyn ...

Github user johnmccabe commented on a diff in the pull request:

https://github.com/apache/brooklyn-dist/pull/31#discussion_r58755124
  
--- Diff: vagrant/src/main/vagrant/servers.yaml ---
@@ -41,7 +41,9 @@ servers:
 box: ubuntu/wily64
 ram: 2048
 cpus: 4
-ip: 10.10.10.100
--- End diff --

Omitting the IP is what switches to host only networking, if you specify an 
address here it results in Virtualbox creating a new virtual subnet to put the 
guest on.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] brooklyn-dist pull request: use host only networking for brooklyn ...

Github user johnmccabe commented on a diff in the pull request:

https://github.com/apache/brooklyn-dist/pull/31#discussion_r58748964
  
--- Diff: vagrant/src/main/vagrant/servers.yaml ---
@@ -41,7 +41,9 @@ servers:
 box: ubuntu/wily64
 ram: 2048
 cpus: 4
-ip: 10.10.10.100
+forwarded_ports:
+ - guest: 8081
+   host: 8081
--- End diff --

If it's a process running in userspace then it will override the mapped 
port which will become accessible if the overriding process is halted.
Give me a second and I'll check the output from `:auto_correct`


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] brooklyn-dist pull request: use host only networking for brooklyn ...

2016-04-06 Thread aledsage

Github user aledsage commented on a diff in the pull request:

https://github.com/apache/brooklyn-dist/pull/31#discussion_r58748556
  
--- Diff: vagrant/src/main/vagrant/servers.yaml ---
@@ -41,7 +41,9 @@ servers:
 box: ubuntu/wily64
 ram: 2048
 cpus: 4
-ip: 10.10.10.100
+forwarded_ports:
+ - guest: 8081
+   host: 8081
--- End diff --

If someone has another process listening on port 8081, then presumably 
`vagrant up brooklyn` will fail with a nice message? Is that ok to assume they 
won't have anything else running there? I think yes.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Aled Sage


Thanks Sam,

That is now cherry-picked into 0.9.0. Agree it is low-risk and worth 
including.


---
Can someone please review and test John's 
https://github.com/apache/brooklyn-dist/pull/31? It looks low risk and 
simple.


Aled


On 06/04/2016 17:29, Sam Corbett wrote:

If I've not missed the cutoff I'd like
https://github.com/apache/brooklyn-server/pull/99 to be included.

On 6 April 2016 at 17:19, Richard Downer  wrote:


On 6 April 2016 at 17:17, Aled Sage  wrote:

I believe that Richard is discussing which Geoff about which

brooklyn-client

platform artifacts are required. Once that is agreed and sorted, then I
think we are good to go.

I've checked Geoff's PR and it's nice and simple: Mac 64bit, Windows
and Linux 32bit. The 32bit executables execute on 64bit machines so
there's no benefit to 64bit artifacts. To the best of my knowledge
there has never been a 32-bit Intel-based Mac, and even if there was,
it's obsolete.

So it's ready to go IMO.

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread John McCabe

Likewise can https://github.com/apache/brooklyn-dist/pull/31 be considered
for inclusion.
/John
On Wed, 6 Apr 2016 at 17:29 Sam Corbett 
wrote:

> If I've not missed the cutoff I'd like
> https://github.com/apache/brooklyn-server/pull/99 to be included.
>
> On 6 April 2016 at 17:19, Richard Downer  wrote:
>
> > On 6 April 2016 at 17:17, Aled Sage  wrote:
> > > I believe that Richard is discussing which Geoff about which
> > brooklyn-client
> > > platform artifacts are required. Once that is agreed and sorted, then I
> > > think we are good to go.
> >
> > I've checked Geoff's PR and it's nice and simple: Mac 64bit, Windows
> > and Linux 32bit. The 32bit executables execute on 64bit machines so
> > there's no benefit to 64bit artifacts. To the best of my knowledge
> > there has never been a 32-bit Intel-based Mac, and even if there was,
> > it's obsolete.
> >
> > So it's ready to go IMO.
> >
>

[GitHub] brooklyn-dist pull request: use host only networking for brooklyn ...

Github user johnmccabe commented on the pull request:

https://github.com/apache/brooklyn-dist/pull/31#issuecomment-206465976
  
fyi @neykov this is related to you comment about the web console url not 
being correct for Vagrant.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] brooklyn-ui pull request: Update editor placeholder

2016-04-06 Thread tbouron

Github user tbouron commented on the pull request:

https://github.com/apache/brooklyn-ui/pull/24#issuecomment-206456324
  
We will probably have to cheat as the textarea is hidden by CodeMirror. 
AFAIK, we can still bound to those events but from the CodeMirror container 
rather than the textarea.

I'll investigate more tomorrow


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] brooklyn-ui pull request: Update editor placeholder

2016-04-06 Thread m4rkmckenna

Github user m4rkmckenna commented on the pull request:

https://github.com/apache/brooklyn-ui/pull/24#issuecomment-206451829
  
My thinking was binding to the dragenter / dragover / dragleave events from 
the textarea assuming the codemirror code propagates said events.

Then adding / removing a class from the textarea OR changing the drag icon 


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[jira] [Resolved] (BROOKLYN-173) WinRM executeScript failed: httplib.BadStatusLine: ''


 [ 
https://issues.apache.org/jira/browse/BROOKLYN-173?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Aled Sage resolved BROOKLYN-173.

   Resolution: Fixed
Fix Version/s: 0.9.0

Has not been seen since we upgraded to the new winrm4j (that does not use 
pywinrm).

> WinRM executeScript failed: httplib.BadStatusLine: ''
> -
>
> Key: BROOKLYN-173
> URL: https://issues.apache.org/jira/browse/BROOKLYN-173
> Project: Brooklyn
>  Issue Type: Bug
>Affects Versions: 0.9.0
>Reporter: Aled Sage
> Fix For: 0.9.0
>
>
> When deploying a simple Windows 7zip app to a vcloud-director environment 
> (over NAT), the install command failed: {{httplib.BadStatusLine: ''}}
> After this error, I tested executing another batch command over the same 
> WinrmMachineLocation+VanillaWindowsProcess, and that worked:
> {noformat}
> mgmt.getEntityManager().getEntity("wV0Kbv2T").getLocations()[0].executeScript("powershell
>  -command \"C:install7zip.ps1\"").getStdOut()
> {noformat}
> The error was:
> {noformat}
> Failed after 44.2s: failed to execute shell script, 10 errors including: 
> Traceback (most recent call last):
>   File "__pyclasspath__/winrm/__init__.py", line 42, in run_cmd
>   File "__pyclasspath__/winrm/protocol.py", line 310, in 
> get_command_output
>   File "__pyclasspath__/winrm/protocol.py", line 328, in 
> _raw_get_command_output
>   File "__pyclasspath__/winrm/protocol.py", line 190, in send_message
>   File "__pyclasspath__/winrm/transport.py", line 82, in send_message
>   File "__pyclasspath__/winrm/transport.py", line 82, in send_message
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/urllib2.py",
>  line 127, in urlopen
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/urllib2.py",
>  line 410, in open
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/urllib2.py",
>  line 522, in http_response
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/urllib2.py",
>  line 442, in error
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/urllib2.py",
>  line 382, in _call_chain
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/urllib2.py",
>  line 896, in http_error_401
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/urllib2.py",
>  line 872, in http_error_auth_reqed
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/urllib2.py",
>  line 885, in retry_http_basic_auth
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/urllib2.py",
>  line 404, in open
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/urllib2.py",
>  line 421, in _open
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/urllib2.py",
>  line 382, in _call_chain
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/urllib2.py",
>  line 1214, in http_open
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/urllib2.py",
>  line 1187, in do_open
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/urllib2.py",
>  line 1187, in do_open
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/httplib.py",
>  line 1045, in getresponse
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/httplib.py",
>  line 409, in begin
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/httplib.py",
>  line 373, in _read_status
> httplib.BadStatusLine: ''
> org.apache.brooklyn.util.exceptions.CompoundRuntimeException: failed to 
> execute shell script, 10 errors including: Traceback (most recent call last):
>   File "__pyclasspath__/winrm/__init__.py", line 42, in run_cmd
>   File "__pyclasspath__/winrm/protocol.py", line 310, in 
> get_command_output
>   File "__pyclasspath__/winrm/protocol.py", line 328, in 
> _raw_get_command_output
>   File "__pyclasspath__/winrm/protocol.py", line 190, in send_message
>   File "__pyclasspath__/winrm/transport.py", line 82, in send_message
>   File "__pyclasspath__/winrm/transport.py", line 82, in send_message
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/urllib2.py",
>  line 127, in urlopen
>   File 
> "/home/amp/cloudsoft-amp-3.0.0-rc2/lib/amp/jython-standalone-2.7-b3.jar/Lib/urllib2.py",
>  line 410, in open
>   File 
>

Jenkins build is back to normal : brooklyn-ui-master #10

See

[jira] [Resolved] (BROOKLYN-249) errors using external config for JcloudsLocation


 [ 
https://issues.apache.org/jira/browse/BROOKLYN-249?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Aled Sage resolved BROOKLYN-249.

   Resolution: Fixed
 Assignee: Aled Sage
Fix Version/s: 0.9.0

> errors using external config for JcloudsLocation
> 
>
> Key: BROOKLYN-249
> URL: https://issues.apache.org/jira/browse/BROOKLYN-249
> Project: Brooklyn
>  Issue Type: Bug
>Reporter: Aled Sage
>Assignee: Aled Sage
> Fix For: 0.9.0
>
>
> Using 0.10.0-SNAPSHOT (also affecting 0.9.0-rc2), I hit problems using 
> external config with JcloudsLocation.
> First it failed to resolve the external config for the ssh publicKeyData and 
> privateKeyData.
> After fixing that, it failed to rebind to my persisted state that included 
> the external config in the location (it tried to coerce the 
> {{DeferredSupplier}} to a String).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Resolved] (BROOKLYN-246) Unable to add byon locations


 [ 
https://issues.apache.org/jira/browse/BROOKLYN-246?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Aled Sage resolved BROOKLYN-246.

   Resolution: Fixed
Fix Version/s: 0.9.0

> Unable to add byon locations
> 
>
> Key: BROOKLYN-246
> URL: https://issues.apache.org/jira/browse/BROOKLYN-246
> Project: Brooklyn
>  Issue Type: Bug
>Affects Versions: 0.9.0
>Reporter: John McCabe
>Priority: Blocker
> Fix For: 0.9.0
>
>
> Running both the most recent 0.9.0-SNAPSHOT dist from maven and a freshly 
> build 0.10.0-SNAPSHOT from master I'm seeing the following behaviour.
> 1. Attempting to add a BYON location via jsgui is rejected with the following 
> error:
> {code}
> Could not resolve item 'testingid': Transformer for Brooklyn OASIS CAMP 
> interpreter gave an error creating this plan: No class or resolver found for 
> location type byon
> {code}
> 2. Trying the the generated JSON via the composer also fails:
> {code}
> brooklyn.catalog:
>   version: 0.0.1
>   items:
>   - id: testingid
> itemType: location
> item:
>   type: byon
>   brooklyn.config:
> displayName: testingdisplayName
> user: testinguser
> password: testingpassword
> privateKeyFile: testingprivateKeyFile
> privateKeyPassphrase: testingprivateKeyPassphrase
> hosts:
> - 10.10.10.102
> {code}
> 3. *BUT* this works:
> {code}
> brooklyn.catalog:
>   version: 0.0.1
>   items:
>   - id: testingid
> itemType: location
> item:
>   type: byon:(hosts="10.10.10.102")
>   brooklyn.config:
> displayName: testingdisplayName
> user: testinguser
> password: testingpassword
> privateKeyFile: testingprivateKeyFile
> privateKeyPassphrase: testingprivateKeyPassphrase
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Resolved] (BROOKLYN-248) Incorrect name in derived locations


 [ 
https://issues.apache.org/jira/browse/BROOKLYN-248?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Aled Sage resolved BROOKLYN-248.

   Resolution: Fixed
 Assignee: Aled Sage
Fix Version/s: 0.9.0

> Incorrect name in derived locations
> ---
>
> Key: BROOKLYN-248
> URL: https://issues.apache.org/jira/browse/BROOKLYN-248
> Project: Brooklyn
>  Issue Type: Bug
>Affects Versions: 0.9.0
> Environment: OSX
>Reporter: Duncan Godwin
>Assignee: Aled Sage
> Fix For: 0.9.0
>
>
> The following location displays "Test Location 1" for both "Test Location 1" 
> and "Test Location 2" when you use it to launch an application.
> See the screenshot here: 
> https://s3.amazonaws.com/drigodwin-misc/brooklyn-248-ss.png
> {code}
> brooklyn.catalog:
>   version: 0.2.0
>   items:
>   - id: test-location-1
> name: "Test Location 1"
> itemType: location
> item:
>   type: byon:(hosts="10.10.10.102")
>   brooklyn.config:
> displayName: testingdisplayName
> user: testinguser
> password: testingpassword
> privateKeyFile: testingprivateKeyFile
> privateKeyPassphrase: testingprivateKeyPassphrase
>   - id: test-location-2
> name: "Test Location 2"
> itemType: location
> item:
>   type: test-location-1
>   brooklyn.config:
> displayName: testingdisplayName2
> user: testinguser2
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[GitHub] brooklyn-ui pull request: Update editor placeholder

2016-04-06 Thread tbouron

Github user tbouron commented on the pull request:

https://github.com/apache/brooklyn-ui/pull/24#issuecomment-206447871
  
The drag & drop feature is provided by CodeMirror. I'm not sure it can be 
customised, I'll need to check that.

That being said, the content of the file is inserted where your cursor / 
mouse cursor drop the file, so having a classic overlay "drop file here" won't 
work here.



---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Richard Downer

On 6 April 2016 at 17:17, Aled Sage  wrote:
> I believe that Richard is discussing which Geoff about which brooklyn-client
> platform artifacts are required. Once that is agreed and sorted, then I
> think we are good to go.

I've checked Geoff's PR and it's nice and simple: Mac 64bit, Windows
and Linux 32bit. The 32bit executables execute on 64bit machines so
there's no benefit to 64bit artifacts. To the best of my knowledge
there has never been a 32-bit Intel-based Mac, and even if there was,
it's obsolete.

So it's ready to go IMO.

[jira] [Resolved] (BROOKLYN-245) Deadlock from ControlledDynamicWebAppCluster, plus thread modifying attribute


 [ 
https://issues.apache.org/jira/browse/BROOKLYN-245?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Aled Sage resolved BROOKLYN-245.

   Resolution: Fixed
 Assignee: Aled Sage
Fix Version/s: 0.9.0

> Deadlock from ControlledDynamicWebAppCluster, plus thread modifying attribute
> -
>
> Key: BROOKLYN-245
> URL: https://issues.apache.org/jira/browse/BROOKLYN-245
> Project: Brooklyn
>  Issue Type: Bug
>Reporter: Aled Sage
>Assignee: Aled Sage
> Fix For: 0.9.0
>
>
> With master (i.e. would also affect 0.9.0)...
> When running {{org.apache.brooklyn.qa.load.LoadTest}} several times, I hit 
> the deadlock below.
> The test deploys several apps concurrently, each using 
> {{ControlledDynamicWebAppCluster}} (via the Java-app 
> {{SimulatedTheeTierApp}}). This deadlock could happen in other uses of 
> {{ControlledDynamicWebAppCluster}}, or in other entities depending if/when 
> they synchronize on the entity themselves.
> {noformat}
> Java stack information for the threads listed above:
> ===
> "brooklyn-execmanager-H7Adif6C-157":
> at 
> org.apache.brooklyn.core.sensor.AttributeMap.asMap(AttributeMap.java:92)
> - waiting to lock <0x0007d3504b60> (a 
> java.util.Collections$SynchronizedMap)
> at 
> org.apache.brooklyn.core.entity.AbstractEntity$BasicSensorSupport.getAll(AbstractEntity.java:1129)
> at 
> org.apache.brooklyn.core.entity.AbstractEntity.getAllAttributes(AbstractEntity.java:1003)
> at 
> org.apache.brooklyn.core.mgmt.rebind.dto.MementosGenerators.newEntityMementoBuilder(MementosGenerators.java:194)
> at 
> org.apache.brooklyn.core.mgmt.rebind.BasicEntityRebindSupport.getMementoWithProperties(BasicEntityRebindSupport.java:69)
> at 
> org.apache.brooklyn.core.mgmt.rebind.BasicEntityRebindSupport.getMemento(BasicEntityRebindSupport.java:61)
> at 
> org.apache.brooklyn.core.mgmt.rebind.BasicEntityRebindSupport.getMemento(BasicEntityRebindSupport.java:1)
> at 
> org.apache.brooklyn.core.mgmt.rebind.PeriodicDeltaChangeListener.persistNowInternal(PeriodicDeltaChangeListener.java:417)
> at 
> org.apache.brooklyn.core.mgmt.rebind.PeriodicDeltaChangeListener.persistNowSafely(PeriodicDeltaChangeListener.java:352)
> at 
> org.apache.brooklyn.core.mgmt.rebind.PeriodicDeltaChangeListener.persistNowSafely(PeriodicDeltaChangeListener.java:346)
> at 
> org.apache.brooklyn.core.mgmt.rebind.PeriodicDeltaChangeListener$1$1.call(PeriodicDeltaChangeListener.java:215)
> at 
> org.apache.brooklyn.core.mgmt.rebind.PeriodicDeltaChangeListener$1$1.call(PeriodicDeltaChangeListener.java:1)
> at 
> org.apache.brooklyn.util.core.task.BasicExecutionManager$ScheduledTaskCallable$1.call(BasicExecutionManager.java:436)
> at 
> org.apache.brooklyn.util.core.task.BasicExecutionManager$SubmissionCallable.call(BasicExecutionManager.java:519)
> at java.util.concurrent.FutureTask.run(FutureTask.java:262)
> at 
> java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
> at 
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
> at java.lang.Thread.run(Thread.java:745)
> "brooklyn-execmanager-H7Adif6C-16":
> at 
> org.apache.brooklyn.core.entity.AbstractEntity$BasicSubscriptionSupport.getSubscriptionContext(AbstractEntity.java:1488)
> - waiting to lock <0x0007d34abc20> (a 
> org.apache.brooklyn.entity.webapp.ControlledDynamicWebAppClusterImpl)
> at 
> org.apache.brooklyn.core.entity.AbstractEntity$BasicSensorSupport.emitInternal(AbstractEntity.java:1164)
> at 
> org.apache.brooklyn.core.entity.AbstractEntity.emitInternal(AbstractEntity.java:2021)
> at 
> org.apache.brooklyn.core.sensor.AttributeMap.update(AttributeMap.java:133)
> at 
> org.apache.brooklyn.core.sensor.AttributeMap.modify(AttributeMap.java:162)
> - locked <0x0007d3504b60> (a 
> java.util.Collections$SynchronizedMap)
> at 
> org.apache.brooklyn.core.entity.AbstractEntity$BasicSensorSupport.modify(AbstractEntity.java:1106)
> at 
> org.apache.brooklyn.core.entity.lifecycle.ServiceStateLogic.updateMapSensorEntry(ServiceStateLogic.java:150)
> at 
> org.apache.brooklyn.core.entity.lifecycle.ServiceStateLogic.clearMapSensorEntry(ServiceStateLogic.java:102)
> at 
> org.apache.brooklyn.core.entity.lifecycle.ServiceStateLogic$ComputeServiceIndicatorsFromChildrenAndMembers.updateMapSensor(ServiceStateLogic.java:558)
> at 
> org.apache.brooklyn.core.entity.lifecycle.ServiceStateLogic$ComputeServiceIndicatorsFromChildrenAndMembers.onUpdated(ServiceStateLogic.java:477)
> at 
>

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


 [ 
https://issues.apache.org/jira/browse/BROOKLYN-244?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Aled Sage resolved BROOKLYN-244.

   Resolution: Fixed
Fix Version/s: 0.9.0

> Blueprint Composer: text disappears after few seconds
> -
>
> Key: BROOKLYN-244
> URL: https://issues.apache.org/jira/browse/BROOKLYN-244
> Project: Brooklyn
>  Issue Type: Bug
>Reporter: Aled Sage
> Fix For: 0.9.0
>
>
> On Brooklyn 0.9.0-SNAPSHOT...
> When I open the Blueprint Composer and start typing (particularly if I've 
> pasted a blueprint), then sometimes a few seconds later the text all 
> disappears leaving me with a blank text area.
> I suspect (talking briefly to Thomas) that it could be refreshing after the 
> catalog has finished loading - the longer the catalog takes to be retrieved, 
> then the longer the delay before any text is lost.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Aled Sage

Thanks Mark for reviewing brooklyn-ui PR #24. That is now merged into
master and cherry-picked into 0.9.0.
(I completely agree with Mark's comments in #24, which can be addressed
in a future PR just in master).

---
I believe that Richard is discussing which Geoff about which
brooklyn-client platform artifacts are required. Once that is agreed and
sorted, then I think we are good to go.

Last chance before the RC: any more bugs that need addressed or PRs that
you think need merged/cherry-picked?

Aled

On 06/04/2016 17:03, Aled Sage wrote:

Thanks Thomas.

I've cherry-picked PR #23.

---
The change in #24 looks sensible (without it, no-one would know that
drag-and-drop of text-files is possible in the Blueprint Composer).

Can someone please review and comment on the PR, and let us know if
you agree it's low-risk for including in 0.9.0.

Aled

On 06/04/2016 16:13, Thomas Bouron wrote:

I also have two candidates to include:
https://github.com/apache/brooklyn-ui/pull/23 and
https://github.com/apache/brooklyn-ui/pull/24

Thoughts?

On Wed, 6 Apr 2016 at 15:51 Geoff Macartney <
geoff.macart...@cloudsoftcorp.com> wrote:

Would it be worth including CLI Go vendoring -
https://github.com/apache/brooklyn-client/pull/16? <
https://github.com/apache/brooklyn-client/pull/16?> Simplifies the
build

for the CLI (no need for extra tool apart from Go itself).

Gnu PGP key - http://is.gd/uI

On 6 Apr 2016, at 15:47, Aled Sage wrote:

Hi all,

Svet: agree with brooklyn-client/pull/14 and brooklyn-client/pull/15.

I've cherry-picked those.

We have reviewed and incorporated the following fixes into master and

the 0.9.0 branch, ready for the next release candidate:

* BROOKLYN-249:
https://github.com/apache/brooklyn-server/pull/103
* BROOKLYN-248:
https://github.com/apache/brooklyn-server/pull/104
* BROOKLYN-246:
https://github.com/apache/brooklyn-server/pull/102
* https://github.com/apache/brooklyn-client/pull/14
* https://github.com/apache/brooklyn-client/pull/15

Anything else that needs done before the next 0.9.0 release candidate?

Aled

On 06/04/2016 09:45, Svetoslav Neykov wrote:

+1 to include 102 & 103.

Also suggest including [1] - cuts down on the number of cli binaries
substantially and fixes the name on Windows. Shaves off close to
20mb from

the release. [2] is minor but good to have (documents the go 1.6
requirement).

Svet.

[1] https://github.com/apache/brooklyn-client/pull/14 <

https://github.com/apache/brooklyn-client/pull/14>

[2] https://github.com/apache/brooklyn-client/pull/15 <

https://github.com/apache/brooklyn-client/pull/15>

On 5.04.2016 г., at 20:10, Aled Sage wrote:

Thanks all.

John has fixed this [1]. I'll review, merge, and cherry-pick to the

0.9.0 branch.

Given we are producing another release candidate anyway, can we

include the fix for BROOKLYN-249 (see [2,3]). Can someone please review
that and give their opinion for whether it should be included in the
0.9.0

release?

Aled

[1] https://github.com/apache/brooklyn-server/pull/102
[2] https://issues.apache.org/jira/browse/BROOKLYN-249
[3] https://github.com/apache/brooklyn-server/pull/103

On 05/04/2016 10:30, John McCabe wrote:

Note that it also breaks the add location wizard in jsgui.

On Tue, 5 Apr 2016 at 10:26 Richard Downer

wrote:
After discussing with John, I agree that BROOKLYN-246 is a
blocker.

The catalog file that the Brooklyn Vagrant distributable uses is

using

the broken format, so it won't work - and we really don't want our
simplest-possible-get-started to require manual modifications
before
it'll work. It also breaks the documented way of using BYON[1]
so we

are at high risk of breaking user's working configurations.

Therefore due to this critical bug, I am cancelling the vote.

Richard.

[1]

https://brooklyn.apache.org/v/latest/ops/locations/index.html#byon

On 4 April 2016 at 18:11, John McCabe wrote:

@all, I hit an issue with location handling while following up on

@svets

comments - https://issues.apache.org/jira/browse/BROOKLYN-246

Looks like a blocker.

On Mon, 4 Apr 2016 at 14:35 John McCabe

wrote:

@svet, latest isn't required, 1.8+ should suffice.

On Mon, 4 Apr 2016 at 12:53 Svetoslav Neykov <
svetoslav.ney...@cloudsoftcorp.com> wrote:

+1 binding

* Tested the .zip & .tar.gz dists
* Tested CLI
* Tried a simple blueprint
* Tested the vagrant artifact - "vagrant up" *fails* because
it's

looking

at the release url

https://www.apache.org/dyn/closer.lua?action=download=brooklyn/apache-brooklyn-0.9.0/apache-brooklyn-0.9.0-bin.tar.gz

Once released should work.
Unrelated, but do we require the absolute latest vagrant

version on

purpose? Can we relax this version requirement?

Svet.

[GitHub] brooklyn-ui pull request: Update editor placeholder

2016-04-06 Thread asfgit

Github user asfgit closed the pull request at:

https://github.com/apache/brooklyn-ui/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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] brooklyn-ui pull request: Update editor placeholder

2016-04-06 Thread m4rkmckenna

Github user m4rkmckenna commented on the pull request:

https://github.com/apache/brooklyn-ui/pull/24#issuecomment-206444894
  
LGTM

**MINOR**
* Might be worth adding styling for dragover
* Might be worth verifying the type of the dropped file


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: [CANCEL][VOTE] Release Apache Brooklyn 0.9.0 [rc2]

2016-04-06 Thread Geoff Macartney

That's fine by me.

cheers
Geoff



Gnu PGP key - http://is.gd/uI


> On 6 Apr 2016, at 16:58, Aled Sage  wrote:
> 
> Hi Geoff,
> 
> I think we should *not* include it in the 0.9.0 release. It improves the 
> build process, but does not change the final artifacts. I think for rc3 we 
> should stick to just cherry-picking bug fixes.
> 
> If anyone disagrees then please shout out.
> 
> Aled
> 
> 
> On 06/04/2016 15:50, Geoff Macartney wrote:
>> Would it be worth including CLI Go vendoring - 
>> https://github.com/apache/brooklyn-client/pull/16? 
>>   Simplifies the build 
>> for the CLI (no need for extra tool apart from Go itself).
>> 
>> 
>> 
>> Gnu PGP key - http://is.gd/uI
>> 
>> 
>>> On 6 Apr 2016, at 15:47, Aled Sage  wrote:
>>> 
>>> Hi all,
>>> 
>>> Svet: agree with brooklyn-client/pull/14 and brooklyn-client/pull/15. I've 
>>> cherry-picked those.
>>> 
>>> We have reviewed and incorporated the following fixes into master and the 
>>> 0.9.0 branch, ready for the next release candidate:
>>> 
>>> * BROOKLYN-249:
>>>   https://github.com/apache/brooklyn-server/pull/103
>>> * BROOKLYN-248:
>>>   https://github.com/apache/brooklyn-server/pull/104
>>> * BROOKLYN-246:
>>>   https://github.com/apache/brooklyn-server/pull/102
>>> * https://github.com/apache/brooklyn-client/pull/14
>>> * https://github.com/apache/brooklyn-client/pull/15
>>> 
>>> Anything else that needs done before the next 0.9.0 release candidate?
>>> 
>>> Aled
>>> 
>>> 
>>> On 06/04/2016 09:45, Svetoslav Neykov wrote:
 +1 to include 102 & 103.
 
 Also suggest including [1] - cuts down on the number of cli binaries 
 substantially and fixes the name on Windows. Shaves off close to 20mb from 
 the release. [2] is minor but good to have (documents the go 1.6 
 requirement).
 
 Svet.
 
 
 [1] https://github.com/apache/brooklyn-client/pull/14 
 
 [2] https://github.com/apache/brooklyn-client/pull/15 
 
 
 
 
> On 5.04.2016 г., at 20:10, Aled Sage  wrote:
> 
> Thanks all.
> 
> John has fixed this [1]. I'll review, merge, and cherry-pick to the 0.9.0 
> branch.
> 
> Given we are producing another release candidate anyway, can we include 
> the fix for BROOKLYN-249 (see [2,3]). Can someone please review that and 
> give their opinion for whether it should be included in the 0.9.0 release?
> 
> Aled
> 
> [1] https://github.com/apache/brooklyn-server/pull/102
> [2] https://issues.apache.org/jira/browse/BROOKLYN-249
> [3] https://github.com/apache/brooklyn-server/pull/103
> 
> 
> On 05/04/2016 10:30, John McCabe wrote:
>> Note that it also breaks the add location wizard in jsgui.
>> 
>> On Tue, 5 Apr 2016 at 10:26 Richard Downer  wrote:
>> 
>>> After discussing with John, I agree that BROOKLYN-246 is a blocker.
>>> The catalog file that the Brooklyn Vagrant distributable uses is using
>>> the broken format, so it won't work - and we really don't want our
>>> simplest-possible-get-started to require manual modifications before
>>> it'll work. It also breaks the documented way of using BYON[1] so we
>>> are at high risk of breaking user's working configurations.
>>> 
>>> Therefore due to this critical bug, I am cancelling the vote.
>>> 
>>> Richard.
>>> 
>>> [1]https://brooklyn.apache.org/v/latest/ops/locations/index.html#byon
>>> 
>>> On 4 April 2016 at 18:11, John McCabe  wrote:
 @all, I hit an issue with location handling while following up on 
 @svets
 comments - https://issues.apache.org/jira/browse/BROOKLYN-246
 
 Looks like a blocker.
 
 On Mon, 4 Apr 2016 at 14:35 John McCabe  wrote:
 
> @svet, latest isn't required, 1.8+ should suffice.
> 
> On Mon, 4 Apr 2016 at 12:53 Svetoslav Neykov <
> svetoslav.ney...@cloudsoftcorp.com> wrote:
> 
>> +1 binding
>> 
>> * Tested the .zip & .tar.gz dists
>> * Tested CLI
>> * Tried a simple blueprint
>> * Tested the vagrant artifact - "vagrant up" *fails* because it's
>>> looking
>> at the release url
>> 
>>> https://www.apache.org/dyn/closer.lua?action=download=brooklyn/apache-brooklyn-0.9.0/apache-brooklyn-0.9.0-bin.tar.gz
>> <
>> 
>>> https://www.apache.org/dyn/closer.lua?action=download=brooklyn/apache-brooklyn-0.9.0/apache-brooklyn-0.9.0-bin.tar.gz
 .
>> Once released should work.
>>Unrelated, but do we require the absolute latest vagrant version

[GitHub] brooklyn-server pull request: Support removal of catalog entries f...

2016-04-06 Thread geomacy

Github user geomacy commented on a diff in the pull request:

https://github.com/apache/brooklyn-server/pull/100#discussion_r58729416
  
--- Diff: 
core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogBomScanner.java
 ---
@@ -100,54 +103,82 @@ public ManagementContext getManagementContext() {
 return managementContext;
 }
 
+/**
+ * Scans the bundle being added for a catalog.bom file and adds 
any entries in it to the catalog.
+ *
+ * @param bundle The bundle being added to the bundle context.
+ * @param bundleEvent The event of the addition.
+ *
+ * @return The items added to the catalog; these will be tracked 
by the {@link BundleTracker} mechanism
+ * and supplied to the {@link #removedBundle(Bundle, 
BundleEvent, Iterable)} method.
+ */
 @Override
-public Long addingBundle(Bundle bundle, BundleEvent bundleEvent) {
+public Iterable> addingBundle(Bundle 
bundle, BundleEvent bundleEvent) {
+return scanForCatalog(bundle);
+}
 
-final BundleContext bundleContext = 
FrameworkUtil.getBundle(CatalogBomScanner.class).getBundleContext();
-if (bundleContext == null) {
-LOG.info("Bundle context not yet established for bundle {} 
{} {}", bundleIds(bundle));
-return null;
-}
 
-scanForCatalog(bundle);
+@Override
+public void removedBundle(Bundle bundle, BundleEvent bundleEvent, 
Iterable> items) {
+if (!items.iterator().hasNext()) {
+return;
+}
+LOG.debug("Unloading catalog BOM entries from {} {} {}", 
bundleIds(bundle));
+List exceptions = MutableList.of();
+final BrooklynCatalog catalog = 
getManagementContext().getCatalog();
+for (CatalogItem item : items) {
+LOG.debug("Unloading {} {} from catalog", 
item.getSymbolicName(), item.getVersion());
+
+try {
+catalog.deleteCatalogItem(item.getSymbolicName(), 
item.getVersion());
+} catch (Exception e) {
+LOG.warn("Caught {} unloading {} {} from catalog", new 
String [] {
+e.getMessage(), item.getSymbolicName(), 
item.getVersion()
+});
+exceptions.add(e);
+}
+}
 
-return bundle.getBundleId();
+if (0 < exceptions.size()) {
+throw new CompoundRuntimeException(
--- End diff --

Hm, I guess I have been assuming that this will just result in a message 
being logged by Karaf as opposed to it causing a failure - I will have to test 
that out.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] brooklyn-client pull request: Use Go 1.6 vendoring instead of Gode...

2016-04-06 Thread m4rkmckenna

Github user m4rkmckenna commented on the pull request:

https://github.com/apache/brooklyn-client/pull/16#issuecomment-206434349
  
LGTM :: Warnings are gone :+1:


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Build failed in Jenkins: brooklyn-ui-master #9