Re: [HELP] maven build smoke-test bootstrapping.

2017-06-29 Thread Clebert Suconic
So, you first build at root, then you should be able to use the smoke-tests.. that's how I meant it to be. I just tried from scratch and it worked.. can you try and let me know? On Fri, Jun 30, 2017 at 12:06 AM, Clebert Suconic wrote: > you first build at root > > meaning: > > go to ./artemis-c

Re: [HELP] maven build smoke-test bootstrapping.

2017-06-29 Thread Clebert Suconic
you first build at root meaning: go to ./artemis-checkout mvn install it should work... although I reckon I always do build mvn iinstall -DskipTests=true at the root On Thu, Jun 29, 2017 at 11:25 PM, Timothy Nodine wrote: > Are the smoke tests not supposed to be under the root directory dur

Re: [HELP] maven build smoke-test bootstrapping.

2017-06-29 Thread Timothy Nodine
Are the smoke tests not supposed to be under the root directory during a build? Clebert Suconic wrote: You have to first build the project at the root. The smoke tests needs the distribution built. It is a smoke test hence it acts on the servers directly. On Thu, Jun 29, 2017 at 3:16 PM Timo

[GitHub] activemq-artemis pull request #1366: ARTEMIS-1252 Add service loading of pas...

2017-06-29 Thread gaohoward
Github user gaohoward commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1366#discussion_r124948436 --- Diff: artemis-commons/src/main/java/org/apache/activemq/artemis/utils/PasswordMaskingUtil.java --- @@ -108,22 +110,29 @@ public static Hash

Re: [HELP] maven build smoke-test bootstrapping.

2017-06-29 Thread Clebert Suconic
You have to first build the project at the root. The smoke tests needs the distribution built. It is a smoke test hence it acts on the servers directly. On Thu, Jun 29, 2017 at 3:16 PM Timothy Nodine wrote: > I attempted a maven build from scratch and it failed; I got the > following error mes

[GitHub] activemq-artemis issue #1295: ARTEMIS-1185 Inter-Process Journal Sampler Pro...

2017-06-29 Thread franz1981
Github user franz1981 commented on the issue: https://github.com/apache/activemq-artemis/pull/1295 @clebertsuconic yep, let's talk tomorrow. I've just tried now but I've no issues at all so probably there is something I'm missing... --- If your project is set up for it, you can reply

[GitHub] activemq-artemis issue #1295: ARTEMIS-1185 Inter-Process Journal Sampler Pro...

2017-06-29 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1295 @franz1981 I tried.. cannot run .. because of the lock... lets just talk over tomorrow.. --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] activemq-artemis issue #1295: ARTEMIS-1185 Inter-Process Journal Sampler Pro...

2017-06-29 Thread franz1981
Github user franz1981 commented on the issue: https://github.com/apache/activemq-artemis/pull/1295 @clebertsuconic but I've run it already concurrently with a server...Simply I've used a different bash console. That's how I've collected the data I've used for the doc I've n

[GitHub] activemq-artemis issue #1295: ARTEMIS-1185 Inter-Process Journal Sampler Pro...

2017-06-29 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1295 @franz1981 you are misunderstanding me.. the CLI won't ever run concurrently with the server.. it's an issue on your CLI.. you are holding the lock on the server.. hence you're not

[GitHub] activemq-artemis issue #1295: ARTEMIS-1185 Inter-Process Journal Sampler Pro...

2017-06-29 Thread franz1981
Github user franz1981 commented on the issue: https://github.com/apache/activemq-artemis/pull/1295 @clebertsuconic Maybe I've not understand your comment...but to capture data I use lock-free inter process communication primitives,hence no lock is involved because locks works only in

[GitHub] activemq-artemis issue #1295: ARTEMIS-1185 Inter-Process Journal Sampler Pro...

