[jira] [Updated] (BROOKLYN-439) Existing `userMetadata` cannot be used with WinRM deployments on certain clouds

[ https://issues.apache.org/jira/browse/BROOKLYN-439?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mike Zaccardo updated BROOKLYN-439: --- Environment: AWS Vcloud Director Description: *Applies to AWS and Vcloud Director

[jira] [Created] (BROOKLYN-439) Existing `userMetadata` cannot be used with WinRM deployments

Mike Zaccardo created BROOKLYN-439: -- Summary: Existing `userMetadata` cannot be used with WinRM deployments Key: BROOKLYN-439 URL: https://issues.apache.org/jira/browse/BROOKLYN-439 Project:

[GitHub] brooklyn-library issue #34: Fix sed command that update the MySQL configurat...

Github user tbouron commented on the issue: https://github.com/apache/brooklyn-library/pull/34 Close in favor or #76 (my PR wasn't really working anyway) --- 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

[GitHub] brooklyn-library pull request #34: Fix sed command that update the MySQL con...

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

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

See

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

See

[GitHub] brooklyn-server issue #480: Config self reference fix

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/480 Pretty sure the failures reported [here](https://builds.apache.org/job/brooklyn-server-pull-requests/1728/) are unrelated: ``` *

[GitHub] brooklyn-server issue #531: initial work to support HttpEntity

Github user andreaturli commented on the issue: https://github.com/apache/brooklyn-server/pull/531 I will open a PR for docs ASAP, thanks @ahgittin for merging it --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] brooklyn-server pull request #555: Add DurationPredicates

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/555 --- 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 #531: initial work to support HttpEntity

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/531 --- 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 issue #531: initial work to support HttpEntity

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/531 very nice @andreaturli -- and excellent tests. is there a corresponding PR against the docs to explain how to use this? looks good so merging --- If your project is set

[GitHub] brooklyn-server issue #546: Sequencer entity

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/546 I feel like this will soon be superseded, by being able to point at some shared-scoped atomic integer and request a `getAndIncrement` directly in YAML as part of `start`. Haven't looked

[GitHub] brooklyn-server issue #554: Ensured the \n character is escaped when escapin...

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/554 -1 I think some shells don't respect `\n` so this could break things all over the place. The norm in shells is for double-quoted strings to be allowed to have a new line, so we

[GitHub] brooklyn-server issue #555: Add DurationPredicates

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/555 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if

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

See Changes: [alex] migrate config inheritance to new classes [alex] document better config inheritance semantics [alex] implement inheritance of config default values [alex] uncomment another assertion about inherited config

[jira] [Commented] (BROOKLYN-267) brooklyn.parameter default value not picked up via inherited config

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

[GitHub] brooklyn-server pull request #462: Inherit config default values

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

Github user sjcorbett commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/529#discussion_r101334570 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/BasicLocationNetworkInfoCustomizer.java --- @@ -0,0 +1,472 @@

[GitHub] brooklyn-server pull request #462: Inherit config default values

Github user neykov commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/462#discussion_r101306815 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/guava/Maybe.java --- @@ -457,6 +457,8 @@ public boolean equals(Object obj) {

[GitHub] brooklyn-server issue #462: Inherit config default values

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/462 Good spot. Will fix and 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

[GitHub] brooklyn-server issue #462: Inherit config default values

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/462 Could cause an issue if absents are used as keys with the intention that they all be different but that would be weird. Have checked our hashCode and it's a constant for absents

[GitHub] brooklyn-server pull request #462: Inherit config default values

Github user geomacy commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/462#discussion_r101298270 --- Diff: core/src/main/java/org/apache/brooklyn/core/objs/BrooklynDynamicType.java --- @@ -175,20 +179,29 @@ protected static void

Re: [DISCUSSION] Determining which Brooklyn node address to advertise

> What is the benefit of the "management-private-" group? It seems > superfluous to me if "management-private-ingress-" will contain the > public IPs of the members of the cluster. Needed for in-cloud access. Even if the public IP is white-listed, local access will use the private IP and will

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

See -- [...truncated 22131 lines...] === [GC 715984K->480455K(759296K), 0.0160060 secs] Tests run: 2029, Failures: 1, Errors: 0, Skipped: 0, Time elapsed:

Re: [DISCUSSION] Determining which Brooklyn node address to advertise

Thanks for clarifying. It was this cross-cloud case I was considering. What is the benefit of the "management-private-" group? It seems superfluous to me if "management-private-ingress-" will contain the public IPs of the members of the cluster. On 15/02/2017 14:16, Svetoslav Neykov wrote:

Re: [DISCUSSION] Determining which Brooklyn node address to advertise

My response below was based on my thinking about in-cloud access between Brooklyn and the managed machines. When the access is cross-cloud we indeed need _c_ security groups. Each one containing _n_ records with the public IPs of the HA cluster members. These can be managed by Brooklyn though.

[GitHub] brooklyn-server pull request #559: Activity api include background

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

Re: [DISCUSSION] Determining which Brooklyn node address to advertise

> You propose that the manager of the Brooklyn HA cluster maintain at least _c_ > security groups (one per security group scope - e.g. AWS EC2 region - per > cloud). Not quite. It's the number of "clouds HA cluster is deployed to". That's the number of availability zones - usually a low number

[GitHub] brooklyn-server issue #559: Activity api include background

Github user tbouron commented on the issue: https://github.com/apache/brooklyn-server/pull/559 Thanks @ahgittin! Tested works great, LGTM. PS: Jenkins failure looks unrelated: ```

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

See

Re: [DISCUSSION] Determining which Brooklyn node address to advertise

Interesting problem Svet. Your proposal is a neat way of sidestepping the problem of updating many security groups as the set of HA nodes changes. Let me rephrase it to see if I understand correctly. Suppose we have: * _n_ Brooklyn servers in an HA cluster (that could be across many clouds /

entity/.../activities endpoint minor API change for background tasks

Hi All- In [1], @tbouron and I add an "includeBackground" query parameter to the `/activities//children` endpoint. This is useful as you can see which tasks were "submitted by" the given task, even if they aren't explicit sub-tasks. Typically these are less interesting eg message

[GitHub] brooklyn-server pull request #521: fix unexpected AddChildrenEffector behavi...

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

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

See -- [...truncated 22129 lines...] Tests run: 2029, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 181.453 sec <<< FAILURE! - in TestSuite

[GitHub] brooklyn-server pull request #516: Add unit tests for Karaf features

Github user m4rkmckenna commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/516#discussion_r101255832 --- Diff: karaf/features/src/main/feature/feature.xml --- @@ -260,7 +260,7 @@ https://github.com/square/okhttp/pull/2246

[GitHub] brooklyn-server pull request #516: Add unit tests for Karaf features

Github user m4rkmckenna commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/516#discussion_r101255259 --- Diff: karaf/itests/pom.xml --- @@ -0,0 +1,180 @@ + + +http://maven.apache.org/POM/4.0.0;

[GitHub] brooklyn-server pull request #516: Add unit tests for Karaf features

Github user m4rkmckenna commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/516#discussion_r101255165 --- Diff: karaf/itests/pom.xml --- @@ -0,0 +1,180 @@ + + +http://maven.apache.org/POM/4.0.0;

[GitHub] brooklyn-server issue #557: transfer icons from catalog items to entities

Github user drigodwin commented on the issue: https://github.com/apache/brooklyn-server/pull/557 This looks fine, what are the sources and licenses of the icons @ahgittin? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] brooklyn-server issue #557: transfer icons from catalog items to entities

Github user drigodwin commented on the issue: https://github.com/apache/brooklyn-server/pull/557 This looks fine, what are the sources and licenses of the icons @ahgittin? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] brooklyn-server pull request #558: BROOKLYN-433: regex/obj config key constr...

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

[jira] [Commented] (BROOKLYN-433) YAML-based config constraint to support regex

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

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

See -- [...truncated 2050 lines...] 2017-02-15 09:39:24,763 INFO TESTNG PASSED: "Surefire test" - org.apache.brooklyn.util.text.QuotedStringTokenizerTest.testQuoting() finished in 1 ms 2017-02-15

[GitHub] brooklyn-client pull request #39: adds flag for connecting to servers using ...

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