Very nice :) Thank you!
Am 20.12.2018 um 18:34 schrieb Bruno Baptista:
That's great! :)
Bruno Baptista
https://twitter.com/brunobat_
On 20/12/18 17:25, Jean-Louis Monteiro wrote:
Build is green, yeah.
https://ci.apache.org/builders/tomee-trunk-ubuntu-jvm8
Great community effort. Thanks everyone
--
Jean-Louis Monteiro
http://twitter.com/jlouismonteiro
http://www.tomitribe.com
On Wed, Dec 19, 2018 at 10:56 PM Daniel Cunha <[email protected]>
wrote:
Hi Roberto,
I can't reproduce the issue in both 7.0.x and 7.1.x with my patch
applied,
it seems passing as expected:
[INFO]
[INFO] -------------------------------------------------------
[INFO] T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.tomee.TomEECliIT
Must specify either a plain text to encrypt or a ciphered value to
decrypt.
usage: cipher [options] <value>
Uses a password cipher implementation to encrypt/decrypt a value.
-c,--cipher <c> Specifies the password cipher implementation to use
(default is Static3DES).
-d,--decrypt Switches command line tool to decrypt.
-e,--encrypt Switches command line tool to encrypt (default).
-h,--help Lists these options and exits.
Available ciphers are: cdi
[INFO] Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time elapsed:
1.144 s - in org.apache.tomee.TomEECliIT
[INFO] Running org.apache.tomee.RemoteTomEEEJBContainerIT
Dec 19, 2018 6:52:57 PM org.apache.openejb.client.EventLogger log
INFO: RemoteInitialContextCreated{providerUri=
http://localhost:46355/tomee/ejb}
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time
elapsed: 3.25
s - in org.apache.tomee.RemoteTomEEEJBContainerIT
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 7, Failures: 0, Errors: 0, Skipped: 0
Is possible to trigger build from buildbot for my PR?
Em qua, 19 de dez de 2018 às 11:36, Roberto Cortez
<[email protected]> escreveu:
The CDI TCK runs after the OpenEJB Core, so you should be able to
see the
debug information you added.
I’m not sure if you noticed it, but I did add a @Ignore to the
AutoConnectionTracker test until we figure out what is wrong.
On 19 Dec 2018, at 13:51, Jean-Louis Monteiro <
[email protected]>
wrote:
Roberto, I saw you commented the test in which I added some more
stuff
to
test.
Do you have some inputs after the run?
Or are we still stuck on CDI ?
--
Jean-Louis Monteiro
http://twitter.com/jlouismonteiro
http://www.tomitribe.com
On Wed, Dec 19, 2018 at 12:44 PM Daniel Cunha <[email protected]>
wrote:
I saw your comment and it's weird.
I'll look it again, the patch should fix exactly the test which you
are
getting error. :)
Thanks for review.
Em qua, 19 de dez de 2018 8:08 AM, Roberto Cortez
<[email protected] escreveu:
Hi Daniel,
I’m having test failures in the PR updates for 7.0.x and 7.1.x. Can
you
confirm? Left you a note in the PR’s.
Cheers,
Roberto
On 18 Dec 2018, at 15:18, Jean-Louis Monteiro <
[email protected]>
wrote:
I pushed some debugging information on the other test and I'll be
monitoring the built
--
Jean-Louis Monteiro
http://twitter.com/jlouismonteiro
http://www.tomitribe.com
On Tue, Dec 18, 2018 at 4:16 PM Daniel Cunha
<[email protected]
wrote:
Nice... Thanks!
Em ter, 18 de dez de 2018 11:48 AM, Roberto Cortez
<[email protected] escreveu:
Hi Daniel,
I didn't yet for the other branches. I'll do it today.
On Tuesday, December 18, 2018, 1:37:34 PM GMT, Daniel Cunha <
[email protected]> wrote:
Hi Roberto,
I saw that you merged the changes on master, did you see the
others
changes
for branch 7.0.x and 7.1.x?
Em seg, 17 de dez de 2018 às 18:07, Roberto Cortez
<[email protected]> escreveu:
Thank you Daniel,
I’ll review and merge.
Yeah it seems it gets stuck in the CDI TCK. I saw it
happening a
couple
of
days before, but now it happened again. Not sure what is wrong
there,
since
we used to pass it.
On 17 Dec 2018, at 20:16, Daniel Cunha <[email protected]>
wrote:
Hi Roberto,
the fix which I send is to make sure that the test is using
the
correct
version for libs.
For example, here
https://github.com/apache/tomee/pull/287/files#diff-88785d5b00bc141a892b2e21e6aaabcaL70
we have the version of the lib on the code.
Which means that, if you try run it on TomEE 7.0.x it will
never
works,
because for that build, it will have openejb-core-7.0.x and
not
openejb-core-8.0.0-SNAPSHOT.
That is the issue, we need to get the correct reference for
the
lib
to
make
sure that the command will works.
About the test stuck, reading the stacktrace that you send,
seems
something
during TCK test. Is it?
Em seg, 17 de dez de 2018 às 17:08, Roberto Cortez
<[email protected]> escreveu:
Hi Daniel,
Can you please explain what the issue was?
Also, it seems our build is getting killed for taking to long
due
to
some
tests being stuck?
Dec 17, 2018 12:06:54 PM
org.apache.openejb.client.EventLogger
log
INFO: RemoteInitialContextCreated{providerUri=
http://localhost:40395/tomee/ejb}
Dec 17, 2018 12:10:27 PM
org.apache.openejb.client.EventLogger
log
WARNING:
RequestFailed{server=http://localhost:40395/tomee/ejb
}
JNDI_LOOKUP:/openejb/DeployerBusinessRemote {error=Cannot
open
object
input
stream to server (OEJP/4.6) : null}
Dec 17, 2018 12:10:27 PM
org.jboss.cdi.tck.impl.testng.ProgressLoggingTestListener
beforeInvocation
INFO: Invoke
FinalMethodWithInheritedStereotypeInterceptorTest.testFinalClassWithInheritedStereotypeInterceptor:
1,203/1,571 Failed tests: 59 (476)
Dec 17, 2018 12:10:27 PM
org.apache.openejb.arquillian.common.TomEEContainer undeploy
WARNING: d18bed2dab5751811c186784503e2ad69ae6c9.war was not
deployed
Dec 17, 2018 12:10:27 PM
org.jboss.cdi.tck.shrinkwrap.ArchiveBuilder
build
INFO: Test archive built [info:
org.jboss.cdi.tck.tests.interceptors.definition.interceptorCalledBeforeDecorator.InterceptorCalledBeforeDecoratorTest,
time: 16 ms]
Dec 17, 2018 12:10:28 PM
org.apache.openejb.client.EventLogger
log
INFO: RemoteInitialContextCreated{providerUri=
http://localhost:40395/tomee/ejb}
command timed out: 18000 seconds without output running
['mvn',
'-U',
'--show-version', '--fail-at-end',
'-Djava.io.tmpdir=/tmp/tomee-trunk-ubuntu-jvm8', 'clean',
'install',
'-Pall-adapters', '-Dsurefire.useFile=false',
'-DdisableXmlReport=true',
'-Dopenejb.arquillian.debug=true',
'-DTOMEE_LOCK_FILE=/tmp/tomee.port.lock'], attempting to kill
process killed by signal 9
program finished with exit code -1
elapsedTime=38754.464253
Last log is from 12:10:28. Process was killed around 17:10:30
PM.
No
logs
in between...
On 17 Dec 2018, at 17:27, Daniel Cunha
<[email protected]
wrote:
Hey folks,
we found an issue on TomEECliIT test.
I've opened fixes for test in all branch, please take a look
on
it.
Fixes:
master -> https://github.com/apache/tomee/pull/288
7.1.x -> https://github.com/apache/tomee/pull/287
7.0.x -> https://github.com/apache/tomee/pull/286
Em sex, 14 de dez de 2018 às 12:36, Jean-Louis Monteiro <
[email protected]> escreveu:
Please wait a minute lol
I'd rather have you help me if needed.
First I don't have the feeling I spent time for nothing
on it
and
second,
at least I will learn something.
--
Jean-Louis Monteiro
http://twitter.com/jlouismonteiro
http://www.tomitribe.com
On Fri, Dec 14, 2018 at 4:33 PM Jonathan Gallimore <
[email protected]> wrote:
I added this test, and it was pretty stable. Happy to help
look
into
it.
On Fri, Dec 14, 2018 at 10:44 AM Roberto Cortez
<[email protected]>
wrote:
It seems the build just randomly fails with this test:
AutoConnectionTrackerTest.test
[ERROR] Failures:
[ERROR] AutoConnectionTrackerTest.test:198 expected:<1>
but
was:<0>
Sometimes it passes, sometimes it fails. I suspect it may
be
related
with
some timing issues / box speed of the build bot. I’m
running
the
test
non
stop on my box (already executed 100+ times) and I
couldn’t
replicate
the
issue:
Any ideias?
On 12 Dec 2018, at 16:25, Jean-Louis Monteiro <
[email protected]>
wrote:
I'll take the first one first
--
Jean-Louis Monteiro
http://twitter.com/jlouismonteiro
http://www.tomitribe.com
On Wed, Dec 12, 2018 at 5:21 PM Roberto Cortez
<[email protected]>
wrote:
Thank you JL,
At least this test is failing:
org.apache.openejb.config.rules.CheckMissingClassTest
And this one seems to fail randomly.
org.apache.openejb.resource.AutoConnectionTrackerTest
Cheers,
Roberto
On 12 Dec 2018, at 16:18, Jean-Louis Monteiro <
[email protected]>
wrote:
I have merged a couple of PR and I'm running a full
build
locally.
I'll
work on fixes any issues if I find some but I'll
report
first
here
--
Jean-Louis Monteiro
http://twitter.com/jlouismonteiro
http://www.tomitribe.com
On Wed, Dec 12, 2018 at 3:33 PM Roberto Cortez
<[email protected]>
wrote:
Hey Jon,
The following commit:
https://github.com/apache/tomee/commit/4d535f969e9769f0403fddd100b085a0d1de15a9
<
https://github.com/apache/tomee/commit/4d535f969e9769f0403fddd100b085a0d1de15a9
Is failing the test:
org.apache.openejb.config.rules.CheckMissingClassTest
Can you help?
Cheers,
Roberto
On 12 Dec 2018, at 04:41, Frankie <
[email protected]
wrote:
Hi Cesar,
This is a valuable piece of information.
That's what I thohght, too when I read the
information
on
IRC
...
Thanks that you want to preserve it on the website.
The JIRA task is now assigned to you.
Frankie
--
Sent from:
http://tomee-openejb.979440.n4.nabble.com/TomEE-Dev-f982480.html
--
Daniel "soro" Cunha
https://twitter.com/dvlc_
--
Daniel "soro" Cunha
https://twitter.com/dvlc_
--
Daniel "soro" Cunha
https://twitter.com/dvlc_
--
Daniel "soro" Cunha
https://twitter.com/dvlc_
--
Richard Zowalla
Lindenstraße 21
74348 Lauffen am Neckar
