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

[Build] Cache spotless p2 dependencies when baking ci image #118523

Merged
merged 2 commits into from
Dec 12, 2024

Conversation

breskeby
Copy link
Contributor

The eclipse formatter used by spotless is resolved at runtime and not declared as
gradle dependency. Therefore we need to run the spotless task to ensure we have
the dependencies resolved as part of our ci image baking.

This should avoid issues with connecting to p2 repos we have experienced lately
in our ci environment

@breskeby breskeby requested a review from a team as a code owner December 12, 2024 07:44
@elasticsearchmachine elasticsearchmachine added v9.0.0 needs:triage Requires assignment of a team area label labels Dec 12, 2024
The eclipse formatter used by spotless is resolved at runtime and not declared as
gradle dependency. Therefore we need to run the spotless task to ensure we have
the dependencies resolved as part of our ci image baking.

This should avoid issues with connecting to p2 repos we have experienced lately
in our ci environment
@breskeby breskeby force-pushed the cache-eclipse-formatter-deps branch from 95c412f to 8f62046 Compare December 12, 2024 07:52
@breskeby breskeby added >non-issue :Delivery/Build Build or test infrastructure Team:Delivery Meta label for Delivery team labels Dec 12, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

@elasticsearchmachine elasticsearchmachine removed the needs:triage Requires assignment of a team area label label Dec 12, 2024
@breskeby breskeby added needs:triage Requires assignment of a team area label v7.17.21 v8.17.1 v8.18.0 auto-backport Automatically create backport pull requests when merged and removed needs:triage Requires assignment of a team area label labels Dec 12, 2024
@breskeby breskeby changed the title [Build] Cache eclipse formatter p2 dependency when baking ci image [Build] Cache spotless p2 dependency when baking ci image Dec 12, 2024
@breskeby breskeby changed the title [Build] Cache spotless p2 dependency when baking ci image [Build] Cache spotless p2 dependencies when baking ci image Dec 12, 2024
@breskeby breskeby enabled auto-merge (squash) December 12, 2024 08:03
@@ -301,7 +301,10 @@ allprojects {
if (project.path.contains(":distribution:docker")) {
enabled = false
}

if (project.path.contains(":libs:cli")) {
Copy link
Contributor

Choose a reason for hiding this comment

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

I assume we are just arbitrarily picking a project here, yes?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes

@breskeby breskeby merged commit 9503afc into elastic:main Dec 12, 2024
16 checks passed
breskeby added a commit to breskeby/elasticsearch that referenced this pull request Dec 12, 2024
…#118523)

The eclipse formatter used by spotless is resolved at runtime and not declared as
gradle dependency. Therefore we need to run the spotless task to ensure we have
the dependencies resolved as part of our ci image baking.

This should avoid issues with connecting to p2 repos we have experienced lately
in our ci environment

* Revert "[Build] Declare mirror for eclipse p2 repository (elastic#117732)"

This reverts commit c35777a.
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
7.17 Commit could not be cherrypicked due to conflicts
8.17
8.x

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 118523

breskeby added a commit to breskeby/elasticsearch that referenced this pull request Dec 12, 2024
…#118523)

The eclipse formatter used by spotless is resolved at runtime and not declared as
gradle dependency. Therefore we need to run the spotless task to ensure we have
the dependencies resolved as part of our ci image baking.

This should avoid issues with connecting to p2 repos we have experienced lately
in our ci environment

* Revert "[Build] Declare mirror for eclipse p2 repository (elastic#117732)"

This reverts commit c35777a.
breskeby added a commit that referenced this pull request Dec 12, 2024
#118591)

The eclipse formatter used by spotless is resolved at runtime and not declared as
gradle dependency. Therefore we need to run the spotless task to ensure we have
the dependencies resolved as part of our ci image baking.

This should avoid issues with connecting to p2 repos we have experienced lately
in our ci environment

* Revert "[Build] Declare mirror for eclipse p2 repository (#117732)"

This reverts commit c35777a.
breskeby added a commit that referenced this pull request Dec 12, 2024
#118593)

The eclipse formatter used by spotless is resolved at runtime and not declared as
gradle dependency. Therefore we need to run the spotless task to ensure we have
the dependencies resolved as part of our ci image baking.

This should avoid issues with connecting to p2 repos we have experienced lately
in our ci environment

* Revert "[Build] Declare mirror for eclipse p2 repository (#117732)"

This reverts commit c35777a.
maxhniebergall pushed a commit to maxhniebergall/elasticsearch that referenced this pull request Dec 16, 2024
…#118523) (elastic#118593)

The eclipse formatter used by spotless is resolved at runtime and not declared as
gradle dependency. Therefore we need to run the spotless task to ensure we have
the dependencies resolved as part of our ci image baking.

This should avoid issues with connecting to p2 repos we have experienced lately
in our ci environment

* Revert "[Build] Declare mirror for eclipse p2 repository (elastic#117732)"

This reverts commit c35777a.
maxhniebergall pushed a commit to maxhniebergall/elasticsearch that referenced this pull request Dec 16, 2024
…#118523) (elastic#118593)

The eclipse formatter used by spotless is resolved at runtime and not declared as
gradle dependency. Therefore we need to run the spotless task to ensure we have
the dependencies resolved as part of our ci image baking.

This should avoid issues with connecting to p2 repos we have experienced lately
in our ci environment

* Revert "[Build] Declare mirror for eclipse p2 repository (elastic#117732)"

This reverts commit c35777a.
sarog pushed a commit to sarog/elasticsearch that referenced this pull request Jan 22, 2025
…#118523) (elastic#118591)

The eclipse formatter used by spotless is resolved at runtime and not declared as
gradle dependency. Therefore we need to run the spotless task to ensure we have
the dependencies resolved as part of our ci image baking.

This should avoid issues with connecting to p2 repos we have experienced lately
in our ci environment

* Revert "[Build] Declare mirror for eclipse p2 repository (elastic#117732)"

This reverts commit c35777a.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Automatically create backport pull requests when merged backport pending :Delivery/Build Build or test infrastructure >non-issue Team:Delivery Meta label for Delivery team v7.17.21 v8.17.1 v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants