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 <[email protected]>
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 <[email protected]>
Signed-off-by: David Matějček <[email protected]>
- throws an exception if it was not possible to delete some files. Signed-off-by: David Matějček <[email protected]>
- 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 <[email protected]>
…better max pool size logic. Fix incorrect javadoc.
Fixed JmacHttpsTest - certificate authentication
Signed-off-by: David Matějček <[email protected]>
Signed-off-by: David Matějček <[email protected]>
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 <[email protected]>
Signed-off-by: Alexander Pinčuk <[email protected]>
Signed-off-by: Alexander Pinčuk <[email protected]>
Signed-off-by: Alexander Pinčuk <[email protected]>
Webpage update for 7.0.14
Signed-off-by: Alexander Pinčuk <[email protected]>
Signed-off-by: Alexander Pinčuk <[email protected]>
Signed-off-by: Alexander Pinčuk <[email protected]>
Signed-off-by: Alexander Pinčuk <[email protected]>
…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 <[email protected]>
Signed-off-by: Alexander Pinčuk <[email protected]>
Signed-off-by: Alexander Pinčuk <[email protected]>
…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 <[email protected]>
Signed-off-by: Alexander Pinčuk <[email protected]>
Signed-off-by: Alexander Pinčuk <[email protected]>
Integrate JaCoCo 0.8.12
Signed-off-by: Alexander Pinčuk <[email protected]>
Integrate Jersey 3.1.6
Enable CDI Lang Model TCK
Signed-off-by: Alexander Pinčuk <[email protected]>
Update Jakarta RESTful Web Services TCK to 3.1.5
Signed-off-by: Alexander Pinčuk <[email protected]>
Signed-off-by: Alexander Pinčuk <[email protected]>
Signed-off-by: Alexander Pinčuk <[email protected]>
Signed-off-by: Alexander Pinčuk <[email protected]>
Signed-off-by: Alexander Pinčuk <[email protected]>
…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 <[email protected]>
Signed-off-by: Arjan Tijms <[email protected]>
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.