Skip to content
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

Capture build scans on ge.apache.org to benefit from deep build insights #11767

Merged
merged 1 commit into from Oct 10, 2023

Conversation

clayburn
Copy link
Contributor

@clayburn clayburn commented Oct 9, 2023

@vvivekiyer - It was nice to meet you at Community over Code this weekend. This is the PR we discussed to publish Build Scans to ge.apache.org.

This PR publishes a build scan for every CI build and for every local build from an authenticated Apache committer. The build will not fail if publishing fails.

The build scans of the Apache Pinot project are published to the Develocity instance at ge.apache.org, hosted by the Apache Software Foundation and run in partnership between the ASF and Gradle. This Develocity instance has all features and extensions enabled and is freely available for use by the Apache Pinot project and all other Apache projects.

On this Develocity instance, Apache Pinot will have access not only to all of the published build scans but other aggregate data features such as:

Dashboards to view all historical build scans, along with performance trends over time
Build failure analytics for enhanced investigation and diagnosis of build failures
Test failure analytics to better understand trends and causes around slow, failing, and flaky tests
Please let me know if there are any questions about the value of Develocity or the changes in this pull request and I’d be happy to address them.

@xiangfu0
Copy link
Contributor

xiangfu0 commented Oct 9, 2023

Thanks @clayburn !

@@ -58,6 +58,7 @@ jobs:
cache: 'maven'
- name: Linter Test
env:
GRADLE_ENTERPRISE_ACCESS_KEY: ${{ secrets.GE_ACCESS_TOKEN }}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this GE_ACCESS_TOKEN an apache org secret?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct, it is an apache org secret managed by the infra team

@codecov-commenter
Copy link

codecov-commenter commented Oct 9, 2023

Codecov Report

Merging #11767 (da1f212) into master (24af80d) will increase coverage by 48.63%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master   #11767       +/-   ##
=============================================
+ Coverage     14.45%   63.08%   +48.63%     
- Complexity      201     1117      +916     
=============================================
  Files          2342     2342               
  Lines        125917   125917               
  Branches      19370    19370               
=============================================
+ Hits          18205    79441    +61236     
+ Misses       106170    40824    -65346     
- Partials       1542     5652     +4110     
Flag Coverage Δ
integration <0.01% <ø> (ø)
integration1 <0.01% <ø> (ø)
integration2 0.00% <ø> (ø)
java-11 63.05% <ø> (+48.63%) ⬆️
java-17 62.95% <ø> (+48.51%) ⬆️
java-20 62.95% <ø> (+48.53%) ⬆️
temurin 63.08% <ø> (+48.63%) ⬆️
unittests 63.08% <ø> (+48.63%) ⬆️
unittests1 67.22% <ø> (?)
unittests2 14.42% <ø> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1514 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@xiangfu0 xiangfu0 merged commit 17ffa3e into apache:master Oct 10, 2023
19 of 20 checks passed
@vvivekiyer
Copy link
Contributor

Thanks @clayburn for creating this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants