[GitHub] aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state

2019-01-16 Thread GitBox
aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state URL: https://github.com/apache/brooklyn-server/pull/1010#discussion_r248245151 ## File path: parent/pom.xml ## @@ -581,6 +581,11 @@ org.ops4j.pax.web

[GitHub] aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state

2019-01-16 Thread GitBox
aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state URL: https://github.com/apache/brooklyn-server/pull/1010#discussion_r248245634 ## File path: pom.xml ## @@ -183,11 +183,13 @@ 6121 +6.10

[GitHub] aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state

2019-01-16 Thread GitBox
aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state URL: https://github.com/apache/brooklyn-server/pull/1010#discussion_r248243963 ## File path:

[GitHub] aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state

2019-01-16 Thread GitBox
aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state URL: https://github.com/apache/brooklyn-server/pull/1010#discussion_r248238589 ## File path: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogUpgradeScanner.java

[GitHub] aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state

2019-01-16 Thread GitBox
aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state URL: https://github.com/apache/brooklyn-server/pull/1010#discussion_r248246789 ## File path: utils/common/src/test/java/org/apache/brooklyn/util/javalang/ReflectionsTest.java

[GitHub] aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state

2019-01-16 Thread GitBox
aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state URL: https://github.com/apache/brooklyn-server/pull/1010#discussion_r248244616 ## File path:

[GitHub] aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state

2019-01-16 Thread GitBox
aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state URL: https://github.com/apache/brooklyn-server/pull/1010#discussion_r248246191 ## File path:

[GitHub] aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state

2019-01-16 Thread GitBox
aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state URL: https://github.com/apache/brooklyn-server/pull/1010#discussion_r228574716 ## File path:

[GitHub] aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state

2019-01-16 Thread GitBox
aledsage commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state URL: https://github.com/apache/brooklyn-server/pull/1010#discussion_r228573401 ## File path: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogUpgradeScanner.java

[GitHub] kemitix commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state

2019-01-16 Thread GitBox
kemitix commented on a change in pull request #1010: BROOKLYN-605 rebind fails for historic state URL: https://github.com/apache/brooklyn-server/pull/1010#discussion_r248307907 ## File path: core/src/main/java/org/apache/brooklyn/core/catalog/internal/CatalogUpgradeScanner.java

[GitHub] aledsage commented on issue #1010: BROOKLYN-605 rebind fails for historic state

2019-01-16 Thread GitBox
aledsage commented on issue #1010: BROOKLYN-605 rebind fails for historic state URL: https://github.com/apache/brooklyn-server/pull/1010#issuecomment-454845886 Thanks @kemitix - looks great. Merging now. This is an automated

[GitHub] asfgit merged pull request #1010: BROOKLYN-605 rebind fails for historic state

2019-01-16 Thread GitBox
asfgit merged pull request #1010: BROOKLYN-605 rebind fails for historic state URL: https://github.com/apache/brooklyn-server/pull/1010 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] kemitix commented on issue #1010: BROOKLYN-605 rebind fails for historic state

2019-01-16 Thread GitBox
kemitix commented on issue #1010: BROOKLYN-605 rebind fails for historic state URL: https://github.com/apache/brooklyn-server/pull/1010#issuecomment-454819030 @aledsage Updated as requested. This is an automated message from

[GitHub] tbouron commented on a change in pull request #169: Fixes JBoss 6 & 7 logos in catalog

2019-01-18 Thread GitBox
tbouron commented on a change in pull request #169: Fixes JBoss 6 & 7 logos in catalog URL: https://github.com/apache/brooklyn-library/pull/169#discussion_r248994554 ## File path: software/webapp/src/main/resources/catalog.bom ## @@ -62,7 +62,7 @@ brooklyn.catalog:

[GitHub] asfgit commented on issue #169: Fixes JBoss 6 & 7 logos in catalog

2019-01-18 Thread GitBox
asfgit commented on issue #169: Fixes JBoss 6 & 7 logos in catalog URL: https://github.com/apache/brooklyn-library/pull/169#issuecomment-455498928 Can one of the admins verify this patch? This is an automated message from the

[GitHub] nakomis opened a new pull request #169: Fixes JBoss 6 & 7 logos in catalog

2019-01-18 Thread GitBox
nakomis opened a new pull request #169: Fixes JBoss 6 & 7 logos in catalog URL: https://github.com/apache/brooklyn-library/pull/169 Fixes JBoss 6 & 7 logos in catalog This is an automated message from the Apache Git Service.

