[GitHub] brooklyn-server issue #908: Added a DashboardAggregator

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/908 test failure fixed, only thing left to decide is whether this is the correct place or not ---

[GitHub] brooklyn-server pull request #908: Added a DashboardAggregator

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/908 Added a DashboardAggregator Added a DashboardAggregator. This aggregator will pull together dashboard sensors from it's children and merge them together. This however, does

Re: Brooklyn REST API - omitting fields in JSON objects

Hello, I agree with Thomas here. This seems like an API breaking change, and should be reserved for V2 if it at all. I lean towards reverting. Regards, Graeme On 8 November 2017 at 10:01, Thomas Bouron wrote: > Hi all. > > I'm not a fan of excluding fields

[GitHub] brooklyn-server pull request #884: Add brooklyn-cli as a feature to karaf

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/884 Add brooklyn-cli as a feature to karaf https://github.com/apache/brooklyn-server/pull/882 added brooklyn-cli to a catalog.bom. This causes brooklyn to fail to start, as karaf can't

[GitHub] brooklyn-library pull request #139: Update visitors-creation-script.sql

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-library/pull/139 Update visitors-creation-script.sql Need all privileges for three tier app You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] brooklyn-library pull request #136: Fix Library Feature File

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-library/pull/136 Fix Library Feature File **Issue** When building Brooklyn karaf, brooklyn-library-catalog is not included in the system folder. This means that on startup Karaf goes to a remote

[GitHub] brooklyn-server pull request #869: Reverted code for selecting subnet hostna...

Github user Graeme-Miller closed the pull request at: https://github.com/apache/brooklyn-server/pull/869 ---

[GitHub] brooklyn-server issue #869: Reverted code for selecting subnet hostname for ...

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/869 Spoke to @aledsage and we have agreed reverting #849 is the way to go. I have created #871 to do this ---

[GitHub] brooklyn-server pull request #871: Revert "Retrieve the right AWS metadata f...

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/871 Revert "Retrieve the right AWS metadata for private hostname" This reverts commit 8877368aa4c04e21eecd167538f878c4e0681900. Please see discussion on this

[GitHub] brooklyn-server issue #869: Reverted code for selecting subnet hostname for ...

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/869 @aledsage did a quick investigation and it looks like you are right. In AMP 4.8, our main.uri (which is highlighted in the UI) would be based off of the host.subnet.hostname so

[GitHub] brooklyn-server pull request #869: Reverted code for selecting subnet hostna...

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/869 Reverted code for selecting subnet hostname for non aws clouds A change was made to how Brooklyn retrieves the subnet hostname in https://github.com/apache/brooklyn-server/pull/849

[GitHub] brooklyn-server issue #869: Reverted code for selecting subnet hostname for ...

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/869 @tbouron and @andreaturli it would be good to get your opinion on this. ---

Re: [PROPOSAL] Delete "classic-mode" (always use Karaf)

+1 On 9 October 2017 at 12:56, Thomas Bouron wrote: > +1 > > On Mon, 9 Oct 2017 at 12:39 Geoff Macartney > wrote: > > > +1 > > > > On Mon, 9 Oct 2017 at 12:38 Duncan Godwin com> > > wrote: > > > > >

[GitHub] brooklyn-server pull request #848: Fix for NPE in BasicBrooklynCatalog

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/848 Fix for NPE in BasicBrooklynCatalog When adding a catalog item, through an exception if there is no brooklyn.catalog in the YAML You can merge this pull request into a Git

[GitHub] brooklyn-server pull request #845: Fix BasicBrooklynCatalog create Spec

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/845 Fix BasicBrooklynCatalog create Spec If the BasicBrooklynCatalog fails to load a catalog item, it will now fall back to the type registry. You can merge this pull request into a Git

[GitHub] brooklyn-server pull request #832: Add ManagementNodeStateListener support

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/832#discussion_r139903337 --- Diff: core/src/main/java/org/apache/brooklyn/core/mgmt/internal/ManagementNodeStateListenerManager.java --- @@ -0,0 +1,175

