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

[Apache Jenkins Server]

See 

--
[...truncated 3.23 MB...]
2017-04-21 01:01:31,940 INFO  TESTNG INVOKING: "Surefire test" - 
org.apache.brooklyn.core.mgmt.persist.DeserializingClassRenamesProviderTest.testRenameNoopIfNotInProperties()
2017-04-21 01:01:31,940 INFO  TESTNG PASSED: "Surefire test" - 
org.apache.brooklyn.core.mgmt.persist.DeserializingClassRenamesProviderTest.testRenameNoopIfNotInProperties()
 finished in 0 ms
2017-04-21 01:01:32,288 INFO  TESTNG 
===
Surefire test
Tests run: 2132, Failures: 1, Skips: 0
===
[GC (Allocation Failure)  579246K->354309K(767488K), 0.0183574 secs]
[GC (Allocation Failure)  579077K->354046K(766976K), 0.0139297 secs]
[GC (Allocation Failure)  578302K->354184K(767488K), 0.0097567 secs]
[GC (Allocation Failure)  578440K->354097K(767488K), 0.0122674 secs]
[GC (Allocation Failure)  578353K->355143K(767488K), 0.0184880 secs]
Tests run: 2132, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 225.114 sec 
<<< FAILURE! - in TestSuite
testGroupWithMatchingFilterReturnsEverythingThatMatches(org.apache.brooklyn.entity.group.SequenceGroupTest)
  Time elapsed: 0.142 sec  <<< FAILURE!
java.lang.AssertionError: entity=Application[jcr7md78]; attribute=Sensor: 
sequence.value (java.lang.Integer) expected [1] but found [2]
at 
org.apache.brooklyn.entity.group.SequenceGroupTest.testGroupWithMatchingFilterReturnsEverythingThatMatches(SequenceGroupTest.java:144)

2017-04-21 01:01:34,607 INFO  Brooklyn shutdown: stopping entities 
[TestEntityImpl{id=xp2sy31nl7}, TestEntityImpl{id=xeg7s9fds6}, 
BlockingEntityImpl{id=hc0nso6ixs}]

Results :

Failed tests: 
  SequenceGroupTest.testGroupWithMatchingFilterReturnsEverythingThatMatches:144 
entity=Application[jcr7md78]; attribute=Sensor: sequence.value 
(java.lang.Integer) expected [1] but found [2]

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

[JENKINS] Recording test results
[INFO] 
[INFO] Reactor Summary:
[INFO] 
[INFO] Brooklyn REST JavaScript Web GUI ... SUCCESS [01:37 min]
[INFO] Brooklyn Server Root ... SUCCESS [  1.392 s]
[INFO] Brooklyn Parent Project  SUCCESS [  4.706 s]
[INFO] Brooklyn Test Support Utilities  SUCCESS [  6.920 s]
[INFO] Brooklyn Logback Includable Configuration .. SUCCESS [  4.115 s]
[INFO] Brooklyn Common Utilities .. SUCCESS [ 27.358 s]
[INFO] Brooklyn API ... SUCCESS [  4.952 s]
[INFO] CAMP Server Parent Project . SUCCESS [  2.616 s]
[INFO] CAMP Base .. SUCCESS [  6.454 s]
[INFO] Brooklyn Test Support .. SUCCESS [  4.565 s]
[INFO] Brooklyn REST Swagger Apidoc Utilities . SUCCESS [  4.457 s]
[INFO] Brooklyn Logback Configuration . SUCCESS [  3.875 s]
[INFO] CAMP Server  SUCCESS [  8.520 s]
[INFO] Brooklyn Felix Runtime . SUCCESS [  7.650 s]
[INFO] Brooklyn Groovy Utilities .. SUCCESS [ 10.643 s]
[INFO] Brooklyn Core .. FAILURE [04:14 min]
[INFO] Brooklyn Policies .. SKIPPED
[INFO] Brooklyn WinRM Software Entities ... SKIPPED
[INFO] Brooklyn Secure JMXMP Agent  SKIPPED
[INFO] Brooklyn JMX RMI Agent . SKIPPED
[INFO] Brooklyn Jclouds Location Targets .. SKIPPED
[INFO] Brooklyn Software Base . SKIPPED
[INFO] Brooklyn CAMP .. 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 Test Framework  SKIPPED
[INFO] Brooklyn OSGi init . SKIPPED
[INFO] Brooklyn Karaf . SKIPPED
[INFO] Jetty config fragment .. SKIPPED
[INFO] Apache Http Component extension  SKIPPED
[INFO] Brooklyn Karaf Features  SKIPPED
[INFO] Brooklyn Karaf Shell Commands .. SKIPPED
[INFO] Brooklyn Karaf Tests ... SKIPPED
[INFO] Brooklyn Library Root 

Re: Brooklyn website

[Thomas Bouron]

Thanks Andrea! I don't mind removing the feather but I thought we were
obliged to add it to the website as Brooklyn is an apache project, isn't it
right?

Also, if we put only the Brooklyn logo on the navigation bar, I'm worried
that the first section of the homepage won't work anymore. We will need to
put something else there. If you have any ideas, I gladly take suggestions
:)

Best.

On Thu, 20 Apr 2017 at 20:26 Andrea Turli 
wrote:

> I think it's a great contribution, much better on a small screen like
> smartphone. Still `Apache Brooklyn` doesn't fit nicely though. Also maybe
> instead of the feather, we could keep a brooklyn logo on the bar that
> remains when you scroll down, wdyt?
>
> Il 20/apr/2017 18:12, "Thomas Bouron"  ha
> scritto:
>
> > Hi Brooklyners
> >
> > I just pushed a PR[1] which implement the changes I was proposing earlier
> > in this thread. I also pushed a demo of the new website and landing page
> on
> > my fork[2]
> >
> > Best.
> >
> > [1] https://github.com/apache/brooklyn-docs/pull/173
> > [2] https://tbouron.github.io/brooklyn-docs/
> >
> > On Tue, 18 Apr 2017 at 09:38 John McCabe  wrote:
> >
> > > +1 on Thomas's mock up, looks great.
> > >
> > > Being able to see something in action before trying it can be useful,
> > > perhaps something asciinema-like (https://asciinema.org/a/46498).
> > >
> > > On Tue, 18 Apr 2017 at 08:21 Thomas Bouron <
> > > thomas.bou...@cloudsoftcorp.com>
> > > wrote:
> > >
> > > > On Tue, 18 Apr 2017 at 00:32 Alex Heneveld <
> > > > alex.henev...@cloudsoftcorp.com>
> > > > wrote:
> > > >
> > > > >
> > > > > Wow - really like this!  Very clean and elegant.
> > > > >
> > > >
> > > > Thanks!
> > > >
> > > >
> > > > > I think we should /show/ a little bit more what we mean by
> > > > > modeling/deploying/managing -- eg for modelling (code snippet),
> and a
> > > > > graphic w cloud icons for deploy, and maybe a UI screenshot for
> > manage
> > > ?
> > > > >
> > > >
> > > > I'm not sure I agree with that. I deliberately went for less because
> I
> > > > believe less is more, especially on the homepage. Adding a code
> > snippet,
> > > > screenshots abd logos will do the reverse IMO. You started this
> thread
> > by
> > > > saying that Brooklyn is always considered as complicated, doing this
> > will
> > > > reinforce this feeling IMO.
> > > >
> > > > That being said, we could add a page to have a UI tour, with a link
> to
> > it
> > > > in the "Get Started" menu item?
> > > >
> > > >
> > > > > And not sure about curl-to-bash for install -- some people are
> > > > > uncomfortable with that (in any case we should curl from an apache
> > site
> > > > > for a release, not github).
> > > >
> > > >
> > > > I used github as an example, this is not a real URL.
> > > >
> > > >
> > > > > But some one-line quick start would be
> > > > > nice.  A docker image?  (And can say jump to Get Started for other
> > > > > flavours of installs.)
> > > > >
> > > >
> > > > Why not for the docker image. I went for the curl command because
> > that's
> > > > the only one that does not require the user to install anything else.
> > If
> > > > you think about it, a quick start guide that starts by "Install
> docker"
> > > or
> > > > "Install vagrant AND VirtualBox" is far from a great way to start
> for a
> > > > first-time Brooklyn user. Now I might be wrong about this and if it
> is
> > > the
> > > > case, then I'm happy to change it (again, that's just a prototype)
> > > > Regarding the other flavours of install, it's specified in a link,
> > below
> > > > the shell command ;)
> > > >
> > > > Lastly instead of the brooklyn logo font, can you just use the big
> logo
> > > > > PNG that is currently in use on the site?
> > > >
> > > >
> > > > I could do that but:
> > > >
> > > >1. the quality is not as nice as pure text
> > > >2. from an semantic/SEO point of view, it's better to use the text
> > as
> > > >crawler will be able to read it
> > > >
> > > > Also, I don't know if you noticed but I rearranged the menu items. I
> > was
> > > > planning to do this + update all the web dependencies i.e. bootstrap,
> > > > jquery, etc
> > > >
> > > > Best
> > > > --
> > > >
> > > > Thomas Bouron • Senior Software Engineer @ Cloudsoft Corporation •
> > > > https://cloudsoft.io/
> > > > Github: https://github.com/tbouron
> > > > Twitter: https://twitter.com/eltibouron
> > > >
> > >
> > --
> >
> > Thomas Bouron • Senior Software Engineer @ Cloudsoft Corporation •
> > https://cloudsoft.io/
> > Github: https://github.com/tbouron
> > Twitter: https://twitter.com/eltibouron
> >
>
-- 

Thomas Bouron • Senior Software Engineer @ Cloudsoft Corporation •
https://cloudsoft.io/
Github: https://github.com/tbouron
Twitter: https://twitter.com/eltibouron

Re: Brooklyn website

[Andrea Turli]

I think it's a great contribution, much better on a small screen like
smartphone. Still `Apache Brooklyn` doesn't fit nicely though. Also maybe
instead of the feather, we could keep a brooklyn logo on the bar that
remains when you scroll down, wdyt?

Il 20/apr/2017 18:12, "Thomas Bouron"  ha
scritto:

