Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/852#discussion_r144297930
--- Diff:
core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogInitialization.java
---
@@ -128,164 +146,255 @@ public void
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/852#discussion_r144315009
--- Diff: launcher/src/test/resources/rebind-test-empty-catalog.bom ---
@@ -0,0 +1,25 @@
+#
+# Licensed to the Apache Software Foundation
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/852#discussion_r144315802
--- Diff:
launcher/src/test/java/org/apache/brooklyn/launcher/AbstractBrooklynLauncherRebindTest.java
---
@@ -0,0 +1,183
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/852#discussion_r144296911
--- Diff:
core/src/main/java/org/apache/brooklyn/core/catalog/internal/BasicBrooklynCatalog.java
---
@@ -1808,6 +1808,42 @@ public void addItem
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-server/pull/862
retest this please
---
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-server/pull/777
@ahgittin NP, I don't contest the why, I'm sure there was a good reason
(confirmed by the above explanation) I was just surprised that this wasn't
flagged before for downstream project
GitHub user tbouron opened a pull request:
https://github.com/apache/brooklyn-dist/pull/104
WIP - Improve RPM/DEB packages
The aim of this PR is to:
1. use Karaf launcher
2. improve the package to to handle properly installations and upgrades
This is spirit
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-dist/pull/104#discussion_r138617661
--- Diff:
karaf/apache-brooklyn/src/main/filtered-resources/etc/brooklyn.cfg ---
@@ -0,0 +1,24
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-dist/pull/104
@drigodwin Comment addressed except for the `Depends` package (see my
comment above)
---
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-dist/pull/105
retest this please
---
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-dist/pull/104#discussion_r138617340
--- Diff: deb-packaging/deb/control/control ---
@@ -19,6 +19,6 @@ Version: [[version]]
Section: misc
Priority: optional
Architecture
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/819#discussion_r138585632
--- Diff:
core/src/main/java/org/apache/brooklyn/entity/group/DynamicFabric.java ---
@@ -50,13 +51,20 @@
+ "for each
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/819#discussion_r138584451
--- Diff:
core/src/main/java/org/apache/brooklyn/core/entity/Attributes.java ---
@@ -52,11 +52,15 @@
// TODO these should switch
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/819#discussion_r138584470
--- Diff:
core/src/main/java/org/apache/brooklyn/core/entity/Attributes.java ---
@@ -52,11 +52,15 @@
// TODO these should switch
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/819#discussion_r138585654
--- Diff:
core/src/main/java/org/apache/brooklyn/entity/group/DynamicFabric.java ---
@@ -50,13 +51,20 @@
+ "for each
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/210#discussion_r138633409
--- Diff: guide/start/running.md ---
@@ -182,7 +182,7 @@ Apache Brooklyn should now have been installed and be
running as a system servic
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/210#discussion_r138629659
--- Diff: guide/ops/configuration/https.md ---
@@ -21,7 +21,7 @@ The passwords above should be changed to your own values.
Omit those arguments
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/210#discussion_r138631937
--- Diff: guide/ops/high-availability/high-availability-supplemental.md ---
@@ -12,34 +12,46 @@ and provides an example of how to configure a pair
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/210#discussion_r138628703
--- Diff: guide/ops/configuration/brooklyn_cfg.md ---
@@ -214,7 +204,7 @@ or
See [HTTPS Configuration](https.html) for general information
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/210#discussion_r138631625
--- Diff: guide/ops/high-availability/high-availability-supplemental.md ---
@@ -12,34 +12,46 @@ and provides an example of how to configure a pair
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/210#discussion_r138628440
--- Diff: guide/ops/configuration/brooklyn_cfg.md ---
@@ -33,7 +28,7 @@
brooklyn.webconsole.security.user.admin.password=AdminPassw0rd
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/210#discussion_r138631452
--- Diff: guide/ops/high-availability/high-availability-supplemental.md ---
@@ -12,34 +12,46 @@ and provides an example of how to configure a pair
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/210#discussion_r138632642
--- Diff: guide/ops/security-guidelines.md ---
@@ -37,7 +37,7 @@ relevant mount points, disks and directories.
For credential storage
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/210#discussion_r138633481
--- Diff: guide/start/running.md ---
@@ -205,19 +205,19 @@ The application should then output its logs to
`/var/log/brooklyn/apache-brookly
Now
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/210#discussion_r138629521
--- Diff: guide/ops/configuration/cors.md ---
@@ -0,0 +1,45 @@
+---
+title: CORS Configuration
+layout: website-normal
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/210#discussion_r138633454
--- Diff: guide/start/running.md ---
@@ -195,7 +195,7 @@ Apache Brooklyn should now have been installed and be
running as a system servic
$ sudo
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/210#discussion_r138633009
--- Diff: guide/start/brooklyn.properties ---
@@ -17,7 +17,7 @@
# under the License.
#
# This is Brooklyn's dot-properties file
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/210#discussion_r138631341
--- Diff: guide/ops/high-availability/high-availability-supplemental.md ---
@@ -12,34 +12,46 @@ and provides an example of how to configure a pair
GitHub user tbouron opened a pull request:
https://github.com/apache/brooklyn-dist/pull/105
Update Brooklyn vagrant
This updates the vagrant configuration to:
- use the latest CentOS
- use the new RPM package to install Brooklyn.
It requires #104 to be merged first
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-dist/pull/110#discussion_r140713075
--- Diff: release/verify_brooklyn_rc.sh ---
@@ -105,65 +115,74 @@ for ARCHIVE in $(find * -type f ! \( -name '*.asc' -o
-name '*.md5' -o -name
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/212#discussion_r140823147
--- Diff: guide/ops/upgrade.md ---
@@ -0,0 +1,356 @@
+---
+title: Upgrade
+layout: website-normal
+---
+
+This guide provides
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/212#discussion_r140824045
--- Diff: guide/ops/upgrade.md ---
@@ -0,0 +1,356 @@
+---
+title: Upgrade
+layout: website-normal
+---
+
+This guide provides
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/212#discussion_r140823246
--- Diff: guide/ops/upgrade.md ---
@@ -0,0 +1,356 @@
+---
+title: Upgrade
+layout: website-normal
+---
+
+This guide provides
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/212#discussion_r140822950
--- Diff: guide/ops/upgrade.md ---
@@ -0,0 +1,356 @@
+---
+title: Upgrade
+layout: website-normal
+---
+
+This guide provides
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-docs/pull/212
@aledsage Comments addressed
---
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-dist/pull/110
retest this please
---
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-server/pull/821
BTW, I just realised that there is no tests coverage for the new endpoints
`Entity Adjuncts` and `Entity Adjuncts Config`. Can you add that please
@ahgittin ?
---
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-server/pull/834
Indeed, just rebased on latest master @ahgittin
---
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/829#discussion_r141829990
--- Diff:
policy/src/main/java/org/apache/brooklyn/policy/ha/ServiceRestarter.java ---
@@ -133,6 +135,10 @@ protected synchronized void
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/829#discussion_r141826617
--- Diff:
policy/src/main/java/org/apache/brooklyn/policy/ha/ServiceRestarter.java ---
@@ -133,6 +135,10 @@ protected synchronized void
GitHub user tbouron opened a pull request:
https://github.com/apache/brooklyn-dist/pull/111
Fix vagrant installation folder when downloading dist version
The name of the tar.gz archive is different for the dist version (i.e.
`apache-brooklyn--bin`) and therefore, the name
GitHub user tbouron opened a pull request:
https://github.com/apache/brooklyn-library/pull/132
Update package for chef bean due to the move to brooklyn-library
This fixes a little mistake I let in when I reviewed #129
You can merge this pull request into a Git repository by running
GitHub user tbouron opened a pull request:
https://github.com/apache/brooklyn-docs/pull/218
Add link to download the br CLI
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/tbouron/brooklyn-docs patch-3
Alternatively you can
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-server/pull/846
LGTM. Tested by launching Brooklyn in both karaf and classic mode: couldn't
see any issue in the logs
---
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/849#discussion_r142135330
--- Diff:
locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/JcloudsLocation.java
---
@@ -2957,7 +2957,7 @@ String
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/849#discussion_r142150833
--- Diff:
locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/JcloudsLocation.java
---
@@ -2957,7 +2957,7 @@ String
GitHub user tbouron opened a pull request:
https://github.com/apache/brooklyn-server/pull/849
Retrieve the right AWS metadata for private hostname
In most conditions for AWS deployments (machine sshable) Brooklyn was
retrieving the `public-hostname`
[metadata](http
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/849#discussion_r142158986
--- Diff:
locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/JcloudsLocation.java
---
@@ -2957,7 +2957,7 @@ String
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/849#discussion_r142170104
--- Diff:
locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/JcloudsLocation.java
---
@@ -2957,7 +2957,7 @@ String
GitHub user tbouron opened a pull request:
https://github.com/apache/brooklyn-library/pull/133
Update config keys' deprecated names
Since Brooklyn 0.12.0 ([this
PR](https://github.com/apache/brooklyn-server/pull/819) precisely) some config
keys' name have been deprecated generated
GitHub user tbouron opened a pull request:
https://github.com/apache/brooklyn-docs/pull/219
Update config keys' deprecated names
Since Brooklyn 0.12.0 ([this
PR](https://github.com/apache/brooklyn-server/pull/819) precisely) some config
keys' name have been deprecated generated
GitHub user tbouron opened a pull request:
https://github.com/apache/brooklyn-server/pull/851
Update config keys' deprecated names
Since Brooklyn 0.12.0 ([this
PR](https://github.com/apache/brooklyn-server/pull/819) precisely) some config
keys' name have been deprecated generated
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/849#discussion_r142173795
--- Diff:
locations/jclouds/src/main/java/org/apache/brooklyn/location/jclouds/JcloudsLocation.java
---
@@ -2829,16 +2829,11 @@ protected String
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-server/pull/821
Cool, thanks @ahgittin. I'm reviewing it right now.
---
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-server/pull/826
Looks like something is wrong on Jenkins @aledsage
```
org.apache.brooklyn.camp.brooklyn.VanillaSoftwareProcessYamlTest.testAlternativeServiceUpPolling()
Build timed out (after
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-server/pull/821
@ahgittin The description says `This is then just one commit (currently)
which:` but there is still 1000+ additions. Is this up to date and rebase on
master?
---
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141604573
--- Diff:
rest/rest-resources/src/main/java/org/apache/brooklyn/rest/resources/AdjunctResource.java
---
@@ -0,0 +1,264 @@
+/*
+ * Licensed
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141595976
--- Diff:
rest/rest-api/src/main/java/org/apache/brooklyn/rest/api/AdjunctApi.java ---
@@ -0,0 +1,233 @@
+/*
+ * Licensed to the Apache
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141590904
--- Diff: core/src/main/java/org/apache/brooklyn/core/policy/Policies.java
---
@@ -81,11 +85,27 @@ public void setEntity(EntityLocal entity
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141604725
--- Diff:
rest/rest-resources/src/main/java/org/apache/brooklyn/rest/resources/AdjunctResource.java
---
@@ -0,0 +1,264 @@
+/*
+ * Licensed
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141604605
--- Diff:
rest/rest-resources/src/main/java/org/apache/brooklyn/rest/resources/AdjunctResource.java
---
@@ -0,0 +1,264 @@
+/*
+ * Licensed
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141604635
--- Diff:
rest/rest-resources/src/main/java/org/apache/brooklyn/rest/resources/AdjunctResource.java
---
@@ -0,0 +1,264 @@
+/*
+ * Licensed
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141596041
--- Diff:
rest/rest-api/src/main/java/org/apache/brooklyn/rest/api/AdjunctApi.java ---
@@ -0,0 +1,233 @@
+/*
+ * Licensed to the Apache
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141594992
--- Diff:
rest/rest-api/src/main/java/org/apache/brooklyn/rest/api/AdjunctApi.java ---
@@ -0,0 +1,233 @@
+/*
+ * Licensed to the Apache
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141602792
--- Diff:
rest/rest-api/src/main/java/org/apache/brooklyn/rest/domain/CatalogLocationSummary.java
---
@@ -26,6 +26,8 @@
import
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141602780
--- Diff:
rest/rest-api/src/main/java/org/apache/brooklyn/rest/domain/CatalogItemSummary.java
---
@@ -36,6 +36,8 @@
* see also, subclasses
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141588659
--- Diff: api/src/main/java/org/apache/brooklyn/api/sensor/Feed.java ---
@@ -43,7 +43,7 @@
/**
* True if everything has
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141605849
--- Diff:
rest/rest-resources/src/main/java/org/apache/brooklyn/rest/transform/CatalogTransformer.java
---
@@ -71,6 +71,8 @@
import
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141595948
--- Diff:
rest/rest-api/src/main/java/org/apache/brooklyn/rest/api/AdjunctApi.java ---
@@ -0,0 +1,233 @@
+/*
+ * Licensed to the Apache
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141589379
--- Diff:
core/src/main/java/org/apache/brooklyn/core/entity/EntityInternal.java ---
@@ -189,7 +189,7 @@
void remove(AttributeSensor
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141602725
--- Diff:
rest/rest-api/src/main/java/org/apache/brooklyn/rest/domain/CatalogEnricherSummary.java
---
@@ -27,6 +27,8 @@
import
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141601297
--- Diff:
rest/rest-api/src/main/java/org/apache/brooklyn/rest/api/PolicyApi.java ---
@@ -36,6 +36,8 @@
@Api("Entity Policies")
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141602815
--- Diff:
rest/rest-api/src/main/java/org/apache/brooklyn/rest/domain/CatalogPolicySummary.java
---
@@ -27,6 +27,8 @@
import
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141590729
--- Diff: core/src/main/java/org/apache/brooklyn/core/policy/Policies.java
---
@@ -81,11 +85,27 @@ public void setEntity(EntityLocal entity
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141607044
--- Diff:
rest/rest-resources/src/main/java/org/apache/brooklyn/rest/util/BrooklynRestResourceUtils.java
---
@@ -130,6 +143,36 @@ public Policy
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141604648
--- Diff:
rest/rest-resources/src/main/java/org/apache/brooklyn/rest/resources/AdjunctResource.java
---
@@ -0,0 +1,264 @@
+/*
+ * Licensed
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141602765
--- Diff:
rest/rest-api/src/main/java/org/apache/brooklyn/rest/domain/CatalogEntitySummary.java
---
@@ -27,6 +27,8 @@
import
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r141602412
--- Diff:
rest/rest-api/src/main/java/org/apache/brooklyn/rest/domain/AdjunctSummary.java
---
@@ -0,0 +1,150 @@
+/*
+ * Licensed
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-server/pull/821
@ahgittin Sorry, I did miss your comment. I was suggesting type to be
consistent with the `DELETE` method. I guess `adjunctType` is fine for now
(`superType` is really confusing for me
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-server/pull/849
@andreaturli Looks like it works as expected :)
https://user-images.githubusercontent.com/2082759/31118084-71611280-a82c-11e7-9862-dabf86a0cfbb.png;>
---
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/821#discussion_r142349155
--- Diff:
rest/rest-api/src/main/java/org/apache/brooklyn/rest/api/AdjunctApi.java ---
@@ -0,0 +1,233 @@
+/*
+ * Licensed to the Apache
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-server/pull/851
retest this please
---
GitHub user tbouron opened a pull request:
https://github.com/apache/brooklyn-library/pull/123
Convert AnsibleEntity and SaltEntity to be real SoftwareProcess
# Background
Salt and Ansible are currently installed by Brooklyn and:
- are used in masterless mode
- use
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-library/pull/126#discussion_r142680824
--- Diff:
software/nosql/src/main/java/org/apache/brooklyn/entity/nosql/mongodb/sharding/MongoDBRouterClusterImpl.java
---
@@ -56,12 +60,17
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-library/pull/126#discussion_r142681857
--- Diff:
software/webapp/src/main/java/org/apache/brooklyn/entity/proxy/nginx/NginxControllerImpl.java
---
@@ -178,6 +178,7 @@ protected void
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-library/pull/126#discussion_r142681908
--- Diff:
software/webapp/src/main/java/org/apache/brooklyn/entity/proxy/AbstractNonProvisionedControllerImpl.java
---
@@ -216,6 +218,7
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/857#discussion_r143427971
--- Diff:
core/src/main/java/org/apache/brooklyn/core/catalog/internal/BasicBrooklynCatalog.java
---
@@ -671,6 +671,8 @@ private void
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/864#discussion_r145357727
--- Diff: core/src/main/java/org/apache/brooklyn/core/entity/Entities.java
---
@@ -302,305 +292,190 @@ public static boolean isTrivial(Object v
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/864#discussion_r145348481
--- Diff: core/src/main/java/org/apache/brooklyn/core/entity/Entities.java
---
@@ -302,305 +292,190 @@ public static boolean isTrivial(Object v
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/864#discussion_r145347765
--- Diff: core/src/main/java/org/apache/brooklyn/core/entity/Dumper.java ---
@@ -0,0 +1,379 @@
+/*
+ * Licensed to the Apache Software
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/864#discussion_r145348576
--- Diff: core/src/main/java/org/apache/brooklyn/core/entity/Entities.java
---
@@ -302,305 +292,190 @@ public static boolean isTrivial(Object v
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/864#discussion_r145348526
--- Diff: core/src/main/java/org/apache/brooklyn/core/entity/Entities.java
---
@@ -302,305 +292,190 @@ public static boolean isTrivial(Object v
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/231#discussion_r150210273
--- Diff: guide/blueprints/_elect-primary-policies.md ---
@@ -0,0 +1,146 @@
+
+There are a collection of policies, enrichers, and effectors
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-library/pull/134#discussion_r150188836
--- Diff: software/messaging/src/main/resources/catalog.bom ---
@@ -25,6 +25,56 @@ brooklyn.catalog:
type
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-library/pull/134#discussion_r150189303
--- Diff: software/messaging/src/main/resources/catalog.bom ---
@@ -25,6 +25,56 @@ brooklyn.catalog:
type
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/228#discussion_r150210136
--- Diff: guide/ops/upgrades/index.md ---
@@ -0,0 +1,22 @@
+---
+title: Upgrades
+---
--- End diff --
Front-matter
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-server/pull/830
@andreaturli Might be completely off-base here but wouldn't it make more
sense for this to be in the clocker project
---
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-server/pull/750
@grkvlt Did you had the time to looks at @aledsage's comments for this PR?
---
Github user tbouron commented on the issue:
https://github.com/apache/brooklyn-server/pull/856
@ahgittin Do you think you can look at the comments above?
---
Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-docs/pull/238#discussion_r151662096
--- Diff: guide/locations/_AWS.md ---
@@ -42,6 +42,21 @@ Below are examples of configuration options that use
values specific to AWS EC2
301 - 400 of 655 matches
Mail list logo