[GitHub] brooklyn-server pull request #832: Add ManagementNodeStateListener support

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/832#discussion_r139721438 --- Diff: core/src/main/java/org/apache/brooklyn/core/mgmt/internal/ManagementNodeStateListenerManager.java --- @@ -0,0 +1,175

[GitHub] brooklyn-client pull request #54: Fixed setting config. Added setting sensor...

Github user Graeme-Miller closed the pull request at: https://github.com/apache/brooklyn-client/pull/54 ---

[GitHub] brooklyn-library pull request #127: Update visitors-creation-script.sql

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-library/pull/127 Update visitors-creation-script.sql An issue was created by #812. That PR removed the password needed to create the user brooklyn'@'localhost You can merge this pull

[GitHub] brooklyn-server issue #823: Fix creation script

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/823 I have tested this on a MySQLNode and it works ---

[GitHub] brooklyn-server pull request #823: Fix creation script

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/823 Fix creation script An issue was created by https://github.com/apache/brooklyn-server/pull/812. That PR removed the password needed to create the user brooklyn'@'localhost

[GitHub] brooklyn-server pull request #818: More on adjunct highlights

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/818#discussion_r138817948 --- Diff: core/src/main/java/org/apache/brooklyn/core/objs/AbstractEntityAdjunct.java --- @@ -568,6 +570,98 @@ public void setUniqueTag

[GitHub] brooklyn-server pull request #818: More on adjunct highlights

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/818#discussion_r138651745 --- Diff: core/src/main/java/org/apache/brooklyn/core/objs/AbstractEntityAdjunct.java --- @@ -568,6 +570,98 @@ public void setUniqueTag

[GitHub] brooklyn-server pull request #818: More on adjunct highlights

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/818#discussion_r138642145 --- Diff: core/src/main/java/org/apache/brooklyn/core/objs/AbstractEntityAdjunct.java --- @@ -568,6 +570,98 @@ public void setUniqueTag

[GitHub] brooklyn-server pull request #818: More on adjunct highlights

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/818#discussion_r138642446 --- Diff: core/src/main/java/org/apache/brooklyn/core/objs/AbstractEntityAdjunct.java --- @@ -568,6 +570,98 @@ public void setUniqueTag

[GitHub] brooklyn-server pull request #818: More on adjunct highlights

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/818#discussion_r138642216 --- Diff: core/src/main/java/org/apache/brooklyn/core/objs/AbstractEntityAdjunct.java --- @@ -568,6 +570,98 @@ public void setUniqueTag

[GitHub] brooklyn-server pull request #818: More on adjunct highlights

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/818#discussion_r138642085 --- Diff: core/src/main/java/org/apache/brooklyn/core/objs/AbstractEntityAdjunct.java --- @@ -568,6 +570,98 @@ public void setUniqueTag

[GitHub] brooklyn-server pull request #818: More on adjunct highlights

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/818#discussion_r138638872 --- Diff: core/src/main/java/org/apache/brooklyn/core/objs/AbstractEntityAdjunct.java --- @@ -568,6 +570,98 @@ public void setUniqueTag

Re: REST API: Policy endpoint addition

PR Created: https://github.com/apache/brooklyn-server/pull/813 On 8 September 2017 at 12:08, Graeme Miller <gra...@cloudsoft.io> wrote: > Geoff, agree we should be adding it to adjunct, that'll be looked at as > part of a stage 2 (Alex is going to pick this up). Not sure about the te

[GitHub] brooklyn-server pull request #813: Added highlights to policy endpoint

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/813 Added highlights to policy endpoint Policies now return highlights from the REST API. Highlights are a map of ids -> (description, time, task id). They are used to expose m

Re: REST API: Policy endpoint addition

t;"taskId": 1345 >> }, >> ... >>} >> }] >> >> or maybe even better, something like this: >> [{ >> ... >>"highlights": [ >> { >>"name": "Last Confirmation", >>

REST API: Policy endpoint addition

12345689, "taskId": 1345 }, ... } }] Please shout if you have any problems with this, otherwise I'll submit a PR shortly with this change. Regards, Graeme Miller