2017-06-29 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1295 `` It is able to capture samples from a running live serve``` ^^ ok that then confirms an issue with your PR. Your CLI is trying to lock the server.. so you wouldn't ever be

[GitHub] activemq-artemis issue #1295: ARTEMIS-1185 Inter-Process Journal Sampler Pro...

2017-06-29 Thread franz1981
Github user franz1981 commented on the issue: https://github.com/apache/activemq-artemis/pull/1295 @clebertsuconic It is able to capture samples from a running live server, but the server must be first started with the proper configuration. It is simply not possible to make it lig

[GitHub] activemq-artemis issue #1295: ARTEMIS-1185 Inter-Process Journal Sampler Pro...

2017-06-29 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1295 @franz1981 How i thought it worked.. you would be able to enable profile on a live server.. and be able to capture stuff from the running server. if that's not how it works.

[GitHub] activemq-artemis issue #1295: ARTEMIS-1185 Inter-Process Journal Sampler Pro...

2017-06-29 Thread franz1981
Github user franz1981 commented on the issue: https://github.com/apache/activemq-artemis/pull/1295 @clebertsuconic I've fixed the typo error and about "how it works" we could talk tomorrow about it :+1: I'm happy you like the doc :) --- If your project is set up for it, you can

[GitHub] activemq-artemis pull request #1366: ARTEMIS-1252 Add service loading of pas...

2017-06-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/1366 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feat

[GitHub] activemq-artemis issue #1366: ARTEMIS-1252 Add service loading of password c...

2017-06-29 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1366 Thanks --- 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 wi

[HELP] maven build smoke-test bootstrapping.

2017-06-29 Thread Timothy Nodine
I attempted a maven build from scratch and it failed; I got the following error message in the smoke tests: [ERROR] Could not locate suitable Artemis.home on either [directory redacted] or [directory redacted] [ERROR] Use the binary distribution or build the distribution before running the exa

[GitHub] activemq-artemis pull request #1366: ARTEMIS-1252 Add service loading of pas...

2017-06-29 Thread roelens8
Github user roelens8 commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1366#discussion_r124877640 --- Diff: artemis-commons/src/main/java/org/apache/activemq/artemis/utils/PasswordMaskingUtil.java --- @@ -108,22 +109,34 @@ public static HashP

Re: [DISCUSS] Removing the Web Console

2017-06-29 Thread Daniel Kulp
> On Jun 29, 2017, at 1:59 PM, Michael André Pearce > wrote: > > I believe we could make a simple activemq branding jar/war with the selected > new logo ;) once decided without too much trouble. > > I'd be happy to step up to try do this if no objectors. I discussed previously, what would NE

Re: [DISCUSS] Removing the Web Console

2017-06-29 Thread Michael André Pearce
I believe we could make a simple activemq branding jar/war with the selected new logo ;) once decided without too much trouble. I'd be happy to step up to try do this if no objectors. > On 29 Jun 2017, at 16:51, Clebert Suconic wrote: > > Speaking the plain truth... The issue is that the hawti

[GitHub] activemq-artemis pull request #1295: ARTEMIS-1185 Inter-Process Journal Samp...

2017-06-29 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1295#discussion_r124868500 --- Diff: docs/user-manual/en/tools.md --- @@ -220,5 +233,115 @@ COMMANDS With --bindings option, The folder used for bindi

[GitHub] activemq-artemis pull request #1295: ARTEMIS-1185 Inter-Process Journal Samp...

2017-06-29 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1295#discussion_r124857265 --- Diff: docs/user-manual/en/tools.md --- @@ -220,5 +233,115 @@ COMMANDS With --bindings option, The folder used for bindi

[GitHub] activemq-artemis pull request #1378: ARTEMIS-1261 Adjust default confirmatio...

2017-06-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/1378 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feat

[GitHub] activemq-artemis pull request #1347: ARTEMIS-1221 Duplicated ID causes Large...

2017-06-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/activemq-artemis/pull/1347 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feat

Re: [DISCUSS] Removing the Web Console

2017-06-29 Thread Clebert Suconic
Speaking the plain truth... The issue is that the hawtio console that was used years ago.. looked like a commercial product outside of apache I think that if we clear that now.. if it looks an apache look & feel.. people wouldn't have an issue with it. That would require some cleanup.. move to a

Re: [DISCUSS] Removing the Web Console

2017-06-29 Thread Michael André Pearce
I don't think this is a blocker, for example artemis uses jboss logging, this doesn't have a notice file I think we just have to preserve them if present and for asf projects themselves eg artemis itself it's policy to provide one for the asf work. Sent from my iPhone > On 29 Jun 2017, at 01:0

Re: [DISCUSS] Removing the Web Console

2017-06-29 Thread W B D
I've been using hawtio alongside the classic web console in ActiveMQ 5.x and have been quite happy with it. I find it easier to use for many common operations as well as for general monitoring, and it is definitely a gap in the current Artemis distribution, so I would certainly welcome built-in haw

[GitHub] activemq-artemis issue #1347: ARTEMIS-1221 Duplicated ID causes LargeMessage...

2017-06-29 Thread gaohoward
Github user gaohoward commented on the issue: https://github.com/apache/activemq-artemis/pull/1347 @clebertsuconic Thanks! :) --- 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 enab

[GitHub] activemq-artemis issue #1347: ARTEMIS-1221 Duplicated ID causes LargeMessage...

2017-06-29 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1347 whoa.. that's a nice Fix!!! Very nice --- 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 h

[GitHub] activemq-artemis issue #1336: ARTEMIS-1224 - change the journal file size to...

2017-06-29 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1336 @mnovak1 I don't think that's an issue: the min size is not really an issue. min journal size is 1K (because of headers and stuff).. if someone sets 1024 where the align is

[GitHub] activemq-artemis pull request #1366: ARTEMIS-1252 Add service loading of pas...

2017-06-29 Thread clebertsuconic
Github user clebertsuconic commented on a diff in the pull request: https://github.com/apache/activemq-artemis/pull/1366#discussion_r124801911 --- Diff: artemis-commons/src/main/java/org/apache/activemq/artemis/utils/PasswordMaskingUtil.java --- @@ -108,22 +109,34 @@ public static