> Hi Brooklyners
>
> I just pushed a PR[1] which implement the changes I was proposing earlier
> in this thread. I also pushed a demo of the new website and landing page on
> my fork[2]
>
> Best.
>
> [1] https://github.com/apache/brooklyn-docs/pull/173
> [2] https://tbouron.github.io/brooklyn-docs/
>
> On Tue, 18 Apr 2017 at 09:38 John McCabe  wrote:
>
> > +1 on Thomas's mock up, looks great.
> >
> > Being able to see something in action before trying it can be useful,
> > perhaps something asciinema-like (https://asciinema.org/a/46498).
> >
> > On Tue, 18 Apr 2017 at 08:21 Thomas Bouron <
> > thomas.bou...@cloudsoftcorp.com>
> > wrote:
> >
> > > On Tue, 18 Apr 2017 at 00:32 Alex Heneveld <
> > > alex.henev...@cloudsoftcorp.com>
> > > wrote:
> > >
> > > >
> > > > Wow - really like this!  Very clean and elegant.
> > > >
> > >
> > > Thanks!
> > >
> > >
> > > > I think we should /show/ a little bit more what we mean by
> > > > modeling/deploying/managing -- eg for modelling (code snippet), and a
> > > > graphic w cloud icons for deploy, and maybe a UI screenshot for
> manage
> > ?
> > > >
> > >
> > > I'm not sure I agree with that. I deliberately went for less because I
> > > believe less is more, especially on the homepage. Adding a code
> snippet,
> > > screenshots abd logos will do the reverse IMO. You started this thread
> by
> > > saying that Brooklyn is always considered as complicated, doing this
> will
> > > reinforce this feeling IMO.
> > >
> > > That being said, we could add a page to have a UI tour, with a link to
> it
> > > in the "Get Started" menu item?
> > >
> > >
> > > > And not sure about curl-to-bash for install -- some people are
> > > > uncomfortable with that (in any case we should curl from an apache
> site
> > > > for a release, not github).
> > >
> > >
> > > I used github as an example, this is not a real URL.
> > >
> > >
> > > > But some one-line quick start would be
> > > > nice.  A docker image?  (And can say jump to Get Started for other
> > > > flavours of installs.)
> > > >
> > >
> > > Why not for the docker image. I went for the curl command because
> that's
> > > the only one that does not require the user to install anything else.
> If
> > > you think about it, a quick start guide that starts by "Install docker"
> > or
> > > "Install vagrant AND VirtualBox" is far from a great way to start for a
> > > first-time Brooklyn user. Now I might be wrong about this and if it is
> > the
> > > case, then I'm happy to change it (again, that's just a prototype)
> > > Regarding the other flavours of install, it's specified in a link,
> below
> > > the shell command ;)
> > >
> > > Lastly instead of the brooklyn logo font, can you just use the big logo
> > > > PNG that is currently in use on the site?
> > >
> > >
> > > I could do that but:
> > >
> > >1. the quality is not as nice as pure text
> > >2. from an semantic/SEO point of view, it's better to use the text
> as
> > >crawler will be able to read it
> > >
> > > Also, I don't know if you noticed but I rearranged the menu items. I
> was
> > > planning to do this + update all the web dependencies i.e. bootstrap,
> > > jquery, etc
> > >
> > > Best
> > > --
> > >
> > > Thomas Bouron • Senior Software Engineer @ Cloudsoft Corporation •
> > > https://cloudsoft.io/
> > > Github: https://github.com/tbouron
> > > Twitter: https://twitter.com/eltibouron
> > >
> >
> --
>
> Thomas Bouron • Senior Software Engineer @ Cloudsoft Corporation •
> https://cloudsoft.io/
> Github: https://github.com/tbouron
> Twitter: https://twitter.com/eltibouron
>

[jira] [Created] (BROOKLYN-484) JBoss7 entity restart fails (launch ssh session returns before process running?)

[Aled Sage (JIRA)]

Aled Sage created BROOKLYN-484:
--

 Summary: JBoss7 entity restart fails (launch ssh session returns 
before process running?)
 Key: BROOKLYN-484
 URL: https://issues.apache.org/jira/browse/BROOKLYN-484
 Project: Brooklyn
  Issue Type: Bug
Reporter: Aled Sage
Priority: Minor


With version 0.11.0-rc1...

We've seen a failure of the {{restart}} effector for {{JBoss7Server}}. The 
post-launch step failed (waiting for the url to be reachable/responsive).

Unfortunately there's no additional debugging information available - the VMs 
are gone, and the debug log is not available.

However, I've identified a reason why this might happen.

On {{start}}, the {{JBoss7SshDriver.launch}} script will redirect stdout/stderr 
to a file named {{console}}, and will then wait for that file to say 'starting'.

Importantly, there is an old comment saying:
{noformat}
// We wait for evidence of JBoss running because, using SshCliTool,
// we saw the ssh session return before the JBoss process was fully 
running
// so the process failed to start.
{noformat}

On {{restart}}, it stops the process, and then calls {{JBoss7SshDriver.launch}} 
again. However, it appends to the file {{console}}. Therefore when it checks if 
the file says 'starting' it will return immediately. This means the ssh session 
could return before the JBoss process was fully running.

A solution would be to change launch, to first move the previous {{console}} 
file. This would mean the subsequent calls to the {{launch}} script would wait 
for the process to be running.

This same problem would also apply to other entities, such as 
{{TomcatSshDriver.launch}}.

A way to reproduce this would probably be to repeatedly call the {{restart}} 
effector (waiting for serviceUp to be true again between each). It almost 
always works - I've personally only seen this failure once.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Created] (BROOKLYN-483) Brooklyn cannot initiate first SSH connection on Azure

[Thomas Bouron (JIRA)]

Thomas Bouron created BROOKLYN-483:
--

 Summary: Brooklyn cannot initiate first SSH connection on Azure
 Key: BROOKLYN-483
 URL: https://issues.apache.org/jira/browse/BROOKLYN-483
 Project: Brooklyn
  Issue Type: Bug
Affects Versions: 0.11.0
Reporter: Thomas Bouron


I'm trying to deploy a very simple blueprint
{code}
location: azure-sng-minimal
services:
  - type: org.apache.brooklyn.entity.software.base.EmptySoftwareProcess
{code}

on a minimal Azure location:
{code}

brooklyn.catalog:
  id: azure-sng-minimal
  itemType: location
  name: "Azure Southeast Asia (SNG) Minimal"
  item:
type: jclouds:azurecompute
brooklyn.config:
  regionId: Southeast Asia
  identity: /Users/thomasbouron/.ssh/azure.p12
  credential: 
  endpoint: 
https://management.core.windows.net/341751b0-f348-45ce-9498-x
{code}

Brooklyn is able to provisioned the VM but fails to initiate the first SSH 
connection:

{code}
2017-04-19 10:38:23,702 INFO  334 o.a.b.r.r.ApplicationResource 
[qtp235840421-124] Launched from YAML: location: azure-sng-minimal
  - type: "elasticsearch-node-tests_19042017_103735:1.0.2-SNAPSHOT" -> 
BasicApplicationImpl{id=b6lgwpmnrj} (Task[start]@tfqRf4JU)
2017-04-19 10:38:23,858 INFO  129 o.a.b.e.s.b.l.MachineLifecycleEffectorTasks 
[ager-iWm4jv3o-17] Starting VanillaSoftwareProcessImpl{id=f7iv5s8u2x}, 
obtaining a new location instance in JcloudsLocation[Azure Southeast Asia (SNG) 
Minimal:/amp/azure.p12@azkm52sknu] with ports [22, 9330, 9200]
2017-04-19 10:38:24,760 INFO  125 o.a.b.l.j.JcloudsLocation [ager-iWm4jv3o-17] 
Creating VM 
azurecompute:https://management.core.windows.net/341751b0-f348-45ce-9498-x@VanillaSoftwareProcessImpl{id=f7iv5s8u2x}
 in JcloudsLocation[Azure Southeast Asia (SNG) 
Minimal:/users/thomasbouron/.ssh/azure.p12@azkm52sknu]
2017-04-19 10:39:09,962 INFO  106 j.compute [user thread 0] Cloud Service 
(brooklyn-oonk80-root-elastic-b6lg-elasticsearchnod-f7iv-caa) created with 
operation id: 76008c93b18d74c797f17958fa336eb6
2017-04-19 10:39:36,713 INFO  106 j.compute [user thread 0] Deployment created 
with name: brooklyn-oonk80-root-elastic-b6lg-elasticsearchnod-f7iv-caa
2017-04-19 10:41:53,917 INFO  106 j.ssh [user thread 1] << 
(jclouds:pw[cdfb4472683ad9b4f7e159935dbc439a]@40.71.103.245:22) error acquiring 
{hostAndPort=40.71.103.245:22, loginUser=jclouds, ssh=null, 
connectTimeout=6, sessionTimeout=6} (attempt 1 of 50): Exhausted 
available authentication methods
2017-04-19 10:41:56,108 INFO  106 j.ssh [user thread 1] << 
(jclouds:pw[cdfb4472683ad9b4f7e159935dbc439a]@40.71.103.245:22) error acquiring 
{hostAndPort=40.71.103.245:22, loginUser=jclouds, ssh=null, 
connectTimeout=6, sessionTimeout=6} (attempt 2 of 50): Exhausted 
available authentication methods
2017-04-19 10:41:58,946 INFO  106 j.ssh [user thread 1] << 
(jclouds:pw[cdfb4472683ad9b4f7e159935dbc439a]@40.71.103.245:22) error acquiring 
{hostAndPort=40.71.103.245:22, loginUser=jclouds, ssh=null, 
connectTimeout=6, sessionTimeout=6} (attempt 3 of 50): Exhausted 
available authentication methods
...
{code}

Looks like the default {{loginUser}} picked by jClouds is wrong. I know I can 
specify my own {{loginUser}} but I would expected jClouds to do this for me on 
basic locations



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Re: Brooklyn website

[Thomas Bouron]

Hi Brooklyners

I just pushed a PR[1] which implement the changes I was proposing earlier
in this thread. I also pushed a demo of the new website and landing page on
my fork[2]

Best.

[1] https://github.com/apache/brooklyn-docs/pull/173
[2] https://tbouron.github.io/brooklyn-docs/

On Tue, 18 Apr 2017 at 09:38 John McCabe  wrote:

> +1 on Thomas's mock up, looks great.
>
> Being able to see something in action before trying it can be useful,
> perhaps something asciinema-like (https://asciinema.org/a/46498).
>
> On Tue, 18 Apr 2017 at 08:21 Thomas Bouron <
> thomas.bou...@cloudsoftcorp.com>
> wrote:
>
> > On Tue, 18 Apr 2017 at 00:32 Alex Heneveld <
> > alex.henev...@cloudsoftcorp.com>
> > wrote:
> >
> > >
> > > Wow - really like this!  Very clean and elegant.
> > >
> >
> > Thanks!
> >
> >
> > > I think we should /show/ a little bit more what we mean by
> > > modeling/deploying/managing -- eg for modelling (code snippet), and a
> > > graphic w cloud icons for deploy, and maybe a UI screenshot for manage
> ?
> > >
> >
> > I'm not sure I agree with that. I deliberately went for less because I
> > believe less is more, especially on the homepage. Adding a code snippet,
> > screenshots abd logos will do the reverse IMO. You started this thread by
> > saying that Brooklyn is always considered as complicated, doing this will
> > reinforce this feeling IMO.
> >
> > That being said, we could add a page to have a UI tour, with a link to it
> > in the "Get Started" menu item?
> >
> >
> > > And not sure about curl-to-bash for install -- some people are
> > > uncomfortable with that (in any case we should curl from an apache site
> > > for a release, not github).
> >
> >
> > I used github as an example, this is not a real URL.
> >
> >
> > > But some one-line quick start would be
> > > nice.  A docker image?  (And can say jump to Get Started for other
> > > flavours of installs.)
> > >
> >
> > Why not for the docker image. I went for the curl command because that's
> > the only one that does not require the user to install anything else. If
> > you think about it, a quick start guide that starts by "Install docker"
> or
> > "Install vagrant AND VirtualBox" is far from a great way to start for a
> > first-time Brooklyn user. Now I might be wrong about this and if it is
> the
> > case, then I'm happy to change it (again, that's just a prototype)
> > Regarding the other flavours of install, it's specified in a link, below
> > the shell command ;)
> >
> > Lastly instead of the brooklyn logo font, can you just use the big logo
> > > PNG that is currently in use on the site?
> >
> >
> > I could do that but:
> >
> >1. the quality is not as nice as pure text
> >2. from an semantic/SEO point of view, it's better to use the text as
> >crawler will be able to read it
> >
> > Also, I don't know if you noticed but I rearranged the menu items. I was
> > planning to do this + update all the web dependencies i.e. bootstrap,
> > jquery, etc
> >
> > Best
> > --
> >
> > Thomas Bouron • Senior Software Engineer @ Cloudsoft Corporation •
> > https://cloudsoft.io/
> > Github: https://github.com/tbouron
> > Twitter: https://twitter.com/eltibouron
> >
>
-- 

Thomas Bouron • Senior Software Engineer @ Cloudsoft Corporation •
https://cloudsoft.io/
Github: https://github.com/tbouron
Twitter: https://twitter.com/eltibouron

[GitHub] brooklyn-docs pull request #173: Update apache brooklyn website's landing pa...

[tbouron]

GitHub user tbouron opened a pull request:

https://github.com/apache/brooklyn-docs/pull/173

Update apache brooklyn website's landing page + styles

Demo is visible here: https://tbouron.github.io/brooklyn-docs/

The most visible changes are on the landing page, it matched the prototype 
I sent on the ML. I restyled a bit the top menu + make it work on mobile 
devices. This means that to make the dropdown appear, you now need to click on 
the menu item instead of just hovering over it.

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

$ git pull https://github.com/tbouron/brooklyn-docs feature/website-landing

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

https://github.com/apache/brooklyn-docs/pull/173.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 #173






---
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-server pull request #643: Don't fail white list loading if invalid ...

[neykov]

GitHub user neykov opened a pull request:

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

Don't fail white list loading if invalid bundles present

If we add a non-bundle jar through `brooklyn.libraries` subsequent white 
list loads fail.

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

$ git pull https://github.com/neykov/brooklyn-server fix/white-list-load

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

https://github.com/apache/brooklyn-server/pull/643.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 #643






---
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] [Commented] (BROOKLYN-482) Config Key default value (in yaml) of type List is mutable

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/BROOKLYN-482?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15976923#comment-15976923
 ] 

ASF GitHub Bot commented on BROOKLYN-482:
-

GitHub user aledsage opened a pull request:

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

BROOKLYN-482: yaml config key default vals immutable



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

$ git pull https://github.com/aledsage/brooklyn-server BROOKLYN-482

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

https://github.com/apache/brooklyn-server/pull/642.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 #642


commit 15da8e50aa4cb8618da3948cbefa614c50877bc5
Author: Aled Sage 
Date:   2017-04-20T15:34:50Z

BROOKLYN-482: yaml config key default vals immutable




> Config Key default value (in yaml) of type List is mutable
> --
>
> Key: BROOKLYN-482
> URL: https://issues.apache.org/jira/browse/BROOKLYN-482
> Project: Brooklyn
>  Issue Type: Bug
>Reporter: Aled Sage
>Priority: Minor
>
> In version 0.11.0-rc2...
> As described in https://github.com/apache/brooklyn-docs/pull/172, it's 
> important that config key default values are immutable.
> However, in a yaml catalog item or blueprint if you declare a config key of 
> type {{java.util.List}} (or Set, Collection or Map) with {{[]}} then that is 
> mutable.
> For example:
> {noformat}
> brooklyn.catalog:
>   itemType: entity
>   items:
>   - id: entity-with-keys
> item:
>   type: server
>   brooklyn.parameters:
>   - name: my.list.key
> type: java.util.List
> default: ["myDefaultVal"]
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[GitHub] brooklyn-server pull request #642: BROOKLYN-482: yaml config key default val...

[aledsage]

GitHub user aledsage opened a pull request:

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

BROOKLYN-482: yaml config key default vals immutable



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

$ git pull https://github.com/aledsage/brooklyn-server BROOKLYN-482

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

https://github.com/apache/brooklyn-server/pull/642.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 #642


commit 15da8e50aa4cb8618da3948cbefa614c50877bc5
Author: Aled Sage 
Date:   2017-04-20T15:34:50Z

BROOKLYN-482: yaml config key default vals immutable




---
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] [Created] (BROOKLYN-482) Config Key default value (in yaml) of type List is mutable

[Aled Sage (JIRA)]

Aled Sage created BROOKLYN-482:
--

 Summary: Config Key default value (in yaml) of type List is mutable
 Key: BROOKLYN-482
 URL: https://issues.apache.org/jira/browse/BROOKLYN-482
 Project: Brooklyn
  Issue Type: Bug
Reporter: Aled Sage
Priority: Minor


In version 0.11.0-rc2...

As described in https://github.com/apache/brooklyn-docs/pull/172, it's 
important that config key default values are immutable.

However, in a yaml catalog item or blueprint if you declare a config key of 
type {{java.util.List}} (or Set, Collection or Map) with {{[]}} then that is 
mutable.

For example:

{noformat}
brooklyn.catalog:
  itemType: entity
  items:
  - id: entity-with-keys
item:
  type: server
  brooklyn.parameters:
  - name: my.list.key
type: java.util.List
default: ["myDefaultVal"]
{noformat}




--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Created] (BROOKLYN-481) Brooklyn is not starting bundles in "brooklyn.libraries"

[Svetoslav Neykov (JIRA)]

Svetoslav Neykov created BROOKLYN-481:
-

 Summary: Brooklyn is not starting bundles in "brooklyn.libraries"
 Key: BROOKLYN-481
 URL: https://issues.apache.org/jira/browse/BROOKLYN-481
 Project: Brooklyn
  Issue Type: Improvement
Reporter: Svetoslav Neykov


Bundles referenced in "brooklyn.libraries" should be started, not just resolved.




--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Created] (BROOKLYN-480) Brooklyn will accept invalid bundles in "brooklyn.libraries"

[Svetoslav Neykov (JIRA)]

Svetoslav Neykov created BROOKLYN-480:
-

 Summary: Brooklyn will accept invalid bundles in 
"brooklyn.libraries"
 Key: BROOKLYN-480
 URL: https://issues.apache.org/jira/browse/BROOKLYN-480
 Project: Brooklyn
  Issue Type: Bug
Reporter: Svetoslav Neykov


Adding any jar file to {{brooklyn.libraries}}, even if not a bundle, will be 
accepted atm. Invalid boms should be rejected, resulting in catalog item 
addition failure.
Also should roll back the already installed bundles.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Created] (BROOKLYN-479) NPE when using byon location

[Svetoslav Neykov (JIRA)]

Svetoslav Neykov created BROOKLYN-479:
-

 Summary: NPE when using byon location
 Key: BROOKLYN-479
 URL: https://issues.apache.org/jira/browse/BROOKLYN-479
 Project: Brooklyn
  Issue Type: Bug
Reporter: Svetoslav Neykov


Blueprint:
{noformat}
location: byon
services:
- type: org.apache.brooklyn.entity.proxy.nginx.NginxController
{noformat}

Error:
{noformat}
Illegal parameter for 'location' (byon); not resolvable: NullPointerException
{noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Created] (BROOKLYN-478) Unable to add preconfigured byon locations (no hosts)

[Svetoslav Neykov (JIRA)]

Svetoslav Neykov created BROOKLYN-478:
-

 Summary: Unable to add preconfigured byon locations (no hosts)
 Key: BROOKLYN-478
 URL: https://issues.apache.org/jira/browse/BROOKLYN-478
 Project: Brooklyn
  Issue Type: Bug
Affects Versions: 0.10.0
Reporter: Svetoslav Neykov


Trying to add an "abstract" byon location with shared config and then specify 
just the hosts in extending locations. When trying to add the bom:

{noformat}
brooklyn.catalog:
  version: "0.0.1-SNAPSHOT"
  itemType: location
  items:
  - id: networking-common
item:
  type: byon
  brooklyn.config:
user: broo
privateKeyFile: ~/.ssh/broo
  - id: networking-lb-tier
item:
  type: networking-common
  brooklyn.config:
hosts:
- 51.140.116.28
{noformat}

I get the following error:

{noformat}
Transformer for Brooklyn OASIS CAMP interpreter gave an error creating this 
plan: No class or resolver found for location type byon
{noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Commented] (BROOKLYN-449) Intermittent NoClassDefFoundError deploying a redis blueprint

[Graeme Miller (JIRA)]

[ 
https://issues.apache.org/jira/browse/BROOKLYN-449?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15976569#comment-15976569
 ] 

Graeme Miller commented on BROOKLYN-449:


There is an image here: cloudsoft.tv:6000/amp-groovy-issue that contains an AMP 
with this issue

> Intermittent NoClassDefFoundError deploying a redis blueprint
> -
>
> Key: BROOKLYN-449
> URL: https://issues.apache.org/jira/browse/BROOKLYN-449
> Project: Brooklyn
>  Issue Type: Bug
>Reporter: Duncan Grant
> Attachments: amp.debug.log, amp.info.log, karaf.log, karaf output
>
>
> I've seen the following error occasionally during redis deployment.  Most of 
> the time it seems to work.
> java.lang.NoClassDefFoundError: Could not initialize class 
> org.codehaus.groovy.runtime.InvokerHelper
> I'm deploying https://github.com/brooklyncentral/brooklyn-redis-server using 
> the catalog.tests.bom
> Sorry - the logs are in kibana and I don't know how to export them.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Updated] (BROOKLYN-449) Intermittent NoClassDefFoundError deploying a redis blueprint

[Graeme Miller (JIRA)]

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

Graeme Miller updated BROOKLYN-449:
---
Attachment: karaf output

also attached a collection of karaf commands from a running issue with this 
error

> Intermittent NoClassDefFoundError deploying a redis blueprint
> -
>
> Key: BROOKLYN-449
> URL: https://issues.apache.org/jira/browse/BROOKLYN-449
> Project: Brooklyn
>  Issue Type: Bug
>Reporter: Duncan Grant
> Attachments: amp.debug.log, amp.info.log, karaf.log, karaf output
>
>
> I've seen the following error occasionally during redis deployment.  Most of 
> the time it seems to work.
> java.lang.NoClassDefFoundError: Could not initialize class 
> org.codehaus.groovy.runtime.InvokerHelper
> I'm deploying https://github.com/brooklyncentral/brooklyn-redis-server using 
> the catalog.tests.bom
> Sorry - the logs are in kibana and I don't know how to export them.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

[jira] [Updated] (BROOKLYN-449) Intermittent NoClassDefFoundError deploying a redis blueprint

[Graeme Miller (JIRA)]

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

Graeme Miller updated BROOKLYN-449:
---
Attachment: karaf.log
amp.info.log
amp.debug.log

log files from when this issue was hit

> Intermittent NoClassDefFoundError deploying a redis blueprint
> -
>
> Key: BROOKLYN-449
> URL: https://issues.apache.org/jira/browse/BROOKLYN-449
> Project: Brooklyn
>  Issue Type: Bug
>Reporter: Duncan Grant
> Attachments: amp.debug.log, amp.info.log, karaf.log
>
>
> I've seen the following error occasionally during redis deployment.  Most of 
> the time it seems to work.
> java.lang.NoClassDefFoundError: Could not initialize class 
> org.codehaus.groovy.runtime.InvokerHelper
> I'm deploying https://github.com/brooklyncentral/brooklyn-redis-server using 
> the catalog.tests.bom
> Sorry - the logs are in kibana and I don't know how to export them.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

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

[Apache Jenkins Server]

See 

[GitHub] brooklyn-server pull request #617: Persist management plane ID

[asfgit]

Github user asfgit closed the pull request at:

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


---
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-server issue #617: Persist management plane ID

[geomacy]

Github user geomacy commented on the issue:

https://github.com/apache/brooklyn-server/pull/617
  
LGTM; Jenkins build break was unrelated, I rebuilt it (build number 2007) 
and it succeeded. Will merge.


---
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-server-master #539

[Apache Jenkins Server]

See 


Changes:

[richard] LICENSE updates

--
[...truncated 5.68 MB...]
[GC (Allocation Failure)  639702K->514333K(719872K), 0.0245620 secs]
[GC (Allocation Failure)  643357K->517311K(722432K), 0.0109029 secs]
[GC (Allocation Failure)  649919K->518117K(720896K), 0.0091277 secs]
2017-04-20 08:41:55,998 INFO  TESTNG PASSED CONFIGURATION: "Surefire test" - 
@BeforeClass 
org.apache.brooklyn.camp.brooklyn.catalog.CatalogMakeOsgiBundleTest.setUp() 
finished in 826 ms
2017-04-20 08:41:55,998 INFO  TESTNG INVOKING: "Surefire test" - 
org.apache.brooklyn.camp.brooklyn.catalog.CatalogMakeOsgiBundleTest.testCatalogBomFromBundleWithManifest()
2017-04-20 08:41:56,029 WARN  Request for bundle 
'org/apache/brooklyn/camp/brooklyn/catalog/osgi/catalog-bundle-1/META-INF/MANIFEST.MF/Manifest-Version'
 will be ignored, loading ' 1.0' as no framework available
[GC (Allocation Failure)  650725K->519414K(727552K), 0.0165605 secs]
2017-04-20 08:41:56,128 INFO  Started application 
BasicApplicationImpl{id=iqjy6j8va0}
2017-04-20 08:41:56,131 INFO  TESTNG FAILED: "Surefire test" - 
org.apache.brooklyn.camp.brooklyn.catalog.CatalogMakeOsgiBundleTest.testCatalogBomFromBundleWithManifest()
 finished in 131 ms
java.lang.AssertionError: attribute=Sensor: a.sensor (java.lang.String); 
val=null
at org.apache.brooklyn.test.Asserts.fail(Asserts.java:758)
at org.apache.brooklyn.test.Asserts.assertTrue(Asserts.java:737)
at 
org.apache.brooklyn.core.entity.EntityAsserts.assertAttribute(EntityAsserts.java:103)
at 
org.apache.brooklyn.camp.brooklyn.catalog.CatalogMakeOsgiBundleTest.assertBasic1DeploysAndHasSensor(CatalogMakeOsgiBundleTest.java:138)
at 
org.apache.brooklyn.camp.brooklyn.catalog.CatalogMakeOsgiBundleTest.testCatalogBomFromBundleWithManifest(CatalogMakeOsgiBundleTest.java:129)
2017-04-20 08:41:56,139 INFO  TESTNG INVOKING CONFIGURATION: "Surefire test" - 
@AfterMethod 
org.apache.brooklyn.camp.brooklyn.catalog.CatalogMakeOsgiBundleTest.cleanUpButKeepMgmt()
2017-04-20 08:41:56,139 INFO  Stopping application 
BasicApplicationImpl{id=iqjy6j8va0}
2017-04-20 08:41:56,140 INFO  Stopped application 
BasicApplicationImpl{id=iqjy6j8va0}
2017-04-20 08:41:56,143 INFO  TESTNG PASSED CONFIGURATION: "Surefire test" - 
@AfterMethod 
org.apache.brooklyn.camp.brooklyn.catalog.CatalogMakeOsgiBundleTest.cleanUpButKeepMgmt()
 finished in 4 ms