[jira] [Created] (BROOKLYN-531) k8s DockerContainer location public ip

Graeme Miller created BROOKLYN-531: -- Summary: k8s DockerContainer location public ip Key: BROOKLYN-531 URL: https://issues.apache.org/jira/browse/BROOKLYN-531 Project: Brooklyn Issue Type

[jira] [Created] (BROOKLYN-530) InitScript for Windows machine doesn't run when USER_METADATA_STRING or USER_METADATA_MAP are defined

Graeme Miller created BROOKLYN-530: -- Summary: InitScript for Windows machine doesn't run when USER_METADATA_STRING or USER_METADATA_MAP are defined Key: BROOKLYN-530 URL: https://issues.apache.org/jira/browse

[jira] [Created] (BROOKLYN-529) Windows location keyPair issue

Graeme Miller created BROOKLYN-529: -- Summary: Windows location keyPair issue Key: BROOKLYN-529 URL: https://issues.apache.org/jira/browse/BROOKLYN-529 Project: Brooklyn Issue Type: Bug

[GitHub] brooklyn-server issue #796: Disabled AUTO_WRAP_CATALOG_YAML_AS_BUNDLE

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/796 no longer required --- 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

[GitHub] brooklyn-server issue #798: Fix template forward reference

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/798 I have tested locally and can confirm that this fixes the issue that I saw. This PR adds a test for that issue, but thought it was best to double check. This code looks good

[GitHub] brooklyn-server pull request #796: Disabled AUTO_WRAP_CATALOG_YAML_AS_BUNDLE

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/796 Disabled AUTO_WRAP_CATALOG_YAML_AS_BUNDLE You can merge this pull request into a Git repository by running: $ git pull https://github.com/Graeme-Miller/brooklyn-server

[GitHub] brooklyn-docs issue #205: use ~~~ rather than ```

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-docs/pull/205 @m4rkmckenna updated --- 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