[GitHub] ahgittin commented on issue #1024: Implement REST and UI security as filter not JAAS LoginModule, start OAuth

2019-01-15 Thread GitBox
ahgittin commented on issue #1024: Implement REST and UI security as filter not JAAS LoginModule, start OAuth URL: https://github.com/apache/brooklyn-server/pull/1024#issuecomment-454503413 Note the following pull requests are needed simultaneously with this one: *

[GitHub] ahgittin opened a new pull request #138: new version of cxf seems to require this setting

2019-01-15 Thread GitBox
ahgittin opened a new pull request #138: new version of cxf seems to require this setting URL: https://github.com/apache/brooklyn-dist/pull/138 feels like it _shouldn't_ be needed, but we get a glassfish class not found error otherwise, and this fixes it by pointing at cxf instead

[GitHub] ahgittin commented on issue #138: new version of cxf seems to require this setting

2019-01-15 Thread GitBox
ahgittin commented on issue #138: new version of cxf seems to require this setting URL: https://github.com/apache/brooklyn-dist/pull/138#issuecomment-454504635 thanks to @jcabrerizo for finding this solution to the boot failure!

[GitHub] asfgit merged pull request #76: bump resteasy to be compatible with new jersey

2019-01-15 Thread GitBox
asfgit merged pull request #76: bump resteasy to be compatible with new jersey URL: https://github.com/apache/brooklyn-client/pull/76 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, comm...@brooklyn.apache.org

[GitHub] asfgit merged pull request #138: new version of cxf seems to require this setting

2019-01-15 Thread GitBox
asfgit merged pull request #138: new version of cxf seems to require this setting URL: https://github.com/apache/brooklyn-dist/pull/138 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, comm...@brooklyn.apache.org

[GitHub] asfgit merged pull request #117: require login to webconsole for webapps

2019-01-15 Thread GitBox
asfgit merged pull request #117: require login to webconsole for webapps URL: https://github.com/apache/brooklyn-ui/pull/117 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, comm...@brooklyn.apache.org

[GitHub] asfgit merged pull request #1024: Implement REST and UI security as filter not JAAS LoginModule, start OAuth

2019-01-15 Thread GitBox
asfgit merged pull request #1024: Implement REST and UI security as filter not JAAS LoginModule, start OAuth URL: https://github.com/apache/brooklyn-server/pull/1024 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list,

[GitHub] ahgittin opened a new pull request #76: bump resteasy to be compatible with new jersey

2019-01-15 Thread GitBox
ahgittin opened a new pull request #76: bump resteasy to be compatible with new jersey URL: https://github.com/apache/brooklyn-client/pull/76 fix need for csrf header now that that is respected in tests needed following https://github.com/apache/brooklyn-server/pull/1024

[GitHub] ahgittin commented on issue #1024: Implement REST and UI security as filter not JAAS LoginModule, start OAuth

2019-01-15 Thread GitBox
ahgittin commented on issue #1024: Implement REST and UI security as filter not JAAS LoginModule, start OAuth URL: https://github.com/apache/brooklyn-server/pull/1024#issuecomment-454624501 @jcabrerizo and I have reviewed each other's work on this. Lots of minor version bumps and tweaks

[GitHub] tbouron merged pull request #169: Fixes JBoss 6 & 7 logos in catalog

2019-01-21 Thread GitBox
tbouron merged pull request #169: Fixes JBoss 6 & 7 logos in catalog URL: https://github.com/apache/brooklyn-library/pull/169 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] tbouron merged pull request #1025: remove WIP oauth provider

2019-01-21 Thread GitBox
tbouron merged pull request #1025: remove WIP oauth provider URL: https://github.com/apache/brooklyn-server/pull/1025 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] jcabrerizo commented on issue #1025: remove WIP oauth provider

2019-01-21 Thread GitBox
jcabrerizo commented on issue #1025: remove WIP oauth provider URL: https://github.com/apache/brooklyn-server/pull/1025#issuecomment-456002612 LGTM. Next step, finish the implementation of the `SecurityProvider` to use Oauth authentication

[GitHub] tbouron commented on a change in pull request #117: require login to webconsole for webapps

2019-01-16 Thread GitBox
tbouron commented on a change in pull request #117: require login to webconsole for webapps URL: https://github.com/apache/brooklyn-ui/pull/117#discussion_r248204904 ## File path: modularity-server/features/pom.xml ## @@ -63,46 +63,11 @@

[GitHub] tbouron commented on a change in pull request #117: require login to webconsole for webapps

2019-01-16 Thread GitBox
tbouron commented on a change in pull request #117: require login to webconsole for webapps URL: https://github.com/apache/brooklyn-ui/pull/117#discussion_r248205106 ## File path: ui-modules/features/pom.xml ## @@ -71,46 +71,11 @@

[GitHub] ahgittin opened a new pull request #1025: remove WIP oauth provider

2019-01-16 Thread GitBox
ahgittin opened a new pull request #1025: remove WIP oauth provider URL: https://github.com/apache/brooklyn-server/pull/1025 will be added back when working it's not needed for #1024 (except for testing) and probably confusing to have in, in its current state. those who need it for

[GitHub] ahgittin commented on issue #1024: Implement REST and UI security as filter not JAAS LoginModule, start OAuth

2019-01-15 Thread GitBox
ahgittin commented on issue #1024: Implement REST and UI security as filter not JAAS LoginModule, start OAuth URL: https://github.com/apache/brooklyn-server/pull/1024#issuecomment-454429862 Note: Bumps some of the versions (see

[GitHub] asfgit merged pull request #126: update order in pom so we get the preferred versions

2019-01-16 Thread GitBox
asfgit merged pull request #126: update order in pom so we get the preferred versions URL: https://github.com/apache/brooklyn-dist/pull/126 As this is a foreign pull request (from a fork), the diff has been sent to your commit mailing list, comm...@brooklyn.apache.org

[GitHub] duncangrant opened a new pull request #1028: Add missing $

2019-01-23 Thread GitBox
duncangrant opened a new pull request #1028: Add missing $ URL: https://github.com/apache/brooklyn-server/pull/1028 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] tbouron merged pull request #1028: Add missing $

2019-01-23 Thread GitBox
tbouron merged pull request #1028: Add missing $ URL: https://github.com/apache/brooklyn-server/pull/1028 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the

[GitHub] grkvlt opened a new pull request #118: [WIP] Parameter UI additions

2019-01-23 Thread GitBox
grkvlt opened a new pull request #118: [WIP] Parameter UI additions URL: https://github.com/apache/brooklyn-ui/pull/118 Adding code to handle the `brooklyn.parameters` section in a blueprint. Currently needs more work on editing and handling raw JSON data.

[GitHub] duncangrant opened a new pull request #1026: Add cxf-karaf-commands dependency

2019-01-23 Thread GitBox
duncangrant opened a new pull request #1026: Add cxf-karaf-commands dependency URL: https://github.com/apache/brooklyn-server/pull/1026 This is a dependency of cxf that was introduced by the version change to 4.2.7. This adds cxf commands to the karaf shell such as listing endpoints.

[GitHub] kemitix commented on a change in pull request #1026: Add cxf-karaf-commands dependency

2019-01-23 Thread GitBox
kemitix commented on a change in pull request #1026: Add cxf-karaf-commands dependency URL: https://github.com/apache/brooklyn-server/pull/1026#discussion_r250136326 ## File path: karaf/features/src/main/feature/feature.xml ## @@ -208,6 +208,7 @@ cxf-jaxrs

[GitHub] kemitix commented on a change in pull request #1026: Add cxf-karaf-commands dependency

2019-01-23 Thread GitBox
kemitix commented on a change in pull request #1026: Add cxf-karaf-commands dependency URL: https://github.com/apache/brooklyn-server/pull/1026#discussion_r250138782 ## File path: karaf/features/src/main/feature/feature.xml ## @@ -208,6 +208,7 @@ cxf-jaxrs

[GitHub] kemitix commented on a change in pull request #1026: Add cxf-karaf-commands dependency

2019-01-23 Thread GitBox
kemitix commented on a change in pull request #1026: Add cxf-karaf-commands dependency URL: https://github.com/apache/brooklyn-server/pull/1026#discussion_r250136326 ## File path: karaf/features/src/main/feature/feature.xml ## @@ -208,6 +208,7 @@ cxf-jaxrs

[GitHub] tbouron commented on a change in pull request #1026: Add cxf-karaf-commands dependency

2019-01-23 Thread GitBox
tbouron commented on a change in pull request #1026: Add cxf-karaf-commands dependency URL: https://github.com/apache/brooklyn-server/pull/1026#discussion_r250137380 ## File path: karaf/features/src/main/feature/feature.xml ## @@ -208,6 +208,7 @@ cxf-jaxrs

