-
Notifications
You must be signed in to change notification settings - Fork 332
Publish build scans to develocity.apache.org for build insights #2559
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -109,6 +109,11 @@ pluginManagement { | |
| } | ||
| } | ||
|
|
||
| plugins { | ||
| id("com.gradle.develocity") version "4.1.1" | ||
| id("com.gradle.common-custom-user-data-gradle-plugin") version "2.3" | ||
| } | ||
|
|
||
| dependencyResolutionManagement { | ||
| repositoriesMode = RepositoriesMode.FAIL_ON_PROJECT_REPOS | ||
| repositories { | ||
|
|
@@ -126,3 +131,15 @@ gradle.beforeProject { | |
| version = baseVersion | ||
| group = "org.apache.polaris" | ||
| } | ||
|
|
||
| val isCI = System.getenv("CI") != null | ||
|
|
||
| develocity { | ||
| server = "https://develocity.apache.org" | ||
| projectId = "polaris" | ||
| buildScan { | ||
| uploadInBackground = !isCI | ||
| publishing.onlyIf { it.isAuthenticated } | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sending scans when authenticated SGTM... Still, it might be worth mentioning it in the main README for awareness. WDYT?
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Apache's Develocity instance requires authentication. (See my other comment).
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I added a bit to the README about build scan publication to the README. |
||
| obfuscation { ipAddresses { addresses -> addresses.map { _ -> "0.0.0.0" } } } | ||
| } | ||
| } | ||
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.
Would it be possible to use https://github.com/apache/polaris/blob/main/gradle/baselibs.versions.toml for version management (as we do for other build plugins)?
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.
It's a "settings plugin", so sadly no.
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.
@snazy is correct. Unfortunately this is not doable