-
-
Notifications
You must be signed in to change notification settings - Fork 8.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merge March 8 security tests into their main class #7700
Conversation
01a6103
to
d18a0ef
Compare
|
||
@Test | ||
@Issue("SECURITY-2823") | ||
public void verifyUploadedPluginPermission() throws Exception { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we ignore this test until it is stabilised please?
I can reproduce it 100% with:
mvn test -Dtest=hudson.PluginManagerTest#verifyUploadedPluginPermission
(on macOS)
No matter how long you wait the file is not there, possibly it gets deleted too quickly or its going somewhere else?
It passes in IntelliJ though
(have tried a bit but haven't figured it out)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(although this test does pass for me on master weirdly although failing I think most of the time on CI)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests pass for me locally with Java 11 on RHEL 8, Java 11 on Ubuntu 22.04, and Java 11 on Debian testing with the command:
git clean -xffd; mvn clean -DforkCount=1C -Dtest=PluginManagerTest,DirectoryBrowserSupportTest,ComputerSetTest,ExecutorTest,SearchTest clean verify
PluginManagerTest
fails for me with Java 17 on RHEL 8, Java 17 on Ubuntu 22.04, and Java 17 on Debian testing with the same command.
The Java 17 failures reported:
PluginManagerTest.verifyUploadedPluginPermission:757 » ConditionTimeout Condition with hudson.PluginManagerTest was not fulfilled within 10 seconds.
Apparently on Java 17, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PluginManagerTest
is now passing with Java 17 on RHEL 8, Java 17 on Ubuntu 22.04, and Java 17 on Debian testing. Thanks very much!
I recommend that we skip the 24 hour merge window for this and instead merge it as soon as CI passes. |
+1
… On 10. Mar 2023, at 13:35, Mark Waite ***@***.***> wrote:
I recommend that we skip the 24 hour merge window for this and instead merge it as soon as CI passes.
—
Reply to this email directly, view it on GitHub <#7700 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADGDOVMGGNQAK47OXRGVXDDW3MN2PANCNFSM6AAAAAAVVCWABY>.
You are receiving this because you are subscribed to this thread.
|
(cherry picked from commit 9d97938)
This PR aims 2 goals:
Testing done
Modified tests successfully run on Java 11 and Java 17 on Ubuntu 22.04, RHEL 8.7, and Debian testing
Proposed changelog entries
N/A
Proposed upgrade guidelines
N/A
Submitter checklist
@Restricted
or have@since TODO
Javadocs, as appropriate.@Deprecated(since = "TODO")
or@Deprecated(forRemoval = true, since = "TODO")
, if applicable.eval
to ease future introduction of Content Security Policy (CSP) directives (see documentation).Desired reviewers
@daniel-beck
Maintainer checklist
Before the changes are marked as
ready-for-merge
:upgrade-guide-needed
label is set and there is a Proposed upgrade guidelines section in the pull request title (see example).lts-candidate
to be considered (see query).