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

Add configuration to send build scans to https://ge.apache.org #810

Merged
merged 2 commits into from
Jun 11, 2023

Conversation

clayburn
Copy link
Contributor

No description provided.

@guylabs
Copy link

guylabs commented Mar 13, 2023

Hi @ppalaga,

the Apache foundation has a Gradle Enterprise instance running at https://ge.apache.org. This instance can be used for any Apache project and projects can opt in for free.

This change enables using this instance of Gradle Enterprise for the Maven daemon project. Once merged, you get faster builds instantaneously due to Gradle's performance accelerations techniques for Maven, like build caching (remote build caching will come in summer for Apache projects).

It will as well publish build scans for every execution of a build on CI as well as on any local developer machine to get insights into the build, ease investigation of failures, reason about outcomes and so forth.

You can read more about it here.

If you have any questions feel free to ping us.

Looking forward to have the Maven daemon project using ge.apache.org.

Regarding the integration test failures with these changes, I'll need to check with @oehme as we might need to reopen #725. We'll keep you posted.

@oehme
Copy link
Contributor

oehme commented Mar 14, 2023

The integ tests have their working dir inside the main project's target folder. So when they search for the .mvn directory and they don't have one themselves, they find the dir you just added. The fix would be to make sure that each integ test has at least an empty .mvn directory, so it doesn't search upwards. We do the same in the integ tests of our extension.

@guylabs
Copy link

guylabs commented Mar 24, 2023

@ppalaga any update on this? Would you be interested in a short call to answer questions and show you potential benefits of using Gradle Enterprise?

@ppalaga
Copy link
Contributor

ppalaga commented Mar 29, 2023

Sorry gentlemen, no time at all for mvnd this month.

@guylabs
Copy link

guylabs commented May 4, 2023

Hi @ppalaga, gentle ping :)

Some of the integration tests to do not have `.mvn` directories and
search up the project structure until they find the `.mvn` directory
of the root project.

This change adds `.mvn` directories with empty `maven.config` files so
that the sample projects in VCS will be as close as possible to those
executed during integration testing
@clayburn
Copy link
Contributor Author

clayburn commented Jun 9, 2023

I've rebased this with the latest changes in master and with the latest plugin version. Let me know if there are any questions or more information needed.

@gnodet
Copy link
Contributor

gnodet commented Jun 9, 2023

I've rebased this with the latest changes in master and with the latest plugin version. Let me know if there are any questions or more information needed.

Is there any maven based project at Apache using this extension so far ? It seems there's only beam which is using gradle.

@clayburn
Copy link
Contributor Author

clayburn commented Jun 9, 2023

I've rebased this with the latest changes in master and with the latest plugin version. Let me know if there are any questions or more information needed.

Is there any maven based project at Apache using this extension so far ? It seems there's only beam which is using gradle.

You can filter on just Maven projects, but Pulsar, Curator, and SCIMple are currently using it. Beam just happens to constitute a large percentage of the scans right now.

@gnodet gnodet merged commit 0e07a38 into apache:master Jun 11, 2023
4 checks passed
@clayburn clayburn deleted the connect-to-ge.apache.org branch June 12, 2023 13:44
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

5 participants