Skip to content

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

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

Merged
merged 2 commits into from
May 24, 2023

Conversation

clayburn
Copy link
Contributor

@clayburn clayburn commented May 9, 2023

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

Description

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

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

Currently, the Apache OpenWhisk build is configured to publish build scans to the publicly available scans.gradle.com. This pull request enhances that functionality by instead publishing build scans to ge.apache.org. On this Gradle Enterprise instance, Apache OpenWhisk 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

If interested in exploring a fully populated Gradle Enterprise instance, please explore the Maven builds already connected to ge.apache.org, the Spring project’s instance, or any number of other OSS projects for which we sponsor instances of Gradle Enterprise.

Please let me know if there are any questions about the value of Gradle Enterprise or the changes in this pull request and I’d be happy to address them.

Related issue and scope

  • I opened an issue to propose and discuss this change (#????)

My changes affect the following components

  • API
  • Controller
  • Message Bus (e.g., Kafka)
  • Loadbalancer
  • Scheduler
  • Invoker
  • Intrinsic actions (e.g., sequences, conductors)
  • Data stores (e.g., CouchDB)
  • Tests
  • Deployment
  • CLI
  • General tooling
  • Documentation

Types of changes

  • Bug fix (generally a non-breaking change which closes an issue).
  • Enhancement or new feature (adds new functionality).
  • Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

  • I signed an Apache CLA.
  • I reviewed the style guides and followed the recommendations (Travis CI will check :).
  • I added tests to cover my changes.
  • My changes require further changes to the documentation.
  • I updated the documentation where necessary.

@codecov-commenter
Copy link

codecov-commenter commented May 11, 2023

Codecov Report

Merging #5411 (487541d) into master (72bb2a1) will decrease coverage by 0.31%.
The diff coverage is 66.66%.

❗ Current head 487541d differs from pull request most recent head adf5163. Consider uploading reports for the commit adf5163 to get more accurate results

@@            Coverage Diff             @@
##           master    #5411      +/-   ##
==========================================
- Coverage   76.83%   76.52%   -0.31%     
==========================================
  Files         241      241              
  Lines       14630    14630              
  Branches      616      616              
==========================================
- Hits        11241    11196      -45     
- Misses       3389     3434      +45     
Impacted Files Coverage Δ
...enwhisk/core/entity/InstanceConcurrencyLimit.scala 75.00% <ø> (ø)
.../openwhisk/core/entity/IntraConcurrencyLimit.scala 90.32% <ø> (ø)
...che/openwhisk/core/loadBalancer/LeanBalancer.scala 0.00% <0.00%> (ø)
...pache/openwhisk/core/invoker/InvokerReactive.scala 53.84% <ø> (-17.70%) ⬇️
.../org/apache/openwhisk/core/controller/Limits.scala 91.30% <100.00%> (ø)
...la/org/apache/openwhisk/core/invoker/Invoker.scala 72.15% <100.00%> (ø)

... and 9 files with indirect coverage changes

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

@clayburn
Copy link
Contributor Author

Hi, is anything needed to get this reviewed and merged?

@style95
Copy link
Member

style95 commented May 24, 2023

@clayburn Thank you for the contribution.

@dgrove-oss dgrove-oss merged commit 37605b4 into apache:master May 24, 2023
@clayburn clayburn deleted the connect-to-ge.apache.org branch May 24, 2023 15:43
clayburn added a commit to clayburn/openwhisk that referenced this pull request Jun 20, 2023
This change allows GitHub Actions builds to submit build scans to ge.apache.org by authenticating those builds. The access key has been stored as an organizational secret by the ASF Infrastructure team in the Apache GitHub organization. The access key is not available to workflows triggered from forks.

This builds on the changes in apache#5411
dgrove-oss pushed a commit that referenced this pull request Jul 6, 2023
This change allows GitHub Actions builds to submit build scans to ge.apache.org by authenticating those builds. The access key has been stored as an organizational secret by the ASF Infrastructure team in the Apache GitHub organization. The access key is not available to workflows triggered from forks.

This builds on the changes in #5411
mtt-merz pushed a commit to mtt-merz/openwhisk that referenced this pull request Oct 22, 2023
mtt-merz pushed a commit to mtt-merz/openwhisk that referenced this pull request Oct 22, 2023
This change allows GitHub Actions builds to submit build scans to ge.apache.org by authenticating those builds. The access key has been stored as an organizational secret by the ASF Infrastructure team in the Apache GitHub organization. The access key is not available to workflows triggered from forks.

This builds on the changes in apache#5411

(cherry picked from commit be3f6d6)
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.

4 participants