rmuir commented on PR #13164:
URL: https://github.com/apache/lucene/pull/13164#issuecomment-1981725768

   I tried it out, i don't want to conflict with your changes here, but if we 
add this to main policy:
   ```
   --- a/gradle/testing/randomization/policies/tests.policy
   +++ b/gradle/testing/randomization/policies/tests.policy
   @@ -61,6 +61,8 @@ grant {
   
      // TestLockFactoriesMultiJVM opens a random port on 127.0.0.1 (port 0 = 
ephemeral port range):
      permission java.net.SocketPermission "127.0.0.1:0", 
"accept,listen,resolve";
   +  // Replicator tests connect to ephemeral ports
   +  permission java.net.SocketPermission "127.0.0.1:1024-", "connect,resolve";
   
      // read access to all system properties:
      permission java.util.PropertyPermission "*", "read";
   ```
   
   Then we can remove the entire `replicator-tests.policy` and its associated 
if-then-else in `randomization.gradle` and just have one of these files. 
replicator tests must be run with `-Dtests.nightly` as a lot of them are 
monstrous, but it seems to work. so it might make it less eye-gouging :)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to