[GitHub] duncangrant commented on a change in pull request #1026: Add cxf-karaf-commands dependency

2019-01-23 Thread GitBox
duncangrant commented on a change in pull request #1026: Add cxf-karaf-commands dependency URL: https://github.com/apache/brooklyn-server/pull/1026#discussion_r250141286 ## File path: karaf/features/src/main/feature/feature.xml ## @@ -208,6 +208,7 @@ cxf-jaxrs

[GitHub] jcabrerizo opened a new pull request #1027: Added config keys for Oauth

2019-01-23 Thread GitBox
jcabrerizo opened a new pull request #1027: Added config keys for Oauth URL: https://github.com/apache/brooklyn-server/pull/1027 Some new ConfigKeys added to mange the authentication using Oauth This is an automated message

[GitHub] tbouron commented on issue #1027: Added config keys for Oauth

2019-01-23 Thread GitBox
tbouron commented on issue #1027: Added config keys for Oauth URL: https://github.com/apache/brooklyn-server/pull/1027#issuecomment-456760002 Thanks for the PR @jcabrerizo ! However, the `OAuthProvider` introduced by #1024 was WIP and removed in #1025. So I think these config keys

[GitHub] tbouron merged pull request #1026: Add cxf-karaf-commands dependency

2019-01-23 Thread GitBox
tbouron merged pull request #1026: Add cxf-karaf-commands dependency URL: https://github.com/apache/brooklyn-server/pull/1026 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] tbouron edited a comment on issue #1027: Added config keys for Oauth

2019-01-23 Thread GitBox
tbouron edited a comment on issue #1027: Added config keys for Oauth URL: https://github.com/apache/brooklyn-server/pull/1027#issuecomment-456760002 Thanks for the PR @jcabrerizo ! However, the `OAuthProvider` introduced by #1024 was WIP and removed in #1025. So I think these config

[GitHub] rdowner commented on issue #136: Avoid build depending on Docker by default

2018-12-18 Thread GitBox
rdowner commented on issue #136: Avoid build depending on Docker by default URL: https://github.com/apache/brooklyn-dist/pull/136#issuecomment-448259624 Can I suggest that the Docker artifact is produced in its own project, instead of pigging-backing off the karaf build project?

[GitHub] geomacy opened a new pull request #136: Avoid build depending on Docker by default

2018-12-16 Thread GitBox
geomacy opened a new pull request #136: Avoid build depending on Docker by default URL: https://github.com/apache/brooklyn-dist/pull/136 To avoid the build failing if Docker is not installed and running, put the docker image build into a profile, so build with mvn clean install

[GitHub] ahgittin commented on issue #963: disable jclouds authorizePublicKey for extra ssh public key data

2018-12-14 Thread GitBox
ahgittin commented on issue #963: disable jclouds authorizePublicKey for extra ssh public key data URL: https://github.com/apache/brooklyn-server/pull/963#issuecomment-447275653 Just had a report of a user who is disabling the ssh init options and using `extraSsh...` to configure the

[GitHub] geomacy opened a new pull request #275: Change git-wip-us.apache.org to gitbox.apache.org

2018-12-14 Thread GitBox
geomacy opened a new pull request #275: Change git-wip-us.apache.org to gitbox.apache.org URL: https://github.com/apache/brooklyn-docs/pull/275 This follows move to gitbox, see https://lists.apache.org/thread.html/3d87de62fa3330222a972ed98e63907107ca5951f0bdf099972ae426

[GitHub] geomacy commented on issue #275: Change git-wip-us.apache.org to gitbox.apache.org

2018-12-14 Thread GitBox
geomacy commented on issue #275: Change git-wip-us.apache.org to gitbox.apache.org URL: https://github.com/apache/brooklyn-docs/pull/275#issuecomment-447492195 We probably need a bit more of a rewrite, but this is a minimal start.

[GitHub] tbouron commented on issue #136: Avoid build depending on Docker by default

2018-12-17 Thread GitBox
tbouron commented on issue #136: Avoid build depending on Docker by default URL: https://github.com/apache/brooklyn-dist/pull/136#issuecomment-447784220 I agree with @duncangrant, we should do opt-in or opt-out for all, i.e. RPM/DEB/Go/Docker. However, this looks good @geomacy! But

[GitHub] kemitix commented on issue #136: Avoid build depending on Docker by default

2018-12-17 Thread GitBox
kemitix commented on issue #136: Avoid build depending on Docker by default URL: https://github.com/apache/brooklyn-dist/pull/136#issuecomment-447787567 I much prefer the Opt-In approach. How many dev boxes are configured to build *both* rpm and deb packages? As a default

[GitHub] geomacy commented on issue #136: Avoid build depending on Docker by default

2018-12-17 Thread GitBox
geomacy commented on issue #136: Avoid build depending on Docker by default URL: https://github.com/apache/brooklyn-dist/pull/136#issuecomment-447783349 I agree that it's inconsistent - actually I was just thinking over the weekend that the same reasoning applies to the rpm/deb and go

[GitHub] duncangrant commented on issue #136: Avoid build depending on Docker by default

2018-12-17 Thread GitBox
duncangrant commented on issue #136: Avoid build depending on Docker by default URL: https://github.com/apache/brooklyn-dist/pull/136#issuecomment-447782289 So we opt out of building the go client, rpm build, and deb build. So this is a bit inconsistent but I think I'd be happy to merge

[GitHub] ahgittin opened a new pull request #117: require login to webconsole for webapps

2018-12-14 Thread GitBox
ahgittin opened a new pull request #117: require login to webconsole for webapps URL: https://github.com/apache/brooklyn-ui/pull/117 the REST API has always been secured but we now secure these static assets also; this gives a better experience on login, and it will be even more

[GitHub] tbouron closed pull request #275: Change git-wip-us.apache.org to gitbox.apache.org

2018-12-15 Thread GitBox
tbouron closed pull request #275: Change git-wip-us.apache.org to gitbox.apache.org URL: https://github.com/apache/brooklyn-docs/pull/275 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] tbouron commented on issue #117: require login to webconsole for webapps

2018-12-14 Thread GitBox
tbouron commented on issue #117: require login to webconsole for webapps URL: https://github.com/apache/brooklyn-ui/pull/117#issuecomment-447371396 @ahgittin This should be ok for basic auth but I think will fail for any other kind of auth (if implemented in the future) All resources are

[GitHub] ahgittin commented on issue #117: require login to webconsole for webapps

2018-12-14 Thread GitBox
ahgittin commented on issue #117: require login to webconsole for webapps URL: https://github.com/apache/brooklyn-ui/pull/117#issuecomment-447379194 good spot @tbouron -- i was trying to do things the "right way" with a JaaS login module but that seems geared around the `` indeed is

[GitHub] ahgittin commented on issue #117: require login to webconsole for webapps

2018-12-14 Thread GitBox
ahgittin commented on issue #117: require login to webconsole for webapps URL: https://github.com/apache/brooklyn-ui/pull/117#issuecomment-447363483 fixed a bug reported by @frogfather where one snippet had been added twice but have noticed that while this works for the first launch,

[GitHub] geomacy closed pull request #137: Docker build plugin version

2019-01-08 Thread GitBox
geomacy closed pull request #137: Docker build plugin version URL: https://github.com/apache/brooklyn-dist/pull/137 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] geomacy closed pull request #1023: Add version for docker build plugin

2019-01-08 Thread GitBox
geomacy closed pull request #1023: Add version for docker build plugin URL: https://github.com/apache/brooklyn-server/pull/1023 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] geomacy closed pull request #136: Avoid build depending on Docker by default

2019-01-08 Thread GitBox
geomacy closed pull request #136: Avoid build depending on Docker by default URL: https://github.com/apache/brooklyn-dist/pull/136 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] geomacy commented on issue #136: Avoid build depending on Docker by default

2019-01-07 Thread GitBox
geomacy commented on issue #136: Avoid build depending on Docker by default URL: https://github.com/apache/brooklyn-dist/pull/136#issuecomment-451867575 @rdowner @duncangrant @kemitix @tbouron how do you think that looks now?

[GitHub] tbouron commented on a change in pull request #136: Avoid build depending on Docker by default

2019-01-07 Thread GitBox
tbouron commented on a change in pull request #136: Avoid build depending on Docker by default URL: https://github.com/apache/brooklyn-dist/pull/136#discussion_r245587504 ## File path: docker-image/pom.xml ## @@ -0,0 +1,104 @@ + + +http://maven.apache.org/POM/4.0.0;

[GitHub] ahgittin opened a new pull request #1024: Implement REST and UI security as filter not JAAS LoginModule, start OAuth

2019-01-09 Thread GitBox
ahgittin opened a new pull request #1024: Implement REST and UI security as filter not JAAS LoginModule, start OAuth URL: https://github.com/apache/brooklyn-server/pull/1024 The JAAS LoginModule seems only to work with BASIC and related auth schemes; it is insufficient for things like

[GitHub] ahgittin commented on issue #117: require login to webconsole for webapps

2019-01-09 Thread GitBox
ahgittin commented on issue #117: require login to webconsole for webapps URL: https://github.com/apache/brooklyn-ui/pull/117#issuecomment-452650378 Changed to use the `BrooklynSecurityProviderFilter` introduced in https://github.com/apache/brooklyn-server/pull/1024

[GitHub] duncangrant opened a new pull request #1023: Add version for docker build plugin

2019-01-08 Thread GitBox
duncangrant opened a new pull request #1023: Add version for docker build plugin URL: https://github.com/apache/brooklyn-server/pull/1023 Allows https://github.com/apache/brooklyn-dist/pull/136 review comments to be addressed - i.e. moving docker plugin version to shared pom in

[GitHub] duncangrant opened a new pull request #137: Docker build plugin version

2019-01-08 Thread GitBox
duncangrant opened a new pull request #137: Docker build plugin version URL: https://github.com/apache/brooklyn-dist/pull/137 Incorporates and addresses review comment in #136 Moves version to brooklyn-server/pom.xml Requires https://github.com/apache/brooklyn-server/pull/1023

[GitHub] geomacy commented on issue #136: Avoid build depending on Docker by default

2018-12-18 Thread GitBox
geomacy commented on issue #136: Avoid build depending on Docker by default URL: https://github.com/apache/brooklyn-dist/pull/136#issuecomment-448367088 I'll have a look at that This is an automated message from the Apache

[GitHub] ahgittin opened a new pull request #1032: make /logout just log out

2019-01-26 Thread GitBox
ahgittin opened a new pull request #1032: make /logout just log out URL: https://github.com/apache/brooklyn-server/pull/1032 previously it redirected to the user-specific logout; but some clients didn't respect that, and assumed the logout had happened. not convinced there is a good

[GitHub] ahgittin opened a new pull request #1033: Share sessions among all bundles so we can share auth

2019-01-26 Thread GitBox
ahgittin opened a new pull request #1033: Share sessions among all bundles so we can share auth URL: https://github.com/apache/brooklyn-server/pull/1033 this takes the first encountered session handler and caches it, also setting the sessonPath on the cookies so cookies are valid for

[GitHub] aledsage commented on issue #1033: Share sessions among all bundles so we can share auth

2019-01-27 Thread GitBox
aledsage commented on issue #1033: Share sessions among all bundles so we can share auth URL: https://github.com/apache/brooklyn-server/pull/1033#issuecomment-457965480 Jenkins test failures look related, e.g. `BrooklynRestApiLauncherTest.testServletStart`: ```

[GitHub] ahgittin opened a new pull request #120: Fix logout [DO NOT MERGE]

2019-01-27 Thread GitBox
ahgittin opened a new pull request #120: Fix logout [DO NOT MERGE] URL: https://github.com/apache/brooklyn-ui/pull/120 update logout to use angular `$http` and actually to call the server to log out previously it wasn't calling to the server (not sure where it was going wrong but

[GitHub] ahgittin commented on issue #120: Fix logout [DO NOT MERGE]

2019-01-27 Thread GitBox
ahgittin commented on issue #120: Fix logout [DO NOT MERGE] URL: https://github.com/apache/brooklyn-ui/pull/120#issuecomment-457978118 this PR includes a lot of exploration, and i'm _still_ exploring. it's proving quite hard to get the browser to forget or re-question its cached

[GitHub] ahgittin commented on issue #1033: Share sessions among all bundles so we can share auth

2019-01-27 Thread GitBox
ahgittin commented on issue #1033: Share sessions among all bundles so we can share auth URL: https://github.com/apache/brooklyn-server/pull/1033#issuecomment-457978368 Thanks @aledsage ; fixed the obvious problem but there are still some tests that need TLC. Top priority for the

[GitHub] ahgittin commented on issue #1032: make /logout just log out

2019-01-27 Thread GitBox
ahgittin commented on issue #1032: make /logout just log out URL: https://github.com/apache/brooklyn-server/pull/1032#issuecomment-457977863 experimenting to see what makes sense the browser seems to keep the local cached creds, _even_ on a 401 unauthorized, so not sure if there's

[GitHub] jcabrerizo commented on issue #1033: Share sessions among all bundles so we can share auth

