uschindler commented on code in PR #12293:
URL: https://github.com/apache/lucene/pull/12293#discussion_r1200504015


##########
gradle/ge.gradle:
##########
@@ -0,0 +1,26 @@
+def isGithubActions = System.getenv('GITHUB_ACTIONS') != null
+def isJenkins = System.getenv('JENKINS_URL') != null
+def isCI = isGithubActions || isJenkins
+
+gradleEnterprise {
+    server = "https://ge.apache.org";

Review Comment:
   Maybe just improve the check and use a regex on the JENKINS_URL to figure 
out we run on correct server and only enable it there.
   
   It is important that the build does not fail in "other" CI environments, 
because there are many environments where Lucene code is built as our large 
quality framework: Elastic, Opensearch, Policeman Jenkins (JDK variant and 
Garbage Collector testing),.... They should not need a key. So lets's maybe 
only enable it for Jenkins of ASF. 
   
   Github CI is only to validate Pull requests, it is also not needed there.



-- 
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