Jenkins build is back to normal : brooklyn-master-build-docker #24

See

[GitHub] brooklyn-server pull request #903: Delete bundle: handle pre-installed karaf...

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/903 ---

[GitHub] brooklyn-server issue #903: Delete bundle: handle pre-installed karaf bundle...

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/903 non-det test failure: org.apache.brooklyn.core.entity.lifecycle.ServiceStateLogicTest.testStopsNicelyToo LGTM, merging ---

Build failed in Jenkins: brooklyn-master-build-docker #23

See -- [...truncated 8.82 MB...] [INFO] --- maven-site-plugin:3.4:attach-descriptor (attach-descriptor) @ brooklyn-all --- [INFO] [INFO] --- apache-rat-plugin:0.12:check (def

Build failed in Jenkins: brooklyn-master-build-docker #22

See -- [...truncated 8.78 MB...] [INFO] --- maven-site-plugin:3.4:attach-descriptor (attach-descriptor) @ brooklyn-all --- [INFO] [INFO] --- apache-rat-plugin:0.12:check (def

[GitHub] brooklyn-server pull request #899: Fix force remove bundle referenced by oth...

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/899 ---

[GitHub] brooklyn-server issue #899: Fix force remove bundle referenced by other bund...

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/899 perfect, merging ---

Build failed in Jenkins: brooklyn-master-build-docker #21

See -- [...truncated 10.14 MB...] [INFO] [INFO] --- apache-rat-plugin:0.12:check (default) @ brooklyn-all --- [INFO] Enabled default license matchers. [INFO] Will parse SCM ig

Build failed in Jenkins: brooklyn-master-build-docker #20

See -- [...truncated 8.79 MB...] [INFO] --- maven-site-plugin:3.4:attach-descriptor (attach-descriptor) @ brooklyn-all --- [INFO] [INFO] --- apache-rat-plugin:0.12:check (def

[GitHub] brooklyn-server issue #899: Fix force remove bundle referenced by other bund...

Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/899 Oops, I meant `201 Created` rather than 202. Agreed - I've changed it to return 200 (see most recent commits). Can you review again please @ahgittin? ---

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

See

[GitHub] brooklyn-dist pull request #105: Update Brooklyn vagrant to use RPM

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-dist/pull/105 ---

[jira] [Commented] (BROOKLYN-545) Rebind failure after using cluster with `enableAvailabilityZones`

[ https://issues.apache.org/jira/browse/BROOKLYN-545?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16260979#comment-16260979 ] ASF GitHub Bot commented on BROOKLYN-545: - Github user asfgit closed the pull re

[GitHub] brooklyn-server pull request #902: BROOKLYN-545: Fix DynamicCluster.zoneFail...

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/902 ---

[GitHub] brooklyn-server issue #899: Fix force remove bundle referenced by other bund...

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/899 It's definitely inconsistent to give a 400 on `IGNORING_BUNDLE_AREADY_INSTALLED` but a 202 on `IGNORING_BUNDLE_FORCIBLY_REMOVED`. i don't think there's any compelling reason to make that

[GitHub] brooklyn-server pull request #899: Fix force remove bundle referenced by oth...

Github user aledsage commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/899#discussion_r152321945 --- Diff: launcher/src/test/java/org/apache/brooklyn/launcher/AbstractBrooklynLauncherRebindTest.java --- @@ -184,7 +190,87 @@ protected File newT

[GitHub] brooklyn-server issue #899: Fix force remove bundle referenced by other bund...

Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/899 @ahgittin One thing I haven’t done is to behave differently if `?force=true` is used (to fail, rather than just returning a code that indicates their request to add the bundle was ignored

[GitHub] brooklyn-server pull request #903: Delete bundle: handle pre-installed karaf...

GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/903 Delete bundle: handle pre-installed karaf bundles Previously, if you tried to delete a bundle that was pre-installed in brooklyn karaf then the REST pai call would return a 500 response:

[GitHub] brooklyn-server issue #899: Fix force remove bundle referenced by other bund...

Github user ahgittin commented on the issue: https://github.com/apache/brooklyn-server/pull/899 looks great. really nice tests. a few minor comments, that's all. ---

[GitHub] brooklyn-server pull request #899: Fix force remove bundle referenced by oth...

Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/899#discussion_r152286086 --- Diff: launcher/src/test/java/org/apache/brooklyn/launcher/AbstractBrooklynLauncherRebindTest.java --- @@ -184,7 +190,87 @@ protected File newT

[GitHub] brooklyn-server pull request #899: Fix force remove bundle referenced by oth...

Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/899#discussion_r152285616 --- Diff: core/src/main/java/org/apache/brooklyn/core/mgmt/ha/OsgiBundleInstallationResult.java --- @@ -98,6 +102,12 @@ void setIgnoringAlreadyIns

[GitHub] brooklyn-server pull request #899: Fix force remove bundle referenced by oth...

Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-server/pull/899#discussion_r152285416 --- Diff: core/src/main/java/org/apache/brooklyn/core/mgmt/ha/OsgiArchiveInstaller.java --- @@ -437,13 +463,19 @@ private synchronized void close(

Jenkins build is back to normal : brooklyn-master-build-docker #19

See

Build failed in Jenkins: brooklyn-master-build-docker #18

See -- [...truncated 8.78 MB...] [INFO] --- maven-site-plugin:3.4:attach-descriptor (attach-descriptor) @ brooklyn-all --- [INFO] [INFO] --- apache-rat-plugin:0.12:check (def

Build failed in Jenkins: brooklyn-master-build-docker #17

See Changes: [thomas.bouron] Add dockerfile for CI build -- [...truncated 9.07 MB...] [INFO] [INFO] --- apache-rat-plugin:0.12:check (default) @ brooklyn-all --

[GitHub] brooklyn-server issue #902: BROOKLYN-545: Fix DynamicCluster.zoneFailureDete...

Github user drigodwin commented on the issue: https://github.com/apache/brooklyn-server/pull/902 retest this please ---

[jira] [Commented] (BROOKLYN-545) Rebind failure after using cluster with `enableAvailabilityZones`

[ https://issues.apache.org/jira/browse/BROOKLYN-545?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16260619#comment-16260619 ] ASF GitHub Bot commented on BROOKLYN-545: - Github user drigodwin commented on th

[GitHub] brooklyn-dist pull request #115: Add dockerfile for CI build

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-dist/pull/115 ---

[GitHub] brooklyn-client pull request #65: Add dockerfile for CI build

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-client/pull/65 ---

[GitHub] brooklyn pull request #13: Add dockerfile for CI build

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn/pull/13 ---

Build failed in Jenkins: brooklyn-ui-master-docker #3

See Changes: [thomas.bouron] Add dockerfile for CI build -- [...truncated 24.52 KB...] /usr/build/target/brooklyn-jsgui-1.0.0-SNAPSHOT/assets/js/libs/jquery.js /usr/

Build failed in Jenkins: brooklyn-ui-master-docker #2

See Changes: [alex.heneveld] fix username in quotes -- [...truncated 50.82 KB...] /usr/build/target/brooklyn-jsgui-1.0.0-SNAPSHOT/assets/js/libs/jquery.js /usr/build

[GitHub] brooklyn-ui pull request #49: Add dockerfile for CI build

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-ui/pull/49 ---

[GitHub] brooklyn-ui pull request #50: fix username in quotes

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-ui/pull/50 ---

Build failed in Jenkins: brooklyn-master-build-docker #16

See -- [...truncated 4.81 MB...] 2017-11-21 10:05:53,791 INFO TESTNG PASSED CONFIGURATION: "Surefire test" - @BeforeMethod org.apache.brooklyn.core.location.dynamic.clocker.

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

See

Build failed in Jenkins: brooklyn-master-build-docker #15

See -- [...truncated 8.76 MB...] [INFO] --- maven-site-plugin:3.4:attach-descriptor (attach-descriptor) @ brooklyn-all --- [INFO] [INFO] --- apache-rat-plugin:0.12:check (def

Build failed in Jenkins: brooklyn-server-master-docker #6

See Changes: [alex.heneveld] minor tidies for better logging and comments, discovered in testing -- Started by an SCM change [EnvInject] - Loading node environm

[GitHub] brooklyn-server pull request #901: minor tidies for better logging and comme...

Github user asfgit closed the pull request at: https://github.com/apache/brooklyn-server/pull/901 ---

[GitHub] brooklyn-server issue #901: minor tidies for better logging and comments

Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/901 LGTM; merging. ---

[GitHub] brooklyn-server issue #894: Adds SetLimitsCustomizer for centos/rhel

Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/894 retest this please ---

[GitHub] brooklyn-server issue #899: [WIP] Fix force remove bundle referenced by othe...

Github user aledsage commented on the issue: https://github.com/apache/brooklyn-server/pull/899 retest this please jenkins timed out (the final test being run started within a few seconds of the suite being aborted, so I don't think it's caused by that test hanging): ```

[jira] [Commented] (BROOKLYN-545) Rebind failure after using cluster with `enableAvailabilityZones`

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

[GitHub] brooklyn-server pull request #902: BROOKLYN-545: Fix DynamicCluster.zoneFail...

GitHub user aledsage opened a pull request: https://github.com/apache/brooklyn-server/pull/902 BROOKLYN-545: Fix DynamicCluster.zoneFailureDetector default val Fixes https://issues.apache.org/jira/browse/BROOKLYN-545 Below is an extract from the comments in the code, explain

Jenkins build is back to normal : brooklyn-client-master-docker #2

See

Build failed in Jenkins: brooklyn-master-build-docker #14

See -- [...truncated 8.77 MB...] [INFO] --- apache-rat-plugin:0.12:check (default) @ brooklyn-all --- [INFO] Enabled default license matchers. [INFO] Will parse SCM ignores for