2019-01-28 Thread GitBox
jcabrerizo commented on issue #1033: Share sessions among all bundles so we can share auth URL: https://github.com/apache/brooklyn-server/pull/1033#issuecomment-458178665 I've been testing with @richardcloudsoft the Web UI and the BR command line and it works perfectly using

[GitHub] aledsage commented on a change in pull request #1033: Share sessions among all bundles so we can share auth

2019-01-28 Thread GitBox
aledsage commented on a change in pull request #1033: Share sessions among all bundles so we can share auth URL: https://github.com/apache/brooklyn-server/pull/1033#discussion_r251413877 ## File path:

[GitHub] aledsage commented on a change in pull request #1033: Share sessions among all bundles so we can share auth

2019-01-28 Thread GitBox
aledsage commented on a change in pull request #1033: Share sessions among all bundles so we can share auth URL: https://github.com/apache/brooklyn-server/pull/1033#discussion_r251416007 ## File path: rest/rest-resources/pom.xml ## @@ -220,10 +215,9 @@

[GitHub] [brooklyn-server] jcabrerizo commented on issue #1061: Granularity entitlement control

2019-03-29 Thread GitBox
jcabrerizo commented on issue #1061: Granularity entitlement control URL: https://github.com/apache/brooklyn-server/pull/1061#issuecomment-478052574 Messages changed. Thanks for the feedback This is an automated message from

[GitHub] [brooklyn-ui] jcabrerizo commented on a change in pull request #130: Fix entitlement messages

2019-03-29 Thread GitBox
jcabrerizo commented on a change in pull request #130: Fix entitlement messages URL: https://github.com/apache/brooklyn-ui/pull/130#discussion_r270449717 ## File path: ui-modules/app-inspector/app/views/main/inspect/summary/summary.controller.js ## @@ -188,7 +188,11 @@

[GitHub] [brooklyn-ui] tbouron merged pull request #130: Fix entitlement messages

2019-03-29 Thread GitBox
tbouron merged pull request #130: Fix entitlement messages URL: https://github.com/apache/brooklyn-ui/pull/130 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [brooklyn-ui] tbouron commented on issue #129: UI for defining parameters

2019-04-02 Thread GitBox
tbouron commented on issue #129: UI for defining parameters URL: https://github.com/apache/brooklyn-ui/pull/129#issuecomment-478991360 >yeah, i buy that. i don't see an easy way to do it with validation for the case where duplicate keys are set in yaml. (we already did it if you tried to

[GitHub] [brooklyn-server] bd2019us opened a new pull request #1062: [SECURITY] change Executors.newCachedThreadPool() to newFixedThreadPool(int)

2019-03-31 Thread GitBox
bd2019us opened a new pull request #1062: [SECURITY] change Executors.newCachedThreadPool() to newFixedThreadPool(int) URL: https://github.com/apache/brooklyn-server/pull/1062 Since the size of 'sockets' can be large, cached thread pool has a high risk in running out of memory and causes

[GitHub] [brooklyn-ui] tbouron commented on issue #118: [WIP] Parameter UI additions

2019-04-03 Thread GitBox
tbouron commented on issue #118: [WIP] Parameter UI additions URL: https://github.com/apache/brooklyn-ui/pull/118#issuecomment-479432992 As @ahgittin, this one is superseded by #129 so closing it But thanks @grkvlt for starting it 

[GitHub] [brooklyn-ui] tbouron closed pull request #118: [WIP] Parameter UI additions

2019-04-03 Thread GitBox
tbouron closed pull request #118: [WIP] Parameter UI additions URL: https://github.com/apache/brooklyn-ui/pull/118 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [brooklyn-server] kemitix commented on issue #1051: [SECURITY] Bump karaf.version from 4.2.2 to 4.2.3

2019-03-25 Thread GitBox
kemitix commented on issue #1051: [SECURITY] Bump karaf.version from 4.2.2 to 4.2.3 URL: https://github.com/apache/brooklyn-server/pull/1051#issuecomment-476392021 @aledsage Needs a little more work. Testing had been pretty basic with just brooklyn-server. I'm seeing a maven enforcer

[GitHub] [brooklyn-docs] tbouron commented on issue #280: Add text to comply with Apache naming guidelines

2019-03-27 Thread GitBox
tbouron commented on issue #280: Add text to comply with Apache naming guidelines URL: https://github.com/apache/brooklyn-docs/pull/280#issuecomment-477116689 I'm also happy with this @geomacy ! This is an automated message