2017-04-20 08:41:56,143 INFO  TESTNG INVOKING CONFIGURATION: "Surefire test" - 
@AfterMethod 
org.apache.brooklyn.camp.brooklyn.catalog.CatalogMakeOsgiBundleTest.clearFeatureEnablement()
2017-04-20 08:41:56,143 INFO  TESTNG PASSED CONFIGURATION: "Surefire test" - 
@AfterMethod 
org.apache.brooklyn.camp.brooklyn.catalog.CatalogMakeOsgiBundleTest.clearFeatureEnablement()
 finished in 0 ms
2017-04-20 08:41:56,144 INFO  TESTNG INVOKING: "Surefire test" - 
org.apache.brooklyn.camp.brooklyn.catalog.CatalogMakeOsgiBundleTest.testCatalogBomFromBundleWithManualManifest()
2017-04-20 08:41:56,151 WARN  Request for bundle 
'org/apache/brooklyn/camp/brooklyn/catalog/osgi/catalog-bundle-1/META-INF/MANIFEST.MF/Manifest-Version'
 will be ignored, loading ' 1.0' as no framework available
2017-04-20 08:41:56,248 INFO  Started application 
BasicApplicationImpl{id=ofq2k86st3}
2017-04-20 08:41:56,248 INFO  TESTNG PASSED: "Surefire test" - 
org.apache.brooklyn.camp.brooklyn.catalog.CatalogMakeOsgiBundleTest.testCatalogBomFromBundleWithManualManifest()
 finished in 104 ms
2017-04-20 08:41:56,249 INFO  TESTNG INVOKING CONFIGURATION: "Surefire test" - 
@AfterMethod 
org.apache.brooklyn.camp.brooklyn.catalog.CatalogMakeOsgiBundleTest.cleanUpButKeepMgmt()
2017-04-20 08:41:56,249 INFO  Stopping application 
BasicApplicationImpl{id=ofq2k86st3}
2017-04-20 08:41:56,254 INFO  Stopped application 
BasicApplicationImpl{id=ofq2k86st3}
2017-04-20 08:41:56,256 INFO  TESTNG PASSED CONFIGURATION: "Surefire test" - 
@AfterMethod 
org.apache.brooklyn.camp.brooklyn.catalog.CatalogMakeOsgiBundleTest.cleanUpButKeepMgmt()
 finished in 7 ms
2017-04-20 08:41:56,256 INFO  TESTNG INVOKING CONFIGURATION: "Surefire test" - 
@AfterMethod 
org.apache.brooklyn.camp.brooklyn.catalog.CatalogMakeOsgiBundleTest.clearFeatureEnablement()
2017-04-20 08:41:56,256 INFO  TESTNG PASSED CONFIGURATION: "Surefire test" - 
@AfterMethod 
org.apache.brooklyn.camp.brooklyn.catalog.CatalogMakeOsgiBundleTest.clearFeatureEnablement()
 finished in 0 ms
2017-04-20 08:41:56,256 INFO  TESTNG INVOKING: "Surefire test" - 
org.apache.brooklyn.camp.brooklyn.catalog.CatalogMakeOsgiBundleTest.testCatalogBomLoadsFileInBundle()
2017-04-20 08:41:56,260 WARN  Request for bundle 
'org/apache/brooklyn/camp/brooklyn/catalog/osgi/catalog-bundle-1/META-INF/MANIFEST.MF/Manifest-Version'
 will be ignored, loading ' 1.0' as no framework available
2017-04-20 08:41:56,338 INFO  Started application 
BasicApplicationImpl{id=pch2spak3x}

[GitHub] brooklyn-server pull request #630: LICENSE updates

[asfgit]

Github user asfgit closed the pull request at:

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


---
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-server pull request #617: Persist management plane ID

[geomacy]

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

https://github.com/apache/brooklyn-server/pull/617#discussion_r112392507
  
--- Diff: 
core/src/main/java/org/apache/brooklyn/core/mgmt/internal/LocalManagementContext.java
 ---
@@ -224,6 +224,7 @@ public String getManagementPlaneId() {
 public void setManagementPlaneId(String newPlaneId) {
 if (managementPlaneId != null && 
!managementPlaneId.equals(newPlaneId)) {
 log.warn("Management plane ID changed from {} to {}", 
managementPlaneId, newPlaneId);
+log.debug("Management plane ID changed from {} to {}", new 
Object[] {managementPlaneId, newPlaneId}, new RuntimeException("Stack trace for 
setManagementPlaneId"));
--- End diff --

Hm, interesting!  That's new to me - a good one to know!


---
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.
---