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

Publish build scans to develocity.apache.org #1714

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

clayburn
Copy link
Contributor

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.

@gmcdonald
Copy link

ping @apache/samza-committers - this is required as the host ge.a.o has been replaced with develocity.a.o.
If not merged soon then INFRA will go ahead, thank you!

@mynameborat
Copy link
Contributor

@clayburn Looks like the CI build is failing. The failure cause isn't apparent from the logs.

@clayburn
Copy link
Contributor Author

@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.

@clayburn
Copy link
Contributor Author

Actually, I just went ahead and updated it, but I can back it out if you don't want the change in this PR.

@clayburn
Copy link
Contributor Author

@mynameborat - does this resolve the concern?

@gmcdonald
Copy link

We are out of time really and so unless someone else can get to it, I'll merge this myself on Monday

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.

3 participants