[GitHub] brooklyn-docs pull request #205: use ~~~ rather than ```

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-docs/pull/205 use ~~~ rather than ``` This keeps this page in line with other pages. Also it was causing issues in a downstream project. You can merge this pull request into a Git repository

[GitHub] brooklyn-server pull request #770: Add null checks to Azure default network

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/770 Add null checks to Azure default network You can merge this pull request into a Git repository by running: $ git pull https://github.com/Graeme-Miller/brooklyn-server

[GitHub] brooklyn-server pull request #755: Added endpoint test boms

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/755 Added endpoint test boms You can merge this pull request into a Git repository by running: $ git pull https://github.com/Graeme-Miller/brooklyn-server add_endpoint_test_boms

[GitHub] brooklyn-server issue #745: Azure ARM default network fixes from PR suggesti...

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/745 @neykov I have rebased --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] brooklyn-server pull request #747: BROOKLYN-520: better error if type missin...

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/747#discussion_r124574322 --- Diff: camp/camp-brooklyn/src/main/java/org/apache/brooklyn/camp/brooklyn/spi/creation/BrooklynComponentTemplateResolver.java

[GitHub] brooklyn-server issue #745: Azure ARM default network fixes from PR suggesti...

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/745 @neykov that's a good point about the default networking functionality being different in 2.0.1. I'd still recommend merging this PR as it is useful for downstream projects that use

[GitHub] brooklyn-server pull request #745: Azure ARM default network fixes from PR s...

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/745 Azure ARM default network fixes from PR suggestions Important Note: Azure ARM doesn't currently work with the master of Brooklyn. Currently, after a Azure ARM VM is created, JClouds

[GitHub] brooklyn-server issue #742: DefaultAzureArmNetworkCreator improvements

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/742 lgtm thanks @aledsage --- 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-docs pull request #200: Fixed advanced example by shipping blueprin...

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-docs/pull/200 Fixed advanced example by shipping blueprints in repo You can merge this pull request into a Git repository by running: $ git pull https://github.com/Graeme-Miller/brooklyn-docs

[GitHub] brooklyn-server pull request #742: DefaultAzureArmNetworkCreator improvement...

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/742#discussion_r123482786 --- Diff: locations/jclouds/src/test/java/org/apache/brooklyn/location/jclouds/networking/creator/DefaultAzureArmNetworkCreatorTest.java

[GitHub] brooklyn-server pull request #742: DefaultAzureArmNetworkCreator improvement...

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/742#discussion_r123482551 --- Diff: locations/jclouds/src/test/java/org/apache/brooklyn/location/jclouds/networking/creator/DefaultAzureArmNetworkCreatorTest.java

[GitHub] brooklyn-server pull request #739: A default network is now created when dep...

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/739#discussion_r123230837 --- Diff: locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/networking/creator/DefaultAzureArmNetworkCreator.java

[jira] [Commented] (BROOKLYN-519) Persisting yaml blueprint with $brooklyn:entitySpec stores all configs

[ https://issues.apache.org/jira/browse/BROOKLYN-519?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16057288#comment-16057288 ] Graeme Miller commented on BROOKLYN-519: You mentioned 0.10.0 of brooklyn. Have you confirmed

[GitHub] brooklyn-server issue #739: A default network is now created when deploying ...

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/739 The failure was due to my test class having an un-approve licence. I have updated the class and pushed- this should force a retest --- If your project is set up for it, you can reply

[GitHub] brooklyn-server pull request #739: A default network is now created when dep...

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/739 A default network is now created when deploying to Azure ARM In an upcoming release of JClouds, support for Azure ARM will be added. The JClouds code for Azure ARM will place each

[GitHub] brooklyn-client issue #42: Improve login handling.

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-client/pull/42 Just want to chime in with a different opinion. I think we should merge as is, and not implement the functionality to allow users to switch between multiple hosts. For me

[GitHub] brooklyn-client issue #54: Fixed setting config. Added setting sensors

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-client/pull/54 Actually, maybe all that is needed is some clarity about how to use this in the help documentation and some better error information. I was fixing this because of this Jira

[GitHub] brooklyn-client pull request #54: Fixed setting config. Added setting sensor...

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-client/pull/54 Fixed setting config. Added setting sensors You can merge this pull request into a Git repository by running: $ git pull https://github.com/Graeme-Miller/brooklyn-client

Re: Is the deploy user available via the API?

If it is the Brooklyn user you're talking about, it should not be difficult to add. Just amend this line to include the user. You can get the

[jira] [Commented] (BROOKLYN-469) CLI: Panic when login attempted with malformed URL

[ https://issues.apache.org/jira/browse/BROOKLYN-469?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16040832#comment-16040832 ] Graeme Miller commented on BROOKLYN-469: [~sjcorbett] I just tested this and got the following

Re: [VOTE] Accept contribution of Brooklyn Container Service

+1 On 31 May 2017 at 12:43, Geoff Macartney wrote: > +1 binding > > On Wed, 31 May 2017 at 12:19 Andrew Kennedy < > andrew.kenn...@cloudsoftcorp.com> wrote: > > > +1 Binding > > > > This is a really useful addition to Brooklyn. It will allow users to > target

[GitHub] brooklyn-client pull request #42: Improve login handling.

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-client/pull/42#discussion_r118496514 --- Diff: cli/io/config.go --- @@ -70,3 +98,111 @@ func (config *Config) Read() { dec := json.NewDecoder(fileToRead) dec.Decode

[GitHub] brooklyn-client pull request #42: Improve login handling.

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-client/pull/42#discussion_r118458597 --- Diff: cli/io/config.go --- @@ -70,3 +98,111 @@ func (config *Config) Read() { dec := json.NewDecoder(fileToRead) dec.Decode

[GitHub] brooklyn-client pull request #42: Improve login handling.

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-client/pull/42#discussion_r118458240 --- Diff: cli/br/brooklyn.go --- @@ -30,27 +30,14 @@ import ( "path/filepath" ) -func getNetworkCredentials

[GitHub] brooklyn-client pull request #42: Improve login handling.

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-client/pull/42#discussion_r118455814 --- Diff: cli/commands/login.go --- @@ -53,6 +58,30 @@ func (cmd *Login) Metadata() command_metadata.CommandMetadata

[GitHub] brooklyn-client pull request #42: Improve login handling.

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-client/pull/42#discussion_r118458766 --- Diff: cli/br/brooklyn.go --- @@ -30,27 +30,14 @@ import ( "path/filepath" ) -func getNetworkCredentials

[GitHub] brooklyn-client pull request #52: Delete requrest now only fail is an unsucc...

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-client/pull/52 Delete requrest now only fail is an unsuccessful code is returned You can merge this pull request into a Git repository by running: $ git pull https://github.com/Graeme-Miller

[GitHub] brooklyn-docs pull request #186: Add todo

Github user Graeme-Miller closed the pull request at: https://github.com/apache/brooklyn-docs/pull/186 --- 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

[jira] [Commented] (BROOKLYN-472) YAML anchors cause failure when adding YAML blueprint to catalog

[ https://issues.apache.org/jira/browse/BROOKLYN-472?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16020957#comment-16020957 ] Graeme Miller commented on BROOKLYN-472: [~Nakomis] could you confirm that this is because

[GitHub] brooklyn-docs pull request #189: Split catalog

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-docs/pull/189 Split catalog You can merge this pull request into a Git repository by running: $ git pull https://github.com/Graeme-Miller/brooklyn-docs split_catalog Alternatively you can

[GitHub] brooklyn-docs pull request #186: Add todo

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-docs/pull/186 Add todo You can merge this pull request into a Git repository by running: $ git pull https://github.com/Graeme-Miller/brooklyn-docs addTodo Alternatively you can review

[GitHub] brooklyn-docs pull request #180: Update bundle upload

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-docs/pull/180#discussion_r117458781 --- Diff: guide/blueprints/catalog/index.md --- @@ -375,153 +375,120 @@ the latest non-snapshot version will be loaded when an entity

Code Contribution: Container Service

are in favour of this, we can provide more deatails about the code and put it to a vote. Regards, Graeme Miller [1] https://docs.cloudsoft.io/ccs/locations/index.html

[GitHub] brooklyn-docs pull request #180: Update bundle upload

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-docs/pull/180 Update bundle upload You can merge this pull request into a Git repository by running: $ git pull https://github.com/Graeme-Miller/brooklyn-docs updateBundleUpload Alternatively

[GitHub] brooklyn-server issue #605: Add series of meta effectors for composite opera...

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/605 @grkvlt this PR is rather large, and missing tests. I suggest it is closed for now and we work out a way to break this down. Some ideas for how to break it down

Re: Faster PR reviews

iple thousands of lines added is in my opinion getting too big to review- and you're not the only one to create a PR like that! Regards, Graeme Miller [1] In the interests of transparency, full data set here: https://docs.google.com/spreadsheets/d/1zDXyhs9AOW1s3f 0Zhd3oZlAW4nKdn7hLPd-118zwPhw

[GitHub] brooklyn-server issue #645: Persist OSGi bundles

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/645 I have briefly reviewed the code and I am in the process of testing it. If you guys are happy to merge go ahead- I'll keep on with testing just in case --- If your project is set

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

[ 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

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

[ 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

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

[ 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

Re: Next release candidate

Hi Richard, Duncan Grant, should this PR be merged in as it fixes node.js requirements: https://github.com/apache/brooklyn-library/pull/98 Also, there are two fixes here for rebind: https://github.com/apache/brooklyn-server/pull/633 https://github.com/apache/brooklyn-server/pull/634 Regards,

[GitHub] brooklyn-server pull request #636: Enable older Java versions

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/636#discussion_r111724361 --- Diff: utils/common/src/main/java/org/apache/brooklyn/util/ssh/BashCommands.java --- @@ -621,6 +621,7 @@ private static Object getFlag(Map

[GitHub] brooklyn-server issue #634: BROOKLYN-474: rebind to policy with private cons...

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/634 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

[GitHub] brooklyn-server issue #623: BundleMaker copy fix

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/623 @drigodwin no, it doesn't. If we were creating a zip from scratch we would do what I have done here, and create a zip entry with only the name. The zip code would then figure out

[GitHub] brooklyn-server issue #623: BundleMaker copy fix

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/623 Can you provide steps to re-produce @m4rkmckenna quickest way to replicate is to use the brooklyn cli from here: https://github.com/apache/brooklyn-client/pull/44. Using that CLI

[GitHub] brooklyn-server pull request #623: BundleMaker copy fix

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/623 BundleMaker copy fix When cloning a zip, we were using the ZipEntry from the input zip. This causes issues, as the info in the input ZipEntry is not necessarily correct

[GitHub] brooklyn-client pull request #44: Added the ability to upload a zip via the ...

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-client/pull/44 Added the ability to upload a zip via the CLI @geomacy would you please review? You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] brooklyn-server pull request #616: Fixed BlobStoreContextFactoryImpl. It can...

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/616 Fixed BlobStoreContextFactoryImpl. It can now have dsl in its config You can merge this pull request into a Git repository by running: $ git pull https://github.com/Graeme

[GitHub] brooklyn-server issue #594: Revert "add original failing test from @sjcorbet...

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/594 Ok, so the circular dependency is: * dependency has been added between brooklyn-rest-server and brooklyn-software-nosql * brooklyn-rest-server is a part of brooklyn-server

[GitHub] brooklyn-server pull request #594: Revert "add original failing test from @s...

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/594 Revert "add original failing test from @sjcorbett" This reverts commit d7cbaeed46d7fedf8e49df015baa812352060f45. That commit added a dependency on brooklyn-soft

[GitHub] brooklyn-server issue #592: Fix publication of EC2 host address in DefaultCo...

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/592 Tested locally, works as expected: ![screen shot 2017-03-13 at 17 24 51](https://cloud.githubusercontent.com/assets/3742990/23866871/1a04afc0-0812-11e7-90bb-755c3667f3b1.png

[jira] [Created] (BROOKLYN-447) waitForSshable deprecated without replacement

Graeme Miller created BROOKLYN-447: -- Summary: waitForSshable deprecated without replacement Key: BROOKLYN-447 URL: https://issues.apache.org/jira/browse/BROOKLYN-447 Project: Brooklyn Issue

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

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

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

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-server/pull/554 @aledsage has pointed out a way to get around the problem I am having, so I no longer need this PR. However, I think it might still be useful @neykov, the use case here

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

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-server/pull/554 Ensured the \n character is escaped when escaping a string for bash You can merge this pull request into a Git repository by running: $ git pull https://github.com/Graeme

[GitHub] brooklyn-docs pull request #147: Added knwon issues section to azure arm

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-docs/pull/147 Added knwon issues section to azure arm You can merge this pull request into a Git repository by running: $ git pull https://github.com/Graeme-Miller/brooklyn-docs

[GitHub] brooklyn-docs pull request #142: Added docs for useMachinePublicAddressAsPri...

GitHub user Graeme-Miller opened a pull request: https://github.com/apache/brooklyn-docs/pull/142 Added docs for useMachinePublicAddressAsPrivateAddress You can merge this pull request into a Git repository by running: $ git pull https://github.com/Graeme-Miller/brooklyn-docs

[GitHub] brooklyn-client issue #38: Exclude cli/.glide/* (cache) from rat + git

Github user Graeme-Miller commented on the issue: https://github.com/apache/brooklyn-client/pull/38 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

[GitHub] brooklyn-client pull request #38: Exclude cli/.glide/* (cache) from rat + gi...

Github user Graeme-Miller commented on a diff in the pull request: https://github.com/apache/brooklyn-client/pull/38#discussion_r97389393 --- Diff: cli/pom.xml --- @@ -101,6 +101,7 @@ vendor

[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

[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

  1   2   >