[GitHub] brooklyn-library pull request #87: Follow API changes in https://github.com/...

GitHub user neykov opened a pull request: https://github.com/apache/brooklyn-library/pull/87 Follow API changes in https://github.com/apache/brooklyn-server/pull/520 You can merge this pull request into a Git repository by running: $ git pull

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

See -- [...truncated 58197 lines...] [JENKINS] Archiving

[GitHub] brooklyn-server issue #520: Limit parallelism of start/stop steps on Softwar...

Github user sjcorbett commented on the issue: https://github.com/apache/brooklyn-server/pull/520 Jenkins is failing on `SoftwareProcessEntityLatchTest.testConcurrency` and `testReleaseableLatchBlocks` By the way, can we get rid of `maxConcurrentChildCommands` when this is

[jira] [Commented] (BROOKLYN-427) JcloudsLocation is not releasing its customizers

[ https://issues.apache.org/jira/browse/BROOKLYN-427?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15830329#comment-15830329 ] Geoff Macartney commented on BROOKLYN-427: -- See

[jira] [Commented] (BROOKLYN-427) JcloudsLocation is not releasing its customizers

[ https://issues.apache.org/jira/browse/BROOKLYN-427?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15830327#comment-15830327 ] ASF GitHub Bot commented on BROOKLYN-427: - GitHub user geomacy opened a pull request:

[GitHub] brooklyn-server pull request #530: JcloudsLocation is not releasing its cust...

GitHub user geomacy opened a pull request: https://github.com/apache/brooklyn-server/pull/530 JcloudsLocation is not releasing its customizers (BROOKLYN-427) preRelease and postRelease of JcloudsLocationCustomizers are not being called as expected during the release of a

[jira] [Created] (BROOKLYN-427) JcloudsLocation is not releasing its customizers

Geoff Macartney created BROOKLYN-427: Summary: JcloudsLocation is not releasing its customizers Key: BROOKLYN-427 URL: https://issues.apache.org/jira/browse/BROOKLYN-427 Project: Brooklyn

Re: Default VM Image

+1 I don't know of anyone who would be negatively impacted by the defaults changing from ubuntu to centos. --- Switching from ubuntu 14.04 to 16.04 as the default ubuntu might well break some blueprints though (as I've seen examples that just use `osFamily: ubuntu`, without an explicit

[GitHub] brooklyn-server pull request #529: LocationNetworkInfoCustomizer

GitHub user sjcorbett opened a pull request: https://github.com/apache/brooklyn-server/pull/529 LocationNetworkInfoCustomizer `LocationNetworkInfoCustomizer` is a special customiser used by `JcloudsLocation` to determine the address and credentials with which a location should be

[GitHub] brooklyn-server issue #520: Limit parallelism of start/stop steps on Softwar...

Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/520 Addressed comments, let's discuss API backwards compatibility. --- 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

Re: Default VM Image

Agree with Mark McKenna's proposal to make it configurable. No strong opinion on what the default value should be, though. Maybe Ubuntu 14.04, for compatibility with blueprints that assume Ubuntu. Any examples of those in the wild? Guglielmo Nigri Senior Engineer @ Cloudsoft Corporation On 19

[GitHub] brooklyn-docs pull request #140: Add Azure Classic and ARM docs

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-docs/pull/140 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] brooklyn-docs issue #140: Add Azure Classic and ARM docs

Github user geomacy commented on the issue: https://github.com/apache/brooklyn-docs/pull/140 Looks good at a quick scan for general issues (can't comment on the specifics of the settings/config for Azure as such), so if Graeme is happy then I'm happy to merge. --- If your project

[GitHub] brooklyn-server pull request #520: Limit parallelism of start/stop steps on ...

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/520#discussion_r96905127 --- Diff: software/base/src/main/java/org/apache/brooklyn/entity/software/base/SoftwareProcessDriverLifecycleEffectorTasks.java --- @@ -169,7 +173,7

[GitHub] brooklyn-docs pull request #140: Add Azure Classic and ARM docs

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/140#discussion_r96904988 --- Diff: guide/ops/locations/_azure-ARM.md --- @@ -0,0 +1,92 @@ +--- +section: Azure Compute ARM +section_type: inline

Re: Default VM Image

Duncan, Do we have any data on what our users are using? That should be our driver for change :-) I am a bit concerned what effect making this change would have on existing blueprints that don't specify an OS requirement. If a user writes such a blueprint and makes it Ubuntu-dependent, would a

Re: Default VM Image

Makes sense I guess; there have been some issues with Ubuntu 16.04 (at least with Vagrant) so Centos as default might be better than Ubuntu 16? On Thu, 19 Jan 2017 at 15:37 Aleksandr Vasilev < aleksandr.vasi...@cloudsoftcorp.com> wrote: > I'm in favor of adding Ubuntu 16.04 and CentOS 7 with

Re: Default VM Image

I'm in favor of adding Ubuntu 16.04 and CentOS 7 with score setting higher than previous versions. Best Regards, Aleksandr Vasilev DevOps Engineer | Cloudsoft Corporation On 20 January 2017 at 00:17, Duncan Godwin wrote: > Hi All, > > Should we change the

Default VM Image

Hi All, Should we change the default image to CentOS 7 over Ubuntu 14.04? Apache Brooklyn, by default currently favours Ubuntu 14.04 and 12.04 LTS images when no preference is selected. This is because of the code in BrooklynImageChooser here:

[GitHub] brooklyn-docs pull request #140: Add Azure Classic and ARM docs

Github user drigodwin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/140#discussion_r96877239 --- Diff: guide/ops/locations/_azure-ARM.md --- @@ -0,0 +1,92 @@ +--- +section: Azure Compute ARM +section_type: inline

[GitHub] brooklyn-docs pull request #140: Add Azure Classic and ARM docs

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/140#discussion_r96874537 --- Diff: guide/ops/locations/_azure-classic.md --- @@ -0,0 +1,235 @@ +--- +section: Azure Compute Classic +section_type: inline

[GitHub] brooklyn-docs pull request #140: Add Azure Classic and ARM docs

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/140#discussion_r96873514 --- Diff: guide/ops/locations/_azure-classic.md --- @@ -0,0 +1,235 @@ +--- +section: Azure Compute Classic +section_type: inline

[GitHub] brooklyn-docs pull request #140: Add Azure Classic and ARM docs

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/140#discussion_r96874644 --- Diff: guide/ops/locations/_azure-classic.md --- @@ -0,0 +1,235 @@ +--- +section: Azure Compute Classic +section_type: inline

[GitHub] brooklyn-docs pull request #140: Add Azure Classic and ARM docs

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/140#discussion_r96876434 --- Diff: guide/ops/locations/_azure-ARM.md --- @@ -0,0 +1,92 @@ +--- +section: Azure Compute ARM +section_type: inline

[GitHub] brooklyn-docs pull request #140: Add Azure Classic and ARM docs

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/140#discussion_r96873539 --- Diff: guide/ops/locations/_azure-classic.md --- @@ -0,0 +1,235 @@ +--- +section: Azure Compute Classic +section_type: inline

[GitHub] brooklyn-docs pull request #140: Add Azure Classic and ARM docs

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/140#discussion_r96876124 --- Diff: guide/ops/locations/_azure-ARM.md --- @@ -0,0 +1,92 @@ +--- +section: Azure Compute ARM +section_type: inline

[GitHub] brooklyn-docs pull request #140: Add Azure Classic and ARM docs

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/140#discussion_r96874259 --- Diff: guide/ops/locations/_azure-classic.md --- @@ -0,0 +1,235 @@ +--- +section: Azure Compute Classic +section_type: inline

[jira] [Commented] (BROOKLYN-426) NoClassDefFoundError from whitelisted bundle causes catalog listing to fail

[ https://issues.apache.org/jira/browse/BROOKLYN-426?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15830044#comment-15830044 ] ASF GitHub Bot commented on BROOKLYN-426: - Github user m4rkmckenna commented on a diff in the

[GitHub] brooklyn-server pull request #528: BROOKLYN-426: avoid propagating NoClassDe...

Github user m4rkmckenna commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/528#discussion_r96872949 --- Diff: core/src/main/java/org/apache/brooklyn/util/core/LoaderDispatcher.java --- @@ -44,6 +44,10 @@ return

[GitHub] brooklyn-docs pull request #140: Add Azure Classic and ARM docs

GitHub user drigodwin opened a pull request: https://github.com/apache/brooklyn-docs/pull/140 Add Azure Classic and ARM docs Adds Azure Classic and Azure ARM usage details to the documented locations. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] brooklyn-server pull request #519: Add experimental CORS server support

Github user bostko commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/519#discussion_r96859105 --- Diff: rest/rest-resources/src/main/java/org/apache/brooklyn/rest/filter/CorsImplSupplierFilter.java --- @@ -0,0 +1,81 @@ +/* + *

[GitHub] brooklyn-docs pull request #129: Update: winrm connectivity limitations

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-docs/pull/129 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] brooklyn-docs pull request #129: Update: winrm connectivity limitations

Github user drigodwin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/129#discussion_r96854401 --- Diff: guide/yaml/winrm/index.md --- @@ -513,13 +513,14 @@ Additional logs may be created by some Windows programs. For example, MSSQL crea

[GitHub] brooklyn-server issue #520: Limit parallelism of start/stop steps on Softwar...

Github user neykov commented on the issue: https://github.com/apache/brooklyn-server/pull/520 @sjcorbett It's generalizing `DslDeferredFunctionCall.resolve` to be reused in the newly added code. When using DSL the actual value we want to resolve could be wrapped in several

[GitHub] brooklyn-server issue #520: Limit parallelism of start/stop steps on Softwar...

Github user sjcorbett commented on the issue: https://github.com/apache/brooklyn-server/pull/520 The bulk of these changes look good. It would help me if you gave some context for 112269ae484dcc5283d9cabc8de2e987f013e79f. --- If your project is set up for it, you can reply to this

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

See

[jira] [Commented] (BROOKLYN-426) NoClassDefFoundError from whitelisted bundle causes catalog listing to fail

[ https://issues.apache.org/jira/browse/BROOKLYN-426?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15829835#comment-15829835 ] ASF GitHub Bot commented on BROOKLYN-426: - GitHub user aledsage opened a pull request:

[GitHub] brooklyn-server pull request #528: BROOKLYN-426: avoid propagating NoClassDe...

GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/528 BROOKLYN-426: avoid propagating NoClassDefFoundError See https://issues.apache.org/jira/browse/BROOKLYN-426 for details. Tested this with a badly behaving bundle, and confirmed

[GitHub] brooklyn-server pull request #527: Adds test for catalog-effector from OSGi ...

GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/527 Adds test for catalog-effector from OSGi bundle You can merge this pull request into a Git repository by running: $ git pull https://github.com/aledsage/brooklyn-server

[jira] [Commented] (BROOKLYN-426) NoClassDefFoundError from whitelisted bundle causes catalog listing to fail

[ https://issues.apache.org/jira/browse/BROOKLYN-426?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15829834#comment-15829834 ] Aled Sage commented on BROOKLYN-426: A simple fix might be to change

[jira] [Created] (BROOKLYN-426) NoClassDefFoundError from whitelisted bundle causes catalog listing to fail

Aled Sage created BROOKLYN-426: -- Summary: NoClassDefFoundError from whitelisted bundle causes catalog listing to fail Key: BROOKLYN-426 URL: https://issues.apache.org/jira/browse/BROOKLYN-426 Project:

[GitHub] brooklyn-server issue #520: Limit parallelism of start/stop steps on Softwar...

Github user sjcorbett commented on the issue: https://github.com/apache/brooklyn-server/pull/520 Compilation fails for me with the following error: ``` [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.3:testCompile (default-testCompile) on

[GitHub] brooklyn-server pull request #520: Limit parallelism of start/stop steps on ...

Github user sjcorbett commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/520#discussion_r96848337 --- Diff: software/base/src/main/java/org/apache/brooklyn/entity/software/base/SoftwareProcessDriverLifecycleEffectorTasks.java --- @@ -169,7

[GitHub] brooklyn-server pull request #520: Limit parallelism of start/stop steps on ...

Github user sjcorbett commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/520#discussion_r96845410 --- Diff: core/src/main/java/org/apache/brooklyn/core/sensor/MaxConcurrencySensor.java --- @@ -0,0 +1,91 @@ +/* + * Licensed to the Apache

[GitHub] brooklyn-docs pull request #130: Update release process notes for release 0....

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-docs/pull/130 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] brooklyn-server pull request #520: Limit parallelism of start/stop steps on ...

Github user sjcorbett commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/520#discussion_r96844228 --- Diff: core/src/main/java/org/apache/brooklyn/util/core/task/ValueResolverIterator.java --- @@ -0,0 +1,173 @@ +package

[GitHub] brooklyn-server pull request #524: Misc improvements

Github user sjcorbett commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/524#discussion_r96843229 --- Diff: rest/rest-server/src/main/java/org/apache/brooklyn/rest/filter/BrooklynPropertiesSecurityFilter.java --- @@ -148,8 +148,11 @@ protected

[GitHub] brooklyn-server pull request #526: Add removeSecurityGroup by id

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/526 --- 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

[GitHub] brooklyn-server pull request #524: Misc improvements

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/524#discussion_r96841464 --- Diff: rest/rest-server/src/main/java/org/apache/brooklyn/rest/filter/BrooklynPropertiesSecurityFilter.java --- @@ -148,8 +148,11 @@ protected

[GitHub] brooklyn-server issue #526: Add removeSecurityGroup by id

Github user sjcorbett commented on the issue: https://github.com/apache/brooklyn-server/pull/526 Looks sensible. It would be good to see some tests for SecurityGroupEditor in a subsequent pull request. --- If your project is set up for it, you can reply to this email and have your

[GitHub] brooklyn-docs pull request #129: Update: winrm connectivity limitations

Github user drigodwin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/129#discussion_r96837520 --- Diff: guide/yaml/winrm/index.md --- @@ -515,11 +515,9 @@ Known Limitations ### Use of Unencrypted HTTP -Brooklyn is

[GitHub] brooklyn-docs pull request #129: Update: winrm connectivity limitations

Github user drigodwin commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/129#discussion_r96837642 --- Diff: guide/yaml/winrm/index.md --- @@ -515,11 +515,9 @@ Known Limitations ### Use of Unencrypted HTTP -Brooklyn is

[GitHub] brooklyn-docs pull request #138: Use full configkey name in docs rather than...

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-docs/pull/138 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] brooklyn-docs issue #138: Use full configkey name in docs rather than the sh...

Github user tbouron commented on the issue: https://github.com/apache/brooklyn-docs/pull/138 This follow [the proposal of deprecating the `@SetFromFlag` annotation](http://mail-archives.apache.org/mod_mbox/brooklyn-dev/201611.mbox/browser) --- If your project is set up for it, you

[jira] [Commented] (BROOKLYN-359) Remove from docs SL Object Store

[ https://issues.apache.org/jira/browse/BROOKLYN-359?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15829704#comment-15829704 ] ASF GitHub Bot commented on BROOKLYN-359: - Github user asfgit closed the pull request at:

[GitHub] brooklyn-docs pull request #139: BROOKLYN-359: fix softlayer swift example f...

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-docs/pull/139 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] brooklyn-server pull request #519: Add experimental CORS server support

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/519#discussion_r96816167 --- Diff: rest/rest-resources/src/main/java/org/apache/brooklyn/rest/filter/CorsImplSupplierFilter.java --- @@ -0,0 +1,81 @@ +/* + *