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

Unify handling of custom Gradle User home in build tool tests #89304

Merged
merged 2 commits into from Aug 15, 2022

Conversation

breskeby
Copy link
Contributor

@breskeby breskeby commented Aug 12, 2022

  • keep guh separated from test project dir
  • unify folder handling

as a side effect when keeping the project dir for debugging failed tests we do not copy the whole GUH which usually isn't providing any additional help for debugging those

@breskeby breskeby self-assigned this Aug 12, 2022
@breskeby breskeby added :Delivery/Build Build or test infrastructure >enhancement Team:Delivery Meta label for Delivery team v7.17.6 labels Aug 12, 2022
@breskeby breskeby marked this pull request as ready for review August 12, 2022 09:19
@breskeby
Copy link
Contributor Author

Should reduce risk of running into #89285

@elasticsearchmachine
Copy link
Collaborator

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

@breskeby breskeby merged commit c4c1802 into elastic:main Aug 15, 2022
breskeby added a commit to breskeby/elasticsearch that referenced this pull request Aug 17, 2022
…c#89304)

- keep guh separated from test project dir
- unify folder handling

as a side effect when keeping the project dir for debugging failed tests we do not copy the whole GUH which usually isn't providing any additional help for debugging those
# Conflicts:
#	build-tools/src/testFixtures/groovy/org/elasticsearch/gradle/fixtures/AbstractGradleFuncTest.groovy
jan0sch pushed a commit to wegtam/elasticsearch that referenced this pull request Aug 18, 2022
…c#89304) (elastic#89402)

Backports the following commits to 7.17:
 - Unify handling of custom Gradle User home in build tool tests (elastic#89304)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Delivery/Build Build or test infrastructure >enhancement Team:Delivery Meta label for Delivery team v7.17.6 v8.5.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants