Merge master into 8.0 #24904
Merged
Merge master into 8.0 #24904
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…better max pool size logic. Add lock, add notifyWaitingThreads call for better max pool size logic. No longer getResources while the pool is being resized. And inform waiting threads when resources are marked as failed.
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Integrate OW2 ASM 9.7
…ant equals - Note that the equals method is not symmetric here - Exousia uses Set.contains -> equals to check the authorization. Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
- throws an exception if it was not possible to delete some files. Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
- The file is owned by the domain, not by the application and not by the test. - The server will notice that the file vanished at some point in time, but we cannot rely on that. - Another reliable solution is to stop-domain - delete lib file - start-domain - Also it is a bad practice to use the same library name for different contents. Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
…better max pool size logic. Fix incorrect javadoc.
Fixed JmacHttpsTest - certificate authentication
Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
Signed-off-by: David Matějček <david.matejcek@omnifish.ee>
Added staging repo to runtests.sh + archiving rotated server.log files on jenkins
Fixes eclipse-ee4j#23483 Add lock, add notifyWaitingThreads call for better max pool size logic.
7.0.14 branch merge
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Webpage update for 7.0.14
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
…ven_plugin_8_0_2 Integrate Maven Git Commit Id plugin 8.0.2
…lugin_3_13_0 Integrate Maven Compiler plugin 3.13.0
…ugin_3_6_3 Integrate Maven Javadoc plugin 3.6.3
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
…lugin_2_16_2 Update Versions Maven plugin to 2.16.2
…xrefs Fixes cross-references in Administration Guide
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Integrate JaCoCo 0.8.12
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Integrate Jersey 3.1.6
Enable CDI Lang Model TCK
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Update Jakarta RESTful Web Services TCK to 3.1.5
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
…1_3_1 Integrate JBoss Class File Writer 1.3.1.Final
Integrate Jakarta JSON Binding 3.0.1
Signed-off-by: Alexander Pinčuk <alexander.v.pinchuk@gmail.com>
Signed-off-by: Arjan Tijms <arjan.tijms@omnifish.ee>
dmatej
reviewed
Apr 16, 2024
| @@ -867,6 +868,7 @@ | |||
| <artifactId>maven-failsafe-plugin</artifactId> | |||
| <version>3.2.5</version> | |||
| <configuration> | |||
| <rerunFailingTestsCount>3</rerunFailingTestsCount> | |||
There was a problem hiding this comment.
Why this shitty thing again? That means waiting for the build forever ...
There was a problem hiding this comment.
I often get a failure, and then I need to manually re-run it. Better to have it automated. It warns when the test is flakey.
There was a problem hiding this comment.
Hmm, in Jenkins same test failed four times in a row. Something wrong in the connection pool...
There was a problem hiding this comment.
The weird thing is that locally it can just build the branch. I've to investigate what I'm doing wrong. Probably something silly...
There was a problem hiding this comment.
It always fails on Jenkins since the previous PR. I think you should not merge it and start screaming that something is wrong with master ;-)
There was a problem hiding this comment.
Flaky test = something is broken, this is an attempt to hide it.
There was a problem hiding this comment.
I have found another issue with java home, I have to fix it first, then I will take a look what is wrong with pools. But I really oppose this change - Jenkins is here to report every issue and keep us safe, not to "make it green, anyhow".
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.