-
Notifications
You must be signed in to change notification settings - Fork 333
Publish build scans to develocity.apache.org #1714
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
base: master
Are you sure you want to change the base?
Conversation
|
ping @apache/samza-committers - this is required as the host ge.a.o has been replaced with develocity.a.o. |
|
@clayburn Looks like the CI build is failing. The failure cause isn't apparent from the logs. |
|
@mynameborat - it is unrelated, but I think you are seeing a failure due to an outdated version of the wrapper validation action: gradle/wrapper-validation-action#174 I don't mind updating it as part of this PR if you like, just let me know. But this change certainly is not at fault. |
|
Actually, I just went ahead and updated it, but I can back it out if you don't want the change in this PR. |
|
@mynameborat - does this resolve the concern? |
|
We are out of time really and so unless someone else can get to it, I'll merge this myself on Monday |
|
Just to verify, this test (and all tests) are passing for me locally: https://ge.solutions-team.gradle.com/s/53e6zqb52qpgq/tests/task/:samza-azure_2.12:test/details/org.apache.samza.system.azureblob.producer.TestAzureBlobSystemProducer/testMutipleThreadSendFlushToSingleWriter?top-execution=1 |
1f9859f to
4b8d596
Compare
|
@mynameborat - I was just looking through old PRs and saw this still opened. I rebased, can we re-run the tests now? |
This PR migrates the Samza project to publish Build Scans to the the new Develocity instance at develocity.apache.org.
Additionally, this PR migrates from the legacy Gradle Enterprise plugin to the renamed Develocity plugin and sets a projectId for use by Develocity.