clayburn commented on code in PR #2264:
URL: https://github.com/apache/solr/pull/2264#discussion_r1490136801


##########
gradle/ge.gradle:
##########
@@ -27,6 +31,25 @@ gradleEnterprise {
         obfuscation {
             ipAddresses { addresses -> addresses.collect { address -> 
"0.0.0.0"} }
         }
+
+        tag("JDK" + System.getProperty("java.version").find(/\d+/)) // major 
version
+        value("Java Vendor", System.getProperty("java.vendor"))
+        value("Java Version", System.getProperty("java.version"))
+
+        // Jenkins job name, less redundant "Solr" parts
+        String jenkinsJobName = System.getenv("JOB_NAME")
+        if (jenkinsJobName) { // looks like "Solr/Solr-Smoketest-9.5"
+            tag(jenkinsJobName.replaceAll(/Solr\W/,"")) // like "Smoketest-9.5"
+        }
+
+        // 
https://docs.github.com/en/actions/learn-github-actions/variables#default-environment-variables
+        if (System.getenv("GITHUB_BASE_REF")) { // only exists for PRs
+            tag("PR")
+        }
+        String ghWorkflowName = System.getenv("GITHUB_WORKFLOW")
+        if (ghWorkflowName) {
+            tag(ghWorkflowName)

Review Comment:
   The Common Custom User Data Gradle Plugin already adds this as a custom 
value called `CI workflow`



##########
gradle/ge.gradle:
##########
@@ -27,6 +31,25 @@ gradleEnterprise {
         obfuscation {
             ipAddresses { addresses -> addresses.collect { address -> 
"0.0.0.0"} }
         }
+
+        tag("JDK" + System.getProperty("java.version").find(/\d+/)) // major 
version

Review Comment:
   For what it's worth, these JDK details are already captured in the [build 
scan](https://ge.apache.org/s/iria4ppalnbdo#infrastructure). This information 
will show up in build scan comparisons as well. But capturing as tags and value 
is nice if you want the ability to filter on these values.



##########
gradle/ge.gradle:
##########
@@ -27,6 +31,25 @@ gradleEnterprise {
         obfuscation {
             ipAddresses { addresses -> addresses.collect { address -> 
"0.0.0.0"} }
         }
+
+        tag("JDK" + System.getProperty("java.version").find(/\d+/)) // major 
version
+        value("Java Vendor", System.getProperty("java.vendor"))
+        value("Java Version", System.getProperty("java.version"))
+
+        // Jenkins job name, less redundant "Solr" parts
+        String jenkinsJobName = System.getenv("JOB_NAME")
+        if (jenkinsJobName) { // looks like "Solr/Solr-Smoketest-9.5"
+            tag(jenkinsJobName.replaceAll(/Solr\W/,"")) // like "Smoketest-9.5"

Review Comment:
   The Common Custom User Data Gradle Plugin already adds this as a custom 
value called `CI job` (but without the filtering you've added).



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to