[GitHub] [brooklyn-ui] ahgittin commented on issue #129: UI for defining parameters

2019-03-27 Thread GitBox
ahgittin commented on issue #129: UI for defining parameters URL: https://github.com/apache/brooklyn-ui/pull/129#issuecomment-477125795 i've addressed 1,2,4, and applied a similar toolbar cleanup to configuration (also use labels a la palette instead of checkboxes for the filters). 3 is

[GitHub] [brooklyn-docs] geomacy commented on issue #280: Add text to comply with Apache naming guidelines

2019-03-27 Thread GitBox
geomacy commented on issue #280: Add text to comply with Apache naming guidelines URL: https://github.com/apache/brooklyn-docs/pull/280#issuecomment-477065009 hi @rdowner @tbouron could I ask for another review, what do you think of the look now?

[GitHub] [brooklyn-ui] ahgittin commented on issue #129: UI for defining parameters

2019-03-27 Thread GitBox
ahgittin commented on issue #129: UI for defining parameters URL: https://github.com/apache/brooklyn-ui/pull/129#issuecomment-477229926 did some testing and it all works (6 is happy), except a default value of `''` is being set for things so `required` has no effect. i've fixed that

[GitHub] [brooklyn-ui] ahgittin opened a new pull request #129: UI for defining parameters

2019-03-26 Thread GitBox
ahgittin opened a new pull request #129: UI for defining parameters URL: https://github.com/apache/brooklyn-ui/pull/129 has a field-based mode and JSON mode builds on #118 , makes it mergeable ideally flaws in the PR can be tracked here but addressed in follow-on PRs unless

[GitHub] [brooklyn-ui] ahgittin commented on issue #118: [WIP] Parameter UI additions

2019-03-26 Thread GitBox
ahgittin commented on issue #118: [WIP] Parameter UI additions URL: https://github.com/apache/brooklyn-ui/pull/118#issuecomment-476723168 Superseded by #129 This is an automated message from the Apache Git Service. To

[GitHub] [brooklyn-server] aledsage opened a new pull request #1060: BROOKLYN-613: failing rebind test for AutoScalerPolicy config using DSL

2019-03-25 Thread GitBox
aledsage opened a new pull request #1060: BROOKLYN-613: failing rebind test for AutoScalerPolicy config using DSL URL: https://github.com/apache/brooklyn-server/pull/1060 This has a failing test to demonstrate https://issues.apache.org/jira/browse/BROOKLYN-613 (with the unit test marked

[GitHub] [brooklyn-ui] tbouron merged pull request #128: Solver error showing entitlements messages

2019-03-25 Thread GitBox
tbouron merged pull request #128: Solver error showing entitlements messages URL: https://github.com/apache/brooklyn-ui/pull/128 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [brooklyn-ui] jcabrerizo opened a new pull request #128: Solver error showing entitlements messages

2019-03-25 Thread GitBox
jcabrerizo opened a new pull request #128: Solver error showing entitlements messages URL: https://github.com/apache/brooklyn-ui/pull/128 The error message from the exception thrown when the user has been not entitled to one action is wrapped on an object and was not showed properlly in

[GitHub] [brooklyn-server] kemitix commented on issue #1051: Bump karaf.version from 4.2.2 to 4.2.3

2019-03-25 Thread GitBox
kemitix commented on issue #1051: Bump karaf.version from 4.2.2 to 4.2.3 URL: https://github.com/apache/brooklyn-server/pull/1051#issuecomment-476274745 Sourced from The GitHub Security Advisory Database. > Moderate severity vulnerability that affects org.apache.karaf:karaf and

[GitHub] [brooklyn-library] houshengbo closed pull request #171: DO NOT MERGE: Just verify how the pipeline job work in Apache, many thanks

2019-03-25 Thread GitBox
houshengbo closed pull request #171: DO NOT MERGE: Just verify how the pipeline job work in Apache, many thanks URL: https://github.com/apache/brooklyn-library/pull/171 This is an automated message from the Apache Git

[GitHub] [brooklyn-library] houshengbo opened a new pull request #171: DO NOT MERGE: Just verify how the pipeline job work in Apache, many thanks

2019-03-25 Thread GitBox
houshengbo opened a new pull request #171: DO NOT MERGE: Just verify how the pipeline job work in Apache, many thanks URL: https://github.com/apache/brooklyn-library/pull/171 This is an automated message from the Apache Git

  1   2   3   4   5   6   7   8   9   10   >